Add parsing for Helmholtz EM examples from exchange with Michael Götte

https://github.com/nomad-hzb/nomad-baseclasses/tree/main/baseclasses/characterizations/electron_microscopy

Four examples with the following SHA256 were shared, file names were anonymized:

c32ae7943c345f43888572112c8500c0c453e678c46e3d1de26da7406e2a1f81  *.tif
f04b5828ad54d0c6ddae357ffdc853e871f520acffa603ce7eacb3ca0285e65c  *.tif
f341a0ef8d284458a9436550c4c33d2d63402827a87c73c58e4352a638f70a9e  *.dm3
5bbf51afa36b1d14ed43f3a0fdc090c639690a52408205b1a81f4cf2c944910d  *.nxs

The analysis brought about the following conclusions:

c32ae7, tif somehow exported from a Zeiss Merlin instrument, many typical Zeiss metadata tagv2 34118 (ascii) and 34119 (utf16), will be implemented via #31
f04b58, tif as it was discussed in https://groups.google.com/g/hyperspy-users/c/1h9CC0egolQ this is an instance of an essentially deprecated proprietary file format from (ThermoFisher), which offers limited contextualization, will not be implemented for now unless a substantiated feature request is made as an issue explaining why this should be supported in NOMAD Oasis, a ThermoFisher TIFF parser exists, newer instruments write data that is better matching this parser
f341a0, dm3, will be implemented via #12
5bbf51, an HDF5 file that uses NeXus via NXroot and base classes only, that file gives only an interval in time to narrow down which actual base class versions were used, an application definition was not used, it is recommended to present this use case in one of the area B task force meetings, contact @sanbrock if support for this particular format should be supported further


Write and test BlockNode, ContentsNode

From interface described in #49, depends on #72. Should be rebased/merged when #72 lands.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@27845ec). Click here to learn what that means.
The diff coverage is 31.45%.

@@            Coverage Diff            @@
##             master      #73   +/-   ##
=========================================
  Coverage          ?   81.66%
=========================================
  Files             ?        7
  Lines             ?      758
  Branches          ?        0
=========================================
  Hits              ?      619
  Misses            ?      139
  Partials          ?        0

Impacted Files
Coverage Δ

apacheconfig/__init__.py
100% <100%> (ø)

apacheconfig/wloader.py
29.75% <29.75%> (ø)

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 27845ec...db18f4a. Read the comment docs.

@sydneyli I assume this PR is not blocked on me, presently. ;-)

@sydneyli I assume this PR is not blocked on me, presently. ;-)

Yep, it's back to me! Hoping to get to this today.

Re: ContentsNode vs BlockNode:
It's a good point. I do see what you mean that ContentsNode vs BlockNode are really only unique in that contents has no tag, and effectively acts as a root node (or as a child of BlockNode), and BlockNode is intermediary.
Here's what the current (above) implementation uses the above for, in terms of logic separation:

BlockNode isolates the logic for manipulating & accessing the tag name & values.
BlockNode keeps track of the whitespace preceding the first tag. ContentsNode keeps track of the whitespace after the final scalar in the group (like, the whitespace before EOF, or before the closing tag of a block)

Having these (particularly the tag manipulation) separated by class type seems useful. What if we have BlockNode subclass a generic ContentsNode, and maybe rename them something like MidNode and RootNode, respectively? Something that'd come up is that the whitespace property of the classes would conflict, so ContentsNode would have to keep a unique trailing_whitespace property.

Having these (particularly the tag manipulation) separated by class type seems useful. What if we have BlockNode subclass a generic ContentsNode, and maybe rename them something like MidNode and RootNode, respectively? Something that'd come up is that the whitespace property of the classes would conflict, so ContentsNode would have to keep a unique trailing_whitespace property.

That sounds like a legit design to me. Alternatively to subclassing, may be we could have an abstract or a base class declaring the interfaces and/or common implementation, then RootNode and BlockNode would have their own unique stuff.
In other words:
AbstractNode -> CommonNodeMixIn + RootNode
AbstractNode -> CommonNodeMixIn + BlockNode
or
NodeBase -> RootNode
NodeBase -> BlockNode
The mix-in piece would be there to keep abstract class defining just the interfaces, not any real code. On the other hand, that's not strictly required with Python customs, AFAIK.
WDYT?

Alternatively to subclassing, may be we could have an abstract or a base class declaring the interfaces and/or common implementation, then RootNode and BlockNode would have their own unique stuff.

Subclassing might make things a bit simpler here, since BlockNode has a lot of unique functionality over RootNode, but not the other way around. That is, the set of things RootNode should be able to do is actually a subset of the things BlockNode can do.

Thanks for the review! I did a pass on the documentation. Currently working on error handling documentation & tests, as well testing for unicode support.

(Just a note in case you missed-- this and the unicode PR are good for another round when you get the chance!)

(Just a note in case you missed-- this and the unicode PR are good for another round when you get the chance!)

Hey, and thank you for the ping!
I am not always sure at which point it's back to me (I am checking out the PRs from occasionally), so explicit communication is good!

Let's merge this PR - I think it's good enough and I do not want to block you.
I've left a couple of nits for your consideration (I am never short of nits). If you consider them worth addressing, I guess you could just push a quick PR.
Also, perhaps at some point we should have a CHANGELOG entry explaining this undertaking. I am just not sure at which PR in the chain it's would be best to note.
Thank you!

Thanks @etingof -- appreciate it! I agree with your notes on consistency between the namings, so i'll push a PR for that soon.


Show web content checkbox does not apper

For Web pure html/css and AngularDart developers is important to have the web content option.
In previous Dartpad release there was this option.
See documentation: Dartpad
Please, return this option again!

You're correct that the checkbox for showing web content is not in the current design. You can still create HTML/CSS/Dart examples, though. Just click the "New Pad" link at the top of the page, and when you select "Dart," flip the switch for HTML:

You'll get the same three-editor experience you're used to.


Task Boundary and Grid re-implementation

Description
As of RapiD v2, we do not draw the task border supplied to us via .gpx file in the url params.  Traditionally, .gpx files have been used as a means for tasking manager to say 'edit within this area, but not outside of it'. These .gpx files could specify arbitrary geometries, not just bounding boxes, to limit edits.
In turn, RapiD would display these borders and limit edits to happen only inside of them.
In RapiD v1, we also had the ability to subdivide a square task boundary into a 2x2, 3x3, 4x4, 5x5, or 6x6 grid (a RapiD-only feature, not something in iD). These were purely cosmetic and just helped mappers 'see' how much they had left to map as they methodically proceeded to map a large boxed area.
Now, we still have the machinery around parsing the .gpx files in place, but we need to ensure that the task border is drawn correctly. Furthermore, we need to detect if the .gpx specifies a rectangular shape and allow the user to subdivide it with the 2x2 <--> 6x6 grid.
The Task boundary predates rapid (iD had it as of tag 2.15.3)
Here's the original pull requests from RapiD v1 to add this task grid, for some context.
https://github.com/facebook/RapiD/pull/26
https://github.com/facebook/RapiD/pull/135

Fixed this while i was taking care of #913 - the grid should work once again. Grid lines have a transparency and are thinner than the task boundary:
https://user-images.githubusercontent.com/1887955/231550519-85795967-9885-47c7-9e20-56cdd49e5721.mov


Changing an alias should invalidate the old alias

What is the current behavior?
When changing a service alias, queries to that alias keeps being routed to the same service.
When another service applies the alias, queries are still routed to the old service
If applicable, share the query, mutation or subscription for reproduction
api.graph.cool/simple/v1/some-old-alias

What is the expected behavior?
When an alias is removed, queries should stop working. When a new service applies the alias, it should start receiving queries.

Can I add something to this? It would be great if deploying with an alias would automatically remove the old alias. This would improve zero-downtime deployments.

Changing an alias will now remove the old one from the cache and queries should be routed correctly now.

@do4gr Could you elaborate a bit? Does that mean that deploying a service with an alias that was already used will silently remove the alias from the old service, and set it on the new service?

I'm still getting this locally:
ERROR: A project with the alias 'test1' already exists. Aliases are globally unique. Please try something else.

{
  "data": {
    "addProject": null
  },
  "errors": [
    {
      "locations": [
        {
          "line": 2,
          "column": 9
        }
      ],
      "path": [
        "addProject"
      ],
      "code": 2034,
      "message": "A project with the alias 'test1' already exists. Aliases are globally unique. Please try something else.",
      "requestId": "eu-west-1:system:cj9rmezxm006y0139gnvszwqc"
    }
  ],
  "status": 200
}


add tests for update token info

Signed-off-by: lukelee-sl<EMAIL_ADDRESS>Description:
Add tests for updateToken.
The tests cannot be executed as they depend on completion of the following pr in services: https://github.com/hashgraph/hedera-services/pull/3738
Updated HederaTokenService.sol and IHederaTokenService.sol to add call for performing the update.  The corresponding .json files have not yet been updated awaiting completion of the pr referenced above.
Related issue(s):
Fixes #410
Notes for reviewer:
Checklist

[ ] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

Codecov Report

Merging #462 (e000518) into main (f87403b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #462   +/-   ##
=======================================
  Coverage   76.46%   76.46%
=======================================
  Files          12       12
  Lines         922      922
  Branches      143      143
=======================================
  Hits          705      705
  Misses        165      165
  Partials       52       52

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

This issue can be closed as it is a duplicate of pr #527


[vinxi] set content type header on page response

PR Checklist
PR Type
What kind of change does this PR introduce?

[x] Bugfix

What is the current behavior?

On GitHub Codespaces page responses return plain text.
What is the new behavior?
Set Content-Type header in createPageEvent().
Other information

Did not add it to server/spa/handler.tsx because its never used
Apperently when returning a string the header is set automatically

Thanks... I have refactored stuff since so I'm not going to merge the PR directly, but I will apply your fix.


Specify the version of importlib-metadata

Currently, python lint fails on the main branch such as https://github.com/launchableinc/cli/actions/runs/5528890710/jobs/10086219577. This PR fixes it

@shibuiwilliam Could you double check, just in case?


Add a get_static_info rpc for TUI API

Currently the tuigen is failing randomly during CI run with the following error:
2022-06-08 11:16:24,640 root         ERROR    GRPC_ERROR: basic_string::erase: __pos (which is<PHONE_NUMBER>3709551615) > this->size() (which is 0)

