CTRL-R history search inserting number* in zsh

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce
I think this is regression as I found this pull request:
https://github.com/junegunn/fzf/pull/2140
$ fzf --version
0.45.0 (202401011)

When i use ctrl-r:

When i select it:

When i select history without * (e.g not shared):

To reproduce simply:
setopt SHARE_HISTORY

then have some shared history (from running command in diff shell) and then use ctrl-r

Looks like reverting this commit fixes it: https://github.com/junegunn/fzf/commit/2671259fdb3094a5eac528f291c1fdadbcf9eda9
But obviously not ideal

Does this patch fix the problem?
diff --git a/shell/key-bindings.zsh b/shell/key-bindings.zsh
index 8a77047..492cac3 100644
--- a/shell/key-bindings.zsh
+++ b/shell/key-bindings.zsh
@@ -103,8 +103,8 @@ fzf-history-widget() {
   local ret=$?
   if [ -n "$selected" ]; then
     num=$(awk '{print $1}' <<< "$selected")
-    if [[ "$num" =~ ^[1-9][0-9]*$ ]]; then
-      zle vi-fetch-history -n $num
+    if [[ "$num" =~ '^[1-9][0-9]*\*?$' ]]; then
+      zle vi-fetch-history -n ${num/\*}
     else # selected is a custom query, not from history
       LBUFFER="$selected"
     fi

/cc @danztran

if [[ "$num" =~ '^[1-9][0-9]*\*?$' ]]; then
+      zle vi-fetch-history -n ${num%\*}

It does:

Thanks, is there any other pattern we should handle?

zle vi-fetch-history -n ${num%*}

Is this truly necessary? In my testing, zle vi-fetch-history appears to handle the optional asterisk correctly. It was primarily the above regex that was overly strict and send the line to the LBUFFER.

Is this truly necessary?

Probably not. But the official document for vi-fetch-history doesn't mention anything about allowing non-numeric suffixes, so I think it's safer to do it even though it's not needed now.

vi-fetch-history (unbound) (G) (unbound)
Fetch the history line specified by the numeric argument. This defaults to the current history line (i.e. the one that isn’t history yet).

Probably not. But the official document for vi-fetch-history doesn't mention anything about
allowing non-numeric suffixes, so I think it's safer to do it even though it's not needed now.

Indeed, you're correct. The argument is definitely intended to be a number.
man 1 zshzle
#  widget [ -n num ] [ -f flag ] [ -Nw ] [ -K keymap ] args ...
# 	Invoke  the  specified widget.  This can only be done when ZLE is active; nor-
# 	mally this will be within a user-defined widget.

# 	With the options -n and -N, the current numeric argument  will  be  saved  and
# 	then  restored  after  the  call to widget; `-n num' sets the numeric argument
# 	temporarily to num, while `-N' sets it to the default, i.e. as if  there  were
# 	none.
# …

Although, the -n argument's parsing appears to take place in the zle_thingy.c file [^1],
vifetchhistory utilizes this argument[^2].  This process uses the Standard C Library function
atoi[^3].
man 3 atoi
# atoi, atoi_l -- convert ASCII string to integer
# …

zmod.mult = atoi(num);

Input
Output

12
12

34*
34

*56
0

-78
-78

+9
9

'hello'
0

[^1]: sourceforge.net - Src/Zle/zle_thingy.c
[^2]: sourceforge.net - Src/Zle/zle_hist.c
[^3]: opensource.apple.com - atoi.c

I am on zsh and still getting the number asterisk pattern on version 0.46.1 when selecting failed commands.
Example:
~
❯ fzf --version
0.46.1 (3c0a6304)

~
❯  1754* cargo test

Is this config related or should it be fixed, as I noticed the commit that addresses this for zsh is tagged 0.46.1?

I am on zsh and still getting the number asterisk pattern on version 0.46.1 when selecting
failed commands.

I couldn't reproduce the issue. Could you please confirm that the correct key-bindings.zsh file is
being sourced from your .zshrc, and that ⌃ Control + R is assigned to
fzf-history-widget?
bindkey | grep -F '^R'
"^R" fzf-history-widget

The function should match the one you see in the link when you run the whence -c … command:
 whence -c fzf-history-widget

https://github.com/junegunn/fzf/blob/1833670fb91dfa17047af320ce7f015e0d500ba6/shell/key-bindings.zsh#L97-L118
The environment variables FZF_CTRL_R_OPTS and FZF_DEFAULT_OPTS are being read for other
configurations. You could also share these assignments.

After confirming all the above steps, let's attempt to debug the issue. Open the key-bindings.zsh
file and enabling xtrace, which prints commands and their arguments as they are executed. Trigger
this by pressing the hotkey combination ⌃ Control + R. This should provide us
with some insight into the problem.
--- a/key-bindings.zsh
+++ b/key-bindings.zsh
@@ -101,6 +101,7 @@ fzf-history-widget() {
   selected="$(fc -rl 1 | awk '{ cmd=$0; sub(/^[ \t]*[0-9]+\**[ \t]+/, "", cmd); if (!seen[cmd]++) print $0 }' |
     FZF_DEFAULT_OPTS="--height ${FZF_TMUX_HEIGHT:-40%} ${FZF_DEFAULT_OPTS-} -n2..,.. --scheme=history --bind=ctrl-r:toggle-sort,ctrl-z:ignore ${FZF_CTRL_R_OPTS-} --query=${(qqq)LBUFFER} +m" $(__fzfcmd))"
   local ret=$?
+  set -o xtrace
   if [ -n "$selected" ]; then
     num=$(awk '{print $1}' <<< "$selected")
     if [[ "$num" =~ '^[1-9][0-9]*\*?$' ]]; then
@@ -110,6 +111,7 @@ fzf-history-widget() {
     fi
   fi
   zle reset-prompt
+  set +o xtrace
   return $ret
 }
 zle     -N            fzf-history-widget

# example result after enabling 'xtrace'
+fzf-history-widget:4> selected=' 8280* cargo test'
+fzf-history-widget:6> local ret=0
+fzf-history-widget:7> [ -n ' 8280* cargo test' ']'
+fzf-history-widget:8> num=+fzf-history-widget:8> awk '{print $1}'
+fzf-history-widget:8> num='8280*'
+fzf-history-widget:9> [[ "$num" -regex-match '^[1-9][0-9]*\*?$' ]]
+fzf-history-widget:10> zle vi-fetch-history -n 8280
+fzf-history-widget:15> zle reset-prompt


[Event Request] Table 11401 "CBG Statement Line" - OnBeforeInitRecordFunction

We would like to request an event in the InitRecord function for the CBG Statement Line. We would like to modify and add additional data when initializing the CBG Statement Line through this function.
LastRecord by var so we can modify lastrecord so that in the end of the function the values are not overwritten.
 procedure InitRecord(LastRecord: Record "CBG Statement Line")
begin
    GetCBGStatementHeader;
    OnBeforeInitRecordFunction(Rec, LastRecord, CBGStatement); //  <<<New event>>>

    "Statement Type" := CBGStatement."Account Type";
    "Statement No." := CBGStatement."Account No.";
    "VAT Type" := "VAT Type"::" ";

    case CBGStatement.Type of
        CBGStatement.Type::Cash:
            if Date = 0D then
                Date := WorkDate;
        CBGStatement.Type::"Bank/Giro":
            if Date = 0D then begin
                if LastRecord.Date = 0D then
                    Date := CBGStatement.Date
                else
                    Date := LastRecord.Date;
            end;
    end;

    "Account Type" := LastRecord."Account Type";
    "Applies-to Doc. Type" := LastRecord."Applies-to Doc. Type";
    "Amount incl. VAT" := LastRecord."Amount incl. VAT";

[IntegrationEvent(false, false)]
local procedure OnBeforeInitRecordFunction(var CBGStatementLine: Record "CBG Statement Line"; var LastRecord: Record "CBG Statement Line"; CBGStatement: Record "CBG Statement");
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 48444.


retry buttons for blog webmentions

...like we currently have for backfeed responses. examples:

https://brid.gy/tumblr/belp.audio
https://brid.gy/tumblr/antipolar.tumblr.com

@chrisaldrich requested this now that we have medium support (#506).


Dynamic functions ignore variants

Description
When describing a stylesheet using dynamic functions, variants are ignored.
The linked repository includes two branches demonstrating the issue on a simple Text component with variants.
In both branches the component is used in the identical way:
import { Text } from '../ui/text'
import { View } from 'react-native'

export default function HomePage() {
  return (
    <View style={{ padding: 70 }}>
      <View>
        <Text variant="title">Title</Text>
      </View>
      <View>
        <Text variant="body">Body</Text>
      </View>
    </View>
  )
}

The main branch contains the component with stylesheets declared as an object as follows:
import { Text as NativeText, TextProps as NativeTextProps } from 'react-native'
import { createStyleSheet, useStyles } from 'react-native-unistyles'

type TextProps = NativeTextProps & {
  variant?: 'body' | 'title'
  color?: 'neutral' | 'primary'
}

export function Text({ variant = 'body', color = 'neutral', ...props }: TextProps) {
  const { styles } = useStyles(stylesheet, {
    variant,
    color,
  })

  return <NativeText {...props} style={styles.root} />
}

const stylesheet = createStyleSheet((theme) => ({
  // NOTE - `root` is an object
  root: {
    fontStyle: 'italic',

    variants: {
      variant: {
        title: {
          fontSize: 90,
        },
        body: {
          fontSize: 18,
        },
      },
      color: {
        neutral: {
          color: theme.colors.neutral,
        },
        primary: {
          color: theme.colors.primary,
        },
      },
    },
  },
}))

The result is as expected. Both texts are italicized. The first one is of a large size, while the second one is smaller:

The dynamic_funcs branch contains the same component, but the stylesheet is defined as a function:
import { Text as NativeText, TextProps as NativeTextProps } from 'react-native'
import { createStyleSheet, useStyles } from 'react-native-unistyles'

type TextProps = NativeTextProps & {
  variant?: 'body' | 'title'
  color?: 'neutral' | 'primary'
}

export function Text({ variant = 'body', color = 'neutral', ...props }: TextProps) {
  const { styles } = useStyles(stylesheet, {
    variant,
    color,
  })

  return <NativeText {...props} style={styles.root()} />
}

const stylesheet = createStyleSheet((theme) => ({
  // NOTE - `root` is a function
  root: () => ({
    fontStyle: 'italic',

    variants: {
      variant: {
        title: {
          fontSize: 90,
        },
        body: {
          fontSize: 18,
        },
      },
      color: {
        neutral: {
          color: theme.colors.neutral,
        },
        primary: {
          color: theme.colors.primary,
        },
      },
    },
  }),
}))

The result of the above code is weird. Both texts are still italic, meaning that the common fontStyle: 'italic' was applied. However, the variants are completely ignored, reverting text sizes and colors to their default values:

Steps to reproduce

Clone https://github.com/b4lk0n/unistyles-dynamic-functions-variants
Install dependencies via yarn
Staying on the main branch, launch the app by yarn ios
Observe the large "Title" and smaller "Body" words, both are italic
git switch dynamic_func
Reload app
Observe the variants styles for the words "Title" and "Body" have gone

Snack or a link to a repository (optional)
https://github.com/b4lk0n/unistyles-dynamic-functions-variants
Unistyles version
2.0.0-rc.2
React Native version
0.72.6
Platforms
iOS
Engine
Hermes
Architecture
Fabric (new)

Thanks for the detailed report, I will take a look 🙏

Published with RC.3

Haha, I immediately knew where the issue was 😃   Thank you for pointing it out. That's the first bug reported for 2.0!


[Event Request] Table 37 "SalesLine" - Procedure "FindResUnitCost" - "OnAfterInitResCostOnFindResUnitCost"

see #6575
Please add a new event raiser to the procedure "FindResUnitCost" of table 37 "Sales Line".
OLD CODE:
local procedure FindResUnitCost()
var
    ResCost: Record "Resource Cost";
begin
    ResCost.Init();
    ResCost.Code := "No.";
    ResCost."Work Type Code" := "Work Type Code";
    CODEUNIT.Run(CODEUNIT::"Resource-Find Cost", ResCost);
    OnAfterFindResUnitCost(Rec, ResCost);
    Validate("Unit Cost (LCY)", ResCost."Unit Cost" * "Qty. per Unit of Measure");
end;

NEW CODE:
local procedure FindResUnitCost()
var
    ResCost: Record "Resource Cost";
begin
    ResCost.Init();
    // start kvspsa - Event Raiser
    OnAfterInitResCostOnFindResUnitCost(Rec, ResCost);
    // stop kvspsa
    ResCost.Code := "No.";
    ResCost."Work Type Code" := "Work Type Code";
    CODEUNIT.Run(CODEUNIT::"Resource-Find Cost", ResCost);
    OnAfterFindResUnitCost(Rec, ResCost);
    Validate("Unit Cost (LCY)", ResCost."Unit Cost" * "Qty. per Unit of Measure");
end;

NEW PUBLISHER:
// start kvspsa - Event Publisher
[IntegrationEvent(false, false)]
local procedure OnAfterInitResCostOnFindResUnitCost(var SalesLine: Record "Sales Line"; var ResourceCost: Record "Resource Cost")
begin
end;
// stop kvspsa

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Mhui show hide fix

Fixes issue with escape stack priority getting all sorts of funky. Doesn't call show/pop as frequently before.

LGTM!


Fix issue with tutorials and add tutorial template

The main issue with rendering tutorials was in publish.js, view.render was expecting a filename property on the tutorialData parameter.

Would be nice to have some sort of tutorial listing on the left.  I will get it added.


Finalize homepage views with data and filters

Title: Finalize homepage views with data and filters
User Stories

As a user, I want to be able to see my posts on the homepage

I expect to see dates, URLs, and plural words properly formatted

completed.


Undocumented/unexplained behavior with account linking and JIT provisioning

Setup
Details are provided in [1].
Issue
I changed the 'Username Claim URI' configuration to 'id', so that username is provisioned with Facebook user ID, as well as the link is always created between IS username and Facebook user ID.
I did federated account linking with my Facebook ID to an account in the PRIMARY user store. 3. 3. However, when I login through Facebook using same ID, a new user account is always provisioned in MYSQL user store but sends back the linked account in PRIMARY user store.
Following message in console is printed:
"[2020-01-05 21:33:32,751]  INFO {org.wso2.carbon.identity.application.authentication.framework.handler.provisioning.impl.DefaultProvisioningHandler} -  An association already exists for user:<PHONE_NUMBER>1684052. Skip association while JIT provisioning"
There seems to be no use case behind this behavior and it seems to be undocumented/unexplained.
[1] "[IAM] Issues and Inconsistencies in Federated Flows" in<EMAIL_ADDRESS>

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!

Must be fixed with https://github.com/wso2-enterprise/iam-product-management/issues/328


Installer Getting Flagged as Virus (Windows)

Describe the bug
I'm trying to install the Depth AI Viewer on Windows. However, Windows Security is flagging the program as a Trojan and preventing the program from being installed. I tried to download the .exe and the .whl and it got flagged both times.
To Reproduce
Steps to reproduce the behavior:

Download and run the installer
Run after install
Gets flagged while installing dependencies

Expected behavior
Not to be flagged as malware
Screenshots
For https://github.com/luxonis/depthai-viewer/releases/download/v0.2.7/depthai_viewer-0.2.7-cp38-abi3-win_amd64.whl

For https://github.com/luxonis/depthai/releases/download/v3.6.0/DepthAI-setup-v3.6.0.exe

I could reproduce it, but hadn't had the time to investigate the issue. Thanks for reporting.

@zrezke I just downloaded the DepthAI installer to check it out before deciding if I would buy a camera. I'm getting the same AV report. It seems to only happen with version 0.2.7. Here's the VirusTotal results: https://www.virustotal.com/gui/file/6bce2a0da8bd933be3d4e840dae0d23e41794bf19cd8241b00b1136d25a3e602
When I scanned the same file in VirusTotal for 0.2.6, I got no hits, so that should narrow down the source.

Actually, I take that back. Running a re-scan on 0.2.6 shows it has issues as well.

Thanks for looking into it! I could narrow it down to 0.1.5 being the first version with this issue. 👀

Fixed the issue with a bump to the rust toolchain. Bumped from 1.74 to 1.76. I think the issue is resolved because of this: https://github.com/rust-lang/rust/pull/118013/
But I don't understand why the depthai-viewer which uses rust version 1.72 isn't flagged (depthai'viewer<=0.1.4).

@zrezke TBH I'm not really sure how that could have caused the issue to be honest, that seems to be about an extra security feature, but the detection was for a specific Trojan (but oddly, Microsoft's public database does not have any info about the one that defender identifies). But I'm not really an expert in this sort of thig.
I personally was leaning towards it either being something accidentally pulled in through a dependency, or a false positive.
On a related note, I'd strongly recommend that you bump the Rust toolchain to at least 1.77.2, as that fixed a 10/10 severity vulnerability. More info here. Though trying to more closely follow the latest release might be even more optimal.

I did a little more digging, and I found that the specific signature that is being triggered was only added to Windows Defender on November 20th, as part of the 1.421.399.0 update, but Microsoft has not yet added any additional details about this specific Trojan, so it may be a legitimate detection for a new virus that was pulled in through a dependency.


Script editor minor change and docs

Q
A

Improvement
yes

What's in this Pull Request?
Added title property for edit mode and more docs on deployment options.

thx @wobba, appreciated. Good clarifications about the usage in noscript scenarios.


Llama 2 7b chat model output quality is low

I have a finetuned llama 2 7B chat model which I am deploying to an endpoint using DJL container. After deploying when I tested the model, the model output quality has degraded (The output seems to be echoing same answer for some questions asked).
Before using DJL container, I was using TGI container and the model was working absolutely fine.
I understand there could be difference in the way of inferencing for both these containers but is there a way of overriding the inference code.
Following is the sample prompt that I am using to prompt the model:
"[INST] <>
Respond only with the answer and do not provide any explanation or additional text. If you don't know the answer to a question, please answer with 'I dont know'.Answer should be as short as possible.
<>
Below context is text extracted from a medical document. Answer the question asked based on the context given.
Context: {text}
Question: {question} [/INST]"
The model is finetuned on the above mentioned prompt so we need to inference in such a way that it comprehends this format of the prompt and gives the answer.
Any resources/suggestions would be really helpful.

Could you provide your deployment config? Trying to help here. Logs will also help

I had used a serving.properties file which has the following configurations
1 engine=MPI
2	option.task=text-generation
3	option.trust_remote_code=true
4	option.tensor_parallel_degree=1
5	option.model_id={{model_id}}
6	option.dtype=fp16
7	option.tgi_compat=true
8	option.rolling_batch=lmi-dist
My endpoint config is very simple:
{
"VariantName": "variant1",
"ModelName": model_name,
"InstanceType": "ml.g5.24xlarge",
"InitialInstanceCount": 1,
"ModelDataDownloadTimeoutInSeconds": 3600,
"ContainerStartupHealthCheckTimeoutInSeconds": 3600,
}
Also please note here, I am not facing any errors while deploying, the deployment is successful but the output formats are different.
Expected output according to the DJL documentation for TGI compatible output feature:
[
{
"generated_text": "Deep Learning is a really cool field"
}
]
What I am getting:
{
"generated_text": "Deep Learning is a really cool field"
}
ALso the quality of output degraded significantly with DJL container as compared to TGI container

could you share a sample prompt you use and parameters? And exepcted output if possbile?

I have mentioned the sample prompt in the issue description. Mentioning below again for reference:
"""[INST] <>
Respond only with the answer and do not provide any explanation or additional text. If you don't know the answer to a question, please answer with 'I dont know'.Answer should be as short as possible.
<>
Below context is text extracted from a medical document. Answer the question asked based on the context given.
Context: {text}
Question: {question} [/INST]"""
Expected output
if question is: What is patient name ?
Model response : [{'generated_text : 'John H'}]
I am using a fine tuned model which is trained on the above mentioned format of prompt and answer


Update role field in AddMember UI

Indeed we can use the KRoleField

Fixed with https://github.com/kalisio/kdk/commit/986072c16a52bb682463199fa2e0171125539632


[andriod] Invalidate SvgView when view hierarchy changes

Problem
Dynamically added children does not shown on Svg view.
Steps to reproduce
import React from 'react'
import { Button } from 'react-native';
import Svg, { Line } from 'react-native-svg'

class Puzzle extends React.Component<{}, {data: number[], current: number}> {
  state = { data: [], current: 1 }

  onAdd = () => this.setState(state => ({
    data: [...state.data, state.current],
    current: state.current + 1,
  }));

  render() {
    return (
      <View style={{ flex: 1 }}>
        <Svg width={100} height={100}>
          {
            this.state.data.map(p => (
              <Line
                key={p}
                x1={0}
                y1={0}
                x2={50}
                y2={p * 10}
                storkeWidth={8}
                stroke="#000"
              />
            )
          }
        </Svg>
        <Button title="add" onPress={this.onAdd} />
      </View>
    )
  }
}

It renders no line even I clicked button.
DOM tree has no problem,

But no lines are visible.

Expected behavior
Should show 4 lines.

It only happens when I wrote without dynamically adding children components.
Code below works fine.
import React from 'react'
import { Button } from 'react-native';
import Svg, { Line } from 'react-native-svg'

class Puzzle extends React.Component<{}, {data: number[], current: number}> {
  state = { data: [], current: 1 }

  onAdd = () => this.setState(state => ({ data: [...state.data, state.current], current: state.current + 1 }));

  render() {
    return (
      <View style={{ flex: 1 }}>
        <Svg width={100} height={100}>
          <Line x1={0} y1={0} x2={50} y2={10} storkeWidth={8} stroke="#000" />
          <Line x1={0} y1={0} x2={50} y2={20} storkeWidth={8} stroke="#000" />
          <Line x1={0} y1={0} x2={50} y2={30} storkeWidth={8} stroke="#000" />
          <Line x1={0} y1={0} x2={50} y2={40} storkeWidth={8} stroke="#000" />
        </Svg>
        <Button title="add" onPress={this.onAdd} />
      </View>
    )
  }
}

Solution
Invalidate parent SvgView when view hierarchy changes.

@mu29  Thanks for this! Haven't had time to test this yet, also I notice you have "storkeWidth" instead of "strokeWidth" could this be related to the issue as well?

Ah ok, it's fine as is. Hopefully I'll have some time this weekend for looking at this.

Thank you for your care 😊 Have a good weekend!


Creating and removing shapes doesnt remove the WebGL Context

I have thousands of polygons i am rendering, and i need to be able to show/hide them and also update the opacity of them. Currently i am creating them any time an update needs to happen like this:
let glifyPolygons = glify.shapes({ 				map: map, 				data: colouredGrids, 				click: (e: any, feature: GeoJSON.Feature) => { 					handleGridClick(feature); 				}, 				color: (index: any, shape: any) => hexToRgb(shape.properties.color, transparency[0] / 100), 			});
Then any time a i need to hide or change the opacity, i remove it:
glifyPolygons.remove()
then redraw the polygons with the new opacity. However when i show/hide the polygons alot or update the opacity too frequently I get a warning in the console:
WARNING: Too many active WebGL contexts. Oldest context will be lost.
Then other shapes i am drawing with glify start to be deleted. It seems like the WebGL contexts arent being removed.  I found a quick solution to show/hide by  doing this:
if (layer && !state) { 		layer.remove(); 	} else if (layer && state) { 		layer.addTo(map); 	}
but the polygon data can also change so i need to be able to create new shape objects.
How can i fix this?

duplicate of #24 ?
related  to #67

@jackbarrie1341 as indicated above, this is a duplicate, so I will close it to keep the issue list clean.
Check out this hook I made to streamline the updating of your glify layers.


Upgrading Ramen (hub, MC)

Are there any operator prerequisites in code?
(from HackMD list)
Also marked as "get help"

Closing as stale.


feat(DTFS2-6715): migrate get team members in tfm to rest

Introduction
We want to replace TFM API's TeamMembers GraphQL query with a REST endpoint.
Resolution
In TFM API, I've created a new endpoint (GET /v1/teams/{teamid}/members) to get members of a team.
This validates that {teamId} is one of the allowed team ids, then calls DTFS Central API's GET /v1/users/team/{teamId} and returns the _id, firstName, and lastName of the results.
I have changed TFM UI to use the new rest endpoint instead of the GraphQL query, and tested the UI pages that use this functionality still work (I've commented this functionality on the ticket)

Codecov Report

Merging #2147 (a226c0b) into feat/DTFS2-6182/replace-graphql-query-facility-in-tfm-with-rest (53d6661) will increase coverage by 1.04%.
The diff coverage is 30.00%.

@@                                         Coverage Diff                                         @@
##           feat/DTFS2-6182/replace-graphql-query-facility-in-tfm-with-rest    #2147      +/-   ##
===================================================================================================
+ Coverage                                                            19.75%   20.79%   +1.04%
===================================================================================================
  Files                                                                 1238     1135     -103
  Lines                                                                28748    27308    -1440
  Branches                                                              4545     4186     -359
===================================================================================================
+ Hits                                                                  5678     5680       +2
+ Misses                                                               20248    19019    -1229
+ Partials                                                              2822     2609     -213

Flag
Coverage Δ

api-test
?

ui-test
65.39% <85.71%> (-0.01%)
:arrow_down:

unit-test
2.27% <13.04%> (+0.05%)
:arrow_up:

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

Files
Coverage Δ

trade-finance-manager-api/src/v1/swagger.js
0.00% <ø> (ø)

trade-finance-manager-api/src/v1/teams/teams.js
100.00% <100.00%> (ø)

...lers/case/amendments/leadUnderwriter.controller.js
97.82% <100.00%> (ø)

...inance-manager-ui/server/controllers/case/index.js
57.67% <100.00%> (ø)

...ollers/case/underwriting/lead-underwriter/index.js
100.00% <100.00%> (ø)

...llers/case/amendments/amendmentTasks.controller.js
22.22% <0.00%> (ø)

trade-finance-manager-api/src/v1/routes.js
0.00% <0.00%> (ø)

...nance-manager-api/src/v1/validation/validateIds.js
0.00% <0.00%> (ø)

trade-finance-manager-api/src/v1/teams/routes.js
0.00% <0.00%> (ø)

... and 105 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


broken homeworld selection in world list when leaving a room via alt+L

leave any room via alt+L
notice homeworld is always selected after leaving any room
click Load World, notice nothing happens
select any other room, Load World now works again

I consider this a feature more than a bug :smile:

I consider this a feature more than a bug smile

Until you realize that "Load World" tries to load some non-existent world. 👀


Generators can return undefined values

Summary
Should the return type of yield* a be TReturn | undefined when in strict mode as opposed to TReturn (where TReturn is the value of the TReturn type argument to a's Iterable type instantiation)?
Or should the type of IteratorReturnResult.value be changed to TReturn | undefined when in strict mode (and potentially the type of Generator.return(value: TReturn) be relaxed to Generator.return(value?: TReturn))?
Have you considered these scenarios? (A github microsoft site search for "generator return undefined" yields no results).
Background
Generators will return undefined values when:

next() is called after the generator has already returned its return value:

function* gen(): Generator<number, string> {
    yield 42;
    return "done";
}

const a = gen();

console.log(a.next()); // { value: 42, done: false }
console.log(a.next()); // { value: 'done', done: true }
console.log(a.next()); // { value: undefined, done: true }

2.1) the generator is interrupted via a break statement of for-of loop on the generator
const nestedIterableIterator: IterableIterator<string, string, unknown> = {
    [Symbol.iterator]() { return this; },

    next() {
        return { value: "42", done: false }
    },

    return(value: string) {
        console.log("return called with", value);
        return { value, done: true }
    }
}

function* generator() {
    yield* nestedIterableIterator;
    return "never";
}

const it = generator();

it.next();
it.return("passthrough"); // return called with passthrough

for (const x of generator()) {
    break; // return called with undefined
}

2.2) the generator is manually interrupted by calling its return method without arguments:
function* gen(): Generator<number, string> {
    yield 42;
    yield 43;
    return "done";
}

const a = gen();

console.log(a.next()); // { value: 42, done: false }
console.log(a.return(undefined as unknown as string)); // { value: undefined, done: true }
console.log(a.next()); // { value: undefined, done: true }

The generator is attempted to be iterated over using yield* after it has already returned:

function* gen(): Generator<number, string> {
    return "done";
}

function* consumer() {
    const a = gen();

    console.log(yield* a); // done
    console.log(yield* a); // undefined
}

consumer().next();

Effect
The type signature of Generator.return guards against 2.2, and there is no way to get the return value from a for-of loop.
That leaves scenarios 1 and 3, of which 1 is likely uncommon, 3 is probably the only viable scenario and only in the case of user error...
⚙ Compilation target
ES2015
⚙ Library
lib.es2015.generator.d.ts
Missing / Incorrect Definition
Generator.return and IteratorReturnResult.value
Sample Code
See above.
Documentation Link
No relevant documentation.

The same obviously applies in an equivalent manner to AsyncGenerator


Resource path fails with namespaced models and resources.

I'm trying to use JSONAPI in an engine and I've defined DokiCore::Tenant (the model) in doki_core/app/models/tenant.rb and DokiCore::TenantResource in doki_core/app/resources/tenant_resource.rb. When I attempt to serialize to hash, I encounter the following error:

NoMethodError: undefined method tenant_path' for #<Module:0x007f9d04208778> from /Users/typeoneerror/.rvm/gems/ruby-2.2.2@doki/gems/jsonapi-resources-0.6.1/lib/jsonapi/link_builder.rb:77:in public_send'

The resource uses model_name to let it know where the model actually is:
module DokiCore
  class TenantResource < JSONAPI::Resource
    model_name 'DokiCore::Tenant'
  end
end

I'm trying to output the hash for a tenant like so:
tenant = DokiCore::Tenant.find(1);
resource = DokiCore::TenantResource.new(tenant, nil);
serializer = JSONAPI::ResourceSerializer.new(DokiCore::TenantResource);
serializer.serialize_to_hash(resource);

which is where the error happens.
How can I get the links to work correctly and/or disable them? I assume this is there it adds the URL to the resource as a link under the "links" key in the outputted json.

Should note that I am not using jsonapi_resource in routes or the controllers yet. I just wanted to serialize to the json-api format. Is this not possible at the moment? Will I have to transition all my rails stuff to use the routes and controllers?

Oh, I think I sorted this out. My API is namespaced in my routes so I had to apply the same namespace to my resources...
tenant = DokiCore::Tenant.find(1);
resource = DokiCore::API::V1::TenantResource.new(tenant, nil);
serializer = JSONAPI::ResourceSerializer.new(DokiCore::API::V1::TenantResource);
serializer.serialize_to_hash(resource);

Sorry for the fault report!


Can't unpublished notes by Digital Garden Center

When I open the Digital Garden Center, there's no list of published notes and unpublished notes. So I don't know how to unpublished the notes without delete the notes directly in Github Repos.
Could you please guide me about this. Very appreciate for it.

Just to make sure, you are clicking the plus button to the left of the "Published" header? Can you send a screenshot?

Yeah, I can't see any notes in the center, and when I click on the button of "Delete notes from garden", there's nothing happened (I have removed the dg-publish: true attribute in the fronmatter). So I am not sure whether it's due to my incorrect operation.
Thank you

Hmm ,weird 🤔 Haven't seen this before, so I'm not sure what's going on. Could you open the developer tools in Obsidian, open the console, and see if any errors appear when opening the publication center? Or when you are expanding any of the lists?
Here's a GIF of how to find the console, in case you haven't opened it before:

Sorry for I can't understand these information, so I uploaded the picture. Please check the picture. Thank you!

No worries, a picture was what I needed. Just forgot to write it 😅
Hmm okay. Seems the plugin can't find your repo. Can you verify that your GitHub repo name and your username in the digital garden settings are correct? And that they don't contain any spaces before or after them?

Yes, I think it is right. Because I can use the command in obsidian to publish the notes and I can see them in my github repo. 🤔

I think I’ve found the issue. When the publication center checks your files in github, it assumes your branch name is «main». I see that your branch is named «master». I’ll hopefully get a fix out later today, so that you may use whatever name you prefer. The plugin should just look at whatever you have set as default.

It works now! So wonderful! Thank you very much!

🎉


vocab_size for char embed?

Hi thanks for sharing.
From the paper, it seems they are doing a character level embedding. but from running main.py, i got a word level model instead? Not sure whether this is consistent with the paper specification?
209  	        model.zero_grad()
210  ->	        output = model(data, target_mask)
211  	        loss, last_loss = model.criterion(output, target)

data.max()
# tensor(32647, device='cuda:0')
model.vocab_size
# 33278

I am no longer developing this repository. If you get consistent results with character level, feel free to send pull request


Cantaloupe sprint 2 staging

https://github.com/TAMULib/IRIIIFService/pull/151
https://github.com/TAMULib/IRIIIFService/pull/152

Note: The partial revert removes part of the commit 41731b8dc41b20cbf61fdf0e440e18bb1e933f34.
Should any problems be discovered during QA, then it may be a good idea to fully revert commit 41731b8dc41b20cbf61fdf0e440e18bb1e933f34 and then test the QA process again.
Alternatively, a more conservative approach by just merging the more conservative changes could also be done. This would also preserve the unit tests.
Here is a conservative changes branch as an alternative to this here:

https://github.com/TAMULib/IRIIIFService/tree/cantaloupe_sprint_2_staging-conservative_changes
Commit hash: 2c3f651d7a269721f636d7c89e2ef46639ab8a9c

Hey all, adding some thoughts to try and bring context and a bit of history so this can get merged and we have good history when we pick this up in the future.
Let's start with some manifests for items that were not problematic before Kevin's work.  An example is this. If we curl and diff that resource with the original image and an image based on this PR, the results are exactly the same.  Another one is this.  If we do the same thing, the diff between both resources is exactly the same.
What does that show?  We don't have any regressions that would impact things that are currently working.
Next, let's look at a problematic manifest response.  Originally, an HTTP Get gave us this:
RDF not found! https://api-dev.library.tamu.edu/fcrepo/rest/3b/6f/c3/25/3b6fc325-f6ca-41d8-b91e-8c5db3be8c13/basbanes-exhibit-texts-todd-magpietest_objects/15/pages/page_0/files/Basbanes-Grolier%20talk_1.jpg/fcr:metadata

After the work here, we get:
{
  "@context" : "http://iiif.io/api/presentation/2/context.json",
  "@id" : "https://api-dev.library.tamu.edu/iiif-service/fedora/presentation/3b/6f/c3/25/3b6fc325-f6ca-41d8-b91e-8c5db3be8c13/basbanes-exhibit-texts-todd-magpietest_objects/15",
  "@type" : "sc:Manifest",
  "attribution" : [ "Basbanes, Nicholas A., 1943-", "in copyright - Educational Use Permitted; For more information see:  http://rightsstatements.org/vocab/InC-EDU/1.0/" ],
  "description" : [ "Annotated typescript of Basbanes's lecture, \"Brandy for Heroes,\" delivered as part of the Grolier Club's \"The Collector & the Bookseller: Fellowship of the Book\" symposium, April 19, 1995.", "From the professional archives of Nicholas A. Basbanes, now held by Cushing Memorial Library & Archives, Texas A&M University." ],
  "label" : "Annotated typescript draft of lecture to Grolier Club",
  "logo" : "https://library.tamu.edu/assets/images/tamu-logos/TAM-PrimaryMarkB.png",
  "metadata" : [ {
    "label" : "creator",
    "value" : "Basbanes, Nicholas A., 1943-"
  }, {
    "label" : "subject",
    "value" : [ "Grolier Club", "Book collectors", "Book collecting", "Booksellers and bookselling", "Lectures" ]
  }, {
    "label" : "rights",
    "value" : "in copyright - Educational Use Permitted; For more information see:  http://rightsstatements.org/vocab/InC-EDU/1.0/"
  }, {
    "label" : "format",
    "value" : "reformatted digital"
  }, {
    "label" : "language",
    "value" : "eng"
  }, {
    "label" : "type",
    "value" : "Lectures"
  }, {
    "label" : "title",
    "value" : "Annotated typescript draft of lecture to Grolier Club"
  }, {
    "label" : "created",
    "value" : "1995"
  }, {
    "label" : "abstract",
    "value" : "Annotated typescript of Basbanes's lecture, \"Brandy for Heroes,\" delivered as part of the Grolier Club's \"The Collector & the Bookseller: Fellowship of the Book\" symposium, April 19, 1995."
  }, {
    "label" : "type",
    "value" : "Text"
  }, {
    "label" : "context",
    "value" : "3b/6f/c3/25/3b6fc325-f6ca-41d8-b91e-8c5db3be8c13/basbanes-exhibit-texts-todd-magpietest_objects/15"
  } ],
  "sequences" : [ {
    "@id" : "https://api-dev.library.tamu.edu/iiif-service/fedora/sequence/3b/6f/c3/25/3b6fc325-f6ca-41d8-b91e-8c5db3be8c13/basbanes-exhibit-texts-todd-magpietest_objects/15",
    "@type" : "sc:Sequence",
    "canvases" : [ ],
    "label" : "Annotated typescript draft of lecture to Grolier Club"
  } ]
}

Is this valid? No, but it is JSON-LD which is a big improvement. It's not valid according to Tripoli because:
* 'canvases' MUST have at least one entry @ data['sequences'][0]['canvases']
* manifest SHOULD have thumbnail field. @ data['thumbnail']

This makes sense.  My two cents is that we merge this and keep this history around to remember how far we made it through this work.


main and improved circuit methods

Merged into my master which is in synch with upstream master, then new PR.

Will release tonight.


Prevent the user from decreasing the Opportunity amount to an amount that is less than the sum of paid payment amounts.

This branch is a rename of an older branch. This is the previous PR:
https://github.com/SalesforceFoundation/Cumulus/pull/4380
Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

**lurch: attach W-033109

Tracking W-033109


Create LOGO for 10secondsofcode

Create LOGO for 10secondsofcode

Create the logo for 10secondsofcode and and it will bring the meaning of 10seconds understanding the concepts.
Use your creativity and design.

Mind if I jump on this?

I added the logo

Mind if I jump on this?

Sure....take it

Hello 🦊 I create a logo for your project with a meaning of "fast", hope you like!

Hello  I create a logo for your project with a meaning of "fast", hope you like!

@mandyellow It is looking nice and if possible could you please change into https://10seconds.netlify.com theme color like BG - #6156B5 and font color as white..

Yeah @elangovanshanthi, no problem! If the logo it's okay, I can upload a PNG and SVG version for you. Just tell me where I should upload 😄

Nice :) Inside logo folder and we will use it. thanks

Done! :happy:


Not able to generate QR code

I have deployed this repo on our stage server initially it was working but now it is not working. I am not able to generate QR code start-session api is going infinite. in my local i took pull and after npm run dev there was a code to install new dependencies in console. after i done that it started working. so can you please provide that npm code to install new dependencies?

Install the operating system dependencies and also the Google Chrome browser.
Note: use node18 or version 18 and remove old sessions and create a new one.


Request for separate zoom account for Jenkins

Jenkins community has been sharing the CDF zoom account but now we are at a point where with more CDF meetings and Jenkins working groups there is more overlap in schedules. e.g CDF MLOps SIG and Jenkins advocacy and outreach meeting often overlap - whoever gets there first can run their meeting.
Now would be great to set up Jenkins with a separate way to run zoom that can be used for:

Jenkins GSoC meetings (ramping up)
Advocacy & outreach SIG
Pipeline authoring SIG
Docs SIG
Platform SIG
Chinese internationalization sig, etc

The Jenkins governing board can manage how access is shared, am happy to be point of contact to help bootstrap this.

Hi @danlopez00 . Any updates on that?

Completed. @oleg-nenashev and @tracymiranda I will send creds over different medium.


Storm should have efficient serialization for nested data structures

e.g., lists, maps, etc.

I hope that Kryo will solve this issue as well. Issue for Kryo open here:
https://github.com/nathanmarz/storm/issues/40

Implemented for 0.6.0


Publish new crate versions

Description of change
This is to get the recent CRT fixes out for the PyTorch connector. I
also bumped the mountpoint-s3-crt-sys crate version to be in lockstep
with mountpoint-s3-crt.
Does this change impact existing behavior?
No.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

I'm pretty sure the crate check workflow is just broken — it doesn't work when changing the versions of multiple crates in one PR. Here's the output we actually care about:
[19:20:30] ~/c/mountpoint-s3 (crate-release-240307|✔) $ cargo package --no-verify -p mountpoint-s3-crt-sys
   Packaging mountpoint-s3-crt-sys v0.6.2 (/local/home/bornholt/code/mountpoint-s3/mountpoint-s3-crt-sys)
    Packaged 2804 files, 50.7MiB (9.9MiB compressed)


Type declaration, f-string use, merge ENT functions, refactoring

pronotepy\ent.py:

added type for args
updated log string with the use of f-string
added/reworked a function called ent() that merges 3 ENT

pronotepy\pronoteAPI.py:

updated log string with the use of f-string
refactored some if else statement and generator

Should we properly close the requests.Session()?
I wanted to put them in with blocks, but I don't know if any function use the returned Object to make other actions on it.
Let me know, I can easily indent them in with block, so they are properly closed after use.

Should be good.
Another point to see: https://github.com/bain3/pronotepy/pull/104#issuecomment-1060608474 can be done in another PR

Oh yes I forgot about that. I'm not sure what happens to the cookies though... If they stay there then we can close them for sure.

I'm not sure what you mean by that. You can get the data parameter through inspecting the network traffic when you do stuff on the demo(/your) pronote. Compression and encryption was done later on because someone needed it.
Oh and yeah we should do something about the cookiejar_from_dict functions everywhere. I don't think we need them and can just use session.cookies, but I have no way to check if it still functions so I have not touched it.

@TheGeeKing It don't close #56 Because some others ENTs have the same system

@TheGeeKing It don't close #56 Because some others ENTs have the same system

I will update them soon.

As you want I thought of doing it but if you prefer to do it, do it. @TheGeeKing Warn me please so that I don't do something uselessly.

@TheGeeKing I have another idea.  It's a pretty big job.  So we could report here when we find a resemblance between several ENTs.  And then we divide the tasks according to everyone's availability because we both seem rather active.

Because I thought that Xiloe do the work for ENT but maybe he has not a lot of time.

@TheGeeKing I have another idea.  It's a pretty big job.  So we could report here when we find a resemblance between several ENTs.  And then we divide the tasks according to everyone's availability because we both seem rather active.

Are you speaking about #101 ? I took a look at it yesterday but will need to take more time to understand everything.
I can find some free time to work on things but I can't guarantee anything.

@TheGeeKing J'ai une autre idée. C'est un gros boulot. On pourrait donc signaler ici quand on trouve une ressemblance entre plusieurs ENT. Et puis on se répartit les tâches selon les disponibilités de chacun car nous semblons tous les deux plutôt actifs.

Tu parles du #101 ? J'y ai jeté un œil hier mais il faudra prendre plus de temps pour tout comprendre.
Je peux trouver du temps libre pour travailler sur des choses mais je ne peux rien garantir.

No I speak about ENT.  I'm working alone for the moment on the QR code and I would prefer that we don't make duplicates😭.  I understand for free time it's the same for me that's why I propose to share the tasks of ENT.  But for the QR code I take care of it.

Nice translate. Haha. So what's your idea ?

@TheGeeKing My idea is to to bring together ENTs that have ressemblance. So we say here when we find ressemblance and then we say who code the function to bring together ENTs

It was the goal since the start no?

Yes but I believed that you didn't understand

I just came on this thread to offer my help initially 😅


pycubelut

hello
Thank you for the program.
I download
pycubelut, but I see that is a .exe file with a nice help,
it is posible import the class or function inside another py file or only it is a exe file.
At the moment, as the luts are thought to be unprotected, we are looking at how to supply the luts in an array of luts
reggards

Hello,
You can use the following python commands once you install the package.

import pycubelut
mylut = CubeLUT("test.cube")
pycubelut.process_image("image.jpeg", "output/folder", 0, mylut, False)

hello thank you for all I will check.
And it is posible recover the cube in array and stay in memory .
it is posible declare like a variable. I will llike create a .py that have all the luts and import to another python that process the image
It is numpy?
LUT_3D_SIZE 32

0.0980 0.0392 0.0863
0.1176 0.0471 0.0941
0.1255 0.0471 0.0941
0.1451 0.0510 0.0980
0.1647 0.0549 0.0980
0.1725 0.0549 0.0980
0.1922 0.0588 0.0980
0.2196 0.0588 0.1059
0.2314 0.0588 0.0980
0.2471 0.0627 0.1.....

mylut = pycubelut.CubeLUT("test.cube")
pycubelut.CubeLUT that is waiting to receive, how can I create the object that is waiting to receive.
reggards

hello,
I put the lut inside a position in array  like this.
import pycubelut

luts=[["""0.0980 0.0392 0.0863
0.1176 0.0471 0.0941
0.1255 0.0471 0.0941
0.1451 0.0510 0.0980
0.1647 0.0549 0.0980
.....
1.0000 0.9725 0.9490
1.0000 0.9725 0.9490
0.9961 0.9765 0.9490
1.0000 0.9765 0.9529"""]]

mylut = CubeLUT(luts[0])
pycubelut.process_image("C:\Users\video-boda\Documents\photofriends\hot_folder\14062015-_DSC0825.jpg", "C:\Users\video-boda\Documents\photofriends\hot_folder\14062015-_DSC0825_prueba.jpg", 0, mylut, False)

But this not works say me
SyntaxError: (unicode error) 'unicodeescape' codec can't decode bytes in position 2-3: truncated \UXXXXXXXX escape
Any method to realease this?
Reggards

You will need to change the following from:
with open(lut_path, 'r') as lut_file:
            for line in lut_file.readlines():

to
      for line in lut_file:

and fix all the indenting for the function.
Also, manually set the size of the lut somewhere in that function with:
self.size = <the integer you want here>

Hello, I change the code but have problems to execute in multithread. This not works
"""pycubelut v0.2 === Author === Yoonsik Park<EMAIL_ADDRESS>=== Description === A library and standalone tool to apply Adobe Cube
LUTs to common image formats. This supports applying multiple LUTs and
batch image processing. The CubeLUT class has complete Cube LUT parsing
and transform capabilities. """

import logging
import numpy as np
from colour.algebra.interpolation import table_interpolation_tetrahedral, table_interpolation_trilinear
from scipy.interpolate import RegularGridInterpolator
import os
from multiprocessing import Pool
import todos_luts
from PIL import Image
import ctypes
from multiprocessing import Process, Manager
global pid_dic
pid_dic={}
from winreg import *
import threading

class CubeLUT:
	def __init__(self, lut_path):
		self.data = None
		self.dim = 0
		self.size = 0
		self.title = ""
		self.filename =""
		self.domain_min = (0., 0., 0.)
		self.domain_max = (1., 1., 1.)
		self._read_cube_lut(lut_path)

	def _read_cube_lut(self, lut_path):
		"""HE MODIFICADO ESTO PAAR QUE COJA TEXTOS"""
		logging.debug("Importing lut: " + lut_path[1])
		self.filename = lut_path[0]
		print (self.filename)
		lineas=lut_path[1].splitlines()
		lut_data = []
		for x in lineas:
			if x == '#' or x.strip() == '':
				# skip comment/empty line
				continue
			if "TITLE" in x:
				self.title = ' '.join(x.split()[1:])
			elif "DOMAIN_MIN" in x:
				self.domain_min = tuple(float(x) for x in x.split()[1:])
			elif "DOMAIN_MAX" in x:
				self.domain_max = tuple(float(x) for x in x.split()[1:])
			elif "LUT_1D_SIZE" in x:
				self.dim = 1
				self.size = int(x.split()[1])
				raise Exception('1D not supported: ' + self.filename)
				# assert self.size <= 65536 and self.size >= 2
			elif "LUT_3D_SIZE" in x:
				self.dim = 3
				self.size = int(x.split()[1])
				# assert self.size <= 256 and self.size >= 2
			else:
				lut_data.append([float(num) for num in x.split()])
		# convert self.data to numpy format
		self.data = np.array(lut_data)
		x = self.size
		self.data = self.data.reshape([x, x, x, 3], order='F')
		if self.domain_min != (0., 0., 0.) or self.domain_max != (1., 1., 1.):
			logging.warning("nonstandard domain for LUT, output may be wrong")

	def transform_tetrahedral(self, image, in_place=False):
		"""Returns the transformed <image>, using the LUT from <self.data>.
		This uses tetrahedral interpolation, which is more accurate and
		slower than trilinear interpolation. If <in_place> is True, the
		<image> array will be directly modified.

		<image> is the 2D RGB array (i.e. 3D array)
		<in_place> specifies if <image> should be transformed directly
		"""
		if not in_place:
		    image = np.copy(image)

		for i, row in enumerate(image):
		    image[i] = table_interpolation_tetrahedral(row, self.data)
		return image

	def transform_trilinear(self, image, in_place=False):
		"""Returns the transformed <image>, using the LUT from <self.data>.
		This uses trilinear interpolation, which is faster than tetrahedral
		interpolation. If <in_place> is True, the <image> array will be
		directly modified.

		<image> is the 2D RGB array (i.e. 3D array)
		<in_place> specifies if <image> should be transformed directly
		"""
		if not in_place:
			image = np.copy(image)

		for i, row in enumerate(image):
			print (i)
			image[i] = table_interpolation_trilinear(row, self.data)
		return image

	def _buggy_transform_trilinear(self, image, in_place=False):
		"""Trilinear interpolation using scipy interpolation methods. Slower
		than the <transform_trilinear> method, and has bugs.

		<image> is the 2D RGB array (i.e. 3D array)
		<in_place> specifies if <image> should be transformed directly
		"""
		if not in_place:
		    image = np.copy(image)

		# save image shape for later
		orig_image_shape = image.shape
		# change 2D RGB -> 1D RGB
		image = image.reshape([-1, 3])
		# equally spaced grid from 0 to 1
		x = np.linspace(0, 1, self.size)
		# for RGB channels respectively
		for i in [0, 1, 2]:
		    i_func = RegularGridInterpolator((x, x, x), self.data[:, :, :, i])
		    image[:, i] = i_func(image[:])
		return image.reshape(orig_image_shape)

def process_image(image_path, output_path, thumb, luto, log):

	print (image_path, output_path, thumb, luto, log)
#	logging.info("Processing image: " + image_path)
	image_name, image_ext = os.path.splitext(image_path)
	if image_ext.lower() == '.tif' or image_ext.lower() == '.tiff':
		# im = tiff.imread(image_path)
		logging.warning("tiff file not supported yet, continuing...")
		return
	else:

		# attempt to open file as an image
		try:
			im = Image.open(image_path)
		except IOError:
			logging.info(image_path + " not an image file, continuing...")
			return

		if (im.mode != 'RGB'):
			im = im.convert('RGB')

		if thumb > 0:
			new_dims = (int(im.size[0] * thumb / max(im.size)),int(im.size[1] * thumb / max(im.size)))
			im = im.resize(new_dims, Image.BICUBIC)
			image_ext = "_thumb" + image_ext

		im_array = np.asarray(im, dtype=np.float32) / 255
		luto.transform_trilinear(im_array, in_place=True)

		im_array = im_array * 255
		new_im = Image.fromarray(np.uint8(im_array))
		lutname = luto.filename.replace(' ', '_')
		if output_path is None:
			new_im.save(image_name + '_' + lutname + image_ext,quality=95)
		else:
			new_im.save(output_path + os.path.basename(image_name) +'_' + lutname + image_ext, quality=95)

image_queue = []

luterio=[]
#   todos_luts.lutito   is array of 3d LUTS

for x in todos_luts.lutito:
	luts_=CubeLUT(x)
	image_queue.append(("./DSC0818.jpg", " ", 0,luts_, False))

with Pool(processes=2, maxtasksperchild=20) as pool:
	print ("procesamos")
	pool.starmap(process_image, image_queue)

for this part
with Pool(processes=2, maxtasksperchild=20) as pool:
	print ("procesamos")
	pool.starmap(process_image, image_queue)

I can't really help you with multiprocessing, but let me know if there are any other problems.

ok thank you


Bug: Sorting by package name does not make sense

Why would "PyFITS" be listed first when I sort by ascending alphabetical order by package name?

capitalization

But, but... still doesn't make sense to me... :sweat_smile:

Maybe de-capitalize before sorting?

I don't think that's possible, I didn't write the sorting function, it's going through google viz


Honour GS use_ext_dns and redundant VR VIP

This patch addresses two issues:

On redundant VR setups, the primary resolver being handed out to instances is the guest_ip (primary IP for the VR). This might lead to problems upon failover, at least while the DHCP lease doesn't update (because the primary resolver will be checked first until times out, however it'll be gone upon failover).

If Global Setting use_ext_dns is true, we don't want the VR to be the primary resolver at all.

logic makes sense, one remark about the code made. starting integration tests on it.

should this not go on 4.7?

one other concern; should the behaviour be more unified between regular vr and vpc somehow? it diverting here, can we prevent that?

@DaanHoogland The patch is ported from 4.7 yes.
The whole VR and VPC should be unified anyway :-)) I noticed there is CsGuestNetwork.get_dns() but we're not there yet, so not sure yet if we can apply same logic.

@ntavares are you saying this is already on 4.7 or on your own 4.7 branch? If it is on 4.7 it should get to master automatically.

Ooops sorry about the confusion. It's on my own 4.7 branch. 4.7 is also missing this, but I wasn't sure how to create the PR for it.

Ah, ok. Plese do make it to 4.7. It's the same way as to master except you need to make some extra choices during creation. Create it on your fork of 4.7 and when creating your PR you need to make sure 4.7 is selected on the apache/cloudstack side as well.

I understand that, my question is which one of the 1000 branches is 4.7? :)
-NT
A Sáb, 7/05/2016, 18:38, Daan Hoogland<EMAIL_ADDRESS>escreveu:

Ah, ok. Plese do make it to 4.7. It's the same way as to master except you
need to make some extra choices during creation. Create it on your fork of
4.7 and when creating your PR you need to make sure 4.7 is selected on the
apache/cloudstack side as well.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/apache/cloudstack/pull/1535#issuecomment-217650221

:) the one with the name '4.7'.

