Separate billing code by providers

We require EAs to sign the eMOU again if the public provider changes, even if the account coding was previously signed with a different provider.

Now Azure products don't have accountCoding in the requestedProject collection in DB:

The modal for Azure still has AWS in the text. We have a separate task for that from earlier: https://app.zenhub.com/workspaces/app-dev-team-646fdb4400ed8c0021aceeb4/issues/gh/bcgov/platform-services-registry/3642

Otherwise works as expected.
I thought of another corner case when while create request the billing code exists and eMOU is signed, but the user uses another email for Expense Authority product. Now Registry lets through with exemption for signing eMOU. It will be a very rear case and may be left as a responsibility of the Cloud Director to check it, but still possible from the Registry side.


Swiper Classes

Hi Good Day,
I am a new to vue-awesome-swiper. My swipers are working fine in Chrome and Firefox but not in IE. The problem is that the classes of the swipers in IE are not complete.
The classes in IE are only swiper-container swiper-container-initialized
I hope you can help me with my problem.
Thanks :)

Good Day :)
I solve this issue of mine by downgrading may swiper from 6.5.4 to 4.5.1


changing position/size of State-wise COVID-19 Cases India

In order to use the slider user needs to scroll down and then scroll up again to see the data even with browser on full screen,It would be more convenient if the whole map card is visible in the screen without scrolling .

Yeah this should be be an easy one. Can you please also let me know what browser/device is this?

Yeah this should be be an easy one. Can you please also let me know what browser/device is this?

yes,its vivaldi


Don't use export * to workaround rollup bug

Has this update made it into a release somewhere? Would love to have this available in my build.

I just came across this same error. Great to see a solution already implemented, can't wait for the release!

@justinfagnani is there anyway we could get a bug fix release on this? Even if it wasn't the right answer long term, this proves to be a pretty big blocker to my team moving forward with a number of projects. It seems like the "fix" at the polymer-cli level either isn't working or isn't going to be released, and the conversation at the RollUp level is still ongoing, so any help you could provide at this level via bug fix release or "next" release or what not would be greatly appreciated.

lit-html 0.13.0 now released.


Revert "chore(deps): bump @gelatonetwork/relay-sdk from 4.0.0 to 5.0.0 (#189)"

This reverts commit 3a1d7332475493f7119af8c8127676ae58867d6f.
5.0.0 seems to have introduced a bug regarding bigint serialisation (which started using with 5.0.0). Currently the service functionality is broken because of that.
Once we confirm the source of the issue and have it addressed we can bump relay-sdk version again.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
501

Relevant Lines:
676

💛  - Coveralls


fix: update to eye v4.11.2

fix: update to eye v4.11.2

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

npm package (@latest dist-tag)
GitHub release
v6.1.3

Your semantic-release bot :package::rocket:


How to publish the App to Azure App Service ?

How to publish the App to Azure App Service ? I guess no need put all to Azure App Service, right ?

I would like it if the example was more clear on how to do this.


SyntaxError: invalid syntax

$ nix-shell ./env
building '/nix/store/4vkpzlns71i5hfk746yd6yf5r3378i2s-mach_nix_file.drv'...
Traceback (most recent call last):
  File "/nix/store/lqsqmiand28kv5zzcark1dg1pdzi8s0b-python3-3.8.9-env/lib/python3.8/site-packages/resolvelib/resolvers.py", line 185, in _merge_into_criterion
    crit = self.state.criteria[name]
KeyError: 'requests'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/nix/store/nykbdrk2wq5gi0248nmzsvjr520j6mhc-dljl5bzw3cvzg0c5vfrmn8z9cnz5rnla-source/mach_nix/generate.py", line 98, in <module>
    main()
  File "/nix/store/nykbdrk2wq5gi0248nmzsvjr520j6mhc-dljl5bzw3cvzg0c5vfrmn8z9cnz5rnla-source/mach_nix/generate.py", line 63, in main
    expr = generator.generate(reqs)
  File "/nix/store/nykbdrk2wq5gi0248nmzsvjr520j6mhc-dljl5bzw3cvzg0c5vfrmn8z9cnz5rnla-source/mach_nix/generators/overides_generator.py", line 35, in generate
    pkgs = self.resolver.resolve(reqs)
  File "/nix/store/nykbdrk2wq5gi0248nmzsvjr520j6mhc-dljl5bzw3cvzg0c5vfrmn8z9cnz5rnla-source/mach_nix/resolver/resolvelib_resolver.py", line 63, in resolve
    result = resolvelib.Resolver(Provider(self.nixpkgs, self.deps_provider), reporter).resolve(reqs, max_rounds=1000)
  File "/nix/store/lqsqmiand28kv5zzcark1dg1pdzi8s0b-python3-3.8.9-env/lib/python3.8/site-packages/resolvelib/resolvers.py", line 413, in resolve
    state = resolution.resolve(requirements, max_rounds=max_rounds)
  File "/nix/store/lqsqmiand28kv5zzcark1dg1pdzi8s0b-python3-3.8.9-env/lib/python3.8/site-packages/resolvelib/resolvers.py", line 280, in resolve
    name, crit = self._merge_into_criterion(r, parent=None)
  File "/nix/store/lqsqmiand28kv5zzcark1dg1pdzi8s0b-python3-3.8.9-env/lib/python3.8/site-packages/resolvelib/resolvers.py", line 187, in _merge_into_criterion
    crit = Criterion.from_requirement(self._p, requirement, parent)
  File "/nix/store/lqsqmiand28kv5zzcark1dg1pdzi8s0b-python3-3.8.9-env/lib/python3.8/site-packages/resolvelib/resolvers.py", line 80, in from_requirement
    candidates = provider.find_matches(requirement)
  File "/nix/store/nykbdrk2wq5gi0248nmzsvjr520j6mhc-dljl5bzw3cvzg0c5vfrmn8z9cnz5rnla-source/mach_nix/resolver/resolvelib_resolver.py", line 42, in find_matches
    matching_versions = filter_versions(all, req.specs)
  File "/nix/store/nykbdrk2wq5gi0248nmzsvjr520j6mhc-dljl5bzw3cvzg0c5vfrmn8z9cnz5rnla-source/mach_nix/cache.py", line 12, in cache_wrapper
    result = func(*args, **kwargs)
  File "/nix/store/nykbdrk2wq5gi0248nmzsvjr520j6mhc-dljl5bzw3cvzg0c5vfrmn8z9cnz5rnla-source/mach_nix/versions.py", line 94, in filter_versions
    versions = list(filter(lambda v: eval(f'v {op} ver', dict(v=v, ver=ver)), versions))
  File "/nix/store/nykbdrk2wq5gi0248nmzsvjr520j6mhc-dljl5bzw3cvzg0c5vfrmn8z9cnz5rnla-source/mach_nix/versions.py", line 94, in <lambda>
    versions = list(filter(lambda v: eval(f'v {op} ver', dict(v=v, ver=ver)), versions))
  File "<string>", line 1
    v === ver
        ^
SyntaxError: invalid syntax
builder for '/nix/store/4vkpzlns71i5hfk746yd6yf5r3378i2s-mach_nix_file.drv' failed with exit code 1
error: build of '/nix/store/4vkpzlns71i5hfk746yd6yf5r3378i2s-mach_nix_file.drv' failed
(use '--show-trace' to show detailed location information)

Hey, I'm really dont know how to fix this?

Hey could you tell the parameters which you used to create the ./env ? Which python version, which requirements, etc.


Add documentation for running mongodb under usecases.

The configuration files for running mongodb is available under https://github.com/openebs/openebs/tree/master/k8s/demo/mongodb
The usecases doc https://github.com/openebs/openebs/blob/master/documentation/source/install/install_usecases.rst, should be updated with content for running mongodb, like the other examples  - percona, crunchy and jupyter.

Resolved as part of PR #870


BaseCompat account.status gives unknown error on 0.4.0 while code was working on production using old SDKs

