[gardening] Use consistent naming of .rst files in docs/ ("CamelCase.rst")

Fix some cases where "Camel Case.rst" is used instead of "CamelCase.rst".
File counts:

Files in repo: 10 242
Files in repo with spaces in file name: 10
Files in repo with .rst suffix: 74
Files in repo with .rst suffix and spaces in file name: 10

In addition to the consistency argument: spaces in file names create unnecessary gotchas when it comes to scripting - see below.
Before this commit:
$ find . -name "*.rst" | xargs ls -dl > /dev/null
ls: cannot access ./docs/Dependency: No such file or directory
ls: cannot access Analysis.rst: No such file or directory
ls: cannot access ./docs/proposals/ObjC: No such file or directory
ls: cannot access Interoperation.rst: No such file or directory
ls: cannot access ./docs/proposals/C: No such file or directory
ls: cannot access Pointer: No such file or directory
ls: cannot access Argument: No such file or directory
ls: cannot access Interop.rst: No such file or directory
ls: cannot access ./docs/proposals/archive/Memory: No such file or directory
ls: cannot access and: No such file or directory
ls: cannot access Concurrency: No such file or directory
ls: cannot access Model.rst: No such file or directory
ls: cannot access ./docs/proposals/C: No such file or directory
ls: cannot access Pointer: No such file or directory
ls: cannot access Interop: No such file or directory
ls: cannot access Language: No such file or directory
ls: cannot access Model.rst: No such file or directory
ls: cannot access ./docs/proposals/rejected/Bridging: No such file or directory
ls: cannot access Container: No such file or directory
ls: cannot access Protocols: No such file or directory
ls: cannot access to: No such file or directory
ls: cannot access Class: No such file or directory
ls: cannot access Clusters.rst: No such file or directory
ls: cannot access ./docs/archive/Namespace: No such file or directory
ls: cannot access Level: No such file or directory
ls: cannot access Vars: No such file or directory
ls: cannot access and: No such file or directory
ls: cannot access Top: No such file or directory
ls: cannot access Level: No such file or directory
ls: cannot access Code.rst: No such file or directory
ls: cannot access ./docs/archive/Objective-C: No such file or directory
ls: cannot access Interoperability.rst: No such file or directory
ls: cannot access ./docs/Pattern: No such file or directory
ls: cannot access Matching.rst: No such file or directory
ls: cannot access ./docs/Failable: No such file or directory
ls: cannot access Initializers.rst: No such file or directory
$

After this commit:
$ find . -name "*.rst" | xargs ls -dl > /dev/null
$

@jrose-apple Any objections?

The main one is that some of these files have internal links, and I'm kind of surprised you didn't see errors when trying to build the docs after making this change.
We've also linked directly to some of these files on GitHub, so we might be breaking some of those links.

@jrose-apple I tried searching for internal links/references to those ten files but I couldn't find any. Do you know where to look for such references? I tried with the following git grep commands:
$ git grep "Dependency Analysis"
docs/DependencyAnalysis.rst:Dependency Analysis
$ git grep "Failable Initializers"
$ git grep "Pattern Matching"
docs/PatternMatching.rst:Pattern Matching
docs/contents.rst:   Pattern Matching
include/swift/SIL/PatternMatch.h://===--- PatternMatch.h - SIL Pattern Matching Infrastructure ---*- C++ -*-===//
$ git grep "Level Vars and Top Level Code"
$ git grep "Objective-C Interoperability"
docs/ObjectInitialization.rst:Objective-C Interoperability
docs/archive/Objective-CInteroperability.rst:Objective-C Interoperability
docs/proposals/Initialization.rst:Objective-C Interoperability
docs/proposals/rejected/Constructors.rst:Objective-C Interoperability
docs/proposals/valref.rst:Objective-C Interoperability
$ git grep "C Pointer Argument Interop"
$ git grep "C Pointer Interop Language Model"
$ git grep "ObjC Interoperation"
$ git grep "Memory and Concurrency Model"
docs/proposals/archive/MemoryAndConcurrencyModel.rst:Swift Memory and Concurrency Model
$ git grep "Container Protocols to Class Clusters"
docs/proposals/rejected/BridgingContainerProtocolsToClassClusters.rst:Bridging Container Protocols to Class Clusters
$ git grep "Level Vars and Top Level Code"

The changes passes utils/build-script -RT, but perhaps that is not the relevant test?

@jrose-apple Turns out there was a reference to Pattern Matching.rst in docs/contents.rst. That reference has now been updated to reflect the new naming :-)

@jrose-apple Fixed a reference to Dependency Analysis.rst too.
Turns out that I was building without sphinx-build installed, so the doc building part was skipped by the build script :blush: Sorry about that.
I've now verified the changes with a proper sphinx-build setup. And sphinx-build runs without errors (make in docs/).
I've now squashed my commits and all references should be correct now :-)

Okay, then it just depends on what we think about breaking links to these documents (none of which get linked to that often). @gribozavr?

I don't mind breaking those links, exactly for the reason you mention.

All right, then I guess consistency should win. Thanks, @practicalswift!


Problems specifying arguments to Docker wrapper for reconstruction

Hello--thanks in advance for any help to what should be a simple issue.
We're trying to run a reconstruction pipeline on qsiprep preprocessed data. We had no trouble running the preprocessing pipeline. For reconstruction, I tried copying virtually the same code as the docs:

Here, there are a couple problems: first off, not all the arguments are interpreted. Second, it says output-resolution is required (which I'm fine with inputting) but if it is required for reconstruction, that should be part of the example code in the docs. When copy-and-pasting the command I wrote above, here's what happens:

So, I reran a few times and realized that the wrapper stops inputting arguments after --recon-spec (for whatever reason). So, I tried running a command with --recon-spec as the last argument. It took in all the arguments but I wound up with this error:

Ok...that's strange because in the docs, bids_dir is one of the example arguments.
So, I tried specifying the positional arguments the way I did when running preprocessing:

The program actually started and it rans BIDS validation. But then quickly returned this error:

One lingering question I have is why --participant_label does not have to be specified for the reconstruction pipeline. Or does it? In the example code in the docs, it is not specified. (I tried running it with and without this argument and got the same result).
If someone could shed some light on why we're having trouble running, we would really appreciate it. If we missed something in the docs or are making a simple error, I sincerely apologize. Happy to provide more info.
Thanks so much!

I have also noticed the same errors in the docs - they seem to be slightly outdated in places. Indeed, it seems like --bids_dir, --output_dir, and --analysis_level can only be inferred from position.
Here is the command I have had success with:
qsiprep-docker
mohammad/testing/BIDS
mohammad/testing/reconOutput
participant
-i pennbbl/qsiprep
--output-resolution 2
--fs-license-file /mohammad/testing/license.txt
-w /mohammad/testing/intermediateWork
I would wager that the error you're getting is either because 1) you didn't specify the analysis_level or 2) you didn't specify a work directory with the -w flag. Try including those and see if you get the error
It may help at times to refer to fmriprep's documentation and usage syntax, as most of qsiprep seems to try to follow it as closely as possible: https://fmriprep.org/en/stable/
Note that I am also a complete novice so no guarantees that the above command is the be-all and end-all correct one. Hope this helped!

@mohammadrashid0917 thank you so much, brother! Specifying the positional arguments as you suggested and adding a working directory solved things. I would strongly recommend the devs update the example reconstruction code in the docs.
Thanks again, Mohammad, for taking the time to help a stranger!


[Pipelines] Internal Server Error

This request produces an internal server error:
curl -X POST -H "Content-Type: application/json" -H "Cache-Control: no-cache" -H "Postman-Token: 4add53f9-81bd-585a-94cf-2fd03be15e7c" -d '[
{
  "method": "POST",
  "endpoint": "http://api.freme-project.eu/current/e-entity/freme-ner",
  "parameters": {
     "language": "en",
     "dataset": "dbpedia"
  },
  "headers": {
    "content-type": "text/html",
    "accept": "text/turtle"
  },
  "body": "
Welcome to Leipzig!
"
}
]' "http://api-dev.freme-project.eu/current/pipelining/chain"

Please find out what is wrong.

The problem is, that the endpoint  http://api.freme-project.eu/current/e-entity/freme-ner does not exist. But this error message should be forwarded to the user.

@jnehring This is not a problem of the pipelines service, but of error handling in live version. Please try this pipeline (calls /e-entity/freme-ner at api-dev):
curl -X POST -H "Content-Type: application/json" -H "Cache-Control: no-cache" -H "Postman-Token: 4add53f9-81bd-585a-94cf-2fd03be15e7c" -d '[
{
  "method": "POST",
  "endpoint": "http://api-dev.freme-project.eu/current/e-entity/freme-ner",
  "parameters": {
     "language": "en",
     "dataset": "dbpedia"
  },
  "headers": {
    "content-type": "text/html",
    "accept": "text/turtle"
  },
  "body": "
Welcome to Leipzig!
"
}
]' "http://api-dev.freme-project.eu/current/pipelining/chain"

I think the pipeline should return which API endpoint has caused the failure. In a pipeline with multiple steps it is often not easy to find out which step causes the problem.

The output should look like this
{

  "exception": "eu.freme.common.exception.PipelineFailedException",
  "path": "/pipelining/chain",
  "message": "The pipeline has failed in step 1, request to URL /e-entity/freme-ner/documents",
  "nested-exception": {
    "exception": "eu.freme.common.exception.InternalServerErrorException",
    "path": "/pipelining/chain",
    "message": "eu.freme.common.persistence.model.SerializedRequest.unParametrize(Ljava/util/Map;)V",
    "error": "Internal Server Error",
    "status": 500
    "timestamp":<PHONE_NUMBER>994
  }
  "error": "Internal Server Error",       //forwarded from nested exception
  "status": 500,      //forwarded from nested exception
  "timestamp":<PHONE_NUMBER>994
}

@jnehring should be implemented, can we close this?

Works fine, thanks.


Both Hildir and Haldor can no longer be spawned in

I remember during Mistlands update this was possible, and had a map with a custom merchant area.
They are now gone after Mistlands update, as well as when I attempt to place them back in, it doesn't properly stay.  I assume the game devs have some how muddled this up from the Ashlands update for some reason.

I see now that there must be new requirements for these NPCs to spawn on ground specifically.  I had in a building before so I guess the floor interfered with them being able to spawn.

Was misunderstanding of new mechanics.  Listed what I had thought was the culprit above.


Add support for test engine jqwik

