Converted all html file extensions and formatting in docs/resources folder from html to md

Fixes

Fixes #333  by @ Murdock9803

Description

This PR addresses the issue of inconsistencies in the documentation resource files located in the docs/_resources directory. Currently, the  files in this folder are in two formats: Markdown (.md) and HTML (.html). This dual format leads to confusion among contributors and complicates the contribution process.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Hi @Dev-JoyA Thanks for doing this!
There are a number of locations where extra space is present preceding the first lines of paragraphs, and that would need to be corrected overall.
Also, please verify that any of the html that is removed and replaced with markdown is not losing a class or changing its ultimate html structure.

Hi @Dev-JoyA Thanks for doing this!
There are a number of locations where extra space is present preceding the first lines of paragraphs, and that would need to be corrected overall.
Also, please verify that any of the html that is removed and replaced with markdown is not losing a class or changing its ultimate html structure.

I would correcting the issue with the spacing in another commit
I ensured i replaced  unordered list with its relevant markdown synthax, same with  and every other html structure are correctly replaced with markdown synthax.

@Dev-JoyA if you reply to #333 I can assign you.


"allOf" reference present in child Dto but missing on identical sibling Dto

Hello! Our application has this user structure (Simplified, the AbstractUserDto has more fields):
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, property = "type")
@JsonSubTypes({
        @Type(
                value = InternalAuthenticatedUserDto.class,
                name = "internal-authenticated"),
        @Type(
                value = RemoteAuthenticatedUserDto.class,
                name = "remote-authenticated")})
@Schema(
        discriminatorMapping = {
                @DiscriminatorMapping(
                        value = "internal-authenticated",
                        schema = InternalAuthenticatedUserDto.class),
                @DiscriminatorMapping(
                        value = "remote-authenticated",
                        schema = RemoteAuthenticatedUserDto.class)})
