Replace time with an atomic counter

I'm not sure it is necessary for the system to guarantee uniqueness and monotonicity for sequence numbers issued by different dispatchers. Generally, any given dispatch tree should only have one dispatcher that is tasked with assigning sequence numbers, and the others should use the *forward methods instead, re-using the sequence numbers. This should probably be added to the documentation.
Lifting that restriction (obviously) obviates the need for atomics. However, I very intentionally chose time over a counter, because it means that, in theory, if the application is terminated and then re-started, old sequence numbers will not be re-used. As you pointed out in #1, we could use Instant for this, though I don't know if its guarantees about monotonicity holds across executions. Does it use something like Linux's CLOCK_MONOTONIC under the hood? Given the relaxed semantics above, it also doesn't need to be shared as you mentioned in #1.

Instant uses CLOCK_MONOTONIC under the hood, but it doesn't expose that information. Instead one has to create a relative point with Instant::now. Whereas std::time exposes a UNIX_EPOCH for SystemTime
It sounds like time suits your needs if you're desiring a monotonic clock that persists program resets. Note that time's precise_time_ns is generally based on time since system start, so it won't persist across system resets
As an aside you should use unwrap_or_else instead of unwrap_or with precise_time_ns so that you aren't querying time when an explicit timestamp exists

Mmm, that's unfortunate about Instant. I know about the no-reboot limitation of precise_time_ns, and it's sad, but there doesn't seem to be any good replacement (if I could get the absolute time out of Instant, that might be it).
Good point about unwrap_or_else. I'd be happy to accept a PR for that.

If the lifetime of the counter exceeds the lifetime of the program, then it'd seem best to store a counter inside the struct & expose a method of at least initializing the struct with a specified counter. Then persistence across executions becomes the responsibility of the user

That's a good point, and arguably a better design than relying on time. I'd be happy to merge a PR that moves to a counter, and adds a startAt argument (or something similar). Note that the counter still doesn't need to be atomic though.

Updated with 2 functions set_sequence & get_sequence. I wasn't sure how to store a struct-bound sequence number that'd be accessible to users throught Dispatcher while existing for DispatchInner
A more implementation-opaque method to global set/get functions would be to move them into Dispatcher. Even if they don't use the &mut self parameter for now

I think I would make the initial sequence an (ideally optional) argument to the constructor for Dispatcher. Adding a mechanism for reading the current sequence number directly is trickier, since the Dispatcher doesn't have access to the DispatcherInner. However, it might be that such a method isn't really necessary, since clients can observe the timestamps that come out at the end to see what the counter is currently at.
I'd also want to get rid of the atomics. They add unnecessary overhead given that synchronized counters between dispatchers does not need to be supported.

@serprex: looks good. Would you mind also adding a doc to new_with_seed, and maybe mention the behavior of the counter in the docs (no correlation between dispatchers, starts from zero unless seed is given, monotonically increasing)?

Took a stab at it; I'm not the best explainer

Close, though "dispatcher increments its counter & uses that as the sequence number" should not be prefixed with "When a timestamp is not supplied". It increments a counter and (s/&/and/) uses that as the sequence number regardless of whether a seed was supplied by the user or not.

No, or_else only executes counter += 1 if ts is None

Ah, I read "when a timestamp is not supplied" as "when a seed is not supplied (to this function)". The behavior you are describing there is already written elsewhere (namely that the system will use either user-supplied times, or the system will keep a counter).

I'm being confused by your discussing an optional parameter when Rust lacks support for function overloading & default parameters & variadic parameters

Let me see if I can write it more clearly.
The way the code is written, there are four possible ways to use the library:

