refactor: replace zod schema deprecated function nonempty() with min(1)

Closes
🎯 Changes

✅ Checklist

[ ] I have followed every step in the contributing guide.
[ ] If necessary, I have added or updated the tests related to the changes made.
[ ] If necessary, I have added documentation related to the changes made.

Current dependencies on/for this PR:

main

PR #69   👈

This stack of pull requests is managed by Graphite.


[Feature Request]: Make Assets Extractor Efficient

In Main.java make the assets extractor efficient

no longer relevant


music

cant seem to get the music to work

Pleas close this issue and re-ask in #1053

You cannot self-host on Windows 7, fyi.


Index Page Load Failure on Main Branch Due to Missing demo.js

Issue
Unable to correctly load the index page on the main branch, which blocks subsequent operations.
Error Details
Flask reports a "404 Not Found" error for the missing demo.js:

"GET /static/js/demo/demo.js HTTP/1.1" 404 -

This occurs because the demo.js file is absent in the main branch.
Steps Taken
The demo.js file from the human-human/llm branch located at overcookedgym/overcooked-flask/static/js/demo/demo.js has been copied to the corresponding location on the main branch.
New Error After Fix
After fixing the missing demo.js file, a KeyError occurs on the playtesting page with the message:

"POST /bc/predict/ HTTP/1.1" 500 -
Traceback (most recent call last):
...
File "overcookedgym/overcooked-flask/app.py", line 165, in predict
data_json["npc_index"],
KeyError: 'npc_index'

Thanks for your asking.
I think you may missed some part of the README.md
In the "1. How to setup" of "Getting Started", these is a sequence of command you need to run such as "npm install" and "npm run build". That is to build the overcooked_ai game core.
Indeed, in the human-human/llm branch, I leave the demo.js in it. However, this demo.js is different from the built demo.js in main branch, that's why I said you need to re-run the "npm build". I'm so sorry that I didn't express it more clearly, which makes you confuse.

Thanks for your asking. I think you may missed some part of the README.md
In the "1. How to setup" of "Getting Started", these is a sequence of command you need to run such as "npm install" and "npm run build". That is to build the overcooked_ai game core.
Indeed, in the human-human/llm branch, I leave the demo.js in it. However, this demo.js is different from the built demo.js in main branch, that's why I said you need to re-run the "npm build". I'm so sorry that I didn't express it more clearly, which makes you confuse.

It works, thanks for the reminder.


Install major version symlink of shared objects.

This change addresses the issue discussed in #1202 and should make it possible in the future to update minor versions without requiring re-linking binaries.

Thanks, this will make things nicer going forward.
I'll merge after we tag v1.2.0

I don't see a problem getting it into 1.2.0.


fix(scipts): update prettierignore

Description
Please explain the changes you made here.
Which issue(s) does this PR fix

Fixes https://issues.redhat.com/browse/RHIDP-1414

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/cherry-pick 1.1.x


soroban-rpc: rename simulateTransaction to preflight

see discussion

@janewang do you think this is something that we still want to do? Or should we close this?

Yes we could keep simulateTransaction. I will open a doc ticket to align from the docs.


Fixed Page Height

max-height added to each page css so the div doesn't keep increasing with respect to content.
div resize must also be done.

Thanks!
First, we have to differentiate between screen and print mode. These CSS lengths only affect the screen mode, and thus the proposed enhancement is only for the screen mode, too. In print, everything is fine already.
Regarding the proposed enhancement, I don’t think it will work that way. Setting min-height instead of just height was a trade-off that had to be implemented because, in screen mode, when you put in more content than fits the page, it will just overflow. The content will not magically skip to the next DOM element, which would be what we need here. Such a visual effect is only possible with CSS Paged Media, which you can see in print mode. In our simulation in screen mode, that does not work, and we would rather have pages that are too long than content that is shown between pages (in the empty space).


Document usage of problem details in the client app

I was looking for advice, how to use ProblemDetails on the receiver end.
The class itself is defined in Microsoft.AspNetCore.Mvc and is contained in the Http.Abstractions assembly.
Both parts make me wonder, how we'd use ProblemDetails in a client application.
This might be a non issue, but I'd rather not include the aforementioned assembly into a Blazor-WASM project or a desktop application.
Perhaps it would be useful to make some samples using ProblemDetails with generated clients via nswag.
Also the scenario of a shared class for the "http models" would be nice.

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

ID: 3f6504cd-a059-c5d4-73fb-61222ef1c6e0
Version Independent ID: 1b42352c-a34f-7703-aa3e-d7e3156bd0bd
Content: Handle errors in ASP.NET Core web APIs
Content Source: aspnetcore/web-api/handle-errors.md
Product: aspnet-core
Technology: aspnetcore-webapi
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Thanks for the suggestions, we'll be reviewing this document.


Adds Initial Infra Manager

Adds initial Infra Manager implementation.
xref: https://github.com/envoyproxy/gateway/issues/41
Signed-off-by: danehans<EMAIL_ADDRESS>

Codecov Report

Merging #178 (00578a3) into main (e20d7ad) will decrease coverage by 1.63%.
The diff coverage is 40.90%.

@@            Coverage Diff             @@
##             main     #178      +/-   ##
==========================================
- Coverage   61.74%   60.11%   -1.64%
==========================================
  Files          20       23       +3
  Lines        1814     1968     +154
==========================================
+ Hits         1120     1183      +63
- Misses        632      715      +83
- Partials       62       70       +8

Impacted Files
Coverage Δ

internal/cmd/server.go
24.32% <ø> (ø)

...ternal/infrastructure/kubernetes/serviceaccount.go
21.42% <21.42%> (ø)

internal/infrastructure/kubernetes/infra.go
35.13% <35.13%> (ø)

internal/ir/infra.go
62.29% <62.29%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.


Feature/canvas measure text

