POSTing to a page without actions should result in a 405

Describe the bug
https://twitter.com/bradgarropy/status/1617733494490423298
If you POST to a page without an action, you should get a 405 response, but instead you get a 200. I could have sworn this already happened, but it doesn't — we only 405 in different circumstances
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-cvxk7v?file=src%2Froutes%2F%2Bpage.svelte,svelte.config.js&terminal=dev
Logs
No response
System Info
System:
    OS: macOS 12.5.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 442.55 MB / 32.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.17.0 - ~/.nvm/versions/node/v16.17.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v16.17.0/bin/yarn
    npm: 8.15.0 - ~/.nvm/versions/node/v16.17.0/bin/npm
  Browsers:
    Chrome: 109.0.5414.87
    Chrome Canary: 111.0.5553.0
    Firefox: 109.0
    Safari: 15.6.1
  npmPackages:
    @sveltejs/adapter-auto: ^1.0.0 => 1.0.2
    @sveltejs/adapter-cloudflare: ^1.0.0 => 1.1.0
    @sveltejs/adapter-cloudflare-workers: ^1.0.0 => 1.0.6
    @sveltejs/adapter-netlify: ^1.0.0 => 1.0.5
    @sveltejs/adapter-node: ^1.0.0 => 1.1.4
    @sveltejs/adapter-static: ^1.0.0 => 1.0.5
    @sveltejs/adapter-vercel: ^1.0.0 => 1.0.5
    @sveltejs/kit: ^1.0.0 => 1.2.5
    svelte: ^3.55.1 => 3.55.1
    vite: ^4.0.4 => 4.0.4

Severity
annoyance
Additional Information
No response

Problem's this line here, https://github.com/sveltejs/kit/blob/3f1706bfc3dcf216e0c5473133e6bd6686f156df/packages/kit/src/runtime/server/page/actions.js#L116
leaf_node.server isn't there when the file is absent so none of the handling for actions here ever takes place


Country code hard code value issue fixed

Please Verified and merged with the master

Verified and merged with the master


Upgrade Module to Terraform 0.12+ Syntax

Would be nice to update this module to HCL2 / Terraform 0.12 syntax and publish a new release to Github / the Terraform registry to enable similar upgrades of some config dependent on it.

FYI @alvin-huang 😄


remove unused variable worker-instance-type

Now that we use a MixedInstancesPolicy it's no longer possible to
specify a worker instance type.  So this should be removed.
Created as draft while I first remove usages from the cluster-config repos

Just waiting on https://github.com/alphagov/verify-cluster-config/pull/120 at this point


What leads to TFMs being appended to some project names?

For multitargeting projects only one TFM is chosen randomly for indexing. I suppose we should turn it off?

Oh, that's a simple explanation. Not sure why I thought more of these were multitargeting. I don't mind there being a reminder that I'm missing out on other TFMs so I can keep in mind that after following a reference I might be taken to the wrong TFM.
My preferred target (highest TFM version) is usually listed last in my <TargetFrameworks> element. Would you recommend that I start reversing that order, or would it make sense to have HtmlGenerator use the last/highest TFM when it can figure that out instead of being random?

I think right now it takes the first TFM and I’d think the first TFM is the “primary” one, but I might be wrong. I suppose we could accept a TargetFrameworkPrecedenceList that lists them in order...

