Bug with hidden navigation bar

If the presenting ViewController's navigation bar is hidden and you cancel the gesture, the child's navigation bar will also get hidden even though it shouldn't.
Tested on iOS 9 and iOS 8.4.

Steps to reproduce with the example project:

make the presenting ViewController's navigation bar hide itself¹
run the app and tap Push
start swiping back from anywhere
push the ViewController back to the left. Do this by letting go and using the inertia, the bug doesn't occur if you swipe all the way back smoothly

Result: The first time you do this, the navigation bar will get hidden. The second time it will get shown again, etc.
¹ e.g. by adding this method to ViewController.m:
- (void)viewWillAppear:(BOOL)animated {
    [super viewWillAppear:animated];

    NSUInteger stackCount = [self.navigationController.viewControllers count];
    if (stackCount == 1) {
        [self.navigationController setNavigationBarHidden:true animated:true];
    } else {
        [self.navigationController setNavigationBarHidden:false animated:true];
    }
}

anyone solved this?

I also have this issue.

(void)animationEnded:(BOOL)transitionCompleted
{
// restore the toViewController's transform if the animation was cancelled
if (!transitionCompleted) {
self.toViewController.view.transform = CGAffineTransformIdentity;
//some hack here
if (nil == [self.fromViewController.navigationController.navigationBar superview] ) {
//if have bug on ios9 and early, may do some work here
[self.fromViewController.navigationController setNavigationBarHidden:NO animated:YES];
}
}
}

this fixed

Anyone Solved it on Swift?

Well I solved it, just make the view controller as child of navigation controller


Add blade directive

Usage
// https://commonmark.thephpleague.com/2.4/extensions/embed/#usage
// https://github.com/ueberdosis/commonmark-embed-extension#usage
// needs embed extension for youtube links
@markdown('https://www.youtube.com/watch?v=eX2qFMC8cFo')

@markdown
# My title

