Issue with /docs/tutorials/kubernetes-basics/deploy-interactive/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
"kubectl proxy" hung
After "+" a new terminal to try POD_NAME, the size of display < that of the terminal that bottom lines could not be seen
Proposed Solution:
Don't know
Page to Update:
https://kubernetes.io/docs/tutorials/kubernetes-basics/deploy-interactive/

@fulltopic Thanks for opening this issue.
@BenHall 👋 Possibly related to #2493?

@zacharysarah It does look like it, however I thought I had resolved this.
@fulltopic Could you please tell me your browser, os and screen resolution please?

Thanks @fulltopic
This bug matches closely other bugs reported against the use of kubectl proxy in the tutorial's hosted at Katacoda. I have a PR pending to resolve, and am closing this bug to focus conversation on the earliest report of this issue: https://github.com/kubernetes/kubernetes.github.io/issues/1561
Please comment at https://github.com/kubernetes/kubernetes.github.io/issues/1561, or subscribe to that bug, to get updates as we work on resolving this.


New Rule: PDF file linking to Password Protected File from first-time sender

h/t to @bfilar for the org extraction idea 💰


11/10 is also a funny fraction

Also, did you know branch names can have / in them? I didn't.

420/10 is funny too :smoking: :leaves:  :leaves:  :maple_leaf: :leaves: :leaves: :+1:

I did know about the '/' thing; I've seen projects use it for namespacing like "tmp/branch-name" before.


Mouse Movement Problems

Hi,
I really like your app..  I have the Google Maps Configurable  example running locally and I am [NaN,0] from  console.log(windMap.renderer.pointFromXY(x, y));  as I move my mouse across the window.  Is this missing data for a problem with canvas?  I am willing to work on fixing this problem but need some insight.
Thanks

Hmm not sure why that would be happening. If you check out the example being served here:
https://damp-reaches-87397.herokuapp.com/
And click around, you'll see X/Y coordinates printed out in the console.
That is just serving this page:
https://github.com/dannycochran/windable/blob/master/examples/googleMaps/configurable.html
It's possible if you are showing the map in a smaller container, you need a different way to get the x/y coordinate than how I'm doing it.

When I zoom out to Google Earth level 1 on  https://damp-reaches-87397.herokuapp.com/ the wind data does not display on Google Chrome.  The wind does display for level 1 on Safari...  I read one of your comments about a memory leak in Chrome so I tried another browser.
Thanks for responding to me..


Usability: Base chart name can be an issue for local helm chart repositories

The chart istio/base makes sense in the context of istio helm repositories. However some organizations choose to setup their helm repos to hold upstream helm chart for different purposes.
If a organization wants the helm chart in their repositories this can cause a name conflict, or inaccuracy of the name to define what this package is about.
E.g. : my-upstream-repo/base would not make sense.
An alternative could be adding istio- prefix to the base helm chart name.
[ ] Ambient
[X] Docs
[X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[X] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane

Why not my-upstream-repo/istio/base?

Actually realizing that may only work with OCI registries?

I think you are right. It would not work on non OCI repos.


Check RenderingDevice availability to display LightmapGI configuration warnings

Follow-up to https://github.com/godotengine/godot/pull/87386
and https://github.com/godotengine/godot/pull/91172.

We can now check whether RenderingDevice can be created (which is not guaranteed when using the Compatibility rendering method), so the warning can be displayed only when relevant.
This also disables the Bake Lightmaps button with a tooltip if baking is not available.
Preview
Ignore the GPU model name (it obviously supports Vulkan), I just changed the condition to get the dialog to show up.

Edit: Now displays dedicated warnings if lightmapper_rd was disabled at compile-time, including a specific message for mobile platforms.

This look’s great though i think maybe an additional comment should be added for the android editor when trying to bake since it would confuse people why they can’t bake lightmap gi if they can use the vulkan mobile backend.

How often is get_configuration_warnings called? Its after every property change right? If so, we need to cache the result of can_create_rendering_device() somewhere or else this is going to make things very slow

I've just checked by benchmarking how time it takes to call DisplayServer::get_singleton()->can_create_rendering_device() 1000 times, and it takes 1 microsecond or less on a debug editor build. It's cached now, as the result of this function can't change during a session anyway.
Node::get_configuration_warnings() is called once per LightmapGI node present in the current scene tree when the tree changes or when the scene is reloaded, so it's not called that often.

I've just checked by benchmarking how time it takes to call DisplayServer::get_singleton()->can_create_rendering_device() 1000 times, and it takes 1 microsecond or less on a debug editor build. The method is already cached in its base implementation, as the result of this method can't change during a session.

Did you test this with the Compatibility renderer? That's where it might be expensive.
If there's already a RenderingDevice then it's basically just checking the singleton's existence so that would indeed be fast.

Did you test this with the Compatibility renderer? That's where it might be expensive.

Indeed, I just tried and it takes much longer to run there. Caching this result avoids the issue entirely: https://github.com/godotengine/godot/pull/97698

Thanks!


Indication of invalid style

Now that styles are auto applied, it would be beneficial to have some type of indication that the style is invalid. Otherwise the learner is left confused on why their style is not being applied.
For example a incorrect style that looks correct on first glance:
flex-direction: row-reversed;
(An error I make all too much.)
A problem like that would be solved by some type of indicator that a line in invalid. (Perhaps an icon to the right of the invalid line?)

Eventually I'd love to parse the answers and add some tailored, localized hints for common errors (like row-reversed). I'm collecting some analytics data on that.
In the meantime, minor improvements could still be made to the feedback. There's subtle feedback in that the "Next" button changes from a disabled to active state. Maybe the color could also be changed from red to green when the level is solved.

I did notice that the next button changes state, but I wonder if that is too subtle of a feedback. (Especially for beginners.) Which is why I think some way of highlighting the offending line would be helpful.


remark for github org and team separation

What this PR does / why we need it:
Syntax of authentication using GitHub teams changed with Concourse 4.x.
In Concourse 3.x teams have been specified using a slash, e.g. my-org/my-team
With Concourse 4.x you must use a colon, e.g. my-org:my-team.
It took me quite some time to figure this out !

/ok-to-test

/lgtm

/retest

/test pull-charts-e2e


Add NeurST results to leaderboard

We report our ASR results on GigaSpeech XL using toolkit NeurST.

@mct10 Sorry I overlooked the section where you added the results and had to revert the PR. The first section (baselines) is actually reserved for the results we provided in the papers, and for the new results, we should add them to the second section, see below

Could you fix that and create another PR? You may want to fill out that contributor tab as well (e.g., ByteDance?)

@chenguoguo  Thanks for feedback. We will fix that.
By the way, should we add our results before or after Wenet?


Docs: It fixes documenttation to options props at README.md

@lucaslimasouza Thank you for opening a pull request. We changed the API in v.7 and now the options prop is renamed to searchOptions 👍


Quickstart

I can write quickstart page.

Work on the website needs to be made. I don't need another issue to keep track the individual task, the big TODO  in the quick-start page is enough.


save as image after render pdf

can i save as image after render pdf?

Once rendering is finished, you can save the canvas as a PNG image using canvas.toDataURL("image/png");. Closing as answered (and out-of-scope for this project).


feat: added daily rates api

Context

fixes #535

Changes

maybe we can find a better place for the daily rates outside of rapyd, since it is no longer using it. But it is ok for now.


Support for generating g-code?

I've had a 3D printer for a little while now and I'm generally frustrated with the state of slicing software. I was thinking it might be fun to try to write a slicer in Rust at some point. Would you be interested in adding support for generating g-code to this crate? Specifically I think it'd be nice to have a type-safe API to represent commands (which I think you basically already have as the output of the parser) and some API to serialize them to strings. This might also want some helper APIs on top of the command types to make it easy to create certain kinds of commands.
What do you think about this? I know the crate is currently #![no_std] (for good reasons), so maybe this could be an optional feature? Something like a slicer is less likely to be concerned about not allocating at all, certainly.

It should be quite possible to add something for generating gcode while keeping the library #![no_std], it's just a case of using something along the lines of the Write trait.
Otherwise I could add a with_std feature that enables using this crate with std.


HREF links do not always work when simulateTouch: true

Check that this is really a bug

[X] I confirm

Reproduction link
https://finisterre.com
Bug description
This is a very intermittent bug, but when simulateTouch: true on non-touchscreen devices (e.g. desktops), sometime the HREF links on slides do not work. The user does not navigate to the intended URL.
This is not currently deployed to production.
Expected Behavior
HREF links should take you to the correct web-page.
Actual Behavior
No response
Swiper version
8.2.2 and above
Platform/Target and Browser Versions
macOS Chrome 105 (and more generally)
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[x] I'm willing to open a PR

Increase threshold parameter

Thank you - we are not currently using threshold, so I am unclear how much to "increase" it by, as we have no baseline. I have chosen 10, as a bit of a random choice. Please advise?


Create directory when overwrite option is set

In 0.32.2, I found that I need to create the "destination" directory first, in order for this call to succeed:
                    reader.WriteEntryToDirectory(destination, new ExtractionOptions()
                    {
                        ExtractFullPath = true,
                        Overwrite = overWrite
                    });

This was working fine with 0.28.2, which creates the "destination" directory if it does not exists. Is this the same issue as #614 ? I have the impression it should have been fixed in 0.29.0... Is that correct?

From what I can see, the 0.29 change checks that you have already created it before calling WriteEntryToDirectory

Maybe overwrite should always create the directory.  Can't recall the reason for change at the moment


🐛 Should check if has selected log file before reading contents

This PR aims to resolve an issue where you unselect a log file, it throws a FileNotFoundException with the message of "File does not exist at path ."

thanks!


Condition for transition to float

I often observe the following behaviour:
When reaching MAX_CELL_VOLTAGE and the current is dropping towards zero the cells with higher voltage come down and the cells with lower voltage come up, then crossing each other and drifting away again before the balancer brings them finally all together.
At the point of time when they are crossing (celldiff<CELL_VOLTAGE_DIFF_KEEP_MAX_VOLTAGE_UNTIL) the battery pack seems to be balanced an the timer of MAX_VOLTAGE_TIME_SEC starts. So after MAX_VOLTAGE_TIME_SEC the control_voltage is lowered to FLOAT-Voltage no matter how well they are together at this point of time (as long as they are not more than CELL_VOLTAGE_DIFF_TO_RESET_VOLTAGE_LIMIT away from each other).
In reality the battery is not completely balanced and MAX_VOLTAGE is stopped to early:

My proposal would be to introduce one more threshold for cell-deviation that causes MAX_VOLTAGE_TIME_SEC to restart if celldiff gets above this threshold:

So celldiff has to be smaller than CELL_VOLTAGE_DIFF_KEEP_MAX_VOLTAGE_TIME_RESTART for MAX_VOLTAGE_TIME_SEC to start the transition to float.
As default value CELL_VOLTAGE_DIFF_KEEP_MAX_VOLTAGE_TIME_RESTART can be choosen much bigger than CELL_VOLTAGE_DIFF_KEEP_MAX_VOLTAGE_UNTIL so this change has no impact for existing configurations and to prevent that some configurations (where the celldiff has a lot of noise) would never change to float.
For my configuration it has a very positiv effect:

Nice catch :-)