LGTM, tested on 4.7

would like to see it on 4.7 first otherwise: LGTM code reviewed and tested
CI result:
only the ping outside wolrd failure, which is not due to any code in this PR
1535.results.network.txt

Here: https://github.com/apache/cloudstack/pull/1536

@ntavares  can you close this one?

@DaanHoogland closed.


Indentation error in Markdown syntax

##View ####Google Official Videos ->
View
Google Official Videos
I think you just forgot it.
Please update it.

fixed


ComfyUI fix

Should fix AttributeError: module 'comfy.k_diffusion.sampling' has no attribute 'sample_sample_euler_negative'. Also fixes mistakes in setattr calls

thank you very much :)


arrow problem

hi
thanks for good library
i have two problem with arrow
1- I cant set arrow to left corner for rtl text;
2- see this:
http://s8.picofile.com/file/8364272676/2019_06_21_16_39_43.jpg
how can fix  this?
thanks a lot

Hi there. The arrow ignores the RTL attributes. I'll fix it in the next release.

thanks
what about 2th problem:
http://s8.picofile.com/file/8364272676/2019_06_21_16_39_43.jpg
how can fix this?

Changing list entries background is not possible either in the current version.


Visualisation fix for #233

XRTK - Mixed Reality Toolkit Change Request
Overview
Initial check-in of Visualisation fix for #233