This is a [link to our website](https://spatie.be)

```php
echo 'Hello world';
```
@endmarkdown

Looks good, could you also send a PR to mention this directive in our docs? I'll already merge and tag an release.


Update branch for release

Automated changes by create-pull-request GitHub action

needed to revert a commit before having the bot create this


List of backlinks are not reported in new MD files

Describe the bug
I have created a set of new notes with wikiinks. The list of reference pages of the note I'm in is not getting computed and displayed in the backlinks section of the page.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://epic-johnson-6b6441.netlify.app/notes/easter egg/'
Click on 'Random Notes' in the backlinks section.
List of backlinks in the backlink section below is empty. It says no backlinks found even though the page I'm in has backlinks.

Expected behavior
I expect to see a list reference pages of the note I'm in, in the backlinks section of the page.
Environment (please complete the following information):

OS & Version: windows 10
Browser : chrome

Additional context
Refer this sample link
https://epic-johnson-6b6441.netlify.app/notes/easter egg/

I can't find any pages linking to /notes/random-notes or /notes/subnotes/random-notes. The backlinks section is just a list of all notes which contain a link to the current page.

@ronykris I'm closing this, since you haven't responded. If you have further information, feel free to reopen it.


refactor: Rename options of access_property

Description:
Extracted from https://github.com/dudykr/stc/pull/1042to make review easier.
Related issue (if exists):

This was wrong


what does lgt_reg mean, is it the entropy maximization term H(q)?

at costs.py line54-59
      # compute src->trg raw scores for batch on this gpu
      raw_scores = torch.mm(r_src, r_trg).float()
      raw_scores = raw_scores.reshape(n_batch_gpu, n_batch, n_locs)
      raw_scores = raw_scores / n_rkhs ^ 0.5
      lgt_reg = 5e-2 * (raw_scores ^ 2.).mean()
      raw_scores = tanh_clip(raw_scores, clip_val=self.tclip)

what dose lgt_reg mean?
how to calculate entropy maximization term H(q) of Eqn.(6) in the paper?

The mixture-based representation code is not currently part of this repo, so there are no entropy terms in the costs implemented here. The lgt_reg term is the regularization on the NCE logits mentioned in the last paragraph of Section 3.3 of the paper (arxiv v2, from July 8th).
The aim of the logit regularization and tanh clipping is to limit the dynamic range of the NCE logits, thereby mitigating some gradient spikes and forcing the encoder to be more efficient about how it arranges the representations in the latent space. This is similar to the annealed cosine similarities used in other recent contrastive representation learning papers, where the temperature parameter controls the dynamic range of the NCE logits.

Thanks, it's clear


Publish on Windows Store

Version: 0.5.6
Should publish (for access with private url for starters) on Windows 10/11 Store (can't on Win8 since Delphi uses Desktop Bridge that was introduced in Win10)

available (not discoverable for now) at:
https://www.microsoft.com/store/apps/9NVM0CHRN7PH
and
test


[1.7] Fix up repo path

Both cri-o and cri-tools have moved from the kubernetes-incubator organization to kubernetes-sigs.  Tests for PRs against this branch are failing.
This is an attempt to help get those, #1966 in particular, passing.

/ok-to-test

LGTM

/retest

LGTM

LGTM
/retest

@nalind looks like there is a mismatch with the crictl version https://storage.googleapis.com/origin-federated-results/pr-logs/pull/kubernetes-sigs_cri-o/2076/test_pull_request_crio_integration_fedora/1585/artifacts/testout.txt

What does the 1.7 here mean?

@mrunalp still working on this? Looks like it needs another path update since we moved again :)

Do we even still support 1.7 release?  Can we just close this?

yeah we should close this as we don't support 1.7 anymore.


UserFields not working

Hi There,
I'm passing the UserFields with authorize API call. but I can't get it in response.

anything I'm doing wrong ? which fields should i use to post the UserFields ?
Can you help me to check it ?
Thanks.

You weren't doing anything wrong - user fields were not supported.
I've made a quick change on branch issue13 that allows user fields to be sent using the format you have given, except you must remove the "userField" key. You may also use key value pairs, so either of these will work:
    'userFields' => [
        'foo' => 'bar',
        [
            'name' => 'Test',
            'value' => 'Test111',
        ]
    ],

I have just tried this with the Hosted Page API, and see the fields in the merchant email:
========== MERCHANT DEFINED =========
x_customer_type : individual
foo : bar
Test : Test111

Could you try that branch and see if it works for you. I haven't tried the other APIs yet, but it should work.  I just had half an hour to knock this together, and will finish it off during the week, with some documentation, with your help.

ok great, thanks

You'll need to update academe/authorizenet-objects too, as this involves a small fix to that package.

Hello,
I just double-checked the issue13 branch using the purchase method and both the name/value and key-pair syntax worked for me.
Is there anything I can help with to get this pulled in?
Thanks!

I've merged and tagged for release.  If you could confirm release 3.1.2 works, please go ahead and close this issue. Thank you.

ok , I will check it. thanks for your support.

 Everything looks like it's working. Thanks!


Error installing in VS Code 1.64.2

Thanks for this extension, I'm currently getting the following error while installing it:
Can't install 'wallacegibbon.erlang-indent-with-emacs' extension because it is not compatible with the current version of Visual Studio Code (version 1.64.2).
Any idea what could be the cause?

I'm sorry but this extension only works for VSCode newer than "1.68.0".  (Which is defined in the auto-generated package.json)
Is it convenient for you to upgrade the VSCode to a newer version?

It works for me now after upgrading. Mind if I explicitly add the version requirement to the README so it doesn't bite others?

I have updated the README. Thank you for reporting this. 🤝


May I know what is backdoor_server port used for?

Hi sevdokimov,
There is a setting log-viewer.backdoor_server.port=9595 in config.conf.
May I know what it is used for? Can I mask it?

This port is needed if you have several instances on Log viewer in a cluster. Log viewer can open logs located on different nodes, in this case, one log viewer loads parts of logs from other logviewer instances using log-viewer.backdoor_server.port port. This port should not be accessible outside the cluster.


AHOCODER implementation

Hi There!
I am trying merlin with ahocoder and as I got outputs with extensions '.cc' '.fv' and '.fo'  I tried to change configure.py based on given parameters but I always got the below error
merlin/src/run_merlin.py", line 645
acoustic_worker.prepare_nn_data(in_file_list_dict, nn_cmp_file_list, cfg.in_dimension_dict, cfg.out_dimension_dict)
^
IndentationError: unindent does not match any outer indentation level
I tried to solve it but It seems like I do not see something so any help would be great.
Thanks in advance.

Python syntax is sensitive to indentation levels and your IndentationError is likely related to that.
I don’t know which code editor you use, but Ubuntu’s default text editors are pretty tedious for Python. Maybe give vscode or emacs a try?

Thank you for the help now the syntax problem seems to be solved. However I have another issue.
Traceback (most recent call last):
File "/home/akfalc/merlin/src/run_merlin.py", line 1336, in
main_function(cfg)
File "/home/akfalc/merlin/src/run_merlin.py", line 680, in main_function
global_mean_vector = normaliser.compute_mean(nn_cmp_file_list[0:cfg.train_file_number], 0, cfg.cmp_dim)
File "/home/akfalc/merlin/src/frontend/mean_variance_norm.py", line 137, in compute_mean
features, current_frame_number = io_funcs.load_binary_file_frame(file_name, self.feature_dimension)
File "/home/akfalc/merlin/src/io_funcs/binary_io.py", line 64, in load_binary_file_frame
fid_lab = open(file_name, 'rb')
IOError: [Errno 2] No such file or directory: '/home/akfalc/merlin/egs/voice_conversion/s1/experiments/jmk/acoustic_model/inter_module/nn_cc_fo_vuv_fv_254/arctic_a0001.cmp'
well as I know ahocoder's outputs have different dimension than world vocoder that's why I have changed something on confıguration.py  but above error appears and I couldn't solve it yet.
Any help will be appriciated.


Document what variables are replaced in user data

This is documented on wiki now. Generating this into inline help would be best.

This has been done as part of https://github.com/jenkinsci/openstack-cloud-plugin/pull/135

Right, keeping this open for connecting the request population and doc generation to make sure it is always up to date.


⚠ SceneTime has degraded performance

In 553112c, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 27512 ms

Resolved: SceneTime performance has improved in 90c1d9c.


Can I make a request to yandex for getting token?

how can I use module social/backends/yandex.py (OAuth) in my script?
In my case I need making request to yandex for getting token by my login and password.

This app is intended to be used with a web framework, the recent split into python-social-auth might help to reach this goal, but as is, there's no support for it yet.


Cannot download preprocessed dataset

Hi, thank you for your great work!
I am highly interested in your work, and am trying to develop on your work.
However, I cannot download preprocessed dataset because it is an empty folder.
Below is a link you put in your README file.
Thanks in advance.
Complete preprocessed datasets can be found [here](https://drive.google.com/drive/folders/1k7uqPbsI6ZROcZYA_shmebXEnHh-q8dV?usp=sharing).

Hi, sorry for the inconvenience! The former data file was accidentally lost. We have updated a new version of code with the same datasets, please refer to the new main branch for details. The original version was moved to the original branch. Thanks for your interest, please get in touch with me if there's any issue with the code or paper.

I will check the main brunch.
Thanks!


Can't install openssl

I am having problem to install openssl. I have the latest vcpkg. Could someone help?
C:\NetSense\netsense.core\tools\vcpkg>vcpkg install openssl
The following packages will be built and installed:
openssl[core]:x86-windows
Starting package 1/1: openssl:x86-windows
Building package openssl[core]:x86-windows...
-- CURRENT_INSTALLED_DIR=C:/NetSense/netsense.core/tools/vcpkg/installed/x86-windows
-- DOWNLOADS=C:/NetSense/netsense.core/tools/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=C:/NetSense/netsense.core/tools/vcpkg/packages/openssl_x86-windows
-- CURRENT_BUILDTREES_DIR=C:/NetSense/netsense.core/tools/vcpkg/buildtrees/openssl
-- CURRENT_PORT_DIR=C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/.
-- Using cached C:/NetSense/netsense.core/tools/vcpkg/downloads/openssl-1.0.2n.tar.gz
-- Testing integrity of cached file...
-- Testing integrity of cached file... OK
-- Extracting done
-- Applying patch C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/ConfigureIncludeQuotesFix.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/ConfigureIncludeQuotesFix.patch done
-- Applying patch C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/STRINGIFYPatch.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/STRINGIFYPatch.patch done
-- Applying patch C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/EmbedSymbolsInStaticLibsZ7.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/EmbedSymbolsInStaticLibsZ7.patch done
-- Applying patch C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/RemoveNonASCIIChar.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Applying patch C:/NetSense/netsense.core/tools/vcpkg/ports/openssl/RemoveNonASCIIChar.patch done
-- Configure x86-windows-rel
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):
Command failed: C:/Windows/PERL;Configure;enable-static-engine;enable-capieng;no-ssl2;-utf-8;VC-WIN32;--prefix=C:/NetSense/netsense.core/tools/vcpkg/packages/openssl_x86-windows;--openssldir=C:/NetSense/netsense.core/tools/vcpkg/packages/openssl_x86-windows;-FS
Working Directory: C:/NetSense/netsense.core/tools/vcpkg/buildtrees/openssl/x86-windows-rel/openssl-1.0.2n
See logs for more information:
Call Stack (most recent call first):
ports/openssl/portfile.cmake:65 (vcpkg_execute_required_process)
scripts/ports.cmake:72 (include)
Error: Building package openssl:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: openssl:x86-windows
Vcpkg version: 0.0.103-2018-02-07-21de65f40e04df86410952ce03f4eaa48a053a68
Additionally, attach any relevant sections from the log files above.

run vcpkg in Visual Studio developer promt for VC 2017 (or 2015)

Hi @Churan2018 , thanks for posting this issue. I can install this package successfully on my machine with the latest vcpkg. So this issue should be fixed now. Please open a new issue if this is still a problem for you.


BUG: scratchpad autostart doesn't respect scratchpad_minwidth

if you start and scratchpad autostarts, the scratchpad doesn't die even if it doesn't have enough room as specified by scratchpad_minwidth. resizing the window even a bit will cause the window to die.
presumably the check that the window has enough room isn't fired out autostart or someth i dunno

more generally, asking the scratchpad to open when there isn't enough room still lets it open. Then resizing the window causes the window to close cause there's not enough room. Honestly I dunno what a good solin could look like ngl.


Loadbalancer "Host IP" configuration is lost on next "edit configuration".

Rancher versions:
rancher/server: 1.6.10
rancher/agent: 1.2.6
Infrastructure Stack versions:
healthcheck: 0.3.3
ipsec: 0.1.4
network-services: 0.2.6
scheduler: 0.6.3
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
Client:
Version:      17.09.0-ce
API version:  1.32
Go version:   go1.8.3
Git commit:   afdb6d4
Built:        Tue Sep 26 22:42:18 2017
OS/Arch:      linux/amd64
Server:
Version:      17.09.0-ce
API version:  1.32 (minimum version 1.12)
Go version:   go1.8.3
Git commit:   afdb6d4
Built:        Tue Sep 26 22:40:56 2017
OS/Arch:      linux/amd64
Experimental: false
Operating system and kernel: (cat /etc/os-release, uname -r preferred)
NAME="Ubuntu"
VERSION="16.04.3 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.3 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial
Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node, external database
Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Edit loadbalancer
Turn on "Show host IP Addresses"
Specify "Host IP" for any particular 'Public' rule.
Click on "Edit" button to finalize changes.
Edit loadbalancer again, turn on "Show host IP Addresses" and see that "Host IP" configuration fields are empty even though they were specified on last edit. The expected result is to see the lastly saved "Host IP" configuration.

Results:

I believe the original value of the service had no entry for the host ip. I believe it would be preserved if it was originally there. Can you confirm?

@aemneina, If I create an new entry "Service Rule" and specify "Host IP", save it, edit loadbalancer configuration again and the field is empty on UI. Maybe it's preserved somewhere, however the there is no value in UI and if you save the configuration again, so the empty value is saved.

Confirmed, ports are present in launchConfig but not (re)-used when editing the LB in the UI.

The same issue exists for "Host IP" configured for service itself in "Port Map" section.

we need sourceIp field in portrule. It's not in portrule schema.
Even ports are in  launchConfig . But UI doesn't know how to map them.
This should be working once server side populates the sourceIp

Not that simple.  The balancer container doesn't know the IP the request was for because it's always the 10.42.x.y IP.  The problem here is that the 1.x UI allows expressing things that can't actually be represented in the API, and therefore can't be restored back on edit.  You cannot route a single target port to different targets based on the (incoming) destination IP.

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.

Close in favor of https://github.com/rancher/rancher/issues/11223


Add 2048 to main

Description
The game of 2048: Move blocks around to build up a score.
Test Instructions
Emulator
2024 Swadge hardware
Ticket Links
#244
Also, can close 2048 / 2048_anim branches
Readiness Checklist

[x] I have run make format to format the changes
[x] I have compiled the firmware and the changes have no warnings
[x] I have compiled the emulator and the changes have no warnings
[x] I have run make cppcheck and checked that cppcheck_result.txt has no warnings for the changes
[x] I have added doxygen comments to any code used by more than one Swadge mode. This includes /*! \file comments with Design Philosophy, Usage, and Example sections for new headers.
[ ] I have run make docs and checked that doxy_warnings.txt has no warnings for the new code

See my tweaks here: https://github.com/AEFeinstein/Super-2024-Swadge-FW/pull/285/commits/ed28e405d8638d570fb20282ee8bf4deca500259
I'm going to merge this, but not delete the branch so we can update music later and anything else.


Flutter Android Arm64

Recently i bought lenovo duet chromebook, i setup flutter in that machine.
flutter and android sdk is installed but android sdk not installed.
i googled lot of article unable to find the android sdk for arm 64

The Github issue tracker is for bugs and features requests for the Flutter SDK. Help and support questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community .


Simplified and leverage the Microsoft Identity App Sync .NET tool

Updated the tutorial to use the .NET tool: Microsoft Identity App Sync tool
Removed the extended sample code as this isn't needed to get things running - A separate document needs to be created to address how to call an API other than MS Graph that's protected by Azure AD.

@henrymbuguakiarie

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

NOTE: all the validation warnings and suggestions will need to be cleared/fixed before we can merge this.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@shoatman - Can you respond to the requested changes or let us know if you would like us to commit them?
@jmprieur, @henrymbuguakiarie - After the changes are made, please add a sign-off command to signal that the PR is ready to merge.

@Court72 - I made the requested changes... and they are in my branch in my fork... however the PR does not appear to have picked up those commits... best to recreate the PR?  Other suggestion?

#sign-off


Fix import TextDecoder in output for nodejs config

Quick typo fix. When compiling for --nodejs the output is incorrect for importing TextDecoder from Node util.

Oh dear, sorry about that!


How to use header.

Can you please explain how to use the fixed header with this layout please,

Are you referring to floating headers ? I'll push an update for floating header.

Are you referring to floating headers ? I'll push an update for floating headers. We had to implement something similar.

yes i am referring the same.

@arasu01 have a look at latest commit.


ovn-nbctl: Logical_Router does not contain a column whose name matches "load_balancer"

commit 902ae6415436be9a1806ed6193b1a23fce187375 is broken:
ovn-nbctl --timeout=5 -vconsole:off --db=tcp:<IP_ADDRESS>:6641 set logical_router GR_<IP_ADDRESS> load_balancer=ea966794-ca55-4db7-95a9-eff3cd4739e9
ovn-nbctl: Logical_Router does not contain a column whose name matches "load_balancer"

And ovn-nb really doesn't have this column:
ovn-nbctl --timeout=5 -vconsole:off --db=tcp:<IP_ADDRESS>:6641 list logical_router
_uuid               : cb65892f-b62c-43ce-a78b-daf952abc6e5
enabled             : []
external_ids        : {}
name                : "GR_<IP_ADDRESS>"
nat                 : []
options             : {chassis="7d5ea627-475c-425c-ac1b-705ec1c160a9"}
ports               : [e0f1ff69-b8d4-447d-b285-fdc13112faa3]
static_routes       : [ce6968f9-a26a-4472-9c0c-2b62331d6f3c]

@RostyslavFridman
You will need to upgrade OVS to the latest tip of branch-2.6 (or master). It came as part of a bug fix.


Message Requests

Hi!
Is it possible to get and allow/decline Message Requests in v1?
Thanks!

If this is not implemented yet I'll be glad to do it by myself.
Could somebody provide traffic snapshots please?


XfxEntry numeric keyboard doesn't have decimals on iOS

Bug
XfxEntry shows "NumberPad" and not "DecimalPad" when its keyboard type is set to "Numeric"

Version Number of Control: 1.1.3
Device Tested On: iPhone 7
Simulator Tested On:

Affects

[x] iOS
[ ] Android

Expected Behavior
The Numeric keyboard type should be able to take any character to make a "number," including the decimal point and hyphen.
Actual Behavior
The keyboard is just a 0-9 whole number keyboard
Steps to reproduce the Behavior
put an XfxEntry control on a ContentPage and specify the keyboard to be "Numeric"

I will look into this, thanks for the report

Sure thing.

Is there any change on this point?


Introduce mobile tags

To avoid forcing users to update on each release, you could use the actions tagger to update the mobile tags (or implement similar solution).
Examples of use of it https://github.com/coactions/dynamic-matrix/blob/main/.github/workflows/release.yml#L10-L14 -- if you check versions you will see how the short mobile tags are kept updated.

To avoid forcing users to update on each release

I don't, even without these horrible bad-practice moving tags.
I did it the proper way using branches.
My docs also show how you just use v1 or v2 to get the latest v1 or v2 version.
For a full explanation why this is majorly wrong, please see https://github.com/Vampire/setup-wsl/issues/44#issuecomment-1375288292

Btw. I even documented it in the readme: https://github.com/Vampire/setup-wsl#version-numbers :-)


Add link to blog post

Review Error for somic @ 2020-06-19 14:57:03 UTC
User must have write permissions to review

Review Error for MrSaints @ 2020-06-19 15:22:49 UTC
User must have write permissions to review


UnitTests fail when there's a comment in local.settings.json

To reproduce add a comment to settings file
{
  "IsEncrypted": false,
  "Values": {
    "AzureWebJobsStorage": "UseDevelopmentStorage=true",
    "FUNCTIONS_WORKER_RUNTIME": "dotnet",
  // test comment
    }
}

and run a test with  using var test = FunctionTester.Create<Startup>();

Resolved in package v1.0.13.


add topic hacktoberfest

Could we get the topic/tag hacktoberfest added to this repository so that pull requests merged during october get credited and attention drawn to this repository.
See https://github.com/topics/hacktoberfest and https://hacktoberfest.digitalocean.com/faq
===
Please at me @GoEddie so I receive a notification

Happy to add it but if you are doing it just for October fest better find another repo as I won’t have time to review any changes before next week.
Probably the 2nd nov will be my first chance so I didn’t want to raise your hopes :)

@lastlink @GoEddie We already have that topic added to this repository, so I'm not quite sure what more we can do here.

Hang on I thought this was a different repo! This is nothing to do with me :)

;) I think we can close this then.


Fix an inconsistency in the document

This is a trivial PR. Please see the file changed.
Previous inconsistency:

Doc
https://github.com/ljbkusters/python-wigner-distribution/blob/0852cc7e90389c0fe5f61116ea8e2de873d6c921/wignerdpy/toolkits/wigner_toolkit.py#L27-L29
Code
https://github.com/ljbkusters/python-wigner-distribution/blob/0852cc7e90389c0fe5f61116ea8e2de873d6c921/wignerdpy/toolkits/wigner_toolkit.py#L77

Thank you for your contribution, I will have a look at it as soon as I have the time to.


feat: generate interactive base library import examples via mo-doc

This PR sets the groundwork for generating interactive examples in the base library documentation.
Progress:

[x] Add a --name <string> flag to mo-doc to enable examples using the given Vessel package name
[x] Include an interactive import Module "mo:name/...Module"; code snippet at the top of each generated Markdown file. The goal here is to give developers a sandbox in case an example is missing (and to provide a quick copy/paste import snippet)

If you name the import fragments, then the function examples can include them in the code block specifier. See the doc/Readme.md for the syntax.

Great idea; thanks!

This should be ready to merge as a preparatory step for mass-producing base library examples.


No module named pip in aarch64-linux

OS: Ubuntu 22.04.4 LTS (aarch64)
VSCode version:  1.89.1
CodeLLDB version: v1.10.0
Compiler: N/A
Debuggee: N/A
I'm trying to install some python packages in the embedded python environment bundled by the plugin through LLDB: Command prompt, as suggested here. However, the pip command fails since it seems that it is not installed in the embedded environment:
(lldb) command script import /home/ubuntu/.vscode-server/extensions/vadimcn.vscode-lldb-1.10.0/adapter/scripts/console.py

Extra commands available:
    pip        - Manage Python packages.
    debug_info - Show module debug information.

(lldb) pip
Traceback (most recent call last):
  File "/home/ubuntu/.vscode-server/extensions/vadimcn.vscode-lldb-1.10.0/adapter/scripts/console.py", line 11, in pip
    runpy.run_module('pip', run_name='__main__', alter_sys=True)
  File "/home/ubuntu/.vscode-server/extensions/vadimcn.vscode-lldb-1.10.0/lldb/lib/python3.9/runpy.py", line 206, in run_module
    mod_name, mod_spec, code = _get_module_details(mod_name)
  File "/home/ubuntu/.vscode-server/extensions/vadimcn.vscode-lldb-1.10.0/lldb/lib/python3.9/runpy.py", line 141, in _get_module_details
    raise error("No module named %s" % mod_name)
ImportError: No module named pip

I tried this same thing in a different system, with Ubuntu x86_64, and it works there. Digging a little more, it seems that the aarch64-linux plugin lacks a site-packages folder in the python installation:
❯ ls ~/.vscode-server/extensions/vadimcn.vscode-lldb-1.10.0/lldb/lib/python3.9/site-packages
ls: cannot access '/home/ubuntu/.vscode-server/extensions/vadimcn.vscode-lldb-1.10.0/lldb/lib/python3.9/site-packages': No such file or directory

Bumping this since I spent quite a bit of time trying to debug the same issue. I'm on version 1.11.0 but using the same OS, and also don't have a site-packages folder:
ls ~/.vscode-server/extensions/vadimcn.vscode-lldb-1.11.0/lldb/lib/python3.9/site-packages
ls: cannot access '/home/ubuntu/.vscode-server/extensions/vadimcn.vscode-lldb-1.11.0/lldb/lib/python3.9/site-packages': No such file or directory


CB-10865 Run ios native tests on Travis

Fixed tests project build failure.
Updated cordova-ios tests dependency version to latest published.
Jira issue

LGTM. Thanks for improving testing!

It might make life easier to put these guys in a loop.

@dblotsky, I believe the whole tests file should be updated in this case and propose to do it as a separate task to avoid conflicts in #101
Created CB-10919 Refactor camera plugin ios native tests


When shaders to be built

Build shaders only

Build shaders then copy .spv to the assets directory. They will be hot-reloaded. cargo-xtask will be good to do so?

Build the game

Build shaders before build the game itself. build.rs will be good to do so.

cargo-xtask and build.rs will have same code.
Invoking cargo-xtask from build.rs may cause problems like dead-lock.
It should be tiny library crate.

done


Pioneer probing for the syil LNC6800

Great macros and awesome youtube videos. You guys are the best. Here is my experience with a pioneer probing system.
From my experiences setting up and running these macro programs there seems to be a slight difference in how different brands of probes operate. The examples and Youtube videos show the silver cnc probe in use.
When using the pioneer probe M10719 (Probe on) and M10720 (Probe Off) need registered in the control for macro functionality.

yes, I agree with that. The macros could be modified to include a parameter in the PROBECONFIG to optionally include the probe-on and probe-off mcodes.
We'd welcome a PR with those changes.


New Feature Request - Allow base types to be specified

In a fairly large implementation, with over 20 content types, it would be useful if all of the generated classes inherited from a common base type so we can add global members and have methods that have the base type as a parameter which could be used by all of the derived types. Also extension methods would be available. Would this be possible? I haven't looked at the code base, and I'd be willing to take a stab at this one if it is approved.

Hi @dconder ,
it's actually by design that the generated classes don't inherit from a common base type. We wanted to keep the models POCO.
What I would recommend in your case is extending the models using partial classes.
So in case of the Dancing Goat site it could look like this:
CommonBaseType.cs
public class CommonBaseType
{
... shared members
}

CommonBaseTypeExtender.cs
public partial class Article : CommonBaseType {}
public partial class Brewer : CommonBaseType {}
public partial class Cafe : CommonBaseType {}
public partial class Brewer : CommonBaseType {}

Would that work for you?

Hi Petr,
Yep, totally understand you can inherit using the partial. I was just thinking that it would be nice to have an optional base class that only appears if you tell it to. I just checked and we're at 28 content types. It would be nice to be able to just generate and have everything happen with one click. If this isn't something you'd like to add, no worries. I've already written the code and tests for fun, so I can just run with my fork.

It's definitely a valid requirement. I'm just trying to better understand the context.
I need to check with my team and see whether there wouldn't be any unexpected consequences (e.g in terms of strongly-typed deserialization, etc.). We might also need to adjust the best practices for working with models (or at least say what's the preferred approach), etc. We'll get back to you soon.
@JanLenoch @Simply007 @alesk-kentico @TomHruby @sutrkiller @huluvu21 what are your thoughts guys?

Hi @dconder , in your particular case, did you just make the generator to add the  : CommonBaseType to the declaration of the model class? Or, did you also make it generate that base class? If so, were its members populated with KC data? Were the shared members modeled using content snippets (in KC)? Thanks!

Hi @JanLenoch I added a new parameter to the command line "--baseclass xxxxx". If this was set, the code generater adds " : xxxxx" to the output of each generated POCO. I also added tests to make sure it works and only happens when --baseclass is set.

Hi @dconder,
is it possible to share the logic you have in the base class?
I agree with @petrsvihlik to have the generated classes as POCO, but I might be missing something when I don't see actual base class implementation.
I believe that common logic for all base classes could be implemented differently.
I am concerned about one thing:  When hiding some members in generated classes that are present in the base class. There is no way without the base class code to determine whether to use a new keyword for member declaration.
public class MyBaseClass
{
    public const string Codename = "baseclassCodename";
}

public class KCModel : MyBaseClass
{
    public (new) const string Codename = "kc_model";
}

This might not be your case, but it could happend.
If we omit new keyword all the times, it is fine for compiling, but it is not a good practice to omit that. In the SDK Type provider, this should not lead to some problems, but I do not see that as a clean implementation. -> Partial class approach does not allow to define member twice.

Hi @Simply007, the base class mainly just contains helper methods. For example, we have some complex logic for determining an items URL, or which image should be rendered at given places. We could use Extension methods and keep the base class empty as well. So it wouldn't have any overrides to the existing generated classes. I

After giving it a thought we decided to stick with the customization model outlined above.
That means we don't want to include base class types (or any other unnecessary directives) directly in the model classes. All customization should be done via partial classes.
If we are about to have the --baseclass xxxxx parameter implemented in this project, it should be done in the way presented in my first comment - CommonBaseTypeExtender.cs.
Please let us know if you're still interested in implementing it, @dconder.

@petrsvihlik Sure, I'd be happy to help implement it. Let me know if you'd like me to proceed.

@dconder absolutely! If you accept the invitation that I've just sent you, I'll be able to assign the issue to you. (So that nobody can steal it from you :))

Cool thanks. For tests, how deep would you like them to go? Check that the base class exists, or actually check that it contains the expected code? Should I also create a test that checks that the base class does NOT exist if the parameter to create it is not set?

I would check whether the class contains expected code (compare it against a fixture, similarly to other tests). Checking the "negative" scenario is nice to have.

fixed in #70
thank you for the contribution @dconder !


Add Customer payment detail ----> money get from customer and money return back to customer

Example:  they pay 100$ and they give back to them 20$ we must store data

CREATE TABLE tbl_payment_user (
pay_usr_id int AUTO_INCREMENT PRIMARY KEY,
pay_usr_amount double NOT NULL,
pay_usr_amount_return double ,
pay_usr_rate double,
cur_id int
)

alter table tbl_payment
add COLUMN pay_usr_id int

pay_usr_rate is calculate for customer loan khmer money but pay dollar and for customer loan dollar but pay khmer

----loan rate will get from tbl_rate : column--> rate_amount

CREATE TABLE tbl_rate (
rate_id int AUTO_INCREMENT PRIMARY KEY,
rate_amount double NOT NULL
)

run sql then insert sample data rate_amount = 4000


PV1 updates from the official HL7 v2 to FHIR mapping

User Story:
As ResportStream, I want to follow the HL7 org's HL7 v2 to FHIR mapping as close as possible, so our FHIR data conforms to a community created spec.
Description/Use Case
We followed the published HL7 v2 to FHIR spec to generate our HL7 v2 to FHIR and FHIR to HL7 v2 conversions.  The original mappings are published via their HL7 v2 to FHIR mapping pages. HL7 has published updates to the mappings via published via Google Docs (these Google docs are now the official delivery of the mappings).  We want to update any mappings used by the UP now while we are not processing data as doing it later can impact data already processed by the pipeline.
Risks/Impacts/Considerations

The result of changing the mappings while we are processing data in UP is the same as updating the COVID 19 schemas.  Replaying a piece of data may fail as the mappings are different, or a mapping change while data is being processed may cause an issue in the data as well. A future plan to be able to update these mappings is needed, such as adding mappings to the HL7 v2 to FHIR conversion first for 60 days, then to the FHIR to HL7 v2 after.
Mappings labeled as under construction can be a risk, so bring up to the team for review before implementing them.

Dev Notes:

You need to carefully inspect our mapping docs vs the new mappings.
Comparing the mappings for a segment will take you down to inspecting the mappings for data types which are reusable in other segments, so coordinate with other developers inspecting other segments to limit duplicate work.
You will also need to verify extension names. E.g. some are at the original website linked from the resource that uses them.  We may be able to reduce the number of reportstream.cdc.gov or ibm.com extensions being used in favor of official extension names.
Some of the mappings are not possible either because of limitations in our conversion library or due to repetitions having multiple values for data that otherwise if supposed to go into just one FHIR property (e.g. Provenance to store multiple Observation values). See the comments in our mapping docs, but you may need to ask the team for feedback on those.
Skip any HL7 fields that we do not have data for, otherwise you have nothing to test with.
Do not change the source HL7 v2 test data for the integration tests as they are based on real data and we want to keep those as pristine as possible.

Acceptance Criteria

The PV1 segment ReportStream mapping docs are updated with the latest HL7 v2 to FHIR mapping changes as appropriate. We need to make sure we are still functionally lossless, so some mappings may not be appropriate for this mission.
The Linux4Health PV1 HL7 v2 to FHIR UP conversion mappings are updated with the latest HL7 v2 to FHIR mapping changes as appropriate.
The RS PV1 FHIR to HL7 v2 UP conversion mappings are updated with the latest HL7 v2 to FHIR mapping changes as appropriate.
The Data Tests integration test data is updated to reflect the changes to the conversion mappings.

Here are the notes from my initial passover:

PV 1.2 - Unsure if the Statuses are being used to flip this on or off properly
PV1.6, PV1.11, PV1.12, PV1.21-PV1.35, PV1.37, PV1.40, PV1.42, PV1.43, PV1.46, PV1.51, PV1.52,    - Says No data and there is nothing in the code, this is fine if Carlos says it is
PV1.7,PV1.8,PV1.9 - There are a lot of places this says it should be but I only see it under Encounter.yml - participant_1.vars.practitionerSpec
PV1.15 - needs to be updated is in specialArrangement
PV1.18 - needs to be updated is in Encounter.extension_5
PV 1.19 - needs to be updated is in Encounter.identifier_1.vars.valueIn
PV 1.41 - needs to be updated is in Encounter.extension_4.constants.vars.value

Attaching a spreadsheet that is updated with the columns that need doc changes or I have questions on after Luis answered some of my questions from above.
https://app.zenhub.com/files/304423150/e9b1cfd0-7236-4753-ad6a-4564e1df85f2/download


Add dontOptimizeAway to std.datetime.stopwatch

As mentioned on #5367, it would be great if the benchmarking function wouldn't allow compilers to optimize its to-be-tested functions away.
There are a couple of tricks to avoid this behavior, at least:

assigning return value to __gshared
using the doNotOptimizeAway trick (see below or #2995)
@optStrategy("none) in LDC
...

I guess this might be a controversial change, so I am very interested to hear your opinions. What strategy to avoid the optimization do you prefer?

To add to @klickverbot's suggestions, I highly recommend watching this talk on benchmarking. The key point is that in addition to preventing the compiler from deleting the function call altogether, we need to make sure we're not interfering the the measurements (by adding unrelated code) and we're not tempering with the compiler's optimization abilities.
Otherwise, we'll be measuring something different from the real world.
One of the techniques shown in presentation is the following:
void escape(void* p)
{
    asm volatile("" : : "g"(p) : "memory");
}

void clobber()
{
    asm volatile("" : : : "memory");
}

And as @klickverbot says, we need an analogous solution in D that works with DMD, GDC and LDC.

Why don't we just ask the compiler devs to add a new pragma to the language to solve this?

Why don't we just ask the compiler devs to add a new pragma to the language to solve this?

CC @WalterBright @yebblies @UplinkCoder @ibuclaw

tbh. you don't really need it for dmd :)
doing an empty switch in a function will disable the inliner.

On a more serious note: We'd need to control somewhat which optimizations we disable,
Which is possible for DMD, but I don't know about LDC / GDC.
How much control do you guys have about the optimizer ?

We should add a module called benchmark of which doNotOptimizeAway would be a primitive. One question is whether this should go in druntime instead of phobos. It is highly platform-dependent so that would indicate druntime as the best place.
At any rate, portable interfaces for non-portable things are what the stdlib is about, so I'm in favor of adding this.

The barrier itself could be an intrinsic of druntime.  Then doNotOptimizeAway can still remain in phobos, itself just a user of the intrinsic.

The barrier itself could be an intrinsic of druntime. Then doNotOptimizeAway can still remain in phobos, itself just a user of the intrinsic.

Just aiming for the simplest thing - i.e. make it part of druntime.

It seems like the only things blocking this is the getpid problem and an issue of future implementation. D doesn't support any platforms atm where the getpid call would be expensive. Secondly, since this is a private function, there's no reason this can't be swapped out when a better option presents itself.
I say we move forward with this.

 From what I've seen, that prevents the function from being optimized out.

Can pragma(inline, false) be applied to lambda's? That a very common use-case for benchmark.

Can pragma(inline, false) be applied to lambda's? That a very common use-case for benchmark.

I have no idea. I never use lambdas for benchmarking, because I always end up benchmarking stuff that's too complicated for that to make sense. Function attributes can be used on lambdas though, so it would make sense if a pragma like that can be used on them, and if it can't, then I'd say that it makes good sense to create an enhancement request for it. But if I'm not mistaken, and pragma(inline, false) actually does the job properly, it seems to me like a nice, simple solution to this problem.

Well, even if it did, the argument is that the standard library should do this for the user automatically, e.g. Phobos should make it hard for the user to get the wrong result in their benchmark.

BTW there's also a very good explanation of clobber, escape and doNotOptimizeAway on StackOverflow.
However, it seems like asm allows an even easier solution:
asm
{
    call [fun + i];
    ret;
}

https://godbolt.org/g/WRFWNv (as you can see pragma(inline) doesn't work)

an issue of future implementation.

Yeah, in theory future implementation could optimize asm away, but

there's no overhead with this change anymore
`benchmark at the moment doesn't guarantee that it avoids optimization
dmd will never get so smart
if ldc or gdc ever get so smart, they could special-case this function or provide a custom API/method to avoid optimization, but as far as I see it, it's a problem of the future.

if ldc or gdc ever get so smart, they could special-case this function or provide a custom API/method to avoid optimization, but as far as I see it, it's a problem of the future.

@wilzbach - Luckily we have attributes for asm statements.  In gdc, all asm blocks prevent optimizations unless you mark it as pure.

Btw, on a quick look it seems all these solutions are meant to prevent dead code elimination. However, is there any solution to prevent 'result caching' for pure functions or inlined functions?
foreach (_; 0 .. n)
{
     escape(fun[i]());
     clobber();
}

Couldn't this still be rewritten to this:
auto tmp = fun[i]();
foreach (_; 0 .. n)
{
     escape(tmp);
     clobber();
}
?

However, is there any solution to prevent 'result caching' for pure functions or inlined functions?

pure is a non-issue so long as the function isn't called multiple times within the same statement. As I understand it, for better or worse, the results of a pure function are never really cached. The current statement may be adjusted such that the pure function is only called once, but once multiple statements are involved, the pure function will be called multiple times.

pure is a non-issue so long as the function isn't called multiple times within the same statement. As I understand it, for better or worse, the results of a pure function are never really cached.

That's not true for gdc. If a function is strong pure (i.e. pure in the GCC backend sense), the GCC backend heavily optimizes calls to such functions including caching, merging calls, etc. (Of course the compiler must be able to 'see' two identical calls, but using inlining and maybe even lto compilers are quite good at that nowadays)

Thinking about this some more, clobbering (all!) input arguments before calling the function should work for pure functions and even inlining problems. As the compiler no longer knows the exact values of arguments or whether these changed since the last iteration, it can not assume anything about the return value. So it has to always execute the complete function. (Or when inlining, the part of the function actually depending on the input value)
@ibuclaw  does the memory clobber (asm volatile("" : : : "memory");) already take care of this, or do we explicitly have to 'write' the arguments in another asm block?
However, benchmark only uses functions which do not take any parameters. If these are pure (could also be inferred), then by definition any call is expected to return exactly the same value. So we'd need a different approach to defeat compiler optimizations there.

Why was this merged?

Because it was approved two weeks ago.

...and because I didn't read carefully enough. @ibuclaw @klickverbot My apologies.

Can we make this forward to a new set of druntime intrinsics or something along the lines instead?

With this soft opening, I mistook it as idle musing rather than a verdict on the viability of the PR. Can I get away with blaming this on cross-Atlantic differences in communication style?

Because it was approved two weeks ago.

I only see one approval. Two weeks and one year ago. :-)


Better UI

come on this is 2018 not 2004

included: move status to a statusbar and add a url label

whoops

Should mention that I tried to use a ToolStrip but I couldn't stretch/spring the address bar. Will investigate

Added a status bar https://github.com/dotcomboom/gopherit/commit/122d86a0d002cb72e4c74be994548e660d3cfbbf

Table layouts: https://github.com/dotcomboom/gopherit/commit/884543a4c328b44d460c66f3b72b29157265a572 and https://github.com/dotcomboom/gopherit/commit/36f72155bd6d08ee6bc3317d9922ea98e4fd80e2

Mapper and Settings moved to status bar: https://github.com/dotcomboom/gopherit/commit/7df4a771e65cbd5205a4b23c458d81bbd71633ba

Bookmarks adjustable, I did something that makes the settings window scale up for no reason which is annoying https://github.com/dotcomboom/gopherit/commit/25d8f9ae936221446bcfa385754b9bdd00c8c546


Zip file generated on Mainframe cannot be extracted on Windows

(Issue from email discussion)
We are trying to use your zip4j library in order to create a zip file on an IBM z/OS Mainframe.
The file needs to be password protected with AES256 encryption.
We successfully deployed the java application using zip4j and the zip creation works without any errors.
The problem we encounter is that the created zip file cannot be unzipped using WinZip after transfer from the mainframe to a window machine.
If the unzipping is done on the mainframe it works as expected which leads us to believe that possibly something with the encryption is not compatible across the different platforms.
If zipping is done without encryption we do not encounter this issue and we are able to unzip the file using WinZip which was zipped on the mainframe.

Fix included in v2.11.2 released today


w3-css-slideshow category fulfilled

I was going through the CSS categories, and found out this slideshow section to be empty, as best of my knowledge I have filled the file with necessary content in a briefed way, and provided genuine links to author's site from which I have gathered data.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

This is an invalid PR.

Here, the format of the source of the PR is in XXXXXXX-patch-xx which clearly means that the XXXXXX is a person. This is a side-effect where we as mods should avoid making any branches on the main repository and always work on our forks.
And they are not the same as the OP in this case.
So, we can close these without further action.
/cc @freeCodeCamp/moderators

Thanks for the reply, sorry for the mistake as I am a newbie. :)

@SumanRoX not a problem!  If you have any questions or concerns please feel free to ask and try again.  We look forward to your future contributions!


Fix mcda method bugs

This PR fixes a couple of bugs found in mcda methods during testing.
VIKOR method has been returning NaNs in the case where some weights are zero. This is due to the formulation of the Q score, which requires division by (column max - column min). This has been fixed by adding a line in the VIKOR method which removes any zero columns.
ROV method was found to be incorrectly reversing site ranks. An example of this is :
results.scores =
 0.6371641507442964
 0.6941436816327351
 0.877935879420678
 1.0066403672998092
 1.033670846678704
 1.5902842580982375
 1.7501562536882735
 0.5729368306847074
 0.4478149807103405
 0.4484346568894094

results.bestIndex =2

In this case, site 2 is neither the highest nor lowest score. This suggests either a bug in JMcDM, or that the method searches for something other than a max/min score in ranking alternatives. Either way, for now it is best to remove ROV.

ROV method was found to be incorrectly reversing site ranks.

Without knowing how the ROV (Range of Value) method works I'm not sure we can determine this is the issue.
Going by the scores alone, it doesn't look like a reversal issue, but judging by the name of the method, it does seem it does not search for a given min/max. Not saying we need to keep it in, but we do need to justify eventually why we ignore/remove it.

ROV method was found to be incorrectly reversing site ranks.

Without knowing how the ROV (Range of Value) method works I'm not sure we can determine this is the issue. Going by the scores alone, it doesn't look like a reversal issue, but judging by the name of the method, it does seem it does not search for a given min/max. Not saying we need to keep it in, but we do need to justify eventually why we ignore/remove it.

Agree , I can look into it more. It's not so much the fact that it is working differently to other methods, it's that results.BestIndex!=maximum(scores), which causes the order to be reversed when ranking when it maybe shouldn't be. On  checking the non-reversed order of ranks, it lines up well with the ranks of other methods, suggesting the method is correct if not reversed. But because the 'results.bestIndex' is neither the max or the min, I'm not sure how to interpret this. Perhaps the method is meant to find a compromise/middle value or something.


moved internal/allocator.Pool into internal/xsync package

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Attention: Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.

Project coverage is 41.66%. Comparing base (a424af4) to head (1c5fe2f).

Files
Patch %
Lines

internal/xsync/pool.go
0.00%
11 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1331      +/-   ##
==========================================
- Coverage   42.17%   41.66%   -0.51%
==========================================
  Files         322      323       +1
  Lines       34274    28978    -5296
==========================================
- Hits        14454    12075    -2379
+ Misses      19353    16436    -2917
  Partials      467      467

Flag
Coverage Δ

go-1.20.x
43.92% <0.00%> (-0.61%)
:arrow_down:

go-1.21.x
43.93% <0.00%> (-0.59%)
:arrow_down:

go-1.22.x
41.63% <0.00%> (-0.51%)
:arrow_down:

macOS
41.66% <0.00%> (-0.02%)
:arrow_down:

ubuntu
41.61% <0.00%> (-0.06%)
:arrow_down:

unit
41.66% <0.00%> (-0.51%)
:arrow_down:

windows
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


LookUpEditor with Multiple = true cascading programmatically in a Form

I have Row Fields as
`[DisplayName("Sites")]
    [LookupEditor(typeof(SiteLookUp), Multiple = true), ClientSide
    [LinkingSetRelation(typeof(UserSitesRow), "UserId", "SiteId")]
    public List<Int32> Sites
    {
        get { return Fields.Sites[this]; }
        set { Fields.Sites[this] = value; }
    }
    [ForeignKey("[xyz].[Customer]", "CustomerId"), LeftJoin("cust"), NotNull]
    [LookupEditor(typeof(CustomerLookUp)), LookupInclude]
    public Int32? CustomerId
    {
        get { return Fields.CustomerId[this]; }
        set { Fields.CustomerId[this] = value; }
    }`

I want to cascade Sites to CustomerId in a Form.
So i added following in xyzDialog.ts constructor
(this.form.Sites as any).cascaseFrom = "CustomerId";
this.form.Sites.cascadeField = "CustomerId";
It's not working, so tried with following code
this.form.CustomerId.changeSelect2(e => {
var customerId = this.form.CustomerId.value;
if (!Q.isEmptyOrNull(customerId))
{
var sites = xyz.Master.SiteRow.getLookup().items.filter(x => x.CustomerId == parseInt(customerId));
this.form.Sites.clearItems();
sites.forEach(x => this.form.Sites.addItem(x.SiteId.toString(), x.SiteName));
}
});
it gives runtime error.
how to cascade both the fields programmatically.
It works in case if i write
[DisplayName("Customer Name"), LookupEditor(typeof(CustomerLookUp))]
    public Int32 CustomerId { get; set; }

    [LookupEditor(typeof(SiteLookUp), Multiple =true, CascadeFrom ="CustomerId",CascadeField ="CustomerId")]

    public List<Int32> Sites { get; set; }

in a xyzForm.cs, but want it to be done programmatically, as i require this cascading to work based on some condition.
regards
Shailesh

Whatnis cascaseFrom

sorry it's a typo error.
thanks
regards
Shailesh

Can you please share code ..
thanks in advance.

It was a typo error and it has been resolved. thank you.
regards
Shailesh

Can u please share working code. I'm having trouble to run same code.
Thanks

Dear mustafalotiya,
i have entered following code in constructor of a xyzdialog.ts
this.form.childId.cascadeFrom = CustomerId;
this.form.childId.cascadeField = CustomerId;
where childId is cascaded to CustomerId;
regards
Shailesh


Fix erroneously failing ChainDB QSM test

Description
There is a difference in behaviour of iterator over dead forks between the ChainDB state machine model and the system-under-test. The reason is that the iterator for the model does not take into account that blocks on a dead fork might have been garbage collected. This PR does two things:

Adapt the model to take into account potentially garbage-collected blocks in its iterator implementation
Add a unit test to bound the behaviour of iterators over dead forks for the system-under-test and the model.

Closes #3999.
Checklist

Branch

[x] Commit sequence broadly makes sense
[x] Commits have useful messages
[x] The documentation has been properly updated

Pull Request

[x] Self-reviewed the diff
[x] Useful pull request description at least containing the following information:

What does this PR change?
Why these changes were needed?
How does this affect downstream repositories and/or end-users?
Which ticket does this PR close (if any)? If it does, is it linked?

[x] Reviewer requested

bors r+


Consider dual publishing commonjs

Description
Hi,
So you've recently made this module ESM only.
Whilst I support the general move towards ESM; realistically moving our entire estate to support ESM is a mammoth task.  The way that unfortunately manifests for me is that I'll either need to fork this module and make it non-esm, or find an alternative (or write my own), because I don't see us managing to migrate an entire legacy estate to support esm any time soon.
Appreciate you may not want to do this, if so please just close the issue and i'll have my answer and know which direction I need to take :)
OpenAPI specification (optional)
No response
Configuration
No response
System information (optional)
No response

Nope, I think we will want this @Stono 🤝 interestingly, you're the only one who complained so far, I was expecting more reports sooner

Yeah perhaps if/when more people move across to the library you might get some more issues.
However realistically i think most people who are on the original library (commonjs) will see the fork, see it's esm, and just abort the migration and leave the old library in place, rather than raising it!

@Stono Would you be able to help out with this. Or provide a repository for reproduction. Unfortunately I do not think my initial fix will solve this

Have pushed changes which I was able to test and appear to fix this issue. Was able to properly generate with both CJS and ESM, when before my changes it would fail with CJS

@Stono This should be now fixed, can you give it a try? Awesome work @jordanshatford 🎉


Provide Twitter geography search snippet completion

Adds a snippet if trying to get completion for the twitter-search
component. The required query parameters (latitude, longitude, radius
and distanceMetric) are provided with placeholder values.
Signed-off-by: Marcelo Henrique Diniz de Araujo<EMAIL_ADDRESS>

Can you provide an animated image in the description of the PR to show how it is working in VS Code?


Add .each and .each_with_index to various macro types

Extracted from 131fcba77536997be90f7ca9b214a05ee77faab7 and 63e37f03dfa0be78f0b78799473598cd24b57c94

Since #9091 is in a holding pattern; I figured it would good to at least add some of macro methods that it was adding.  I was actually able to reimplement my DI refactor (minus the compiler passes) with just this PR, and a lot of duplication.  This will satisfy my needs until #9091 can go through a proper design phase.

Isn't this redundant with macro {% for?
Though I'm happy to add this, I'd rather have one or the other eventually, not both

Isn't this redundant with macro {% for?

No because:
{% begin %}
  {% hash = {"one" => {data: [[2, 2, 2], 2, 3]}, "two" => {data: [4, 5, 6]}} %}

  {% for key, value in hash %}
    {% arguments = value[:data].map do |v|
         if v.is_a? ArrayLiteral
           c = 0
           # Can't iterate over the inner array since you cant nest macro expressions
           c
         else
           v + 1
         end
       end %}
    {{pp arguments}}
  {% end %}
{% end %}

This enables that.  I'm sure you could rewrite this to use {% for .. but It would be quite an unwieldy beast.

Can I get another review on this?

Anything left to do?  Would be nice to get this into the nightly build.

Thanks @Blacksmoke16


Meta to stderr

Adds --errmeta option which tells the lib to output --info and --rejected information (ie, non-CSS output) to stderr instead of stdout. Allows users to easily filter out stuff that isn't the final CSS, as in:
$ purifycss my.css my.html my.js --rejected --errmeta >purified.css 2>purified.meta

Hey @kennyt - is there any appetite for this? We're using this fork and it's very helpful to us as part of our scripting flow.


No-GUI option does not generate any output

Hi,
after switching to the new extractor it seems --nogui and/or -g do not create neither the standard file artifacts.json nor a with option -o specified output file.
Regards Efro

This was fixed in https://github.com/raid-toolkit/extractor/commit/14b3fb4312775cb18ad65c321c1c0a56f9764b9f.
Thanks for the report.


(Kia EU) API Error handling + refactor

Continued work from https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/pull/83

Add API exceptions
Perform API error handling
Refactor: simplify headers
Linting

Passed tests:

cached request
forced request
stop climate
stop charging

Thanks! I hope I did not break anything :)
One question: what was the intended purpose of the "stage" argument in the exeption constructor of https://github.com/Hyundai-Kia-Connect/hyundai_kia_connect_api/pull/83 ?
I just noticed one of my exception calls is without this argument, so it'll crash.

Stage was from code I was using as a template. They used it as a way to pass details into the exceptions.   Such as stage of login.


when I import source code to idea , this is not working

but mvn install -DskipTests it's working

Don't cross post (you opened this in stack overflow). Activate the spring maven profile.


Allow for attaching of assets to releases

Since publishing will create a release with the changelog, it'd be nice to also add an option to add an asset to the release too.

What assets are you wanting to upload?

Depends on the project but it would be zip files, binaries, or other non-source code assets.

I've got uploads of the dist/ folder working in #177, do you need to be able to upload things from a command-line argument or will that suffice?

@AlphaMycelium Sorry for the delay in my response. I'm not using pypi but I do need to zip up a subdirectory and attach that as a release. The issue I'm running into is that because semantic release bumps the version number, I have to zip after the version is bumped. I guess I could:

Run semantic-release version
Zip up and store in /dist
Run semantic-release publish

But is that the best option here?

@alandtse That would work.
However, if you have a setup.py (even if you aren't using PyPI), it is possible to add a custom command to automate the creation of the zip, and then use the build_commands option I added in #185 to run it automatically before publishing.


The versions of the Canvas and MDA in the download links seem to be old

I have updated the Core Creator Kit and am now updating the Canvas and MDA apps. the version that is being downloaded for these is MDA: CreatorKitReferencesMDA_1.0.20221205.3_managed
Canvas: CreatorKitReferencesCanvas_1.0.20221205.1_managed
The versions I currently have are from 3 months ago and are:
CreatorKitReferencesCanvas
3 months ago
1.0.20230421.1
CreatorKitReferencesMDA
3 months ago
1.0.20230421.1
Are there updated versions for these?
Thank you

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

ID: 9a869972-ce34-c5c9-c591-fd25347181c1
Version Independent ID: 9a869972-ce34-c5c9-c591-fd25347181c1
Content: Set up the Creator Kit - Power Platform
Content Source: power-platform/guidance/creator-kit/setup.md
Service: power-platform
Sub-service: guidance
GitHub Login: @denise-msft
Microsoft Alias: demora

Thanks for your comment, @DS2019. This issue has been assigned to the doc author for review.


php artisan admin:make UserController --model=App\User

Hello, i just installedlaraevl-admin , everything was good , and when i jumped to quick start and ran php artisan admin:make UserController --model=App\User for windows , i faced with error model does not exists , what is the solution ? thanks

Try
php artisan admin:make "App\\User"

php artisan admin:make Website\OrdersController --model=App\Models\Orders
thats how I do it.

Hi, I was able to do it with:
php artisan admin:make UsersController --model='App\Models\User'

Just add quotes in the model path.

Yes, add quotes is the point.


add support for jsonschema

[ ] differentiate between the 2 schema types
[ ] some basic validation rules to get started with
[ ] support for schema references
[ ] support for proper same type promotion based on restrictions

Hi @gabriel-tincu , do you have rough estimated timeline to support JSON schema? Thanks!

Hello @Samlinxia , sorry for the late response. The support has been introduced by https://github.com/aiven/karapace/pull/171 and https://github.com/aiven/karapace/pull/192.

@hackaugusto , thank you for the update


Fcom 203

Ticket FCOM-203 Fixes to the ToJson() method, now method returns data in a correct form and it is usable in frends platform

This PR fixes issue #7


Add Spell Checker

As the right-click context menu is provided by the system adding the spell checker isn't supposed to cause issues. An example of this implementation is Unigram chat search right-click menu (designed for Anniversary Update and above, so different from the one in Notepad).

Currently not planning on adding this, unless I find a method in the future that allows me to add this in a non-intrusive way.

What do you mean by intrusive?

The red squiggely line. For some that use Notepad as a writing application this may be wanted, but for stuff I personally use the app for it's usually not wanted. So I want to look how I can find a way to make both parties happy when I have some spare time. ;)

Personally I would rather have it, but at the end you are the developer and it's up to you to decide what is more suited for your needs.

Well... If it's a common .exe program, you can minify the mechanism of plugin into file existence detection: if plugin.dll exists, enable it and display its function; else, disable it and hide everything about its existence to the users.

Well... If it's a common .exe program, you can minify the mechanism of plugin into file existence detection: if plugin.dll exists, enable it and display its function; else, disable it and hide everything about its existence to the users.

That's unrelated, since there are different APIs for uwp

I've added a toggle to enable the Spell Checker in Settings in the latest commit (
9b017e4). It's set to off by default, but you can enable it by toggling it on.
Happy editing! 😊


Feat: Disable on custom buffer

Requested feature
I checked in the docs and there isn't a way to say that on 'startify' buffer to not print the status line like for chadtree as example.
Motivation
In this way I can disable in symbols-outline and in startify.

Do you wan;t to disable entire statusline or just some component ?


Template support

Template support
First of all, thanks for creating this plugin. I am web designer for Open Law Library. We have selected to use it with our project since it proved to be best for our needs, especially because it had the possibility for custom templates and styles. With a bit of customization, we were able to achieve something like this

Our template
Our template looked like this one:
const defaultTemplate = `
<transition-group
	:name="transitionName"
	:class="outerClass"
	tag="div"
>

			<button v-if="!message.options.important"
				type="button"
				class="button button--icon button--transparent"
				data-dismiss="alert"
				aria-label="Close message"
				@click.stop.prevent="destroyFlash(index)"
			>

			</button>

</transition-group>
`;

Styles and icons were custom made and were included in the main SCSS file.
Is it possible to create separate .vue component with message template, scripts and styles, which can be reused every time messages are called? I guess that templating would be much easier in this way.

Hello ! That is something I really would like to do. Did you find a workaround to be able to provide a template ?

I don't remember why this didn't work for us in the end, but we moved on to another library Vue Toasted where we achieved the result from the image.

I don't remember why this didn't work for us in the end, but we moved on to another library Vue Toasted where we achieved the result from the image.


Use Material Inkwell + Add related theme options

What's Changed

FEAT Use Material's InkWell instead of GestureDetector for SidebarXCell, which adds
a default hover color and a splash animation when pressed.
FEAT Add secondaryHoverColor, splashColor, selectedSplashColor,
highlightColor, selectedHighlightColor, focusColor, and selectedFocusColor
properties to SidebarXTheme
FEAT Enable default toggle button interaction colors hoverColor, highlightColor, and focusColor

Highlight, Focus, and Hover, and Splash colors

Default toggle button is now improved for keyboard tab navigation

Thanks for the approval. Before you merge, I just noticed a typo in the readme. It should be selectedHoverColor instead of secondaryHoverColor

This is getting stale, so I'm going close the PR. Feel free to take any of the changes and incorporate them into the codebase when you have time.


docs(README): add "customRegExpToFindKeys" attribute to "TypeScript" README section

Description
Add missing "customRegExpToFindKeys" attribute to README "TypeScript" config example
Check List

[x] The commit message is formatted
[x] New functionality includes testing
[x] New functionality has been documented
[x] All user-facing changes accompanied by the correspondent documentation
[x] All tests pass
[x] PR include resolved issues
[x] PR include description

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Figure out why we are not getting the last 2 weeks of job information.

Currently not getting new job information to the public live instance (trss.adoptium.net), and we believe the machine has enough space (as its been recently cleaned up).  Last results look to be from mid-February.
Recently build jobs were reconfigured / named differently, though we do not expect that this should affect the ability for TRSS to monitor a top-level build pipeline.
(related: https://github.com/adoptium/aqa-test-tools/issues/622 - is there a way to see logs despite having no direct access to the machine?)
This can turn into a 'documentation issue', where we could add some docs about troubleshooting for these cases.

TRSS was in a weird state: the services were not running. However, the node processes were running. And no TRSS log files under /var/log/. I suspect that the disk space issue caused TRSS at a bad state. And log files were removed during the disk free-up.
I restarted TRSS services and https://trss.adoptium.net/ is back online. I can see the latest build from today (Mar 4th).

And log files were removed during the disk free-up.

What makes you say that? The files in /var/log/TRSS*.log.? are still present as they were before.

TRSS was in a weird state: the services were not running. However, the node processes were running.

Hmmm the services were still showing as running on Friday if this is a valid way to query it - am I misinterpreting this or were you looking at the status in a different way?
root@trss:~# systemctl status TRSSFrontend
● TRSSFrontend.service - LSB: forever-service startup script for TRSSFrontend
   Loaded: loaded (/etc/init.d/TRSSFrontend; generated)
   Active: active (running) since Mon 2021-11-22 21:26:28 UTC; 3 months 10 days ago
     Docs: man:systemd-sysv-generator(8)
    Tasks: 11 (limit: 4915)
   CGroup: /system.slice/TRSSFrontend.service
           └─10637 /usr/bin/node /home/jenkins/openjdk-test-tools/TestResultSummaryService/frontend.js --configFile=/data/db/credentials/trssConf.json

Warning: Journal has been rotated since unit was started. Log output is incomplete or unavailable.
root@trss:~# systemctl status TRSSBrontend
Unit TRSSBrontend.service could not be found.
root@trss:~# systemctl status TRSSBackend
● TRSSBackend.service - LSB: forever-service startup script for TRSSBackend
   Loaded: loaded (/etc/init.d/TRSSBackend; generated)
   Active: active (running) since Mon 2021-11-22 21:26:17 UTC; 3 months 10 days ago
     Docs: man:systemd-sysv-generator(8)
    Tasks: 11 (limit: 4915)
   CGroup: /system.slice/TRSSBackend.service
           └─10532 /usr/bin/node /home/jenkins/openjdk-test-tools/TestResultSummaryService/backend.js --configFile=/data/db/credentials/trssConf.json

I restarted the services last Friday. TRSS was in a bad state and had no log before I restarted it.

This is no longer an issue, can be re-opened if required.


Some [ID] entries in the text version do not adhere to the documented format

e.g.
[ID]
3MKH_d3mkhd1
should this be
3MKH_3_D or 3MKH_1_D?
Is that chain_id correct?
looking at the pdb entry for 3MKH, 3MKH_4_D seems most likely without looking further into it.
In text/casp11/training30 there are 3661 of these either malformed or undocumented IDs.

https://github.com/aqlaboratory/proteinnet/issues/1#issuecomment-375270286

Thanks! Missed that somehow.


How do I generate text

What is wrong?
I dont know how to generate text using brain.js, I would like to train the neural net with text, e.g a book. Then ask the bot to generate an output. E.g generate a book
How important is this (1-5)?
1, I think this would be really funny if given the right input. It's not really important.
Other Comments
I love this framework, thanks for creating something so awesome.

maybe you're looking for markov chains?
e.g. https://github.com/substack/node-markov

Hmm not exactly what I'm looking for, very close however.


In the study listing page, add to analysis do not respect artifact visbility

if in a study you have to BIOM artifacts, and one is public and the other is private, when you click "Add to analysis" it will add the samples from both the public and the private biom.

Fixed in https://github.com/biocore/qiita/pull/2002


sync'd videos not sync'd on first playback

See details here: https://github.com/stanfordnmbl/opencap-viewer-private/issues/132

@antoinefalisse I think this is fixed in #18. Can you check? I tried to "break" it but seems like it is always synchronized now.


No log for passwords

Hi,
Please add no_log: true to task "Template auth config" in roles/vmauth/tasks/configure.yml because it may contain sensitive information.

Hi @kkrasnov1
This was added as a part of v0.58.1 release.
Closing as completed.


Fix panic in jsonpath func with empty string value

WHAT
This pull request fixes panic in jsonpath func with empty string value.
( also include changes by go mod tidy )
WHY
Before this change, you can get panic with empty string value.
➜  stein git:(fix-jsonpath-empty-string) ✗ make test
?       github.com/b4b4r07/stein        [no test files]
?       github.com/b4b4r07/stein/lint   [no test files]
?       github.com/b4b4r07/stein/lint/internal/policy   [no test files]
=== RUN   TestGJSONFunc
=== RUN   TestGJSONFunc/simple.json
=== RUN   TestGJSONFunc/simple_kubernetes.yaml
=== RUN   TestGJSONFunc/string_value.yaml
=== RUN   TestGJSONFunc/number_value.yaml
=== RUN   TestGJSONFunc/object_value.yaml
=== RUN   TestGJSONFunc/tuple_value.yaml
=== RUN   TestGJSONFunc/strict_string.json
=== RUN   TestGJSONFunc/strict_boolean.json
=== RUN   TestGJSONFunc/strict_boolean.json#01
=== RUN   TestGJSONFunc/strict_boolean.json#02
=== RUN   TestGJSONFunc/empty_string.json
    --- FAIL: TestGJSONFunc/empty_string_value.json (0.00s)
        jsonpath_test.go:113: unexpected error: panic in function implementation: runtime error: index out of range [0] with length 0
            goroutine 54 [running]:
            runtime/debug.Stack(0xc000166d80, 0x20, 0xc000180180)
                /usr/local/go/src/runtime/debug/stack.go:24 +0xab
            github.com/zclconf/go-cty/cty/function.errorForPanic(...)
                /Users/dragon3/pkg/mod/github.com/zclconf/go-cty@v0.0.0-20181231001355-67e3da15e430/cty/function/error.go:44
            github.com/zclconf/go-cty/cty/function.Function.Call.func1(0xc000166d80, 0xc000166da0)
                /Users/dragon3/pkg/mod/github.com/zclconf/go-cty@v0.0.0-20181231001355-67e3da15e430/cty/function/function.go:239 +0x11f
            panic(0x1348820, 0xc000180180)
                /usr/local/go/src/runtime/panic.go:679 +0x1b2
            github.com/b4b4r07/stein/lint/internal/policy/funcs.GJSONFunc.func2(0xc000184140, 0x1, 0x1, 0x13c3fa0, 0xc0000ae181, 0xc0000ae181, 0x0, 0x0, 0x104f215, 0x1577898, ...)
                /Users/dragon3/src/github.com/b4b4r07/stein/lint/internal/policy/funcs/jsonpath.go:87 +0xa7a
            github.com/zclconf/go-cty/cty/function.Function.Call(0xc00018e540, 0xc000184140, 0x1, 0x1, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
                /Users/dragon3/pkg/mod/github.com/zclconf/go-cty@v0.0.0-20181231001355-67e3da15e430/cty/function/function.go:243 +0x508
            github.com/b4b4r07/stein/lint/internal/policy/funcs.GJSON(0x1369512, 0x17, 0xc0000ae840, 0xa, 0xa, 0x13c3fa0, 0xc0000ae181, 0x13143a0, 0xc000148180, 0x10836cc, ...)
                /Users/dragon3/src/github.com/b4b4r07/stein/lint/internal/policy/funcs/jsonpath.go:42 +0x12d
            github.com/b4b4r07/stein/lint/internal/policy/funcs.TestGJSONFunc.func1(0xc00018a400)
                /Users/dragon3/src/github.com/b4b4r07/stein/lint/internal/policy/funcs/jsonpath_test.go:111 +0x16a
            testing.tRunner(0xc00018a400, 0xc00017c210)
                /usr/local/go/src/testing/testing.go:909 +0x19a
            created by testing.(*T).Run
                /usr/local/go/src/testing/testing.go:960 +0x652
=== RUN   TestGrep
=== RUN   TestGrep/Match
=== RUN   TestGrep/NotMatch
--- PASS: TestGrep (0.00s)
    --- PASS: TestGrep/Match (0.00s)
    --- PASS: TestGrep/NotMatch (0.00s)
=== RUN   TestWc
=== RUN   TestWc/OneLineWithCharOption
=== RUN   TestWc/OneLineWithWordOption
=== RUN   TestWc/OneLine
--- PASS: TestWc (0.00s)
    --- PASS: TestWc/OneLineWithCharOption (0.00s)
    --- PASS: TestWc/OneLineWithWordOption (0.00s)
    --- PASS: TestWc/OneLine (0.00s)
FAIL
FAIL    github.com/b4b4r07/stein/lint/internal/policy/funcs     1.537s
?       github.com/b4b4r07/stein/lint/internal/policy/loader    [no test files]
?       github.com/b4b4r07/stein/lint/internal/policy/terraform [no test files]
=== RUN   TestDuplicatedNode
--- PASS: TestDuplicatedNode (0.00s)
=== RUN   TestRemoveNotExistEdge
--- PASS: TestRemoveNotExistEdge (0.00s)
=== RUN   TestWikipedia
--- PASS: TestWikipedia (0.00s)
=== RUN   TestCycle
--- PASS: TestCycle (0.00s)
PASS
ok      github.com/b4b4r07/stein/lint/internal/topological      (cached)
?       github.com/b4b4r07/stein/pkg/logging    [no test files]
FAIL
make: *** [test] Error 1

This is because getJSON returns if the value is empty string (e.g. {"ok": ""}) then switch b[0] panic with index out of range error.

👀
Thank you for the contribution 🙇 I'll review this soon 🙇

Tagged and uploaded as new release: https://github.com/b4b4r07/stein/releases/tag/v0.2.11

Thanks!


Data-Tier Application Wizard menu item on CMS opens connection dialog even if connected

Connect to CMS and expand to registered server node
Right click on server and select Data-tier Application Wizard
Connection dialog pops up instead of using connection from node (the dialog is populated with correct information though)

Note this seems to work fine for OE (launches the wizard directly)

This issue is actually a DacFx issue and not Data Explorer. It's tracked here - https://github.com/microsoft/azuredatastudio/issues/3486.

Closing as duplicate of #3486 based on previous comment.


Enable SPARQL Federation with Credentials

Checked through unit tests and created a custom unit test for testing against a local blazegraph instance. No issues


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 861 ms

Resolved: Bofrost.at performance has improved in 606c193 after 11 minutes.


Links break if they contain underscores

Example:
[a b c](/a_b_c)

Becomes:

a b c

Hello there, i found same problem here...
>>> import markdown
>>> MARKDOWNX_MARKDOWN_EXTENSIONS = [
...     'markdown.extensions.extra',
...     'markdown.extensions.nl2br',
...     'markdown.extensions.smarty',
...     'markdown.extensions.fenced_code',
...
...     # Custom markdown extensions.
...     'app_forum.utils.extensions.urlize',
...     'app_forum.utils.extensions.mention',
...     'app_forum.utils.extensions.emoji',
... ]
>>> md = markdown.Markdown(extensions=MARKDOWNX_MARKDOWN_EXTENSIONS)
>>> md.convert('![](/media/markdownx/affbbda4-054e-4fd3-b263-eb4982289d01.jpg)@[falcon] ')
'
\x02klzzwxh:0000\x03\x02klzzwxh:0001\x03
'
>>>
>>> # but if has `space` between `)` and `@` it work well.
>>> md.convert('![](/media/markdownx/affbbda4-054e-4fd3-b263-eb4982289d01.jpg) @[falcon] ')
'
 @falcon
'
>>>

My Extention
import markdown
from django.contrib.auth.models import User

BASE_LINK = '//mysite.com/profile/'
MENTION_RE = r'\@\[(?P<username>\S+)\]'
# MENTION_RE = r'(?<!\!)\@\[([^\]]+)\]' -> m.group(2)

class MentionPattern(markdown.inlinepatterns.Pattern):

    def __init__(self, pattern, markdown_instance):
        markdown.inlinepatterns.Pattern.__init__(self, pattern)
        self.markdown_instance = markdown_instance

    def handleMatch(self, m):
        username = m.group('username')

        """Makesure `username` is registered."""
        if username in [u.username for u in User.objects.all() if u.is_active]:
            url = '{0}{1}/'.format(BASE_LINK, username)
            el = markdown.util.etree.Element('a')
            el.set('href', url)
            el.set('class', 'direct-mention-link')
            el.text = markdown.util.AtomicString('@' + username)
            return el

class MentionExtension(markdown.Extension):

    def extendMarkdown(self, md, md_globals):
        """ Setup `mention_link` with MentionPattern """
        md.inlinePatterns['mention_link'] = MentionPattern(MENTION_RE, md)

def makeExtension(*args, **kwargs):
    return MentionExtension(*args, **kwargs)

@agusmakmun it appears that your issue is completely unrelated to this issue. When I run your input without your extension, the output is completely reasonable:

@[falcon]

Which suggests you have a bug in your extension. For an explanation of why you are getting strings like klzzwxh:0002 in your output, see #354.
Note that this forum is for reporting bugs in Python-Markdown, not for asking support questions. If you need any assistance building an extension, please ask for assistance on the mailing list. If you find an actual bug in Python-Markdown (rather than your code), then please open a new issue.


alien_cake_addict example - game board does not appear

Bevy version
master 7be096a
Operating system & version
Windows 10
What you did
Running the alien alien_cake_addict sample multiple times.
What you expected to happen
Tile game board appears every time.
What actually happened
Running the alien alien_cake_addict sample multiple times does not show the game board at all times.
Rather, all boxes are piled up in what appears to be the world origin.
Additional information
There may be a race condition.

Yeah this is probably related to the recently merged #1144. Does it work if you check out a commit prior to that?

We anticipate a number of "system order" related issues due to the new executor logic, which we plan on fixing over the next week or so.

Thanks for the heads up!
Indeed it works on b71ada03465a0343b4ead1b2896f44587e719a47

Thanks for checking! Knowing that is super helpful.


[Feature] Support satellite images.

NeRFs for satellite images are attracted community interest, e.g., satnerf. Do you have a roadmap for supporting satellite images in the nerfstudio?

Would be interested in seeing this functionality.

You should be more precise when you ask for "support".
Do you mean:

Supporting satellite-specific camera models (e.g. pushbroom)
Being robust to images acquired at different times, with different shadows - nerfstudio uses appearance embedding from Nerf-W, so it can somewhat deal with that
Being robust to transient objects (clouds) - nerfstudio doesn't model these. Nerf-W does
Supporting large-scale environments (a la block-nerf)
Something else?


OpenAPI specs don't return typed responses in any API with a "raw" parameter

Using CollectionsResource as an example, any of the resources that use the raw parameter generate an OpenAPI spec that loses the type information. ResponseWrapper is a generic data holder.
When generating any clients in a typed language (e.g. Java, Kotlin) from the specs the generated have two current limitations

In many of the methods the @ApiOperation is missing and so the generated stubs have a void return type.
With the recent fix on the CollectionsResource https://github.com/stargate/stargate/pull/1100, the new return type is invalid if called with raw=true

I would propose that anyone using the OpenAPI spec to generate API clients to Stargate is using a language where they would default to raw=true so as to have typed data in their application.
As an example making this change from ResponseWrapper to DocCollection in CollectionsResource
@GET
@ApiOperation(
    value = "List collections in namespace",
    response = DocCollection.class,
    responseContainer = "List")
@ApiResponses(
    value = {
      @ApiResponse(code = 200, message = "OK", response = DocCollection.class),
      @ApiResponse(code = 401, message = "Unauthorized", response = Error.class),
      @ApiResponse(code = 500, message = "Internal server error", response = Error.class)
    })

generates
data class DocCollection (
    /* The name of the collection. */
    @SerializedName("name")
    val name: kotlin.String,
    /* Whether an upgrade is available. Use the 'upgrade a collection' endpoint with the upgrade type to perform upgrade. */
    @SerializedName("upgradeAvailable")
    val upgradeAvailable: kotlin.Boolean? = null,
    /* The upgrade type, if an upgrade is available. */
    @SerializedName("upgradeType")
    val upgradeType: DocCollection.UpgradeType? = null
) {

    /**
     * The upgrade type, if an upgrade is available.
     * Values: SAI_INDEX_UPGRADE
     */
    enum class UpgradeType(val value: kotlin.String) {
        @SerializedName(value = "SAI_INDEX_UPGRADE") SAI_INDEX_UPGRADE("SAI_INDEX_UPGRADE");
    }
}

    @GET("v2/namespaces/{namespace-id}/collections")
    suspend fun getCollections(@Header("X-Cassandra-Token") xCassandraToken: kotlin.String,
        @Path("namespace-id") namespaceId: kotlin.String,
        @Query("raw") raw: kotlin.Boolean? = null): Response<DocCollection>