The user created the dispatcher using new_with_seed, and uses .send() to send values (i.e., does not use .forward(). The system will assign each sent item a monotonically increasing timestamp starting at the value provided to new_with_seed.
The user created the dispatcher using new, and uses .send() to send values. In this case, the system will also assign each sent item a monotonically increasing timestamp, except that the timestamps start at 0.
The user created the dispatcher using new_with_seed, but uses .forward(), not .send() to send values. Since forward() items already have timestamps, the dispatcher counter will not be used.
The user created the dispatcher using new, but uses .forward(). Result is the same as above.

Elsewhere in the documentation, it is explained that for any given dispatcher, either .forward() or .send() should be used. Mixing and matching the two is not supported. Specifically, in the create description where I differentiate between serializing and forwarding mode (admittedly, that text should refer explicitly to forward and send).
However, you can choose to use either .forward() or .send() regardless of whether you called new or new_with_seed. The docstring for new_with_seed currently reads:

When a timestamp is not supplied, the dispatcher increments its counter & uses that as the sequence number. A program may maintain monotonic sequence numbers between executions by tracking those received, storing them on exit, & restoring state with new_with_seed.

This, to me, reads as: if you do supply a timestamp when creating a dispatcher (i.e., you use new_with_seed, not new), the dispatcher will not keep a counter. This, in turn, implies that you cannot use .send() with a dispatcher created with new_with_seed, which is obviously not true. Presumably what you meant by the first sentence is that if a timestamp is not supplied with an update (i.e., .send() is used), the a counter will be kept. However, this is not clear from the text as it stands, and is doubly confusing given that it is in the docstring for new_with_seed specifically.
My suggestion is to make the docstrings for both new and new_with_seed very similar. That is, copy the docstring from new to new_with_seed, and change the first sentence to read (new part highlighted):

Creates a new clocked dispatch whose automatically assigned sequence numbers start at a given value.

and then add a paragraph saying something like:

This method is useful for programs that wish to maintain monotonic sequence numbers between multiple executions of the application. Such an application should track received sequence numbers, store the latest one upon exiting, and then use this method to resume the sequence numbers from that point onward upon resuming.

Looks great! Would you mind also adding a simple test case for this (basically just that the first received update has the timestamp of the counter), and then I'll merge?

Thanks for all the work @serprex! We got there in the end :+1:


add more clarity for windows 10 in-place upgrade to windows 11

[Enter feedback here]
Hi team,
according to this doc: https://learn.microsoft.com/en-us/troubleshoot/azure/virtual-machines/in-place-system-upgrade
windows 10 is supported for in-place system upgrade. (cannot feedback on this link)
however, there is no document mentioned if Azure windows 10 can be upgraded to windows 11 directly.
assuming that the windows 10 is supported for in-place upgrade, we can download iso and click install without issue?
there are several things need to clarify:

current Azure marketplace image for windows 11 is Gen 2 only, is it Gen 1 VM supported?
if take Gen 2 Windows 10 VM for example, download windows 11 from - Download Windows 11 (microsoft.com)

setup failed at checking since azure VM does not have TPM

if we take Gen 2 Windows 10 created with trusted launch, which has vTPM, in this scenario, download windows 11 ISO and install successfully. however, VM offer and plan still remains at windows 10. is this recommended?

what is the official best practice to upgrade windows 10 to windows 11?

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

ID: 780935e4-9fdd-d41a-c4ae-69e1cf118653
Version Independent ID: 7d156aed-66b7-e010-0647-6064aa54f58b
Content: Windows in-place upgrade - Azure Virtual Machines
Content Source: articles/virtual-machines/windows-in-place-upgrade.md
Service: virtual-machines
GitHub Login: @cynthn
Microsoft Alias: cynthn

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

@daisygithuba
I'm going to assign this to the document author so they can take a look at it accordingly.
@cynthn
Can you please check and add your comments on this doc update request as applicable.

@daisygithuba As far as the Gen 1 v Gen 2 piece, this artcile: https://learn.microsoft.com/en-us/windows/whats-new/windows-11-requirements#virtual-machine-support says: "In-place upgrade of existing generation 1 VMs to Windows 11 isn't possible."
I'm still working to find a good doc on client OS in-place upgrades.

Hi @cynthn.,

I'm still working to find a good doc on client OS in-place upgrades.

Can we get the good doc about this topic? I'm waiting for it... I believe that this doc will contain the following topics.

Supportability about in-place upgrade from Win10 to Win11

The current doc doesn't mention about the supportability of this path clearly

The supported way to upgrade Win10 to Win 11

Best regards.

Thanks for your contribution to our documentation
We sincerely apologize for the delayed response. Unfortunately, we have been unable to review this issue in a timely manner. However, we are making overall enhancements to our content. We are closing this issue for now as there has been no activity for a while. If you feel that the problem persists, please respond to this issue with additional information.
?
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Logic behind docker-compose-files folder

Description
I'm not sure I get the logic behind the fabric/consensus/docker-compose-files directory being how it is today, and the included Markdown file (which should probably be moved to the Wiki) doesn't really clarify things.
This directory currently has nine YAML files.
I see the need for compose-consensus-4-{batch, classic, sieve}.yml so we're three files down - good. Based on what I read in the Markdown file, compose-consensus-4.yml is actually the, err, true docker-compose.yml and it extends compose-consensus-4-defaults.yml. So:

compose-consensus-4-batch.yml
compose-consensus-4-classic.yml
compose-consensus-4-sieve.yml
compose-consensus-4-defaults.yml
compose-consensus-4.yml

Are accounted for.
Questions

What role does the existing docker-compose.yml (and compose-defaults.yml) serve?
Is there a reason why we're keeping compose-consensus-4-links.yml in a separate file and we don't merge it with compose-consensus-4-defaults.yml?

Expected Behavior
As a developer, I would expect the docker-compose.yml file to do what compose-consensus-4.yml and compose-consensus-4-links.yml currently do.
There's always a chance I'm missing something.

@kchristidis I would ask a more general question - Why do we store any Docker-compose files in this repository at all?
The peer and memberservc networks we create are so uniform and consistent that they could/should all be created by the application of a single script, which would generate and then execute a custom Docker-compose file based on the user's requirements. Creating a Docker-compose network of 39 peers running Sieve consensus with security and profiling should be as simple as executing
dockerComposeNetwork -security -sieve -profile 39

For an existence proof of this approach see the busywork script userModeNetwork
which supports the above (using simple processes rather than containers), plus correctly handles the startup sequencing issues discussed in #1234, and also creates a JSON encoding of the network configuration for use by other tools.

The fabric/consensus/docker-compose-files are leftovers from obc-peer when we needed a way to startup 4 peers for unit testing. corecode started with docker-compose.yml and compose-default.yml. tuand27613 copied from those 2 and created the compose-consensus-4 files.
If we don't see the need for these anymore,  I've got no issues with nuking from the repo.

@rameshthoomu  @bmos299  ^^^


Package request: zfs-replicate

zfs-replicate is currently at 3.0.0 but the most recent version in nixpkgs is 1.2.3.
@alunduil Could you please update the package?

I've just updated zfs-replicate to 3.1.0 and will put together a pull request for this version.  If you're comfortable, you can submit version bumps in the future, and I should be able to sign off on them.

This package is currently on the latest 3.2.13 version


display_and_projection: remove image processing

Basic image manipulations section should go in the introductory part

Made this notebook more or less obsolete now.


Transpile error

this file is not in any tranpilation loop hence few users are seeing bug for => operator

Failed pictures (2): failed-pictures.zip

:postbox: Posted by artipost.io, through Travis build 357457578, for commit 6119803.

Oh, I guess it's because of #698.


Environment Setup: docker_nginx doesn't not start - [emerg] 1#1: host not found in upstream

Hi! I'm trying to startup the projects using the latest version of vscode, dockers over windows 10...
I following the next steps successfully withouts errors but the

Change the sql sa password
Compile all projects successfully
Then I edit my system32/drivers/etc/hosts file and add the <IP_ADDRESS> wallet.local
Run the truscert-windows.ps1 to add the localhost certificate
Run docker-compose build & docker-compose -f docker-compose.yml -f docker-compose.override.yml up -d

Do I forgot any steps?
All was fine but the the next containers don't start:

docker_nginx logs:
/docker-entrypoint.sh: Configuration complete; ready for start up
2020/11/19 02:28:54 [emerg] 1#1: host not found in upstream "accounts-api" in /etc/nginx/nginx.conf:41
nginx: [emerg] host not found in upstream "accounts-api" in /etc/nginx/nginx.conf:41
Obviously I can't browser any https://wallet.local:8081/ website....
I attached the nginx_config.json configuration file
docker_nginx.log
accounts logs:
standard_init_linux.go:211: exec user process caused "no such file or directory"
docker_accounts-api-seeds logs
Build started...
Build succeeded.
Server=sql1;User Id=sa;Password=XXXX;Database=Accounts;
Applying migration '20200821064304_InitialCreate'.
Done.
NOTE: I verified the SQL connections and tables, all works fine.
I supossed that the main issue is the nginx but I don't know how to fix it!
PD: I install and uninstall some times the dockers images and delete all containers and start again, without success.
Thanks

Hey @maxiar,
Thank you reaching us!
I suspect the issue is on accounts-api. It did not find the required /https/localhost.crt file.
https://github.com/ivanpaulovich/clean-architecture-manga/blob/20973ff83d88ef8b39085d1c7f8bf767a0a113e3/accounts-api/src/WebApi/entrypoint.sh#L2
Please check the "shared drivers" in Docker desktop or run docker inspect accounts-api and look for
"Mounts": [
            {
                "Type": "bind",
                "Source": "C:\\repos\\clean-architecture-manga\\.docker\\https",
                "Destination": "/https",
                "Mode": "ro",
                "RW": false,
                "Propagation": "rprivate"
            }
        ]

To narrow your findings try  docker-compose -f docker-compose.yml -f docker-compose.override.yml up -d accounts-api

Hi! I don't have the "shared drivers" in Docker desktop, this option is only enabled for Windows Containers (https://github.com/docker/for-win/issues/7199#issuecomment-643390851).
I'm running in a Linux Containers with WSL2 activated:

I verified the "docker inspect accounts-api" I found this:
    "Mounts": [
        {
            "Type": "bind",
            "Source": **"C:\\Projects\\Personal\\clean-architecture-manga\\.docker\\https",**
            "Destination": "/https",
            "Mode": "ro",
            "RW": false,
            "Propagation": "rprivate"
        }
    ],

And the files are located correctly:

Note that: In the previous post I tell you that only run the "Run the truscert-windows.ps1 to add the localhost certificate", may be I missing something...
How to test or execute this command into accounts-api?
cp /https/localhost.crt /usr/local/share/ca-certificates/localhost.crt


operator project-quay (3.9.0-nightly.20230616)

Add 3.9.0-nightly.20230616 to community operators

@quay-devel please rebase this PR

/merge possible

/merge possible

/merge possible


Use promocmp for fuzz

This has EnvoyEndpoint proto
Please provide a description of this PR:

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
53d2b6280bf7e8718fad63f3bb564d748ccb11c2
link
unknown
/test integ-distroless

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Features/camera logs

-Camera-related logs management: captures errors from the built-in error listeners from zxing-scanner (scanError, scanFailure) as well as from qr http requests and from the fetching/parsing from the storage. Other errors captured by zxing-scanner are intercepted though console.error. Stack-traces are not stored due to length limitations when sending (see below).
-Logs are stored through storage service (which uses Ionic storage --indexedDB).
-Logs are shown in Settings>View Logs>Camera logs
-Last logs are sent from Settings>Send logs through mailTo. Only 1400 characters, which is aprox. the minimum size allowed by local email services.
-Feature-flag conditioned (logs-enabled)
-Also it was added the deactivation of the camera when living barcode-scanner when credentials page is left.

Added tests with Jest. Only those which are directly related to the camera logs feature are taken into account in the test execution and coverage.


Use local version of cbuild during tests

When running integration tests, a version of cbuild is downloaded from an official repo.
Why not build and use a local version in the tests? It seems kind of counter-productive to have work with local changes but still download a pre-build binary as it limits my flexibility and increases the complexity of the test-suite.

Just to be clear cbuild is maintained in a separate repository https://github.com/Open-CMSIS-Pack/cbuild/ and therefore the automated tests refer to a previously released version. Maybe we need some way to override this when running these tests locally.

@soumeh01 So, it's not a restriction on testing the local changes but I would say it has a negative effects on readability and complexity. So if I would statue my use case:
"As a developer, I wan't to easiliy reproduce the outcome of a integration test"
The current tests runs a binary that only exists during the test-execution instead of genereting a command line for cbuildgen+cmake which I can copy to directly.

@jlonnberg Thanks for raising this issue.
I believe there is room for improvement and we should think about the best environment setup we could have without degradation of tests coverage, both locally and when running on CI.
Currently every tool is released separately, for this reason also the binaries of dependent tools are downloaded from their releases instead of building them locally.
We could think about setting up the development environment having cbuild or cmsis-toolbox as the entry point and add the dependent tools as submodules. The integration tests could be moved to the top level repo. Ideally this approach should be kept in sync with the idea of having individual releases as well, for example by using tagged submodules instead of just checking-out the main branches heads. Also the full integration tests should run for every relevant tool.
Ideas, alternatives, best practices recommendations are welcomed!

@brondani I have not embraced the cbuild repo in depth yet but some ideas from my side would be:

The integration tests in devtools run the tests using cbuildgen+cmake which are the active components that are needed for the test-suite.
If cbuild is just a wrapper for the other tools, why not write a testsuit for cbuild that checks that it generates correct commandlines for the underlying tools? This gives a clear separation between the repos but will perhaps trigger compatibility issues.

@jlonnberg can this be closed or are there still issues with it?

@ReinhardKeil From what I know, nothing has been done to change the current behaviour. This is a design-question and I believe that the current test-system can be improved and easier to use. I believe that binaries are still downloaded based on stale versions  of cbuild when testing as well as larger parts of the tests are enclosed in shell scripts forcing the usage of bash on windows instead of opting for a more flexibile launguage, e.g., python.

While the test system might be not easy to use, lets decide if it would be worth investing.  Why should the test system improve? How would be the more flexible way look like? When and why this is needed? What is the effort?

This is now covered in https://github.com/Open-CMSIS-Pack/devtools/issues/1508


Ignore Ruff "F403" (allow star imports)

Strict linting for imports can be really annoying especially when first getting started with a new library. I think it would be beneficial to ignore this rule in the default template.

Per discussion in Slack, playground has this ignored, but this is now the production-oriented template and it's desired to keep these rules to align with Python best practices.


Provide light dom props children

This commit provides a resolution for issue #41, applying a strategy similar to the one used in pull request #56. The commit ensures that the children of a component, such as 'FooComponent', are correctly rendered even without using the Shadow DOM.
An example component which should render now correct in the not shadow-dom:
function FooComponent(props) {
    return (
        <Fragment>

My Heading

            {/* Render the children inside this div */}

{props.children}

        </Fragment>
    );
}

I've played around with this a bunch and this works great :+1:
But I have encountered one issue: when I don't explicitly pass shadow: false as the option the child components are getting rendered double.
Testcase
Components:
const ParentComponent: Preact.FunctionalComponent = (props) => {
  return (

Headline

      { props.children }

  );
}

const ChildComponent: Preact.FunctionalComponent = (props) => {
  return (

Child

  );
}

Markup:
<x-parent>
    <x-child></x-child>
    <x-child></x-child>
</x-parent>

✅ With explicit shadow: false - Works Great
register(ParentComponent, 'x-parent', [], { shadow: false });
register(ChildComponent, 'x-child', [], { shadow: false });

Results in:
<x-parent>

Headline

    <x-child>

Child

    </x-child>
    <x-child>

Child

    </x-child>

</x-parent>

🚫 Without passing any options - Renders children both in a <slot> and as a direct child
register(ChildComponent, 'x-child', []);

Results in:
<x-parent>

Headline

    <slot>
      <x-child>

Child

      </x-child>
      <x-child>

Child

      </x-child>
    </slot>

  <x-child>

Child

  </x-child>
  <x-child>

Child

  </x-child>
</x-parent>

Reason
I think the reason for this is the check for the shadow-Option in Line 202 & Line 206.
Maybe the determination of shadow mode should be consolidated with Line 13 to a local variable and then used throughout:
const isShadow = options ? options.shadow === true : false;

Would love to get this merged! Just ran into this issue. For now, I've added a global style: [slot] { display: none !important; }, but have to explicitly state all slots:
<x-button>
MyButton
</x-button>

Would be great to just have:
<x-button>
MyButton
</x-button>

Any news about that?

It's funny, in practice I've found I never actually use this. I always need at least two slots in my CEs, never just one child. I've gone back to the official build for now.

I was wondering if porting such a popular library like Ant Design would be possible using web-components... Here's an example usage: https://codepen.io/djalmajr/pen/OJdoxqQ
But this "little problem" makes it unfeasible.

Bumped into the same problem with duplicated children components. Any intention to finalize and merge this one?
cc @paull39 @bspaulding


Show initial progressbar when starting

Fixes #63.

Many tests fail because logarithmic scaling doesn't work for the value 0 - any ideas how to address this?

Since it doesn't work for 0 I think the best solution would be to exclude that value with an if statement.
We can simply assume that the log of 0 = 0 in this case.

Any idea what's going on with the coverage? I get 100% coverage if I run py.test locally on my machine...

Everything looks good :) The appveyer build can be ignored for the time being, I'll look at that when I push a new release soon.
Thank you very much for the help!