One possible issue is the huge number of API calls to get child-menu names and help-strings at each level. We can replace those with a single get_static_info rpc call similar to the datamodel and settings API.

ADO 682015


Add admin interface

As an admin, I would like to be able to access and update database information easily.
To achieve this with a reasonable lift, we could look at using nestjs-admin - https://nestjs-admin.com/docs/install

https://github.com/marmelab/react-admin

We are using internal.io for now


Export to Json/Import from Json issue.

I created an event for when my shop is open and excluded holidays from it by removing them manually.   When I save it to json, then reload the json file, the removed dates show back up.  Below is the seriesIgnoreDates line from when I saved it, it appears to list the dates twice, then 3 times, then 4 times.
"seriesIgnoreDates":[04/06/2022,04/06/2022,01/10/2022,01/10/2022,01/10/2022,26/11/2022,26/11/2022,26/11/2022,26/11/2022,24/10/2022,24/10/2022,24/10/2022,24/10/2022,24/10/2022,05/08/2022],
I tried saving it again after removing the extra events and saving it and I got.
"seriesIgnoreDates":[0.00370919881305638,0.0003297065611605671,0.0003297065611605671,0.0003090999010880316,0.0003090999010880316,0.0003090999010880316,0.0005440158259149357,0.0005440158259149357,0.0005440158259149357,0.0005440158259149357,0.0011869436201780415,0.0011869436201780415,0.0011869436201780415,0.0011869436201780415,0.0011869436201780415,0.0011689596259329197,0.0011689596259329197,0.0011689596259329197,0.0011689596259329197,0.0011689596259329197,0.0011689596259329197,30/04/2022,04/06/2022,04/06/2022,01/10/2022,01/10/2022,01/10/2022,26/11/2022,26/11/2022,26/11/2022,26/11/2022,24/10/2022,24/10/2022,24/10/2022,24/10/2022,24/10/2022,05/08/2022],
I tried removing all the extra dates from the saved file and it still doesn't exclude the excluded dates.

After some investigation into the issue, I have found the reason that ignore dates don't get imported.  This will be fixed in v1.4.3.
Thanks again for your feedback!

Just released v1.4.3.
Enjoy!


chore: bdd-gh-1586631715 to 0.0.1

chore: Promote bdd-gh-1586631715 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


[Internal] Move letter grade math to calculateGrades func

Overview of Proposed Changes
Instead of recalculating the letter grade based on the GPA in the searchResultsTable component, store this data like the gpa is stored in calculateGrades.
Details
Changes are needed in and searchResultsTable the dashboard
Reference Skedge for an example:
https://github.com/UTDNebula/skedge/blob/bd999e5ea43893e600aef44f6110e62a0a79796c/src/pages/index.tsx#L86-L93

Is what I did in https://github.com/UTDNebula/utd-trends/pull/301/commits/bc04095ed7563f53bb437e0b066b065cf67bcd5d fine?

Yeah, that works!


Import a printed map

Select and image of printed map (jpg), then have the user select the location of two beacons/landmarks on the map. This will allow TS to determine the latitude and longitude of all points on the map and see their current location / nearby beacons.
ex. Take a picture of a trail sign with the you are here marker, and you would need to enter one more location (maybe determined by map scale)

Workflow:
User clicks the maps icon on the navigation page / tools. They can select a pre-existing map or add a new one.
If they select a preexisting map, it loads the image and plots the current location, any beacons on the map, and the destination if available. The user can long-press the map to share a location, create a beacon, or navigate to that point (temp beacon). While navigating, the same navigation panel will be present at the bottom. The user can also drag or zoom in/out of the map. The user can select a new beacon to navigate to by clicking on it on the page and selecting 'Navigate'. Finally, the user can choose a new map to show by either pressing back or by clicking a close icon in the top right.
If the user chooses to add a new map, they will be asked to give it a name, select an image (map needs to be flat with north facing up). Upon opening the map, they will need to enter the location of two beacons (on a diagonal to each other). They can either select an existing beacon or enter a location manually. Once they do that, they can start using the map.
On the map select screen, users can also delete a map or edit it (name, image, locations).

Such a good idea!
I just think that it may be hard to take a flat with north facing up photo of a trail sign or similar outdoor map :wink:.
So the best option of this feature would be to additionally transform a crooked image using 3 or 4 beacons - like QR scanner apps. But I guess it will require additional work and image processing library :/

@Leopardus4  Thanks! So far so good, I have a fully working prototype (releasing in experimental mode). I haven't had an issue getting the picture facing properly as most maps have north at top of the signs and it is fairly easy to get a flat shot of it. As for printed maps, it is a bit more tricky since you can rotate it yourself. I'm going to eventually allow you to rotate the map - and I do have some experience with image processing and transformation.


Deploy needed objects for prometheus metrics when the operator comes up

This PR ensures that when the operator is deployed, all the proper objects are placed so prometheus can scrape the custom metrics.
-The operator creates a certman-operator service to expose the metrics over on deploy
-The operator creates a certman-operator servicemonitor to tell prometheus how to digest the metrics
-Removes the operator-sdk metrics service for now
-Includes a prometheus-k8s role for the certman-operator namespace that has permissions to get, list and watch services, pods and endpoints.
-Includes a prometheus-k8s rolebinding to allow the prometheus-k8s serviceaccount from the openshift-monitoring namespace to use the aformentioned role.
When the operator and the role/rolebindings are deployed, the certman-operator namespace must be given the following label to get everything working: openshift.io/cluster-monitoring="true"

/hold

/hold cancel

/lgtm

/hold

/lgtm

@tparikh Ping

/lgtm

/hold cancel


[DS-3217] fix more than one test

Revised a few tests to make them work no matter the time of day (they were failing between 23UTC and 24UTC). Used Calendar instead of Date, thanks to a hint from LuizClaudioSantos, set the time zone on the Calendar object thanks to a hint from Helix84. Swapped out some old string hacking for a nice standard SimpleDateFormat object.

https://jira.duraspace.org/browse/DS-3217

https://jira.duraspace.org/browse/DS-3217

this PR replaces #1448

Tested using Vagrant-DSpace:
sudo service vboxadd-service stop && sudo date --set 23:01:00 #set the time to 23UTC to trigger the error
date #confirm the time is set
mvn test -Dmaven.test.skip=false
Confirmed working.

+1 after testing and code inspection
I set my local time to just after 11am NZ time (corresponding to just after 23 UTC). Without this patch, there are test failures that are clearly related to date issues (installItem, checking whether "install" date matches expectations). With this patch, and time still in the hour between 23 and 24 UTC, all tests pass on current master.

+1 via code review. Looks good, and I trust the testing by @hardyoyo and @aschweer


nixos copySystemConfiguration: enable by default

Motivation for this change
system.copySystemConfiguration copies /etc/nixos/configuration.nix into the system derivation at build time.  This can be very useful for debugging, but is much more useful if it was enabled before problems were encountered; however, users are not likely to realize they need this feature until it is too late.
Previously, this feature was not enabled by default because it did not work with chroots (see #7974).  That issue was fixed in e1901a1.

Your PR message contains useful information about why we should have this. IMHO, that belongs in the commit message as well. (Btw, if the commit started out with that message, github would automatically copy it to the PR message when you open the PR.)
How does it work for users with modularized configuration.nix? Does only the configuration.nix file itself get copied, or also all referenced files?

Indeed we cannot enable this option by default because <nixos-config> is not always used, e.g. in NixOps deployments.

Looks to me like this could be closed?

How does it work for users with modularized configuration.nix? Does only the configuration.nix file itself get copied, or also all referenced files?

As noted in the option's description, it's simple and only copies that single file, so the general usability seems rather limited to me.

Hi everyone, and thanks for the input.  With your explanations, it certainly makes sense why it's not enabled by default.  However, I still wonder if there is a way we can help beginners with this, since they're they ones most likely to need it and least likely to know about it.  Just today, there was a post on the mailing list where someone didn't know about this option until it was too late.
@edolstra Perhaps it would make sense to change nixos-generate-config to set this option to true, while keeping the default as false.  That way, people following the instructions for their first install would get it, while NixOps and such would not be impacted.  I'm happy to figure it out and send a PR if this is a good solution.

I was the one that accidentally deleted my /etc/nixops/configuration.nix and asked on the mailing list if there was a way to restore it. Sadly, there wasn't but I was able to more-or-less restore it by snippets (1 and 2) and most of the remaining by memory. :smiley:
While I do see why it is set to false by default (because it would give a false sense of security[1]), I do agree with @ryantrinkle that it would be a good idea to have it set to true in newly generated configs. Perhaps with a comment that unambiguously states that only /etc/nixos/configuration.nix is backed-up.
[1]: which can probably be solved with #709 on Nix: deterministic evaluation of expressions/derivations which will make it possible to determine what files configuration.nix includes and to back those up as well.

[1] would be like using a sledgehammer to crack a nut.

@vcunat fair enough, but it's probably useful for other situations as well. But lets see if we can find a proper nutcracker for this one. 😄


C API Exception

Hi,
The C API method "TF_GraphGetTensorNumDims" throws an exception with message "Node X was not found in the graph" even for valid nodes that are in the graph. This tends to happen with the outputs of particular ops (e.g., reshape and matmul). I think it may have to do with shape inference after looking at the C API implementation but I'm not sure what's wrong. Is there something I need to do to enable shape inference when compiling the shared library? I thought that was enabled by default. Or is there something else that's broken?
Thank you,
Anthony

Shape inference should be happening. Please do fill out the full issue template, in particular the library version being used and a code snippet that reproduces the problem.
What you're saying does sound like a bug, but a reproducible example will go a long way :)
(FWIW, the basic unittest passes, so if there is a bug, we'd love to update the unittests to catch that)
Thanks!

@asimshankar I'll try to create a reproducible example using the Java API. Truth be told, I came about this problem while developing the Scala API and don't have a simple code snippet you can use to reproduce this. One thing that might help for your unit tests is that the problem doesn't occur for many ops. A minimal example that causes the problem for me (using Scala syntax but still real enough) is this:
val images = placeholder(TFUInt8, Shape(-1, 28, 28))
val vectorizedImages = reshape(images, Shape(-1, 784))

vectorizedImages is an Output with Java API semantics. If you now try to call vectorizedImages.shape, you will get the exception I am describing, saying that Node Reshape was not found in the graph. I hope this can help you reproduce the problem, but if it doesn't I can look into creating a code sample using the Java API.
As for the version, I tried this with both the master branch version (current one) and the r1.1 branch. I compiled without --config=opt as this currently causes another problem for which I am currently creating another issue. :)
Thanks!

Small update: Compiling the current master branch code with the --config=opt flag does not resolve the issue.

@asimshankar After trying to replicate the problem with a simple example in the Java API I found out the following:

The reason the previous example was failing was that I was passing a tensor with shape [2, 1] as the second input to Reshape. This needs to be a one-dimensional tensor. I assume that this made this line in the C API fail silently. Thus the shape refiner never registered the op.
I noticed the same happening with other ops. For example, passing a one-dimensional tensor to MatMul results in the same behavior.
Shouldn't we have an error thrown in the native library whenever invalid inputs or attributes are provided for the ops being constructed?

Thanks!

Ah, thanks for tracing this down @eaplatanios ! Sounds like we forgot to clean this up (the precondition mentioned in the comment in the line you pointed to has been met I believe, so we just have to remove that).
I'll try to get that cleared up.
Thanks!

@asimshankar No worries. I'm actually curious. What is the desired behavior for when somebody creates a "Reshape" op using a 2-D tensor provided for the shape? That's just an example, but in general, shouldn't the native library check the op arguments and throw an exception at graph construction time if something is wrong?

@eaplatanios : Yes, the desired behavior would be to have the addition of the "Reshape" operator (which was provided invalid arguments) throw an exception. If we didn't ignore the error, that is what would happen. The reason we ignored the error was a temporary one to ease transition, that's taken care of, so after my fix, the Java call to OperationBuilder.build() will throw an exception.
Hope that makes sense.

@asimshankar That makes total sense! Thanks for the prompt responses! :)


