Incompatibility Issue with PostgreSQL Version (11) in Docker Image and snapshot version (14)

Tried to restore 13.2 snapshot from https://update-cardano-mainnet.iohk.io/cardano-db-sync/index.html#13.2/
The dump is for PostgreSQL 14, but the docker image has PostgreSQL 11.21
OS
Ubuntu 20.04
Versions
cardano-db-sync version: <IP_ADDRESS>
PostgreSQL version: 14

Build/Install Method
Docker image from ghcr.io/intersectmbo/cardano-db-sync:<IP_ADDRESS>

Run Method
Docker, command used: docker run -it --entrypoint bash ghcr.io/intersectmbo/cardano-db-sync:<IP_ADDRESS>

Additional Context
Attempting to use PostgreSQL version 14 snapshots with the current Docker image, which only includes PostgreSQL version 11.21.
Problem Report
After pulling the latest Docker image for cardano-db-sync (version <IP_ADDRESS>) from the GitHub Container Registry and attempting to verify the PostgreSQL version, it was discovered that the included PostgreSQL version is 11.21. The expectation, based on the operational requirements for using PostgreSQL version 14 snapshots, was that PostgreSQL 14 would be included in the image.
Command Used:
docker run -it --entrypoint bash ghcr.io/intersectmbo/cardano-db-sync:<IP_ADDRESS>

Output Received:
bash-5.2# pg_restore --version
pg_restore (PostgreSQL) 11.21

That's a different error which was fixed in <IP_ADDRESS>. Check merged pr
If you want to work with postgres 14 and sync from snapshot you can use my custom Dockerfile.
ARG version="<IP_ADDRESS>"

FROM ghcr.io/intersectmbo/cardano-db-sync:${version} as base

# Second stage: Start from a minimal Debian or Alpine base
FROM debian:buster-slim

ARG version
ENV VERSION=${version}
LABEL VERSION=${version}
LABEL BUILD_DATE="2024-05-06"

# Copy everything from the distroless image
COPY --from=base / /

# Install necessary packages for adding repositories
RUN apt-get update && apt-get install -y wget gnupg

# Add PostgreSQL's official repo
RUN echo "deb http://apt.postgresql.org/pub/repos/apt/ buster-pgdg main" > /etc/apt/sources.list.d/pgdg.list

# Import the repository signing key
RUN wget --quiet -O - https://www.postgresql.org/media/keys/ACCC4CF8.asc | apt-key add -

# Update apt and install PostgreSQL client
RUN apt-get update && apt-get install -y postgresql-client-14

STOPSIGNAL SIGINT

ENTRYPOINT ["/entrypoint.sh"]