packages not available

using conda to create with the requirements.txt file gives this error:
Solving environment: unsuccessful attempt using repodata from current_repodata.json, retrying with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed

and lists all lines in the requirements.txt file for packages that it can't find.
doing a cross check, I was able to install numba in conda with this conda install numba
could you provide the channels, or a requirements file in pip format?

@igbeller
Thanks for trying. If you cannot create an environment with conda, you can try the following instructions:
`conda create --name ZMM-TTS  python=3.8
conda activate ZMM-TTS
conda install  blas=1.0=mkl  pytorch==1.12.1 torchvision==0.13.1 torchaudio==0.12.1 cudatoolkit=11.3  pytorch-mutex==1.0=cuda  libgcc-ng=11.2.0=h1234567_1 libgfortran-ng=11.2.0=h00389a5_1 libgfortran5=11.2.0=h1234567_1 numpy-base==1.23.5  -c conda-forge -c pytorch
pip install speechbrain==0.5.14 text2phonemesequence==0.1.4 matplotlib==3.6.2 g2p  pandas==1.5.2 g2p-en==2.1.0 epitran==1.24 praat-parselmouth==0.4.3 pyworld==0.3.2 pycwt==0.3.0a22 python-speech-features==0.6 pstring==1.0.2
pip install transformers==4.30.2 keras==2.13.1 librosa==0.9.2   # mkl==2021.4.0
pip install tensorflow-gpu==2.2.0 unidecode==1.2.0 pypinyin==0.48.0
pip install  tensorboardx==2.6 tensorboard==2.2.2
`
If you are still facing any issues, please feel free to ask.


[Feature]: MoViNet-A6 Model

Is there an existing issue for this?

[X] I have searched the existing issues.

Is your feature request related to a problem? Please describe.
Thanks for this wonderful project!  Code of MoViNet-A0~A5 have already in the project, will you release MoViNet-A6 in the future (The largest model of MoViNet series)?
Describe the solution you'd like.
No response
Describe an alternate solution.
No response
Anything else? (Additional Context)
No response

/assign @jaelgu
any idea

@Hou9612 According to the paper, you can easily ensemble MoviNets A4&A5 to get A6. To get inference results of A6, just average logits by A4&A5.

@Hou9612 According to the paper, you can easily ensemble MoviNets A4&A5 to get A6. To get inference results of A6, just average logits by A4&A5.

Got it! I will read the paper again and try it, thanks very much for your reply.


fix: Add accept option to merged config

Fixes #1340

How would such a test look like? Do we assert on the correct accepted status codes after merging the CLI values with the config file values?

Sure, just something simple like https://github.com/Techassi/lychee/pull/1.

Thanks for fixing the clippy error. 👍

Oh I wanted to include the serde support :/


Output command returns an error

When using the -o flag for a License, it returns a no such file or directory error.
This should be addressed pronto.

Actually a non-issue, used the wrong flag...


fix: Fix update collaboration empty response (box/box-openapi#452)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 92.376%

Files with Coverage Reduction
New Missed Lines
%

box_sdk_gen/internal/base_object.py
2
85.22%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.02%

Covered Lines:
10250

Relevant Lines:
11096

💛  - Coveralls


./kepler/common bind in dev directory does not exist

What happened?
The following bind contains a source url that does not exist:
- type: bind
source: ./kepler/common/var/lib/kepler/data
target: /var/lib/kepler/data
If users were to run this normally an error would be encountered.
What did you expect to happen?
A kepler/common directory should already exist with the same data directory as in kepler
How can we reproduce it (as minimally and precisely as possible)?
Run docker compose up in the manifests/compose/dev directory
Anything else we need to know?
The format of the local models used in kepler seems completely different than the trained models produced by the model server. Thus, I think it would be better if this bind is removed now that models can be transferred locally with filepath in the Model Config.
Kepler image tag

latest

Deployment

[X] Model server
[ ] Estimator
[ ] Online trainer
[ ] Offline trainer
[ ] Profiler

Kepler model server image tag if deployed

Kepler estimator image tag if deployed

Kepler online trainer image tag if deployed

Kepler offline trainer image tag if deployed

Kepler profiler image tag if deployed

Kubernetes version

$ kubectl version
# paste output here

Install tools

Kepler deployment config

For on kubernetes:
$ KEPLER_NAMESPACE=kepler

# provide kepler configmap
$ kubectl get configmap kepler-cfm -n ${KEPLER_NAMESPACE}
# paste output here
# provide kepler model server configmap if Kepler Model Server is deployed
$ kubectl get configmap kepler-model-server-cfm -n ${KEPLER_NAMESPACE}
# paste output here
# provide kepler deployment description
$ kubectl describe deployment kepler-exporter -n ${KEPLER_NAMESPACE}

For standalone:
put your Kepler command argument here

@sthaha


Task 80 and 81, Stats variables.

Hello,
I am using this plugin with stats variables but i get errors in console from infoboardreborn.
Here you can find the errors:
http://pastebin.com/gBdrAVZS
Board doesn't work, please say me how to fix this problem.
Stats work correctly.
Thanks :)

I get the same error when hooking in pvpstats and Stats3, the board just dissapears all together.

Me too with pvpstats.
Please update the plugin!

Please, this would make me 100% on the plugin!


Ship a source-buildable .NET Core 2.1 RC

We will deliver a high-quality RC source-build that moves predictably to RTW.
To do so, we need to:

[x] maintain close sync with prodcon builds during RC (SHA syncing), ensuring offline builds are green.
[x] ensure the behaviour gaps between the Microsoft distributed product are reduced and of minimal impact to consumers of source-build.
[x] hold the line on using SDK 2.1.300 as the toolset to build the RC source-build
[x] opportunistically, we will improve the engineering of source-build by removing patches.

The CLI version should be 2.1.300-preview2-008530 which is what we shipped for preview2.

@dleeapho Are any of the "done criteria" able to be marked as complete?  I'm providing an update today and want to have an up-to-date status on these.

@shawnro this epic is currently in-flight.

Released as tag v2.1.0-rc1 https://github.com/dotnet/source-build/tree/v2.1.0-rc1


Skip Github Actions caching for now

Cache has been failing on Windows recently. Going to just skip it for now. Dependencies shouldn't be too bad.

Funnily it looks like it's actually 10min faster this way :D


怎么连接STM32和Raspberry Pi3

Hi, @doublejouker
The STM32 MCU board on our minibot has built-in USB serial port,
so we just connect Pi3 through USB 2.0.


Can't override default attributes with css-class or mj-class