manager/UI: Show the status of instance managers

We've added the instance manager as a part of Longhorn deployment now, but currently, there is no way to know if the instance manager is ready to serve.
We can add a field in the node status: Readiness: Ready, Deploying.
Ready means all the existing (compatible) engine images have been deployed on the node and instance manager of the related engine images are running.
Otherwise, it's deploying.
From UI, click Readiness column of the node to see a matrix of engine images and engine instance manager/replica instance manager to see which one is not ready. We can also show the readiness of the deployed engine image here.
E.g. it can look like this for one node:

Engine
Engine Image
Instance Manager - Engine
Instance Manager - Replica

v0.6.0
Ready
Running
Starting

v0.5.0
Incompatible
Unknown
Unknown

In order for this to work, we need to expose the status of Instance Manager.

Backend merged at https://github.com/longhorn/longhorn-manager/pull/370

@smallteeths Now the backend shows up the managerType to be replica or engine.
UI needs to read from InstanceManager list and EngineImage list. The engine name will be used to match the two.

Validation: PASSED
Steps to test:

Longhorn (master):

On Settings/Engine Image page, Click "Deploy New Image", Enter longhornio/longhorn-engine:v0.5.0
On Node page, Readiness column

Expected Result:
(1) Instance Manager State should be Ready
(2) Clicking on Instance Manager State, a matrix of engine images and engine instance manager/replica should appear and should contain the following:

Engine
Engine Image
Instance Manager - Engine
Instance Manager - Replica

longhornio/longhorn-engine:v0.5.0
Incompatible
Unknown
Unknown

longhornio/longhorn-engine:master
ready
running
running

(3) If an instance manager engine/replica pod is not yet ready, its status should be starting


Typo in the construction algorithm

Going by the text,
This set is reduced by retaining only centroids with a count less than 4 \delta q (1 - q) n

Going by the code,
c_i.count + w_n \leq 4 n \delta q(c_i) (1 - q(c_i))

The latter condition is not only incorrect, it renders the minimum subsequently taken completely useless, as w_n \leq Max(i) - c_i.count already. This leads to pumping the exact value a lot even if the capacity would be exceeded a little bit.
Also relevant is the fact that if c_i already exists in the set then all boundaries on its size are ignored.

Can you point to exactly what you are referring to in the text and code?
Line number would be nice. File name alone would be sufficient.
On Mon, Sep 19, 2016 at 6:11 PM, Alexander Sedov<EMAIL_ADDRESS>wrote:

Going by the text,
This set is reduced by retaining only centroids with a count less than 4 \delta q (1 - q) n
Going by the code,
c_i.count + w_n \leq 4 n \delta q(c_i) (1 - q(c_i))
The latter condition is not only incorrect, it renders the minimum
subsequently taken completely useless, as w_n \leq Max(i) - c_i.count
already. This leads to pumping the exact value a lot even if the capacity
would be exceeded a little bit.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tdunning/t-digest/issues/73, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPSesk-PaXtLIUw9FHYx6dnmRvhcSrtks5qrrQvgaJpZM4KAq6E
.

I’m quoting histo.pdf.

20 сент. 2016 г., в 15:37, Ted Dunning<EMAIL_ADDRESS>написал(а):
Can you point to exactly what you are referring to in the text and code?
Line number would be nice. File name alone would be sufficient.
On Mon, Sep 19, 2016 at 6:11 PM, Alexander Sedov<EMAIL_ADDRESS>wrote:

Going by the text,
This set is reduced by retaining only centroids with a count less than 4 \delta q (1 - q) n
Going by the code,
c_i.count + w_n \leq 4 n \delta q(c_i) (1 - q(c_i))
The latter condition is not only incorrect, it renders the minimum
subsequently taken completely useless, as w_n \leq Max(i) - c_i.count
already. This leads to pumping the exact value a lot even if the capacity
would be exceeded a little bit.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tdunning/t-digest/issues/73, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPSesk-PaXtLIUw9FHYx6dnmRvhcSrtks5qrrQvgaJpZM4KAq6E
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/tdunning/t-digest/issues/73#issuecomment-248288767, or mute the thread https://github.com/notifications/unsubscribe-auth/AAW_XaL9TI1SD2QvH_aMMgIkKg6Kv7Qtks5qr9OegaJpZM4KAq6E.

Thanks.
On Tue, Sep 20, 2016 at 3:01 PM, Alexander Sedov<EMAIL_ADDRESS>wrote:

I’m quoting histo.pdf.

20 сент. 2016 г., в 15:37, Ted Dunning<EMAIL_ADDRESS>написал(а):
Can you point to exactly what you are referring to in the text and code?
Line number would be nice. File name alone would be sufficient.
On Mon, Sep 19, 2016 at 6:11 PM, Alexander Sedov <
<EMAIL_ADDRESS>wrote:

Going by the text,
This set is reduced by retaining only centroids with a count less than
4 \delta q (1 - q) n
Going by the code,
c_i.count + w_n \leq 4 n \delta q(c_i) (1 - q(c_i))
The latter condition is not only incorrect, it renders the minimum
subsequently taken completely useless, as w_n \leq Max(i) - c_i.count
already. This leads to pumping the exact value a lot even if the
capacity
would be exceeded a little bit.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tdunning/t-digest/issues/73, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAPSesk-
PaXtLIUw9FHYx6dnmRvhcSrtks5qrrQvgaJpZM4KAq6E>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <
https://github.com/tdunning/t-digest/issues/73#issuecomment-248288767>,
or mute the thread <https://github.com/notifications/unsubscribe-
auth/AAW_XaL9TI1SD2QvH_aMMgIkKg6Kv7Qtks5qr9OegaJpZM4KAq6E>.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/tdunning/t-digest/issues/73#issuecomment-248294483,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPSejJjQ6Tkxg3cqLtdzMNx1gFLS_Knks5qr9kegaJpZM4KAq6E
.

I spotted this too. The text in histo.pdf says

This set is reduced by retaining only centroids with a count less than 4 \delta q (1-q)n ...

The algorithm 1, on page three however has:
3:  S = {c_i : | c_i.mean - x | < z  \land c_i.count + w_n \le 4 n \delta q(c_i) (1 - q(c_i)) };
4: while S \ne {} /\ w_n > 0 :
5: Sample c_j ~ Uniform(S);
6: \Delta w = min (4n \delta q(c_j) (1 - q(c_j)) - c_j.count, w_n)

Is there a type on line three, should we pick \land c_i.count \le 4 n \delta q(c_i) (1 - q (c_i)), i.e. centroids where there is still space to add some weight. Otherwise the \Delta w on line 6 is always equal to w_n, by the definition of S:
forall c in S, c.count + w_n < threshold, min(threshold - c.count, w_n) = w_n, as
c.count + w_n < threshold => w_n < threshold < c.count


Wrong property type in Portal

The dashboard properties are expected to be "integer" not "number".

@jjbfour , I saw you updated the portal swagger last time and can you please check if this is swagger defect?

@msHich , could you have a look at this issue?

This issue has fixed by pr: https://github.com/Azure/azure-rest-api-specs/pull/8829.


Abfall chart at 624px detail labels are too close together (photo)

click detail button also on the inside

make inside clickable


Bootstrap Multiselect click issue

When i use yadcf in combination with the bootstrap multiselect the stopPropagation() is not working.
Every time that the bootstrap dropdown (ul-element) opens after i click the dropdown button the datatable column sorting is triggered.
When i use the default multiselect of chosen select then there a no problems.
I checked the code generated for the select element, but this had the onclick and onmousedown events like on the normal select elements without bootstrap multiselect.

please provide a test page with your preferred js playground site

https://jsfiddle.net/3hyzorps/6/
When clicking on the select in the first column the ASC/DESC is triggered.

Hi,
There is a special way to apply third party select plugins to yadcf, see the following jsbin to understand how its done, but for some unknown reason the multi select plugin not being opened on - I can't understand why, I created another jsbin in which I apply the the multiselect manually (without using the yadcf way) and it seems to work fine, If you will find the reason it does not open when called from yadcf - let me know

Found the reason, need to comment the following in the yadcf code

 else if (selectType === 'custom_select') {
            selectElementCustomInitFunc($selectObject);
            //$selectObject.next().attr("onclick", "yadcf.stopPropagation(event);").attr("onmousedown", "yadcf.stopPropagation(event);");
        }

