Fix missing tests in EventsRegisterTest

First, read https://zulip.readthedocs.io/en/latest/events-system.html so that you understand the Zulip events system; the new application feature tutorial may also be helpful.  At the end of that description, I mention how we can test whether we have complete coverage for EventsRegisterTest; I did that check recently and found some gaps:

changing realm waiting period setting
changing realm default language setting
removing the various bot default streams
users added to an invite-only stream
changing the avatar source for a human user

There are also these gaps which are not really a problem because the register codebase doesn't fetch that data, but we might want to add a test just to confirm that nothing changing is correct:

sending messages
adding/removing emoji reactions

Each of these tests is probably <10 lines of code, mostly cribbing off the existing tests, but may not be ideal for someone not used to reading somewhat complex Python code (so good for beginners-to-Zulip, not beginners-to-Python).

@timabbott Is this open to work? I would like to work on it 👷

Sure, go for it!

@timabbott Thanks!

Today I add tests for (and where necessary, fixed) all of these except for two issues:

email change
sending presence

(to be merged shortly).
I'm going to open follow-up issues for both of those specific issues, and am closing this meta-issue.


Incorrect merging of model classes

I'm not sure what is going on here (feel free to rephrase the title accordingly), but there is something wrong happening in a (not so)edge case.
Here you have a minimal reproducer description:
openapi: 3.0.3
info:
  title: A bug reproducer
  version: v1
paths:
  "/one":
    patch:
      requestBody:
        content:
          application/json:
            schema:
              $ref: "#/components/schemas/ComponentTwo"
      responses:
        "200":
          description: OK
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/ComponentOne"
components:
  schemas:
    WithOne:
      type: object
      properties:
        one:
          type: string
    WithTwo:
      type: object
      properties:
        two:
          type: string
    WithThree:
      type: object
      properties:
        three:
          type: string
    ComponentOne:
      type: object
      allOf:
        - $ref: "#/components/schemas/WithOne"
    ComponentTwo:
      type: object
      allOf:
        - $ref: "#/components/schemas/WithOne"
        - $ref: "#/components/schemas/WithTwo"
        - $ref: "#/components/schemas/WithThree"