which is much more useful than
data class ResponseWrapper (
    /* Response data returned by the request. */
    @SerializedName("data")
    val data: kotlin.Any? = null
)

@GET("v2/namespaces/{namespace-id}/collections")
suspend fun getCollections(@Header("X-Cassandra-Token") xCassandraToken: kotlin.String,
    @Path("namespace-id") namespaceId: kotlin.String,
    @Query("raw") raw: kotlin.Boolean? = null): Response<ResponseWrapper>

I'd speculate that any clients accessing the doc api are not accessing it by generating OpenAPI client stubs because in fact the return types are missing for some methods.  It might be preferable to use the oneof part of the OAS 3.0 spec to specify the two different return types whether or not raw=true, but AFAIK there is no way via the Swagger annotations to automate this.  Therefore I'd request that the spec defaults as though raw=true and specifies the correct types, rather than the wrapper.

FWIW it does look like the annotations in swagger-core 2.x (OAS 3.0) do support oneOf
    @GET
    @Path("/oneOf")
    @Operation(summary = "Test inheritance / polymorphism",
            responses = {
                    @ApiResponse(description = "bean answer",
                            responseCode = "200",
                            content = @Content(
                                    mediaType = MediaType.APPLICATION_JSON,
                                    schema = @Schema(
                                            oneOf = { MultipleSub1Bean.class, MultipleSub2Bean.class }
                                    )
                            )
                    )
            })