This pull request adds support for the property-based testing engine jqwik. jqwik is a junit platform test engine. Lifecycle Hooks in Jqwik are still experimental. I will ask Johannes Link to have a look at this PR. Maybe he has some additions.

[x] The module should use as few dependencies as possible,

The module has jqwik as single compile time dependency, apart from the testconainers project.

[x] Regarding libraries, either:

they should be compileOnly if they are likely to already be on the classpath for users' tests (e.g. client libraries or drivers)
they can be implementation (and thus transitive dependencies) if they are very unlikely to conflict with users' dependencies.

[x] If client libraries are used to test or use the module, these MUST be legal for Testcontainers developers and Testcontainers users to download and use.

Jqwik uses the Eclipse Public License - v 2.0

[ ] Every module MUST have a dedicated documentation page containing:

[x] A high level overview
[x] A usage example
[x] If appropriate, basic API documentation or further usage guidelines
[x] Dependency information
[ ] Acknowledgements, if appropriate

I borrowed some code from Jupiter extension. Should I acknowledge this?

[x] Consider that many users will not read the documentation pages - even if the first person to add it to a project does, people reading/updating the code in the future may not. Try and avoid the need for critical knowledge that is only present in documentation.

Annotations are similar to the ones of the Jupiter extension

I talked to Johannes. I am going to add the AroundPropertyHook, so that containers are restarted after each try.

I added support for restart after each try

The tests are failing because of a 503. Tests are passing @kiview @bsideup @rnorth

Sorry for not commenting sooner - as you can see we're quite underwater with the number of PRs at the moment, and we're focused on the urgent issue of Docker Hub rate limiting coming up.
We saw https://github.com/jlink/jqwik-testcontainers/issues/1 and we think that's a good approach for everybody. Kotest did similarly.

@rnorth I pushed the code to https://github.com/jlink/jqwik-testcontainers


Invalid test module name

During our automated builds of the newest Swift snapshots for the Sandbox, we run into the following issue while incorporating BlueSocket:
swift-build: error: the module at Tests/Socket has an invalid name ('Socket'): the name of a test module has no 'Tests' suffix
fix: rename the module at 'Tests/Socket' to have a 'Tests' suffix

This began with the 7-29 snapshot a week ago. Sounds like a quick fix?

Hey Jim- you need to use his master-next branch. Since 7/29 has broken Tests, Bill hasn't upgraded yet.

Thanks for the heads up, I'll change our automation.

I'll hopefully be upgrading either over the weekend or Tuesday when I get back to work. The master-next branch is 7/29 compatible with the exception of tests on Linux.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1357 ms

Resolved: Bofrost.at performance has improved in 34c7217 after 7 minutes.


Support postcss-apply

@dantman Fixes #4. CC @MoOx in case cssnext needs updated.

If this is a patch and is delivered as a patch (following semver spec), cssnext will get the patch automatically.

I assumed so but just looping you in 🙏

Working in my build so far.

Thanks! Could it be tagged to propagate to cssnext?

Hi @jonathantneal will this get tagged soon ? I'm trying to use @apply in my project, but this issue prevents me from doing so

+1
"Expected pseudo-class.." when using the documented @apply function with code similar to:
:root {
	--foo: {
		display: block
	};
}


DRILL-3786: Query with window function fails with IllegalFormatConver…

…sionException

+1. Looks good.


peter jacksons king kong unexpected error

Describe the bug
peter jacksons king kong has unexpected error when booting nvidia remix
Remix 0.2
rtx 3070 ti
latest GPU drivers
other
Attach logs!
NvRemixBridge.log
d3d9.log
SettingsApplication_d3d9.log
KingKong8_d3d9.log
CheckApplication_d3d9.log
there are 3 ways of opening the game so included all 3 logs
To Reproduce
Steps to reproduce the behavior:
click on exe
Expected behavior
game should run
Screenshots
If applicable, add screenshots to help explain your problem.

Hello! Will you please test this on the latest release and let us know if the issue persists? Thank you!

Thank you for reaching out. We're going to close this ticket due to an extended period with no response. If the issue persists for you on the latest build, please open a new ticket.


Add default.nix

This is probably fine.
Collecting digitalmarketplace-utils==28.0.2 from<EMAIL_ADDRESS>(from -r requirements.txt (line 9))
  Cloning https://github.com/alphagov/digitalmarketplace-utils.git (to 28.0.2) to /private/var/folders/fg/vl0n7j2d0d9fcn__37bq6g5m000132/T/pip-build-96jvu1k6/digitalmarketplace-utils
fatal: unable to access 'https://github.com/alphagov/digitalmarketplace-utils.git/': error setting certificate verify locations:
  CAfile: /etc/ssl/cert.pem
  CApath: none
Command "git clone -q https://github.com/alphagov/digitalmarketplace-utils.git /private/var/folders/fg/vl0n7j2d0d9fcn__37bq6g5m000132/T/pip-build-96jvu1k6/digitalmarketplace-utils" failed with error code 128 in None
make: *** [Makefile:32: requirements-dev] Error 1

