(Geo)Zarr sub-resources in /coverage, and CIS JSON

Zarr is typically organized in the cloud as a directory structure of resource files, as opposed to a single file.
What does this mean for implementing this as a representation of /coverage?
Would the content then be sub-directories of /coverage?
What does an application/x-zarr negotiated media type typically return?
Does this sub-resource pattern mean that we should resurrect sub-resources in the context of CIS JSON, to require in addition to /coverage:
/coverage/domainset, /coverage/rangetype, /coverage/rangeset, /coverage/metadata instead of the current profile= query parameter in that requirement class, but ONLY for when CIS-JSON support is declared in /conformance ?

How Zarr is internally organized should not be important. We should use the common parametres ot extract information from the coverage that are agnostic of the file format.
I'm not able to understand why the first sentence is related with the rest of the discussion and the resucitation of "/coverage/"
In case that we request a zarr as a format, can we only use a MIME multipart response or a zip files (with or without compression) to support the retrieval of several files in one? Both support files folders and byte arrays.

@joanma747 In general, I think multipart response are a pain to deal with. Zip could be one option (though we would need a Zarr+zip media type), but I also think the whole idea of Zarr being cloud-friendly is normally mapping it to separate files that allow to efficiently serve it and access individual parts of it (somewhat similar to COG range request).
Although the standard itself seems to say that the key/value store (file name / file content) can be implemented any which way, from https://wiki.earthdata.nasa.gov/display/ESO/Zarr+Format:

Zarr can be viewed as the cloud based version of HDF5/NetCDF files as it follows a similar data model.  Zarr does not come in a single file as NetCDF or HDF5 does but as a directory with chunks of data in compressed binary files and metadata describing the binary content in external JSON files.

So based on all of this, I am wondering if it should mean that our (Geo)Zarr conformance class in coverages is not a single file unlike other representations like GeoTIFF or netCDF, but individual files (as defined by Zarr) inside /coverage/ directory.
And I am making the parallel to CIS JSON if we already introduce encoding-specific sub-resources about the desire to access only the "domainset" or "rangetype" property that we had before that currently I changed to a /coverage?f=json&profile=domainset,rangetype request.

I see.
I still think that:

We should use common parametres to extract information from the coverage that are agnostic of the file format.
So, requesting "files in a folder" is not the way WCS worked and it should not be the way OGC API coverages works.

If you want to implement Zarr as "files in a folder" then you should define the OGC Zarr API (that is not the OGC API Coverages)
and do
/collections/{collectionId}/Zarr/file
I have the same opinion with COG. We should not serve a COG in a OGC API Coverages. COG uses HTTP range and the client is in control of the bytes traffic. It does not require an API and adding an API only messes with the original idea.

@joanma747 About COG see https://github.com/opengeospatial/ogcapi-coverages/issues/93#issuecomment-1111158575 :)
For COG, it's rather easy to add support for HTTP range to a /coverage GeoTIFF representation (and cannot potentially conflict with other encoding resources, since it does not define new resources).
For Zarr as files in a folder, I am not convinced one way or the other.
If this is what could allow pointing a Python XArray client to an OGC API - Coverages /coverage resource, I think it is a very big argument "for" (while avoiding having to define a competing separate API), but there are certainly valid arguments against in terms of "messing up" the clean resource representation defined by OGC API - Coverages.

it's rather easy to add support for HTTP range to a /coverage
That is true, but does this make sense?.

The whole purpose of the OGC API coverages is to forget about the internal structure of the coverage and request the data based on geospatial filters and other filters.
If I have to build a client that transforms all that in to byte positions before I do a HTTP -range request, where is the value of the OGC API Coverages. It is simply better to forget about it and consider that HTTP-range is your protocol. No API needed.

@joanma747 The value / sense in that is being able to support both typical OGC API - Coverages clients that implement parameters like subset and scale-factor (and/or coverage tiles) as well as those clients that just understand COG at the same OGC API end-point. In my opinion, that makes data easier to find by having everything in one place, one API that supports both use cases, and is somewhat of an answer to justify OGC APIs to those asking 'What's the point of OGC API? We don't need an API, we have COG'.
It would be the same idea for supporting a Zarr directory that you can point an XArray to /collections/{colletionId}/coverage.

@joanma747 I am not sure having separate APIs for specific encodings/formats is a good idea. For example, a CDB data store could use a UNIX file system (traditional CDB approach) in which content is structured in a hierarchy of file folders (based on tiling/LoD rules). A variety of formats/encodings are used and more will be added. As a developer defining clients that access a CDB data store, I want to simply have one API that accesses any vector data stype and one API that accesses any coverage type.
Actually, I would really like one API that rules them all :-) so that I can simple say "Give me content in this geography area with appropriate metadata so I can then process further". Unfortunately, the OGC API design/architecture does not support this!

Actually, I would really like one API that rules them all :-) so that I can simple say "Give me content in this geography area with appropriate metadata so I can then process further"
@cnreediii Doesn't API-EDR do that?

@jerstlouis, @joanma747 , @cnreediii The key issue should be no the detailed format and structure in the cloud or on disks, but the metadata that is exposed for processing - this should be the same and consistent across OGC APIs .
PS NetCDF3 and NetCDF4/HDF5 have completely different internal structures - the first is a multidimensional array, the latter a hierarchy of objects (which may be multidimensional arrays). The OGC APIs should hide this.

What is the mechanism to know the internal file structure in the first place? Without knowing it, you cannot request individual chunk files by name.
And then the client should be "aware" of this structure and what "chunk" corresponds to what spatial area. The Zarr structure should not change when you do a subsetting (a new Zarr should not be created) so in practice, when requesting individual subfiles you would not use subsetting or scaling parameters.

@joanma747 Indeed, that is the question! Back in the day when I helped design and implement a GIS API, we had a simple call to the server that asked what formats/encodings the server supported for output. We used a controlled vocabulary so that the server provided a list of one or more formats and then the client "knew" what could be returned. Sort of sounds like some of the W*S Standards :-) Whether the returned format/encoding conformed to the rules of a given format, such as IEGS or SIF, that was a different question :-)

SWG 2024-01-24: Since we do not have enough feedback / experience on how best to implement Zarr as a coverage representation in OGC API - Coverages, and uncertainty about the usefulness of either internal or (multi-web resources) space partitioning blocks potentially conflicting or being redundant with subsetting mechanisms, I propose to remove the Zarr requirement class from Part 1, with the option to add it later if we receive more feedback.
Alternatively, if anyone has input on Zarr and would like to propose how best to go about it, please discuss this in this issue or at the upcoming Code Sprint February 13-15, https://github.com/opengeospatial/ogcapi-coverages/wiki/February-2024-OGC-API-‐-Coverages-Virtual-Code-Sprint .

SWG 2024-08-21: We now have a draft Zarr requirement class using a Zip container files for the /coverage response, with the internal Zarr structure contained within it.
Please provide feedback on this approach if you plan on implementing Zarr in your Coverages implementation.


[WIP] Migrate to annotationProcessor feature in gradle 4.6

This PR fixes followings.

rewrite to annotationProcessor configuration.
See: https://docs.gradle.org/4.6/release-notes.html#convenient-declaration-of-annotation-processor-dependencies
remove unnecessary apt plugin

Note
This PR has dependency #793 .

@wreulicke I did merged #793 into master. As I notice that the title have [wip], do you still have remaining tasks on this PR?

@muga thank you for your review.
blocked by #793.
i just rebased.

Thank you, merged.


bug: no Pending State for transactions with nonce too high

Bug Report
Kakarot RPC version:
v0.6.2

TL;DR
It is "surprising" for SDKs and tools that they can't send many transactions in any order with various nonces, as Kakarot does not have a "pending mempool" where txs await their nonce "too high" to become valid. Instead, Kakarot refuses nonce too highs at the mempool level.
Probable fix: to be discussed within the team
Current behavior:
Transactions that reach the RPC with a Nonce too high will fail asynchronously due to Starknet feeder gateway refusing txs with nonce too high. Leaving user without feedback regarding their tx and why it failed.

Expected behavior:
On Ethereum, each transaction from an account is assigned a sequential nonce. The nonce is a counter that represents the number of transactions sent from the account. It's crucial for preventing transaction replay attacks and ensuring transactions are processed in order.
When a transaction with a too high nonce reaches an Ethereum node's mempool (the waiting area for transactions before they are included in a block), the transaction will not be immediately executed or included in a block. This is because transactions from the same account must be processed in strict nonce order. Here's what typically happens:
Pending State:
The transaction with the too high nonce is considered "pending" and remains in the mempool. It waits for all transactions with lower nonces from the same account to be executed first. This ensures the sequential order of transactions.
Queuing:
Nodes may keep the transaction in a queue (also part of the mempool) specifically for transactions that cannot yet be processed due to nonce gaps. This queue helps nodes manage transactions that are out of order but expected to be valid eventually.
Stuck Transactions:
If transactions with the necessary nonce values to fill the gap do not appear, the high nonce transaction remains stuck in the mempool. It can stay there until it's dropped due to a timeout (nodes do not keep transactions in their mempool indefinitely) or until the nonce gap is filled.
Manual Intervention:
Users may notice that their transaction is stuck due to a too high nonce. They can intervene by either sending transactions with the missing nonce values or, in some cases, sending a zero-value transaction with a higher gas price to themselves to fill the nonce gap.
Dropping:
Eventually, if a transaction remains in the mempool without being able to be processed (due to nonce issues or otherwise), nodes may drop it to free up space. This dropping threshold varies between different Ethereum clients and their configurations.
Rebroadcasting:
Users can rebroadcast a dropped transaction if they still want it to be executed, provided they ensure that the nonce sequence is correct this time.
In summary, a transaction with a too high nonce does not disrupt the network but requires either the missing transactions to be broadcast or some form of manual intervention to proceed.

Steps to reproduce:

git clone git@github.com:bajpai244/on-chain-whitelist.git
git checkout feat/add-scripts

Fill .env
PRIVATE_KEY=
SEPOLIA_RPC_URL=https://sepolia-rpc.kakarot.org