Thanks a lot for merging this commit!

Thank you for this great improvements :-)


"Go to definition" does not work if caret is at the end of a qualified symbol, right before dot

(current master)
If caret (| here) is at the end of a symbol, Go to definition does not work
module Module1 =
    let foo x = x

let _ = Module1|.foo 1

It does work if caret is at start or in the middle:
let _ = |Module1.foo 1
let _ = Mod|ule1.foo 1

Odd, I thought that was fixed.  I'll take a look

Ok, it was only for a long identifier Module1.foo name.  Fixed in #1855


[Feature]: Add Rotational Cipher exercise

This PR adds the Rotational Cipher exercise from the problem specifications repo.
I might need to fix the unreadable intrinsic function chain later, but right now I don't think it will be an issue since it's just for the CI test.

All checks passed, so everything seems alright.

Actually, it wasn't alright. The tests were not running for the Rotational Cipher

line 14: cobc: command not found

The CI on Linux is failing with a command not found for cobc. That's weird.


TYPO: HIST field bit in example pseudo-code are reversed.

In example encoder pseudo-code has this section:
if (InfoIsBranchTaken(info))
encoHIST = (encoHIST << 1) | 0; // Mark branch as taken
else
encoHIST = (encoHIST << 1) | 1; // Mark branch as not-taken
Bits 0 and 1 are reversed. Current reference code (and everywhere else in the spec ...) HIST bit=1 means 'taken' and 0 means 'not-taken'.
This is considered a typo.

Fixed (by direct edit).


chore(qwik-city/middleware): add TextEncoderStream polyfill and tests

add _TextEncoderStream_polyfill and tests

@wmertens when you get time can you review

yeah the props are there only to match the previous polyfill. so everything is just-in-case

Oh, sure. But anyway here's TypeScript flavored version of TextEncoderPolyfill based on Node.js' implementation, just in case:
/**
 * TextEncoderStream polyfill based on Node.js' implementation https://github.com/nodejs/node/blob/3f3226c8e363a5f06c1e6a37abd59b6b8c1923f1/lib/internal/webstreams/encoding.js#L38-L119 (MIT License)
 */
export class TextEncoderStream {
  #pendingHighSurrogate: string | null = null

  #handle = new TextEncoder()

  #transform = new TransformStream<string, Uint8Array>({
    transform: (chunk, controller) => {
      // https://encoding.spec.whatwg.org/#encode-and-enqueue-a-chunk
      chunk = String(chunk)

      let finalChunk = ""
      for (const item of chunk) {
        const codeUnit = item.charCodeAt(0)
        if (this.#pendingHighSurrogate !== null) {
          const highSurrogate = this.#pendingHighSurrogate

          this.#pendingHighSurrogate = null
          if (codeUnit >= 0xDC00 && codeUnit <= 0xDFFF) {
            finalChunk += highSurrogate + item
            continue
          }

          finalChunk += "\uFFFD"
        }

        if (codeUnit >= 0xD800 && codeUnit <= 0xDBFF) {
          this.#pendingHighSurrogate = item
          continue
        }

        if (codeUnit >= 0xDC00 && codeUnit <= 0xDFFF) {
          finalChunk += "\uFFFD"
          continue
        }

        finalChunk += item
      }

      if (finalChunk) {
        controller.enqueue(this.#handle.encode(finalChunk))
      }
    },

    flush: controller => {
      // https://encoding.spec.whatwg.org/#encode-and-flush
      if (this.#pendingHighSurrogate !== null) {
        controller.enqueue(new Uint8Array([0xEF, 0xBF, 0xBD]))
      }
    }
  })

  get encoding() {
    return this.#handle.encoding
  }

  get readable() {
    return this.#transform.readable
  }

  get writable() {
    return this.#transform.writable
  }

  get [Symbol.toStringTag]() {
    return "TextEncoderStream"
  }
}

looks good lets try it

@octet-stream thanks for finding this better polyfill


onQueryChange getting stale query, not receiving result of transformQuery

React Query Builder version
6.5.5
Platform
No response
Description
Hi RQB team! Thanks so much for building this great library
I am running into an issue in my code where the lifecycles and rendering cycles seem to be messed up. I have a controlElement for fieldSelector that runs transformQuery and then sets it's result as a query.
When logging, the result looks correct. The problem is, onQueryChange is then run with what seems to be a stale query. Have people run into this before? Thanks so much and let me know if you need any more information.
A business use case for what I want is to to reset multiple rules within a rule group when one rule is changed. I want to reset the operators and the values for all sibling rules when a single rule field is updated.
Thanks again!
Reproduction
const [query, setQuery] = useState<any>()

return <QueryBuilder
   query={query}
   onQueryChange={(query) => {
   // THIS QUERY IS STALE
return   setQuery(query)
   }}
   controlElements={
      fieldSelector: (props) => {
      return
        <MaterialValueSelector
          {...props}
          value={value}
          handleOnChange={(value) => {
            const result = transformQuery(query, {
              ruleProcessor: (r) =>
                ruleProcessor(
                  r,
                  r.path?.slice(0, 2)?.join("-") === parentPathCompositeKey
                    ? value
                    : r.value
                ),
              ruleGroupProcessor: (rg) => {
                if (rg.path?.join("-") === parentPathCompositeKey) {
                  return {
                    ...rg,
                    combinator:
                      combinators[value as Fields]?.[0]?.name ?? rg.combinator,
                  };
                }
                return rg;
              },
            });
            console.log(
              "FIELD SELECTOR CHANGE",
              JSON.stringify(result, null, 2) // THIS IS ALL CORRECT
            );
            setQuery(result);
            const fieldValueMapCp = new Map(fieldValueMap);
            fieldValueMapCp.set(parentPathCompositeKey, value);
            setFieldValueMap(fieldValueMapCp);
            return props.handleOnChange(value);
          }}
        />
      }
   }
/>

My temporary solution for this is to keep a ref of the result and use that as the onQueryChange (resetting it after we setQuery)...but still wondering if this is a race condition on my end that is more easily solvable? Thanks

You've got a LOT going on there. A few RQB-specific problems but mostly just non-idiomatic React code.
After a quick review, here are some notes:

Type useState with RuleGroupType:const [query, setQuery] = useState<RuleGroupType | undefined>();

Alternatively, set an initial query that is already typed as RuleGroupType:const [query, setQuery] = useState(initialQuery);

You can pass a useState setter like setQuery directly to onQueryChange:return <QueryBuilder query={query} onQueryChange={setQuery} />;

Do not define components within other components.

Move all your controlElements outside the component they're assigned in.

The last point leads to the main issue: calling "external" state handlers (setQuery, setFieldValueMap) within an RQB custom component.

Either let RQB handle its own state or handle it yourself, but don't do both.
props.handleOnChange updates the internal RQB query and calls the function you passed to onQueryChange (setQuery in your case) with the result.

Calling setQuery ahead of props.handleOnChange will cause problems because they're ultimately updating the same state variable, probably with different starting points.
useState setters are not synchronous, so when you call setQuery, the corresponding state variable query is not updated immediately. This is why it seems "stale" in your case when onQueryChange is called.

I don't really know what's going on with those rule and group processors, but if you want to make those changes to the query and the field update from the value selector, I suggest using the update method on the query before passing it to transformQuery, like this:
const result = transformQuery(update(query, "field", value, props.path), {
  ruleProcessor,
  ruleGroupProcessor,
});

Important note: query in the example above should be retrieved from RQB state using props.schema.getQuery(), not your own state. This ensures you're working with the latest version (the same one RQB has just rendered). To update the entire query and not just the field property of the given rule, use props.schema.dispatchQuery()
const MyValueSelector = (props: ValueSelectorProps) => {
  const handleOnChange = (value: any) => {
    const query = props.schema.getQuery();
    const result = transformQuery(
      update(query, 'field', value, props.path),
      { ruleProcessor, ruleGroupProcessor }
    );
    props.schema.dispatchQuery(result);
    // DO NOT call props.handleOnChange() here
    // since `update` already handled it.
  };
  return <MaterialValueSelector {...props} handleOnChange={handleOnChange}>;
}

@jakeboone02 this is super thorough, thank you. I apologize, I omitted some of the typing and nuances of the code to be less verbose so I can see how this snippet comes off riddled with code smells 😂
One thing I had a question about: I've noticed that dispatchQuery and getQuery are in the newer version of RQB (v7)...are they available in 6? We're behind in react so can't update to 7 at the moment.
I'll take a look at the rest of your suggestions and implement them ASAP. They make sense to me and I appreciate all your help!