In testing, found an issue with assigning models.
A) The Unity API must have updated as the previous Prefab check was failing (model could not be assigned) - Fixed
B) Even with the model assigned, the override model was not used - investigating
C) Walking the code, if a specific controller has specified to use the platform default, this is not being honoured - investigating

All issues resolved, including the OculusTouch pointer issue which was mapped incorrectly to an axis instead of a button


Allow setting a custom cache key

For scenarios where developers already manage token requests and caching for themselves, they could choose to pass a custom implementation of the AccessTokenCache, they may initialize their cache using identifiers that are relevant in their applications e.g. their application's user Ids etc. These user IDs could differ from the userID on the Graph that this lib uses, causing cache retrieval to fail.
Allowing devs to setCustomCacheKey on a TokenRequestContext allows the access token provider to check the cache using the custom key.
part of https://github.com/microsoftgraph/msgraph-sdk-php/issues/1407

Second thought this could lead to issues where the wrong user's token is fetched/identity swaps if we rely on the developer's keys.
Seems like a better idea to get them to setCacheKey which derives a unique key based on the access token then using getCacheKey() on the TokenRequestContext as the identifier during cache hydration as explained here


Docs for observer system

Doc strings and new export for observer system.
Docs pages for observer system and DMRGObserver

Codecov Report