then, run:
make deploy
Then look into the RPC logs, you will see that one of the two transactions failed, and upon querying the SN feeder gateway, it was refused because of Nonce error

Retry service in RPC to be able to debounce pending txs in mongo and shoot them again
Prune pending txs when they are mined (update some "mined" field or delete it, tbd)
Decide on max retry
Limit invalid txs as early as possible in send_raw_transaction, e.g. check chain_id, only check synchronous attributes (not nonce, not balance, etc.)


Hovering a node does not produce very informative visuals.

Describe the bug

Currently when we hover on a node it highlights all the links which have atleast 1 node as

Hovered node
Nodes which can be reached with hovered node as source node
Nodes which can be reached from nodes in step 2 as source nodes and so on..

Hovering nodes highlight the specified links. But it is very difficult for a viewer to guess the logic happening behind the scene. Hence it is not soo informative for viewers. (Even I was not able to guess the logic without reading the code). See the screenshots section.
The more informative logic will be to highlight the source and destination nodes (that are the neighbouring nodes) of the hovered node with different styles for source and destination. For example with different colors.
To Reproduce

Go to website
Zoom the website to have a clear view of nodes and links.
Hover at different nodes (atleast 7-8 nodes)
Can you guess the logic behind the hover-node event ?

Expected behavior

We should show links where hovered link is source node and where hovered link is destination node with different styles, so that the user should not have to view the direction of link from the link text. Hence making it more informative.
Screenshots

Different hovered nodes
Almost all links are highlighted

Only 1 link is highlighted

Desktop (please complete the following information)

OS: Windows 10
Browser: Chrome
Version: 80

@kgodey can I work on this issue?

@kgodey  The label "not ready for work" is automatically added. How can I remove it?

One idea could be to offer different ways to highlight nodes and having something like a control panel, where the user is able to select suitable options for them to traverse the graph. Maybe you can add this to your proposal.

@BharatNischal All issues are automatically tagged "not ready for work" until a project maintainer reviews it and decides whether we'll accept a PR for the issue.

To answer your question, you cannot remove it, @soccerdroid will remove it once it's ready for you to work on.

Hi @BharatNischal . This feature was added in order to check how connected, central or relevant a node is in the graph. This is why the direct neighbors, as well as the neighbors of the neighbors, are highlighted. Though I also see your point that it might be confusing if a node has a lot of connections. I like the idea of having different colors. Do you think you can modify a bit your idea and leave this feature as it is?

@soccerdroid  How about giving different colors to the outlinks of different nodes. This will keep the feature of how connected a node is and at the same time makes it more informative
Eg
1 -> [2,3,4] all links in red color
2 -> [5,6] all links in blue color
4 -> [7] all links in yellow
and so on
The colors are just example I will choose relevant one.
what you say?

Hi @BharatNischal . This feature was added in order to check how connected, central or relevant a node is in the graph. This is why the direct neighbors, as well as the neighbors of the neighbors, are highlighted. Though I also see your point that it might be confusing if a node has a lot of connections. I like the idea of having different colors. Do you think you can modify a bit your idea and leave this feature as it is?

@soccerdroid How about giving different colors to the outlinks of different nodes. This will keep the feature of how connected a node is and at the same time makes it more informative
Eg
1 -> [2,3,4] all links in red color
2 -> [5,6] all links in blue color
4 -> [7] all links in yellow
and so on
The colors are just example I will choose relevant one.
what you say?

@soccerdroid  If you think its a good solution then please assign me to the issue

To me, a better way to do this would be to make the edges arrows (rather than plain lines).

To me, a better way to do this would be to make the edges arrows (rather than plain lines).

@soccerdroid  @mathemancer  I think its a good suggestion. Should I start working on this feature?
Also Do I include the below suggested feature or not?

Hi @BharatNischal . This feature was added in order to check how connected, central or relevant a node is in the graph. This is why the direct neighbors, as well as the neighbors of the neighbors, are highlighted. Though I also see your point that it might be confusing if a node has a lot of connections. I like the idea of having different colors. Do you think you can modify a bit your idea and leave this feature as it is?

@soccerdroid How about giving different colors to the outlinks of different nodes. This will keep the feature of how connected a node is and at the same time makes it more informative
Eg
1 -> [2,3,4] all links in red color
2 -> [5,6] all links in blue color
4 -> [7] all links in yellow
and so on
The colors are just example I will choose relevant one.
what you say?

@soccerdroid If you think its a good solution then please assign me to the issue

To me, a better way to do this would be to make the edges arrows (rather than plain lines).

@soccerdroid @mathemancer I think its a good suggestion. Should I start working on this feature?
Also Do I include the below suggested feature or not?

Hi @BharatNischal . This feature was added in order to check how connected, central or relevant a node is in the graph. This is why the direct neighbors, as well as the neighbors of the neighbors, are highlighted. Though I also see your point that it might be confusing if a node has a lot of connections. I like the idea of having different colors. Do you think you can modify a bit your idea and leave this feature as it is?

@soccerdroid How about giving different colors to the outlinks of different nodes. This will keep the feature of how connected a node is and at the same time makes it more informative
Eg
1 -> [2,3,4] all links in red color
2 -> [5,6] all links in blue color
4 -> [7] all links in yellow
and so on
The colors are just example I will choose relevant one.
what you say?

@soccerdroid If you think its a good solution then please assign me to the issue

I am not sure if a combination of colors is a solution to the issue. I think @mathemancer suggestion is a good starting point. I'll assign this to you.


python3Packages.oauth2client: fix license

Description of changes
This package was under Apache License 2.0 all that time: https://github.com/googleapis/oauth2client/blob/master/LICENSE.
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks for noticing!


Manually create metric descriptor at view registration time.

Some of our metrics are recorded rarely. Prior to this change, we rely
on OpenCensus stackdriver exporter to create the metric descriptor for
us, which means some metric descriptor will not be created long after
the code change, blocking the creation of monitoring and alerting.
Note the CreateMetricDescriptor call is idempotent and safe to call
repeatedly.

see static analysis error - check for correct version of the import

see static analysis error - check for correct version of the import

/lgtm

/lgtm


Dictionaries to use resources not hardcoded files

A quick hack to make dictionaries a resource rather than hardcoded links so that the commands will work independent of where you are on the file system. Long term we will have dictionaries neatly sorted in a separate repo to solve this elegantly.
This closes #32

Can one of the admins verify this patch?

Abandoning because we deprecate the 0.2.7 branch in favour of dev


Time Data Does Not Match Format

Until very recently, the integration was working perfectly. Yesterday, all my mail sensors suddenly became unavailable and I've tried several times to uninstall and reconfigure the integration to no avail. Here's the error I'm seeing in the logs:
This error originated from a custom integration.
Logger: custom_components.mail_and_packages
Source: helpers/update_coordinator.py:219
Integration: Mail and Packages (documentation, issues)
First occurred: 9:34:35 AM (1 occurrences)
Last logged: 9:34:35 AM
Error fetching Mail and Packages (outlook.com) data: time data '* Tuesday, Ju' does not match format '%A, %B %d'
I'm thinking Outlook updated something in their time formatting that caused the integration to stop working. Any thoughts?

Environment (please complete the following information):

OS: [e.g. HassOS/Raspbian/CentOS]
Type of device: [e.g. RPi3/NUC/Synology]
Home Assistant version: [e.g. 0.105.5]
Hassio/Docker/Core?
Component version: [e.g. 0.1.2]

OS: HassOS
Type of Device: HP Pavilion Laptop (host machine)
HA Version: 2021.6.6
Hassio on Virtualbox VM
Component version: 0.3.2

Give the latest beta a try it will likely clear that error up.

I don't see anything beyond 0.3.2 in HACS, even with "show beta" enabled (would it be master?):

would it be master?

No
I think HACS has throttled down it's refresh time, so you'll need to 'Update Info' for the repo after turning on the "show betas".

Tried it (version 0.3.3-b11), it worked for about a minute then my sensors were unavailable again. New errors are showing up now though, the most similar being this:
This error originated from a custom integration.

Logger: custom_components.mail_and_packages.helpers
Source: custom_components/mail_and_packages/helpers.py:1229
Integration: Mail and Packages (documentation, issues)
First occurred: 2:43:05 PM (18 occurrences)
Last logged: 2:53:26 PM

International dates not supported. (unconverted data remains: Track)
International dates not supported. (time data '* Tuesday, Jul' does not match format '%A, %B %d')

Before that there was an IMAP login error and a timeout but I think those resolved themselves.

Can you enable debugging and post the log so we can get a clearer picture?

Okay, I can still post the logs if you'd like me to, but after viewing them, I think I found the source of the problem.
Everything works normal until the integration checks for Amazon packages. The error is a result of the emails that are being forwarded from gmail where the date is shown as "Tue, Jul 20" (see image below).

I reconfigured the integration and did not include any Amazon sensors and that fixed the issue. However, I had to delete the day-old email that was causing this issue before reconfiguring again and including Amazon package sensors. I'll go ahead and change the email on my Amazon account as this issue seems to be caused by forwarding from Gmail.
What's odd is that when this sensor fails, all of them fail as well. Is there a way to get it so that it only makes the sensor that's throwing the error unavailable?

Everything works normal until the integration checks for Amazon packages. The error is a result of the emails that are being forwarded from gmail where the date is shown as "Tue, Jul 20" (see image below).

That's not the date that's used when parsing the Amazon emails it's the date after will arrive:,estimated delivery date is:,guaranteed delivery date is:,Arriving:,Arriverà: in the email body.

What's odd is that when this sensor fails, all of them fail as well. Is there a way to get it so that it only makes the sensor that's throwing the error unavailable?

No it's an all or nothing.


Allow ELBv2s to register independently of containers.

This PR makes some changes to the way ELBv2 registrations work:

New option EUREKA_ELBv2_ONLY_REGISTRATION - defaults to true.  This works in line with the other ELBv2 options, which also must be set appropriately.
If you override, and set it to false, then the previous behaviour of adding the ELB hostname and port to each individual container registration will happen.
Each ELBv2 endpoint (DNS name and port combination) will be registered in eureka, but containers will not.
Each time a new container associated with the ELB endpoint is started, registrator will send a Reregister to eureka, updating the metadata.
During deploys, the metadata will be updated once for each new container to start.  As such, the newest metadata always wins.
Heartbeats are still piggybacked onto container lifecycles.  As such, heartbeats will be sent to a given ELB endpoint as many times as there are associated containers running.  It would be possible to alter --ttl and -ttl-refresh registrator startup options to compensate and reduce the number of heartbeats if desired.
A ELBv2 will not be explicitly removed once registered with eureka.  Heartbeats will simply stop, as and when associated containers die.  Once all associated containers stop heartbeating (that would be all within an ECS target group), the record would expire based on the --ttl registrator startup value.

Hi @StevenACoffman you might want to take a look at this.  It'll give you a bit more context on what changes we have made to ELBv2 registrations.

looks good to me otherwise

@chrisgilbert Thanks! I'm going to need a bit of time to digest some of the implications. I will play with it in our environment a bit this weekend. Looks great!


fe and be changes. Change application.

Describe your changes
Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] If it is a core feature, I have added thorough tests.
[x] Do we need to implement analytics?
[ ] Will this be part of a product update? If yes, please write one phrase about this update.

Codecov Report
Attention: Patch coverage is 0% with 4 lines in your changes are missing coverage. Please review.

Project coverage is 5.88%. Comparing base (bc76528) to head (0ee6d28).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             main     #32      +/-   ##
=========================================
- Coverage   13.33%   5.88%   -7.46%
=========================================
  Files           5       6       +1
  Lines          15      17       +2
  Branches        1       1
=========================================
- Hits            2       1       -1
- Misses         13      16       +3

Flag
Coverage Δ

backend
0.00% <0.00%> (ø)

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

Files
Coverage Δ

frontend/src/app/app.component.ts
100.00% <ø> (ø)

...ul/backend/weather/controller/WeatherController.kt
0.00% <0.00%> (ø)

.../com/tul/backend/weather/service/WeatherService.kt
0.00% <0.00%> (ø)

...kotlin/com/tul/backend/auth/base/SecurityConfig.kt
0.00% <0.00%> (ø)


Error when compiling  idl/export.h

/opt/ros/iron/include/idl/string.h:19:10: fatal error: idl/export.h: No such file or directory
WHen compiling, it gives this error in Ubuntu 22, ROS2 Iron.
Any idea why? Tried installing rosidl but nothing changed, Is there something I'm missing?

@RDaneelOlivav for my iron installation at least there is no idl package listed at all in the /opt/ros/iron/include path. Where is this package from?
Seems like maybe your ROS 2 iron installation is corrupted.


Add Casdoor docs to Minio

Casdoor already supported Minio by: https://github.com/casdoor/casdoor/issues/409
Next we want to list the progress in Minio community, e.g., writing a docs page like: https://github.com/minio/minio/blob/master/docs/sts/keycloak.md

@Abingcbc

https://github.com/minio/minio/pull/14185

PR merged


JoErNanO: Watch topformtemplates.com

JoErNanO requests the watch of the watch_keyword topformtemplates\.com. See the MS search here and the Stack Exchange search here.

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

Approved by tripleee in Charcoal HQ


Remove version: Microsoft.Edge version 114.0.1823.82

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: Package still available;


Update pydro v0.1.2

Not sure why it fails, there's almost no difference with v0.1.1.

The error seems unrelated:
TypeError: importPackages is not a function\n    at make_pyjs

My guess is an incompatibility between pyjs and the pyjs test runner.

this is related to some empack change. I investigate now

https://github.com/emscripten-forge/pyjs-code-runner/pull/4 this should fix it!


[Backport]HBASE-25848: Add flexibility to backup replication in case replication filter throws an exception

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
24m 36s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ branch-1 Compile Tests _

-1 :x:
mvninstall
10m 25s
root in branch-1 failed.

+1 :green_heart:
compile
0m 48s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
compile
0m 54s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
checkstyle
2m 10s
branch-1 passed

+1 :green_heart:
shadedjars
3m 50s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 55s
branch-1 passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 49s
branch-1 passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+0 :ok:
spotbugs
3m 41s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 37s
branch-1 passed

_ Patch Compile Tests _

-1 :x:
mvninstall
2m 13s
root in the patch failed.

+1 :green_heart:
compile
0m 48s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javac
0m 48s
the patch passed

+1 :green_heart:
compile
0m 54s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
javac
0m 54s
the patch passed

+1 :green_heart:
checkstyle
1m 51s
the patch passed

+1 :green_heart:
whitespace
0m  1s
The patch has no whitespace issues.

+1 :green_heart:
shadedjars
3m 37s
patch has no errors when building our shaded downstream artifacts.

-1 :x:
hadoopcheck
2m  6s
The patch causes 10 errors with Hadoop v2.8.5.

-1 :x:
hadoopcheck
5m  1s
The patch causes 10 errors with Hadoop v2.9.2.

+1 :green_heart:
javadoc
0m 38s
the patch passed with JDK Azul Systems, Inc.-1.8.0_262-b19

+1 :green_heart:
javadoc
0m 50s
the patch passed with JDK Azul Systems, Inc.-1.7.0_272-b10

+1 :green_heart:
findbugs
3m 34s
the patch passed

_ Other Tests _

-1 :x:
unit
178m 32s
hbase-server in the patch failed.

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

248m 41s

Reason
Tests

Failed junit tests
hadoop.hbase.mapreduce.TestLoadIncrementalHFiles

hadoop.hbase.mapreduce.TestLoadIncrementalHFilesUseSecurityEndPoint

hadoop.hbase.mapreduce.TestSecureLoadIncrementalHFiles

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3291/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3291

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 02a5c510a9f5 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-home/workspace/Base-PreCommit-GitHub-PR_PR-3291/out/precommit/personality/provided.sh

git revision
branch-1 / ddac801

Default Java
Azul Systems, Inc.-1.7.0_272-b10

Multi-JDK versions
/usr/lib/jvm/zulu-8-amd64:Azul Systems, Inc.-1.8.0_262-b19 /usr/lib/jvm/zulu-7-amd64:Azul Systems, Inc.-1.7.0_272-b10

mvninstall
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3291/1/artifact/out/branch-mvninstall-root.txt

mvninstall
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3291/1/artifact/out/patch-mvninstall-root.txt

hadoopcheck
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3291/1/artifact/out/patch-javac-2.8.5.txt

hadoopcheck
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3291/1/artifact/out/patch-javac-2.9.2.txt

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3291/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3291/1/testReport/

Max. process+thread count
4298 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3291/1/console

versions
git=1.9.1 maven=3.0.5 findbugs=3.0.1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Add configuration to run MAVSDK on the autopilot side

This adds a way for MAVSDK to advertise itself as an autopilot.
@julianoes: I hardcoded the system id to 1. Is that a valid first implementation, or should I look into having it as a parameter somewhere?

I hardcoded the system id to 1. Is that a valid first implementation, or should I look into having it as a parameter somewhere?

FWIW IMO it is a valid first implementation.

Ok, merging for now, that can be improved later.
Regarding the QGC filters, that's documented here: https://github.com/mavlink/MAVSDK/issues/917. Makes sense to me to solve that in a different PR.


feat: Added sign-in/sign-out button

Is it supposed to be outside of the navbar?

Yep, according to figma:

Ohh okay my bad. I was looking at the register and profile pages so I did not notice it.


Set max_width on rustfmt to 90

Motivation
Right now our .editorconfig sets max_line_length to 90, but our formatter doesn't enforce that
Proposal
Make sure the formatter enforces it
Test Plan
Run formatter + CI

main

#1452   👈

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

Join @andresilva91 and the rest of your teammates on  Graphite


View Source button is 404ing

On this page if I click the "view source" I go to a page url that looks like:
https://github.com/Google/WebFundamentals/tree/material-branch/src/content/en/updates/posts/2015/11/devtools-digest-cds-edition.markdown
The problem is the branch material-branch which doesn't include the new post, but it does exist on master. This could be as simple as merging master into material-branch.
Related to #2304

Thanks for the bug report.
The config files need updating to master:
https://github.com/google/WebFundamentals/search?utf8=✓&q=WFGithubBranch

I have a bug fix coming in for this.

@gauntface shall we still keep staging at material-dot ?


AS-910: fetch TOS markdown from SAM

Take a look at some places where we use withBusyState and withErrorReporting, you should be able to simplify the useOnMount here.

Thanks! I took a stab at it.


Can you support cucumber.js is chioce for unit test and ui test

What problem does this feature solve?
I know mocha is good unit test tool.But I think cucumber.js is better then mocha.I don't have any idea how to integration cucumber with karma.Can you support this in the cli or teach me how to integration both.
What does the proposed API look like?
see above

I personally have never used cucumber and issues are not where questions like this should be posted to.
Please make sure to read the Issue Reporting Guidelines before opening new issues. The issue list only accepts bug reports and feature requests. Questions should be posted to the Discord chat,  forum or StackOverflow. Thanks.

I think this is new feature.I want cli have new choice for cucumber.js......

@kinanson, vue-cli initialises your project with whatever template you want, if you want to have cucumber option available you can just clone vuejs-templates/webpack and make your own template with cucumber support. Anyway, this is not a vue issue so locking this conversation.


enhance(examples): improve examples and getting started

Previously our examples needed to download 5GB Hyde dataset which is not a great way to get started. This is fixed and examples are cleaned up to be useful for newcomers.

I think this is a good start! I have a bunch of larger restructuring ideas for the docs but I would build these on top of what you did here. Good to merge from my side.


Update pre-commit workflow

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (483a1e1) to head (cda44e4).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #246   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           68        68
  Lines         1395      1395
  Branches       104       104
=========================================
  Hits          1395      1395

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Upgrade to CRA 5

