[BUG] Typescript bindings - Nested structs, undefined structs, ByteArray

Describe the bug
Some issues in my generated bindings other than those listed in #2715.
To Reproduce

Clone tournament_component
run, cd tournaments && cd contracts && sozo build --typescript
inspect, contracts/bindings/typescript/models.gen.ts + contracts.gen.ts

or find the file here:

models.gen.ts
contracts.gen.ts

Nested structs
It seems the bindings produced in models.gen.ts do not define types in the Schema when nested.
Current behavior
    AdventurerModel: {
      fieldOrder: ["adventurer_id", "adventurer"],
      adventurer_id: 0,
      adventurer: {
        fieldOrder: [
          "health",
          "xp",
          "gold",
          "beast_health",
          "stat_upgrades_available",
          "stats",
          "equipment",
          "battle_action_count",
          "mutated",
          "awaiting_item_specials",
        ],
        health: 0,
        xp: 0,
        gold: 0,
        beast_health: 0,
        stat_upgrades_available: 0,
        stats: Stats,
        equipment: Equipment,
        battle_action_count: 0,
        mutated: false,
        awaiting_item_specials: false,
      },
    },

Expected behavior
    AdventurerModel: {
      fieldOrder: ["adventurer_id", "adventurer"],
      adventurer_id: 0,
      adventurer: {
        fieldOrder: [
          "health",
          "xp",
          "gold",
          "beast_health",
          "stat_upgrades_available",
          "stats",
          "equipment",
          "battle_action_count",
          "mutated",
          "awaiting_item_specials",
        ],
        health: 0,
        xp: 0,
        gold: 0,
        beast_health: 0,
        stat_upgrades_available: 0,
        stats: {
          fieldOrder: [
            "strength",
            "dexterity",
            "vitality",
            "intelligence",
            "wisdom",
            "charisma",
            "luck",
          ],
          strength: 0,
          dexterity: 0,
          vitality: 0,
          intelligence: 0,
          wisdom: 0,
          charisma: 0,
          luck: 0,
        },
        equipment: {
          fieldOrder: [
            "weapon",
            "chest",
            "head",
            "waist",
            "foot",
            "hand",
            "neck",
            "ring",
          ],
          weapon: { fieldOrder: ["id", "xp"], id: 0, xp: 0 },
          chest: { fieldOrder: ["id", "xp"], id: 0, xp: 0 },
          head: { fieldOrder: ["id", "xp"], id: 0, xp: 0 },
          waist: { fieldOrder: ["id", "xp"], id: 0, xp: 0 },
          foot: { fieldOrder: ["id", "xp"], id: 0, xp: 0 },
          hand: { fieldOrder: ["id", "xp"], id: 0, xp: 0 },
          neck: { fieldOrder: ["id", "xp"], id: 0, xp: 0 },
          ring: { fieldOrder: ["id", "xp"], id: 0, xp: 0 },
        },
        battle_action_count: 0,
        mutated: false,
        awaiting_item_specials: false,
      },
    },

Undefined input structs
I have one case in my contracts.gen.ts where an input struct is not defined (it's not stored in a model).
Current behavior
  const tournament_mock_registerTokens = async (
    snAccount: Account,
    tokens: Array<Token>
  ) => {
    try {
      return await provider.execute(
        snAccount,
        {
          contractName: "tournament_mock",
          entrypoint: "register_tokens",
          calldata: [tokens],
        },
        "tournament"
      );
    } catch (error) {
      console.error(error);
    }
  };

Expected Behaviour
interface Token {
    token: string
    token_data_type: models.TokenDataType
}

const tournament_mock_registerTokens = async (
  snAccount: Account,
  tokens: Array<Token>
) => {
  try {
    return await provider.execute(
      snAccount,
      {
        contractName: "tournament_mock",
        entrypoint: "register_tokens",
        calldata: [tokens],
      },
      "tournament"
    );
  } catch (error) {
    console.error(error);
  }
};

ByteArray
Currently inputs for ByteArray are typed as string in contracts.gen.ts. This is correct for models.gen.ts as they are indexed as so, however for call inputs this results in deserialization errors.
Current behavior
  const tournament_mock_createTournament = async (
    snAccount: Account,
    name: BigNumberish,
    description: string,
    startTime: BigNumberish,
    endTime: BigNumberish,
    submissionPeriod: BigNumberish,
    winnersCount: BigNumberish,
    gatedType: models.Option,
    entryPremium: models.Option
  ) => {
    try {
      return await provider.execute(
        snAccount,
        {
          contractName: "tournament_mock",
          entrypoint: "create_tournament",
          calldata: [
            name,
            description,
            startTime,
            endTime,
            submissionPeriod,
            winnersCount,
            gatedType,
            entryPremium,
          ],
        },
        "tournament"
      );
    } catch (error) {
      console.error(error);
    }
  };

Expected behavior
  const tournament_mock_createTournament = async (
  account: Account,
  name: BigNumberish,
  description: ByteArray,
  startTime: BigNumberish,
  endTime: BigNumberish,
  submissionPeriod: BigNumberish,
  winnersCount: BigNumberish,
  gatedType: CairoOption<GatedTypeEnum>,
  entryPremium: CairoOption<Premium>
) => {
  try {
    return await provider.execute(
      account,
      {
        contractName: "tournament_mock",
        entrypoint: "create_tournament",
        calldata: [
          name,
          description,
          startTime,
          endTime,
          submissionPeriod,
          winnersCount,
          gatedType,
          entryPremium,
        ],
      },
      "tournament"
    );
  } catch (error) {
    console.error(error);
  }
};

Where ByteArray is generated by:
import { byteArray} from "starknet"

const byteArrayDescription = byteArray.byteArrayFromString(tournamentDescription)

Should be solved by #2832 + dojo.js 1.0.7. Please @starknetdev don't hesitate to reopen as necessary.


 Closing.

Github should also be used for producing quality software but since you
broke that rule I didn't think you would care about the others.
On Mon, Feb 10, 2020 at 11:06 PM Lonnie Ezell<EMAIL_ADDRESS>wrote:

Github is for tracking bugs. Closing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bcit-ci/CodeIgniter/issues/5905?email_source=notifications&email_token=AAAXTKDMVRP7RFCZRETNCNTRCIP3VA5CNFSM4KSZZBNKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELLE6SI#issuecomment-584470345,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAXTKAGOJRLXBK3GVPNPJDRCIP3VANCNFSM4KSZZBNA
.


keycloak_quarkus: use absolute path for certificate files

If you want to use a certificate for both Keycloak and Nginx, the cert files are quite unlikely to be under keycloak.home
Even in case you want to keep the under Keycloak path, you can always define the variable:
   keycloak_quarkus_cert_file: "{{ keycloak.home }}/conf/my_keycloak_cert.pem"
   keycloak_quarkus_key_file: "{{ keycloak.home }}/conf/my_keycloak_key.pem"

I am ok with the change, but you'll need to update meta/argument_specs, defaults/main.yml and README as well

This change will break existing configurations, unless they are using the default value. Not sure how to warn users about this

I'm not sure on why CI is failing now

Found the issue here:
https://github.com/ansible-middleware/keycloak/blob/main/molecule/quarkus/converge.yml#L12
(the molecule test playbook was not using default and broke).
The certs are created in the prepare step here:
https://github.com/ansible-middleware/keycloak/blob/main/molecule/quarkus/prepare.yml#L9
Please amend the test config so it passes

LGTM (again) thanks for contributing


python38Packages.asyncwhois: 0.3.0 -> 0.3.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.asyncwhois is: "Python module for retrieving WHOIS information"
meta.homepage for python38Packages.asyncwhois is: "https://github.com/pogzyb/asyncwhois"
meta.changelog for python38Packages.asyncwhois is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.3.1 with grep in /nix/store/d906w94692dkfqdjaxaf9lpzi1zb9acq-python3.8-asyncwhois-0.3.1

directory tree listing: https://gist.github.com/67d42f26a3a16c17d45e403521b5da03

du listing: https://gist.github.com/4c935374c03ef3044e7fecce0d41e8e2

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.asyncwhois
python39Packages.asyncwhois

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d906w94692dkfqdjaxaf9lpzi1zb9acq-python3.8-asyncwhois-0.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.asyncwhois https://github.com/r-ryantm/nixpkgs/archive/3a8aa8dab1aa58dbf056dc7a5e9017332cf4113b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d906w94692dkfqdjaxaf9lpzi1zb9acq-python3.8-asyncwhois-0.3.1
ls -la /nix/store/d906w94692dkfqdjaxaf9lpzi1zb9acq-python3.8-asyncwhois-0.3.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python38Packages.asyncwhois
python39Packages.asyncwhois

Maintainer pings
cc @fabaff for testing.

Result of nixpkgs-review pr 116580 at 3a8aa8da run on aarch64-linux 1

2 packages built successfully:

python38Packages.asyncwhois
python39Packages.asyncwhois


feat(api): implement Page.waitForNetworkIdle()

Which will wait for there to be no network requests in progress during the idleTime before resolving.
One case where this is useful is if you want to take a screenshot after adding a css class that results in more assets being downloaded (background images etc).
refs #3083
TODO

[x] The tests currently use a public url, because for some reason when using "/digits/*.png" the RequestFinished/RequestFailed events aren't fired, and the request isn't removed from _requestIdToRequest, even though the fetch() call resolves. Any idea why? Is this a chromium bug?
[ ] Is firefox support needed too?

any status on this?

I'm also interested in this. Super useful feature when want to do some action after everything has finished loading.

Just found this https://github.com/jtassin/pending-xhr-puppeteer looks exactly what's needed.

I'm trying to implement something similar and also realise that sometimes the RequestFinished/RequestFailed events aren't fired. No idea for the reason yet.

@jschfflr I brought it back up to date, but it still passes locally with the remote urls but fails with the digit ones :/

The tests currently use a public url, because for some reason when using "/digits/*.png" the RequestFinished/RequestFailed events aren't fired, and the request isn't removed from _requestIdToRequest, even though the fetch() call resolves. Any idea why? Is this a chromium bug?

I'll have a look if this is still the cause later

Fixed it :)
But I think there may be a memory leak/bug.
Currently https://github.com/puppeteer/puppeteer/blob/8d1a697fc3ef10300d886beb961a45ea1451c427/src/common/NetworkManager.ts#L423-L434 is not called if the response is from the cache, meaning anything from the cache stays in
https://github.com/puppeteer/puppeteer/blob/8d1a697fc3ef10300d886beb961a45ea1451c427/src/common/NetworkManager.ts#L111
I wonder if there should be a _forgetRequest call in
https://github.com/puppeteer/puppeteer/blob/8d1a697fc3ef10300d886beb961a45ea1451c427/src/common/NetworkManager.ts#L414-L421
?