Merging #546 (3204dd3) into master (8653523) will increase coverage by 0.39%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master     #546      +/-   ##
==========================================
+ Coverage   80.50%   80.89%   +0.39%
==========================================
  Files          41       41
  Lines        4314     4099     -215
==========================================
- Hits         3473     3316     -157
+ Misses        841      783      -58

Impacted Files
Coverage Δ

src/mps/observer.jl
94.73% <66.66%> (ø)

src/broadcast.jl
59.23% <0.00%> (-8.58%)
:arrow_down:

src/physics/site_types/tj.jl
58.92% <0.00%> (-6.08%)
:arrow_down:

src/physics/sitetype.jl
89.65% <0.00%> (-3.51%)
:arrow_down:

src/mps/dmrg.jl
79.16% <0.00%> (-2.09%)
:arrow_down:

src/physics/site_types/spinone.jl
84.31% <0.00%> (-1.97%)
:arrow_down:

src/indexset.jl
72.59% <0.00%> (-1.55%)
:arrow_down:

src/physics/autompo.jl
91.95% <0.00%> (-1.24%)
:arrow_down:

src/decomp.jl
88.62% <0.00%> (-0.76%)
:arrow_down:

... and 25 more

Continue to review full report at Codecov.

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


how can ı give variable size images to lasagne network ?