I noticed that using canvasCtxt.measureText() is much faster than using an svg (at least on Chrome; I haven't tried this in any other browsers yet), so I thought maybe you'd be interested in this code.

Hi @bbloomf, as a matter of fact this is more or less how I originally implemented all of the text measuring: the chant context held on to a canvas context which it used for text measuring, even when outputting to SVG. I was trying to squeeze more performance out of the layout pipeline, and while it mostly works (canvas text measuring is not quite as precise as measuring SVG text nodes, but usable in most cases), the bigger issue is that the canvas doesn't support strings with more advanced typography. For example, canvas small caps uses faux small caps not true smcp glyphs.
I seem to also recall that there is no way to actually get baseline measurements via the canvas, meaning that text alignment needs kludges (like the "this.bounds.height = this.fontSize * 1.2" in the pull request).
But you're right, it's much faster using the canvas to measure text, since there is no DOM manipulation involved. Maybe it could be implemented as an optional feature for those preferring speed over output quality/features?

Yes, it looks like there are additional text metrics that may someday be supported by canvasCtxt.measureText() but aren't supported in any browsers right now, that would return various measurements relating to the vertical axis: https://developer.mozilla.org/en-US/docs/Web/API/TextMetrics
I do think it would be nice to have the option of preferring either speed or precision.

I just changed this to allow either strategy for measuring the text based on an optional argument passed into the ChantContext constructor.  Let me know what you think.

Yeah, that looks good. I think I'd prefer that the logic of text measuring (based on the strategy)  be encapsulated in the ChantContext, but that could be refactored later too... Thx!


Missing X_4P_4DAC1_WFBYTE symbol

Checklist:

[x] I updated to the latest spin2 version available
[ ] I have restarted VSCode and the problem still exists

Version of Spin2 Extension with the issue: Current
Last working version (if known):
Hardware, Operating System, VSCode version: Does it really matter?

Description of problem:

@Wuerfel21 (as long as I'm fixing things... "should have checked before releasing last, sigh") I thought i'd find and fix this too.  Thanks for reporting this.

repaired. Fixes avail in v1.3.9 and later.

Speaking of, what came of the whole "regex debug parsing" thing?
I did some research then and figured that matching balanced brace sequences is impossible with regex....
But not impossible with the oniguruma engine that VSC uses, which allows for recursive patterns...

I still need/want to make it better.
We should setup zoom call... I'm interested to hear what you've found out.

Sure, just shoot me a mail when you have time


fix logic in JError.custom

Signed-off-by: André Roth<EMAIL_ADDRESS>

I'll merge that after upgrade python on production servers. Looks good but I use python 3.4

Thank you!


Missing POI data in SDS public data sources

Hi there,
I have been exploring this SDS API since it can list the x closest entities from a given geopoint. I'm particularly interested in looking for grocery store entities from the NAVTEQNA data source (https://msdn.microsoft.com/en-us/library/hh478192.aspx). However, it seems that some stores are not available in this database. For instance, in Canada there is a chain of stores called "Real Canadian Superstore". This is not found in the database when using the following query
http://spatial.virtualearth.net/REST/v1/data/f22876ec257b474b82fe2ffcb8393150/NavteqNA/NavteqPOIs?spatialFilter=nearby({my location})&$filter=EntityTypeID%20eq%20'5400'&$select=EntityID,DisplayName,Latitude,Longitude,__Distance,AddressLine,Locality,CountryRegion&$top=6&key={bingMapsKey}

How would one go about getting the NAVTEQNA data source updated with new entity data?
Cheers,
Daniel

This is a public API that is maintained by NAVTEQ. Any one can make a data source public and manage it themselves. It is unlikely that the NAVTEQ data source will be updated any time soon, however the Bing Maps team is planning on releasing some point of interest API's in 2018.


Consider making AR process work with multi-dimensional arrays

I think it can be made to work, but as written probably would not work "as expected". So for now I've made things stricter about 1D arrays and added tests that the errors get raised (in e0273ae)

Originally posted by @dylanhmorris in https://github.com/CDCgov/PyRenew/pull/380#discussion_r1725697048

This was closed by #439


Range setters - how to handle it?

Hey!
I'm using ranged properties (as per your SDK for homie on esp8266).
zoneNode.advertiseRange("on", 1, 3).settable(zoneOnHandler);
What would be the standard message to MQTT to send range 1 with value X?
eg. devices/c40f46e0/zone/on[1]/set ??
Sorry I just couldn't find any documentation on how to handle it.
Kind regards,
Luke

The range index is appended with an underscore.
As example:
Actual MQTT messages from my [heating control] (https://github.com/euphi/ESP_Heizungscontroller).
The set message is from Openhab and the reply of the homie-esp8266 device:
devices/hzgctrl/Relais/in_9/set ON
devices/hzgctrl/Relais/in_9 ON

For four information, the constructor of the node is:
RelaisNode::RelaisNode() :
		HomieNode("Relais", "switch16"), relais_bitset(0x0000), io(0x20) {
	advertiseRange("in",1,16).settable();
}

and the resulting properties message:
devices/hzgctrl/Relais/$properties in[1-16]:settable

Hi Euphi,
It all makes sense now! Thanks.
Regards,
Luke


Flipped environment background texture when using viewport + render tex or viewport stretch display mode

Here's a simple scene that is setup to show the issue:
https://dl.dropboxusercontent.com/u/3292249/flipped_texture_env_BG.zip
Run the scene first and you'll see the texture is the correct way up
Now go to project settings > Display and set stretch_mode to 'viewport'
Now run the scene again and you'll see that the texture is flipped vertically
This is probably related to the same issue that caused  #2180

@sketchyfun Could you have a look that the fix that was just pushed might not invalidate your already merged PR #2180? (e.g. if reduz fixed the root issue and your PR was fixing a consequence of it).

@akien-mga I wondered that too, but I just built and checked it and it both environment background texture and billboard sprites display correctly when using viewports, so looks like it's fine

Great, thanks for checking :)


V2 - Can't select "Fully Shaded" in CLE category on Add Tree screen

On the Add new tree screen, in the Crown Light Exposure dropdown list, you're not able to select "Fully Shaded" as an option.
I believe this is due to the fact that the value for this option is 0, and the logic in the select menu and the tree benefits calculator is dependent on non-falsey values. I tried a few things, but I'm not sure where the change should be made.

I'm working on this... I thought it would be simple, but it ended up leading me down a long rabbit hole...
When I fix one thing, something else breaks, and so on... I learned that I'm not a fan of the !! operator...


Image and Text become blurry after Convert to PDF

Apologize if there is already an answer somewhere that I haven't found it.
I have an HTML page with 595px width, after convert to pdf, the Image and Text become blurry.
my options setting of pdf:  { format: 'Letter'}
The screenshot of Web page:

The screenshot of PDF

Thanks in advanced!

I am seeing the same thing.

:( I haven't experienced this. What's the resolution of the images you're using. Is the text part of the image? Or separate text? If text gets blurry it might be some font issue. Blurry images are a completely different one.

@marcbachman I have the same issue with images, I have two base64 png in my page that get blurry. Is it because of the original png quality or their dpi? Can we set the dpi for generated pdf?

@marcbachmann Thanks for the reply.
The images and text are separated, images are png files with the size of 6kb. I can't find a good solution to it, only to increase the size of text font and images' size to 20kb to compensate the losing quality.


index out of bounds for tuple

The code at http://play.crystal-lang.org/#/r/bk5 causes error index out of bounds for tuple {Int32}. How can I reference tuple arguments that might not exist? Apparently an if isn't enough.

You define a method overloads with two or more arguments

In this particular case, sure - and I have since done so. But in general, there may be cases where, say, a function returns a tuple, and we need to extract values from it. There ought to be a way to handle tuples whose number of elements is not necessarily known at compile-time, don't you think?

There is a special operator for that. See http://play.crystal-lang.org/#/r/bmo

Jason

—
Jason Waldrip
Chief Technology Officer
brandfolder.com
<email|<EMAIL_ADDRESS><office |<PHONE_NUMBER>>
<mobile |<PHONE_NUMBER>>
On Aug 16, 2015, 5:45 PM -0600<EMAIL_ADDRESS>wrote:

In this particular case, sure - and I have since done so. But in general, there may be cases where, say, a function returns a tuple, and we need to extract values from it. There ought to be a way to handle tuples whose number of elements is not necessarily known at compile-time, don't you think?
—
Reply to this email directly orview it on GitHub(https://github.com/manastech/crystal/issues/1195#issuecomment-131641082).

Example using method overloads to find the length of a tuple at compile time: http://play.crystal-lang.org/#/r/bmy

@jwaldrip I didn't know the '?' Would fix this particular problem. Thanks.

@bjmllr Interesting solution.


[2.6.3-p1] Make webhook certificate auto renew ahead of expiration

https://github.com/rancher/rancher/issues/36079

Validation Environment

Component
Version / Type

Rancher version
v2.6.3-patch1-rc2

Rancher cluster type
Docker

Docker version
20.10.7, build 20.10.7-0ubuntu5~20.04.2

Helm version
v2.16.8-rancher1

Downstream cluster type
N/A

Downstream K8s version
N/A

Browser type
Google Chrome

Browser version
97.0.4692.99 (Official Build) (x86_64)

Validation steps

Find the secret called cattle-webhook-tls in the cattle-system namespace

Starting from the default Rancher homepage at /dashboard/home
Click hamburger menu -> click local -> click Storage -> click Secrets -> set search to All Namespaces -> find cattle-webhook-tls -> click ellipsis (three dots) -> click Edit Config
Copy the Certificate contents
You can also get the certificate by just running the following command against the local cluster

kubectl get secrets -n cattle-system cattle-webhook-tls -o json | jq -r ".data.\"tls.crt\"" | base64 -d

Copy the certificate contents and decode them. You can use a site like https://certificatedecoder.dev

Details about the initial cert:

Cert
Last 10 characters of cert
Issued on
Expires on

Initial cert before any changes
wJOyTgPhl6
Mon Jan 24 2022 17:52:12
Thu Jan 22 2032 17:52:12

Delete the certificate secret

In the UI you can click the ellipsis (three dots) and select Delete

Go to the rancher-webhook deployment in the cattle-system namespace

Click hamburger menu -> click local -> click Workload -> click Deployments
for the rancher-webhook deployment listed under Namespace cattle-system -> right click & select Edit Config
Under Environment Variables click Add Variable creating one with these details

Type: Key/Value Pair
Variable Name: CATTLE_NEW_SIGNED_CERT_EXPIRATION_DAYS
Value: 2

Click Save

Check that the cert has been recreated with a new 2 day expiration, using either method from step #1

Cert
Last 10 characters of cert
Issued on
Expires on

Initial cert before any changes
wJOyTgPhl6
Mon Jan 24 2022 17:52:12
Thu Jan 22 2032 17:52:12

Cert after adding 2 day env var
ktDb4lwbCm
Mon Jan 24 2022 17:52:12
Wed Jan 26 2022 18:09:04

Wait 6 hours then check the cert again

Cert
Last 10 characters of cert
Issued on
Expires on

Initial cert before any changes
wJOyTgPhl6
Mon Jan 24 2022 17:52:12
Thu Jan 22 2032 17:52:12

Cert after adding 2 day env var
ktDb4lwbCm
Mon Jan 24 2022 17:52:12
Wed Jan 26 2022 18:09:04

After waiting 6 hours
J628SaIg==
Mon Jan 24 2022 17:52:12
Jan 27 2022 06:09:04

To get a 10-year certificate, update the deployment again by removing the environment variable and save

RESULT
The cert updates after waiting 6 hours, and deleting the environment variable resets the cert back to 10 years

Release Note
New and existing rancher-webhook deployments will automatically renew their TLS certificates when the expiration date is 30 days or less. See #36177.


Fixtype cast bug about dateType and dateTimeType

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[x] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 34.73 seconds
load time: 544 seconds
storage size:<PHONE_NUMBER>7 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230119112217_clickbench_pr_84154.html


Multiple clusters on the same machine ?

Is it possible to start multiple clusters with ccm ? I'm on Mac OS X.

You can create multiple clusters, but they generally can't be running at the same time, because of bound ip/ports.
Use ccm list to view all your current clusters, and ccm switch to choose the active cluster.

Is it not possible to change the ip/ports that is bound to clusters, to enable multiple clusters on the same machine?

Yes, that is possible

@NihalHarish
You can create multiple clusters like below but you can't start all at once. Only one can be active at a time.


Documentation on createLevelLabel is confusing

Which version of Serilog.Sinks.Grafana.Loki are you using?
v7.1.0
Which version of .NET are you using?
net6.0
Describe the bug
The docs here:

https://github.com/serilog-contrib/serilog-sinks-grafana-loki/wiki/Application-settings
https://github.com/serilog-contrib/serilog-sinks-grafana-loki/wiki/Visualizing-log-level-in-Grafana

clearly state that:

The level label always won't be created while using ILabelAwareTextFormatter
Which includes the LokiJsonTextFormatter.

But why is the example appsettings in https://github.com/serilog-contrib/serilog-sinks-grafana-loki/wiki/Using-LokiJsonTextFormatter-and-Grafana-Loki-json-parser
using both
"textFormatter": "Serilog.Sinks.Grafana.Loki.LokiJsonTextFormatter, Serilog.Sinks.Grafana.Loki",

and
"createLevelLabel": true,

if createLevelLabel is ignored in this case?
I tried variations using createLevelLabel in combination with filtrationLabels etc. but createLevelLabel does not seem to have an effect when using LokiJsonTextFormatter ?
While the JSON parsing (using | json  in Grafana is working as described, I would like to have level as a real Loki Label (for usage in stream selector because it is easier to type), as it was pre 7.0,
But I guess this is not possible in any way?
To Reproduce
Use the configuration in https://github.com/serilog-contrib/serilog-sinks-grafana-loki/wiki/Using-LokiJsonTextFormatter-and-Grafana-Loki-json-parser
including
"labels": [
    {
        "key": "app",
        "value": "web_app"
    }
],
"filtrationMode": "Include",
"filtrationLabels": [
    "app"
],
"outputTemplate": "{Timestamp:dd-MM-yyyy HH:mm:ss} [{Level:u3}] [{ThreadId}] {Message}{NewLine}{Exception}",
"createLevelLabel": true,

A loki label level is never created (as per design?)
Expected behavior
There should be some way to restore the behavior from pre 7.0 to get a real loki level label?
We MUST use the JSON formatter because of cardinality and to keep the log properties.
Log/SelfLog output or exception with stacktrace
No response
Application or code sample, which could be used to reproduce a bug
No response
Additional context
Also, does outputTemplate do anything when using LokiJsonTextFormatter ? It did not look like it but maybe I got confused.
I have read the documentation

[X] I have read Common Problems section in project wiki

I remembered I've already kind of answered this myself in https://github.com/serilog-contrib/serilog-sinks-grafana-loki/pull/16#issuecomment-774980316
So if I use
          "createLevelLabel": true,
          "filtrationMode": "Include",
          "textFormatter": "Serilog.Formatting.Compact.RenderedCompactJsonFormatter, Serilog.Formatting.Compact",

I'm back to the desired behavior. What's not clear for me @mishamyte :

What is the advantage of using LokiJsonTextFormatter in contrast to e.g. RenderedCompactJsonFormatter ? What features does it have that RenderedCompactJsonFormatter or CompactJsonFormatter does not ?

Hi @Mario-Hofstaetter,
The discussion was here.
After that I have received a lot of appeals about this implicit behavior. So I decided to change it v7.0.
It is mean that if you are using json-based formatter, you also use json pipeline in Loki.
This leads to smaller count of labels.

But why is the example appsettings in https://github.com/serilog-contrib/serilog-sinks-grafana-loki/wiki/Using-LokiJsonTextFormatter-and-Grafana-Loki-json-parser
using both
"textFormatter": "Serilog.Sinks.Grafana.Loki.LokiJsonTextFormatter, Serilog.Sinks.Grafana.Loki",

and
"createLevelLabel": true,

Thank you, it's a typo (first there was simple formatter). Will fix it.

What is the advantage of using LokiJsonTextFormatter in contrast to e.g. RenderedCompactJsonFormatter ? What features does it have that RenderedCompactJsonFormatter or CompactJsonFormatter does not ?

You could compare all named formatters, using their source code:

LokiJsonTextFormatter
RenderedCompactJsonFormatter


Need to migrate from NLTK to PyTextRank

https://spacy.io/universe/project/spacy-pytextrank
https://spacy.io/usage/spacy-101
https://spacy.io/usage/linguistic-features
https://spacy.io/usage/linguistic-features#noun-chunks
https://spacy.io/api/doc#sents
https://www.kaggle.com/neomatrix369/summariser-pytextrank-class
https://jcharistech.wordpress.com/2018/11/28/natural-language-processing-with-textacy-spacy/

Fixed


TSC Standard Library Overlap

While running tsc over my app, with new-javascript installed, I ran into a few different compilation errors, many of them being overlapped declare class and interface definitions that are already defined in the standard library, which are already defined using the interface declaration type.
I can try to help with this too, wasn't sure how to work on it exactly though. I thought I'd let you know about it before I got too far into it.
I took this screenshot while running tsc -w ./index.d.ts inside a cloned copy of new-javascript on my local machine.

The problem was mostly that I referenced the WebWorker lib for some stuff which resulted in the WebWorker type declarations overriding the DOM ones. I now split everything up into index.d.ts for DOM stuff and worker.d.ts for workers. Some stuff like navigator.userAgentData is available both in the main thread and in workers, so I added the navigator extensions directly into the index.d.ts and worker.d.ts files.
If you update to<EMAIL_ADDRESS>and make sure your TypeScript and TSC are on version 4.9.4 it should theoretically work; please let me know if it doesn't.

Worked perfectly! No more errors at all. Thanks for fixing it! Sorry to spam with all of these issues, haha.


Cache embeddings

Is your suggestion for improvement related to a problem? Please describe.
We have implemented storing of chat history, but currently we haven't implemented storing of embeddings.
Describe the solution you'd like

When user opens a .bib library, a background process should start that generates the embeddings.
Embeddings are stored in a local user directory.
User can access the AI chat only at the end of embeddings generation. Or when all the embeddings for one BibEntry are generated.

Additional context

When user adds a new BibEntry or a LinkedFile the process starts again, but will generate embeddings only for the additions.
[IMPORTANT ONLY FOR NEXT ITERATION] When user changes an EmbeddingModel, all the embeddings should be invalidated.

Challenges

We have two choices for when to allow the user to chat with entries:

When all the embeddings for the library are generated.
Pros: easy.
Cons: the user have to wait.
When the embeddings for one BibEntry are generated the user should have the ability to chat with it. Other generation tasks for other entries should continue in background.
Pros: users doesn't have to wait.
Cons: synchronization. Will it be okay to read embeddings while chatting and in the same time writing embeddings for new entries?

Okay, let's start with simple:

Let's store all embeddings at run-time and throw them away at exit.
When user opens a database, run a background task that will generate embeddings in background.
While embeddings are being generated, the user can't chat with AI.
At the end of generation, user can chat with AI.

Umm, I failed.
So, at first I thought it would be a GUIPostOpenAction, but I need a PreferencesService and AiService, they are not awailable in GUIPostOpenAction classes.
Where to put this code?


abnormal tooth mineralization - subterms

abnormal tooth mineralization MP:0002817
Add new child terms:
abnormal enamel mineralization
DEF: anomaly in the process by which calcium salts, mainly carbonated hydroxyapatite, are deposited into the dental enamel
SYN: abnormal enamel calcification
SYN: enamel mineralization defects
SOURCE: GO:0070166
abnormal dentin mineralization
DEF: anomaly in the process by which calcium salts are deposited into dentin
SYN: abnormal dentin calcification
SYN: dentin mineralization defects
SOURCE: GO:0097188
abnormal cementum mineralization
DEF: anomaly in the process by which calcium salts, mainly carbonated hydroxyapatite, are deposited into the initial acellular cementum
SYN: abnormal cementum calcification
SYN: cementum mineralization defects
SOURCE: GO:0071529

Added:
abnormal enamel mineralization MP:0030450
abnormal dentin mineralization MP:0030451
abnormal cementum mineralization MP:0030452


Change "WORKSPACE_BASE" environment variable in "Quick Start" docs

This WORKSPACE_BASE=$(pwd)/workspace
If you run echo $WORKSPACE_BASE in terminal it will print nothing! (which means the variable not set.)
It should be export WORKSPACE_BASE=$(pwd)/workspace
Now if you run echo $WORKSPACE_BASE in terminal it will print /home/kali/workspace (which means the variable in now set)
I think this will fix this problem: Comment #2332238664

@mamoodi
My way is more comprehensive as if you try it on your environment it will work, but when I try yours on my environment it doesn't work and of course -e WORKSPACE_MOUNT_PATH=$WORKSPACE_BASE \  variable will be certainly empty in that case (while the end-user don't know what is the problem!)

Updated README and docs. Thank you!
Closing.


Repeatedly building a failed results in weird io error

It seems like repeatedly building a recipe when it's failing eventually results in some weird behavior. I've seen both
Error:   × FileSystem error: 'Io error. Look inside err_kind for more details.'

and
Error:   × IO Error: failed to create directory `/private/tmp/ioerror/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-
  │ build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/
  │ rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/
  │ output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/
  │ work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-
  │ build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/
  │ rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/
  │ output/bld/rattler-build_cool_1706803408/work/output/bld/rattler-build_cool_1706803406/work/output/bld/rattler-build_cool_1706803404/
  │ work/output/bld/rattler-build_cool_1706803404/work/recipe`
  ├─▶ failed to create directory `/private/tmp/ioerror/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-
  │   build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/
  │   bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/
  │   work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-
  │   build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/
  │   bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/
  │   work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-build_cool_1706803409/work/output/bld/rattler-
  │   build_cool_1706803409/work/output/bld/rattler-build_cool_1706803408/work/output/bld/rattler-build_cool_1706803406/work/output/bld/
  │   rattler-build_cool_1706803404/work/output/bld/rattler-build_cool_1706803404/work/recipe`
  ╰─▶ File name too long (os error 63)

repro
On OSX, not sure if this might be relevant
mkdir -p /tmp/repro/recipe
cd /tmp/repro
curl -JLo recipe/recipe.yaml https://gist.githubusercontent.com/benmoss/5b7b0dc23edda98fc34a2f5eb987f045/raw/3e4a933a07660995887c8d6bd2412e1c1db8c8dd/meta.yaml

rattler-build build -r recipe/recipe.yaml
< Cannot solve the request...>

rattler-build build -r recipe/recipe.yaml
< IO error >

It seems like if I am doing this in /tmp/ I get the File name too long error, whereas if I'm in another directory (/Users/bmoss40/workspace/rattler-build-stuff) I get the Io error. Look inside err_kind for more details error.

Looks like this is the same as https://github.com/prefix-dev/rattler-build/issues/450


graphite: call the right util module

This change is

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

Comments from Reviewable


expose the event for onChange

I want to call event.preventDefault for onChange function, but since the onChange function does not receive the event object, I am unable to do it.
I have a keyboard listener on document, so whenever I click delete, the keyboard listener on the document is being triggered. If we could call e.preventDefault in onChange function, I think this could have been prevented

@mohammedzamakhan I also need the native (synthetic React) event object to use NumericInput with Controller in react-hook-form. Did you find an alternative solution, a walk-around? Thanks.


Rollup and modern tooling compatibility

Bug report 🐞
Geist doesn't work with Rollup, and modern tooling like Snowpack, due to problems in styled-jsx specifically mentioned here and here.
This is a great UI library, but that the fact that it can only run with Webpack is defeating the purpose of minimalistic and modern ESM compatibility that this library can offer. Is using styled-jsx essential for this library? It doesn't seem very well maintained.
Version & Environment

Rollup and Snowpack

Expection
The behavior I expect is to get bundled without bugs.
Actual results (or Errors)
I got an error: here and here.

About ESM:
I tried to compile a bundle of esm using babel, it also seems to work if you need esm mode, but the output package is still not concise enough.
About rollup:
My initial thought was to wait for rollup to fix the issue and then I would follow up with an upgrade, now it seems that this problem still exists and has not been solved, which is a technology selection error. Due to a large amount of code that depends on styled-jsx, there is no easy way to move to other libraries yet.
I evaluated the workload of removing styled-jsx when I added the esm module, the simplest solution would require changes to each component, test case, and server-side rendering module, so, at least in version 2.x I have not removed the idea of styled-jsx.
A sad conclusion 😢 , but the good thing is that users can still use esm bundle, it's just that it will be a bit larger than the best practice package. (I would guess that the volume of useless code is about 10% - 25%)

For any new comments, please reopen it.


Needed: Collection Object Quick form: Quick sequence

As a curator, I need to be able to link collection objects, extracts, and sequences in a Collection Object relationship with Origins defined, but one of those objects is a data class "Sequence", and the other is data class "Extract.
Use case: I remove a leg from a wasp that is pinned and send it for DNA extraction. I give the pinned wasp a catalog number and the extract an identifier of some sort. I then want to link the leg extract to the pinned specimen via catalog number and extract identifiers and specify the origin relationship that one is extracted from another. I then get sequences back, and want to link those sequences to both the extract (leg), and the original collection object (pinned specimen).

I imagine @kandakoj would be interested in this too?

There is an interface for existing sequences/extracts:

From a "quick" perspective I think this is good enough.  I think the parallel example is that we need Quick new sequence for Extract.  In both cases I'm not completely convinced that "Quick" is the way to manage sets of sequences and extracts.  I"m going to close for now in lieu of a Task - New sequence issue.


Endpoint-admins cannot manage resources restricted to other users

Bug description
If a resource is restricted to a non-admin user, e.g. container restricted to helpdesk user eve then endpoint-admins cannot manage it.
Expected behavior
Endpoint admins should be able to manage restricted resources.
Steps to reproduce the issue:

Restrict a resource to a non-admin user e.g. container to helpdesk user eve
Login as an endpoint-admin and try to delete this container
Recieve error Failure access denied to resource

Technical details:

Portainer version: 1.22.2
Docker version (managed by Portainer): 19.03
Platform (windows/linux): Linux (proxmox)
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer): portainer agent stack
Browser: firefox/chrome

Additional info:
Does not affect volume browse

Related to https://github.com/portainer/portainer/pull/3337


feat: add docs for mounting to element directly

Description

Adds docs for mounting an element using a DOM element instead of a selector.

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.51.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add handling for undefined group names in legacy docs

Because they predate the use of sassdoc for documentation, older versions of
neat (~>'1.8') do not have groups defined for the various features. This
caused the group names to read as "Undefined". This change checks to see if
they are undefined and if it is, the group heading will be omitted.
https://github.com/thoughtbot/neat/issues/564
Old

New

One comment, but otherwise LGTM!


NTR: microglial cell of the cerebral cortex

see
https://www.ncbi.nlm.nih.gov/geo/query/acc.cgi?acc=GSM1658188
def: "Microglial cell that is part of the cerebral cortex." []

see
https://gitlab.isb-sib.ch/Bgee/expression-annotations/issues/20#note_5982
and use of 'Annotation Extension'


Add variable importance measures

Look at the different variable importance measures supported by XGBoost and add those calculations to forust.

This paper could be useful for implementing an unbiased feature importance measure
https://arxiv.org/pdf/2305.10696.pdf

Basic feature importance measures have now been implanted (gain, cover, and weight). Closing this, and will open a separate ticket to keep track of the unbiased gain.


Renewal time ignored on static image

Checklist:

[x] I updated to the latest version available
[x] I cleared the cache of my browser

Release with the issue: 5.0.0-rc1
Last working release (if known): 4.0.x
Browser and Operating System: HassOS 2023.4 and Latest Chrome on whatever (multiple OSes)

Description of problem:
Camera view 'continually' refreshes (i.e. live view instead of static view). This happened after the upgrade to 5 RC1 and clicking through lovelace to upgrade them all. The 'defaulted' to 10 seconds on update_seconds but even now on 0 (as shown below) it does not yield (it streams regardless of setting).

Javascript errors shown in the web inspector (if applicable):

Additional information:
type: custom:frigate-card
cameras:
  - camera_entity: camera.somecamera
    view:
      default: snapshot
      timeout_seconds: 30
image:
  mode: camera
view:
  default: image
  update_seconds: 0
menu:
  buttons:
    frigate:
      enabled: true
    live:
      enabled: true
    clips:
      enabled: true
    snapshots:
      enabled: true
    image:
      enabled: true
    download:
      enabled: false
    fullscreen:
      enabled: true
    camera_ui:
      enabled: false
  style: hidden
  position: right
live:
  controls:
    next_previous:
      style: none
    thumbnails:
      mode: none
  draggable: true
  lazy_load: true
  lazy_unload: never
dimensions:
  aspect_ratio_mode: static
media_viewer:
  controls:
    next_previous:
      style: none

@CoMPaTech I see some things invalid (but harmless) in that config. Can you clarify what you want to happen?

@CoMPaTech I see some things invalid (but harmless) in that config. Can you clarify what you want to happen?

It's an older 'always upgraded' version of the card. What I expected to happen/used to work up to the upgrade:

Card showing latest 'static' image (live, not latest snapshot)
Maybe updating every 10 seconds, but rather just static and only refresh on page refresh.
All the other cruft is just to map less/more icons into the view

Maybe I should start over fresh, but I might not be the only one running into the upgrade issue?

Maybe I should start over fresh, but I might not be the only one running into the upgrade issue?

You need this:
image:
  mode: camera
  refresh_seconds: 0

The update_seconds controls how often the entire card re-renders, which you do not want. The image view will automatically refresh its image every 1 second (in high performance mode), or 10 seconds (in low performance mode). The above config will force it to never refresh (verified working on v5.0.0-beta.1).

I think most of the defaults came through toggling switches back and forth in the past. The update_seconds I already looked at and it doesn't change it for me, all camera's keep updating at approx 1fps. I did found how to restore the functionality though:
image:
  mode: camera
  refresh_seconds: 0

Changing the update_seconds doesn't yield - refresh does. So setting it to the below does still images with 10 second refreshes
image:
  mode: camera
  refresh_seconds: 0
view:
  default: image
  update_seconds: 10

So it's working as intended again, feel free to close and/or should we figure something wrt why it change they way it used to work?

Changing the update_seconds doesn't yield - refresh does. So setting it to the below does still images with 10 second refreshes

I don't understand what you mean with "doesn't yield"? If you want the image to refresh every 10 seconds, why does this not work:
image:
  mode: camera
  refresh_seconds: 0
view:
  default: image
  [ NOTHING ELSE HERE]

I think I once set update_seconds to 0 and it did produced a still image, that doesn't work (at least now) and indeed you need the refresh_seconds.


Crash when getting directions to "Монастырь Картезианцев"

Steps:

Search for "Монастырь Картезианцев".
Open map.
Get directions.

Expected:

Directions are set.

Actual:

Crash.

Closed in f335002.


operators hive-operator (1.2.3641-2a02c5d)

Update Hive community operator channel(s) [['alpha']] to 1.2.3641-2a02c5d
/hold

/hold cancel

/merge possible

/merge possible

/merge possible

/merge possible


File.open autocompletes enum from _File

Godot version:

3.2.3 rc4
Issue description:

Wait, this is in 3.2? @ThakeeNathees maybe this related to one of your changes to code completion?

this is reproducible since 3.2.0-stable. possibly https://github.com/godotengine/godot/pull/26990 will fix this.

Oh, so it's older. But the namespace change won't be added to 3.2. This needs a workaround like what happens for other cases.

Still in 3.4 beta. I can see many newcomers get confused when they see this in completion popup then get the error

This is fixed in 4.0 by using a proper namespace for these core singletons. But a workaround for 3.x would still be good.

Not reproducible in v3.6.stable.official [de2f0f147], closing this as completed.


Small adjustments

Summary | Résumé

Signature: add default value for type and variant.
Fix topic menu focus border for menu btn

@ethanWallace the has-link prop for the signature component currently doesn't have a default value, but the Storybook story shows the default value as false. Should we set the default value to false in the component as well?

Yes, let's set it to the default value in the component as well

@ethanWallace added it in!


wrong value while entering 1 with leading zeros

https://user-images.githubusercontent.com/24309704/125603528-0c14e638-4545-4dcf-8633-cf20c76a7c8d.mp4
This is the error I am facing, while entering zeros, it is incorrectly behaving

@iabduul7 can you please share your code?

@iabduul7 can you please share your code?

the same code written in readme file.
 <CurrencyInput       value={value}       onChangeValue={setValue}       prefix="$"       delimiter=","       separator="."       precision={2}       onChangeText={(formattedValue) => {         console.log(formattedValue); // $2,310.46       }}     />

@CaioQuirinoMedeiros Is there any update about this problem?

I'm sorry but so far I couldn't reproduce this behavior... maybe you guys could provide some more info...

try to set  precision={0}


The link for downloadable PDF documentation is not working

I have tried to download the documentation in PDF from the official site (http://flask.pocoo.org/) and the link there (http://flask.pocoo.org/docs/0.12/.latex/Flask.pdf) is returning error 404.

Duplicate of https://github.com/pallets/flask-docs/issues/3, and several others in this repo: https://github.com/pallets/flask/search?q=PDF&type=Issues&utf8=✓


Dump Failed: Err: {DS_ERROR_EXE_NOT_FOUND}

What i do wrong?


added fake handcuffs

couldnt use unequip since it wouldnt sync with the client.

Can you put a message for when the traitor buys this item, saying how to remove the fake handcuffs, since most players buying it might not know the command at first.

Sure !


test: e2e debuggability improvemnts

Follow up for #281
Modifications
This PR includes a few improvements for the current e2e debuggability which are:

outputting pod changes for each resource and sorting all output into separate folders
adding the option to set the env variable DISABLE_WATCH to disable watches for all resources and controllers
refining the output to include fractional seconds and format better overall

Verification
Ran make test-e2e locally to view the new output artifacts.

Oh, sorry I didn't clarify earlier but can we have the disableWatch flag also prevent the capture of the numaplane and numaflow log files?

Oh, sorry I didn't clarify earlier but can we have the disableWatch flag also prevent the capture of the numaplane and numaflow log files?

and maybe instead of calling it disableWatch, we could just call it disableTestArtifacts or something?


Ability to pass custom target to "CMake: Build Target" command (aka cmake.buildWithTarget) as argument

Brief Issue Summary
I would like to configure a task which would depend on building a specific target. It is possible to call a command from a task using "inputs", tasks.json would look something like this:
{
  "version": "2.0.0",
  "tasks": [
    {
      "label": "Build Target",
      "command": "echo ${input:buildTarget}",
      "type": "shell",
      "problemMatcher": []
    }
  ],
  "inputs": [
    {
      "id": "buildTarget",
      "type": "command",
      "command": "cmake.buildWithTarget",
      "args": "targetName"
    }
  ]
}

Unfortunately, cmake.buildWithTarget does not support receiving target as argument, instead it unconditionally prompts user for target selection:
https://github.com/microsoft/vscode-cmake-tools/blob/2878725633841f0e63499bd51b1e26291c104884/src/cmakeProject.ts#L2066-L2067
This is not acceptable in my use case, because there is a specific target I want to build and I don't want to bother workspace users with target selection (which would be non-user-friendly and error-prone).
Could you please make cmake.buildWithTarget accept target as argument?
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

@i-ky Thank you for the request / suggestion! I have placed it on the backlog, but if you want this feature quickly, we would certainly welcome an open-source contribution!
Thanks! 😊

@i-ky i made the #4206 PR. It would be nice if you could test it to confirm it does the job.


[NEW] File list

Closes https://github.com/RocketChat/Rocket.Chat.Android/issues/1205

@filipedelimabrito Looking great! Just one thing: the download button on images doesn't work? I tried here several times and no feedback is provided.

@rafaelks https://github.com/RocketChat/Rocket.Chat.Android/issues/1307


There is no way to paste clipboard content

Gvim has + and * for clipboard content. But pynvim does not have these registers and it means, that there is no way to insert content into pynvim from clipboard.

On X11 platforms xsel (or xclip) is required for clipboard function, do you have any of these? (BTW clipboard is handled in core neovim and is not dependent On pynvim)

Thanks, after sudo apt install xsel these registers started to work, but :registers still don't show +´ and *`. Strange.
Another strange thing is that vim does not have these registers, but gvim has and it works without xsel or xclip.
And if clipboard functionality depends on external executables does not look like neo, it smells like oldish thing.

:registers still don't show +´ and*`. Strange.

It's a known issue, already in the issue tracker.

but gvim has and it works without xsel or xclip.

gvim links to X libraries. Unlike gvim, Neovim does not have a compilation-time dependency on X. Vim (with clipboard support) does. Neovim also supports user-extension of clipboard support.

And if clipboard functionality depends on external executables does not look like neo, it smells like oldish thing.

Vim depends on external executables for:

spawning processes
completing filenames
generating diffs

Neovim is simply externalizing more things than Vim, and simplifying code as a consequence. And its not like Vim doesn't depend on anything. As @justinmk explained, if you don't have the libx11-dev package when compiling, Vim won't have clipboard. From the user perspective, adding clipboard to Neovim is a matter of doing apt-get install xsel vs apt-get install libx11-dev before compiling Vim


Code Changes

Code Changes
Base on audit report

[ ] https://github.com/opendcs/rest_api/issues/79

Work on this issue being performed under USACE W912HQ23F0243 Task 5 has been completed. This issue will be kept open to track parallel development efforts.


Please carefully look up to this here  as setIsActive is not defined yet but called inside code at line 163

Please carefully look up to this here setIsActive is not defined yet at line 163, isActive is only passing by props from Accordion, I know that what I did it's not best solution,I added this lines for proper understanding and updated setIsActive to setCheckActive to remove errors, Please take necessary steps so that learners like myself don't get confused by ReferenceError of variables which isn't declared yet and got curious about why it was like that

Hey there @KishorKumarParoi! 👋
The error in this code is intentional. It's showing the second step of a 3-step refactoring to lift state up.
That said, I agree that this may be confusing. Perhaps it would be a good idea to add a comment next to the confusing code? Something like this:
function Panel({ title, children, isActive }) {
  return (

{title}

      {isActive ? (

{children}

      ) : (
        <button onClick={() => setIsActive(true)}>
        // 👆 This onClick handler doesn't work yet, because we removed setIsActive in step 1
          Show
        </button>
      )}

  );
}

What do you think?

Hey there @KishorKumarParoi! 👋
The error in this code is intentional. It's showing the second step of a 3-step refactoring to lift state up.
That said, I agree that this may be confusing. Perhaps it would be a good idea to add a comment next to the confusing code? Something like this:
function Panel({ title, children, isActive }) {
  return (

{title}

      {isActive ? (

{children}

      ) : (
        <button onClick={() => setIsActive(true)}>
        // 👆 This onClick handler doesn't work yet, because we removed setIsActive in step 1
          Show
        </button>
      )}

  );
}

What do you think?

Yes I am agreed with you

@KishorKumarParoi how about you update the PR with these changes then?

@KishorKumarParoi how about you update the PR with these changes then?

Updated Thank you

Hope this one is going to be merged soon


Code generation for inline queries

Describe the solution you'd like
In order to use codegen-based tools, I need to write queries far from the code. But I wanna write queries to be inlined inside the code AND the type declarations generated are automatically bound.
Tools like graphql-let are useful, but TypeScript's inference feature makes things even smarter.
I know how to do this, but I am not sure where is the right place to.
Describe alternatives you've considered
example:
// $ExpectType UserData
useQuery(`
  query User($id: String!) {
    user(id: $id) {
      id
      username
      fullName
      email
    }
  }
`);

// $ExpectType PostData
useQuery(`
  query Post($id: String!) {
    post(id: $id) {
      id
      category
      title
      description
    }
  }
`)

// genarated code looks like this:
type QueryResult<Query extends string> = (
  Query extends UserQuery ? UserQueryResult :
  Query extends PostQuery ? PostQueryResult :
  any
);

// client's useQuery should be overwriten
function useQuery<Query extends string>(query: Query): QueryResult<Query>;

// extracted from useQuery(`...`)
type UserQuery = `
  query User($id: String!) {
    user(id: $id) {
      id
      username
      fullName
      email
    }
  }
`;

type UserQueryResult = {
  _brand: "User",
};

type PostQuery = `
  query Post($id: String!) {
    post(id: $id) {
      id
      category
      title
      description
    }
  }
`;

type PostQueryResult = {
  _brand: "PostData",
};

This code can be easily generated through codegen plugin, but finally, the client's useQuery type declaration must be overwritten.
Additional context

I'm in the middle of figuring this out, with a WIP branch in the middle of a personal project: https://github.com/primitiveconcept/ludumdare46/compare/codegen?expand=1
The main problem I'm having is that tools other than graphql-code-generator only support this:
gql``

not one of these:
gql<"Type">``
gql(``)

I haven't found a way to get TypeScript to extract the literal value from a tagged template literal; the input has to be TemplateStringsArray, so the extracted generic from it ends up being string. That leaves: make gql a function, or give gql a type param. Neither one of these is really supported by any other tools (in editor or otherwise).

Your more immediate question:

I know how to do this, but I am not sure where is the right place to.

There's a "hooks" config property that lets you run anything after codegen completes. I'm using that here:
https://github.com/primitiveconcept/ludumdare46/compare/codegen?expand=1#diff-d6ac51fc77e5e52e810e1ab39e41c764R6-R10
to run a Babel plugin here:
https://github.com/primitiveconcept/ludumdare46/compare/codegen?expand=1#diff-69dd8ef9ba9d159595a7606869f7b8ecR1-R21
which adds an exported type containing all operations by name.
For your approach, typescript-react-apollo will generate code like this:
export const UserFieldsFragmentDoc = gql`
fragment UserFields on User {
  id
  username
  role
}

You can use Babel traverse to find those and build up your type object, or convert them to what you need.
Otherwise, you're in custom plugin land. Good luck.

Our tool for extracting queries is graphql-tag-pluck (https://github.com/ardatan/graphql-toolkit/tree/master/packages/graphql-tag-pluck). I looks for gql template usages and tries to extract strings from it.
If you wish to write it inline, without gql, you can use the magic GraphQL comment (it also detected by most IDEs), like that:
useQuery(/* GraphQL */`
  query Post($id: String!) {
    post(id: $id) {
      id
      category
      title
      description
    }
  }
`)

This way, graphql-tag-pluck will be able to find it as well.


Bump bundled ruby version to 2.7.6.

Please let me know if this needs any changes! 🙂

Thanks for the contribution Alex! This is now a little out of date, and I'm working on getting us fully up to 3.2 in the near future, so I'm gonna go ahead and close this out for now.


No support for time-generated-field header

According to this article https://docs.microsoft.com/en-us/azure/log-analytics/log-analytics-data-collector-api#create-a-request time-generated-field can be used to specify actual data timestamp.
Would be very useful for my scenario.

created pull request https://github.com/ealsur/HTTPDataCollectorAPI/pull/4

Great addition, thanks a lot!

Nuget package has been updated


Add test for API stability for a CausalPy example in Causal Inference and Discovery in Python by Aleksander Molak

On pages 304-307 there is an example using CausalPy in Causal Inference and Discovery in Python by @AlxndrMlk. We should create a new pytest file which tests the API to ensure that people running this code in the future are still able to replicate the results.
It should probably be done in a new test file, something like test_api_stability.py.
A choice will have to be about reading in the data remotely or placing it internally in this repo.
To be clear, I'm not proposing that we add that example to our docs, at least not at this point :)

Sounds great @arthurmello! Feel free to make a start - I'll assign you to this issue now.

Thanks @drbenvincent! I just did the PR, looking forward to your review.


[HxInputDate] Add DisplayMonth to HxInputDate

Hello,
I have a requirement to add the HxCalendar.DisplayMonth parameter to the HxInputDate.
More than happy to implement and raise a PR.
That is quite a simple change.
Part 2 is for HxInputDateRange, it would be a bit more complex as the whole idea is to optionally change the DisplayMonth of the EndDate when the StartDate is changed while EndDate is still null.  Should it be a flag along the lines of end date is after start date?  Think holiday reservation or event booking.
Any thoughts before I start implementation?
Thanks.

Do you want the displayed month in the dropdown to default to something other than the current month (Today) when no Value is set? If so, we should name the new parameter CalendarDisplayMonth.
For the HxInputDateRange, I expect the user experience to be such that when the EndDate isn't set, it displays the same month as the StartDate. To be honest, I thought our current implementation already did this. It would be a bug if it defaults to the current month (Today).
I'd appreciate a pull request on this (both ;-)).
cc @crdo @jirikanda

The attached PR fixes the bug and also provides the enhancement.


Baking app freezes device from sleep

@greenknot @pscott
v2.2.11
This is perfectly repeatable; happens constantly. I've installed the 2.2.11 tezos baking app and started baking/endorsing on my test network. After a few minutes the screen turns off. Baking/endorsing continues to function.
Press either button of the nano s to wake up the screen. Press buttons to scroll through baking info screens. Within 1-3 screens the entire device locks up.
Prior to the UX rework (<= 2.2.9, this was not an issue). This freezing issue has also been mentioned by several other bakers in the Tezos bakers slack channels.
What's the proper way to debug this issue on the device?

Firmware 1.6.1
MCU 1.12

We've seen this issue and have added this task to our taskboard. Thanks for notifying us!

@pscott More reports from community about device freezes

I've noticed using the baking app 2.2.11, the Ledger only goes into screensaver mode after you BAKE a block, even endorsing does not send the screen to sleep. After that bake, the screen will go dark, but still baking. Just don't touch any buttons, or the Ledger will freeze or be invisible to "list connected ledgers."


Fix install issues

This PR fixes the issues that we were having with installation on latest versions of ember. It ditches the use of the fork for pace.js and use the legit source since it's working fine now.

well done! :+1:


How do I cache only nodes that are members of ways with a certain tag?

I am aware of using the NodeLocationsForWays handler to cache node locations so that I can later access the location information from the NodeRef objects. If I understand correctly, it seems to me that this will cache ALL the node locations, which of course results in rather (unnecessarily) large files. My question is, is there a way to only cache node locations, for example, that are part of a way that has Tag:natural=coastline instead of all the nodes in the .pbf file?

Sure you can do that. You have to read the file at least twice then, check all ways for the tag you are interested in, remember the node ids you need, then re-read the file, this time only calling the NodeLocationsForWays handler for the nodes you have remembered. It depends very much on your exact use case whether that makes sense or whether it is faster to just remember all nodes. (But that, as you mention, will need more memory.)
It might be easier in your case to just use osmium-tool to filter out all coastlines (osmium tags-filter input.osm.pbf w/natural=coastline -o out.osm.pbf) and then work with that filtered file, no need to write something special for that.


Adding canvas removes cursor:pointer

Or rather, adding 'image-cropper' to the canvas element, not happening if i add it to a div.
Also, the entire page is one big accordion, if that could contribute to the issue.
Any ideas?

I have fixed #6 Does this resolve this issue for you as well?

yes but the issue with el.style.cursor = 'nwse-resize';  gets called when im hovering over a random p-element nowhere near the canvas. Actually all the resizes

i have 2 canvases, one works, the other one dont have any cursor-changes when cropping, instead the cursor-changes happen on another part of the page on a fixed element.

Ah yes I see, I have created a test with two image croppers with two canvases and can see only one of them has the cursor-changes. I will investigate this.

Very cool, thanks =)

I have created a new service that injects into the directive allowing intelligent decision making when displaying the cursors on a page with multiple directives. I tested it with two canvas images and it seemed to work well. Let me know how you go with it and if it is good will mark it as a release.

That helped with the changes to the cursor on the second canvas, but i still get cursor changes on the element that is next to the canvas with properties float:left and position:fixed.

Updated the code to set the cursor style at the root html element. This should fix your issue.

Perfect! Thanks

Although, now the mouse wont change when hovering the crop area

Hmm I am having trouble replicating this issue. So far with the previous problems I have created two canvases (with directive attached) and have set the styles so that they floated right and another div was floating left with a fixed position. However this setup is working fine for me with hovering functioning correctly. Is there anything else on your page that would be setup differently to my test case?

Sorry, my bad. cursor: auto rule in foundation.css that borked it. Thanks!

If you have cursor:auto or default on the body for example, it seems to break it.

The issue with the random cursor-changes is back. However, they disappear after i chose a picture for the canvas. Only using one canvas. Been adding alot of css-changes which may have caused it, im am unsure to which changes made the behavior return.

Are you setting cursor in your css anywhere? Also, while investigating this I found a bug where the cursor logic was happening before an image had loaded. I have fixed this and maybe this will help?

Thanks again! That did the trick it seems :D


Remove NUnit Software from Copyright; add MIT license.

Fixes #136
Only @CharliePoole or @OsirisTerje should merge
We are still discussing this

What we have generally done in the past is update the date in the source when that file has a significant change. I'll do another commit applying your comments and now adding your name as discussed.

OK, I think this is now ready.

Thanks @jnm2! Lots of bugs in my global replacement software. 😈

@OsirisTerje I made a judgement to just leave a single year in each file (last year a major change was made) and to do the full range of years for each of us in the top-level copyright statements.
@nunit/core-team This PR of mine can be considered as my assigning shared copyright over the nunit-vs-adapter software to Terje, recognizing the scope of his contributions. If it should be necessary at some point, I'll do something more formal.

@OsirisTerje Does this look OK to you now?


Update CFStream doc with CFRunLoop poller info

Update CFStream doc with CFRunLoop poller related information.
Also updated the objc contact to @stanley-cheung

#22019


Find a way to "hot reload" / make it so you don't have to restart the app with ./scripts/code.sh to see your changes

Context
Find a way to "hot reload" / make it so you don't have to restart the app with ./scripts/code.sh to see your changes
This is NEEDED
Solution

TLDR: https://blog.hediet.de/post/hot_reload_for_vs_code_extension_development

setup project by reading Contributing docs
run yarn watch in a terminal - wait for it to compile everything (around 2 mins), it then shows "finished compiling", keep it running.
open another terminal to run the app ./scripts/code.sh
now every time you make a change, yarn watch will pick up those changes and will only compile that, almost instantly. (save your changes manually)
reload the windows to see the changes. (Command Palette -> reload window)

changes to both code editor itself and extension (submodule) are compiled or hot reloads.
you can use yarn watch-client if you are only making change to the code editor.

Disable 'Auto Save' feature, save your changes manually. (auto save triggers yarn watch on every change, causing it to hang or crash).

Thank u himanshu! Added to the docs


Compiler overflow evaluating the requirement <&_ as std::ops::Mul<&_>>::Output

It's possible to cause a compiler overflow by trying to multiply a reference without a concrete type. The minimal example below is duplicated both on 0.10.1 from crates.io and master from the repository.
extern crate nalgebra;
use nalgebra::Vector3;

pub fn thing() {
    let f: f32 = 1.0;
    let x1: Vector3<f32> = &Vector3::<f32>::z() * f; // Fine

    // overflow evaluating the requirement `<&_ as std::ops::Mul<&_>>::Output`
    let x2: Vector3<f32> = &Vector3::z() * f;
}

The full error message can be found here: https://gist.github.com/djmcgill/d4afe13a3c5b4145897be0916496f73e

I get an even more mysterious but similar explosion in this program:
extern crate nalgebra as na;

fn main() {
    let height: f64 = 40.0;
    // overflow evaluating the requirement `<&_ as std::ops::Add<&_>>::Output`
    for y in -(height / 2)..(height / 2) {
        println!("aiee: {}", y);
    }
}

Output here: https://gist.github.com/icefoxen/3812c3ba0dfe7a3dad4e3196c4ca1d66
Thank you!  :heart:


This alleviates the requirement that "." be on the include path.

I was seeing APC errors without this change and we run PHP w/o "." on the include path.

Not sure if this helps...

@EvanDotPro I guess set_include_path('') would reproduce this, right @SocalNick?
Anyway, I think altering the include path is not up to the framework nor the default application skeleton...
@SocalNick PHP defaults are usually fine, and I think you should eventually fix your environment manually in your exceptional case (set_include_path)

I think the point is that he wants to be able to have a clean (read: empty) include path (@SocalNick correct me if I'm wrong). I just simply wasn't able to reproduce this in my environment.

@EvanDotPro fine with empty include path, not fine (for me) if the include path has everything but ..

Sorry I didn't close this. I reverted this change at some point and don't seem to be getting the errors anymore. Our ops team uses Chef to deploy these boxes, so something could have changed, but I can't be certain.


Support to synchronize dirty changes across windows

VSCode Version: Code 1.10.1 (653f8733dd5a5c43d66d7168b4701f94d72b62e5, 2017-03-02T00:27:09.350Z)
OS Version: Darwin x64 16.4.0
Extensions:

Extension
Author
Version

beautify
HookyQR
0.8.8

python
donjayamanne
0.5.8

rest-client
humao
0.12.3

vscode-icons
robertohuertasm
7.3.0

Steps to Reproduce:

open a window A and open a file F in that window
open a new window B(Shift-command-N), drag that file F to window B
now file F on both window A and B, change content of F on window A without save, the unsaved content doesn't display on window B.

cc @Tyriar

We do not support this today, so marking as feature request.

Something we'd have to deal with for this is what happens when a hot exit restore happens and it contains different content to the current "workspace copy":

Set "files.hotExit": "onExitAndWindowClose"
Open window with folder a
Edit file foo
Close window with folder a
Open window with folder b
Edit file foo
Open window with folder a, at this point folder a's backups would be restored (changes in step 3) which are different to the current changes (in step 6).

https://github.com/Microsoft/vscode/issues/11866


Fix use of wrong UNICHARSET

Signed-off-by: Stefan Weil<EMAIL_ADDRESS>

This should fix issue #1205. I found it by adding an assertion more up in the call stack. There might be more similar code using the wrong UNICHARSET, so I'll try to add more assertions to find it.


bug: \t-\tp broken

Hi! I came across this bug on BabelMark, as I'm writing a markdown CST for C#.
this lib writes an <ul> when having the input text \t-\tp (\t being a tab and p being paragraph content.
Version: 0.8

https://babelmark.github.io/?text=	-	It+offers

Yup, that looks like a bug. Thanks for reporting it!


Dependency within NuGet is wrong for protobuf

protobuf-net (≥ <IP_ADDRESS>8 && < 3.0.0)
However current version of protobuf is <IP_ADDRESS>8
I'm guessing this is a minor typo.  :)

deployed

Lightning fast!   Well done.

FYI i have some proposed change in the licensing model for NuGet packages under the NServiceBusExtensions Org https://discuss.particular.net/t/proposed-change-in-the-licensing-model-for-nuget-packages-under-the-nservicebusextensions-org


setup kanban board

Project link: https://github.com/ciraganenicole/Recipe-app/projects/1
Team:

Ayoub Chahir
Nicole Ciragane
Solomon Hagan

Hi Team,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Figures from preprint

I've uploaded the schematics from the preprint in PNG format for inclusion in the TemplateFlow documentation assets. If you think the source SVGs would be better, I can upload those instead and open a new PR. The SVGs contain some Inkscape-specific metadata that I'll need to remove first, but this is apparently a straightforward process.

Looking forward to the SVGs :)


More modules (metal_machine and metal_firewall)

Still needs unit tests, but I will add this for now anyway.


iframely-container size

Just a feature request:
It would be nice to have an option in the admin panel to set max width of the iframely-container.
Cheers!

Is there a reason having this option in admin panel would be more convenient than altering it directly in the CSS for the class name?


Handling of join button

To give us flexibility to not remove join buttons for each event that has passed, this PR introduces a simple scripts that hides buttons after the events have gone.
This script is currently contained in _includes but should maybe go to assets and be loaded as a script and not inline. Waiting on opinions.
Further I don't use any jQuery here, as it currently does not seem to be used on the website.

Further I don't use any jQuery here, as it currently does not seem to be used on the website.

jQuery is loaded by default due to the minimal mistakes scheme (see https://mmistakes.github.io/minimal-mistakes/docs/javascript/). So I would not worry about using it.

jQuery is loaded by default due to the minimal mistakes scheme (see https://mmistakes.github.io/minimal-mistakes/docs/javascript/). So I would not worry about using it.

I didn't found it from inspecting the loaded js files. It also complaint about wrong syntax as I was just trying with jQuery in the first place ;) However, adapting it to jQuery again shouldn't be an issue, if we like to.

This is ready to be merged, isn't it?

This is ready to be merged, isn't it?

Might be the case :) at least I tried to think about arrays etc.

Ah wait, sorry

no, it's all fine :D Thanks a lot @eileen-kuehn!


Feature: Item Kind Highlight groups

It would be cool if we could specify/overwrite for each Item Kind our own Highlights
So for instance item kind Function is 'Blue' and snippet is 'orange'

How to realize it? If you have an idea, feel free to post it.
(CmpKindFunction, CmpKindInterface ...)?

Correct kind of like this:
CmpKind .. kind_name
on the setup function we could introduce a table:
kind = {
  Function = "#3d3d3d",
  Snippet = "...",
  ...
}

The table is used to compose highlights

thanks!

It would be cool if we could specify/overwrite for each Item Kind our own Highlights So for instance item kind Function is 'Blue' and snippet is 'orange'

I do it by creating custom highlight groups
https://github.com/Dan7h3x/NvimPy


Document how to use bootloader

While the protocol is well-documented, there is no documentation on how to use it.
In the programmer directory it says to run tinyprog -l, however it doesn't say how to use tinyprog.
There is a setup.py, however it doesn't work:
❯ python .\setup.py
Traceback (most recent call last):
File ".\setup.py", line 4, in <module>
    from setuptools import setup, find_packages
ModuleNotFoundError: No module named 'setuptools'

There is a directory called tinyprog, and surprisingly Python seems to recognize this, however it doesn't work either:
❯ python tinyprog
Traceback (most recent call last):
File "D:\obj\Windows-Release\37amd64_Release\msi_python\zip_amd64\runpy.py", line 193, in _run_module_as_main
  File "D:\obj\Windows-Release\37amd64_Release\msi_python\zip_amd64\runpy.py", line 85, in _run_code
  File "tinyprog\__main__.py", line 10, in <module>
    from six.moves.urllib.request import urlopen
ModuleNotFoundError: No module named 'six'

It's telling you you don't have setuptools installed. setuptools is required to run setup.py for many, if not the majority, of setup.py scripts out there.
This is all completely standard fare for installing Python packages, it shouldn't be particularly surprising or hard to figure out.


feat: filter discovery to selected streams

Feature scope
Taps (catalog, state, stream maps, etc.)
Description
It is commonly advantageous to filter stream discovery to only selected streams. This can be for both performance and cost (e.g. Snowflake metadata credits) reasons. Full context is captured here.
In several popular taps, this is solved by providing database, schema and table settings, accepting comma-separated names to be used for filtering. Whilst this is adequate, it isn't ideal as it is both verbose (when selecting many itemised tables) and error-prone (when managing long lists of itemised tables, and also translating from table name to stream name).
An improved solution is proposed in that same issue:

My preference would be to use the existing selection syntax and mechanisms to source the table filter list, with a flag to enable/disable the behaviour (i.e. discover all). I think 'limit discovery to selected streams' is a helpful default, but it should also be overridable. This would effectively rely on the catalog to supply the filter, rather than config.json, but means we aren't duplicating/polluting config.json with stream-level config. We should also interpret no provided catalog as a case to discover all.

@kgpayne interesting idea! How were you thinking we'd pass the selection criteria to the tap during discovery? Since usually catalogs are the product of discovery so it wouldnt exist prior to that.
This could be specific to my implementation of environments but if we did use the existing selection syntax I think I'd still need to set the selections multiple times (i.e. once per environment) so I can inject my alternative databases and schemas, right? Ideally for most use cases I'd be able to set the selection criteria once in the top level config then override the database/schema somehow in each environment config. That would help me avoid selections getting out of sync across my environments.
Am I understanding correctly? Do you have an idea of how I could get around that?

@kgpayne, @pnadolny13 - What if we built something like a native exclusion_filter into the SDK, that would be somewhat compatible with and analogous to --exclude in Meltano?
Does this meet the requirement?

@aaronsteers I really like this proposal 🙌 Will add comments there 👍

I wrote this up as another option here:

https://github.com/meltano/sdk/issues/1240

As I tried to imagine doing this during discovery via traditional selection logic, I kept running into the fact that you have to first run discovery in order to deselect anything from it.

@pnadolny13 I would expect this feature would solve the 'selection criteria per env' case 🙂 e.g.
environments:
  - name: prod
    env:
     SOURCE_DB_ID: db1_live
  - name: uat
    env:
     SOURCE_DB_ID: db1_uat
plugins:
  extractors:
    - name: tap-mysql-db1
      inherit_from: tap-mysql
      select:
        - ${SOURCE_DB_ID}-table1.*
        - ${SOURCE_DB_ID}-table2.*
        - ${SOURCE_DB_ID}-table3.*
      metadata:
        "*":
          replication-method: LOG_BASED

Still relevant

Very much relevant still, we are facing this https://github.com/MeltanoLabs/tap-postgres/issues/215

Seems to be a good way to handle this that would allow us to adhere to the singer spec is to have meltano pass the select, metadata (etc) fields as configuration options to the tap via config.json . The tap would use these to limit discovery properly it'd take a bit of work on the sdk side to mimic meltano but this is probably easiest on meltano users while still allowing the tap to run without meltano

@visch that is my strong preference 😅 I would love for the SDK to implement Meltano-style selection, including the supporting of wildcards etc.
There is some crossover with #1350 too - selection isn't the only stream-wise config that folks want to apply.

Contributor

@visch that is my strong preference 😅 I would love for the SDK to implement Meltano-style selection, including the supporting of wildcards etc.
There is some crossover with #1350 too - selection isn't the only stream-wise config that folks want to apply.

@kgpayne  I read through #1350 and unless I'm missing something I think my idea here may be easier than adding a whole new configuration option. From the tap side it'd look like a configuration option as you mention, maybe the streams key makes sense maybe not (you'll see why it may matter in a sec). From Meltano's side you'd just define your normal select criteria (stole @pnadolny13  's example)
plugins:
  extractors:
  - name: tap-gitlab
    variant: meltanolabs
    pip_url: git+https://github.com/meltanolabs/tap-gitlab.git
    select:
      projects.*:
      merge_requests.*:
      issues.*:
      '!issues.description':
      '!issues.title':
      '!merge_requests.description':
      '!merge_requests.title':
environments:
- name: userdev
  config:
    plugins:
      extractors:
      - name: tap-gitlab
        config:
          groups: meltano
          start_date: '2020-01-01T00:00:00Z'

The difference is that the config that meltano would pass to Meltano would look different
before changes config.json passed to tap
{
"groups": "meltano",
"start_date":  '2020-01-01T00:00:00Z'"
}

after changes config.json passed to tap
{
"groups": "meltano",
"start_date":  '2020-01-01T00:00:00Z'",
"_select":  [
"sourcedbid12345-table1.*",
"sourcedbid12345-table2.*",
"sourcedbid12345-table3.*",
]
}
```

The beauty of this is that:
1. There's no duplication for Meltano users, stuff just "works".
2. Taps still follow the singer spec

Downsides
1. There's a spec that `meltano` owns that's now duplicated in the `sdk` versioning this isn't trivial as you'd have to maintain two versions, but this seems better than the other alternative

@visch re: downside 1. - I'd be happy for the SDK implementation of selection to take precedence over the Meltano. I.e. we advertise "selection" as a capability for SDK-based Taps and 'turn off' the Meltano selection engine in favour of pass-through. For legacy/non-SDK Taps (that don't advertise a "selection" capability) Meltano would step in as it currently does 🙂

just chiming in here - I would also find this feature very useful as I have been trying to come up with a custom workaround for this on a specific tap!


Add LossDynamicsAnalyzer for noisy label detection

Summary

Ticket no. 103075
Added LossDynamicsAnalyzer for noisy label detection.
Added Jupyter notebook example for this feature.
Added ImagenetWithSubsetDirsExporter to support this feature.

How to test
I added some tests for this change.
Checklist

[x] I have added unit tests to cover my changes.
[ ] I have added integration tests to cover my changes.
[x] I have added the description of my changes into CHANGELOG.
[ ] I have updated the documentation accordingly

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below).

# Copyright (C) 2023 Intel Corporation
#
# SPDX-License-Identifier: MIT

Codecov Report
Patch coverage: 90.35% and project coverage change: +0.02 :tada:

Comparison is base (bddf3cb) 78.73% compared to head (8f5b996) 78.76%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #928      +/-   ##
===========================================
+ Coverage    78.73%   78.76%   +0.02%
===========================================
  Files          215      218       +3
  Lines        25478    25575      +97
  Branches      5085     5100      +15
===========================================
+ Hits         20061    20145      +84
- Misses        4255     4263       +8
- Partials      1162     1167       +5

Flag
Coverage Δ

macos-11_Python-3.8
?

ubuntu-20.04_Python-3.8
78.76% <90.35%> (+0.04%)
:arrow_up:

windows-2019_Python-3.8
?

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

Impacted Files
Coverage Δ

...ms/noisy_label_detection/loss_dynamics_analyzer.py
88.50% <88.50%> (ø)

datumaro/plugins/data_formats/imagenet.py
91.76% <92.30%> (+0.51%)
:arrow_up:

datumaro/__init__.py
100.00% <100.00%> (ø)

datumaro/components/algorithms/__init__.py
100.00% <100.00%> (ø)

...nents/algorithms/noisy_label_detection/__init__.py
100.00% <100.00%> (ø)

datumaro/components/algorithms/rise.py
88.73% <100.00%> (+0.07%)
:arrow_up:

datumaro/components/format_detection.py
93.58% <100.00%> (ø)

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Make nvcc code generation target configure by env var

Like ChainerX: https://github.com/chainer/chainer/blob/c8e47f7e8463c84cedb6ac12ccbad582d6321ce5/chainerx_cc/CMakeLists.txt#L168-L170

Jenkins, test this please.

Jenkins CI test (for commit 4d3b05c4f010d62f9daf92a049fa2e8a789f6cc2, target branch master) failed with status FAILURE.

The failure seems to be a precision problem. Going to merge.


Consider slimming down the make stuff

Why does this project require make to build?

Most of the build/installation instructions are from before Cargo existed.  You can use Cargo to build the project (although make is still required due to the build process for stdbuf).  However, installing requires make because (as far as I know) Cargo does not support the creation of symlinks to another binary or the installation of man pages (which will eventually be written).

(although make is still required due to the build process for stdbuf).

There is nothing in that crate's makefile that couldn't be done directly in build.rs AFAIK.
Man pages does need something more than cargo, but the README should make it clear one can just invoke cargo to build.

Yeah, the README instructions do need to be updated.
I'm not really satisfied with stdbuf's build process, so I'll see if there's a better way using build.rs later.

I've updated the README in #1105.

I'm not really satisfied with stdbuf's build process, so I'll see if there's a better way using build.rs later.

Why is libstdbuf needed in the first place, can't we just setup the IO stuff from stdbuf's main function?

setvbuf() doesn't affect programs started using exec() functions, so we need to use the constructor attribute and LD_PRELOAD to actually modify the buffering in the executed program.

make will no longer be required to build stdbuf once #1106 lands.


Update to v0.4.0

Issue #, if available:
Description of changes:

Update version to v0.4.0
Will merge once ready to release.

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

Job PR-1589-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1589/1/index.html

Job PR-1589-3 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1589/3/index.html

Missing updates in https://github.com/awslabs/autogluon/blob/master/tabular/src/autogluon/tabular/predictor/predictor.py#L2812

Missing updates in https://github.com/awslabs/autogluon/blob/master/tabular/src/autogluon/tabular/predictor/predictor.py#L2812

No, this is correct. Version caching wasn't implemented until v0.3.1, that is what the warning is referring to.

Missing updates in https://github.com/awslabs/autogluon/blob/master/tabular/src/autogluon/tabular/predictor/predictor.py#L2812

No, this is correct. Version caching wasn't implemented until v0.3.1, that is what the warning is referring to.


[do not merge] POC for generate-terraform

See the README to review the POC
Feel free to comment on this PR if you have questions/feedback.

@juandiegopalomino is the issue that if we export more than one service, then we would have the opta-k8s-service-helm duplicated in each?
The same argument can be made about TF module, if a user export more than one services, the aws_k8s_service/tf_module would be duplicated in each output directory.
Here is how the POC currently works
opta generate-terraform aws-env.yaml
   # output
   ./aws-env # contain aws env terraform json file, modules

opta generate-terraform my-service1.yaml
   # output
   ./my-service1 # contain service1 terraform file, modules, and opta-k8s-service-helm

opta generate-terraform my-service2.yaml
   # output
   ./my-service1 # contain service2 terraform file, modules, and opta-k8s-service-helm

Here is what we can do to prevent duplication:
opta generate-terraform aws-env.yaml
   # output
   ./aws-env # contain aws env terraform json file
   ./modules # contain opta tf modules

opta generate-terraform my-service1.yaml
   # output
   ./my-service1 # contain service1 terraform
   ./modules # contain opta tf modules
   ./helm # contain helm templates

opta generate-terraform my-service2.yaml
   # output
   ./my-service2 # contain service2 terraform
   ./modules # contain opta tf modules
   ./helm # contain helm templates

if we wanted to be fancy we could also do something like
opta generate-terraform aws-env.yaml my-service1.yaml my-service2.yaml
   # output
   ./aws-env # contain aws env terraform json file
   ./my-service1 # contain service1 terraform
   ./my-service2 # contain service2 terraform
   ./modules # contain opta tf modules
   ./helm # contain helm templates

Let me know what you think?


fim-config-set home equivalent in new refactored flatimage?

Before, this command existed:
./arch.flatimage fim-config-set home '"$FIM_FILE_BINARY".home
This is really neat for making flatimage even more portable & not cluttering host's ${HOME} folder.
Is this just undocumented at the moment or the feature is not yet available in refactored flatimage?
Love the overall refactor btw, good job!

Hello @fiftydinar , thanks! That is accomplished with fim-env, from the steam example:
  # Optionally remove access to real HOME (it is disabled by default)
  "$IMAGE" fim-perms del home
  # Configure custom HOME
  "$IMAGE" fim-env add 'USER=steam' \
    'HOME=/home/steam' \
    'XDG_CONFIG_HOME=/home/steam/.config' \
    'XDG_DATA_HOME=/home/steam/.local/share'

@ruanformigoni I see how it's done now in Steam example, thanks!