plumber breaks with node-sass 4.9.4 on macOS Mojave

Hello jamonserrano!
I've been using plumber for a while now using both grunt (node-sass, older version) and gulp (gulp-sass). In my current project I'm using strictly npm scripts to build my front-end layer and I'm having trouble with plumber causing node-sass to crash when compiling.
The first to issues was regarding variables. I had to move line 97 and 98 to be above line 96 eding up with this order
$line-height: $line-height * $grid-height;
$baseline-from-bottom: ($line-height - $font-size) / 2 + ($font-size * $baseline);
$corrected-baseline: round($baseline-from-bottom);
$baseline-difference: $corrected-baseline - $baseline-from-bottom;
`
But I also get a more serious error on line 115 and 117 saying

"message": "0.943remrem isn't a valid CSS value.",
"formatted": "Error: 0.23575remrem isn't a valid CSS value.\n        on line 106 of src/vendor/baseline/plumber.scss\n        from line 13 of src/app.scss\n>> \t\t$padding-top: - $baseline-difference * $grid-height;\n\n   ----------------^\n"

That's referring to these two lines of code
$padding-top: (1 - $baseline-difference) * $grid-height;
$padding-bottom: $baseline-difference * $grid-height;
It seems like $grid-height is only getting the unit "rem" but without a value.
Have you seen this error before? Not sure how to proceed to fix it.
Thanks in advance!

Hi @rhedman, thanks for the feedback! Can you share your plumber settings, the relevant parts in your code and the first error that made you reorder the rows? The second error is likely caused by the row reorder as now both $line-height and $baseline-difference are units (e.g. 2rem) and their product will have remrem or rem*rem unit.
I've rebuilt the plumber documentation page with node-sass 4.9.4 and didn't see any errors so it's possible that you have a use case that is not covered yet.

Thanks for your reply
Here's my plumber defaults
@include plumber-set-defaults(
$grid-height: rem($base-unit),
$baseline: $body-baseline,
$font-size: 2,
$line-height: 3,
$leading-top: 0,
$leading-bottom: 4
)
Which these variables:
$body-baseline: .198;
// Size base
$size-base: 20px !default;
// Base units
$base-unit: $size-base / 2 !default;
And this function
@function rem($size) {
@return ($size / $size-base) * 1rem;
}
They are included in the project like this
// Vendor
@import './vendor/baseline/plumber';
// Settings
@import './base/functions';
@import './base/variables';
This is the same setup I've used in all my other projects (I have also tried to change $grid-height in the plumber defaults to have a static value of 1rem without any luck)
And finally this is the error message I get (after changing everything back in my plumber sass.
""message": "Undefined variable: "$corrected-baseline".",
"formatted": "Error: Undefined variable: "$corrected-baseline".\n        on line 96 of src/vendor/baseline/_plumber.scss, in mixin plumber\n from line 4 of src/components/typography/_typography.scss\n from line 16 of src/app.scss\n>> \t$baseline-difference: $corrected-baseline - $baseline-from-bottom;"

OK, I think I just figured this one out after hours of scratching my head. It's my SASS-lint config (with autofix) that keeps breaking plumber. It changes the order som som code blocks thus causing them to end up in the wrong order.
Thanks for your time (and sorry for wasting it) and thanks for Plumber. It's a great tool I use daily :)

I tried that and my linter accepts that fine but the auto-fixer doesn't and changes the file anyway. It also ignores my config where I specify that all filer in the vendor-folder should be ignored. I will stop using the auto-fixer for now.


Add more demonstration gifs on plugin page (generating serial ver uid, swagger annotations)

Just Alt + Enter on the field or mapping function name @MairwunNx


Add documentation for test cases with images

Would like to add README files for each sub-directory in computational_geometry/resources/test so that it's clear where those polygons come from, and provide a listing of all those provided, including images of each so it's easy to reference.

Depends on #17 and #20 so that the images look sensible.


Added links for Create Account, Log In, and Search Results pages. Cre…

Added links for Create Account, Log In, and Search Results pages. Created basic search results table

Everything is looking good!!


Update param description for 'workspaceID'

Description
The description of 'workspaceId' is rather non-descriptive today. It says its the ID to a LAW. That's a no-brainer, but for what...
... that's the important info on this parameter.

Duplicate issue

#752


Hardcoded default marker set label name

Currently is not possible to change default marker set label "Markers".

True, it's not currently possible to change it in the main config.
You can, however, edit the existing /plugins/dynmap/markers.yml file and change the label there.


Fix hanging IteratorExecutor. HiveDataset will skip bad tables instea…

…d of failing the entire job.
This PR fixes two issues:

IteratorExecutor does not shutdown executor on a failure, making job hang.
HiveDatasetFinder fails if a single table is bad. Fix will skip and report bad tables.

@ydai1124 can you review? Thanks.

Coverage increased (+0.2%) to 46.328% when pulling bac7ac9a6519361dd09132f654e05353468145c5 on ibuenros:distcp-hive-badtablefix into 9e8f96ce3732eaae22f94ee83c4e33c4fc2760d2 on linkedin:master.

@ydai1124 addressed comments.

LGTM

Coverage increased (+0.08%) to 46.196% when pulling dec4a9e3a54a8658701fac9a6a10f4df52cd8583 on ibuenros:distcp-hive-badtablefix into 9e8f96ce3732eaae22f94ee83c4e33c4fc2760d2 on linkedin:master.


Support assigning K8s Name to custom resources

Is your feature request related to a problem?
In the UI I'm seeing K8s Name: Unresolved for an Alert relating to an ArgoCD Application CRD that I defined.
On Slack the following information was shared on how K8s Name is defined:
    Deployment: deployment, namespace
    DaemonSet: daemonset, namespace
    StatefulSet: statefulset, namespace
    Job: job_name, namespace
    Pod: pod, namespace
    HorizontalPodAutoscaler: horizontalpodautoscaler, namespace
    Node: node

Since my alert doesn't have any of these labels, the K8s Name is unresolved.
Describe the solution you'd like
That information should probably be added to the docs.
Furthermore, I would like a solution where the user can manually assign the alert a robusta K8s name, perhaps through a special alert label.

Fixed with https://github.com/robusta-dev/robusta/pull/1170
@djfinnoy you can now customize the labels that are used for mapping. Does that help with the manual assignment in your case?


GraphQL - Phase 2

NOTE TO REVIEWERS: Feel free to ignore generated files -- anything named generated.go or ending in _gen.go.
What's new?
Schema updates

Queries and mutations can be tagged with the @authRequired directive, which will reject unauthenticated requests. This is similar to the AuthRequired middleware we apply to authenticated REST routes.

Replaced ErrSessionExpired and ErrNoCookie with ErrNotAuthorized, which has an errorType field denoting what kind of auth failure was encountered. This should make it easy to add auth failures to any return type union just by adding ErrNotAuthorized.

Adopted a convention for return type unions: XyzOrError. It sounds kind of dumb, but I've found it's a useful way to describe operations that will return a union with a return type or an error. E.g. the Viewer query now returns ViewerOrError, which is a union of Viewer and all of the different error types the query might return.

Auth updates

Cookies have a 10-year MaxAge now so they won't expire before their JWT does

GraphQL routes marked with @authRequired will return appropriate auth errors based on cookie/JWT status

Some minor refactoring of the way we store auth info in the request context -- auth context keys (user_id, authenticated) are namespaced to avoid accidental key name collisions (now auth.user_id and auth.authenticated), and there are helper methods to set those fields instead of letting other packages set them directly.

Resolvers

Viewer and UserByUsername root queries are available. Lots of other fields are available or partially available, but there's a lot here that's still in-progress. If a field comes back null or returns an unhandled error, it probably isn't implemented yet.

GraphQL database lookups are cached via dataloaders, which makes it much easier to write modular/reusable resolvers. Dataloaders can also be used to batch database queries, but I haven't set that up yet.

Other Notes

Schema types like ImageNFT and VideoNFT are difficult to implement at the moment because we're not storing media type info on the backend. We will be storing this information when we switch from Opensea to our own indexer, but we may have to wait until then to fully implement this approach in GraphQL.

can you confirm it's okay for everyone to see a given user's wallets?
yup! totally cool


Omitting room from Slack JCasC configuration leads to "java.lang.IllegalArgumentException: Project Channel or Slack User ID must be specified."

Jenkins and plugins versions report

Environment

Jenkins: 2.319.3
OS: Linux - 4.18.0-372.26.1.el8_6.x86_64
---
ace-editor:1.1
ant:1.11
antisamy-markup-formatter:2.7
apache-httpcomponents-client-4-api:4.5.13-1.0
authentication-tokens:1.4
basic-branch-build-strategies:1.3.2
blueocean:1.24.8
blueocean-autofavorite:1.2.4
blueocean-bitbucket-pipeline:1.24.8
blueocean-commons:1.24.8
blueocean-config:1.24.8
blueocean-core-js:1.24.8
blueocean-dashboard:1.24.8
blueocean-display-url:2.4.1
blueocean-events:1.24.8
blueocean-git-pipeline:1.24.8
blueocean-github-pipeline:1.24.8
blueocean-i18n:1.24.8
blueocean-jwt:1.24.8
blueocean-personalization:1.24.8
blueocean-pipeline-api-impl:1.24.8
blueocean-pipeline-editor:1.24.8
blueocean-pipeline-scm-api:1.24.8
blueocean-rest:1.24.8
blueocean-rest-impl:1.24.8
blueocean-web:1.24.8
bootstrap4-api:4.6.0-3
bouncycastle-api:2.25
branch-api:2.1044.v2c007e51b_87f
caffeine-api:2.9.3-65.v6a_47d0f4d1fe
checks-api:1.7.2
cloudbees-bitbucket-branch-source:2.4.4
cloudbees-folder:6.714.v79e858ef76a_2
command-launcher:1.2
conditional-buildstep:1.4.1
config-file-provider:3.8.1
configuration-as-code:1.55.1
configuration-as-code-groovy:1.1
credentials:<IP_ADDRESS>
credentials-binding:1.27.1
display-url-api:2.3.5
docker-commons:1.18
durable-task:493.v195aefbb0ff2
echarts-api:5.0.2-1
favorite:2.4.1
font-awesome-api:5.15.2-2
generic-webhook-trigger:1.84.2
git:4.11.3
git-client:3.11.0
git-server:1.10
github:1.34.1
github-api:1.303-400.v35c2d8258028
github-branch-source:2.11.4
github-oauth:0.39
google-oauth-plugin:1.0.6
groovy:2.4
handy-uri-templates-2-api:2.1.6-1.0
htmlpublisher:1.25
jackson2-api:2.13.3-285.vc03c0256d517
javadoc:1.0
javax-activation-api:1.2.0-3
javax-mail-api:1.6.2-2
jaxb:2.3.6-1
jdk-tool:1.0
jenkins-design-language:1.24.8
jira:3.7.1
jjwt-api:0.11.2-5.143e44951c52
job-dsl:1.77
jquery3-api:3.6.0-1
jsch:<IP_ADDRESS>
junit:1.53
kubernetes:1.31.2
kubernetes-client-api:5.12.1-187.v577c3e368fb_6
kubernetes-credentials:0.9.0
kubernetes-credentials-provider:1.199.v4a_1d1f5d074f
lockable-resources:2.11
mailer:414.vcc4c33714601
mapdb-api:<IP_ADDRESS>
matrix-auth:2.6.8
matrix-project:771.v574584b_39e60
maven-plugin:3.7
mercurial:2.15
metrics:<IP_ADDRESS>
oauth-credentials:0.4
okhttp-api:4.9.2-20211102
openshift-client:1.0.35
openshift-login:1.0.26
openshift-sync:1.0.53
pam-auth:1.6
parameterized-trigger:2.43.1
pipeline-build-step:2.16
pipeline-github:2.8-138.d766e30bb08b
pipeline-graph-analysis:1.10
pipeline-input-step:2.12
pipeline-milestone-step:1.3.1
pipeline-model-api:1.9.3
pipeline-model-definition:1.8.4
pipeline-model-extensions:1.9.3
pipeline-rest-api:2.15
pipeline-stage-step:2.5
pipeline-stage-tags-metadata:1.8.4
pipeline-utility-steps:2.12.0
plain-credentials:1.7
plugin-util-api:2.4.0
popper-api:1.16.1-2
prometheus:2.0.10
pubsub-light:1.13
run-condition:1.3
scm-api:602.v6a_81757a_31d2
script-security:1138.v8e727069a_025
slack:625.va_eeb_b_168ffb_0
snakeyaml-api:1.29.1
sse-gateway:1.24
ssh-credentials:1.19
sshd:3.0.4
structs:308.v852b473a2b8c
subversion:2.15.4
timestamper:1.20
token-macro:293.v283932a_0a_b_49
trilead-api:1.0.13
variant:1.4
windows-slaves:1.0
workflow-api:1153.vb_912c0e47fb_a_
workflow-basic-steps:2.20
workflow-cps:2660.vb_c0412dc4e6d
workflow-cps-global-lib:566.vd0a_a_3334a_555
workflow-durable-task-step:2.35
workflow-job:1145.v7f2433caa07f
workflow-multibranch:716.vc692a_e52371b_
workflow-scm-step:2.13
workflow-step-api:625.vd896b_f445a_f8
workflow-support:813.vb_d7c3d2984a_0

What Operating System are you using (both controller, and any agents involved in the problem)?
Jenkins running on OpenShift Container Platform 4.11.
Reproduction steps

Configure Slack using JCasC. Omit the room key. E.g. the minimal config would be:

unclassified:
  slackNotifier:
    teamDomain: mydomain
    tokenCredentialId: slack-api-token

Bring up a new Jenkins with this JCasC config
Try to use slackSend from a pipeline without specifying a channel

Expected Results
Slack message goes to default channel associated with token.
Actual Results
23:17:50  java.lang.IllegalArgumentException: Project Channel or Slack User ID must be specified.
23:17:50  	at jenkins.plugins.slack.StandardSlackService.<init>(StandardSlackService.java:140)
23:17:50  	at jenkins.plugins.slack.workflow.SlackSendStep$SlackSendStepExecution.getSlackService(SlackSendStep.java:451)
23:17:50  	at jenkins.plugins.slack.workflow.SlackSendStep$SlackSendStepExecution.run(SlackSendStep.java:307)
23:17:50  	at jenkins.plugins.slack.workflow.SlackSendStep$SlackSendStepExecution.run(SlackSendStep.java:258)
23:17:50  	at org.jenkinsci.plugins.workflow.steps.SynchronousNonBlockingStepExecution.lambda$start$0(SynchronousNonBlockingStepExecution.java:47)
23:17:50  	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
23:17:50  	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
23:17:50  	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
23:17:50  	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
23:17:50  	at java.base/java.lang.Thread.run(Thread.java:829)
23:17:50  Finished: FAILURE

Anything else?
It appears that this is fixed by just adding room: "" to the JCasC config. Another way to work around this is to go to the Jenkins configuration page and just save it without changing anything; an empty room tag will be added to the Slack config on-disk.
So it sounds like there might be faulty logic somewhere that distinguishes between a null vs empty string for the room when it shouldn't.

Is there a reason you aren’t specifying a room?
While web hooks do work some functionality won’t work with them.

We use the same JCasC config to bring up multiple instances of Jenkins for different pipelines. They each send notifications to different Slack rooms and have their own separate tokens. We've been able to avoid having to conditionalize/templatize JCasC configs so far to keep things simple. Everything seems to work, so we're not using any functionality of slackSend that explicitly requires a room.

but you can just create one bot user, and one token. invite the bot to the room and then override the room as you need it. a room in your slack send config is a lot clearer than a token

it's accidental

Ahh, that's unfortunate. In that case we'll have to throw more configuration at it. It's confusing though, because when you set this up on the Slack app side, it asks for a channel to post to. Is there any purpose to that?

and as far as I know the webhooks you are using are likely deprecated: api.slack.com/legacy/custom-integrations/messaging/webhooks#migrating-from-legacy

We're using the Jenkins CI app in Slack and get the token it generates from there. Is that using the legacy webhooks API?

don't you currently need to provide configuration to override the token? or are you doing that at a folder level?

The Jenkins credential is named the same (so is embedded in the generic JCasC dropin), but it's created differently per-pipeline.

no that's not the webhooks one that's the legacy slack built-in one.
see more here:
https://github.com/jenkinsci/slack-plugin#bot-user-mode


Oculus Quest hands not working on steam

Well, I'm developing a videogame for the oculus quest, and I had 0 problems working on the engine, but once I try to play SuperHot on steam, it does not recognize my hands, and I can see them behind me not moving at all.

Can you play other games on steam?

Can you play other games on steam?

SUPERHOT has this issue with me too, the hands are on the floor, detached from the controllers  so I cannot interact with anything.

same problem here, nothing i try fixes the problem

Any luck you guys? My hands seem to spawn on the ground, I've not found any solution via google.


How to get face embedding / encoding?

Congrats for this nice project @ipazc!
I see the output of your algorithm is something like:
[
    {
        'box': [277, 90, 48, 63],
        'keypoints':
        {
            'nose': (303, 131),
            'mouth_right': (313, 141),
            'right_eye': (314, 114),
            'left_eye': (291, 117),
            'mouth_left': (296, 143)
        },
        'confidence': 0.99851983785629272
    }
]

i.e. it gives the bounding box, the keypoints (nose, mouth, eye, etc.).
But how to get an face embedding / face encoding, to be able to do face identification?

I may have misunderstood what you are asking, but this project isn't about getting face encodings for recognition purposes. For that, you will need something else, such as https://github.com/ageitgey/face_recognition.

@MattyB95 Maybe I misunderstood this project, but I thought MTCNN + FaceNet not only allows face detection, but also recognition / identification, by assigning a 128D vector embedding to each face.
https://github.com/davidsandberg/facenet
Is it correct @ipazc?

@josephernest I'm not completely familiar with FaceNet but wouldn't that be providing the face recognition/identification encodings. This project would be more about cropping the image to the facial region for that purpose, but I will let @ipazc give his verdict :)

MTCNN is only used to detect faces in an image,
The value in the the "box" are the corners for the face box in an image
To get the face encoding you will need to pass the the pixels resulted from cropping the image to a face corners to one of the models that are used for face encoding, as facenet, deepface,....

@imnimn Is there a Python implementation that packs: MTCNN for cropping the face box + facenet or deepface for the encoding?

@imnimn Is there a Python implementation that packs together: MTCNN to get the face box + facenet or deepface for the encoding?

Please check this article:
https://arsfutura.com/magazine/face-recognition-with-facenet-and-mtcnn/,
It is implementation is in this repository
https://github.com/arsfutura/face-recognition


More updates to Verify

This PR further improves Verify implementation in the SDK, adding documentation, validation, missing fields etc.

Codecov Report

Merging #410 (cadce54) into main (38f0f6c) will increase coverage by 0.20%.
The diff coverage is 97.36%.

@@             Coverage Diff              @@
##               main     #410      +/-   ##
============================================
+ Coverage     90.81%   91.01%   +0.20%
- Complexity     1662     1675      +13
============================================
  Files           286      285       -1
  Lines          4638     4632       -6
  Branches        204      210       +6
============================================
+ Hits           4212     4216       +4
+ Misses          338      329       -9
+ Partials         88       87       -1

Impacted Files
Coverage Δ

.../java/com/vonage/client/redact/RedactEndpoint.java
100.00% <ø> (ø)

...main/java/com/vonage/client/verify/BaseResult.java
0.00% <ø> (ø)

...java/com/vonage/client/verify/ControlEndpoint.java
100.00% <ø> (ø)

...java/com/vonage/client/verify/ControlResponse.java
100.00% <ø> (ø)

...com/vonage/client/verify/SearchVerifyResponse.java
91.66% <ø> (ø)

...in/java/com/vonage/client/verify/VerifyClient.java
91.17% <ø> (+2.94%)
:arrow_up:

...com/vonage/client/verify/VerifyControlCommand.java
100.00% <ø> (ø)

...n/java/com/vonage/client/verify/VerifyDetails.java
96.66% <0.00%> (-3.34%)
:arrow_down:

...java/com/vonage/client/verify/VerifyException.java
100.00% <ø> (ø)

...n/java/com/vonage/client/verify/VerifyRequest.java
92.30% <ø> (ø)

... and 73 more


Methods, self and ad-hoc inheritance

An idea for adding methods call syntax to Dyon. A self argument name after two underscores __ tells Dyon the function can be called as a method. There is no self keyword, so you can use either self or another name for the argument.
Example:
fn person__name(name: str) -> Person {} {
    return {name: name}
}

// Use `self` after two underscores `__` to tell Dyon that this is a method.
fn say_something__self(p: Person {}) {
    println(p.name + ": Doh!")
}

// You can use `self` for argument name, but you don't have to.
fn walk__self(mut self: Person {}) { ... }

fn main() {
    homer := person(name: "Homer Simpson")
    // Call method.
    homer.say_something()
    // Alternative calling syntax.
    say_something(Person: homer)
    // Use `mut` when the method mutates the object.
    mut homer.walk()
}

To know when an object is mutated, you need to use mut in front of the variable name when calling a method.
The self argument must have an ad-hoc type. This makes it possible to declare methods with same names but for different types. Dyon replaces self with the name of the ad-hoc type.
You can only call a method when the type checker can prove an ad-hoc type for a variable.
The self keyword could be added to syntax highlighting to make it easier to read the code, but does not need any special meaning in the language, only in the function name.
mut block
When a method mutates the object, you can use a mut block to call multiple methods with .:
mut window {
  .set(title: "Hello world")
  .set(size: [200, 300])
}
// Can not call `.title()` inside `mut` block.
println(window.title())

This is how you declare a set method:
fn set__self_title(mut window: Window {}, title: str) { ... }
fn set__self_size(mut window: Window {}, size: [f64]) { ... }

get methods just use the names of the property:
fn title(window: Window {}) -> str { ... }
fn size(window: Window {}) -> [f64] { ... }

Ad-hoc inheritance
Methods could be extended with a form of ad-hoc inheritance.
When the method is not found for an ad-hoc type, a nested ad-hoc type can be used.
fn bar__self(a: Bar {}) { ... }

// `Foo` inherits from `Bar`
fn foo__self(a: Foo Bar {}) {
    a.bar()
}

First Dyon looks in the table Foo for the method bar, and because it is not found, it looks up in the table Bar.
If a method for a nested ad-hoc type is the same for the nested type, an error is shown to require the keyword override on the new function and virtual or abstract on the super function.
// Allow sub types to override the behavior.
fn virtual bar__self(a: Bar {}) { ... }

// Override behavior from super function.
fn override bar__self(a: Foo Bar {}) { ... }

The abstract keyword is used to specify a method for an ad-hoc type that is later specified by sub types. The body must be empty.
fn abstract bar__self(a: Bar {}) -> str {}

fn override bar__self(a: Foo Bar {}) -> str { ... }

This is a bit too complicated right now. Will have to wait for later.


dict<K,V> does not have const operator[]

It might be nice to have a const operator[] in the dict class, for accessing elements from const dicts.

T& dict<>::operator[](const K &key) creates the entry with its default constructor when it is not already in the dict. You can't do that with a const dict<>. Use const T& dict<>::at(const K &key) const to access an already existing entry (throws std::out_of_range for non existing entries).


Makes sure that phantomjs.path is valid. Related to #506.

Proposed changes in this pull request
Checks to make sure phantomjs is valid path.
Checklist

[x] New tests have been added
[x] npm test passes successfully
[ ] Documentation has been updated

@amir20 to review

@elias-winberg can you also review?


How to normalize the nuclear distance channel?

Hi, great work!
Image data downloaded from those image_data0x.npy files has un-normalized nuclear distance channel. Would be nice if you can provide the way that to normalize the channel to -1 and 1.
Thanks.

Hi,
We didn't normalize nuclear distance and I think you shouldn't. Instead, it is rescaled by 0.01.
https://github.com/royerlab/cytoself/blob/0133ab6c45d990cbfb1ab25361facd2c43940a04/cytoself/datamanager/opencell.py#L69

Hi @li-li-github.
Thanks for your advice. As suggested, I have successfully achieved nuclear segmentation with StarDist.

But, I'm also curious about how to get the nucleus distance matrix from the segmented picture. Could I reference the code for computing nucleus distance? Thanks.

Hi @JimmyCai91 and @Yuangaga
I'm very sorry for the late reply.
After checking my old code and recalling my memory, I think what I meant was
making -1 <= the values in the nuclear distance channel =< 1
So it's not "normalized."
This was done by rescaling the channel with 0.01. Throughout the whole dataset that we are offering in this repo, the min value of the nuclear distance channel after rescaling is -1.053 and there are only 118 pixels that are lower than -1, so they should be negligible.
The max value is well below 1.
The idea of making the nuclear distance in [-1, 1] was because, without rescaling, the large values in the nuclear distance channel would overwhelm other channels, resulting in unsuccessful training.
I hope this answered your question.

Hi @li-li-github ,
Thanks for the reply. It reads great!
I will close this issue.
Thanks,

@Yuangaga if you need, feel free to reopen this issue.


Bump k8s bits to the latest available version

Bump k8s bits to the latest available version
https://github.com/openshift/cluster-api-provider-powervs/blob/main/go.mod#L21:L28

/remove-lifecycle stale

Currently the version 1.22 is supported as like other providers

fixed via #42


memcpy error when trying to run script without docker

Tried copy-initalization instead of memcpy, but does not work because one variable is a char and the other is a RemReqType. Should this memcpy error just be ignored?
transport/message.cpp: In member function ‘virtual void YCSBClientQueryMessage::copy_from_buf(char*)’: ./system/helper.h:206:29: error: ‘void* memcpy(void*, const void*, size_t)’ writing to an object of non-trivially copyable type ‘class ycsb_request’; use copy-assignment or copy-initialization instead [-Werror=class-memaccess]   memcpy(&v, &d[p], sizeof(v)); \

I'm having the same issue.
It starts with my MakeFile not building the ./rundb and ./runcl executables using gcc version 9.3.0 (Ubuntu 9.3.0-17ubuntu1~20.04) to target x86_64-linux-gnu.
g++ -c -Wall -g -gdwarf-3 -std=c++0x -rdynamic  -I. -I./benchmarks -I./client/ -I./transport -I./system -I./statistics -I./blockchain -I./smart_contracts  -I./data_structures -Ideps/jemalloc-5.1.0/include -Ideps/nng-1.3.2/include -Ideps/boost_1_67_0 -Ideps/crypto -I./db -Ideps/sqlite-autoconf-3290000/build/include -D NOGRAPHITE=1 -Werror -Wno-sizeof-pointer-memaccess  -o obj/config.o config.cpp
g++ -c -Wall -g -gdwarf-3 -std=c++0x -rdynamic  -I. -I./benchmarks -I./client/ -I./transport -I./system -I./statistics -I./blockchain -I./smart_contracts  -I./data_structures -Ideps/jemalloc-5.1.0/include -Ideps/nng-1.3.2/include -Ideps/boost_1_67_0 -Ideps/crypto -I./db -Ideps/sqlite-autoconf-3290000/build/include -D NOGRAPHITE=1 -Werror -Wno-sizeof-pointer-memaccess  -o obj/ycsb_query.o benchmarks/ycsb_query.cpp
In file included from deps/boost_1_67_0/boost/mpl/aux_/na_assert.hpp:23,
                 from deps/boost_1_67_0/boost/mpl/arg.hpp:25,
                 from deps/boost_1_67_0/boost/mpl/placeholders.hpp:24,
                 from deps/boost_1_67_0/boost/mpl/bind.hpp:24,
                 from deps/boost_1_67_0/boost/mpl/lambda.hpp:18,
                 from deps/boost_1_67_0/boost/parameter/parameters.hpp:13,
                 from deps/boost_1_67_0/boost/lockfree/detail/parameter.hpp:13,
                 from deps/boost_1_67_0/boost/lockfree/detail/freelist.hpp:25,
                 from deps/boost_1_67_0/boost/lockfree/queue.hpp:22,
                 from ./system/pool.h:6,
                 from ./system/global.h:29,
                 from ./system/query.h:4,
                 from benchmarks/ycsb_query.cpp:1:
deps/boost_1_67_0/boost/mpl/assert.hpp:188:21: error: unnecessary parentheses in declaration of ‘assert_arg’ [-Werror=parentheses]
  188 | failed ************ (Pred::************
      |                     ^
deps/boost_1_67_0/boost/mpl/assert.hpp:193:21: error: unnecessary parentheses in declaration of ‘assert_not_arg’ [-Werror=parentheses]
  193 | failed ************ (boost::mpl::not_<Pred>::************
      |                     ^
cc1plus: all warnings being treated as errors
make: *** [Makefile:71: obj/ycsb_query.o] Error 1

I then ignore warnings arising from boost in my deps/boost_1_67_0/boost/mpl directory.
and it brings me to #17 above.

Yeah I made those changes that it asked for regarding the parentheses and yup, got at the same error. I guess I should just use the docker setup? But, ugh, hard to set that up across multiple systems though...

resolved


export lines not covered

It would appear that export is the only thing not covered in my code. Shouldn't export be ignored or something? What's going on here?

is that a screenshot from the browser? or external report?
we know that the exports are being transpiled
what version of ember? ember-cli-blanket?

@jschilli that is a screenshot from Code Climate.
I'm on Ember 2.3.1 and ember-cli-blanket 0.8.0
Is it supposed to run coverage on the transpiled code and not the original code?
My options look like this, I might have messed up:
var options = {
  modulePrefix: 'ember-math-helpers',
  filter: '//.*ember-math-helpers/.*/',
  antifilter: '//.*(tests|template).*/',
  loaderExclusions: [],
  enableCoverage: true,
  cliOptions: {
    reporters: ['lcov'],
    autostart: true,
    lcovOptions: {
      outputFile: 'lcov.dat',
      renamer: function (moduleName) {
        var expression = /^ember-math-helpers/;
        return moduleName.replace(expression, 'addon') + '.js';
      }
    }
  }
};

Okay, I looked at localhost:4200/tests, and it is transpiled. Now the uncovered part is the actual inside of the function. I'm not sure how it's not covered though. My test correctly does the addition, and it passes, so I am not sure why it is listed as not covered.
test('addition works', function(assert) {
  let result = add([20, 10]);
  assert.equal(result, 30);
});

@jschilli looks like this may be related to https://github.com/sglanzer/ember-cli-blanket/issues/105
This code is clearly exectued, but not marked as covered. I saw the fix there was to downgrade loader.js. Is there a way to fix this or update the library so we don't have to downgrade?

This should be fixed with 0.9.1


ServerBuilder::SetDefaultCompressionLevel doesn't work

What version of gRPC and what language are you using?
c++
What operating system (Linux, Windows, …) and version?
macOS
What runtime / compiler are you using (e.g. python version or version of gcc)
What did you do?
use ServerBuilder to construct a server with SetDefaultCompressionLevel()
What did you expect to see?
the server send compressed data
What did you see instead?
the data does not been compressed.
Anything else we should know about your project / environment?
How to
Please insert code maybe_default_compression_level_.is_set = true; in function ServerBuilder::SetDefaultCompressionLevel()
ServerBuilder& ServerBuilder::SetDefaultCompressionLevel(
    grpc_compression_level level) {
  maybe_default_compression_level_.is_set = true;  // code to be inserted
  maybe_default_compression_level_.level = level;
  return *this;
}

A pull request would be welcome next time :)


merge Extrality/trow into trow-registry/trow

This includes all the changes/improvements made in https://github.com/Extrality/trow (see https://github.com/Extrality/trow/pulls).
Closes #345, #343, #340, #329, #279, #250, #339, #330, #229, #189, #168

I'd really like to re-enable cosign. I don't really see the problem with signatures.

Thanks !

cosign and no-tls were pretty useful I thought

cosign may be added back later but it needs some work, I want to avoid polluting the package page with signatures like what happened in https://github.com/Extrality/trow/pkgs/container/trow.
no-tls is still there: before, trow looked for certificates in /etc unless --no-tls was given. Now you need to use the --tls option, which allows passing arbitrary paths to certificates

Ok, great. Very happy to hear about tls :)
I should manage to look into cosign at some point. I might also manage to look into simplifying the docker build. Both of these things have cross-over with my current work.


Add sgdisk to the inmutable-rootfs module setup

Fixes #1632
Signed-off-by: Itxaka<EMAIL_ADDRESS>

Up to my understanding this will not hit system/cos unless you bump the initrd package too.

I'm just trying to understand why it started failing suddenly out of nowhere. It's very confusing.

Related to https://github.com/rancher/elemental-toolkit/pull/1631

Related to https://github.com/rancher/elemental-toolkit/pull/1631

Well I hope so! 🤣

Related to #1631

Well I hope so! rofl

it was just to put a link for (self-)navigation through issues and associated PRs 🤣

Related to #1631

Well I hope so! rofl

it was just to put a link for (self-)navigation through issues and associated PRs rofl

I mostly meant that the link its worng as its pointing to itself :)
Sorry, still sleepy, did not finish the comment lmao. I put the correct Fixes stanza in the first comment :+1:

I'm just trying to understand why it started failing suddenly out of nowhere. It's very confusing.

Blerg, now I see it, it was dropped by mistake 2 PRs ago but then on my EFI PR initrd was bumped. So thats when the errors appear. Ok, clear now, bumping initrd.

I mostly meant that the link its worng as its pointing to itself :)
Sorry, still sleepy, did not finish the comment lmao. I put the correct Fixes stanza in the first comment +1

🤦🏼 I thought I put a link to the issue... the one still sleepy it's me 😪  🤣

I'm just trying to understand why it started failing suddenly out of nowhere. It's very confusing.

I am the one to blame here, I removed the sgdisk from initrd while cleaning up few packages mostly due to bootloader changes. I though it was a leftover. In fact, sgdisk should not be used, as we relay on parted, however then I found this (I had completely forgot it): https://github.com/rancher/elemental-cli/blob/b918db3867add2cd4a2ea0cff748638b04556fdf/pkg/partitioner/disk.go#L125-L143

I'm just trying to understand why it started failing suddenly out of nowhere. It's very confusing.

I am the one to blame here, I removed the sgdisk from initrd while cleaning up few packages mostly due to bootloader changes. I though it was a leftover. In fact, sgdisk should not be used, as we relay on parted, however then I found this (I had completely forgot it<EMAIL_ADDRESS>
Yeah I finally found it. My confusion was that I wrongly did a git history and thougth that the sgdisk was not part of the initrd for a long long time, so I was very confused on why would it happen now when we bumped the initrd several times. Until I found out that it was a very recent slip and then it make sense :D

This is important information which isn't captured in the PR :-(
We should add a comment in module-setup.sh to notice that sgdisk can be dropped once parted has catched up.


_userManager.ResetPasswordAsync(...) is returning Invalid Token validation message

App: ASP.NET MVC Core 1.1.1 with Individual User Accounts
Identity Core
Entity Framework Core
Visual Studio 2017 ver 15.3.3
When clicking on Forgot Password link, the following code successfully sends an email with a generated link. Clicking on the link inside the email body correctly displays the Reset Password form.  But after entering login name, new password etc and clicking on Reset button, the validation error Invalid Token shows up.
Body of my Email that App sent:
Please reset your password by clicking here: link

Note: The PasswordReset functionality works fine if I am logged in and want to reset my password to something else.
The only difference between following code and the default code that VS2017 generates in AccountController is that we're not using Email attribute as a user name, we have changed
[Required]
[EmailAddress]
public string Email { get; set; }

to
[Required]
public string UserName { get; set; }

etc. etc., and have extended the user properties as follows:
ApplicationUser.cs:
// Add profile data for application users by adding properties to the ApplicationUser class
public class ApplicationUser : IdentityUser
{
    public string UserFullName { get; set; }
    [Column(TypeName = "varchar(50)")]
    public string UserEmaill { get; set; }
    [Column(TypeName = "varchar(15)")]
    public string UserCity{ get; set; }
}

Controller:
[HttpPost]
[AllowAnonymous]
[ValidateAntiForgeryToken]
public async Task<IActionResult> ForgotPassword(ForgotPasswordViewModel model)
{
    if (ModelState.IsValid)
    {
        var user = await _userManager.FindByNameAsync(model.UserName);
        string sUserEmail = user.UserEmail; //I Added this. Note the email is not the user name in my app
        if (user == null)
        {
            // Don't reveal that the user does not exist or is not confirmed
            return View("ForgotPasswordConfirmation");
        }

        // For more information on how to enable account confirmation and password reset please visit http://go.microsoft.com/fwlink/?LinkID=532713
        // Send an email with this link
        var code = await _userManager.GeneratePasswordResetTokenAsync(user);
        var callbackUrl = Url.Action("ResetPassword", "Account", new { userId = user.Id, code = code }, protocol: HttpContext.Request.Scheme);
        await _emailSender.SendEmailAsync(sUserEmail, "Reset Password", $"Please reset your password by clicking here: link");
        return View("ForgotPasswordConfirmation");
    }

    // If we got this far, something failed, redisplay form
    return View(model);
}

[HttpPost]
[AllowAnonymous]
[ValidateAntiForgeryToken]
public async Task<IActionResult> ResetPassword(ResetPasswordViewModel model)
{
    if (!ModelState.IsValid)
    {
        return View(model);
    }
    var user = await _userManager.FindByNameAsync(model.UserName);
    if (user == null)
    {
        // Don't reveal that the user does not exist
        return RedirectToAction(nameof(AccountController.ResetPasswordConfirmation), "Account");
    }
    var result = await _userManager.ResetPasswordAsync(user, model.Code, model.Password);
    if (result.Succeeded)
    {
        return RedirectToAction(nameof(AccountController.ResetPasswordConfirmation), "Account");
    }
    AddErrors(result);
    return View();
}

UPDATE:
I read this famous article  on the above issue. But the article is more than 2 years old and none of the following scenarios apply to my case:

Password Reset Token vs Email Confirmation Token: I've verified that I'm using Password Reset Token; moreover, the input parameter purpose in the signature public virtual async Task<string> GenerateUserTokenAsync(string purpose, TKey userId) is not used in GeneratePasswordResetTokenAsync(...) of ASP.NET Core 1.1.1.

Token is changed in transit: Author is talking about using Url-Encoding  such as: code = System.Web.HttpUtility.UrlEncode(code); but ASP.NET Core does not have System.Web available.

Security Stamp is Null: I've verified that each user in my ASPNETUsers table has SecurityStamp value not null.

Machine Keys are not matching: Token was generated on the same desktop machine (windows 10) where it's being validated (just minutes after it was generated).

UPDATE 2
After reading a response from @claudioct, I installed NuGet package  for using UrlEncode. I then tried the following but to no avail

I tried code = System.Net.WebUtility.UrlEncode(code); after the line var code = await _userManager.GeneratePasswordResetTokenAsync(user); inside ForgotPassword(...) POST method; but still the same error. I then also tried var code = System.Net.WebUtility.UrlDecode(model.Code); before the line var result = await _userManager.ResetPasswordAsync(user, code, model.Password); inside ResetPassword(...) POST method; same error again.

Can use System.Net.WebUtility.UrlEncode("string") and System.Net.WebUtility.UrlDecode("string") to resolve #2;

@claudioct I tried your suggestion but still the same error. I've added UPDATE 2 in my post above.

If you're using ModelBinder, try to not use System.Net.WebUtility.UrlDecode("string") .
I guess Net Core do the dirty work for us.

@claudioct  Yes, I've read that at different places online to not to use Decode since, according to some, it may even bring back the issue you are trying to resolve. That's why, I mentioned in my UPDATE 2, that first I tried using Encoding part only. But after getting the same error, I tried Decoding part, as well, but still the same error.

Did you debugged to make sure you're trying to change password with correct token?

@claudioct In my Note in the post above I mentioned that the PasswordReset functionality works fine if I am logged in and want to reset my password to something else. Now the same functionality does not work when a generated link is clicked on the email body. I don't know how I would debug outside the visual Studio (please note that we are now not inside the Visual Studio anymore; it's PasswordReset form when I click on the link in the email). Browser F12 mode does not give any info not even any error.

@HaoK

This is really up to your code. We give you a code which must match. How you render the tag in your email is up to you. UrlEncode is wrong for example, because you're passing it as a query string, and UrlEncode is for path parts.


As a developer, I want to manage the versioning of my deployment/application.

Problem
We want to manage the different versions of our applications and test them progressively (for example, a frontend developer wants to test the version of the backend that is in the testing phase on a PR). We need to have a deployment for each version.
Solution
If a user creates a PR, an automatic deployment should be carried out with the correct version (via the Dockerfile, based on the specific file of each directory. For example, for the frontend, it should refer to the package.json). Afterward, the URL of this deployment should be displayed in the pull request (for example, https://v100.nachet.ninebasetwo.xyz/ for the frontend and https://v100.nachet.ninebasetwo.xyz/api for the backend). Subsequently, a developer could test https://"x".nachet.ninebasetwo.xyz at any time.
How to do it

Automatic deployment of an image version (application) with ArgoCD to Kubernetes.
Tag the Docker image with the version specified in the specific file (for example, package.json for the frontend).
For routing, modify the LB Ingress NGINX to point to a specific version. For example, if a frontend application is on version v1.0.0, its URL would be https://v100.nachet.ninebasetwo.xyz/. When someone clicks on this URL, the backend will automatically communicate with https://v100.nachet.ninebasetwo.xyz/api (v100 being the corresponding version for the backend).

Summary

Each PR for each application has a URL. The frontend URL always points to the corresponding version of the backend (https://v100.nachet.ninebasetwo.xyz/ -> https://v100.nachet.ninebasetwo.xyz/api). The LB needs to be configured for this.
A new tag is added to the Dockerfile (the version tag).

Regarding the Dockerfiles of the projects, ARGs need to be retained at build time (for example, docker build...).
Steps:

[ ] Modify this github action to tag the image with the corresponding tag of the application
[ ] Create a new github action to create the deployment via ArgoCD and display the new URL inside the PR

[ ] Deploy ArgoCD
[ ] Automate the deployment (secrets included) to kubernetes
[ ] Display the new URL inside the PR

[ ] Configure the LB Ingress NGINX to point the frontend to the specific backend version (e.g https://v100.finesse.ninebasetwo.xyz -> https://v100.finesse.ninebasetwo.xyz/api)

review ADR: https://github.com/ai-cfia/dev-rel-docs/blob/main/adr/006-database-change-management.md
also : https://www.ietf.org/rfc/rfc1034.txt#:~:text=They%20must%20start%20with%20a,be%2063%20characters%20or%20less.&text=information%2C%20which%20may%20be%20empty.)

They must start with a letter, end with a letter or digit, and have as interior characters only letters, digits, and hyphen. There are also some restrictions on the length. Labels must be 63 characters or less. information, which may be empty.

I suggest substituting . for - so v2.3.30 becomes v2-3-30


proposal: net/http: add a generic mechanism for storing arbitrary data in Request

There is a demand to hold arbitrary data in the request scope, such as passing data between multiple handlers or storing parsed path variables by an HTTP router (e.g., #60227). This proposal suggests a general mechanism for storing arbitrary data in the Request struct.
Background
Currently, the recommended approach for associating arbitrary data with a Request is to use Request.WithContext or Request.Clone along with context.WithValue:
req = req.WithContext(context.WithValue(userCtxKey, user))
// or
req = req.Clone(context.WithValue(userCtxKey, user))

The problem with this approach is that both methods create a copy of the Request (either shallow or deep), resulting in a performance hit. Additionally,  users need to be cautious to avoid using the old Request.
It is worth mentioning that the primary purpose of Request.WithContext and Request.Clone is to enable users to control the entire lifetime of a request and its response: obtaining a connection, sending the request, and reading the response headers and body. Storing arbitrary data doesn't look like a good fit.
Before the context package was introduced (i.e., before Go 1.7), storing arbitrary data in Request was basically not possible. So as a workaround, HTTP routers or web frameworks designed alternative Handler variants, like func(ResponseWriter, *Request, map[string]string), to pass their parsed path variables along with ResponseWriter and Request. Or simply abandon Handler and introduce a Context struct of their own. The result is added complexity.
The #60227 also suggested a general mechanism to store arbitrary data in Request:
package http

type Request struct {
	...
	Vars map[string]any
	...
}

However, in my opinion, the usage of the exported map[string]any has the following problems:

The user is required to perform a nil check before using it, which can be cumbersome.
The presence of the exported map introduces the risk of data tampering by other packages.
The use of string keys can lead to collisions, especially as it is also designed to store parsed path variables.

Proposal
I propose to add an unexported map[any]any field named "vaules" to the Request struct. This field is used to hold arbitrary data within the request scope. Additionally, we also need add two exported methods to the Request struct: func(key any) (value any, ok bool) as the getter and func(key, value any) as the setter for Request.values.
Implementation
This proposal will introduce the following API changes to the net/http package:
package http

type Request struct {
	...
	values map[any]any
	...
}

// Value returns the value set in the r for the given key, or nil if no value is
// present. The ok result indicates whether value was found in the r.
func (r *Request) Value(key any) (value any, ok bool) { ... }

// Set sets the value to the r for the given key.
//
// The provided key must be comparable and should not be of type string or any
// other built-in type to avoid collisions between packages. Users should define
// their own types for keys.
func (r *Request) SetValue(key, value any) { ... }

CC @neild @bradfitz

The problem with this approach is that both methods create a copy of the Request (either shallow or deep), resulting in a performance hit.

https://github.com/golang/go/issues/48811 Request.SetContext was declined due to

Request is explicitly immutable. That's why we have Request.WithContext.

Does this proposal make request mutable? :thinking:


[PokémonTabletopAdventures_v3]: Adds level tracker to hybrid characters

Changes / Comments

Added a level field to the hybrid (or Pokémon (Character Class)) character type.
Altered some CSS ever so slightly to ensure the new "Level / Honors" label is displayed on one line for the default font size.
Updates the README accordingly.

Roll20 Requests

[x] Does the pull request title have the sheet name(s)? Include each sheet name.
[x] Does this add or change functional aesthetics (such as layout or color scheme)?

Character Sheet Info Roll20 Internal Use only.


(docs) - fix docs.  (feat) - add key === pathname.

documentation for adding custom Route render was wrong.
also, i added the key === pathname for the default component.
different routes of the same path component need to be re-rendered.  without the key, the router doesn't know to re-render.

The implementation of this has changed.


[BD-46] docs: improve design and usability of Icons page

Description
Features and fixes:

according to big number of icons it required time to render all the icons. To avoid instant rendering of 2000+ icons react-window was used
search against 2000+ icons is also a heavy operation. To avoid multiple searches debounce from lodash was used
preview of the icon is added on the top of the table. If user scrolls to the bottom of the table and clicks on some icon then he is auto-scrolled to the preview.
added styles
fix for the SearchField: re-rendering of the Wrapper inside the SearchField makes the inner components to unmount and mount again that causes problems when writing a text in the input. Added useCallback to prevent re-rendering

Deploy Preview
Icons page
Merge Checklist

[ ] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[x] Does your change adhere to the documented style conventions?
[ ] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[x] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[ ] Were your changes tested in the example app?
[x] Is there adequate test coverage for your changes?
[ ] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @monteri! I've created BLENDED-1314 to keep track of it in Jira.   More details are on the BD-46 project page.
When this pull request is ready, tag your edX technical lead.

@monteri 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

:tada: This PR is included in version 20.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Azure Marketplace List and GetAgreement API returns errors

Bug Report

import path:  services/marketplaceordering/mgmt/2015-06-01/marketplaceordering
SDK version  v46.0.0
Go version: go version go1.14.2 darwin/amd64

What happened?

MarketplaceAgreementsClient.List and  MarketplaceAgreementsClient.GetAgreement API return errors
Here is my code

  market "github.com/Azure/azure-sdk-for-go/services/marketplaceordering/mgmt/2015-06-01/marketplaceordering"

  client := market.NewMarketplaceAgreementsClient(subscriptionID)
  client.Authorizer = authorizer

  agreements, err := client.List(ctx)
  if err != nil {
  	log.Errorf("error being encountered is ", err.Error())
  }

  agreementTerms, err := client.GetAgreement(ctx, publisher, offer, sku)
  if err != nil {
  	log.Errorf("error being encountered is ", err.Error())
  }

Error with List API:

marketplaceordering.MarketplaceAgreementsClient#List: Failure responding to request: StatusCode=200 -- Original Error: Error occurred unmarshalling JSON - Error = 'json: cannot unmarshal object into Go value of type []marketplaceordering.AgreementTerms

Error with GetAgreement API:

marketplaceordering.MarketplaceAgreementsClient#GetAgreement: Failure responding to request: StatusCode=404 -- Original Error: autorest/azure: error response cannot be parsed: \"The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.\" error: invalid character 'T' looking for beginning of value

Hi @akning-ms do we include this kind of bugs in the S360 KPI?

Yes. it should be in our live validation report. but seems it is not shown due to no traffic. @leni-msft to help take a look

@ArcturusZhang This issue has already been detected in live validation report and created as S360 action item: https://portal.azure-devex-tools.com/amekpis/correctness/detail?errorId=580A0264-1553-4CA9-910D-C4891AFA65E2

@ArcturusZhang @leni-msft , Is this issue still active?


[4.x] Godot flies out when I hide the window

Tested versions
Godot 4.1.3, Godot 4.2.1
System information
Godot v4.1.3.stable - Windows 10.0.19045 - Vulkan (Mobile) - dedicated Radeon RX 560 Series (Advanced Micro Devices, Inc.; 31.0.14001.45012) - Intel(R) Core(TM) i5-4570 CPU @ 3.20GHz (4 Threads)
Issue description
When I click the "minimize" button in a running application, godot displays many errors in the console and crashes.
In the example, I use this plugin, it’s unlikely that this is the problem, since this happens in all modes except “compatibility”
Errors in Forward+:

Errors in Mobile:

Steps to reproduce

Run my example and run it in Forward+ or Mobile mode
Minimize a window

Wait about 15 seconds and you will see a huge number of errors

Minimal reproduction project (MRP)
demoGD.zip

Please try confirming that this happens without the plugin, that's an important thing to consider

Please try confirming that this happens without the plugin, that's an important thing to consider
It's almost impossible :(
I repeated everything that was done in the example project, drawing the Viewport in sprite2d and texturerect, nothing happens, all the main rendering process is in GDCubismUserModel.
If there were problems in compatibility mode, I would be sure that this is a plugin problem, but in compatibility mode everything works

Please try confirming that this happens without the plugin, that's an important thing to consider

It's almost impossible :(
I repeated everything that was done in the example project, drawing the Viewport in sprite2d and texturerect, nothing happens, all the main rendering process is in GDCubismUserModel.
If there were problems in compatibility mode, I would be sure that this is a plugin problem, but in compatibility mode everything works

That doesn't mean it isn't specific to the plugin though, so if it happens with the plugin and not otherwise it's very clearly related to the plugin, so the first step is to make sure the plugin isn't causing it, unless the plugin triggers a bug in the engine

I'll try to contact the plugin developer and ask him to chime in on this issue

I'd say to report it there instead first, if it doesn't happen at all without it

This is a matter under discussion, and it appears that there may be an issue on the side of the Godot Engine.

There might be a bug similar to the one I reported before at https://github.com/godotengine/godot/issues/81476.

The issue of increasing memory usage when minimizing the window can be reproduced by a script that explicitly sets a property (transparent_bg) on an initialized SubViewport.
extends Node3D

const NODE_COUNT: int = 32

var ary_n: Array[Node3D]
var ary_v: Array[SubViewport]

func _ready():
    for i in range(NODE_COUNT):
        var n = Node3D.new()
        self.ary_n.append(n)

        var v = SubViewport.new()
        self.ary_v.append(v)

        n.add_child(v)
        add_child(n)

func _process(_delta):
    for n in self.ary_n:
        for v in n.get_children():
            n.remove_child(v)
        remove_child(n)

    var index = 0
    for n in self.ary_n:
        var v: SubViewport = ary_v[index]
        v.transparent_bg = true

        n.add_child(v)
        add_child(n)
        index += 1

This code does not render anything, but when the window is minimized, static memory starts to be used.
It is used up at an incredible speed, so it is recommended to restore from minimization immediately.
This behavior has been confirmed on the Windows and macOS versions of Godot Engine 4.1.2.

@AThousandShips

It appears that minimizing the window leads to additional issues.
When the window is minimized, dynamically generated ArrayMeshes are not discarded, leading to a memory leak as they continue to accumulate in memory.
Although there is no verification code, it seems that a memory leak occurs only when the window is minimized if the following process is performed on a MeshInstance2D added to SubViewport:
Array ary;
ary.resize(Mesh::ARRAY_MAX);
ary[Mesh::ARRAY_VERTEX] = gen_vertex(...);
ary[Mesh::ARRAY_TEX_UV] = gen_uv(...);
ary[Mesh::ARRAY_INDEX] = gen_index(...);

Ref<ArrayMesh> ary_mesh;
ary_mesh.instantiate();
ary_mesh->add_surface_from_arrays(Mesh::PRIMITIVE_TRIANGLES, ary);

...

node->set_mesh(ary_mesh)

This issue is different from the one reported above, but it is a bug that occurs around SubViewport, so I added it here first.

I have reported this issue along with reproducible code at the following link: Godot Engine Issue #90030
Since I have isolated the problem and simplified the code, I feel it would be okay to close this one.


Captcha handling for  Bloomberg feeds

When accessing a free feed from Bloomberg I get blocked by their captcha verification
I’m using iPhone 11 / iOS 16.3/ NetNewsWire 6.1 (build 6110)
The feed:
https://www.bloomberg.com/opinion/authors/ARbTQlRLRjE/matthew-s-levine.rss
the result

The problem is that when you hit the Reader view button, it actually is a robot reading that web page. Or a program, anyway, which is the same thing in this context.
The Reader view works by downloading the page and trying to figure out what the article text is. It’s a program doing that and not a person. Their captcha is working as expected.
I can’t imagine a way around this short of writing code that can fool the captcha, which seems unlikely.


Error when running in docker container

Bug description
when I try to use prisma within a docker container so It will be bundled with my dart api to the cloud I get this error when running npx prisma generate
 => ERROR [build 10/13] RUN npx prisma db pull                              0.7s
------
 > [build 10/13] RUN npx prisma db pull:
0.680 node:internal/fs/utils:351
0.680     throw err;
0.680     ^
0.680
0.680 Error: ENOENT: no such file or directory, open '/app/node_modules/.bin/prisma_schema_build_bg.wasm'
0.680     at Object.openSync (node:fs:596:3)
0.680     at Object.readFileSync (node:fs:464:35)
0.680     at /app/node_modules/.bin/prisma:16:12782
0.680     at /app/node_modules/.bin/prisma:2:355
0.680     at Object.<anonymous> (/app/node_modules/.bin/prisma:242:122)
0.680     at Module._compile (node:internal/modules/cjs/loader:1356:14)
0.680     at Module._extensions..js (node:internal/modules/cjs/loader:1414:10)
0.680     at Module.load (node:internal/modules/cjs/loader:1197:32)
0.680     at Module._load (node:internal/modules/cjs/loader:1013:12)
0.680     at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:128:12) {
0.680   errno: -2,
0.680   syscall: 'open',
0.680   code: 'ENOENT',
0.680   path: '/app/node_modules/.bin/prisma_schema_build_bg.wasm'
0.680 }
0.680
0.680 Node.js v18.19.0
------
Dockerfile:29
--------------------
  27 |
  28 |     # Generate Prisma Client
  29 | >>> RUN npx prisma db pull
  30 |     RUN npx prisma generate
  31 |
--------------------
ERROR: failed to solve: process "/bin/sh -c npx prisma db pull" did not complete successfully: exit code: 1

How to reproduce

make a docker file like the following :

# Official Dart image: https://hub.docker.com/_/dart
# Specify the Dart SDK base image version using dart:<version> (ex: dart:2.17)
FROM dart:stable AS build

WORKDIR /app

# Copy pubspec.yaml and pubspec.lock
COPY pubspec.* ./

# Get dependencies
RUN dart pub get

# Copy everything else
COPY . .

# Install Node.js and npm
RUN apt-get update && apt-get install -y nodejs npm

# Update npm to the latest version
RUN npm install -g npm@latest

# Install Prisma
RUN npm install @prisma/client prisma --save-dev

# Copy Prisma schema
COPY prisma ./prisma

# Generate Prisma Client
RUN npx prisma db pull
RUN npx prisma generate

# Compile the server.dart file to native code
RUN dart compile exe bin/server.dart -o bin/server

# Copy Prisma Engine deps so to `/runtime/`
RUN FILES="libz.so libgcc_s.so libssl.so libcrypto.so"; \
    for file in $FILES; do \
    so="$(find / -name "${file}*" -print -quit)"; \
    dir="$(dirname "$so")"; \
    mkdir -p "/runtime${dir}"; \
    cp "$so" "/runtime$so"; \
    echo "Copied $so to /runtime${so}"; \
    done

FROM scratch

COPY --from=build /runtime /
COPY --from=build /app/bin/server /app/bin/
COPY --from=build /app/.env /app/.env
# COPY --from=build /app/db/test.db /app/db/test.db
COPY --from=build /app/prisma-query-engine /app/prisma-query-engine

ENV TZ=Asia/Shanghai

WORKDIR /app
CMD ["./bin/server"]

try to build it by running : docker build -t example  .

Expected behavior
to generate prisma binary successfully as I am developing on mac machine and deploying to a linux server so it should be compiled again on deployment
Prisma information
// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema

// Looking for ways to speed up your queries, or scale easily with your serverless or edge functions?
// Try Prisma Accelerate: https://pris.ly/cli/accelerate-init

generator client {
  provider      = "dart run orm"
}

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

model User {
  id    Int     @id @default(autoincrement())
  email String  @unique
  name  String?
}

Environment & setup

OS: macos
Database: postgresql
Node.js version: v21.5.0

Prisma Version
^5.10.2

Hey @egyleader, have you tried to also copy the missing file?  prisma_schema_build_bg.wasm is required for the CLI to work properly.

to generate prisma binary successfully as I am developing on mac machine and deploying to a linux server so it should be compiled again on deployment

This is not the problem here. The prisma_schema_build_bg.wasm is platform independent.

0.680 Error: ENOENT: no such file or directory, open '/app/node_modules/.bin/prisma_schema_build_bg.wasm'

This error message is curious though - this file is never present at this location. It should only be in node_modules/prisma/build/ where the prisma CLI should read it from. It is highly surprising that npx prisma db pull gets that wrong.
Any idea what could be going on here?

I ran into this issue yesterday whilst messing around with archiving node_modules. This was outside of any containerisation but hopefully this is a useful data point to add here.
The default state of node_modules/.bin after a fresh yarn install was:

Noting that this is a link pointing to the location inside the prisma package.
I archived the node_modules and then unarchived it again. Running prisma generate resulted in the same error as originally mention in this issue:
Error: ENOENT: no such file or directory, open '/app/node_modules/.bin/prisma_schema_build_bg.wasm'

I realised that the issue for me was that the link was not being carried over during the archive/unarchive process and instead I would end up with:

Admittedly I was up to shenanigans and in my case simply correcting the archive/unarchive to correctly handle the links corrected this issue.
Perhaps checking the state of the links in node_modules/.bin would be useful debugging step for this issue?

Ran into this issue with AWS Beanstalk. I had to add a step in the deploy process to delete the prisma package and reinstall it.
.ebextensions/prisma.config:
container_commands:
  01_remove_broken_prisma:
    command: "rm -rf node_modules/prisma"
  02_npm_install:
    command: "npm install"
  03_prisma_generate:
    command: "npx prisma generate"
  04_prisma_migrate:
    command: "npx prisma migrate deploy"

to generate prisma binary successfully as I am developing on mac machine and deploying to a linux server so it should be compiled again on deployment

This is not the problem here. The prisma_schema_build_bg.wasm is platform independent.

0.680 Error: ENOENT: no such file or directory, open '/app/node_modules/.bin/prisma_schema_build_bg.wasm'

This error message is curious though - this file is never present at this location. It should only be in node_modules/prisma/build/ where the prisma CLI should read it from. It is highly surprising that npx prisma db pull gets that wrong.
Any idea what could be going on here?

This happened to me when archiving the build. The symlinks in node_modules/.bin where resolved instead of being copied over.


Could you please change the Starrite ore texture?

To be honest, the texture looks childish, especially when compared to the superior-looking ones like unobtainium (so it does not generate?) carmot, mythril, quadrillum. Starrite can use one of these textures if not used by a few more ores already.
I can create a new texture based on the new vanilla ores textures from 1.17 snapshots if you would be interested.
Btw tantalite ore is nearly invisible when spawned in the neighborhood of the stone. The texture is good but the colors could be different.

Textures for ores are currently being worked on by Tector, but due to personal issues he had to step away for a while.
If you want to contribute a new texture I would be very grateful! Starrite being "childish" is an interesting thought considering it is mainly a placeholder texture made from downscaling a star.
Tantalite is also subject to change, as I personally hate the current colour of the ore.

I noticed that Stormyx uses the same texture as Gems from More Gems mod. Maybe Stormyx texture could be rotated by 90/270 or 180 degrees?

Also, what about using official (from 1.17 snapshots) copper texture?

Closing because too cluttered.

Stormyx actually uses the emerald ore texture. It should mostly be fine considering it is in the nether.
I have considered refactoring some of the new ore textures into Mythic Metals. We'll see which ones make it through.

In coppers case it will plainly be removed in the 1.17 version as it is not needed at all. The block textures will be updated to reflect this.


cmake issue

make -j package_rpm
Looks like something wrong with cmake.
RPM build errors:

CMake Error at /usr/local/share/cmake-3.20/Modules/Internal/CPack/CPackRPM.cmake:1816 (message):
RPM package was not generated!
/root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM
Call Stack (most recent call first):
/usr/local/share/cmake-3.20/Modules/Internal/CPack/CPackRPM.cmake:1887 (cpack_rpm_generate_package)
CPack Error: Error while execution CPackRPM.cmake
CPack Error: Problem compressing the directory
CPack Error: Error when generating package: opae
make[3]: *** [CMakeFiles/package_rpm.dir/build.make:71: CMakeFiles/package_rpm] Error 1
make[2]: *** [CMakeFiles/Makefile2:848: CMakeFiles/package_rpm.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:855: CMakeFiles/package_rpm.dir/rule] Error 2
Removed cmake completely.
Rebuild the entire cmake with 3.19.
Scanning dependencies of target opae.io-package
[100%] Built target opae.io-package
[root@centosrt build]# make -j package_rpm
Scanning dependencies of target package_rpm
everything is done.
on Make make -j package_rpm
RPM build errors:

CMake Error at /usr/local/share/cmake-3.19/Modules/Internal/CPack/CPackRPM.cmake:1816 (message):
RPM package was not generated!
/root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM
Call Stack (most recent call first):
/usr/local/share/cmake-3.19/Modules/Internal/CPack/CPackRPM.cmake:1887 (cpack_rpm_generate_package)
CPack Error: Error while execution CPackRPM.cmake
CPack Error: Problem compressing the directory
CPack Error: Error when generating package: opae
make[3]: *** [CMakeFiles/package_rpm.dir/build.make:77: CMakeFiles/package_rpm] Error 1
make[2]: *** [CMakeFiles/Makefile2:957: CMakeFiles/package_rpm.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:964: CMakeFiles/package_rpm.dir/rule] Error 2
make: *** [Makefile:205: package_rpm] Error 2
In the error file, I see this.   and Compiled without Example because HSSI failing to compile
i.e I used cmake  ..  -DCPACK_GENERATOR=RPM -DOPAE_BUILD_SAMPLES=NO
error: File not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/bin/hssi
error: File not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/bin/dummy_afu
error: File not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/bin/hello_fpga
error: Directory not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/src/opae
error: File not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/src/opae/*
File not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/bin/hssi
File not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/bin/dummy_afu
File not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/bin/hello_fpga
Directory not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/src/opae
File not found: /root/build3/opae-sdk/build/_CPack_Packages/Linux/RPM/opae-2.0.1-Linux/usr/src/opae/*

Hi @spyroot, thanks for your query. In order for the RPM build to be successful, as you've encountered, the applications must be built as they are noted in the RPM spec.  For reference, you can have a look at scripts/build-rpms.sh.


recursiveCheckPricelist blowing stack

I guess with big enough price lists, the recursion in the check pricelist is causing an exception. https://github.com/TF2Autobot/tf2autobot/blob/9fddc53823e762f7f1c859afa067005fcdc1b672/src/classes/Listings.ts#L299

Stack trace:
RangeError: Maximum call stack size exceeded
    at String.replace (<anonymous>)
    at interpolate (C:\tf2autobot\node_modules\pluralize\pluralize.js:75:16)
    at C:\tf2autobot\node_modules\pluralize\pluralize.js:89:20
    at String.replace (<anonymous>)
    at replace (C:\tf2autobot\node_modules\pluralize\pluralize.js:88:17)
    at sanitizeWord (C:\tf2autobot\node_modules\pluralize\pluralize.js:119:38)
    at Function.plural (C:\tf2autobot\node_modules\pluralize\pluralize.js:149:14)
    at pluralize (C:\tf2autobot\node_modules\pluralize\pluralize.js:177:46)
    at Currencies.toString (C:\tf2autobot\node_modules\@tf2autobot\tf2-currencies\lib\currencies.js:60:18)
    at replaceDetails (C:\tf2autobot\dist\classes\Listings.js:384:38)
    at Listings.getDetails (C:\tf2autobot\dist\classes\Listings.js:400:23)
    at Listings.getDetails (C:\tf2autobot\dist\classes\Listings.js:440:37)
    at Listings.getDetails (C:\tf2autobot\dist\classes\Listings.js:440:37)
    at Listings.getDetails (C:\tf2autobot\dist\classes\Listings.js:440:37)
    at Listings.getDetails (C:\tf2autobot\dist\classes\Listings.js:440:37)
    at Listings.getDetails (C:\tf2autobot\dist\classes\Listings.js:440:37)

Aiyoo

Had to revert to 4.7.0

For now, please edit your details.buy and details.sell to maybe only have one %name%, and total characters less than 150.
I think this is because the bot called getDetails method two times for each buy and sell order, because the generated details might be more than 200 characters.
v4.8.0 introduced such checking, and since v4.9.0, the archived listings are also counted, so yeah.

Okay I'll give that a try

any good news?

You're welcome. It still needs some optimization. I will try to research it later.

Yeah something like just slice the string if the length is too long and
blerp a warning
Sent via mobile
On Thu, May 5, 2022, 8:59 AM IdiNium @.***> wrote:

You're welcome. It still needs some optimization. I will try to research
it later.
—
Reply to this email directly, view it on GitHub
https://github.com/TF2Autobot/tf2autobot/issues/1095#issuecomment-1118518105,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHZXR7MFMCLM763DQVTHYLVIPA4BANCNFSM5VBZTPMA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Yeah, like that simplicity. I will consider it.


Feature: Franklin's Scaling by bosses defeated

Tested by defeating bosses and confirming the HP, Level, and XP/GP scaled accordingly for all battles.

Merged with 1.3.0 https://github.com/ff6wc/WorldsCollide/pull/64


关于global_sfm分支application编译中colmap问题

作者你好，我按照你给的global_sfm步骤，已成功完成GraphOptim的编译安装，但是在编译application时遇到colmap版本问题，想请问一下作者使用的是哪个版本的colmap。谢谢。

Hi, the compilation of the global_sfm branch is not thoroughly tested (I only tested it on Ubuntu 20.04), therefore I did not merge it into the main branch. I plan to fix the issue this year. I'd like to help you. Before that, could you show me the compilation output?

okay. Would you mind submitting a PR to fix this issue? For example, we can obtain the version number of COLMAP to use different interfaces.

I'm sorry. I don't know how to do. After I reduced the colmap version to 3.4, a new issue occurred. colmap was only supported as a library after 3.5, but 3.5 uses EstimateRelativePose instead of EstimateWithRelativePose, so when compiling the code in the "application", the colmap library could not be found. I don't know how to solve this problem.

Alright. You may try my branch instead: https://github.com/AIBluefisher/colmap .

Keep this issue opening if the problem is still not solved.

作者您好，我使用了你提供的colmap，能够成功对“application”进行编译，然后按你提供的命令（“./applications/run_global_sfm.sh $DATASET_PATH $OUTPUT_PATH $VOC_TREE_PATH $MOST_SIMILAR_IMAGES_NUM”）运行，运行时出现了以下错误，采用的图片是我自己上传的19张图片，MOST_SIMILAR_IMAGES_NUM设置为5

I have sent it to you via email


[Feature] Add MonoFlex Head

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
This PR will add monoflex head, edge_fusion_module, ellip_gaussain_gen.
Modification
This PR will add monoflex head, edge_fusion_module, ellip_gaussain_gen.
BC-breaking (Optional)
no
Use cases (Optional)
n/a
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

The PR is being revised frequently, For now, it's not ready to be reviewed.

After discussion, this PR will be refactored to re-write these pipelines as functions so that it is more convenient to use them in other models.


[New Feature]: DROP DOWN NAV BAR

🎮 feature Request
feature logic and basic description
The nav bar is static, when we scroll down the page the nav bar will also come.
please @Durgesh4993 sir, assign this to me under GSSOC'24.
Point down the features
feature points
it will continuously show the nav bar when we scroll up and down.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@IpsitaPrusty Could you please explain more what you want to do, if possible you can also add screenshot.

@IpsitaPrusty Could you please explain more what you want to do, if possible you can also add screenshot.

Sure @Durgesh4993 sir, There is a navbar on the first page right,

inside the navbar there are some elements so, when we hover on that button it will come with a box for better user's settings and notifications. From user's point of view it will be helpful in different ways:
-Easy access
-organized structure
-interactive elements and so on.
please @Durgesh4993 sir assign this to me.

@Durgesh4993 please assign this to me

plz assign it to me

Hi, I have been following your project and I would love to contribute, can you please assign me this issue.


Move to quarkus-resteasy-client based artifacts

Move to quarkus-resteasy-client based artifacts
There is big rename going on in Quarkus main / Quarkus 3.9 Let's use the new naming and not rely on relocations which can be deleted any day
002-quarkus-all-extensions fail is unfortunately expected, it's under investigation atm.
It seems that GraalVM upgrade is triggering passing full classpath and it fails on https://github.com/quarkusio/quarkus/blob/main/core/deployment/src/main/java/io/quarkus/deployment/recording/BytecodeRecorderImpl.java#L681 for applications with big classpath

So the rename happened already - https://github.com/quarkusio/quarkus/pull/39159


load images in the cluster without pushing them ()

Solved in https://github.com/keptn/lifecycle-toolkit/pull/375


Use explicit file types for module examples

In "The module compiler option" section of the TS docs, it is difficult to understand that each pair of codeblocks represents a source file and built file. It is obvious in the markdown code due to the fact that the language is specified in the code block, but that is not transparent to the reader of the site. For example, the ES... example section has an identical input and output which is confusing for most even given the context of the section it is in or even the page they are on.
This PR attempts to fix that by adding filename comments above each pair of code blocks that specify that it is an input file (.ts) or a built file (.js). The syntax chosen matches the syntax from the "Type-only imports and exports" section on the same page.
As an example, here is the existing ES... section.
import x, { y, z } from "mod";
import * as mod from "mod";
const dynamic = import("mod");
console.log(x, y, z, mod, dynamic);

export const e1 = 0;
export default "default export";

import x, { y, z } from "mod";
import * as mod from "mod";
const dynamic = import("mod");
console.log(x, y, z, mod, dynamic);

export const e1 = 0;
export default "default export";

@microsoft-github-policy-service agree


Package all tests during setup to foster testability of installed Modin

If we're going to ship some tests we better ship all of them, so users would be able to run the full suite after installation without requiring to use source distribution.
cc @anton-malakhov  @devin-petersohn

https://github.com/intel-go/modin/pull/92

@anton-malakhov how come it's fixed if the fix is not in this repo? 😝


updated static download link to latest version

the download link was pointing to an specific version of the MSI, updated the link so that users always get the latest version.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LanaChin PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Investigate Low Level API Building and client.events

Low level api building ruined unit tests and client.events required the use of multi-threading and didn't accomplish what was needed. The work-around implemented simply prints the docker output as it would have if it was run on the CLI.
However, one issue is that on failure, the docker output is not printed out. I don't think currently there is a solution to this issue.

I think I will revisit this later myself. The output it most useful when there is a build failure. I will reassign this to me.

using the events functionality doesn't provide any information, only more information on success.
However the low level api provides accurate error detail on failure.
testing.py has the low level API
testing2.py is the current method in development
(bimage2) jnsproul@RENCI_QDNWVQP2M4 timage % python testing.py
[b'{"stream":"Step 1/2 : FROM non_existent_image:latest"}\r\n{"stream":"\n"}\r'
b'\n',
b'{"errorDetail":{"message":"pull access denied for non_existent_image, reposi'
b"tory does not exist or may require 'docker login': denied: requested access "
b'to the resource is denied"},"error":"pull access denied for non_existent_ima'
b"ge, repository does not exist or may require 'docker login': denied: request"
b'ed access to the resource is denied"}\r\n']
(bimage2) jnsproul@RENCI_QDNWVQP2M4 timage % python testing2.py
Traceback (most recent call last):
File "/Users/jnsproul/Repositories/timage/testing2.py", line 5, in
image = client.images.build(path=".", rm=True, tag='second/v1')
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/jnsproul/miniconda3/envs/bimage2/lib/python3.12/site-packages/docker/models/images.py", line 304, in build
raise BuildError(chunk['error'], result_stream)
docker.errors.BuildError: pull access denied for non_existent_image, repository does not exist or may require 'docker login': denied: requested access to the resource is denied

Events does not add anything useful to output on build failure


dnsmasq metrics: How many addresses are read from each file.

Example output:
Apr 15 16:37:02 dnsmasq[27269]: read /etc/hosts - 3 addresses
Apr 15 16:37:02 dnsmasq[27269]: read /etc/hosts.consul - 1827 addresses

Everytime that's done - a metric should be sent with a tag of filename.

Closed by https://github.com/darron/goshe/commit/b59cf00bf1ec247464bffdbc8b29de88985b4825


Replaced deprecated function from all uses

Forgot to update all files in the previous pull request (https://github.com/dwgoon/sfa/pull/1)

Thanks again :)


New substance misuse question

Context
https://trello.com/c/rgaujgUs/663-health-new-question-did-they-have-any-past-issues-with-substance-misuse-before-custody
E2E PR: https://github.com/ministryofjustice/hmpps-community-accommodation-tier-2-e2e/pull/66
I created this branch from 660 (PR #458) so that should be merged before this is.
Changes in this PR
Screenshots of UI changes
Before

After

Release checklist
As part of our continuous deployment strategy we must ensure that this work is
ready to be released at any point. Before merging to main we must first
confirm:
Pre merge checklist

[ ] Are any changes required to the e2e tests?
[ ] If you've added a new route, have you added a new
auditEvent? (see server/routes/apply.ts for examples)
[ ] Are there environment variables or other infrastructure configuration which needs to be included in this release?
[ ] Are there any data migrations required. Automatic or manual?
[ ] Does this rely on changes being deployed to the CAS API?

Post merge checklist
Once we've merged it will be auto-deployed to the dev environment.
Find the build-and-deploy job in CircleCI.
Should a release fail at any step, you as the author should now lead the work to
fix it as soon as possible.

 Health: new question "Did they have any past issues with substance misuse before custody?"


Write readme stuff for Helm 2 deprecation

Helm 2 is now deprecated and the recommended version is Helm 3.
We need to announce Helm 2 is deprecated for our chart and update our documents to use Helm 3 by default.

https://github.com/Kong/charts/pull/251 is the last remaining breaking change I wanted to get in before 2.0. That PR still includes support for legacy proxy and admin stuff, so it will be released in 1.x and should be available for a couple months before we remove all deprecated stuff (features with NOTES.txt warnings) entirely in 2.x.
Helm does not provide information about its version to the template engine, so we can't add a Helm 2 deprecation warning inside the chart itself; that will only be announced in the changelog. We furthermore don't have immediate plans to make changes that will break Helm 2 support, as templated CRDs will remain (they're necessary for CRD-only releases), albeit defaulted to false. We can use the new Chart.yaml format, which is only supported by Helm 3, but there's no pressing reason to do so.

https://github.com/Kong/charts/pull/251 is the last remaining breaking change I wanted to get in before 2.0. That PR still includes support for legacy proxy and admin stuff, so it will be released in 1.x and should be available for a couple months before we remove all deprecated stuff (features with NOTES.txt warnings) entirely in 2.x.
Helm does not provide information about its version to the template engine, so we can't add a Helm 2 deprecation warning inside the chart itself; that will only be announced in the changelog. We furthermore don't have immediate plans to make changes that will break Helm 2 support, as templated CRDs will remain (they're necessary for CRD-only releases), albeit defaulted to false. We can use the new Chart.yaml format, which is only supported by Helm 3, but there's no pressing reason to do so.

https://github.com/Kong/charts/pull/258 covers the remaining deprecations we want in place before the cutover.

https://github.com/Kong/charts/pull/258 covers the remaining deprecations we want in place before the cutover.

remaining work: #260

remaining work: #260


Assess whether warning icons should use alert role

assess whether or not the alert icons (such as insufficient project points) should use the alert role causing their rendering to interrupt any other text being read by a screenreader.

the warning icons already use the alert role, although the chrome plugin screen reader only seems to read them when the page is initially loaded


Fixed the stucked popover of the

Hi Eduardo.
Thank you for the Great product - the kor-ui !
I had some troubles with the popover of  the <kor-input type="select">, when it is a child of the <kor-modal>, therefore I found and fixed the cause - the popover's listeners were bound to the document, while they should be bound to the kor-input's parent element.
Regards.

Hi Vladmir, thank you so much for the catch and for proposing the solution! I will include this fix on the next patch.


Implement MVP

Description
We want to allow users to create, edit and validate Table Schemas from a web interface.
We have an earlier version of a similar library here, for Data Packages. While we consider this implementation to be a bit flawed, it does show all the moving parts and useful implementation approaches for building out tableschema-ui.
The most important component is the use of json-editor to generate web forms out of JSON Schema files.
Tasks

[ ] Write a short spec on the approach / design
[ ] Create a new Table Schema
[ ] Edit an existing Table Schema
[ ] Validate a Table Schema
[ ] Download as a tableschema.json descriptor
[ ] Tests

@roll there is no requirement to use JSON Editor - the new UI is quite different from a normal form, and JSON Editor is designed around auto-generating "normal" web forms.

@pwalsh
I think It's better to close this repo (at least on this stage) and do all the work in datapackage-ui repo. It will save a lot of time on syncing between repos and of other overhead. For users it will be the same instead of tableschemaUI.EditorSchema there will be datapackageUI.EditorSchema.

https://github.com/frictionlessdata/datapackage-ui/issues/234

MOVED
https://github.com/frictionlessdata/datapackage-ui/issues/234


Rethink default keybinding CTRL + T

The default keybinding for invoking Navigate to definition (ctrl + t) shadows VSCode's keybinding for Go to Symbol in Workspace.
I currently have your Navigate to definition on alt + t and jump back to prev... on alt + shift + t.
Otherwise, great extension!

Thanks. For now, I don't want to change the default that has already been set, and as hinted, it's fairly easy to change default keybindings on a per user basis to suit.
P.S: Personally, I use ctrl+click if I want to use VSCode's symbol navigation.


Flexible rhts test Makefile->Type to tag conversion

The idea is to let user to convert "Type" field from the rhts test Makefile into tags section of main.fmf
This would require some filtering like:

consider ","  "/"  "\s" as dividers
types like: kerneltier1, KernelTier1, kernelTier1 should be converted into KernelTier1
accept "."  "_"  and "-" in the type name, for ex Multihost.1.3 or gre_offload

This feature would eliminate a lot of extra work with setting tags when importing tests from existing repo.

We've shortly discussed this on the hacking session yesterday. The problem is that not everybody needs/want to import the Type. The best solution (for the future) could be to allow users to enable features like this in their config (together with regexp matching rules and similar stuff).
For now the proposed solution is to introduce a new option, let's say --type which would explicitly enable the type -> tag conversion. It could be a list of types which should be converted to corresponding tag. The default would be multihost as that's the current behaviour:
tmt test import --type multihost

Not sure about the lowercase/uppercase/camelcase handling. Probably I would recommend that the desired tag name would be provided in the option and it would catch/match case-insensitive all detected types. So for the @juk's use case it would be:
tmt test import --type KernelTier1

Which would cover kerneltier1, KernelTier1 and kernelTier1 variants. Optionally, a special key could be supported to convert all detected types:
tmt test import --type all

@lukaszachy, @FrNecas, @hegerj, @dkarpele, @juk, what are your thoughts on this?

@juk, ok, thanks for the feedback. Should be quite easy to implement. @lukaszachy, @FrNecas, @hegerj, @dkarpele, anybody willing to look into this? Any other comments or suggestions here?

Hi, I will take a look on this issue.

Hi @psss,
to clarify, I think it should be done like this:
tmt test import = tmt test import --type multihost (current behaviour tmt/convert.py#334)
tmt test import --type = tmt test import --type multihost


[$500] IOU - Edited amount for a paid request does not revert to initial amount after closing error message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.1-7
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:

Navigate to staging.new.expensify.com.
[User A] Request money from User B.
[User A] Open IOU details page > Click Amount.
[User B] Pay the request.
[User A] Enter a new amount and save it.
[User A] Close the error message.

Expected Result:
The amount in IOU details page should revert to the original amount after closing the error message.
Actual Result:
The amount in IOU details page does not revert to the original amount after closing the error message. It only reverts after refreshing the app.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/5965ccbb-5690-4eae-a520-ab2d5e918e45

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01674cdc2bbfceb3e4
Upwork Job ID:<PHONE_NUMBER>277539328
Last Price Increase: 2023-11-28

Proposal
Please re-state the problem that we are trying to solve in this issue.
IOU - Edited amount for a paid request does not revert to initial amount after closing error message
What is the root cause of that problem?
When User B paid the request money, We don't have any logic to dismiss edit amount modal in the User A
What changes do you think we should make in order to solve the problem?
I suggest we should add a condition to check if isSettled is true, we will dismiss amount edit modal or display not found page
What alternative solutions did you explore? (Optional)

Thanks for the proposal @bernhardoj.
Can you please help me understand later part of your proposal. After we revert set to merge, what issue will we face and how your proposal will solve the issue?

@sobitneupane can you please update this?

Whoops!  I got it Greg!

@sobitneupane can you complete the checklist?

Offer sent @sobitneupane can you please accept that as well as complete the above checklist?

Sorry for the delay @slafortune. I am running quite busy. I will try to get to it before weekend.

Thanks for the update @sobitneupane , I'll make sure to come back to this on Monday.

coming back to this Monday.

@sobitneupane will you be able to get this off your plate in the next day or two?

I will complete the checklist by EOD.

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ]  [@sobitneupane] The PR that introduced the bug has been identified. Link to the PR:

https://github.com/Expensify/App/pull/27609

[ ]  [@sobitneupane] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

https://github.com/Expensify/App/pull/27609/files#r1475723305

[ ]  [@sobitneupane] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

[ ]  [@sobitneupane] Determine if we should create a regression test for this bug.

Yes.

[ ]  [@sobitneupane] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Request money from user B
Press the IOU preview twice to open the IOU detail
Press the Amount field to open the edit amount page
From user B, pay the request
Back to user A, edit the amount to any amount
Verify you will get an error message
Verify the amount is reverted to the paid amount

Do we agree 👍 or 👎

[ ] @bernhardoj paid $500 via Upworks
[ ] @sobitneupane is due $500 via NewDot

$500 approved for @sobitneupane based on summary above.


Empty attributes in a  tag are not printed by Element::ToString unless they are in a custom XML namespace

Environment

OS Version: macOS: 13.6.6-x86_64
Source or binary build? source build of sdf14 as of c9779cb7f19bac92b512e213b7fd50dbcf2c8c8d (just after merge of https://github.com/gazebosim/sdformat/pull/1407)

Description

Expected behavior: as of #1407, attributes within a <plugin/> set to an empty string "" can now be parsed without emitting errors, so the empty attributes should be included in the string generated by Element::ToString.
Actual behavior: Only empty attributes in a custom XML namespace will be printed by Element::ToString.

Steps to reproduce

Apply the following patch (from https://github.com/gazebosim/sdformat/pull/1407#issuecomment-2088798546):

diff --git a/test/integration/plugin_attribute.cc b/test/integration/plugin_attribute.cc
index 7f4f9f4c..b66c9da1 100644
--- a/test/integration/plugin_attribute.cc
+++ b/test/integration/plugin_attribute.cc
@@ -65,4 +65,9 @@ TEST(PluginAttribute, ParseAttributes)
     EXPECT_EQ(value->Get<std::string>(""),
               std::string("value"));
   }
+  {
+    auto pluginToString = plugin->ToString("");
+    EXPECT_NE(std::string::npos, pluginToString.find("empty_attribute=''"))
+      << pluginToString;
+  }
 }

Build and run INTEGRATION_plugin_attribute test
Observe test failure

[ RUN      ] PluginAttribute.ParseAttributes
/Users/scpeters/ws/dome/src/sdformat/test/integration/plugin_attribute.cc:70: Failure
Expected: (std::string::npos) != (pluginToString.find("empty_attribute=''")), actual:<PHONE_NUMBER>3709551615 vs<PHONE_NUMBER>3709551615
<plugin name='example' filename='libexample.so'>
  <user attribute='attribute'/>
  <value attribute='attribute'>value</value>
</plugin>

[  FAILED  ] PluginAttribute.ParseAttributes (56 ms)

Output

As I noted in https://github.com/gazebosim/sdformat/pull/1407#issuecomment-2088798546, I don't have a good idea for how to fix this. I'm going to close this as "won't fix", but feel free to comment here if you have an idea and we can reconsider it.


add feature to allow datafusion cli to list columns from a table

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
add feature to allow datafusion cli to list columns from a table
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

closed by #1231


Startup config file not found

Hi,
Is there something specific that I need to do to get the containers to have a persistent config?
I'm have a  csr and vmx image in containerlabs and when I destroy and deploy the lab again, the config is always wiped out to basic one written by netlab.
According to CSR boot trace,
2023-07-07 15:50:12,785: launch     TRACE    Startup config file /config/startup-config.cfg is not found
Other images , i.e EOS, no problems.
Docker version 24.0.3
clab version: 0.42.0
Br,
Heikki

Hi,
yes, a startup-config needs to be provided, but given you're using netlab I am not sure how it is done there.
@jbemmel might know

Netlab is intent based - what's described in the config file gets applied. If Netlab is not (made) aware of pre-existing configs that should override things, it shouldn't leave those in place.
If that happens for EOS, that would be a bug...
To achieve this in general (not tested) you may be able to declare the container as 'unmanaged' and provide a clab.startup-config with the desired configuration

I'm using containerlabs and I used vrnetlab to create the containers for vmx and csr.
I assumed the startup config would be written to a file on the docker host, like with the other "native" containerlab images.
If not, I can save the configs some other way.

@HeikkiLavaste
for VM based images it doesn't do that for these images. If you need your lab to come up in a predefined step then you have to save the intended config and provide it in clab file via startup-config instruction.

Feel free to re-open this issue if new data is available.