Thanks for updating the commit!
It does indeed look like a memory leak. But calling _forgetRequest in _onResponseReceived might cause problems because there's also the ResponseReceivedExtraInfo event that might arrive before or after ResponseReceived.
Can you file an issue for that and assign it to me as there's currently some work being done to improve support for ResponseReceivedExtraInfo to unblock rolling the latest Chromium.

No problem, it's nice to finally get this added :)
Here's the ticket: https://github.com/puppeteer/puppeteer/issues/7560. I don't have permission to assign it to you.
Also noticed you turned auto merge on, but it's not approved yet

@tjenkinson I just saw this test flake on the mac bot:
https://github.com/puppeteer/puppeteer/runs/3585119647?check_suite_focus=true
Could you take a look into why that happened and address the issue? Thanks!

That's weird. I wonder if it lagged in a few places and it went over idleTime 🤔
Opened https://github.com/puppeteer/puppeteer/pull/7564 to see if that helps

Yeah I think the bots are not very powerful so timing might be problematic in some places. But it looks like the fix solved it :)


Flickering after opening a window(s)

every time i open other application or tab to other application the taskbar turn back into left taskbar for a sec then turn to center again.

I updated from <IP_ADDRESS> to <IP_ADDRESS> and have seen that too.

Happening in <IP_ADDRESS> too

What build of Windows?

To avoid duplicate issues i'll close this one.
https://github.com/ChrisAnd1998/TaskbarX/issues/298


[BUG] Error popup with eeveeSpotify

Describe the bug
Every time i run eeveeSpotify i have a popup error: Impossible to open the link, etc..
How to fix this error? Thanks

Seems a problem with link tried to open from safari. I have seen a new popup before that about safari link but i made a mistake and clicked on yes. Now i am searching to see again that popup to clicked on no button but also installing again app i can’t see it

Error translate: the link that you tried to open seems to be a Spotify link, but Spotify can’t open it.

This issue has been around for a year and we have yet found the cause. But since it's not actually causing problem, just ignore it.

This issue has been around for a year and we haven't yet found the cause. But since it's not actually causing any problem, just ignore it.

Ok :(

@khanhduytran0 @hugeBlack hi, this is a reply from the eevee dev:
https://github.com/whoeevee/EeveeSpotify/issues/558#issuecomment-2525232670
I know it's not a "BIG" problem/bug but it's a bit tedious honestly and LiveContainer is a really stunning app. Can we try to fix this? Seems that Spotify/LC are trying to open a link. I think you can fix this, please.

I did some research today and fixed this issue. Try latest build of 3.1.0.

I did some research today and fixed this issue. Try latest build of 3.1.0.

I have no words... Fixed!
Are you the best developer in this world?
Thank you so much for your big support!


/vlmeval/dataset/image_mcq.py", line 181, in evaluate
[rank0]:     answer_map = {i: c for i, c in zip(meta['index'], meta['answer'])}
[rank0]:   File "/data/miniconda3/envs/env-3.9.2/lib/python3.9/site-packages/pandas/core/frame.py", line 4102, in getitem
[rank0]:     indexer = self.columns.get_loc(key)
[rank0]:   File "/data/miniconda3/envs/env-3.9.2/lib/python3.9/site-packages/pandas/core/indexes/base.py", line 3812, in get_loc
[rank0]:     raise KeyError(key) from err
[rank0]: KeyError: 'answer'

Can you provide the running command?

If you are evaluating a private benchmark, please check your dataset file xxx. tsv fields correct, such as 'answer' instead of 'ans'. Please refer to Development Doc for More detailed information.

Am runing on MME only got this error

I try to reproduce this error. This is my command torchrun --nproc-per-node=8 run.py --model InternVL-Chat-V1-5 --data MME. Everything is OK. And MME benchmark should correspond to the image_yorn.py file instead of image_mcq.py. Please check your code repo.

Then that's weired, I didn touched anything inside vlmkit, those tsv are automatically downloaded, there must be an key error at some where in some dataset.

Maybe you can try to pull our latest code. We have fixed some known minor bugs. If you still encounter other problems, please do not hesitate to contact us and give us some detailed information to fix them. Thank you.

Hi, @lucasjinreal , I tried to reproduce the problem but failed (I'm evaluating llava_v1.5_7b on MME), with latest main branch.

Duplicate of https://github.com/open-compass/VLMEvalKit/issues/378


Editing pasted text

How can i edit pasted text inside willPaste event function?
I need something like pastedText.bold(), but if i try to use built-in javascript methods it pasting
"text" instead of bold "text".

The event object for the willPaste event has a fragment property which contains a document fragment with the data that will be inserted into the document for the paste. You can modify the contents of this fragment to change what will be pasted.


default method

if there's a method just like :
var baseRequest = require('request');
baseRequest.defaults({
timeout: 5
});
so we may not set the timeout and other params everytime

Interesting, not sure Fetch Spec has this feature or not, but does copying this default option objects help? Or maybe we could expose the request options somehow in the future.
https://github.com/bitinn/node-fetch#options

Fetch spec doesn't have this feature. I don't think it's really that useful since you can just do something like this:
const options = {
	method: 'GET',
	timeout: 0
	// Other options
};

fetch('https://amazon.com', options);
fetch('https://google.com', options);

Please reopen if necessary ;)


Chrono slimes (My take)

New Slime Suggestion

What will this slime look like?

What kind of slime is it?

[X ] Ranchable Slime - A slime that can be ranched
[ ] Utility Slime - A slime that provides some functionality to the ranch
[ ] Pet Slime - A slime that can follow you around and do some tasks

Diet & Favorite Food

Plort

Favorite Toy

Slime Technical Sheet
Type

[ ] Normal – Just your ordinay slime.
[X ] Mutation – Rare versions of other slimes, like the Albino which is a mutation of the Pink Slime.
[ ] Ancient – Slimes that have lived for a very long time, like the gold slime, it is an ancient being almost as old as the planet
[ ] Fossil – Slimes that can only be found as fossils (or in the case of the saber as descendants). It will be possible to use their DNA, from the fossils, to bring them back.
[ ] Godly – Slimes that have great power and can use some energy similar to magic. They are seen as gods by the other slimes as they keep the slime world alive with their energy.
[ ] Special – Slimes like the Tarr or Twinkle slime are seen as special cause they aren’t normal and don’t fit into any other category.

Mutation of what slime? (If you choose Mutation above)

Behavior

[X ] Passive – Does not damage the rancher in any way. The aggitation will not cause difference in behaviours.
[ ] Neutral – Might damage the rancher indirectly. The aggitation might cause aggressive behaviour similar to the rolling of the rock slime.
[ ] Aggressive – Attacks the rancher directly.

Risk for the rancher? (If you choose something other than Passive above)

Activity Pattern

[ ] Diurnal – Only survives in the light / during the day
[ ] Nocturnal – Only survives in the dark / during the night
[X ] Metatunal – Survives in the Dark and Light
[ ] Crepuscular – Is active only during Dusk and Dawn. Disappears during the inactive time.
[ ] Matitunal – Is active only during Dawn. Disappears during the inactive time.
[ ] Vespertine – Is active only during Dusk. Disappears during the inactive time.

Can become a largo?

[ ] Yes - This slime can become a largo with any slime that normally can.
[X ] Specific - This slime can only become a largo with specific slimes.
[ ] No - This slime cannot become a largo.

Largo Specification (If you choose specific above)

Location?

Condition to find?

Environment?

[X ] Any Kind - Anywhere independent of the type of environment
[ ] Land – Anywhere on land, does not need a special condition for it
[ ] Water – Anywhere on water, does not need a special condition for it
[ ] Deep Water – On water, requires a place where the water pressure is higher (deep water)
[ ] Lava – On land, requires Lava or a region that contains lava to survive
[ ] Arid – On land, requires a arid/dry zone to survive
[ ] Other - Make a suggestion of a new environment

Environment Specification (If you choose other above)

Temperature

[ ] Cold – Requires cold temperatures (below 15°C/59°F)
[ ] Hot – Requires hot temperatures (above 30°C/86°F)
[ ] Moderated – Requires moderated temperatures (between 10°C/50°F and 35°C/95°F)
[X ] Indifferent – It doesn’t matter the temperature

Vaccable?

[X ] Yes - Can be vacced normally
[ ] Upgrade - Requires an upgrade to be vacced
[ ] Conditional - Can only be vacced under certain conditions
[ ] No - This slime can't be vacced

Vac Upgrade or Condition (If you choose Upgrade or Conditional above)

Synergies

Possible names for the slime? (Optional)

Additional context

Oh wait
what?

there cannot be any space between [x]

there cannot be any space between [x]

Time to edit again

Just going to reference #62 and close this one.

Just going to reference #62 and close this one.


Connection issues handling

Part of example client:
while True:
    children = await client.get_children(path, watch=True)
    ...
    await client.wait_for_events([WatchEvent.CHILDREN_CHANGED], path)

Client 1 lost connection to the server
Client 2 added new children to the path
Client 1 repaired connection

In this case Client 1 doesn't get event so he won't continue the loop to the next iteration to update children list. Is there any proper way to handle such connection problems?

Ok, thanks!


Document properties for Windows 8.1

@jozefizso : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Export improvements

Hi! Thank you for helping out with Embedded Graphics development! Please:

[x] Check that you've added passing tests and documentation
[x] Add a simulator example(s) where applicable
[x] Add a CHANGELOG.md entry in the Unreleased section under the appropriate heading (Added, Fixed, etc) and appropriate crate (embedded-graphics, simulator, tinytga, tinybmp) if your changes affect the public API
[x] Run rustfmt on the project
[x] Run ./build.sh (Linux/macOS only) and make sure it passes. If you use Windows, check that CI passes once you've opened the PR.

PR description
Tweaks the prelude and top-level crate exports to now look like this:

Top level crate:

As mentioned in one of the commits, I now have to make the draw_target module public for the DrawTarget reexported item in lib.rs to be created as a link by rustdoc.
Closes #358

I would prefer to remove the draw_target and drawable modules entirely, because having them as re-export in the root of the docs doesn't make them more discoverable. I've prepared a commit that moves them to the root:

https://github.com/rfuest/embedded-graphics/tree/export-improvements
It is a breaking change, but all types are in the prelude and if that is used to import these types it won't actually break the users code.

Your change makes more sense, thanks for that. I cherry picked it into this branch, so it's ready for another look. A small PR for once!

Interesting that linkchecker didn't recognize the broken link. I've tried to run this:
linkchecker --check-extern --ignore-url=^http target/doc/embedded_graphics/index.html

This seems to catch the error without causing a lot of http requests.

Very strange behaviour... I incorporated the extra args into build.sh which caught a few more broken links as well. All fixed and pushed. I ran build.sh locally which passes now, so hopefully CI does the same.

Well spotted. Fixes pushed.

Thanks!


The intro paragraphs about push and pull models are confusing.

These paragraphs are confusing:
You can work with the Azure Cosmos DB change feed using either a push model or a pull model. With a push model, a client requests work from a server and has business logic for processing a change. However, the complexity in checking for changes and storing state for the last processed changes is handled on the server.
With a pull model, a server requests work, often requesting it from a central work queue. The client, in this case, not only has business logic for processing changes but also storing state for the last processed change, handling load balancing across multiple clients processing changes in parallel, and handling errors.
Here is why they are confusing:
I am trying to understand how I am supposed to interact from my code with the change feed. While doing that, based on the paragraphs, I have problem trying to answer these questions:

Who is doing the pushing? The change feed, or my code?
Who is doing the pulling? The change feed, or my code?
What does it mean "to request work from X"? Does it mean invoking some method on X (aka pushing a request for work to X) ?
Does requesting work mean "please process this work for me and let me know when you are done" or does it mean "hey, I can do some work for you, please send it to me" ? These are opposite meanings, and it is not clear which one is it.
Who plays the role of server or client here? My code, or the change feed? Or does it flip around depending on the context (perhaps on who is initiating the pull/push) ?
For a moment I thought that the client is my code and server is the change feed, but then it says that in the 2nd paragraph that the server requests work from some central work queue. So now we have three actors here: my code, the change feed, and... central work queue? Who is what?

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

ID: 3eeb8721-2a9d-3972-c431-359909a3c4f2
Version Independent ID: 32f88bb1-00ee-fbf2-a285-b1ceb51e9340
Content: Reading Azure Cosmos DB change feed
Content Source: articles/cosmos-db/read-change-feed.md
Service: cosmos-db
GitHub Login: @timsander1
Microsoft Alias: tisande

@konrad-jamrozik We are checking this and we will get back to you.

Hi @konrad-jamrozik. That paragraph is just intended to be a general introduction to push model vs pull architectures. Could you keep reading the doc and let me know if that covers your questions? I'm happy to cover any remaining questions here.

@timsander1 Here is my current best bet on the meaning:
The client is my code.
The server is the change processor.
In case of the push model (paragraph 1), the change processor pushes the notification to my code there is work to be done. This is written as a client requests work from a server which appears to me to be false? I mean, the client is not requesting anything, the change feed processor is pushing the notification, isn't it?
In case of the client model (paragraph 2), the change processor requests work from the central work queue, but this is internal implementation detail hidden from my client code. I can now query the change feed (or pull from it) to process the change. Thus, in this paragraph requesting work has different meaning.
Am I correct in my interepretations?

@timsander1 even if this is intended as very high level broad introduction, I would still say it would be good to have consistent meaning of the phrase to request work and make sure it makes intuitive sense, that is, if X request work from Y it means that X invokes something on Y, and Y doesn't do anything actively, just receives the call and starts processing. If my thinking is correct, the meaning of requesting work is now opposite to that, causing my confusion.

How about a rewrite like this (diff in bold)
You can work with the Azure Cosmos DB change feed using either a push model or a pull model. With a push model, a server pushes work to a client that has business logic for processing this work. However, the complexity in checking for work and storing state for the last processed work is handled on the server.
With a pull model, the client has to pull the work from the server. The client, in this case, not only has business logic for processing work but also storing state for the last processed work, handling load balancing across multiple clients processing work in parallel, and handling errors.

Ah, I see what you mean. I'll submit a docs update to reword that section tonight. I agree that the meaning of "request work" is a little different in paragraph 1 and paragraph 2 and that is unclear.
But yes, your assumptions are correct. For the second paragraph - "With a pull model, a server requests work, often requesting it from a central work queue." In this case, the "central work queue" is the change feed. With the pull model, you request changes from the change feed at your own pace, the changes aren't "pushed" to your code.

@timsander1 awesome, thx! :)