I can't override a default value with a css-class or mj-class but I can by adding the attributes directly to the mjml tag which doesn't work for me with my use case. I need to use css-class because I want the styles to change according to the width of the screen.
What I want to do specifically is add a border to an overall column, but remove the top or bottom border if a certain width is reached.
You can replicate with this code and the editor.
<mjml>
  <mj-head>
    <mj-title> Responsive table </mj-title>
    <mj-attributes>
      <mj-class name="data-table" width="50%" />
      <mj-style inline="inline">
        .data-table { width: 50%; border-radius: 24px; border: "1px
        solid #000"}
      </mj-style>
      <mj-style inline="inline">
        @media (max-width: 450px) {
        .data-table:first-child { border-bottom: none; }
        .data-table:nth-child(2) { border-top: none; } }
      </mj-style>
    </mj-attributes>
  </mj-head>
  <mj-body>
    <mj-section>
      <mj-column css-class="data-table">
        <mj-text>Key</mj-text>
      </mj-column>
      <mj-column css-class="data-table">
        <mj-text>Value</mj-text>
      </mj-column>
    </mj-section>
  </mj-body>
</mjml>

Classes of either kind should be able to override the default styling

It's not really clear for me what you want to do here.
But both your mj-style are in mj-attribute so it won't be applied.
Not sure about that synthax : border: "1px solid #000" probably want border: 1px solid #000; ?
And you can't inline media query so you should remove that inline attribute on the 2nd mj-style.

Aside from the removal of inline: "inline" is there anything else I can do to ensure that the media query is applied properly? This isn't removing the borders like I'd hoped

 <mj-style >
        @media (max-width: 450px) {
        .data-table:first-child { border-bottom: none; }
        .data-table:nth-child(2) { border-top: none; } }
</mj-style>

Interesting, so !important should be added to any media query ask? That worked btw, much appreciated

You should avoid !important as much as possible. But when you don't have really the choice to then you can use it. This should be explained here https://developer.mozilla.org/en-US/docs/Web/CSS/Specificity


Error on build server with ExcelReader.DataSet.pdb

I am trying to use your tool to create a test suite and we encountered an error on our build server -
"Indexed source information could not be retrieved from \ExcelDataReader.DataSet.pdb. Error: Symbol indexes could not be retrieved. "
After some internal discussion we think it has to do with the DebugType setting since we use an older TFS server(TFS 2012).
Version Information:
ExcelDataReader v3.6.0
ExcelDataReader.DataSet v3.6.0
I believe newtonsoft.Json had a similar issue and was able to fix this. Is there anything that can be done to resolve this on our build server?

Hi, I have this very same issue - did you find a solution to it in the end? ; its all hunky dory on my PC, but I must be missing a step to get the pdb included somehow either by reference or wix component maybe?  I'm also using ancient TFS 2012

Indexed source information could not be retrieved

Did you test these solutions?
https://github.com/JoshClose/CsvHelper/issues/535
https://github.com/fluentassertions/fluentassertions/issues/767

Hi, I see, thanks for those links (especially on a Saturday) very much appreciated.


molecule does not honour --destroy=never and prunes ephemeral data anyway (appeared after #1739)

Issue Type

Bug report

Molecule and Ansible details
ansible --version && molecule --version
ansible 2.7.10
  config file = None
  configured module search path = [u'/home/jenkins/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.5 (default, Jun 20 2019, 20:27:34) [GCC 4.8.5 20150623 (Red Hat 4.8.5-36)]
molecule, version 2.22

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Detail any linters or test runners used:
N/A
Desired Behaviour
When running molecule test --destroy=never do not prune molecule ephemeral data, even when destroy stage is a part of a scenario.
Actual Behaviour
Molecule configuration excerpt:
scenario:
  name: default
  test_sequence:
  - dependency
  - syntax
  - lint
  - create
  - converge
  - idempotence
  - verify
  - destroy

molecule test --destroy=never --all does not honour --destroy=never and prunes ephemeral data when destroy is a part of a scenario in the molecule configuration.
Output excerpt:
Verifier completed successfully.
--> Scenario: 'default'
--> Action: 'destroy'
Skipping, '--destroy=never' requested.
--> Pruning extra files from scenario ephemeral directory

Described behaviour contradicts the purpose of NEVER destroying an environment parameter regardless of what stages were defined in the scenario.

Pruning based on presence of destroy stage in the molecule configuration appeared after https://github.com/ansible/molecule/pull/1739

cc @seandst

Yep: https://github.com/ansible/molecule/blob/575bf4b5a5205562811ff4ca18e35ec94a25f53e/molecule/command/base.py#L145-L146 < destroy --never is not taken into account when deciding to prune based on the destroy step.
I'll put my general thoughts here if anyone's interested in the wall of musings... :)
I don't immediately recall if it's easy to get at the click opts/args from the command-line at this point in the process, but probably some thoughtfulness about the decision to prune is needed. The quickest fix might be to put a prune kwarg onto execute_scenario, since scenario.sequence and the command args both appear to be available when it's called in execute_cmdline_scenarios.
As pruning is needed (or not) per-scenario, the scenario config might be a reasonable place to keep track of whether or not to prune at the end, instead of basing it on the sequence members after the fact.
My memory is fuzzy, but I suspect that a major reason for execute_scenario to have been created was to deal with some crazy unreadable indentation that happens in the try/except in execute_cmdline_scenarios, and also for testability.
Speaking of testability, the failure case reported by this issue is supposed to be covered: https://github.com/ansible/molecule/blob/master/molecule/test/unit/command/test_base.py#L166
But that test is invalid because the mocked execute_scenario masks the prune call.
So yeah, I'll fix up the test to fail like it should, and then fix up the code so it passes again. I'll also take a quick peek at some of the other tests related to this stuff and see what other bad mocks I may have made.


ROX-21383: add clusterid to the listening endpoints table

Description
This PR adds clusterid to the listening_endpoints table. This PR also makes it so that if a listening endpoint arrives in central and it already exists without the clusterid, it is set. Currently the API returns the clusterid, but that is obtained by joining to the process_indicators table, and it is not guaranteed to have it. Now the clusterid is obtained from the listening_endpoints table if it has it, and from the process_indicators table otherwise.
The main purpose of adding the clusterid is to improve SAC for listening_endpoints.
This PR is related to the one for adding namespace https://github.com/stackrox/stackrox/pull/9182
Checklist

[x] Investigated and inspected CI test results
There are two failures, but they are known flakes
[x] Unit test and regression tests added
[x] Inspected the listening_endpoints table after deploying
[x] Inspected the listening_endpoints table after upgrading
- [ ] Evaluated and added CHANGELOG entry if required
- [ ] Determined and documented upgrade steps
- [ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

There are no large user facing changes and there is no impact to the upgrade.
Testing Performed
Inspecting the database after deploying
$ git checkout jv-ROX-21383-add-clusterid-to-the-listening_endpoints-table

$ ./deploy/k8s/deploy-local.sh

central_active=# select * from listening_endpoints ;
-[ RECORD 1 ]------+-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
id                 | 1080d340-3b56-4c76-a530-b735c2c86a47
port               | 8080
protocol           | 1
closetimestamp     |
processindicatorid | 9a234838-862a-5c87-a054-1d740731f47f
closed             | f
deploymentid       | 09148667-b386-455a-b2bc-653035b77113
poduid             | 57ea0d08-e30e-53e2-a985-510aacbbe4ba
clusterid          | 4223f74f-4190-4280-af71-84abe31fb182
serialized         | \x0a243130383064

central_active=# select count(*) from listening_endpoints ;
-[ RECORD 1 ]
count | 69

central_active=# select count(*) from listening_endpoints where clusterid is null ;
-[ RECORD 1 ]
count | 0

Inspecting the database after upgrading
git checkout master

$ ./deploy/k8s/deploy-local.sh

git checkout jv-ROX-21383-add-clusterid-to-the-listening_endpoints-table

$ make tag
4.3.x-478-g360fec1d79

ks edit deployment central-db

central_active=# select * from listening_endpoints ;
-[ RECORD 1 ]------+-----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
--------------------------------------------------------------------------------------------------------------------------------
id                 | 8158fc0b-9f9a-4d67-be9c-01d7972f4746
port               | 10054
protocol           | 1
closetimestamp     |
processindicatorid | 5dbb954d-377b-522f-b4ea-07d536195fc8
closed             | f
deploymentid       | b3820578-3900-4e43-8d21-cf2f5fd5a1d9
poduid             | 11b23052-264e-5c1c-bce4-eda9318f77a5
serialized         | \x0a2438313538666330622d396639612d346436372d626539632d30316437393732663437343610c64e18012a2435646262393534642d333737622d353232662d623465612d303764353336313935666338422462333832303537382d3339
30302d346534332d386432312d63663266356664356

ks edit deployment central

central_active=# select * from listening_endpoints ;
-[ RECORD 1 ]------+------------------------
id                 | 68df7b7f-b3ac-4523-916e-64ece91007cc
port               | 6061
protocol           | 1
closetimestamp     |
processindicatorid | d3b183db-5664-5e61-b511-325038d826f0
closed             | f
deploymentid       | b3820578-3900-4e43-8d21-cf2f5fd5a1d9
poduid             | 11b23052-264e-5c1c-bce4-eda9318f77a5
serialized         | \x0a2436

central_active=# select count(*) from listening_endpoints where clusterid is null;
-[ RECORD 1 ]
count | 0

central_active=# select count(*) from listening_endpoints where not clusterid is null;
-[ RECORD 1 ]
count | 78

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/retest

/retest

/retest


Fixed #25161 -- Encouraged users to install the latest release in docs

Not sure the wording is perfect, but it's a starting point. Feel free to improve.

Thanks @timgraham, excellent suggestions, I've updated the patch.

:+1:

merged in 600fb7c51b22bc0d9138f580bb5e5abcfedc76ee, thanks!


onChange does not fire when using the shortcuts

I've noticed that when using the shortcuts will not trigger the onChange function. The input does get updated with the range from the shortcuts, but the function does not fire.
It works fine if I select a range from the calendar manually.
I'm not sure if I missed something from the docs (I can replicate it with code from the demo page), or if this is a bug.

Hi 👋 @eduardlleshi
Thank you very much for reporting this problem.
I'm really sorry for the delay. I'm new to open source contribution with a few packages to maintain right now. I am making an effort to respond as quickly as possible.
I will fix these issues very quickly.

@eduardlleshi i have already solved the problem.


RE7 and RE8 Head Orientation

both games looks fantastic in VR
but to walk/run ist a bit strange. most VR games have Head Orientation instead of Controller Orientation
Is it possible to add Head Orientation in REFramework Menu?

Fixed in #319


Make CircleCI fail on sphinx warnings

closes #40

Nice 👍

@choldgraf can you trigger a readthedocs build, for some reason the merge didn’t trigger it

Done


Upgrade request (GET ws://)

I'm having several problems building a reverse proxy  for an application that makes the following protocol upgrade request:
GET ws://<IP_ADDRESS>:6565/?reconnectionToken=[redacted] HTTP/1.1
Connection: Upgrade
Upgrade: websocket
Sec-WebSocket-Key: 4l2gyUWXMc6gwhW26k3OAw==

mitm-proxy error:
HTTP protocol error in client request: Invalid request scheme: ws

It is probably not fully RFC compliant or most libraries do not include ws and wss as protocols.
If you can add support for these protocols it could be very useful. (I also accept workaround of all kinds).

After reading the RFC of HTTP protocol, I found my answer. So I'm closing this issue as unnecessary.

First of all can you name the application so we can publicly shame it? :smile:
Anyway, I don't think this is valid at all so I'm not sure this will (or can) be added to mitmproxy.
Here's your workaround: start a second mitmproxy in reverse mode in from of your regular one and correct these nasty packets.
mitmdump --rawtcp --mode reverse:http://localhost:8081 --tcp-hosts '.*' --scripts addon.py

import mitmproxy.tcp;

def tcp_message(flow: mitmproxy.tcp.TCPFlow):
  flow.messages[-1].content = flow.messages[-1].content.replace(b'GET ws://', b'GET http://', 1)

I can't name the application but I have to say that it has caused me quite a few headaches. Thanks for the workaround, I hadn't thought about it.
(for me we can close the issue too).

Feel free to close it then, I'm not a maintainer.


Cannot find dictionary

Hey,
Looking forward to adding your gem to my app, but I'm running into trouble running it on a mac. How do you install dictionaries? Is there a particular format I should be looking for? I'm new to the NLP world so sorry if these are newbie questions.

FFI::Hunspell.dict('en_GB') {|d| d.check? "dog"}
RuntimeError: unable to find the dictionary "en_GB" in any of the directories

Kind Regards,
Jack

I just installed the gem (on a mac) and it's still unclear what I need to do to get the basic english dictionary. If I have to download a separate file and put it somewhere, how would that work in a production environment? Or something like Heroku, where you don't have direct access to the file system?


Create eursc.txt

github_pull_request_explanation.txt
Initial creation

For the record: the file linked above contains this clarification:
the university official website URL, if different from the domain you are submitting
https://www.eursc.eu/ is the domain for all EU schools
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the university
https://www.eursc.eu/en/European-Schools/studies/syllabuses
ICT is offered in all primary cycles (P1-P5) and the first 5 of the secondary cycles(S1-S5)
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.
https://www.eursc.eu/Documents/2017-02-D-20-en-3.pdf page 21/48
/lib/domains/eu/eursc.txt
European schools
Ecoles européennes
Europäische Schulen

@mirena99 I can not add this domain to this repository due to too broad scope of schools covered by this domain. I am not sure that it is the best solution, because many schools in the EU have their own email domains, and some of them are added to this repository already.
Also, the report you are referring to does not shed a light on who and how provide people with an access to the email in this domain. You also may know that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age, and it's not easy to ensure that all the schools in the EU will comply with this term (and even be notified about it), because if we include this domain into the repository, ALL the students and teachers of the schools will be granted with the free license automatically, on the base of possessing an email in this domain.
Please let us know what do you think about it. You can make a public comment on Github, just browse to https://github.com/JetBrains/swot/pull/3627 and leave your comment there. We may need to consider another acceptable solution. It's unlikely that we are able to add such a domain to this repository.

Hello,
I think there might be a slight misunderstanding. This domain represents the 15 schools which bear the name "European School" (see https://www.eursc.eu/en/European-Schools/locations). It does not at all mean that it includes ALL schools in the countries of the European Union.
Regarding the numbers, please refer to the report https://www.eursc.eu/Documents/2016-10-D-2-en-2.pdf:
page 11 - total number of students (26691),
page 15 - number of secondary students, i.e. age 12-18 (13943)
page 19 - number of staff (1401).
Staff and secondary students only get a personal email, so ~15000 is the max number of licences.
Please advise if this is an acceptable number to be covered. I understand completely your constrains and will be looking for another solution if needed.
Thanks a lot,
Mirena

@mirena99 I am sorry for a delay with processing your request. Thank you for the additional details. They have helped us to identify your email domain as similar to school district's ones, from the UK and the US, some of which are included into this repository. Thus, we decided to add the domain which you have suggested as well. Thank you for your explanation and the patience. Pull request is now merged.


CI: Update the environment to ubuntu-20.04

The Ubuntu 16.04 environment of GitHub Actions, which is currently used by our CI workflows, has been retired last month. This PR updates our CI workflows to use the Ubuntu 20.04 environment.

@bors-servo r+

:pushpin: Commit db1019f has been approved by jdm

:hourglass: Testing commit db1019faf3efdcfb8e7749a59f337174d95c1d72 with merge 2053318e819bb0693fd9a5480ae876e04c37934f...


error: invalid command 'develop'

hello，I have the same problem, how did you resolve this? Thanks

how did you resolve this?

我也遇到了同样的问题，请问您是怎么解决的@chexqi

将setup.py中的
from distutils.core import setup
修改为
from setuptools import setup

将setup.py中的
from distutils.core import setup
修改为
from setuptools import setup


Add webCompat feature

Task/Issue URL: https://app.asana.com/0/1202552961248957/1205083099335524/f
Description
This PR adds the webCompat content scope script feature.
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3358   👈

This comment was auto-generated by Graphite.

@joshliebe I was trying to check this PR (by testing first) however, the step  the images next to each story load always fails as it always show the placeholder


Add Steven to top level owners

I am proposing we add @stevenctl to the top level owners to help reduce the bus factor a bit. Largely this just means approving go.mod changes.
I specifically assigned Lin to avoid same company approval, although I don't think company diversity here is that important since the role is pretty minor

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-istiodremote-k8s-tests_istio
622a9f59eca6e29cfe1a8cfcd1d32485262b6c2c
link
true
/test integ-istiodremote-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Ops-files for controlling deployment name and kubo version.

Signed-off-by: Leah Hanson<EMAIL_ADDRESS>What this PR does / why we need it:
Adds ops-files for controlling deployment name and kubo version.
When deploying multiple clusters on the same bosh director, it is necessary to give them different names and sometimes to specify different kubo versions for each one. These ops-files make it easier to change these settings because you can put the values in vars-files rather than changing the base manifest manually every time.
How can this PR be verified?
Use these opsfiles, see that you can create a deployment named something other than cfcr or with an arbitrary version.
Is there any change in kubo-deployment?
No
Is there any change in kubo-ci?
No
Does this affect upgrade, or is there any migration required?
No
Release note:

NONE

Hi @benmoss
We have ops-file to rename deployment:
https://github.com/cloudfoundry-incubator/kubo-deployment/blob/master/manifests/ops-files/rename.yml
and two ops-files to use custom kubo-release versions:
https://github.com/cloudfoundry-incubator/kubo-deployment/blob/master/manifests/ops-files/misc/dev.yml - use latest.
https://github.com/cloudfoundry-incubator/kubo-deployment/blob/master/manifests/ops-files/kubo-local-release.yml - use local repo.
In general, we don't support using custom final versions and dev versions should be covered by two ops-files.
We have also a separate ops-file in our CI for testing purposes:
https://github.com/cloudfoundry-incubator/kubo-ci/blob/master/manifests/ops-files/set-kubo-release-version.yml
Feel free to reuse it for your tests.


Remove Dropdown (v1) entirely

We no longer need this since Dropdown V2 is a thing. Once we remove it, we can remap the import for components/Dropdown to use DropdownV2!

completed in https://github.com/helpscout/hsds-react/tree/v3

@plbabin You're a hero 🏆


Use CWD for relative paths

When using a relative path, it would make more sense to use the current working directory rather than the Micro-Manager installation folder. Futhermore, expending the user (~) would be nice.

Thanks for the PR @impact27!


Update metadata with correct homepage, license etc.

This helps in providing better information on websites like pypi. For example compare https://pypi.org/project/pyodbc/ with https://pypi.org/project/soda-sql/

@vijaykiran Looks like https://pypi.org/project/soda-sql/ borrows README file content. Having updated docs all over the place, maybe there is better content to borrow to update in pypi? Let me know if I can help with the content there!

I think the best option is to sync README of the repo. It’ll be automatically published on pypi.


运行和打包都失败

`PS C:\Users\admin\Desktop\vue-cesium-webpack-starter-master> yarn serve
yarn run v1.22.15
$ vue-cli-service serve
INFO  Starting development server...
98% after emitting CopyPlugin
ERROR  Failed to compile with 1 error                                                                                1:43:18 ├F10: PM┤
error  in ./node_modules/cesium/Source/ThirdParty/zip.js
Module parse failed: Unexpected token (6365:57)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|
|       function getWorker(options = {}) {

          return new Worker(new URL(workerData.scripts[0], import.meta.url), options);

|       }
|`

Hi guys, thanks for your feedback.

What system platform?
What is the Node version?
Can you describe your problem in detail?

@liuzhenghe30265 最近会花时间看一看


Comparing anything with Null results in Null instead of True or False

From MS Access 2016, in the VBA Direct window (Ctrl+G), typing 'print 1<>2' will result in printing "True", but 'print 1<>Null' or 'print 1=Null'  will both result in Null, which used to result in True/False in previous versions. This means having tests like this in if-statements results in the script taking the wrong route. A very damaging bug if you're used to comparing text boxes that also can be empty (Null). Of course the Vartype test can solve this, but that means you have to go through all the code and redo all testing.
print 1<>2
True
print 1<>Null
Null
print 1=Null
Null
print Null=Null
Null
This bug is new in Microsoft Access 2016 MSO (16.0.9330.2073) 32-bits.

@SiemEik Thanks for pointing this out. I'll try to track down an Access writer to investigate. Can you provide a link to documentation that references this?

I am fairly sure it has always been like this. Comparing with null yields null. I have checked that the print behaviour you describe above is consistent with 2013 (and I believe earlier versions as well, but don't have a VM handy)

Print IsNull(Null)

or if you prefer
Print Nz(1 <> Null, True)
Print Nz(1 <> Null, False)

also input boxes have empty strings not Null see:
https://msdn.microsoft.com/en-us/vba/language-reference-vba/articles/inputbox-function
Print InputBox("Enter Nothing") = ""

and form textbox controls in 2013 return null when empty but the VBA throws an error dialogue:

Thanks for your contribution! This content has migrated to https://github.com/MicrosoftDocs/VBA-Docs. Please go to the new repo, and if you still see the issue occuring there, please open a brand new issue. Closing this issue now. Regards, lindalu


replacing clone by deepCopy does not fix clone implementation

In version 0.1.2. the fix in lines 14-21 were removed. This fix seems to be necessary, even with owl-deepcopy. Without the fix I get the following error:
TypeError: Cannot read property 'numAsyncPres' of undefined
  at model.module.exports._lazySetupHooks (/home/donnut/dev/js/node_modules/mongoose/node_modules/hooks/hooks.js:149:49)
  at model.module.exports.pre (/home/donnut/dev/js/node_modules/mongoose/node_modules/hooks/hooks.js:113:10)
  at model.Document._doQueue (/home/donnut/dev/js/node_modules/mongoose/lib/document.js:1256:21)
  at model._registerHooks (/home/donnut/dev/js/node_modules/mongoose/lib/document.js:1231:8)
  at model.registerHooks [as _registerHooks] (/home/donnut/dev/js/node_modules/mongoose/lib/model.js:704:37)
  at model.Document (/home/donnut/dev/js/node_modules/mongoose/lib/document.js:69:8)
  at model.Model (/home/donnut/dev/js/node_modules/mongoose/lib/model.js:34:12)
  at new model (/home/donnut/dev/js/node_modules/mongoose/lib/model.js:1668:11)

when instantiating a new mongoose.model. With the fix in place, all works fine.

Looks like it is fixed


Will anything break if Hashable got Eq as a superclass?

Hashable and Eq have to be compatible for unordered-containers to work properly, so having Eq as superclass of Hashable seems natural to me.
My question: is there any non-contrived use case for Hashable which will be ruled out by having Eq superclass? Bonus points if that use case is in realm of hashing-based data structures, because "This class (Hashable) exists for the benefit of hashing-based data structures."
EDIT: I'm asking here first, for smaller audience. Will later ask on libraries list & other fora.

It's hard to imagine one applicable to hashing-based data structures. For one that isn't, maybe something like computable reals?

computable reals

You mean that they cannot be Eqd? But can they be hashed either?

Depends on your requirements. Obviously if your numbers are all the same
out to N places then you'll get a lot of collisions.
On Mon, Aug 30, 2021, 5:05 AM Oleg Grenrus @.***> wrote:

computable reals
You mean that they cannot be Eqd? But can they be hashed either?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/haskell-unordered-containers/unordered-containers/issues/318#issuecomment-908174209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOOF7IAIEEADFMVZPSRCELT7NCXJANCNFSM5DBNILHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I think you're asking a very good question, and your suggestion may well be
a good idea.
On Mon, Aug 30, 2021, 4:57 AM Oleg Grenrus @.***> wrote:

Hashable and Eq have to be compatible for unordered-containers to work
properly, so having Eq as superclass of Hashable seems natural to me.
My question: is there any non-contrived use case for Hashable which will
be ruled out by having Eq superclass? Bonus points if that use case is in
realm of hashing-based data structures, because "This class (Hashable)
exists for the benefit of hashing-based data structures."
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/haskell-unordered-containers/unordered-containers/issues/318,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOOF7M7H5EUWB377OGYHI3T7NBZNANCNFSM5DBNILHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

But what that hash would be useful for? Specifically what algorithm (polymorphic in Hashable type) would be applicable to this (because if it's not polymorhic, you don't need to use Hashable).

It was pure speculation. I know of none.
On Mon, Aug 30, 2021, 5:13 AM Oleg Grenrus @.***> wrote:

But what that hash would be useful for? Specifically what algorithm
(polymorphic in Hashable type) would be applicable to this (because if
it's not polymorhic, you don't need to use Hashable).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/haskell-unordered-containers/unordered-containers/issues/318#issuecomment-908179962,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOOF7OTNMRSMKWZS3TXVALT7NDSPANCNFSM5DBNILHA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

https://hackage.haskell.org/package/hashable-<IP_ADDRESS> is on Hackage.
If the bounds are simply relaxed to allow hashable-<IP_ADDRESS> everything compiles fine and all tests pass.
However another alternative or additional option is to make a new release with hashable >=<IP_ADDRESS> lower bound and cleaned up type-signatures.

I was imprecise, wigh GHC-7:
% cabal build -w ghc-7.10.3 --constraint='hashable >=1.4'
Resolving dependencies...
Build profile: -w ghc-7.10.3 -O1
In order, the following will be built (use -v for more details):
 - unordered-containers-<IP_ADDRESS> (lib) (first run)
Configuring library for unordered-containers-<IP_ADDRESS>..
Preprocessing library for unordered-containers-<IP_ADDRESS>..
Building library for unordered-containers-<IP_ADDRESS>..
[1 of 9] Compiling Data.HashMap.Internal.List ( Data/HashMap/Internal/List.hs, /code/other-haskell/unordered-containers/dist-newstyle/build/x86_64-linux/ghc-7.10.3/unordered-containers-<IP_ADDRESS>/build/Data/HashMap/Internal/List.o )
[2 of 9] Compiling Data.HashMap.Internal.Unsafe ( Data/HashMap/Internal/Unsafe.hs, /code/other-haskell/unordered-containers/dist-newstyle/build/x86_64-linux/ghc-7.10.3/unordered-containers-<IP_ADDRESS>/build/Data/HashMap/Internal/Unsafe.o )
[3 of 9] Compiling Data.HashMap.Internal.Array ( Data/HashMap/Internal/Array.hs, /code/other-haskell/unordered-containers/dist-newstyle/build/x86_64-linux/ghc-7.10.3/unordered-containers-<IP_ADDRESS>/build/Data/HashMap/Internal/Array.o )
[4 of 9] Compiling Data.HashMap.Internal ( Data/HashMap/Internal.hs, /code/other-haskell/unordered-containers/dist-newstyle/build/x86_64-linux/ghc-7.10.3/unordered-containers-<IP_ADDRESS>/build/Data/HashMap/Internal.o )

Data/HashMap/Internal.hs:536:10-48:
    Could not deduce (transformers-<IP_ADDRESS>:Data.Functor.Classes.Eq1
                        (HashMap k))

as Eq1 instance is given only for base-4.9.
I'd suggest just dropping GHC-7 support. It's simplest.

I'd suggest just dropping GHC-7 support. It's simplest option.

Dropping support for GHC < 8 sounds good to me. If it turns out that some users really need continued support for GHC-7, we can consider a more complex solution.

I'm missing something. What's the purpose of this change? And how does an Eq superclass for Hashable relate to Eq1 for HashMap k?

analogously Eq1 is a superclass of Hashable1. HashMap is Hashable1, but not always (GHC <8) Eq1.
The proposed change is just restrict the unordered-containers versions so Eq1 (HashMap k) always exists.

Alternatives are:

adding a dependency on transformers (and transormers-compat) to have Eq1 always.
removing Hashable1 instances (breaking change)

I've started working on removing the compat code for GHC < 8. So far, this looks like a very nice cleanup. We can even get rid of an entire module: https://github.com/haskell-unordered-containers/unordered-containers/blob/master/Data/HashMap/Internal/Unsafe.hs

I'm not using 7.x anymore.

And getting rid of hideously dangerous code is always good.

Support for GHC < 8 was removed in #323. #324 will bump our bound on hashable.

Is there an ETA for a release?

@phadej I'll upload a release once https://github.com/haskell-unordered-containers/unordered-containers/pull/327 is merged.
So, probably Monday or maybe Tuesday.


website-demo breaks when xviz vertices contain #/accessors/x

Hi,
I am using this tool to convert and visualize nutonomy scenes, after converting the dataset into xviz format and try to load it in the website demo, I am getting a blank page. The error comes from filterVertices in normalizeXVIZPrimitive and it breaks when the input is not a number array.

Not sure if it is a bug with visualizer or with xviz writer library.
Thanks in advance!

So it seems that the actual problem is with unknown object class missing styles. Checking the colormap in xviz conversion script solved my problem here.


header: race: core_listener.go

This is no bueno because:

a screenshot is not searchable
the screenshot does not show which test triggers this


Allow FacetGrids use in heatmaps

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[x] (If applicable) Documentation has been added / updated (for bug fixes / features).
[ ] (If applicable) Tests have been added.
[x] CHANGELOG.rst has been updated (with summary of main changes).

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added.

What kind of change does this PR introduce?

fg.heatmap can now accept plot_kw = {"row":"dim1", "col":"dim2"} like other functions in Figanos!

Does this PR introduce a breaking change?
No
Other information:

Are "row" and "col" the only arguments we can pass to the Seaborn facetgrids in plot_kw?
I tried using height and aspect in plot_kw and it didn't work (https://seaborn.pydata.org/generated/seaborn.FacetGrid.html).

With how I programmed it, yes. I may have misunderstood the general strucutre in Figanos, right now plot_kw is used both in sns.FacetGrid and sns.heatmap, so some options don't work in one function or the other. I will check how other functions handle this.

Usually plot_kw allows you to access the function argument of the xarray plot which also allows the xarray facetgrid arguments.
However, it doesn't always work well and this is why I put some warnings with fig_kw and facetgrids that not all options are available. I think if you add a warning saying only col and row are accepted for the seaborn faccetgrid options in plot_kw it will work great.

Here is my hacky solution, just look at allowed keywords for each function
        kws_hm = set(inspect.signature(sns.heatmap).parameters)
        kws_fg = set(inspect.signature(sns.FacetGrid).parameters)
        plot_kw_keys = list(plot_kw.keys())
        plot_kw_fg = {k: v for k, v in plot_kw.items() if k in kws_fg.intersection(plot_kw_keys)}
        plot_kw_hm = {k: v for k, v in plot_kw.items() if k in kws_hm.intersection(plot_kw_keys)}
# then use plot_kw_fg for sns.FacetGrid, and plot_kw_hm for sns.heatmap
``

```python
# height is only sns.FacetGrid, cmap is only sns.heatmap
fg.heatmap(da, plot_kw = {"col": "time", "height":6, "cmap":"Spectral_r"})

Oops, I realized I had not pushed a final commit (one line of code) in my github branch @Zeitsperre anything I could do that doesn't involve simply a new PR? I guess not?


Troubles

Hello. There was such a problem: when receiving LuckPerms data through Vault, the following error occurs (screen attachments), and such a problem is only with your plugin, the rest of the rights systems transmit data without errors. What can be wrong?

Error: https://ibb.co/KVYHzBJ

Help me, please!

This is not directly LP related.
The issue is that a plugin tries to check for permissions on the main server thread which should be avoided since it can cause lags or other performance issues.
LP intentionally returns a RuntimeException with the following message (As seen in the image):
The operation to load user data for '<UUID>' was canceled by LuckPerms. This is NOT a bug.
The lookup request was made on the main server thread. It is not safe to execute a request to load data for offline players from the database in this context.
If you are a plugin author, please consider making your request asynchronously.
Alternativelym, server admins can disable this catch by setting 'vault-unsafe-lookups' to true in the LP config, but should consider the consequenzes (lag) before doing so.

So in order to "fix" this, you have two options:

Let the developer of the plugin (In this case it's either CommandNPC or your SkyBlock plugin) that made that unsafe check know about this issue to hope he fixes it or
Set this option here to true:
https://github.com/lucko/LuckPerms/blob/ffe6c24b53e75af2aa94def451c183795026fd36/bukkit/src/main/resources/config.yml#L506-L515


Enriching the guidance for APIs needing asynchronous support for long-running requests

Learn Build status updates of commit 12ca64f:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

docs/best-practices/api-implementation.md
:bulb:Suggestion

Details

docs/best-practices/api-implementation.md

Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'architectures' is not a valid Microsoft alias.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@EdPrice-MSFT
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Learn Build status updates of commit 7d98e39:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

docs/best-practices/api-implementation.md
:bulb:Suggestion

Details

docs/best-practices/api-implementation.md

Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'architectures' is not a valid Microsoft alias.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit a0ff90f:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

docs/best-practices/api-implementation.md
:bulb:Suggestion

Details

docs/best-practices/api-implementation.md

Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'architectures' is not a valid Microsoft alias.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 74fe5dd:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

docs/best-practices/api-implementation.md
:bulb:Suggestion

Details

docs/best-practices/api-implementation.md

Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'architectures' is not a valid Microsoft alias.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 84c09c9:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

docs/best-practices/api-implementation.md
:bulb:Suggestion

Details

docs/best-practices/api-implementation.md

Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'architectures' is not a valid Microsoft alias.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#Sign-Off
"architectures" is a valid alias.

Learn Build status updates of commit 69015bb:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

docs/best-practices/api-implementation.md
:bulb:Suggestion

Details

docs/best-practices/api-implementation.md

Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'architectures' is not a valid Microsoft alias.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Lost all memos after updating docker image

Describe the bug
I followed the instructions in the official documentation and after pulling and starting the new image, none of my previous notes are available to read. I did make a back up of the db as the instructions say. Do I just replace the current memos_prod.db with my backup?
Steps to reproduce

have months worth of notes

docker stop memos && docker rm memos

cp -r ~/.memos/memos_prod.db ~/.memos/memos_prod.db.bak

docker pull ghcr.io/usememos/memos:latest

docker run -d
--init
--name memos
--publish 5230:5230
--volume ~/.memos/:/var/opt/memos
ghcr.io/usememos/memos:latest

and then the notes are gone

Screenshots or additional context
No response

I am sorry about your lost, that happened to me once before as well. I initially thought it was because of I did not mount the volume.

The same. I can login, but "Failed to fetch memo list".

Same happened to my instance.

I can access back my old notes by rolling back to docker tag 0.15.2.

That worked for me as well. Thanks for the suggestion!

I can access back my old notes by rolling back to docker tag 0.15.2.

That's right. I can access when rolling back to 0.15.0, but I wonder how we can upgrade.

I can access back my old notes by rolling back to docker tag 0.15.2.

That's right. I can access when rolling back to 0.15.0, but I wonder how we can upgrade.

I find the log shows ERROR   cmd/memos.go:57 failed to migrate db ... migrate error: -- webhook and I manage to switch to 0.18 by removing table webhook in db.


2D Regression start point

I have a dataset and corresponding data flag for a Wafer. The structure is as follows like :
Data_set sample
[[NaN NaN NaN NaN .....3456 .8765 .56454 .....NaN NaN ]
     [NaN NaN NaN .....765 .3456 .8765 .56454 ..... NaN NaN]
-----
     [NaN NaN NaN NaN... .3456 .8765 .56454 -----NaN NaN]]

and the Data_flag is as follows :
  [[0 0 0 0 ....1 1 1.... 0 0]
    [0 0 0 ....1 1 1 1 ....0 0]
----
    [0 0 0 0 ...1 1 1.. 0 0]]

So the in the DataFlag the NaN is replaced by 0 and the other position is replaced by 1.
I want to select a few positions from the Data_set based on the value of the Data_flag (where the value is 1) and then I want to predict the other position value based on data_flag.
I want to solve using this framework.
I'm confused about which solution is applicable for me form https://gpy.readthedocs.io/en/deploy/_modules/GPy/examples/regression.html#multioutput_gp_with_derivative_observations
I also tried [here]
https://github.com/stocon/pycon2016
I'm rocky and do a lot of search for the perfect start point of such a problem. please suggest start point
Thank you

Sorry, your problem isn't entirely clear. Would you be able to post a code snippet and some sample data so we can be more helpful?


Create a tag

Glad somebody has this image out there. Mind creating a v1.0.0 tag so it's easy to know as a consumer I'm always pulling the same image?

Hi Marshmellow,
I could, but actually I've some concerns about it.
If I'd tag the Image, this would freeze the state of the nginx. So if you and others are still using the image in 2 years it will work, but there is a high chance of being exploitable.
I try to allways use latest stuff with my Images. So for example:
You have a running and working marvambass/nginx-registry-proxy you can tag it yourself locally.
Two months later you pull a new version of my latest image and test if it works. If so you use the new one - if not you open a new ticket here, and i try to fix it.
You have to keep in mind, this images comes with an included reverse proxy with htaccess and ssl - its intended for direct internet connections. In my opinion you should never use a old version of a online service (just because it works) better fix the applications to get it run with the newer version.
unfortunatly there are no unit testing frameworks for docker out there, which would ensure a special functionality
if you use my latest image, you should be safe, if allways try to keep everything working and updated.

if you need tags, fork this project


⚠ ClamAV has degraded performance

In c7d2be5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: ClamAV performance has improved in 247f8ae.


Commenting out image pin from helm

Signed-off-by: jonahjon<EMAIL_ADDRESS>Issue #, if available:
*Description of changes:
This comments out the image pin from the helm requires.yaml from a previous PR. This might some more testing, as it's messed up in the staging release.

/cherrypick release-0.11


Allow to define the locale for date-formats, renewed

Renewed PR #65  after the change in the main-repo to react. Same functionality: allow to define the locale in the date-Options

several broken issues now in the clock panel in v7, will test this change while debugging

I accidentially commited the fix for #60 into this PR. Sorry.


#Bug menu for non-standard notification sound are disappear.

Expected behavior
Like in version 5.600, in sound menu there are 2 tabs: melody, music (in Huawei smartphone) - it's standard phone menu.
Actual behavior
In newer verions 5.714-5.716 there are non-native sound menu - there are only basic notification sounds and NO music and other sounds.
Steps to reproduce

in versions 5.600 go to settings-> mailbox settings ->notifications->melody - there are 2 tabs: "melody" and "music"
in versions 5.716 in settings "new mail ringtone"

there are no any tabs, there are only one list of sounds.

Environment
K-9 Mail version: 5.714-5.716
Android version: 7.0, EMUI 5.0.3
Account type (IMAP, POP3, WebDAV/Exchange):
Please take some time to retrieve logs and attach them here:

In 5.600 we were using the system's notification sound picker which apparently is heavily modified by Huawei. This system component was deprecated and we're using the suggested replacement now. For most Android versions the old and the new component look exactly the same.
We have no plans to change this.

So, how can i use other sounds?

In my Android 7.0 i was able to chose custom sound with old component, but cannot do this with your new component. It's is same Android, same EMUI, same smartphone. So i think - it's bug, because it was working in 5.6, but now it doesn't work.


Ho to connect the inverter to serial 1?

I am using the MasterPower inverter Omega 5000 (MP-5K-24) at home, and I would love to connect a photon to it in order to expose the data logs but I am not quite sure how to connect the serial port of the photon to the inverter.
Can you help me with this? Some pictures might help as well.
Thank you

Your inverter should have come with a cable that exposes the serial with a DB9 connector. The cable should look like the following:
https://www.amazon.com/Cisco-72-3383-01-Console-Cable-RJ45-to-DB9/dp/B000GL3MOY
To connect to the photon you would require a RS232 to TTL converter, like the one here.


Missing Class: Google Http MediaFileUpload while uploading Image(s) manually

Recently after updating to Version 3.2.2 we've been unable to Upload new pictures to our website through the normal Media uploader. We've been unable to reproduce this on the dozen or so other websites we're using which have yet to update yet.
The Server is reporting the following error:
[Wed Jun 08 14:11:19.027388 2022] [proxy_fcgi:error] [pid 12724:tid<PHONE_NUMBER>91392] [client <IP_ADDRESS>:44512] AH01071: Got error 'PHP message: PHP Fatal error:  Uncaught Error: Class 'Google\\Google_Http_MediaFileUpload' not found in /var/www/vhosts/WEBSITE/httpdocs/wp-content/plugins/wp-stateless/lib/classes/class-gs-client.php:234
Stack trace:
#0 /var/www/vhosts/WEBSITE/httpdocs/wp-content/plugins/wp-stateless/lib/classes/class-utility.php(352): wpCloud\\StatelessMedia\\GS_Client->add_media()
#1 /var/www/vhosts/WEBSITE/httpdocs/wp-includes/class-wp-hook.php(307): wpCloud\\StatelessMedia\\Utility::add_media()
#2 /var/www/vhosts/WEBSITE/httpdocs/wp-includes/plugin.php(189): WP_Hook->apply_filters()
#3 /var/www/vhosts/WEBSITE/httpdocs/wp-includes/post.php(6597): apply_filters()
#4 /var/www/vhosts/WEBSITE/httpdocs/wp-admin/includes/image.php(351): wp_update_attachment_metadata()
#5 /var/www/vhosts/WEBSITE/httpdocs/wp-admin/includes/image.php(161): wp_create_image_subsizes()
#6 /var/www/vhosts/WEBSITE/httpdocs/wp-admin/includes/ajax-actions.php(2469): wp_update_image_su...', referer: https://WEBSITE/wp-admin/media-new.php

To confirm that this wasn't a bad install as we have uninstalled and re-installed from WordPress' Plugin Store/Shop.
All other Websites we are using are on 3.1.1

Fixed.
Will be released in version 3.2.3

Released. Closing the ticket.
Please let us know if you have any further questions


Flutter app restarts after it is minimised [Only for iPad, Working for other Devices]

Steps to reproduce
Requirement: Test these steps only on an iPad
Step 1: Open the app and go to any screen
Step 2: Minimize the app with any of the screens
Step 3: Open the app from the minimized state by clicking on the app icon or from the recent app.
Expected results
App is restarted and started from the beginning splash screen.
Actual results
It should open from the same state when we minimize the app.
Code sample
Not needed
Screenshots or Video
Flutter Wonderous app:
iPhone [Working]: Video Link
iPad [Not Working]: Video Link
Logs
Not needed
Flutter Doctor output
Not needed

When an app gets put in the background in iPadOS, it gets rapidly resized and screenshotted at different sizes, so that if placed in multitasking view, it looks correct right away. It's possible the dynamic layout/Navigation for different screen sizes is being reset.

Same issue here

@moffatman Might be possible when we are rotating the device but what about when we are in stable condition like portrait mode and resume from the background? because I cross-checked with other iOS natively built apps in iOS which are resuming from where we left off while we moved to the background.
You can also play with any iOS native app on iPad which works similarly to the iPhone.

Thanks for the report @pinkal-simform
I tried couple of sample apps (default counter and navigation) using latest master version and running on ipad simulator (since I don't have physical ipad) and I observed that the state is preserved upon opening the app as shown below:
https://github.com/flutter/flutter/assets/67046386/d012be9f-3f73-41c8-9ede-b35fc2efabf3
Can you check using same sample apps on the device to see if you still get same behavior or not ? Or is it something specific to wonderous app you are using ?

@darshankawar Please check in release mode, sometime it is working fine in debug mode.

@darshankawar I tried Flutter new project, here is the result in a real iPad.
iPad video: link

So it's not the problem I think above; because the new project is really simple and doesn't have display-size-based-navigation

Thanks for the update. Currently I don't have physical ipad device to replicate / verify this further, but based on the report and video attached, I'll keep the issue open for team's tracking.

iPad 12 Pro with iPad OS 17.5.1
Fast App Termination was on. When I turn off, then app state was maintained. Can you share me, what is the default state when developer settings is off in any iPhone or iPad?
Flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.19.6, on macOS 14.4.1 23E224 darwin-arm64, locale
    en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.3)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.2)
[✓] VS Code (version 1.88.0)
[✓] Connected device (2 available)
    ! Error: Browsing on the local area network for iPhone. Ensure the device is
      unlocked and attached with a cable or associated with the same local area
      network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code
      -27)
    ! Error: Browsing on the local area network for SF-IPAD-005. Ensure the
      device is unlocked and attached with a cable or associated with the same
      local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code
      -27)
[✓] Network resources

• No issues found!

By default Fast App Termination is not on. It is a setting used by developers when they want to test state restoration. You can find out more about state restoration in https://api.flutter.dev/flutter/services/RestorationManager-class.html.
Given that this was due Fast App Termination being on, I'll be closing this as working as intended.
Thank you


FB started 400ing /me/posts API requests for some users

started about 24h ago, around the same time as #393. maybe same root cause: timing implies that the FB v2.0 API apocalypse may be to blame, and/or that we don't have the right permissions for those users.
example log:
Fetching https://graph.facebook.com/v2.2/me/posts?offset=0&limit=50&access_token=...... kwargs {'headers': {'If-None-Match': u'"71aab0b83a454f8018c8c6e3dd07f13419985fee"'}}
Error 400, response body: Bad Request
HTTP Error 400: Bad Request
Traceback (most recent call last):
File "/base/data/home/apps/s~brid-gy/3.383971133541294147/tasks.py", line 109, in post
source_updates.update(self.poll(source))
File "/base/data/home/apps/s~brid-gy/3.383971133541294147/tasks.py", line 161, in poll
cache=cache)
File "/base/data/home/apps/s~brid-gy/3.383971133541294147/facebook.py", line 104, in get_activities_response
resp = self.as_source.get_activities_response(group_id=SELF, **kwargs)
File "/base/data/home/apps/s~brid-gy/3.383971133541294147/activitystreams/facebook.py", line 227, in get_activities_response
resp = self.urlopen(url, headers=headers, parse_response=False)
File "/base/data/home/apps/s~brid-gy/3.383971133541294147/activitystreams/facebook.py", line 1204, in urlopen
timeout=appengine_config.HTTP_TIMEOUT)
...
HTTPError: HTTP Error 400: Bad Request

follow #393 for @kylewm's and@diplix's great work so far. also thanks to @armingrewe for first reporting this!

For some reason didn't spot this earlier, but it looks like there's a lot more impacted by the FB API change:
I'm also using the Jetpack Share plugin on my photoblog. That used to show how many times an entry was liked on FB. Those numbers have now completely disappeared for all entries. For Twitter and Pinterest the numbers still show (G+ never showed), but FB is gone.
No idea if this helps you guys, but I thought you might find it interesting.

thanks for the info! definitely interesting. apocalypse indeed. :/

this is getting acute. :(

woot, FB granted the new permissions!

That seems to have fixed it. I just "re-linked" my FB account and it's starting to find the links. Quite a lot coming through now, I'll be busy approving this evening.
Kind of related, Jetpack have released a new version today as well, that's fixed the like count.

currently disabling ("pausing") FB accounts hit by this. we try to send them an FB notification, like usual, but that API call is getting 403ed with {"error":{"message":"(#200) Cannot send notifications to a user who has not installed the app","type":"OAuthException","code":200}}. sigh.

reverted that change. all affected accounts should be disabled and notified (or at least attempted).


[Bug]: Fake timers not working on custom environment

Version
29.5.0
Steps to reproduce

Clone repo https://github.com/ferenckv/jest-fake-timers-test
run npm install
run npm run test

Expected behavior
Example test should pass.
Actual behavior
Example test fail.
Additional context
No response
Environment
System:
    OS: Linux 5.15 Ubuntu 18.04.6 LTS (Bionic Beaver)
    CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
  Binaries:
    Node: 16.10.0 - ~/.nvm/versions/node/v16.10.0/bin/node
    npm: 8.15.0 - ~/.nvm/versions/node/v16.10.0/bin/npm
  npmPackages:
    jest: ^29.5.0 => 29.5.0

Looking at your reproduction, I think all works as expected. You create global.dater in your custom environment. Any environment is a sandbox and fake timers cannot access / know anything outside of it. Hence there is no way to figure out that your custom API is based on global.Date object.
In other words, global.Date which you use to create global.dater and global.Date in the sandboxed environment are two separate things. Only the sandboxed one is touched by fake timers. For me it seems that all works as expected.


Use available threads when nb threads is zero

Resolves #358
/cc @kermitt2

Coverage increased (+0.006%) to 36.72% when pulling 81d85012b4ed3d52b70994d7ac8c52803896c297 on de-code:use-available-threads-when-nb-threads-is-zero into fad45a5f3e134dfb37190e424004b4f37d3aa679 on kermitt2:master.


Provide ability for clients to add HTTP headers

Requirement - what kind of business use case are you trying to solve?
We would like to provide service owners the ability to submit spans from their service running in their development environment over to the production jaeger-collectors, bypassing the jaeger-agent.
The reason for bypassing the jaeger-agent is to make the development experience of submitting Jaeger spans as seamless as possible by removing the need for service owners to clone the jaeger repository and run a local agent. It should "just work".
Problem - what in Jaeger blocks you from solving the requirement?
When services, such as jaeger-collector, are deployed to production, they are assigned ephemeral IP addresses and ports when listening for HTTP requests.
In order for caller services (whether calling from development or production) to reach specific endpoints of callee services, a side-car process is used to route the outbound request to the correct IP and port which is achieved by providing the:

caller service name
target service name
target HTTP endpoint

Currently, jaeger-client-go does not provide a capability for clients to flexibly add headers to the outbound HTTP request.
The impact of not having this capability is that users will need to clone the jaeger repository and run the jaeger-agent locally in order for local spans to be submitted to the jaeger-collector.
Proposal - what do you suggest to solve the problem or improve the existing situation?

The proposal is to add the following attribute to the ReporterConfig allowing clients to configure their own HTTP headers:
// Header instructs the reporter to add these headers to the http request when reporting spans
Header http.Header `yaml:"headers"`

Pass the header to the HTTPTransport in ReporterConfig.newTransport():
	case rc.CollectorEndpoint != "":
		httpOptions := []transport.HTTPOption{transport.HTTPBatchSize(1), transport.HTTPHeader(rc.Header)}
		if rc.User != "" && rc.Password != "" {
			httpOptions = append(httpOptions, transport.HTTPBasicAuth(rc.User, rc.Password))
		}
		return transport.NewHTTPTransport(rc.CollectorEndpoint, httpOptions...), nil

Then in HTTPTransport.send(spans []*j.Span), append the custom headers to the HTTP request:
	req.Header.Set("Content-Type", "application/x-thrift")
	// Use Add instead of Set to avoid overwriting existing headers
	for k := range c.header {
		req.Header.Add(k, c.header.Get(k))
	}

Any open questions to address
Is it the right approach to Add headers rather than Set?

sgtm
Perhaps the config option should be a map[string]string rather than http.Header, which allows setting it in the config file (you had `yaml:"headers" struct tag), and generally has less ambiguity about multi-value headers.
Re Add vs. Set, since we're fully in control of creating the HTTP request, they are almost equivalent, although using Set would allow overriding other headers, like the Content-Type just above that. I would go with Set.

Thanks @yurishkuro! Will do as suggested.


Spanish translation

Hi, I've been working with the Spanish translation. Now it's 100% complete and I've also corrected several string that were wrong.
Can you update the web to use the new translations?
Thanks!
Ignacio

Thank you for this, @ignaciolep ! It's much appreciated. We'll pull in the latest translations as we approach the next release candidate. I'll leave this open until we've done that.


[Build/CI] Extending the set of AMD tests with Regression, Basic Correctness, Distributed, Engine, Llava Tests

Enabling the above mentioned tests.

@simon-mo Will do. Thank you!


feat: create a thread and post

This PR adds an option to create threads and posts.
I'm not sure how we'll handle the permission check, but for now, I've implemented the check in the controller. Although lack of permissions results in dd('Unauthenticated') 😅  At some point we will have to change this.
I have split the code into different controllers and I'm planning to do so also with DiscussionController (but later).

@michalsn , I think it is better to add this button(start ...) in themes\default\master.php so that it is present on all pages.
In addition, we can add a shortcut key(e.g. shift+N+D) to jump .
see here .

You might be right. However, I'm not entirely sure it should be always available - e.g. a Members page.