I figured there were some missing pieces 🤣
Unfortunately those methods are only in v7, but the workaround for v6- is to pass query down through the context prop and completely rely on external state for updates.
I linked to some relevant examples in #296 and also discussed something similar in #604.

hey @jakeboone02 , trying to refactor this behemoth of a component i've created using all utility functions and external state management. Here's a basic issue im running into.....I have two rule group levels (level 1 is + Audience, level 2 is + Parameter)....when I click + Parameter, I want a default rule to show up.
The problem with this code is, the default field seems to show up, but the + Rule button disappears when using the add functionality you described in the documentation.
setQuery here is just a dispatch to a store.
Is there something i'm missing here? Attached is the code and screenshots:
const addGroupAction = (
  props: ActionWithRulesAndAddersProps,
  onClick: (e: any) => void
) => {
  return props.level === 2 ? null : (
    <MaterialActionElement
      {...props}
      label={EGroupTitle[props.level] ?? "+ Group"}
      sx={{
        "&:hover": {
          backgroundColor: "#007DFF",
        },
      }}
      handleOnClick={onClick}
    />
  );
};

<ReactQueryBuilder
            fields={fields}
            query={query}
            showCombinatorsBetweenRules
            autoSelectOperator
            resetOnFieldChange
            onQueryChange={(updatedQuery) => {
              setQuery(
                updatedQuery as RuleGroupType<
                  RuleType<string, string, any, string>,
                  string
                >
              );
            }}
            controlElements={{
              /** PROBLEM CODE BELOW */
              addGroupAction: (props) =>
                addGroupAction(props, (e: any) => {
                  if (props.level === 1) {
                    // DO NOT run handleOnClick here, 'add' handles changes for us
                    return setQuery(
                      add(query, defaultRule, props.path, {
                        combinatorPreceding: "and",
                      })
                    );
                  }
                  return props.handleOnClick(e);
                }),
              /** IGNORE CODE BELOW THIS LINE */
              addRuleAction,
              removeGroupAction: (props) =>
                props.level === 2 ? null : <MaterialActionElement {...props} />,
              removeRuleAction: (props) => {
                return props.level === 2 ? null : (
                  <MaterialActionElement {...props} />
                );
              },
            }}
          />

the UI looks like this:

the desired UI should be this:

BASICALLY, when you add a rule group on the second level, I want there to be a default rule that shows up. Adding the prop addRuleToNewGroups doesn't work because that automatically adds new rules to every group. I want to just add it to the second level.
Reminder that I'm using v6 here so I don't have access to the utility methods that were described above (getQuery, dispatchQuery) etc.
That's why I am using the add methods provided
Thanks again for all your help here, I really appreciate it!

I don't think you need a custom addGroupAction event handler in the component itself like that. Try this:
Working example
const onAddGroup = (rg: RuleGroupType) => ({
  ...rg,
  rules: [{ field: 'firstName', operator: '=', value: '' }],
});

export const App = () => {
  const [query, setQuery] = useState(initialQuery);

  return (
    <QueryBuilder
      fields={fields}
      query={query}
      onQueryChange={setQuery}
      onAddGroup={onAddGroup}
    />
  );
};

Sorry it took so long to respond to this, btw!

Awesome thanks @jakeboone02


Country informational page mobile view

Right side data container doesn't  flex properly.

Closing due to already having #239


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/86636

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/86675

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/86684

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


[Bug]: Data tab, under Supporting, Geographic Areas: Clipboard inhibiting ASCII codes

Steps to reproduce the bug
1. Data tab, under Supporting, Geographic Areas

If I try to type in a GA with a diacritic such as 'San José' using the ASCII code 'Alt+130', there will be an autofill from the the clipboard since the shortcuts in the clipboard begin with 'Alt'.

At some point I had copied 'San Jose' or part of it so the GA fills in using the clipboard when I type 'Alt'. see additional screenshot

@bpescador, @chrisgrinter, @rich-keller

Screenshot

Expected behavior
When I type an ASCII code for a diacritic or special character, it should not include previously copied data from the clipboard.
Additional Screenshots
No response
Environment
Sandbox
Sandbox Used
No response
Version
build d19176ebb on 2023-01-10T02:04:07Z
Browser Used
Version 108.0.5359.125


chore: add volunteer KrishnaTheCoder756

chore: add volunteer KrishnaTheCoder756

Thanks for making the changes @KrishnaTheCoder756. Welcome to the community!


[Vote] Welcome new PMC member XiaoHu Zhang joining Sharding-Sphere

Orange Finance arrange a full time developer @beckhampu who is responsible for APM of Sharding-Sphere for several months. His pull requests are #817,  #856 and  #958, and he is already in Sharding-Sphere's committer list.
Orange Finance have already establish a new department which name is innovation center, and plan to as a sponsor to support Sharding-Sphere's development continually.
Today, we will invite XiaoHu Zhang who is an director of innovation center for Orange Finance to join into Sharding-Sphere PMC. We believe that his joining will make Sharding-Sphere better and better.
All PMC members, please vote for this decision:
@haocao
@wu-sheng
@hanahmily
@PegasusS
Voters can either respond "agree, yes, +1", "disagree, no, -1", or "abstain".

+1
Welcome and glad we have a new partner and sponsor. Look forward our cooperations.

agree, yes, +1

Welcome,+1

Welcome,+1

Welcome,Tiger!  +1

All pass, welcome XiaoHu Zhang join us


Ran mvn javadoc:fix

fixed javadoc with javadoc tool - no manual modification - shall be amended with business description
organized imports

Automatic javadoc generation has added wrong authorship of files. This ought to be corrected.

Thanks, this looks good.
The only issue I see is that the version is the default of $Id:$Id but this is probably something that can be changed with the <defaultVersion> tag when doing a release.


intersection types and possibly undefined errors causing additional, invalid missing prop errors

Flow 0.30.0
/* @flow */

import React, {Component} from 'react'
import * as Immutable from 'immutable'

export type Props = {
  calibration: Immutable.Map<any, any>,
}

type PointProps = Props & {
  pointIndex: number,
}

export class Point extends Component<void, PointProps, void> {
  static validate(props: PointProps): Object {
    const {pointIndex, calibration} = props
    const outputValue = calibration.getIn(['points', pointIndex, 'y'])
    return {outputValue: {error: `'${outputValue}' is not a valid number`}, valid: false}
  };
  render() {
    return null
  }
}