I have a set of images in different sizes. I would like to sample fixed size patches per batch from these images. However as far as I see from the code, nolearn interface expects to have a fixed numpy array dataset. Is there any way to handle this in nolearn?

I believe this is something that is not directly supported by the Lasagne framework. At the very last, when the images are passed through the BatchIterator to the layers, they should have equal size, since the weight parameters of the net have fixed size.
Right now, I don't quite understand what you want to achieve, but it sounds like your problem could be solved if you pre-process the image data so that the patches are extracted beforehand and normalized in size. As mentioned, the BatchIterator could do this job if you change the transform method.

One way to do this is to do manual batch generation in a loop that contains net.fit().  Here's an example to illustrate this:
net = NeuralNet(
    #...,
    max_epochs=1,
    )

for X, y in generate_patches():
    try:
        net.fit(X, y)
    except StopTraining:
        break

Here I'm raising a custom StopTraining exception in my EarlyStopping code that then stops the training loop.
Alternatively, you can sample from patches inside your BatchIterator code.  The X that you feed into net.fit() could be whole images (or filenames?) and the batch iterator would then return the patches.

Are there performance issues with the net.fit loop?


Flush scheduled jobs on schedule()

Before scheduling jobs, all scheduled jobs in queue should be flushed. This is to prevent duplication of scheduled jobs and jobs not in the current schedule definition to become unmanaged.
Use queue.flushScheduled().