Oauth scopes end up in 'scope' property instead of 'scopes' property

If I lock an endpoint down using a few scopes like:
 class MyAwesomeAPI < Grape::API
    oauth2 'foo', 'bar
    get '/with_no_description' do
       { hello: 'undescribed world' }
    end

 end

Then those scopes do not get picked up because for some reason, they end up put into the api_context.options[:route_options][:auth] hash under the key [:scope], not the key [:scopes].  Therefore, the has_auth_scopes? method in WineBouncer::AuthStrategies::Default always returns false (once again since no scopes are in [:scopes], everything is in [:scope], and all endpoints end up being locked down with only the default scope.
I believe the bug is only with the new release (with the new DSL, version 0.3.0), so reverting back to the old style auth: DSL (version 0.2.0) fixes the issue.
Thanks,
Greg

Facing the same problem. Only strategy swagger works well with scopes.

Temporarily fix it by monkey patching using
module WineBouncer
  module AuthStrategies
    class Default
      def auth_scopes?
        endpoint_authorizations &&
          endpoint_authorizations.key?(:scope) && endpoint_authorizations[:scope].any?
      end

      alias_method :has_auth_scopes?, :auth_scopes?

      def auth_scopes
        endpoint_authorizations[:scope].map(&:to_sym)
      end
    end

    class Protected
      def auth_scopes?
        endpoint_authorizations &&
          endpoint_authorizations.key?(:scope) && endpoint_authorizations[:scope].any?
      end

      alias_method :has_auth_scopes?, :auth_scopes?

      def auth_scopes
        endpoint_authorizations[:scope].map(&:to_sym)
      end
    end
  end
end

Thank you, ill have a fix within 20 minutes :+1:
There was no spec that handles custom scopes. So I added these.

Default scope should work without the monkeypatch.
@guidoprincess, @calfzhou care to verify?

it works!

Yep, its better.  Feel free to close this and thank you as always!

Thank you for reporting :)