Hello! I'm working on integrating the Agora SDK for KinFit, which is already running on kin for quite some time. I updated the kin-ios version in the podfile to 0.4.0, rebuilt and now the account.status function does not work anymore.
Here is the relevant part of the code (I added some asdasd so I can more easily find it in the logs):
        ...
        print("asdasdQ")
        print(self.account)
        self.account!.status { (status: AccountStatus?, error: Error?) in
            print("asdasdX")
            print(status)
            print(error)
            guard let status = status else { return }

            ...

Here are the logs:
asdasdQ
Optional(KinSDK.KinAccount)
2020-11-21 10:15:45.440887+0100 kinfit[66572:7024944] KinBase.KinAccountContext::info::getAccount(forceUpdate:)
2020-11-21 10:15:45.443145+0100 kinfit[66572:7024944] KinBase.KinService::debug::[Request][V3]====
CreateAccountRequest(accountId: "GAA7C3Z2ATHIFIVS2PNKT2ZJLDKP3TOPF5GZKO3QTTZZO2U35NIK3QHY")
=====[Request][V3]
2020-11-21 10:15:45.443657+0100 kinfit[66572:7024896] KinBase.AgoraGrpcProxy::debug::AgoraGrpcProxy::request::<APBAccountV3CreateAccountRequest 0x600002e081e0>: {
    account_id {
      value: "GAA7C3Z2ATHIFIVS2PNKT2ZJLDKP3TOPF5GZKO3QTTZZO2U35NIK3QHY"
    }
}
2020-11-21 10:15:45.997364+0100 kinfit[66572:7024944] KinBase.AgoraGrpcProxy::debug::AgoraGrpcProxy::response::<APBAccountV3CreateAccountResponse 0x600002e7ab80>: {
    result: EXISTS
    account_info {
      account_id {
        value: "GAA7C3Z2ATHIFIVS2PNKT2ZJLDKP3TOPF5GZKO3QTTZZO2U35NIK3QHY"
      }
      sequence_number:<PHONE_NUMBER>6927104
    }
}
2020-11-21 10:15:45.998102+0100 kinfit[66572:7024944] KinBase.KinService::debug::[Response][V3]====
CreateAccountResponse(result: KinBase.CreateAccountResponse.Result.exists, error: nil, account: nil)
=====[Response][V3]
asdasdX
nil
Optional(KinBase.KinService.Errors.unknown)

So as you see, everything is working nicely (Agora tells the account already exists), but when I do account.status I receive an Optional(KinBase.KinService.Errors.unknown)
Any ideas?
Thanks!

Just checked, the issue is not resolved on 0.4.1

resolved

resolved


binding-functions encoded in ASCII on Windows

The default is for Powershell to generate UTF-16 with BOM, and setting to UTF-8
also generates a BOM. (I understand newer versions of Powershell have a UTF8
with no BOM option, but that wasn't available for me.) The BOM is a problem
because Java for example does not discard the BOM, requiring workarounds like
Commons IO.
This limits function names in the C API to the ASCII character set, which
currently appears to be a reasonable limitation.

Coverage remained the same at 100.0% when pulling e6ff28970229f79da8fbdb7d677ace4c868821f8 on isaacbrodsky:binding-functions-ascii into 9c4cdac70eb5ac733737d952ae8ce12d17f13b8b on uber:master.

I confirmed in H3-Java / Appveyor that this does work: https://github.com/isaacbrodsky/h3-java/tree/bump-3.4.0-test-with-newer-h3


Content overlap with Background-color on homepage

Problem:
The main hero section content overlaps with background colour in mobile view
Screenshot:

Normal Behaviour
The text should only be in blue portion as background colour and font colour is same i.e white
Conclusion
I would like to fix the issue allow me to make a PR for the same.

@kunalvishnoi The documentation website is unfortunately not open source, so there's not an easy way to allow you to fix it. I'll make a note of it. Thank you!


[FLINK-13656] [sql-parser][table-planner][table-planner-blink] Bump Calcite dependency to 1.21.0

The Changes
SQL Parser
Some refactoring in flink-sql-parser
Flink Planner

org.apache.calcite.rel.core.Match#partitionKeys has changed from List<RexNode> to ImmutableBitSet, since CALCITE-1935,
org.apache.calcite.rel.core.Match#copy has been removed since CALCITE-1935 part1.
org.apache.flink.table.api.stream.sql.JoinTest#testRightOuterJoinEquiAndLocalPred, #testLeftOuterJoinEquiAndLocalPred, org.apache.flink.table.api.batch.sql.JoinTest#testFullOuterJoinEquiAndLocalPred, #testRightOuterJoinEquiAndLocalPred, #testLeftOuterJoinEquiAndLocalPred plan has changed, but it is still correct. since CALCITE-3101
ConverterRule that use RelOptRule#description was affected because the
default description generation rule has changed since CALCITE-3115.
org.apache.flink.table.api.stream.table.AggregateTest#testMultiDistinctAggregateSameFieldOnHopWindow, org.apache.flink.table.api.stream.sql.DistinctAggregateTest#testMultiDistinctAggregateSameFieldOnHopWindow max distinct was removed. Since CALCITE-3159

Blink Planner

The plan change mostly because of the same reason with flink planner
SqlFunction was copied from CALCITE because of CALCITE-3360
org.apache.flink.table.planner.plan.batch.sql#testMinusWithNestedTypes IS NOT DISTINCT FROM is not expanded now since CALCITE-3174
FlinkTypeFactory#createDecimalQuotient was moved to FlinkTypeSystem#deriveDecimalDivideType due to CALCITE-3187

Thanks @KurtYoung for the nice review, my personal CI has passed https://travis-ci.org/danny0405/flink/builds/590222711, but the Flink one fails due to a failure plugin download.


Refactor medical imaging datasets for experiments

Hi @constantinpape,
There's quite a lot of changes in this PR, mostly focused on:

Structuring the (relevant) medical imaging datasets (with docstrings, standard format of functions, etc)
Ensure exclusive train-val-test splits for all datasets.

GTG from my side!


Code enhancement: Replaced explicit synchronized access to a hashmap with a concurrent map.

What changes were proposed in this pull request?
Refactored access to a HashMap through an explicit external lock with a implicit internal lock. Thread safe access to the map is maintained.
How was this patch tested?
Ran Unit-tests

LGTM.

This is not a bug fix. Addressed a TODO for a general code enhancement. This code is not directly related to any specific feature work.

It seems this PR has stalled a bit since the last activity a few weeks ago.  @varunkatta what's next here?

Just back to this PR...I thought this changed was merged long back. I will address the last few comments today.

@satybald I addressed your comments. Do you want to take a quick look at the diff?
@ash211  Next steps, if there no more changes requested by reviewers is to merge this PR.

Going to bring in after https://github.com/apache-spark-on-k8s/spark/pull/459

@varunkatta this is ready for rebase -- please fix merge conflicts


Added Nightwatch tests

@reshadn I'd recommend taking a look at this! It'll change our testing 4evvvaaaa
https://github.com/nightwatchjs/nightwatch

Wow that's pretty awesome! I hadn't come around to asking about testing yet. This looks great.

@reshadn yeah it's super cool - we have simple and fast automated tests.... once this works lol

OMG it passed!!! @ahmedre it PASSEDDDDDDDD


Support for grammar "Gherkin" and extension "feature"

This language? https://github.com/cucumber/cucumber/wiki/Gherkin
Could you provide a sample input and output?

Yes it is :)
https://github.com/cucumber/cucumber/wiki/Gherkin

What do you mean by "a sample input and output"? An example of un-formatted file and his fixed version?

Yep.
If the sample is similar to existing languages then we could support it easier. If it's different then we have an example to compare to our existing examples.
—
Sent from Mailbox
On Fri, Jul 17, 2015 at 9:35 AM, Armand Abric<EMAIL_ADDRESS>wrote:

What do you mean by "a sample input and output"? An example of un-formatted file and his fixed version?
Reply to this email directly or view it on GitHub:
https://github.com/Glavin001/atom-beautify/issues/377#issuecomment-122261698

Here it is: https://gist.github.com/Spy-Seth/3206c3015db5eb22fd54

Great! looks good!

Published to v0.28.9

Excellent news! :dancers:


invalid key

autogpt_1     | openai.error.AuthenticationError: Incorrect API key provided: "sk-uaFv*****************************************5sj". You can find your API key at https://platform.openai.com/account/api-keys.
call-auto-gpt_autogpt_1 exited with code 1

This  is what exceeded quota looks like 2023-10-08 20:49:00,271 WARNING  Please double check that you have setup a PAID OpenAI API Account\ . You can read more here: https://docs.agpt.co/setup/#getting-an-api-key 2023-10-08 20:49:08,406 INFO  message='OpenAI API response' path=https://api.openai.com/v1/chat/co\ mpletions processing_ms=None request_id=c98379dc5396b4075c261fe8375fe9c9 response_code=429 2023-10-08 20:49:08,406 INFO  error_code=insufficient_quota error_message='You exceeded your curre\ nt quota, please check your plan and billing details.' error_param=None error_type=insufficient_qu\ ota message='OpenAI API error received' stream_error=False

This is resolved the issue was how the key was being quoted


SSL Certificates Not Generating

Recently got my setup moved over to an SSL/Advanced setup.  Only issue I am having is that it won't generate the certs.
I get this error:
proxy_1      | time="2020-04-08T19:55:14Z" level=error msg="Unable to obtain ACME certificate for domains "teslamate.klepean.com": unable to generate a certificate for the domains [teslamate.klepean.com]: acme: Error -> One or more domains had a problem:\n[teslamate.klepean.com] acme: error: 400 :: urn:ietf:params:acme:error:connection :: Fetching http://teslamate.klepean.com/.well-known/acme-challenge/d4YFXm7S35pO6u74ug993nqTFzks_hNPvWs1M_tD9mc: Error getting validation data, url: \n" providerName=tmhttpchallenge.acme routerName=teslamate@docker rule="Host(teslamate.klepean.com)"
It is identical for both of my domains.
Any ideas?

Do you have something else running on port 80? Can you fetch  the mentioned URL in a browser?

Nothing else on port 80.  No, the url doesn't seem to exist.

I am using the 1.19 dev version if that matters.  I even made a clean instance with 1.19 dev, and I am getting the same errors.

Just tried a clean instance with the official 1.18.2 release, same error...

Is there a way to just manually generate the cert, and place it somewhere without having to rebuild my entire setup?

Nothing else on port 80. No, the url doesn't seem to exist.

You have to set up the DNS entry with your hosting provider or whatever DNS service you're using.

For what it's worth, there are a few threads on the let's encrypt community forum that cover this error, for example:
https://community.letsencrypt.org/t/certificate-not-being-created-due-to-error-getting-validation-data/70863

Thanks for the response.  I have dns records for my site, and the site itself works from the internet.  I have firewall setup to forward both 80 and 443.  The site works fine from the internet with the exception of saying the connection isn't secure.

I have created a certificate manually, and I have it as a PEM file.  This one file contains my cert, as well as the full chain.  How can I use the advanced Docker configuration, but use this pem file instead of having it attempt to auto retrieve?

Wow, got this figured out.  My NAS was redirecting the requests via UPNP.  Disabled that, and all is well.


How to invoke Closure ?

output:  object(Closure)#2 (0) { }
how to invoke Closure function?
namespace Router;

class Router
{

    protected method;
    protected url;
    protected rules;

    public function __construct()
    {
        let this->method = $_SERVER["REQUEST_METHOD"];
        let this->url = $_SERVER["PATH_INFO"];
        let this->rules = [];
    }

    public function add(string! route, func)
    {
        let this->rules[route] = func;
    }

    public function run()
    {
        var route, func;

        for route, func in this->rules {
            if route == this->url {
                var_dump(func);
                //how to invoke func?
                break;
            }
        }
    }
}

<?php

$router = new \Router\Router();

$router->add('/hello', function () {
    echo "hello";
});

$router->run();

I want to invoke func method? How?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/37443616-how-to-invoke-closure?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

{func}();
it work!

how about call_user_func?


Use HTML5 native client form validation. Closes #1013

Wherever that is possible to sync client HTML5 form validation with the
rules used on the server to validate user input, use type, min length
and required when applicable:

add type=email, whenever email input is expected
add minlength attribute, whenever input lenght is checked
add required, whenever input is checked against empty values.

Thanks!

Thank you!


USB PCDC communication device can not send data with exactly 64 bytes

If you send exactly 64 bytes by using R_USB_Write(...), data is not sent immediatley to the usb bus. (but later on with the next write)
I used the the r_usb_pcdc middleware to send usb data. (R01AN2030EJ0130)
The problem seems to be in the r_usb_plibusbip.c Line 371
if (g_usb_pstd_data_cnt[pipe] < (uint32_t)size)

should be replaced by
if (g_usb_pstd_data_cnt[pipe] <= (uint32_t)size)

In my opinion the check is wrong because if size = 64 (internal buffer size) and the data sent count is also 64 the transmission is not terminated correctly and the data is not sent. (remaining bytes is 0) In that case the bus is NOT in continuous mode.
RX65 Hardware Manual:
BVAL Bit (Buffer Memory Valid)
The BVAL bit should be set to 1 when data has been completely written to the FIFO buffer on the CPU side for the pipe
selected using the CURPIPE[3:0] bits (selected pipe).
When the selected pipe is in the transmitting direction, set the BVAL bit to 1 in the following cases. Then, the USB
switches the FIFO buffer from the CPU side to the SIE side, thus enabling transmission.

To transmit a short packet, set the BVAL bit to 1 after data has been written.
To transmit a zero-length packet, set the BVAL bit to 1 before data is written to the FIFO buffer.
When data of the maximum packet size has been written for the pipe in continuous transfer mode, the USB sets the
BVAL bit to 1 and switches the FIFO buffer from the CPU side to the SIE side, thus enabling transmission.
Writing 1 to the BVAL bit should be done while the FRDY flag is 1 (set by the USB).
When the selected pipe is in the receiving direction, do not set the BVAL bit to 1.

This problem is reproduceable.

Witch layer is responsible for sending that 0 packet. How do I know on application layer about max packet size. Driver internal you know about that and hold the information e.g. USB_WRITESHRT, USB_WRITEEND.
I think the basic driver should add the zero-length packet itself if required but currently this is not done.
I found no hint about that behaviour in the driver documentation or in the PCDC sample program.
Von: TakahiroMuranaka @.***
Gesendet: Freitag, 12. März 2021 01:22
An: renesas/rx-driver-package
Cc: gurkerlbaron; Mention
Betreff: Re: [renesas/rx-driver-package] USB PCDC communication device can not send data with exactly 64 bytes (#19)
Please send the zero-length packet after sending 64 bytes data.
When you sent the data of the max packet size, you need to send the zero-length packet at next packet.
(The max packet size is 64 bytes in Full-speed, 512 bytes in USB High-speed.)
In USB communication, the last packet must be the short packet. A short packet means a packet whose data size is less than the max packet size. So, If the data size is a multiple of the max packet size, then the last packet must be a zero-length packet.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/renesas/rx-driver-package/issues/19#issuecomment-797145117 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ADD64SL4ANX5ONLSGTDQTELTDFNC7ANCNFSM4Y35FMNQ .  https://github.com/notifications/beacon/ADD64SN4AR4UHPLAQ7UANKTTDFNC7A5CNFSM4Y35FMN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOF6BXQHI.gif

Zero length packet must be sent in the application program. Zero length packet is sent by R_USB_Write function.
Please be sure to specify USB_NULL(zero) to the 3rd argument of R_USB_Write function when sending zero-length packet.
e.g)
R_USB_Write(p_ctrl, g_buf, USB_NULL);
In additional, the max packet size is specified in USB specification.
The max packet size is 64 bytes in Full-speed, 512 bytes in High-speed.

@TakahiroMuranaka Can we close this issue? We have updated RX Driver Package to v131 now, if we still have issue related this thread, let's discussing further.

I have confirmed with @TakahiroMuranaka to close this issue. Done.


Installation Guides rework

I added the following guides:
https://jeffschering.github.io/mmdocs/monolith/install/install-ubuntu-1404.html
https://jeffschering.github.io/mmdocs/monolith/install/install-ubuntu-1604.html
https://jeffschering.github.io/mmdocs/monolith/install/install-rhel-66.html
https://jeffschering.github.io/mmdocs/monolith/install/install-rhel-71.html

Hi @esethna I think I've addressed everything. Let me know if I missed anything.

Thanks Jeff, thanks Eric, I will be next up for final review or merge if everything looks good, just need to carve out time for the review

@JeffSchering just starting now,
Sorry, just to check, are the following up-to-date? Apologies for the slow review, we're low on staff during the holidays and couldn't get the review in prior.
https://jeffschering.github.io/mmdocs/monolith/install/install-ubuntu-1404.html
https://jeffschering.github.io/mmdocs/monolith/install/install-ubuntu-1604.html
https://jeffschering.github.io/mmdocs/monolith/install/install-rhel-66.html
https://jeffschering.github.io/mmdocs/monolith/install/install-rhel-71.html

Hi @it33 ... Yes, I updated them just now.

Hi @it33 ... Yes, I updated them just now.

@JeffSchering really great work here, mostly minor nits here and there.
This will make our install guides significantly better, thank you for working on them!!
One quick note: For code blocks such as attached below, wondering if we'd be able to combine the text with the inline code so that the flow of the document is smoother?

For instance, for the first line, we could write
Run ``sudo chmod 664 /etc/systemd/system/mattermost.service`` to make the service executable.

or
Make the service executable: ``sudo chmod 664 /etc/systemd/system/mattermost.service``

Open for discussion and for hearing your thoughts.
Thanks again, excellent work!

Also, if you can help add a note about user accounts and parameters to our style guides, that would be great. Would you like me to create a GitHub issue for us to track? Or have you already started on it?

Yes, I've already started on it, but go ahead and create a GitHub issue so that we can have the proposed changes visible to everyone.

Great, added here https://github.com/mattermost/docs/issues/772 @JeffSchering

@JeffSchering Are these docs updated?
https://jeffschering.github.io/mmdocs/monolith/install/install-ubuntu-1404.html
https://jeffschering.github.io/mmdocs/monolith/install/install-ubuntu-1604.html
https://jeffschering.github.io/mmdocs/monolith/install/install-rhel-66.html
https://jeffschering.github.io/mmdocs/monolith/install/install-rhel-71.html
It appears that some of the changes might not have been pushed for those pages

@jasonblais Thanks for reminding me. I updated them just now.


Missing Ornn Items

Ornn Items are not included in the Item data from DDragon, but do exist in the game. But, since the Item data is not included in DDragon, we're missing item images eg. Sandshrike's Claw, ID=7000. The Ornn Item ID's are between 7000 - 7022 (inclusive). Looking at the CDragon data as well, fields like requiredAlly should be set to Ornn, active should be set to true, and store should be set to false. This means that the some fields are incorrect and other fields like icon in the Item model would be unpopulated. These items are edge cases, but I think these should be added in the item set.
The general steps I would do is:

Get all the item items that are forgeable by Ornn in his ability passive description
Create a dictionary of Ornn items using CDragon i.e. {7000: "Sandshrike's Claw",...}. CDragon Ornn item names follow %i:ornnIcon% <item_name>. So if the name matches this expression, we know it's an Ornn item.
Loop through the Ornn items and follow the same process as if it were a regular item.

Should these items be accounted for, and if so, what could we do about missing fields like icon and simpleDescription?

So I've written down the code below (which will be in __main__.py. The only problem is it's just the same code repeated twice (it's exactly the same with for d in ddragon) and we import re in __main__.py:
import re # additional import

def _name_to_wiki(name: str):
  ... # unchanged function
def main():
  for d in ddragon:
    ... # unchanged steps
  for cdrag in cdragon:
    ornn_item_match = re.search(r"(?:%i:ornnIcon% )([A-Za-z0-9' \-]+)", cdrag["name"])
    if ornn_item_match is not None:
      ornn_item_name = ornn_item_match.groups()[0]
      item = _name_to_wiki(ornn_item_name)
      item.name = ornn_item_name
      item.id = cdrag["id"]
      item.icon = "" # what would we do here?
      item.builds_from = cdrag["from"]
      item.builds_into = cdrag["to"]
      item.simple_description = "All stats have been improved." # this si the caption of the item
      item.required_ally = "Ornn" # you can't forge ornn items without Ornn as your ally
      item.required_champion = cdrag["requiredChampion"]
      item.shop.purchasable = False
      if item is not None:
        jsonfn = os.path.join(directory, "items", str(item.id) + ".json")
        with open(jsonfn, "w", encoding="utf8") as f:
          j = item.__json__(indent=2, ensure_ascii=False)
          f.write(j)
        jsons[item.id] = json.loads(item.__json__(ensure_ascii=False))
        print(item.id)
  jsonfn = os.path.join(directory, "items.json")
    with open(jsonfn, "w", encoding="utf8") as f:
      json.dump(jsons, f, indent=2, ensure_ascii=False)
  del jsons
  return

if __name__ == "__main__":
    main()
    print("Hello! What a surprise, it worked!")

Any suggestions?

Awesome, this will be a great addition. Overall it looks good. As usual, I have a handful of small things that would be useful to update:

CDragon should have the icon data, e.g. /lol-game-data/assets/ASSETS/Items/Icons2D/6662_Tank_T4_FrostfireGauntlet.png. I don't remember the prefix but it's in Cass or in Pyot if not in Cass.
It's too bad CDragon doesn't have a simple description. Let's do something like f"An Ornn-updated item built from {item.name}". When people ask, "What item is that?" we want to use whatever answer they will be given, which in my opinion is what I wrote.
Let's use f-strings when we can, and please replace str(item.id) + ".json" with f"{item.name}.json".
Let's make sure to use black for formatting.
Please add a comment above if item is not None: because it is not immediately clear to me what the intention of that if block is.
The re import is no problem, it's built into Python and is part of the standard library.
Please capitalize the first work in comments. It's just a personal thing I like and I think it makes the code more readable. I haven't been super strict about that in the past but if you're making changes anyways I'd appreciate it.

Otherwise it looks good! It'd be nice if we could reduce the code duplication but this is a unique case so I'm okay with having duplicate code. If it becomes a problem, we can fix it in another iteration.
Thanks for your work here!!

Thanks for your help! I really do appreciate the structured way of contributing to the project, it keeps it consistent and concise for readers and contributors. To note the things you've talked about in the previous comment:

CDragon has the icon data for the original item, but the actual icon with the Masterwork overlay is not present. The path for the item icon images in CDragon are in https://raw.communitydragon.org/latest/plugins/rcp-be-lol-game-data/global/default/assets/items/icons2d/. Taking the Sandshrike's Claw (ID=7000) example, iconPath: "/lol-game-data/assets/ASSETS/Items/Icons2D/6693_Assassin_T4_ProwlersClaw.png" is the same as Prowler's Claw iconPath
Yeah, it's a shame, but I will change the simple description to the one you've noted. It makes much more sense than the temporary one I've set 👍
I took the same lines of str(item.id) + ".json" in line 81 of items/__main__.py. I will change that line as well to use f-strings.
Will do. I wanted it to fit better in the GitHub comments section
if item is not None: is the same as line 80, but I guess in the mentioned code, it's not necessary since there will always be a populated item. This will be removed as well.
Great! I thought that it wouldn't 'fit' in __main__.py
What do you mean about capitalizing the first work in comments?

I've also looked around and I found this exact issue on Riot Games/developer-relations. This wasn't an issue before the new items, but I guess as temporary fix we can include the above code.
Thanks again for your help!

Hmm, for the icons... they do exist in http://raw.communitydragon.org/10.24/game/data/items/icons2d/ but they are named terribly - you can ctrl+f for "ornn" and find some of them, but others don't have his name in the filename. They also exist on the wikia with a better formatted name: https://static.wikia.nocookie.net/leagueoflegends/images/5/52/Upgraded_Aeropack_item.png which you can find using the inspect tool from https://leagueoflegends.fandom.com/wiki/Ornn/LoL
If you want to set the icon to None for this PR that's fine, but we should link to the correct icon at some point.
"capitalizing the first work" should have been "capitalizing the first word". So # you can't forge ornn items without Ornn as your ally can be changed to # You can't forge ornn items without Ornn as your ally. Typically I follow all the styles in the Python Cookbook by David Beazley, and you'll note that his code snippets read more like a text document than "code", that's how well formatted they are.
Riot is typically slow to update DDragon, although they have been faster to update it when images are missing. It will be a few weeks at least I'd guess though.

I already added ornn items in a branch on my fork. You don't need to add it.

@vorpal56 We talked on the discord about icon images, just leave it set to None for now. We'll fix it later.

@jjmaldonis Ok great. I'll have @dryancd handle the PR. Sorry for not looking at the files in CDragon more thoroughly, there's just so many different files and folders and I was just clicking through each one hoping I would find it 😅

Nah, it's my fault @vorpal56 I didn't create a pr for it. You can still create a pr with your stuff and I'll compare it with mine and see if I need to fix anything.


Add Database Migration Support for SurrealDB

What is the issue?
Currently, our project does not have a mechanism for managing database migrations for SurrealDB. Database migrations are essential for versioning and tracking changes to the database schema over time. As our project evolves, we need a way to apply schema changes consistently across different environments (development, staging, production) without manually executing SQL statements.
How can you resolve it step-by-step?

Evaluate Migration Libraries: We have two options to consider:

surrealdb-migrations: A migration library specifically designed for SurrealDB.
surrealgo_migrate: A pure Go migration library for SurrealDB.

Choose a Migration Library: After evaluating the features, documentation, and community support of both libraries, decide which one better suits our project's needs.

Install the Migration Library: Follow the installation instructions provided by the chosen library's documentation.

Set up the Migration Directory: Create a dedicated directory within our project to store migration files. A common convention is to have a migrations directory at the root level.

Create Initial Migration: Generate the initial migration file that represents the current state of the database schema. This migration file should be named with a descriptive prefix and a timestamp (e.g., 001_initial_schema.surreal).

Write Migration Files: As we make changes to the database schema, create new migration files with descriptive names and timestamps (e.g., 002_add_user_table.surreal, 003_create_index_on_users.surreal). These files should contain the SQL statements necessary to apply or revert the schema changes.

Integrate Migration Commands: Depending on the chosen migration library, integrate the necessary commands or scripts into our project's build process or deployment pipeline. This could involve running migration commands during application startup, deployment, or through dedicated scripts.

Document Migration Process: Update our project's documentation to include instructions on how to run migrations, create new migrations, and manage the migration history.

Test Migrations: Set up a testing environment and ensure that migrations can be applied and reverted successfully without any issues.

Commit and Push Changes: Once everything is working as expected, commit the changes to version control and push them to the remote repository.

References/Links

surrealdb-migrations: A migration library for SurrealDB.
surrealgo_migrate: A pure Go migration library for SurrealDB, with examples.
Database Migrations: Wikipedia article explaining the concept of database migrations.

By following these steps, we can introduce database migration support for SurrealDB in our project, ensuring a consistent and maintainable approach to managing schema changes across different environments.

Hi, sorry what is it that you are looking to do ?

sorry for not adding context.
I was think that we could setup this starter project to also include the migrations using that https://github.com/Odonno/surrealdb-migrations
there is also a pure golang migrations project for surreal DB that we could also look at: https://github.com/JohnKinyanjui/surrealgo_migrate
this woudl then allow developers to have things setup for migration in https://github.com/sbshah97/surrealdb-go-starter-project
Let me know what you think.

This looks interesting and promising. Do you want to put out a MR?

Hi, I've updated the description by a bit, hopefully that helps.

Yes it’s much better .

This looks interesting and promising. Do you want to put out a MR?

Would like to but right now am chockers with work , so I added it so others could also pick it up


clean resource before apply new

Fix: https://github.com/open-cluster-management-io/registration-operator/issues/254
Related PRs:
https://github.com/open-cluster-management-io/registration-operator/pull/250
https://github.com/open-cluster-management-io/registration-operator/pull/252
https://github.com/open-cluster-management-io/registration-operator/pull/234
Signed-off-by: haoqing0110<EMAIL_ADDRESS>

/assign @qiujian16 @elgnay


Block out some code that breaks VS2022 builds

Fixes the issue reported in https://github.com/zeek/zeek/issues/2565
@eladsolomon-ms Does this seem like a reasonable-enough fix for the time being?

Sure, reasonable enough
We haven't tried building in VS2022 yet due to other dependencies we currently have.
I'll make sure to update once we do, and share a PR when we have the fix for VS2022.


ActiveModel::ForbiddenAttributesError raised when Hashie::Mash used with ActiveRecord

Hashie::Mash responds to any respond_to? *? with true, which can cause problems. For instance when performing attribute mass assignment, ActiveRecord checks if the attributes respond to permitted?, then raises ActiveModel::ForbiddenAttributesError if permitted? isn't truthy.
Hashie's behavior: https://github.com/intridea/hashie/blob/master/lib/hashie/mash.rb#L224
ActiveRecord's behavior: https://github.com/rails/rails/blob/v4.2.0/activemodel/lib/active_model/forbidden_attributes_protection.rb#L20
The result is that one can't use Hashie (Grape's params in my case) with ActiveRecord. My work-around right now is to convert all Hashie::Maps to Hashs before handing them off to AR. But that seems like a poor solution. Ideally hashie_obj#respond_to? :permitted? should return false.
Using hashie 3.4.0 and activerecord 4.2.0.

I think you need https://github.com/Maxim-Filimonov/hashie_rails. Can you please confirm?

I'm not actually using rails, but yes that seems to fix it. Thx.

@jhollinger Are you using ActiveModel? That's the specific part of Rails that deals with this issue.

Yep, just ActiveRecord + Grape on Rack.

That gem is misnamed, but it does what it needs to do, see https://github.com/Maxim-Filimonov/hashie_rails/pull/3. Would appreciate any updates to the docs/README/whatever to remove the "rails" part of things.


After starting the server it automatically shuts down and shows this error

Thx fixed, just run as admin ;)