Decided to leave this out of Integreat and simplify queue interface instead. Integreat's only interaction with the queue is pushing actions to it and getting actions back through the dispatch method.
Flushing scheduled jobs are still essential, but should happen directly on the queue.


Adding IntendedFor for fieldmaps in json

Relate to #69. See here for an explanation why this attribute is important in fMRIPrep processing.

phasediff.json has been updated under /rawdata. Code and script for updating the files can be found at /fMRI under the slice_timing branch.

I suggest that we put these scripts in /bids since they and are operating on the BIDS /rawdata folder.
Indeed, everything that is working on the BIDS rawdata should be in the /bids folder.
(Then we keep routines that generate data that go into /derivatives folder their appropriate locations, like /fMRI, /dMRI or sMRI)

Could the following errors possibly be due to not specifying the full path?
<EMAIL_ADDRESS>
1: [ERR] 'IntendedFor' field needs to point to an existing file. (code: 37 - INTENDED_FOR)
		./sub-NENAH002/fmap/sub-NENAH002_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH002/func/sub-NENAH002_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH004/fmap/sub-NENAH004_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH004/func/sub-NENAH004_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH005/fmap/sub-NENAH005_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH005/func/sub-NENAH005_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH007/fmap/sub-NENAH007_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH007/func/sub-NENAH007_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH007/fmap/sub-NENAH007_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH007/func/sub-NENAH007_task-rest_dir-AP_run-2_bold.nii.gz
		./sub-NENAH010/fmap/sub-NENAH010_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH010/func/sub-NENAH010_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH011/fmap/sub-NENAH011_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH011/func/sub-NENAH011_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH011/fmap/sub-NENAH011_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH011/func/sub-NENAH011_task-rest_dir-AP_run-2_bold.nii.gz
		./sub-NENAH012/fmap/sub-NENAH012_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH012/func/sub-NENAH012_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH014/fmap/sub-NENAH014_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH014/func/sub-NENAH014_task-rest_dir-AP_run-1_bold.nii.gz
		... and 31 more files having this issue (Use --verbose to see them all).

	Please visit https://neurostars.org/search?q=INTENDED_FOR for existing conversations about this issue.

Could the following errors possibly be due to not specifying the full path?
<EMAIL_ADDRESS>
1: [ERR] 'IntendedFor' field needs to point to an existing file. (code: 37 - INTENDED_FOR)
		./sub-NENAH002/fmap/sub-NENAH002_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH002/func/sub-NENAH002_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH004/fmap/sub-NENAH004_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH004/func/sub-NENAH004_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH005/fmap/sub-NENAH005_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH005/func/sub-NENAH005_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH007/fmap/sub-NENAH007_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH007/func/sub-NENAH007_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH007/fmap/sub-NENAH007_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH007/func/sub-NENAH007_task-rest_dir-AP_run-2_bold.nii.gz
		./sub-NENAH010/fmap/sub-NENAH010_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH010/func/sub-NENAH010_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH011/fmap/sub-NENAH011_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH011/func/sub-NENAH011_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH011/fmap/sub-NENAH011_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH011/func/sub-NENAH011_task-rest_dir-AP_run-2_bold.nii.gz
		./sub-NENAH012/fmap/sub-NENAH012_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH012/func/sub-NENAH012_task-rest_dir-AP_run-1_bold.nii.gz
		./sub-NENAH014/fmap/sub-NENAH014_acq-gre_run-1_phasediff.nii.gz
			Evidence: sub-NENAH014/func/sub-NENAH014_task-rest_dir-AP_run-1_bold.nii.gz
		... and 31 more files having this issue (Use --verbose to see them all).

	Please visit https://neurostars.org/search?q=INTENDED_FOR for existing conversations about this issue.

Solved by replacing sub-NENAH012/func/ with func/.


"generated by" comment includes path to req2flatpak

# Generated by /home/cbm/.local/bin/req2flatpak --requirements Pillow==9.4.0 --target-platforms 310-x86_64 310-aarch64 --outfile python3-Pillow.yaml

That /home/cbm/.local/bin/ is correct but not very professional and not really relevant to almost everyone reading the file.
Some nice way to get just the exe name from argv?  or we have to do something like pathlib.Path(argv[...]).name?
I remember also seeing ./ like:
# Generated by ./req2flatpak --requirements

which is probably the same fix...

hi!
I agree, this would be a nice improvement.
For the implementation: the Pathlib.Path.name method should do the job, compare, e.g., "Extract file name from path" on stackoverflow.

If you run things like python3 -c req2flatpak ... then arg.sysv[0] will be -c.   https://docs.python.org/3/library/sys.html#sys.argv
I think pathlib.Path.name is still ok in that case...

We could use __file__ as well.

Things I'm trying:
        a1 = [pathlib.Path(x).name if i == 0 else x for i, x in enumerate(sys.argv)]

        a2 = [pathlib.Path(sys.argv[0]).name]
        a2.extend(sys.argv[1:])

        a3 = [pathlib.Path(__file__).name]
        a3.extend(sys.argv[1:])

        output_stream.write("# Generated by " + " ".join(argv) + "\n")