Interesting. I think the postgresql-setup.sh needs to use the absolute path in nix store!
something like: /nix/store/*/bin/sed

Interesting. I think the postgresql-setup.sh needs to use the absolute path in nix store!

I don't think this is necessary, we should just be able to add the missing paths here: https://github.com/IntersectMBO/cardano-db-sync/blob/master/nix/nixos/cardano-db-sync-service.nix#L259

And it seems this problem is still present in ghcr.io/intersectmbo/cardano-db-sync:<IP_ADDRESS>

Yes, this is true. We're going to prepare a docker image with this fix in a few days

How is everyone creating/restoring snapshots from docker? There are a few options I can think of:

The postgresql-setup.sh script (has commands like create/restore snapshot)
Environment variable RESTORE_SNAPSHOT

If everyone is using (1), it would probably be better to add it to the image (or maybe a new image) along with all its dependencies.

The postgresql-setup.sh script is already part of the image. I think it's in /nix/store.
And yes I'm setting the RESTORE_SNAPSHOT var.

I run using the following command:
export NETWORK=mainnet &&\
docker pull cardanofoundation/cardano-graphql-server:8.0.3-${NETWORK} &&\
docker pull cardanofoundation/cardano-graphql-background:8.0.3-${NETWORK} &&\
docker pull cardanofoundation/cardano-graphql-hasura:8.0.3 &&\
docker pull cardanosolutions/cardano-node-ogmios:v6.2.0_8.9.0-${NETWORK} &&\
RESTORE_SNAPSHOT=https://update-cardano-mainnet.iohk.io/cardano-db-sync/13.2/db-sync-snapshot-schema-13.2-block-10272678-x86_64.tgz \
docker-compose up -d

Still encountering: /nix/store/sipivivjip8mq92inydn2g95d35wn8vs-postgresql-setup.sh: line 194: sed: command not found

This should be fixed here (as long as you're using postgresql:14): https://github.com/IntersectMBO/cardano-db-sync/pkgs/container/cardano-db-sync/214567424?tag=<IP_ADDRESS>-docker


Database

Setup database, created insert functionality, stopped server from taking pictures, wrote super basic get query from db (by os)

lgtm


[22.11] armcord: 3.1.4 -> 3.1.7

armcord: 3.1.4 -> 3.1.7
(cherry picked from commit 716bc884badd59a9f7cd2669a4e463bff3c25013) #227937
I'm unsure about whether this backport should get merged, as I consistently get a crash when I press the screenshare button on wayland, that appears to be a regression in v3.1.5, and due to now trying to use setDisplayMediaRequestHandler(), and then having upgraded electron version to 22 in v3.1.7 There is a "legacy" screen capture setting that doesn't, but even with this enabled, it still crashes.
Their build server has an electron 21 build, but it isn't versioned by releases (only has folder for stable and dev) which means we can't get our builds from there.
I think something is missing from the nix build environment that breaks screen capture on electron 22, but I can't figure out what.
Description of changes

3.1.6 -> 3.1.7

Updated Electron
Disabled automatic Wayland checks
This update may fix some of the issues that you might've encountered when using screenshare in past ArmCord versions (v3.1.5-6).

3.14 -3.1.5-> 3.1.6

Fixed arRPC not starting
Updated arRPC enabling process scanning on Linux
Rewritten the capturer
Performance tweaks
Updated translations
Added disable-autogain

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.05 Release Notes (or backporting 22.11 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

[x] Fits CONTRIBUTING.md.

this should probably be cherry picked from 83765a4a84cca6fcd0f64b19941542efe1869f0c instead of the merge commit

for now though:
Result of nixpkgs-review pr 228399 run on x86_64-linux 1

1 package built:

armcord

@wiryfuture https://github.com/ArmCord/ArmCord/commit/09fcd77803519bd376aceac20e50008f669d07bc

It was still crashing for me, regardless of whether I had legacy enabled or not.


Keep Docker state between builds

Greatly reduces incremental build time for "make docker".

Codecov Report

Merging #1164 (9f33e2f) into main (15d5c76) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1164   +/-   ##
=======================================
  Coverage   99.98%   99.98%
=======================================
  Files         305      305
  Lines       19964    19964
=======================================
  Hits        19962    19962
  Misses          2        2

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


add to Bower

add bower config file and list in bower page search:
http://bower.io/search/
Searching for 'H5F'
returns this project:
https://github.com/dsheiko/HTML5-Form-Shim

It's already on npm but if you wanna do a pr to put in on bower be my guest.

bower looks for h5f.js instead of h5f.min.js because the component.json is out of date and there is no .bower.json file


Collapsable filters for mobile

Hello,
We are building a more mobile friendly version of our filters and therefor we want to create collapsable filter boxes. Since we are already using Bootstrap 3 at our site I wanted to use their collapse functionality.
When I add the needed html via jQuery to the instantsearch generated html it doesn't work. It seems that it isn't picking up the click event.
When I add this:
this.addEventListener("click", function(){

            $($(this).data('target')).collapse('toggle');
        });

The collapse part works, but it seems that instantsearch is no longer picking up the events..
You can see what I mean at https://markputs.nl/algolia.php (make your screen small, javascript can be found in Algolia.js)
Thanks.

@Maaark thanks for sharing you're use case. It seems that you are changing the content of the dom generated by the instant search widget (l.696 of algolia.js for example), and this will mess with the internal React rendering. If you change the outer dom it should be ok though.

@bobylito What do you mean exactly with the outerdom? What element? Since we need to have a specific structure in order to have collapse functionality

I use "outer DOM" as opposed to the DOM that's inside the widget. If you create DOM wrappers that fits the rules for your scripts that would work.

But the issue is that I need to alter the react generated html to give it the right structure for the collapse functionality.

Actually, there is an option to make the refinement list widget collapse already built-in. https://community.algolia.com/instantsearch.js/documentation/#refinementlist Would that work for you?

@bobylito I tried that one but it didn't work. I reapplied it now, so you can see that it's not working and perhaps find the bug.
To make it a bit more complicated, we want to show the minimalized version only to devices with a window width < 600px. Bigger sizes should get our current version, with all options visible. Is that something we can do?

@bobylito Can you confirm to me that it's a bug or did we do something wrong?

I will make a proper example to test and check the behaviour and see if we're on the same page.

Hi @Maaark if you experienced any issue while using the collapsable option can you please demonstrate it somehow then we can work on it? Thanks

@vvo You can see what I mean at https://markputs.nl/algolia.php. Make your screen 400px, click at filter resultaten and all of those refinement lists are collapsable: true. See for example that Merken area.

@bobylito @vvo Any idea when you guys can have a look at this?

@Maaark I had a look right now, I can see the collapsible option seems not to work in your case. Could you try to first update instantsearch.js to the latest version? Your are using an old version (1.6.x), we are at 1.8.x.
Then if it's still not working I would advise you to try to reproduce the bug in a isolated simple webpage first, it will help you understand what can go wrong.

And is it possible to make the collapsable functionality work with predefined window width limits? So for example we want to be the refinement lists only collapsable when the window width is < 768px?

Once the collapsible option is working in a simple webpage, you should try using our predefined CSS classes to apply some CSS media queries on top of them so that the widget are collapsed only based on the viewport.
See our CSS file (https://cdn.jsdelivr.net/instantsearch.js/1/instantsearch.css):
.ais-root__collapsible .ais-header {
  cursor: pointer;
}

.ais-root__collapsed .ais-body, .ais-root__collapsed .ais-footer {
  display: none;
}

It's an old issue and we haven't encountered it again. Please reopen an issue if you're experiencing it again.


Use "fake" instead of "null"

this commit fixes #20

I'll manually add it in the next commit since this also has some bugs I fixed, causing conflicts


Use ⌘ consistently

What's being changed (if available, include any code snippets, screenshots, or gifs):
Some Mac shortcuts used Command  while others used ⌘
The symbol is well-known, shorter, and (to my eyes at least) “parses” easier than the reading the word “Command”.
So, might as well be consistent with it.
Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Sorry to ask is this a software get rid of this stupid android knox
security I have on my fone some has put it on there and it say it has an it
addmin that controls what I do on my fone and now I can't get into my
accounts that how I came across use looking for anyone or ways to do it my
self to get rid of it but I'm not that clue up so I really need help
Show quoted text
On Sun, 25 Sep 2022, 4:54 am Zearin, @.***> wrote:

What's being changed (if available, include any code snippets,
screenshots, or gifs):
Some Mac shortcuts used Command while others used ⌘
The symbol is well-known, shorter, and (to my eyes at least) “parses”
easier than the reading the word “Command”.
So, might as well be consistent with it.
Writer impact (This section is for GitHub staff members only):

This pull request impacts the contribution experience

I have added the 'writer impact' label
I have added a description and/or a video demo of the changes
below (e.g. a "before and after video")

You can view, comment on, or merge this pull request online at:
https://github.com/github/docs/pull/20843
Commit Summary

2251503
https://github.com/github/docs/pull/20843/commits/2251503948705a6b0f596bea0a6af2ece6664ef2
Use ⌘ consistently

File Changes
(1 file https://github.com/github/docs/pull/20843/files)

M content/get-started/using-github/github-command-palette.md
https://github.com/github/docs/pull/20843/files#diff-cf8f679772b4e2ebffb93e4934eb1b4e2b98bc1a9bbfc48491aeda1f7d7b63c1
(18)

Patch Links:

https://github.com/github/docs/pull/20843.patch
https://github.com/github/docs/pull/20843.diff

—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/20843, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3GBVW7QXUW2E2IZGO4P5E3V74P4DANCNFSM6AAAAAAQUWIFWA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sorry to ask is this a software get rid of this stupid android knox
security I have on my fone some has put it on there and it say it has an it
addmin that controls what I do on my fone and now I can't get into my
accounts that how I came across use looking for anyone or ways to do it my
self to get rid of it but I'm not that clue up so I really need help
Show quoted text
On Sun, 25 Sep 2022, 4:54 am welcome[bot], @.***> wrote:

Thanks for opening this pull request! A GitHub docs team member should be
by to give feedback soon. In the meantime, please check out the contributing
guidelines https://github.com/github/docs/blob/main/CONTRIBUTING.md.
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/20843#issuecomment-1256998678, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A3GBVW73VNNSCJT5KK2R6OLV74P2JANCNFSM6AAAAAAQUWIFWA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I don't think these must to change to ⌘ from Command. The guidelines specifically instructs to use full words instead of symbols

My mistake! thanks for correcting me 👍

@Zearin Thanks for opening this PR! ✨
We appreciate the thought you put into this, but as @AnilSeervi pointed out in our guidelines, our practice is to use the full word "Command". If you see instances where that is not the case, feel free to open a PR for that fix!
I'm going to go ahead and close this PR. Thank you for interest and passion for improving the docs! 💛


More batched issues

General comments

the text should explain what the dashed box means in the patterns. it seems that it always encapsulates the entities
that are part of the result (either in vanilla or in aggregated form). right?

Comments for specific queries/operations
complex-read / 3:

what is the minimum length of the path? can src and dst be directly connected?

complex-read / 4:

the text prescribes to "find the sum and max of transfers (edge2 and edge3)". is this determined as "sum(edge2.amount) + sum(edge3.amount)"?
is the query only returns the other account of the maximum value, why is there a need to sort on the sumEdge2Amount and sumEdge3Amount attributes?
how should ties be handled?

complex-read / 11(Will be removed):

can persons l2p1, ..., l3p3 invest in companies such as l1c2, l1c3, l2c2, etc.? the pattern suggests that this is not
possible but it would be logical... please clarify which one is the case?

More comments

What is the rationale for rounding float values to 3 decimal places? (that they usually express ratios, so this is basically a percentage value in the form of xx.x%?)
The Path data type does not consider multi-edges: traces on different edges are indistinguisable based on their return Path.
"The mechanism is also called PER_NODE_LIMIT somewhere" -> where?
The paragraph "About the truncation" could use a rewrite with better structure and illustrations. It can be understood
in its current for but given that this is one of the distinguishing features of this benchmark, it should be explained
more clearly.

It should be mentioned that "truncation" is a form of deterministic sampling

currentTime
Several queries use the "currentTime" parameter. Where does the value of this parameter come from: the data set, the driver (e.g. Java's System.currentTimeMillis()), or the DBMS (e.g. Cypher's date() function)? If it's a parameter, the patterns should indicate it as such with ${currentTime}.
Path queries with conditions over multiple edges
Queries that require "all timestamps in the transfer trace are in ascending order" or the "upstream" edge are difficult to explain in plain Cypher (or GQL or SQL/PGQ) because they require some memory.
One can think of this as an extra choke point which requires support for the category of queries "Regular expression with memory" as described in this paper: https://dl.acm.org/doi/abs/10.1145/2274576.2274585. Even if it's not defined as a choke point, it's worth including this reference in the specificiation.

Will fix

General comments

the text should explain what the dashed box means in the patterns. it seems that it always encapsulates the entities
that are part of the result (either in vanilla or in aggregated form). right?

Comments for specific queries/operations
complex-read / 3:

what is the minimum length of the path? can src and dst be directly connected?

More comments
currentTime
Several queries use the "currentTime" parameter. Where does the value of this parameter come from: the data set, the driver (e.g. Java's System.currentTimeMillis()), or the DBMS (e.g. Cypher's date() function)? If it's a parameter, the patterns should indicate it as such with ${currentTime}.
Path queries with conditions over multiple edges
Queries that require "all timestamps in the transfer trace are in ascending order" or the "upstream" edge are difficult to explain in plain Cypher (or GQL or SQL/PGQ) because they require some memory.
One can think of this as an extra choke point which requires support for the category of queries "Regular expression with memory" as described in this paper: https://dl.acm.org/doi/abs/10.1145/2274576.2274585. Even if it's not defined as a choke point, it's worth including this reference in the specificiation.

Will discuss

Comments for specific queries/operations
complex-read / 4:

the text prescribes to "find the sum and max of transfers (edge2 and edge3)". is this determined as "sum(edge2.amount) + sum(edge3.amount)"?
is the query only returns the other account of the maximum value, why is there a need to sort on the sumEdge2Amount and sumEdge3Amount attributes?
how should ties be handled?

complex-read / 11(Will be removed):

can persons l2p1, ..., l3p3 invest in companies such as l1c2, l1c3, l2c2, etc.? the pattern suggests that this is not
possible but it would be logical... please clarify which one is the case?

More comments

What is the rationale for rounding float values to 3 decimal places? (that they usually express ratios, so this is basically a percentage value in the form of xx.x%?)

The Path data type does not consider multi-edges: traces on different edges are indistinguisable based on their return Path.

"The mechanism is also called PER_NODE_LIMIT somewhere" -> where?

The paragraph "About the truncation" could use a rewrite with better structure and illustrations. It can be understood
in its current for but given that this is one of the distinguishing features of this benchmark, it should be explained
more clearly.

It should be mentioned that "truncation" is a form of deterministic sampling

Will discuss

Comments for specific queries/operations
complex-read / 4:

the text prescribes to "find the sum and max of transfers (edge2 and edge3)". is this determined as "sum(edge2.amount) + sum(edge3.amount)"?
is the query only returns the other account of the maximum value, why is there a need to sort on the sumEdge2Amount and sumEdge3Amount attributes?
how should ties be handled?

complex-read / 11(Will be removed):

can persons l2p1, ..., l3p3 invest in companies such as l1c2, l1c3, l2c2, etc.? the pattern suggests that this is not
possible but it would be logical... please clarify which one is the case?

More comments

What is the rationale for rounding float values to 3 decimal places? (that they usually express ratios, so this is basically a percentage value in the form of xx.x%?)

The Path data type does not consider multi-edges: traces on different edges are indistinguisable based on their return Path.

"The mechanism is also called PER_NODE_LIMIT somewhere" -> where?

The paragraph "About the truncation" could use a rewrite with better structure and illustrations. It can be understood
in its current for but given that this is one of the distinguishing features of this benchmark, it should be explained
more clearly.

It should be mentioned that "truncation" is a form of deterministic sampling

TODO

explain dashed box
add RPQ with memory chokepoint
Rewrite truncation paragraphs.

General comments

the text should explain what the dashed box means in the patterns. it seems that it always encapsulates the entities
that are part of the result (either in vanilla or in aggregated form). right?

Path queries with conditions over multiple edges
Queries that require "all timestamps in the transfer trace are in ascending order" or the "upstream" edge are difficult to explain in plain Cypher (or GQL or SQL/PGQ) because they require some memory.
One can think of this as an extra choke point which requires support for the category of queries "Regular expression with memory" as described in this paper: https://dl.acm.org/doi/abs/10.1145/2274576.2274585. Even if it's not defined as a choke point, it's worth including this reference in the specificiation.


When i run python3 main.py i am having this error!

That's because I forgot to add rich module in requirements.txt. But now I should work fine, just use git pull where you cloned the repo and run pip install -r requirements.txt, then you should be good to go.

I'm closing this issue, if the error still persists, just comment here, and I'll reopen it.


Figure out why the attached tf_gather.py TensorFlow script is slow going on with this tf_gather.py script

From a community partner:
"Apparently TF is rather slow in reductions and mutliplications of vectors. Run this example and you will see very low CPU usage. I know I can run this with sparse vectors. It is probably optimized. I am disappointed this didn't work."
tf_gather.py.zip

comments from @samjabrahams:

When you put sess.run() inside a loop, you're having TensorFlow jump between the Python and C++ layers 100000 times, which just exacerbates the communication bottleneck. In typical TensorFlow code, you end up with a large amount of operations being run with a single call to run()

I see TensorFlow using up all of a single core with the provided code. The code is iterating through and running a bunch of simple, single threaded operations sequentially, so that's why you don't see large overall CPU usage

I've attached some modified code: it places 100000 multiplies to be run in sequence in a single graph and then runs the whole thing with a single call to sess.run(). It doesn't use multiple cores, however. On my computer, the script finishes running in<PHONE_NUMBER>15, or approximately 0.0083 seconds per series of operations created with func()

Check out the discussion here on the GitHub repo about creating code to test out multiple-CPU usage. The basic idea is to create a TensorFlow Queue along with helpful built-in functionality to create a bunch of coordinated threads to run the code.

one_session_call.py.zip

response from community member:
"It is a problem that I have seen in TF, in general. I was discussing it with a friend from . TF is very good at dense linear algebra. In fact if you run the same code with a matrix product or a hadamard product you will see very good CPU utilization.
Try to increase the size of this example and you would expect that most of the time will be spent on the reduce_sum. It doesn't happen. Switching and serialization still dominate. I am not sure that it actually does parallelize the reduce.
In fact with Sam's way it just executes all the nodes in parallel but not each node. In general I think that TF parallelize only linear algebra nodes. Also sometimes it can parallelize in GPU but not in GPU.
The bottom line is that if you want to run things fast it is not as simple as you think. You need to get deep down to control flow or use the trick with the queue which is not really a great paradigm. Also pushing all the data in the graph is not a wise thing as you copy a lot."


SDIT-1041: 🔧 Move to new dev prison api in cloud platform

Thanks @petergphillips. Will merge these changes in.


Update demos

API Demos:
Events:

[ ] Make a login page that hides menu items

Id:

[ ] Show multiple menus diff IDs

Input:

[ ] Show different kinds of inputs

Items:

[ ] Show different kinds of items
[ ] Fav musicians

ItemSliding:

[ ] Show different kinds of sliding items
[ ] Add to Option as well

LocalStorage:

[ ] Button block messes up icons
[ ] Select boxes

Menu:

[ ] Multiple Menus

Navbar:

[ ] Show some navbars

RadioButton/RadioGroup:

[ ] Make multiple radio lists

Refresher:

[ ] Pull to refresh does not work in demo

Searchbar:

[ ] Show different kinds of Searchbars

Segment:

[ ] Show different kinds of segments

Select:

[ ] Random selects

SQLStorage:

[ ] Look into this

Tab/Tabs:

[ ] Show different tab scenarios

Textarea:

[ ] Different textareas

Title:

[ ] Update toolbar scenarios to look better

Random:

[ ] Right now the demos that pass parameters actually are only passing one parameter. We should use a list with selects so they can pass multiple parameters and then show the code on the next page.

Examples:
Modals
Nav Params
Moved some to these issues:
https://github.com/driftyco/ionic-site/issues/395
https://github.com/driftyco/ionic-site/issues/394

@drewrygh I'm currently working on searchbar/segment but if you want to tackle any others on this list just coordinate with me. :smile:

The only remaining task here should be part of the ionic-native tasks


Enlarge the width of the select dropdown for the list of endpoints in the Digma Metrics report

Issues: having the long names of endpoints makes it hard to see the search results

Let's make the width of the selector results dynamic. The maximum width should be based on the longest endpoint name.


How to make window and document accessible in files in include/ui?

The proposed structure for imports suggests to have a imports/ui directory for UI related files. However, meteor-typescript does not set the web.browser environment in this directory. It is only included into the imports/client directory.
It took me a while to figure out why typescript was complaining about the use of window and document.
The simple solution is to put all UI related files under imports/client. But I wonder if there is another way to tell typescript that the files are only loaded into the browser?
Or at maybe document that you should put all client files into the import/client directory.

I think UI related files don't  always mean they are of the client's ones in an isomorphic environment as Meteor. Some UI parts can be rendered on the server in such case.
yes, meteor-typescript doesn't set web.browser because it just compiles according to the parameters provided by Meteor compilers API, which means if Meteor says that files under imports/client is of web.browser it compiles accordingly.

I do agree -- I just followed the structure proposed by the meteor team.
Would you mind a PR that adds this to the documentation of this plugin? I have not tried - is any directory called client browser code or only code /client and /imports/client?


[BUG] UI API called from background thread

Describe the bug
Passing fetchToken=true in join channel call leads to 'UI API called from background thread' warning message from iOS and eventual crash.
Workaround is for app to manually get token and pass to Agora SDK instead of relying on Agora SDK to automatically fetch token.
To Reproduce
Steps to reproduce the behavior:

From main thread, call - AgoraViewer.viewer.join(channel: channeId, as: role, fetchToken: true, uid: uid)

Expected behavior
AgoraUIKit should process fetchToken result in main thread
Screenshots

Smartphone (please complete the following information):

Device: iPhone 13
OS: iOS 15
Version 4.0.0-preview.12

thanks for the bug report! looks like the success handler for .join needs to come back to the main thread.
i'm currently working on the 1.8.1 release, so I'll add the patch in there and roll that out to the preview soon after

Just pushed an update 4.0.0-preview.13 that should fix your issue. Please note that since this version the library now has two separate parts, AgoraUIKit and AgoraRtmControl - double check that both of these libraries are included in your app target if using Swiftpm.
We'll be switching to 4.0.0-beta.1 soon, once some of the extensions are updated to work with it by the way


Sort out by Views [JAVASCRPT]

On hover of text - "Total Views" put a small icon/button that if tapped, will sort the list by the person who got the least amount of views, to the most amount of views, from the list.

@KendallDoesCoding can i work on this? if no one is working

I can do this by both using library and By coding JS in classical way as we do? You can let me have this one

@uuc110 to take the view I need the API key for which for me it's not possible to share my Private API key🤷♂ so I don't think any another way to solve this? well not in my knowledge.

Hi @KendallDoesCoding , I will be able to pick this up, you can assign this feat to me if open.

Hi @KendallDoesCoding , I will be able to pick this up, you can assign this feat to me if open.

Sure.

@KendallDoesCoding to take the view I need the API key for which for me it's not possible to share my Private API key🤷♂ so I don't think any another way to solve this? well not in my knowledge.

which software is it for API key? There is another way which I was working on the other day but left it half.

I was trying to use google's Youtube Data API v3

Bro. The views is on the website in the lqst column we just needed to sort
out in order by views.
Eg:
5 million
2 million
The 2 million as to come on top. This issue doesnt need any API just JS
code.
On Fri, 31 Mar 2023, 10:35 am Sourabh Kushwah, @.***>
wrote:

@KendallDoesCoding https://github.com/KendallDoesCoding to take the
view I need the API key for which for me it's not possible to share my
Private API key🤷♂ so I don't think any another way to solve this? well
not in my knowledge.
which software is it for API key? There is another way which I was working
on the other day but left it half.
I was trying to use google's Youtube Data API v3
—
Reply to this email directly, view it on GitHub
https://github.com/KendallDoesCoding/youtubers-birthdays/issues/7#issuecomment-1491301967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTOW2PBG6OSVN5GZMZTW6ZQYFANCNFSM6AAAAAAWJNZ7BI
.
You are receiving this because you were mentioned.Message ID:
@.***>

Bro. The views is on the website in the lqst column we just needed to sort out in order by views. Eg: 5 million 2 million The 2 million as to come on top. This issue doesnt need any API just JS code.
…
On Fri, 31 Mar 2023, 10:35 am Sourabh Kushwah, @.> wrote: @KendallDoesCoding https://github.com/KendallDoesCoding to take the view I need the API key for which for me it's not possible to share my Private API key🤷♂ so I don't think any another way to solve this? well not in my knowledge. which software is it for API key? There is another way which I was working on the other day but left it half. I was trying to use google's Youtube Data API v3 — Reply to this email directly, view it on GitHub <#7 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUGJUTOW2PBG6OSVN5GZMZTW6ZQYFANCNFSM6AAAAAAWJNZ7BI .  I Dropped the PR thinking its about API thing, When I already did it😂 Anyway. Thanks for correcting me

Oh, it's not API related. Sorry that I wasn't clear in the issue
description.
Thanks,
Kendall Does Coding
Portfolio
https://ncc-net.com/click?redirect=https%3A%2F%2Fkendalldoescoding.gq&dID=1680253291744&hashId=060de6478a47358f5315814fdd49f493dff92988d90dca1ba25e30911a8a60f18777e7&linkName=Portfolio
Github
https://ncc-net.com/click?redirect=https%3A%2F%2Fgithub.com%2FKendallDoesCoding&dID=1680253291744&hashId=060de6478a47358f5315814fdd49f493dff92988d90dca1ba25e30911a8a60f18777e7&linkName=Github
Youtube
https://ncc-net.com/click?redirect=https%3A%2F%2Fkendalldoescoding.gq%2FcodingYT&dID=1680253291744&hashId=060de6478a47358f5315814fdd49f493dff92988d90dca1ba25e30911a8a60f18777e7&linkName=Youtube
On Fri, 31 Mar 2023 at 11:42, Sourabh Kushwah @.***>
wrote:

Bro. The views is on the website in the lqst column we just needed to sort
out in order by views. Eg: 5 million 2 million The 2 million as to come on
top. This issue doesnt need any API just JS code.
… <#m_7724634788709762059_>
On Fri, 31 Mar 2023, 10:35 am Sourabh Kushwah, @.> wrote:
@KendallDoesCoding https://github.com/KendallDoesCoding
https://github.com/KendallDoesCoding https://github.com/KendallDoesCoding
to take the view I need the API key for which for me it's not possible to
share my Private API key🤷♂ so I don't think any another way to solve
this? well not in my knowledge. which software is it for API key? There is
another way which I was working on the other day but left it half. I was
trying to use google's Youtube Data API v3 — Reply to this email directly,
view it on GitHub <#7 (comment)
https://github.com/KendallDoesCoding/youtubers-birthdays/issues/7#issuecomment-1491301967>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTOW2PBG6OSVN5GZMZTW6ZQYFANCNFSM6AAAAAAWJNZ7BI
https://github.com/notifications/unsubscribe-auth/AUGJUTOW2PBG6OSVN5GZMZTW6ZQYFANCNFSM6AAAAAAWJNZ7BI
.  I
Dropped the PR thinking its about API thing, When I already did it😂
Anyway. Thanks for correcting me
—
Reply to this email directly, view it on GitHub
https://github.com/KendallDoesCoding/youtubers-birthdays/issues/7#issuecomment-1491353255,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTMMPVBYR2WUVKAB6ETW6ZYWBANCNFSM6AAAAAAWJNZ7BI
.
You are receiving this because you were mentioned.Message ID:
@.***>

A issue about a API is #4, not this one!

Any update on this @iONBain

@iONBain - any update?

@KendallDoesCoding - Hey, just a doubt about the sorting issue as most of the details are in .html file, it might be hard to pull and sort those. If you got any tips on how to do it would love to know.
PS -  would be good if we create an array of the details for better usability.
Thank you.

@KendallDoesCoding - Hey, just a doubt about the sorting issue as most of the details are in .html file, it might be hard to pull and sort those. If you got any tips on how to do it would love to know. PS - would be good if we create an array of the details for better usability.
Thank you.
Hey,
Thanks for bringing this to my notice...

Can you create a issue suggesting to create a array for the details and then we can go from there?

Can I get this Assigned ? Also I would need more details on what you exactly want to achieve and where to add the sort button

Can I get this Assigned ? Also I would need more details on what you exactly want to achieve and where to add the sort button here is my discord-harshsingh9215

Hey! @iONBain was assigned to this, but I haven't heard from him on this from a long time, if he doesn't reply. I will assign this to you.
@iONBain - any update?

Why this codebase doesn't contain a ".html" file?

@KendallDoesCoding  -  I would like to work on this one - Can you assign this to me?

@KendallDoesCoding - I would like to work on this one - Can you assign this to me?

Hey. Sorry, @MayankBansal12 is working on this. You can help him with it, if he requires. Otherwise, you can work on other issues related to the project. Thanks

Why this codebase doesn't contain a ".html" file?

Hi @CodeWithPrajjwal
We are using the ejs template in Express to generate dynamic HTML content, so we don't have .html files, but in public folder, you can see .ejs files.

@KendallDoesCoding - I would like to work on this one - Can you assign this to me?

Hi @shivamsethi,
So, what approach you are looking to use to sort YouTubers?

I think we can use the mongoDB inbuilt functions to sort the data based on views or category. @MayankBansal12 - Let me know if you are not working on this, i can work on it Or we can work together.

Hey @Lucifer-zone, you can go ahead and raise a PR for the same.

Hi @KendallDoesCoding - Could you assign this to me

Hi @KendallDoesCoding - Could you check this PR - https://github.com/KendallDoesCoding/youtubers-birthdays/pull/102
I am getting a strange issue while Deepsource verification and not able to add you as reviewer

Hey @Lucifer-zone, your PR #102 has been merged. It does allow sorting based on views however, we need to add sorting for a category. On selecting a specific category name, say Gaming or Tech, all the related entries are shown. Currently sorting is working based on views.

Hi @MayankBansal12 - I have added the sorting on category column too where if we click on category column, values of category get sorted based on lexicographical order. It was working for me. Could you try again by clearing the cache.
I will also try once.

Sorting on category is working for me, Please try again

I have checked the category sorting too. What I meant was if I click on any category, say Tech or Gaming, then all the items related to that category get listed up...something like that. Currently, it gets sorted based on lexicographic order which is fine.


show info and edit ability per rule in policies

On the "Add/Delete Filter on Policies" as rules are edited some value doesn't show. For instance if saying that psnr.Y exceeds a cutoff number of 1. The '1' doesn't appear after the rule is supplied. There should be a way to view and edit existing rules of a policy (so far they can only be deleted).
Also Filter2 should only be shown contextually if average-difference is selected, else it has no meaning.

display and selection is both fixed.. and updated on server.

I noticed that it doesn't work quite well.

It is somewhat fixed now. I think.

My recent issue is related, I think.

Okay i will look more carefully. (I thought it works fine now..) For the meantime, it would be great you can tell me the case things doesn't work as it should.

due to no update.


Make sure 'All' is toggled on (instead of 'Basic'). What does this means . Iam not able to build the project

I am also not able to integrate this project with Xcode
After following all the steps mentioned I am getting
ld: warning: directory not found for option '-L/Users/sushant/Library/Developer/Xcode/DerivedData/Goalwise-cenftntedkqaxebrfbvguvbsiuya/Build/Products/Debug-iphoneos/Mixpanel'
ld: library not found for -lMixpanel
clang: error: linker command failed with exit code 1 (use -v to see invocation)
React Native Version: 0.47.2

To find the Header Search Paths options you need to choose the All option in the second row here:


Use mjs and avoid mixing CJS (require) with ESM. Do not overwrite config.resolve.alias.vue if pre-set.

works good. Improved the PR title, this fixes #53


Support generating an Electron App

Does it make sense and is it possible to automatically generate a packaged electron app from our stuff? The app would still require a python backend running somewhere, which does not have to be the client (due to websockets), but it should be possible to wrap up the html and js into an app that's made installable through the electron packaging system.
More investigation needed.

Have you seen this?
https://www.fyears.org/2015/06/electron-as-gui-of-python-apps.html

I was thinking about something along those lines. I need to make time to go over your link in more detail. Thanks a lot for the info.

I've decided to do this directly with the Chromium Embedded Framework. Already have a working prototype in MacOS using the adding-embedded-chromium branch, where instead of spawning the default browser I launch a version of the cefsimple browser. This will remove all browser-like window elements (address bar, navigation buttons, etc.), making it look like a real application. Some testing appreciated.
I don't want to get into talking directly to CEF through C bindings though, so everything will stay with websockets for greater flexibility.
Now I just need to find a good way of generating cross-platform installable binaries. I know these exist, just haven't made time to research my options yet.


Nrfx 5977 extend mbox sample

Print on what board/core sample is executing.
Print how many bytes of data can be sent in the mbox message (show use of mbox_mtu_get_dt()).
Print how many channels are available for incoming and/or outgoing messages (show use of mbox_max_channels_get_dt()).
Add sample configuration 'nrf54h20_app_ppr' where:

cpuapp sends messages to cpuppr vevif channel 15,
cpuppr sends messages to cpuapp bellboard channel 18.

Add sample configuration 'nrf54h20_rad_app' where:

cpuapp sends messages to cpurad bellboard channel 12,
cpurad sends messages to cpuapp bellboard channel 18.

Upstream PR: https://github.com/zephyrproject-rtos/zephyr/pull/75033

Tested manually on NCS code base.
Both configurations are working on my desk :smile:
./zephyr/scripts/twister --ninja --inline-logs --jobs 1 -T zephyr/samples/drivers/mbox --platform nrf54h20dk/nrf54h20/cpuapp --device-testing --device-serial /dev/serial/by-id/usb-SEGGER_J-Link_001051165753-if00 -v -v --west-flash="--erase"
./zephyr/scripts/twister --ninja --inline-logs --jobs 1 -T zephyr/samples/drivers/mbox --platform nrf54h20dk/nrf54h20/cpurad --device-testing --device-serial /dev/serial/by-id/usb-SEGGER_J-Link_001051165753-if02 -v -v --west-flash="--erase"
(this timeouts unless serial port on cpuapp is opened; most likely due to printk and UART HW flow control)
OR
west build -b nrf54h20dk/nrf54h20/cpuapp --pristine --test-item sample.drivers.mbox.nrf54h20_app_ppr .
west build -b nrf54h20dk/nrf54h20/cpurad --pristine --test-item sample.drivers.mbox.nrf54h20_rad_app .
west flash --erase


DDT-759: Logic for POST shipping-instructions

also looks goog to me (with the TODO fixes mentioned in the code ;-)


Updates to get support on modern GJS

This PR fixes polydock so it works with GJS 1.72. Doing this required some changes to the pnpm installation so I could build from source with changes to pnpm, as well as changing tsconfig to target a more modern js version. Finally I also needed to make some slight app changes to match modern GJS subclassing

Sorry for the late merge!
I no longer use polydock myself, since I moved to wayland.

@folke can you please release this. latest version is still broken.


Enhancement

This is not an issue but rather an enhancement suggestion. First of thank you Step-up-labs for the API it's really helpful to a lot of C# developers.
I would like to suggest a log in or sign up enhancement using the Firebase SMS authentication. This will be really helpful as it's not currently supported unless you go native especially for Xamarin Forms developers.
Thank you.

Duplicate of https://github.com/step-up-labs/firebase-authentication-dotnet/issues/58


Fix disabled unit tests

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2828/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2832/
Test PASSed.

alluxio-bot, merge this please


@import "@material/ripple/mixins"; causes error build in MDC-103 Web

When I import "@material/ripple/mixins" in home.scss I get this error below
ERROR in ./home.scss
Module build failed:
@forward "@material/animation" as mdc-animation-;
^
Can't find stylesheet to import.
╷
1 │ @forward "@material/animation" as mdc-animation-;
│ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
╵
<EMAIL_ADDRESS>1:1  @import
stdin 5:9                                              root stylesheet

I had this problem, too. The solution is in issue #232:
In the @import, change "@material/ripple/mixins" to "@material/ripple/mdc-ripple"


fix bug in maxout function

The line "shape[axis] = -1" will make the shape wrong when dealing with batches with arbitrary sizes.
if the shape of input tensor is [None, ... , num_channels, ... ], "shape[axis]=-1" together with "shape+=[num_channels // num_units]" will make the shape become [None, ... , -1, ... , num_channels // num_units]. But when reshape the input tensor, the "None" element in shape will make "-1" become "None", not "num_units".

@martinwicke can I merge this?

Check with @yifeif but I think yes.

You are good if it's for non-master branches.

MacOS flake. Re-running.

Going to merge this fix into 1.9 so we can start official release builds. Please follow-up with @pavithrasv comment and add a test case with None please though.
Thanks!


Add metadata option to config of GCS publisher so headers such as cache-control can be set

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Problem description
Using the GCS publisher I did not see a way of adding metadata to the GCS object.
This causes files to revert to default metadata, including the cache-control header, which is set to a 3600 second default cache.
RELEASES.json is affected by this header as well.
I want my cache-header to be set to no-store so that I can push updates instantly and not have to wait for cache to expire.
Attached is a pull request that adds metadata as a config option.
https://github.com/electron/forge/pull/3682
Proposed solution
This pull requests adds a metadata option to the config of the GCS publisher.
https://github.com/electron/forge/pull/3682
Alternatives considered
I can go into GCS and update the metadata after every publish but I would rather set this on the bucket upload itself.
Additional information
No response

Closed by #3682


[GDP] Move existing Grant Details content to match new placement

Why is this issue important?
Sub-task of Grant Detail Page sprint.
Current State
Content placement matches what is in the old modal in GrantDetailsLegacy.vue
Expected State
Change layout of existing content in GrantDetails.vue to match the new placement in the design
Implementation Plan
No response
Relevant Code Snippets
No response

Design file for Grant Details, to be reviewed on Mon 1/15:
https://www.figma.com/file/yQx55VTr5awPYClHIJCz8y/Grant-Details?type=design&node-id=1110-2419&mode=design

I think this is now covered by other issues


adding ios-safari screenshots

PR is adding capability to take ios-screenshots. These will be used as goldens.

I see a couple of issues:

Every screenshot has a horizontal line at the top of the screenshot, as if the header height is not configured correctly.
Some images do not seem to be cropped correctly. For example, https://github.com/flutter/goldens/pull/110/files#diff-38216bc9db8e3dc32cc55edc3fa069f4 is very tall, so I'd expect it to be cropped on the bottom if it doesn't fit the iPhone screen, but instead, it seems to be cropped on the right side.

I see a couple of issues:

Every screenshot has a horizontal line at the top of the screenshot, as if the header height is not configured correctly.
Some images do not seem to be cropped correctly. For example, https://github.com/flutter/goldens/pull/110/files#diff-38216bc9db8e3dc32cc55edc3fa069f4 is very tall, so I'd expect it to be cropped on the bottom if it doesn't fit the iPhone screen, but instead, it seems to be cropped on the right side.

I don't see any horizontal line, can you send me a screenshot or a name or something I can use?
https://raw.githubusercontent.com/flutter/goldens/master/engine/web/backdrop_filter_clip.png
https://raw.githubusercontent.com/flutter/goldens/master/engine/web/draw_vertices_triangle_fan.png

I see a couple of issues:

Every screenshot has a horizontal line at the top of the screenshot, as if the header height is not configured correctly.
Some images do not seem to be cropped correctly. For example, https://github.com/flutter/goldens/pull/110/files#diff-38216bc9db8e3dc32cc55edc3fa069f4 is very tall, so I'd expect it to be cropped on the bottom if it doesn't fit the iPhone screen, but instead, it seems to be cropped on the right side.

I adjust the scaling as we discussed offline. I also removed the header from the images that have them. PHAL

Thanks for the review!


build: Update Vulkan-Headers to 1.3.257

No generated code changes.

CI Vulkan-ExtensionLayer build queued with queue ID 2531.

CI Vulkan-ExtensionLayer build # 591 running.

CI Vulkan-ExtensionLayer build queued with queue ID 2541.

CI Vulkan-ExtensionLayer build # 592 running.

CI Vulkan-ExtensionLayer build # 592 passed.


ALVR breaks in the nwest version when using HEVC codec

Description
When connecting with HEVC codec selected, screen shows SteamVR lobby(void thing, not room) for a moment and then freezes. Also sometimes some colors flash before that.
Update: Sometimes it brings you to steam home and then heavily lags and freezes.
General Troubleshooting

[x] I carefully followed the instructions in the README and successfully completed the setup wizard
[x] I read the ALVR Wikis here and here

Environment
Hardware
CPU: Intel(R) Xeon(R) CPU E3-1270 V2
GPU: AMD RX Vega 56
Audio: ?
Installation
ALVR Version: 19.1.0
SteamVR Version: 1.24(.7)
Install Type:

[x] Packaged (exe, deb, rpm, etc)
[ ] Portable (zip)
[ ] Source

OS Name and Version (winver on Windows or grep PRETTY_NAME /etc/os-release on most Linux distributions): Windows 10 22H2

it works with h264 codec but looks awful


Properly add JDBC_EXTENSIONS pragma (#760)

Instead of relying of changing ifdefs use the variable decralation as anchor.
This fixes #760

@gotson Here it is.

Thanks a lot, works like a charm.


[WIP] Tenant Creation

Description
This pr adds the logic for the creation/deletion of Capsule Tenants during the Liqo peering/unpeering process.
How Has This Been Tested?

[x] locally on KinD
[x] new unit test

Pull Request Test Coverage Report for Build 982898526

64 of 83 (77.11%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 47.968%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/mapperUtils/mapper.go
0
3
0.0%

internal/resource-request-operator/resourceRequest_controller_methods.go
50
56
89.29%

internal/resource-request-operator/resourceRequest_controller.go
14
24
58.33%

Totals

Change from base Build 982150830:
0.1%

Covered Lines:
9904

Relevant Lines:
20647

💛  - Coveralls


This issue is happening again. Can anyone check it?

          This issue is happening again. Can anyone check it?

Originally posted by @ZuhaibSohail163 in https://github.com/mui/material-ui/issues/28689#issuecomment-1710217476

Hey @ZuhaibSohail163, thanks for the report 😊
I cannot reproduce the behavior by changing to the Russian language, as described in https://github.com/mui/material-ui/issues/28689. Could you provide repro steps for your case? What browser and language are you using? Any additional information might help.

Sorry, it was due to my browser settings. Now it is working alright. Thanks for the response @DiegoAndai .

No problem!


Ctrl-P is not recieved in vim

Windows Terminal version (or Windows build number)
1.7.1033.0
Other Software
NVIM v0.5.0-dev+1256-gd19348923
Steps to reproduce

Open vim
Bind something to ctrl-p : :nmap <Ctrl-p> :echo "Ctrl P triggered"
Press Ctrl-P

Expected Behavior
Expecting to see "Ctrl P triggered" message
Actual Behavior
Nothing is shown

Tried unbinding it via
{
    "command": "unbound",
    "keys": "ctrl+p"
}

With no luck

Thanks a lot! TLDR; It has nothing to do with Windows Terminal.
I had an extension in my browser that blocks desired key mappings. I configured it to block Ctrl-P and Ctrl-W, so when my vim reflexes poped up, i wouldn't close any tab or start printing it.
For some unknown reason extension also captured said keystrokes when browser was minimized.
I'm very sorry for bothering, thanks for suggested method of debugging it via wsl. The moment i pressed Ctrl-P and saw no keycode was exactly what made my brain click.

Glad you were able to figure it out!
<off topic>

For some unknown reason extension also captured said keystrokes when browser was minimized.

wow that sounds dangerous. Imagine if a malicious extension globally bound all keys, but then sent them through as if nothing happened. It could install itself as a global keylogger 😨

Yeah, that sounds ominous. Will make an issue in Brave repo, maybe that's unintentional behavior.


Clean up dead code

I didn't touch any public or codegen'd declarations, only internal or lower declarations that weren't used in anywhere in this project, including tests.
I also opted to keep some of the unused code that serves as local development helpers, things like recursiveDescription since that seems best to keep around even if not used.
I'm happy to re-add anything that's useful to keep around.
By the way, these unused declarations were found using a SwiftLint rule that's still in development, partly powered by SwiftSyntax itself.

@swift-ci test

Sorry @CodaFi, forgot to update for CMake. That's done now and builds cleanly for me with cmake -B build -G Ninja -S . -DCMAKE_BUILD_TYPE=Release and cmake --build build.

That last failure is interesting.
Undefined symbols for architecture x86_64:
  "(extension in SwiftSyntax):SwiftSyntax.SyntaxProtocol.withoutTrivia() -> A", referenced from:
      swiftASTGen.evaluateMacro(sourceFilePtr: Swift.UnsafePointer<Swift.UInt8>, sourceLocationPtr: Swift.UnsafePointer<Swift.UInt8>?, expandedSourcePointer: Swift.UnsafeMutablePointer<Swift.UnsafePointer<Swift.UInt8>?>, expandedSourceLength: Swift.UnsafeMutablePointer<Swift.Int>) -> Swift.Int in Macros.swift.o
ld: symbol(s) not found for architecture x86_64

swiftASTGen.evaluateMacro is referring to some code that I had removed, but my tool couldn't detect that. I'm not even sure where that's happening or if it's possible to detect this statically.

@swift-ci test

@swift-ci test

@CodaFi looks like CI's back from vacation so I think this could be merged now.

@swift-ci test


Shift from unittest to pytest

Is your feature request related to a problem? Please describe.
Unittest is cool, but pytest is cooler. It packs more features and as this continues to grow I can see myself leveraging them further for automated testing.
Describe alternatives you've considered
Sticking with unittest. Its a meh thing, I want to change and further my own skills

This has been done in the rewrite.


BFS AND DFS TRAVERSAL IN C++

this my first contribution in open source through SSOC'23

when this will be merged and is this contribution added to ssoc??

You should first raise an issue and after you are assigned, then only you open a PR


Runner Parallel execution creates OOM when executed with large number of csv rows

I have checked the following:

[X] I use the newest version of bruno.
[X] I've searched existing issues and found nothing related to my issue.

Describe the bug
I have several 1000 row csv with 5 columns
In my collection I have 3 requests
Each request is executed 1000 times
When runner executes it allocates memory and ports for all 3000 requests all at once and pretty much creates a dos on bruno and my computer until it falls over when allocating over 5GB of memory and 3000 ports.
can you please enable the option to run serially.. its usual to offer a api app the option to only always run in parallel
.bru file to reproduce the bug
No response
Screenshots/Live demo link
https://discord.com/channels/1036297729606963331/1036367269804130314/1284753638857904251
https://discord.com/channels/1036297729606963331/1036367269804130314/1284749828085121024

I am able to work around this by using the cli which does execute serially.

Thanks for submitting this @bstupid, we've logged it as a feature request as we do not currently support serial execution of .csv via the CLI.

@Rreyn-Bruno your miss reading the issue. with Runner via the GUI if you have an csv driving execution anything more than 300 rows creates a massive issue and OOM error for Bruno and makes runner pretty much useless.
With CLI which DOES NOT REQUIRE a fix is the work around because it only run's serially and does not create this issue. This is how Runner should work but does not offer this option.
Runner only runs in parallel which is the problem.
Anyone coming from PM will expect runner to work serially, It doesnt make sense for runner to only run in parallel where it directly impacts Bruno.. it basically locks up or crash's before allowing users access to the Bruno gui. usually after many hours.

Support for Serial execution will be available in the upcoming v1.35.0 release which is set to be released tomorrow.


Add Travis config for Python SDK tests

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Adds a basic Travis config for installing and running tests with tox. tox is only configured to setup.py test. This will provide a basic level of presubmit coverage.

R: @jasonkuster, @silviulica, @davorbonaci

Beautiful! LGTM.
(We should do the same on Jenkins separately.)

Yuhuu!!! LGTM

@davorbonaci please take another look.

LGTM.

Also, feel free to squash and rebase at this point.


Two public ways to get session's Fluent version

Is it just a matter of renaming version to _version or can we remove it altogether?

We can remove the usage of version altogether and can instead access get_fluent_version() always. There are couple of reasons why I think this: 1. version is not available in meshing sessions, so it is confusing for users, 2. version in session_solver falls back to use get_fluent_version only internally.

@seanpearsonuk
solver.version is specifically used for codegen. We will rename it to _version. Otherwise we will have to pre-process Fluent version something like if float(solver_session.get_fluent_version()[:-2]) < 23.0: for codgen.


Update Chrome Webstore version (20.2.18) to latest version

The Chrome Webstore release is at version 20.2.18 and the master is at 20.2.28 as of this date.
https://chrome.google.com/webstore/detail/refined-github/hlepfoohegkhhmjieoechaddaejaokhf
No screenshots/URL since this is not an issue with the plugin, however, it is an issue with a release platform.

Chrome takes a long time to release updates


install error

OS: Mac v12.5
Python: 3.10.6
Install script is throwing this error in the seed dcs stage.
$ python --version
Python 3.10.6
$ make install
...
python -m ambuda.seed.dcs
Fetching latest data ...
Cloning into '/Users/kvic/src/ambuda/data/ambuda-dcs'...
remote: Enumerating objects: 47, done.
remote: Counting objects: 100% (34/34), done.
remote: Compressing objects: 100% (25/25), done.
remote: Total 47 (delta 21), reused 20 (delta 9), pack-reused 13
Receiving objects: 100% (47/47), 10.10 MiB | 96.00 KiB/s, done.
Resolving deltas: 100% (25/25), done.
Already on 'main'
Your branch is up to date with 'origin/main'.
HEAD is now at 7622c22 Fix double tab in meghadutam parse data
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File<EMAIL_ADDRESS>line 86, in _run_code
    exec(code, run_globals)
  File "/Users/kvic/src/ambuda/ambuda/seed/dcs.py", line 123, in <module>
    run()
  File "/Users/kvic/src/ambuda/ambuda/seed/dcs.py", line 102, in run
    add_parse_data(path.stem, path)
  File "/Users/kvic/src/ambuda/ambuda/seed/dcs.py", line 87, in add_parse_data
    for slug, blob in iter_parse_data(path):
  File "/Users/kvic/src/ambuda/ambuda/seed/dcs.py", line 68, in iter_parse_data
    raise ValueError(f'Line "{line}" must have exactly three fields.')
ValueError: Line "gOtamaH	gOtama	pos=n,g=m,c=1,n=s" must have exactly three fields.
make[1]: *** [db-seed-basic] Error 1
make: *** [install] Error 2```

Thanks, will fix soon. For now, delete the error check.


Update CartId in the Local Storage When Customer Add a Product To the Cart

Hyva checkout needs the cartId (the mask id) to be stored in local storage. The path is mage-cache-storage.cart.cartId. So whenever the customer add a product to the cart, we need to update the customerData which is going to update by Magento for the frontend contains the cartId within it.

This is implemented by adding the module https://github.com/hyva-themes/magento2-graphql-tokens as part of hyva-checkout.
You can see the related merge request here: #35


Greyed out profile

somehow managed to get a invisible modal when going from drop -> my profile with a evm wallet connected
scroll is disabled and clicking anywhere closes the modal and everything is alright
https://github.com/user-attachments/assets/8c213ce6-73e9-49a0-a6ec-8aa9935c76c7

should by fixed by #10834


Schema compare - filter Database schemas

**Is your feature request related to a problem?
N/A
Describe the solution or feature you'd like
Add a drop down filter field to filter SQL Database schemas. Azure Synapse dedicated SQL pool can contains a lot of schemas. Schema compare is too slow to compare all tables, views…
With a schema option filter, we could just compare one or more specifics schemas.
Describe alternatives you've considered
Additional context

closing as duplicate of https://github.com/microsoft/azuredatastudio/issues/12964


Use version 2.0.0-beta2 engine components

:tada: This issue has been resolved in version 2.1.0 :tada:
The release is available on:

GitHub Release
NuGet Package
Chocolatey Package


Add default attribute to the derive macro for Properties

Is your feature request related to a problem? Please describe.
If an optional property isn't present, it may not always be desirable to set it to the default value. Sometimes a boolean property should default to true or an integer should default to a non-zero number. Also, optional callbacks are a pain right now because defaulting them to a no-op behaviour could be unexpected to a developer (so yew doesn't implement Default for them)
Describe the solution you'd like
Add a default property to the derive macro that behaves similarly to serde's default attribute https://serde.rs/field-attrs.html#default--path
Ideally, the default can be specified with an expression and if no expression is given, falls back to Default::default().
Any property that has a default attribute is implicitly optional.
Examples:
#[derive(Clone, Properties)]
struct Props {
  #[props(default: 3),
  countdown: usize,

  #[props(default: Callback::noop()),
  on_click: Callback<()>,

  #[props(default: true),
  display: bool,

  #[props(default),
  highlight: bool,
}

It's supposed to be default = …, not default: …, right?

@jplatte I believe either is supported by Rust now. I thought default: ... looked nicer but default = ... is fine too

Yeah, either should be possible. I've just never seen the colon syntax used in attributes. AFAIK all builtin attributes use = (e.g. #[path = "foo.rs"], #[cfg(feature = "feat")] ).

@jplatte @mdtusz @AlephAlpha @hgzimmerman I just updated this issue to be a new proposal for how we handle props and would like input. This builds upon the work that @AlephAlpha already did in #881

I like the new proposal :)
We could maaaybe have #[prop_or_else(Callback::noop)] (fn) and #[prop_or(true)] (value) to mirror Options / Results API, but I don't know if that really makes things simpler.

Good call, that would probably be more in line with expected behavior. It'd be closer to Option's API

No comments from me. Looks like a great improvement for usability and grok-ability!

I'm trying to implement this. Should we support the old syntax for backward compatibility?

No need, would complicate things too much


Changes to json file in JSONServiceRegistry are not dynamically loaded into CAS Server

Closes #2166
Changes:

recreate the ordered Set
add tests for this cases

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.David Rodriguez seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.


update 1.0 e2e to master with 1.0 accommodations

1.0 accommodations

volumesnapshot used is v1 in master. revert to v1beta1 in 1.0.

Codecov Report

:exclamation: No coverage uploaded for pull request base (oadp-1.0@a6dcd34). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             oadp-1.0     #830   +/-   ##
===========================================
  Coverage            ?   37.50%
===========================================
  Files               ?       14
  Lines               ?     3066
  Branches            ?        0
===========================================
  Hits                ?     1150
  Misses              ?     1816
  Partials            ?      100

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

/retest

This is failing because in 1.0 we do not have https://github.com/openshift/oadp-operator/blob/master/docs/scripts/dc-restic-post-restore.sh

https://github.com/openshift/openshift-velero-plugin/pull/141 does not exist in oadp-1.0 so update this to go back to two stage restic restore.

Closing for now. Might revisit later.


feat(config): sync smtp and email config to remote

What kind of change does this PR introduce?
part 5 of https://github.com/supabase/cli/pull/2814
What is the new behavior?
Syncs email and custom smtp config to remote.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

8 of 16 (50.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 59.751%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/start/start.go
8
16
50.0%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.9%

Totals

Change from base Build<PHONE_NUMBER>0:
-0.07%

Covered Lines:
6385

Relevant Lines:
10686

💛  - Coveralls

@avallete I'm moving email template to a separate PR because adding Content field also requires updating the local nginx/kong server where these templates are served.


Using Adalam Matcher

Recently the Adalam matcher was merged, but I don't seem to be able to use it. Does it only work for certain types of features? If so, which?
I am running off the current head at the time of writing which is 972b57fe695c4e3b3634c9140acec58060de7ecc.
The error I receive, during reconstruction.main is:
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
Cell In [12], line 1
----> 1 model = reconstruction.main(sfm_dir, images, sfm_pairs, feature_path, match_path)

File ~/Hierarchical-Localization/hloc/reconstruction.py:118, in main(sfm_dir, image_dir, pairs, features, matches, camera_mode, verbose, skip_geometric_verification, min_match_score, image_list, image_options, mapper_options)
    116 image_ids = get_image_ids(database)
    117 import_features(image_ids, database, features)
--> 118 import_matches(image_ids, database, pairs, matches,
    119                min_match_score, skip_geometric_verification)
    120 if not skip_geometric_verification:
    121     estimation_and_geometric_verification(database, pairs, verbose)

File ~/Hierarchical-Localization/hloc/triangulation.py:93, in import_matches(image_ids, database_path, pairs_path, matches_path, min_match_score, skip_geometric_verification)
     91 if min_match_score:
     92     matches = matches[scores > min_match_score]
---> 93 db.add_matches(id0, id1, matches)
     94 matched |= {(id0, id1), (id1, id0)}
     96 if skip_geometric_verification:

File ~/Hierarchical-Localization/hloc/utils/database.py:200, in COLMAPDatabase.add_matches(self, image_id1, image_id2, matches)
    199 def add_matches(self, image_id1, image_id2, matches):
--> 200     assert(len(matches.shape) == 2)
    201     assert(matches.shape[1] == 2)
    203     if image_id1 > image_id2:

AssertionError:

AdamLAM is compatible only with features based on DoG, which has per-point orientation and scale, so sift or sosnet in hloc.extract_features. When trying to use it with other features (e.g. SuperPoint), it should fail at
https://github.com/cvg/Hierarchical-Localization/blob/972b57fe695c4e3b3634c9140acec58060de7ecc/hloc/matchers/adalam.py#L45

@Skydes Thanks for the reply. It doesn't seem to be this.  The same error appears to happen with both sosnet and sift.
I'm just working off the pipeline_SfM notebook, with the only change being swapping the feature and matcher confs.

I think it may be crashing while trying to add empty matches to the database?

Might just be putting a plaster on a deeper problem, but this appears to fix it:
https://github.com/cvg/Hierarchical-Localization/pull/240

This is an unrelated error, try to run the whole process from scratch on a empty directory.


Set pointer-events: none to TileLayer and all non-iteractive layers

Can I ask what the reason was for removing pointer events from tile layers?
Only the removal of "view image" etc from the context menu has caused a certain amount of complaints on openstreetmap.org and if I knew the reason for this change it would make it easier to decide whether to override the CSS and re-enable them there.

Off the top of my head, I don't think there is a strong rationale or obvious use case.
From my personal point of view, preventing pointer events on gridlayers/tilelayers would prevent some weird edge cases such as a mobile browser zooming to a tile's extent on double-tap. It should be possible that the touch-action CSS will work for this, but there might be some browser weirdness. This means that overriding the pointer-events CSS rule should be a safe option and not cause any problems.
Also Leaflet does not expect users to attach pointer event handlers to gridlayers/tilelayers, so the easiest way to prevent problems (or additional logic) with event bubbling has been to avoid events in gridlayers altogether.
Please also note that https://github.com/Leaflet/Leaflet/pull/5290 would get rid of <img>s in the DOM altogether, by stitching them in a <canvas> like OpenLayers does. It's the only way to get rid of https://github.com/Leaflet/Leaflet/issues/3575 I'm afraid. This is planned, and the sensible idea is to have it on by default, but can be disabled.

Thanks @IvanSanchez. Sounds like it might be best to avoid that then and add something (more sophisticated) to our new context menu then.


Do you know of any repos / lists of sites tracking consumer scores?

There was a recent any article about websites tracking consumer scores and how to request your data from them.
Do you know if there is any repo listing websites such as those? When Googling the closest thing I found right away was this repo, which has quite a different purpose.

I think Cracked Lab has some information on these kind of companies.


add isPrinted property to Stencil to switch between printed and stenc…

this is, what I would otherwise write into CR :)
there is a new attribute isPrinted to decide, which SVG to use when displaying stencil.
(accompanied with reset() method to allow the once-printed stencil to reinitialise)
https://user-images.githubusercontent.com/2682941/231067023-20a955e7-73fa-438d-8c67-a93113bba6ef.mov

Well the problem is that now we miss creation of stencil completely, therefore we do not explain screen printing much. But I think since we switched from big squeegee to small squeegee there is not much we can do

you are right. I think that creating of the stencils can be mentioned in the step description. but for now let's take them as "give once". we are just "printing" another version. stencils were already made.


Adaptive Card Designer - Host App for Federated Search Channel

Creating host app for Federated Search Channel:
The change is for new requirement to include preview card for Federate Search Channel in adaptive card designer app.
Added new card in designer template for Federated Search Channel.
Developers can use this to preview the adaptive card response on Search channel before actually deploying to prod.
Verified manually the layout . Attaching the screen shot as per current configuration.

Microsoft Reviewers: Open in CodeFlow

Hi David ,
I had no instructions related to Federated Search changes so I had copied directly from Outlook. I have resolved the review comments.
But to your point to revert to the previous the formatting , for example I reverted the changes did to app.ts  and removed // eslint-disable-next-line no-console.
And after that it is not allowing to commit hence I had to make the changes I have made to existing files. Please find the error below:

git -c user.useConfigOnly=true commit --quiet --allow-empty-message --file -
[STARTED] Preparing...
[SUCCESS] Preparing...
[STARTED] Running tasks...
[STARTED] Running tasks for */!(package-lock)
[STARTED] eslint --fix --max-warnings=0 C:/MyProjects/AdaptivecardRepo/AdaptiveCard…
[FAILED] [91meslint --fix --max-warnings=0 C:/MyProjects/AdaptivecardRepo/AdaptiveCards/source/nodejs/adaptivecards-designer-app/src/app.ts C:/MyProjects/AdaptivecardRepo/AdaptiveCards/source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.ts[39m [2m[FAILED][22m
[SUCCESS] Running tasks...
[STARTED] Applying modifications...
[SKIPPED] Skipped because of errors from tasks.
[STARTED] Reverting to original state because of errors...
[SUCCESS] Reverting to original state because of errors...
[STARTED] Cleaning up...
[SUCCESS] Cleaning up...

[91m[91m×[91m eslint --fix --max-warnings=0 C:/MyProjects/AdaptivecardRepo/AdaptiveCards/source/nodejs/adaptivecards-designer-app/src/app.ts C:/MyProjects/AdaptivecardRepo/AdaptiveCards/source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.ts:[39m
C:\MyProjects\AdaptivecardRepo\AdaptiveCards\source\nodejs\adaptivecards-designer-app\src\app.ts
52:5  error  Unexpected console statement  no-console
✖ 1 problem (1 error, 0 warnings)
husky - pre-commit hook exited with code 1 (error)
Kindly let me know how can I resolve this.
Best Regards,
Nayana.
From: David Claux @.>
Sent: Wednesday, February 23, 2022 10:39 PM
To: microsoft/AdaptiveCards @.>
Cc: Nayana Ramakanth @.>; Mention @.>
Subject: Re: [microsoft/AdaptiveCards] Adaptive Card Designer - Host App for Federated Search Channel (PR #7061)
@dclaux requested changes on this pull request.
Beyond my comments, the changes modify the formatting of certain existing files. Please revert to the previous formatting.

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813111687&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270565677|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=9Cd7ZSN7kWq8%2F8disJy1MNAOWV5BvNJ%2B0SjAIbvn%2BRA%3D&reserved=0:

+import * as Adaptive from "adaptivecards";

+import { HostContainer } from "../host-container";
+import * as hostConfig from "../../hostConfigs/federated-search.json";

+export class FederatedSearchContainer extends HostContainer {

constructor(name: string, styleSheet: string) {

super(name, styleSheet);

this.actionsRegistry.unregister("Action.Submit");

this.elementsRegistry.unregister("Input.Text");

this.elementsRegistry.unregister("Input.Date");

this.elementsRegistry.unregister("Input.Time");

this.elementsRegistry.unregister("Input.Toggle");

this.elementsRegistry.unregister("Input.ChoiceSet");

this.elementsRegistry.unregister("Input.Number");

this.actionsRegistry.register("Action.Http", Adaptive.HttpAction);

Why would Federated search support Action.Http? Shouldn't FS only support Action.Execute? Action.Http was used solely by Outlook and we have now moved to the universal action model (Action.Execute) which is what FS should use.

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813112778&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270565677|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=3raKPA224PtxdpEsItmeAg0FS52aJd1guG%2BNOR%2B%2FAMU%3D&reserved=0:

constructor(name: string, styleSheet: string) {

super(name, styleSheet);

this.actionsRegistry.unregister("Action.Submit");

this.elementsRegistry.unregister("Input.Text");

this.elementsRegistry.unregister("Input.Date");

this.elementsRegistry.unregister("Input.Time");

this.elementsRegistry.unregister("Input.Toggle");

this.elementsRegistry.unregister("Input.ChoiceSet");

this.elementsRegistry.unregister("Input.Number");

this.actionsRegistry.register("Action.Http", Adaptive.HttpAction);

this.cardHost.classList.add("fedsearch-card");

}

public renderTo(hostElement: HTMLElement) {

hostElement.classList.add("outlook-frame");

Why use an "outlook" branded frame when this HostContainer is for Federated Search? Plus this class doesn'r even seem to be defined in your css.

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813114367&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270565677|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=d7RNb5lQnMoDTpFU9DqhDNewSvSAUxvZt6H2I%2BRZ4lc%3D&reserved=0:

this.actionsRegistry.register("Action.Http", Adaptive.HttpAction);

this.cardHost.classList.add("fedsearch-card");

}

public renderTo(hostElement: HTMLElement) {

hostElement.classList.add("outlook-frame");

hostElement.appendChild(this.cardHost);

}

public initialize() {

super.initialize();

Adaptive.GlobalSettings.useMarkdownInRadioButtonAndCheckbox = false;

}

private parsePadding(source: any): Adaptive.PaddingDefinition {

This seems to have been copied straight from the Outlook host container. I doubt Federated Search cards support explicit padding (and if they do they probably shouldn't). This method likely should be removed.

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813115459&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270565677|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Rx57xOUEdhN4zy%2BlZhArFeZeO6HETmDNQ%2FKXyhPPO8U%3D&reserved=0:

     Adaptive.parseEnum(Adaptive.Spacing, source["right"], Adaptive.Spacing.None),

     Adaptive.parseEnum(Adaptive.Spacing, source["bottom"], Adaptive.Spacing.None),

     Adaptive.parseEnum(Adaptive.Spacing, source["left"], Adaptive.Spacing.None)

   );

 }

}

return null;

}

public parseElement(

element: Adaptive.CardElement,

source: any,

context: Adaptive.SerializationContext

) {

if (element instanceof Adaptive.Container && typeof source["rtl"] === "boolean") {

This isn't necessary. The rtl property has been an official AC property for a while and is parsed automatically.

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813116193&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270565677|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=w15xFjfWh5ydyulCnlpQJIqB4nc3ZSrFUKxzjUI9VxA%3D&reserved=0:

 }

}

return null;

}

public parseElement(

element: Adaptive.CardElement,

source: any,

context: Adaptive.SerializationContext

) {

if (element instanceof Adaptive.Container && typeof source["rtl"] === "boolean") {

 element.rtl = source["rtl"];

}

if (element instanceof Adaptive.AdaptiveCard) {

All this needs to be removed... It's something that only Outlook supports.

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813116356&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270565677|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=GP4egeYq%2Bi7XJ2nn01IxtTkYxhYpkhSSB0BeFLvVFbg%3D&reserved=0:

   const actionResources = source["resources"]["actions"] as Array<any>;

   actionResources.forEach((action) => {

     this.actionsRegistry.createInstance(actionResources["type"], context.targetVersion);

     if (action) {

       action.parse(actionResources, context);

       action.setParent(card);

       actionArray.push(action);

     }

   });

 }

}

if (element instanceof Adaptive.Image) {

Remove

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813116476&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270565677|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=AEy3YS9hLEKwmJVjeInRoIer6DlVZb9xUCBEwIB226Y%3D&reserved=0:

     if (action) {

       action.parse(actionResources, context);

       action.setParent(card);

       actionArray.push(action);

     }

   });

 }

}

if (element instanceof Adaptive.Image) {

 element.backgroundColor = source["backgroundColor"];

}

if (element instanceof Adaptive.Container) {

Remove

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813116604&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270565677|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=sbVIyf3USm2ugkw7VEAMwAxQ%2BXUImGszlR3gIZkw3zI%3D&reserved=0:

 }

}

if (element instanceof Adaptive.Image) {

 element.backgroundColor = source["backgroundColor"];

}

if (element instanceof Adaptive.Container) {

 const padding = this.parsePadding(source["padding"]);

 if (padding) {

   element.padding = padding;

 }

}

if (element instanceof Adaptive.ColumnSet) {

Remove

In source/nodejs/adaptivecards-designer/src/containers/federated-search/federated-search-container.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23discussion_r813116964&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270615664|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=1xx3kCXVGx1X4l3EIzwqPwtsWZJkPtmdzfizNqy7Dh8%3D&reserved=0:

 if (padding) {

   element.padding = padding;

 }

}

if (element instanceof Adaptive.ColumnSet) {

 const padding = this.parsePadding(source["padding"]);

 if (padding) {

   element.padding = padding;

 }

}

}

public anchorClicked(element: Adaptive.CardElement, anchor: HTMLAnchorElement): boolean {

Remove
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FAdaptiveCards%2Fpull%2F7061%23pullrequestreview-891388848&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270615664|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=BIJIrZoFhZVJ%2FKf0mxSm3InrJ1INrhaRX%2BPSW0xTXG8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPXKLOBOHW26SOCFANDCLVDU4UIBVANCNFSM5PEDJFXQ&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270615664|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=Qrs3sbIfCThHhjf97%2B%2FHWGDPBqwnZcUND01niuUg7Kk%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270615664|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=zfZm073vR7EA4m2K5xJIqoDpQfYKJSkOS7XJhFuBXGI%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|Nayana.Ramakanth%40microsoft.com|fb8e9b1e41a24182680308d9f6ef24b6|72f988bf86f141af91ab2d7cd011db47|0|0|637812329270615664|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=nbGYJbfjxdaRsbqtC%2FgsfEv5HQu%2BgZpAfql68RgImr8%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@naramaka sorry, but I have no idea how reverting to the original formatting could possibly prevent the code from building. I'm not going to be able to help with that.
Thanks for making the code changes. However, without a new commit, I can't review your last changes.

@naramaka sorry, I had missed it (don't know how...)

@naramaka just to be suere - this is the styling you're going for in search for buttons?

Asking because they don't necessarily look like buttons I tend to see in other places, like in Office.com for instance:

Also, the font used in your buttons isn't the same as the font used in the rest of the card.

@naramaka just to be suere - this is the styling you're going for in search for buttons?
Asking because they don't necessarily look like buttons I tend to see in other places, like in Office.com for instance:
Also, the font used in your buttons isn't the same as the font used in the rest of the card.

@dclaux There is no specification yet. I am submitting initial configuration so that we can improve it as and when we have the clarity. But if you do have any default specification then let me know I can change.

@licanhua - can you review?


[material-ui][Autocomplete] Display options provided to the options prop even if loading is true.

fix #41633
Display loading message even if the options exist.

[x] I have followed (at least) the PR section of the contributing guide.

@ZeeshanTamboli please make sure that fixes, especially regressions are covered with tests, otherwise we have no guarantee that this will not break again.

@ZeeshanTamboli please make sure that fixes, especially regressions are covered with tests, otherwise we have no guarantee that this will not break again.

Aplogies. Created a PR for adding a test: #41675.

@ZeeshanTamboli @mnajdova Thanks for merging. Could this fix be applied to v5 as well? Is there a plan to backport it?

@nekoya Here's the PR link that should be merged for v5: #41677.


Found  NATIPs conf option, but don't think its taking effect.

Running mox in a homelab environment, natted.
Is this correct in the email details, trail ?
<html>
<body>
<!--StartFragment-->
10mins 6s ago | 90517ms |   |   |   | dialing smtp server: dial tcp <IP_ADDRESS>:0-><IP_ADDRESS>:25: i/o timeout
-- | -- | -- | -- | -- | --

<!--EndFragment-->
</body>
</html>

Seems like it should be the public facing ip address ?

The NATIPs option only tells mox that it is running in a NAT configuration, and what the actual external IPs are. Mox uses those IPs for its self-checks. It does not influence how outgoing smtp connections are made. So the IPs in the dial error message look reasonable. It's not great that you're seeing a timeout... Hope that doesn't happen for all outgoing connections.
What were you expecting the NATIPs config option to do?

The NATIPs option only tells mox that it is running in a NAT configuration, and what the actual external IPs are. Mox uses those IPs for its self-checks. It does not influence how outgoing smtp connections are made. So the IPs in the dial error message look reasonable. It's not great that you're seeing a timeout... Hope that doesn't happen for all outgoing connections.

What were you expecting the NATIPs config option to do?
Just trying to figure out why i cannot get outbound email to work, actually and inbound email as well.

Yea i tried DNS Checker, and another 1 that wants to charge for the info, but neither seemed to be of much help.
The free on did provide
Reverse DNS does not match SMTP Banner.

But i couldn't find any further info , on what that means, or how to repair
My ISP is Cox, and i've never seem to got much help from them. To tell you the truth i don't think their very smart about any of this.
They just want you to use what you pay for as run of the mill, and not have to help at all.

Would this cause a problem ?
`
IP: <IP_ADDRESS>
Hostname: ip68-12-151-20.ok.ok.cox.net.
`
Even though i'm trying to claim it on my dns ?

Could you give some more details of the error messages you are seeing? There's not enough to go on now.
Is it incoming and outgoing email that isn't working?
It will help to see logging from mox. You can enable trace logging to get all the protocol details.
FWIW, I cannot connect to <IP_ADDRESS> on port 25. You may be able to connect from inside your network, but from outside it appears blocked.

Yes neither incoming nor outgoing email is occurring.
I will be happy to provide whatever logs you need, just point me in the right direction.
Also here's what i see from telnet to the smtp port.
helo collinsinfotech.com
250-webserver.collinsinfotech.com
250-PIPELINING
250-SIZE 104857600
250-STARTTLS
250-ENHANCEDSTATUSCODES
250-8BITMIME
250-LIMITS RCPTMAX=1000
250 SMTPUTF8

Don't know it helps.
I guess nevermind. it looks as if my isp is actively blocking port 25
Is there anyway of changing the port , although i wouldn't know how the outside world would know to use a different port   :(

Dear sir.
Thank you very much for your help.
Is It possible to use a public dynamic IP behind Nat? Is it possible to specify a Fqdn in the IPs settings like this:
IPs: mail server.domain.com, <IP_ADDRESS>
NATIPs: <IP_ADDRESS>
What would it happen if after the server starts the pubt IP changes? Would it be necessary to restart the mail sever?


Gas tracks failed

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (307576d 2022-08-29T00:05:05.72678Z)
What command(s) is the bug in?
forge test
Operating System
macOS (Apple Silicon)
Describe the bug
I report issue#2984. After upgrade revertTo works well. But when I use the same config as mentioned in #2984, the gas output are all zero like below:
[PASS] testClaimReward() (gas: 0)
[PASS] testReceive(uint256) (runs: 1024, μ: 0, ~: 0)

After more tests I found that it's not gas tracking failed. It's the whole test not run at all (so it's always 0)


[Flink-3007] Implemented a parallel version of Hidden Markov Models. It uses a par…

Implemented a parallel version of Hidden Markov Models. It uses a parallel version of the Baum-Welch algorithm to train the model and uses the Viterbi algorithm to predict the sequence labels.
The prediction quality of my examples is low. The reason for this is probably the small size of the model prameters

Hi @HilmiYildirim, Thanks for opening pull request. I would like to shepherd this PR. Could you rename this PR to include JIRA issue number? I think "[FLINK-3003] Implement a parallel version of the Hidden Markov Model" would be better.

3003 is wrong, its [FLINK-3007]

@uce Thanks for correction. :-)

I have renamed the pull request and adapted the source code so that all checks have passed.

Thanks for your contribution @HilmiYildirim. I had some comments.
I'm not so sure about the correctness of the implementation. Where did you get the testing data from @HilmiYildirim. Is that from some published source?
The current implementation lacks the integration with the FlinkML's pipelining mechanism. Furthermore, it only works on integers. What if the observations are characters for example.
The PR needs more documentation for better understanding the code and maintaining it.
Could we maybe add a smaller test file. One of the files is 2MB large. This should definitely be smaller.
I think this PR needs a little bit of more effort to get in shape.

Closing since flink-ml is effectively frozen.


v1.3.x - strange perl unit test failure

Description
An unrelated change detected a failure on a perl unit test. The regularly triggered v1.3.x build does not show this error.
Full log can be found here: http://jenkins.mxnet-ci.amazon-ml.com/blue/rest/organizations/jenkins/pipelines/incubator-mxnet/branches/PR-14163/runs/3/nodes/932/steps/1230/log/?start=0
The relevant part:
t/test_gluon_trainer.t ......
1..30
ok 1
ok 2
ok 3
ok 4
ok 5
ok 6
ok 7
ok 8
ok 9
ok 10
ok 11
ok 12
ok 13
ok 14
ok 15
ok 16
ok 17
ok 18
ok 19
ok 20
ok 21
ok 22
ok 23
ok 24
ok 25
Failed 5/30 subtests

@sergeykolychev could you maybe weigh in?

@mxnet-label-bot add [Test]

@perdasilva I'll take a look, thanks!

@perdasilva This is pretty strange for me too, seems like the script only run 25 tests for some reason but did not fail in any other way. Never seen that one, likely related to some change in env. Let's see if the failure persists.

@sergeykolychev thanks for looking into it. Let's keep this issue around to see if anyone else runs into it?

Sure

No activity. I'll close this issue.


Allow special characters for bank holiday names

In Germany there is a bank holiday called "Buß- und Bettag".
Trying to add this Bak Holiday gives the following error: New name of New Bank Holiday should contain only letters and numbers
As e.g. "Buß- und Bettag" is an official bank holiday name both umlauts and hyphens should be allowed for names of bank holidays.

ouch..
Thanks for reporting

Initially there was an idea to store only "clean" data in DB.
But then I changed mind and decided to go with approach: allow users to store any data they want as long as it is rendered in safe way. And handlebar does it nicely. So that should protect from XSS attacks.
Regarding SQL injections, sequalise should take care about it.
So now the app does not restrict users from submitting only particular set of characters.
Hence this issue should be fixed.


Relax :application validator to allow hyphens

Also improve the error message to explain what characters are allowed, and include the offending value.
Addresses #1512.
Note: I could not find any existing test for the :application validator, and I wasn't sure how to add it. However since this is a small regex and string change, perhaps it can be merged as-is.

:+1:


DNN-32726: avoid null reference exception when call to Globals.LinkClick method.

Fix #2967

Any chance we could get this retarged or submitted for 9.4.1?

Closing this one in favor of #2988
Thanks @zyhfish


Add a Table to Feature the Contributors.

Add a table in README to feature all the contributors

@all-contributos add @aromalanil for code and doc

@all-contributors add @aromalanil for code and doc