but without it muti  select will trigger the sorting of the th, you should contact multiselect author and ask him on how you should solve this,
using the following code does not help either

	$filterSelector.multiselect({ onDropdownShow: function(event) {
                yadcf.stopPropagation(event);
            }});


Inserting into a container has incorrect code

https://github.com/thejameskyle/babel-handbook/blob/master/translations/en/plugin-handbook.md#user-content-toc-inserting-into-a-container
The code should not have the .get('body'), as that returns the actual body array. unshift/pushContainer is a method on the path.

@kenzierocks cool with making a pr to change it? looks like my mistake 😄

Yea, I'll do it tonight.


Individual padding value for each series

Describe your suggested feature
Could you add the ability to customize padding for each series? Not that I want to fiddle with the exact padding, like getting 17% instead of 20% that the app provides, but in the sense that Manga A can have 15% padding, Manga B can have 20%.
I'd to read manga in landscape mode, but each manga is different. Some with more text, and thus requires less padding, so I can read the texts better.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.9.3.
[X] I will fill out all of the requested information in this form.

Personally not interested in this, I believe an issue is already open in Main Tachi


Add upload list to Welcome welcome screen

Add ability to upload files on Welcome screen of Messaging Center similar to Chat screen.
MOB-1722

!merge


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 195 ms

Resolved: ClamAV performance has improved in d039f44.


Storage.Preferences - Clarify when to use Default

There is Storage.Preferences and Storage.Preferences.Default, both offering the Get and Setmethods for storing preferences.
Please document the difference between the two of them.
In the documentation you call Set on Storage.Preferences.Default but Get on Storage.Preferences.
When I do the same in my project, the Get call always just returns the passed default value, never the actual value I stored by a previous Set call.
In code:
// as used in the documentation:
Preferences.Default.Set("test1", new DateTime(2002, 01, 01));
var y1 = Preferences.Get("test1", new DateTime(1999, 01, 01));
// does not work, y1 is now the passed default value 01/01/1999

// but when calling like this
Preferences.Set("test2", new DateTime(2002, 01, 01));
var x1 = Preferences.Get("test2", new DateTime(1999, 01, 01));
// it does work, x1 is as expected 01/01/2002

Tested on Maui for Android, VS 17.3.3

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

ID: a973c0ae-6fc0-2218-2809-2d5973373ce2
Version Independent ID: a973c0ae-6fc0-2218-2809-2d5973373ce2
Content: Preferences - .NET MAUI
Content Source: docs/platform-integration/storage/preferences.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

@tschramme86 Thank you for opening this issue up. They are the same at the moment. The .Default way is the new way to access the default implementation of the preferences interface. The example should be using that.


[BUG] nextUrl is not using basePath or base_redirect_url in OpenID authentication

What is the bug?
When logging in using base_redirect_url (e.g https://my-domain/basePath/), the redirection after successful login is done to "/" instead of base_redirect_url. When adding /app/home/ to initial login request  (e.g. https://my-domain/basePath/app/home), the redirect is done properly.
version
Opensearch 2.2.1
How can one reproduce the bug?
Steps to reproduce the behavior:
Configure opensearch and opensearch dashboards instance to use openid and use a proxy such as nginx to add the params:
server.basePath: "/yourbasepath"
server.rewriteBasePath: true
Try logging in using the base url (e.g https://mydomain/yourbasepath).
What is the expected behavior?
If not request.query.nextUrl then we should redirect to base_redirect_url and if base_redirect_url is None then we redirect to "/".
Potential correction:
/server/auth/types/openid/routes.ts
line
should be:
nextUrl: request.query.nextUrl || getBaseRedirectUrl(this.config, this.core, request) || '/'

[Triage] Thanks for filing

linking issues #1099 and https://github.com/opensearch-project/security/issues/2095

when can we expect a fix for this issue?

@prathaptce A fix for this issue is included in the 2.5.0 release.

Closing this issue as https://github.com/opensearch-project/security-dashboards-plugin/issues/926 was merged.


Lp metric

Closes #773
Generalizes the LpDistance metric to be higher-order. See notes in issue.

Current dependencies on/for this PR:

main

PR #763

PR #765

PR #767

PR #769

PR #771

PR #774   👈

This comment was auto-generated by Graphite.

Closing as this turned out to be un-necessary. Not deleting, as it might be useful for later work.


Add pet images

[ ] Seek permission from developer before using their assets.
[ ] Add images to cards.

Email sent


Type definitions missing from release

Thanks for merging #20 to add TypeScript definitions, but it looks like the file is not included in the released package for v1.0.1.
I'm adding TypeScript support to an existing project, and saw that you had just released 1.0.1 with types, but after upgrading it in my project the .d.ts file is still missing.
Based on<EMAIL_ADDRESS>I can see that it contains the release notes for 1.0.1, but index.d.ts is not present.
So, it seems like there's a setting missing in package.json to include that file when packaging it for release.

Fixed in 1.1.0 I believe. Closing!


Add custom icon pack

Add a custom icon pack with popular icons of apps e.g. linked in, github, etc.

Assign me this issue.

I would like to add the custom icon pack! Please assign me this issue

@swat1998 I would like to work on this issue, please assign me this issue

@AjinkyaTaranekar  Go forward


add activation layer creators.

add activation layer factory and hardmish function.

/assign @oneseer

/approve
/lgtm

/assign @junjiejiangjjj

Codecov Report

Merging #268 (efd3f73) into main (7cda316) will increase coverage by 0.99%.
The diff coverage is 78.40%.

@@            Coverage Diff             @@
##             main     #268      +/-   ##
==========================================
+ Coverage   70.75%   71.74%   +0.99%
==========================================
  Files         171      175       +4
  Lines        5834     5922      +88
  Branches      600      607       +7
==========================================
+ Hits         4128     4249     +121
+ Misses       1500     1465      -35
- Partials      206      208       +2

Impacted Files
Coverage Δ

towhee/models/layers/activations/hardmish.py
45.45% <45.45%> (ø)

towhee/models/utils/create_act.py
61.76% <61.76%> (ø)

towhee/models/layers/activations/__init__.py
100.00% <100.00%> (ø)

towhee/tests/models/utils/create_act.py
100.00% <100.00%> (ø)

towhee/models/layers/activations/hardsigmoid.py
53.84% <0.00%> (+53.84%)
:arrow_up:

towhee/models/layers/activations/gelu.py
58.33% <0.00%> (+58.33%)
:arrow_up:

towhee/models/layers/activations/mish.py
58.33% <0.00%> (+58.33%)
:arrow_up:

towhee/models/layers/activations/hardswish.py
58.33% <0.00%> (+58.33%)
:arrow_up:

towhee/models/layers/activations/tanh.py
60.00% <0.00%> (+60.00%)
:arrow_up:

towhee/models/layers/activations/swish.py
60.00% <0.00%> (+60.00%)
:arrow_up:

... and 2 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 7cda316...efd3f73. Read the comment docs.


grunt-stryker exports the unpublished Gruntfile.js

Summary
In package grunt-stryker, The property "main" (package.json) sets "Gruntfile.js". But there isn't Gruntfile.js in published files. Gruntfile.js isn't in source code, it has been removed from the e51a1f4 commit.

Thanks for reporting this. I've been thinking about this for a while: do we still need grunt support? It still has some downloads/month, but who is using it?

I don't use it (and I don't know how Grunt works). I found this problem by doing the https://github.com/stryker-mutator/stryker-js/pull/4405 pull request.


Design for the API

Requirements
The carbonQL API may internally have 3 APIs.

Energy API  (E)

Marginal intensity API (I)

Embodied emission API

Energy API
Request message will be of the format
{
          getEnergyCPU
               {
                   device type (cloud/laptop/non-cloud)
                   cloud provider name
                   OEM (in case of laptop - dell, HP)
                   region
                   number of vCPU
                  duration
                  durationunit
               }
            getEnergyStorage
                 {
                     device type (cloud/laptop/non-cloud)
                     cloud provider name
                       OEM (in case of laptop- dell,HP)
                     region
                     storage type (hdd/ssd)
                    storage usage
                   storage size unit (TB/GB/MB)
                    duration
                    durationunit
                 }
            getEnergyMemory
                 {
                     device type (cloud/laptop/non-cloud)
                     cloud provider name
                     OEM (in case of laptop- dell,HP)
                     region
                    memorytype (DDR4/DDR3/DDR5)
                    memoryusage
                   memory size unit (MB/GB)
                    duration
                    durationunit
                 }
                getEnergyNetworking
                     {
                          device type (cloud/laptop/non-cloud)
                          cloud provider name
                         region
                         data size
                         data size unit (KB/MB/GB)
                         duration (data transmission duration)
                         durationunit
                     }
}

Marginal Carbon Intensity API
To get the marginal intensity value of the software. This is the increased carbon intensity value due to running of the software. For this we need to understand
a) Region and electricity grid where the hosting infrastructure of the software is getting it's power from
b) the energy intensity mix of the grid
c) the duration for which the software is running or was running.
All these parameters should be passed into the API.
For example
a. Duration in minutes/hours.
b.  The Cloud region where the software is hosted - central us, central India, us west 3 etc.  If it is an on-premise server or laptop/desktop, then this will be the carbon intensity value from the attached electricity grid.
The output from the API will be in gCo2eq/Hr.
Request message will be of the format
{
getCarbonIntensity
     {
         grid region
         duration
     }
}

Embodied Emissions API
Request message will be of the format
{
          getEmbodiedEmissionsCPU
               {
                   device type (cloud/laptop/non-cloud)
                   cloud provider name (aws/azure)
                   OEM (in case of laptop - dell, HP)
                   processorName
                    number of vCPU

               }
            getEmbodiedEmissionsStorage
                 {
                     device type (cloud/laptop/non-cloud)
                     cloud provider name
                     OEM (in case of laptop- dell,HP)
                     storage type (hdd/ssd)
                    storage size
                     storage size unit (TB/GB/MB)
                 }
            getEnergyMemory
                 {
                     device type (cloud/laptop/non-cloud)
                     cloud provider name
                     OEM (in case of laptop- dell,HP)
                    memorytype (DDR4/DDR3/DDR5)
                    memorysize
                   memory size unit (MB/GB)

                 }
                getEnergyNetworking
                     {
                          device type (cloud/laptop/non-cloud)
                          cloud provider name
                          region
                     }
}