I was liked a1 but maybe __file__ is closer in semantics to what we want (?)

Seems __file__ is not always defined?   https://stackoverflow.com/questions/9271464/what-does-the-file-variable-mean-do

More docs on file:
https://docs.python.org/3.9/library/importlib.html?highlight=__file__#importlib.machinery.ModuleSpec.origin

Name of the place from which the module is loaded, e.g. “builtin” for built-in modules and the filename for modules loaded from source. Normally “origin” should be set, but it may be None (the default) which indicates it is unspecified (e.g. for namespace packages).

https://docs.python.org/3/reference/datamodel.html

The pathname of the file from which the module was loaded, if it was loaded from a file. The __file__ attribute may be missing for certain types of modules, such as C modules that are statically linked into the interpreter. For extension modules loaded dynamically from a shared library, it’s the pathname of the shared library file.

I think you can safely use __file__. in my experience, __file__ is reliably defined if python scripts are loaded from the filesystem. (it is not defined in scenarios where python scripts are compiled into an executable binary... but that does not matter for what we want to do, because compiling to a binary is not at all our intended runtime scenario).
Alternatively, you could print the arguments as parsed by argparse. But there does not seem to be a built-in way for this, compare e.g. https://stackoverflow.com/q/34992524

I think you can safely use __file__. in my experience, __file__ is reliably defined if python scripts are loaded from the filesystem.

This is my experience to and also what a was able to grasp from the docs.


Upgrading heapster to v0.17.0

cc @piosz @dchen1107

How about removing mount of ssl volumes? Should be fixed by that https://github.com/GoogleCloudPlatform/heapster/pull/382. I can do it in a follow up PR as well.
LGTM Thanks a lot for doing it!

@piosz: Done. PTAL

@k8s-bot: retest this please

Ping @dchen1107

LGTM. Re-run shippable.

@k8s-bot test this [testing build queue, sorry for the noise]

LGTM from me as well. Thanks for the change!

Automatic merge from SubmitQueue


Allow spans in help to provide structured suggestions

Depending on the failure mode, it may be possible to suggest a fix to the user.
In those cases, it'd very nice to be able to show them a PR-like code suggestion - what to remove and/or what to add with respect to a span pointing at their code.
The precedent I'm looking at here is Rust's suggestion machinery (here).
Going with a practical example:

I'd love for that code snippet in help to be rendered using the same style of the code snippets above, with ::<GenericType> in green to stress that it's an addition.
Is this a usecase you'd be interested to support?

how do you feel about using the Advice diagnostic level for this, actually? That's what it was originally meant for, after all.

@LukeMathWalker how did your experiment go?


✨ Provide label for unembargoed flaws

OSIDB-2937 Confusing Embargo badges
Checklist:

[x] Linting passed
[x] Type checks passed
[x] Tests suite passed
[x] Commits consolidated
[ ] Changelog updated
[-] Test cases added/updated
[x] Jira ticket updated

Summary:

Changes:

Provide badge for unembargoed flaws
Prevent edge case embargo label showing when flaw isn't embargoed (invalid data)

Considerations:
[Replace with any additional considerations, notes, or instructions for reviewers.]

I am just thinking, why does this logic needs to be so complicated? Flaw.embargoed == true -> embargoed Flaw.embargoed == false -> unembargoed (public)
Or am I missing something? I don't think anything with the dates is not needed for the logic.

I'm not really sure about this PR, tbh--it stemmed from a presumably transient data integrity issue specific to one environment. Do you think there is a need for an Unembargoed label?

I am just thinking, why does this logic needs to be so complicated? Flaw.embargoed == true -> embargoed Flaw.embargoed == false -> unembargoed (public)
Or am I missing something? I don't think anything with the dates is not needed for the logic.

I'm not really sure about this PR, tbh--it stemmed from a presumably transient data integrity issue specific to one environment. Do you think there is a need for an Unembargoed label?

Can you please elaborate a little bit? I don't quite follow about the integrity issue. Also I don't think the label unembargoed is really needed, every Flaw is either emabrgoed or unembargoed, so either embargoed label or no label should be enough, right?

Okay, if that's the single source of truth then yes. Upadted lmk if it looks good @JakubFrejlach


WIP: add version to workflow configs

TODO:
-[ ] add parse tests to handle all cases ...
-[ ] document it in docs

// TODO: should we require a version number -> in therms of UX we should not, in terms of strong typisation we should


openLimitedPhotoLibraryPicker callback when done

Feature request
Hey, great package, it's been really helpful to develop my app. I'm currently using the openLimitedPhotoLibraryPicker method to handle a case where users want to update their selection. I would find really helpful a callback when the user is done with updating his selected photos.
Why it is needed
Since I can't know when the user is done by using .then() it's hard for my app to know when to re get the camera roll pictures to see the new selected pictures in the result.
Possible implementation
Adding a onFinish callback param to the function would do the job.
Or .then() should be called only when selection is finished
Thanks again for this package, have a good day!

@Nazort The native code does not offer this kind of callback. The only way to do it is currently to wait for the AppState status change. This could be done on the native side, but also on the JS side.

@zoontek Thanks for the quick answer and advice, I will implement something using AppState then

I've tried doing something using AppState but with a listenener listening on state change, it does not get trigger when the modal is opened or closed, the app stays active at all time.

I also tried with AppState listeners with no luck. Also with navigation and layout events. I found no way of detecting that the user is done with updating his selected photos. Can we reopen the issue?

Can not find out a solution for this issue. Can someone help me?

If native code does not offer this kind of callback, and the app state does not change, then how other apps like Instagram do it?

From Apple's documentation: "Any changes the user applies to their limited library selection trigger a PHPhotoLibraryChangeObserver update that you can observe to see the changes to their selection."

I think openLimitedPhotoLibraryPicker is with no use without the callback after selection.

Sounds like a PR might prove the point and stand a chance of integration!

why this issue was closed? Has this callback been implemented?

@andvalsol I don't recall seeing a PR from you?
The original author closed it, because they were satisfied:

Thanks for the quick answer and advice, I will implement something using AppState then

And the comment trail appears to have died here:

Sounds like a PR might prove the point and stand a chance of integration!

All of this is self-evident from reading the comments.


Eureka client does not get the list of servers (but registers to Eureka Server) when dockerised.

I have the following service-XXXX (eureka server) It has @EnableEurekaServer in main class
applciation.yml
eureka:
instance: localhost
client:
  register-with-eureka: false
  fetch-registry: false
  serviceUrl:
    defaultZone: http://localhost:8761

pom.xml

org.springframework.cloud
spring-cloud-starter-netflix-eureka-server

RAXXXX (eureka -client) it has @EnableDiscoveryClient although I understand this is optional
application.yml
eureka:
    client:
        register-with-eureka: true
        fetchRegistry: true
        serviceUrl:
            defaultZone: http://localhost:8761/eureka/

pom.xml

org.springframework.cloud
spring-cloud-starter-netflix-eureka-client

RAXXXX (eureka-client) These are python services and they registered
Note :-
I see all of these are registered in http://localhost:8761/eureka/
I also can see RAXXXX is able to get the "current list of Servers" with out any issues
Issue: The problem is RAXXXXX is not able to get the "current list of Servers" if all the above three services are dockerised
My docker-compose.yml below
version: '3.8'
services:
  service-registry-XXXX:
    image: c360saascr.azurecr.io/service-registry-XXXX
    ports:
      - "8761:8761"
  risk-analysis-XXXX:
    image: c360saascr.azurecr.io/risk-analysis-XXXX
    ports:
      - "8080:8080"
    command: -Xms12g -Xmx12g -XX:ActiveProcessorCount=8 -Deureka.client.serviceUrl.defaultZone=http://service-registry-service:8761/eureka -verbose:gc -XX:+PrintGCDetails -XX:+HeapDumpOnOutOfMemoryError   -XX:HeapDumpPath=heapdumps -jar risk-analysis-handler-0.0.1-SNAPSHOT.jar
  risk-analysis-XXXX:
    image: c360saascr.azurecr.io/risk-analysis-XXXX
    ports:
    - "5001:5001"
    command: http://service-registry-XXXX:8761 risk-analysis-service5001 5001 risk-analysis-service-5001.log
    restart: on-failure

For security concern I need to add XXXX.
What I tried?

Initially I had three different containers with --link in each. Upon reading a stackoverflow post I created docker-compose.yml
Tried register-with-eureka: true and fetchRegistry: true
Tried docker inspect and replaced -Deureka.client.serviceUrl.defaultZone=http://service-registry-service:8761/eureka with proper ip
In docker-compose.yml tried creating a network and attached that to all the three services.
I see c.netflix.config.ChainedDynamicProperty being called twice in local and once if dockerised. Sadly second time it brings the list of servers (in my local). Wondering this might be the problem but how to make it call twice?

Please learn how to properly format code and logs.
localhost in default zone is not correct.

I have it as -Deureka.client.serviceUrl.defaultZone=http://service-registry-service:8761/eureka as u see in the docker-compose.yml
command: -Xms12g -Xmx12g -XX:ActiveProcessorCount=8 -Deureka.client.serviceUrl.defaultZone=http://service-registry-service:8761/eureka -verbose:gc -XX:+PrintGCDetails -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=heapdumps -jar risk-analysis-handler-0.0.1-SNAPSHOT.jar

I'm unsure how to proceed and assume there is a problem in the docker configuration or networking.

They all are in the same network. Does the below seem strange?
I see c.netflix.config.ChainedDynamicProperty being called twice in local and once if dockerised. Sadly second time it brings the list of servers (in my local). Wondering this might be the problem but how to make it call twice?

:man_shrugging: That's in upstream netflix code. I have no idea why it would be different in docker.

Thats the only difference I see between my local instance and dockerised instance

@hariraogotit, please provide a minimal, complete, verifiable example that reproduces the issue, along with a README containing the steps necessary to run the app locally and in docker - I will have a look at it then.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Enhancedstatuscodes backend

backend.go was missing the new enhanced status codes and the tests for response.go were not in travis

Sorry, forgot to commit the tests. will close the pull request and create a new one


DSC JSSATEN info updated

Solves #33

Hey @naman360, thanks for the PR! Let me quickly update the site to incorporate the new leads as well, to make sure we have a well formed information panel for all chapters along with their historical data. Will merge right after!

yeah fine!


Directory structure review

So, the simple-lang repo, comprises of the environments and the modules needed for the installation of simple. But, the environment I believe should be on it's own. The main repo should only install the compiler and it's core modules leaving out the enviroment. This, enables developers work on each environment in its' repo instead of committing into the base branch, that can cause pollution.
Therefore, I propose that the new directory structure for the main repository should be.

.github
build
examples
modules
simple
...

Perhaps, In the future, the examples folder might be removed too, it really shouldn't be installed or be part of the main installation process.