:(

God damn what's up with your mac?


NEED Some help about getting started your ARIMA API!thx(*￣︶￣)

Hi, @signaflo !
Sorry to bother you. I have to say you've done so wonderful work that it helped a lot on ARIMA. As you know , there are so few materials about Arima in Java, no matter the realization of the model. Your work has benefited me a lot.
BUT, I still get some problems and I sincerely ask for your help. The question is --- as my ability is not so good ,I could find the main function about the use of the Arima model to predict (I mean the  specific process ), so can you tell me how to use your API to run an example?
My second question is, is there function considering checking the time series's stability, which is the foudation of  the use of Arima.
I will appreciate your reply if you can spare some time to help me.
Good Day!

Hi @JayVae, and thank you.
Could you take a look at this page and let me know what further questions you have?

@signaflo SO so so much thanks for your reply.
I have read the page on wiki , and I've benefited a lot.But I still have some questions:

First, if we want to use ARMA, we need to check the stationarity test, or we need stationarity test to detemine the difference parameter d ,so can you give me some advice how to do this work, is there any open source code to do so?
Second, on how to determine  the parameters of p,q of ARIMA, I think we can write a loop of different parameter set and use the BIC or AIC to determine which is best. Do you have any advice?
Third, in your example on wiki page, the order is (0, 1, 1, 0, 1, 1) , does this mean that the data model is either non-seasonal ARIMA or seasonal ARIMA?

I will appreciate your reply if you can spare some time to help me.
Good Day!


Expand large type signatures

When you're working with a typed language, such as TypeScript, and you create a large enough type when you hover over an implementation you will see truncated types marked by .... I think it'd be a welcome addition to be able to expand these truncated types to allow viewing the full type signature.

Being investigate upstream with https://github.com/microsoft/TypeScript/pull/31384 (which is about errors but would also apply to hovers)


[ENG-3943] Revert dialog z-index and change file-action behavior

Ticket: ENG-3943
Feature flag: n/a

Purpose

Revert z-index of dialog backgrounds to prevent bad behaviors associated with dropdowns in modal windows

We bumped the z-index up due to some issues with mobile view for Files Page Phase II in this PR here: https://github.com/CenterForOpenScience/ember-osf-web/pull/1529 to address the problem found here: https://www.notion.so/cos/Files-Page-Phase-II-Bugs-Found-2daf0384e75f43c18bef48c60e4b58be#02af2801c0f243b99b4c8be9ab0aef28

Prevent issue where opening the "Resource Type" dropdown using the 🔽 down caret would immediately close the dropdown options

Summary of Changes

Revert z-index for dialog background to 900
Close dropdown when selecting a file/folder action to prevent them from showing up simultaneously and obscuring each other

Screenshot(s)
Prevents dropdown content from being hidden behind blurred background as seen here:

Should also avoid views like this:

Side Effects

QA Notes

The sharing and embed options for files/folders shows up on top of the existing folder/file action dropdown menu in mobile view (behavior unchanged), but opening the delete, move/copy, and rename modals should automatically close the file/folder action dropdown menu

I especially like that you added all the dropdown.closes here.

Yea, it felt much less like a problem with the modal or dropdown components, but moreso that we just shouldn't be having two modal style windows up simultaneously


🏫 Simplify affiliation IDs for JATS

This modifies affiliation ids that appear in JATS to be simpler and uniform, by default aff-1, aff-2, etc.
It also removes the attempted author/affiliation deduplication (which previously didn't work as intended anyway).

For authors, they simply shouldn't be deduplicated - you want to know all the different authors on each subarticle. Since we removed author ids from JATS (#662) there is no longer any JATS validation worry.
Regarding affiliations, we do not deduplicate these either: the implication of this is that identical affiliations in different articles/subarticles will appear separately in each. However, they will also be given a different id, so JATS will be happy. I think it would be nicer if affiliations were compared across articles and deduplicated, but the current behaviour matches citations (identical citations appear in each article). Good enough for now.

Really, what we need to do is rely on the JATS library to build a JATS-like unist tree, then we can apply all these id changes as post-processing unist transforms, rather than finessing the inputs prior to JATS serialization...

Nice, looking forward to improvements in the Jats library. :)


cmd/baseserver: incorporate baseserver version + Makefile

From a discussion offline in which a teammate wasn't sure
if their PR was working, I suspect it was because perhaps
we were using a stale version of baseserver.
Now we can investigate what version their binary was built
against since we can bake that git commit hash in, and also
added a Makefile to install as well as cross compile binaries
for different architectures.
Note: The recommended way of building/installing baseserver
is make install or make as that will bake in all the version
information and help with future diagnosis.

Exhibit

$ make install 1&>/dev/null 2&>/dev/null;baseserver version
0.0.1:a9df9aeef4

/cc @mappum @ebuchman

This also brings up the question of how to handle the difference in versioning a lib and its various binaries.
So far our standard is to keep versioning information in version pkg. Should we have some standard for keeping the versions of binaries in there too? How do we distinguish binary versions from lib version?
@zramsay want to weigh in too ?

https://github.com/tendermint/coding/issues/16

I like the idea a lot, but would make some changes to work with all binaries.  Easier to code than write English...
Please look at #230 and let me know what you think.

The cross-compilation already happens in dist directory, built in docker containers.
I took the parts of the git hash in the version and the linker flags and put that in #230
Closing this PR in favor of that, but thanks for the idea.


Shapes.AddTextbox ignores the anchor object

It seems that you removed the optional anchor parameter from the methods Shapes.AddTextbox, .AddShape, .AddCanvas, ...
In Word 2010 you could add a range parameter to specify the range of the anchor. It no longer works in Word 2016. It seems that you removed it from the documentation as well.
Is there any alternative solution? The AddTextbox method would be quite useless if you could no longer specify the anchor.
I also found an "unanswered" question in the MSDN forums referring to this: https://social.msdn.microsoft.com/Forums/office/en-US/f175055e-7b40-4bcb-8409-d4e0910022f4/documentshapesaddtextbox-ignores-anchor-word-2013?forum=worddev

@makzr So sorry for the delayed answer. Have you tried the Shape.Anchor property? I think that's what you're looking for. If not, please reopen and let me know. Thanks, linda


Cursor handle isn't under cursor in text field

Open Gallery, go into Expansion Panels, tap into Trip name, then tap into the Trip name text field.
Notice how the cursor is at the right of the string, but the cursor handle is at the left of the string.
(The expectation is that the cursor handle is under the cursor.)

cc @mpcomplete @dragostis

Might be that I haven't used the input widget the right way.

Seems like most of the input bugs are reproducible by simply creating this:
new Input(
  hintText: 'asda',
  labelText: 'item.name',
  value: new InputValue(
    text: 'item.value'
  ),
  onSubmitted: (InputValue val) {
    setState(() {
      // item.value = val.text;
    });
  }
)

This is a duplicate of #5987.  Re-creating the InputValue each time you build the Input widget doesn't do what you want.  You need to use the input value you get in the onChanged callback,

@abarth Partially. This fails even without a value.


Fix bug https://github.com/amdegroot/ssd.pytorch/issues/234

Modify model loading and saving path, set 'viz' as global variable, fix bug amdegroot#234 for Visdom window display, remove redundant parameter in 'criterion', logging more info, fix PyTorch 0.4.0+ compatibility.

Unfortunatelly, i get the next error with the trained model derived from this modifications:
python eval.py --trained_model=checkpoints/COCO.pth
/home/joselito92/Projects/thesis/ssd/punrong/ssd.pytorch/ssd.py:34: UserWarning: volatile was removed and now has no effect. Use `with torch.no_grad():` instead.
  self.priors = Variable(self.priorbox.forward(), volatile=True)
Traceback (most recent call last):
  File "eval.py", line 425, in <module>
    net.load_state_dict(torch.load(args.trained_model))
  File "/home/joselito92/Projects/thesis/ssd/punrong/lib/python3.7/site-packages/torch/nn/modules/module.py", line 777, in load_state_dict
    self.__class__.__name__, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for SSD:
        size mismatch for conf.0.weight: copying a param with shape torch.Size([804, 512, 3, 3]) from checkpoint, the shape in current model is torch.Size([20, 512, 3, 3]).
        size mismatch for conf.0.bias: copying a param with shape torch.Size([804]) from checkpoint, the shape in current model is torch.Size([20]).
        size mismatch for conf.1.weight: copying a param with shape torch.Size([1206, 1024, 3, 3]) from checkpoint, the shape in current model is torch.Size([30, 1024, 3, 3]).
        size mismatch for conf.1.bias: copying a param with shape torch.Size([1206]) from checkpoint, the shape in current model is torch.Size([30]).
        size mismatch for conf.2.weight: copying a param with shape torch.Size([1206, 512, 3, 3]) from checkpoint, the shape in current model is torch.Size([30, 512, 3, 3]).
        size mismatch for conf.2.bias: copying a param with shape torch.Size([1206]) from checkpoint, the shape in current model is torch.Size([30]).
        size mismatch for conf.3.weight: copying a param with shape torch.Size([1206, 256, 3, 3]) from checkpoint, the shape in current model is torch.Size([30, 256, 3, 3]).
        size mismatch for conf.3.bias: copying a param with shape torch.Size([1206]) from checkpoint, the shape in current model is torch.Size([30]).
        size mismatch for conf.4.weight: copying a param with shape torch.Size([804, 256, 3, 3]) from checkpoint, the shape in current model is torch.Size([20, 256, 3, 3]).
        size mismatch for conf.4.bias: copying a param with shape torch.Size([804]) from checkpoint, the shape in current model is torch.Size([20]).
        size mismatch for conf.5.weight: copying a param with shape torch.Size([804, 256, 3, 3]) from checkpoint, the shape in current model is torch.Size([20, 256, 3, 3]).
        size mismatch for conf.5.bias: copying a param with shape torch.Size([804]) from checkpoint, the shape in current model is torch.Size([20]).

You can see the PR files' changes.


fix(takeLast): fix takeLast behavior to emit correct order

closes #1407
This PR tries to fix behavior of takeLast by changing ring buffer indexing routines. Maybe I miss something, but so far it looks work correctly.
/cc @staltz for visibility.

Found one failure cases, marking as blocked for now.

Nice! Thanks for the fix

Updated PR to use marble diagram for test cases.

Merged with 73eb658.


[nrf noup]: Revert "[nrf fromtree] cmake: use the warnings_as_errors ...

flag for cpp files"
Some nRF Connect SDK samples are generating warnings for C++ code. nRF Connect SDK CI treats warnings as errors for code, but this was previous not active for C++ source code.
Therfore revert the upstream warning-as-errors flag for C++ code. Revert this commit when nRF Connect SDK C++ source code is clear from warnings.
This reverts commit f3e3cedba999b1c6364e97f6660490d297d45ea4.

@tejlmand Revert should have no nrf noup tag, please update

@tejlmand Revert should have no nrf noup tag, please update

fixed.


[Addition] World Pregenerators

Checklist

[X] I've checked that my mod suggestion hasn't already been added to one of the lists.
[X] I've checked the latest issues to make sure that this mod hasn't already been suggested.

What is your mod's name?
Chunk Pregenerator
What is your mod's main version?
Other (Forge)
What is your mod's other versions?
1.8.X or earlier, 1.12.X, 1.15.X (Forge), 1.16.X (Forge), 1.18.X (Forge), 1.19.X (Forge)
What is your mod's type?
Enhancement
What side dose your mod need to run on?

Worldgen lag is a real issue. Even in Singleplayer, and worldpregeneration is actually a really important thing you should do.
This can turn it from "unplayable" to "playable" especially if your core count (including threads) is stuck at 4 or less or the CPU isn't the newest anymore... And even then its suggested to do so.
That's why I am suggestion ChunkPregenerator.
Which supports 1.4.7-1.7.x-1.8.9-1.10.x-1.11.x-1.12.x-1.14.x-1.15x-1.16x-1.18x-1.19.x
Also another mod that I would suggest is Chunky
Which is another pregenerator that provides a lot of performance gains and covers
forge-fabric-customservers
while chunkpregenerator is dedicated for forge.
(OPTIONAL) Give some extra information about the mod.
Chunk Pregenerator is a tool for generating your world before you actually play it.
It also includes maintenance tools such as:

Chunk Deletion/Trimming
Performance Tracking (up to 1.12)
World Maintenance tools (up to 1.12)
Retro-generation that isn't relying on the Player Itself.
Harddrive protection (1.14 or newer where it becomes nessesary)
Memory Leak fixes that become apparent when pregeneration.

And a few other things.

I've added your mod to performance mods as i think that is a better fit then Enhancement. Love your mods, Keep up the good work :D

@NordicGamerFE
Small extra note since i saw this.
Know issues + Fixes are actually tracked by myself.
Here you find both lists.
1.12 or older: https://github.com/TinyModularThings/Chunk-Pregenerator-Issue-Tracker/issues/1
1.14 or newer: https://github.com/TinyModularThings/Chunk-Pregenerator-Issue-Tracker/issues/2


Dismiss Pulsator and CGPoint position not working.

There's two things that I can't seem to get working or can't figure out. First, the .position protocol doesn't recognize the CGPoint value and is always set to the upper left corner of the screen. Next, how do I dismiss the pulsator after starting it?

Same issue for me too

@andrewfashion have you figured it out yet???

Here's how I got it working...

Declare a Pulsator on MyViewController:

let pulsator = Pulsator()

In viewDidLoad, set your pulsator properties what view you want to add it to, etc.

yourImageView.layer.superlayer?.insertSublayer(pulsator, below: yourImageView.layer)
pulsator.numPulse = 3
pulsator.backgroundColor = UIColor.red.cgColor
pulsator.radius = containerView.bounds.width / 2

If you don't have viewDidLayoutSubviews configured, do so...

override func viewDidLayoutSubviews() {
	super.viewDidLayoutSubviews()
	view.layer.layoutIfNeeded()
	pulsator.position = yourImageView.center
}

Wherever you want it to start pulsing, then call pulsator.start(). Wherever you want it to stop, call pulsator.stop()


[DEN-34] Refactor Edge CNAMEs resource management to use SDK

Refactored Edge CNAME (formerly cname) to use SDK API client
Updated schema to match current Edge CNAME API (added dir_path, enable_custom_reports and removed origin_type)
Renamed cname resource to edgecname to reduce confusion with route/dns
Updated Edge CNAME resource example file to match updated schema
Added go generate Edge CNAME documentation
Updated go generate WAF Scopes documentation
Delete cname_client in favor of using SDK
Corrected some copyright messages in customer resource

Looks good, however we should consider adding "flatten" and "expand" functions to the edgecname code and add unit tests for them. Maybe a future clean-up task?

@stevenpaz good point, I will add to backlog


the editor note tap sound thingy

What feature do you want to get added on the base engine?
it scares me when it's on full spanking volume
there is a problem with the note sound in the chart editor because it's a little way too loud so make a setting option where you can make it so you can optimize the volume of the sound
To test your sight, and reliability, please select the option of what should NOT be requested.
Proper credit that was forgotten

I know there is already an option in the charter editor but it doesn't save

actually there is an option to mute the sounds entirely, and it DOES save.

i am not going to add a full option for it, because it's unnecessary.

nope that's something different from what I'm trying to talk about


The images being sliced vertically. How can I make it horizontal.

Hi, thanks for the amazing model.
I added the intermediate images of the grid anchor. I see the images are sliced vertically. How can I make it horizontal?
Original Image:

Intermediate anchor sliced image ("grid_9")

Hi, @rjmehta1993 , sry, I don't quite understand "the images are sliced vertically". As shown in your sliced image, the raw image is cropped with the 3x3 shape and then horizontally concatenated.

Hey, thanks for the quick reply. If you see the slicing, the slice is breaking the context of columns.
If I avoid slicing by keeping grid_1, the resolution is very low. The slicing breaks the column and index of the table, thus if I ask a simple question "What is the management fee for column ITD?" It fails to answer.
Can I slice it horizontally? Something like in the bottom.

Hi, @rjmehta1993 , sry, we currently don't support cropping a global image into multiple rectangular.
But, I think the failure of our model to answer the question is not due to the cropping but the Non-standard row-column relationships in this image.
Actually, I have tried parsing texts in the image with our HuggingFace demo, our model could well organize texts as the structure in the image, as shown below. This validates that the 2x3 cropping doesn't influence the model to understand texts at the same row.

I agree. The problem is not parsing or recognizing the text. But the problem is the spatial relation of column and index on the question.
It is not able to map the column and index to the question if the column is far from the index.
Not sure why. But thanks for this amazing model.

I agree. The problem is not parsing or recognizing the text. But the problem is the spatial relation of column and index on the question.
It is not able to map the column and index to the question if the column is far from the index.
Not sure why. But thanks for this amazing model.

Thanks for your appreciation. Such non-standard col-row relationship is still a challenging problem. We will further enhance the general structure understanding abilities in our future work~


[AMS] Fix #14687: Mixed resource group and account name in command "az ams streaming-endpoint show" #14687

Description

Switched resource group and account name in output string.
Testing Guide

Run the following command with an incorrect streaming endpoint name to see the error string that was changed
az ams streaming-endpoint show --resource-group [rg]--account-name [accountName]--name [incorrectName]
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x ] The PR title and description has followed the guideline in Submitting Pull Requests.