Changelog: https://github.com/facebook/create-react-app/blob/main/CHANGELOG.md
Webpack 5 migration guide (hopefully mostly hidden from us by CRA): https://webpack.js.org/migrate/5/
Migration concerns:

We use craco for the theme package. They have a prerelease version available that supports CRA 5. If it doesn't just work then we can skip it initially (branch only), hacking deployment/index.ts as required to just import the default deployment package and reverting package.json to use react-scripts.
Web workers: the search web worker uses worker-loader. We should remove this in favour of webpack 5's built-in worker support: https://webpack.js.org/guides/web-workers/
Check browser compatibility of the emitted code. I think it'll be browserlist driven, this is a good opportunity to make sure that makes sense.
Compare output file structure, hashes, locations code splitting for language bundles, web worker, typeshed, etc.

Blocks #558

Closed by #568.


Connecting the client to the server

Is there any way to specify the hostname of the server for ollama run to connect to? I can use the HTTP API, but what about the cli client? I haven't been able to find this info in the docs.

Set the OLLAMA_HOST envvar to connect to a different host.


Concurrent download limit problems

If you try to set the download limit to something that is above FetchConst.MAX_DOWNLOADS_LIMIT, the method silently sets the value to the default value, which is 1.
This is problematic partially because MAX_DOWNLOADS_LIMIT isn't public, so you're relying on the developer to use second hand knowledge to not call that method with a value greater than 7.
Why is there a hard coded limit to how many files can be downloaded concurrently? Is the number 7 chosen arbitrarily or based on some data? I would much prefer if you laid out a recommendation for the maximum number of downloads in the documentation but allow the developer to use whatever number they choose.

@jacobtabak My initial thought was to limit concurrent downloads to provided the best user experience possible(Thinking overall system performance). I set max to 7 because I felt it was a good fit. I do not have data to back the max limit, and was waiting on the community to provide feedback.

I feel like a "one size fits all" limit of 7 may be lacking since Androids come in all shapes and sizes. I also feel strongly that the silent failure when attempting to set a limit over the hard-coded cap should be addressed.

@jacobtabak Agreed. Ill remove the limit and update the documentation. Thanks!

done. This will be in version 1.1.2 in fetch which will be released in the next few days.


chore(clang-format): restore cmake for clang-format

Purpose

For reviewers

Related Github issues

Changes

Tests

Deployment Considerations
Version Notes
{ major | minor | patch }

NOTE: Include the version notes in the squash commit message

I've add if(APPLE) to keep find_program behavior the same on non apple systems. Also committed changes that clang-format made

Tests are also failing on the develop branch, so not that changes caused it for sure(it would be very strange if so). Can't understand how that could happen after successful pipe in the last merged to develop request


Bug/Correcting comments and documentation