Errors:
react-sandbox/scratch/flowissues.js:16
 16:     const {pointIndex, calibration} = props
                            ^^^^^^^^^^^ property `calibration`. Property cannot be accessed on any member of intersection type
 16:     const {pointIndex, calibration} = props
                                           ^^^^^ intersection
  Member 1:
   10: type PointProps = Props & {
                         ^^^^^ Props
  Error:
   18:     return {outputValue: {error: `'${outputValue}' is not a valid number`}, valid: false}
                                            ^^^^^^^^^^^ undefined (too few arguments, expected default/rest parameters). This type cannot be added to
   18:     return {outputValue: {error: `'${outputValue}' is not a valid number`}, valid: false}
                                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ string
  Member 2:
   10: type PointProps = Props & {
                                 ^ object type
  Error:
   16:     const {pointIndex, calibration} = props
                              ^^^^^^^^^^^ property `calibration`. Property not found in
   10: type PointProps = Props & {
                                 ^ object type

This only seems to hapeen if PointProps is an intersection type.  The errors about calibration not being part of the shape are wrong and go away if the string template error is fixed.
I can also repro this way:
/* @flow */

import React, {Component} from 'react'
import * as Immutable from 'immutable'

export type Props = {
  calibration: Immutable.Map<any, any>,
}

type PointProps = Props & {
  pointIndex: number,
}

export class Point extends Component<void, PointProps, void> {
  static validate(props: PointProps): Object {
    const {pointIndex, calibration} = props
    const outputValue = calibration.getIn(['points', pointIndex, 'y'])
    return outputValue.toFixed(1)
  };
  render() {
    return null
  }
}

Errors:
react-sandbox/scratch/flowissues.js:16
 16:     const {pointIndex, calibration} = props
                            ^^^^^^^^^^^ property `calibration`. Property cannot be accessed on any member of intersection type
 16:     const {pointIndex, calibration} = props
                                           ^^^^^ intersection
  Member 1:
   10: type PointProps = Props & {
                         ^^^^^ Props
  Error:
   18:     return outputValue.toFixed(1)
                  ^^^^^^^^^^^^^^^^^^^^^^ call of method `toFixed`. Method cannot be called on possibly undefined value
   18:     return outputValue.toFixed(1)
                  ^^^^^^^^^^^ undefined (too few arguments, expected default/rest parameters)
  Member 2:
   10: type PointProps = Props & {
                                 ^ object type
  Error:
   16:     const {pointIndex, calibration} = props
                              ^^^^^^^^^^^ property `calibration`. Property not found in
   10: type PointProps = Props & {
                                 ^ object type

So this is a bug in the immutable.js.flow file you're probably using. See if changing these two lines:
https://github.com/facebook/immutable-js/blob/master/type-definitions/immutable.js.flow#L55-L56
to this single line:
getIn<T>(searchKeyPath: ESIterable<any>, notSetValue?: T): T;

helps.
Detailed explanation: when you don't provide a notSetValue arg in your call, you're implicitly passing undefined, so Flow selects the first case with T = void.

@avikchaudhuri I think you're missing the point.  The question is why the extraneous errors like this were getting triggered:
 16:     const {pointIndex, calibration} = props
                            ^^^^^^^^^^^ property `calibration`. Property cannot be accessed on any member of intersection type
 16:     const {pointIndex, calibration} = props
                                           ^^^^^ intersection

These errors are wrong and have nothing to do with immutable.js.

OK, reopening to see how we can do this better. In general, when one branch of an intersection fails for whatever reason, we have to try the other branch. In this case the second branch is doomed to fail, and the first fails because of a declaration bug. I agree it doesn't match expectations, since it's clear in our heads that the second branch should never succeed. But to Flow the two branches are equally to blame.

What you're saying made sense at first but on more thought it doesn't.  I can see how for a union type, A | B, if the A branch fails, then it should retry the code assuming the value is of type B.  But with an intersection type A & B, if something fails according to the conditions of A, it's not right to retry assuming the value is just of type B, because the value must also be of type A, right?
The more I think about this the hazier it seems to me.  Obviously for union types flow just needs to check if one branch of the union passes.  So at first I would think that with intersection types, all branches have to pass -- but of course the branch lacking calibration would never pass -- so I'm guessing flow does something more complicated for shape intersections, like combining the props present?
In any case no value of type Props & { calibration: ... } can lack a calibration field, so isn't it a bug if a branch of flow checking believes it doesn't have a calibration?


New version: taers232c.GAMADV-XTD3 version 6.67.27

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Exception during executable launch operation System.AggregateException: One or more errors occurred. (An error occurred trying to start process 'C:\GAMADV-XTD3\gam.exe' with working directory 'D:\TOOLS'. Operation did not complete successfully because the file contains a virus or potentially unwanted software.)  ---> System.ComponentModel.Win32Exception (225): An error occurred trying to start process 'C:\GAMADV-XTD3\gam.exe' with working directory 'D:\TOOLS'. Operation did not complete successfully because the file contains a virus or potentially unwanted software.    --- End of inner exception stack trace ---    at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)

(Automated response - build 643.)

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping @Exorcism0666, I'll get a notification.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Manual Validation ended with:

8:43:43 AM: Detected 1 Defender (with signature version 1.403.2631.0) errors: 8:43:43 AM: Trojan:Win32/Wacatac.B!ml

(Automated response - build 629.)

Hi @Exorcism0666,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 649.)

/AzurePipelines run


Maintainer access to the Plugins Team

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the issue has not already been raised

Issue
This is a core plugin and the Plugins Team doesn't have access to it
CC @Uzlopak if you can change it

@fastify/leads

This is a library. It falls under the @fastify/libraries team.

Checked, and it is part of @fastify/libraries. Closing :)


feat: versioned gateway api

Declares v1 for gateway apis with the following schema: gateway-http-host/version/endpoint
After this change, clients should prefer to call gateway-http-host/v1/endpoint. The non-versioned api with the format gateway-http-host/endpoint is still supported but will likely will deprecate this, or arbitrarily change the apis as necessary
Consuming api from fedimint-ln-client
This PR proposes that the gateway should register a versioned api url as part of LightningGateway payload. The client can

use the api as is, assuming compatibility. This is the current client behaviour.
check the version before constructing compatible payloads
ignore a gateway that doesn't have compatible api version

closes #3904

Will gateway clients will only support 1 version of the gateway API? This works slightly differently than the consensus API versioning which supports a range of versions IIRC but we're comparing REST and JSON RPC so might not be apples-to-apples.
Do we expect that gateway API to serve multiple versions? For instance, once we add v2 would we continue serving v1 (like you do here with old un-versioned routes)?
What happens if a gateway-cli running this version of the code encounters a v2 API? Will they see a generic 404 error or a more helpful message telling them to upgrade?

Will gateway clients will only support 1 version of the gateway API?

Clients can consume multiple versions of the API, but we should start simple where the client assumes a single  / non-versioned gateway API

This works slightly differently than the consensus API versioning which supports a range of versions IIRC but we're comparing REST and JSON RPC so might not be apples-to-apples

More reason to build a JSON RPC for gatewayd and overlay it with this REST API #2293 . Adding this to 0.4.0 or 0.5.0 milestones

Do we expect that gateway API to serve multiple versions? For instance, once we add v2 would we continue serving v1 (like you do here with old un-versioned routes)?

Gatewayd will serve multiple api versions, but we should be hesitant / resistant to add new versions

What happens if a gateway-cli running this version of the code encounters a v2 API? Will they see a generic 404 error or a more helpful message telling them to upgrade?

Not included in current PR but we should include checks for automatic fallback / error handling for incompatible api calls. maybe we can use un-versioned routes to build compat between versions

Rebasing to include https://github.com/fedimint/fedimint/pull/4005 once it's merged should fix flake CI job

@Kodylow , could you take a look at this PR? you've have good context on axum servers

Will go ahead and approve, extra call (as any other changes) can still be addded in a follow-up.


More parameters to make bacula::cient working on Solaris

Please have a look, to the following changes

make config template a paramenter
default to linux package names and paths

Example usage on solaris with CSWbaclua package

  class { 'bacula::client':
     director_server   => 'director',
     director_password => 'xxxxxxx',
     client_package    => 'CSWbaculaclient',
     package_provider  => "pkgutil",
     client_conf       => "/opt/csw/etc/bacula/bacula-fd.conf",
     client_service    => "cswbacula",
     working_dir       => "/opt/csw/var/bacula/working",
     pid_dir           => "/opt/csw/var/bacula/run"
  }

This is pretty awesome.  Thanks.  Can you update the parameters list in the README and maybe add a section about how to manage it in Solaris?

Sure. Do you think it'd make sense to introduce top scope equivalents to all new bacula::client parameters?

@pesimon Sorry I've been lagging on this. It's been fairly busy around here.  Yes, I think all parameters should have top scope equivalents.  My goal is to keep this module configurable from the top scop/ENC level as well as class parameters.  I'm also trying to stick to having a single point of entry, being the bacula class.  This makes delegation to sub-classes easier and makes things like validation easier.


winston: change from ava to tap for testing

It adds coverage testing (including mandating 100% coverage with the
tap "--100" option). As well, I find that node-tap is slightly more
reliable on identifying a particular failing test when an exception
is raised.
coverage
% npx tap --100
 PASS  test.js 13 OK 74.95ms

  🌈 SUMMARY RESULTS 🌈

Suites:   1 passed, 1 of 1 completed
Asserts:  13 passed, of 13
Time:     1s
----------|----------|----------|----------|----------|-------------------|
File      |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
----------|----------|----------|----------|----------|-------------------|
All files |      100 |      100 |      100 |      100 |                   |
 index.js |      100 |      100 |      100 |      100 |                   |
----------|----------|----------|----------|----------|-------------------|

a test failing with an exception
For example, I was working on the ecs-winston-format module and hit an exception in one of the tests:
><EMAIL_ADDRESS>test /Users/trentm/tm/ecs-logging-js/loggers/winston
> standard && ava -v

  ✔ Should produce valid ecs logs
  ✔ Bad ecs log (on purpose)
  ✔ Should not change the message
  ✔ Should not change the log level
  ✔ Should append any additional property to the log message
  ✔ Keys order

  Uncaught exception in test.js

  TypeError: Converting circular structure to JSON

  › JSON.stringify (<anonymous>)
  › $main (eval at build (node_modules/fast-json-stringify/index.js:149:20), <anonymous>:261:51)
  › Format.ecsTransform [as transform] (index.js:52:18)
  › DerivedLogger._transform (node_modules/winston/lib/winston/logger.js:305:29)
  › DerivedLogger.Transform._read (node_modules/readable-stream/lib/_stream_transform.js:177:10)
  › DerivedLogger.Transform._write (node_modules/readable-stream/lib/_stream_transform.js:164:83)
  › doWrite (node_modules/readable-stream/lib/_stream_writable.js:409:139)
  › writeOrBuffer (node_modules/readable-stream/lib/_stream_writable.js:398:5)
  › DerivedLogger.Writable.write (node_modules/readable-stream/lib/_stream_writable.js:307:11)
  › DerivedLogger.log (node_modules/winston/lib/winston/logger.js:244:14)

  ✔ http request and response (req, res keys)
  ✖ test.js exited with a non-zero exit code: 1
  ─

  7 tests passed
  1 uncaught exception
npm ERR! Test failed.  See above for more details.

with ava -c1:
% ./node_modules/.bin/ava -c 1

  Uncaught exception in test.js

  TypeError: Converting circular structure to JSON

  › JSON.stringify (<anonymous>)
  › $main (eval at build (node_modules/fast-json-stringify/index.js:149:20), <anonymous>:261:51)
  › Format.ecsTransform [as transform] (index.js:52:18)
  › DerivedLogger._transform (node_modules/winston/lib/winston/logger.js:305:29)
  › DerivedLogger.Transform._read (node_modules/readable-stream/lib/_stream_transform.js:177:10)
  › DerivedLogger.Transform._write (node_modules/readable-stream/lib/_stream_transform.js:164:83)
  › doWrite (node_modules/readable-stream/lib/_stream_writable.js:409:139)
  › writeOrBuffer (node_modules/readable-stream/lib/_stream_writable.js:398:5)
  › DerivedLogger.Writable.write (node_modules/readable-stream/lib/_stream_writable.js:307:11)
  › DerivedLogger.log (node_modules/winston/lib/winston/logger.js:244:14)

  ─

  7 tests passed
  1 uncaught exception

with tap (after conversion):
% npx tap
 FAIL  test.js
 ✖ Converting circular structure to JSON

  test: http request and response (request, response keys)
  stack: >
    JSON.stringify (<anonymous>)

    $main (eval at build (node_modules/fast-json-stringify/index.js:149:20), <anonymous>:261:51)

    Format.ecsFormat (index.js:52:18)

    DerivedLogger._transform (node_modules/winston/lib/winston/logger.js:305:29)

    DerivedLogger.Transform._read (node_modules/readable-stream/lib/_stream_transform.js:177:10)

    DerivedLogger.Transform._write (node_modules/readable-stream/lib/_stream_transform.js:164:83)

    doWrite (node_modules/readable-stream/lib/_stream_writable.js:409:139)

    writeOrBuffer (node_modules/readable-stream/lib/_stream_writable.js:398:5)

    DerivedLogger.Writable.write (node_modules/readable-stream/lib/_stream_writable.js:307:11)

    DerivedLogger.log (node_modules/winston/lib/winston/logger.js:244:14)
  type: TypeError
  tapCaught: uncaughtException

 RUNS  test.js 1 failed of 11 17s
...
Suites:   0 of 1 completed
Asserts:  1 failed, 9 passed, of 11
Time:     30s
...

other comparison notes
Unfortunately node-tap is much bigger:
% du -sh node_modules
145M	node_modules

vs:
% du -sh node_modules
 64M	node_modules

Both ava and node-tap do a good job presenting code snippets and diffs on assertion failures.
I like that node-tap test files can be run without the runner. I.e. they are run without a special environment. As well the runner runs them out of process, which can help with multiple test files competing with each other.

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #29 opened

Start Time: 2021-01-14T00:56:26.619+0000

Duration: 6 min 56 sec

Commit: 739a23f94da57a3ac21938e62f113a0c8a2cfb4b

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #29 opened

Start Time: 2021-01-14T00:56:26.619+0000

Duration: 6 min 56 sec

Commit: 739a23f94da57a3ac21938e62f113a0c8a2cfb4b


Monitor the "ssl" and "ssl 2 way" with prometheus

What did you do?
Took a look on blackbox_exporter to observer the "ssl" possibility to obtain certificate information, but in my opinion are not so detailed.
What did you expect to see?
The possibility to onitor the "ssl 2 way" the certificate details per expiration, self-signed or not, security, if they have an intermediate certificate if the communication is permitted between to project componets communication between them based on 2 certificates in "ssl 2 way" approach.
Will you try to improve or to have something in the future?
<<Feature request>>
Environment
Dev/ Pre-Production / Production

System information:
CentOS Linux release 7.3.1611 (Core)
Linux 3.10.0-514.6.2.el7.x86_64 x86_64

Prometheus version:
2.0

Alertmanager version:

Prometheus configuration file:

Alertmanager configuration file:

Logs:

This would be a feature request against the blackbox exporter, but I think you're really looking for a cert auditing tool which is not a goal of the blackbox exporter.

I was looking on the internet and based on experience I m trying to have on prometheus something to really help on this matter doesn’t matter if is blackbox_exporter or not. In Centreon or Nagios I could check the certificate or some particularities in a nice view. What I consider and look here is to have in Prometheus something more complex based on the fact that I don’t want to use 3 tools to monitor a server. I want to have it all in prometheus. I don’t need an auditing tool only to know if this will be developed or not in Peometheus “detailed ssl”, “detailed ssl 2 way”, otherwise based in the needs I will choose another solution that covers this needs instead of Prometheus.
Kind regards,


How can I install k2 with version 1.9.dev20211101.

https://github.com/k2-fsa/icefall/blob/14c93add507982306f5a478cd144e0e32e0f970d/.github/workflows/run-pretrained-transducer-stateless.yml#L36
Thanks for the great work before I ask question.
Here is my problem. I want use icefall for stateless transducer experiment, it requires torchaudio 0.10.0, it means we need PyTorch 1.10.0.
But I can't find any specific wheel package version from conda or pip . So I guess we need install from code, but I don't know how to checkout k2 version == 1.9.dev20211101.
Hope someone can help me. Thanks.

Could you first refer to https://k2-fsa.github.io/k2/installation/from_source.html
or
https://k2-fsa.github.io/k2/installation/for_developers.html
to install k2 from source?

So I guess we need install from code, but I don't know how to checkout k2 version == 1.9.dev20211101.

Just using the master of k2 is fine.

Will remove the constraint of using torchaudio 0.10.0 soon so that you can also use PyTorch versions, e.g., 1.7.1

https://github.com/k2-fsa/icefall/blob/14c93add507982306f5a478cd144e0e32e0f970d/.github/workflows/run-pretrained-transducer-stateless.yml#L36
is for CI to run inference, which does not need to compute the transducer loss. Hence, it does not depend on torchaudio 0.10.0.  Currently, only the training code needs torchaudio 0.10.0.

Could you first refer to https://k2-fsa.github.io/k2/installation/from_source.html or https://k2-fsa.github.io/k2/installation/for_developers.html to install k2 from source?

So I guess we need install from code, but I don't know how to checkout k2 version == 1.9.dev20211101.

Just using the master of k2 is fine.
Will remove the constraint of using torchaudio 0.10.0 soon so that you can also use other PyTorch versions, e.g., 1.7.1

Thanks for you reply. I have tried to compile k2, but it failed with some exception like this:
/home/huanglk/anaconda3/envs/k2/lib/python3.8/site-packages/torch/include/torch/csrc/api/include/torch/nn/cloneable.h: In instantiation of ‘std::shared_ptr<torch::nn::Module> torch::nn::Cloneable<Derived>::clone(const c10::optional<c10::Device>&) const [with Derived = torch::nn::CrossMapLRN2dImpl]’:
/tmp/tmpxft_000077ff_00000000-6_pytorch_context.compute_86.cudafe1.stub.c:16:27:   required from here
/home/huanglk/anaconda3/envs/k2/lib/python3.8/site-packages/torch/include/torch/csrc/api/include/torch/nn/cloneable.h:57:59: error: invalid static_cast from type ‘const torch::OrderedDict<std::basic_string<char>, at::Tensor>’ to type ‘torch::OrderedDict<std::basic_string<char>, at::Tensor>&’
/home/huanglk/anaconda3/envs/k2/lib/python3.8/site-packages/torch/include/torch/csrc/api/include/torch/nn/cloneable.h:69:61: error: invalid static_cast from type ‘const torch::OrderedDict<std::basic_string<char>, std::shared_ptr<torch::nn::Module> >’ to type ‘torch::OrderedDict<std::basic_string<char>, std::shared_ptr<torch::nn::Module> >&’

ENV:
gcc version 5.5.0 20171010 (Ubuntu 5.5.0-12ubuntu1~16.04)
cmake version 3.20.5
Python 3.8.12 (default, Oct 12 2021, 13:49:34)
PyTorch 1.10.0
CUDA toolkit 11.1
cuDNN 8.3.1

That same error is mentioned here https://github.com/NVIDIA/apex/issues/805
where it says it can be resolved by upgrading to gcc 7.
Of course, it would be more ideal if the cmake files figured out the problem (I'd have thought that torch's cmake
package or whatever would notice this, if it were an incompatibility).

The same error that prevents GitHub actions from building k2 with torch 1.10 when using ubuntu 16.04.
As Dan suggested, using GCC 7 will fix it; k2 is also using GCC 7 to build conda packages against torch 1.10.
See
https://github.com/k2-fsa/k2/blob/master/scripts/conda/k2/meta.yaml#L26
    - gcc_linux-64=7

Thanks for your suggestion. It resolved by upgrading to gcc 7.


DS-690 - [Admin Panel] Scroll bar is required under the users tab when we open a user through the admin panel

Add scrollbar to the users tab under Admin panel.

@jugnu-appveen


The listr2 version might be a bit behind with the latest support version

Description

Steps to reproduce
Hi there,
I have checked with lint-staged latest version package.json file, and I have noticed that you were using version ^5.0.7 for package listr2, which is a bit old compared with what listr2 latest version which is ^6.3.1.
I noticed that if I am using lint-staged 13.2.1 to do the lint checking which will give me an error, I guess it is because of the listr2 5.0.7 is still using CommonJS mudle system to compile the code, which is display some error similar like below debug log shown:

ONLY if when developer try to do the lint stage check for all files including node_modules.
Thus, I would like to ask is that possible to update the listr2 version on your side?
Debug Logs
Error [ERR_REQUIRE_ESM]: require() of ES Module /home/Workspace/node_adc/node_modules/supports-color/index.js from /home/kamoraes/Workspace/node_adc/.git/hooks/commit-msg not supported.
Instead change the require of index.js in /home/Workspace/node_adc/.git/hooks/commit-msg to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/home/kamoraes/Workspace/node_adc/.git/hooks/commit-msg:8:23) {
  code: 'ERR_REQUIRE_ESM'
}

expand to view
Error [ERR_REQUIRE_ESM]: require() of ES Module /home/Workspace/node_adc/node_modules/supports-color/index.js from /home/kamoraes/Workspace/node_adc/.git/hooks/commit-msg not supported.
Instead change the require of index.js in /home/Workspace/node_adc/.git/hooks/commit-msg to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/home/kamoraes/Workspace/node_adc/.git/hooks/commit-msg:8:23) {
  code: 'ERR_REQUIRE_ESM'
}

Environment

OS:
Node.js:
lint-staged:

Thank you man ~~

cc @damengrandom

Is there any workaround for this while the package didn't get updated?

Is there any workaround for this while the package didn't get updated?

Is there any workaround for this while the package didn't get updated?

For those who are getting the ESM error, remove the lockfile (yarn.lock | package-lock.json) and reinstall all packages. To me, it was the only way to make it work.
It'll break if you install any new package, but just repeating the process will make it work again.

<EMAIL_ADDRESS>has now been released an includes the latest version of listr2. It also dropped support for Node.js 14.


PP-12222 card numbers in cardid

WHAT YOU DID
I've added examples of the bin range data to src/main/resources, defaulted the configuration to look at those files, allowed the configuration to point to files within the classpath, provided worldpay's example cards (from their public documentation) into the format we expect to receive Worldpay bin ranges in.  This should allow us to overwrite the files on their way to deployment and get rid of the git submodule approach.
In theory this should not break any existing behaviour/deployments as each environment previously had to provide the location of these files and if they continue to do so the app should run in the same way as before.  What it allows is for us to remove that configuration and rely on the files built into the jar which will be replaced with real data on concourse.
How to test

Remove any configuration you have for worldpayDataLocation, discoverDataLocation and testCardDataLocation, it should pick up the files I've provided
Add worldpayDataLocation, discoverDataLocation and testCardDataLocation back in and it should pick up the files you specify

Code review checklist
Logging

[ ] Should we log the file location at start?
[ ] Should we log when a file can't be found or is the Exception enough?

Documentation

[ ] When we start using this functionality (closer to the end of the epic) we'll need to update the documentation around how bin ranges are collected, processed and put live.

Introduced default values for worldpayDataLocation, discoverDataLocation and testCardDataLocation which point to newly added files (based on files which already existed in this repo and information from public Worldpay sites)

Outstanding question:
Is there a better way to handle the file paths that are relative to the classpath?  I tried a number of things before deciding to do it the long way round, I'm now comfortable with my approach but if there's a smoother option that I've missed I'd appreciate hearing it.  The relevant sections are the default values and the function I wrote to handle absolute file paths vs classpath relative file paths.

I've tested the cards from the following places:

Our own documentation
http://support.worldpay.com/support/kb/bg/testandgolive/tgl5103.html
https://developer.worldpay.com/docs/access-worldpay/testing/payments-test-values

I've written my own lines in the CSV for these based just on the information on the websites, this is to avoid bringing any metadata which is not public into a public codebase.
I've written these cards into the built-in Worldpay data so that it's usable locally without any risk of it being used in the prod data.


SPEC file for google-compute-engine-oslogin

Can't find spec-file for google-compute-engine-oslogin. Should't it be in specs directory? Or google-compute-engine-oslogin is a product of some other spec-file?

It is currently in its own directory (https://github.com/GoogleCloudPlatform/compute-image-packages/blob/master/google_compute_engine_oslogin/packaging/rpmbuild/SPECS/google-compute-engine-oslogin.spec)... we do need to migrate it and had plans to do so. I'll keep this issue open to track that.

oh, thank you


I didnt figure out where defined @ins instance variables

Considering that this issue is over a year old and lacking crucial information, I'm going to close for now. Feel free to re-open using this template should you need to follow-up.


error CS0012 GraphClient.Extensions.IEntityBase is defined in an assembly that is not referenced

I added 2 extra fields to the General Ledger Setup table and the page.
tableextension 2048890 tableextension2048890 extends "General Ledger Setup"
{
    fields
    {
        field(50000;"Excise Decimal Places";Text[5])
        {
            CaptionML = ENU='Excise Decimal Places',
                        FRB='Nombre Décimales Accises',
                        NLB='Accijnsdecimalen';
            Description = 'T6428';
            InitValue = '2:2';

            trigger OnValidate();
            begin
                CheckDecimalPlacesFormat("Excise Decimal Places");
            end;
        }
        field(50001;"Excise Rounding Precision";Decimal)
        {
            CaptionML = ENU='Excise Rounding Precision',
                        FRB='Précision Arrondi Accises',
                        NLB='Accijnsafrondingsprecisie';
            DecimalPlaces = 0:5;
            Description = 'T6428';
            InitValue = 0.0001;

            trigger OnValidate();
            begin
                MESSAGE(gtxtc_RestartProgram);
            end;
        }
    }
        gtxtc_RestartProgram  : TextConst ENU='You must close the program and start again in order to activate the unit-amount rounding feature.',FRB='Vous devez quitter le programme et redémarrer pour pouvoir activer la fonction d''arrondi montant unité.',NLB='U moet het programma afsluiten en opnieuw starten om de prijsafrondingsfunctie te activeren.';
}

pageextension 2049051 pageextension2049051 extends "General Ledger Setup"
{
    layout
    {
        addafter(Application) //need new tab, not new group in tab
        ////addafter("Unrealized VAT") //= previous field
        {
            group(Product)
            {
                CaptionML = ENU='Product',
                            FRB='Product',
                            NLB='Product';
                field("Excise Decimal Places";"Excise Decimal Places")
                {
                }
                field("Excise Rounding Precision";"Excise Rounding Precision")
                {
                }
            }
        }
    }
}

When I want to publish my extension, I get the following error:
[2018-01-04 10:46:37.72] Sending request to http://localhost:7049/Product/dev/apps
[2018-01-04 10:46:42.21] The request for path /Product/dev/apps failed with code InternalServerError. Reason: c:\ProgramData\Microsoft\Microsoft Dynamics NAV\110\Server\MicrosoftDynamicsNavServer$Product\source\PageExtension\PageExtension2049051.cs(103,13) : error CS0012: The type 'Microsoft.Azure.ActiveDirectory.GraphClient.Extensions.IEntityBase' is defined in an assembly that is not referenced. You must add a reference to assembly 'Microsoft.Azure.ActiveDirectory.GraphClient, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
c:\ProgramData\Microsoft\Microsoft Dynamics NAV\110\Server\MicrosoftDynamicsNavServer$Product\source\PageExtension\PageExtension2049051.cs(103,197) : error CS1061: 'Microsoft.Dynamics.Nav.BusinessApplication.PageExtension2049051' does not contain a definition for 'GetDecimalString' and no extension method 'GetDecimalString' accepting a first argument of type 'Microsoft.Dynamics.Nav.BusinessApplication.PageExtension2049051' could be found (are you missing a using directive or an assembly reference?)

Wait, what?
Nowhere in the page nor table is 'Microsoft.Azure.ActiveDirectory.GraphClient.Extensions.IEntityBase' used and GetDecimalString has no occurences when I search for that function/String.
(There are also other objects in the project, but they have no value for this problem.)
Specially:
When I remove the field "Excise Rounding Precision" from the page, the extension builds and publishes perfectly!
Using Microsoft_Application_11.0.19394.0.app & Microsoft_System_11.0.12925.0.app, no dependencies, Al Language 0.12.15355

Thank you for solving this issue. :)
But how can I get this fix?
On the NAV Development Preview – February Update site is written:
Please note, that the improvements announced in this blog post are not available in Dynamics NAV 2018 and the following cumulative updates of Dynamics NAV 2018.
Guess what I was using when reporting this issue? Dynamics NAV 2018!

You would need to file a support case for NAV 2018 and the fix will be backported to the next CU release.


Include note that script parameters don't overwrite default script values

From Feedback Hub:

If there is a default parameter in the script, we’ll enumerate that in our parameter UI and users can set it, but we don't overwrite the default value (since we never modify the script directly). You can think of this as “pre-populated suggested values” are provided in the UI, but SCCM doesn't provide access to “default” values at run-time. This can be worked around by the Script that is written.

Reference internal bug 515526

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

ID: 7c7d1db7-6812-ff3c-9aac-0f55800ed665
Version Independent ID: a4499cf7-ed5a-710c-66e0-005563b9c015
Content: Create and run scripts - Configuration Manager
Content Source: sccm/apps/deploy-use/create-deploy-scripts.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

This information was added under the parameters section. The changes should be published in the next day or so.
#please-close


Errors when running ios.sh (ios-configurator) in Linux

The following errors can be seen when running the ios.sh file in a linux environment.
--------------------------------------------------------------------------
WSO2 IoT Server is configured via localhost as the product has SSO enabled by default. When configuring WSO2 IoT Server with iOS you need to make it IP based instead of localhost. This script will configure WSO2 IoT Server with the IP

Please enter the IP that you need to replace (if you are trying out IoTS server for the first time this will be localhost
localhost
ios.sh: 201: ios.sh: [[: not found

Please enter your current IP
<IP_ADDRESS>
ios.sh: 210: ios.sh: [[: not found
--------------------------------------
All your  + localhost +  IP's are replaced with  +<IP_ADDRESS>
--------------------------------------
Changing <IoT_HOME>/conf/carbon.xml
sed: can't read : No such file or directory
sed: can't read : No such file or directory
Completed!!
Changing <IoT_HOME>/conf/app-manager.xml
sed: can't read : No such file or directory
Completed!!
Changing <IoT_HOME>/conf/identity/sso-idp-config.xml
sed: can't read : No such file or directory
Completed!!
Changing <IoT_HOME>/conf/identity/sso-idp-config.xml
sed: can't read : No such file or directory
Completed!!
Changing <IoT_HOME>/conf/iot-api-config.xml
sed: can't read : No such file or directory
Completed!!
Changing <IoT_HOME>/repository/deployment/server/jaggeryapps/devicemgt/app/conf/app-conf.json
Completed!!
Changing <IoT_HOME>/repository/deployment/server/jaggeryapps/api-store/site/conf/site.json
Completed!!
Changing <IoT_HOME>/repository/deployment/server/jaggeryapps/portal/configs/designer.json
Completed!!
Changing  <IoT_HOME>/conf/api-manager.xml
found
sed: can't read : No such file or directory
Completed!!
Changing <IoT_HOME>/conf/etc/webapp-publisher-config.xml
sed: can't read : No such file or directory
Completed!!
Changing  <IoT_HOME>/bin/iot-server.sh
Completed!!

This also does not copy the public certificate to the conf/identity/identity-providers/iot_default.xml

Same was observed when run as bash.
$ bash ios.sh


Refactoring Login& Contact component

Refactoring the Login & Contact as per the proposed style.
Note- Please make sure to follow all the guidelines mentioned in the proposed style properly!

@Kajol-Kumari Please assign me this issue. I assure you to complete this as I have completed working on one issue assigned to me.

Would like to work on this issue under SWOC and DWOC.


Ensure all cw20 sends use Binary not Option<Binary>

ex: https://github.com/CosmWasm/cosmwasm-plus/blob/main/packages/cw20/src/msg.rs#L16
This was updated in cw20-base, but I forgot the cw20 spec. Let's check all usage in all packages and ensure they match

Even inside cw20 messages differ :)
cw20-0.6.1/src/receiver.rs:
pub struct Cw20ReceiveMsg {
    pub sender: String,
    pub amount: Uint128,
    pub msg: Binary,
}

cw20-0.6.1/src/msg.rs:
    Send {
        contract: String,
        amount: Uint128,
        msg: Option<Binary>,
    },


Clean dropins

It would be helpful to implement a task in ant as described in WASdev/ci.maven#9 so we can clean the droppins dir if necessary

@jgawor
This issue is still open but it was already merged, is anything else missing?

No, I don't think so.


[Feature - Sparrow Server] Add current block height

Maybe I haven't identified where I can find the current block-height in Sparrow Server 1.7.1 but it would be nice to have the information displayed at the upper right hand corner (behind the Connected status).

Hover your mouse on the connection indicator on bottom right corner. It will display your connection's block height.

In the server version?

I believe @RequestPrivacy is referring to Sparrow Terminal.

Facepalm. Long day.

Added in ff90a2c3.


Module stays invisible if label is initially blank

Using Waybar v0.10.4-1 on Hyprland and Arch Linux, tested on Waybar v0.10.4-6-g003dd3a9 and this still happens
Making a module that would use playerctl to display information about what's playing. If there wasn't anything playing when Waybar started up (i.e when "playerctl status" prints "No players found") the module would not be shown, and the script would call "playerctl status" every 10s until there's a player, which is what's expected. Then the module should appear with the information from playerctl, what happens instead is that the label gets set, but the module stays hidden. I know the label is set because you can hover over it's area and see a tooltip saying whatever the label is saying. If you killed Waybar and started it again while there was a player, the module would be visible and showing the script's output normally, it could even reappear after getting blanked out when the player stops.
As a workaround, I'm making the module show filler text like "playerctl.sh" for 1s when starting, and it works perfectly fine now, but I would rather not do that, so, help me out pls.
tl;dr the module would not appear ever if it started as blank (there being no text) but being given text later. the module can appear and disappear just fine if it was not blank to begin with, but preferably it gets to be blank
Here is the module in the config:
"custom/playerctl": {
        "format": "{}",
        "exec": "$HOME/.config/waybar/playerctl.sh",
        "on-click": "playerctl play-pause"
    },

This is the shell script I have now, with the workaround:
#!/bin/sh

format="{{ title }} - {{ artist }} {{ duration(position) }}"
playing_icon=""
paused_icon=" "
sleep_time=10

# below was just an `echo ""` but changed for workaround
echo "playerctl.sh"
sleep 1

while true
do
    while true
    do
        player_status=$(playerctl status)
        prefix=""
        if [ "$player_status" = "Playing" ]; then
            prefix="$playing_icon"
        elif [ "$player_status" = "Paused" ]; then
            prefix="$pause_icon"
        else
            echo ""
            break
        fi
        info="$(playerctl metadata --format "$format")"
        echo "$prefix$info"
    done
    sleep $sleep_time
done

Still not sure if this is a bug with Waybar or my stuff :p

I can't reproduce any issues using the same Waybar commit. Of course, I've commented out the echo "playerctl.sh". :wink: What I notice is that it can take a while with the 10s sleep but I suppose you already knew that. I'm using this style and config:
#custom-test-output {
    background-color: #268bd2; /* blue */
}

#custom-test-output.disabled {
    background-color: transparent;
}

{
    "layer": "top", // Waybar at top layer
    "position": "top", // Waybar position (top|bottom|left|right)
    "height" : 20,
    "modules-center": ["custom/test-output"],

    "custom/test-output": {
        "format": "{}",
        "exec": "./script-3496.sh",
        "on-click": "playerctl play-pause"
    },
}

Starting waybar without anything playing:

Then, when starting song, after a while:

That said, the fans spin up. I'd probably add a sleep to the inner loop as well. Maybe not 10s, but something.
I don't really have any idea what's wrong, but here are two ideas:

try json. Reason being, while the module does handle single line output, the manpage really says the expected format is text\ntooltip\nclass. This might just be how I feel about it, but I think Json offers better clarity for this than a 3 line output.
I suppose you've considered and discarded the mpris module?

That said, the fans spin up. I'd probably add a sleep to the inner loop as well. Maybe not 10s, but something.

I didn't hear my fans spinning before because another fan was being too loud for me to hear my computer crying.

try json. Reason being, while the module does handle single line output, the manpage really says the expected format is text\ntooltip\nclass. This might just be how I feel about it, but I think Json offers better clarity for this than a 3 line output.

I tried to fix the script with this idea, but I don't know which manpage you're talking about, and also just didn't know how to.

I suppose you've considered and discarded the mpris module?

Yeah, I both considered thoroughly, and also decided to discard the mpris module, that I totally knew about before writing my own thing.
With that being said, I think I'll just start using the mpris module, as it seems more generally functional and reliable than my sh script.
I should read more, and I doubt this is an actual Waybar bug.

I tried to fix the script with this idea, but I don't know which manpage you're talking about, and also just didn't know how to.

You should be able to access the custom module's manpage with man waybar-custom on the command line. Manpages and wiki haven't always been perfectly synced, or sometimes stuff is missing there, still, I think using both is a good place to get information.

I suppose you've considered and discarded the mpris module?

Yeah, I both considered thoroughly, and also decided to discard the mpris module, that I totally knew about before writing my own thing. With that being said, I think I'll just start using the mpris module, as it seems more generally functional and reliable than my sh script. I should read more, and I doubt this is an actual Waybar bug.

:smile: Well, there can be valid reasons to use a custom module where a native module exists, I guess for missing features, bugs, etc. That's why I worded it that way.

Thank you for the clarification!


doc: update regression test section to use Python 3

Signed-off-by: Lorenzo Fontana<EMAIL_ADDRESS>What type of PR is this?
/kind content
Any specific area of the project related to this PR?
/area documentation
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #165
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign fntlnz
You can assign the PR to them by writing /assign @fntlnz in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

I'll put an hold on this until we merge the PR on falco.
/hold

/hold

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leodido
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leodido]

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

LGTM label has been added.  Git tree hash: 1b10078f882019a778a1e90927ac8b882b69e962

https://github.com/falcosecurity/falco/pull/1093 has been merged int
/hold cancel


View restriction permission confusing and poorly documented

When you go to Page->Permissions from the toolbar, you'll see global permissions display like:

A green checkbox under 'Can view', yay, that means that users in the Content Editors group 'can view' the page!
But when you click into the details, the language seems to be the opposite:

Now the view is 'restricted'?
The documentation doesn't clarify matters.

View restrictions can be set-up from the View restrictions formset on any cms page. Once a page has at least one view restriction installed, only users with granted access will be able to see that page. Mind that this restriction is for viewing the page as an end-user (front-end view), not viewing the page in the admin interface!

View restrictions are also controlled by the CMS_PUBLIC_FOR setting. Possible values are all and staff. This setting decides if pages without any view restrictions are:

viewable by everyone – including anonymous users (all)
viewable by staff users only (staff)

It's unclear if 'front-end-view' means 'front-end editing view' or just what normal users see when visiting the site without being logged in as a special admin user (aka published pages only).
It's also unclear without reading the source code how a global settings.py type setting interacts with admin configurable 'global' settings. If CMS_PUBLIC_FOR = 'all' and I click 'view restricted' what does that mean? Obviously we can't check any view permissions for anonymous users. What if I change it to 'staff'?

What actually makes it possible to obtain the link "(global)" in the permissions display?

Note correct documentation URL: http://docs.django-cms.org/en/support-3.0.x/topics/permissions.html


Issue when jquery is loaded in the footer

Hi,
The modal is loaded before jQuery when jQuery is placed in the footer (with roots theme framework by example). Issue is easily fixed by wrapping the content of load_footer_script() (line ~77 to line ~183 in frontend/class-frontend.php) with a pure JavaScript "ready function" (document.addEventListener('DOMContentLoaded', function(){ /* CONTENT */ }); I can issue a pull request if you want or you can implement a "cleaner" version if you prefer.
Thank you,
BT

Hi,
thanks for your hint!
I'm already using a "ready function" (https://github.com/kevinweber/wbounce/blob/master/frontend/class-frontend.php#L82). So I don't know what I should change to solve your issue. Feel free to send a working pull request.
Have you tried the plugin's option "Load script in footer"? You find it on the options page of the plugin.
Best regards,
Kevin

Hey,
Sorry I was not clear in the first message. The problem come from the fact that the ready function need jquery and jquery is loaded after. In the footer the order is:
your modal
your modal javascript (the one that use the ready function)
jquery
modernizr, bootstrap, etc
I assume you hook to the start of the footer (when the option is selected in the option panel) and add your code. I fixed it by replacing the jQuery ready function with a plain and pure JavaScript "ready" function. I will send you a pull request Monday when I'm at work or this weekend if I ever get time to set up a wamp stack and a fresh WordPress (just re installed windows). It's gonna be easier to see what I'm talking about.
Honestly I don't think it affects a lot of people. We used Roots starter theme to create our own and I think it messes up with your plugin when all scripts are loaded in the footer.
Beside that, nice plugin, even if I don't like sites that use popup like that haha. I'm not the one choosing what we do, so for what it does, it does it well.
Thank you,
BT


add vec_type_trait implementation

📌 Checklist before creating the PR

[ ] I have created an issue for this PR for traceability
[ ] The title follows the standard format: [doc/gemini/tensor/...]: A concise description
[ ] I have added relevant tags if possible for us to better distinguish different PRs

🚨 Issue number

Link this PR to your issue with words like fixed to automatically close the linked issue upon merge
e.g. fixed #1234, closed #1234, resolved #1234

📝 What does this PR do?

Summarize your work here.
if you have any plots/diagrams/screenshots/tables, please attach them here.

This PR add VecTypeTrait and related components for vec ld,st,compute
💥 Checklist before requesting a review

[ ] I have linked my PR to an issue (instruction)
[ ] My issue clearly describes the problem/feature/proposal, with diagrams/charts/table/code if possible
[ ] I have performed a self-review of my code
[ ] I have added thorough tests.
[ ] I have added docstrings for all the functions/methods I implemented

⭐ Do you enjoy contributing to Colossal-AI?

[ ] 🌝 Yes, I do.
[ ] 🌚 No, I don't.

Tell us more if you don't enjoy contributing to Colossal-AI.

Better to use Squash and Merge (You might want to set it as default)

Better to use Squash and Merge (You might want to set it as default)

ok


chore: update time-rs

What kind of change does this PR introduce?
Fix build for rust 1.80
What is the current behavior?
https://hydra.nixos.org/build/271860153/nixlog/3
What is the new behavior?
Compile correctly
Additional context
Just update time-rs.

LGTM. Thanks for contributing! 😄


share pdf insted of downloading

instead of downloading as a PDF i want to generate PDF while sharing via social media

i want to cretae one pdf and insert data into that pdf and share that pdf it should not download into the device.please provide any solutions for that.
thanks in advance
regards
manoj

@manojrejinthala Sounds like you added that issue to the wrong plugin. That the plugin for printing HTML stuff.


MigrationStrategy in separate dart file

hello, i couldn't see from here, https://moor.simonbinder.eu/docs/advanced-features/migrations/, but i'm wondering about: how can i have my beforeOpen function in another dart file for example? i want to have that seeded data inserts somewhere else.
with the below statement, i don't know how i can get "todos" known in this new file. it's that new file that i want to import in database.dart, and then provide the function [from that new file] for the beforeOpen function.
await into(todos).insert(const TodosCompanion(title: Value('first todo')));
thanks,

You can create an extension on your database class or alternatively you need to hand over a parameter referencing the database instance this to your migration class/function.
extension DatabaseMigrations on MyDatabase {
   Future<void> runMigrationX() async {
    await into(todos).insert(const TodosCompanion(title: Value('first todo')));
  }
}

thanks @kuhnroyal ; that worked very well with the extension class. thanks for the example code snippet!


Set up Kotlin end-to-end tests

Replicating Swift/Objective-c end-to-end tests in Kotlin/Java.

Depends on https://github.com/envoyproxy/envoy-mobile/issues/1263


Android app with MACE asks for payment even with an active subscription

Android app with MACE asks for payment even with an active monthly subscription, then logs-out the account when declining to pay with the VPN sill connected.
This behavior doesn't manifest on the play-store variant.
PIA with MACE Version 3.9.1.
Android OS version 9

Hey there,
We've made a few attempts at replicating your issue but can't seem to make it happen on our end. Would you be able to provide a little more information to help us out here?
When it "asks for payment" could you let us know exactly what the dialog says (assuming it's a dialog).
What are the exact steps to get this to pop-up? For example, does it happen after trying to connect to a server, when you launch the app, etc.
If you can let us know those two things we can dig a little deeper into this and hopefully reproduce it and resolve the issue.

Hey there,
We've made a few attempts at replicating your issue but can't seem to make it happen on our end. Would you be able to provide a little more information to help us out here?
When it "asks for payment" could you let us know exactly what the dialog says (assuming it's a dialog).
What are the exact steps to get this to pop-up? For example, does it happen after trying to connect to a server, when you launch the app, etc.
If you can let us know those two things we can dig a little deeper into this and hopefully reproduce it and resolve the issue.

@jaspws
Sure, let me give you more information about my OS. I am running Lineage OS based on android 9 with microg to get notifications. My bootloader is unlocked but my ROM is not rooted.
I can't recall the exact text about the notification because now it has auto-renewed my subscription. More or less the text notification pop's up below the status bar a few days before my renewal period asking to renew my payment and all this time I am connected to your service. If I just clear the notifications without tapping on it, everything proceeds as usual but I I tap on it, it opens open the app asking me for renewal and then when declining the payment as I mentioned before it locks me out without logging me out of the service.
This notification appears twice before the renewal period comes to pass.

@jaspws
Sure, let me give you more information about my OS. I am running Lineage OS based on android 9 with microg to get notifications. My bootloader is unlocked but my ROM is not rooted.
I can't recall the exact text about the notification because now it has auto-renewed my subscription. More or less the text notification pop's up below the status bar a few days before my renewal period asking to renew my payment and all this time I am connected to your service. If I just clear the notifications without tapping on it, everything proceeds as usual but I I tap on it, it opens open the app asking me for renewal and then when declining the payment as I mentioned before it locks me out without logging me out of the service.
This notification appears twice before the renewal period comes to pass.

Thanks for that info. I just have one more question.
You mentioned the issue happened on the MACE build, but did you happen to have an account purchased through the Play Store? If that's the case, it's very very unlikely you'll experience this issue again and we're pretty sure it's already been resolved.

Thanks for that info. I just have one more question.
You mentioned the issue happened on the MACE build, but did you happen to have an account purchased through the Play Store? If that's the case, it's very very unlikely you'll experience this issue again and we're pretty sure it's already been resolved.

@jaspws
I created an account last spring via the website with a monthly subscription. I didn't know till December last year that you had builds with MACE in it. I switched to it and experienced the bug.

@jaspws
I created an account last spring via the website with a monthly subscription. I didn't know till December last year that you had builds with MACE in it. I switched to it and experienced the bug.


Is the link attribute a style or functional method?

Links do not work (label1 > Hello World, and label2 > Dog). This is on both device and simulator. I'll keep looking..

@daviddelmonte this is only for styling. you can use this tutorial on how to make the attributed string respond to link.
https://www.hackingwithswift.com/example-code/system/how-to-make-tappable-links-in-nsattributedstring
Cheers 👍

Many thanks!

On Aug 26, 2019, at 12:16, Michael Henry<EMAIL_ADDRESS>wrote:

@daviddelmonte this is only for styling. you can use this tutorial on how to make the attributed string respond to link.
https://www.hackingwithswift.com/example-code/system/how-to-make-tappable-links-in-nsattributedstring
Cheers 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


include LS regression animation in video/tutorial

https://gist.github.com/jfaganUK/0f38d414d9a132d029c366ac310a2a06
http://www.jessefagan.com/content/images/2019/10/advertising_album_sales_regression_animation_06.gif

link 2 is 404 error, we can use link 1 and create a another material for the website.


Validation of user fails on second login and beyond when using LDAP

Steps to reproduce:

Configure LDAP
Login with a valid LDAP username and password
Logout
Try to login with the same LDAP username and password again
Second login and beyond always fails with error:

{ [ValidationError: Validation failed]
  message: 'Validation failed',
  name: 'ValidationError',
  errors:
   { username:
      { [ValidatorError: Expected username to be unique]
        message: 'Expected username to be unique',
        name: 'ValidatorError',
        path: 'username',
        type: 'user defined',
        value: 'username' },
     email:
      { [ValidatorError: Expected email to be unique]
        message: 'Expected email to be unique',
        name: 'ValidatorError',
        path: 'email',
        type: 'user defined',
        value<EMAIL_ADDRESS>} } }

Hi @parente, thank you for sharing this problem with us.
Could you try updating the LDAP plugin to the lastest version: npm update lets-chat-ldap
Before trying again, you'll need to remove the LDAP user already in the database - it might be easier to just drop the database.

If you just want to drop the user:
mongo
use MYDATABASE
db.users.remove({})

@sibartlett<EMAIL_ADDRESS>fixed the problem for me. Thanks!


[Snyk] Upgrade postman-collection from 4.2.0 to 4.2.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade postman-collection from 4.2.0 to 4.2.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-09-11.

Release notes

Package name: postman-collection

4.2.1 - 2023-09-11Release v4.2.1

4.2.0 - 2023-08-03Release v4.2.0

    from postman-collection GitHub release notes

Commit messages

Package name: postman-collection

ee2cc8d Merge branch 'release/4.2.1'
159a22b Release v4.2.1
908e442 Update dependencies
1b3e2e8 Allow description as a property in PostmanVariable to be updated (#1332)
89e1049 Merge branch 'release/4.2.0' into develop

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report

Merging #22 (a00ef0c) into main (93c8b69) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage   67.08%   67.08%
=======================================
  Files          12       12
  Lines         240      240
  Branches       61       61
=======================================
  Hits          161      161
  Misses         68       68
  Partials       11       11

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Compatibility with "Laravel 5 Modules"

Modules generated by https://github.com/pingpong-labs/modules has:

view() function instead of \Theme::view()
templates namespacing fails (example: \Theme::view('mymodule::index')) because it looks for template mytheme::mymodule instead of index template inside mymodule::mytheme

General question: why templates are not fallback to theme-outside files? It would be comfortable to have basic templates (for each module), but with ability to be rewritten in scope of theme.
Is themes extending/fallback planned to be implemented?

I'm interested in this as well. I'll pitch into development where I can but this is outside of my scope of expertise.


Add Queen Ethelburga's Collegiate

@rhysperry111 Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Open question about goint the cluster mode

So, the idea is to have a cluster where each node is able have a chunk of the actors.
If, for example, there are 100.000 actors and 5 instances in the cluster, then there should be 20.000 actors in each server.
Things I need to think (and where help is very welcomed):

What happens if a server crashes? Should we replicate the enqueued messages in order to not lose messages?
Raft? I'm checking it and it seems good.
Should the consensus algorithm sync messages or just where is each actor?
Until what point should the actor inboxes being persistent?
Is it worth it to have a cluster where if any instance dies are messages lost?

Many decisions!

This project is great. I think we can refer to some settings of Orleans:

What happens if a server crashes? Should we replicate the enqueued messages in order to not lose messages?

Using raft, I believe it can solve this problem

Raft? I'm checking it and it seems good.

Yes

Should the consensus algorithm sync messages or just where is each actor?

I think it should be the location of each actor. We can consider using the redis clustering algorithm to calculate the slot according to the actor ID, and the slot distribution of each node of the raft synchronization cluster. In this way, we can control the size of the raft log, which is independent of the number of actors

Until what point should the actor inboxes being persistent?

You mean actor survival time?, I think we can give a fixed cache through the LRU algorithm

Is it worth it to have a cluster where if any instance dies are messages lost?

Yes, I think there is no problem. If you receive, you need to understand the risk of message loss. This is normal, but we need to ensure that if a node leaves the cluster, you need to forcibly shut down the node, because the cluster may be able to transfer the actor of this node to other nodes.
above

Hey @Cloud33! Thanks for the feedback and encouraging!
I'm not having free time to work on this, so I don't know how much I can commit to do anything tbh.
Maybe I will be able to take it in the future.
Again, thanks!