[x ] I adhere to the Command Guidelines.

AMS


BUGFIX: set a recipient for Approval Email

smtpmailer requires a valid recipient address and not just a list of bcc addressees

Thank you for your contribution here. I just wanted to let you know that we’re looking to improve our management of issues and pull requests, with a goal to providing clear direction on how Symbiote contributes to the open source community. As part of this ongoing effort:

issues and pull requests that have been open prior to December 2017 with no recent activity are being closed

this is included

if you can confirm this remains applicable for SS3.6+ please do open it again

remaining issues and pull requests will be

categorised

on a clear path to resolution (whatever the next step may be).

This ultimately brings us one step closer to our ambition; being able to prioritise current issues and contributions in a timely manner. If you have any questions, please see https://www.symbiote.com.au/contributing for further information.


MTT table entry with poison data

when smmtt or iomtt in use, it is possible that MTT table walk gets poisoned data, what exception should be raised in this case?

The hart would raise a hardware-error exception.
The IOMTT is expected to report the attempted poison consumption to the RAS subsystem and abort the DMA transaction. If the aborted transaction is a write then the IO bridge may discard the write; the details of how the write is discarded are implementation defined. If the IO protocol requires a response for write transactions (e.g., AXI) then a response as defined by the IO protocol may be generated by the IO bridge (e.g., SLVERR on BRESP - Write Response channel). For PCIe, for example, write transactions are posted and no response is returned when a write transaction is discarded. If the faulting transaction is a read then the device expects a completion. The IO bridge may provide a completion to the device. The data, if returned, in such completion is implementation defined; usually it is a fixed value such as all 0 or all 1. A status code may be returned to the device in the completion to indicate this condition. For AXI, for example, the completion status is provided by SLVERR on RRESP (Read Data channel). For PCIe, for example, the completion status field may be set to "Unsupported Request" (UR) or "Completer Abort" (CA).

The hart would raise a hardware-error exception.
The IOMTT is expected to report the attempted poison consumption to the RAS subsystem and abort the DMA transaction. If the aborted transaction is a write then the IO bridge may discard the write; the details of how the write is discarded are implementation defined. If the IO protocol requires a response for write transactions (e.g., AXI) then a response as defined by the IO protocol may be generated by the IO bridge (e.g., SLVERR on BRESP - Write Response channel). For PCIe, for example, write transactions are posted and no response is returned when a write transaction is discarded. If the faulting transaction is a read then the device expects a completion. The IO bridge may provide a completion to the device. The data, if returned, in such completion is implementation defined; usually it is a fixed value such as all 0 or all 1. A status code may be returned to the device in the completion to indicate this condition. For AXI, for example, the completion status is provided by SLVERR on RRESP (Read Data channel). For PCIe, for example, the completion status field may be set to "Unsupported Request" (UR) or "Completer Abort" (CA).

In latest spec chapter 4.2. The process of hart mtt table walk is listed and what exception should be reported are listed too(although poisoned data is not considered here). However, for iomtt, things are different for handling  access fault or poisoned data or reserved bits, is it possible to add some descriptions to clarify these situations in spec?

However, for iomtt, things are different for handling access fault or poisoned data or reserved bits, is it possible to add some descriptions to clarify these situations in spec?

See #87

this PR #87 is now merged.


modify unit test of storage module

Bugfix

Modify unit test of storage module

          1 Message

:book:
Executed 47 tests, with 0 failures (0 unexpected) in 54.052 (54.177) seconds

  Generated by :no_entry_sign: danger


Upgrade to EKS 1.25

Bump EKS version to 1.25

Hi @svennam92

Is there going to be a branch with EKS 1.25 that we can test the automation labs?
Do you want each area (Automation, Observability, etc..) to create a tracking issue with owner, or you want to track all modules on this issue?

@csantanapr As a rough plan, we start with creating a branch that bumps the version to 1.25, and run an e2e test suite. For test failures we can open issues and tag them with the appropriate label for each working group. We should also reference this issue so we have a view of all pending issues.

Should we make this one EKS 1.26?
Should each module section start testing with EKS 1.26 to see if there are any updates to be made to allow for the upgrade?

I think we decided to stay on an n-1 version for the workshop, so let's stick with 1.25 for now. I believe @ROunofF may have an update on the upgrade process

Completed by #606


[0.10.1] fix OSX environment path to allow haxe execute haxelib and neko

This fix haxe compilation & compiler code completion contribution. Allow to execute haxe/haxelib/neko on Mac OSX. Should be included to 0.10.1

should fix #370


Initial build out of form validation repo

Is your feature request related to a problem? Please describe.
This is the initial build out of the repo with the validation methodology from auro-input.
Describe the solution you'd like
All validation code from auro-input is moved into this repository.
Functionality to be made generic so that it's syntax makes sense regardless of the component that uses it.
Related ticket: auro-input: Move Validation functionality to new repo

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jason-capsule42:x: Jason BakerJason Baker seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


new Chart throws RangeError: minimumFractionDigits value is out of range.

Expected Behavior
Should work normally
Current Behavior
Throws an error RangeError: minimumFractionDigits value is out of range. from new Chart(...).
This is the same as #9019 which was already closed.
It happens when occasionally, and calculateLabelRotation gets a NaN value. This happens when Math.asin(Math.min(maxHeight / maxLabelDiagonal, 1)) is NaN, as you cannot Math.asin on a value less than -1.
This issue propagates into NaN margins and messes up everything until the next update call.
Possible Solution
The constraint should be Math.max(Math.min(maxHeight / maxLabelDiagonal, 1), -1).
Adding the Math.max(..., -1) here fixes it.
The fix is easy, and mathematically correct, as the range for asin is -1 to 1.
Steps to Reproduce
I actually had a hard time reproducing this. This happens ocassionally but not consistently, with a line (time series) chart, and the canvas not being in the dom yet. Although the canvas has a default size of 300x150 to specs.
I'm sorry I can't throw in here a codepen that shows the issue. I guess this has to do with lots of environmental parameters, perhaps including the current time.
Context
Environment

Chart.js version: 3.4.1
Browser name and version: Chrome 91.0.4472.124

I still got this kind of error in version 3.9.1

``> I still got this kind of error in version 3.9.1
try this
scale: {      y: {           ticks: {                callback: function (val, index) {                  return '${this.getLabelForValue(Number(val))}%';                },           }      } }

If you found this bug. See https://github.com/chartjs/Chart.js/issues/10673 for a potential solution.

@brixbimboavengoza's comment stopped the error for me, but the y axis only showed one tick (the topmost one), which wasn't usable to me.
Turns out, based on @kurkle's comment here, you can turn off swcMinify to fix this.
Problem is, swcMinify: true reduces my build time by almost half. And I didn't want to let that go.
I took a bit of a risk and changed the next version to the latest canary (15 as of now) as of today and that fixed it just fine! As of 12.3.2 canary 10, the swc minifier version was updated which included the fix.

i just upgrade my project to nextjs 12.3.1 and facing this error too,
Thanks for your words to let me understand what is the problem come form

This is for nextjs
@brixbimboavengoza's comment stopped the error for me, but the y axis only showed one tick (the topmost one), which wasn't usable to me.
Turns out, based on @kurkle's comment here, you can turn off swcMinify to fix this.
Problem is, swcMinify: true reduces my build time by almost half. And I didn't want to let that go.
I took a bit of a risk and changed the next version to the latest canary (15 as of now) as of today and that fixed it just fine! As of 12.3.2 canary 10, the swc minifier version was updated which included the fix.

i just upgrade my project to nextjs 12.3.1 and facing this error too,
Thanks for your words to let me understand what is the problem come form

@jayantbh @kelvin9314 sorry this fix was not really helpful, found out that the real problem comes from the versions I'm using.
I'm using Next JS with chakra ui. Below we're the versions that works for me
You can try to align these version into yours then run yarn/npm install again
(disregard packages that are non-existent for your case)

"chart.js": "^3.9.1",
"react-chartjs-2": "^4.3.1",
"next": "12.1.6",
"react": "18.1.0",
"@chakra-ui/react": "^1.8.8",
"@emotion/react": "11.9.0",
"@emotion/styled": "11.8.1",

Spent hours trying to debug this using various suggestions but in the end using "next": "^12.3.2-canary.32" in package.json is what fixed it.

``> I still got this kind of error in version 3.9.1
try this scale: { y: { ticks: { callback: function (val, index) { return '${this.getLabelForValue(Number(val))}%'; }, } } }

This fixed for me, thanks

We are actually seeing this issue and we are unfortunately pinned to a next.js compiler version that does not have SWC.
We are on:
"chart.js": "3.5.1",
"react-chartjs-2": "3.0.5",
"next": "10.0.8",

i encountered a similar error in version 4.4.0
(but about  maximumFractionDigits),
with the following setup:
const options = {
...
locale: 'hi',
ticks: {
format: {
style: 'currency',
currency: 'INR',
currencyDisplay: 'symbol',
minimumFractionDigits: 2,
//            maximumFractionDigits: 2,
},
},
...
};
my workaround was explicitly setting maximumFractionDigits (uncomment above)


make android error