b3dm example not working

Hello, for me the b3dm example didn't works.
TypeError: Cannot read property 'batchTable' of undefined
TypeError: Cannot read property 'batchTable' of undefined
at http://localhost:44716/js/3dcitydb-web-map-master/js/b3dmlayer.js:227:54
Whats my fault?
is b3dm better than gltf? and how i get these files from 3DCityDB?
For me it is not really clear, which cesium instance i should use to show performant and interact (select) of buildings with tiles.

Hello Stretch345,
We haven't yet updated the B3DM Dataset for the currently used Cesium Version.
you can either wait until we update the dataset or can go back to the following commit.
https://github.com/3dcitydb/3dcitydb-web-map/tree/c47d108aefedffac628e60d157afb79f45a9ec0b
There the B3DM example should still work.
At the moment it is not possible to create theses files from an 3DCityDB.
In one to two weeks a new Importer/Exporter will be released with support for GLTF.
The 3DCityDB to B3DM Konverter is a closed Source internal implementation from virtualcitySYSTEMS.

Hallo jbo023,
does the exports of 3dcitydb only works with your own cesium viewer https://github.com/3dcitydb/3dcitydb-web-map or should it work with the cesium 3d-tile branch?

The 3DCityDBWeb-Map is just an Wrapper around Cesium to guarantee the same api for two different datasets.