This only corrects some errors in the comments (also present at https://sdk.sphero.com/community-apis/javascript-sdk/).
Also, there the example Stream Odometer should point to this code.

All tests running smoothly here. Think the shasum check fail is not my responsiblity. Despite, I haven't changed code.


tsne_utils import error: try recompiling

How can I solve the below error? I already had compass running in the same env, but now it stopped - no idea why.
Traceback (most recent call last):
  File "/home/icb/karin.hrovatin/miniconda3/envs/compass/bin/compass", line 33, in <module>
    sys.exit(load_entry_point('compass-sc', 'console_scripts', 'compass')())
  File "/home/icb/karin.hrovatin/miniconda3/envs/compass/bin/compass", line 25, in importlib_load_entry_point
    return next(matches).load()
  File "/home/icb/karin.hrovatin/miniconda3/envs/compass/lib/python3.8/importlib/metadata.py", line 75, in load
    module = import_module(match.group('module'))
  File "/home/icb/karin.hrovatin/miniconda3/envs/compass/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/icb/karin.hrovatin/miniconda3/envs/compass/lib/python3.8/site-packages/Compass/compass/main.py", line 28, in <module>
    from .compass.penalties import eval_reaction_penalties, compute_knn
  File "/home/icb/karin.hrovatin/miniconda3/envs/compass/lib/python3.8/site-packages/Compass/compass/compass/penalties.py", line 4, in <module>
    from .extensions import tsne_utils
  File "__init__.pxd", line 181, in init compass.compass.extensions.tsne_utils (compass/compass/extensions/tsne_utils.c:5875)
ValueError: numpy.ndarray has the wrong size, try recompiling. Expected 80, got 88

I have not run into this error and the tsne utils have been left as-is for a long time, so I would guess that this might be caused by changing/updating numpy as the tsne utils rely upon numpy's C code. I would suggest re-installing Compass to force the tsne utils to be linked correctly or installing Compass in the environment somewhere else and copying the tsne files over.
Does this happen when using the gaussian penalty diffusion or just when running Compass? I'm assuming it's the latter above.
If you are not using the gaussian penalty diffusion you could also remove the tsne extension and the calls that use it/import it in the penalties computation.

I have tried reinstalling both numpy, Compass, and cython, but did not work. I solved it by making a new env and reinstalling all from scratch.


difficulty to understan sbatch_cluster.py

Dear Tommy,
thank you very much for your contribution in the field of the scRNA, I find really great that you freely share all these stuff to the community.
I am a newbie on both scRNA and snakemake but I have a hunger for learning, I realized a pipeline similar to yours to run snakemake with cellranger count and I was about to submit it to the cluster when I encountered your sbatch_cluster.py. It is a couple of days that I am struggling in understanding the utility and difference of this wrapper compared to the --cluster "qsub " modality which in your script is in pyflow-cellranger.sh.
I would be very grateful if you could explain this to me because I am not finding a lot of documentation, neither in the manual of snakemake.
All the best,
Ale

Hi @raveancic,
the sbatch_cluster.py is used for more control of the jobs.
read this https://groups.google.com/forum/#!topic/snakemake/1QelazgzilY. I learned from there.
use
https://github.com/crazyhottommy/pyflow-cellranger/blob/master/pyflow-cellranger.sh  it is relatively easier.


Make install can't copy the manpages if there's a space in current folder name

...
created directory `copy/deps/openssl/ssl/man/man3'
created directory `copy/deps/openssl/ssl/man/man5'
created directory `copy/deps/openssl/ssl/man/man7'
/bin/sh: -c: line 0: syntax error near unexpected token `copy/deps/openssl/ssl/man/man$sec/$fn.${sec}'
/bin/sh: -c: line 0: `pod2man="`cd ./util; ./pod2mantest /usr/local/bin/perl`"; here="`pwd`"; filecase=; if [ "darwin64-x86_64-cc" = "DJGPP" -o "darwin64-x86_64-cc" = "Cygwin" -o "darwin64-x86_64-cc" = "mingw" ]; then 	filecase=-i; fi; set -e; for i in doc/apps/*.pod; do 	fn=`basename $i .pod`; 	sec=`/usr/local/bin/perl util/extract-section.pl 1 < $i`; 	echo "installing man$sec/$fn.${sec}"; 	(cd `/usr/local/bin/perl util/dirname.pl $i`; 	sh -c "$pod2man 	--section=$sec --center=OpenSSL 		--release=1.0.1q `basename $i`") 	>  /Users/mylittlemac/Downloads/mydir copy/deps/openssl/ssl/man/man$sec/$fn.${sec}; 	/usr/local/bin/perl util/extract-names.pl < $i | 	(grep -v $filecase "^$fn\$"; true) | 		(grep -v "[	]"; true) | 	(cd /Users/mylittlemac/Downloads/mydir copy/deps/openssl/ssl/man/man$sec/; 		 while read n; do 			PLATFORM=darwin64-x86_64-cc $here/util/point.sh $fn.${sec} "$n".${sec}; 		 done); done; set -e; for i in doc/crypto/*.pod doc/ssl/*.pod; do 	fn=`basename $i .pod`; 	sec=`/usr/local/bin/perl util/extract-section.pl 3 < $i`; 	echo "installing man$sec/$fn.${sec}"; 	(cd `/usr/local/bin/perl util/dirname.pl $i`; 	sh -c "$pod2man 		--section=$sec --center=OpenSSL 		--release=1.0.1q `basename $i`") 		>  /Users/mylittlemac/Downloads/mydir copy/deps/openssl/ssl/man/man$sec/$fn.${sec}; 	/usr/local/bin/perl util/extract-names.pl < $i | 		(grep -v $filecase "^$fn\$"; true) | 		(grep -v "[	]"; true) | 		(cd /Users/mylittlemac/Downloads/mydir copy/deps/openssl/ssl/man/man$sec/;  while read n; do 			PLATFORM=darwin64-x86_64-cc $here/util/point.sh $fn.${sec} "$n".${sec}; 		 done); done'
make: *** [install_docs] Error 2

Hello DesWurstes,
Issue was not seen, when tested the latest source on x86. Tried to reproduce issue by git cloning to DIR with space, "make install" also worked OK to DIR with name space.
Can you please close the issue?
Thanks,
Santhosh Kumar R,
Intel India


Issue viewing clips gallery and snapshot gallery

On the frigate hass card when I click on either the snapshots gallery or the clips gallery I get the following message
Could not find client for frigate instance id: frigate. Check troubleshooting
When I click on troubleshooting the webpage appears but with no help with this error.
I am running hass-card v2.2.2 Frigate 0.94 on a seperate docker server and run HA on a raspberrypi
Is this not possible when running frigate on a different device?
Many thanks

Do you have the Frigate integration installed and working: https://github.com/blakeblackshear/frigate-hass-integration
Is there anything interesting in the Home Assistant logs pertaining to Frigate?
Did you change the default MQTT client_id as configured here: https://docs.frigate.video/configuration/index/
If yes, you have to set the client_id configuration value in the card to match.

(HA and Frigate on different machines is absolutely supported, in fact the card exclusively talks to HA)

Thanks for your super quick response.
here was me struggling and I thought I had copied the default config, but I had changed the client_id when I setup frigate months ago. Now I have added the change all is working perfectly
Very much appreciate all you work


Adopt case-prov, adding first provenance renders to Oresteia

This patch series does what the title requests, and makes one adjustment to the Spear Phishing example.  New files following the name pattern $(example_name)-prov.ttl are generated using case_prov_rdf, and this caused the hand-written file spear_phishing-prov.ttl to be overwritten.  So, as a side effect of this patch series, that file has been translated to JSON-LD at spear_phishing-prov.json.

Absent other feedback, I'm fine with this merging once CI passes.


apply transforms to normalized attributes

currently it is not applying transforms to attributes this fixes it

I signed it!


Sorting on jQuery DataTables does not seem to work

Clicking the sort button does not seem to work when data is displayed in jQuery DataTables.

Not an issue, sorting has to be enabled for jQuery DataTables.  Updated documentation.


Change to new inference server

Change from admission-controller to smarter-inference
Signed-off-by: Alexandre Peixoto Ferreira<EMAIL_ADDRESS>

LGTM


Recommended configuration for mounting a project into a custom docker service

When creating a new custom docker-compose based service I want to mount the project as a volume. Does following the example here mount the project it in the same way the web container does? e.g. fixes uid/guids, mutagen functionality etc
Thank you!

I hope taht was a good-enough answer @justafish - closing this for now, happy to continue the conversation here or elsewhere.


Disable P2SH

~Right now v1 forces P2SH encoding, at least for MPMA transactions. Both v1 and v2 should default to multi-sig, since~ P2SH is unreliable (https://github.com/CounterpartyXCP/counterparty-core/issues/2270 https://github.com/CounterpartyXCP/counterparty-core/issues/1383). (It's also probably more expensive in many cases, because it uses multiple transactions.) All of this is a stopgap, given we're gonna be doing all encoding using witness data in the near future (cf. https://github.com/CounterpartyXCP/counterparty-core/issues/1375).

apparently this actually has something to do with memo, memo_in_hex, and the version of bitcoin core / python-bitcoinlib being used...

So it turns out that P2SH no longer works with recent versions of Bitcoin Core.
In addition, there were two other bugs in the P2SH implementation:

https://github.com/CounterpartyXCP/counterparty-core/issues/2287
https://github.com/CounterpartyXCP/counterparty-core/issues/2288

Plus we also ran into: https://github.com/CounterpartyXCP/counterparty-core/issues/2286

done here https://github.com/CounterpartyXCP/counterparty-core/pull/2290


Fixed old gallery application version properties

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[x] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@grizzlytheodore these are the changes that were request to make all API version of gallery application version in line with what is present on the PIR CAPS back end.
The feature related to gallery applications version (VMApps) was not released in the state that it was in the old API.

@grizzlytheodore these are the changes that were request to make all API version of gallery application version in line with what is present on the PIR CAPS back end.
The feature related to gallery applications version (VMApps) was not released in the state that it was in the old API.

/azp run

/azp run

does the failing check need to be resolved or is it expected?

does the failing check need to be resolved or is it expected?

@msyyc could you please take a look at the breaking changes on python SDK?

@msyyc could you please take a look at the breaking changes on python SDK?

The .Net SDK CI failure is not due to this PR, it is due to a PR that was merged: https://github.com/Azure/azure-rest-api-specs/pull/11202

The .Net SDK CI failure is not due to this PR, it is due to a PR that was merged: https://github.com/Azure/azure-rest-api-specs/pull/11202

@msyyc Yuchao Yan FTE could you please take a look at the breaking changes on python SDK?

The breaking change is ok

@msyyc Yuchao Yan FTE could you please take a look at the breaking changes on python SDK?

The breaking change is ok


Cursor jumps to the second to last position

Using v1.0.1  - When I type in one-way-input my cursor movies back a space... easier shown than described:
https://ember-twiddle.com/7f03b5ab62ff1a340debc81734b1264e
Type anything.

This should now be fixed in 1.0.2

Confirmed.  Thanks!


Move the current post to drafts

In order to create new post and use the old ones as example, move all the post to drafts.
By this way the post will be available in local while developing

http://jekyllrb.com/docs/drafts/


Question about Tokenizer

Why does the tokenizer tokenize Cl as just C?
sequence = "[O-][Cl+3]([O-])([O-])[O-]"
print(tokenizer.tokenize(sequence))
['[', 'O', '-', ']', '[', 'C', '+', '3', ']', '(', '[', 'O', '-', ']', ')', '(', '[', 'O', '-', ']', ')', '[', 'O', '-', ']']

However, the tokenizer performs differently for Li and Br.
sequence = "[Li+].[Br-]"
print(tokenizer.tokenize(sequence))
['[', 'Li', '+', ']', '.', '[', 'Br', '-', ']']

Thank you for your help!

Hi @himanshisyadav, I tried your example and Cl is tokenized to be Cl, not C.
sequence = "[O-][Cl+3]([O-])([O-])[O-]"
print(tokenizer.tokenize(sequence))
['[', 'O', '-', ']', '[', 'Cl', '+', '3', ']', '(', '[', 'O', '-', ']', ')', '(', '[', 'O', '-', ']', ')', '[', 'O', '-', ']']

Maybe you can check your code where the tokenizer is imported and defined. Let me know if you still have any question.

@ChangwenXu98 Thank you for your quick response! I'll check my code.


Update MSFT_xADDomainController.psm1

Parameter SiteName is being passed as empty string instead of $Null when not present thus passing the check on 135.  Modified line accounts for both states.

Hi @Merto410, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@Merto410 Can you please add a bullet point describing your change under the Unreleased section in the Readme as described here?

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

@kwirkykat As requested README.md updated

Reviewed 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


option to return only first result on ambiguous grammars

While I work to figure out why my grammar is ambiguous, it would be great to be able to tell nearley that I just want the first result that it gets (so far that's been working fine). Right now I'm getting ~25k results and while I'd like to fix my grammar, I'd also like to shortcut the process while I figure it out.
Somewhat related: is there an easy way to figure out where the ambiguities stem from?

Can't you just do the_result_i_care_about = results[0]? Unfortunately, I can't just "throw out" ambiguities because they might get resolved later on in the parse.
Figuring out where ambiguity stems from is HARD (as in "factoring big numbers is hard" kind of hard). My recommendation is that you use nearley-test to print out the ambiguous results on small inputs to see how they differ. Use this information to isolate which nonterminals can be parsed ambiguously. Often this is quite unobvious—I guess it's the price you have to pay for an expressive parser that parses whatever you throw at it!

So, I did manage to figure it out. I had two stray whitespace terminals that, depending on indentation, gave me between 48 and 25k results. Removing the bad entries solved a LOT of my problems. However, I still felt it was worth asking the question, just in case it was possible. I don't really understand the intricacies of parsers if you can't tell, haha.
Thanks for the update! I'll close this since my questions were answered.


architecture-updates

Fixes # .
Overview
Updated architecture with a section on isomorphic, and included slides and notes from presentations.
Data Model / DB Schema Changes
<N/A if no changes>
Environment / Configuration Changes
<N/A if no changes>
Notes

🎆


Add slovak language file

@pvagner
I added following string resources.

dialog_accept
dialog_close
send_title
send_close_button
send_exit_confirmation_message
send_overwrite_confirmation_message

Can you make these strings?

thanks @wa2c for accepting. I have now translated everything that's available in main as of now.


Support for streams in tiddlyclip

https://github.com/buggyj/tiddlyclip-plugin/issues/35
As mentioned in the issue linked above, is it possible to add support for streams in tiddlyclip. Is there anything you can do from your part regarding this. If not, feel free to close this issue.

I've had a quick look and it seems that the work involved is entirely tiddlyclip specific and not even in normal wiki syntax. As such unfortunately I don't think there is anything I can do to be of assistance.
Adding a node to an existing tiddler involves adding a parent field in the new node pointing to the parent tiddler, and adding the new node to the stream-list field of the parent. You may want to consider running actions in tiddlywiki after clipping to move the tiddler created by tiddlyclip into a Streams structure.

Can anything be done using text slicer plug in or the excise text editor toolbar button so that the excised text becomes the child nodes of parent tiddler

Refer to this discussion in case anyone is interested


Integrate the gopath hacks

Signed-off-by: Arghya Sadhu<EMAIL_ADDRESS>Fixes #453

Proposed Changes
Use GOPATH hacks to codegen anywhere.

Release Note

Docs

I think the reason why it doesn't work is that you should first update the deps and then update the script

@slinkydeveloper once #448 gets merged it will work after rebase


beforeLoad prop? for dispalying loading sign or image

there is a onLoad prop for completion of loading of the 3d object. Is possible to add a beforeLoad prop for displaying "Loading..." or similar indications?

 Problem solved.
Create state of loading and setLoading with initial state of ture.
Let onLoad = {function A}
funtion A  change loading state from true to false.


Sanitize generated comments

We're seeing issues with the codegen in which comments containing */ literals aren't being escaped. (An example can be found in google.longrunning.Operations.ListOperations).
KotlinPoet has an open issue for sanitizing comments, so for now, borrow their implementation in Wire.

(In draft because I'm still testing this 😅)

The fix looks sensible to me. I'm not sure why whitespace is modified, and I believe @ should be escaped as well, because it has meaning in KDoc, but not in proto.
A test for this would be very nice to have. What about a proto file with a service that has offending comments attached? I think it should be generated automatically when put into protoc-gen-connect-kotlin/proto/buf, and produce a Kotlin client in protoc-gen-connect-kotlin/src/test/java. Then we just need a test case in protoc-gen-connect-kotlin/src/test/kotlin/PluginGenerationTest.kt to make sure that the generated code compiles.

ok, added a simple test case in https://github.com/bufbuild/connect-kotlin/pull/39/commits/09cd4bbaee02cdf57a1acd6b7b82d1ad784b926e - happy to extend it in any way that would make us more confident.

generated code looks like this FWIW, which is valid kotlin:
// Code generated by connect-kotlin. DO NOT EDIT.
//
// Source: buf/evilcomments/v1/evilcomments.proto
//
package buf.evilcomments.v1

import build.buf.connect.Headers
import build.buf.connect.MethodSpec
import build.buf.connect.ProtocolClientInterface
import build.buf.connect.ResponseMessage
import build.buf.connect.http.Cancelable
import kotlin.Unit

public class EvilCommentsServiceClient(
  private val client: ProtocolClientInterface,
) : EvilCommentsServiceClientInterface {
  /**
   *  This comment contains characters that should be escaped.
   *  @ is valid in KDoc, but not in proto.
   *  Comments in KDoc use C-style block comments, so */ and /* should be escaped.
   *  \[ and \] characters should also be escaped.
   */
  public override suspend fun evilComments(request: Evilcomments.EvilCommentsRequest,
      headers: Headers): ResponseMessage<Evilcomments.EvilCommentsResponse> = client.unary(
    request,
    headers,
    MethodSpec(
    "buf.evilcomments.v1.EvilCommentsService/EvilComments",
      buf.evilcomments.v1.Evilcomments.EvilCommentsRequest::class,
      buf.evilcomments.v1.Evilcomments.EvilCommentsResponse::class
    ),
  )

  /**
   *  This comment contains characters that should be escaped.
   *  @ is valid in KDoc, but not in proto.
   *  Comments in KDoc use C-style block comments, so */ and /* should be escaped.
   *  \[ and \] characters should also be escaped.
   */
  public override fun evilComments(
    request: Evilcomments.EvilCommentsRequest,
    headers: Headers,
    onResult: (ResponseMessage<Evilcomments.EvilCommentsResponse>) -> Unit,
  ): Cancelable = client.unary(
    request,
    headers,
    MethodSpec(
    "buf.evilcomments.v1.EvilCommentsService/EvilComments",
      buf.evilcomments.v1.Evilcomments.EvilCommentsRequest::class,
      buf.evilcomments.v1.Evilcomments.EvilCommentsResponse::class
    ),
    onResult
  )

}


Reflect 4.0.9 release

Checklist

[x] User relevant changes are recorded in CHANGES.txt
[x] Touched code is covered by tests
[x] Documentation has been updated if necessary
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

retest this please
the error suggests a temporal glitch

retest this please

retest this please

retest this please


Update to only push to Docker on tagged releases

Closes #731
Code Changes

[x] Update when the docker workflow runs

Steps to Confirm

[ ] list any manual steps taken to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[ ] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)
[ ] documentation issue created (tag docs-team to complete issue separately)

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
Docker will now only build and push on tagged release versions. Additionally I'll go through and clean up our dockerhub to get rid of the dirty versions

Looks good to me @ThomasLaPiana ! Thanks for tackling this 🙌🏽 (and mentioning the dirty version cleanup as well)
Is there any benefit to combining this with the pypi publish workflow at all? I can' think of one off hand but figured I would throw it out there

I briefly thought of that as well, but couldn't think of a particularly good reason to do it so decided not to even though part of me feels like it intuitively makes sense. Will leave as-is for now though, thanks for the quick review!


Fix: ktlint java

:loudspeaker: Type of change

[ ] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring

:scroll: Description

:bulb: Motivation and Context

:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #230 (c2a2290) into main (0b781b1) will increase coverage by 2.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #230      +/-   ##
==========================================
+ Coverage   80.53%   82.62%   +2.09%
==========================================
  Files          51        5      -46
  Lines        1582      213    -1369
==========================================
- Hits         1274      176    -1098
+ Misses        308       37     -271

Impacted Files
Coverage Δ

dart/lib/src/noop_hub.dart

dart/lib/src/protocol/message.dart

dart/lib/src/scope.dart

dart/lib/src/protocol/sdk_version.dart

dart/lib/src/protocol/sentry_runtime.dart

dart/lib/src/protocol/request.dart

dart/lib/src/throwable_mechanism.dart

dart/lib/src/protocol/sentry_exception.dart

dart/lib/src/protocol/sdk_info.dart

dart/lib/src/protocol/device.dart

... and 36 more

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 0b781b1...c2a2290. Read the comment docs.


Tumblr Search does not seem to be working

The following has been submitted:
Tumblr Search does not seem to be working.
Seeing this error in the logs:
20211123 06:55:24.081 Err TumblrSearchCrawler.CrawlPageAsync: Microsoft.CSharp.RuntimeBinder.RuntimeBinderException: 'System.Dynamic.ExpandoObject' does not contain a definition for 'timelines'
at CallSite.Target(Closure , CallSite , Object )
at System.Dynamic.UpdateDelegates.UpdateAndExecute1[T0,TRet](CallSite site, T0 arg0)
at CallSite.Target(Closure , CallSite , Object )
at TumblThree.Applications.Crawler.TumblrSearchCrawler.d__10.MoveNext() in C:\projects\Tumblthree\src\TumblThree\TumblThree.Applications\Crawler\TumblrSearchCrawler.cs:line 119

It doesn't affect safe keywords.
For the other keywords you need to be signed in to be able to search them and download the posts.


How can we do paginations with redux in this boilerplate?

I tried to implement paginations for user's repositories. Did anyone here try this or have any idea how to do it ?

+1 this redux thing is hell new for me so any good example would help thinking in redux. old jquery routines X_X

@kidjp85 @sascha1337 https://github.com/reactjs/redux/tree/master/examples/real-world
the real-world example contains pagination

I've never needed to implement pagination, but following the example @tsm91 posted is a good start!
I'll close this issue since it isn't directly related to the boilerplate, but feel free to continue commenting.


Pull up to speed (Pt. 2)

Closes https://github.com/conda-forge/artview-feedstock/pull/4
This should hopefully not suffer from Python 3.x failures. Absorbed your changes @pelson.

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.

Passing. 😄

In the name of the ARTView developers thank you very much for this repository. @nguy Can you add some kind of Acknowledgement in README or in the docs?

Will do and also wanted to add a big thank you!


[Feature Request]: Want to add a feed back form.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
For this web platform there should be a Feedback form where user can submit there feedback about this platform, like experience, information, accability etc.
Use Case
I add a 5 star along with name and comment based feedback  form, Which I attached in footer. After clicking on that button a pop up opened at the middle of the screen. visitor can able to left there opinion. After that It display a greeting. Just like attached video.
It helps to improve the platform more better.
Benefits
No response
Add ScreenShots
https://github.com/user-attachments/assets/fea23dac-db0a-45ad-b382-f24e696eadd3
https://github.com/user-attachments/assets/6245c3fa-1b89-4889-9b4f-b095507e3330
@Soujanya2004  Please assign me this under label-3 . It's supposed to be an mandatory feature of that type of platform.
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@Suvadip-sana and where will all the feedback be displayed

You tell me that.
I thought for now it's only display in alert format. After that I add it's full backend functionality. Where I built a new mongo db model / route, then in a new page called 'feedback' where all the feedback display in grid card format..
@Soujanya2004 let me know what you think about. For now I built it's front end along with functionality of vanilla JS.

@Suvadip-sana i thought of adding a mail id and send the response to Gmail
yeah u can work on frontend for now
add this in homepage after faq section for now

@Suvadip-sana yeh that's nice actually, you can raise issue and work on it

@Soujanya2004   yaa. thanks.  Currently I complete rest of the other ISSUE that I raised here. Hope this related all ISSUE will be assigned to me.

@Suvadip-sana sure


Delays downloading FakeNetMini.exe on Windows

Downloading the FakeNetMini.exe file by browsing to a URL such as http://blah.com/hahahax.exe sporadically hits some sort of delay, and sometimes can even time out.
@iphelix indicates this has been observed prior to now in other browsers and it was dismissed as a transient latency issue perhaps due to the tendencies of browsers to download lots of extraneous content upon startup.
I am tracking this issue in case it is a problem with FakeNet-NG itself, and if it becomes a recurring theme, we can prioritize and address it.
Details:

Windows 7 Professional SP1 amd64
IE8 (version 8.0.7601.17514).
Installed:

python-2.7.12.amd64
Git-2.14.1-32-bit
gvim80-586
VCForPython27

See image below (the "Navigation Canceled" message applies to the page prior to this browse; no content is shown because this browse went directly to the download sequence in IE).

I think it could be a browser issue. Firefox download instantaneously while IE is lagging behind. Do they approach binary downloads in a different way? File parts?


[Fix] selected note bg color

The line in Standard Notes source code setting this color is here. Official themes use this variable to control the bg color when user selects a note. For example, look here to check how Midnight does the same. So I believe this PR fixes #2
I tested the fix partially by changing the css directly in developer tools in the Standard Notes app and it works fine for single and multi note selection. This change matches the background color to the one before (in the project's README). Refer to the image below for more details:

@luisstd I cannot test this fully without hosting the dist somewhere. If you have an option to test it, please let me know if there are any issues with the fix. Thanks!

Thank you for the fix! I will update the hosted reference for downloading the theme. I'll close the issue if it works, otherwise I will revert to the 1.0 release. But I believe this should work, I double checked with the default themes as well and I think that the missing variable is what's causing the issue.


Questions / issues about cross-compiling

Hey,
I've been trying to do some cross-compiling and I have a few concerns about how the system works. Please excuse me if I got something wrong or the solution already exists.
Issue 1
I don't see a good way to conditionally do things only when the package is cross-compiled. I'm aware of crossAttrs but I don't think this is generally a solution. I think it should be possible to do this:
nativeBuildInputs = [foo bar] ++ optional crossCompiling [baz];
preConfigure = ''
  foo
'' + optionalString crossCompiling ''
  baz
'';

Issue 2
When I tried to use nativeBuildInputs=[autoconf automake] within crossAttrs, the build got broken because the configure script was no longer being passed the essential --build and --host options. I suppose the cross derivation wrapper doesn't consider this possibility.
Issue 3
I have seen stdenv ? cross used as a condition for cross compiling but this is not correct most of the time, because it is also true when the native package within a cross-stdenv is being compiled!
Issue 4
Often you need to do this:
configureFlags = "--with-foo=${foo} --with-bar=${bar}"

But you need to have the crossDrv passed when cross compiling. I'm pretty confident it will result in the native outputs being passed in case of a cross-compiled package. There should be a way to do this transparently so that crossDrv path is passed in case of cross-compiling. Actually I think this is the primary reason why cross-compiled NixOS doesn't work - it just picks up the native builds of all packages.

Note, I think the only serious issue is that it is (apparently) not possible to specify different build inputs (all 4 kinds) depending on whether we're cross-compiling. I say this after looking at makeStdenvCross. The modified mkDerivation will work just with the *buildInputs that were passed (which cannot possibly be dependent on cross-compiling) and also doesn't seem to account properly for any *buildInputs in crossAttrs.

An example would be packages that need to have their own native build in nativeBuildInputs when cross-compiling, because it uses some of its own binaries during the build, which it doesn't know how to build when cross-compiling.

For your last issue, just set selfNativeBuildInput = true;
On Sun, May 15, 2016, 3:34 AM Ambroz Bizjak<EMAIL_ADDRESS>wrote:

An example would be packages that need to have their own native build in
nativeBuildInputs when cross-compiling, because it uses some of its own
binaries during the build, which it doesn't know how to build when
cross-compiling.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/15461#issuecomment-219277655

I analyzed the worst of this on https://github.com/NixOS/nixpkgs/issues/14965.
I think that during build time at least you can reliably detect cross-compilation by if [ -z "$crossConfig" ]; then...

1: Do hostPlatform != buildPlatform or similar.
3: I absolutely agree and that attribute is on its way out.  hostPlatform != buildPlatform is more correct and more clear.
4: Cross derivations are now the default so this issue is fixed.
That just leaves 2. I didn't think we automatically passed --build and the moment, just --host. I'll hopefully be cleaning up that soon.

Number 2 was probably due to crossAttrs.configureFlags wiping out the hack, not the extra nativeBuildInputs. Those are now added anyways, controlled by configurePlatforms. See https://github.com/NixOS/nixpkgs/blob/master/pkgs/stdenv/generic/make-derivation.nix#L28-L30.
That means all 4 are solved and this can be closed!


Return Restarting kernel in Spyder IDE when run the code sim = habitat_sim.Simulator(cfg)

Habitat-Sim version
vx.x.x
Habitat is under active development, and we advise users to restrict themselves to stable releases. Are you using the latest release version of Habitat-Sim? Your question may already be addressed in the latest version. We may also not be able to help with problems in earlier versions because they sometimes lack the more verbose logging needed for debugging.
Main branch contains 'bleeding edge' code and should be used at your own risk.
Docs and Tutorials
Did you read the docs? https://aihabitat.org/docs/habitat-sim/
Did you check out the tutorials? https://aihabitat.org/tutorial/2020/
Perhaps your question is answered there. If not, carry on!
❓ Questions and Help
when I run the code sim = habitat_sim.Simulator(cfg) in spyder IDE, it returns Restarting kernel... and stops running, how to cope with this problem?

Hey @man469,
We can't really support IDE specific issues, but it sounds like the process is crashing casuing a kernel restart. Possibly it could be running out of process memory loading the scenes or there may be mis-configured system dependencies. Hard to know without more details.

Hello, I have the same error: my kernel crashes when I call habitat_sim.Simulator(cfg).
I use jupyter notebook on a Ubuntu latest with 60GB of memory and 8GB of RAM

Hello, same problem when running habitat-sim 0.3.1 nightly via Jupyter notebook on Ubuntu 22.04 with 64G RAM. Here is the error from terminal:
[W 2024-10-22 16:14:07.246 ServerApp] 404 GET /api/kernels/4dc214dd-8f29-4f94-a3e6-43241d684b99/channels?session_id=3b52f31c-94e3-484a-9a5f-22f6bbab0747 (<IP_ADDRESS>): Kernel does not exist: 4dc214dd-8f29-4f94-a3e6-43241d684b99
[W 2024-10-22 16:14:07.249 ServerApp] 404 GET /api/kernels/4dc214dd-8f29-4f94-a3e6-43241d684b99/channels?session_id=3b52f31c-94e3-484a-9a5f-22f6bbab0747<EMAIL_ADDRESS>4.25ms referer=None
[W 2024-10-22 16:14:07.251 ServerApp] 404 GET /api/kernels/4dc214dd-8f29-4f94-a3e6-43241d684b99?1729628047249 (<IP_ADDRESS>): Kernel does not exist: 4dc214dd-8f29-4f94-a3e6-43241d684b99
[W 2024-10-22 16:14:07.251 ServerApp] wrote error: 'Kernel does not exist: 4dc214dd-8f29-4f94-a3e6-43241d684b99'
Traceback (most recent call last):
File "/home/ml/anaconda3/envs/habitat/lib/python3.9/site-packages/tornado/web.py", line 1790, in _execute
result = await result
File "/home/ml/anaconda3/envs/habitat/lib/python3.9/site-packages/jupyter_server/auth/decorator.py", line 73, in inner
return await out
File "/home/ml/anaconda3/envs/habitat/lib/python3.9/site-packages/jupyter_server/services/kernels/handlers.py", line 75, in get
model = await ensure_async(km.kernel_model(kernel_id))
File "/home/ml/anaconda3/envs/habitat/lib/python3.9/site-packages/jupyter_server/services/kernels/kernelmanager.py", line 501, in kernel_model
self._check_kernel_id(kernel_id)
File "/home/ml/anaconda3/envs/habitat/lib/python3.9/site-packages/jupyter_server/services/kernels/kernelmanager.py", line 532, in _check_kernel_id
raise web.HTTPError(404, "Kernel does not exist: %s" % kernel_id)
tornado.web.HTTPError: HTTP 404: Not Found (Kernel does not exist: 4dc214dd-8f29-4f94-a3e6-43241d684b99)
[W 2024-10-22 16:14:07.252 ServerApp] 404 GET /api/kernels/4dc214dd-8f29-4f94-a3e6-43241d684b99?1729628047249<EMAIL_ADDRESS>0.93ms referer=http://localhost:8888/notebooks/Habitat/content/habitat-sim/Navigation.ipynb

In my case, the problem was solved as shown here: https://github.com/facebookresearch/habitat-sim/issues/1941


Log Analytics resource group

Does the Log Analytics Workspace have to be in the same Resource Group as the resource being monitored? (i.e. can I have a Batch Service in one RG that sends logs to Log Analytics in another RG in the same subscription?)
If so, how do I reference this in the ARM template?

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

ID: b8f43189-3418-fd30-0528-550157d8366d
Version Independent ID: 049e3b0e-6f10-6985-c719-05fe43fc00b8
Content: Create diagnostic setting in Azure using Resource Manager template
Content Source: articles/azure-monitor/platform/diagnostic-settings-template.md
Service: azure-monitor
Sub-service: ****
GitHub Login: @bwren
Microsoft Alias: bwren

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

Thanks for reaching out. Here is an example for diagnostic setting that might help you.
{     "$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#",     "contentVersion": "<IP_ADDRESS>",     "parameters": {         "batchaccountName": {             "type": "String",             "metadata": {                 "description": "Name of the Batch Account that will be created."             }         },         "settingName": {             "type": "String",             "metadata": {                 "description": "Name of the setting. Name for the diagnostic setting resource. '."             }         },         "storageAccountName": {             "type": "String",             "metadata": {                 "description": "Name of the Storage Account"             }         },         "workspacerg": {             "type": "String",             "metadata": {                 "description": "Log Analytics workspace RG for the Log Analytics workspace to which logs will be sent."             }         },         "workspacename": {             "type": "String",             "metadata": {                 "description": "Log Analytics workspace name for the Log Analytics workspace to which logs will be sent."             }         }     },     "variables": {},     "resources": [         {             "type": "Microsoft.Batch/batchAccounts",             "apiVersion": "2019-08-01",             "name": "[parameters('batchaccountName')]",             "location": "[resourceGroup().location]",             "dependsOn": [],             "properties": {                 "autoStorage": {                     "storageAccountId": "[resourceId('Microsoft.Storage/storageAccounts', parameters('storageAccountName'))]"                 },                 "poolAllocationMode": "BatchService"             },             "resources": [                 {                     "type": "providers/diagnosticSettings",                     "apiVersion": "2017-05-01-preview",                     "name": "[concat('Microsoft.Insights/', parameters('settingName'))]",                     "dependsOn": [                         "[resourceId('Microsoft.Batch/batchAccounts', parameters('batchaccountName'))]"                     ],                     "properties": {                         "name": "[parameters('settingName')]",                         "workspaceId": "[resourceId(parameters('workspacerg'), 'Microsoft.OperationalInsights/workspaces', parameters('workspacename'))]",                         "logs": [                             {                                 "category": "ServiceLog",                                 "enabled": true,                                 "retentionPolicy": {                                     "days": 0,                                     "enabled": false                                 }                             }                         ],                         "metrics": [                             {                                 "timeGrain": "PT1M",                                 "enabled": true,                                 "retentionPolicy": {                                     "enabled": false,                                     "days": 0                                 }                             }                         ]                     }                 }             ]         }     ] }
We will close the issue for now. If there are further questions , please revert and we will be glad to assist you.


Setting stringValue or displayText in table cell resets horizontal alignment to left

I have .odt file with table called statsTable.
Cells at row 1 have horizontal alignment set to RIGHT.
Cells at row 2 have horizontal alignment set to CENTER.
When I set cell values programmatically by .setStringValue or .setDisplayText, horizontal alignment is reset to LEFT and need to set alignment again.
try (final InputStream input = Main.class.getResourceAsStream("stats.odt")) {
            final var doc = OdfDocument.loadDocument(input);
            final var odfTable = doc.getTableByName("statsTable");
            odfTable.getRowByIndex(1).getCellByIndex(1).setStringValue("test");
            odfTable.getRowByIndex(1).getCellByIndex(2).setDisplayText("hello world!");
            odfTable.getRowByIndex(2).getCellByIndex(1).setDisplayText("next test");
            odfTable.getRowByIndex(2).getCellByIndex(2).setDisplayText("very next test");
            // need to set alignment of each cell, because it is reset to LEFT
            odfTable.getRowByIndex(2).getCellByIndex(2).setHorizontalAlignment("center");
            ...
            doc.save(new File("manipulated.odt"));
        }

Am I missing something? Is this expected behavior? I would expect horizontal alignment to be unchanged while setting the cell value.
Using version 0.11.0 and .odt file with table was created with LibreOffice Writer 7.4.

It does not sound like any desired behavior.
Just tested a bit adding new content in LibreOffice Writer and Calc.

In Writer: all new cell content is added left - likely in writing direction (as European I write from left ro right, so it is left aligned).
In Calc: Numbers are aligned right (as the calculation direction is still from their arabian origin right to left) and Text is aligned left.

Nevertheless, if something is middle aligned it should stay this way.
Most of the doc API was a gift from IBM Beijing, which was later forked to Simple API, which already abondoned due to heavy copy/pasted duplicated code to ODFDOM.
I would be happy if you could provide a pull-request.
It usally helps to start with a test that fails and the patch makes the regression test run.
Thanks in advanced, Marek,
Greetings from Berlin,
Svante


PDFs are being scanned when they shouldn't be.

I am not setting the filetype
-i, --fileTypes
With
node --max-old-space-size=6000 --no-deprecation purple-a11y/cli.js -u https://www.whitehouse.gov -c 2 -s same-domain -p 50  -a none --blacklistedPatternsFilename ./pa-gTracker-exclude-medicare.csv -k "Random<EMAIL_ADDRESS>But I am still finding PDFs in the list of URLs crawled. This shouldn't be the case..  If the default is html only then I shouldn't see any PDFs (or other docs) in my results.

Hi @mgifford, can I check which version of Purple A11y are you using to run the scan? E.g. 0.9.46, or newer (i.e. directly from GitHub).
I have not been able to replicate the issue of pdfs scanned when default strategy is html-only


[forge] compatibility test on cloud backend(78)

Motivation
Added a compatibility test on cloud backend
Test Plan
cargo test --package compatibility-test --test forge-compatibility

/canary

:sunny: Canary successful

/land


Fix/useless file

Why are these changes needed?

Delete useless files
add .env and dotenv
update dependencies

Related issue number (if applicable)
#6 #27 #5

LGTM


[WIP] Lazyload sentry

This is breaking, as we're changing API. It also needs to be tested.

@tiberiuichim @avoinea do we need upgrade guide entry for this one?


chore: Add HashMap docs

Description
Problem*
Summary*
Adds docs for https://github.com/noir-lang/noir/pull/4242
I've also edited the interface for entries, keys, and values to return BoundedVecs instead of arrays with optional elements. This is much more natural I think. I originally intended this for a separate PR, but I didn't want to write documentation for the old version of these functions only to immediately have to edit them afterward.
Additional Context
Documentation*
Check one:

[ ] No documentation needed.
[x] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

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

Looks like using the snippets script messes up the formatting in the snippets: https://65e5eac2f2a1ab3dd25576d3--noir-docs.netlify.app/docs/dev/noir/standard_library/containers/hashmap

Generally looks good from a skim over the code. I'll do a proper read tomorrow morning and merge.

Looks like using the snippets script messes up the formatting in the snippets:

Let's open up another Issue to address that. (Also noticed the links to source code link to master instead of a specific commit, which in the get case the line numbers no longer highlight the right portion of code as the file was updated.)

@Savio-Sou

which in the get case the line numbers no longer highlight the right portion of code as the file was updated.)

The lines of code are correct in the PR, so they'll link to the correct place once merged at least. Going forward they can break when looking at old versions of the docs though if we don't update the line numbers on the snippets there.


iOS builds with cocoapods crash with space in app name

Reported on community.
Fuse Version 1.3 and older.
MacOS Sierra.
Building with -DCOCOAPODS
Fuse does not let me build with -DCOCOAPODS if the .unoproj file has spaces.
This is the error I get:
Nanaimo::Reader::ParseError - [!] Dictionary missing ';' after key-value pair for "name", found "t"
    #  -------------------------------------------
    #
    #          5A4940FB151D17AD000ACE3F /* data */ = {isa = PBXFileReference; lastKnownFileType = folder; path = data; sourceTree = "<group>"; };
1738>          FC61D3661F1CECFC00DC7109 /* test the app.entitlements */ = {isa = PBXFileReference; lastKnownFileType = text.plist.entitlements; name = test the app.entitlements; path = test the app/test the app.entitlements; sourceTree = "<group>"; };
                                                                                ^
    #          F00570010000000000000001 /* include/Uno.h */ = {isa = PBXFileReference; name = Uno.h; path = "include/Uno.h"; sourceTree = "<group>"; };
    #          F00570010000000000000003 /* include/Uno/_config.h */ = {isa = PBXFileReference; name = _config.h; path = "include/Uno/_config.h"; sourceTree = "<group>"; };
    #  -------------------------------------------

To reproduce this just make a new app, name it my app (must contain spaces) and build for iOS with -DCOCOAPODS. I had no trouble with spaces in 1.2.1.

@eksperts: This is a uno-problem, not a fuselibs one.

@kusma moved


Add issue and PR templates

Closing out my old issues I no longer have context on. If anybody else is still seeing this, I'd encourage you to file a new issue with more information. Cheers!


[Hyperparameter Inquiry] Can't Reproduce ACE05 Results

Hi Haoyu,
Nice work on this MRE one-pass! I noticed that the commands in README.md are for SemEval 2018.  (We are especially curious about the hyperparameters.)
Thank you in advance for your information!
Best,
Zhijing

The ACE dataset is using the same set of hyper parameter as SemEval except for the number of epochs is 15 instead. However, due to the nature of the randomness of BERT finetuning, we simply run all experiment for 5 times and pick up the best performed one.


Deeplabv3+voc does not train after loading the model

The deeplab v3 version does not train after loading the model. It's stuck at 0% forever as shown below:
(deeplab-yude) u6617221@anaconda:~/Models/semantic-segmentation-codebase/experiment/deeplabv3+voc$ python train.py
/students/u6617221/Models/semantic-segmentation-codebase/model/resnet101s-03a0f310.pth loaded.
Use 6 GPU
0%|                                                                                | 0/30000 [00:00<?, ?it/s]

Hello, has the problem been solved? I have a similar problem. By the way, I used 4 GPUs.


$(...).tapTarget is not a function

I get error when I click for open feature discovery
here is the reference link http://materializecss.com/feature-discovery.html

Please make sure you include jQuery before Materialize and not (again) after this and call the method on documentready when all scripts have loaded.
Can you provide a codepen?

Are you using version 0.98.2? I'm using version 0.98.0, which doesn't have this feature.

https://cdnjs.cloudflare.com/ajax/libs/materialize/0.97.8/js/materialize.min.js

Well, 0.97.8 is an old version and this was added / fixed in a newer version. The docs are always for the latest release.

Please try if it works with 0.98.2.

Okay Thank you so much for help

Okay Thank you so much for help

In the latest version it also gives problems

This error in console in the line:

Hi @briancaldass,
Which version do you use? Does an element with the class tap-target exist?

i have same problem, i fix this with include jquery on end of list import
import M from 'materialize-css/dist/js/materialize.min.js'
import * as $ from 'jquery';

import M from 'materialize-css/dist/js/materialize.min.js'
import * as $ from 'jquery';

Depending on your version you may need jQuery. Materialize 1.0.0 does not require jQuery anymore and uses cash internally.


Documentation is missing for most methods

See https://godoc.org/github.com/samuel/go-zookeeper/zk. It goes against best-practises, but more importantly I don't know

if I can send in a nil ACL list to Conn.Create.
what type of string Conn.Create returns.
what flags are valid for Conn.Create.
why there are different variable groups for errors and not just one variable group.

There are probably more examples, but that's a good start.

...can I somehow avoid setting version when deleting?


Feature/akot15

Added the an implementation of the fully SPS scheme FSP2 presented by Abe et al. [1]
[1] Abe et. al.: Fully Structure-Preserving Signatures and Shrinking Commitments
https://eprint.iacr.org/2015/076.pdf

I pause the review for now and wait for a response to the above comments.

I pause the review for now and wait for a response to the above comments.

I am doing this wrong, because I don't see any comments?

I pause the review for now and wait for a response to the above comments.

I am doing this wrong, because I don't see any comments?

Sorry it was still pending.

I think the above hints will fix the issues and make it cleaner. Maybe you have to apply the hints at other places too (didn't check all of it).

Also all tests are ✅ if I run them. Good work 🎉


What is the relationship between sandbox, executor and runtime?

I was recently reading the code for substrate wasmtime, but I ran into some problems. I don't really understand the difference between runtime and sandbox.
For example, in the file substrate/client/executor/wasmtime/src/runtime.rs, we can call the method call() of WasmtimeInstance to execute the functions in the wasm instance. Also, in the substrate/client/executor/wasmtime/src/host.rs file, is the invoke() function implemented for HostContext also executing functions in the wasm instance?
What is the difference between the runtime instance and the sandbox instance?
Thanks!

I think this is a great question for our Stackexchange :point_right: https://substrate.stackexchange.com/
Would you mind asking it over there again?

No problem!


add white, add opt for more TIM chans, add opt for gamma correction

Can you check if these changes are valid? Thanks

Hi Frank,
thanks for PR. I really like that you've added support for more timers. I'm quite busy now and also do not have hardware and set-up to test it now :( I hope I'll get to this PR later and merge it.
I also would like to merge all my WS2812 libraries for L0, F1, F3, F4 to a single one so it will be also better managable but not sure when I get to it.
Thanks

I missed a a few spot and fixed it in the commit just now. I got it working... if I keep my multimeter on the data pin... damn thing doesn't like 3.3V data signals. Need another week for a buffer chip to confirm that it's ok with all sorts of animations.
Take your time, if people see this pull request somewhere, that's enough of a hint to them.

I have completed my project that uses this code
https://github.com/frank26080115/my-laser-projects/tree/master/neopixel_word_clock

Thanks @frank26080115 for sharing your project.
I hope I find time to set-up hardware with L0 and merge it and test it. It may take some time.