This has been implemented


Ability to scroll to selected item?

Hi, I am using this in virtual mode, as I have potentially very large lists. Also, I have just single selection, where I preselect an item if it was previously selected.
Is there a way to have the list automatically scroll to the selected item when it is out of the opening view (ie it is some way down the list)?

for the moment is not posible, but i think it´s a great idea for single selection.

Any implementation on this?

Hi, was just asked about this again by our testers (so another nudge). Or even some workaround to do it?


MTL-1811 Incorrect FSopt Fails to Mount Partitions in SLES15SP4

Summary and Scope

Fixes MTL-1811

Issue Type

Bugfix Pull Request

allocsize=13107 is invalid and has been ignored (perhaps discarded) by xfsprogs in SLES15SP2 and SLES15SP3, in SLES15SP4 this value prevents XFS from mounting.
SP3:
ncn-w003:~ # rpm -q --whatprovides /sbin/mkfs.xfs
xfsprogs-4.15.0-4.52.1.x86_64

SP4:
ncn-w001:~ # rpm -q --whatprovides /sbin/mkfs.xfs
xfsprogs-5.13.0-150400.1.9.x86_64

Somehow between xfsprogs-4.15 and 5.13 the handling of how allocsize is enforced has been made more strict. It is not known if our bad value was being honored at all in previous versions of xfsprogs, but we know in our current xfsprogs for SP4 that it causes a fatal error.
This PR changes the value to 131072, a valid value which is 2^17.
Prerequisites

[x] I have included documentation in my PR (or it is not required)
[x] I tested this on internal system (x) (if yes, please include results or a description of the test)

Idempotency

Risks and Mitigations
What is less risky, or more risky now - or if your mod fails is there a new risk?

Confirmed to work, all k8s NCNs on redbull booted successfully with their partitions after baking this RPM into their images.

Seems fine, but it'd be great to see comments here explaining why we need each of these options, and in particular why this size for this option.

It was general xfs optimization, I'll do more digging and add details to the JIRA.


Changed files for comments

Pull Request Test Coverage Report for Build 737

165 of 204 (80.88%)  changed or added relevant lines in 20 files are covered.
42 unchanged lines in 8 files lost coverage.
Overall coverage remained the same at 80.838%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/commons/util/PasswordUtil.java
33
34
97.06%

src/main/java/seedu/address/logic/commands/EditCommand.java
4
5
80.0%

src/main/java/seedu/address/logic/commands/SignupCommand.java
3
4
75.0%

src/main/java/seedu/address/model/event/Event.java
7
8
87.5%

src/main/java/seedu/address/storage/Comments.java
0
1
0.0%

src/main/java/seedu/address/model/event/Status.java
15
17
88.24%

src/main/java/seedu/address/storage/XmlAdaptedEvent.java
10
12
83.33%

src/main/java/seedu/address/model/event/Comment.java
7
10
70.0%

src/main/java/seedu/address/logic/commands/AddCommentCommand.java
0
4
0.0%

src/main/java/seedu/address/model/ModelManager.java
6
11
54.55%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/address/model/event/Phone.java
1
90.0%

src/main/java/seedu/address/model/util/SampleDataUtil.java
1
94.74%

src/main/java/seedu/address/model/user/Username.java
1
90.0%

src/main/java/seedu/address/logic/comments/Comments.java
1
96.43%

src/main/java/seedu/address/model/ModelManager.java
2
87.5%

src/main/java/seedu/address/storage/XmlAdaptedEvent.java
2
95.65%

src/main/java/seedu/address/ui/EventPageFormatter.java
6
0.0%

src/main/java/seedu/address/logic/commands/AddCommentCommand.java
28
0.0%

Totals

Change from base Build 735:
0.0%

Covered Lines:
1911

Relevant Lines:
2364

💛  - Coveralls


Update build status on release-1.4

Update build status after migrating tests

/test ?

/test metal3-ubuntu-e2e-integration-test-release-1-4

/test metal3-ubuntu-e2e-integration-test-release-1-4

/test metal3-ubuntu-e2e-integration-test-release-1-4

Closed in favor of https://github.com/metal3-io/ip-address-manager/pull/504 since release-1.4 is not longer supported after releasing-1.7


halodi-unity-package-registry-manager 1.0.3

fixed using references
bumped package version
bumped deps

@jamestkpoon please review and make a release when you can :)

@jespersmith please review and make a release when you can :)

@jespersmith could you please add git tag for 1.0.3 🙏


Ongoing development

This PR is a bit of a mixed bunch of different things, reflecting a lot of work from feedback from Eszter.
Other changes include:

desktop chat view has list of chats, plus messages for selected chat
should have a max width on pages, so it at least doesn't look broken on desktop
some important bug fixes for message flagging
disable messaging flagging for user chats (direct messages) - not relevant
menu blurs main site when open
highlight currently active links
compact message tabs on mobile
refined button colours (less dramatic hover/focus colours)
bug fix on conflicting modal logic for tasks vs other modals
various design tweaks
redirect to first user chat when visiting chat overview on desktop (it's a bit clunky... clientside redirect after loading... 🙏)
updates avatars, and command to reimport them, will update existing ones, or add new ones as needed
nav menu used to load via htmx, now is just included directly... not sure why it used to use htmx... 🤞
nicely styled file inputs

Some screenshots
Nicely styled file inputs

Desktop chat view with panels

Small tabs on mobile

I made a few more small fixes:

removing now-unused the check nav route
ensure background is white under topic tabs (some css margin collapsing was making it show the blue fade)
have a title tooltip over the relative dates on tasks to show exact date

So, I'll merge now :)


Rotating Causes Collider to Jitter on Y-Axis on Remote Client

For reasons I'm currently not able to investigate right now. Rotating the player object causes the Collider game object to experience drifting on the Y-Axis. This results from the delayed loop and some constraint behavior or incorrect understanding.

The issue appears to be unrelated to my setup. Rather likely a bug with VRChat?

https://github.com/Zexxx/Smart-Floor-Collider/assets/21136842/450426da-bf49-4abd-9ed2-ac1374d17b78


(Feature) display 3 prices for Crowdsale page for Dutch Auction strategy

Problem: crowdsale page displays only current price of the token. It could be useful to see start and end prices also.
Solution: display start and end prices of crowdsale in the crowdsale page as well

@vbaranov As I understand it is displayed as price is how many tokens can be bought per ETH.
I ended up with a layout, that may respect the concepts but I need confirmation:
Dutch Auction

Or we can replace 'Price' with 'Rate' to make it less confusing and display only one value.
@dennis00010011b thoughts?

@fernandomg I like this layout, but in the new layout for TW 2.0 redesign, the rates will be displayed over the tickers on the scale. I think display both price and rate will overload the page.

Or we can replace 'Price' with 'Rate' to make it less confusing and display only one value.

This one looks more preferable to me


[hotfix] fix dead link

Purpose of this pull request
as title

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

@liunaijie #7435 has been fixed, thanks.


feat(tidb): skip ci on some images file change on  tidb lts 6.1 & 6.5

skip ci on some images file change on  tidb lts 6.1 & 6.5

/hold
Take effect after the release of v7.6.0

/lgtm
/approve

/unhold

/unhold


2 Pages for the same thing: EventBus and EventStream

Currently these pages contain the same thing. Im guessing this is a leftover from some rename action. Can I safely assume the EventStream page can be removed?
(since the eventbus one contains a specific link back to the JVM docs)

Yes, please consolidate

PR was Merged


bug(drag-and-drop): Scrolling a parent container while dragging an item breaks placeholder/dropspot position

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
When using drag-and-drop without scroll (but some parent is scrollable), the drag placeholder seems to get out of sync with the cursor when you scroll while dragging.
Reproduction
https://stackblitz.com/edit/d1tv4h-4a6ghz?file=src%2Fapp%2Fcdk-drag-drop-custom-placeholder-example.html
Steps to Reproduce:

Start dragging an item
Scroll down, and notice the item's placeholder/dropspot is now out of sync with the cursor

Expected Behavior
The placeholder/dropspot for dragging should always match where the user's cursor is
Actual Behavior
The placeholder/dropspot for dragging becomes out of sync with the user's cursor when scrolling while dragging. When dropping at the cursor, the item doesn't drop in that place.
Environment

Angular: 15
CDK/Material: 15.2.9
Browser(s): All
Operating System (e.g. Windows, macOS, Ubuntu): All

You have to set cdkScrollable on the scrollable container so the CDK knows that it's being scrolled. Here's a working example: https://stackblitz.com/edit/d1tv4h-jimys6?file=src%2Fapp%2Fcdk-drag-drop-custom-placeholder-example.html


Issues loading on first setup

Hello!
I've been trying to follow the README to get this working locally before trying it in prod, but I can't seem to get my data to load into the database at all. So far, my code looks like this:
var GTFS = require('gtfs-sequelize');

var downloadConfig = {
  gtfsUrl: 'http://transitfeeds.com/p/nj-transit/409/latest/download',
  downloadsDir: 'downloads'
};

var gtfs = GTFS(downloadConfig);
gtfs.downloadGtfs(function() {
  //download has finished callback
  console.log("Download complete, adding gtfs to database.");
  var pgConfig = {
    database: 'postgres://gtfs_sequelize:gtfs_sequelize@localhost:5432/gtfs-sequelize-test',
    downloadsDir: 'downloads',
    gtfsFilename: 'gtfs.zip',
    sequelizeOptions: {
      logging: false
    }
  };

  var gtfs = GTFS(pgConfig);
  gtfs.loadGtfs(function() {
    //database loading has finished callback
    console.log("Finished loading to database");
  });
});

However, I keep receiving the following error:
path.js:7
    throw new TypeError('Path must be a string. Received ' + inspect(path));
    ^

TypeError: Path must be a string. Received undefined

What exactly am I doing wrong here? I'm using node version lts/boron if that helps.

Thanks for reporting.
Unfortunately a lot is going on here that is the fault of this library.

The README docs are wrong and need to be updated.  The config option gtfsFilename should be gtfsFileOrFolder.
The url you are querying does a redirect to the actual file, which this library doesn't yet support
The url it redirects to downloads via https, which this library can't handle
This particular feed is the kind that omits the calendar.txt file and instead includes everything in calendar_dates.txt.  This library also can't handle that, but at least that is a know issue (#53)

I should probably work on fixes for all these things, but it's low priority for me at the moment.  If you want to take a shot at it I'm happy to accept a PR.


Roll Flutter Engine from fcb40168f159 to 8d1789a43319 (1 revision)

https://github.com/flutter/engine/compare/fcb40168f159...8d1789a43319
2024-01-06<EMAIL_ADDRESS>Roll Skia from 38c029ddd665 to 514dd58cabfc (1 revision) (flutter/engine#49575)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


"extends" schema resolution now handles extending a type that isn't a…

…t the
root of the schema file
Resolves #425

Great, thanks for submitting this and including a test case :D