@srini1978 this is why I like GraphQL as the framework for this project. GraphQL is like an API façade in front of multiple other "things", perfect when you want to create a single API that secretly behind it might have dozens, hundreds of other end points or bit of code that return data and it allows for flexibility on the calling side. When they make a call to the API they can request data in multiple different formats nested calls, all kinds of things.
How our architecture would be built if it was in GraphQL is that we'd define resolvers, see: https://www.apollographql.com/docs/apollo-server/data/resolvers/. A resolver is like an atomic piece of code that returns one part of the API request, so we'd have a resolver for CarbonIntensity that takes some inputs and returns some responses.
Out of the box we might ship with a resolver for free public data.
If someone installs the WattTime package it adds a WattTime resolver to the chain and tries to use that first and if nothing is returns falls back to other resolvers in the chain.
If we can keep this in mind, I'm not 100% on this but my hunch is telling me this could be a good fit for this project.

The GitHub API is a GraphQL API, this is how you'd form requests to the API: https://docs.github.com/en/graphql/guides/forming-calls-with-graphql

@srini1978 we need to decide what we do with this issue and all related issues as we moved to framework/sdk creation and API-based approach is not we pursue now.

Hi @srini1978 I believe this issue can be closed now, the conversation has moved onwards and most of this is either incorporated into other docs/specs/conversations or has been superseded. Let me know if you want to reopen.


Can't start the Vulcan project using the quick-start-from-nodejs template

Environment

Version:

cli version: 0.4.0
core version: 0.4.0
build version: 0.4.0
serve version: 0.4.0

OS: macOS

node: v16.17.0

What's wrong?

Can't start the Vulcan project using the quick-start-from-nodejs template
After further inspection using vulcan serve,  there was an error message showing: Error  Cannot find module 'bcryptjs'

What's the correct behavior?
It seems like bcryptjs is missing inside the package.json file; and at least the server can be successfully started with "bcryptjs": "^2.4.3" installed.
Reproducing steps

installed VulcanSQL CLI from NPM
vulcan init --template=quick-start-from-nodejs
vulcan start --watch -> we can found that the server is not running at localhost:3000
vulcan serve -> there should be an error message as described above

Thanks for the issue feedback! We will fix it in the next version


Feature Request: smart quotes

I don't know if this is a common thing in markdown or not, so maybe it should be an option instead of a default feature, but it would be nice if the previewer showed smart quotes in its preview. When markdown is entered on sites like Tumblr it does the smart quotes -- though, as I say, I don't know if it's applicable in all situations where markdown is used.
For example:
"this text won't be rendered with smart quotes"

would become:
“this text won’t be rendered with smart quotes”

There exist at least three different types of "smart quotes", I think it's related to the language-settings:

“these smart quotes are your example”
«these smart quotes are used in France»
„These quotes are used in Germany‟
The the document-language had to be known, and then the corresponding locals be loaded.


chore: init zk-kit.circom repository

This PR initializes a new repo for hosting Circom circuits, copying the circuits from the ZK-Kit repository as part of a new structure of the ZK-Kit packages. ZK-Kit will have a repository per language.
Re https://github.com/privacy-scaling-explorations/zk-kit/issues/282

the utils package circuit aren't building from compile root script. Got the following error:
<EMAIL_ADDRESS>Process started
<EMAIL_ADDRESS>===| Compiling the circuit |===
<EMAIL_ADDRESS>Error: No such circuit in ./circuits.json

the utils package circuit aren't building from compile root script. Got the following error:

I should have solved it! Let me know 👍🏽

yes, works fine - thank you for the quick fix

amazing, makes sense to have one package per circuit - the configs are lit, just a few nits!

Thank you so much! Applied all changes you suggested 👍🏽


关于docker运行plex的问题

我用了 docker hub中的 linuxserver/plex 和 plexinc/pms-docker 两个镜像都试过了，在容器“其它配置”里使用任何参数都会造成失败。
Plex Media Server first run setup complete
[cont-init.d] 40-plex-first-run: exited 0.
[cont-init.d] 50-plex-update: executing...
[cont-init.d] 50-plex-update: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
exec: fatal: unable to exec -p: No such file or directory
-p exited 111
Starting Plex Media Server.
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] syncing disks.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.
root@Openwrt:/mnt/sda3#

其中错误为 exec: fatal: unable to exec -p: No such file or directory，任何参数包括--networkk什么的都会报错

日志上已经说了，你映射的文件或文件夹不存在


Fix pps with documentation

This PR's intent is to fix #266
A Documentation token is added to the lexer, so that we can print the ones between an OCaml declaration and gospel specifications directly as attributes. The other are printed as documentation comments, so that OCaml parser deal with them appropriately.

CI is green-ish (the failure is a git-fetch one).
So PR is ready for review.

Suggestion: maybe the docs preprocessor should apply to OCaml ASTs rather than strings; that looks simpler to implement.

I'm working on a ppx rewriter that turns the Gospel attributes into documentation attributes (here for now). But this ppx relies on having the Gospel content in Gospel attributes, that is it relies on gospel pps.
The issue I've stumbled on (#266) is that the current gospel preprocessor (gospel pps) detaches some of the documentation (if we want the gospel attributes in the OCaml AST, we may lose some of the documentation ones).
This PR proposes a fix to that issue, so that we have all the documentation attributes in the OCaml AST, even after gospel pps.


Raspberry pi 4 armv7 base images

To make it work with rpi, you've to use armv7 base images for both the app and mongo.
Then point your browser to http://raspberrypi:8080 on the network as before

Thanks for the pull requests. Can add instructions for using baby-tracker with a Raspberry Pi (ARM device) to the README?


Add support for Windows containers

Following the release of Ryuk 0.7.0, this adds basic support for Windows containers as a first step for #5621.
The behaviour on Linux and macOS should not be changed in any way, and Windows support depends on the daemon reporting a "windows" operating system.

A WCOW compatible image of https://github.com/testcontainers/helloworld is required for the ResourceReaperTest to succeed on Windows containers.

A WCOW compatible image of https://github.com/testcontainers/helloworld is required for the ResourceReaperTest to succeed on Windows containers.

Now as pull request prepared at https://github.com/testcontainers/helloworld/pull/5


[Bug][framework] PIPELINE_MAX_PARALLEL is not working

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The total number of running pipelines stays at 1 not matter what value is given by PIPELINE_MAX_PARALLEL

What do you expect to happen
total number of running pipelines can reach up to the given PIPELINE_MAX_PARALLEL
How to reproduce

set PIPELINE_MAX_PARALLEL in .env file or by EnvVar
create multiple pipelines
observe pipelines statuses from the database

Anything else
No response
Version
main
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It looks cause by #3986 and fixed.


Links from various places are broken: http://dai.cs.rutgers.edu/dai/s/acknowledgments

This is linked to from various menus and also from other web pages.
So we should make sure that URL has content.
If you can point that URL instead to this one:  http://www.bu.edu/asllrp/dai-credits.html
I can update that page...
But since there are links to that page from so many different places, we should keep that URL working.
Thanks!

The link to Terms of Use is also broken:  http://dai.cs.rutgers.edu/dai/s/terms

This was the original content of that page:
The data available from these pages can be used for research and education purposes, but cannot be redistributed without permission.
Commercial use, without explicit permission, is not allowed, nor are any patents and copyrights based on this material.
Those making use of these data in research must, in resulting publications or presentations, cite, as appropriate:
C. Neidle et al. (2017) The ASLLRP SignStream® 3 Corpus, Boston University, Boston, MA.
C. Neidle et al. (2017) The ASLLRP Sign Bank, Boston University, Boston, MA.
and this URL:  http://www.bu.edu/asllrp/about-dai2.html .

We should add a few more references.  I'll add those in a separate comment.  Thanks!

This link is also broken - again referenced from multiple places:
http://dai.cs.rutgers.edu/dai/s/overview

I have updated the http://www.bu.edu/asllrp/dai-credits.html page.  So the easiest thing would be to have http://dai.cs.rutgers.edu/dai/s/acknowledgments  just redirect to that page.  Then all the links to http://dai.cs.rutgers.edu/dai/s/acknowledgments can be left as is.  Thanks.
likewise I have added a  “terms of use” page:
http://www.bu.edu/asllrp/dai-terms.html

You can eitalsoer redirect from http://dai.cs.rutgers.edu/dai/s/terms
   to that page (or else replicate the page above at the Rutgers URL).

The HOME link is also broken.

The Search menu is also broken.

Remove overview page from all dai pages


Telemetry got changed to True silently.

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Linux Mint 20.3 Una 64-bit Linux Mint 20.3 Una 64-bit
- Continue: v0.9.132 (pre-release)
- IDE: Version: VSCodium 1.89.0

Description
While tweaking things in the config.json I noticed the Telemetry setting had reverted to "true" without warning.
To reproduce
Dunno how long it has been like this. I always opt-out of telemetry whenever presented with the choice, or otherwise see it hidden in the settings.
Log output
No response

@TiagoTiago I'm going to fix this immediately, there's absolutely no world in which we want this to happen.
If you can share any more details about what seemed to happen it would help me immensely to solve this quickly.
I will update soon! Very sorry this happened

I'm sorry, I don't know what more details I can give; I just noticed it was set to true while I am confident I would've left it at false. Over time I've occasionally seen the extension asking VSCodium to be restarted, I imagine to apply some update, I didn't double-check the settings every time that happened, so I don't even know if it happened recently or not. I remember there was one time maybe some weeks ago where the extension received an update and forgot the models I had set and I had to add them again, perhaps that the could have been the moment when the telemetry setting also go reset, but I didn't think to check at the time, so it's just a guess.

@TiagoTiago this is good information, I appreciate that. I do recall that we had made the mistake of rewriting models in config.json for a brief instant in time. This was a release that was followed up almost immediately with a fix, but chances are that a few folks would have had the misfortune of needing to add back their old setup. What I'm looking into now is whether this would have also included resetting telemetry.
Testing is showing that the current setup does not override allowAnonymousTelemetry: false, so under the assumption that the problem happened after selecting an option on the new onboarding screen, this to me seems like the reassurance needed to know that the problem has been solved.
If you wanted to validate this, these are the steps that I am following:

Set "allowAnonymousTelemetry": false in config.json
Click the "?" icon in the bottom right corner of Continue
You'll see this screen, where you can click "Open Tutorial" in the top right

Select any of the options on the onboarding screen (I've tested with all 3), and click "Continue"
Open config.json to see that we've populated the model attributes, but that "allowAnonymousTelemetry": false remains intact

Wiped out everything when I clicked on the Ollama option and tried to proceed with the wizard (I had gone the manual route before using TextGenWebUI, so I decided to try differently this time to be sure I could see if it changed anything in the json; apparently not only it did, but it did too much); telemetry remained false though, so at least that's good.

Ok, telemetry being the same is the important part. I made a small update here to avoid overwriting the entire models list like you described: https://github.com/continuedev/continue/commit/6f8528a5346c75ddfe05abcc3f43e6076dd97791
I want to be completely sure before I go ahead and close this issue—does it seem that there is anything still outstanding?

I can't test the new patched version at the moment; but if both the original telemetry reset issue and the wiping of already setup models have been fixed, then I don't see need for this bug report to remain open.
I won't be asking for the little bit of data that has been collected to be erased because if the collection is as restricted as described at the "What we track" section, then providing sufficient additional data to identify what came from me will only serve to speed up de-anonymization efforts; what leaked, is already leaked, can't do anything about it that won't make matters worse.

@TiagoTiago I'm going to fix this immediately, there's absolutely no world in which we want this to happen.
If you can share any more details about what seemed to happen it would help me immensely to solve this quickly.
I will update soon! Very sorry this happened

Why isn't telemetry opt-in?

Probably a good compromise between the developer's desire to have the most telemetry possible, and the user rights, would be to have it so when the setting is not initialized, prompt the user on install/update to pick between "Enabled" and "Disabled", with both at first set as empty radio buttons, so that by default it remains uninitialized until the user makes a decision. Having a default that's either of the options will bias the end result as people have a tendency to leave settings at the default value. In short, neither opt-in nor opt-out, just plain opt :)

Closing this issue because the original concern has been addressed. Thank you both for your feedback


I don't think this is needed

I think you can just do the following to do this natively within AWS SAM:
Specify the lambda resource
  LambdaFunction:
    Type: AWS::Serverless::Function
    Properties:
      CodeUri: lambdas/
      FunctionName: !Ref RedirectLambdaName
      Handler: RedirectLambda.handler
      Role: !GetAtt RedirectLambdaFunctionRole.Arn
      Runtime: nodejs12.x
      AutoPublishAlias: live

Then in the CloudFront resource, for the lambda function arn:
              LambdaFunctionARN: !Ref LambdaFunction.Version

You may not realize this, but this plugin has nothing to do with SAM. This plugin is for the Serverless framework.


Flutter run crashes with specific project name on linux device

Steps to Reproduce

Create a project named "plugins"
Use flutter run
Flutter crashes with exception

Logs
CMake Error at /usr/share/cmake-3.16/Modules/FindPkgConfig.cmake:463 (message):
  A required package was not found
Call Stack (most recent call first):
  /usr/share/cmake-3.16/Modules/FindPkgConfig.cmake:643 (_pkg_check_modules_internal)
  flutter/CMakeLists.txt:28 (pkg_check_modules)

Exception: Unable to generate build files

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel dev, 1.24.0-10.2.pre, on Linux, locale fr_FR.UTF-8)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for
      more details.
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[!] Android Studio (not installed)
[✓] IntelliJ IDEA Community Edition (version 2020.1)
[✓] Connected device (3 available)