Generating code with Kiota we obtain (examples in C# for your convenience but applies to all).
expected:
public class ComponentOne : WithOne, IParsable

not expected:
public class ComponentTwo : WithOne, IParsable

this is surprising, as it seems that ComponentTwo is being considered equal to ComponentOne for some reason.
The expected output for ComponentTwo should be(code trimmed for clarity):
    public class ComponentTwo : IAdditionalDataHolder, IParsable
    {
        public IDictionary<string, object> AdditionalData { get; set; }
        public string? One { get; set; }
        public string? Two { get; set; }
        public string? Three { get; set; }

If ComponentOne is not used ComponentTwo gets generated with the expected shape(it took forever to minimize 😅 ).

Vincent is out this week on vacation. @andrueastman  could you investigate?

Kiota is building an inheritance index here of a type and the types that depend on it from the allOf and using this as a mean of building the discriminator mapping.
When the WithOne type is created, it tries to build the discriminator mappings (therefore initialize its derived types in the codeDom which are ComponentOne and ComponentTwo) by calling GetCodeTypeForMapping
https://github.com/microsoft/kiota/blob/4335b26876e9aa2cbdbd8cf0b189469212d60bd5/src/Kiota.Builder/KiotaBuilder.cs#L2020
However, when this is done, this is done by passing WithOne as the baseClass parameter (as this is being done from the context of the base class trying to create discriminator therefore assuming there are no other in between the inheritance tree).
To resolve this, the baseClass parameter could maybe be removed and the call to AddModelDeclarationIfDoesntExist replaced with CreateModelDeclarations as it will look up the schema and should the correct allOf entry as this is its responsibility.

@andreaTP Just to confirm this issue is related to the comment at https://github.com/microsoft/kiota/issues/4346#issuecomment-2020409106?

Just to confirm this issue is related to the comment at https://github.com/microsoft/kiota/issues/4346#issuecomment-2020409106?

I have no idea ...

Why is Kiota using inheritance for allOf when the spec explicitly states that it's not allowed? From the spec:

allOf can not be used to "extend" a schema to add more details to it in the sense of object-oriented inheritance. Instances must independently be valid against "all of" the schemas in the allOf. See the section on Extending Closed Schemas for more information.

It's meant to be used to compose types, effectively copying the properties of the base allOf.

Why is Kiota using inheritance for allOf when the spec explicitly states that it's not allowed? From the spec:

allOf can not be used to "extend" a schema to add more details to it in the sense of object-oriented inheritance. Instances must independently be valid against "all of" the schemas in the allOf. See the section on Extending Closed Schemas for more information.

It's meant to be used to compose types, effectively copying the properties of the base allOf.

Unfortunately this is one of the instances where using a validation specification as an IDL shows limitations. Effectively kiota implements the status quo of "OAS chose to use JSON Schema, lots of OAS specs use allOf to model inheritance even though it wasn't designed for that, people expect that convention to work".


Dev atan2

Further work on issue #19, replaces PR #21
Due to std::min causing ODR-usage, for C++11/C++14, we need namespace scope variable declarations; this need is deprecated in C++17. Instead, I've replaced fvar::order_sum and fvar::depth as static constexpr data members with aliases of std::integral_constant. This removes the need to use deprecated code in C++17 builds while keeping the code clean and portable.

This PR is made moot by PR #22 and by PR #24

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev-atan2@7818e27). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             dev-atan2      #23   +/-   ##
============================================
  Coverage             ?   65.49%
============================================
  Files                ?        9
  Lines                ?     2498
  Branches             ?        0
============================================
  Hits                 ?     1636
  Misses               ?      862
  Partials             ?        0

Impacted Files
Coverage Δ

...lude/boost/math/differentiation/autodiff_cpp11.hpp
100% <ø> (ø)

include/boost/math/differentiation/autodiff.hpp
100% <ø> (ø)

Continue to review full report at Codecov.

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


Support recovering from invalid URLs using search

When launching a search from an invalid URL (like, invalid network or invalid layer), we should just jump to the global search page.
Implemented on top of #1145.

This is no longer necessary, so closing.


How to backup my wallet from macOS?

Hello!
I found how to export keys on komodo support platform.
But I'm not sure if this method is also true for wallet on MAC.
I didn't see any "export keys" on my site.

What's the problem?

in native mode wallet.dat is your backup
make a copy of it

Thank you very much!
I also found where is the wallet on MAC


Eye Blink Counter

Is your feature request related to a problem? Please describe.
-- I want to create a system that could be stopped through the eye blinks of the user.
Describe the solution you'd like
-- The system would detect the eye blink counts of the user. Whenever it detects a double blink of the user, the system will be stopped.
Describe alternatives you've considered
-- The alternatives can be through voice commands. A 'stop' command would also be applicable.
Approach to be followed (optional)
-- I'll use the Opencv python library to detect the eye blinks.
Please assign this issue to me. In addition, I'm a GSSOC'23 open source contributor.

hey this is not required as it is already implemented


Account delete causes Error

This happens because the method ICEndPoint removeConfiguration:  receives an EndPoint, but expects an Dictionary

The expected dictionary is supposed to hold at least the following fields:
username, port, address, name, ssl
Each ICEndpoint holds name and ssl.
It also holds an authenticationInfo object, which itself holds a username.
It also holds an serverInfo object, which itself holds a port and host. hostis equal to theaddress` here.
We could create a method which builds up the expected dictionary in endpoint.


Show THE SQUIRREL

Add the squirrel icon when an update is available.
See https://github.com/atom/atom/pull/10377 for the discussion that prompted this.
/cc @atom/feedback

I think it'd be good to add a spec for this little dude, similar to the one from release notes.

Yeah definitely :+1:

This is ready for :eyes: now.

Alright, I think this is really and truly ready :pray:

Let's do this! :rocket:

Thanks for fixing this long-standing pain-point. :heart:

:ship:


Don't use DTO classes when generating interfaces

Fixes #2341. The Typescript DataConversionCode relies on a class to be generated. If there is no need to call this, as is the case when using interfaces, we can just parse the JSON result.

I think the initial idea of UseDtoClass == false is that they are not generated but you need to provide/write them yourself.
So this would be quite a breaking change.

v13.2.5


Generate the JAVA SDK failure

I try to run these commands:

autorest readme.md  --csharp --azure-arm --tag=package-2022-05-10-privatepreview --output-folder=csharp_sdk/
autorest readme.md  --python --azure-arm --tag=package-2022-05-10-privatepreview --output-folder=python_sdk/

They are all succeeded.
But, when I try to run this command:

autorest readme.md  --java --azure-arm --tag=package-2022-05-10-privatepreview --output-folder=java_sdk/

It show the error like:

AutoRest code generation utility [cli version: 3.6.1; node: v16.13.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest
info    | AutoRest core version selected from configuration: ^3.7.0.
info    |    Loading AutoRest core      'C:\Users\qinqiangxu\.autorest\@autorestcore@3.8.4\nodemodules\@autorest\core\dist' (3.8.4)

info    |    Installing AutoRest extension '@autorest/java' (^4.0.24 -> 4.0.24)
info    |    Installed AutoRest extension '@autorest/java' (^4.0.24->4.0.24)
error   |   Error: ENOENT: no such file or directory, scandir 'C:\Users\qinqiangxu\.autorest\@autorestjava@4.0.24\nodemodules\@autorest\java'
error   | Autorest completed with an error. If you think the error message is unclear, or is a bug, please declare an issues at https://github.com/Azure/autorest/issues with the error message you are seeing.
fatal   | Failed to install extension '@autorest/java' (^4.0.24)

Java has lots of flags that is required for either data-plane for control-plane.
It is always recommended to use Azure pipeline for this.
See https://github.com/Azure/azure-sdk-for-java/wiki/[Management]--OpenAPI-specs for control-plane
See https://github.com/Azure/azure-sdk-for-java/wiki/Protocol-Methods-Quickstart-with-AutoRest for data-plane

Thanks, but if our API spec is still in the private repo, and if we just want to generate the SDK in the local, but without PR.
Is there any way to do that?
BTW, I want to generate the Control Plane SDK/CLI

If you can make a PR (just a draft PR), SDK automation will also generate it for you, if you have this swagger-to-sdk part in your readme.md (and include azure-sdk-for-java item)
https://github.com/Azure/azure-rest-api-specs/blob/main/documentation/samplefiles/samplereadme.md#swagger-to-sdk

If you only only do in local, go with this python script
https://github.com/Azure/azure-sdk-for-java/wiki/[Management]--OpenAPI-specs#use-python-script

If you can't use the python script, you can try this
autorest --version=3.8.4 --use=<EMAIL_ADDRESS>--java output-folder=<> --namespace=<> --azure-arm --generate-samples --fluent=lite

Make sure you have Java runtime equal or higher than 11
java -version

Finally, it is work with this command, thanks.
autorest --version=3.8.4 --use=<EMAIL_ADDRESS>--java --output-folder=java_sdk/ --namespace=Microsoft.AppComplianceAutomation --azure-arm --generate-samples --fluent=lite readme.md


Using repositoryMethod should imply inverseSide

Lately I omitted on purpose mappedBy on ReferenceMany using repositoryMethod since there was no owning side and using count() on such uninitialized PersistentCollection was stubbornly returning 0.

Unless it makes sense to persist/remove documents via such collections (using cascade) - it surely is possible (tested that)? In that case PersistentCollection::count should be fixed (https://github.com/doctrine/mongodb-odm/blob/df142796394681d0f77416ec92c66972ce0ba14b/lib/Doctrine/ODM/MongoDB/PersistentCollection.php#L522)

IMO, repositoryMethod collections are inverse-side, and should probably be considered read-only if we ever get around to implement that (I know we discussed it for standard inverse-side collections).
This issue seems like a fix for a previous oversight, so :+1: from me.

I'm also :+1: on this. As for making inverse collections read-only, there was no consensus on this yet ;)


Tables uploading with "." at end of table name

Can be fixed pretty easily by creating a list of commands like:
select
'ALTER TABLE "'||table_name||'" RENAME TO "'||substr(table_name,1,length(table_name) - 1)||'";'
from information_schema.tables where table_name like '%.'

But it would be nice if this didn't happen

Fixed in v2.0.1


Feature/cc mt service authentication

@rvsape tested "npm run dev" and works for me normally. What to do because no errors to fix for me

For me it looks fine. i have tested using npm start and npm run check. npm run dev is giving error which needs to be fix

missing dependency? npm i ts-node


az aks get-credentials failed with error : socket.gaierror: [Errno -3] Temporary failure in name resolution

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
``
Errors:

To Reproduce:
run
az aks get-credentials --resource-group MyResourceGroup --name MyClsutername

Put any pre-requisite steps here...
``

Expected Behavior
should get the credentials  , instead getting the below error
The command failed with an unexpected error. Here is the traceback:
HTTPSConnectionPool(host='login.microsoftonline.com', port=443): Max retries exceeded with url: /72f988bf-86f1-41af-91ab-2d7cd011db47/oauth2/v2.0/token (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fd9dc051a60>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution'))
Traceback (most recent call last):
File "/opt/az/lib/python3.8/site-packages/urllib3/connection.py", line 174, in _new_conn
conn = connection.create_connection(
File "/opt/az/lib/python3.8/site-packages/urllib3/util/connection.py", line 73, in create_connection
for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
File "/opt/az/lib/python3.8/socket.py", line 918, in getaddrinfo
for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -3] Temporary failure in name resolution
During handling of the above exception, another exception occurred:
Environment Summary
Linux-4.4.0-22000-Microsoft-x86_64-with-glibc2.29, Ubuntu 20.04 LTS
Python 3.8.13
Installer: DEB

azure-cli 2.36.0

Extensions:
aks-preview 0.5.39

Dependencies:
msal 1.17.0
azure-mgmt-resource 20.0.0

Additional Context

route to CXP team

@asafsofer-sl Thanks for reaching out to us and sharing this feedback. Looking at the error message, it seems like the issue is at the Network socket layer causing the connection failure. Could you confirm if there are any proxy / firewall in your corporate network ? Could you please test it from a different network and check if you still face the same issue ? Awaiting your reply.

Hi @navba-MSFT  , Yes you are totally right , look like there was security update on my laptop causing my FW to block the access.
anyway resolved now , after i regain the access by changing my FW configuration .


TypeError: Cannot set property length of # which has only a getter

This commit/line on the next branch introduced a change which is breaking my app.
Not really sure why, any ideas?

Should be fixed now


New substrate DB migration

This includes adding feed_id @boxhock ?

@ebarakos Yes, should be a migration for this: https://github.com/smartcontractkit/external-initiator/blob/feature/fluxmonitor/store/database.go#L345-L347


Building conda package from source code on pypi

Hi :)
I am currently trying to build an anaconda package from the source code you uploaded on pypi.org. Since it is on pypi it is supposedly quite easy to do so ^^.
For reference, i followed this official guide: conda-build skeleton.
During testing the build scripts expects the following path in the folder og.fmodpy: /about/version.txt
What is the purpose of og.fmodpy? You mentioned somewhere that this folder contains old code. Is it necessary to make your module work?

No it is not necessary and I should remove it. og_fmodpy is the original version of this code from years ago. It's ugly code, but it allows for a special case of passing subroutines as arguments  and that's the only reason I kept it around.
I'll remove it from the repository and publish a new version in a minute.

New version 1.3.0 is out, og_fmodpy is removed. Hope that helps!

Thanks :) I'll try again this week.


capturing deletions

Greetings,
I've noticed that alignments break over small deletions.  Is there a way to control the size of deletion an alignment can contain?
Thank you,
Zev

Hi, Zev
In short, there is no way to control acceptable insertion / deletion size. Although the X-drop threshold might be used for the purpose setting very small value, it does not seem to be a good choice because it also splits alignments on low-identity regions.
Actually, the detailed answer depends on the length of the deletion.

If it is shorter than 25 bases, the behavior might came from a bug in the alignment routine. I would appreciate if you could show the actual sequence pair that reproduce the case (and options provided to the program).

Unfortunately, if it is longer than 25 bases, the behavior is a limitation of the alignment routine. The program uses a 32-cell fixed wide banded alignment with adaptive steering technique. The algorithm is confirmed by experiment that dropping indels longer than 25bases while capturing perfectly shorter than it. (The line BW = 32, the fourth line from the left, in the Figure 2(d) shows the trend: https://github.com/ocxtal/adaptivebandbench ) Since the reason of the algorithm selection is the good performance and efficiency of the adaptive band algorithm, i'm sorry but the limitation will not be alleviated in the future...😢

Thanks,
Hajime

Hi, Zev
Minialign is now updated to version 0.3.2. In this release some bugs in the chaining routine, which made the chained path collapsed when it reached the head of the query sequence, are fixed. The chaining parameters, side lengths of the parallelogram window, are now modifiable with '-L' and '-H' flags (and the defaults are also enlarged to 5000, in order not to split chain around low-identity regions). I'm glad if you could test this new version.
Thank you.
Hajime

@ocxtal Sorry I didn't reply sooner.  Thank you for the updates.  I will re-run the alignments after the thanksgiving holiday.  What parameters would you suggest for -L and -H to maximized INDEL/SV detection.

Hi, Zev
Recommended -L and -H settings is difficult (since I'm not familiar with indel/SV calling...), hmm...

I confirmed by experiment that the default parameters -L5000 and -H5000 are large enough to stride low-identity regions in real PacBio and ONT data.
Smaller values affect on the contiguity of the resulting alignments but larger value does not. Nor on the other alignment qualities except for the calculation time.
Large indels (>= 25 bases) could not be captured in any resulting alignment due to the limitations of the alignment routine (as described in my first reply).

Currently I believe that large indel detection should be resolved in the postprocess of the local alignment and could be a preprocess of the SV detection program. However, if you say the large indels must be captured in the local alignment stage, I'll consider adding indel detection algorithm (alignment linking and gap filling) as a postprocess of the calculation of the alignment set.
Regards,
Hajime Suzuki

Hi, Zev
Just now I have figured out that the problem is: the extension alignment terminated just before the indels and the following matching regions were not reported...! (I am sorry to be late to understand...😢) I have confirmed the phenomenon on my simulated data and I'll add downstream-rescuing algorithm in the next release.
Thanks,
Hajime

Hi, Zev,
I'm sorry for my delayed reply. I've just pushed the minor update, 0.4.2, with a downstream alignment rescuing algorithm. The algorithm still fails collecting alignments after short indels, it performs much better than the previous release, 0.4.1. Please try it out.
Here are pileups of my test data.

minialign-0.4.1 (default params)

minialign-0.4.2 (default params)

bwa-mem (default params), as a reference
Thanks,
Hajime Suzuki


Dynamically change language

Hello,
is there a way to set language 'externally', for example passing a variable in url parameters?
I basically wrapped rainloop in a page and i'd like to set language per user
Thanks

Fast answer in No.
But, you may do it by using SSO functionality
<?php

// Enable RainLoop Api and include index file
$_ENV['RAINLOOP_INCLUDE_AS_API'] = true;
include '/var/www/rainloop/index.php';

$ssoHash =<EMAIL_ADDRESS>'super-password', array(
    'Language' => 'ru_RU'
));

// redirect to webmail sso url
\header('Location: http://yourdomain.com/rainloop/?sso&hash='.$ssoHash);

Unfortunately. it doesn't work right now (additional language parameter), but it will be fixed in the next version.

This code should work in the last version.


modification to new citation

Adding emphasis to title for readability of citation

+1. Love the emphasis.

+1


feat(range): add rh-range

What I did

[X]  Made rh-range its own branch so that rh-audio-player can continue without the component.

Resolve accessibility issues identified by axe DevTools:

[ ] aria-label attribute cannot be used on a rh-range with no valid role attribute
[ ] aria-hidden element must not be focusable nor contain focusable elements

Testing Instructions

Test range with axe DevTools.
Set value of range using keyboard only.
Set value of range using mouse.

Notes to Reviewers

@nikkimk this may need some revisiting, since it's old. Do you mind closing if this needs to be? Or provide more context if we're keeping this open? Thanks!

Closing this PR as stale


feat(ulimits): additional ulimit options

Description

add as, cpu, maxlogins, msgqueue, rss, sigpending options to the user_ulimit resource

Related Issue

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

Can you add some tests in for these new options please?

Hi @johnmccrae there are additions for the kitchen-tests section - this is currently the only place that ulimit options are being interrogated. Is there something you'd like me to add?

Hi @johnmccrae there are additions for the kitchen-tests section - this is currently the only place that ulimit options are being interrogated. Is there something you'd like me to add?

@chrisminton Thanks for pointing those out. I think we're good with those.

@chrisminton Can you add the test that Phil requested?

@chrisminton Looks like the Buildkite build is broken due to an old configuration file. Can you rebase on main, please?

Approving workflows and tagging for community PR review.


Installation problem on Mac for python 3

Hi I am not able to install pycrs due to the following error:
$ pip install pycrs
Collecting pycrs
  Using cached PyCRS-0.1.2.zip
    Complete output from command python setup.py egg_info:
    running egg_info
    creating pip-egg-info/PyCRS.egg-info
    writing pip-egg-info/PyCRS.egg-info/PKG-INFO
    writing top-level names to pip-egg-info/PyCRS.egg-info/top_level.txt
    writing dependency_links to pip-egg-info/PyCRS.egg-info/dependency_links.txt
    writing manifest file 'pip-egg-info/PyCRS.egg-info/SOURCES.txt'
    warning: manifest_maker: standard file '-c' not found

    error: package directory 'pycrs\elements' does not exist

Cheers

The repo copy has been fixed.  I had the same problem under Linux.

Thanks, @micahcochran:
Confirm the latest master branch works for me as well.
Cheers

@micahcochran @guziy I have now updated the pipy version with the newest repo, so pip install should work now along with all the latest bugfixes and py3 support. Also added contributors in the docs.

@karimbahgat Thank you!


Fix broken link on equality information page

Fixes a broken link that should go to the list of harmonised standards on the Government Analysis Function's website. We use the correct link further up the page

Closing cos I didn't notice Calvin open the exact same PR 5 minutes earlier 🙃


build(main): release 1.2.9

:robot: I have created a release beep boop
1.2.9 (2024-09-15)
Continuous Integration

Update version.js (59e424f)

Miscellaneous Chores

Remove unused genversion dependencies and scripts (abbbcc7)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.2.9
:sunflower:


add hdp 2.6.2-205

Add support for HDP 2.6.2-205 and <IP_ADDRESS>-40.
http://s3.amazonaws.com/public-repo-1.hortonworks.com/index.html#/HDP/centos7/2.x/updates/<IP_ADDRESS>
http://s3.amazonaws.com/public-repo-1.hortonworks.com/index.html#/HDP/centos7/2.x/updates/<IP_ADDRESS>

👍 lgtm pending travis

Travis was failing to detect rvm 2.1.4 because it only exists in Precise. My first fix was to update the rvm version to 2.1.5 and while that worked my final fix was to set the distribution to precise and match the Hadoop cookbook.
https://github.com/caskdata/hadoop_cookbook/blob/master/.travis.yml


Accept completion word by word and line by line

Hi Krassowski,
I am so excited and impressed that we could have the inline completion for the Jupyterlab!!
During the trial under the Jupyterlab 4.1 pre-release, I found the code completion was smooth and fast. However, it seems that completion can only be accepted chunk by chunk (across several lines). there were so many completions. Many times, they are not 100% percent what I want, and such many completion lines can be disatractive.
therefore, it would be great if we could accept the completion word by word and line by line, like in VScode GitHub copilot.
Best regards

Thank you for the feedback @EricHUANG970823. I agree, we should have these. Since acceptance is implemented in JupyterLab itself, would you mind opening an enhancement request over at https://github.com/jupyterlab/jupyterlab/issues?

Thanks. New enhancement request has been opened via her https://github.com/jupyterlab/jupyterlab/issues/15531#issue-2041413602

Thank you!

Closing as this is tracked in https://github.com/jupyterlab/jupyterlab/issues/15531.


queries: clojure has-type? updated to kind-eq?

A suggested fix for the nvim-treesitter breaking change, which renames the has-type? query to kind-eq? query
https://github.com/nvim-treesitter/nvim-treesitter/commit/a80fe081b4c5890980561e0de2458f64aaffbfc7
From a text search of the nvim-treesitter-sexp project, there was only one has-type text match, so I changed that to kind-eq
This fix seems to work with the setup described in #6  (Neovim 0.10.0 and AstroNvim config) and with the latest (as of 3 days ago) nvim-treesitter plugin.
I dont known if this is the right way to solve the issue, but I can confirm it works.
Resolve #6

Looks right to me, the predicate just needed renaming. Good job, I'll merge


Fix incorrectly overridden project groups by JvmTestSuite.

Motivation:
group was set in JvmTestSuite's configuration to change the test task's group, but Project.group was changed unintentionally. https://github.com/line/armeria/blob/e15c7b7809db6d77c6798c237707e942fceb51bb/core/build.gradle#L178-L180
The overridden group caused to publish Armeria artifacts to Verification org in Sonatype snapshot storage.
https://github.com/line/armeria/actions/runs/5010142242/jobs/8979748543#step:8:1273
Modifications:

Override the group of a test task in testTask.configure block.

Result:
Fixed unintentionally overridden Gradle project group information.

Thanks for the quick review. 🙇♂

The publishing task successfully finished.


[BEAM-2212] Cherry-pick #2956 into release-2.0.0 branch

R: @davorbonaci
May not be worth CPing, it's just improving LOGs and error messages. Thoughts?

Changes Unknown when pulling 1e2b6dbaacde05d4074172df0ed77a6ffbac83b6 on dhalperi:cp-2956 into ** on apache:release-2.0.0**.


Project Logo?

If not, feel free to close this ticket :-)
In the mean time, I'll share some ideas I had ...

I'm pasting some sketches I did exploring what might be interesting. I had a few others that I didn't add here:

one, an all-caps "SLICK" with the "L" replaced with a stylised lambda (looks like an inverted "Y")
another was an italicised all-caps "SLICK" similar to the new "GO" logo, complete with three dashes.

Here are the sketches I finished:

The three textual ones only have one finished; I chose to demonstrate on the font I found the nicest (obviously inspired by the Lisp "worm" logo).
TBH, one of these really "do it" for me ... though, I'd kinda like to experiment more with the "slick when wet" one:

get rid of the "CAUTION"
explore more stylised renditions of the lambda and the three dashes
maybe go a little more abstract, in general ... dunno

I like the "slick when wet" idea (but I agree it still needs some work - says I, who has no graphical design skills whatsoever :). What about a "Gopher in parentheses"?

Oh, that's a great idea! Should be able to adapt this one here quite easily:

https://github.com/golang-samples/gopher-vector/blob/master/gopher.ai

Okay, here are some iterations on that theme:

What cracks me up about the "Lisp eyes" variants is the quasi-psychedelic nature reminiscent of Jake from "Adventure Time" 😆

I love the direction where this is heading! :)

So, taking more inspiration from "Adventure Time", in particular:

May I present, ~Jake~ Slick the Gopher:

"It's soooooo beautiful ...."

Here's the .svg file I'm working with ... there are lots of layers for each part and variation, so simply hiding them should let you easily examine different options:

https://www.dropbox.com/s/dujo1hlwfi2ioyr/sketch-7.svg?dl=0

I've only tested the file with Affinity Designer; the layers should be preserved across platforms and graphics applications (my Inkscape no longer works, so I can't test it on that).


pkg/smi,kubernetes: unexport package scoped types

Description:
Makes a few exported types and variables that are
package scoped to be scoped within the package.
Signed-off-by: Shashank Ram<EMAIL_ADDRESS>
Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [ ]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?
No

Codecov Report

Merging #2566 (c183a22) into main (b819b3e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2566   +/-   ##
=======================================
  Coverage   62.70%   62.70%
=======================================
  Files         152      152
  Lines        7027     7027
=======================================
  Hits         4406     4406
  Misses       2604     2604
  Partials       17       17

Flag
Coverage Δ

unittests
62.70% <100.00%> (ø)

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

Impacted Files
Coverage Δ

pkg/kubernetes/types.go
0.00% <ø> (ø)

pkg/kubernetes/client.go
87.12% <100.00%> (ø)

pkg/smi/client.go
77.44% <100.00%> (ø)

Continue to review full report at Codecov.

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


fix(obstacle_avoidance_planner): fix lat_dist_to_front_bound comparison

Signed-off-by: Mehmet Dogru<EMAIL_ADDRESS>Description
Currently drivable area check for traj footprints in MPT doesn't work. Because lat_dist_to_front_bound (lateral distance of footprint corners to front bound) is always positive.

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kosuke55 @takayuki5168 to please review

@purewater0901 Could you check this PR.

Yes, you're right! Thank you for your contribution!

Thank you @purewater0901 :pray:
If anybody from the code owners can review as well, we can merge it :100:

@mehmetdogru I think @kosuke55 just approved this PR, so you can merge it


[BUG]: arrayConnectivityPollRate Argument is required

Describe the bug
Initially started without these arguments and tested the SDC network down scenario on worker node and found that Podmon was taking no action.  But then after adding them it works fine.
- "-arrayConnectivityPollRate=5"
- "-arrayConnectivityConnectionLossThreshold=3"
The document mentions that they are optional, shouldn’t it be a mandatory parameter ?
Also if they are not configured what is the default value for polling and threshold ?
To Reproduce
Steps to reproduce the behavior:

Deploy CSI without these two arguments.
Bring down SDC(storage array) network.
Validate if podmon is taking action on the pods to cleanup.
...
n. Step n See error

Expected behavior
Exepect podmon to take action even when the arguments are not configured.
System Information (please complete the following information):

OS/Version: SLES 15.1 (15SP1)
Kubernetes Version: 1.20.4
Additional Information...

Additional context
Add any other context about the problem here.

Both Ameer and I tried to recreate this issue, but could not. In my setup here were the arguments to podmon container:
Containers:
podmon:
Container ID:  docker://f5f4e784f5db5f669166940d7cc65dd8e68d842df38d9d24fb5bb781be44cd5c
Image:         docker.io/dellemc/podmon:v0.1.0
Image ID:      docker-pullable://dellemc/podmon@sha256:4532aaf77fbf66148ee641a3221660ab026fb108711a86a380885de0908f6428
Port:
Host Port:
Args:
-csisock=unix:/var/run/csi/csi.sock
-labelvalue=csi-vxflexos
-mode=controller
State:          Running
I ran this on a system with two interfaces per node and failed only the array interface and the fail-overs completed normally.
If you see this behavior persisting, can you please reproduce it and sent the tar file generate by the collect-logs.sh script after trying a failover as well as the podmon section of you values.yaml or myvalues.yaml used to start the driver. Thanks, Tom

Thanks for the response Tom and team, yes we shall come back with logs.  In the meantime, could you please confirm your environment details you tested it and see it working ?
OS:
k8 version:
Steps: (Executed to test)

Surely.
OS: Centos 7.8, running in Vsphere VM, two interfaces per VM, one for the k8s control plane, the other public to the array.
k8s version: 1.20.5
Steps:

Uninstall/re-install driver with the changed podmon block definition shown above.
Deploy some protected pods.
Verify in in the vxflexos namespace (i.e. the driver) the active (i.e. pod that holds the leases and is elected leader) is emitting the polling messages. Note- these are only emitted for nodes that are protecting pods. We don't ask the driver about nodes that have no protected pods.
We deploy a script test/sh/bounce.ip to bounce the interface(s) we desire to every worker node. It is configured on each node to only bounce the interface to the vxflex (interfaces list in the script). Note the interface goes down for a time and then automatically comes back up.
Run the script test/sh/failnodes.sh to fail the particular worker node. It does this by invoking the bounce.ip script on that node.
Observe using script tools/mon.sh that a) the node never goes NotReady, but b) the protected pods on that node are moved to another node and start.

Hello Team,
This was re-tested without explicitly defining the arguments and it works fine now without any error.  Hence this can be closed.

Thank you... closing.


Fixes future::Shared's Send/Sync bounds (Issue #952)

Send/Sync where implemented even if Shared
should not have been Send/Sync (Issue #952)

I took a look at the failed checks, and it seems they are
unrelated to the pull request, basically it seems as if
some nightly parts broke, as there are:

errors wrt. proc_macro
errors wrt. ui/error message tests

Thanks for the fix!


Add tracking attributes to links in card block

Add tracking attributes to links in card block

Resolves: MWPW-134082
Test URLs:

Before: https://main--milo--ruchika4.hlx.page/drafts/ruchika/card-analytics?martech=off
After: https://analytics--milo--ruchika4.hlx.page/drafts/ruchika/card-analytics?martech=off

CC

Before: https://main--cc--adobecom.hlx.page/creativecloud/animation/discover?martech=off
After: https://main--cc--adobecom.hlx.page/creativecloud/animation/discover?milolibs=analytics--milo--ruchika4&martech=off

@hparra listed some valuable info on how to get all the PR checks green: https://github.com/orgs/adobecom/discussions/997
Please this edit future PRs accordingly, I added the correct label for now


initial docs for make commands

@rhatdan
@MichaelClifford
These READMEs can be templated in the future, but for now I've duplicated information

There are a few podman build and podman run commands that should also be replaced with their make build/run targets, I didn't test those though so didn't replace them here.

LGTM


Remove Old Signs After Placing New Ones

In s:Make(), it looks like all signs are removed before new ones are added. This causes the sign column to briefly flash off and on after Neomake finishes running, which is a bit annoying to the user. Would be nice if Neomake placed new signs before removing the old ones to prevent this.

Hmm, I agree that it's a bit jarring to have the list disappear, then reappear, but I think that maybe it would be even better if when the old gutter is removed, show a blank gutter but no errors, then once neomake completes, replace the blank gutter with the new gutter with the new signs, creating like a buffer

Closing as dupe of #516.
@theamazingfedex

but I think that maybe it would be even better if when the old gutter is removed, show a blank gutter but no errors, then once neomake completes, replace the blank gutter with the new gutter with the new signs, creating like a buffer

That is the current behavior isn't it?
Will have to investigate, but I thought it would clear signs when the job starts already?!

The signs do clear in the gutter when the job starts, but I think the main issue @jebaum was getting at is the jarring effect caused by the gutter being removed and redrawn when jobs are run.

So it should not replace existing signs, but keep them in case they would appear at the same place again?

The issue is that the gutter is removed when the job begins and is immediately redrawn (if there are still errors). This causes the window to shift, which is quite jarring. If it's possible to keep the gutter open until the job completes and removing it only if there are no errors/warnings, I think that would fix the issue.
In addition, it would be nice if the quickfix buffer acted the same way. In that it would only disappear once a job finishes successfully.

So I looked into this issue a little bit more. Being unfamiliar with vim's development tools, I didn't know about the sign command, and how neomake uses it. Apparently this was an issue in mainline vim that was resolved recently (see: https://github.com/vim/vim/issues/117).
I'll file an issue upstream to see if this can be added to neovim as well, but I don't believe anything needs to be done on your end @blueyed.

Thanks for investigating!
The patch should come to Neovim "automatically", no need to create an issue.
I still think we can improve this (place signs later, do not delete+readd).


Finalize and test V2 NFT Storefront code

Context
We have a new proposal for a second version of NFT storefront. We need to finish it and make sure it is thoroughly tested
Definition of Done

Thoroughly review and discuss https://github.com/onflow/temp_nft-storefrontV2, looking for bugs and potential improvements.
Make a PR in the main nft-storefront repo with the finalized contract.
Write tests in https://github.com/onflow/nft-storefront/tree/main/lib/go/test

@albeethekid @satyamakgec Have y'all found a time to discuss this and finalize the proposal yet? It would be great if Satyam could start working on testing it.


[DC-2130] Refactor DRC street address comparison

@nishanthpp93
Thank you so much for reviewing this PR, and thanks for your time yesterday for the discussion! Could you please have a look at it again? Please let me know if you have questions or anything. Thanks!

@nishanthpp93 Thanks again for the review! I solved the merge conflict. Please let me know if there is anything else!


No version found for react/http

Could not find dev version from react/http which satisfies this. Has reactphp libraries changed since your blog post
Tried with this and also with hand made composer.json:
composer require "react/http:^0.5@dev" phly/react2psr7
Problem 1
    - The requested package react/http ^0.5@dev exists as react/http[0.3.x-dev, dev-WyriHaximus-php7-hhvm-travis, dev-compat-guzzle-deps, dev-master, dev-streaming-multipart, v0.1.0, v0.1.1, v0.2.0, v0.2.1, v0.2.2, v0.2.3, v0.2.6, v0.3.0, v0.4.0, v0.4.1] but these are rejected by your constraint.
  Problem 2
    - Installation request for phly/react2psr7 0.1.1 -> satisfiable by phly/react2psr7[0.1.1].
    - phly/react2psr7 0.1.1 requires react/http ^0.5@dev -> no matching package found.

The same problem. There is no such version in repo.

Try to alias dev-master of react/http to 0.5, i.e., "react/http": "dev-master as 0.5". I think this is what the Readme tries to say in the installation section.

Try to alias dev-master of react/http to version 0.5, i.e., "react/http": "dev-master as 0.5". I think this is what the Readme tries to say in the installation section.

Imho not working. Neither master nor 0.5 seem to have the require getFiles()or getBody()methods. Too bad, but imho not usable right now?

@andig, @back-2-95, et al: React has changed since I wrote the blog post, and this is the danger of creating a library against something pre-1.0.
I'll see if I can identify what needs to change in order to make this work.

Might make sense to wait for 0.5 or 0.5.1- it recently seems to have gaind some traction again ;)

Ugh... the changes to remove getFiles() and getBody() mean that this library will likely require a parser of some sort, which I was hoping to avoid. I'll wait and see if they re-introduce those methods before doing too much work here.


Add adapt credentials endpoint

Pull Request Template
Prerequisites

This relates to #167 and implements the adapt_credentials endpoint.

Motivation
One step closer to implementing the issue #167
Approach
It forwards the additional endpoint adapt_credentials to the service broker.
Pull Request status

[x] Initial implementation
[x] Unit tests

Third-party code
n/a
Work in Progress label
n/a

@dzahariev as discussed with @loewenstein

Pull Request Test Coverage Report for Build 1473

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 89.302%

Totals

Change from base Build 1469:
0.003%

Covered Lines:
2813

Relevant Lines:
3150

💛  - Coveralls

FYI @fmui


fix: ensures environment variables are imported

What is this PR doing?
PR https://github.com/Shopify/function-examples/pull/104 removed the automatic loading of the environment variables that the shopify cli generates when an app is deployed which in turn broke the discount-functions-sample-app as those variables are required in some of the pages (example).
This PR adds that functionality back with a simplified approach. It uses the standard config options that Vite provides to automatically load any environment variable prefixed with SHOPIFY and the default VITE_.
Important to note that this approach loads and exposes ALL environment variables prefixed with SHOPIFY_ to the frontend which includes the API_SECRET as well. If that isn't desired just let me know and I will change the approach.
🎩
I tested this on my development store both locally and in prod.

Like you mentioned, it looks like the SHOPIFY_API_SECRET is also passed to the frontend

I noticed that SHOPIFY_API_SECRET wasn't on the .env and thought it was weird that it was still being exposed to the frontend. So I did some digging and apparently Vite will also expose values from process.env that starts with the prefixes from the envPrefix list.
So unfortunately I don't see another way besides picking the variables we want manually, like we were doing before.


opslevel_service: tag validation fails with datasource or variable

versions:
$ terraform version
Terraform v1.8.4
on linux_amd64
+ provider registry.terraform.io/opslevel/opslevel v1.2.1

reproduce:
terraform {
  required_version = ">= 1.0"
  required_providers {
    opslevel = {
      source  = "OpsLevel/opslevel"
      version = ">= 1.2.1"
    }
  }
}

provider "opslevel" {
  api_token = "..."
}

resource "terraform_data" "test" {
  input = "bar"
}

resource "opslevel_service" "db" {
  name        = "jpo-service"
  owner       = "team-sre-emea"
  # illustrates the correct key:value format:
  description = "foo:${terraform_data.test.output}"
  # CAUSES ERROR:
  tags        = ["foo:${terraform_data.test.output}"]
}

Expected:
service created with tag foo:bar
Actual:
$ terraform apply -auto-approve
╷
│ Error: Invalid Attribute Value
│
│   with opslevel_service.db,
│   on main.tf line 25, in resource "opslevel_service" "db":
│   25:   tags        = ["foo:${terraform_data.test.output}"]
│
│ Attribute tags list must contain elements with 'key:value' format, got: 1

Notes:

any datasource will trigger the error, terraform_data is just for simplicity (our actual use-case is the aws_rds_cluster datasource)
using a variable instead of a datasource also triggers the error:variable "test" {
  type = string
  default = "bar"
}

resource "opslevel_service" "db" {
  #...
  tags        = ["foo:${var.test}"]
}

there is no plan output whatsoever, therefore it seems validation is running before any datasource read occurs
commenting-out the tags = ... line allows the service to be created with a description that illustrates the correct key:value format

👋 Hi @jpo-tu, thank you for raising this issue! The fix for this is in and will be part of the next release.


uv pip compile|install can't handle --no-binary lxml option in requirements file

This obviously works on pip + pip-tools which we use.
Our relevant base.in:
# other dependencies

# see https://github.com/onelogin/python3-saml/issues/292 and
# https://bugs.launchpad.net/lxml/+bug/1960668 -> we can avoid this by compiling lxml
# against the system libxml2
--no-binary lxml
lxml

# other dependencies

which results in the base.txt:
#
# This file is autogenerated by pip-compile with Python 3.10
# by the following command:
#
#    ./bin/compile_dependencies.sh
#
--no-binary lxml

# ... actual dependencies

pip-tools can produce this file, but uv pip compile doesn't understand this option:
(open-forms) ➜  open-forms git:(chore/1052-remove-defusedxml) uv pip compile requirements/base.in
error: Unexpected '-', expected '-c', '-e', '-r' or the start of a requirement in `requirements/base.in` at position 306

Details
Repository is https://github.com/open-formulieren/open-forms so you can take a look at the requirements files yourself (and the ./bin/compile_dependencies.sh script).
$ python --version
Python 3.10.13
$ uv --version
uv 0.1.2

Running in a pre-existing virtualenv.

I should've searched better, this is related to #1332 and #977

Ah yeah, I think this is a separate issue. We don't support all flags in requirements.txt, only a subset. We could consider adding this one.

+1
All my projects with postgres I use a smaller lib from psycopg2 and uv doesn't support --no-binary yet
psycopg2==2.9.9 --no-binary psycopg2

@felipewove -- Just to clarify, we do support --no-binary on the command-line, like uv pip install psycopg2==2.9.9 --no-binary psycopg2. We just don't support it within a requirements file yet.

@charliermarsh got it, thanks!
I have it automated on my containers, so I'll make some workaround just for some env to test UV ;)

Hi is there any update on supporting --no-binary in a requirements file?

I can probably add support for this, it's come up multiple times now.


Front-End Week Timeline

Building navigational timeline which will change the state of the app on week selections.

Accordion working as expected. need to add some sort of arrow on the left of the timeline to match design. Possibly need to rework state variables for integration.  Restyle everything later.

Accordion working as expected. need to add some sort of arrow on the left of the timeline to match design. Possibly need to rework state variables for integration.  Restyle everything later.


configure pnpm changeset

@aka-toxa play locally with changeset flow:

pnpm changeset within the root
choose package to update version for and fill up all steps
pnpm changeset version - apply changes/bump all deps/provide changelog
merge to main - pipeline should cover everything else

Looks good, I will try this on my local machine and then merge it. Thank you for your contribution!

@zzz1ck
LGTM but can you please add pnpm changeset to the pre-commit hook?

husky doesn't support user input on pre commit :(


Swap to singleton function

Thanks for the awesome construct, this little PR addresses two issues when I encountered when I tried to use it in a multi-stack app:

Removes the hardcoded function name, allowing CDK to control the name (this is best practice).

Using a SingletonFunction so apps with multiple stacks don't need to deploy multiple copies of the same lambda.

Finally, would you consider matching your release versions to match the CDK dependency version? This makes it easier for users to install the correct version of this lib to match their underlying CDK version. We use this to great effect here.

@BDQ
Thanks for this contribution!  I appreciate the help in making this construct more robust.  👍👍
I am currently setting up the CI/CD GitHub Action for this repo.  After I complete this, I'll work on merging this.  Thanks!

@jmb12686 delighted to see your onboard with the versioning suggestion too!


Correctly track machines (localhost)

Tracking of connected machines should be done by Mac address or something more reliable than hostname and IP (as is the current case).
Sometimes, connecting to self renders two separate entries (localhost and the LAN IP), which is incorrect due to unreliable (self-)identification.

Test on mobile devices as well


Refactor Module -> Pallet

The term module, used a lot in FRAME is historically linked to SRML.
Not a big deal, and probably lots of people are used to the current naming, but it is technically wrong. Now the term module is replaced with pallet. Hence,

decl_module! -> decl_pallet!.
struct Module -> struct Pallet.

I think both can be addressed by https://github.com/paritytech/substrate/issues/5678 cc @thiolliere

I don't think that we should touch any of the old code.
You can bring in your ideas in the proc-macro conversion pr, but we should not break the old code. Let it stay and we will deprecate it anyway with the merge of the new proc-macro macros.

Yes I think the PR that closes https://github.com/paritytech/substrate/issues/5678 should also close this.

Sounds like we can close this issue directly? There is no real reason to have 100  issues open for one pr. If you want something specific, I propose that you comment on the original issue or directly in the pr.


Request logo

freedom in the crypto world is important, the most important goal is to make a strong foundation in our tokens.
making tokens is easy, very difficult, how to achieve that goal can be made and realized real.
we have made a planning project that we have tested in a work simulation, and we believe our tokens are one of the future, which will make all happy.
We are a team that is experienced in the field of crypto trading and we see many tokens that are not too fast in market action, and we see that and want to realize our dreams so that everyone is free in crypto trading on some market crypto
(we are a team experienced in crypto trading, and created the crypto freedom project)
The main purpose
A. Creating crypto trading convenience (RXCC)
B. Crypto advantages that can be determined
C. All control is in the hand
https://cryptofreedom.ml
<EMAIL_ADDRESS>https://twitter.com/RXCCrypto
https://t.me/cryptofreedomgroup
https://medium.com/@RXCCcryptofreedom
https://github.com/RXcryptofreedom
gas limit: 60000

你好，代币信息提交有误，请参考此教程 https://github.com/consenlabs/token-profile/blob/master/README.zh-CN.md，
建议和你们的技术和设计协作完成资料的提交。


Refactor extension points

Description
On top of #2.

Move Extension to the SDK root, matching OpenSearch Java code.
Moved api/Extension and api/ActionExtension. Now, an extension implements a collection of extension points via multiple inheritance.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dbwiddis Take a look? This matches closer what we have in the Java code.


add osm view for merging healthsites and osm

this PR continue PR : https://github.com/healthsites/healthsites/pull/1057
Will fix the travis test
PR updated:

update docker osm to deploy it automatically (although i'm still not sure how to link the database, instead using IP adrress for now)
create view to merge osm data and healthsites

API:

The admin:

How admin detail looks

and also able to import osm data from node and way

Add 2 API
osm and also v2
osm is getting osm healthcare, with additional attributes from healthsites that is not found on osm

Coverage increased (+2.7%) to 42.017% when pulling 3af01d4d6654c213e0218f23f3397b3ba8c5caad on meomancer:api-v2 into 09a01d80504a51e71756ddd0151f3fae52d9dcac on healthsites:develop.

Add 2 API
osm and also v2
osm is getting osm healthcare, with additional attributes from healthsites that is not found on osm

Hi is this screenie duplicated?

Updated schema of API:

@meomancer can we get this merged please and work with @nixilla to get it deployed on the staging server


can i add tests?

i noticed there aren't any tests, can i add some using jest? love the project 👍

@ImLunaHey ... I am working on a new version 6 (will take still some time - hope to make first release in December) and there I have some basic tests. If ok, would you like to have a look on that as soon as I have a first version out?

@sebhildebrandt is there a branch i can follow? i noticed the v6 branch is very out of date on here.

@sebhildebrandt do you have date set for the release or just going by ear?

@ImLunaHey I am planning to make first beta version available in the second week of January. I am currently making some more smaller things in version  5 and in parallel try to make V6 working and reattach all changes that happened since I have started with V6. I had a longer break working on this due to private reasons in my family but finally got back to work on it. The code you see in the V6 branch is pretty old, I didn't pushed for a long time but I did a lot of work in the meantime. As soon as I see that code quality is good, I will push it again.

@sebhildebrandt any chance you have a version of v6 you're ready to push?

@sebhildebrandt any chance you have a version of v6 you're ready to push?


Sharing via API

Is it possible to share via Power BI REST APIs?

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

ID: 21f47742-00cf-eac0-7880-18eb5e988003
Version Independent ID: 0010dce8-b1e0-b1b4-9840-ff2c3e32b53e
Content: Share Power BI dashboards and reports with coworkers and others - Power BI
Content Source: powerbi-docs/service-share-dashboards.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @alexb5dh -- Thanks for your question! I don't know the answer. I'll check and get back to you.

Hi, @alexb5dh -- here's your answer: Sharing, as in directly sharing a report with a user is not available, but sharing as in adding a user to a workspace is: https://docs.microsoft.com/en-us/rest/api/power-bi/groups/addgroupuser
That currently supports adding a user as an admin only.
Hope that helps!


Migrate release notes

Release notes for the older versions should be migrated from internal Adobe github.

@filmaj @stevengill do you have a script to automate this?

You mean the annotations attached to releases as listed over in https://git.corp.adobe.com/Spectrum/spectrum-css/releases ?

Hmm those seem to have been migrated over just fine? https://github.com/adobe/spectrum-css/releases

It is just showing release notes for the latest versions added manually.

Ah yes, I see. @trieloff has blogged about how to move issues/labels/milestones on our very own tech blog. In there we reference this project: https://github.com/ahadik/git_mover
I could take a look at adding migrating release notes support to it?

I'll run it

Done. Thanks @filmaj


Automatically deploy test applications on each commit

We should automatically deploy test applications on each commit. We can do this as part of our main build, or in a separate build triggered on each commit. This will help with things like testing Xamarin and multiple browsers.

Also, add Commit hash endpoint to both apps

Repo specific tasks are done for this. Now we need to work on the automated publish per commit to dev


UpdateList.prototype.remove issue

remove: function (child)
{
var index = this._list.indexOf(child);
    if (index !== -1)
    {
        this._list.splice(index, 1);
    }

    return child;
},

I think should push the child into _pendingRemoval list but not directly remove from the active list? otherwise _pendingRemoval becomes useless?
and if use _pendingRemoval I think the indexOf check is unnecessary here because later it will be checked in the preUpdate method.
you think? or is this designed originally like so?

It should go in the pendingRemoval array.

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


cheat doesent starting Connected to active game error

am open the cheat but Connected to active game always no

@MTMHA fixed at https://github.com/Blaumaus/le_chiffre/releases/tag/v1.4.24


Add in support for retina gravatars

see my PR #25

Thanks - looks great

This is now released as version 0.7.2


helm server install should install dm into “helm” namespace

This is a polish item from #474

@michelleN I can take this one on if you want.

Fixed by #544


ImportError: cannot import name 'opentype' on ubuntu 16, python3.6

Im getting error after upgrading from ubuntu 14 to ubuntu 16. My development server is on ubuntu 14 which has no issue, but now i upgraded my local system with ubuntu 16, and i started getting this error for pip install --upgrade google-cloud-storage package.
Tried pip install --upgrade pyasn1-modules. but same error.
Here are system details and error traceback:

Google Cloud Storage

Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.4 LTS
Release:	16.04
Codename:	xenial

3.Python 3.6.3, pip 10.0.1

$ pip show google-cloud-storage output:
Name: google-cloud-storage
Version: 1.8.0
Summary: Google Cloud Storage API client library
Home-page: https://github.com/GoogleCloudPlatform/google-cloud-python
Author: Google LLC
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.5/site-packages
Requires: google-api-core, google-cloud-core, google-resumable-media
Required-by:

pip freeze output
aioredis==1.0.0
asgiref==2.3.0
asn1crypto==0.24.0
async-timeout==2.0.1
attrs==17.4.0
autobahn==18.4.1
Automat==0.6.0
boto3==1.7.7
botocore==1.10.7
cachetools==2.0.1
certifi==2018.4.16
cffi==1.11.5
channels==2.1.1
channels-redis==2.1.1
chardet==3.0.4
click==6.7
constantly==15.1.0
cryptography==2.2.2
daphne==2.1.1
decorator==4.3.0
Django==2.0.4
django-cors-headers==2.2.0
django-fcm==0.1.1
django-filter==1.1.0
django-material==1.2.3
django-simple-history==2.0
django-uuidfield==0.5.0
djangorestframework==3.8.2
djangorestframework-jwt==1.11.0
docutils==0.14
fcm-django==0.2.18
future==0.16.0
geocoder==1.38.1
geographiclib==1.49
geopy==1.13.0
google-api-core==1.1.1
google-auth==1.4.1
google-cloud-core==0.28.1
google-cloud-storage==1.8.0
google-resumable-media==0.3.1
googleapis-common-protos==1.5.3
hiredis==0.2.0
hyperlink==18.0.0
idna==2.6
incremental==17.5.0
ipgetter==0.7
jmespath==0.9.3
msgpack==0.5.6
mysqlclient==1.3.12
Pillow==5.1.0
protobuf==3.5.2.post1
pyasn1==0.4.2
pyasn1-modules==0.2.1
pycparser==2.18
pyfcm==1.4.5
PyJWT==1.6.1
python-dateutil==2.7.2
pytz==2018.4
ratelim==0.1.6
requests==2.18.4
requests-toolbelt==0.8.0
rsa==3.4.2
s3transfer==0.1.13
six==1.11.0
Twisted==17.9.0
txaio==2.10.0
urllib3==1.22
zope.interface==4.5.0

Traceback

[Wed Apr 25 18:43:12.082126 2018] [wsgi:error] [pid 25603] Traceback (most recent call last):
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/auth/crypt/rsa.py", line 20, in <module>
from google.auth.crypt import _cryptography_rsa
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/auth/crypt/_cryptography_rsa.py", line 40, in <module>
raise ImportError(_IMPORT_ERROR_MSG)
ImportError: cryptography>=1.4.0 is required to use cryptography-based RSA implementation.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/django/core/handlers/exception.py", line 35, in inner
response = get_response(request)
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/django/core/handlers/base.py", line 113, in _get_response
resolver_match = resolver.resolve(request.path_info)
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/django/urls/resolvers.py", line 494, in resolve
for pattern in self.url_patterns:
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/django/utils/functional.py", line 36, in __get__
res = instance.__dict__[self.name] = self.func(instance)
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/django/urls/resolvers.py", line 536, in url_patterns
patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/django/utils/functional.py", line 36, in __get__
res = instance.__dict__[self.name] = self.func(instance)
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/django/urls/resolvers.py", line 529, in urlconf_module
return import_module(self.urlconf_name)
File "/usr/lib/python3.5/importlib/__init__.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "<frozen importlib._bootstrap>", line 986, in _gcd_import
File "<frozen importlib._bootstrap>", line 969, in _find_and_load
File "<frozen importlib._bootstrap>", line 958, in _find_and_load_unlocked
File "<frozen importlib._bootstrap>", line 673, in _load_unlocked
File "<frozen importlib._bootstrap_external>", line 665, in exec_module
File "<frozen importlib._bootstrap>", line 222, in _call_with_frames_removed
File "/home/friendcolony/python_workspace/pycharm_workspace/FriendColony/FriendColony/urls.py", line 21, in <module>
from user_mgmt.views import GetAccessTokenURL, fb_user_login_with_id, otp_verify, reset_password_link, \\
File "/home/friendcolony/python_workspace/pycharm_workspace/FriendColony/user_mgmt/views.py", line 68, in <module>
from google.cloud import storage
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/cloud/storage/__init__.py", line 40, in <module>
from google.cloud.storage.client import Client
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/cloud/storage/client.py", line 22, in <module>
from google.cloud.client import ClientWithProject
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/cloud/client.py", line 27, in <module>
from google.oauth2 import service_account
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/oauth2/service_account.py", line 77, in <module>
from google.auth import _service_account_info
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/auth/_service_account_info.py", line 22, in <module>
[from google.auth import crypt
[File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/auth/crypt/__init__.py", line 39, in <module>
[from google.auth.crypt import rsa
[File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/auth/crypt/rsa.py", line 27, in <module>
[from google.auth.crypt import _python_rsa
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/google/auth/crypt/_python_rsa.py", line 26, in <module>
from pyasn1_modules.rfc2459 import Certificate
File "/home/friendcolony/python_workspace/Virtual_EnvPy36/lib/python3.6/site-packages/pyasn1_modules/rfc2459.py", line 20, in <module>
from pyasn1.type import opentype
ImportError: cannot import name 'opentype'

Any Help..?

Your version of pyasn1 it too old. I'm not sure why pip install --upgrade pyasn1 isn't fixing it, but that is the root of the problem.  pyans1-modules requires pyasn1>=0.4.1,<0.5.0.
It's even weirder that your pip freeze output lists 0.4.2 as the pyans1 version, so I really don't know what's occurring here.
I could not reproduce with Ubuntu 16.04 and the apt-provided Python 3.5. My pip freeze looks like this:
cachetools==2.0.1
certifi==2018.4.16
chardet==3.0.4
google-api-core==1.1.1
google-auth==1.4.1
google-cloud-core==0.28.1
google-cloud-storage==1.8.0
google-resumable-media==0.3.1
googleapis-common-protos==1.5.3
idna==2.6
pkg-resources==0.0.0
protobuf==3.5.2.post1
pyasn1==0.4.2
pyasn1-modules==0.2.1
pytz==2018.4
requests==2.18.4
rsa==3.4.2
six==1.11.0
urllib3==1.22

I'm going to go ahead and close this, but I'm happy to keep discussing and working through this with you.


3.0.3 fails to compile

steps to reproduce:

pick a Linux host
pick a non-privileged user account that has no rights to write into the root directory
run:
$ mkdir -p build
$ cd build
$ cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/home/username/install/ArangoDB &&
make all install

expected:
it should compile and install into specified directory
actually:
it failed to install in attempt to create /init.d directory in the root
cause:
cmake script uses ETCDIR which I didn't find among GNUInstallDirs. Latter has SYSCONFDIR. Did you by mistake type ETCDIR instead of SYSCONFDIR?
workaround:
add -DETCDIR=/home/username/install/ArangoDB/etc
into commandline arguments for cmake

ArangoDB tries to install itself as a service and requires root rights.
You can create a package with make package or cpack -G STGZ (performed in build folder) and then install in any location with write access.
I mean the title is wrong; "Fails to install" would be better.

servusoft please re-read the report once again.
It has nothing to do with root. It has nothing to do with installation.
It's merely cmake script related problem.

Ok, the fault on 3.0.3 confirmed:
'# get
git clone -b 3.0 git://github.com/arangodb/arangodb.git
'# config
cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/home/user/install/ArangoDB
'# compile use 4 core
make -j4
'#install
make install >inst.log 2>err.log
'# err.log:
CMake Error at cmake_install.cmake:64 (file):
  file cannot create directory: /init.d.  Maybe need administrative
  privileges.

make: *** [install] Error 1

However on 3.0.x (devel branch) is no error was apparently fixed.

@dmitri-- I think we already fixed that in devel, we removed parts doing those special directory calculations of the directories
Can you retry with devel to confirm?

tried the clone and yeah it  worked, no problems with make install trying to create directory in root.
The only two things concern me:

is it possible to avoid rewriting rpath for the compiled binaries while running  "make install"? This /lib64 dir for stdlibc++ and libgcc render the binaries useless.

check these 3 lines in the daemon config:
[cluster]
...
arangod-path = @SBINDIR@/arangod@PROGRAM_SUFFIX@
dbserver-config =<EMAIL_ADDRESS>coordinator-config =<EMAIL_ADDRESS>

What are @SBDIR@, @SYSCONFIGDIR@ and @PROGRAM_SUFFIX@?

@dmitri-- do the cmake options help with (1) https://cmake.org/Wiki/CMake_RPATH_handling ?

regarding these config lines, they were leftovers from 2.8;  If you have a look at etc/arangodb3/arangod.conf.in - there are more of them which are still used and thus replaced during the install process.
In windows some of these markers are replaced during arangodb starttime, since you can't tell the install directory during configure / compile time.

does this still persist with current devel? Else may we close this?
Meanwhile we have:
https://github.com/arangodb-helper/build-docker-containers
which we use for our internal CI;  It could be used to aid compiling.

Major work was done on the cmake build system; so please open a new bug if something similar still persists. Closing.


Automatic PR from RestAPI-PR1849

Generated from RestAPI PR: https://github.com/Azure/azure-rest-api-specs/pull/1849

Codecov Report

Merging #1540 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1540      +/-   ##
==========================================
+ Coverage   54.23%   54.23%   +<.01%
==========================================
  Files        3585     3585
  Lines       88235    88233       -2
==========================================
- Hits        47853    47852       -1
+ Misses      40382    40381       -1

Impacted Files
Coverage Δ

...ation/models/provider_operations_metadata_paged.py
80% <ø> (ø)
:arrow_up:

...mgmt/authorization/models/classic_administrator.py
50% <ø> (ø)
:arrow_up:

...mgmt/authorization/models/role_definition_paged.py
80% <ø> (ø)
:arrow_up:

...thorization/models/provider_operations_metadata.py
40% <ø> (ø)
:arrow_up:

...re/mgmt/authorization/models/provider_operation.py
44.44% <ø> (ø)
:arrow_up:

...azure/mgmt/authorization/models/role_assignment.py
50% <ø> (ø)
:arrow_up:

...authorization/azure/mgmt/authorization/__init__.py
100% <ø> (ø)
:arrow_up:

...authorization/models/role_assignment_properties.py
66.66% <ø> (ø)
:arrow_up:

...zure/mgmt/authorization/models/permission_paged.py
100% <ø> (ø)
:arrow_up:

...ization/models/classic_administrator_properties.py
66.66% <ø> (ø)
:arrow_up:

... and 19 more

Continue to review full report at Codecov.

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

Doc only


ENH: Change default order of filters

This filter order matches better with requirements of IEC-61260-1 2014
Furthermore, the order of a bandpass filter should be divided by two.

@felipeacsi
in #160 I made the filters use 'sos' instead of 'ba'. This (supposedly) gives better results, especially at lower frequencies.
In the room module, an implicit order of 3 was used for the bandpass filter. This was also the default value up to now. For a bandpass filter the order should be divided by 2, therefore order 3 is an invalid value.
A good value of the order is 8 for bandpass filters and 4 for lowpass filters (see de9b899).
Now, making that change to the room module results in quite different values. What do you think about this? Is this relevant for what you use your implementation for?

@felipeacsi  I moved the relevant commit in a separate PR, #167

The change is good, the past filters weren't good to be honest. In fact, I had several errors in low frequencies.
I think the values in test_room.py can be updated.

Indeed, scipy 0.14 improved the accuracy of filter design:
http://docs.scipy.org/doc/scipy-dev/reference/release.0.14.0.html#scipy-signal-improvements
And now with scipy 0.16, it's possible to filter with Second-Order Sections which I'm using here now.
http://docs.scipy.org/doc/scipy-dev/reference/release.0.16.0.html#scipy-signal-improvements
I'll update the module and the tests. Thanks.


Format of clip_data from TSV

Hi Zhang lab,
I wanted to get a better understanding of the format of the TSV clip data from the link provided via https://zhanglabnet.oss-cn-beijing.aliyuncs.com/prismnet/data/clip_data.tgz.
The first column seems to show the Emsemble ID, followed by the sequence of basepairs. The next column seems slightly ambiguous so I am guessing it is the icShape analysed data? Then there are two columns that follow with single numbers. May I know what these last two columns are? And also where is the eCLIP binding data actually stored? Is it a per base pair binding score or an overall binary binding label of either 1 or 0?
Thank you.

Yes, you are right. These two columns are the binding score (clipDB) and the binding label (1 or 0).

Thank you for getting back promptly.
Ok so from what I understand, the binding score is aggregated over the entire length of 100bp and not any specific region?
If that is the case, how was the analysis in Figure 4 performed whereby the region specific binding was depicted? Are the scores in those diagrams binding probability for the entire sequence?
Also, from the generate data shell script, there is an option for generate the data in a binary or non binary manner, for most of the paper, the scores are continuous so do I need to change the option of is_bin == "0" to reproduce the results in the study?

That's right! You could change the option of is_bin == "0" to test the continuous case.
The length of binding regions (collected by POSTAR database)  are mostly ~30bp (with the binding score ), we extend to 101bp in this study. I directly copy the binding site data collection and processing strategy from the PrismNet paper below.

RBP binding site data collection and processing
RBP binding sites from CLIP-seq were collected from POSTAR and ENCODE (eCLIP).35,100 In total, we collected 269 CLIP-seq datasets for 56 RBPs from POSTAR,100,101 as well as 392 eCLIP datasets for 134 RBPs from ENCODE.35 To ensure that the CLIP data sets used in our study are of high-quality and consistent, we downloaded the binding sites from the ENCODE project and a published database (POSTAR), in which the binding sites have been generated using a uniform pipeline; that is, we did not use the binding site data from the original publications (as called by different labs using various tools, pipelines, and parameters).
For a RBP with CLIP experiments in different cell lines, we constructed a PrismNet model for each cell line separately. For a RBP with multiple CLIP experiments in one cell line, we only chose one experiment of the highest quality (first filtered by the number of experiment replicates, then ranked by average sequencing depth among replicates). For any CLIP experiment of more than one replicate, we combined the overlap binding sites from all replicates. Specifically, we performed replicate normalizations, summing up and then merging them to use all the information of each replicate. The scores of each peak were normalized to [0, 1]. Overlapping binding peaks (at least 1nt) were then merged with the summed peak signals to yield a single peak.
We defined each resulted peak as a binding site. The length of a binding site was unified to 101 nt, where a region shorter than 101 nt was expanded from the middle to both sides and a longer region was cut from both sides to the middle. Finally, the top 5000 binding sites with the highest signals and at least 40% icSHAPE scores coverage were kept for the training and testing of PrismNet, as positive samples.

I see, thank you for the really detailed explanation, it makes much more sense to me now.
One final question, from the study it was also shown the PrismNet trained on IGF2BP1 binding in K562-CLIP was able to generalize and accurately predict peaks on the EIF3F transcipt in HepG2 cells. From a broader prespective, is this model able to generalize and accurately predict binding on other cell lines which might be more distant like SHSY5Y or RD? I think the rationale for asking this is to get a better understanding as to how far PrismNet is able to generalize the learned relationships between RBP binding and RNA structure and to what extents can inferences be made? Not really thinking of a clear cut obective threshold but a general sensing...

hello,
I have the same question:
is this model able to generalize and accurately predict binding on other cell lines which might be more distant like SHSY5Y or RD?
I just have peak regions, how to use the model to predcit what RBPs bind to these regions?


bottles: 51.5 -> 51.6

Description of changes
https://github.com/bottlesdevs/Bottles/releases/tag/51.6
I didn't want to do it because of the regression https://github.com/bottlesdevs/Bottles/issues/2897, but vkd3d-proton 2.9 released, and it won't apply correctly on Bottles 51.5.

Things done

Built on platform(s)

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

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

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

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

[x] package build on x86_64-linux
[x] executables tested on x86_64-linux

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2265


[PE-D][Tester C] Mark command could be renamed to something more accurate

Just a minor nitpick, off the bat using the application, I was expecting the command to update the history of the elderly to be something like call

Labels: severity.VeryLow type.FunctionalityBug
original: ghos7ie/ped#5

You are right, it's just a nit pick


Discussion: Which fields to include to edit in metadata creation (vs editing?)

We probably need a discussion of which fields we want to offer the user to edit after the first creation of the metadata, and which we will allow them to edit when they are setting up the metadata.
My suggestion would be that all fields are shown to the user at creation (so adding a field to the view in this PR) with only Original Name locked down. Once the metadata has been stored in the database only Display Name and Description are editable (in which case we should remove either Original Name or Title from the file as it stands here).

@seedcase-project/developers
Originally posted by @K-Beicher in https://github.com/seedcase-project/seedcase-sprout/issues/273#issuecomment-1999368568

Yes, I think we should not scare the user with too much information.
Arisa was confused with just two names and since we have added a third.

We decided to start the web app from scratch after the major design/architecture decisions and changes we made. No longer relevant.


Unable to find location of "Disable AADIP integration in M365 Defender

🚨 Looks like you arrived from the internal review site
🚨 Do not enter Microsoft confidential information here
Hello team,
We've received a request related to https://review.learn.microsoft.com/en-us/azure/sentinel/whats-new#heads-up-microsoft-365-defender-now-integrates-azure-active-directory-identity-protection-aadip where the customer would like to "Disable AADIP integration at the source, in your Microsoft 365 Defender portal.". We're unable to find clear documentation around this, more specificaly, where in M365 Defender can we find this option?
The advisory here would be to add an URL with step-by-step instructions.
Thank you,

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

ID: 95a6bcf5-2399-cbdf-aed4-12fd4a3fd887
Version Independent ID: 2dc6c43b-20af-5169-71ef-f89af205d3e8
Content: What's new in Microsoft Sentinel
Content Source: articles/sentinel/whats-new.md
Service: microsoft-sentinel
Sub-service: microsoft-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

@asdcristi
Thanks for your feedback! We will investigate and update as appropriate.

@asdcristi
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

@yelevin
Can you please check and add your comments on this doc update request as applicable.

@asdcristi Thanks for your question! Apparently this feature has been revised and this option is no longer available. The documentation has been updated (in the last few hours!) to reflect this. The updated instructions can be found here:
What's New: Microsoft 365 Defender now integrates Azure Active Directory Identity Protection (AADIP)
I'm going to close this issue now, but please feel free to comment if you have additional questions.
#please-close

@yelevin Thank you for the update.
I can still see an issue on the updated documentation:
In the "Microsoft 365 Defender now integrates Azure Active Directory Identity Protection (AADIP)" section's table,
We have under Action in Microsoft Defender "Don't use Microsoft 365 Defender for AADIP alerts:
Choose the Turn off all alerts option for AADIP integration.". However we cannot find any page where we are advised on how to do so.
While on the Sentinel's side, it's stated "Leave enabled those Microsoft Security analytics rules that create incidents from AADIP alerts."
Could you please check with the team if we can reference an article where we have step by step instructions on how to Turn off all alerts option for AADIP integration in Defender?
Thank you,
Cristian

@asdcristi @SaibabaBalapur-MSFT
Here's an article in the Microsoft 365 Defender documentation that explains what to do.
https://learn.microsoft.com/en-us/microsoft-365/security/defender/investigate-alerts?view=o365-worldwide#configure-aad-ip-alert-service
I've added a link to this document from the document on which this issue was opened.

Thank you very much @yelevin , process looks clear now!


Fix #347 - remove redundant intro screen

Note branch is wrongly titled. This fixes #347

Going to merge--no impact change. Looks like a couple false positive test failures (timeouts) on the al_exhibit code which isn't related to this change.


dependencies motmetrics==1.2.0 and nuscenes-devkit==1.1.1 clashes

Hi guys,
impressive work! I am in the process of reproducing some results but think I found a dependency issue:
Pip complained and I verified with the requirements file of the nuscenes-devkit 1.1.1. These two libraries whose versions are specified in the requirements file are not compatible. Pip complained:

ERROR: Cannot install -r requirements.txt (line 11) and motmetrics==1.2.0 because these package versions have conflicting dependencies.
The conflict is caused by:
The user requested motmetrics==1.2.0
nuscenes-devkit 1.1.1 depends on motmetrics<=1.1.3

My solution at the moment is to install nuscenes-devkit 1.1.3 instead but I am not sure yet if that doesn't break something. I will update this ticket if I find something not working.

Hi @SaschaHornauer, thanks for your interest in our QD-3DT!
This dependency issue was expected and meant to install motmetrics==1.2.0.
We recommend installing motmetrics==1.2.0 throughout the running and evaluation on datasets in one virtual environment and preparing another virtual environment for the evaluation requirements (which is motmetrics==1.1.3) of nuScenes devkit. Please refer to INSTALL.md for an example of setting up your virtual environment.
I'm also looking forward to seeing nuscenes-devkit support a higher version of motmetrics, but please use two versions interchangeably for the current implementation.
I'll keep this ticket closed to help others in searching for the same issue. If you find anything not working, please create another ticket. Thanks!


CAMEL-14710: camel-jcache Tests are failing due to hazelcast upgrade

This was braked due to https://github.com/hazelcast/hazelcast/commit/9063ee227dfadce4c6e799193f53bc66ea255d51


Conan doc

Adds a basic example of how to consume the library through Conan (#1449).
I have cleaned up the shields; can be reverted if the old format was better.

Coverage remained the same at 99.834% when pulling cd54065a5935f14376ecf08a0407d9426246405b on offa:conan_docs into 33d11e10699bae03ba2a58a280e92494f4fa0d82 on cpputest:master.

@claremacrae I have updated to link to point to conan center (again).

Thanks!


Separate PoseEstimation() object from NWBFile() metadata?

I'm interested in using the first half of convert_h5_to_nwb to save with different metadata from DataJoint Elements.
Would you be interested in a PR that separated out the pieces that generated PoseEstimation() object?

@CBroz1 Check out https://github.com/DeepLabCut/DLC2NWB/pull/10 for one attempt at doing something like this but need traction upstream to make it happen
CC: @h-mayorquin, point person on the effort there

Ah - missed that. Yes, I think _write_pes_to_nwbfile from that PR would do what I needed. I'll keep an eye on the progress there.


I would like to create an EOS account and facing errors. Please help me out.

Is my command line right?
daeyeonjo@daeyeonjo-All-Series:~/eos/contracts$ cleos --url http://node2.liquideos.com:8888 system newaccount --stake-net 30 --stake-cpu 30 --buy-ram 10 g44tiojygige nowcoincount EOS838q5ocmCU1tfHJYyaRmd2jPXHaD5Dr6fcDc6JYYdUBh9obzLc EOS838q5ocmCU1tfHJYyaRmd2jPXHaD5Dr6fcDc6JYYdUBh9obzLc
2029616ms thread-0   asset.cpp:79                  from_string          ] 3010012 asset_type_exception: Invalid asset
Asset's amount and symbol should be separated with space
{}
thread-0  asset.cpp:42 from_string
2029616ms thread-0   asset.cpp:79                  from_string          ] from: 10
Error 3010012: Invalid asset
Error Details:
Asset's amount and symbol should be separated with space
rethrow

I checked other discussion about this issue, tried the following and got another error.
daeyeonjo@daeyeonjo-All-Series:~/eos/contracts$ cleos --url http://node2.liquideos.com:8888 system newaccount --stake-net '30.0000 EOS' --stake-cpu '30.0000 EOS'--buy-ram '10.0000 EOS' g44tiojygige nowcoincount EOS838q5ocmCU1tfHJYyaRmd2jPXHaD5Dr6fcDc6JYYdUBh9obzLc EOS838q5ocmCU1tfHJYyaRmd2jPXHaD5Dr6fcDc6JYYdUBh9obzLc
ERROR: ExtrasError: [EOS838q5ocmCU1tfHJYyaRmd2jPXHaD5Dr6fcDc6JYYdUBh9obzLc]
Create an account, buy ram, stake for bandwidth for the account
Usage: cleos system newaccount [OPTIONS] creator name OwnerKey [ActiveKey]
Positionals:
creator TEXT                The name of the account creating the new account (required)
name TEXT                   The name of the new account (required)
OwnerKey TEXT               The owner public key for the new account (required)
ActiveKey TEXT              The active public key for the new account
Options:

I tried another command line and got the following error.
Error 3050003: eosio_assert_message assertion failure
Any idea to solve this matter?
daeyeonjo@daeyeonjo-All-Series:~/eos/contracts$ cleos --url http://node2.liquideos.com:8888 system newaccount --stake-net '30.0000 EOS' --stake-cpu '30.0000 EOS' --buy-ram-kbytes 8  g44tiojygige nowcoincount EOS838q5ocmCU1tfHJYyaRmd2jPXHaD5Dr6fcDc6JYYdUBh9obzLc EOS838q5ocmCU1tfHJYyaRmd2jPXHaD5Dr6fcDc6JYYdUBh9obzLc
260959ms thread-0   main.cpp:429                  create_action        ] result: {"binargs":"a09863fe5197096190a7a6683a8a389d00200000"} arg: {"code":"eosio","action":"buyrambytes","args":{"payer":"g44tiojygige","receiver":"nowcoincount","bytes":8192}}
262428ms thread-0   main.cpp:429                  create_action        ] result: {"binargs":"a09863fe5197096190a7a6683a8a389de09304000000000004454f5300000000e09304000000000004454f530000000000"} arg: {"code":"eosio","action":"delegatebw","args":{"from":"g44tiojygige","receiver":"nowcoincount","stake_net_quantity":"30.0000 EOS","stake_cpu_quantity":"30.0000 EOS","transfer":false}}
Error 3050003: eosio_assert_message assertion failure

The g44tiojygige account only has 41.5705 EOS available.  You're trying to stake 60 EOS.  Free up some of the 335 EOS staked or buy more EOS.

Thank you very much for your comment.


Update metadata.md

update to v2.0.0

@wheresaddie I just directly push to update the json value in the article, reflecting Substrate v2 changes. Hope you don't mind 😄

Also, JSON syntax doesn't support commenting. That's why github diff viewer showing the JSON as red on // .... I don't have a good solution on how to resolve this, and just let it be for now.


Memory Issues

When working with the framework for a while it will seemingly randomly give memory errors on sudo shell_manager package xxxx which will persist until things will eventually free up (generally around 3-4 minutes later). When opening up ps aux it shows many (at least 40-50) sets of the following entry:
vagrant   6819  0.0  0.9<PHONE_NUMBER>8 ?        S    02:49   0:00 sshd: vagrant@notty
root      6836  0.5  0.9<PHONE_NUMBER>4 ?        Ss   02:50   0:01 sshd: vagrant [priv]

Can you do top when this happens?
On Wed, Aug 22, 2018, 11:08 PM Alex Fulton<EMAIL_ADDRESS>wrote:

When working with the framework for a while it will seemingly randomly
give memory errors on sudo shell_manager package xxxx which will persist
until things will eventually free up (generally around 3-4 minutes later).
When opening up ps aux it shows many (at least 40-50) sets of the following
entry:
vagrant   6819  0.0  0.9<PHONE_NUMBER>8 ?        S    02:49   0:00 sshd: vagrant@notty
root      6836  0.5  0.9<PHONE_NUMBER>4 ?        Ss   02:50   0:01 sshd: vagrant [priv]
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/picoCTF/picoCTF/issues/143, or mute the thread
https://github.com/notifications/unsubscribe-auth/AONIp9LqFDhotnL-oeIh_RAj244KupY5ks5uThyzgaJpZM4WIwkh
.

@al3xfulton I have seen a similar situation happening on my test VMs. Based on watching htop on the VMs, I think this could be due to /picoCTF-env/bin/python3.6 /picoCTF-env/bin/daemon_manager -d /picoCTF/picoCTF-web/daemons cache_stats share_instances running on the web server.


[BUG] SCSS and JS not compiling on "packer start"

Describe the bug
Hello and thanks for creating this great tool. Love it. I found a bug when running "packer start", its not compiling scss and js, but in terminal i see that changes are detected. But when running packer build, changes are then compiled and working as expected. Can you please help?
To Reproduce
Steps to reproduce the behavior:

run packer start
change scss or js
nothing changes, new files are not compiled both on localhost with hmr or preview via shopify admin
close terminal session
run packer build
run packer deploy
changes are compiled and visible

Expected behavior
When you change scss or js file, it get compiled and changes visible on localhost or preview via admin
Screenshots
If applicable, add screenshots to help explain your problem.
System (please complete the following information):

OS: osx
Node Version: v14.16.0
NPM Version: v7.7.6
Browser: all

Packer (please complete the following information):

Version: latest

Theme (please complete the following information):

[Shopify Packer theme]
Version: latest

With the recent upgrade to webpack 5 internally I have noticed this issue and a few others running in development mode (start/watch) in some of the recent versions, can you run packer -v and post the exact version?

@hayes0724 Shopify Packer - v2.0.12

Thanks for the feedback, I was able to recreate and this was a quick fix. The source maps were incompatible with hmr.
fixed in v2.0.13

@hayes0724 can you please write me a quick explanation how to update that on existing project? or i need to re-create the whole project? thanks for this!

Packer should be installed globally (unless you need a specific version for one project) so use global update/upgrade command
yarn
yarn global upgrade @hayes0724/shopify-packer

npm
npm update -g @hayes0724/shopify-packer

@hayes0724 thank you so much, this tool is so sick! Best i tried. Really appreciate it!

@hayes0724 sorry for bothering you, unfortunately i update to 2.0.13 and its still not working for me, i will now try to recreate the project and try

You may need to delete the dist folder before running start. Webpack clean sometimes leaves files on development build unless they are deleted. It used to delete everything after each compile but I disabled this since it increase start/watch dev time significantly

I have the same issues. I'm running on v.2.1.3.

@marcelaerts actually updating did not fix the issue for me, i needed to start a new packer project, did not find a way to fix it on current project, i think something with assets was killing my webpack