I have tried the run this project with make android after make prepare-android cmd but I get below error.
is there an additional step for grade??
rustdev316:~/Projects/rustdev/uniffi-rs-fullstack-examples/hello$ make android cd platforms/android/UniffiRustExample && ./gradlew cargoBuild —info Error: Could not find or load main class org.gradle.wrapper.GradleWrapperMain Caused by: java.lang.ClassNotFoundException: org.gradle.wrapper.GradleWrapperMain make: *** [Makefile:44: android] Error 1 rustdev316:~/Projects/rustdev/uniffi-rs-fullstack-examples/hello$

Could you please send the full error trace?

hello
sorry for the late reply
That is the whole error in the terminal after make android

or is there a way to get a full error trace??

@andodeki sorry, I'm not familiar with Android. You need to google it to get full stack trace before I can help

i have not made changes to the project in this repo locally I just cloned it and prepared the env for android using prepare-android cmd and cd into hello folder and run make android Then I got the error.

@andodeki sorry for the delay.
I found my old laptop and the root cause:
hello/platforms/android/UniffiRustExample/local.properties is missing. Please create this file and add contents like:
sdk.dir=/opt/homebrew/share/android-sdk
ndk.dir=/opt/homebrew/share/android-ndk

(these paths above assume that you installed Android SDK and NDK using homebrew).

these are sdk and ndk installation paths right?

Yeah.

@andodeki How's it going?

hey @imWildCat i did that but i get an error
$ java -version openjdk version "1.8.0_345" OpenJDK Runtime Environment (IcedTea 3.24.0) (Alpine 8.345.01-r0) OpenJDK 64-Bit Server VM (build 25.345-b01, mixed mode)
$ export ANDROID_HOME=/home/rustdev316/.android $ export NDK_HOME=/home/rustdev316/.android/android-ndk-r20 $ make android cd platforms/android/UniffiRustExample && ./gradlew cargoBuild --info Error: Could not find or load main class org.gradle.wrapper.GradleWrapperMain make: *** [Makefile:44: android] Error 1 $
below is the local.properties file
$ cat platforms/android/UniffiRustExample/local.properties sdk.dir=/home/rustdev316/.android ndk.dir=/home/rustdev316/.android/android-ndk-r20

https://stackoverflow.com/a/21674403/2949840
Could you try to run ./grawdlew cargoBuild --debug --stacktrace

cd platforms/android/UniffiRustExample && ./gradlew cargoBuild --debug --stackstrace
Error: Could not find or load main class org.gradle.wrapper.GradleWrapperMain
Caused by: java.lang.ClassNotFoundException: org.gradle.wrapper.GradleWrapperMain

@andodeki are you on macOS? Could you try to install java 11 instead?
brew install openjdk@11

java -version
openjdk version "11.0.17" 2022-10-18
OpenJDK Runtime Environment (build 11.0.17+8-alpine-r0)
OpenJDK 64-Bit Server VM (build 11.0.17+8-alpine-r0, mixed mode)

export JAVA_HOME="/usr/lib/jvm/java-11-openjdk"

Still the same error?

yeah still the same error actually that was the installed java11 from the beginning

@andodeki do you have git-lfs and ran git lfs pull?

it builds but i get a differents error

build file '/home/ubuntu/Projects/rustdev/uniffi-rs-fullstack-examples/hello/platforms/android/UniffiRustExample/build.gradle': 14: only buildscript {} and other plugins {} script blocks are allowed before plugins {} blocks, no other statements are allowed

Sorry I don’t have more resource on this.
Please go ahead with Google

i managed to solve the error by chaning how build.gradle is written but now i get a different error
i think it needs python but its picking python by default instead of python3.
how can i set it to use python3 ??

uniffi-rs-fullstack-examples/hello/platforms/android/UniffiRustExample/build/linker-wrapper/linker-wrapper.sh: line 4: python: command not found

You can easily google it.
I'm not your free teacher.


RDK-34519 [DAC] Update BundleGen to support latest Dobby functionality

Please also rebase your changes so git history only contains the relevant commits

In latest patchset changes are rebased


added pyarrow/numpy dtype literals and allowed str | DtypeObj as input for Series.astype

[x] Closes #733

[x] Closes #747

[x] Tests added: Please use assert_type() to assert the type of any return value

pandas nullable UInt data types were missing

numpy type code literals

numpy alternative literals (half, short, double, etc.)

pyarrow literals

added DtypeObj | _str to final astype overload (resulting in Series[Any])

added few tests

fixed random bug: def __getattr__(self, name: str) -> S1 accidentally using str instead of _str.

@randolf-scholz  tests are failing

added tests for all the types
added ObjectDtypeArg

ObjectDtypeArg: TypeAlias = (
    # Builtin object type and its string alias
    type[object]  # noqa: Y030
    | Literal["object"]
    # Numpy object type and its string alias
    # https://numpy.org/doc/stable/reference/arrays.scalars.html#numpy.object_
    | type[np.object_]
    | Literal["O"]  # NOTE: "object_" not assigned
)