! Doctor found issues in 2 categories.```

Hi @fodil-a
Just tried to reproduce the issue with a new project with plugins, no issues  on dev or master
(base) taha@pop-os:~/AndroidStudioProjects/plugins$ flutterd run -d linux
Downloading linux-x64/linux-x64-flutter-gtk tools...                5.4s
Downloading linux-x64-profile/linux-x64-flutter-gtk tools...      2,395ms
Downloading linux-x64-release/linux-x64-flutter-gtk tools...      2,141ms
Running "flutter pub get" in plugins...                            238ms
Launching lib/main.dart on Linux in debug mode...
Building Linux application...
Waiting for Linux to report its views...                             5ms
Syncing files to device Linux...                                    80ms

Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h Repeat this help message.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).
An Observatory debugger and profiler on Linux is available at:
http://<IP_ADDRESS>:44271/LIU2YnVg7DM=/
Application finished.

flutter doctor -v
[✓] Flutter (Channel dev, 1.24.0-10.2.pre, on Linux, locale en_US.UTF-8)
    • Flutter version 1.24.0-10.2.pre at /home/taha/Code/flutter_dev
    • Framework revision 022b333a08 (2 weeks ago), 2020-11-18 11:35:09 -0800
    • Engine revision 07c1eed46b
    • Dart version 2.12.0 (build 2.12.0-29.10.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /home/taha/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /home/taha/Code/sdk
    • Java binary at: /home/taha/Code/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 11.0.0-2
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.2

[!] Android Studio (not installed)
    • Android Studio not found; download from
      https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup
      for detailed instructions).

[✓] VS Code (version 1.51.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.16.0

[✓] Connected device (3 available)
    • Linux (desktop)  • linux      • linux-x64      • Linux
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome
      87.0.4280.66

! Doctor found issues in 1 category.

Can you please provide your flutter doctor -v your complete flutter run -d --verbose and a complete reproducible minimal code sample
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

yes you can close, the problem was a missing library that i had to install (recent change ?)

yes you can close, the problem was a missing library that i had to install (recent change ?)


Run prerender on a subroute

Is it possible to run prerender on a seperate route?
I want to run prerender on http://localhost:3000/prerender/http://example.com instead of http://localhost:3000.
for technical reasons, I can not set-up a reverse proxy in front of prerender so that's not a possibility.

You can export a PORT environment variable to change which port it uses. If you mean a different route on your own website where localhost:3000/prerender goes to the prerender server and a request to localhost:3000 goes to your normal website, then that's not possible with just the prerender server.

Yeah I basically wanted prerender to listen on a different root path instead of "/". But since this is not supported I will have to find a different workaround.
Would be cool to have an option that can be passed in, root-route or something that would support this.
Feel free to close the issue. Thanks for answering my question 👍

Sure thing!


tweak chatbot test to make test less flakey

Still flakey 😭

Still flakey 😭

This did not fix #53 but #65 did.

This did not fix #53 but #65 did.


Added .ipynb link to the .md file

Description
This will help reader go directly to the code, rather than finding it in the GitHub
Type of change

[x] Mistake in text fixed

Added to SUMMARY.md (delete section if not needed)?

[x] Already Added

@all-contributors add @Abhiroop25902 for documentation


Updating Modular file extension.

Updating Modular's file extension to .maxviz to avoid any conflicts with existing .mod file formats.

Updated test file:
resnet34.maxviz.zip


Avoid resolving objects in getTypeFacts when caller doesn't need that info

This stems from a discord conversation where getTypeFacts spends all of its time resolving structured type members, when all it cared about was whether or not the type was nullable.
It turns out that the vast majority of calls to getTypeFacts only care about a subset of flags (and mainly, nullable-ness), so this may be a perf win.

@typescript-bot perf test this
@typescript-bot pack this

A percent, maybe?

@typescript-bot perf test this full

I'm going to take this out of draft as I think that even if it's at most a percent, we really kinda need a lot of small 1% wins. Curious what you all think here.

@typescript-bot test top100
@typescript-bot user test this
@typescript-bot user test tsserver
@typescript-bot test tsserver top100
@typescript-bot run dt


fix(provider): Count ports for a service only in reservations

Right now the current code counts any service endpoint as consuming a port. This isn't really true for HTTP services, they get multiplexed all via the ingress controller.
This code just counts the endpoints that wind up being a NodePort in Kubernetes are consuming a port.

is this in master and mainnet/main?


Conversion from Python datetime to Julia DateTime fails

using CondaPkg
using PythonCall
using Dates
datetime = pyimport("datetime")
t = datetime.datetime.now()
pyconvert(DateTime, t)

results in
cannot convert this Python 'datetime' to a Julia 'DateTime'

Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:35
 [2] macro expansion
   @ ~/.julia/packages/PythonCall/3GRYN/src/convert.jl:355 [inlined]
 [3] macro expansion
   @ ~/.julia/packages/PythonCall/3GRYN/src/Py.jl:131 [inlined]
 [4] pyconvert(#unused#::Type{DateTime}, x::Py)
   @ PythonCall ~/.julia/packages/PythonCall/3GRYN/src/convert.jl:370

with the minimalist CondaPkg.toml
[deps]
python = "3.9"

but it fails with Python 3.10 and 3.11 too.
This is with CondaPkg v0.2.17 and PythonCall v0.9.10

Hmm I think this is because a Julia DateTime is to millisecond precision whereas a Python datetime is to microsecond precision, and PythonCall currently refuses to do the conversion unless the microseconds are zero.
I could add a conversion rule so that if you specifically ask for a DateTime then it will do the lossy conversion.

Such a conversion would work fine for what I am doing. Thanks.

In the meantime, you can do this to get the time to the nearest second:
using CondaPkg
using PythonCall
using Dates
datetime = pyimport("datetime")
t = datetime.datetime.now()
# t = t.replace(microsecond=0)  # nearest second
t = t.replace(microsecond=(t.microsecond//1000)*1000)
pyconvert(DateTime, t)

Your proposition gives me a stacktrace:
MethodError: no method matching //(::Py, ::Int64)

Closest candidates are:
  //(!Matched::Integer, ::Integer)
   @ Base rational.jl:62
  //(!Matched::Rational, ::Integer)
   @ Base rational.jl:64
  //(!Matched::Complex, ::Real)
   @ Base rational.jl:78

I tried to add pyconvert(t.microsecond, Int) but that does not work either
MethodError: no method matching pyconvert(::Py, ::Type{Int64})

Closest candidates are:
  pyconvert(!Matched::Type{T}, ::Any) where T
   @ PythonCall ~/.julia/packages/PythonCall/qTEA1/src/convert.jl:370
  pyconvert(!Matched::Type{T}, ::Any, !Matched::Any) where T
   @ PythonCall ~/.julia/packages/PythonCall/qTEA1/src/convert.jl:371

But honestly, converting a Python datetime.datetime to DateTime is mundane enough that it should work out of the box. The truncation from microseconds to milliseconds is unavoidable, and the programmer who needs to do that conversion will bite the bullet.

It's pyconvert(Int, t.microsecond).
The pyconvert rules are lossless by default, hence refusing to truncate. However there's an argument that small losses in precision are OK (just as convert(Float16, 1.2) loses precision) so maybe it's OK in this case. I've just been burned too many times by Python happily converting floats to ints.
In the future, I'm planning to add keyword args to pyconvert to allow more flexible conversion, so that e.g. pyconvert(DateTime, t; truncate=true) will allow a loss of precision.


Hardcode operating mode env variable

We had problems with the deployments due to a wrong value in the OPERATING_MODE en variable. So we decided to hardcode to avoid issues with out-of-date config maps. Moreover, we're not able to change the value of config maps in an automatized manner.

Moreover, we're not able to change the value of config maps in an automatized manner.
Oh, why is this? Can't this be a deployment config triggered change?

And btw. Do you think it's a better way to handle it? Maybe still keeping this in the configmap template in our repo and just remove it from the configmap in OS (if it's feasible to support configmaps merging on the OS side cc: @jfchevrette @vpavlin) is better?

@alexeykazakov the best way is to let the owners of the application to manage the properties that are passed as arguments. That isn't possible unless we hardcode all the properties. Regarding the secrets, we could also manage them by using third party (vault, gpg keys, ...). However I understand the situation at this moment.

As long as this is not something that will change across environments (i.e. stage non-clustered, prod clustered) I am fine with this approach and I think it's appropriate solution for this particular problem.
Rule of thumb here is - if you are 100 % sure this value will be same across all the envs (local, stage, prod, hybrid-cloud-of-your-dreams) then feel free to hardcode it. Otherwise, use configMaps

The ideal requirements are the following:

It should be possible to configure the clustered/standalone mode depending on env (both Cluster mode and the number of replicas bc it will fail if there is more then 1 replica in standalone mode)
There should be the default values (clustered mode + 3 replicas). If the env vars is not set in particular environment then the default ones are used.
Devs should be able to change the default values.

Can we achieve all of this if we keep the default values in our template but just don't specify them in the OSIO OSD environments. But if needed we can override them in any environment/cluster we want (with the SD team help)?
If it can take some time to meet all the requirements we can merge this PR for now because it's better than we have now and create separate issues to improve it.

@alexeykazakov

It should be possible to configure the clustered/standalone mode depending on env (both Cluster mode and the number of replicas bc it will fail if there is more then 1 replica in standalone mode)

Whatever is pushed to prod-preview, it would be released in production. Therefore we can only have one environment, consequently a template.

There should be the default values (clustered mode + 3 replicas). If the env vars is not set in particular environment then the default ones are used.

Yeah! we experienced that before, we need more than one replica if we set KC to clustered mode.

Whatever is pushed to prod-preview, it would be released in production. Therefore we can only have one environment, consequently a template.

Probably true for preview vs prod. But we could have other envs. Appsec or some on premise or local cluster which would be nice to control via configmap.

Yeah! we experienced that before, we need more than one replica if we set KC to clustered mode.

Yep. At least 3 replicas for cluster mode and the only one for standalone. Otherwise KC will fail.

We might have situations where we would need to run production in one mode and staging in another.
Example, if there's an issue with clustered mode, we might want to temporarily set production to standalone mode while we debug staging in clustered mode.
So in that case , workflow should be

Deploy standalone in both ( using the proposed way here in this PR )
Manually change staging to clustered mode and proceed with the debugging?

I created a new issue to follow up - https://github.com/fabric8-services/fabric8-auth/issues/45


[Multitag] Move search location inside multitag list wrapper

Problem
It is not possible to correctly style a multitag flounder that spans multiple lines.
Currently
The search input is located outside of the multitag list wrapper. The position of the cursor is set using setTextMultiTagIndent(). This approach does not allow us to have a multitag flounder than spans multiple lines.
Wanted
In multitag mode the search location should be inside the multitag list wrapper. That is, <input class="flounder__input--search"> should be inside <div class="flounder__multi--tag--list">. We no longer need setTextMultiTagIndent()
Tasks

[x] move search location inside multitag list wrapper
[x] update CSS
[x] remove setTextMultiTagIndent() and any associated methods
[x] update tests

Small CSS (z-index) tweak still needed here, reopening.


Problem matcher should use unique onwer if omitted

VSCode Version: 1.3.0
OS Version: any

Currently the owner is set to external. This can cause problems if two matchers are defined with no owner.

Verified in code


Missing yarn prepack step

Environment
I'm using yarn v3 and importing @sentry/capacitor as a Git dependency (this is because I am using my fork).
Steps to Reproduce

Forked this repo
In my package.json dependencies, I now have "@sentry/capacitor": "borgoat/sentry-capacitor#head=yarn-2"
yarn install

Expected Result
I can still import and use @sentry/capacitor, but based on my fork.
Actual Result
I am not getting a correctly packaged dependency. The dist folder is missing, and the library is therefore unusable.
Proposed Fix
Configure  the prepack lifecycle step to perform the library build, instead of prepublishOnly. This is the best practice according to Yarn as far as I can tell ^1.

Note about yarn pack: Although rarely called directly, yarn pack is a crucial part of Yarn. Each time Yarn has to fetch a dependency from a "raw" source (such as a Git repository), it will automatically run yarn install and yarn pack on it to know which are the files to use.

and also

[...] the prepublish hook shouldn't have side effects. In particular don't transpile the package sources in prepublish [...]

If you agree, I can provide a quick PR to address this.

Thank you for pointing out!
As far as I can judge there'll be no side effects to others , since it's a quick fix I opened the PR but feel free to review if you feel like there's anything else that you'd like to see :)

I'm not really sure if that will impact this change or not, but I should mention that I did my tests with yarn 3.5 since we only use that internally. The branch in my fork uses that^1. While this repo is still using v1 which has not received much attention lately, maybe it's worth migrating^2.


Tabs does not work properly for duplicate names

Example of bug: http://jsfiddle.net/kby77myL/1/

Looks good, thanks!


ci: make Meson warnings fatal

New warnings can be hard to notice in CI, since CI will just pass in
that case. Meson sometimes uses warnings for important mistakes, e.g.
invalid option.
Let's turn warnings into errors so that we can spot these more easily.

Note, we need to bump the Meson version to get rid of the meson.build_root() warning.

Ah, this already caught a mistake in the Alpine CI build, good.

Good catch on the alpine CI ;)

I don't mean to be rude, but isn't it a little aggressive to "hard-depend" on a meson version that was just released several days before?  Sure, people who find this (and understand it) can patch it back easily or they may even be able to update their distro's meson, but still...

How is it aggressive that our master (i.e. bleeding edge, unreleased work) depends on other recent releases (not even the latest release at the time of writing this comment) whose features or fixes it needs? In my opinion it seems hypocritical to require the absolute latest unreleased work of a project, but then disallow it from depending on any recent releases.
If your distro does not have a recent meson release, it can easily be fetched with pip.


Adding Middleware

What about adding a middleware for filtering HTTP requests. Application like authenticated, CORS, CSFR, IP Restriction Can be done.

Hi @sunel, thank you for your contribution. I have been thinking about this addition for a while but not able to work on it yet. A PR would be great :)

Great 👍


remove the old UI codebase

Remove the old UI codebase
Description of the changes:

Deletes all the C++/CX files
Retains assets and resources

How changes were validated:

The codebase are gone, the app cannot run so nothing to validate at this stage.
New codebase will be ported here once this PR is checked in.

Hi @hanzhang54 ,
Could you help review this PR by the end of tomorrow? Thanks.

LGTM


How to print accuracy in test_submission.py.?

How to print accuracy in test submission.py.?

test_submission only produces predictions without ground-truth files, so if you like to see the acc/iou metrics, you will need to submit them to https://competitions.codalab.org/competitions/23732?secret_key=dba10d3a-a676-4c44-9acf-b45dc92c5fcf#results

Line 143 in test_submission.py, giving error..
How to resolve this and call this function "def metrics(predictions, gts, label_values=land_classes):".?
I am trying to call this function.
Share a solution for this.

To use metrics() function, you have to gts - the ground truths data.
You may want to use it to evaluate the val set, if so you could refer to evaluate() function defined in lib/utils/measure.py, and it's used like
acc, acc_cls, mean_iu, fwavacc, f1 = evaluate(predictions_all, gts_all, train_args.nb_classes) # line 191, train_R50.py

In which source file is multi perspective embodied?

In which source file is multi perspective embodied?

Are the following enough to print the metrics such as confusion matrix, accuracy, f1score etc?
in test_submission.py, add a new line:
prepare_gt(TEST_ROOT_FOR_GT_ONLY) , where TEST_ROOT_FOR_GT_ONLY = 'dataset/test/'
and comment out the following lines:
96, 98, 102, 140, 143, 144

After these changes, while running test_submission.py :
Traceback (most recent call last):
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 241, in
main()
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 42, in main
tta_real_test(nets, stride=600, batch_size=1,
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 101, in tta_real_test
test_images, test_labels = (loadtestimg(test_set))
ValueError: too many values to unpack (expected 2)

There is no labels provided for test set, so you can not use prepare_gt() for it. test_submission can not computer accuracy/F1 etc

I am working now on a new test dataset with labels (in the required format). Is is possible to compute the metrics (with current code) since I have labels?

The current test_submission.py didn't provide evaluation. You might need to write some code youself to compute the accuracy If you test set has GT files, you can call the function evaluate(predictions, gts, num_classes) provided in the file/lib/utils/measure.py, then you will get all supported scores.

An example code for test_with_ground_truth. you can modify it according to your case.

after modification, i am getting a 'key' error  while reading the gt as follows:
test_labels = (np.asarray(io.imread(label), dtype='uint8') for label in test_set[GT])
Traceback (most recent call last):
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 255, in
main()
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 43, in main
tta_real_test(nets, stride=600, batch_size=1,
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 106, in tta_real_test
test_labels = (np.asarray(io.imread(label), dtype='uint8') for label in test_set[GT])
KeyError: 'gt'

While I try the following modification:
test_labels = (np.asarray(io.imread(label), dtype='uint8') for label in test_set[GT])
for img, id, gt in tqdm(zip(test_files, idlist, test_labels), total=len(test_set[GT]), leave=False):
The following error:
Traceback (most recent call last):
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 260, in
main()
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 43, in main
tta_real_test(nets, stride=600, batch_size=1,
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 131, in tta_real_test
for img, id, gt in tqdm(zip(test_files, idlist, test_labels), total=len(test_set[GT]), leave=False):
File "/usr/local/lib/python3.10/dist-packages/tqdm/notebook.py", line 222, in iter
for obj in super(tqdm_notebook, self).iter(*args, **kwargs):
File "/usr/local/lib/python3.10/dist-packages/tqdm/std.py", line 1081, in iter
for obj in iterable:
File "/scratch/manu/MSCG-Net-master_selftrained/./tools/test_submission.py", line 106, in
test_labels = (np.asarray(io.imread(label), dtype='uint8') for label in test_set[GT])
File "/usr/lib/python3/dist-packages/skimage/io/_io.py", line 48, in imread
img = call_plugin('imread', fname, plugin=plugin, **plugin_args)
File "/usr/lib/python3/dist-packages/skimage/io/manage_plugins.py", line 207, in call_plugin
return func(*args, **kwargs)
File "/usr/lib/python3/dist-packages/skimage/io/_plugins/imageio_plugin.py", line 10, in imread
return np.asarray(imageio_imread(*args, **kwargs))
File "/usr/lib/python3/dist-packages/imageio/core/functions.py", line 221, in imread
reader = read(uri, format, "i", **kwargs)
File "/usr/lib/python3/dist-packages/imageio/core/functions.py", line 136, in get_reader
format = formats.search_read_format(request)
File "/usr/lib/python3/dist-packages/imageio/core/format.py", line 694, in search_read_format
if format.can_read(request):
File "/usr/lib/python3/dist-packages/imageio/core/format.py", line 195, in can_read
return self._can_read(request)
File "/usr/lib/python3/dist-packages/imageio/plugins/dicom.py", line 93, in _can_read
with open(filename, "rb") as f:
PermissionError: [Errno 13] Permission denied: '/core'

The above issue solved when i used the following to read labels:
test_labels = (loadgt(test_set))


Dying breaks the cheat log

When player dies, their golf nbt is corrupted (I have yet to look at the playerdat file right before and right after death, will do tomorrow morning) If you do /golf player @s seeCheatLog after dying (without doing clearCheatLog first), you will get an "Unexpected error" (it's a NullPointerException when you hover over it).

Oh, I thought I already fixed it 🤔 . Will look into it once more, thanks for reporting.

Oh, I thought I already fixed it 🤔 . Will look into it once more, thanks for reporting.


numpy<2 ?

Warning message shows when execute python weights/convert_safetensor_to_pt.py:
A module that was compiled using NumPy 1.x cannot be run in
NumPy 2.1.2 as it may crash. To support both 1.x and 2.x
versions of NumPy, modules must be compiled with NumPy 2.0.
Some module may need to rebuild instead e.g. with 'pybind11>=2.12'.

If you are a user of the module, the easiest solution will be to
downgrade to 'numpy<2' or try to upgrade the affected module.
We expect that some modules will need time to support NumPy 2.

After downgrade to v1.26.4, warning gone.

Yes, numpy version should be v1.26.4. Will update it in the requirements.txt


cts, mts, cjs, mjs, etc extension support

You should probably look over https://github.com/weswigham/TypeScript/pull/67 instead of this, so you're not reviewing two PRs of changes together - this PR is just here to run typescript bot commands on the PR~

@typescript-bot pack this

Merged in #45884


Ability to change font size of Chip text

I am probably just missing something in Inspector and the stylesheet, but is there currently a term e.g., how there is mush-chip-height, or mush-title-font-size?
I am trying to increase their size so they are easier to use on a touch display. I love these lovelace addons - thank you for all the great work!

This will be fixed in the next mushroom release : https://github.com/piitaya/lovelace-mushroom/pull/121
The font-size for span, icon and svg will be adjusted automatically.

Maybe I am missing something, but how do I adjust the font size manually of the chip text?

@Kev1000000 The font-size (and icon size) is automatically adjusted with the chip height. The default chip height is 36px. If you set it to 72px, the font-size will be doubled too.

@Kev1000000 The font-size (and icon size) is automatically adjusted with the chip height. The default chip height is 36px. If you set it to 72px, the font-size will be doubled too.

How can we set this to 72px?

You can do this using theme : https://github.com/piitaya/lovelace-mushroom-themes

You can do this using theme : https://github.com/piitaya/lovelace-mushroom-themes

I'm having the same issue.  How can I override the primary or secondary text font size for a specific card only?  I don't want to change this at the global theme level.  Any help?

How can I override the primary or secondary text font size for a specific card only? I don't want to change this at the global theme level. Any help?

any idea?

It says the theme is not required any more as of 2022.11.  If that is the case how would one change the font for an individual card such as a mushroom entity card?  I've been playing with this for a few hours and it has me stumped.  I can change the background colour like so:
- type: custom:mushroom-entity-card
entity: sensor.upstairs_atc_temperature
fill_container: true
card_mod:
style: |
:host {
--ha-card-background: teal;
}
but have not been successful changing the font size.  Help greatly appreciated.

Ah checkout the mushroom theme, there is a list of settings in there, for example under "card" there is:
mush-card-primary-font-size: 14px;
That will set the text in a card. Here is an example:
      - type: custom:mushroom-entity-card
        entity: sensor.downstairs_atc_temperature
        card_mod:
          style: |
            :host {
              --ha-card-background: green;
              --mush-card-primary-font-size: 26px;
            }

I close it as theme variables can be set for one card using card-mod.


Buffer.validate() runs twice per input.

AcceptAction.validate_and_handle runs Buffer.validate and then Buffer.append_to_history, which also calls Buffer.validate. It seems like this should be eliminated with refactoring or caching.
class AcceptAction(object):
        def validate_and_handle(self, cli, buffer):
        """
        Validate buffer and handle the accept action.
        """
        if buffer.validate():
            if self.handler:
                self.handler(cli, buffer)

            buffer.append_to_history()

class Buffer(object):
    def append_to_history(self):
        """
        Append the current input to the history.
        (Only if valid input.)
        """
        # Validate first. If not valid, set validation exception.
        if not self.validate():
            return

        # Save at the tail of the history. (But don't if the last entry the
        # history is already the same.)
        if self.text and (not len(self.history) or self.history[-1] != self.text):
            self.history.append(self.text)

Thanks for pointing out!
This should fix it: https://github.com/jonathanslenders/python-prompt-toolkit/commit/43a52365cc99ceb71c5f7bc815525d03140d75fd

Thanks, that does it!


Warning in rn 0.25.1 +

hi all,
how to get rid of the warning:
ReactNative.createElement is deprecated. Use React.createElement from the "react" package instead
?

Would you like to send a pull request?

Hi, actually I'm asking how to fix the message.

Are you sure that it is not produced by your code?

yes.
It even occurs  with the example when I change ReactNative version to 0.25.1
2016-05-30 14:24 GMT+02:00 Evgeniy Sokovikov<EMAIL_ADDRESS>
Are you sure that it is not produced by your code?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/skv-headless/react-native-scrollable-tab-view/issues/291#issuecomment-222481380,
or mute the thread
https://github.com/notifications/unsubscribe/AAFe7cBKqkzERixWtExwN0bRvhb6GDMEks5qGtcQgaJpZM4IpeQO
.

Releases Vsersion-0.25.1 had some deprecations.
import React, { Component, View } from 'react-native';

to
import React, { Component } from 'react';
import { View } from 'react-native';

So we got below warning message:
Warning:
ReactNative.createElement is deprecated. Use React.createElement from the "react" package instead

We can easily change with codemod suggest by react-native official release v0.25.1 note.

I don't see where I'm using ReactNative.createElement

@skv-headless I also don't see ReactNative.createElement your module. :+1:
It will be another module.

Should be fixed in master.


Copy-Paste still indents

Fantastic extension - works great when typing new code.
However, when copy-and-pasting a block of code, the Smart Indent still kicks in and it indents the line below those tags again.  Any chance you could add a fix for that, too?
Thanks again!

Glad you find it useful!  You're right, Copy/Paste is still a problem and isn't something I've tackled yet in this extension because I didn't want to clutter up the undo/redo stack.  I think I have an idea of how to get around that problem.  I'll see if I can't have a fix for copy/paste soon.

I just merged a change that should fix copy/paste macro indent issues.  Let me know if you have any issues with it.

Works great!  Passing it around to my team now and including it into our "getting started" docs in the recommended setup section, with full credit to you!  Thanks!

Awesome!  👍


grafana/dashboards/docker_container.json file can be used for k8s containers?

I have defined datasource for my k8s containers in open shift and used the json file from your project
by changing datasource from "Prometheus" to "". it seems to catch the graphs correctly but does not display any data.
Any idea?
Thanks,
Ram

I assume that's because when you change the data source it likely just inserts test walk data.
I suspect you could likely get it working on a k8s cluster but you likely would need to change some of the inputs and perhaps filter on your nodes/pods.
@stefanprodan any insight you might be able to provide? Would the swarmprom repo be a better alternative for this?


Unable to start Prime CMS

Hi.
$ nodemon --unhandled-rejections=strict index.js
[nodemon] 2.0.3
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): *.*
[nodemon] watching extensions: js,mjs,json
[nodemon] starting `node --unhandled-rejections=strict index.js`
[nodemon] clean exit - waiting for changes before restart

This happens in my Kubernetes cluster when I deploy Prime.
Can I set a logging level to see what's going on?

Sure can, just add DEBUG=prime* to your environment variables for that pod/container.
This will likely be the user.meta is not a function error however.


Show proper validation error when --issue --title is used without a body

No, ec420e1 does not fix this

Ok, here we go again, 626857b appears to fix this by just resolving this, but it conflicts in horrible ways with sth. like this:
ideaseed -iT"gporekpogerkp" "grejogerj"

where the user has just forgotten to specify a repo, and instead of failling it tries to be smart and resolve "grejogerj" to the repository name, but in this case it is not the repository but the IDEA, so I'm going to revert 62687b and mark this as wontfix.

626857b reverted in 0d317295a6669cf4961457767884f76b22c3f6ed