I like your reword :). Mind if I use that? You're welcome to submit the PR yourself if you prefer

@timsander1 ok I'll submit a PR and refer to this discussion in it.

Thanks @konrad-jamrozik and @timsander1 it looks like the issue is resolved. Closing it now.
#please-close


AIR Runner: Add support for pipeline simulation

Tracing channel data consumption and production as symbolic tokens.
Clang-format.
Runner unit testing infrastructure.
Make air-runner support for loops with multiple tokens; use multi-token for loops to represent for-loop pipelining.

@fifield in this PR I have fixed the issue with linalg op latency estimation.


query/querier: fix sum() inflated values problem

Fixes the problem with sum and inflated values as outlined here: https://github.com/improbable-eng/thanos/issues/922.
The problem was the following: sum selects the last value of each time series in each window and adds the different dimensions up however our code asks for the downsampled sum value which is equal to all samples added up in either a 5m or 1h window, and adding those up, obviously, results in an inflated value. The practical result was that as if we applied sum_over_time(...[5m]) on top. So the fix is to ask for the last sample in each window in the case of sum instead of the aggregated value.
Also adds an E2E test for a typical setup of one Sidecar connected + one or more
Thanos Store nodes. Tests and shows how the whole thing with sum really works.
Testing: wrote a query like sum(kafka_log_log_value{topic="iam"}) with identical Sidecar/Store nodes and selected Max 5m/1h downsampling. With this fix the values look sane, without - not (it jumps up very high after the downsampled data comes into play).

cc @jjneely. It's funny that we've arrived that the same conclusion even though I haven't looked at the commented that you've made in #922.

Nice find. lgtm :+1:

Where is my approve button?  I want to hit it.  :-)  Thanks!

I think we've reached sufficient consensus that this is correct. @bwplotka feel free to still review, but I'll go ahead and merge :)


Check for StandardRB offenses in CI

Hello !
I see there is Rubocop in the project, but it's not used in CI.
This PR adds a Github action to enable it, if you're interested... (and there will be a lot of files to lint :)

Hey! Ugh sorry, I wish you'd have asked me before, just yesterday I came to the conclusion to use standardrb for this... it's actually in the project as a development dependency already, so you could just change that to something like this?
    - name: Run StandardRB
      run: bundle exec standardrb --format progress

Of course !

Thanks a ton! Now I've yet got to find out why Github Actions insists on running checks that aren't there anymore, and doesn't run new checks 😴

Should I remove Rubocop-related stuff too ?