@Dr-Irv I removed the str overload for now, tests should pass. But I think this still should be discussed, maybe if LiteralString is supported by typecheckers one can do some overload trickery.
Apart from that, since I added ObjectDtypeArg one might consider adding object to S1. (and I'd suggest renaming S1 to something more descriptive like SeriesKind).
Also as a side questions: why do TimestampSeries and TimedeltaSeries, etc. exist? Why don't  Series[Timestamp] / Series[Timestamp] work?

@Dr-Irv I removed the str overload for now, and added the test you suggested. But I think this still should be discussed, possibly when LiteralString is supported by typecheckers one can do some overload trickery to get both dynamical string and typo-checking to work.

I'm still seeing AsTypeArg including str, so will ask for that to be removed.

Apart from that, since I added ObjectDtypeArg one might consider adding object to S1.

Let's handle in a separate PR.   I think if we do that there is a possibility of other errors getting in.

(and I'd suggest renaming S1 to something more descriptive like SeriesKind).

That's a historical artifact.  Open to a PR that would change that.

Also as a side questions: why do TimestampSeries and TimedeltaSeries, etc. exist?  Why don't Series[Timestamp] / Series[Timestamp] work?

This is because mypy can't handle the overloads correctly for the arithmetic operators on Series[Timestamp] and Series[Timedelta] .    See #718 and links to discussion in pyright there.

It looks like the Windows tests are failing, with the new parameterize stuff, you can get a complete list of what is failing.  You may need to test the O/S to get the values appropriate for Windows.

There is another thing that came up: pandas supports numpys extended type codes:
https://numpy.org/doc/stable/reference/arrays.dtypes.html#string-dtype-note
There are hundreds of possible combinations, and for some data types unlimited number of literal strings that are allowed. For example s.astype("S25") implicitly creates a bytes datatype with 25-bytes. any positive number of integer bytes is valid.
The question is: how many of these should be hard-coded as literals, to avoid false-postives as currently str will not be allowed.

There is another thing that came up: pandas supports numpys extended type codes: https://numpy.org/doc/stable/reference/arrays.dtypes.html#string-dtype-note
There are hundreds of possible combinations, and for some data types unlimited number of literal strings that are allowed. For example s.astype("S25") implicitly creates a bytes datatype with 25-bytes. any positive number of integer bytes is valid.
The question is: how many of these should be hard-coded as literals, to avoid false-postives as currently str will not be allowed.

I say none.  The goal of the pandas stubs is to cover the most common use cases.  We try to strike a balance between being too narrow and too wide.  IMHO, having str as a valid type for astype() is too wide and reduces the usefulness of the stubs in the most common use cases.
If someone actually uses these more esoteric strings with pandas, then they can open an issue and we can add them one by one.

I found np.sctypeDict which seems like a good list of aliases to support. I added a test to check that all these aliases are tested. The windows stuff is hopefully fixed as well.

Tests pass now


[zsdl] getMouseFocus and getMouseState

In this case I would go with below. What do you think?
pub const MouseState = extern struct {
   x: i32,
   y: i32,
   button_mask: u32,
};

pub fn getMouseState() MouseState {
...
}

I was tempted to this but couldn't decide. If you think this is better then I will agree! I will change and test change as soon as I get on computer today

Thanks!


Feature Request: Flutter should expose API to configure default windows size (Height, Width) cross platform

For cross platform app, we may want to set a different window size, with height and width for all platform(Windows, Linux, MacOS)

I guess there should be a api expose for us to call when
  runApp(MyApp());

Hi, thanks for filing the issue.
I see there are some issues that addressing the case you described, see #30736, #67708.
Closing as duplication, so please follow up on it for further updates.


Star Wars SWAPI (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)

[x ] I attest that the connector works and I verified by deploying and testing all the operations.
[x ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.

[x ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.

[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Thanks for the Star Wars connector submission! Hoping that people use this to learn how to use the Power Platform :) Given that this week is Thanksgiving week, we will pause certification processes and look at PRs starting Monday, 11/29. Hope you have a great week!

[[certify-connector]]

@paulculmsee Apologies for the delay in response.
Update - Your connector is now in production deployment and reached Test region, and it would be completed and available in all the production regions within 3 weeks. Please let us know if you have any questions, Thanks.

@paulculmsee Apologies for the delay in response.
Congratulations! The deployment of your connector and documentation page is now complete, We have a short survey on the connector certification process that we would like for you to complete. Your feedback is appreciated and will help us improve the program, Thanks.


Update Blockifier to 2.6.0-rc1

Update Blockifier and Native to the latest posible version

Solved on #30


Make color of close button customizable

Hey,
Hope you are doing good.
Thank you for your widget. Please make the color of the close button customizable at this line
Thanks,
Aamir Ali

Thanks, @Rodiii :)


Removes dependencies on individual service binaries from service inspectors

Overview
Service inspectors now work with service interfaces instead of concrete implementations.
For most inspectors this change was trivial. For IMixedRealitySceneService an optional IMixedRealitySceneServiceEditor interface was created for editor-only functionality used by the inspector.
Changes

Fixes: #4825

Verification

As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run mrtk_pr

/azp run mrtk_pr

pulled this and it looks to be displaying the profiles correctly.

looking into the change now

/azp run

/azp run mrtk_pr


Bump to 3.18.9

@kwizart, the ntp patch is now mainline, so I removed our patch
https://kernel.org/pub/linux/kernel/v3.x/ChangeLog-3.18.9
commit 20dcda8d40db528865b1aa5053b39e0a803df693
Author: John Stultz<EMAIL_ADDRESS>Date:   Mon Feb 9 23:30:36 2015 -0800

    ntp: Fixup adjtimex freq validation on 32-bit systems

    commit 29183a70b0b828500816bd794b3fe192fce89f73 upstream.

    Additional validation of adjtimex freq values to avoid
    potential multiplication overflows were added in commit
    5e5aeb4367b (time: adjtimex: Validate the ADJ_FREQUENCY values)

    Unfortunately the patch used LONG_MAX/MIN instead of
    LLONG_MAX/MIN, which was fine on 64-bit systems, but being
    much smaller on 32-bit systems caused false positives
    resulting in most direct frequency adjustments to fail w/
    EINVAL.

    ntpd only does direct frequency adjustments at startup, so
    the issue was not as easily observed there, but other time
    sync applications like ptpd and chrony were more effected by
    the bug.

    See bugs:

      https://bugzilla.kernel.org/show_bug.cgi?id=92481
      https://bugzilla.redhat.com/show_bug.cgi?id=1188074

    This patch changes the checks to use LLONG_MAX for
    clarity, and additionally the checks are disabled
    on 32-bit systems since LLONG_MAX/PPM_SCALE is always
    larger then the 32-bit long freq value, so multiplication
    overflows aren't possible there.

    Reported-by: Josh Boyer<EMAIL_ADDRESS>    Reported-by: George Joseph<EMAIL_ADDRESS>    Tested-by: George Joseph<EMAIL_ADDRESS>    Signed-off-by: John Stultz<EMAIL_ADDRESS>    Signed-off-by: Peter Zijlstra (Intel<EMAIL_ADDRESS>    Cc: Linus Torvalds<EMAIL_ADDRESS>    Cc: Sasha Levin<EMAIL_ADDRESS>    Link<EMAIL_ADDRESS>    [ Prettified the changelog and the comments a bit. ]
    Signed-off-by: Ingo Molnar<EMAIL_ADDRESS>    Signed-off-by: Greg Kroah-Hartman<EMAIL_ADDRESS>


Code styles

Could you add to project the code styles you used in order to create a pull request with some very small but good changes? ^^
Thx mate.
BTW, very good job :-)

Sure man, will do that asap :)

:+1:


SWR responses cached forever in CloudFlare Workers

Environment
CloudFlare Workers
Nuxt 3.4.1
nitro: {
    storage: {
      cache: {
        driver: "cloudflare-kv-binding",
      },
    },
  },

Reproduction
export default cachedEventHandler(
  async () => {
    // Test subrequest
    return await $fetch(
      "https://timeapi.io/api/Time/current/zone?timeZone=Europe/Budapest"
    );
  },
  {
    swr: true,
    maxAge: 60
  }
);

deployed via Wrangler
Describe the bug
The initial request is cached forever.
The expires time is 15 minutes ago from now as I am writing this.
I tested it upto 2 days.
It works on local Wrangler dev perfectly.
It also works on CloudFlare Workers with swr: false.
Additional context
The initial response:
{
   "year":2023,
   "month":4,
   "day":28,
   "hour":15,
   "minute":47,
   "seconds":6,
   "milliSeconds":370,
   "dateTime":"2023-04-28T15:47:06.370526",
   "date":"04/28/2023",
   "time":"15:47",
   "timeZone":"Europe/Budapest",
   "dayOfWeek":"Friday",
   "dstActive":true
}

Cache file from KV:
{
   "expires":1682689686245,
   "value":{
      "code":200,
      "headers":{
         "etag":"W/\"z7FJBuCozK\"",
         "last-modified":"Fri, 28 Apr 2023 13:47:06 GMT",
         "cache-control":"s-maxage=60, stale-while-revalidate=15"
      },
      "body":{
         "year":2023,
         "month":4,
         "day":28,
         "hour":15,
         "minute":47,
         "seconds":6,
         "milliSeconds":370,
         "dateTime":"2023-04-28T15:47:06.370526",
         "date":"04/28/2023",
         "time":"15:47",
         "timeZone":"Europe/Budapest",
         "dayOfWeek":"Friday",
         "dstActive":true
      }
   },
   "mtime":1682689626377,
   "integrity":"MVNRfu1EFA"
}

Logs
No response

Is this solved?

Is this solved?

Yes, it was fixed by this commit: https://github.com/unjs/nitro/commit/421d6255ea4633753101ccaac21a8b89726d0c01
Thank you!


Upgrade/lingo 3.2.0

Fixes LogicException: This ParserOutput contains no text! error.

Hi - it looks like you accidentally included basically all of Taqasta in this PR, if I understand it correctly.


[DOCS0-2] Document all public classes, methods and functions

Use this utility: https://github.com/SuperDuperDB/superduperdb/blob/main/test/unittest/test_docstrings.py
TODO:

[ ] Add a utility to restrict the checkers to the below places.

Prelaunch:

[ ] base.datalayer.Datalayer
[ ] components.component.Component
[ ] components.model.*
[ ] components.datatype.*
[ ] components.vector_index.VectorIndex
[ ] components.schema.Schema
[ ] components.metric.Metric

@jieguangzhou to increase coverage and doc-string quality.


Improve performance and efficiency of Array.Parallel.choose

I've re-implemented the Array.Parallel.choose function so it's more memory-efficient for typical use cases. The current implementation allocates two arrays (of 'U and bool, respectively) of the same length as the input array to hold intermediate results; this wastes a fair bit of memory when the input array is large, and especially when the proportion of 'chosen' elements is small.
The downside is that the new code is somewhat more complex than the existing implementation. I've introduced additional comparer types and the private combineWorkerResults function to merge intermediate results into the final result; this infrastructure can be re-used in the future if we want to add additional functions from the Array module to the Array.Parallel module (e.g., filter, choose2) or retrofit existing functions like Array.Parallel.partition for more efficiency/performance.

Hi @jack-pappas - Cool, do you have perf results?  Also, are new tests needed?

@dsyme I don't have any perf. results, beyond that the new function "seems" to be at least as fast, if not faster. The reduction in memory usage was the main impetus behind this new implementation. I had some cases where I was using Array.Parallel.choose on large arrays of small data types (byte, char, etc.) and choosing only small fractions of the input data, and I realized the existing function was using O(input) memory instead of O(output) memory.
I can put some simple benchmarks together though to compare the performance of the two implementations if you'd like.
I don't think any new tests are needed -- the unit test suite already has tests for Array.Parallel.choose, and those should sufficiently cover this new implementation as well. Although not part of this official repository, I'm using this same implementation in a number of other Array.Parallel functions in ExtCore which have passing unit tests. (I meant to submit them for F# 4.0 and didn't have the time over the holidays, and now we've reached the feature freeze.)

@latkin - As per my other messages, we should draft guidelines on what's needed for a PR for library function perf improvements.
@jack-pappas  - I think we can assume that the guidelines will say "Perf results must be submitted" :)
@latkin - perhaps we should start a branch for post-F#-4.0 PRs?

Thank you for the submission. Per the contribution guidelines, perf optimizations should include tests and quantitative analysis showing reliable gains from the change, which others can reproduce.  Please re-open when tests and analysis are in place.

I tweeted a request that people add the required perf/correctness testing and resubmit https://twitter.com/dsyme/status/559725937940393988


Added Config class to help manage everything with config values

pog
i'll give it a proper look when my server's up, but im sure it's good!


Chromium fails to launch in CentOS 7.2 (linux-503964, sandbox disabled)

$ ./node_modules/puppeteer/.local-chromium/linux-503964/chrome-linux/chrome --headless --no-sandbox --disable-setuid-sandbox
[0928/203844.466538:FATAL:nss_util.cc(632)] NSS_VersionCheck("3.26") failed. NSS >= 3.26 is required.  Core file will not be generated.
[0928/203844.611182:ERROR:broker_posix.cc(43)] Invalid node channel message
[0100/000000.611185:ERROR:broker_posix.cc(43)] Invalid node channel message

Weirdly..... I have NSS installed:
user@machine:/var/project$ sudo yum install nss-util
Loaded plugins: fastestmirror, langpacks, logchanges
Loading mirror speeds from cached hostfile
Package nss-util-3.21.0-2.2.el7_2.x86_64 already installed and latest version
Nothing to do
user@machine:/var/project$ sudo yum install nss-softokn
Loaded plugins: fastestmirror, langpacks, logchanges
Loading mirror speeds from cached hostfile
Package nss-softokn-<IP_ADDRESS>-14.2.el7_2.x86_64 already installed and latest version
Nothing to do
user@machine:/var/project$ sudo yum install nss-sysinit
Loaded plugins: fastestmirror, langpacks, logchanges
Loading mirror speeds from cached hostfile
Package nss-sysinit-3.21.0-9.el7_2.x86_64 already installed and latest version
Nothing to do
user@machine:/var/project$ sudo yum install nss-tools
Loaded plugins: fastestmirror, langpacks, logchanges
Loading mirror speeds from cached hostfile
Package nss-tools-3.21.0-9.el7_2.x86_64 already installed and latest version
Nothing to do
user@machine:/var/project$ sudo yum install nss-softokn-freebl
Loaded plugins: fastestmirror, langpacks, logchanges
Loading mirror speeds from cached hostfile
Package nss-softokn-freebl-<IP_ADDRESS>-14.2.el7_2.x86_64 already installed and latest version
Nothing to do

When launching from Puppeteer I get:
Caught error: Error: Protocol error (Network.enable): Target closed.
Machine info:
LSB Version:	:core-4.1-amd64:core-4.1-noarch:cxx-4.1-amd64:cxx-4.1-noarch:desktop-4.1-amd64:desktop-4.1-noarch:languages-4.1-amd64:languages-4.1-noarch:printing-4.1-amd64:printing-4.1-noarch
Distributor ID:	CentOS
Description:	CentOS Linux release 7.2.1511 (Core)
Release:	7.2.1511
Codename:	Core

Dupe of https://github.com/GoogleChrome/puppeteer/issues/391

This is absolutely not a dup! I've done everything in #391 to try to fix this!

https://github.com/adieuadieu/serverless-chrome/issues/53 suggests you may need to build a newer version of nss.

@ebidel
I will try. Thank you

Just to give an update.
The VM image on the datacenter's CentOS 7.2 image was frozen at 7.2 so I added a new repo sourced to CentOS 7.4 which had NSS 3.28 so I was able to upgrade and get chromium working.

Our Centos7.2 updated library to NSS 3.28 is work!
The CentOS7.4 Release is OK.
If the user is root, by the tips,  use --no-sandbox args :
      const browser = await puppeteer.launch({
          args : ['--no-sandbox']
      });


Disabling Mixer tests using the new TF in K8s.

https://github.com/istio/istio/issues/12750

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
f82cc3a1ea4bc0c2b4c1654d6d77c4d431c754a5
link
/test istio-pilot-multicluster-e2e

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.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nmittler, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/OWNERS [nmittler,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
f82cc3a1ea4bc0c2b4c1654d6d77c4d431c754a5
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
12a902ce48b89a390caceb884fd5e1f3fb64c084
link
/test istio-integ-local-tests

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.

integ local test failure is TestAuthnPermissive flake.

/test istio-integ-local-tests

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
12a902ce48b89a390caceb884fd5e1f3fb64c084
link
/test istio-pilot-multicluster-e2e

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.


Provide separate highscore tracking for competition tables

When a table is part of a competition, provide an option not  to track the highscore into the usual nvram.
VPin MAME supports a reg key for the nvram path in the global settings.
Check if this can be customized for every table and provide a separate pinemhi installation that is only used for these competition tracking.
Let the user now that the table that is played is tracked differently: Provide a pop-up notifcation for competition modes of tables.
(Separate feature request).

VPinMAME does not support a path parameter for every rom. So for now, the existing backup and reset mechanismn must suffice.


[datalake] Add DirectoryClient and FileClient

In this PR I hope to finish migration of existing operations and add some more related ones.
There are some slight changes to current patterns in the hopes to address some of what I gathered from discussions in this repo as well as some thoughts collected while implementing prior changes.
From what I understand in many ways C# often serves as reference implementation for SDKs. As over there I introduced a DirectoryClient and FileClient rather than having file operations live on the FileSystemClient. Not sure if other crates do this as well, but since the file and directory operations all work against the same same REST Api route, I opted for having one operation that supports all options for the specific route and using the builder for multiple operations on multiple clients. Main motivation is to avoid the significant code duplication and reduce maintenance effort.

@ctaggart @rylev @thovoll - I made some somewhat structural changes to the crate and am hoping to get some early feedback if that approach makes sense. There is still a bunch of work in the PR, but the relevant aspects can be seen in the files 'operations/path_put.rs', clients/directory_client.ts, and request_options.rs.
Any guidance if this is a desired direction is greatly appreciated.

One thing that came to my mind is the question how the client should behave for rename operations. Specifically the client can no longer be used to operate on the file that has been renamed, but a new client needs to be created. We do create a client for the target location inside the rename call. SO my question is should a file or directory client update its internal path to still track the renamed location. In that case it could be re-used after renaming?
To be honest I am not even sure what I would expect as a user - on one hand i specified a path and changing that could be surprising, on the other hand if I think of this as a physical file I would feel it still is the same file, just with a different name.

perhaps you could talk more about the motivation behind PathClient? I'm not sure I fully understand why that approach is preferable.

The main motivation is that operations against files and directories are essentially the same in terms of the REST API request. However - taking C# as guideline - we wanted to separate Files and Directories conceptually. So the main motivation is to avoid code duplication since the request mainly just differ in the resource query parameter.

Another question that came up is how to present the results for delete operations. These operations return a continuation token since there seems to be a max number of items to delete in a single request - as such should the delete operation return a stream much like the list ops? From the tech side that seems clear, but I am not sure from the user perspective. On the file system no continuation is needed, so delete would look different on file systems and file / directory.
We do return the token in the response, but right now we treat it like other "one-off" operations.

One thing that came to my mind is the question how the client should behave for rename operations. Specifically the client can no longer be used to operate on the file that has been renamed, but a new client needs to be created. We do create a client for the target location inside the rename call. SO my question is should a file or directory client update its internal path to still track the renamed location. In that case it could be re-used after renaming?
To be honest I am not even sure what I would expect as a user - on one hand i specified a path and changing that could be surprising, on the other hand if I think of this as a physical file I would feel it still is the same file, just with a different name.

The .Net and Java SDKs return a new client when renaming and don't update the internal state of the "old" client, which has the problem you mentioned (and more) but it's a pattern we will have to follow unless there's a Rust specific reason that we can't.

This is an interesting and confusing case. With ADLS Gen2 storage accounts (meaning that hierarchical namespaces are enabled) file and directory deletes and renames are atomic and don't need continuations. However, the ADLS Gen2 REST API can also be used with a storage account that has hierarchical namespaces DISABLED (which means it's not an ADLS Gen2 storage account). This is where continuations come into the picture.
The .Net SDK simply returns the continuation as a header and lets the SDK user deal with it if needed. I think we should do the same here and not return a stream, since using the ADLS Gen2 API with a non-ADLS Gen2 storage account is not the main use case.

The PR is almost ready now and can be more seriously reviewed. As for the questions mentioned above, I followed the points @thovoll  made above, and modelled the responses after the C# implementation.
Mostly I need to clean up the examples and extend the tests. I am also going to try and record transactions for the tests.
I think I finally understand what @rylev meant re the path client. Its probably unnecessarily convoluted to have the builders be generic over file and directory client, and rather implement a base path client to use as inner client in the builders.
There is also two from the delta-rs perspective critical operations missing - read and list - maybe I can add these as well, since work is quite mechanical at this point.
This PR has again become quite large. My hope is that due to the "repetitive" nature of the implementation it is still reviewable. IF not, I can break out additional operations in a separate request.

Nit: Probably should use file_system everywhere instead of filesystem. The REST API does call it Filesystem but the .Net and Java SDKs call it FileSystem.

@roeap this looks really good, thanks for all the work!

Thanks for the feedback @thovoll - i updated parameter names and made the APIs a little bit more generic.

@rylev - we seem to see intermittent test failures and I am trying to figure out why. Essentially most of the time test with REPLAY work, until they don't :D. Trying to figure out the root cause, and if it is even related to the mock framework.
Do you have any suggestions where to look here?

Resolves: #490

@rylev - we seem to see intermittent test failures and I am trying to figure out why. Essentially most of the time test with REPLAY work, until they don't :D. Trying to figure out the root cause, and if it is even related to the mock framework.
Do you have any suggestions where to look here?

What is the error?

you can look at the latest failed run, just before the latest successful one. the only change is re-recording transactions. also in the previos change between last success and new failure the changes "should" not have had any effect. It was just a bit of renaming and making parameter types a bit more permissive. String --> ´into´

@roeap I can reproduce the intermittent failure on my machine. I added some println!s and was able to see what's causing the failure:
     Running tests\file_system.rs (C:\Code\Github\roeap\azure-sdk-for-rust\target\debug\deps\file_system-ef3c18c80a73359f.exe)

running 1 test
----------------------------------------
actual_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
expected_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
----------------------------------------
actual_uri = '"/?resource=account&prefix=azurerustsdk&maxresults=3"'
expected_uri = '"/?resource=account&prefix=azurerustsdk&maxresults=3"'
----------------------------------------
actual_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
expected_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
----------------------------------------
actual_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
expected_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
----------------------------------------
actual_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
expected_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
actual_headers.len() = 4
expected_headers.len() = 3
actual_headers = '[("x-ms-properties", "ModifiedBy=SW90YQ==,AddedVia=QXp1cmUgU0RLIGZvciBSdXN0"), ("content-length", "0"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12")]'
expected_headers = '[("content-length", "0"), ("x-ms-properties", "AddedVia=QXp1cmUgU0RLIGZvciBSdXN0,ModifiedBy=SW90YQ=="), ("x-ms-version", "2019-12-12")]'
----------------------------------------
actual_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
expected_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
actual_headers.len() = 5
expected_headers.len() = 3
actual_headers = '[("x-ms-properties", "ModifiedBy=SW90YQ==,AddedVia=QXp1cmUgU0RLIGZvciBSdXN0"), ("content-length", "0"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12")]'
expected_headers = '[("x-ms-version", "2019-12-12"), ("content-length", "0"), ("x-ms-properties", "AddedVia=QXp1cmUgU0RLIGZvciBSdXN0,ModifiedBy=SW90YQ==")]'
----------------------------------------
actual_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
expected_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
actual_headers.len() = 6
expected_headers.len() = 3
actual_headers = '[("x-ms-properties", "ModifiedBy=SW90YQ==,AddedVia=QXp1cmUgU0RLIGZvciBSdXN0"), ("content-length", "0"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12")]'
expected_headers = '[("x-ms-properties", "AddedVia=QXp1cmUgU0RLIGZvciBSdXN0,ModifiedBy=SW90YQ=="), ("x-ms-version", "2019-12-12"), ("content-length", "0")]'
----------------------------------------
actual_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
expected_uri = '"/azurerustsdk-datalake-file-system?resource=filesystem"'
actual_headers.len() = 7
expected_headers.len() = 3
actual_headers = '[("x-ms-properties", "ModifiedBy=SW90YQ==,AddedVia=QXp1cmUgU0RLIGZvciBSdXN0"), ("content-length", "0"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12"), ("x-ms-version", "2019-12-12")]'
expected_headers = '[("x-ms-properties", "AddedVia=QXp1cmUgU0RLIGZvciBSdXN0,ModifiedBy=SW90YQ=="), ("x-ms-version", "2019-12-12"), ("content-length", "0")]'
Error: CoreError(Policy(MismatchedRequestHeadersCount(7, 3)))
thread 'file_system_create_delete' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: the test returned a termination value with a non-zero status code (1) which indicates a failure', /rustc/59eed8a2aac0230a8b53e89d4e99d55912ba6b35\library\test\src\lib.rs:194:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
test file_system_create_delete ... FAILED

failures:

failures:
    file_system_create_delete

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 15.81s

error: test failed, to rerun pass '--test file_system'

It seems that the "x-ms-version" header starts replicating at some point in the test.
Another lurking problem looks to be that the header value for "x-ms-properties" doesn't match because the semicolon-separated items within the value are ordered differently, e.g.:
actual_headers = '[("x-ms-properties", "ModifiedBy=SW90YQ==,AddedVia=QXp1cmUgU0RLIGZvciBSdXN0"),
'expected_headers = '[("x-ms-properties", "AddedVia=QXp1cmUgU0RLIGZvciBSdXN0,ModifiedBy=SW90YQ=="),'

Thanks for the investigation @thovoll. It looks like the root cause is that Properties are stored in a HashMap which does not maintain consistent order. We can instead use a BTreeMap which keeps its keys in sorted order. This would fix the issue.

Thanks @thovoll  @rylev  for figuring this out ... I updated the implementation of properties accordingly. At leat locally I could not generate the error again after several trials... so looking good so far :)


Support for @UtilityClass

The plugin does not correctly support the UtilityClass annotation.
See attached screenshot.
The annotation makes the inner class AquaTestRuntimeFactory static and so the static method create() is valid but Intellij complains about this.

Dupe of https://github.com/mplushnikov/lombok-intellij-plugin/issues/143

@AlexejK Sorry about that, should have searched properly before opening the issue. Thanks.


Error Core/Config.php array_keys() expects parameter 1 to be array

Describe the bug
Got the following error breaking parts of the GUI:
Error Core/Config.php array_keys() expects parameter 1 to be array, string given (errno=2)

Apparently its related to the function isArraySequential(&$arrayData) which fails when it is triggered on a string as $arrayData
https://github.com/opnsense/core/blob/master/src/opnsense/mvc/app/library/OPNsense/Core/Config.php#L80
When I print out $xmlNodeName on line https://github.com/opnsense/core/blob/master/src/opnsense/mvc/app/library/OPNsense/Core/Config.php#L114
I do get "ca". Which is an empty string. Right before I receive the mentioned error.
Work Around
I've added an is_array check in the function on line 80 and returned an empty string in that case. This helped me work around this issue but I assume it should never get into this situation right?
To Reproduce
Not sure how I got in this situation. Issue was visible using the iperf plugin but seems unrelated. Seems just a code bug.
Expected behavior
Should not run a function that crashes on string value.
Environment
Software version:
FreeBSD 12.1-RELEASE-p11-HBSD #0  74f1f081a1e(stable/20.7)-dirty: Fri Dec  4 13:40:15 CET 2020     root@sensey64:/usr/obj/usr/src/amd64.amd64/sys/SMP amd64
OPNsense 20.7.6 ec006cefe
Plugins os-acme-client-2.1 os-dyndns-1.23 os-iperf-1.0
Time Thu, 17 Dec 2020 05:13:24 +0100
OpenSSL 1.1.1h  22 Sep 2020
PHP 7.3.25
Plugins:
os-acme-client (installed) | 2.1 | 562KiB | Let's Encrypt client |
-- | -- | -- | -- | --
os-dnscrypt-proxy (installed) | 1.8 | 79.1KiB | Flexible DNS proxy supporting DNSCrypt and DoH |
os-iperf (installed) | 1.0 | 24.3KiB | Connection speed tester |
os-maltrail (installed) | 1.5 | 39.9KiB | Malicious traffic detection system |
os-theme-vicuna (installed) | 1.1 | 5.24MiB | The vicuna theme - dark anthrazit |
os-dyndns (installed) | 1.23 | 160KiB | Dynamic DNS Support

was there more data in the traceback? if needed we can safeguard the function, but I do like to understand why it got here in the first place.

never mind, https://github.com/opnsense/core/commit/03f71e97e9b76556d40684300713f91179e5138e moves the previous is_array() into the function which should prevent this anyways. If there is another bug causing this, I'm interested in the next traceback when it happens.

@AdSchellevis : I did trigger the bug report from the GUI, which should include the full trace. Not sure what caused it but  since it crashes after xmlNodeName echo's ca during debugging, I assume it's some kind of "ca" field that is an empty string?
Under system -> Trust -> Authorities , I do have an empty certificate, self signed with no name.
Not sure if this is normal or which plugin created this.
Might be Let's Encrypt which is active. OpenVPN I'm not using, but I did click on "Use a wizard to setup a new server" once but did not go through with the step "Create a New Certificate Authority (CA) Certificate".
Also I did try out Tailscale installation as mentioned in : https://github.com/tailscale/tailscale/issues/787 . Not sure if this one creates a certificate which is visible for OPNsense and which causes this issue.
Should I just remove the empty self signed certificate?

@Bramzor I would remove the empty record indeed, it might be a bug in the let's encrypt plugin as it inserts certs too (although I'm not sure). Sometimes the config history helps in tracking these down when the faulty data was generated recently.


feat: change subcollection delimiter

Change subcollection delimiter
Description
Changed the subcollection delimiter to be + instead of _ as this is much less common as an id format than _.
Fixes #249
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Testing
Confirmed that the bug occurred with the same tests for the unaltered membrane.
Then to confirm the change fixed this bug, used BloomRPC with the local membrane to do tests for getting, setting, and deleting documents. Then tested with a local nitric run by pointing the CLI membrane plugins to my local membrane plugins and performing get, set, and delete against the collection.

Codecov Report

Merging #293 (363e46d) into develop (481e3ac) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop     #293      +/-   ##
===========================================
- Coverage    61.40%   61.35%   -0.05%
===========================================
  Files           43       43
  Lines         4350     4353       +3
===========================================
  Hits          2671     2671
- Misses        1504     1506       +2
- Partials       175      176       +1

Flag
Coverage Δ

unittests
61.35% <0.00%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/plugins/document/document.go
76.22% <0.00%> (-1.64%)
:arrow_down:

Continue to review full report at Codecov.

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


 I use top -Hp slave broker process to find which thread cpu is the highest and use jstack to see what the thread that has the highest cpu is doing, then I get following pictures

as shown in the above pictures, asyncappender-worker-thread has the highest CPU, why does this situation happens?How can I solve this question? Can I solve this question by changing parameters of rocketMq?

@WillemJiang @justinmclean @jimjag @ke4qqq 大佬帮帮忙，我们非功能测试不知道为什么Kill掉其中一台机器的master broker，slave broker的CPU就飙涨

@walterhong I will try with your description and if you have other useful detailed information, please comment it in here, but as your picture shows, the thread that occupies the highest CPU is 10227, it's hexadecimal representation is 27f3, so maybe you should list more information here,

@duhenglucky I use top -Hp 10227, and I get the thread PID that has the highest CPU in process 10227, and then I use ‘printf %x\n to transform this thread PID into its hexadecimal representation,0x2889,and at last I use jstack 10277|grep 0x2889 -A 30, and I find the one of the highest CPU thread name in process 10277 is Async-appended-worker, and the other threads in process 10277 which have as high CPU as Async-appended-worker are  Netty threads such as nettyserverthread, nettyNIOthread, and nettycodecthrrad.

@walterhong What you said seems makes sense. It is recommended to check the relevant configuration of sysctl, especially the related configuration in os.sh. In addition, it is recommended to execute os.sh before service started, which optimizes the default values of some os kernels parameters.

@duhenglucky
I have ran os.sh on every machine that broker is running, but when I killed the master broker, the slave broker cpu was still high. I configure master slave syncronizing mode to async, flushing disk mode to sync. The threads that has high cpu can be categorized into two types, one is nettynioselectorthread, the other is nettyservercodecthread. What the nettyservercodecthread is doing is showing in the following pictures:

the pictures are content of slave broker process in jstack. I find that when master broker is down, slave broker is trying to do something very fast in org.apache.rocketmq.remoting model, which makes the cpu high.

@duhenglucky
I remote debug slave broker that has high cpu,and find that request code of the highest thread in slave broker process is get_broker_runtime_info

@walterhong   did you used the rocketmq-console？and this  page will frequently call this interface and will lead to highest CPU usage.

@duhenglucky
I kill rocketmq-console when slave broker CPU is high, but the CPU does not fall down

@duhenglucky
I kill rocketmq-console when slave broker CPU is high, but the CPU does not fall down

ok, could you provide more information, or you can connect with me by Dingtalk:juicehenry


feat: axiom xor

Description
Added xorconstraint for axiom backend
Problem*

Resolves
Summary*

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Thanks!! This is good for now but I think in the latest halo2 release by axiom xor is added so we should be able to use that directly as well if the version of the dependency is updated https://github.com/axiom-crypto/halo2-lib/blob/400122a6cf074783d0e5ee904a711e75ddfff3d4/halo2-base/src/gates/flex_gate/mod.rs#L569

I'll change then!

In the latest version, Context::new() requires more arguments such as type_id, context_id and copy_manager. How can we do this across places where we instantiate Context?

don't know much about how the new Context work :'(    Im ok with merging this as it is. may be the migration to new versions can happen in another PR


painless C extensions with CFFI

so idk about you, but i think CFFI is pretty painless for interacting with C/C++ code:

you simply compile a .so file, use ffi.dlopen('….so'), ffi.cdef('void myfunc(…)') and are able to call it from python.
it works with PyPy
it can be used with numpy (example)

we just have to think if we need to pass complex data structures, but i assume a few dense and sparse matrices is all you need. we just have to figure that out beforehand, and how to create a sparse matrix from raw memory (AFAIK everything under the sun supports at least column compressed layout, we might want to switch csr_matrix → csc_matrix)

Seems cool to me. Let's give this a try as soon as there's a little bit of time.

As we don't have any extensions anymore we can close this for now. Also, many people start adapting numba. In the context of Scanpy this should usually be enough... let's talk again in case we need it.


Search, Edit and Go To menus disappeared from script editor??

Godot version:
3.2.1 stable
OS/device including version:
Win 10 Lenovo 2-in-1 (physical-keyboard-less tablet mode)
Issue description:
I noticed that the Edit, Search and Go To menus had disappeared from the script editor as I was trying to repro what I thought was a "ctrl+f doesn't move focus to text input" bug - but at that moment the editor had no option/handler for ctrl+f altogether! Clicking around a little, to another tab and back brought the missing menus back. I could actually see them pop into existence as I returned to the faulty script window.
So, not that big of a deal, but weird!
Steps to reproduce:
Sorry, none as of yet. I just reckoned that the oddity deserved an entry in case someone else runs into it.
Oh, just remembered: I had just done "close other tabs" a short while before. Related?
Minimal reproduction project:
Any.

Duplicate of https://github.com/godotengine/godot/issues/38837.


Consider adding a IWebHostBuilder.Configure overload that takes a WebHostBuilderContext

Would be nice/consistent to have this additional overload as it also exists for ConfigureAppConfiguration and ConfigureServices. It would allow us to access the configuration and environment info without having to resolve it ourselves from DI.

@cwe1ss, would you be interested in sending a PR for this?  We can take this in 2.x timeline if it can be an extension method?

Nah sorry, I’m on a sabbatical right now so I try not to do too much programming related stuff.


Flatten / merge for moving content from the old Github there.

Hi Roelant
Would you agree to keeping the sample usage application in a separate repository?
Only having the schema and documentation. I think that the libraries we develop is where we do validation and test out usage. E.g. dp-metalib.net
Would you be happy with that?

Hi Roelant
As per our offline conversation. I am closing this pull request at your request.
Cheers Scott