This is probably not the only tool that will assume that the first TFM has some special significance. (The only other one I'm aware of is the Windows Forms designer.) I'm okay with reordering all my TFMs.
Tackling indexing all TFMs and taking TFMs into account when tracking references is something I'd be more interested in working on than TargetFrameworkPrecedenceList.

If you want a future looking source indexing solution, there's a new one slow-cooking over at https://github.com/ref12/codex. But right now it comes with absolutely zero support, zero guarantees, it won't work, is broken, not worth looking into, not ready for adoption, etc. etc.


[maintenance] Update meta.yaml to reflect code ownership change

Description
This file looks extremely out of date, change to this PR are highly recommended from reviewers.
Changes proposed in this pull request:

update package versions
change owners to reflect CODEOWNERS for conda-recipe

/intelci: run

It's fine but file is not used, so might be right thing would be to remove conda-recipe completely from repo. @Alexsandruss  - what are your thoughts?


Revert "Skip security signals listing tests"

Reverts DataDog/datadog-api-client-java#415
Let's re-enable these tests so we can better understand why they were failing in the first time.

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run


Remove batch size from adapter, infer during inference, and propagate…

Draft for

removing the dependence on batch_size in adapters;
removing the need for the user to specify batch_size and hence inferring batch_size dynamically after adapting the data;
propagating the stage argument through the adapter and controlling the behavior of EMA etc.

The trick was to re-implement broadcast to infer the broadcast dim from another existing array, which will always be the case. Examples:
# Mock simulator with realistic shapes - broadcasting number of observations
def sim(batch_size):
    return dict(
        N=10,
        x=np.random.randn(batch_size, 10, 2),
        theta=np.random.randn(batch_size, 2)
    )

# Default behavior - will only broadcast to shape[:-1]
adapter = bf.adapters.Adapter().broadcast("N", to="theta", batch_dims_only=True)
print(adapter(sim(3))["N"].shape)
# (3, 1)

# Broadcast completely to shape
adapter = bf.adapters.Adapter().broadcast("N", to="theta", batch_dims_only=False)
print(adapter(sim(3))["N"].shape)
# (3, 2)

# Can also easily broadcast non-scalar to the left (along batch_dim)
def sim(batch_size):
    return dict(
        N=np.random.randn(2),
        theta=np.random.randn(batch_size, 2)
    )
adapter = bf.adapters.Adapter().broadcast("N", to="theta", batch_dims_only=False)
print(adapter(sim(3))["N"].shape)
# (3, 2)

# This will be undefined! Shape of N must be (batch_size, 1) to broadcast correctly
def sim(batch_size):
    return dict(
        N=np.random.randn(batch_size),
        theta=np.random.randn(batch_size, 2)
    )
adapter = bf.adapters.Adapter().broadcast("N", to="theta", batch_dims_only=False)
print(adapter(sim(3))["N"].shape)
# ValueError: operands could not be broadcast together

@paul-buerkner @LarsKue Naming of arguments can be improved. Currently, this broadcast has no inverse, but it is possible to implement by tracking the old shapes during forward pass.

Looks good I think.
About the batch_dims_only argument. Since we currently implement it as "everything but the last dim", would it perhaps make more sense to rename it to last_dim with last_dim = False equal to batch_dims_only = True. I still struggle seeing everything else than the first dimension as a batch_dim. Happy to be convinced of course.
One question I have about approximator.sample. I take it we now have to wrap data as [variable], as in (from the linear regression notebook):
conditions = {"y": [lik["y"]], "x": [lik["x"]], "N": [N]}

vs. previously the following worked (with batch_size = 1 as an added argument):
conditions = {"y": lik["y"], "x": lik["x"], "N": N}

I think it is fine to require [variable] for single dataset inference. However, the error one sees otherwsie is quite uninformative. I see:
File ~/Dropbox/Software/bayesflow/BayesFlow/examples/../bayesflow/approximators/continuous_approximator.py:167, in ContinuousApproximator._sample(self, num_samples, inference_conditions, summary_variables)
    [165](https://file+.vscode-resource.vscode-cdn.net/Users/paul-buerkner/Dropbox/Software/bayesflow/BayesFlow/examples/~/Dropbox/Software/bayesflow/BayesFlow/bayesflow/approximators/continuous_approximator.py:165)         inference_conditions = summary_outputs
    [166](https://file+.vscode-resource.vscode-cdn.net/Users/paul-buerkner/Dropbox/Software/bayesflow/BayesFlow/examples/~/Dropbox/Software/bayesflow/BayesFlow/bayesflow/approximators/continuous_approximator.py:166)     else:
--> [167](https://file+.vscode-resource.vscode-cdn.net/Users/paul-buerkner/Dropbox/Software/bayesflow/BayesFlow/examples/~/Dropbox/Software/bayesflow/BayesFlow/bayesflow/approximators/continuous_approximator.py:167)         inference_conditions = keras.ops.concatenate([inference_conditions, summary_outputs], axis=1)
    [169](https://file+.vscode-resource.vscode-cdn.net/Users/paul-buerkner/Dropbox/Software/bayesflow/BayesFlow/examples/~/Dropbox/Software/bayesflow/BayesFlow/bayesflow/approximators/continuous_approximator.py:169) if inference_conditions is not None:
    [170](https://file+.vscode-resource.vscode-cdn.net/Users/paul-buerkner/Dropbox/Software/bayesflow/BayesFlow/examples/~/Dropbox/Software/bayesflow/BayesFlow/bayesflow/approximators/continuous_approximator.py:170)     # conditions must always have shape (batch_size, dims)
    [171](https://file+.vscode-resource.vscode-cdn.net/Users/paul-buerkner/Dropbox/Software/bayesflow/BayesFlow/examples/~/Dropbox/Software/bayesflow/BayesFlow/bayesflow/approximators/continuous_approximator.py:171)     batch_size = keras.ops.shape(inference_conditions)[0]

File /opt/miniconda3/envs/bayesflow/lib/python3.11/site-packages/keras/src/ops/numpy.py:1352, in concatenate(xs, axis)
   [1350](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/bayesflow/lib/python3.11/site-packages/keras/src/ops/numpy.py:1350) if any_symbolic_tensors(xs):
   [1351](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/bayesflow/lib/python3.11/site-packages/keras/src/ops/numpy.py:1351)     return Concatenate(axis=axis).symbolic_call(xs)
...
--> [389](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/bayesflow/lib/python3.11/site-packages/jax/_src/util.py:389)   raise ValueError(f"axis {axis} is out of bounds for array of dimension {num_dims}")
    [390](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/bayesflow/lib/python3.11/site-packages/jax/_src/util.py:390) if axis < 0:
    [391](https://file+.vscode-resource.vscode-cdn.net/opt/miniconda3/envs/bayesflow/lib/python3.11/site-packages/jax/_src/util.py:391)   axis = axis + num_dims

ValueError: axis 1 is out of bounds for array of dimension 1

Do think this error is informative enough to point users in the right direction?

Tests are currently not passing. Could you update this @stefanradev93?
I am also not a huge fan of this change in general. When designing, I explicitly chose to let the batch size be manually supplied from the outside since it is fickle to infer it. It should also always be known when the adapter is called, with the exception being that users apparently do not want to supply a sample size when calling the sample method of the continuous approximator.
Even for that case, I think it would be much more robust to expect the batch size to be passed, unless the user explicitly makes it optional by providing the name of another object that it can be inferred from. I.e. this change should be optional.
Do note that I already have an idea for a better solution: explicitly defined variables which include meta-data (such as which axis is the batch axis).

From a dev perspective, it may be more robust to expect the user to pass the batch_size. But from a user perspective, this makes for a worse user-interface. That is why Stefan and I are both in favor of this more automatic (albeit perhaps not 110% robust) solution.
Your alternative sounds also good potentially, but I would have to see the details and user-interface of this. What is more, I consider these two not mutually exclusive. One might take the batch_size from the simulator if it is specified explicitly there. And if not, infer it as in Stefans proposal.

I think I now have a general enough version of broadcast implemented. It may not yet be pretty, but perhaps we can confirm first if it has all the funcationality we want and need.
broadcast now has two new arguments, expand and exclude:

expand informs where new dimensions should be added to x to match the number of dimensions in to. Can be "left", "right", or an integer or tuple containing the indices of the new dimensions. The latter is need if we want to include a dimension in the middle, which will be required for more advanced cases. By default we expand left.

exclude informs which dimensions (of the dimensions after expansion) should retain their size, rather than being broadcasted to the corresponding dimension size of to. By default we exclude the last dimension (usually the "data" dimension) from broadcasting the size.

@stefanradev93 would you mind checking my implementation and verfying that this covers all our use cases?


✨ Review use of SQLGlot as SQL Rewriter

CF #764
SQLGlot includes a number of heuristic-based optimizations, investigate using SQLGlot to do an initial parse and optimize of provided statements and then output the resultant SQL for parsing in sqlparser-rs.
Some gotchas anticipated are:

Handling of parameter substitutions
Non-standard syntax handling
Some optimizations may not be suitable for this specific use-case (e.g. the way selection and projection pushdowns work)
non SELECT queries

These are either not mentioned in the documentation reviewed so far or appear opinionated in the documentation.
Some advantages are:

We can push some optimizations written in Opteryx but aren't in SQLGlot (e.g. negative eliminations) and be good open source citizens
New optimizations will be added by a larger community rather than all hand-coded by Opteryx developers

First step is to add SQLGlot as an impotent step in the execution flow (get SG to parse everything in the regression suite and see what happens).
Depending on the outcome, either

back out
try parsing back to raw SQL and running through
submit fixes to SG

SQLGlot Parsing Failures
SELECT * FROM $satellites WHERE `name` = 'Calypso'

SELECT * FROM `$satellites` WHERE name = 'Calypso'

SELECT * FROM $satellites WITH (NO_CACHE)

SELECT * FROM $satellites WHERE name ~ '^C.'

SELECT * FROM $satellites WHERE name !~ '^C.'

SELECT * FROM $satellites WHERE name ~* '^c.'

SELECT * FROM $satellites WHERE name !~* '^c.'

SELECT * FROM UNNEST(('foo', 'bar', 'baz', 'qux', 'corge', 'garply', 'waldo', 'fred')) WHERE unnest LIKE '%e%'

SELECT GREATEST ( LIST ( name ) ) as NAMES FROM $satellites GROUP BY planetId

SELECT LEAST ( LIST ( name ) ) as NAMES FROM $satellites GROUP BY planetId

SELECT INTERVAL '1 1' DAY TO HOUR

SELECT * FROM $astronauts WHERE 'Apollo 11' IN missions

SELECT * EXCEPT id FROM $satellites

it's easy to fix the $ by mapping it to a parameter, you can also do that in your own custom dialect.
class CustomDialect(Dialect):
Tokenizer(Tokenizer):
KEYWORDS = {
"$": TokenType.PARAMETER
...

this doesn't fail for me
SELECT
  *
FROM "testdata"."nulls"
WHERE
  "tweet" ILIKE '%TRUMP%'

it should be easy to fix all of these parsing errors in your own custom dialect, i can help you with that if you want, we can chat in my slack
https://join.slack.com/t/tobiko-data/shared_invite/zt-1ma66d79v-a4dbf4DUpLAQJ8ptQrJygg

Thanks @tobymao I've started to write a custom dialect (mainly cribbed from the postgres dialect) and now the biggest set of parsing issues relate to hints (the WITH clause following relation names).

It looks like this may not be feasible to achieve the desired benefit of using it as a SQL rewriter / heuristic optimizer; it appears SQLGlot requires schema information at the Optimization step; however, Opteryx (will) have the schema at the Bind step (which is before its Optimization step) and in ad hoc mode (e.g. CLI querying files) no schema is expected to be available until execution has started.

why can't you just optimize right before execution? you need to understand the schema during execution

Hi Toby, that is the current model - I have an existing parser, lexer, binder, planner, optimizer and executor, I would need to refactor the entire stack to use SQLGlot that late in the process. I was investigating inserting it at the front of the existing chain to perform some initial optimization, convert the result back to SQL and pass it through the existing chain, which I've heard referred to as a SQL Rewriter.
For most queries, there's no schema information until the first data file has been read, for example, if you're running a query against a CSV on the command line. This affects what optimizations are possible (e.g. currently only push predicates in single-table queries).
I really like some of the features in your project though and will look to adopt them - the way you've laid out your examples in the README and how the predicate and projection push-downs are pushed to sub-queries - I expect this will be a huge benefit when querying external SQL-aware data sources.

Thinking about this and why it's so structurally challenging, it has become clear I've not implemented my planning "correctly". Rather than create a logical plan, I jump straight to creating a physical plan.
If I refactor to separate these activities, replacing the parser to SQLGlot should be (mostly) limited to one/two step in the query engine - the binder and the cost-based analyzer.
I'm splitting out these phases because I think it will simply each step anyway, using SQLGlot may reduce some of the refactoring effort so will be looked at as part of this work.

great!

WITH statements appear to be supported now.
outstanding issues appear to be:

generate_series with CIDR ranges aren't supported as they only have one parameter - this is non-standard, fix will be to do this expansion in a different function/type. Postgres appears to have an inet type which can be an IP or CIDR
Interval types (variations of the below)

SELECT INTERVAL '5 6' YEAR TO MONTH
❌Invalid expression / Unexpected token. Line 1, Col: 31.

There are other issues, but these appear to be minor bugs

INET type added to SQLGlot, now need to refactor code for this case in Opteryx.


Additional Functions on IconOptions and on WMSparameters

Add missing CQL_FILTER filter parameter for geoserver in WMSParameter
Give the possibility to set the anchor position

Hello @ikzent ,
thanks for your contribution. Have a look at the remarks in the review. Let me know if I can help!

Looks good so far. Could you please remove the whitespaces in the two marked lines?

Thanks for your efforts!


peer dependency on inquirer missing

Describe the bug
<EMAIL_ADDRESS>installation is warning about a peer dependency that is missing:

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^6.0.0 but none is installed. You must install peer dependencies yourself.

Which makes `npm list -g to throw an error message:

npm ERR! peer dep missing<EMAIL_ADDRESS>|| ^6.0.0, required by<EMAIL_ADDRESS>
To Reproduce
Steps to reproduce the behavior:

Install @aws-amplify/cli by `npm install -g @aws-amplify/cli
See the warning message after the installation

Expected behavior
aws-amplify/cli should be able to install itself without generating any warning and/or error messages.
Desktop (please complete the following information):

OS: macOS 10.13.6
Versions:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Any luck with this issue? I've run into the same problem. It won't display the 'type of app' or let me configure amplify.
OS: mac 10.13.6
node: 11.0.0
yarn: 1.10.1
amplify/cli: 0.1.27, 0.1.29
amplify-react-native: 2.0.8
amplify: 1.1.8
I got the same warning using 1.29. When I rolled back to 1.27 I did not get the error but I still could not get the app init'd. Tried deleting node_mods, clearing yarn cache, xcode cache

@NYCSwan Could you please provide the error stack?

@gjmveloso I don't see this issue with the latest version of the CLI (0.1.38). Please feel free to re-open or comment on this thread if you're still seeing this issue.


Allocator.h:239:7: runtime error: null pointer returned from function declared to never return null

To reproduce run
UBSAN_OPTIONS=halt_on_error=true ~/swift-latest/usr/bin/swift test --sanitize=undefined --filter testObjectiveCTestsAfterInMemoryEdit

Output
Test Case '-[SourceKitLSPTests.DocumentTestDiscoveryTests testObjectiveCTestsAfterInMemoryEdit]' started.
/Users/alex/src/sourcekit-lsp/.command-line-build/checkouts/indexstore-db/lib/LLVMSupport/include/llvm/Support/Allocator.h:239:7: runtime error: null pointer returned from function declared to never return null
/Users/alex/src/sourcekit-lsp/.command-line-build/checkouts/indexstore-db/lib/LLVMSupport/include/llvm/Support/Allocator.h:213:3: note: returns_nonnull attribute specified here
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /Users/alex/src/sourcekit-lsp/.command-line-build/checkouts/indexstore-db/lib/LLVMSupport/include/llvm/Support/Allocator.h:239:7 in
error: Exited with unexpected signal code 6

Synced to Apple’s issue tracker as rdar://129444472


[V3] Translation Service: fix everything around this exercise #1071

Reviewers, please spell/grammar check
Related #1017
Changes

Rename from concept based exercise (promises) to story based exercise translation-service
Add full name to config.json
~Add blurb for promises to config.json~
~Add design document~
Sync concept documents with exercise documents
~Add concept links~
~Add about.md for promises concept~
Fix eslint for typescript files

I'm going to merge this to make the CI run again. A new issue will be created to resolve the TODOs


[bug] name is deprecated

💻 Code

🙂 Expected behavior

No warning.

This was an intended change, I believe. See related discussion: https://github.com/microsoft/TypeScript/issues/18433

This was an intended change, I believe. See related discussion: https://github.com/microsoft/TypeScript/issues/18433

@AviVahl It's actually easier to trigger... please see the newly uploaded picture.

@AviVahl It's actually easier to trigger... please see the newly uploaded picture.

That's a different behavior, where a .ts file without any esm export/import statements is treated as a script. if you add export {} it would treat it as a module. Not to say that this is ideal behavior or anything.

That's a different behavior, where a .ts file without any esm export/import statements is treated as a script. if you add export {} it would treat it as a module. Not to say that this is ideal behavior or anything.

Eventually, I assume we'll be able to look at an .mjs file and say "that's a module", but until then, this is an external behavior.

Eventually, I assume we'll be able to look at an .mjs file and say "that's a module", but until then, this is an external behavior.

@DanielRosenwasser Just to add more info, .js is the same too:

@DanielRosenwasser Just to add more info, .js is the same too:


Added a simple systemd unit example.

Signed-off-by: Valentin Bud<EMAIL_ADDRESS>

I would be in favor of adding something like this.


Fix code coverage filters

What needs fixing in terms of code coverage filters?

Looks like something wrong with configuration, no data is displayed: https://codecov.io/gh/unitycontainer/container


Scaling Sampler by using row level probability

Currently Sampler component requires all data for a class to be loaded into memory in order to do sampling. This unfortunately does not scale well when we have larger dataset which may have 1000s of records per class.
Instead as suggested by @rclough I tried to implement a row level probability where each record is either augmented (oversampling) or deleted (undersampling) based on the count of records for its class vs the goal count for each class.
This leads to less "correctness" as we run random for each row and therefore it's less predictable. However for large datasets it should be correct "enough". It's a tradeoff.
Collaboration with @rclough
Please review deeply to make sure it makes sense as I have less context on this component.
Also fixed up a bit the README and made sure to add the missing proposal file for tracking.

/lgtm

/lgtm

/merge


Use published socket.io without native dependencies

Fixes https://github.com/airportyh/testem/issues/659 https://github.com/airportyh/testem/issues/660

Working on linking the pure packages together.

engine.io-client-pure released

Sorry, but for some reason testem websocket connections aren't working when using the pure version. I'll revert the git dependencies for now and look into that tomorrow again.
WDYT?

@johanneswuerbach keep me posted. I'll take a look tonight aswell

After npm link-ing all pure packages locally and running make build in both clients I still see Uncaught Error: Cannot find module 'buffer' from socket.io.js when opening a browser with testem.

:tada: now everything passes. Sorry about the versioning mess, I'll clean this up tmrw.


Prune metrics: AUC & ROC 5/n

What does this PR do?
prune classes and functional AUC and AUROC
follow-up #6505
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

Hello @Borda! Thanks for updating this PR.

In the file pytorch_lightning/metrics/classification/roc.py:

Line 37:12: W292 no newline at end of file


How to import container to a different storage?

My server on btrfs has broken and now I setup a new one on zfs. Now I want to restore the backups of my containers to the new server. How to do it properly? Perhaps I can just rsync rootfs between the uncompressed backup and the newly created blank container?
The documentation does not mention it, and doing lxc import <container_backup.tar.xz> complains about incompatible storages.

AFAIK you have to edit backup.yml and change the driver option in the pool section (eventually the source path as well if this differs):
pool:
  config:
    source: /var/lib/lxd/storage-pools/default
  description: ""
  name: default
  driver: dir
  used_by: []
  status: Created
  locations:
  - none

rsync the container to `/var/lib/lxd/storage-pools/default/containers/<your_container>
create a symbolic link ln -s /var/lib/lxd/storage-pools/default/containers/<your_container>  /var/lib/lxd/containers/<your_container>
lxd import <your_container> && lxc start <your_container>
But perhaps there is a better way to achieve that.

Just use lxc import -s POOL as listed in the help:
Description:
  Import backups of containers including their snapshots.

Usage:
  lxc import [<remote>:] <backup file> [flags]

Examples:
  lxc import backup0.tar.gz
      Create a new container using backup0.tar.gz as the source.

Flags:
  -s, --storage   Storage pool name

Global Flags:
      --debug            Show all debug messages
      --force-local      Force using the local unix socket
  -h, --help             Print help
      --project string   Override the source project
  -q, --quiet            Don't show progress information
  -v, --verbose          Show all information messages
      --version          Print version number

Oh, actually, wrong answer now that I'm re-reading your question.
It looks like your problem is that your lxc export used --optimized-storage and so you can't re-import this in anything other than a zfs pool as the backup contains zfs binary data.
You could post tar Jtvf backup.tar.xz to confirm this.
If that was indeed the case, you do need a zfs storage pool to import that container, once imported, you can then move it to another pool of another type.


[docs] Update creating-coins.md

Explicitly stating that we must re-publish our module after every change will save the reader a few minutes.

/land


Possibly make some application methods chainable

Hey very nice app architecture framework.  Where I work we have a need to use progressive enhancement quite a bit and I found most other app architecture frameworks didn't make it easy to serve HTML from the server.  We just needed something simple to help organize our JavaScript.  This looks to fit the bill.
On to my question.  We'd like to lazy-load not only the module HTML but also the JavaScript file associated with it.
From the few tests I've done it looks to be possible, we'd just have to make sure each T3 component starts itself rather than using the Box.Application.init method (at least I think that'll work.. if there's a cleaner way... I'm all ears).
With that said, would it be possible to make Box.Application.addModule, addService, etc. return an instance of the application instead of undefined.  This way I could do something like the following:
Box.Application.addModule('some-module', function(context) {

    'use strict';

    return {

        init: function() {

	},

	destroy: function() {

        }
    };

}).start(document.getElementById('some-module'));

This is really more of a stylistic preference but would be nice to have.

Here's some info on dynamically loading modules:
http://t3js.org/docs/guides/dynamic-loading
I don't see any reason we can't add chaining. @j3tan thoughts?

Hey thanks!


Refactor Thingiverse to use ProviderDataIngester

Suggested Improvement

Refactor the Thingiverse provider script to use the new ProviderDataIngester base class
Benefit

More details in #229
Implementation

[ ] 🙋 I would be interested in implementing this feature.

I've posted about next steps for this refactor on the Make WP Blog.

Closing this in favor of #820, see the Make WP post linked above for more context.


[FIX] generateResourcesJson: Include dependencies of XML resources

The XML analysis requires dependency resources to be able to list them
as "required" in the resources.json.

Tests are missing

Coverage increased (+0.005%) to 93.842% when pulling 89b792f41f181b6b7b683f9687f732de19335ba0 on resourceListCreator-fix-missing-XML-dependencies into b0a1875d81296412d90e2b172f51d67614f374b9 on master.

CI is broken since https://github.com/SAP/ui5-builder/commit/61e2e3c43f52eaee7c51c1298c08a07ccc656058

CI should be solved via https://github.com/SAP/ui5-builder/pull/587


Error "Homebridge not initialized." - Where does this log message come from?

Everytime I add my only camera to your Homebridge Plugin camera.ui, the log message [CameraUI] [Cam name]: Homebridge not initialized. appears in Homebridge log.
It doesn't matter if I add the Camera through camera.ui or directly through Homebridge plugin settings.
Without adding a camera the message didn't appear.
Curiously, everything works fine without any problems?!?
[2/14/2022, 8:36:26 AM] Homebridge v1.4.0 (HAP v0.10.0) (Homebridge Cams) is running on port 51587.
[2/14/2022, 8:36:26 AM] [CameraUI] Haustür: Configuring unbridged accessory...
[2/14/2022, 8:36:26 AM] [CameraUI] Haustür: Setting up accessory...
[2/14/2022, 8:36:27 AM] [CameraUI] Accessories refreshed and config.json saved!
[2/14/2022, 8:36:27 AM] Haustür D969 is running on port 44935.
[2/14/2022, 8:36:27 AM] Please add [Haustür D969] manually in Home app. Setup Code: 031-45-154
[2/14/2022, 8:36:35 AM] [CameraUI] Haustür: Setting up camera, please be patient...
[2/14/2022, 8:36:35 AM] [CameraUI] MQTT: New message: Topic: cameras/motion - Data: Haustuer_MotionStop - Type: string
[2/14/2022, 8:36:35 AM] [CameraUI] Haustür: Homebridge not initialized.
[2/14/2022, 8:36:37 AM] [CameraUI] Haustür: Prebuffering scheduled for restart at 2AM: 1043 minutes
[2/14/2022, 8:36:37 AM] [CameraUI] camera.ui v5.0.16 is listening on port 8081 (http)

That error also appears in the dashboard of camera.ui:

Environment

Node.js Version: v16.14.0
Homebridge Version: v1.4.0
Camera UI Version: v5.0.16
Operating System: Raspberry Pi OS 32bit on Raspberry Pi 4
Browser: Latest Chromium 98.0.4758.82

I completely re-installed with a fresh Homebridge Docker container and without copying any configuration made prior to the reinstall.
I didn't found any similar issues within this GitHub project and nowhere else.
Maybe someone can give me a hint what can cause this?

The information comes because camera.ui gets a motion message via MQTT although camera.ui has not finished loading. the message disappears as soon as camera.ui has finished loading (camera.ui v5.0.16 is listening in port 8081 (http))


no 'val' folder in new_orders.zip

Hi,
I tried to launch training with
python train.py ./scannet --dataset scannet --mindepth 0.25
The error is:
"scannet/new_orders/val/valnew_orders_v.list not found"
Then I found out new_orders folder only contains train/test folder

Hi.
I went through this and there seems to be some confusion created by me. I apologize for this.
Answering your other question first, I see and I will upload the test directory. Btw, you can ignore the .list files in the scannet folder. They serve no purpose.
The test split we use has all its files in the val folder. So if you just need to train and test, the train, val folders in scannet are sufficient. For the val split, I need to upload the orders. Instead I can just upload the .pkl file here for your use before I make these changes more concretely in the repo.(). But these val list I uploaded actually has its files in the test folder of scannet which I need to upload first.
We made our train/test/val splits first and later had to adapt to a previous work's splits causing this confusion. Again, if you just need to train to some fixed epochs and test, you can do it now without any new uploads from me.
Once again, I apologize for the confusion.


Destroy VM isn't working with BRIDGE_LIST and UNDEPLOYED VMs

Description
When I removing undeployed VM, I have the next error:
[Z0][VM][I]: New state is ACTIVE
[Z0][VM][I]: New LCM state is EPILOG
[Z0][TM][I]: Command execution failed (exit code: 255): /var/lib/one/remotes/tm/fs_lvm/delete opennebula-oned-7789d84587-6n654:/var/lib/one//datastores/189/7273/disk.0 7273 100

Where opennebula-oned-7789d84587-6n654 is my oned controller hostname.
But my system datastore 189 have BRIDGE_LIST="node1 node2" variable.
To Reproduce
I have two datastores with fs_lvm:
Images one:
NAME="Images Datastore"
ALLOW_ORPHANS="NO"
BRIDGE_LIST="node1 node2 node3 node4"
CLONE_TARGET="SYSTEM"
DRIVER="raw"
DS_MAD="fs"
DS_MIGRATE="YES"
LN_TARGET="SYSTEM"
RESTRICTED_DIRS="/"
SAFE_DIRS="/var/tmp"
TM_MAD="fs_lvm"
TYPE="IMAGE_DS"

System one:
NAME="System Datastore"
ALLOW_ORPHANS="NO"
BRIDGE_LIST="node1 node2"
DISK_TYPE="FILE"
DS_MIGRATE="YES"
RESTRICTED_DIRS="/"
SAFE_DIRS="/var/tmp"
SHARED="YES"
TM_MAD="fs_lvm"
TYPE="SYSTEM_DS"

Then I create VM from it. Then I do undeploy it hard, and then terminate this VM.
Expected behavior
VM successful removed.
Details

Affected Component: Core
Hypervisor: KVM
Version: 5.6.1

Additional context
Add any other context about the problem here.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

@rsmontero please reopen the bug report the flowing patch does not solves problem fully. Sometimes opennebula can call mv script with whole directory target (like 5.8 do):
[Z0][TM][I]: Command execution failed (exit code: 255): /var/lib/one/remotes/tm/fs_lvm/delete opennebula-oned-0:/var/lib/one//datastores/103/64 64 103

In this case:
https://github.com/OpenNebula/one/blob/5af6dba3dde51e88bdf7251c17b99489a7c23557/src/tm_mad/fs_lvm/delete#L44-L47
Variable DS_SYS_ID will be empty, and DST_HOST will not be set:
https://github.com/OpenNebula/one/blob/5af6dba3dde51e88bdf7251c17b99489a7c23557/src/tm_mad/fs_lvm/delete#L83-L94

The second problem is that in this case, even if we will have right host, delete operation will just remove whole directory with the vm, but not destroy vm volumes

Link: https://github.com/OpenNebula/one/issues/2352

Any way this bug was produced from #2352, so we can close it as duplicate.
Sorry for the mess.


The same input, the original model and the compiled model have different outputs

temperature is 0 .1 ,  same input, the original model and the compiled model have different outputs . After several rounds of dialogue, no matter how I ask, the compiled model outputs the same content，But the original model wouldn't .

Can you explain what's the original model and what's the compiled model?
If the original model means the HF and the the compiled model means the TRT LLM, they are expected to be different when you enable sampling. And if you want to get different results on TRT LLM, please setup different random seed.

Close this bug because it is inactivated. Please reopen it if needed.


Pass ValidNavigationMesh to Islands as Arcs.

It sucks that we have to use Arcs, but this is an easy way to allow us to compare nav meshes, and allows callers to keep their own copy to check for deltas.

Codecov Report

Merging #15 (52543c2) into main (1b09b00) will decrease coverage by 0.2%.
The diff coverage is 87.5%.

Impacted Files
Coverage Δ

src/island.rs
82.3% <14.2%> (-17.7%)
:arrow_down:

src/agent.rs
100.0% <100.0%> (ø)

src/avoidance.rs
99.0% <100.0%> (ø)

src/lib.rs
98.1% <100.0%> (+<0.1%)
:arrow_up:

src/nav_data.rs
99.2% <100.0%> (+<0.1%)
:arrow_up:

src/path.rs
100.0% <100.0%> (ø)

src/pathfinding.rs
100.0% <100.0%> (ø)

... and 1 file with indirect coverage changes


Refactor radio.hpp/cpp

Main Changes

Renamed radio to radioEsp32Interface
Updated documentation

Associated Issues

Closes #117

Tests
It has not been tested on the quadcopter yet.

looks good I'll test it out soon before we merge


Add keyboard shortcuts to skip tracks

It'd be really nice if we could bind keys to skip tracks forward or back like right arrow for skip and left arrow for going back.

Might be something to consider in the future but other apps usually provide those capabilities.

@freedomtacos, I personally use Autohotkey to map key combinations to media keys (and many other things as well).
I have these lines in my ahk file to map Win+Space to Play/Pause, Win+Alt+Left to Previous, and Win+Alt+Right to Next.
#Space:: Media_Play_Pause
!#Right:: Media_Next
!#Left:: Media_Prev

Hello, I would like to recommend 3RVX. This is an HUD skinnable and allows to create media hotkeys.
I have dedicated keys in my keyboard and assigned them to it. Very useful and can change the in/decrease of volume per click.
Try it and then say something.


Custom my own context in DndProvider has an error tip

Describe the bug
When I pass a custom property "context" to  DndProvider component, I meet an error.
my code

error tips

and then I find, the rootElement has changed to be my custom context, so it is an error

Any update here? I also encoutered this problem.


#1097  重现

What version of Turborepo are you using?
latest
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Windows
Describe the Bug
This project: https://github.com/munafio/react-vue-turborepo-starter.git
You can repeat the question,
Expected Behavior
#1097
To Reproduce
#1097

Setting "turbo": "=1.2.3" can restore to normal

Hi, if this is an instance of the behavior described in #1097 can you provide the various pieces of information listed out in the final comment?

node v16.15.1
npm 8.19.2
pnpm 7.12.2
First clone the repo, then install the dependencies
npm install

after that, start the development server (apps running in parallel)
npm run dev

cmd return:

<EMAIL_ADDRESS>dev
turbo run dev --parallel

ERROR  failed to calculate global hash: error hashing files. make sure that git has been initialized failed to read git hash-object: exec: "git": executable file not found in %PATH%

Sorry, I'm a newcomer and don't know what to provide. If it's not for this problem, you can close it. Thank you

@dongyanghe
我們已經搵到呢個問題謝謝
We've already found the problem. Thank you!

@dongyanghe Could you make sure that you're using the latest turbo? (npm turbo version should output 1.5.4). I notice that the repo you linked has a checked in lockfile and turbo is at version 1.3.4.
The error message you got seems to be from an older version.


Remove Ant Commander Pro version 4.5

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest

Note: <path> is the name of the directory containing the manifest you're submitting.

Removed Ant Commander Pro version 4.5 as latest version is 4.6
Microsoft Reviewers: Open in CodeFlow

URL: https://www.antcommander.com/versions/AntCommanderPro-4.5.exe
Status Code: 200
(Automated message - build 861)


Fixes the bug that text may not render correctly.

This commit fixes the bug that text may not render correctly when glyph is not found in fallback fonts. This bug is hard to find when rendering western letters because almost every font includes all western letters. But when it comes to Chinese characters, it's easily that a glyph is not found in western fonts and fons__getGlyph() should return NULL until meeting a Chinese font.
Before:

After:

This case also caught up my attention when I did the initial code review, and I came up with the conclusion that not returning NULL is correct, but looks like I was wrong :) I think the function is currently too subtle so that all the cases would be visible.
Firstly, the function should not only return NULL in case of error, i.e. not enough memory. In case it does not find a glyph, it should return empty glyph, and cache that result.
Secondly, I don't like how the function calls itself.
Maybe something like this could work:
	FONSfont* renderFont = font;

	...

	// Try to find the glyph in fallback fonts.
	if (g == 0) {
		for (i = 0; i < font->nfallbacks; ++i) {
			FONSfont* fallbackFont = stash->fonts[font->fallbacks[i]];
			int fallbackIndex = fons__tt_getGlyphIndex(&fallbackFont->font, codepoint);
			if (fallbackIndex != 0) {
				g = fallbackIndex;
				renderFont = fallbackFont;
				break;
			}
		}
		// It is possible that we did not find a fallback glyph.
		// In that case the glyph index 'g' is 0, and we'll proceed below and cache empty glyph.
	}

	...
	scale = fons__tt_getPixelHeightScale(&renderFont->font, size);
	fons__tt_buildGlyphBitmap(&renderFont->font, g, size, scale, &advance, &lsb, &x0, &y0, &x1, &y1);

	...
	fons__tt_renderGlyphBitmap(&renderFont->font, dst, gw-pad*2,gh-pad*2, stash->params.width, scale,scale, g);

That way the miss gets cached earlier too.

@memononen Your solution indeed is much better. 👍  I've tested your code and it works well in my app. I also amended the commit, please let me know if something needs to modify.

Can you also check that the caching works in case of fallback font? When the cache is empty, it should hit the fallback codepath, after that it should always be in the cache.
Can you also test that with some non-existing glyph (i.e. some emoji with a font w/o emojis)?
Does the emoji font on the demo show properly?
https://github.com/memononen/nanovg/blob/master/example/demo.c#L863

Just updated the code and finished the tests. Caching works in case of fallback font. When hitting non-existing glyph, square is rendered as expected, and the emoji font on the demo shows without any problem. ;)

Great! Thanks for the fix and testing.


taxon name input cleanup

We need to clean up the input we receive in the taxon name text box. Delete any leading or trailing whitespace. Replace any internal whitespace (spaces or tabs) with a single space. Otherwise the name lookup will fail, as just happened to me when I left a trailing space on a genus name.

Works great.


RuntimeWarning: Degrees of freedom <= 0 for slice

I am encountering the following warning when using the Panoptic_Evaluator on a SemanticPair:
/home/josef/miniconda3/envs/panoptica/lib/python3.12/site-packages/numpy/core/_methods.py:206: RuntimeWarning: Degrees of freedom <= 0 for slice   ret = _var(a, axis=axis, dtype=dtype, out=out, ddof=ddof, /home/josef/miniconda3/envs/panoptica/lib/python3.12/site-packages/numpy/core/_methods.py:163: RuntimeWarning: invalid value encountered in divide   arrmean = um.true_divide(arrmean, div, out=arrmean, /home/josef/miniconda3/envs/panoptica/lib/python3.12/site-packages/numpy/core/_methods.py:198: RuntimeWarning: invalid value encountered in scalar divide   ret = ret.dtype.type(ret / rcount)
I am running panoptica 0.53 and Python 3.12.

fixed with https://github.com/BrainLesion/panoptica/releases/tag/v0.5.6


jsonschema: add filesystem loader for use with embed

Actually I take it back!
I was going to suggest this:
func SchemaLoader(fs fs.FS) func(string) gojsonschema.JSONLoader {
	js := jsonschema.NewFS(fs)

	return func(name string) gojsonschema.JSONLoader {
		return js.LoadJSONExt(name)
	}
}

But seeing how you've used it here:
//go:embed *.json
var fs embed.FS
var schema = jsonschema.NewFS(fs).LoadJSONExt

It is both generic/decoupled and perfectly concise! 👏 Definitely had not thought of setting the function on the variable


Rpsec types

When you run the tool, the rspecfile that is generated does not apply Rspec types (Models, Controller, etc..)
Something like
# +Department+

Class Department
...
end

Will produce:
describe Department do
...
end

With rspec types:
RSpec.describe Department, :type => :model do

Sorry for such a slow reply and thanks :heart:
It is hardcoded here:
https://github.com/Nedomas/zapata/blob/f847fd28196be0dc4c702bfb252fd60460e276f7/lib/zapata/rzpec/writer.rb#L50
We could make it work give different type depending if its a model or a controller.
Could make a pull request with such switch?
Thank you in any case :hamburger:

I can probably pop something out this weekend if someone else does not get to it and time permits.
The above should change the behavior depending on what the ruby file is describing (Model,View,Controller, Interactor, etc...)
I wanted to use the gem on an undocumented project I inherited. Thanks for the gem either way, it helped with a convoluted Model schema I worked with. I think that would make a killer feature for the future :)

Hey, Michael.
I'm running into it myself too. How did it go for you?

Sorry never got around to this feature. And I just got put on another huge project....
When I get free time, ill revisit.

Let me know if you're still interested in it.


docs: small, subtle text changes to addr DXA3U1 feedback

PR Checklist

[ ] Leverages formatting as provided by https://squidfunk.github.io/mkdocs-material/reference/abbreviations/
[ ] Have viewed and verified edits in rendered form ensuring proper formatting

What are you changing?

    modified:   docs/Docker/overview.md
    modified:   docs/General/loginToRPS.md

Anything the reviewer should know when reviewing this PR?

@kimberlyxdavis After you view my addtl. changes, feel free to merge it in.


Test loftr in aachen

Hi! I’m trying to use your great work to eval loftr in aachen.
But it seems like a huge work, my running state is shown below, It doesn't call the gpu to handle....
100%|██████████| 131615/131615 [00:01<00:00, 99454.57it/s]
0%|          | 0/102817 [00:00<?, ?it/s]
Load match file, existing matches 0
Start matching, total 102817 pairs...
0%|          | 3/102817 [01:27<831:08:45, 29.10s/it]
Is this a normal phenomenon? If not, could you tell me how to solve it?
my args is:
--gpu
0
--config
loftr
--colmap
F:/work/COLMAP-3.8-windows-cuda/COLMAP.bat
--benchmark_name
aachen
my config is as follow, only changed the path of ckpt.
default: &default
class: 'LoFTR'
ckpt: 'E:/work/image-matching-toolbox-main/pretrained/loftr/myout0.ckpt'
match_threshold: 0.2
imsize: -1
no_match_upscale: False
aachen:
<<: *default
match_threshold: 0.0 # Save all matches
pairs: ['pairs-db-covis20.txt', 'pairs-query-netvlad50.txt']
npts: 4096
imsize: 1024
qt_dthres: 4
qt_psize: 48
qt_unique: True
ransac_thres: [20]
sc_thres: 0.2 # Filtering during quantization
covis_cluster: True

Hi @autostudiolea,
I would suggest to debug it first with a pair of images, since the matcher is initialised in the same way. And in theory it automatically use the available GPU device: https://github.com/GrumpyZhou/image-matching-toolbox/blob/76a865780919ae87cabc050b4569cf0feb11007d/immatch/modules/base.py#L49 .

Thank you for your response! I had just configured the virtual environment incorrectly    T T
Now I can finish the task in a day


user12986714: Watch nihaojewelry.com

user12986714 requests the watch of the watch_keyword nihaojewelry\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

nihaojewelry\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


@Begin add styles, reading numfmtid

new branch if my changes to get numfmtid, learning GIT so don't mind me lol,
removed unused vector

@CEXT-Dan  any chance you would like to test my private development branch for the XLStyles implementation?

Branch is no longer private - I pushed it to https://github.com/troldal/OpenXLSX/compare/master...development-aral for everyone to test.

Hi @CEXT-Dan - could you have a look at the XLStyles implementation in the development-aral branch & let me know what functionality from your pull request is missing?
As far as I can tell, the only thing you implemented that I did not yet cover is interpretation of number formats to determine whether something is a date / currency / etc?
The number format functionality is mainly why I kept this pull request open, but there are so many deviations from the main branch in here, that I'd appreciate if you could open a new issue with the functionality that is still missing in the development branch, so that I can track it in a ticket dedicated to what is still "open".


Forward-porting #5450 to main

https://hibernate.atlassian.net/browse/HHH-15632
https://hibernate.atlassian.net/browse/HHH-15633
https://hibernate.atlassian.net/browse/HHH-15635
The second patch didn't apply cleanly, hopefully I didn't screw it up as I edited from commandline. Letting CI figure it out.

Thanks for porting :)

@beikov I see you marked JIRA resolved, but shouldn't these be set as fixed for version 6.2 too?

We agreed that there is no need for also setting 6.2.0 since 6.1.5 should imply that.

Really? That seems a big change to established processes and I don't think
it can work well.
With 6.2 already branched,  how does one distinguish a patch that has only
been pushed to 6.1 and forgot to be applied to main (like you had done) ?
Please ensure that the Jira changelogs are always consistent with the
content of the branch.
On Fri, 28 Oct 2022, 07:40 Christian Beikov, @.***>
wrote:

We agreed that there is no need for also setting 6.2.0 since 6.1.5 should
imply that.
—
Reply to this email directly, view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/5452#issuecomment-1294530121,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAKMTL4SGVAGHKEE672A63WFNYNNANCNFSM6AAAAAARPKP3NA
.
You are receiving this because you authored the thread.Message ID:
@.***>

It's unfortunate that we even maintain two branches for so long.

With 6.2 already branched,  how does one distinguish a patch that has only been pushed to 6.1 and forgot to be applied to main (like you had done) ?

We only resolve JIRA issues when we are done with them. I usually assign 6.1.x if I want it backported and only resolve it once I merged to both branches. IMO it doesn't matter how we do this, we can always forget to do something.

Please ensure that the Jira changelogs are always consistent with the content of the branch.

That's what we are trying :)


mayflower/trusty64-puppet3 no longer hosted at filedump.mayflower.de

Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'mayflower/trusty64-puppet3' could not be found. Attempting to find and install...
    default: Box Provider: virtualbox
    default: Box Version: >= 0
==> default: Loading metadata for box 'mayflower/trusty64-puppet3'
    default: URL: https://vagrantcloud.com/mayflower/trusty64-puppet3
==> default: Adding box 'mayflower/trusty64-puppet3' (v0.7.2) for provider: virtualbox
    default: Downloading: https://vagrantcloud.com/mayflower/boxes/trusty64-puppet3/versions/0.7.2/providers/virtualbox.box
    default: Download redirected to host: filedump.mayflower.de
    default: Download redirected to host: filedump.mayflower.de
    default: Download redirected to host: mayflower.de

virtualbox Externally hosted (filedump.mayflower.de) redirects to mayflower.de.

I got the same issue. Where we can get vagrant box file?

Synthesize needs to be updated to work with [at least] Ubuntu Xenial (16.04) and possibly Python 3.x. Trusty is EOL.

This has been resolved in the latest release.
https://github.com/obfuscurity/synthesize/releases/tag/v4.0.0


Website is down

As of 2022-07-01T01:05:15.817Z, https://geodle.me/ 404s. https://muhashi.github.io/geodle/ is served, but all the static resources it depends on (js, css) 404. Runs fine locally.

Thanks for letting me know so fast! Should be back up now. Turns out the deployment script I use removes custom domain settings which broke the site.


Override parameter type in swagger

Informations

Type
Version

Improvement
4.x

Description
I would like to set a type what I need in swagger API even if the parameter type is different from the swagger API type. My code converted path params from number to object by converters or the middlewares.
for example, source code:
    @PathParams('user') user: User

then output in swagger:
  "parameters": {"in":"path","name":"user","type":"object"}

what I need:
  "parameters": {"in":"path","name":"user","type":"number"}

I guess this issue relates to OpenApiParamsBuilder.
        pathParam = Object.assign({}, pathParam, param.store.get("baseParameter") || {}, {
          type: swaggerType(param.type)
        });

Thanks in advance.

Hi @akkunchoi ,
To do that:
 "parameters": {"in":"path","name":"user","type":"number"}

Just write that:
@PathParams('user') user: number

See you :)

Hi @Romakita. Thanks for your reply. But my explanation was missing.
In my case, the middleware loads a user entity from request.params.user before controller process because of the purpose to share loading process (by the way, Converter does not seem to support async loading).
So, path params type should be entity class, and API path params type should be number.
Here is part of my code.
@Middleware()
export class UserLoaderMiddleware implements IMiddleware {
  constructor(private userRepo: UserRepository) {
  }
  public async use(@Request() request: Express.Request) {
    request.params.user = await this.userRepo.findOneOrFail({id: request.params.user}).catch((err: any) => {
      throw new NotFound(err)
    })
  }
}

export class UserController {
  @Get("/:user")
  @UseBefore(UserLoaderMiddleware)
  show(@PathParams('user') user: User) {
  }
}

This may be a peculiar method. But I think that this is very smart.

I'd like to adopt your idea. Thanks!


⚠ Samsung has degraded performance

In 1d7c090, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2624 ms

Resolved: Samsung performance has improved in c11e484.


🐛 Unable to create auth token

No duplicates 🥲.

[X] I have searched for a similar issue in our bug tracker and didn't find any solutions.

Database
PostgreSQL
What happened?
Original issue https://github.com/spiral/cycle-bridge/issues/85
SQLSTATE[22P02]: Invalid text representation: 7 ERROR:  invalid input syntax for type bytea
CONTEXT:  unnamed portal parameter $5 = '...'

The payload contains an url. Upon json_encode, it will add backslashes to escape the uri e.g http:\/\/<IP_ADDRESS>:8080\/ which I guess somewhat invalid for bytea column. Removing the URL from the payload resolves the issue.
Version
Postgres 16.1
PHP 8.3

In PostgreSQL, the BYTEA data type is designed to store binary data, such as images, sounds, and other raw byte sequences. The BYTEA type is intended for representing data in binary form, and inserting strings with backslashes may cause issues due to the intricacies of byte handling and escaping in SQL.
If you attempt to insert a string with backslashes directly into a BYTEA column, it can lead to confusion, as the backslash () in SQL is used for escaping special characters. Therefore, when inserting a string with backslashes, they should be properly escaped.
Example of a problematic string:
INSERT INTO your_table (binary_column) VALUES ('some data with a slash \');

In this case, PostgreSQL might interpret the backslash as the beginning of an escaped sequence, leading to an error or misinterpretation of the data.
To avoid such situations, you can use the E'...' function, which allows the use of escape sequences in strings:
INSERT INTO your_table (binary_column) VALUES (E'some data with a slash \\');

In this example, E'...' enables the use of an escaped backslash in the string. Note that each backslash must be doubled to prevent interpretation errors.
Or check this function https://www.php.net/manual/en/function.pg-escape-bytea.php

Thank you. However, if you refer to the original issue, I do not have control over how the data is being serialized.
https://github.com/spiral/cycle-bridge/blob/e0bda2ca40c696ecd82de3d49fb32b40307bc63b/src/Auth/Token.php#L50


Error in link formatting for HTML links

Hi,
When I pass the seemingly valid string to Slack::Notifier:
"*On http://localhost:3001/*\n\n*David Cameron's membership has been rejected by Duncan Stuart*\n\n*Rejection Notes*\n\nUnsuccessful PM\n\n"
I get the following error:
expected no Exception, got #<NoMethodError: undefined method `[]' for nil:NilClass> with backtrace:
         # /Users/samuelstickland/.rvm/gems/ruby-2.5.0/gems/slack-notifier-2.3.2/lib/slack-notifier/util/link_formatter.rb:60:in `block in sub_html_links'

It works fine without any HTML links. If there something I am doing wrong or is this a bug?

Hmm.. There is something weird with my string encoding, since in the debugger copying and paste the string contents into itself (i.e. string = "....") allows the code to continue without an error

Although.. doing this in the debugger, inside sub_html_links, just before string.gsub(HTML_PATTERN) do
(byebug) string
"*On http://localhost:3001/*\n\n*ID*: 999\n*Severity*: Keytar farm-to-table iPhone keffiyeh mlkshk vegan blog Pitchfork 8-bit.\nKeytar vinyl twee tofu freegan banh mi.\n"
(byebug) string.gsub(HTML_PATTERN) { puts "#{Regexp.last_match[1]} - #{Regexp.last_match[2]}" }
*** NoMethodError Exception: undefined method `[]' for nil:NilClass

nil
(byebug) str2 = string.clone
"*On http://localhost:3001/*\n\n*ID*: 999\n*Severity*: Keytar farm-to-table iPhone keffiyeh mlkshk vegan blog Pitchfork 8-bit.\nKeytar vinyl twee tofu freegan banh mi.\n"
(byebug) str2.gsub(HTML_PATTERN) { puts "#{Regexp.last_match[1]} - #{Regexp.last_match[2]}" }
*** NoMethodError Exception: undefined method `[]' for nil:NilClass

nil
(byebug) str2 = string
"*On http://localhost:3001/*\n\n*ID*: 999\n*Severity*: Keytar farm-to-table iPhone keffiyeh mlkshk vegan blog Pitchfork 8-bit.\nKeytar vinyl twee tofu freegan banh mi.\n"
(byebug) str2.gsub(HTML_PATTERN) { puts "#{Regexp.last_match[1]} - #{Regexp.last_match[2]}" }
*** NoMethodError Exception: undefined method `[]' for nil:NilClass

nil
(byebug) str2 = "*On http://localhost:3001/*\n\n*ID*: 999\n*Severity*: Keytar farm-to-table iPhone keffiyeh mlkshk vegan blog Pitchfork 8-bit.\nKeytar vinyl twee tofu freegan banh mi.\n"
"*On http://localhost:3001/*\n\n*ID*: 999\n*Severity*: Keytar farm-to-table iPhone keffiyeh mlkshk vegan blog Pitchfork 8-bit.\nKeytar vinyl twee tofu freegan banh mi.\n"
(byebug) str2.gsub(HTML_PATTERN) { puts "#{Regexp.last_match[1]} - #{Regexp.last_match[2]}" }
http://localhost:3001/ - http://localhost:3001/
"*On *\n\n*ID*: 999\n*Severity*: Keytar farm-to-table iPhone keffiyeh mlkshk vegan blog Pitchfork 8-bit.\nKeytar vinyl twee tofu freegan banh mi.\n"
(byebug) str2 == string
true
(byebug) str2.bytes == string.bytes
true
(byebug) str2.encoding
#<Encoding:UTF-8>
(byebug) string.encoding
#<Encoding:UTF-8>

What on earth? I am completely stubbed

Rails 'bug' is here:
https://github.com/rails/rails/issues/3496
The problem seems to be that the Regexp global variables aren't truly global:

Their scope is controlled by ruby in a way that prevents AS::SafeBuffer from being able to provide gsub + block

You've dug in a bit more so I'd trust your judgement, but seems like working with a match object instead of Regexp.last_match would be a good fix? (I think I prefer the readability of that anyway)

Unfortunately I don't believe that will work:

In the block form, the current match string is passed in as a parameter, and variables such as $1, $2, $`, $&, and $' will be set appropriately. The value returned by the block will be substituted for the match on each call.

Which apparently means that you’ll always get a single argument denoting the entire regexp match, rather than all the matched groups as arguments.
For now we're doing this String.new(string) before passing the string to slack-notifier


[docs]: Operator retry is missing the description, and examples

Please check those that apply

[ ] typo
[ ] documentation doesn't exist
[ ] documentation needs clarification
[ ] error(s) in example
[x] needs example

Description Of The Issue
We might want to add an example here.

closed by #4691


Move instance out of sumcheck directory

Prover_instance and verifier_instance shouldn't be in sumcheck/

Closed by https://github.com/AztecProtocol/aztec-packages/pull/8383


Absorption correction for cylindrical samples

Executive summary
Calculate and apply absorption correction for cylindrical samples in Powder Diffraction
Context and background knowledge
For example, see algorithms in Mantid
AbsorptionCorrection
CylinderAbsorption
SNSPowderReduction in Mantid to check when the absorption correction is applied within the whole workflow
Inputs
Scipp data structure from loaded NeXus or csv file(s)
Assumption: the input should be expressed in wavelength
Note that information about the sample (chemical formula, geometry, mass density, number density...) might be stored in the sample log of the file or must have to be provided by the user when using the algorithm.
Methodology
See source code from (Mantid) references above
Assumption:  we consider only cylindrical samples for this first implementation
Acceptance criterion
Absorption correction step is implemented in ESS Diffraction worflow (https://scipp.github.io/ess/instruments/external/powgen/powgen.html)
Outputs
This requirement should provide the absorption correction term and the corrected sample data
Which interfaces are required?
Integrated into reduction workflow, Python module / function
Test cases

As input file, use POWGEN event file provided in Mantid (PG3_4866_event.nxs or PG3_4844_event.nxs) and already used in ESS Diffraction workflow.
In this case, the output from the implementation in Scipp could be compared with what we get from Mantid.
CSV files from DREAM (also available in Scipp documentation)

Comments
Followup Work: Extend this implementation to:

other sample shapes
several components (sample, container)
using other assumptions (like ‘FullPaalmanPings’)

Before implementing, we should consider if the actual correction should be part of ScippNeutron, since it is very generic. ESSdiffraction would then only contain the workflow integration, imported from ScippNeutron.

Removed from essentials milestone because it is unclear whether this should be done during reduction or analysis. Postponing implementation until we have clarity.


week3-server-comments-box

Added comments box to portfolio page.

Whoops, looks like I accidentally stayed on the same branch when I continued working. When I created a pull request, it failed, but IntelliJ still pushed the changes.
Is it alright if you guys just review the changes in the latest commit as well? Or is there a good way to reverse the pushing process?

We can just review the latest commit in this pull request.
I'm looking at the code, and I'm not seeing the change you said you made. Did you forget to add/push them?

Yeah, sorry. I forgot to commit and push my changes.

Hey, Megan. Just wanted to make sure that you looked over the changes made in commit 7823243 before merging.

Thanks for checking Josh. Changes look great :)


Bug: V2版本启动报错@import 'vxe-table/styles/all';

Version
Vben Admin V2
Describe the bug?

Reproduction

System Info
![1735372343074](https://github.com/user-attachments/assets/8d054f3f-c8a5-42d1-941c-f3617c5353df)

Relevant log output
No response
Validations

[X] Read the docs
[X] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[X] I have searched the existing issues and checked that my issue does not duplicate any existing issues.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

不要随意升级依赖


Multiple quiet login requests (different scopes) from an extension to a single AuthenticationProvider are indistinguishable on the Accounts menu

The two quiet logins are now accessible, but the screenshot reveals another issue, which is that the menu item prompts for the two different-scopes requests are identical. I will open a separate issue for this.

Originally posted by @gjsjohnmurray in https://github.com/microsoft/vscode/issues/137613#issuecomment-974840018

/assign @TylerLeonhardt

Perhaps AuthenticationGetSessionOptions could have a new hint?: string property which when passed would replace the cryptic digit 1 in parens at the end of the item text.
Arguably such a hint property could also be used to provide the additional detail text which a non-quiet forceNewSession reauthentication can now achieve by passing forceNewSession: "Why we need a new session". As implemented for #129064 (about to finalize) the extra text can only be added to the re-authenticating variant of the modal dialog. The hint property I am suggesting could also work for the authenticating variant and for the quiet login menu items.

I think we need this first?
https://github.com/microsoft/vscode/issues/124643

I don't think that's a dependency, but I agree that until we have it a situation with multiple quiet logins from the same extension to the same authprovider will misbehave.

True. And yes thanks for opening this one. Something I've wanted to do is something like one of these pages:

Perhaps what we can do is:

for createIfNone: true we can include the scopes in the modal that opens up
for createIfNone: false we can have clicking on the "Sign in" open a modal that displays the scopes as well (essentially showing the createIfNone: true modal

That way:

the experience is consistant
we're very clear to the user what they are being requested for

Later on it would be nice to allow an Auth Provider to provide human readable names for scopes... so for example in the Microsoft Authentication extension, you could ask for Task.ReadWrite but really this is the Microsoft To-Do (read & write) permission. This will help the users understand scopes that don't make sense.
So I like:

add a hint just like forceNewSession
display scopes being requested to the user

have a mechanism for making them user-readable

Any updates on this one? I have a use case which would benefit from the proposed additional features.

Just realized this is covered with other issues now.

add a hint just like forceNewSession

that's https://github.com/microsoft/vscode/issues/117358

display scopes being requested to the user

https://github.com/microsoft/vscode/issues/96609
however this issue has more details so I will dupe https://github.com/microsoft/vscode/issues/96609 to this.


examples/large_table: Fix deprecation warning

This fixes the cargo web start deprecation warning:
WARNING: `/js/app.js` is deprecated; you should update your HTML file to use `/large_table.js` instead!

nice :) thanks!


Fix link to the cirque test guide

Problem
Fix link to the cirque test guide
Change overview
Change link.
Testing
Checked the preview of the README.

@markus-becker-tridonic-com
/home/runner/work/connectedhomeip/connectedhomeip/docs/README.md:9: error: unexpected token TK_EOF as the argument of ref (warning treated as error, aborting now) 12551
Looks like a real failure in your edit

@markus-becker-tridonic-com /home/runner/work/connectedhomeip/connectedhomeip/docs/README.md:9: error: unexpected token TK_EOF as the argument of ref (warning treated as error, aborting now) 12551
Looks like a real failure in your edit

Similar to https://github.com/project-chip/connectedhomeip/pull/6862. Update of Doxygen in the containers?

@markus-becker-tridonic-com - looks like doxygen fails

Our workflow seems to just say: doxygen runs on ubuntu  20.04.
https://github.com/project-chip/connectedhomeip/blob/master/.github/workflows/doxygen.yaml#L31
Do we need to update doxygen? Can't we make it pass with the current version?

Our scripts try to  install doxygen as is. this is probably wasteful (takes time) and installs an old version as well:
https://github.com/project-chip/connectedhomeip/blob/master/scripts/helpers/doxygen.sh#L37
I believe it would be best to create a doxygen dockerhub image and then we have more control.

In 11529 I created a doxygen dockerhub image. Will build it and see about transitioning our workflows to using it.

/rebase

Doxygen passes now, however spellcheck  seems broken in ToT :(

#11540 will fix spellcheck

Doxygen change was  revertted because it does not seem to play nice with GH-pages updates. However this PR now will pass ... but unsure what will happen after merging. @woody-apple ?

/rebase


Open Source：Is the code for your method going to be open source these days?

After reading your paper, I am very interested in the methodology and would like to know when you will open-source it.

Thank you for your attention. But which work are you talking about, MANet? If so, I will open source this work in a week.

OK, I will update as soon as possible. If you are interested, please give me a star.


Use apt keys for installing AppOptics agent

This pulls the apt repo keys for the AO agent install (and removes --allow-unauthenticated).

Works great! Thanks @cce!


Changing name of script after using it in preload() generates "Could not resolve script" error, recoverable only by reloading project

Tested versions
v4.3.beta.custom_build [557f63d03]
Custom build for the precision=double flag
System information
Godot v4.3.beta (557f63d03) - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3050 Ti Laptop GPU (NVIDIA; <IP_ADDRESS>98) - AMD Ryzen 7 5800H with Radeon Graphics (16 Threads)
Issue description
Using a script's path in a const XXX := preload() function, and then renaming the script using Godot's "Rename" menu command, gives a "Error at (xx, yy): Could not resolve script "res://script_name.gd".
The error persists even after retyping the expression. Using "=" instead of ":=" also does not make the error go away. Renaming the script back and using the old name in preload() also does not resolve the issue.
It's undoable. Gives vibes of "Anton Chigurh" in "No Country for Old Men".
Steps to reproduce

Create any script. e.g. "script_a.gd".
In another script, load the first script into a const using preload(). e.g. const ScriptA := preload("res://script_a.gd").
Rename the first script's name. e.g. "script_b.gd".
Error appears in the second script.
Edit the name in the preload() function, rename the first script to original name, close and reopen the second script, and try a few other things, but the error persists.
Reload project, error gone.

Minimal reproduction project (MRP)
The steps above to reproduce the issue are sufficient to display the problem, and no MRP is needed.

I can reproduce the issue on Fedora 40 with 4.3.beta (5708a3a02e00061e03366f2dabf8942df66fedca).
After renaming the first script, and saving the one with preload, this error is printed, somewhat expectedly:
res://main.gd:3 - Parse Error: Preload file "res://script_a.gd" does not exist.
res://main.gd:3 - Parse Error: Cannot infer the type of "ScriptA" constant because the value doesn't have a set type.

At this step:

Edit the name in the preload() function, rename the first script to original name, close and reopen the second script, and try a few other things, but the error persists.

Editing the name in the preload() function to match the new name complains that it couldn't resolve res://script_b.gd.

res://main.gd:3 - Parse Error: Could not resolve script "res://script_b.gd".
res://main.gd:3 - Parse Error: Cannot infer the type of "ScriptA" constant because the value doesn't have a set type.

Renaming the first script to script_a.gd (while keeping that name in the preload call) also doesn't solve the issue, and prints these errors:

Attempt to open script 'res://script_b.gd' resulted in error 'File not found'.
res://main.gd:-1 - Compile Error:

Occasionally it does seem to resolve the error after printing this once.

In 4.2.2-stable, the issue seems similar, though I had to do explicit changes to the scripts to start catching errors, while in 4.3.beta just saving the script without modification was enough.


Support Self-Service Org Deletion

OC-11595 Manage: Self-Service Org Deletion

While this isn't available via manage, it is now available via chef-server-ctl.  I'm going to close this bug here as a manage feature should likely be tracked in other channels.


octo/reviews.lua:271: Expected lua string

Issue Description
Type: bug report or feature request
Describe what happened (or what feature you want)
starting a review gives
Error executing vim.schedule lua callback: ...aham/.config/nvim/plugged/octo.nvim/lua/octo/reviews.lua:271: Expected lua string
Describe what you expected to happen
to start the review like normal
How to reproduce it (as minimally and precisely as possible)

Octo pr list
Octo pr checkout
Octo review start

Tell us your environment
linux, ubuntu 20

Anything else we need to know?

Should be fixed by f0be84f


Missing style for ol/ul on rendered markdown

We probably need to add some margin around the left-side and both top and bottom. Maybe GitHub's style can help here?

Ah, I think we actually need some margin-top on h3 headers too. Check this out:

I don't expect it to be as large as the h2 headers but we probably need some amount.

I am working on this.


Clocker

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[ ✔] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[ ] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

Additionally, if adding a new cask:

[✔ ] Named the cask according to the token reference.
[✔ ] brew cask install {{cask_file}} worked successfully.
[✔ ] brew cask uninstall {{cask_file}} worked successfully.
[ ✔] Checked there are no open pull requests for the same cask.
[✔ ] Checked the cask was not already refused in closed issues.
[✔ ] Checked the cask is submitted to the correct repo.

Welcome, Dr. Doofenshmirtz, and thank you for the submission. In the future, please use x in checklists (no spaces inside the brackets) and Github will make them clickable.

Thanks for the quick review. I've added appcast and zap. Let me know if I'm missing anything.


test: avoid usage of Given/When/Then structure for trivial tests

The goal is to improve tests readability.

Finally, the change is aborted for the following reasons:

This change does not necessarily provide an improvement. The trade-off between shortening trivial tests or always sticking with a given template can be seen as 50/50.
A great argument from @Niiz9 suggests that always being consistent with the application of a pattern like here will implicitely encourages other developers to follow the same structure for the new tests. By relying on it only for a subset of tests, there is a risk of someone interpreting that as "there is not general rule to follow, I can do what I want"... Even if there could be a great reason of deviating from the general rule like here for improving readability.


⚠ RPC server - Harmony has degraded performance

In fab0332, RPC server - Harmony ($HARMONY_RPC_URL) experienced degraded performance:

HTTP code: 0
Response time: 0 ms

Resolved: RPC server - Harmony performance has improved in 5ec942b.


Invalid deployed release prevents upgrade

I have a successful DEPLOYED release which actually contains an error. The specific error is a DeploymentSpec without a selector field. Clearly at some point either the API or helm client, or something, allowed this in. The object in the api has a selector. I'm not sure if that was manually added.
The selector is present in kubectl get deployment thing, but not present in helm get manifest --revision N , where N is a deployed release.
I am now trying to upgrade the deployed release.
Helm attempts to deserialise the last DEPLOYED release, the one with the error. This deserialisation fails. The new deploy cannot continue.
I have tried using --force but things still fail at reading the release.
I'm not quite sure what the correct behaviour should be here. I think I can get things working again by hacking the config maps, but this situation should probably be catered for
perhaps helm edit manifest?

editting labels on config map entries doesn't seem to change the status (shame). Instead I curiously managed to rollback to a failed redploy (one that had failed due to the deserialisation problem), that seemed to work.

Hi @tcolgate
Sorry you are having trouble. Would it be possible to share a chart that displays the problem? Also what version of helm and k8s are you working with?

@tcolgate maybe your issue is related to #1873

It might be worth just closing this, but just to try and clarify a little. The cause of this was very simple. The content of the configmap was not compatible with the currently running API (or atleast whatever tiller was built against), reading the configmap caused an error due to the missing selector. There should probably be a better way of fixing this but I did manage to resolve it by doing a "rollback" to the most recent deploy (which was actually in a failed state).


Updating SignaturePullSecrets for policy controller based on new working

Updating SignaturePullSecrets documentation of the policy controller based on the latest changes by @hectorj2f

@MageshSrinivasulu Please, remember to signoff your commits

@MageshSrinivasulu Please, remember to signoff your commits

@hectorj2f This is the first time I am doing this so took some time to understand the process. Now the check ran fine. Please approve the PR


Cannot compile demography.md with newest msprime

I'm getting
nbclient.exceptions.CellExecutionError: An error occurred while executing the following cell:
------------------
dem = msprime.Demography(populations=[pop0, pop1], migration_matrix=M, events=[migration_rate_change])
ts = msprime.sim_ancestry(samples=[samples0, samples1], demography=dem, sequence_length=1000, random_seed=63461, recombination_rate=1e-7)
------------------

ValueError                                Traceback (most recent call last)
<ipython-input-8-acd1a2c2edb0> in <module>
----> 1 dem = msprime.Demography(populations=[pop0, pop1], migration_matrix=M, events=[migration_rate_change])
ValueError: Population ID should not be set before using to create a Demography

I don't understand why this is, as the raw code seems to run fine for me in a notebook:
import msprime
import numpy as np
from IPython.display import SVG
pop0 = msprime.Population(initial_size=500, growth_rate=0, name='Population0')
pop1 = msprime.Population(initial_size=500, growth_rate=0, name='Population1')
samples0 = msprime.SampleSet(num_samples=3, population='Population0')
samples1 = msprime.SampleSet(num_samples=3, population='Population1')
M = np.array([
[0, 0.05],
[0.02, 0]])
dem = msprime.Demography(populations=[pop0, pop1], migration_matrix=M)
ts = msprime.sim_ancestry(samples=[samples0, samples1], demography=dem, sequence_length=1000, random_seed=17, recombination_rate=1e-7)

We need to update these examples as they aren't idiomatic (or correct, I think) - we don't create objects any more, but rather call methods like add_population etc. The above would be
dem = msprime.Demography()
dem.add_population(name="Population0", initial_size=500)
dem.add_population(name="Population1", initial_size=500)
# Not sure we'd get into asymmetric migration rates so soon, this is quite advanced. Anyway.
dem.set_migration_rate(source="Population0", dest="Population1", rate=0.05)
dem.set_migration_rate(source="Population0", dest="Population1", rate=0.02)
ts = msprime.sim_ancestry({"Population0": 3, "POpulation1": 3}, demography=demography, # etc)

Do you want me to do this or leave it to Georgia in a re-write? At the moment I suspect CI might fail when I make changes to other pages.

I'll leave that to you and @gtsambos.

updating now 👍

Closed in #57


Feature to close HSTracker when Hearthstone closes

https://github.com/bmichotte/HSTracker/issues/357

Could you refactor to use Hearthstone.swift close event ?

I think this is what you mean?

yes ! thanks


Support Falcon uri templates in exempt_routes

This updates process_resource() to use the req.uri_template when checking
against the exempt_routes configuration. This allows one to exempt templated
routes rather than just hard-coded, static ones.
Addresses loanzen/falcon-auth#11

Coverage remained the same at 92.478% when pulling 9dbfc920b3d695e3ea308269324a3282878b3dfc on jcwilson:exempt-uri-templates into 46b45fbe56b1a735c9eb4b169362da7594021b7a on loanzen:master.

Coverage remained the same at 92.478% when pulling 9dbfc920b3d695e3ea308269324a3282878b3dfc on jcwilson:exempt-uri-templates into 46b45fbe56b1a735c9eb4b169362da7594021b7a on loanzen:master.


Support code review per commit

Many times PR contains multiple commits.
The code review of the PR in github web supports to review per commit, while in the VS code extension code review is only supported at the PR level.

Duplicate of https://github.com/microsoft/vscode-pull-request-github/issues/4315


Factor out common attached node behaviors

Attached nodes have common behaviors, including detaching and renaming. We should consider factoring out this common behavior similarly to how Quick Access nodes have certain behaviors factored out. This will reduce the code in each extension that supports attachments and simplify maintenance.

Attached nodes are more complex than QA nodes, because there is one central driving mechanism for QA that can add common actions and such, whereas Attachments are entirely dependent on extension implementations to generate nodes. This will require more investigation to derive a viable solution.


Add stable/unbound

Unbound is a fast caching dns resolver that we use in production kubernetes clusters to provide upstream resolution of private zones through kube-dns. This chart uses an auto-built docker image based on alpine:latest with unbound 1.6.7, as well as bash and bind-tools. It uses googlecontainer/exechealthz as a side car to perform health checks against the unbound server on localhost (allowing the default configuration of this chart to run unbound with restricted network access). Uncompressed image size is ~20MB. Submitted into stable because I believe it meets all technical requirements.

/ok-to-test


Starring

Starring doesn't seem to add a track to your favourites in Spotify anymore.

Just checked and it's working here on Windows + OSX, should be appearing in your Starred playlist.  Libspotify doesn't let us add to "Your music"  unfortunately.
One thing, if the track was already starred, it will not jump to the top of the list.

Checked this again, seems to be working so closing this.  Perhaps song was already starred?

The issue seems to be solved indeed. After your first reply I still didn't succeed in starring...


feat(theme-default): hide aside element when no heading present

Summary
Related Issue
closes: #915

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

We'd better unified all logic that controls UI presentation in this function.
https://github.com/web-infra-dev/rspress/blob/70ebf18fc65ddf40715530767f72d490370446d9/packages/theme-default/src/logic/useUISwitch.ts#L18

We'd better unified all logic that controls UI presentation in this function.
https://github.com/web-infra-dev/rspress/blob/70ebf18fc65ddf40715530767f72d490370446d9/packages/theme-default/src/logic/useUISwitch.ts#L18

get it 👌

Your alumni @9aoy

Your alumni @9aoy

haha, what a surprise 😮❕

We'd better unified all logic that controls UI presentation in this function.
https://github.com/web-infra-dev/rspress/blob/70ebf18fc65ddf40715530767f72d490370446d9/packages/theme-default/src/logic/useUISwitch.ts#L18

I've moved all the logic to useUISwitch, thanks for the review

I think add page.toc.length > 0 && is enough for this PR. The scene of no sidebar data already deal well?

I think add page.toc.length > 0 && is enough for this PR. The scene of no sidebar data already deal well?

with sidebarData.items.length > 0

without sidebarData.items.length > 0

🤔，I added that because I found that Object.keys(sidebar).length > 0 is true even if there is no sidebar in current page

I can not reproduce it.
My case is e2e/fixtures/basic

I can not reproduce it.
My case is e2e/fixtures/basic

It works perfectly when there is nothing else, but when we add more pages like

then we can reproduce this issue

I think this is because localesData.sidebar returns all sidebars in different pages by its path

I thought about it for a while and refered to the layout of Vitepress in such a cene that without headings in a doc.
The logic of this judgment is to reserve the placeholder sidebar and outline aside for the entire document. This is beneficial to the overall layout of the document  and also prevents the feeling of transition when switching each page in the document.
At the same time, there are may be some height calculations logic in document layout that require dynamic layout by relying on some placeholder width.
For a special single page, we are more suitable to use FrontMatter to control the display of related components for a separate page.
我想了一下，并参考了 Vitepress 在没有标题时候的布局。
这样判断的逻辑让文档的整体保留占位的 sidebar 和 outline aside，对于文档整体的布局是有益的，它使得文档中每个页面切换时不会产生瞬移的感觉。
同时，文档布局中可能还有一些高度的计算，需要依靠一些占位的模块来进行动态布局。
对于单页面的情况，我们更加适合使用 FrontMatter 为单独的页面控制相关组件的展示。
@linbudu599 @MaxtuneLee cc What's your opinion?

I thought about it for a while and refered to the layout of Vitepress in such a scene that without headings in a doc.
The logic of this judgment is to reserve the placeholder sidebar and outline aside for the entire document. This is beneficial to the overall layout of the document and also prevents the feeling of transition when switching each page in the document.
At the same time, there are may be some height calculations logic in document layout that require dynamic layout by relying on some placeholder width.
For a special single page, we are more suitable to use FrontMatter to control the display of related components for a separate page.
我想了一下，并参考了 Vitepress 在没有标题时候的布局。
这样判断的逻辑让文档的整体保留占位的 sidebar 和 outline aside，对于文档整体的布局是有益的，它使得文档中每个页面切换时不会产生瞬移的感觉。
同时，文档布局中可能还有一些高度的计算，需要依靠一些占位的模块来进行动态布局。
对于单页面的情况，我们更加适合使用 FrontMatter 为单独的页面控制相关组件的展示。
@linbudu599 @MaxtuneLee cc What's your opinion?

I think ur right, reserving sidebar by default make page transition smoother, and we also have sidebar option to let users choose what they want, from overall consideration, I think this is reasonable to keep the sidebar.

The logic of this judgment is to reserve the placeholder sidebar and outline aside for the entire document. This is beneficial to the overall layout of the document and also prevents the feeling of transition when switching each page in the document.

Agreed 👍

I will close this PR and its associated issues soon, thanks for your attention.


clock %S format specifier displays floating point

          I updated and now my clock's seconds are displaying in floating point representation instead of integer. See attached photo

My clock configuration is:
clock": {
"interval": 1,
"format": "{:%Y-%m-%d %H:%M:%S}",
}

This only happened recently as a result of updating
Probably a result of #2409

I updated and now my clock's seconds are displaying in floating point representation instead of integer.

Works fine for me.

Probably a result of https://github.com/Alexays/Waybar/pull/2409

What makes you think that exactly?
Are you using distribution which falls back to simple clock? By looking at recent commits it looks like dae7794bdfda81460beb26ad90a3beec2ab18bf2 might need to applied to simple clock as well?

By looking at recent commits it looks like https://github.com/Alexays/Waybar/commit/dae7794bdfda81460beb26ad90a3beec2ab18bf2 might need to applied to simple clock as well?

Actually not, it appears that simpleclock.cpp converts time to time_t which is expressed in seconds:
https://github.com/Alexays/Waybar/blob/b7a527b122831235fb36f661d415f5f3ed378d92/src/modules/simpleclock.cpp#L19-L22
Can't really help then. Clock rounds to full seconds so %S shouldn't show any fractional part while simple clock converts to time_t.

so why do you think this is displaying fractional? I didn't change my config through the update

Same on my side, also have just this simple config and get a floating point representation with %S
"clock": {
    "format-alt": "{:%Y-%m-%d}",
    "format": "{:%H:%M:%S}",
    "interval": 1
},

ah just built the master branch and there the problem does not exist anymore

Hi, only for reference:
Arch upgrade: upgraded waybar (0.9.20-2 -> 0.9.22-1)
Fixes this problem for me as well :)

Thanks @manio. That's the first comment from someone affected that has an actual merit.
The reason for fractional seconds, in this case, was distribution packaging issue caused by cherry pick of e9b29613 in https://gitlab.archlinux.org/archlinux/packaging/packages/waybar/-/commit/76fca700672f93e580582b541a0f45b312322b69 while skipping dae7794.


Remove – from Plotly annotations

HTML entities like – are not compatible with Plotly annotations. We need to fix this:

And check to see if there are any other Plotly charts with the same or similar problems.

@cstephen While the PR for this would fix it, I'm puzzled because I can't recreate this either on-screen or in the downloaded charts from the production site.  !!!  What specific conditions does it occur in?

Oh, it looks like this has been fixed in production but not in main (oh dear), I'll track down the relevant commits and we'll figure out what happened.


[Dropdown] Added .inverted variations

Closed Issues
#5276
Description
Adds .inverted variation for dropdowns.
This was originally created by @Banandrew and requested by @raptoria
Original issue https://github.com/Semantic-Org/Semantic-UI/issues/5276
I have also made a PR for the docs to add an example https://github.com/Semantic-Org/Semantic-UI-Docs/pull/389

Any movement on this? It'd be great to be able to use this for dropdowns in inverted menus!

@airhorns coming next update on the fork https://github.com/hammy2899/Fomantic-UI/blob/master/RELEASE-NOTES.md

Any chance to get this merged please?

Hey, could this get merged? It'd be useful to have!

Inmplemented since https://github.com/fomantic/Fomantic-UI/releases/tag/2.4.3
See https://fomantic-ui.com/modules/dropdown.html#inverted


Error handling

I have found a minor issue (unrelated to this pull request) yesterday while reviewing your code changes. We have switched all functions to use the C API instead of the C++ API for performance reasons. However when the primesieve C API is used we need to check errno after each call into libprimesieve to know whether an error has occured. We correctly do this for primesieve.primes(n) and primesieve.n_primes(n). However for all other functions we don't do it. This means that in case an error occurs for e.g. primesieve.count_prime(n) that function will silently return an incorrect result.
The reason for this issue is that C does not support C++ exceptions. When an error occurs inside libprimesieve an exception (which contains a message of what went wrong) is thrown. However in the libprimesieve C API all exceptions are caught (inside libprimesieve) and errno is set instead.

Along these lines, based on some quick testing, it seems the C interface is noticeably faster than the C++ interface. To test, I created another submodule based on the array submodule, and modified it to include the C++ interface rather than C. The performance matched very closely with the base module, with the array submodule based on the C interface performing much better.

Is my understand right that you only measured a speed up when using the C API for primesieve.primes(n) and primesieve.n_primes(n). If so I suggest that we switch back all other functions to the C++ API because there error handling works out of the box.
Originally posted by @kimwalisch in https://github.com/hickford/primesieve-python/pull/30#issuecomment-556978990

I figured a separate issue would be better for an ongoing discussion on this issue.
As noted in the other thread, running count_primes and count_twins in the C version is about 3x faster than in the C++ version.

OK, let me benchmark this as well. If there is really a 3x speedup we will use the C API with proper error handling.

I have now benchmarked primesieve-python-1.4.4 which uses the C++ API vs primesieve-python-2.0.0 which uses the C API on Ubuntu 18.04 x64 using the following benchmark:
"""Tests for primesieve module."""

import primesieve

def test_count_primes():
    """Test count_primes function."""
    assert primesieve.count_primes(100) == 25
    assert primesieve.count_primes(10**6) == 78498
    assert primesieve.count_primes(100, 10**6) == 78473
    primesieve.count_primes(10**10)
    assert primesieve.count_primes(10**9, 10**10) == 404204977

    for x in range(10000000, 10003000):
        primesieve.count_twins(x)

    for x in range(0, 300000):
        primesieve.count_primes(x)

def test_nth_prime():
    """Test nth prime function."""
    assert primesieve.nth_prime(25) == 97
    assert primesieve.nth_prime(26) == 101
    assert primesieve.nth_prime(1, 100) == 101
    assert primesieve.nth_prime(100, 1000000) == 1001311
    primesieve.nth_prime(10**9)

This code does a few large computations but also many tiny computations, so it is a good mix.
Timings are:

primesieve-python-1.4.4: 8.96 seconds
primesieve-python-2.0.0: 7.54 seconds

So using the C API indeed is a bit faster but not 3x. And if we add proper error handling to the C API this will also incur a small slowdown in which case there would not be much benefit in using the C API. I can understand that using the C API for the primesieve.primes(n) and the primesieve.n_primes(n) functions provides a significant speedup as this allows us to avoid converting the C primes array into a python list (which is very slow). However the other functions just return a single integer and hence from my understanding this should perform similarly for the C and C++ APIs.
Can you please let me know what code you have used for your benchmark and also the operating system you have used? For your information: the OpenMP multi-threading of primesieve-python-1.4.4 does not work on macOS and possibly also not on Windows. However the new C++11 multi-threading works on all operating systems. Maybe this could explain the huge performance speedup you have measured? Note that multi-threading is used only for the count functions and the nth_prime function.

I am running on Windows 10, python 3.7.5.
I ran the following at the IPython prompt.  I saw nearly identical performance for count_twins.
# version 2
%timeit count_primes(1e9)
68.8 ms ± 1.51 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

# version 1.4.4
%timeit ps.count_primes(1e9)
186 ms ± 7.63 ms per loop (mean ± std. dev. of 7 runs, 10 loops each)

OK, you measured a nearly 3x speedup. Can you please run the same benchmark but with 1e12 instead of 1e9. And while the benchmark is running have a look at the CPUs in the Windows Task Manager. I think that version 2 uses multi-threading while version 1.4.4 does not use multi-threading.
After thinking about it I even think that the 30% performance speedup I measured are most likely related to libprimesieve improvements and not because the C API is faster than the C++ API for counting primes and the nth_prime function.
The reason why I would prefer to use the C++ API (if there is not much performance difference) is that the error messages are better. Using the C API we can just report that an error occurred while using the C++ API you get detailed error info. Also we don't need to implement error handling for the C++ API.

Further testing has proven your point.  Using the C++ interface and a more recent version of primesieve performed about as well as the C API.  I have submitted a PR to make the change back to the C++ API.

Fixed by https://github.com/hickford/primesieve-python/pull/32


fist commit

🛠 Fixes Issue : ""

👨💻 Changes proposed:

✔ Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.
[ ] I am a participant of Hacktoberfest 2022.

📄 Note to reviewers

📷 Screenshots

Thank you for raising the PR and taking the time to contribute to this repo.
Please provide a brief description of the changes you have made in the comment above, along with screenshots if applicable.