public abstract sealed class AbstractUserDto
              permits InternalAuthenticatedUserDto,RemoteAuthenticatedUserDto {

	private String username;
        private String fullName;

        [...]

public final class InternalAuthenticatedUserDto
        extends AbstractUserDto {
}

public final class RemoteAuthenticatedUserDto
        extends AbstractUserDto {
}

You can see that the concrete user DTOs are exactly the same. But the generated api spec is different.
For the RemoteAuthenticatedUserDto it correctly references the base class:
"FlatRemoteAuthenticatedUserDto": {
  "type": "object",
  "allOf": [
    {
      "$ref": "#/components/schemas/FlatAbstractUserDto"
    }
  ]
},

For the InternalAuthenticatedUserDto the properties are present on the schema itself:
"InternalAuthenticatedUserDto": {
  "type": "object",
  "properties": {
    "username": {
      "maxLength": 63,
      "minLength": 0,
      "type": "string"
    },
    "fullName": {
      "type": "string"
    }
  }
},

This seems to be related to #3312.

CC @Digirik @SimonErm

I'm sorry, I did not notice we were running an outdated version. This seems to be fixed in the latest release :)


Can not open emg brake interface

https://docs.citahub.com/zh-CN/next/cita/system-contract-interface/emergency-brake
version: latest

It works now, have a try?

yes


Model Missing

Hi, thanks for the repo. I'm trying to implement this on my device however the link you provided leads to nowhere. Where can I find the trained lanenet model? Thanks!

https://www.dropbox.com/sh/0b6r0ljqi76kyg9/AADedYWO3bnx4PhK1BmbJkJKa?dl=0

@canozcivelek Try this https://www.dropbox.com/sh/0b6r0ljqi76kyg9/AADedYWO3bnx4PhK1BmbJkJKa?dl=0


ch5/joinMap中VTK的问题

报错，但是依然可以编译通过
-- Boost version: 1.58.0
-- Found the following Boost libraries:
--   system
--   filesystem
--   thread
--   date_time
--   iostreams
--   serialization
--   chrono
--   atomic
--   regex
** WARNING ** io features related to pcap will be disabled
** WARNING ** io features related to png will be disabled
-- The imported target "vtkRenderingPythonTkWidgets" references the file
   "/usr/lib/x86_64-linux-gnu/libvtkRenderingPythonTkWidgets.so"
but this file does not exist.  Possible reasons include:
* The file was deleted, renamed, or moved to another location.
* An install or uninstall procedure did not complete successfully.
* The installation package was faulty and contained
   "/usr/lib/cmake/vtk-6.2/VTKTargets.cmake"
but not all the files it references.

-- The imported target "vtk" references the file
   "/usr/bin/vtk"
but this file does not exist.  Possible reasons include:
* The file was deleted, renamed, or moved to another location.
* An install or uninstall procedure did not complete successfully.
* The installation package was faulty and contained
   "/usr/lib/cmake/vtk-6.2/VTKTargets.cmake"
but not all the files it references.

-- looking for PCL_COMPONENT
-- Could NOT find PCL_COMPONENT (missing:  PCL_COMPONENT_LIBRARY PCL_COMPONENT_INCLUDE_DIR)
-- looking for PCL_COMMON
-- looking for PCL_OCTREE
-- looking for PCL_IO
-- Configuring done
-- Generating done
-- Build files have been written to:

看  https://stackoverflow.com/questions/37369369/compiling-pcl-1-7-on-ubuntu-16-04-errors-in-cmake-generated-makefile
这里的提问好像是 vtk的问题，不知道有没有办法消除这个问题？

依然报错
环境是Ubuntu 16.04
~/D/s/c/joinMap> pcl_viewer map.pcd
The viewer window provides interactive commands; for help, press 'h' or 'H' from within the window.
> Loading map.pcd X Error of failed request:  BadValue (integer parameter out of range for operation)
  Major opcode of failed request:  154 (GLX)
  Minor opcode of failed request:  3 (X_GLXCreateContext)
  Value in failed request:  0x0
  Serial number of failed request:  43
  Current serial number in output stream:  44

上面的问题好像解决了 ： https://askubuntu.com/questions/893922/ubuntu-16-04-x-error-of-failed-request-badvalue-integer-parameter-out-of-range
NVIDIA驱动的问题

change COMPONENT to COMPONENTS from CMake file. Add the s


draft delete should delete docker related artifacts

Opening this issue to keep track of discussion in #379
As @fibonacci1729 brought up, at the moment, draft delete deletes a helm release but doesn't do anything with the Docker artifacts that get exist. There are implications of deleting this such as deleting layers that other images rely on. There are also different strategies for going about doing this in an intelligent manner as @bacongobbler mentioned. See above PR thread for more info.

This might be nice under a feature flag or at least by gating with a yes/no prompt. The user should explicitly opt into this since docker images stored in the daemon and the registry are being used across other tools like what's implied in #541. :)


Update and rename pnpmfile_js.md to .pnpmfile_cjs.md

See https://rushstack.zulipchat.com/#narrow/stream/262513-general/topic/.E2.9C.94.20pnpmfile.2Ejs.20is.20unrecognized/near/252645331
(incidentally, this was changed in the main repo four months ago: https://github.com/microsoft/rushstack/commit/c74d8de61d829f254d57eb7effbedaf5f895feb1) - given rush is a monorepo tool, shouldn't the website be in there too?? Might be a nice way to show off rush deploy type commands too.

Cool, thanks @mmkal for updating this!


Selection issue when all items checked

When all options are selected then spinnerText gets displayed as blank, all values are not appended.

you have to call spinner.setAllText(...) for displaying a text when all items are selected.


Control Saving/Restoring Checkpoint From Training Type Plugin

🚀 Feature

Motivation

Currently, we have CheckpointConnector to deal with logics of restoring/saving checkpoint, but it is highly dependent on training type plugin, especially when we will save/restore sharded states. It might be clearer for training type plugin/accelerator to directly handle these.
Another thing which might worth consolidating is one unified interface for restore. For saving checkpoint, we have clear one save_checkpoint(self, filepath, weights_only: bool = False) , however for restoring, we have

checkpoint_connector.restore_weights() (https://github.com/PyTorchLightning/pytorch-lightning/blob/master/pytorch_lightning/trainer/connectors/checkpoint_connector.py#L82) when training providing resume_from_checkpoint
training_type_plugin.restore_model_state_from_ckpt_path (https://github.com/PyTorchLightning/pytorch-lightning/blob/master/pytorch_lightning/trainer/trainer.py#L1152) when test/validation provide ckpt_path)

(The former depends on the latter, and for the latter, feel bit counter-intuitive for restore_model_state_from_ckpt_path to return checkpoint state though model has already been loaded and whether to load optimizer states)
the main difference is just whether we just restore model states. We could do similarly as
def save_checkpoint(self, filepath, weights_only: bool = False)

for restoring
def restore_checkpoint(self, filepath, weights_only: bool = False)

Pitch

Transfer ownership of load/restore to purely training type plugin (or accelerator), and unify restore_checkpoint interface.
Alternatives
Remain current implementation.

@shuyingsunshine21 would you be open to send a PR with yoour proposal ?

sure, bring this up for brainstorming and need to think a bit more on this!


downgrade directories to v4 again

rustc tidy says
tidy error: invalid license `MPL-2.0` in `option-ext 0.2.0 (registry+https://github.com/rust-lang/crates.io-index)`

@bors r+

:pushpin: Commit e5fd63ca1e7163031209bb5c162b3cf2a6452769 has been approved by RalfJung
It is now in the queue for this repository.

Ah looks like we already have some MPL-2.0 exceptions and it's fine as long as it's only in rustc tooling, not the runtime.
@bors r-


Send more advanced slack messages

Default slack plugin sends only text message with color data. However Slack supports more complex messages. Slack message attachments
With this implementation any message mentioned in Slack messages documentation can be send with this plugin.
There is also an issue about this feature. Slack Message Attachments #277

Nice I could see this being useful.

agree, slack message with changelog / test result would be nice

@ahmetrehaseker any updates here? this is an good feature, we'd like implement it, thanks

This would be a great feature to have, can't wait for it to be finished.

This seems to have been abandoned. @samrocketman can I adopt this? What's the preferred course of action for this? I'm thinking I'd fork the original repo, replicate the changes from this repo with your suggestions and open a new PR. Is that ok?

It's okay to cherry-pick/squash changes and build on them in a new PR preserving authorship.

Alright, here we go. New PR created: #324

Closing this in favor of #324


Fix denominators.

Replace denoms as fitting.

Some sxs to help ease approval:

Sanity check the bar chart latest dates: USA top charts local staging

Title change and denom change in employment rate; denom change for labor force participation rate (local, then staging):

Denom change for UI Claims (local, then staging):

Labor force participation rate splitting: local staging

Denom change for individual income (left: local, right: staging)

[continued]

Labor force close up: note that the other countries chart was originally computed from the WDI Count_Person_InLaborForce with Count_Person as the denom. Now, it is using the WDI StatVar Count_Person_15To64Years_InLaborForce_AsFractionOf_Count_Person_15To64Years

@beets  since there are many calls for me to do other things, I have reverted all date munging to Bo's original functions. I'll try your suggestions later on.


Development builds are not being updated on froggi.es

As outlined in the readme:

Development builds for each commit for Garry's Mod are available on froggi.es.

When going to the link however, the build is only updated to commit https://github.com/Joshua-Ashton/VPhysics-Jolt/commit/067cc7eb4e5a145f4c3be6629f597809a5ca1317.

This is fixed now.

This is an issue again


Helpful typing around producer actor

Addresses some of the concerns pointed out in issue #90.

Codecov Report

Merging #93 into master will decrease coverage by -0.37%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
- Coverage   64.11%   63.74%   -0.37%
==========================================
  Files          16       16
  Lines         496      502       +6
  Branches       35       29       -6
==========================================
+ Hits          318      320       +2
- Misses        178      182       +4

Impacted Files
Coverage Δ

...ala/cakesolutions/kafka/akka/ProducerRecords.scala
7.69% <ø> (-3.42%)
:x:

.../cakesolutions/kafka/akka/KafkaProducerActor.scala
69.23% <ø> (+4.94%)
:white_check_mark:

...ala/cakesolutions/kafka/akka/ConsumerRecords.scala
47.61% <ø> (-11.21%)
:x:

.../cakesolutions/kafka/akka/KafkaConsumerActor.scala
57.2% <ø> (+0.93%)
:white_check_mark:

Continue to review full report at Codecov.

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

👍


Add ability to interpolate values into Headers for HTTP channels

originated from https://github.com/helium/console/issues/760


Dockerstatic: Add packages for uptime, diff and hostname for UBI8

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

ref https://github.com/adoptium/infrastructure/issues/2882#issuecomment-1385769644
These packages are needed on ubi8 for jdk_management tests to pass, see https://github.com/adoptium/infrastructure/issues/2882#issuecomment-1384516670

Im in favour of replacing the ubuntu21 dockerstatic pr check for ubuntu22 as it is more up to date, and we have to ubuntu21 x64 static docker containers in jenkins. Any objections?

Im in favour of replacing the ubuntu21 dockerstatic pr check for ubuntu22 as it is more up to date, and we have to ubuntu21 x64 static docker containers in jenkins. Any objections?

Does that have anything to do with this PR?

@sxa No, I just thought it may need updating


undefined method 'peeraddr' for #<Fluent::PluginHelper::HttpServer::Request

Description:
I get the error undefined method 'peeraddr' for #<Fluent::PluginHelper::HttpServer::Request... when trying to use the protobuf-http input plugin. If I hardcode a peeraddr then it progresses to a similar message undefined method 'header'.... If I hardcode both the content_type and the peeraddr, thus removing both of those method calls, the plugin seems to work as intended, minus actually performing the checks against the actual request peeraddr and headers.
Reviewing both the documentation  for the Fluentd HTTP Server plugin helper, as well as the code, I would agree with the error message, it does not look like those methods are available in the request object.
To Reproduce
Steps to reproduce the behavior:

I'm using the fluent/fluentd:v1.16-debian-1 Docker image
Added apt-get install -y protobuf-compiler and RUN gem install fluent-plugin-protobuf-http to the DockerFile
Using stdout for my output
POST to endpoint using Postman with a valid protobuf message

fluentd.conf
<source>
  @type       protobuf_http
  @id         protobuf_http_input
  @log_level  debug

  bind        <IP_ADDRESS>
  port        8081
  tag         debug.test

  proto_dir   /fluentd/etc
  in_mode     binary
  out_mode    json
</source>

<match **>
  @type stdout
</match>

What's most confusing to me is I'm struggling to find an update that would have caused this to stop working. I'm not a Ruby developer so it might be obvious, but it looks like the HTTP Server plugin helper request object hasn't changed since 2019 so I feel like I must be missing something given the update to the plugin in 2021.

While testing, I also tried adding attr_reader request to the request class so that I could call headers from the Protocol::HTTP::Request class and that worked, i.e. content_type = req.request.headers['content-type']

@dentonk
Thank you for reporting this issue!
I never tested this on Debian. I'll try to take a look at this soon.
Is this how your Dockerfile looks like?
FROM fluent/fluentd:v1.16-debian-1
RUN apt-get update && apt-get install -y protobuf-compiler
RUN gem install fluent-plugin-protobuf-http

If there's anything else, please do include that. Thanks!

Thank you so much for digging into this so quickly and your detailed explanation!! That seems to have done it! I missed that one little detail around async-http vs webrick and could not for the life of me figure out how this ever worked with the async-http stack. I knew I must be missing something, I was just looking in the wrong place. 🙂
I tested with both debian and alpine and was able to get it to work by uninstalling async-http as you suggested.

Awesome! You're welcome! :smile:
Thank you for your verification! :+1:
I looked at the implementation of this plugin and indeed it's using the webrick APIs.
That makes it more compatible with webrick and completely incompatible with async-http.
I'll add a new section in the README for known issues and list its hard dependency on webrick and how to resolve its conflict with async-http. I'll close this issue then. Cheers!

Updated README with #11.


feat(snap): use updated environment variable injection

Signed-off-by: Siggi Skulason<EMAIL_ADDRESS>This PR updates edgex-go to use the new environment variable injection.
apps.<app>.config.<my.env.var> -> setting env var MY_ENV_VAR for an app (e.g. apps.core-data.config.service.port=1000)
config.<my.env.var> -> setting env variable for all apps (e.g. config.service.host=localhost)

If your build fails  due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) N/A - they are in the imported module
[X] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?) N/A

Testing Instructions

Do
sudo snap set edgexfoundry env.core-data.service.host=localhost
sudo snap set edgexfoundry apps.core-data.config.service.port=8080
sudo snap set edgexfoundry config.debug=true

Confirm that all .env files have DEBUG=true
Confirm that the core-data .env file also has both the SERVICE_HOST and SERVICE_PORT settings

$ more /var/snap/edgexfoundry/current/config/core-data/res/core-data.env
export SERVICE_HOST=localhost
export DEBUG=true
export SERVICE_PORT=8080
$ more /var/snap/edgexfoundry/current/config/core-command/res/core-command.env
export DEBUG=true

New Dependency Instructions (If applicable)

Thanks @siggiskulason for this addition. I've added a new PR (https://github.com/edgexfoundry/edgex-go/pull/3986) with some fixes and additions.
I'm going to close this in favor of the other PR.


Metadata fix

Fix bug where going back and forth between metadata and repositories steps breaks metadata
Metadata now properly shows up in the review step (and saves to the submission model)
Laid the groundwork for being able to go back and edit which repositories you want to submit to (i.e. metadata forms will properly update when a link to a repository is deleted)

Known Issues

Going back a step and attempting to remove repositories after you've selected them will not actually remove them. This is being worked on this evening!

This looks good, noted the issues!


RELEASE: v.0.1.0-alpha.28

Coverage remained the same at 100.0% when pulling 756d158e0bb811025a11bd16386dd42ae05ef8ca on dev into 3158f0300d667d5b0eb99ee4a5b9bb51f71cbfd6 on next.


No user feedback when refreshing list of databases

Azure Data Studio Version: 1.32 / 1.33 Insiders
OS Version: Windows 10 "current"

Steps to Reproduce:

In the Connections pane, connect to a server
Expand the "Databases" folder and observe the spinner displayed over the folder icon:

Wait for list of databases to load
Right-click on "Databases" folder and click Refresh. Observe that no spinner is displayed and there is no other obvious user feedback.

Does this issue occur when all extensions are disabled?: Yes

Closing as PR #20621 is merged that should address this issue.


Only entities converted in output

Describe the bug
It appears that the tool is only converting entities to jsonl format, even when other annotation types such as relations have been used in the .ann file and thus also appear in the annotation.conf file.
To Reproduce
Steps to reproduce the behavior:

Git clone the repo
Execute: go run main.go -p "./testData/news" --output "./news.jsonl"
Open news.jsonl
See that in the first line (corresponding to 000-introduction), there are only entities and no relations, for example.

Expected behavior
I expected to see {"Data":"Welcome [...] page.\n","Entities":[[418,426,"Organization"],[...]],"Relations":[[540,545,549,560,"Origin"],[...]]}
Desktop (please complete the following information):

OS: Ubuntu 20.04
bratconverter version: development

Additional context
Maybe it's not yet part of the tool's functionalities but I didn't see it mentioned anywhere :) thanks in advance for your help!

It works now, I must have been doing something wrong before!


Does not implement pact's multi valued http response headers

It seems that multi valued headers are not supported yet. If v is an array then we have to repeat the header.
https://github.com/uglyog/pact-stub-server/blob/934b748809a5fa6fb45d22940e8d9595b43e0a34/src/pact_support.rs#L48
We faced the need of multi valued header while setting multiple cookies.
https://github.com/DiUS/pact-jvm/issues/851
And the issue got fixed. The changes are done in https://github.com/DiUS/pact-jvm/commit/997e24ebdb2b5f420020480fcb607e1448d2da7b

This needs a change to an upstream library. That will need to be released first.

0.3.1 has been released with this fix


travis: Use the original ocaml-ci-scripts script

This should fix the current Travis build errors.

LGTM, thanks!
FWIW this won't fix the Travis build errors because they are due to various Blas/LAPACKE/etc build nonsense -- different packaging on different distros builds and then installs libraries in different places with, in some cases, different symbols :(  But since my PR to quieten git was accepted, we should be back on the mainline rather than my fork, so it's still an improvement.


Update FaIR heat uptake units

[x] Tests added
[x] Documentation added (N/A)
[x] Example added (in the documentation, to an existing notebook, or in a new notebook) (N/A)
[x] Description in CHANGELOG.rst added (single line such as: (`#XX <https://github.com/openscm/openscm-runner/pull/XX>`_) Added feature which does something)

@chrisroadmap

Cool, looks good


feat: implement the test suite for MLOAD opcode

### Tasks
- [ ] Create the trait for the MLOAD opcode category
- [ ] Create the MLOAD opcode function, write it only as a panic_with_felt252('MLOAD not implement yet')
- [ ] Create the test suite for MLOAD, taking Geth's examples
- [ ] Create the test suite for MLOAD, taking Kakarot cairo0 examples
- [ ] Create the test suite for MLOAD, taking the evm.codes examples

@Eikix Can you assign me, pls?

Important update: we are gathering some bugs in the Kakarot v0 codebase, we need to make sure each issue and each PR in Kakarot-ssj is aware of the lists of known bugs. Look at this link everytime you take an issue and check your issue isn't targeted by a known bug.

Important update: we are gathering some bugs in the Kakarot v0 codebase, we need to make sure each issue and each PR in Kakarot-ssj is aware of the lists of known bugs. Look at this tracking issue everytime you take an issue and check your issue isn't targeted by a known bug. Will add this reminder in many places to make sure we keep track of known bugs.


merge master to live-based

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

latest/docs-ref-autogen/dls/fs.yml
:white_check_mark:Succeeded
View (azure-cli-latest)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat(semantic,syntax): add SymbolFlags::ArrowFunction

There are many cases in lint rules where we want to see if a symbol is a
function by checking its SymbolFlags. This is currently not fully possible,
since variables assigned to arrow functions are not distinguished from any other
kind of variable. This PR adds SymbolFlags::ArrowFunction for variables that
are initialized to arrow functions. Symbols that are re-assigned to arrow
functions will not have this flag, but this is acceptable for lint rules.

#4946  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on  Graphite


CTKFlagTextField in storyBoard

Hi,
I am not able to set the textfield in storyboard, it crashes when we set it via storyboard(not programatically). Could you please suggest any solutions for the error?

Please use the template to create new issue.


Publishing files from ArcGIS Online to Geoportal

Hi, Can I publish the files from my ArcGIS Online account to the Geoportal Catalog without using Harvester??
If yes, can you refer the process to me?
Thank you,
Durga

hi, no. Geoportal works with metadata only. the item description in ArcGIS Online can be considered metadata so you could setup your geoportal to index that. Geoportal Server does not host data or files or apps. those will need to be kept elsewhere such as in ArcGIS Online.

Hi @mhogeweg,
For hosting data can you let us know the opensource options which work with Geoportal?

if you're hosting files then IIS or Apache HTTPD work. But I'm confused, you have an ArcGIS Online subscription.


SteamVR Fail on Arch Linux

Bug reports:
Having installed alvr both directly from the alvr repo and also via the alvr aur I get an error in from the SteamVR App saying that a key component isn't working and I need to restart the app.  Reboots, and re-install continues to get same error.
I will attach the vrserver.txt log here.
SteamVR-2021-07-17-AM_11_11_14.txt
ALVR Server appers to connect and see headset fine.  But SteamVR will not stream output to ALVR.  If I run SteamVR without ALVR I get usual cannot find headset and no additional errors.  So assuming that it is something between alvr and steamvr.
Environement
What hardware do you have: AMD 5900, AMD 6800XT, Garuda Linux/Arch Kernel: Linux-Zen-5.12
What version of ALVR and SteamVR have you tried? Latest

It appears this is not actually related to ALVR and is in fact a driver/libsteam_api issue.  It is throwing a code when it tries to access out of bounds memory.
Thanks
KK


:sparkles: Add Level Select screen and logic along with Resetting Level Logic

Add 'LevelSelect.hs' which handles level select screen ui and logic

Reset Level logic here

Added new type 'Level' :: (Int, Environment) in 'Sokoban.hs'.
Restructure event handling in 'Game.hs' to use 'Level' instead of 'Environment' for resetting level logic
Compiles and runs properly (no automated tests run)

Please fix merge conflicts before merging.


support web XR

Describe the problem
now MRTK can work with android ,ios and wearable devices but it still have a bit inconvenient because we need to install the app to device so we really need the web XR feature
Describe the solution you'd like
i have tried demo  by @Rufus31415 in https://github.com/Rufus31415/Simple-WebXR-Unity - including MRTK integration. It's great if this target officially supported in MRTK
Describe alternatives you've considered
Additional context
Existing Repo: https://github.com/Rufus31415/Simple-WebXR-Unity
Demo site: https://rufus31415.github.io/sandbox/simple-webxr-mrtk/

Closing as a duplicate of https://github.com/microsoft/MixedRealityToolkit-Unity/issues/8472. Thanks for submitting! We'll use the other issue for discussion.


Issue on adding emoji between emojis

There is an issue when I want to insert emoji between inserted emojis.
Insert emoji between text is ok but between emojis has issue.
After inserting emoji, text will be converted to a single character like this:

@devbrisk I can't replicate the issue with the example. Could you provide an example to check?

You can see the example in the following :
https://github.com/Fintasys/emoji_picker_flutter/assets/114703704/e2c3ab9d-fa51-47cb-a095-5e13b4720143

@devbrisk Sorry but it seems to be an issue within your implementation. I tried to replicate the issue with example project but I couldn't reproduce it. If you can provide a minimal example where you can reproduce the issue I would look into it.


Outdated Wallet Link Code --> Needs to be updated to Coinbase Wallet SDK

Hi,
I work on the DevRel team at Coinbase and am submitting this issue to inform you that the WalletLink SDK has been deprecated in favor of our new Coinbase Wallet SDK You can find more on that and how to upgrade to the latest version here.
Please let me know if you have any questions!
Thanks,
Freddy

Hi @FreddyMJ3 we have the WalletLink package marked as deprecated and have a Coinbase SDK package available and live.


Restore bashrc and profile from skeleton file if file doesn't exist

fixes: https://github.com/microsoft/vscode-dev-containers/issues/878
Pulls from the skeleton file in our common-debian.sh if the .bashrc and .profile do not exist (or are empty).

LGTM!


SRPMs are built in Copr by default for installations since September 6

Signed-off-by: Frantisek Lachman<EMAIL_ADDRESS>TODO:

[x] Write a new test.
[x] Update the old tests.
[ ] Update documentation

Fixes #1583

RELEASE NOTES BEGIN
SRPMs are built in Copr by default for Packit GitHub app installations since September 6, 2022. For older installations, you can set the  srpm_build_deps config option to use Copr as a builder. Let us know if you hit any issue with the new implementation. We are going to slowly decommission the old implementation and are happy to help with the transition.
RELEASE NOTES END

should we mention in the release notes that this is only about SRPMs for Copr builds and for Koji builds, everything is the same or do you think it is clear?

Good point. I'll rather mention that.


Make Assembler reusable

We currently create assembler instances on-demand e.g.:
Flux<Transaction> transactionFlux = Flux.fromIterable(getCustomers())
    .buffer(10)
    .flatMap(customers -> assemblerOf(Transaction.class)
        .from(customers, Customer::getCustomerId)
        .assembleWith(
            oneToOne(this::getBillingInfoForCustomers, BillingInfo::getCustomerId),
            oneToManyAsList(this::getAllOrdersForCustomers, OrderItem::getCustomerId),
            Transaction::new)
        .using(fluxAdapter()))

but doing so means that a few new intermediate short lived objects (to internally build the assembler) are allocated on each invocation, which could make the GC work a bit more under heavy load with very high velocity data. For most use cases this doesn't have any impact, and we should be careful with premature optimization, but for code where the smallest performance gain is significant, changing the current implementation could potentially lead to small increase in data throughput.
Anyhow, it might not be a bad idea to be able to save the 'recipe' once and just apply it whenever data is coming i.e. change the api to be able to do something that could look like:
Assembler<Flux<Transaction>> transactionAssembler = assemblerOf(Transaction.class)
   .withCorrelationId(Customer::getCustomerId)
   .assembleWith(
       oneToOne(this::getBillingInfoForCustomers, BillingInfo::getCustomerId),
       oneToManyAsList(this::getAllOrdersForCustomers, OrderItem::getCustomerId),
       Transaction::new)
   .using(fluxAdapter()));

Flux<Transaction> transactionFlux = Flux.fromIterable(getCustomers())
   .buffer(10)
   .flatMap(transactionAssembler::assemble);

The final api might not be exactly like the example shown above which is just for illustration purpose.

Creating a reusable Assembler instance will have the undesirable side-effect of losing type inference, so closing the issue, won't fix it. A new issue should probably be created to add a set of static methods to invoke the assembling process as a single function, less intuitive than using the builder, but for performance critical scenarios, this would help.

We can now reuse an Assembler, here is how (example with RxJava adapter):
AssemblerBuilder<Customer, Flowable<Transaction>> = assemblerOf(Transaction.class)
        .withIdExtractor(Customer::getCustomerId)
        .withAssemblerRules(
                oneToOne(AssemblerTestUtils::getBillingInfoForCustomers, BillingInfo::getCustomerId, BillingInfo::new),
                oneToManyAsList(AssemblerTestUtils::getAllOrdersForCustomers, OrderItem::getCustomerId),
                Transaction::new)
        .using(flowableAdapter(single()));

Flowable<Transaction> transactionFlowable = Flowable.fromIterable(getCustomers())
        .buffer(3)
        .flatMap(assembler::assemble);

A new builder chain was added (Assembler.AssemblerBuilder) and the final container type <RC> is captured there.


crash when openmp is used (null pointer in typedef name)

I am using cv2pdb to convert DLLs generated with MinGW-w64 from DWARF to PDB. I recently started using OpenMP in that project. This causes cv2pdb to crash because now addUdtSymbol is being called with a null name from CV2PDB::createTypes, more specifically in these lines:
case DW_TAG_typedef:
	cvtype = appendModifierType(getTypeByDWARFPtr(cu, id.type), 0);
	addUdtSymbol(cvtype, id.name);

For now I just added if(!name) return false; to the beginning of CV2PDB::addUdtSymbol.
Minimal example:
Add the following to a new file, e.g., cv2pdb_crash.cpp:
__declspec(dllexport) void foo(int I)
{
	#pragma omp parallel for
	for(int i = 0; i < I; i++)
		continue;
}

Build a DLL:
g++ -fopenmp -O0 -g -shared  cv2pdb_crash.cpp -o cv2pdb_crash.dll
Run cv2pdb:
cv2pdb -C cv2pdb_crash.dll
Observe crash.

Thanks for the report.
It's probably better to allow adding these "anonymous" UDT. IIRC the types referred to don't work properly in the debugger if there are no corresponding UDT entries.
I have pushed an appropriate patch to master.

Ok. I didn't actually try debugging anything with my workaround.


Missing semicolon

Copied and pasted this just to have a play and noticed there was a semicolon missing on this line which breaks the example.

Thanks!

No problem, thanks for the Laravel integration :smile:

Anytime :wink:


Bunch of errors when installing qiskit with pip on MacOS Sierra

I tried to install qiskit following the documentation page, on MacOS Sierra 10.12.6, python 3.7.2 anaconda distribution, but I get a series of errors.
Notably, the installation is still somewhat successful, in that I can import qiskit afterwards, but still something is clearly going wrong during the installation.
More precisely, what I did was create a new python=3 conda environment, installed some basic packages (numpy, pandas, etc.), and tried to install qiskit via pip install qiskit qiskit-aqua  as instructed in the docs.
See log.txt for a full dump of the relevant terminal output.
The first errors seems to be in this chunk of output:
Building wheel for qiskit (setup.py) ... error
  Complete output from command /Users/666/miniconda3/envs/qiskit/bin/python -u -c "import setuptools, tokenize;__file__='/private/var/folders/kx/l1xhdrl91_jgktjchpxgxfrhbpc7p1/T/pip-install-zgaegu8x/qiskit/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" bdist_wheel -d /private/var/folders/kx/l1xhdrl91_jgktjchpxgxfrhbpc7p1/T/pip-wheel-ibi5iv9i --python-tag cp37:
  running bdist_wheel

  ----------------------------------------
  Failed building wheel for qiskit
  Running setup.py clean for qiskit
  Building wheel for sympy (setup.py) ... done
  Stored in directory: /Users/666/Library/Caches/pip/wheels/6c/59/86/478e3c0f298368c119095cc5985dedac57c0e35a85c737f823
  Building wheel for psutil (setup.py) ... done
  Stored in directory: /Users/666/Library/Caches/pip/wheels/b4/9c/85/b73f594568ccc19c310ec6c61564346dd16c96aa2c35c5849a
  Building wheel for networkx (setup.py) ... done
  Stored in directory: /Users/666/Library/Caches/pip/wheels/68/f8/29/b53346a112a07d30a5a84d53f19aeadaa1a474897c0423af91
  Building wheel for mpmath (setup.py) ... done
  Stored in directory: /Users/666/Library/Caches/pip/wheels/63/9d/8e/37c3f6506ed3f152733a699e92d8e0c9f5e5f01dea262f80ad
  Building wheel for pycparser (setup.py) ... done
  Stored in directory: /Users/666/Library/Caches/pip/wheels/f2/9a/90/de94f8556265ddc9d9c8b271b0f63e57b26fb1d67a45564511
Successfully built sympy psutil networkx mpmath pycparser
Failed to build qiskit
Installing collected packages: ply, mpmath, sympy, urllib3, chardet, idna, requests, ntlm-auth, pycparser, cffi, asn1crypto, cryptography, requests-ntlm, decorator, networkx, jsonschema, marshmallow, marshmallow-polyfield, psutil, pillow, qiskit-terra, qiskit-aer, qiskit, cvxopt, pyobjc-core, pyobjc-framework-Cocoa, scikit-learn, qiskit-aqua
  Running setup.py install for qiskit ... done
Successfully installed asn1crypto-0.24.0 cffi-1.12.2 chardet-3.0.4 cryptography-2.5 cvxopt-1.2.3 decorator-4.3.2 idna-2.8 jsonschema-2.6.0 marshmallow-2.18.1 marshmallow-polyfield-3.2 mpmath-1.1.0 networkx-2.2 ntlm-auth-1.2.0 pillow-5.4.1 ply-3.11 psutil-5.5.1 pycparser-2.19 pyobjc-core-5.1.2 pyobjc-framework-Cocoa-5.1.2 qiskit-0.7.3 qiskit-aer-0.1.1 qiskit-aqua-0.4.1 qiskit-terra-0.7.0 requests-2.21.0 requests-ntlm-1.1.0 scikit-learn-0.20.0 sympy-1.3 urllib3-1.24.1

and a whole lot more errors came out when I tried to install with pip install qiskit visualization qiskit-aqua.

So that failed to build wheel output is expected it was introduced in #119 (and was the key patch for the 0.7.3 release) as part of a larger workaround. It's safe to ignore and should not be fatal. This is why it successfully installed right after it printed the error. You should be able to actually use qiskit after your first pip command succeeded.
As for the other error output in the log. That's actually being caused by you putting visualization in the list of packages. That's not actually a package which is a requirement, or a part of qiskit. Instead it's a separate package on pypi: https://pypi.org/project/visualization/ which has it's own requirements that require building things from source. Which for one reason or another don't build on your system (I'm not going to try to debug it). The docs you linked to say to add qiskit[visualization] to get the extra optional requirements for visualization not visualization (although that doesn't quite work as described in the doc yet, see #12).
I'm going to close this bug because it looks like nothing is misbehaving on the qiskit side. Feel free to re-open if I'm missing something.

@mtreinish

The docs you linked to say to add qiskit[visualization] to get the extra optional requirements for visualization not visualization (although that doesn't quite work as described in the doc yet, see #12).

Should the doc be revised? If so, what do you recommend it say instead?

@derivation that line in the doc should probably be:
pip install -U 'qiskit-terra[visualization]' qiskit-aqua qiskit
at least until we address #12

@mtreinish  I see, that makes sense. I would amend the docs though: the reason I used pip install qiskit visualization is that I tried using the command mentioned in the docs, qiskit[visualization], but that didn't work.
Actually, looking at the output again I see that that error was likely caused by the shell misinterpreting it:
$ pip install qiskit[visualization] qiskit-aqua

zsh: no matches found: qiskit[visualization]

I guess zsh interprets the square brakets as something else?

@lucainnocenti yeah zsh (which I'm also a user of) interprets [] as part of a regex pattern by default. So you'll need to escape it with either \[\] or '[]'. So in that example what I typically do is run pip install -U 'qiskit[visualization]' (although because of #12 you'll find that has no difference from pip install -U qiskit)


Error Flutter/Flutter.h

Hi, I recently came across this error in the plugin and shared preferences. I've tried everything and nothing seems to fix it.
Xcode's output:
↳
    In file included from /Volumes/DATA/Flutter/sdk-flutter/.pub-cache/hosted/pub.flutter-io.cn/flutter_secure_storage-5.0.2/ios/Classes/FlutterSecureStoragePlugin.m:1:
    /Volumes/DATA/Flutter/sdk-flutter/.pub-cache/hosted/pub.flutter-io.cn/flutter_secure_storage-5.0.2/ios/Classes/FlutterSecureStoragePlugin.h:1:9: fatal error: 'Flutter/Flutter.h' file not found
    #import <Flutter/Flutter.h>
            ^~~~~~~~~~~~~~~~~~~

I am closing all older issues. If this issue still exists in the latest version, please let me know.


chore: add Shayna as codeowner

Description

References

Testing

Automation

Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

Test Execution Passed.


dapp console: add mixpanel

closes https://github.com/ethereum-optimism/ecosystem/issues/103
This is the initial PR for Mixpanel setup for Dapp Console. You can change mixpanel project tokens based on environment type (we have two: testing and production).

Current dependencies on/for this PR:

#136  👈
main

This stack of pull requests is managed by Graphite.

potentially also drop the mixpanel directory name for something more generic like event-tracking or something along those lines so the imports also stay generic


[BUG] Unable to edit existing Anomality Detectors after changing Authentication to SAML

Describe the bug
Changing authentication from internal users to SAML authentication causes existing (created with internal user, that is now deleted) anomality detectors to be not editable with administrator access.
Details:
PUT /_dashboards/api/alerting/monitors/id-of-detector?ifSeqNo=1&ifPrimaryTerm=59
-> Response status code 200
-> Response content: {"ok":false,"resp":"[alerting_exception] User has no available detectors"}
To Reproduce
Steps to reproduce the behavior:

Change authentication method to SAML (Azure in this case), Log in with new user with pre-existing anomality detectors
Click on Anomaly detection on sidebar
Click on Detectors
Click on one of Detectors created
Anomality overview click on Edit alert settings
Click on Edit
Do not make any changes and click on Update at the bottom

Expected behavior
Should be able to successfully edit
Plugins
SAML Authentication from Opensearch AWS console enabled
Host/Environment (please complete the following information):

OS: Windows 11, Edge
Version 2.3 (Latest on AWS)

Hi @amitgalitz ,
The "Details" section gives the technical code that I get trying to edit a detector that is visible in the UI.
I can see same detectors as with internal user, but all actions against them causes the "Details" sections error (I looked it up in network tab under developer tools)
About the backend roles - is there an api endpoint that I could call to see which backend role is assigned to specific detector? I can report back when I know which role to add. My gut feeling is that they do not have same roles (cant check from UI since internal user is long gone)

To get all the information related to the user backend role you can use the get detector API for that specific detector (this gets all detector info including the ownership related information):
GET _plugins/_anomaly_detection/detectors/<detector-id>
To do a search on all detectors you can use GET _plugins/_anomaly_detection/detectors/_search with a body including a search query like:
{
  "query": {
    "match_all": {}
  }
}

Added the roles I could find and no change.
However, I realized this is not about detectors. I managed to edit the detector, but not the alert associated with it. (Look at steps to reproduce: Edit alert settings and "Details" API endpoint). So the topic of the issue is misleading, it is error editing alert that is created through detector.

That's a good point, I missed that you went to edit alert settings and this is where you got the issue. In that case it might be a similar logic but with the alerting monitor itself. Once you created an alerting monitor the logic regarding ownership of that monitor is separate from the detector itself. The monitor simply sits on top of the detector and scans it for relative to the alerting logic configured.
I believe alerting has similar backend role logic. I will reach out for confirmation on this and steps to get monitor backend role if that is indeed issue.

@lehmat are you able to edit the alerting settings as Admin user after converting to SAML?

@lehmat, do the user have matching backend roles as the AD detector? If not, it will fail even as admin.
@amitgalitz, there needs to be a code change here to allow admins to access all detectors. Additionally changes might also be needed here to take care of the admin case.

@lehmat, do the user have matching backend roles as the AD detector? If not, it will fail even as admin.

@lezzago I looked at the endpoints given in this ticket and looked up the roles in the response. Added them to my user and retried the operation - no change.
Is there any PUT commands I could do to fix them pointing at my new user? The code changes to enable admin to edit them seems legit, but given AWS is lagging already multiple minor versions behind - I should not wait for the fix to be available

@amitgalitz I am assuming here that AD has logic to allow admins to see all detectors regardless of their backend role. If that is the case, the user might not have the same backend roles and the pieces of code I shared is causing that problem.

We might actually want to fix it on AD and alerting so we allow admin access to everything no matter the backend roles. I am still a little unsure though how we can can access the Anomaly Detector but at the same time when trying to edit the monitor we get the "no detectors found error".

Hi @lehmat could you possible paste the backend roles of the monitor, the detector and the user. Can obviously be redacted but just so we have a better understanding of what the situation is to better understand if its possible to have an API workaround.

@amitgalitz Sure, but I might need help to get what you exactly need. Here is the ones I could think of being useful API call:
GET _dashboards/api/alerting/detectors/sZofY4UB_zoY8BXQij1K
{
    "ok": true,
    "detector": {
        "name": "Cloudtrail",
        "description": "",
        "time_field": "@timestamp",
        "indices": ["log-aws-cloudtrail*"],
        "filter_query": { "match_all": { "boost": 1 } },
        "detection_interval": { "period": { "interval": 5, "unit": "Minutes" } },
        "window_delay": { "period": { "interval": 1, "unit": "Minutes" } },
        "shingle_size": 8,
        "schema_version": 0,
        "feature_attributes": [{ "feature_id": "sJofY4UB_zoY8BXQiT1K", "feature_name": "Region", "feature_enabled": true, "aggregation_query": { "region": { "value_count": { "field": "errorCode.keyword" } } } }],
        "ui_metadata": { "features": { "Region": { "aggregationBy": "value_count", "aggregationOf": "errorCode.keyword", "featureType": "simple_aggs" } }, "filters": [] },
        "last_update_time":<PHONE_NUMBER>914,
        "user": { "name": "<previous-localuser-that-is-deleted>", "backend_roles": [], "roles": ["security_manager", "all_access"], "custom_attribute_names": [], "user_requested_tenant": "Security" },
        "detector_type": "SINGLE_ENTITY"
    },
    "version": 4,
    "seqNo": 3,
    "primaryTerm": 3
}

AND trying to edit this one:
PUT _dashboards/api/alerting/monitors/sZofY4UB_zoY8BXQij1K?ifSeqNo=1&ifPrimaryTerm=59
{"ok":false,"resp":"[alerting_exception] User has no available detectors"}

My current user info:
GET _dashboards/api/v1/auth/authinfo
{
    "user": "User<EMAIL_ADDRESS>backend_roles=[c31a6137-01f3-4a76-bd2b-f18a956f654d], requestedTenant=Security]",
    "user_name"<EMAIL_ADDRESS>    "user_requested_tenant": "Security",
    "remote_address": "<IP_ADDRESS>:33582",
    "backend_roles": ["c31a6137-01f3-4a76-bd2b-f18a956f654d"],
    "custom_attribute_names": ["attr.jwt.roles", "attr.jwt.sub", "attr.jwt.saml_nif", "attr.jwt.nbf", "attr.jwt.exp", "attr.jwt.saml_si"],
    "roles": ["security_manager", "all_access"],
    "tenants": { "Sales": true, "Security": true, "global_tenant": true<EMAIL_ADDRESS>true },
    "principal": null,
    "peer_certificates": "0",
    "sso_logout_url": "https://login.microsoftonline.com/xxxxxx/saml2?SAMLRequest=........"
}

@lehmat, do you have this setting enabled: plugins.anomaly_detection.filter_by_backend_roles?

How do I check this? I have not installed it myself, rather using AWS managed service

@lehmat, do you have this setting enabled: plugins.anomaly_detection.filter_by_backend_roles?

How do I check this? I have not installed it myself, rather using AWS managed service

You can run this in dev tools: GET _cluster/settings?include_defaults=true

It is false

Added code in AD backend to check if user has all_access so admin can view all detectors and it has been released since OS 2.7 https://github.com/opensearch-project/anomaly-detection/pull/850.
Closing this issue, feel free to reopen if you feel that this change doesn't solve the problem here.


clean up retagging recursive traversal a bit

The last refactor left some strange things behind -- no need to have qualify be a separate function, really.

@bors r+

:pushpin: Commit<PHONE_NUMBER>0d6638fc1bc541c2e85c60be325832 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit<PHONE_NUMBER>0d6638fc1bc541c2e85c60be325832 with merge 22cf3fca9b7fe4865199a4ebad697823a9c834c4...


HashedModuleIdsPlugin with externals where multiple keys map to the same value can result in flip-flopping module IDs

Bug report

What is the current behavior?
When using HashedModuleIdsPlugin along with externals where multiple keys in the externals object maps to the same value, the module ID can change depending on which key is encountered first.
This happens because:

The HashedModuleIds plugin generates the module ID based on module.libIdent, which for externals is the path of the import which matches the key in the externals configuration. https://github.com/webpack/webpack/blob/945a9514ca23dfa3438df2ca7abd1238a2ea0c72/lib/ExternalModule.js#L25-L27
This id is stored on the module instance for future reference. https://github.com/webpack/webpack/blob/bc840ec8747ba19c9e87629f2959ed2e65a9ffbe/lib/HashedModuleIdsPlugin.js#L23-L36
Externals that result in the same code share a module instance.

If the current behavior is a bug, please provide the steps to reproduce.

Repro instructions here: https://github.com/lencioni/webpack-externals-module-id-bug-repro
What is the expected behavior?
The hashed module ID does not change when the imports are re-ordered.

Other relevant information:
webpack version:  noticed on 3.10.0, reproduced on 4.19.0
Node.js version: N/A
Operating System: N/A
Additional tools: N/A

hmm yes, I think this is a bigger issue.
Storing userRequest on Module is probably incorrect, since there could be multiple per module. Using it as libIdent is also bad. libIdent is probably also being abused by a few Plugins.
I gonna make some refactoring for this for the next major. I don't think this can be fixed in a non-breaking way in v4.
As workaround avoid these situations, i. e. add comments to the externals to make them different instances.

For my particular use case I am switching to NormalModuleReplacementPlugin.


Documentation not clear

If Font Awesome is not already included in the consuming application, add the following your applications config/environment.js file.

'ember-cli-notifications': {
    includeFontAwesome: true
}

Where exactly in the environment.js file?

Should say
If Font Awesome is not already included in the consuming application, add the following your applications config/environment.js file as a property of the ENV object.


Improve news fragment file name parsing

The new version uses category as the reference point to later infer the issue number and counter value.
As a consequence, it will return (None, None, None) if the base name doesn't contain a valid category.
The new parser allows using file names like 123.feature.1.ext which are convenient when one wants to use an appropriate extension (e.g. rst, md) to enable syntax highlighting.
Closes #1.

Also submitted upstream as https://github.com/hawkowl/towncrier/pull/173.


Added Exporting of Bloat

Was playing around a bit with this package and saw that you forgot to export the newest protection you added.

Oh wow, of course out of all things I forgot that! Thanks 😅


Parsedown eats closing ) after a link

Try this in Parsedown:
### Test ([source](https://google.com))

it will be rendered as

Test (source

PHP Markdown gets it right, and here's a test with GitHub's parser:
Test (source)

I'll see what i can do. Give me by the end of the week.


why some domain's organization field is registrant_organization and other is registrant_org or org

in com entry is org field
in cl is registrant_organization
in jp is registrant_org

why not use one specific field like registrant_organization in all tld domain?

that is a good idea - PR welcome to standardize this field

Closing as no one seems motivated to standardize this field


mega: add cleanup support

Fixes #3138

What is the purpose of this change?
Add cleanup support for Mega remote
Was the change discussed in an issue or in the forum before?
#3138
Checklist

[x] I have read the contribution guidelines.
[x] I have added tests for all changes in this PR if appropriate.
[x] I have added documentation for the changes if appropriate.

[x] I'm done, this Pull Request is ready for review :-)

Also, I really need to remember to run golint.

Nice!  Did you run the integration tests for this?  go test in the backend/mega directory with a TestMega remote setup?

I ran the integration test before committing. There was one error but it was from PublicLink. That will need to be fixed separately.

I ran the integration test before committing. There was one error but it was from PublicLink. That will need to be fixed separately.

The PublicLink test has been broken for a long time.  The feature works, but the test is broken - ideas to fix appreciated!

I've given the travis build a kick - it has been really unreliable recently :-(

Looks like the build passed now. Any idea why it seems to be failing so often?

Looks like the build passed now. Any idea why it seems to be failing so often?

Would not wish to be pushy... are you going to provide a test/branch release with this feature any time soon so I can try it?
Thanks

Sorry, this is on my own repo since I don't have write access here so I can't use the beta release system as it is. I'll try to trigger a build for an unofficial version.

Okay, I built an unofficial version. Available here: https://nc.ydgkim.com/s/ebCoLfGKHaKcQH5
Switch back to ncw's version as soon as possible.

Sorry for the delay merging this.  The code is great - I've just been a bit overloaded - sorry!
I'll merge it now - thank you very much.


[NOP] test/3_branch.js readability improvement

1, assure assert.equal(actual, expected, message) order
2, better line breaks

Oops, forgot to ping you :) @dmitrig01

:+1:


Risk of crash during migration in a emergency cluster

Description of the risk
Sometimes, for various reasons, the operations team upgrades only part of the cluster. In our practice, a cluster usually consists of two parts. But in theory it can have three or more parts.
When there are different versions of the application on the cluster in different instances, this is, of course, an abnormal state of the cluster. However, it is possible to run the migration on all leaders with the "migrator" role.
What do you need
I would like to have some kind of verification mechanism that would not allow migration to be launched if at least one instance has an inappropriate version of the application.

Нашёл такую же задачу от декабря 2021:
https://github.com/tarantool/mfactory/issues/163


[Dashboard] Add revisited footer for different user account types

#14470

We need to update snapshots as well!

And reviewing the usability changes, let's use this PR to open links in a new tab :)


Updating the repo description (cache refresh).

We would like to refresh Burn repository description. It has been updated to "Burn - A Flexible and Comprehensive Deep Learning Framework in Rust" from  "BURN: Burn Unstoppable Rusty Neurons"

By default, it grabs the description from crates.io. if you want a different description, you can add an override for the description field to _data.yml/crates.yaml
PR welcome (and typically merged quicker), but I may try to make the change sometime when not in my phone...

This appears to have been updated by virtue of crates.io at some point. Closing.


ios fuse preview app throws errors

It just doesn't work anymore.
If it is not a totally blank project I keep getting errors like:
Fuse.Reactive.Each.SourceLineNumber' has no setter
Any solutions available?

If you're on a mac (with XCode installed) you could try deploying a custom preview app from the menu under Preview -> Preview on iOS and see if the problem persists

i have the same error in ios preview app and android as well

Unfortunately updating the Fuse Preview app was forgotten during the release of 1.9.0, in the midst of all the changes.
I had a plan to get that done today and also get the process documented. However, there is an error preventing me from doing so, due to some changes before the open sourcing.
Specifically, running the script Source/Preview/App/build.sh now gives me this error:
Compiling syntax tree
Fuse.Simulator/SimulatorClient.uno(149.36): E3102: There is nothing named 'BinaryMessageInbox' accessible in this scope. Are you missing a package reference?
1.72 s

It should be safe to remove the _alsoReceieveMessagesFromPipe variable and all the references to it, including the line causing this error.
(For reference, this class appears to have been copied from https://github.com/fuse-open/fuse-studio/blob/master/Source/Preview/Core/SimulatorClient.uno where this has since been removed)

@BeeWarloc I make a pull request by testing @lorents 's suggestion - both Android & iOS app built locally worked well. Would you kindly review & approve my pull request, and then release a new Preview apps on App Store and Play Store?

Sorry about the delay, an updated version of the preview app compatible with 1.9.0 should now be available on both the Android and iOS app stores.
I have tested that the Android release works, but unfortunately I don't have an iOS device at hand. So I'd appreciate feedback on whether the iOS version actually works or not.

It works, thank you


useDocuments cannot be used with a variable

Hi!
I stumbled upon an issue I couldn't explain. When I run npm run dev in a dummy app with:
<script setup lang="ts">
const foobar = '~/pages'
console.log(useDocuments(foobar).value.length) // it always logs 0
console.log(useDocuments('~/pages').value.length) // it logs 3 (in my case)
</script>

When looking at the transpiled code in my browser, I saw:
import _documents_0 from '/@islands/documents?pattern=ooba';
import _documents_1 from '/@islands/documents?pattern=~/pages'

I could find why this has this behaviour: https://github.com/ElMassimo/iles/blob/80eb10366ddefea217f32d138b8b79e1aea5366e/packages/iles/src/node/plugin/documents.ts#L134
I am deeply troubled by this static analysis of the code, which fails when using useDocuments with a variable. Is it a usual pattern, or a design choice from your end? This seems really flaky to me, and needs at the very least to be documented.
(otherwise, iles is nice :) thanks for the great work)

The reason why I stumbled upon this is because I want to build a wrapper to order the Documents that looks like:
import { ref, computed } from 'vue'
import { useDocuments } from 'iles/dist/client'

export const defaultCompareFn = (x: any, y: any) => x.order - y.order // this is a convention I use to sort my pages in the menu

export const dateCompareFn = (x: any, y: any) => x.date - y.date

export default (globPattern: string, compareFn = defaultCompareFn) => computed(() => useDocuments(
  globPattern).value.sort(compareFn))

I would have liked to re-use this composable for my navbar and my blog posts.

All right, thanks for the swift reply. This is my first project with Vite!

Documented here:


feat: Show error in tab

Issue - https://github.com/tscircuit/snippets-cli/issues/7
https://github.com/user-attachments/assets/a905858b-156f-45c2-87b7-e7a9e68c57d3

@seveibar Looked more onto this, I think we don't even need the eval-worker listener to emit event. Cause the circuit execute function is throwing the error itself with the message which has been passed by the core. So, how about we just catch that error here and remove the listener?

@seveibar Looked more onto this, I think we don't even need the eval-worker listener to emit event. Cause the circuit execute function is throwing the error itself with the message which has been passed by the core. So, how about we just catch that error here and remove the listener?

@imrishabh18 yes that's fine, try to reduce the usage of try catch, e.g. you can do .execute().catch() to increase readability

@seveibar Let me know if this is good to merge or not!

Will add the log and then merge it!


Develop

Merging Gallery changes and several other changes with MASTER BRANCH

by  Joy

Merging Gallery changes and several other changes with MASTER BRANCH

By Joy


Generate a Json Schema $Id or Id field

I am trying to work out how to generate the following JSON Schema snippet with Id. I can't find a way to do with using the settings. I understand that before draft 4 it was $id and that later versions have just id. I can work with either as long as i can specify the id and that it is a valid schema. I am not too fussy if its draft-04 or later as long as its valid for that draft.
{
"$schema": "http://json-schema.org/draft-04/schema#",
"id": "Customer",
"title": "Customer",
"type": "object",
"additionalProperties": false,
"properties": {
"lastName": {
"type": [
"null",
"string"
]
},
"firstName": {
"type": [
"null",
"string"
]
},
C# schema generation settings:
var schema = JsonSchema.FromType(new JsonSchemaGeneratorSettings()
{
SerializerSettings = new JsonSerializerSettings
{
ContractResolver = new CamelCasePropertyNamesContractResolver(),
Converters = {new StringEnumConverter()},
},
});
var schemaAsJson = schema.ToJson();
I would be grateful for any help you can give.

You need to add the additional property "id" to the ExtensionData dictionary of the generated schema or use the [JsonSchemaExtensionData] attribute on the class to add it


RTCanvasCalculator

RTCanvasCalculator.cs creates all the node whenever it calculates a canvas. So if it calculates every frame there's a significant drop in framerate.
What's the proper way to calculate a canvas at runtime?

What do you mean with 'Creates all the nodes whenever it calculates'? There is no copy made or anything, it is directly accessing the asset and performs the calculation on it. Or is this related to the debug that automatically gets called, in which case remove it if you don't need it, it wasn't meant to be called each frame and debug it anyways;)
Please clarify this, thanks!

Steps to recreate:

Add the RTCanvasCalculator component to a GameObject
Add a canvas to it
Add a Debug.Log in a Node's Create method
Call CalculateCanvas  on  every frame

The Debug.Log will be called every frame.
RuntimeNodeEditor doesn't do this. I've tried to pick it apart... but just get lost in this.

Oh, ok so without trying I now know why. It is caused by #72, basically one node of each type is created on initiation, which is in the end bad design but I chose so when I first started this project to be able to overwrite the Create method for each node type (not possible with statics). Hard to overcome but that's the root cause.
The problem that is most noticeable here is then caused by a different problem that is easier to fix. Node initiation consists of two parts, normal and GUI-related initiation, where GUI related initiation can only be called from OnGUI. When using the framework, it is initiating but as it wasn't called in GUI code it can't initiate the GUI, and currently that means it is still flagged as not initiated. So that's why a call to the framework each frame currently causes it to re-initiate each frame.
I'll fix it and report back:)

Great. Thanks

Fixed in 786a804 :)


Add support for displaying custom token address

@lochie would be great if you can take a look at this PR. so close to having it on our app :D

@lochie added an example on testbench

any update?

@lochie would be great if we can get this in too :D


Katex commutative diagram

Hi, commutative diagram \xymatrix does not work in Katex
$$ \xymatrix{
FX \ar[r]^-{Ff} \ar[d]{\eta_X} & FY \ar[d]^{\eta_Y} \
GX \ar[r]-{Gf} & GY\} $$
This is very frequently used in mathematics!

https://codepen.io/aziis98/pen/KJYqZV
I tried making a prototype of this (at the moment this is all an enormous hack). I wanted to see if it was easily possible to parse the xymatrix syntax and compute all the arrow positions and sizes.
I don't really know if it is possible to create a parser plugin for Katex that does more then adding some simple macros but this may be a starting point.
Notes on the Code
I tried to comment the code as better as I could. For now I only support simple arrows "->" and double ones "=>" and the support for arrow labels was just too bad to include here (it is commented out).
The parsing part completely bypasses Katex's parsing and macro system (but I think that it might be relatively simple to convert a Katex Macro/Function - for now I just splitting strings and call Katex's "render(expression, node)" on each sub expression)
Probability the most useful part might be the arrow creation. I compute the position of start & end of the arrow and it's angle and then rotate and translate it's container with CSS. For porting to Katex one would just need a function for creating an arrow of given length - I haven't really understood how stretchy.js works (?) for doing this.

Hi @aziis98. This looks really interesting. I noticed that you had some very insightful comments:
// Without the node already in the DOM it looks like
// it doesn't work. Probably because I can't compute
// all the bounding boxes necessary for this to work.
// Tecnically Katex already knows all the bounds
// of its objects (?)

Actually, KaTeX only knows the vertical bounds of all its objects. It relies on the browser to set most horizontal alignment. For fraction bars, KaTeX has always used a span border and a little CSS trickery to make that span width match the width of the numerator and denominator. PR #670 put SVG images of very long arrows into spans with overflow: hidden; and applied the same CSS trickery to make that span width match the subject matter.
The benefit of that approach is that KaTeX never has to query the browser to find the bounds of any of its objects. KaTeX can finish its work and shut down without any cycles of render, measure, render. And this partly explains why KaTeX is fast.
I can think of no CSS trickery that can find the correct geometry for the diagonal arrows in \xymatrix. We either have to settle for the AMS CD package or we have to take your approach and do at least one render, measure cycle.
I'm glad that KaTeX does not have to do render, measure cycles for most math. For the occasional special case, like commutative diagrams, I think it would be okay to use your approach. Then again, that's probably not my call. It would be interesting to hear what others think.

We've always wondered whether we'd need a measurement cycle, but so far @ronkok has proved otherwise. I could see doing such a measurement in an extension, perhaps even a generic "measurement" extension that can callback other extensions. I cannot see doing it in base KaTeX because it's simply incompatible with the renderToString workflow (e.g. server side).
KaTeX could compute an approximate horizontal geometry, as it does have the width of all the characters. Is there any chance that's accurate enough for arrows? I'm guessing @ronkok's thought about this, but could be worth experimenting.

Good point regarding server side rendering. I like the idea of a measurement extension. It could then also support \rotatebox and \cancelto.

Say we wanted a measurement extension. How to go about that? I have some thoughts.
Say we did the following inside KaTeX proper:

Write a series of non-standard unary functions that display stretchy arrows. Give them names like \@rightarrow{<width>}, \@Rightarrow{<width>}, etc. Each function would return a span containing the arrow’s SVG set inside a box that has a hard width defined. This box would be attached to the baseline.

Write a non-standard \@rotate{<object>}{<angle>}{<width>} function. The object’s width must be an argument, because many KaTeX objects don’t know their own width.

Expose a defineTwoStageMacro function, similar to the existing defineMacro function that was used by the mhchem extension, but with an additional callback that would execute after a measurement is taken.

Then the extension would define a macro for \xymatrix that:

Expands into a {array} environment and passes control to KaTeX.

Waits until the first phase is rendered, and takes the necessary measurements of the DOM.

Calculates the geometry for any diagonal arrows and return a revised {array} environment in the callback function. This second set of code would include things like \<EMAIL_ADDRESS>

Is this sort of thing feasible?

have there been any updates on this? this is the only thing stopping me from always using KaTeX instead of MathJax!

No progress, I'm afraid. I may take this up after we complete the AMS automatically numbered environments such as equation and align. I don't know when that will occur.
How would you feel about getting the AMS {CD} environment instead of \xymatrix?

The {CD} environment is pretty limited, but better than nothing. I suppose it depends on how much simpler it is to implement than \xymatrix. I really agree with the suggestions above, in that avoiding measurement cycles is great, but the idea of an extension that can be enabled simply for commutative diagrams seems really nice. At the moment, I know of at least one project that uses KaTeX for inline maths, but then loads MathJax for any commutative diagrams, so this sort of special treatment for commutative diagrams is already sort of accepted as a necessity I think.

If we're comparing commutative diagram extensions, supporting tikz-cd would be even better. I'm not sure what the difficulty of implementation compared to xymatrix would be, though.

This feels like a really complex thing to add and should probably be done as an extension.

As an update to this, AMS's CD environment is now supported in the master branch, thanks to #2396. So you should give that a try! (e.g. on our demo page) Hopefully this suffices for many people's commutative diagrams.
xymatrix could still be a target, so I'll leave this issue open to track that.  #219 tracks tikz, but that would be a more major undertaking (and likely a contrib module).

How does one use 'master' on the demo page?

@jedbrown Oops, I was wrong: the demo page uses the official release, not master. (I was confused because the website's documentation is compiled using master, but that's done statically, so there's no live version to play with.)  You'll need to build from source and then e.g. yarn start to try out a local demo site -- or install the master branch in your local project.

You can also try out the preview site from the PR which does include the new code in the playground: https://deploy-preview-2396--katex.netlify.app.


How to read stitched image from an acquisition

Hello,
So the code I'm trying to write is supposed to acquire an acquisition and then read the dataset and output the stitched image.
this is the code and the result where it gives the following error: Exception : Method not yet implemented for in progress acquisitions

If I do acq.acquire({'row': 0, 'col': 0}) it still doesn't let me read the stitched image as the final product.
If I use a multi-d acquisition I get the same error but the stage goes through all the locations I'm interested in and then with
img = dataset.read_image() plt.imshow(img)
I get the following

Any help would be greatly appreciated because I've been stuck on this for last 2 weeks and anything I try just doesn't work.
thank you.

As the error says, that function is not yet implemented. Try opening the dataset from its path on disk, as described here


(PUP-4345) Backport Zpool mirror fix for Puppet 3.8

This commit backports 9464fe08 from master, which fixed a longstanding regression in the Zpool resource. Also backported are the corresponding spec and acceptance tests from commits 91243475 and a907ddd8 respectively.

As @joshcooper did in the original ticket, I verified that his updated acceptance test completes successfully in Solaris 11:
bundle exec rake ci:test:git FORK=whopper SHA=PUP-4345/port_zpool_3.8 TESTS=tests/resource/zpool/basic_tests.rb CONFIG=./config/nodes/solaris11.yaml

          - Test Case Summary for suite 'tests' -
   Total Suite Time: 39.27 seconds
  Average Test Time: 39.27 seconds
          Attempted: 1
             Passed: 1
             Failed: 0
            Errored: 0
            Skipped: 0
            Pending: 0
              Total: 1


Wrong query to check OverQuota

The query on the documentation to check for OverQuota (Data collection stopped due to daily limit) is outdated.
The query in the docs states:
_LogOperation | where TimeGenerated >= ago(7d) | where Category == "Ingestion" | where Operation has "Data collection"
However, the "Operation" field is a ID, not a text field. Therefore, nothing will be returned by this query (even if the daily limit was hit) since the filter is wrong.
The correct query should be
_LogOperation | where Category == "Ingestion"  | where Detail contains "OverQuota"
Query is also wrong at: https://docs.microsoft.com/azure/azure-monitor/logs/daily-cap#alert-when-daily-cap-is-reached

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

ID: b6d511e9-3132-c830-8f68-086ec8577480
Version Independent ID: a5ff4b87-395a-3a8a-afda-acdd3f74d528
Content: Monitor health of Log Analytics workspace in Azure Monitor - Azure Monitor
Content Source: articles/azure-monitor/logs/monitor-workspace.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @guywi-ms
Microsoft Alias: guywild

The query in the [docs states)](https://docs.microsoft.com/azure/azure-monitor/logs/daily-cap#alert-when-daily-cap-is-reached) states:
_LogOperation | where Operation =~ "Data collection stopped" | where Detail contains "OverQuota"
However, the "Operation" field is an ID, not a text field. Therefore, nothing will be returned by this query (even if the daily limit was hit).
Same problem as mentioned on the main issue, but on a different doc page.

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

ID: ae90b263-4498-a600-da27-55b3b1fc41d8
Version Independent ID: cea3914e-db4f-0d32-7f7a-d12feaea953e
Content: Set daily cap on Log Analytics workspace - Azure Monitor
Content Source: articles/azure-monitor/logs/daily-cap.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @guywi-ms
Microsoft Alias: guywild

Please note, that is this reporting a similar issue in 2 docs page:
Content Source: articles/azure-monitor/logs/monitor-workspace.md
Content Source: articles/azure-monitor/logs/daily-cap.md

@andrefboliveira Thanks for your comment. We will review the issue and get back to you shortly.

Assigning to content author @guywi-ms for doc-enhancement.

#assign-reviewer: @DaleKoetke

The following reviewer(s) could not be added because they aren't a valid GitHub ID, aren't a valid collaborator on this repo, or are the author of the PR: dalekoetke

@andrefboliveira Thanks for your dedication to our documentation!
We've created an internal work item to update the queries shortly.
We are closing this issue for now, but feel free to comment here as necessary.
#please-close

Hi @guywi-ms
Is there any update on this topic?
I see that both pages still show the wrong query:
https://learn.microsoft.com/azure/azure-monitor/logs/monitor-workspace#operation-data-collection-stopped
https://learn.microsoft.com/azure/azure-monitor/logs/daily-cap#alert-when-daily-cap-is-reached
can you take a look?


Remove call to vTaskDelete from runDemoTask

Description
runDemoTask is created using Iot_CreateDetachedThread API and the
threads created using this API must not call vTaskDelete as it is
automatically called when the thread routine returns. Explicitly calling
vTaskDelete results in a memory leak as the internally allocated
threadInfo_t does not get freed.
This was reported here:
https://github.com/aws/amazon-freertos/issues/989
Checklist:

[x] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks


Typo in assert. Fixes #101

Codecov Report

Merging #102 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #102   +/-   ##
=======================================
  Coverage   99.81%   99.81%
=======================================
  Files          18       18
  Lines        1095     1095
=======================================
  Hits         1093     1093
  Misses          2        2

Impacted Files
Coverage Δ

gpflowopt/pareto.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


timeline for stable release

Hi,
Is there any timeline for a stable release?

Or a roadmap or milestones?

+1

No publicly available timeline, sorry. We're working on it, but can't share those details yet.


Fix typo in Cloudstack buildspec

The Cloudstack tests are all currently skipped due to the wrong indendation in the pre_build commands. Fix the indendation to pass the commands as an array to the pre_build phase.
Issue #, if available:
Description of changes:
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/cherry-pick release-0.18


Spurious unit test failures with explicit copies

For some time we have seen spurious unit test failures for the explicit copy tests involving host accessors. I did some investigating and it turns out there's a massive race condition between the explicit copy and the host accessor that is used to access the data afterwards for validation. However, it might not be hipSYCL's fault:
In those tests, the explicit copy operates on two accessors, at least one of which is a host accessor. The problem is that, since host accessors are not associated with a command group handler, a SYCL implementation cannot know that the accessor is actually used by the copy operation [1]. Because of this, of course it cannot create a dependency between the explicit copy and the host access afterwards since it doesn't know that the accessor is used by the copy. As a result there are no synchronization guarantees whatsoever between the explicit copy and the host access.
I'm uncertain now whether the intent of the SYCL specification is to allow this at all as it doesn't work with the usual rules for accessor dependencies.
@psalz @fknorr Since you added those tests, do you depend on support for this? There are various approaches we could take:

remove those tests entirely, interpreting the SYCL specification as not allowing host accessors in explicit copy invocations
keep the tests, but acknowledge that additional manual synchronization is required by inserting queue::wait() calls before the host access to make sure the memcpy has completed
Add additional dependency semantics for explicit copies: Check the copy() arguments, and when an argument is a host accessor, manually add it to the operation's requirements. It is not yet entirely clear how this can best be implemented as host accessors work somewhat differently than the device accessors that the handler's requirement handling is designed for.

[1] At least by relying on the regular mechanisms for that. See the proposed solutions.
EDIT: The specification also requires that host accessors block other operations on their buffer for the duration of their existence. This is not yet implemented in hipSYCL - if it were, I suspect that using host accessors in explicit copy operation might cause additional problems such as deadlocks.

PR #382 now temporarily disables the tests in question until we have decided on a solution strategy.

I'm uncertain now whether the intent of the SYCL specification is to allow this at all as it doesn't work with the usual rules for accessor dependencies.

No, it is not allowed per spec. While not at all obvious (certainly I didn't know this when I added support for explicit copies originally), it was pointed out to me by someone at Codeplay that explicit copies are considered specialized kernel functions, and it says somewhere in the spec that it's illegal to use host accessors from within kernel functions. Clearly this should be made more explicit in the spec (see also SYCL-Docs#104).

@psalz @fknorr Since you added those tests, do you depend on support for this?

I believe at some point we used this in Celerity, but I just checked, and it looks like we no longer do. So I'd be fine with removing it!


Status plugins

GET /api/v1/status now returns a plugin key:
{
  "node": {...},
  "org": {...},
  "defaults": {...},
  "plugins": {
    "blockchain": [
      {
        "connection": "ethereum"
      }
    ],
    "database": [
      {
        "connection": "sqlite3"
      }
    ],
    "dataExchange": [
      {
        "connection": "ffdx"
      }
    ],
    "identity": [
      {
        "connection": "onchain"
      }
    ],
    "sharedStorage": [
      {
        "connection": "ipfs"
      }
    ],
    "tokens": [
      {
        "connection": "erc20_erc721"
      },
      {
        "connection": "erc1155"
      }
    ]
  }
}

Codecov Report

Merging #689 (a0aa842) into main (1c688c2) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #689   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          317       317
  Lines        19485     19514   +29
=========================================
+ Hits         19485     19514   +29

Impacted Files
Coverage Δ

internal/orchestrator/status.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Support for .Net 8

Is .Net 8 support planned?

I am using it on .Net 8 and have no issues idk how i fixed it but it works

it should work just fine with .NET 8. @TheresasAccount do you have anything specific in mind that's not working in a .NET 8 project?

Fixed it by just adding it to the Maui application and having the page declared there instead of the class library.


create high level development terms where missing

For the Alliance slim it would be helpful to have high level development terms for all anatomical systems. We are missing these terms for a few systems:
endocrine/exocrine gland phenotype - add an abnormal gland development term under abnormal gland morphology and collect the existing development terms from this branch under the new parent.

Consider doing the same for abnormal skin adnexa morphology

added abnormal gland development - MP:0020973

added abnormal gland development - MP:0020973


Update to 2.0.2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@ericdill what happened here?
Also can this go in?

@conda-forge-admin, please rerender

Also can this go in?

soon as CI says its good i guess


Make sure your i element has a closing tag

http://www.freecodecamp.com/challenges/waypoint-add-font-awesome-icons-to-our-buttons has an issue.
It says I am missing a closing tag for my i element, but I cannot find any issues in my code..

Thank you for your detailed issue! I've pushed a fix for this.


fix(TestBackend): prevent area mismatch

TestBackend::new(400, 400) resulted in a Rect with width/height 255,255 and a locally stored width/height of 400,400 which is wrong.
TestBackend::new could have been fixed, but this prevents this kind of error completely as its stored only at a single location now.

Is this a breaking change? TestBackend implements serde derive. Reading from an older version should still work but reading from a newer version will not.


make collate.py executable

Just a proposal: make collate.py executable
Otherwise (when executing python collate.py instead of ./collate.py) the
path from which the encodings and so on are loaded is not correct.

Good suggestion, thanks!


STL-2068: provide new anchor button component

Short Description

new anchor button component created

Changes

adds a new generic anchor button component to use for  all different cases to get rid of the "wild growth"
next step is to use this for share buttons
step after next is to exchange all the other anchor buttons

Feedback

something missing?
Hints:
You have to start storybook to see it in action
the one with the icon and text (play, youtube): hover state for icon has to be set in the parent component where it is used, because colors could be different

How to review this Pull Request
Link to Confluence (internal)

I will close this one in favor of #814


Built-in VB language LT / GT operators may appear not as operator color

Does this issue occur when all extensions are disabled?: Yes

Version: 1.78.2 (user setup)
Commit: b3e4e68a0bc097f0ae7907b217c1119af9e03435
Date: 2023-05-10T14:39:26.248Z
Electron: 22.5.2
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: Yes
Steps to Reproduce:

Create a VB file with the following line:

If JobNameLen < 0 Then

Observe that the < seems to be marked with the mismatched bracket color.

instead of line

Probably the ideal solution is to add an unbalancedBracketScopes value of keyword.operator to the package.json file.
However, it might also be worth noting that I do not understand why the < or > are getting this treatment, because I am not observing them being color matched.

However, it might also be worth noting that I do not understand why the < or > are getting this treatment, because I am not observing them being color matched.

@hediet would you expect angle brackets in VB to get color matching when bracket pair colorization is turned on?

As a quick workaround, I'd suggest to remove <...> as bracket pair from VB (and wait of a PR to configure it properly).


Fix indentation in session doc

Summary
Fix the code example in session doc that did not have proper indentation inside a context.
Details and comments
Fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.658%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2583

Relevant Lines:
3555

💛  - Coveralls


Is this project dead?

105 open issues and 2 PRs with no activity and no new version for over 3 months.
Is this project dead @perak ?

@cyclops24 it's not dead! But is temporary on hold - I got heavy workload last 3 months plus baby @ home. I'm finishing one project for customer and will fo us full power to meteor-kitchen soon.

Hi Perak,
happy to discuss future projects as well once you are done with the happy customer. But obviously don't want to take your focus from m-k ;=)
Thx for the great work

@cyclops24 I'll close this issue :)

@perak Congratulations for good news about baby. :-)
I suggest that if you want, open meteor-kitchen-CLI source for future process.


better warning handling for slack class

@jenshnielsen @nulinspiratie
Changes proposed in this pull request:

Add a custom SlackTimeoutWarning so one can filter the slack timeout messages
Make the default error logging less verbose
Add one more type of error to the except

@peendebak Haha I'm starting to regret my comment on catching a specific error, I didn't realize that there were several possible errors. Looks good though!


Gource?

When will support for Gource be added?

It will appear in the new application.

Когда выйдет новое приложение?


disk IO operation

GreenFunc should provide API to save/load Green's function to/from disk.
The API should provide two options:

For small Green's function, simply save/load as plain text
For large Green's function, use hdf5 format to save/load with data compression. The following link could be useful: https://juliaio.github.io/HDF5.jl/stable/

We decide to use jld2 format, which works automatially.


Simple display is slightly inconsistent

When displaying a Simple error, the found token and expected tokens are displayed slightly inconsistently. The found token is wrapped in apostrophes while the expected ones aren't, EG found 'x' but one of a, y, z was expected.

This should be fixed in 234f024 now.


could not find addr_of in ptr

anyhow used as dependency of some of other dependency in project.
And I have go error:
error[E0433]: failed to resolve: could not find `addr_of` in `ptr`
   --> /home/evgeny/.cargo/registry/src/github.com-1ecc6299db9ec823/anyhow-1.0.40/src/error.rs:606:14
    |
606 |         ptr::addr_of!((*unerased.as_ptr())._object) as *mut E,
    |              ^^^^^^^ could not find `addr_of` in `ptr`

error[E0433]: failed to resolve: could not find `addr_of` in `ptr`
   --> /home/evgeny/.cargo/registry/src/github.com-1ecc6299db9ec823/anyhow-1.0.40/src/error.rs:647:22
    |
647 |                 ptr::addr_of!((*unerased.as_ptr())._object) as *mut E,
    |                      ^^^^^^^ could not find `addr_of` in `ptr`

error: aborting due to 2 previous errors

rustc: nightly-2021-01-30
target: wasm32-unknown-unknown
OS: ubuntu-16.04

I had the same issue, but just with nightly, stable still worked fine and after an update it works with 1.53.0-nightly (07e0e2ec2 2021-03-24) as well.

I just fixed it with anyhow = "=1.0.39"
But for my current library, I can't upgrade it from my toolchain nightly-2021-01-31

Closing as a duplicate of #146. This crate does not support old nightlies; the expectation is that if you are using a nightly toolchain, then you are remaining on top of recent nightly changes.

@dtolnay thanks for your response.
And another solution - just use a fixed version of anyhow if it's impossible to upgrade.


Support editing and deleting messages

Will require sdk support

Server side changes here https://github.com/datastax/astra-assistants-api/pull/17

Example: https://github.com/phact/streaming-assistants/blob/main/tests/test_modify_and_delete_messages.py


Azure Digital Twins Definition Language Parser Readme Issue

Section link

Reason:
1.Await need to be used in async function.
2.[value] is not be parsed, the value is shown as following:

Suggestion:
1.add async before function.
2.[value] should be json string.
@xirzec, @johngallardo, @dw511214992 for notification.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Digital Twins:0.31620476,Azure.Core:0.12696677,EngSys:0.05034725'

@YoDaMa if you agree with these recommendations, @zzhxiaofeng can open a PR with the changes. Just reply here that you agree and add the "help-wanted" label.
Otherwise, please provide your recommendations here or in a PR. Thanks!

I agree with the changes. Feel free to open a PR @zzhxiaofeng

@YoDaMa We should be able to close this issue as per #21200


Renewal check causes excessive CPU usage

We're using lua-resty-auto-ssl to provide HTTPS for a sizable number of domains.
In production we're seeing patterns of elevated CPU utilization on our server instances, roughly every 24 hours, which correlates with a large number of accesses (GET and SET) into the Redis key-value store used as backend for storing the certificates.
Any idea what is causing this? Is nginx perhaps re-cycling workers after a while, at which point the shdict needs to be rebuilt? (But why the SETs?)

By default the renewal check is happening every 24 hours, which probably causes your high load. To make it less often adapt the renew_check_interval setting (default: 86400 seconds). Although I am not sure, if the load you are seeing is expected. How many different certificates/domains do you serve?

@luto Is see 19645 keys in my Redis, so 9822 domains? (assuming one key with :timestamp and one with :latest per domain)
I'll have a look at the renewal code tomorrow, thanks for pointing this out. The spike we're seeing is roughly 50 minutes long, where CPU utilization goes from ca. 2 to 45 percent. Redis accesses go up from ca. 50 to 1500 per minute. (We have two instances connected to the same Redis store.) Do those numbers sound reasonable?

Those numbers look a little high to me. But the instances we're currently running are in the 50-300 domains range. @GUI (the maintainer of this project) probably has a larger setup to compare some numbers.
Note that I have never worked with the renewal code myself. But since this is afaik the only thing running daily, it may very well be the culprit. Doing some back on the envelope calculations with your numbers, I suspect that there is still potential for optimization.

If you want to confirm it's the renewals, it's pretty easy to just set it to every 2nd day or so and see if the spikes move with it. I ran some tests with 30.000 domains (we're planning to eventually run it with all of our +100.000 domains) and 50 minutes sounds reasonable for 10.000 domains. We ended up with some forked code (https://github.com/simplesite/lua-resty-auto-ssl thanks to https://github.com/ryokdy) that limits the renewal checks as we saw it would take well over 10 hours in our case.

It is indeed the renewal check, thank you @luto @brianlund. I will have a look at the fork mentioned.

Extracting the code out of that fork into a PR would be great 😅 seems quite useful.
Cookies for anyone who does it ;)

Can't say no to cookies :) Picked out the code to store certificates and paired it with my own changes to the renewal job.
There is also code for deleting expired certificates, if this gets included I'll create a PR for that.

@brianlund Thank you for the PR! I'll see to test this in the new year.

Should be fixed in the v0.12.0 release thanks to @brianlund's #111 PR. Thanks everyone!


Blog post: Equinix Provider

Description
Adds a blog post on the new(-ish) Equinix provider.
Checklist:

[x] I have reviewed the style guide.
[x] If blogging, I have reviewed the blogging guide.
[x] I have manually confirmed that all new links work.
[x] I added aliases (i.e., redirects) for all filename changes.
[x] If making css changes, I rebuilt the bundle.

@jkodroff Give us a ping when you're done with your edits and okay to publish. Thanks!