I added e25bb22f92af24f8546d60bfc9794c4d7194c34b in, as it fixes StandardRB offenses
standard: Use Ruby Standard Style (https://github.com/testdouble/standard)
  lib/attractor/cli.rb:57:11: Style/IdenticalConditionalBranches: Move `Attractor::HtmlReporter.new(file_prefix: file_prefix, calculators: calculators(options), open_browser: open_browser).report` out of the conditional.
  lib/attractor/cli.rb:59:11: Style/IdenticalConditionalBranches: Move `Attractor::HtmlReporter.new(file_prefix: file_prefix, calculators: calculators(options), open_browser: open_browser).report` out of the conditional.
  lib/attractor/cli.rb:79:11: Style/IdenticalConditionalBranches: Move `Attractor::SinatraReporter.new(file_prefix: file_prefix, calculators: calculators(options), open_browser: open_browser).report` out of the conditional.
  lib/attractor/cli.rb:81:11: Style/IdenticalConditionalBranches: Move `Attractor::SinatraReporter.new(file_prefix: file_prefix, calculators: calculators(options), open_browser: open_browser).report` out of the conditional.

I'll maybe have time to look at your PRs this afternoon, can't guarantee it though...

I took the liberty and fixed some CI issues in #97


portals dissapeared entirely?

ok so i had a world and i ONLY SWITCHED THE END PORTALS to the little model and it did not work so i went in and did /kill @e and now my portals are gone

portals are entities , they die with /kill , so be careful when trying to clear lag on your creations

The scale box type end portal takes time to load and it does not work with low render distance(<5)

Close because outdated.


Add relays param for fetchEvent and fetchEvents

It would be nice to be able to fetch events from certain relays using these helper functions. I have a use case where sometimes I need to fetch events only from one specific relay.

Or would it make more sense to just make a separate NDK instance with that one relay?

Or would it make more sense to just make a separate NDK instance with that one relay?

No, you would pass in a separate NDKRelaySet. subscribe allows it but fetchEvent(s) don't.
Probably this should be a new arg on these two functions as this is conceptually different than a filter option (ie you'll want to this this for publish and that wouldn't make sense to live as a filter option 😂)

Want to make a PR for this?

@erskingardner I can take a crack at it. At a quick glance, it looks like it's an easy update.


Add Miou_unix.write_string

Similarly to Stdlib.output_string this function is a higher-level function for Miou_unix.write except the offset and length do not have to be provided and are instead calculated to take the whole string.

Remplaced by https://git.robur.coop/robur/miou/pulls/52, the Miou_unix.write infers the length of the given string, you actually can write: Miou_unix.write fd str now. Thanks!

ok why not. Although I'm noticing that the use of Option.value ~default:(String.length str - off) does a call to String.length and a substraction for nothing when the length is already given.
Even if the performance cost is small, I would encourage you to use a regular match construct instead


Check the license compatibility to create ballerina oracledb driver with native dependencies

Description:
$Title
Describe your task(s)
Check the license compatibility to use the thin driver bundled with ballerina oracledb module to provide a db driver to the oracledb users.
Related Issues (optional):

This is already done. Hence closing


New feature: auto height

Category

[x] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[x] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

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

Shell

[x] bash
[x] zsh
[ ] fish

I've been proposing a change in kubectx (#143), to adjust the fzf size within temrinal using --height parameter and providing the number of items plus 2.
After discussing the details, we went to the conclusion that it might be interesting to have --height=auto directly implemented in fzf rather than fiddling around with some workaround to achieve the same result.
Would it be acceptable to offer such a value for automatically adjust height when making an interactive choice within the terminal?

I second this proposal.
If fzf knows the input line count, it can use it to render auto height based on that.
Otherwise specifying —height would require calling the target program twice, which isn’t as performant.
Not to mention, the output can change between the two invocations (one for calculating line count vs one from fzf) to the target command. This can cause consistency issues.  It would be better if fzf handled it.

I like the idea but implementing it is going to be non-trivial because:

fzf reads the input asynchronously; it starts the finder before knowing the exact number of candidates as it was inherently designed to work with long-running processes (e.g. find / | fzf)

And because it has a lot of options that affect the number of items that can fit on the screen.  Properly implementing "auto" height without breaking any of these can be quite tricky.

for i in $(seq 100); do echo $i; sleep 1; done |
    fzf --height 50% --header $'multi-line\nheader' \
        --header-lines 3 \
        --border \
        --inline-info \
        --preview 'echo {}' \
        --preview-window up \
        --margin 2,10 \
        --layout reverse

I'm not really bothered by a few extra lines when using a fixed height (e.g. fzf --height 40%), so this is not my priority.

It would be nice to have an option that chooses the height based on the current cursor location. This way if the user runs fzf after C + l or clear, it would use the full screen. But if the cursor is at the bottom of the screen, it could fallback to the default height passed in (i.e. 40% or 60%). Thoughts?

fzf --height=~100%

https://github.com/junegunn/fzf/blob/master/CHANGELOG.md#0340


docs: Rewrite and expand Fleet Manager API endpoints documentation

Description
The current upstream documentation related to KFM Admin Endpoint was very sparse, which did not allow any user of KFM to understand how the KFM Admin endpoints work.
Aside from that, it contained details specific to how we configure it in Red Hat. The upstream repository of KFM and therefore its documentation should not contain any particularities of the KFM deployment(s) in Red Hat.
This PR rewrites and expands the Fleet Manager API endpoints documentation.
This will help users understand how the KFM admin endpoints work, how to configure them and how to interact with them.
Verification Steps

Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA have been completed
[ ] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has been created for changes required on the client side

I created a jira issue to track work on creating documentation for how KFM Admin API endpoints are configured and how to interact with them for our Red Hat specific deployments in stage and production. The issue is: https://issues.redhat.com/browse/MGDSTRM-11077


mksh: fix license

According to official documentation (http://www.mirbsd.org/TaC-mksh.txt), mksh
licensing changes according to build-time configuration. According to it, the
license was changed.

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

According to official documentation (http://www.mirbsd.org/TaC-mksh.txt), mksh
licensing changes according to build-time configuration. According to it, the
license was changed.

I think we should just supply all possible variations and add a comment to check their documentation for details.

Result of nixpkgs-review pr 117048 at 36a43891 run on aarch64-linux 1

1 package built successfully:

mksh

1 suggestion:

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near pkgs/shells/mksh/default.nix:25:3:
   |
25 |   buildPhase = "sh ./Build.sh -r";
   |   ^

According to official documentation (http://www.mirbsd.org/TaC-mksh.txt), mksh
licensing changes according to build-time configuration. According to it, the
license was changed.

I think we should just list all possible variations and add a comment to check their documentation for details.

Well, I think that there is a case for "multi-licensed works". But, by talking with the upstream developers, the license is indeed a combination of all these things (miros+isc+unicode).


Extremely difficult to find a list of supported datatypes

I'm in the process of setting up an InfluxDB instance using an external Unix time stamp. Due to the Unix time 2038 problem I want to be absolutely sure that the datatypes of that field will be int64.
I can not find any information on the supported datatypes in InfluxDB. The only useful information so far is the fact that once you enter your first value into a field, the datatype is locked and can not be changed in the future.

@nii236 in the Fields section of the line protocol docs.

Updated link


Revert "Make tests more resilient to Skia gold failures and refactor flutter_goldens for extensive technical debt removal (#140101)"

Reverts https://github.com/flutter/flutter/pull/140101
That PR somehow made non-matching gold tests not fail at HEAD.
Fixes https://github.com/flutter/flutter/issues/141880

Blocked by https://github.com/flutter/flutter/issues/140169

https://github.com/flutter/flutter/pull/141427

Looks like CI is failing in relation to the crypto dependency

Failures would be fixed by https://github.com/flutter/flutter/pull/141864, but it's currently blocked due to an infra issue. I will push a workaround to this branch...

I just pushed https://github.com/flutter/flutter/pull/141814/commits/df00fe3e4347aa4be5b7fba01663009ad6bd4bfc, which I hope will fix CI

Investigating failures...

Pushed another commit: https://github.com/flutter/flutter/pull/141814/commits/10076530c003f0baccda1ffbd4751ed7ce01bbfb

We should land https://github.com/flutter/flutter/pull/141888 first to test the current system and see if we can reproduce the issue.

I don't think that section really applies here, because this isn't blocking anyone else's work. The tree is green (indeed that's the problem, it was too green), and we can easily keep an eye on the Skia dashboard to catch unexpected regressions while we know the system isn't fully functional.


[FR] Adding calculation of resistance levels

What's the problem of not having this feature?
Do you have plans to add a calculation of resistance levels? Your toolkit is very complete and full featured but this would be nice to add.
Describe the solution you would like
A function to find the resistance levels

Could you add a source explaining what you mean with this? Definitely possible.

Well I used in the past the following code: https://github.com/yongghongg/stock-screener/blob/main/stock_breakout_demo.ipynb and his explanations to the code

I have introduced Support and Resistance levels based on local maxima and minima. Please have a look here: https://github.com/JerBouma/FinanceToolkit/blob/2119a21d3d9b726e8f4c20bd7e202fc7c441abe0/financetoolkit/technicals/technicals_controller.py#L3139-L3239

Works great, thank you!


Move defaults and calculations from CLI *Options to *Config [part 3]

PR Description
The final PR followed up PRs #4614 and #4627

Move default option values from Validator*Options classes to ValidatorConfig
Move default data-path value from DataOptions classes to DataConfig
Adjust all *OptionsTests to check for equivalence of CLI options and config builder

Fixed Issue(s)

Relates to #4601
Documentation

[x] I thought about documentation and added the documentation label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

One interesting issue is that we're now printing:
Teku is shutting down

when running teku --help. I'm guessing we've wound up adding the shutdown hook too soon

Addressing this in #4675


python312Packages.playwrightcapture: 1.27.4 -> 1.27.5

Diff: https://github.com/Lookyloo/PlaywrightCapture/compare/refs/tags/v1.27.4...v1.27.5
Changelog: https://github.com/Lookyloo/PlaywrightCapture/releases/tag/v1.27.5

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 369645

x86_64-linux

:fast_forward: 1 package marked as broken and skipped:

gnuk

:x: 5 packages failed to build:

lacus
lacus.dist
newlib
newlib-nano
spike

:white_check_mark: 9 packages built:

armTrustedFirmwareTools
fusee-launcher
prowlarr
python312Packages.lacuscore
python312Packages.lacuscore.dist
python312Packages.playwrightcapture
python312Packages.playwrightcapture.dist
python313Packages.playwrightcapture
python313Packages.playwrightcapture.dist


disable parallel classloading

#review @bmangold

👍  , was able to get a passing build with no errors, and smoke tested the deployment


Fix minor doc grammar issues

Closes #19495

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "19495-doc-grammar" git@github.com:dbarboza/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

LGTM, thanks!

ping @MHBauer @vdemeester

LGTM 🐰


Can you please provide the colab version of this project

We will provide the colab version in a few weeks. Thanks for your patience.


enhance(gho): suicide death rate

Add the suicide death rate ratio between males and females, using WHO GHO data.
This dataset has data available since 2000 and 2019.

@lucasrodes it looks like you've manually deployed this dataset which already exists. The problem is that there are automatic deploys that overwrite any manual updates of existing datasets. I've merged this to make ETL CI pass again, please open a new PR if you need to make any changes (@spoonerf please make a review even though it's been already closed)


docs: update distributors-announce file

https://github.com/kubernetes/k8s.io/pull/1822 moves the distributors-announce list to a groups.yaml file under committee-product-security ownership, to ensure relevant reviewers are assigned to PRs updating list membership
This PR updates docs to follow suit

/cc @joelsmith @tallclair

Thanks!
/lgtm
/approve


nested lval support for put

The put proc gives an error if you use a nested field reference as the target.  We should fix this as it's easy now.  We have the technology.

While doing this, ensure that #1932 is addressed too.

While doing this, ensure that #1932 is addressed too.

Verified in zq commit 2294ca24 using the zq-sample-data.
The linked PR has tests that include several examples of this, but just to give an easy one, this used to not work (plus the error message had a typo in it :trollface:):
$ zq -f zson 'head 1 | cut id | put id.comment="my favorite flow"' conn.log.gz
id.comment: put currently supports only top-level field assignemnts

Now it works.
$ zq -f zson 'head 1 | cut id | put id.comment="my favorite flow"' conn.log.gz
{
    id: {
        orig_h: <IP_ADDRESS>,
        orig_p: 39681 (port=(uint16)),
        resp_h: <IP_ADDRESS>,
        resp_p: 3389 (port),
        comment: "my favorite flow"
    } (=0)
} (=1)

Thanks @henridf!


Bump versions for release 0.55.0 - april 2024

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3096/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3096/


file manager pages do not appear

I follow all installation instructions and clear commands
my package version is the master
Laravel 5.5

please, what I can do to solve that?

Didn't you forget to register the routes?
Maybe you have disabled the setting in the config and didn't register your own instead?
    /*
    |--------------------------------------------------------------------------
    | Routing
    |--------------------------------------------------------------------------
     */

    'use_package_routes' => false,

Set it to true
    /*
    |--------------------------------------------------------------------------
    | Routing
    |--------------------------------------------------------------------------
     */

    'use_package_routes' => true,

Or Add this line to your routes/web.php
    Route::group(['prefix' => 'filemanager', 'middleware' => ['web', 'auth']], function () {
        \UniSharp\LaravelFilemanager\Lfm::routes();
    });


error handling in chat sample

refs: https://github.com/microsoft/vscode/issues/213710
@jrieken @roblourens the ugly part is this code snippet
if (err.cause instanceof Error && err.cause.message.includes('off_topic')) {
Could we have a specific error.code for off_topic requests?

For reference, here's how the err object looks like

Good question. Makes me wonder if the error throwing is too eager (this is us/me) and if in such cases we should do something else. Or yeah, have one error type for "filtered" but I believe this usually isn't an error reason but just a stop reaons

Makes me wonder if the error throwing is too eager (this is us/me) and if in such cases we should do something else. Or yeah, have one error type for "filtered" but I believe this usually isn't an error reason but just a stop reaons

I think that every extension would handle this in a similar way. And might be better if the model just responds with "Can not help with off topic questions".
Error throwing should happen when model is not available, no more requests etc...


Replacing SSL Cert with Let's Encrypt

Hi,
there is an issue replacing the SSL Cert after 3 months:
TASK [letsencrypt : Create a challenge for ocp2.kimmlingen.org using a account key file.] ***
fatal: [host]: FAILED! => {"changed": false, "msg": "Failure downloading https://acme-v02.api.letsencrypt.org/directory, Request failed: <urlopen error [Errno 101] Network is unreachable>", "other": {}}
Any Idea what that my cause?
 Time deviation with my chronyc of 6 minutes has caused this problem. After correcting it the playbook run smoothly through.


Invalid Refresh Token when using Refresh Token with Device Tracking

I followed the examples for Authentication and I was able to get it to retrieve an access token and refresh token. When trying to use toe refresh token to reauthenticate, it is failing if I have device tracking turned on.  I came across #75 and it looks like someone else had the same issue but never replied with their result from testing. Seems to still be broken if Device Tracking is turned on and returns Amazon.CognitoIdentityProvider.Model.NotAuthorizedException: Invalid Refresh Token.

Appears to be an issue which is not yet fixed (e.g. https://github.com/aws/aws-sdk-net-extensions-cognito/issues/73). This needs to be prioritized by the team.

@ashishdhingra  thank you for the response. This issue was logged about 8 months ago, how does one get it to be prioritized?

Really curious if this is being worked on or if it will be worked on?

I had a similar issue that is described here and in #75. In my case, our user pool is tracking devices. To get the Refresh Token to re-authenticate, I can to confirm the device id first, and attach it to the user, before calling .StartWithRefreshTokenAuthAsync. Here is a sample of the code I used with comments.
`
public async Task LoginTest() {
        //See:https://docs.aws.amazon.com/sdk-for-net/v3/developer-guide/cognito-authentication-extension.html
        //See: https://github.com/aws/aws-sdk-net-extensions-cognito#authenticating-with-secure-remote-protocol-srp

        //Create anonymous credentials to create a Cognito Identify Provider
        AmazonCognitoIdentityProviderClient provider =
            new AmazonCognitoIdentityProviderClient( new Amazon.Runtime.AnonymousAWSCredentials() );

        //Set up the initial request when we don't have a refresh token. Obviously assumes an existing user.
        CognitoUserPool userPool = new CognitoUserPool( poolID, clientID, provider );
        CognitoUser user = new CognitoUser( username, clientID, userPool, provider );
        //This is the only time we need the user's password (assuming the soon to be obtained refresh token stays valid)
        InitiateSrpAuthRequest authRequest = new InitiateSrpAuthRequest() {
            Password = password
        };

        //Send the request to Cognito for authentication
        AuthFlowResponse authResponse = await user.StartWithSrpAuthAsync( authRequest ).ConfigureAwait( false );

        var accessToken = authResponse.AuthenticationResult.AccessToken;
        var refreshToken = authResponse.AuthenticationResult.RefreshToken;
        var deviceKey = authResponse.AuthenticationResult.NewDeviceMetadata.DeviceKey;

        //Now we can get AWS Credentials for the user
        CognitoAWSCredentials credentials =
            user.GetCognitoAWSCredentials( identityPoolID, RegionEndpoint.USEast1 );

        //Above, when we made the StartWithSrpAuthAsync Call, the call also generates a temporary 'Device'
        //Now need to tell Cognito to confirm this temporary device.

        //Device Verification happens locally. the Salt and password verifier gets sent to Cognito in the next .ConfirmDeviceAsync() call
        var deviceVerifier = user.GenerateDeviceVerifier(
            authResponse.AuthenticationResult.NewDeviceMetadata.DeviceGroupKey,
            myRandomDevicePassword,
            username );

        ConfirmDeviceResponse confirmDeviceResponse = await user.ConfirmDeviceAsync(
            accessToken,
            deviceKey,
            "Bob", //this is as creative as I got in naming my device
            deviceVerifier.PasswordVerifier,
            deviceVerifier.Salt );

        //Now attach the newly attached Device to the user. We need this step to successfully complete the StartWithRefreshTokenAuthAsync() call
        //NOTE There may be a better way to do this, but method here is to first create a generic Device, with the known
        //device key. Then Use GetDeviceAsync() to pull the real details from Cognito
        CognitoDevice device = new CognitoDevice(
            deviceKey,
            new Dictionary<string, string>(),
            DateTime.Today,
            DateTime.Today,
            DateTime.Today,
            user );
        await device.GetDeviceAsync();
        user.Device = device;

        //Now pretend we need to fast foward in time and refresh the tokens

        //See: https://github.com/aws/aws-sdk-net-extensions-cognito#authenticating-using-a-refresh-token-from-a-previous-session

        //I'm actually not sure what this step does ..
        user.SessionTokens = new CognitoUserSession( null, null, refreshToken, DateTime.UtcNow, DateTime.UtcNow.AddHours( 1 ) );

        //Create the refresh request object
        InitiateRefreshTokenAuthRequest refreshRequest = new InitiateRefreshTokenAuthRequest() {
            AuthFlowType = AuthFlowType.REFRESH_TOKEN_AUTH
        };

        //CAll Cognito to refresh the token
        AuthFlowResponse authResponse2 = await user.StartWithRefreshTokenAuthAsync( refreshRequest ).ConfigureAwait( false );

        //Now we have a new accessToken and a new refreshToken, both of which need to be re-saved
        var accessToken2 = authResponse2.AuthenticationResult.AccessToken;
        var refreshToken2 = authResponse2.AuthenticationResult.RefreshToken;

        //Again, we can get new credentials for signing API requests
        CognitoAWSCredentials credentials2 =
            user.GetCognitoAWSCredentials( identityPoolID, RegionEndpoint.USEast1 );

        var restAPICredntials2 = await GetRestAPICredentials( user.SessionTokens.IdToken );
    }

`

Yes, attaching the device before initiating the refresh token works on our end. My apologizes for late response.


Add info on daily reports for server

Hi,
This PR follows this discussion https://lists.osgeo.org/pipermail/qgis-psc/2018-June/006331.html (with a slight delay).
Actually, the aim is to talk about daily reports generated for the server (OGC tests and performance reports).
This PR is not ready to be merged. In fact, I would like to:

wait for the dedicated server and deploy WMS CertifSuite and PerfSuite on it
wait for the WFS CertifSuite to be ready

Then, I'll update http links accordingly.
Otherwise, I think that 2 subdirectories should be created for OGC tests:

http://test.qgis.org/ogc_cite/wms_130/
http://test.qgis.org/ogc_cite/wfs_110/

In the same way as for OGC tests, http://test.qgis.org/perf_test/ directory should probably be splitted:

http://test.qgis.org/perf_test/ms_perfs/
http://test.qgis.org/perf_test/graffiti/

I'm not a good writer, so all critics are welcomed :).

Thanks for the review @yjacolin!
I updated the documentation accordingly.


Adds m2m route to WSGI.py

@DanielJMaher Please review.  Critical change to ensure m2m route is available to uwsgi processes.


Copy edits for name of venue, adds definition of CFP

Minor copy edits for header, corrects name of venue (using French capitalization rules)

@choldgraf I don't have permission to merge my changes in. Can you please merge?

Sure thing 👍


Enable remote image cache for prawn-svg

Enable the remote image cache for prawn-svg so that remote images are only downloaded once per document. Alternatively, we may want to add integration with open-uri-cached so that these images are cached between runs as well (with smart update detection). We're shooting for the most optimal behavior.

We should also link this cache between the main and scratch document to avoid having two caches. The same is true for the raster image cache (which is already enabled by default).

We can start by simply enabling the option if the cache-uri attribute is set. I think prawn-svg should integrate it's cache with open-uri/cached, or expose the cache methods. Otherwise, I don't see a way for a deeper integration.


Removing remote dependency

Removing remote dependency on http://d3js.org/d3.v3.min.js?05022016.  Saved the file as part of the code base.  Also removed js/d3.min.js and js/d3.layout.js which look like they are no longer used.

Nice work, figuring this out so quickly


Modulation Mixer and Audio Mixer Param Names

Let wait until after optuna or this will be a pain for me.


How can the specification implementer support the new version？

What is the question:
There is a big difference between the new version and the old version of the specification. How can the specification implementer support the new version?

Hi @skyshaokai good question. As our specification matures we will have these kinds of growing pains but we need
to make sure to keep them to a minimum.
Currently we are at release v0.6 and are planning v0.7 for this summer. We plan to start v1.0 release candidates shortly after that. I think it is realistic that until the spec reaches v1.0 that can be lots of updates to the language given community feature and improvement requests.
We do have plans to shorten our release schedule before v1.0 however so each release would include a smaller set of updates/changes which should make updates for implementors easier (and us as well). In addition we need to do a better job limiting breaking changes (especially around removing existing features).
One thing that would really allow us to become more efficient with all this is community help. We are a 100% community-driven project and its success depends on community contributions.
Having more hands on deck to help us with defining and updating the roadmap, updating sdks, website, vscode plugin, etc etc would be really nice to have. We are looking for community contributors to not only raise issues or add improvements but also help with leading and helping maintain and improving all parts of our ecosystem.
So to answer your question, we have several plans in place to set up a much better release structure going forward and are working hard on it. At the same time we are looking for help to realize these plans faster. Any help is much appreciated!

Thank you for answer.


Menu box overlaps with the statistics info

Open Settings panel -> Misc tab -> enable Show statistics
Let's move the panel a few px up:)

This is likely due to window resolution and not really fixable for every case, so let's forget about it.


[Android] general fixes

updated docs: https://github.com/lwhiteley/react-native-facebook-login/tree/master/android
Changeset:

adding feature for setting login behavior for android
onLoginFound and onLoginNotFound events were added
remove enforcement of email permission, permissions must be passed in button now
add buttonView Property to override default view

Breaking changes:

getCurrentToken has been changed to getCredentials
getCredentials returns an object instead of a string
returned data object now has credentials sub object that has token information similar to IOS version

A couple caveats currently:
LoginBehaviors enum seems to be diff from IOS
Mapping was done but there are still two diferrences :

android has no Browser behavior (IOS does)  [if used, this will default to Native]
android has a NativeOnly behavior (IOS doesn't)

// android interpretation of loginBehaviors
// these will map to the android sdk LoginBehavior enum
FBLoginManager.LoginBehaviors = {
   SystemAccount:"DEVICE_AUTH"
   NativeOnly:"NATIVE_ONLY"
   Native:"NATIVE_WITH_FALLBACK"
   Web:"WEB_ONLY"
}

Should resolve/partially address the following issues:
#103  (Partially addressed, IOS doesn't have this)
#81
#55
#86
#77
#91
#100
#73
if testing, please ensure to be aware of breaking changes

This change is

@skizzo @jamesone test this if you are using the android version please.
@magus where you at?
Can you give some context into the Login behavior enum for the IOS version. seems to be different in the android SDK.

@satya164 you can review this as well

@lwhiteley - Not using Android so can't test. Am on IOS.

I wish to try this commit, but I can't find the way to let me test it, I try to npm install https://github.com/lwhiteley/react-native-facebook-login.git OR npm install https://github.com/magus/react-native-facebook-login.git also fail. Can you show me the way to test this? thanks
I would like to test it on the issue #99 and #92

@vshy108 npm install https://github.com/lwhiteley/react-native-facebook-login.git is the way to go
if the module is already in your package.json then you could just replace the version with https://github.com/lwhiteley/react-native-facebook-login.git
then run
npm update react-native-facebook-login

@lwhiteley I'll try this right now! I'll be back with feedbacks soon!

@lwhiteley Overall looks good to me.

@satya164 thanks ill update the doc, that snippet looks more precise

I'm getting a 'Insufficient permissions' error when I try to login! Any tips on this?

@rafaelv90 if you don't specify permissions in the FBLogin button then you will get that.
before we were defaulting with email, however IOS doesn't do that and that was a issue for portability.
Please refer to the breaking changes in the main body of the PR.
Let me know if that helps

@lwhiteley Yup! That worked! Thanks!
It's working very well in Android, although in iOS when I try to login, a Safari bar is open and after the authorisation it won't close and come back to app!

Thanks for testing @rafaelv90
Im not familiar with the IOS side as yet actually but i assume that may have to do with the IOS sdk and how it works with the IOS platform.
if there isn't an issue, please open one and @magus or someone in the community can better assist

@lwhiteley Worked! I used the login behaviour like this:
loginBehavior={FBLoginManager.LoginBehaviors.SystemAccount}
It uses the Facebook account logged in the iOS device to authorise my app!
Do you know if this behaviour works fine on Android too? Because in Android a modal is opened and closed without throwing some conclusive errors!

@rafaelv90 I havent tested all the behaviors .. just the ones i needed really.
if its not working in android it may be sdk and platform related as well.
hence the reason i wanted folks with the actual issues to test and make recommendations

@lwhiteley I see! Changed the behaviour to:
loginBehavior={FBLoginManager.LoginBehaviors.Web}
It works fine on both platforms and for now it's suit my needs! Thanks for all the help!

@rafaelv90 can you also list the issues that you tested and confirmed are working.
thanks

@lwhiteley The onLoginFound and onLoginNotFound events are working very well both in Android and iOS.
The buttonView property works well, but sadly only for Android at this point! Right now, the only thing I miss is a way to customize the login button for both platforms with the same piece of code!

thanks @rafaelv90
buttonView property is really new.
but we can agree it is very necessary because each project has varying needs for how it wants the button to look
The IOS version may have it in the near future
We just need more contributors @magus  and I wont have all the time in the world to create features :+1:
as you can see these fixes took a while to arrive

@lwhiteley Yeah! It's great already! I'll try some customisations and maybe create a pull request thanks!

@lwhiteley thanks for this nice feature,
then I got a minor UI issue in iOS env, because there is no custom icon feature for iOS yet, which makes it unconsistent to the Android version.
do you have plan to changed the code for iOS ?

correct @weixingsun
There is currently an inconsistency because this feature is new
The difference is that the IOS version is a native button while the Android version is a button that mocks the behavior of the native button so the android version has a bit more freedom.
My initial thoughts on this is to expose two buttons
something like
var {FBLogin, FBLoginNative} = require('react-native-facebook-login');
so FBLogin would be more customizable for both platforms (when the FBLoginManager for both sides match up) and people who just want the native button can use the other API.
Just an initial idea though.
ill have to let @magus weigh in since he is the primary maintainer for the IOS side

Thanks a lot, yes, it's totally reasonable, Let's wait for @magus .

Currently pointing to this PR from my package.json
Waiting for  @magus to merge that
Great work @lwhiteley 👍

Hello,
There's a function to login with behaviour with loginManager?
Thx


Helm install fails due to CRD conflict in otherwise empty cluster

Struggling with a strange issue installing maesh on a fresh k8s 1.17.0 cluster with helm3.0.1. Maesh chart repo updated prior to running install command. Helm fails with "Unable to continue with install: existing resource conflict: kind: CustomResourceDefinition, namespace: , name: traffictargets.access.smi-spec.io" - this CRD (or any other cmi-spec CRD) does not exist in the cluster prior to running helm install.
$ helm version
version.BuildInfo{Version:"v3.0.1", GitCommit:"7c22ef9ce89e0ebeb7125ba2ebf7d421f3e82ffa", GitTreeState:"clean", GoVersion:"go1.13.4"}

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.0", GitCommit:"70132b0f130acc0bed193d9ba59dd186f0e634cf", GitTreeState:"clean", BuildDate:"2019-12-13T11:52:32Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.0", GitCommit:"70132b0f130acc0bed193d9ba59dd186f0e634cf", GitTreeState:"clean", BuildDate:"2019-12-07T21:12:17Z", GoVersion:"go1.13.4", Compiler:"gc", Platform:"linux/amd64"}

$ helm repo update
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "maesh" chart repository
Update Complete. ⎈ Happy Helming!⎈

No smi-related crds are present in the cluster prior to installing. This is a 100% fresh kubeadm-deployed cluster with calico and traefik being the only additions as part of cluster standup:
$ kubectl get crd
NAME                                          CREATED AT
bgpconfigurations.crd.projectcalico.org       2019-12-20T13:20:41Z
bgppeers.crd.projectcalico.org                2019-12-20T13:20:41Z
blockaffinities.crd.projectcalico.org         2019-12-20T13:20:41Z
clusterinformations.crd.projectcalico.org     2019-12-20T13:20:41Z
felixconfigurations.crd.projectcalico.org     2019-12-20T13:20:41Z
globalnetworkpolicies.crd.projectcalico.org   2019-12-20T13:20:42Z
globalnetworksets.crd.projectcalico.org       2019-12-20T13:20:42Z
hostendpoints.crd.projectcalico.org           2019-12-20T13:20:42Z
ingressroutes.traefik.containo.us             2019-12-20T13:24:17Z
ingressroutetcps.traefik.containo.us          2019-12-20T13:24:17Z
ipamblocks.crd.projectcalico.org              2019-12-20T13:20:42Z
ipamconfigs.crd.projectcalico.org             2019-12-20T13:20:42Z
ipamhandles.crd.projectcalico.org             2019-12-20T13:20:42Z
ippools.crd.projectcalico.org                 2019-12-20T13:20:42Z
middlewares.traefik.containo.us               2019-12-20T13:24:17Z
networkpolicies.crd.projectcalico.org         2019-12-20T13:20:42Z
networksets.crd.projectcalico.org             2019-12-20T13:20:42Z
tlsoptions.traefik.containo.us                2019-12-20T13:24:17Z

Trying to install maesh as the first command after the previous results in failure:
$ helm install maesh maesh/maesh
Error: rendered manifests contain a resource that already exists. Unable to continue with install: existing resource conflict: kind: CustomResourceDefinition, namespace: , name: traffictargets.access.smi-spec.io

After the above runs, the trafficsplit and traffictargets crds are present in the cluster:
$ kubectl get crd
NAME                                          CREATED AT
bgpconfigurations.crd.projectcalico.org       2019-12-20T13:33:23Z
bgppeers.crd.projectcalico.org                2019-12-20T13:33:23Z
blockaffinities.crd.projectcalico.org         2019-12-20T13:33:24Z
clusterinformations.crd.projectcalico.org     2019-12-20T13:33:24Z
felixconfigurations.crd.projectcalico.org     2019-12-20T13:33:24Z
globalnetworkpolicies.crd.projectcalico.org   2019-12-20T13:33:24Z
globalnetworksets.crd.projectcalico.org       2019-12-20T13:33:24Z
hostendpoints.crd.projectcalico.org           2019-12-20T13:33:24Z
httproutegroups.specs.smi-spec.io             2019-12-20T13:49:11Z
ingressroutes.traefik.containo.us             2019-12-20T13:35:48Z
ingressroutetcps.traefik.containo.us          2019-12-20T13:35:48Z
ipamblocks.crd.projectcalico.org              2019-12-20T13:33:24Z
ipamconfigs.crd.projectcalico.org             2019-12-20T13:33:24Z
ipamhandles.crd.projectcalico.org             2019-12-20T13:33:24Z
ippools.crd.projectcalico.org                 2019-12-20T13:33:24Z
middlewares.traefik.containo.us               2019-12-20T13:35:48Z
networkpolicies.crd.projectcalico.org         2019-12-20T13:33:24Z
networksets.crd.projectcalico.org             2019-12-20T13:33:24Z
tcproutes.specs.smi-spec.io                   2019-12-20T13:49:11Z
tlsoptions.traefik.containo.us                2019-12-20T13:35:48Z
trafficsplits.split.smi-spec.io               2019-12-20T13:49:11Z
traffictargets.access.smi-spec.io             2019-12-20T13:49:11Z

Helm template only produces one crd item.
helm template maesh/maesh | grep -C 5 traffictargets.access
---
# Source: maesh/templates/smi/smi-access.yaml
apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  name: traffictargets.access.smi-spec.io
spec:
  group: access.smi-spec.io
  version: v1alpha1
  scope: Namespaced
  names:

Is anyone able to shine a light on what's actually failing here? My powers of reasoning are failing me completely.

Same issue for me.

Hey guys,
I'll give it another go and try to reproduce. :)

Hello all!
I can confirm that behavior and let's say its "intended". That chart is helm v2 compatible. If you try to install it with Helm v3, that issue will happen. Our Helm v3 compatible chart is already in master and will be released with Maesh v1.1.
I'll add a note to the docs for the time in between.
Sorry for the inconveniences!


Debug/Test information is missing

Information on how to debug/test the protocol launch needs to be added to this page.
Something like this answer on StackOverflow: https://stackoverflow.com/questions/23110487/how-to-debug-windows-store-app-onactivated-event

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

ID: efc62851-f8af-dde5-2e95-03d59a25ad81
Version Independent ID: bf635b1d-f0b7-1fad-2445-ef9044e65a10
Content: Handle URI activation - UWP applications
Content Source: windows-apps-src/launch-resume/handle-uri-activation.md
Product: uwp
Technology: processes-and-threading
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

Hello. We apologize for the delay in resolving this issue. As we transition to a new feedback system, we are closing all outstanding issues. Any issues that can be addressed will be tracked using a different internal tool, and we regret that we won't be able to provide further updates on this particular matter. We value your input and assure you that all feedback is reviewed. We recommend visiting the Microsoft Q&A site, where your issue may have already been discussed.


Update tunnelIdentifier in capabilities

If a tunnelIdentifier is set in sauceConnectOpts, the same tunnelIdentifier needs to be set for each capability.  Otherwise, the tests fail.
This PR updates lib/sauce-launch-service.js so that tunnelIdentifier is set for each capability if one was set in sauceConnectOpts

Published as v0.4.0


Mariosmsk fix bug getdemandname

Merged upstream dev into branch, and resolved merge conflicts.

Codecov Report

Merging #1 into fix_bug_getdemandname will increase coverage by 0.4%.
The diff coverage is 73.73%.

@@                   Coverage Diff                    @@
##           fix_bug_getdemandname       #1     +/-   ##
========================================================
+ Coverage                  69.84%   70.24%   +0.4%
========================================================
  Files                         22       22
  Lines                       9890     9882      -8
========================================================
+ Hits                        6908     6942     +34
+ Misses                      2982     2940     -42

Impacted Files
Coverage Δ

src/report.c
73.78% <ø> (ø)
:arrow_up:

src/quality.c
95.93% <ø> (ø)
:arrow_up:

src/hydsolver.c
65.78% <ø> (ø)
:arrow_up:

src/smatrix.c
98.51% <ø> (ø)
:arrow_up:

src/qualroute.c
88.28% <ø> (+1.67%)
:arrow_up:

src/hash.c
82.05% <ø> (ø)
:arrow_up:

src/rules.c
34.25% <ø> (ø)
:arrow_up:

src/mempool.c
91.48% <ø> (ø)
:arrow_up:

src/epanet2.c
5.71% <0%> (ø)
:arrow_up:

src/hydstatus.c
73.96% <0%> (ø)
:arrow_up:

... and 13 more

Continue to review full report at Codecov.

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

@Mariosmsk if you merge this PR, it will fix the conflicts between OWA dev and your branch. then I can merge your PR into OWA dev. Make sense?


Add logic to strip the special char for PA

This PR is a rush to fix an issue with bad data that STRAC is sending to us. We have made repeated attempts to get this matter addressed by STRAC, and they are currently working on it, but PA wants this addressed by no later than Friday. I do not view this as the final fix. I think we should instead move the stripping of characters to the ingest side so we can specify things like stripping of vertical tab characters from senders, or the \u0019 from STRAC, etc.  Therefore I have done this instead.
In specific, the character we are stripping for PA is the \u0019 I noted above, which is a unicode control character for END OF MEDIUM, typically reserved for credit card transaction processing.
Test Steps:

Run gradle testIntegration. I have added an integration test for this change. There is a test STRAC file with the \u0019 character in the first and last name of the patient, and the logic strips out the character from both values on serialization. You can run it manually as well.

Changes

Adds logic to do the stripping of the characters for the regex
Adds an integration test

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?
[ ] (For Changes to /frontend-react/...) Ran npm run lint:write?
[x] Added tests?

Process

[ ] Are there licensing issues with any new dependencies introduced?
[ ] Includes a summary of what a code reviewer should test/verify?
[ ] Updated the release notes?
[ ] Database changes are submitted as a separate PR?
[ ] DevOps team has been notified if PR requires ops support?

Fixes

Closes #4250

Pull reviewers stats
Stats for the last 30 days:

User
Total reviews
Median time to review
Total comments

MauriceReeves-usds
29
15h 7m
10

carlosfelix2
26
1h 48m
64

cwinters-usds
22
3h 23m
4

TomNUSDS
16
1h
14

jbiskie
13
1h 9m
0

brick-green-agile6
12
1h 10m
2

kevinhaube
11
1h 2m
1

jimduff-usds
10
3h 8m
2

whytheplatypus
8
1h 2m
9

JosiahSiegel
7
59m
4

sean-usds
6
16m
2

sethdarragile6
5
4h 28m
0

oslynn
4
19h 13m
1

jorg3lopez
3
1h 42m
1

rhood23699
3
1h 57m
0

RickHawesUSDS
2
2h 21m
0

jeremy-page
2
10h 53m
0

clediggins-usds
2
1h 15m
0

ahay-agile6
1
1d 23h 57m
0

victorialb-usds
1
2d 20h 38m
1

snesm
1
16m
0

jh765
1
2h 54m
0

I reviewed and it looks good.  I also ran integration tests and unit tests.  They are good.  Thanks.


ci: migrate from set-output to GITHUB_OUTPUT

The former is deprecated and will stop working soon: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Signed-off-by: JP Simard<EMAIL_ADDRESS>

/retest


Still no media?

just downloaded and run on Ubuntu 20.
Proxy shows connected
But cannot send photos.
Is it supposed to be like this?

You should see the media endpoint on the statistics page (See https://github.com/WhatsApp/proxy#check-your-connection for how to reach that page).
Your WhatsApp proxy page should display "Connected" and it should NOT show "Media: unavailable".
Your Internet provider may be filtering your packets based on ports. The default port for media is 587 and changing it might help. For this, you will need to update the HaProxy configuration to bind to a different port.

Hi
Thanks for support.
Got progress
Now able to receive images but unable to send.
using this to invoke
docker run -it  -p80:80 -p 443:443 -p 5222:5222 -p 8080:8080 -p 8443:8443 -p 8222:8222 -p 8199:8199 -p 587:587 -p 7777:7777 facebook/whatsapp_proxy:latest
Any ideas?
Thanks
Rgds
Richard, Singapore

You should see the media endpoint on the statistics page (See https://github.com/WhatsApp/proxy#check-your-connection for how to reach that page).
Your WhatsApp proxy page should display "Connected" and it should NOT show "Media: unavailable".
Your Internet provider may be filtering your packets based on ports. The default port for media is 587 and changing it might help. For this, you will need to update the HaProxy configuration to bind to a different port and update the media port settings on your phone.

Can I have the WhatsApp proxy IP address you are using?

Testing
On Wed, 22 Nov 2023, 13:34 fungjigroup, @.***> wrote:

Can I have the WhatsApp proxy IP address you are using?
—
Reply to this email directly, view it on GitHub
https://github.com/WhatsApp/proxy/issues/228#issuecomment-1822143055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKWO5FIL6TR3ZRD7CM2EETYFWFIDAVCNFSM6AAAAAA6M6EJGSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRSGE2DGMBVGU
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I am from Hong Kong. Would you mind if you can provide the IP address for us to test as well? My wife is in Mainland China. I want to test if the WhatsApp proxy IP address can be used in Mainland China or not. Thanks!

Try
Anpr.optasia.com.sg
For testing only
On Fri, 24 Nov 2023, 11:14 fungjigroup, @.***> wrote:

I am from Hong Kong. Would you mind if you can provide the IP address for
us to test as well? My wife is in Mainland China. I want to test if the
WhatsApp proxy IP address can be used in Mainland China or not. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/WhatsApp/proxy/issues/228#issuecomment-1825099104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKWO5AWT4337DTFVXV7QRDYGAGJ7AVCNFSM6AAAAAA6M6EJGSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRVGA4TSMJQGQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Thanks! I just test it and it works fine! I ll let my wife in Mainland
China to test it in the coming weekend and I ll let you know the result
later. Anyway, thanks for your help!
On Sun, Nov 26, 2023, 18:56 richardgohth @.***> wrote:

Try
Anpr.optasia.com.sg
For testing only
On Fri, 24 Nov 2023, 11:14 fungjigroup, @.***> wrote:

I am from Hong Kong. Would you mind if you can provide the IP address
for
us to test as well? My wife is in Mainland China. I want to test if the
WhatsApp proxy IP address can be used in Mainland China or not. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/WhatsApp/proxy/issues/228#issuecomment-1825099104,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AOKWO5AWT4337DTFVXV7QRDYGAGJ7AVCNFSM6AAAAAA6M6EJGSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRVGA4TSMJQGQ>

.
You are receiving this because you modified the open/close state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/WhatsApp/proxy/issues/228#issuecomment-1826751258,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCCWOESCW2TDLKS3EPNGCLTYGMN6XAVCNFSM6AAAAAA6M6EJGSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRWG42TCMRVHA
.
You are receiving this because you commented.Message ID:
@.***>

Hello, I try to ping Anpr.optasia.com.sg on my computer and it replies but it cannot connect to the WhatsApp proxy in WhatsApp.
Any ideas?
Thanks!
Best Regards,
Albert, Hong Kong

#281

#281 WhatsApp proxy hasn't been updated for a long time and is outdated. When using curl to connect to port 587, WhatsApp's certificate is displayed, which seems relevant to your issue. According to your description, the session is using a self-signed certificate, while the media part is using WhatsApp's certificate.

Could you please clarify if I'm supposed to host a separate ip:port for media? The thing is I'm forwarding it through free version Ngrok and I can't use two ports simultaniously

the session is using a self-signed certificate, while the media part is using WhatsApp's certificate.

I'm having the same issue. This is what I get with curl on port 443:
* Server certificate:
*  subject: CN=1iCBLvVF330N.net
*  start date: Aug 14 03:42:48 2024 GMT
*  expire date: Aug 12 03:42:48 2034 GMT
*  issuer: CN=gYSkf3n0Uxybzdlam

This is what I get with curl on port 587:
* Server certificate:
*  subject: C=US; ST=California; L=Menlo Park; O=Meta Platforms, Inc.; CN=*.whatsapp.net
*  start date: Jun 20 00:00:00 2024 GMT
*  expire date: Sep 18 23:59:59 2024 GMT
*  issuer: C=US; O=DigiCert Inc; OU=www.digicert.com; CN=DigiCert SHA2 High Assurance Server CA

I'm guessing that at least in China, packets are scanned and HTTPS connections with certificates with CN=*.whatsapp.net are denied.
I tried to use a HTTPS reverse proxy to override the certificate, and curl seems to be able to get all the info. However, WhatsApp couldn't connect to the port for media.


Investigate usability concerns

For example, long codes should be formatted with dashes in between each group of n digits:
geaj-du3e-28eb-87ga-83vd
And the user input method should be customized for this:
____-____-____-____-____

Resolved in #25


Implement GUI testing using TestFX

closes #54, closes #60

Need to merge #51 before merging this PR, also need to request to use TestFX library on the forum.


Add list of houses and their leaders

This is public information, taken from page 5 of the Player's Guide

Sorry, maar ik kan niet approven 😅
Alle publieke informatie over houses bestaat al


update vue js to version 2.5 add required keys to v-for

Happy to find a simple table component. Thanks for that great work.
Tried it and found somme errors while I had<EMAIL_ADDRESS>globally installed.  So I tried to get them out.
In the same time I had to add the records IDs or keys.

Hi Sebastian,
When using vuejs 2.5, there is a need to have a key, otherwise at runtime,
vue produces a warning.
There were no key in the data table, so I added one, the id field, and made
it invisible.  I haven't seen any other field able to be used as unique key.
When applying this change, there is no warning anymore.
Thanks to have checked it.
Benoit
@sebastiandedeyne requested changes on this pull request.

In docs/index.html
https://github.com/spatie/vue-table-component/pull/129#discussion_r170543377
:

         ]"

         sort-by="songs"
         sort-order="asc"
         >

       <table-column show="key" label="id" :hidden=true></table-column>

I don't see why this is necessary?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/spatie/vue-table-component/pull/129#pullrequestreview-99242365,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AECGSSQk-uTT6GHHVEhmAUJ_5ZkKBJrrks5tYoKUgaJpZM4SK7-O
.

Ok, I made the changes myself on master, thanks!


Blog: How to Use Node-RED and Hyperledger Fabric for Enhanced Supply Chain Transparency in Manufacturing

Description
This blog will be great for SEO by focusing on practical blockchain applications in manufacturing. We will provide a practical guide, covering the benefits and drawbacks of integrating Node-RED with Blockchain. The blog aims to deliver actionable insights while addressing potential challenges and advantages, making it a valuable resource for industry professionals.
Epic/Story
No response
Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

We will cover each aspect and explore how it presents various challenges


PLAT-121331: Rename AriaValuetextDecorator.js as AriaValueTextDecorator.js and fix layout in qa-a11y

Rename AriaValuetextDecorator.js as AriaValueTextDecorator.js in qa-a11y sample
Some arc family components overlaps each other in qa-a11y sample. So I adjusted them.
Rename SlotCenter as Center in qa-a11y Arc Picker view so as to not overflow the text.

Additional Consideration

Fix JS Doc error in ThemeDecorator

Enact-DCO-1.0-Signed-off-by: YB Sung<EMAIL_ADDRESS>

When I tap on the item from the side menu, it seems opening other components. So I checked the region and found Marquee height is bigger than the outer layout. Please take a look.

I'll fix it in a separate issue.


Think about forward edges

Following on from https://stackoverflow.com/q/24844338/110081, @feuerbach suggests:
If we think about classic make-style build systems, they are based on back
edges. (I picture the edges in the dependency graph going from source to
target.) We start with a terminal vertex (such as "ALL", "build", "install"
etc.), and traverse the edges backwards to find all the dependencies, and so on.
Some computations (such as mine) are more conveniently expressed in terms of
forward edges. We don't know the exact target; instead we have a bunch of
source files (such as web pages) and the rules specifying how to process them.
This is more similar to a shell script than a Makefile. The only reason to use a
build system such as Shake instead is the short-circuiting behaviour.
When there's a direct correspondence between source and target files (such as
one between .c and .o files), there's almost no difference between the two
approaches. But imagine when the file itself determines the set of output files
(which isn't unrealistic — I think metapost/asymptote have this feature), then
trying to express this forward flow with backward edges can be unwieldy.
I should think about what that means, if you can express it in Shake, if it should be on the build-shootout etc.

Sounds interesting, and something that I probably often want in my tools.
Here is an example, simplified from what I’m doing right now: I have a directory with .log files. From each .log file I want to extract a .csv file, and from that .json report. That’s one obvious normal rule each, but how do I “pull” all the .json reports? For that I use something like
    "reports" ~> do
        logs <- filter looksLikeALogFile  <$> getDirectoryContents
        need [ takeBaseName log <.> "json" | log <- logs ]
    want ["reports"]

With forward edges, maybe this phony rule (which needs to know both the input of the chain of rules, and the output, i.e. the knowledge is duplicatd) could be dropped.

Is this also the 'unknown result' pattern? (#421)

The initial motivation on stack overflow is the lossy information pattern - where information is available in the person who does the need that you want available in the *> rule. That's a bit different to unknown result, but certainly related, it's what I was referring to in https://github.com/ndmitchell/shake/issues/421#issuecomment-184437586.
I think @nomeata is just after a forward build system though, which Shake.Forward might perfectly answer in the next release.


Moving the mobile mirror instance without the borders.

When I do --window-borderless, it's the center of the screen, how do I move it around without needing the titlebar?
Is it possible to move it with commands?

It depends on your window manager. On XFCE, Alt + move allows to move. On Windows, I have no idea.

It depends on your window manager. On XFCE, Alt + move allows to move. On Windows, I have no idea.

on scrcpy, when you do alt+move keys, it just rotates it sadly.

It depends on your window manager. On XFCE, Alt + move allows to move. On Windows, I have no idea.

Also been trying to search around for the solution can't seem to find it, so I went here lol.


Try 4 times more if can't read a frame.

I'm using a webcam, in my scripts, it will start and stop with howdy, but the connection has a delay so howdy can't capture frames forthwith. This won't impact the efficiency but improve the stability, so I'm considering whether l shall push it.

I'm not really sure about this PR, wouldn't a configurable delay be easier?

Maybe this one is simpler to users and more flexible on different camera?


Update to v0.14.0-rc3

Backlinking to purescript-contrib/governance#35

This looks good other than reverting the whitespace changes to the ci.yml file!


Fixes VideostreamGDNative crash on audio_channel=0.

Added an if case to check if the mix_callback exists before running any
of the audio code.
Fixes: #28644

Thanks!

Cherry-picked for 3.1.2.


Any way to install one tool located in my machine ?

im develop one console app and want know if i can install without publish with nuget for testing purpuse.

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

ID: f0c5b03c-2854-8462-57dc-b0239388096e
Version Independent ID: 13b38200-ed54-7bfe-f7cf-099cf1b010f4
Content: .NET Core Global Tools
Content Source: docs/core/tools/global-tools.md
Product: dotnet-core
GitHub Login: @KathleenDollard
Microsoft Alias: mairaw

@ucavalcante
I assume you are creating a global tool with dotnet pack with PackAsTool == true in the project.

You tool is a normal console application. You can test it's functionality by calling it directly or by using unit tests. When that is done...

dotnet pack and use the --output switch to put your package on your local drive
This creates a local NuGet package in the location you specify.

To test the installation of your tool use the --add-source switch on dotnet tool install to find your package. You may also specify a --tool-path if you'd prefer to keep the tool you're testing separated from your other install tools.
Let me know if that isn't enough to get you started.

Going to go ahead and close this as it appears resolved. @ucavalcante, if that isn't the case, please reopen. Thanks!

@KathleenDollard / @JRAlexander , I came here for the same thing. Might be useful to addition to the docs. Currently the --tool-path is documented, but it would be even clearer if you had a specific "How to test your tools locally" section.

What's the default list of feeds the dotnet tool search the package on?

@JohnTortugo
Internally, this uses standard NuGet restore. It uses the normal NuGet.config (or it's default)
@wli3 for more nuance if needed.


pkg/asset/templates: Name the kubeconfig context

Its useful to give the kubeconfig context a name when managing many k8s clusters with configs along a KUBECONFIG path. Until now, I've manually added a name to be able to switch back and forth (kubectl config use-context NAME), so it would be handy if that were done automatically.

Not necessarily opposed to this, but if you're specifying a specific kubeconfig (via env var or otherwise) doesn't it only know about the contexts in that file? So wouldn't adding a context to a single-context file not really change anything?

@aaronlevy the env var KUBECONFIG actually supports multiple files, separated by : so you can have multiple files all with unique context names.

Yep, there is a config file per cluster.

lgtm

This is still desired. You want a current context to be added to address the concerns of https://github.com/coreos/bootkube/pull/63?


Send If-Modified-Since header for stale cache entries

https://github.com/lukechilds/onionoo-node-client/issues/9

Coverage remained the same at 37.5% when pulling 4d347a0b0a0c3914d2e3f9a7d7b44e0e5e4d4ac7 on cache into 98cbfa3bec354375e51e36a8836d77717e6a4433 on master.

Coverage decreased (-1.8%) to 35.714% when pulling f1f173c239328346449b492a12e79b0cae4f24fc on cache into 98cbfa3bec354375e51e36a8836d77717e6a4433 on master.


mount with snapshot doesnt work

so im working with web container with react and i have a mini express app i did to test the snapshot so the arraybuffer is coming from the backend i console.log it  but when i put it in the mount function it doesnt mount any one have a solution i did like in the docs exactly

well i found the solution in case anyone need it  u need to convert it in Uint8Array
const snapshot = await snapshotResponse.arrayBuffer();
const snapshotUint8Array = new Uint8Array(snapshot);
console.log(snapshotUint8Array)
await webcontainerInstance.mount(snapshotUint8Array);

Hi :wave: thanks for opening this issue!
It looks like you have the solution so I will close this issue.
Feel free to open another if you have any other questions!


1st EDA round

[ ] get idea of image sizes (dimensions, color information) -- Alex
[ ] start a class for useful functions -- Dominik
[ ] part of image import: get rid of white bar on the bottom of each picture


Requst Dataset

Hi,
I send request to download the dataset but I didn't receive anything?

Link should've been sent via email. Let me know if you still have not yet received it.

Thank You very much I received the dataset.
في الاثنين، 17 أبريل 2023 في 5:48 م تمت كتابة ما يلي بواسطة Yun Chang <
@.***>:

Link should've been sent via email. Let me know if you still have not yet
received it.
—
Reply to this email directly, view it on GitHub
https://github.com/MIT-SPARK/Kimera-Multi-Data/issues/3#issuecomment-1511514890,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZHQ4Z636R76HFM7G2XZEGTXBVJ5NANCNFSM6AAAAAAXAQYLLM
.
You are receiving this because you authored the thread.Message ID:
@.***>


BusinessArcade

Please fill the following out:

Name: Business Arcade
Homepage url: https://businessarcade.com
Brand Guidelines/Licensing: Online Shopping
Logo: (svg if possible, pngs over 500x200px with transparent backgrounds are also acceptable)

Example:
Name: Business Arcade
Homepage url: https://businessarcade.com
Brand Guidelines/Licensing: http://businessarcade.com/image/cache/banners/May 2018/transparent-220x100.png
Logo: http://businessarcade.com/image/cache/banners/May 2018/transparent-220x100.png

please add this to http://www.typescriptlang.org/community/friends.html


Update MonitoringPage.java

The getDefaultChoice(String eSelectedValue) has been defined in the class. This function is never called in this class. This is also a waste of space and creates complexities in the class
What's in the PR

...

How to test manually

...

Automatic testing

[ ] PR includes unit tests

Documentation

[ ] PR updates documentation

 Quality gate: Passed
 Everything looks good
Details