Use MongoDB 7

Description
Closes #3231.
Supersedes pull request #3652.
Readiness checklist

[ ] I added/updated unit tests (and they pass).
[x] I added/updated integration/compatibility tests (and they pass).
[x] I added/updated comments and checked rendering.
[x] I made spot refactorings.
[ ] I updated user documentation.
[x] I ran task all, and it passed.
[x] I ensured that PR title is good enough for the changelog.
[x] (for maintainers only) I set Reviewers (@FerretDB/core), Milestone (Next), Labels, Project and project's Sprint fields.
[x] I marked all done items in this checklist.

(for maintainers only) I set Reviewers (@FerretDB/core)

The review wasn't requested from the @FerretDB/core team. Because of that, there is no team notification.
Also, please re-request the review from everyone. For example, this PR is not currently on my dashboard.

@rumyantseva PTAL


Updated increment and decrement operations

++ and -- will be removed from the Swift language as of 3.0

Awesome! Thanks @ianrahman ! :smile: :tada:


zsh-abbr: 5.1.0 -> 5.2.0

Description of changes
Updating zsh-abbr to the latest released version.
Changelogs

https://github.com/olets/zsh-abbr/blob/main/CHANGELOG.md#520-2023-10-19

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 262539 run on x86_64-linux 1

1 package built:

zsh-abbr