An KML File with network links referencing GLTFs (will soon be exportable from the opensource Importer/Exporter in the 3dcitydb organization. (Can be used with the Cesium Master branch)
An B3DM Dataset for the 3D-Tiles implementation of Cesium (this uses the 3D-Tiles branch)
The CitygmlToB3DM Converter is an internal implementation from virtualcitySYSTEMS, if you have further interest, you can contact us.

thanks,
ok, i think for the beginning it's the best to use kml.
could you tell me how to color the bildings dynamicly?
I have a json file with the ids of the bildings and a color value.
Is there a given function or does i have to edit a specific function?
I see the setEntityColorByPrimitive in citydbkmllayers. Could this be the right function?
Thanks
Stephan

Hi Stephan,
the dynamic Highlighting (Mouse-Click, Mouse-Over) of 3D-objects has been implemented in the 3DWebclient shipped with this 3dcitydb-web-map API. Please refer the javascript code under this URL:
https://github.com/3dcitydb/3dcitydb-web-map/blob/master/3dwebclient/script.js#L401
A comprehensive User-Guide to using this 3dcitydb-web-map-API and the 3DWebclient (as well as the KML/glTF Exporter Tool) will be released soon.
Zhihang

thanks i will have a look at this cript. I need it not as a click event for one building, but for all buildings at init. I want to color all buildings with a color by a special json file. So i have to process the array of buildings.

ok, i found a solution in citydbKmTilingManager.js -> this.oTask.addListener("checkMasterPool", function (matrixItem, taskQueue) :-)
newKmlDatasource.load(tileUrl).then(function(dataSource) {
						    var entities = dataSource.entities.values;
							for(var i =0; i < entities.length; i++) {
								var e = entities[i];
								console.log(e._name);
								//e.wall.outlineColor = Cesium.Color.RED.withAlpha(0.5);
								if (e._name == "BLDG_0003000e00234d99")
									e._children[0]._polygon._material._color._value.blue = 1;
							}

i have two layers, one extruded, one geometry lod2.
I Think because auf prefetching i get a overload and it crash.
Is it possible to disable prefetching or is there a better possiblility?
buildingLayerGeom.citydbKmlTilingManager.startPrefetching = false;
Didn't disabled it.
Thanks

Hi,
I don't think the prefetching is the main reason for that crash issue you mentioned.
Have you created the extruded and geometry layers by using the Cesium-Branch of the importer/exporter tool?
If so, could you share with me your exported master JSON file?
email<EMAIL_ADDRESS>regards,
Zhihang

Hi Zhihang,
I used your code to color only one building but I ended up with more than one.

What do you propose if I want to add colors to buildings with respect to heat demand?
Regards,
Shahab

Hi Shahab,
the class CitydbKmlLayer has a public function called highlight  (see the link below) which can be used for coloring an arbitrary number of KML/glTF objects.
https://github.com/3dcitydb/3dcitydb-web-map/blob/master/js/CitydbKmlLayer.js#L344
An example usage should somehow look like the JS code below:
citydbKmlLayer.highlight(
{
"building_GMLID1": Cesium.Color.RED,
"building_GMLID2": Cesium.Color.YELLOW,
"building_GMLID3": Cesium.Color.BLUE,
 .....
}
);

Before calling this function, you may need to programmatically determine which color should be assigned to the individual building object according to its heating demand. The conceptual methods for creating such thematic map have been outlined in many research papers in the field of cartography.
By the way, on the Cesium homepage there is an awsome 3D Tiles online demo (you might have already seen it before), where the NYC building objects can be rendered with different colors based on the building height values. However, this web application is not open source.

Hi Zhihang,
I think I'm making a mistake because I could not implement that.
Could you please be more specific?

this is the error:
b3dmlayer.js:510 Uncaught TypeError: Cannot read property 'DEHH_5df9c3a4-2319-473b-9bf8-72d7d0fe0220' of undefined
and this is the code snippet in index.html:

Hi,
i cannot see the code snippet on this page. Could you check your last comment?

Hi
this is the error:
b3dmlayer.js:510 Uncaught TypeError: Cannot read property
'DEHH_5df9c3a4-2319-473b-9bf8-72d7d0fe0220' of undefined
and this is the code snippet in index.html:

On Fri, Oct 7, 2016 at 3:01 PM, Zhihang Yao<EMAIL_ADDRESS>wrote:

Hi,
i cannot see the code snippet on this page. Could you check your last
comment?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/3dcitydb/3dcitydb-web-map/issues/1#issuecomment-252245909,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATz7cNQNdT2yF8xPHrMGfQd10gnlSRH0ks5qxkKcgaJpZM4HMeNz
.

--
Regards,
Shahab Shafieihajiabady

I suppose that the KML dataset you use was exported using the Import/Export tool. In this case, you have to use the CityDBKmlLayer class for loading the KML dataset into the 3dcitydb-web-map.
You may also need to take a look at the code example (see the link below) showing how to create a CityDBKmlLayer instance in JavaScript.
https://github.com/3dcitydb/3dcitydb-web-map/blob/master/examples/kmlManhattanExample/index.html#L97

Dear Zhihang,
As you see in the screenshot, everything is already done.
The project is running and all the layers are added.
I also changed some fields to fit them to our needs.
The question is:
How can I add colors to the building individually?
When we use CitydbKmlTilingManager.js like this:
if (e._name == "DEHH_9d4c8b35-56b8-4fae-9228-46a2686391d8")
e._children[0]._polygon._material._color._value.blue = 1;
it colors all the buildings in the respective tile
But we want to have access to each building's color separately.

Regards,
Shahab
On Fri, Oct 7, 2016 at 3:17 PM, Zhihang Yao<EMAIL_ADDRESS>wrote:

I suppose that the KML dataset you use was exported using the
Import/Export tool. In this case, you have to use the CityDBKmlLayer
class for loading the KML dataset into the 3dcitydb-web-map.
You may also need to take a look at the code example (see the link below)
showing how to create a CityDBKmlLayer instance in JavaScript.
https://github.com/3dcitydb/3dcitydb-web-map/blob/master/
examples/kmlManhattanExample/index.html#L97
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/3dcitydb/3dcitydb-web-map/issues/1#issuecomment-252249390,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATz7cJNo9PL99iChylofAJOKojhp_43wks5qxkZ3gaJpZM4HMeNz
.

--
Regards,
Shahab Shafieihajiabady

Dear Shahab,
As i mentioned in my earlier comment, the best way for coloring individual building object is to use the public function highlight   in the class CitydbKmlLayer. I also utilized this function to implment the highlighting (Mouseover and Mouseclick) function for the 3D web client. The following link take you to the corresponding code fragement:
https://github.com/3dcitydb/3dcitydb-web-map/blob/master/3dwebclient/script.js#L450
regards,
Zhihang

Hi Zhihang,
This is what I put in index.html:
<script>
     CitydbKmlLayer.prototype.highlight(

{
"DEHH_efd4b191-e991-4372-a0ce-7f16e6fc9748": Cesium.Color.RED,
"DEHH_9d4c8b35-56b8-4fae-9228-46a2686391d8": Cesium.Color.YELLOW,
"DEHH_1ab21131-6c4f-4af7-8b88-5bbff054ad1b": Cesium.Color.BLUE
}
);

and this is the error:
CitydbKmlLayer.js:359 Uncaught TypeError: Cannot set property 'DEHH_efd4b191-e991-4372-a0ce-7f16e6fc9748' of undefinedCitydbKmlLayer.highlight @ CitydbKmlLayer.js:359(anonymous function) @ index.html:371
Regards,
Shahab

Hi
I think this part of code has a problem
CitydbKmlLayer.prototype.highlight = function(toHighlight){
for (var id in toHighlight){
this._highlightedObjects[id] = toHighlight[id];
this.highlightObject(this.getObjectById(id));
}
this._highlightedObjects = this._highlightedObjects;
}

Dear Shahab,
the web link below maybe helpful for you get a better understanding of using JavaScript's prototype.
http://stackoverflow.com/questions/1635116/javascript-class-method-vs-class-prototype-method
regards,
Zhihang

Dear Zhihang,
Could you please put the code to color only one building with gmlid using toHighlight function?
I mean the code snippet that I can use either in index file or script file.
Regards,
Shahab


Update type for Subscription supports new fields

Hi,
As I have tested some cases for data feed coming from Instagram by registering the Subscription of the real time updates, I see some new fields are added to the in coming structure.
I see ig package supports this feed by "Update" type, but this additional fields do not seem to be supported by this type.
Now update feed coming like this, that the ones under "data" field looks new ones.
[
{
    "changed_aspect": "media",
    "object": "user",
    "object_id": "2157425226",
    "time":<PHONE_NUMBER>,
    "subscription_id": 19731857,
    "data":
    {
        "media_id": "1063661397067665037_2157425226"
    }
}
]

I do not see this is properly documented in the Instagram API document, but do you have any plans to support this in near future?
Thank you,

Hmm, does that fact that there isn't documentation about that field mean that they're lazy, or that this is an unsupported API?

I suppose they are simply a bit lazy.
The sample is in "Receiving Updates" section in the following URL. It does not include "data" field in there. I could not find the comments for this field in other places either as long as I have checked.
https://instagram.com/developer/realtime/

@cdepillabout What's your opinion about including this field to ig?

Hmm, I checked out the Ruby client and Python client to see how they are handling subscriptions.
The Python client doesn't seem like it has any specific support for handling the data in a subscription callback.
The Ruby client looks like it provides some helper methods for handling callbacks, but it doesn't particularly help you out with the data in a subscription callback.
However, it does have have a test that deals with subscription callback.  Here is the data they are using in their test:
https://github.com/Instagram/instagram-ruby-gem/blob/master/spec/fixtures/subscription_payload.json
It doesn't look like they are using the extra data field in their test.
I'll email instragram and try to find out what's up.


nixos: some improvements for dnscrypt-proxy

Enable systemd support and some additional hardening.

Pushed a squashed series which I hope makes a bit more sense.

Rebased on master to resolve merge conflict.

Rebased & added support for custom upstream resolvers.

I've used these changes for a long time without problems, I believe they should be merged at some point.

Bump.

Thanks for the PR, and sorry for the delay. I think it's a good idea to @ - mention people familiar with the files you're changing, or to ping the nix-dev mailing list. Unfortunately we don't seem to have an effective way to make sure things like this don't fall through the cracks.
(I personally have no idea what dnscrypt-proxy is but these look like careful changes so I merged them.)

(I mean, do that next time you want someone to pay attention to your PR; you don't have to do it now.)


Questions about Adaptive S-Norm for ASV

Thanks for sharing the code. In your paper, it mentioned that the Adaptive S-Norm was applied. But implementation in the utils.py is different from AS-Norm. The code didn't choose some top n cohort but use all (10000 in the code) from the training set to compute the mean and std. It looks like S-Norm, not AS-Norm. I was wondering did you have a chance to compare S-Norm vs AS-Norm in the experiment? Will you add AS-Norm into the code as well?

Thank you for your feedback regarding our paper and repository. We acknowledge the inaccuracies in the paper regarding the measurement and comparison of adaptive s-norms. It was not part of our study, and we do not intend to include adaptive s-norms in the associated repository.
Your attention to detail is appreciated, and we apologize for any confusion caused by these inaccuracies.

Thanks for your quick reply!
Here is another problem I met recently. For ASV task, if I replace the Cross entropy loss with the commonly used AMSoftmax, the training process will get some problems, training loss cannot converge and training ACC keeps very low (lower than 1%). This is very weird. I was wondering if you have tried using AMSoftmax and got a similar observation. Thanks.


No players showing in UI / Ingress

What version of Music Assistant has the issue?
2024.1.0
The problem
Since the addon updated to 2.0.0b96, I can no longer configure any players in the UI. The page is empty. If I click on the speaker icon however then I can see all my receivers.
I’ve tried a complete uninstall of the addon and the integration but it’s still the same.
The providers tab and the core tab appear to be fine.
How to reproduce
Update addon to 2.0.0b96
Lose ability to configure players in the UI
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.2.2
What type of installation are you running?
Home Assistant Supervised
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Fixed in 2.0.0b97 thanks

Broken again in 2.0.0b98

This is working fine for everyone else as far as I am aware. Try b99

Fixed in 2.0.0b100 thanks. I think it was somehow linked to AirPlay receivers not working

Seems broken again in 2.0.0b107
The only player showing is a new speaker just added yesterday (it has 2 versions Google Cast & AirPlay) plus a GoogleCast group, also added yesterday. Others added prior are missing in Settings > Players but are showing in the speaker selection list.


Create a unit test that assert that named plans overwrite previously named plans

Example pseudo-code:
transaction.add(plan: PlanA, to: target, withName: "plan")
transaction.add(plan: PlanB, to: target, withName: "plan")

Should result in PlanA being added, removed, and then PlanB being added.

Was looking into taking a crack at this yesterday evening. Had a look atMDCTransaction:
NS_SWIFT_NAME(Transaction)
@interface MDMTransaction : NSObject

#pragma mark Adding plans to a transaction

/** Associate an plan with a given target. */
- (void)addPlan:(nonnull id<MDMPlan>)plan
       toTarget:(nonnull id)target
    NS_SWIFT_NAME(add(plan:to:));

@end

Would tackling this issue also include adding another method to MDCTransaction which would look something like https://github.com/material-motion/material-motion-runtime-android/blob/develop/library/src/main/java/com/google/android/material/motion/runtime/Transaction.java#L60 ?

Yep! We've got this listed in https://github.com/material-motion/material-motion-runtime-objc/issues/8, which if we could mark it as such would be a blocking issue on this one.

https://github.com/material-motion/material-motion-runtime-objc/issues/79 is a blocking issue as well.

And the spec: https://material-motion.gitbooks.io/material-motion-starmap/content/specifications/runtime/named-plans.html

Closing this out as https://github.com/material-motion/material-motion-runtime-objc/pull/88 has been merged in.


[Bug]: Unable to change the Project Home Folder

Apache Hop version?
2.9.0
Java version?
11
Operating system
Windows
What happened?
Installation details:-
Download and extract the apache-hop-client-2.9.0.zip file from the official page: https://hop.apache.org/download/
apache-hop-client-2.9.0.zip
Java Version:- 11
Reproduction Steps:-

Open HOP GUI
Create a New Project set the local Windows path to the HOP project and Save.
Change the Project Home Folder From Windows Filepath to the Incorrect Filepath and click Save.
Ex: Windows Filespath: D:\ProjectFiles\HOP-Files  ==> D:\ProjectFiles\HOP-Files1
Now try to edit the Project to change the Home Folder to a valid path.
We get the Project Home Folder doesn't exist message.

Issue:- The HOP GUI application is not allowing us to the change "Project Home path" to a valid path after setting an invalid Project Path.

Below is the error message that we get.
Error Message:-
"""HopException:
Error calculating actual project config filename
Project home folder 'D:\ProjectFiles\HOP-Files1' does not exist
Root cause: HopException:
Project home folder 'D:\ProjectFiles\HOP-Files1' does not exist"""
Issue Priority
Priority: 3
Issue Component
Component: Hop Gui

.take-issue

It was not allowing me to Delete the project.

However, I tried restarting the HOP GUI and it reverted to the old settings.


i18n: Italian Translations

Ciao!
Here are the Italian translations!
Thanks for making this awesome project opensource, I'm having a lot of fun poking around in the code :P
I'll try to keep up with any further translations that might be needed.
Cheers!
~S²

Woops! Just noticed Prettier messed with indentation, fixing right away!


remove IPFS dependency

Currently, the only reason we can't replace all IPFS handling  is that we need to fetch each subgraph manifest to get min_block. If we can move that field into the Deployment entity of the network subgraph, then we can remove a big pile of code associated with loading manifests.

resolved by #647


Realm Crash when using in Memory and On Disk realms

I have a problem I don't understand with Realm.
Here is the context:
I use an inMemory realm for the data I just want to use across my app view controllers, but I don't want them to be persisted between launches.
It works fine...but as soon as I added a 2nd Realm, used persistently this one, it started crashing when I add a new field to the data saved in the "inMemory" realm.
I can't figure it out. I have tested without using the persistent Realm, and there I can add/remove fields on my inMemory objects and no crash.
But as soon as the persistent realm is created as well, and I add/remove attributes from one of the "inMemory" object, it crashes straight away.
I am using Realm Objective-C 96.3
I create my inMemory realm this way:
                  instance = [[self alloc] init];
                  RLMRealmConfiguration* config = [[RLMRealmConfiguration alloc] init];
                  config.inMemoryIdentifier = @"WeeloyRealmInMemory";

                  NSError* error;
                  instance._in_memory_realm = [RLMRealm realmWithConfiguration:config error:&error];

I create the persistent one this way :
RLMRealmConfiguration* config = [RLMRealmConfiguration defaultConfiguration];
config.schemaVersion = CURRENT_CONFIG_SCHEMA;

NSError* errorRealm;
RLMRealm* realm = [RLMRealm realmWithConfiguration:config error:&errorRealm];

It crashes on Simulator ios 9.2.1.
I haven't test on physical devices, because I doubt it's device/OS related.
I think it some how points to the same file on the disk, when it shouldn't.
The error is : 0  @"NSLocalizedDescription" : @"Migration is required for object type 'RestaurantRLMObject' due to the following errors:\n- Property '_test_a' is missing from latest object model."
( I tried delete/add a property, it works fine as long as I am using only one realm, but if I have both, it crashes each time :/ ).

I have already sent an email to Realm guys, but in case some one else had had the issue and sorted it out, I wanted to post it here too.

Update on it.
I set up a path on the In-Memory configuration, then I set the in-memory identifier ( so it nullifies the path, but I wanted to be sure, nothing nasty is done by realm with the path).
I have tried like with this update.,...result is that I still get the error when I create the on-disk realm...but if I  increase the CURRENT_CONFIG_SCHEMA number....then the error disappears.
That is not normal behaviour, I have double-checked and the on-disk realm doesn't contain the same objects as the in-memory one...so I still don't understand why the on-disk realm is affected, it makes absolutely no sense.
This happens at the very beginning of the app too, so there's no object in any of the realm either.
If you could help me understand why this is happening ?

This issue is followed by https://secure.helpscout.net/conversation/183705997/3451/?folderId=440773

I  meet the same issue. How to solve it?

@fanxiushan , HelpScout is a ticketing system which Realm used to use for support requests.  It is no longer used.
I took a look over the internal notes and it looks like this user had an error in their schema migration logic.

In my case, I found that when I use both memory and disk realm store, my data structure in memory realm will still store my disk realm (only the schema in disk realm,but no data),so when I modify the memory realm schema(ex:I add a field on exsit schema) . Then I launch app, the app crashed.


[docs] Fixed typo in readme

Fixed a typo mistake in the readme.

Coverage remained the same at 99.295% when pulling e192ca4b92cf3dd013e56d75bc329a5e2106daa0 on fix-typo-in-readme into f01239675778a09f4e1365fd12cd7795797f7144 on master.


Add auth examples to CI

Closes #479.

I still need to add the Python example.

@AlekSi PTAL

@AlekSi do I need to add the python-example as a submodule in a separate PR?


[newrelic-logging] 1.17.1 fails with parser 'cri,docker' is not registered

Bug description
using chart version 1.17.1 with the default config, we are getting this Parser error:
[2023/07/28 12:09:54] [error] [input:tail:tail.0] parser 'cri,docker' is not registered
Version of Helm and Kubernetes
Helm 3.12
k8s 1.25
Which chart?
newrelic-logging 1.17.1
What happened?
Throws an error about the Parser.  Previously the chart set LOG_PARSER to either cri or docker, but now sets it to cri,docker or docker,cri.
Seems the Parser can't handle a list?
What you expected to happen?
cri parsing works without throwing the error
How to reproduce it?
Install using the default configs
Anything else we need to know?

@jimmyR this has been fixed in #1130. Apologies for the trouble! Please try again with the latest chart version (1.18.1), and let me know if you're still encountering a problem. Thanks!


Fire sample

New sample new particle builder with a fire decorator.

Currently blocked by adding the frontend-devtools package.   @StefanRetief will push a fix either here in a separate PR.
Icons need to be updated.
README needs to be written.
Consider rewriting placement tool to not use AccuSnap.

After npm install, npm run build produces:
.\src\frontend-samples\fire-decoration-sample\Particle.ts
Cannot find module: '@bentley/frontend-devtools'. Make sure this package is installed.

That package is not listed in your package.json. I don't think you should introduce a dependency on what is basically a developer-only package, just to get a couple of trivial functions. Copy them instead?

When placing a new emitter, it uses the default settings, but the UI doesn't update. e.g., the new particle does not emit smoke but the smoke toggle remains on.

Not a able to reproduce.  Perhaps there is something with the UI that is confusing?  Please list steps.

I have had little success selecting an emitter.

The way it works is unintuitive, as it functions by getting the closet emitter from the point given by AccuSnap.  If that distance is greater than the effect's range plus half a meter, then returns undefined.  It would work better if it ray cast for the effect's bounding box or something like that.  Any suggestions from you about it?

Can the effect be made more triangular?

I'll revisit that formula.

Do you intend to include snow particle effect sample? Would that be separate? The "viewer feature" being demonstrated is not "fire", it's "particle effects".

@Josh-Schifter and I talked about this some.  The snow would be a different sample.  This just "how to make fire using particles" sample.  If that changes in the future, updates can be made.

Not a able to reproduce. Perhaps there is something with the UI that is confusing? Please list steps
Start app or reload page.
Place an emitter.
Adjust the sliders and enable smoke.
Place another emitter.
The new emitter uses the default "Candle" settings but the sliders and toggles remain unchanged - e.g., smoke still enabled.

The way it works is unintuitive, as it functions by getting the closet emitter from the point given by AccuSnap. If that distance is greater than the effect's range plus half a meter, then returns undefined. It would work better if it ray cast for the effect's bounding box or something like that. Any suggestions from you about it?

ParticleCollectionBuilderParams.pickableId.

@Josh-Schifter and I talked about this some. The snow would be a different sample. This just "how to make fire using particles" sample. If that changes in the future, updates can be made.

Please make it clear that the feature being demonstrated is Particle Effects, not fire simulation, by e.g. renaming the sample to "Particle Effects - Fire". Particle effects are a Viewer Feature; Fire Decorator is just an example of something you can do with that feature.


(WIP) Add test suite

This is a pull request to add tests to winterfell that would fix issue #18 it's currently a early WIP.
I've started using Jest with Enzyme which are a good match for unit testing components, however some feature tests might benefit from a different stack which doesn't auto mock by default.

[x] Add test framework setup
[ ] Create tests for all input types
[ ] Remove dist folder from VCS

Hi @timReynolds.
Thanks for the PR. Just having a quick look over this I've noticed two things:

ReactDOM has been imported as ReactDom, please capitalise the "Dom" part.
Removal of the npm-debug.log file, also worth adding that to the .gitignore.

I'll get some time to look closer at this soon, but until then, thanks for the PR!
Thanks.

Still needs lots of work, just trying to put down the foundation yesterday when I had 10 min.

On 7 Mar 2016, at 17:09, Andrew Hathaway<EMAIL_ADDRESS>wrote:
Hi @timReynolds https://github.com/timReynolds.
Thanks for the PR. Just having a quick look over this I've noticed two things:
ReactDOM has been imported as ReactDom, please capitalise the "Dom" part.
Removal of the npm-debug.log file, also worth adding that to the .gitignore.
I'll get some time to look closer at this soon, but until then, thanks for the PR!
Thanks.
—
Reply to this email directly or view it on GitHub https://github.com/andrewhathaway/Winterfell/pull/57#issuecomment-193350111.

Hi @timReynolds. Is this going to be worked on further? Thanks.


FZ Editor doesn't work in the current master

Steps to reproduce

get latest master
rebuild the whole solution and launch PT
invoke FZ Editor by hotkey, e.g. win+~
it throws an exception and stays in memory forever

Expected Behavior

FZE doesn't stays in memory if it encountered a critical bug, and exits instead
FZE doesn't throw

Notes
The issue is CNR if we rebuild the FZEditor project, so I suspect it could be related to having a shared build dir with other .NET projects
Screenshots

Not sure if this error is related:
CLR:(C:\WINDOWS\Microsoft.Net\assembly\GAC_64\mscorlib\v4.0_<IP_ADDRESS>__b77a5c561934e089\mscorlib.dll)
Rejecting native image because native image dependency
C:\WINDOWS\Microsoft.Net\assembly\GAC_64\mscorlib\v4.0_<IP_ADDRESS>__b77a5c561934e089\mscorlib.
dll had a different identity than expected

is this a good enough reason to try the full .net core 3.1 shift?

Let's try.

update: based on conversations with the team, belief is this is due to a common temp directory.  We also believe this is what caused a unit test to fail on the last CI

tests pulled, stuff works again


TypeScript Fix: Renamed invalid export type MapViewAnimated to Animated

Does any other open PR do the same thing?
No
What issue is this PR fixing?
It allow typescript to import the Animated export. Without it - anyone using typescript won't be able to import the Animated export.
How did you test this PR?
As this change is only applied on the typescript definition - no test is necessary on the functionality of this plugin.
But you can test the typescript support by simply importing the Animated export and transpile using TSC command. Which will allow the typescript import without any issue.
Tested on latest stable Typescript 4.0.3. Also tested the code working by running the app on a Android device on debug mode.

This was recently changed https://github.com/react-native-maps/react-native-maps/commit/acd5ebafc8f063763e24e271db6d4ad3cd7b26da
@aajahid What's the discrepancy here?

@christopherdro - I assume MapViewAnimated was renamed into Animated.
When I tried to use MapViewAnimated ( my IDE suggested there is something available based on the typescript ) - In run time I found there no such thing exported from the react-native-maps library.
Hence I renamed it to Animated and now I can use the Animated export.


For: WIP Controller, figure out who populates various GUIDs

Currently controller (wip) does not populate the GUIDs before talking to the broker, say, when creating a Service Instance. My understanding is that the controller is responsible for this.
There's a TODO in wip controller so if you resolve this, also update / remove the comment.

From @pmorie :  It's the API server's job to create GUIDs for: Instance, Binding. GUIDs for ServiceClass and ServicePlan come from the broker.

@vaikas-google It seems like we should encode this information somewhere in docs and possibly types.go, agree?

Absolutely. I think controller should also check that it's indeed populated as bad things will happen if it's not. Not sure what the guarantees there are however from the API server side, but seems safer to make sure it's there always?


Update pnpm (v7) and Node.js (>=16)

Reason:
When starting out I couldn't start anything because the project demands Node.js <= v16. I'm at Node.js v18 and v20 is also released. So I figured we should basically turn this around now: let's allow Node.js >= v16. Also we can start using pnpm v7 (had some issues with pnpm v8 we can look into later).
Closes [issue link]
Changes made (preferably with images/screenshots):
Check off the following:

[ ] I have reviewed my changes and run the appropriate tests.
[ ] I have have run rush change to add the appropriate change logs.
[ ] I have added/edited docs.
[ ] I have added tutorials.
[ ] I have double checked and DEFINITELY added docs.

Tracking https://github.com/microsoft/rushstack/issues/4033 for pnpm v8

There are some issues that arise when using pnpm 7.
Perhaps it would be best to only update the node version to 16 at this stage

There are some issues that arise when using pnpm 7.
Perhaps it would be best to only update the node version to 16 at this stage

Thanks, closing this in favor of #273.


what is Wide RevNet in your paper?

https://arxiv.org/abs/1707.04585
Papers such as https://arxiv.org/pdf/1907.02544.pdf use Wide RevNets.


Uncaught error at user-stream.js

Hi. I'm suffering this error. No tweets at all in the mirror.
It looks like internet connection error but there are more internet connected modules working without issues. Even restarting the app when RPi is running some minutes (connection established correctly) fails with same error
{ Error: Uncaught, unspecified "error" event. ([object Object])     at Stream.emit (events.js:163:17)     at ClientRequest.<anonymous> (/home/pi/MagicMirror/node_modules/user-stream/                                                                                lib/user-stream.js:173:16)     at emitOne (events.js:96:13)     at ClientRequest.emit (events.js:188:7)     at TLSSocket.socketErrorListener (_http_client.js:306:9)     at emitOne (events.js:96:13)     at TLSSocket.emit (events.js:188:7)     at emitErrorNT (net.js:1272:8)     at _combinedTickCallback (internal/process/next_tick.js:74:11)     at process._tickCallback (internal/process/next_tick.js:98:9)   context:    { type: 'request',      data:       { Error: socket hang up           at TLSSocket.onHangUp (_tls_wrap.js:1087:19)           at TLSSocket.g (events.js:286:16)           at emitNone (events.js:91:20)           at TLSSocket.emit (events.js:185:7)           at endReadableNT (_stream_readable.js:926:12)           at _combinedTickCallback (internal/process/next_tick.js:74:11)           at process._tickCallback (internal/process/next_tick.js:98:9) code: 'E                                                                                CONNRESET' } } }

That error looks like it has to do with the external dependency user-stream. Make sure your auth token credentials are set correctly and you have a good internet connection. If it still doesn't work, you'll have to seek more support from that repo.


Update discipline-scalatest to 2.1.1

Updates org.typelevel:discipline-scalatest from 2.1.0 to 2.1.1.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel", artifactId = "discipline-scalatest" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #76 (62e0dc4) into master (02ad2db) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #76   +/-   ##
=======================================
  Coverage   94.93%   94.93%
=======================================
  Files           1        1
  Lines          79       79
=======================================
  Hits           75       75
  Misses          4        4

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 02ad2db...62e0dc4. Read the comment docs.


Show only "important" Vaadin versions in the compatibility matrix

^title says it all 😀
This would decrease the information overload by just showing the "important" / "supported" releases in the compatibility matrix like e.g. 14,22,23+
Some could argue that the other versions should be accessible.. like some kind of "show more" or remove them all together - I'm fine with both.
Example:

Taken from here https://directory.demo.vaadin.com/addon/multiselect-combo-box

Yes, this would improve the readability. One option is is to use similar grouping to https://caniuse.com/?search=DateTimeFormat

That would be awesome!

Implemented a version of this. How does it feel like?

That looks way better! Thanks a lot


Weapon types

this mod doesn't sync well with the origins mod as every weapon is identified as a sword, even the axes which makes no sense. have you thought about changing this?

in what categories can the weapons get identified else?

well you can add a axes.jason in this:
(https://github.com/Globox1997/MedievalWeapons/tree/1.19/src/main/resources/data/fabric/tags)/items/
divide them between axes and swords. weapons like daggers, staffs (which are basically sticks), and obviously swords would be in the sword.jason while axes (including fransisca), maces, and spears, can all be defined in the axes.jason

Which tag gets exactly used? what is its namespace?
c:swords and c:axes or something else?

I would assume so. if you have the namespace for swords you would just have to replace swords with the word axes. I haven't written a mod before, but I have browsed through a lot of mod codes. (I'm curious that way). and seeing other weapon mods using different json's for each item type. and having the correct extends (ex: extends sworditem, extends axeitem)

the issue I see is, the axes aren't real minecraft axes, they are not intended to chop wood

they don't have to be. hell, you can chop a tree with a Minecraft sword. use a Minecraft axe for combat. but anyone who uses the origin's mod and it's companion Medieval Origins. should they use a origin like Paladin who has a decrease on damage with swords and a damage buff with axes. all the weapons in your mod become useless. and you have some nice weapons to use too.

No plans to change it. Maybe origins provide something to determine between different weapons.


Final UP change communication plan

User story
As a user, I want to know how ReportStream can help me so I can send or receive the data I need.
Background & context
We need a more specific direction and framework for moving forward with communicating about the UP through piloting and slow onboarding this quarter and moving into full release in 2024.
Now that we have tried to get a clear picture of the user's journey and their needs, we can pull together what we know into a framework for actionable plans.
Open questions
Working links
-Strategy deck

Mural space

Acceptance criteria

[x] Work from last sprint finalized into a reference deck/doc/place

Deck with plans


Create USWDS Utility Class Prop Type for use across components

Discussed in https://github.com/MetroStar/comet/discussions/74

Originally posted by jpandersen87 September  7, 2023
It would be useful for all the possible utility classes (not usa- classes, as that's expected to be the component's responsibility and can be given via className by the user otherwise) to be applicable via a prop object similar to how the native style prop works.

Closing due to upcoming changes to USWDS variables and utitlities.


Simple Compose additions/ example

Hi,
thanks for the Re-Release!
I am currently struggeling with the Mumble LDAP Authenticator. The Server is currently Running 1 Release behind on Static x86.
In this Setup, i Build murmur & the LDAP Authenticator from the Git Tag of the Latest Release.
I have used the following Configs, but it doesn't seem to work.
The LDAP seems to can't connect to the murmur server and i cant figure out why. I have tried switching up the IPs (<IP_ADDRESS> / <IP_ADDRESS>) but that doesn't help at all.
docker-compose.yaml
version: "3.7"
services:
  murmur:
    build: ./mumble
    volumes:
    - $PWD/conf/murmur.ini:/etc/murmur/murmur.ini:ro
    - $PWD/murmur.sqlite:/var/lib/murmur/murmur.sqlite:rw
    - $PWD/log/murmur.log:/var/log/murmur/murmur.log:rw
    ports:
    - 6502:6502
    networks:
      ice:
        ipv4_address: <IP_ADDRESS>
  murmur-ldap:
    build: ./mumble-scripts/Authenticators/LDAP
    volumes:
    - $PWD/conf/LDAPauth.ini:/app/LDAPauth.ini:ro
    - $PWD/log/LDAPauth.log:/var/log/murmur/LDAPauth.log:rw
    - $PWD/mumble/src/murmur/Murmur.ice:/app/Murmur.ice:ro
    networks:
      ice:
        ipv4_address: <IP_ADDRESS>

networks:
   ice:
      driver: bridge
      ipam:
        config:
        - subnet: <IP_ADDRESS>/24

LDAPauth.ini
. . .
;Ice configuration
[ice]
host            = <IP_ADDRESS>
port            = 6502
slice           = Murmur.ice
secret          = 4MST5yDtefksy3oLWMMNN
watchdog        = 30
. . .

LDAPauth Dockerfile
FROM python:slim-bullseye
WORKDIR /app
COPY LDAPauth.py LDAPauth.py
RUN apt-get update -y && apt-get install -y libsasl2-dev python-dev libldap2-dev libssl-dev build-essential libbz2-dev
RUN pip3 install python-daemon python-ldap zeroc-ice
ENTRYPOINT python3 LDAPauth.py
EXPOSE 6502

murmur.ini
. . .
ice="tcp -h <IP_ADDRESS> -p 6502"
icesecretread=***********************
icesecretwrite=***********************
;identical

Could you Provide Official examples on how to set it up?

Hm - I have never used docker-compose nor have I ever touched that LDAP-thingy, so I'm afraid that I'm not too much of a help here xD
@azlux do you have any ideas on this?
@Natenom do you happen to have any experience with LDAP?

My Current Issue is related to the connection between the mumble server and LDAPauth via ice.
I have tried so far:
Murmur [Container] -> LDAPauth [Directly on Linux VM] ✔
Murmur [Directly on Linux VM] -> LDAPauth [Directly on Linux VM] ✔
Murmur [Container] -> LDAPauth [Container] ❌
I Really have Issues if the IP of the LDAP Authenticator and the Murmur Server is not the Same.

What about the case
Murmur [Directly on Linux VM] -> LDAPauth [Container]
?
Does that also fail?

I have not tested that since the Static Server is discontinued. If it makes sense i would try it.

Yeah well it would be more in order to complete the test-case set than as a suggested workaround. I was just curious what the outcome would be. Maybe that could hint as to where the problem lies.

If you have a custom config file, you should use the MUMBLE_CUSTOM_CONFIG_FILE variable : https://github.com/mumble-voip/mumble-docker#additional-variables
I thing you issue is because the image don't take your custom config

Okay, i try to change it on my VM.
https://github.com/mumble-voip/mumble/blob/d16ad421d496aefdc68468d1e64425a086d0a32a/scripts/murmur.ini#L65
Do i have to enter here the IP/Hostname of the Authenticator or does it work with an Wildcard or CIDR?

I doubt that it would work with a wildcard as the ICE interface expects a concrete address to bind to (afaik)... I might be wrong though :eyes:

https://doc.zeroc.com/display/Ice34/Proxy+and+Endpoint+Syntax#ProxyandEndpointSyntax-AddressSyntax
<IP_ADDRESS> should work as a IPv4 wildcard. :: for IPv6. * may too; I’m not entirely sure whether the server is considered a “proxy”.

Given that Mumble in container, ldap auth on host works, I don’t see why it’d be a config file not being used issue?
In your posted configuration I see LDAP docker conf has EXPOSE 6502. Doesn’t that mean it’d expect it as an outgoing port, and may not identify it as an incoming port of the network?
https://docs.docker.com/engine/reference/builder/#expose

Never used the Ice Protocol before. I tried it vice versa but it didn't worked. Has anyone of you time to run a Quick Test with my Setup? I really tried much, but either the Authenticator can't connect at all or i get the Error that the Address is already allocated.

I've added a docker-compose example into the wiki : https://github.com/mumble-voip/mumble-docker/wiki
If you want to use the nework for ice, you should use :
ice="tcp -h <IP_ADDRESS> -p 6502" into the config
AND
EXPOSE:
 - 6502

to allow other containers to reach this port.

2022-05-11 10:36:20,914 INFO Shutdown complete
2022-05-11 10:42:58,510 INFO Starting LDAP mumble authenticator
2022-05-11 10:42:58,512 DEBUG Using shared ice secret
2022-05-11 10:42:58,512 INFO Connecting to Ice server (<IP_ADDRESS>:6502)
2022-05-11 10:42:58.513692 LDAPauth.py: error: Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/Ice/__init__.py", line 1548, in doMain
    status = self.run(args)
  File "/app/LDAPauth.py", line 226, in run
    if not self.initializeIceConnection():
  File "/app/LDAPauth.py", line 264, in initializeIceConnection
    adapter = ice.createObjectAdapterWithEndpoints('Callback.Client', 'tcp -h %s' % cfg.ice.host)
  File "/usr/local/lib/python3.10/site-packages/Ice/__init__.py", line 934, in createObjectAdapterWithEndpoints
    adapter = self._impl.createObjectAdapterWithEndpoints(name, endpoints)
Ice.SocketException: Ice.SocketException:
Cannot assign requested address

;Ice configuration
[ice]
host            = 10.10.10.xxx
port            = 6502
slice           = Murmur.ice
secret          = ******************************
watchdog        = 30

version: "3.7"
services:
  murmur:
    environment:
      - MUMBLE_CUSTOM_CONFIG_FILE=/data/murmur.ini
    image: mumblevoip/mumble-server:latest
    volumes:
    - $PWD/conf/murmur.ini:/data/murmur.ini:ro
    - $PWD/murmur.sqlite:/data/murmur.sqlite:rw
    - $PWD/log/murmur.log:/data/murmur.log:rw
    hostname: murmur
    expose:
      - 6502
    ports:
    - 64738:64738
    - 64738:64738/udp
    - 6502:6502/tcp
    - 6502:6502/udp
  murmur-ldap:
    build: ./mumble-scripts/Authenticators/LDAP
    hostname: murmur-ldap
    depends_on:
    - "murmur"
    volumes:
    - $PWD/conf/LDAPauth.ini:/app/LDAPauth.ini:ro
    - $PWD/log/LDAPauth.log:/var/log/murmur/LDAPauth.log:rw
    - $PWD/mumble/src/murmur/Murmur.ice:/app/Murmur.ice:ro

murmur-murmur-1  | Using manually specified config file at /data/murmur.ini
murmur-murmur-1  | All MUMBLE_CONFIG variables will be ignored
murmur-murmur-1  | Running Mumble server as uid=1000 gid=1000
murmur-murmur-1  | "/data" has the following permissions set:
murmur-murmur-1  |   drwxr-xr-x, owner: "mumble" (UID: 1000), group: "mumble" (GID: 1000)
murmur-murmur-1  | Command run to start the service : /usr/bin/mumble-server -fg -ini /data/murmur.ini
murmur-murmur-1  | Starting...
murmur-murmur-1  | <X>2022-05-11 10:42:56.343 SSL: OpenSSL version is 'OpenSSL 1.1.1f  31 Mar 2020'
murmur-murmur-1  | <W>2022-05-11 10:42:56.344 Initializing settings from /data/murmur.ini (basepath /data)
murmur-murmur-1  | <W>2022-05-11 10:42:56.652 MetaParams: TLS cipher preference is "TLS_AES_256_GCM_SHA384:TLS_CHACHA20_POLY1305_SHA256:TLS_AES_128_GCM_SHA256:ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-GCM-SHA256:DHE-RSA-AES256-GCM-SHA384:DHE-RSA-AES128-GCM-SHA256:DHE-RSA-AES256-SHA:DHE-RSA-AES128-SHA:AES256-SHA:AES128-SHA"
murmur-murmur-1  | <W>2022-05-11 10:42:56.690 ServerDB: Opened SQLite database /data/murmur.sqlite
murmur-murmur-1  | <W>2022-05-11 10:42:56.691 ServerDB: Using SQLite's default rollback journal.
murmur-murmur-1  | <W>2022-05-11 10:42:56.693 MurmurIce: Endpoint "tcp -h <IP_ADDRESS> -p 6502 -t 60000" running
murmur-murmur-1  | <W>2022-05-11 10:42:56.693 Murmur 1.4.230 (1.4.230) running on Linux: Ubuntu 20.04.4 LTS [x64]: Booting servers
murmur-murmur-1  | <W>2022-05-11 10:42:56.697 1 => Server listening on <IP_ADDRESS>:64738
murmur-murmur-1  | <W>2022-05-11 10:42:57.072 1 => Registration needs nonempty 'registername', 'registerpassword' and 'registerurl', must have an empty 'password' and allowed pings.
murmur-murmur-1  | <W>2022-05-11 10:43:01.847 Killing running servers
murmur-murmur-1  | <W>2022-05-11 10:43:01.852 1 => Stopped
murmur-murmur-1  | <W>2022-05-11 10:43:01.852 Shutting down
murmur-murmur-1  | <W>2022-05-11 10:43:01.853 MurmurIce: Shutdown complete

In firewalld i Opened Port 6502 tcp and udp. on the Port there is something. I don't know why the Authenticator won't connect and if it's a Docker or Authenticator Issue.
[root@MUMBLE murmur]# telnet 10.10.10.XXX 6502
Trying 10.10.10.XXX...
Connected to 10.10.10.XXX.
Escape character is '^]'.
IceP

murmur-murmur-1  | 2022-05-11 10:42:56.693 MurmurIce: Endpoint "tcp -h <IP_ADDRESS> -p 6502 -t 60000" running

Here i used the Wildcard. The log just shows a random local IP

I've open the ice on <IP_ADDRESS> on my docker compose (+ expose value) on the mumble-server container
I create a new container on same network
I telnet mumble-server 6502 from the other container, result:

root@50efb5a63c69:/# telnet mumble-server 6502
Trying fd00::26...
Trying <IP_ADDRESS>...
Connected to mumble-server.
Escape character is '^]'.
IceP
^C

Ok I start to understand the issue you have, Ice use a open port on mumble-server to register a callback on specific port
So BOTH ice client and ice server need open ports :

6502 for the mumble-server
XXXX (port choose for the callback) on the mumble-ldap.

Easy way : network_mode : "service:mumble-server" on the docker-compose for your service murmur-ldap.
Hard way: expose 6502 on mumble-server and the other port mumble-ldap (and use the name of container instead of fixed IP). But I don't know if you can choose the callback port on your script.

Network mode does it.
I think the Authenticator Configuration needs a better seperation between Host and Client Variables.


how to modify scroll speed  factor

DEFAULT_TOUCH_DRAG_MOVE_RATIO_FWD ，DEFAULT_TOUCH_DRAG_MOVE_RATIO_BCK，DEFAULT_DECELERATE_FACTOR 。 i see this,

Have you tried: https://github.com/EverythingMe/overscroll-decor#effect-behavior-configuration
Specifically, refer to the first numeric argument.

ok thx


Temporarily disable QuestionnaireItemDropDownViewHolderFactoryEspressoTest

QuestionnaireItemDropDownViewHolderFactoryEspressoTest#shouldSetDropDownValueToAutoCompleteTextView
IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1323
Description

Temporarily disable QuestionnaireItemDropDownViewHolderFactoryEspressoTest#shouldSetDropDownValueToAutoCompleteTextView
Improve imports
Use property access syntax
Remove unnecessary explicit type arguments

Alternative(s) considered
NA
Type
Testing
Screenshots (if applicable)
NA
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

Approved! One nit: can you update title to include making Android Studio suggested changes?

thanks - i think the other changes are just cosmetic and less important. so kept the title the same, but added them in the description.


The bucket list should be displayed with a bucket name which has access

@hiteshjoshi can you please assign this to relevant person from BE coders pod

From @Rishabh-Rathod
"Currently S3 Datasource query of list bucket results in all the buckets present irrespective of user having permission for it or not. This issue was to list only buckets which user has permission/access to. This will be useful when we list all buckets to choose from in generate CRUD template."

s3 usage 6m: 148

Stats

Stat
Values

Reach
148

Effort (months)
0.50