Fix User is not navigated to the concierge chat after login on Web

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/20885
PROPOSAL: https://github.com/Expensify/App/issues/20885#issuecomment-1595032312
Tests

Test case 1: Navigate to the concierge chat after login

Log out of Expensify
Go to https://staging.new.expensify.com/concierge
Verify that you are taken to the login screen
Log in to any Expensify account
Verify you're navigated to the concierge chat after login

Test case 2: Navigate to the normal report after login

Go to any chat report => Copy the report URL
Go to any other report (to update latest visit report)
Log out of Expensify
Go the report URL that we copied in step 1
Verify that you are taken to the login screen
Log in to any Expensify account
Verify you're navigated to the chat report that we copied in step 1

[x] Verify that no errors appear in the JS console

Offline tests

Can't test when network is offline
QA Steps

Same as Tests

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://github.com/Expensify/App/assets/9639873/60921b81-0d2a-47e3-93af-ba6265e2f73f

Mobile Web - Chrome

https://github.com/Expensify/App/assets/9639873/deddf3a5-c3aa-4ab6-b93e-7c5ba3b0aacb

Mobile Web - Safari

https://github.com/Expensify/App/assets/9639873/f425f5a2-0131-4d0c-9b79-f3511f2e895f

Desktop

We can't open deeplink on Desktop, there is a bug report here https://github.com/Expensify/App/issues/19581
https://github.com/Expensify/App/assets/9639873/749df7fb-d672-41c2-a594-98297c140461

iOS

https://github.com/Expensify/App/assets/9639873/0c376aa3-d6fd-4ce1-b5b0-6ba283d7a9b7

Android
https://github.com/Expensify/App/assets/9639873/c78d6b2f-e6be-4f06-9899-2b8d9b3db142

@parasharrajat I dont think that is a bug though, that deeplink will not remove the stack and the up button goes to previous page, its like deeplinking to a thread from chat, you should be an option to go back to the chat you came from

@parasharrajat that one is also not related to this PR is it?

@parasharrajat bump

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/mountiny in version: 1.3.42-0 🚀

platform
result

🤖 android 🤖
cancelled 🔪

🖥 desktop 🖥
cancelled 🔪

🍎 iOS 🍎
cancelled 🔪

🕸 web 🕸
cancelled 🔪

@hoangzinh @parasharrajat @mountiny we are seeing last chat for a sec before concierge is loaded even on incognito. Is it expected? https://github.com/Expensify/App/assets/43995119/8dcc75d9-dc15-4e89-b45f-c97682759b3e

@mvtglobally I believe that should not be a blocker

@vitHoracek No need to log as an issue, right?

🚀 Deployed to production by https://github.com/jasperhuangg in version: 1.3.42-26 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Add support for returning authenticated user

As for now, if oauth2_proxy is used only as authenticator, it doesn't return information about authenticated user. This patch introduces header X-Authenticated-User which is returned from proxy upstream. This way further user-based authorization is possible.

@lsiudut I generally think oauth2_proxy should continue to be agnostic of the actual application that is being proxied/authenticated. Having oauth2_proxy set response headers with authentication seems to break that assumption and change that dynamic. I think responsibility for response headers should rest entirely on the proxied application.
I suggest that a better way to implement this would be to enable --pass-basic-auth and have your application behind oauth2_proxy copy X-Forwarded-Email request into X-Authenticted-Email response header.

I understand you completely and I was trying to make my solution this way. I thought that returning name of authenticated user is similar action as forwarding basic auth to application with difference, that we may perform some actions on http server level (nginx and lua authorization in this case). Unfortunately we have setup which makes inserting another proxy very difficult.
If you're finding this solution too invasive just close this PR.

What would be good way to pass authentication data in use case of nginx auth_request directive?

@pawelgocek can you expand upon your question? What are you trying to accomplish?

To pass information about authenticated user name/email to upstream. Right now it is possible when oauth2_proxy works as proxy, but when it's used as authenticator (for auth_request directive) we don't receive any information except authentication status (HTTP 202 or 401).
In our use case nginx is proxying directly to microservices and there's no easy way to inject oauth2_proxy between them. Also we can't use it before nginx, as not all endpoints are suppose to be covered by authenticator. auth_request directive is perfect for us, but we lack info about authenticated user name. We could try to make some hacky solution and proxy authenticated request back to nginx, but we'd like to avoid it.

Actually, this should be possible with the auth_request_set Nginx directive, something like:
auth_request_set $auth_user $upstream_http_x_authenticated_email;
proxy_set_header x-authenticated-email $auth_user;

See also: http://stackoverflow.com/questions/19366215/setting-headers-with-nginx-auth-request-proxy

Yes, but only if oauth2_proxy return X-Authenticated-Email in response :). For now it's not returning such header. My patch adds exactly this functionality.

@lsiudut @mbland ahh. thanks this context is helpful. When using nginx auth is a good argument for setting return parameters so that nginx can pass them upstream. Let me think this a little more.

+1

This looks super-helpful for auth_request users, is there anything else that needs doing to get it landed? :)

This still looks super-helpful for auth_request users (&& required for some grafana users).
Is there anything else that needs doing to get it landed?

I noticed there were merge conflicts.
After resolving those, I noticed travis was busted.
Travis is now fixed in #299.
Travis build and tests now pass.
New pull request is #298

We are currently using the auth_request directive as well which saves us a lot of nginx instances and configuration details for oauth2_proxy. So is there anything still blocking #298 or can we expect it to be merged soon? Otherwise we will have to bite the bullet and either switch to the branch containing the change or start using oauth2_proxy as an actual proxy.

We are also using the auth_request directive and would like to use #298 if it were merged.


Map is not visible after v-show

Hi.
Map works fine but when I'm using it in a multi-tab, map tiles doesn't render normally.
To Reproduce
Steps to reproduce the behavior:

put map in a container
add v-show to container
set initial value of v-show to false
change v-show to true
map doesn't render

Screenshots

Platform

OS: macos
Browser: chrome
Version: 103.0.5060.134

Link to reproduce bug
https://codesandbox.io/s/v-show-warning-forked-ritzum

This seems to be a problem with the reactivity in your app and not with vu3-openlayers. Please make sure the boolean you are using for v-show is a ref().
I provided a playground where you can see, the v-show works:
https://stackblitz.com/edit/vitejs-vite-9wrqog?file=src/components/TheMap.vue


Proof-of-concept: snap routes to roads

Route-based schemes will usually follow existing roads. To make it as smooth as possible for people to sketch, this PR introduces a tool to snap to roads and intersections.
https://user-images.githubusercontent.com/1664407/197337513-a3b6c631-cb0c-4658-9aec-2f466fe4fde7.mp4
How it works
This is a "port" of https://github.com/a-b-street/abstreet/blob/master/apps/game/src/common/route_sketcher.rs from bike.abstreet.org. It reads in a serialized A/B Street map, which does the pathfinding between waypoints and generates the road center-line geometry. The business logic of snapping and responding to input events happens in Rust. A small JS shim makes it work alongside https://github.com/mapbox/mapbox-gl-draw.
Limits/bugs
This is hardcoded to work for a small area in Bristol right now, by shoving 30MB of a map file into the WASM binary! I wanted to at least HTTP fetch a map for this first step, but I'm hitting bizarre serde issues going from JS to WASM right now. Longer term, we need to figure out how to make this work anywhere. A simple, but slow, short-term option is to dynamically fetch OSM data from Overpass, feed into osm2streets, and port the simple pathfinding logic to work off of that instead. https://github.com/a-b-street/abstreet/issues/1009 has more relevant ideas for scaling nation-wide.
You can only draw one snapped route right now. You can click on the result and edit the form as usual. If you then refresh the page, the snapped route acts like a normally drawn one (managed by mapbox-gl-draw) and you can't drag the points around anymore. And if you start a new snapped route, it overwrites the previous route, because they use a fixed ID. I need to figure out how to make the tool work seamlessly with multiple routes at a time. It's tricky, because there's extra state about the waypoints. Do we save that in the geojson as extra info so it survives page reloads, or do we snap just the linestring back to intersections later? (Probably the second!)
Once we solve the above problems and make it scale everywhere easily, this would be a great standalone maplibre plugin to show off osm2streets. CC @BudgieInWA
Next steps
I'll merge this PR, check the github actions works correctly, and post the demo link here. (And then keep iterating on the above problems)

Live at https://acteng.github.io/atip/routes.html?authority=West of England Combined Authority#14.29/51.44965/-2.57773


Correctly dim bot label in dim-bots

Fix: #3520
@fregante I went a bit fast by approving the change you propose.
This will work a lot better, since there is not / in the URL.

Always provide a link where the PR can be tested

Here is a link with a repository with the bot label:
https://github.com/wilau2/harvest-balance-calculator/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc
Here is the inspector open on the label:

I think then it should be a more clear label%3Abot

@fregante updated and tested.

Thanks! It’s a selector by the way, not a regex 👍


TODO - make the location of the keypair configurable.

    Future TODO - make the location of the keypair configurable.

Originally posted by @adambkaplan in https://github.com/tektoncd/results/pull/253#discussion_r994883235

@adambkaplan can you close this issue?

/close
This was completed in #283


Building for Windows on Arm wiki page is outdated

Hi,
I wanted to update the wiki page How to build OpenBLAS for Windows on ARM64 because the instructions are deprecated but the edit button is gone. What is the proper way to get those instructions updated? There is a new flang version and you can now build LAPACK along with OpenBLAS on Windows on Arm and I wanted to mention that.

@everton1984 Not sure how to proceed here without your input - we do not have a Windows-on-Arm system available for testing, and at least the Linaro tracker for "Arm LLVM Toolchain Enhancement" lists an open issue LLVM-760 as holding back OpenBLAS compilation with "flang-new". (This ticket has not been updated since last summer, but the download page at llvm.org has only sketchy flang release notes from 15.0.0 even with the recent release of LLVM16)

@martin-frbg Yeah I thought I had something working but when I got the unit tests to compile one of them are failing so I had to hold on before sending the updated text. Sorry about this delay. I can close this and reopen when I get it solved if you want.

I'd think it depends on how long you expect the delay to be - and if it is something we can help with (without having accesss to a Windows-on-Arm system). I notice the f_check script currently does not know flang-new as an executable name, which may be a sufficient drawback in some circumstances.

I've been fighting against two issues, the first one is that the CMake command add_executable produces a incorrect command line (I'm guessing this is a cmake issue but couldn't pinpoint yet). When I use a add_custom_target with the proper compilation commands, all level two and three unit tests like sblas2, dblas2, etc are hanging (or at least taking and absurd ammount of time). I believe I could get you access to WoA machine if you are interested.

Eeek... not too keen to debug the lovely CMAKE system... is this something in OpenBLAS' own CMakelists.txt ? (Assumed to work on both Arm64 and Windows, just not tested on WoA...). Things hanging could mean problems with multithreading, unless the hangs also happen in single-threaded mode.

hahaha same feelings here, I guess this is a weird combination of cmake + WoA + flang-new, still doing some tests to properly find the culprit. How can I run the tests in single-threaded mode?

Btw, is your flang executable still called flang-new ? While the CMAKE build files do no rely on the f_check I mentioned above, they use whatever CMAKE thinks CMAKE_Fortran_COMPILER is, and cmake/fc.cmake adds some compiler options (only) if the capitalized name is STREQUAL "FLANG". (Not sure if this plays a role for your problem, would mostly affect handling of complex numbers)

Yeah, they are still releasing it as flang-new for some reason I think it is going to be fused with normal flang just later this year. I noticed that as well but it does not seem to make a difference for this particular problem. Thanks for the tips. I will let you know if I make any progress, if I find this to be too complex I will close this issue and re-open it once everything is working and the wiki can be updated.
Btw, I noticed that on the CI for Windows the BUILD_TESTING flag is disabled, is there any reason? Is it something we can help with?

Btw, I noticed that on the CI for Windows the BUILD_TESTING flag is disabled, is there any reason? Is it something we can help with?

IIRC that was just to keep jobs from exceeding their time limit. Maybe I'll try to activate it on Cirrus (once I get the windows_container build working there)

Generally curious about the state of flang(-new) in LLVM16 - is it considered stable on all major LLVM platforms ? I just noticed that at least the "Chocolatey" package for Windows x86_64 (as suggested for Cirrus CI jobs) does not appear to include it. (And as mentioned above, the LLVM download page has only a draft 15.0.0 release note for flang under the 16.0.0 heading, with an empty section entitled "Windows support")

Generally curious about the state of flang(-new) in LLVM16 - is it considered stable on all major LLVM platforms ? I just noticed that at least the "Chocolatey" package for Windows x86_64 (as suggested for Cirrus CI jobs) does not appear to include it. (And as mentioned above, the LLVM download page has only a draft 15.0.0 release note for flang under the 16.0.0 heading, with an empty section entitled "Windows support")

Afaik  flang-new is not being distributed on x86 by default, only for Windows on Arm, at least for now but I believe the plan is to make flang-new the 'main' flang. It's hard to track flang state, it is always a moving target.

@martin-frbg Even with threads disabled *blasN (N >= 2) is hanging. Can I create an Issue for this or at least have a way to contact somebody if I need a bit of help?

Well, you certainly can - but it looks like you are the flang-on-WoA expert here, there's very scant documentation for this compiler and at least I would not know better than you how to debug or trace a program on that system. Perhaps make sure that a pure C build (NOFORTRAN=1 which pulls in f2c-converted LAPACK sources) works first, if you haven't done that already.  Not sure if @mmuetzel can help with his recent experience with flang-new/LLVMFlang on x86_64 Windows (PR #4016) ?

@martin-frbg Trying a pure C build fails for me with a bunch of errors similar to
C:\*\OpenBLAS\lapack-netlib\SRC\slalsa.c(963,25): error: incompatible pointer to integer conversion passing 'integer *' (aka 'int *') to parameter of type 'integer' (aka 'int'); remove & [-Wint-conversion]
            lf = pow_ii(&c__2, &i__1);

is this known or something related to WoA? I am using tag v0.3.21 btw.

Please update to 0.3.23 (or the current state develop branch) so that you do not have to rediscover all the bugs that were fixed since last summer. If you really really absolutely need to use the old version, apply PR #3765 to it.

MSYS2 applies a couple of patches for their OpenBLAS binary package:
https://github.com/msys2/MINGW-packages/tree/master/mingw-w64-openblas
In particular this one might be relevant for you:
https://github.com/msys2/MINGW-packages/blob/master/mingw-w64-openblas/004-aarch64-detection.patch
(Not sure why it hasn't been upstreamed to here.)

Interesting - I'd hazard a guess that it is not built with flang-whatever though ?

You're guessing correctly. It's built using your f2c version of LAPACK with LLVM Clang.

Yeah that's part of the point, I am trying to make native OpenBLAS compile with flang-new with LAPACK everything + testing. I can make it compile with some cmake options tweaks, it's just the tests that are giving me headache.

Fwiw, it's building for me locally with MSYS2's version of Flang 16 on Windows x86 64-bit.
And it passes all tests here.
The flags I used to configure:
cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=ON -DBUILD_STATIC_LIBS=ON -DDYNAMIC_ARCH=ON -DUSE_THREAD=ON -DNUM_THREADS=64 -DTARGET=CORE2 -DC_LAPACK=OFF ..

Additionally, I modified fc.cmake like in #4016:
https://github.com/xianyi/OpenBLAS/pull/4016/commits/2c2306b47dca0eceae9bbd854a59cf9a3af5c482

Although with LAPACK disabled its works fine here as well.

Not sure why you point that out. But please notice that it does build LAPACK with LLVM Flang with the above configuration. I only added -DC_LAPACK=OFF to make absolutely sure it doesn't pick the f2c version of LAPACK.

Although with LAPACK disabled its works fine here as well.

Not sure why you point that out. But please notice that it does build LAPACK with LLVM Flang with the above configuration. I only added -DC_LAPACK=OFF to make absolutely sure it doesn't pick the f2c version of LAPACK.

I pointed that out because I can't read :) I thought it was -DC_LAPACK=ON

Looks like this is really a flang-new + cmake bug, gonna close this issue for now.  I will get this sorted and reopen it when it is ready. Thanks for all the comments.

@everton1984 any news on the status of LLVM17 and especially its flang-new on WoA ?

closing as fixed by #4987 (based on a test build in a Parallels VM on Mac M4, while awaiting actual WoA hardware)


Package a new release with current updates/commits

If you could find the time to package a new release, that would be great. The download centos2alma-0.1.0.zip is lots of commits behind current master.
I would like to do more testing in the coming weeks and i'd like to do that with the most recent version of the toolset.
Thanks,
PS: Or supply instructions on how to get from git clone to ./executable script and we can do it ourselves.

New release is ready
About build instruction. I'm researching a way to provide docker container for build to make build as simple as possible.


Update tested Ruby version

GitHub Pages uses Ruby 2.3.1, so we should presumably test against the same version. Hopefully this will fix the borked tests as well.

Well, the tests run now, but they certainly don't pass.
Baby steps around the office.

This is my fault. I broke the tests in #10 and didn't run them locally so I didn't catch it.
Using defaults kind of breaks a lot of the assumptions the tests relied on. Might be necessary to rethink some things.

@DirtyF Would you mind leaving a review?

@DirtyF: Yeah, the tests here are quite different from the other Jekyll tests.
I wonder if we can improve them.

@jekyllbot: merge

It seems I'm not authorized to push to master?

I am a member of @jekyll/plugin-core.
@jekyll/plugin-core has write access to this repository.
I am unable to merge this PR because GitHub says I am not authorized.
…😶

That's really weird. You see a grey merge button?

You see a grey merge button?

It is grayed out. Next to it is the message "You're not authorized to merge this pull request."

Tried to merge from command line and push to master.

Counting objects: 1, done.
Writing objects: 100% (1/1), 257 bytes | 0 bytes/s, done.
Total 1 (delta 0), reused 0 (delta 0)
remote: error: GH006: Protected branch update failed for refs/heads/master.
remote: error: You're not authorized to push to this branch. Visit https://help.github.com/articles/about-> protected-branches/ for more information.
To https://github.com/jekyll/plugins.git
 ! [remote rejected] master -> master (protected branch hook declined)
error: failed to push some refs to 'https://github.com/jekyll/plugins.git'

Oh, protected branches don't mean you don't have push access to the repo, it means there are specific restrictions about that branch, master.

I just added jekyll/plugin-core to that list at the bottom. Can you try again?


OSError: [WinError 193] %1 is not a valid Win32 application

I am getting this error after this command on anaconda
"OSError: [WinError 193] %1 is not a valid Win32 application"
Running Windows 10 64 bit
Running Anaconda
What to do?

Installed 32 Bit anaconda and not getting this error anymore. A different error but this can be closed


Kafka bulk publish

Description
Added Bulk publish implementation for Kafka.
To be merged after #2024
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
related to https://github.com/dapr/dapr/issues/2218
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Will be updating this PR to completely ignore the traceID field as it is not part of cloudEvent spec https://github.com/cloudevents/spec/blob/v1.0/extensions/distributed-tracing.md and also there is a TODO to delete traceid and keep traceparent alone ... https://github.com/dapr/components-contrib/blob/master/pubsub/envelope.go#L39

Will be updating this PR to completely ignore the traceID field as it is not part of cloudEvent spec https://github.com/cloudevents/spec/blob/v1.0/extensions/distributed-tracing.md and also there is a TODO to delete traceid and keep traceparent alone ... https://github.com/dapr/components-contrib/blob/master/pubsub/envelope.go#L39

How will users trace calls then?

I believe it will be through the trace parent header itself. It contains trace id and parent id in it ...
See https://w3c.github.io/trace-context/#traceparent-header

Based on my understanding,
traceParent header consists of the traceID and parentID fields apart from the version and sample details.
The traceID is ID of the whole trace forest and is used to uniquely identify a [distributed trace](https://www.w3.org/TR/trace-context/#dfn-distributed-traces) through a system from doc.
The parentID sometimes known as spanID is the  ID of this request as known by the caller (in some tracing systems, this is known as the span-id, where a span is the execution of a client request).
So we have two options here for BulkPublish:

BulkPublish is supposed to be a single operation so we use the same traceID, parentID combination across all messages in that operation i.e. the same traceParent header.
The other option is to generate new child span for each event inside the BulkPublish operation since each message in itself can be consumed very differently and need not be consumed in the same fashion. This way we will also be able to identify individual events in the trace though I am not sure that is what a tracing solution should be used for. In this case, there will be a slight change in the parentID or spanID for every event but the overall traceID should be the same. Need to verify that scenario though.

I personally thing we need to verify option 2 and go with that. Will try and verify 2 tomorrow morning and post update here.
Meanwhile @yaron2 @artursouza thoughts?

BulkPublish is seems to be a single operation so we use the same traceID, parentID combination across all messages in that operation i.e. the same traceParent header.

It's a single operation as far as the user is concerned, but that is merely a network optimization. In terms of functionality, each message in the bulk is sent and acked individually, so it should have a separate traceID and parentID for each message.

BulkPublish is seems to be a single operation so we use the same traceID, parentID combination across all messages in that operation i.e. the same traceParent header.

It's a single operation as far as the user is concerned, but that is merely a network optimization. In terms of functionality, each message in the bulk is sent and acked individually, so it should have a separate traceID and parentID for each message.

@yaron2 Yes each event will have its own traceParent header ... the impl PR https://github.com/dapr/dapr/pull/5127 does that only.
Just that acc to the trace context doc a traceParent header contains version-traceID-parentID-flags (parentID is actually the operation ID or the spanID)
There is no separate TraceIDField in either CloudEvent or the TraceContext docs ...

@dapr/maintainers-components-contrib Please review PR related to bulk publish implementation ...


[BUG] No promt "Do you want to install the certificate.." when using --install-certificate true

:information_source: Info

Version: 3.7.0
:speech_balloon: Description

flutter pub run msix:publish --publish-folder-path C:\Temp --install-certificate true does not install the certificate, i have seen in your code a promt for it but its not triggered, i have tried it with and without admin rights.
[  +70 ms] executing: [C:\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[ +137 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[   +1 ms] 4f9d92fbbdf072a70a70d2179a9f87392b94104c
[   +1 ms] executing: [C:\flutter/] git tag --points-at 4f9d92fbbdf072a70a70d2179a9f87392b94104c
[ +126 ms] Exit code 0 from: git tag --points-at 4f9d92fbbdf072a70a70d2179a9f87392b94104c
[        ] 3.3.1
[  +62 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref --symbolic @{upstream}
[  +90 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{upstream}
[        ] origin/stable
[        ] executing: [C:\flutter/] git ls-remote --get-url origin
[  +81 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[ +338 ms] Unable to locate an Android SDK.
[   +3 ms] executing: [C:\flutter/] git rev-parse --abbrev-ref HEAD
[  +73 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[  +98 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +5 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +85 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update.
[        ] Artifact Instance of 'GradleWrapper' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterSdk' is not required, skipping update.
[        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'PubDependencies' is not required, skipping update.
[ +112 ms] Using C:\flutter\.pub-cache for the pub cache.
[   +1 ms] executing: C:\flutter\bin\cache\dart-sdk\bin\dart __deprecated_pub run msix:publish --publish-folder-path C:\Temp --install-certificate true -v
[     0.018s] parsing cli arguments
[     0.190s] validating config values
[     0.192s] validating app installer config values
[     0.193s] validate publish version
[     0.200s] You publishing the same version (0.0.1) as last publish
Do you want to increment it to version 0.0.2 ? (y/N) n
[     2.618s] running "flutter build windows"...
[    15.598s] building msix files...
[    15.598s] validating build files
[    15.606s] cleaning temporary files
[    15.619s] create app icons
[    15.938s] copying VC libraries
[    15.964s] generate appx manifest
[    15.974s] generate package resource indexing files
[    16.152s] packing msix files...
[    16.153s] packing
[    18.372s] cleaning temporary files
[    18.399s] publishing...
[    18.399s] copy msix to versions folder
[    18.420s] generate app installer
[    18.426s] generate app installer web site
appinstaller created: C:\Temp/one_app.appinstaller[+20484 ms] "flutter run" took 20.709ms.
[  +83 ms] ensureAnalyticsSent: 64ms
[   +2 ms] Running shutdown hooks
[   +1 ms] Shutdown hooks complete
[   +1 ms] exiting with code 0

### :scroll: Pubspec.yaml

<!--- Please include your pubspec.yaml file here --->

```msix_config:
  display_name: XXX
  publisher_display_name: XXX
  identity_name: XXX
  publisher: CN=XXXXXX-XXX-XX-80D5-072F17E152F6
  capabilities: internetClient
  store: true

Ah my bad, i thought the cli has a higher priority then the misx_conifg in the Pubspec.yaml. I had store: true in my Pubspec.yaml which then causes msix:publish to not self sign the package.


Allow us to easily get a mapping of topics to brokers

A clear and concise description of what the enhancement is.
We need to have a quick way to figure out which topics are served by which brokers. So it will be helpful if if get broker URL and list of its partitioned topics which it serves from topic name
Describe the solution you'd like
Will use partitioned-lookup of the topic and  sort them by broker URL
Describe alternatives you've considered
Partitioned -lookup of a topic helps but in other way that mapping of topic name and its broker URL.
Additional context
Add any other context or screenshots about the feature request here.

Do you mean than you want a result like  Map<brokerUrl, List<topic-partition>> with a given partitioned-topic name?
If so , I think I can help with this by changing the result of  partitoned-lookup command.

It would be nice if maybe the pulsar-admin supported this when we call topic stats or topic stats-internal?


Decrease the number of digits after the decimal point

I noticed that it depends on the system. Linux has 6 digits (e.g. 120.125368 ms), MacOS has 3 digits. In my opinion this is excessive precision. Could you round it down to one digit?

No, this is up how accurate each measurement is. Having microsecond accuracy is not unusual. It has no impact on float64 precision.

I am not a programmer and have little understanding of what you are talking about. I am looking for a replacement for the default ping on Linux for this reason:
"Why does ping run forever by default in Linux?"
https://unix.stackexchange.com/questions/255405/why-does-ping-run-forever-by-default-in-linux
(although I already found a solution with alias ping=ping\ -c4, but still)
I compiled your application, put it in the PATH and edited the source code (ping.go) beforehand to have 4 requests by default and a timeout of 20 seconds.
Clarification: exactly the default. I know there are options like -c, but I'm too lazy to write them.
So, I got almost a standard ping, except the time is too accurate, which is unpleasant to the eye. See the screenshot:

You don't want to change the accuracy of value in the upstream, ok. But tell me how I can fix the source code (ping.go I guess, or golang dependency files?) so that MY local build is the same as the standard ping. Or a little more accurate, but not 6 digits. For example, 1 digit after the decimal point.
I compiled go-ping project binary with go build and just added it to the PATH, to replace the standard ping.
Understand that I am not a programmer, but I know how to compile. However, I do make minimal edits to the source code. I need to fix my build of your project.
I have small requirements for ping (just occasionally check availability of sites, ip addresses), I do not write my projects using go-ping.

wtf


Weird error with _python_runtime.py when running tests

See here: http://stackoverflow.com/questions/27453908/error-when-running-tests-with-nose-gae
Please let me know if I can provide more info.

Resolved via #12 and https://github.com/Trii/NoseGAE/tree/0.4.0


Cell Content

Hi, maybe this is a real issue not something I'll find the answer to in 5 minutes :)
Is there a way to specify the contents of a cell, meaning the html contents? Can we put a component in there so as to display an image, or just a link?
Thanks,
R

Yes, this is possible. I've update the docs to be more clear, but here is an example of a column renderer, which is what you're looking for:
const columns = [
     {
         name: 'Image',
         dataIndex: 'id',
         renderer: Image,
         editable: false
     }, ...
]

export const Image = ({ column, value, row }) => {
    return (
        <img src= { value } />
        );
};
`

Excellent, Thank you, and the docs are better.  I had seen reneder but thought it was just for, or, well, not sure what I thought it was for.  But I know now!
Thanks,
Raif


Close window on click on copy button in generate SQL dialog

It would be nice, if it was an option to configure the behavior of Generate SQL dialog when user clicks on the copy button.
The goal of the feature is to avoid of clicking on copy and after that on close button. To my mind it's better to have one button "Copy & close" and another one  - "Cancel".

This makes sense. And I don't think that need some config option for that. Extra button maybe..

Verified


Short name for TROPOMI Ozone product doesn't work

I'm trying to run the API for TROPOMI ozone as follows:
results = earthaccess.search_data(     short_name='S5P_L2__O3__PR_HiR',     cloud_hosted=True,     bounding_box=(min_lon, min_lat, max_lon, max_lat),     # point = (max_lon, max_lat),     temporal=("2020-06-01", "2020-06-02")     # count=10 )
but it returns 0 granules even though the short name is correct, as you can see here:
https://disc.gsfc.nasa.gov/datasets/S5P_L2__O3__PR_HiR_2/summary?keywords=TROPOMI_MINDS
Why isn't it working?

Is this your original code? You have a missing parenthesis at the end because of the comments via #
What are your bounding box coordinates?
If I do this e.g.:
results = earthaccess.search_data(short_name='S5P_L2__O3__PR_HiR',
cloud_hosted=True,
bounding_box=(-10, 20, 10, 50))
I get 2533 granules.
What is your result?
Ver 0.9.0


make install fails on Apple Silicon M1 darwin-arm64

Description
Encountered an error when running make install from the development setup instructions.
Related Issues
No response
Questions
No.
Demo
No response
Reproduction
Follow the development setup instructions and run make install.
Expected Results
make install should complete without error.
Actual Results
When running make install on Apple M1, I get the following error related to the electron dependency:
Downloading Electron...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     9    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (22) The requested URL returned error: 404
make: *** [/Users/<redacted>/deps/tmp/electron-v6.0.10-darwin-arm64.zip] Error 22

It looks like electron needs to be at least version 11 to support Apple Silicon/M1/darwin-arm64.
I did some local tests with updating the setup to version 12 and make install was able to complete.
Verifying download...
Success.

Extracting Electron...
Running tests...
v12.2.3

Success.

Version
develop branch
Environments
N/A
Browser Version
No response
Node.js / npm Version
No response
Platform
macos M1
Checklist

[X] Read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.

@walterra Thanks for reporting this issue. Will try and push a fix later this evening.

Should be resolved in https://github.com/stdlib-js/stdlib/commit/d33368119f5cd6d037eed7e1d9f24be3368832a9.
If you still encounter issues when installing repository dependencies, feel free to reopen this issue.


[suggest] add a "search select words in code map" to right click menu

and "search select words in all code map of current project"

You can achieve the same experience already: Ctrl+T and start typing the desired word

errr...
vscode cannot recognize dynamically added object properties in javascript ,  so goto/peek definition mostly not working on my project
I have to use codemap to achieve this


Image fit property

Description
Provide fit property for  component.
The fit prop supports the following types:

aspectFill
aspectFit
stretch

Image source
JIRA
MXSCOMP-217
Completion status

[ ] Android
[x] iOS
[x] Lumin

Resolved for Lumin by: https://github.com/magic-script/magic-script-components-lumin/commit/0f2e90776a601d613b9b3cddbe6c79845950bc0e


decrease the height of the overall scroll view

Even if I don't have anything inside the parallaxscrollview, there is still a lot of white space that's scrollable, how can i change the height size of the overall parallaxscrollview?

Any updates on this?

Same problem.

just set marginBottom of parallaxScrollView equal to header height: (minus)
<ParallaxScrollView
style={{marginBottom: -STICKY_HEADER_HEIGHT}}
....

Is there any solution ?

Bump, still seems unsolved.

@jankoritak There's a PR solving this issue and it's working properly. It's not merged yet so you might need to clone it separately.

@bahaa96 Could you, please, link me the PR?
Btw. I ended up resolving it by patching (removing) footerSpacer, which seems to work in my case.

If you use stickyHeaderHeight={100} for example, it will add like a padding top when you scroll so it doesn't scroll all the way to the top. Increasing the number will increase the padding where it stops scrolling.


Fix mute and unmute

They need to actually alter the value of ChannelActiveMute.  Currently they are NOOPs.

Going to pull this out of the ATMsynth and make just 1 function out of it => toggle mute.
void toggleMute(byte ch) {
ChannelActiveMute ^= (1 << ch );
}


[Bug]: Generated IDs are not unique

Preflight Checklist

[x] I could not find a solution in the documentation, the existing issues or discussions
[x] I have joined the ZITADEL chat

Environment
Self-hosted
Version
2.60.2
Database
CockroachDB
Database Version
23.1
Describe the problem caused by this bug
I'm in the process of setting up IaC for our ZITADEL installations, and it seems there is a significant issue with the ID generation process. The problem is severe, as the generated IDs are not unique when multiple resources are created at once, causing consistency issues.

Operating System
No response
Relevant Configuration
No response
Additional Context
No response

We use sonyflake which requires a machine ID. If the machine ID is not unique, you may experience the problems you are facing. The default is to use the machine's private IP.
We experienced similar problems in GCP where there is not PrivateIP available. You might need to tune the following config section to match with what your deployment provider offers:
https://github.com/zitadel/zitadel/blob/77cd430b3a67cd95ad9deec1e5b44a17638def06/cmd/defaults.yaml#L342-L359

We have both of these options enabled. I can check whether they are unique on all three instances, but I'm positive that's the case.
What "PrivateIp" means in this context, though, there could be multiple network interfaces attached, how ZITADEL knows which one is "private"?

We loop over all interfaces and take the first address with a private prefix:
https://github.com/zitadel/zitadel/blob/50d2b26a28656d5919247b07099a249e0958baf4/internal/id/sonyflake.go#L56-L90

Got you, thanks!
Indeed, two out of three instances had identical private IPs (docker containers running on different hosts).


TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string

I'm having this error when executing yarn:app start:
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined at validateString (internal/validators.js:118:11) at Object.join (path.js:1039:7) at noopServiceWorkerMiddleware (/var/www/html/react-workspaces-playground/node_modules/react-dev-utils/noopServiceWorkerMiddleware.js:14:26) at Layer.handle [as handle_request] (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/layer.js:95:5) at trim_prefix (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/index.js:317:13) at /var/www/html/react-workspaces-playground/node_modules/express/lib/router/index.js:284:7 at Function.process_params (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/index.js:335:12) at next (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/index.js:275:10) at launchEditorMiddleware (/var/www/html/react-workspaces-playground/node_modules/react-dev-utils/errorOverlayMiddleware.js:20:7) at Layer.handle [as handle_request] (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/layer.js:95:5) at trim_prefix (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/index.js:317:13) at /var/www/html/react-workspaces-playground/node_modules/express/lib/router/index.js:284:7 at Function.process_params (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/index.js:335:12) at next (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/index.js:275:10) at handleWebpackInternalMiddleware (/var/www/html/react-workspaces-playground/node_modules/react-dev-utils/evalSourceMapMiddleware.js:42:7) at Layer.handle [as handle_request] (/var/www/html/react-workspaces-playground/node_modules/express/lib/router/layer.js:95:5)
Tools and versions:

Windows 10 Enterprise
Yarn: v1.19.0 (cannot use v1.22.0, due to this error).
Lerna: 3.20.2
@react-workspaces/react-scripts: 3.3.0-alpha-08

According to this stackoverflow topic, this error should be resolved by upgrading the react-scripts version to 3.4.0.

version 3.4.0 needs to be merged into this fork

@F1LT3R Hi could you review this: https://github.com/react-workspaces/create-react-app/pull/19 pull request and merge?

@keshikashviligio did you want to merge 3.4.0 into this fork as @zipzapp suggested?


Support singular matrices?

It seems like the code for non-normal matrices requires the matrix to be non-singular.  Am I missing something?  If not, is there any plans to support non-singular matrices in the near future?

I'd have to look a bit deeper into this. I don't remember putting anything in that requires the matrix to be non-singular. Did you try it? Can you provide a minimal example?

I tried a simple example and you're right that it seems like it works for some singular matrices, but the way the NonNormalAuto is written, my matrix was passed to np.linalg.solve_triangular, which raises an error for singular matrices.  Here is a minimal example:
from pseudopy import NonnormalAuto
import numpy as np

mat = np.array([[1, 0, 1],[0, 0, 0], [0, 0, 1]])
pseudo = NonnormalAuto(mat, 1e-5, 1)

So it's not your input matrix that is passed to solve_triangular but a matrix derived from it with the Schur decomposition. From looking at the code, it seems like the problem arises if you have eigenvalues with multiplicity >1. I need more time to look into this so figuring this out could take a while.
In the meantime, does NonnormalMeshgrid work for you?

Apologies for the delay.
For NonnormalMeshgrid I'm running into problems with the plot method.  I get the following messages:
/opt/anaconda3/lib/python3.7/site-packages/pseudopy/nonnormal.py:122: UserWarning: No contour levels were found within the data range.
  colors=pyplot.rcParams['axes.prop_cycle'].by_key()['color']
/opt/anaconda3/lib/python3.7/site-packages/pseudopy/utils.py:56: UserWarning: Attempting to set identical left == right == 0.020408163265306145 results in singular transformations; automatically expanding.
  pyplot.xlim(numpy.min(vertices.real), numpy.max(vertices.real))

I'm hoping to look at your code more closely and figure out if I can plot the information returned by your function myself.

I got NonnormalMeshgrid working, but I had to plot it myself using a method like in my edit.


Carthage error

*** Skipped building IHProgressHUD due to the error:
Dependency "IHProgressHUD" has no shared framework schemes for any of the platforms: iOS

If you believe this to be an error, please file an issue with the maintainers at https://github.com/Swiftify-Corp/IHProgressHUD/issues/new

@KevinCoderX Currently IHProgressHUD is not available through Carthage. You are welcome to submit a PR with the changes.


Port to newest hyper version

API changed a lot. So it is not trivial.

#107 does some preliminary work for this

done


README: update slack channel to CNCF slack

fixes https://github.com/distribution/distribution/issues/3342

oh! one more location to update (contributing.md); fixing now

oh! one more location to update (contributing.md); fixing now


Allow to exclude some controller actions from authentication.

Add option to Guardian.Plug.EnsureAuthenticated to allow controller actions to be excluded from authentication.
Refers to this issue.
From Readme:
It is also possible to pass a list of actions to be excluded from authentication.
This can be useful for signup. Here you could exclude the :new and :create actions
in the UserController (or whatever handles your signup).
defmodule MyApp.MyController do
  use MyApp.Web, :controller

  plug Guardian.Plug.EnsureAuthenticated, handler: MyApp.MyAuthErrorHandler, except: [:new, :create]
end

@hassox / @scrogson care to take a peek at this?  Looks good to me. 👍

@AlejandroHuerta makes a good point in #180 that this could be accomplished with a guard  which is more of the Elixir way of doing it.
plug Guardian.Plug.EnsureAuthenticated, [handler: MyApp.SessionController] when not action in [:create, :new]

What do you think @ospaarmann?  @hassox or @scrogson any thoughts?

Adding a guard clause would definitely be the right way to do this in Phoenix.

I'd just like to mention that if you all decide to implement the :except it would be reasonable to also allow :only. This would be similar to how phoenix does it.

@doomspork I agree with that. It seems to be more flexible and doesn't introduce more complexity into the plug itself.
It would be nice to add an example to the Readme though, so that less experienced users are made aware of this option. I could create a PR for that, if this helps.

@ospaarmann I am going to close this PR but please feel free to open a PR with README updates 😀

hi @doomspork, I don't see any update on this issue. Do I miss something?


Batch specific updates for undo/redo?

So I have a rather large/complicated store for baseball scoring which has a bunch of actions that could happen. I have some that are just incrementing the ball count where i would like undo to just decrement the ball count by one. Then there are other actions where I change like 4 states, trigger a 1s long animation, then change more states which I would like to be batched into one update for the undo.
Is there a way I can set a flag for the start of sth like a transaction? that way zundo knows that all of those changes should be considered one change.

Hi @KrisLau,
Thanks for your question. Right now, zundo doesn't support that natively, but you could define a handleSet function to batch incoming changes. In this function, you would need to have a value in memory to track the batch of incoming changes, and then after a certain period, call handleSet.

@charkour Would I be able to request that as a new feature? I don't have the skills to implement it myself but I was wondering if maybe something like this would work:
In the temporal options, there would be a enableBatching or sth like that that would accept a boolean. Then they would need to add a state like _transaction or _transactionGroup which would accept a string. If the first update they make to the store is:
{
   _transaction: 'deposit',
   count: state.count + 1
}

If the subsequent update has the _transaction value of 'deposit' still zundo would batch that update together with the previous one (replace the latest entry in pastStates with the current one). However if _transaction had a different value, zundo would treat it like a new entry to the pastStates.

@charkour ?

@charkour ?

Hey thanks, sorry @KrisLau, work has been busy.
I really like your idea, and I think if you use a custom handleSet function, you should be able to batch updates. However, if every update includes updates to every field in the state, you can simply add throttle or debounce from a utility library to effectively batch updates. Using one of those methods will "drop" intermediate state changes and only save changes on a specific time interval.

@charkour No worries, I always appreciate you taking the time to reply! I think I managed to figure it out using handleSet! I created a state called batchWithPrevious and if it's true, I skip over calling handleSet

Glad you figured it out! And thanks for the update.
You're also welcome to create a PR to make batching easier or to add an example of it! Thank you :)

Will do! I'll probably add an example for now and then work on adding a PR later because I'm in the middle of preparing for our app's launch right now hahaha

For sure! If the app is public, I would love to see it once it is launched.

Cool, I'll drop a link to it here once it is launched! Date for launch is up in the air right now haha


Update README.md

Added Oxford comma

Coverage remained the same at 79.009% when pulling 6b51ab93197e798dc5a97db48968a5d5faf408d4 on ajmeese7:master into b6a25e511f80a03ff2d13424f6329975e049139d on gernest:master.

Thanks


feat(tutorial): complete step 5 - hursley https://carbon-tutorial-amaciejk.eu-gb.mybluemix.net

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

@amaciejk can you put the URL to your deployed app in the description please?

Added in the description also: https://carbon-tutorial-amaciejk.eu-gb.mybluemix.net

Looks great! I've registered your completion. Be sure to finish off with the 'wrapping up' section to request your badge and t-shirt.


On ready issue when clicking through from another page.

I'm getting an soundManager.createSound(): Unavailable - wait until onready() has fired. error when I click through to a page with a song. The toneden player shows up fine, it just won't play when you click play. Refreshing the page totally fixes it though.

Nevermind, it was because of rails Turbolinks. :facepunch:


Update consumer.rst

looks like a link is missing

Yes, it is missing. But what exactly your PR adds to this page? May be you want to add the page and point the link to it?

It adds nothing. I just wanted to put your attention to this point, it looks like a link has been lost maybe during some migration.

@dlaptin-softheme your comments about documentation migration are welcome at #759, thank you!


User help please

When I try kubectl command parameters with K they are not recognised, what am I doing wrong?
e.g.:
PS> k get namespace
> kubectl get namespace
Error: unknown command "get namespace" for "kubectl"

Just tried this on my machine and it does not happen.

Try replacing kubcetl with kubectl.exe in the ps1-file. Worked for me.


[BC API 2.0] Updated HttpStatusCode of POST Requests

Some POST Requests have an incorrect response HttpStatusCode.
This PR changes the 200 OK to 201 Created.

Learn Build status updates of commit 4cca6cc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dev-itpro/api-reference/v2.0/api/dynamics_salesCreditMemoLine_create.md
:white_check_mark:Succeeded

dev-itpro/api-reference/v2.0/api/dynamics_salesInvoiceLine_create.md
:white_check_mark:Succeeded

dev-itpro/api-reference/v2.0/api/dynamics_salesOrderLine_create.md
:white_check_mark:Succeeded

dev-itpro/api-reference/v2.0/api/dynamics_salesQuoteLine_create.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks for your contribution, much appreciated!


Implementation of test and parsing for protocol relative uris (#300)

Implemented ability to handle protocol relative URIs (#300 )
You'll have to forgive me because I'm just starting out with rust so some of this might not be idiomatic and the test macro framework was/is a bit confusing.
Adding support for protocol relative uris required:

The assumption that something that indicates scheme is always separated by three characters be turfed.

In places that this had an impact I've introduced locals with a name like offset or delimiter width and adjusted some of the magic 3's to use it instead.

Scheme matching on // for scheme2::parse and parse_exact
Changing how as_str works to be able to output uris without a ":"

I ran benches locally and none of them seemed to be impacted by the additional branching

good call; this is useful and should be considered

Thanks. This looks good to me. I would consider adding a variant to Scheme2 to represent this case instead of checking for len == 0. Thoughts?

I think that makes sense. That was actually how I was approaching it initially but I got a bit thrown around by the design. I'll re implement with Relative as a Scheme2


AST: consider extension and extended type are from the same module if valid alternative module names are equal

Framework authors may move a type along with its extensions from
a high-level framework to a low-level one. We should compare
alternative module names for them to check if they should be considered from
the same module to preserve ABI stability.
rdar://65889766

@swift-ci please smoke test

@slavapestov could you take a look at this for 5.3 branch?


button to change view

I want to put a button in the carousel to go to the next or previous view, is this possible?

Hi @ncampos92, Yes you can. You can use any Touchable component, and im the onPress call the "goToPage" method. For calculating the next/previous item, you can use the "onPageChange" to update the index.
you can wrap the component in a view, and then use position absolute to place your buttons wherever you want.
Let me know if that helps.

Hi @machadogj im trying to call the goToPage method but im getting an error. I call "Carousel.goToPage(index)" and get "undefined is not a function". Can you help me with this?

Hi @ncampos92 you actually need to get a hold of the reference first. Something like this in your render method:
render() {
  return (
    <View.....>
      <Carousel ref={ c => this.carousel = c } .... >....</Carousel>
    </View>
  );
}

And then you can do:
handleOnNext() {
  this.carousel.goToPage(3); //whatever you need here.
}

Thanks @machadogj now i got it to work!

Sounds good!


Doc Link Broken

Hi everyone, Adam Honore (adh@) pointed me at this project.
I went to https://github.com/finos/htc-grid, read through the README and then clicked on the doc link at the bottom of the page: https://awslabs.github.io/aws-htc-grid/. I got a 404 error as a result. What's the correct link to the docs site?

Thank you for raising the issue.
We are working on fixing it.

Issue is fixed for #82


Documentation not working

What happened to the documentation? All the links from the readme.md are giving me 404.

Hey @RVXD, we are in the process of rewriting them for the SS4 version. Please use the 3 branch for SS3 docs

Aha, thanks!


Node JS Support

Right now it depends on XMLHttpRequest() and window, but I want to use it for a node project. If you don't already have a compatible version I can convert it for you.

First of all, I haven't been actively maintaining this repository lately, so it will not work with the new bell schedule and will probably fail spectacularly.
And I also need to move this to my new account from which I am replying.
As for using the library from node, it should be simple enough to make the changes yourself (maintaining a separate branch and copying changes from one into the other would be a lot of work).
The following should make it compatible:

Use some repo to provide XMLHttpRequest (e.g. xmlhttprequest).
Replace window.bellSchedule = { with something like module.exports = {.
Use xmldom to provide DOMParser.

Tell me if this works as I have not yet tested it.
Thanks!

Thanks for the reply. When I have the time I'll test it out and let you know if it still works. IIRC the data format is the same as it was last year. Should be simple enough to convert to node with request & cheerio. I'll let you know when I update it.

The data format is the same except I have to implement the new schedule rotation and collaboration replacements when there isn't any special schedule I can rely on. I have the code the bell schedule website uses though, so this should be pretty easy.
Request and cheerio could also work, just a little more effort as you'll have to change more code to be compatible with their apis.
Good luck!


Implemented Multi-Select Filter Feature for Projects and Organizations

#142
Implemented Multi-Select Filter Feature for Projects and Organizations
Screenshots

@andoriyaprashant Could you please review the updated code and let me know if it satisfies all requirements.

And please resolve the conflicts @Saumya-28


Support FireFox (and Chrome?)

FireFox added mpris2 support some time late last year (2020) and I've been using it with playerctl to great success, but the interface does not support the "Position" property, apparently.
ERROR:__main__:Unknown DBusError encountered during tick
Traceback (most recent call last):
  File "/home/fichte/code/discordrp-mpris/discordrp_mpris/__main__.py", line 71, in run
    await self.tick()
  File "/home/fichte/code/discordrp-mpris/discordrp_mpris/__main__.py", line 105, in tick
    await asyncio.gather(
  File "/usr/lib/python3.9/site-packages/ravel.py", line 2951, in get_prop
    reply.expect_return_objects("v")[0][1]
  File "/usr/lib/python3.9/site-packages/dbussy.py", line 4383, in expect_return_objects
    raise DBusError(self.error_name, self.expect_objects("s")[0])
dbussy.DBusError: org.freedesktop.DBus.Error.NotSupported -- /org/mpris/MediaPlayer2.org.mpris.MediaPlayer2.Player Position is not supported

No idea about Chrome since I haven't tested it, but it seems that discord itself offers a player under a 'chromium' suffied bus that defines /org/mpris/MediaPlayer2 but no interfaces (using QDBusViewer). This will need some investigation.

relevant firefox bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1659199
Probably should still add support here for sources that don't support Position

Even using the plasma integration extension doesn't work since it still tries to access Position from the firefox one sometimes


Choice with space not shown

I am working <choices> that takes the following structure:
<CHOICES>
<CHOICE>New</CHOICE>
<CHOICE>Ready for staffing</CHOICE>
<CHOICE>Draft</CHOICE>
<CHOICE>Approved </CHOICE>
<CHOICE>Rejected</CHOICE>
<CHOICE>Completed</CHOICE>
</CHOICES>
When I debug my app, the choice with space (<CHOICE>Ready for staffing</CHOICE>) is not shown.  I have tried double and single quotes with the same result.  Running Visual Studio 2019, against latest version of SharePoint.

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

ID: 83792d77-9975-fcaf-1869-4e21b47f7d83
Version Independent ID: 7977c542-e1ea-9064-e046-1a64ba65aaa3
Content: CHOICE element (List)
Content Source: docs/schema/choice-element-list.md
Product: office-online-server
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

If you want to create choice field in custom list in SharePoint add-in, the field Schema like below.
<Field Name="ChoiceField" ID="{cf2d0af1-2f4d-4284-b630-813159ed08ef}" DisplayName="ChoiceField" Type="Choice">
      <CHOICES>
          <CHOICE>New</CHOICE>
          <CHOICE>Ready for staffing</CHOICE>
          <CHOICE>Draft</CHOICE>
          <CHOICE>Approved </CHOICE>
          <CHOICE>Rejected</CHOICE>
          <CHOICE>Completed</CHOICE>
      </CHOICES>
</Field>

You can create a new choice field and add choices with space in the UI to check if it works.


Change org.scalactic to testDependencies

Change the lib dependency for scalactic. Move it from nonShadedDependencies to testDependencies

The code change LGTM.  Would you mind building the JAR, using jar -tf to check to make sure scalactic classes are not included in the main JAR, and confirming that here?

@jkbradley I double checked the assembly JAR file. There is no scalactic class anymore.

Thanks!  Merging with master


Feat: Improve Our Blogs page

Describe the feature
Currently Blogs Page  looks like this

We can improve this page by better styling and UI
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@upsaurav12 Can you share some image or something so that I can get to know what enhancement you would do?

Page that gave i gave you For Blog. Currently the box shadow is not looking good with the theme so be can remove that

hi @SUGAM-ARORA  i have created pr improving the page design, please have a look and let me know if any changes is required.


Cargo Command in Lesson 06

The command you have in the tutorial: cargo new --vcs none gl should probably be cargo new --vcs none --lib gl
The directory tree you show implies that you mean to create a new lib, rather than a binary project.

Thanks, fixed.
On Fri, 3 Aug 2018, 18:33 Jeremiah Goerdt<EMAIL_ADDRESS>wrote:

The command you have in the tutorial: cargo new --vcs none gl should
probably be cargo new --vcs none --lib gl
The directory tree you show implies that you mean to create a new lib,
rather than a binary project.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Nercury/rust-and-opengl-lessons/issues/4, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGexNcsZcImR1vmIYBPqwOSESOV6u47ks5uNG0_gaJpZM4VuMyx
.


No Decode Delegate For This Image Format

Hi, I'm using this library to convert a PDF file to PNG image but when I execute my PHP code I get this error:
"Uncaught exception: NoDecodeDelegateForThisImageFormat `PDF' @ error/constitute.c/ReadImage/501"
Throwed in PathToLibrary\Pdf.php:42
PDF line 42 is: $this->imagick = new Imagick($pdfFile); in the construct of the class.
I've tried with some pdfs, and I get the same error with all of them. They only have text.
Thanks for your attention.

For a reason I can't figure out, it seems the latest imagemagick version bundled for homebrew does not embed the pdf delegate anymore :
convert -version
Version: ImageMagick 7.0.7-11 Q16 x86_64 2017-11-12 http://www.imagemagick.org
Copyright: © 1999-2017 ImageMagick Studio LLC
License: http://www.imagemagick.org/script/license.php
Features: Cipher DPC HDRI Modules
Delegates (built-in): bzlib freetype jng jpeg ltdl lzma png tiff xml zlib

The exception may comes from this lack of delegate

I also met this problem on windows.Imagick cannot delegate ghostscript.But I'm not fixed yet.

I have the same problem, thank you for your help.
I searched a lot but did not find the right answer! I did not install **Imagick **and ghostscript properly, but I do not think I did a small job!
I placed C:\ghostscript\bin in my PATH variable in Windows, but still I have got Uncaught exception 'ImagickException' with message NoDecodeDelegateForThisImageFormat when I call readImageFile !
I can do the conversion using convert test.pdf test.jpg command in Windows CMD, but I get this error with the php code:

Fatal error: Uncaught ImagickException: NoDecodeDelegateForThisImageFormat `PDF' @ error/constitute.c/ReadImage/509 in C:\xampp\htdocs\project\vendor\spatie\pdf-to-image\src\Pdf.php:40 Stack trace: #0 C:\xampp\htdocs\project\vendor\spatie\pdf-to-image\src\Pdf.php(40): Imagick->pingimage('C:/xampp/htd...') #1 C:\xampp\htdocs\project\pdfToImaage.php(7): Spatie\PdfToImage\Pdf->__construct('C:/xamp/htd...') #2 {main} thrown in C:\xampp\htdocs\project\vendor\spatie\pdf-to-image\src\Pdf.php on line 40

I have the same problem, thank you for your help.
I searched a lot but did not find the right answer! I did not install **Imagick **and ghostscript properly, but I do not think I did a small job!
I placed C:\ghostscript\bin in my PATH variable in Windows, but still I have got Uncaught exception 'ImagickException' with message NoDecodeDelegateForThisImageFormat when I call readImageFile !
I can do the conversion using convert test.pdf test.jpg command in Windows CMD, but I get this error with the php code:

Fatal error: Uncaught ImagickException: NoDecodeDelegateForThisImageFormat `PDF' @ error/constitute.c/ReadImage/509 in C:\xampp\htdocs\project\vendor\spatie\pdf-to-image\src\Pdf.php:40 Stack trace: #0 C:\xampp\htdocs\project\vendor\spatie\pdf-to-image\src\Pdf.php(40): Imagick->pingimage('C:/xampp/htd...') #1 C:\xampp\htdocs\project\pdfToImaage.php(7): Spatie\PdfToImage\Pdf->__construct('C:/xamp/htd...') #2 {main} thrown in C:\xampp\htdocs\project\vendor\spatie\pdf-to-image\src\Pdf.php on line 40

Version: ImageMagick 7.0.11-4 Q16 x86_64 2021-03-20 https://imagemagick.org Copyright: (C) 1999-2021 ImageMagick Studio LLC License: https://imagemagick.org/script/license.php Features: Cipher DPC HDRI Modules Delegates (built-in): bzlib djvu fftw fontconfig freetype heic jbig jng jp2 jpeg lcms lqr ltdl lzma openexr pangocairo png tiff webp wmf xml zlib
same issue in version 7.0.11-4 OS Debian AWS

This solution works for me.
https://stackoverflow.com/questions/52861946/imagemagick-not-authorized-to-convert-pdf-to-an-image


Update __phpunit_verify() method signature for PHPUnit 7.4

PHPUnit 7.4 has added a parameter to the public interface which breaks implementation compatibility when missing.
This change should not affect PHPUnit <7.3 when having the parameter in the implementation but not the interface.
Fixes #29

Thanks, @chrisminett!
Merged and released with 2.1.2.


Clarify what it means to be "unsupported" by a supports expression.

@JavierMatosD Hopefully this makes it clearer for the customer you were working with.

What I find confusing is how the message prints the requested platform first, i.e.:
atlmfc[core]:x64-linux is only supported on 'windows'

... this reads like "The linux package is only supported on windows".
Not sure how to improve this concisely, maybe
atlmfc[core] is unsupported on x64-linux but supported on 'windows'.
...

(This area also needs improvements when a dependency is unsupported. ATM it doesn't tell the requested feature which asks for unsupported dependency. This is difficult to resolve.)

What I find confusing is how the message prints the requested platform first, i.e.:

That's fair, I'll take a look at this again

This area also needs improvements when a dependency is unsupported.

I agree.


Who developed Go?

Hi, First, thanks for sharing.
And I want to double-check the sentence of
Go, also known as Golang, is an open-source programming language developed by Google.
As I understand, the Go is developed at Google, by Robert Griesemer, Rob Pike, and Ken Thompson. It's not developed by Google as my understanding, although it's started to use in Google.

@ledongthuc  Hello, thanks for your feedback but according to their website, I quote:

Go is an open source, strongly typed, compiled language written to build concurrent and scalable software. The language was invented at Google by Rob Pike, Ken Thomson, and Robert Griesemer. Go is designed for simple, reliable, and efficient software.

yes, that's what I mean. It was developed at Google by Rob, Ken, and Robert. it was not developed by Google.
The term developed by Google is confusing and easier to understand Google invented it. But actually, Rob, Ken and Robert developed it when they were working at Google.

Ok, now I get your point. I will mark as an issue. Will update soon

https://github.com/pedrobertao/golang-interview-prep/commit/ff6379608abab683b9e5e45d9b7683bdccf42219


Docs: Getting started Section

what's wrong in the documentation?
a. Prerequisites:  The prerequisites that users need to have before they can get started with the project. This may include software dependencies, programming languages, libraries, or any other requirements.
b. Commands to run:  The specific commands that users need to run to set up the project, or perform other necessary tasks.
c. Expected behavior: Description of what users should expect to see or achieve when they successfully run the commands and set up the project.
Add screenshots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

hey, @TRIPLE-ADE we already have contributing guide we don't need local setup details in README.md. we need to add the Prerequisites section.
what do you think?

Okay would push that today


[DataLoader2] Saving and restoring initial seed generator

Stack from ghstack:

-> #998

Changes to DataLoader2:

Modifying state_dict to store randomness_state, which includes:

_seed: int
_reset_seed: bool - flag indicating whether _seed needs to be set
_seed_generator - the latest version at the time when state_dict is called
_initial_seed_generator - the versopm that is saved at the beginning of very epoch

Modifying from_state and load_state_dict to restore randomness_state
Adding a method _restore_checkpoint_beginning_of_epoch

This sets self._seed_generator = self._initial_seed_generator, allowing users to re-create an epoch from the beginning.

Considerations
Storing the randomness states provide more flexibility for users to restore as they see fit. The decision to do that should not be controversial.
I decided to make add a new method for checkpointing at the beginning of the epoch, ensure that users are not confused about what randomness is restored by default.
The basic idea is that we want to allow users to restore dl2._seed_generator to the previously saved version. From that point on, they can create a new __iter__ and continue from the beginning of the epoch.

Note that since _seed and _reset_seed are also saved, if the users were planning to use a different seed or if there was a need to re-seed, those remain valid after restoring the checkpoint.
Finally, if users change their mind at any point (after restoring) and want to manual set seed. That seed will override any other behavior and the seed will be used.

SeedGenerator is pickable. No need to add those supporting functions.

Yea, that makes sense. I can take those out.

IIUC, initial state should be the state at the beginning of each epoch but not the beginning of all epochs

I'm expecting __iter__ to be called at the beginning of each epoch, such that this line will be called every time.
self._initial_seed_generator = clone(self._seed_generator)
Does that look right to you?

I am not sure if it's needed to differentiate initial seed and current state of seed generator. After SG seeds the graph, the state of SG is not going to evolve during iteration. All random states should be self-preserved by random DataPipes.
-> It means I don't know if we need to add restore_initial_seed_generator to the argument.
During __iter__, like what we offers for reading_service, whenever seed_generator_state is provided, we can just restore it. Otherwise, we should try to seed the seed_generator either by self._seed or None.

@nivekt has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.

I looked into whether we can refactor ._reset_seed: from DataLoader2 into SeedGenerator, I concluded that is not possible, mainly because the current DL2 behavior wants to use the seed once and don't re-seed until users call the seed method again. Whereas SeedGenerator would re-seed every time its seed method is called.

@nivekt has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.

@nivekt has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.

@nivekt has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


Package name

Hi,
I noticed that you're the owner of the npm package workbench which currently seems to be unused. Would you please consider transferring ownership of the package name to me? I was hoping to use it for my benchmarking library as the name workbench is a portmanteau (sorta) for worker_threads + benchmark, which is my library's main feature.
Thanks,
Jonah Snider

Hi @casesandberg, not sure if you saw my previous message, but I'm still interested in using this package name!


Any iteration possibility? or find key of an item?

Any iteration possibility? or find key of an item?

You can iterate using Remaining. Since this supports arbitrary and variable length objects to be stored, a find would have to do a linear scan.
Is this what you were asking? Did you have an implication strategy in mind?

Since I have not heard back in a while, I'll close this issue. But feel free to reopen and/or send a PR.


Add Solaris target

a.k.a SunOS / SmartOS

Solaris builds work via a vagrant setup. Not in CI yet.


A mistake of running "train_cls.py" with CUDA 10.1 and Pytorch 1.3.0 On win10

Thank you so much for your work. I met A mistake of running "train_cls.py" with CUDA 10.1 and Pytorch 1.3.0 On win10. Could you please  help me?
ValueError: cannot find context for 'forkserver'
Error message:
(pointnet) λ python -m pointnet2.train.train_sem_seg
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\envs\pointnet\lib\runpy.py", line 183, in _run_module_as_main
mod_name, mod_spec, code = get_module_details(mod_name, Error)
File "C:\ProgramData\Anaconda3\envs\pointnet\lib\runpy.py", line 109, in get_module_details
import(pkg_name)
File "C:\Data\Desktop\Pointnet2_PyTorch-master\pointnet2_init.py", line 17, in
from pointnet2 import utils
File "C:\Data\Desktop\Pointnet2_PyTorch-master\pointnet2\utils_init.py", line 8, in
from . import pointnet2_utils
File "C:\Data\Desktop\Pointnet2_PyTorch-master\pointnet2\utils\pointnet2_utils.py", line 11, in
import etw_pytorch_utils as pt_utils
File "C:\ProgramData\Anaconda3\envs\pointnet\lib\site-packages\etw_pytorch_utils_init.py", line 11, in
from .persistent_dataloader import DataLoader
File "C:\ProgramData\Anaconda3\envs\pointnet\lib\site-packages\etw_pytorch_utils\persistent_dataloader.py", line 20, in
_mp_ctx = multiprocessing.get_context('forkserver')
File "C:\ProgramData\Anaconda3\envs\pointnet\lib\multiprocessing\context.py", line 238, in get_context
return super().get_context(method)
File "C:\ProgramData\Anaconda3\envs\pointnet\lib\multiprocessing\context.py", line 192, in get_context
raise ValueError('cannot find context for %r' % method) from None
ValueError: cannot find context for 'forkserver'

This should be fixed now.


Remote Windows

Is there a way to set this up on a remote windows VM? Homebridge on raspberry pi controlling windows VM on same network?

@kesp91 Maybe you can use ssh send the cmd command to the remote windows, such as ssh root@remote_ip 'svcl.exe /Stdout /GetPercent', the remote windows install the OpenSSH and configure the svcl.exe and SoundVolumeView.exe in PATH is need. And you need change the method getAllSpeakers, which the original code get the speaker devices from local file(sound-item.json), you need modify the code, let it get from remote file's content via ssh.


Standardising mavlink message strings

I tried to standardise the mavlink message strings a tiny bit:

No ALL CAPS words (even when the situation is really bad. But let's face it: A lot of messages can indicate something really bad)
Begin message with a capital letter
No punctuation character at end of message
Minor English adjustments

I understand that this is a matter of taste to some extent, but let's see :)

Late to the party. This is great!

Thanks @hamishwillee :grin:


Using the satellizer client sample with nodeJs sample and getting popup problem.

I'm taken the latest sources from github and using the client sample together with the nodeJs sample.
I registered correctly a google api.
The request towards google is functioning, the popup is shown, I provide the consent details and then the home page of the sample app is shown inside inside the popup window and it is not closed.
I have exactly the same problem with twitter, so probably the problem is simply somewhere in the client sample.

When I rollback sattelizer.js to version 0.9.4, without changing anything else, the problem is gone and everything works perfectly.

It would be nice to see if you could post console output of both the popup and main window. Something is causing an error which prevents the popup to be closed.

I cant reproduce systematically. Will come back if the problem reoccurs.
Thanks a lot for this great library.


dev and library code syncs

This feature update brings the library and dev UI to AHQ Store on version 0.8.6

Merging current changes to add the classes

updater
Install Worker


1040 header UI changes

Description
The top navigation bar styling is done. See http://localhost:5050/catalog/ top for changes.
Changes reflects zeplin https://app.zeplin.io/project/62b9dba49dd2b211410e608e/screen/62b9dbf684718712f34fe8d0
Fixes #1040
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[x] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[x] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[ ] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

Hi, Minu, I think there are multiple things to be fixed:

"Contribute" font color is different from the design.
"Contributions Catalog" needs the Block-I logo.
Contribute button appears differently on pages other than the home page.
Login button margin is slightly smaller than what's needed to keep it at the center
Login/logout button font weight needs to be 800 to match the ProximaNova-Bold font.
The header height and hello string width need to be updated based on the design.

We need to get the implementation as close as to the design. Please compare the design with the implementation after making these changes. Thanks.

Converting to draft to fix the changes.

Hi, Minu, I think there are multiple things to be fixed:

"Contribute" font color is different from the design.
"Contributions Catalog" needs the Block-I logo.
Contribute button appears differently on pages other than the home page.
Login button margin is slightly smaller than what's needed to keep it at the center
Login/logout button font weight needs to be 800 to match the ProximaNova-Bold font.
The header height and hello string width need to be updated based on the design.

We need to get the implementation as close as to the design. Please compare the design with the implementation after making these changes. Thanks.

updated changes. Compared to zeplin webpage.

Thanks.
From the Events Manager design examples, there is a lot more space between the name and the logout button in the top navigation bar. I think it can be reduced. The text "Contributions Catalog" does not vertically align to the center. Otherwise, the current header looks fine to me. Thanks.
The top navigation bar's height might need to be increased to match the design, but that'll also be related to the logo conversation. So, it can probably stay as it is for now.

Converting to draft to put the Block-I image

updated styling.
ready for review. @sandeep-ps @bingzhang

The font styles of the Contributions Catalog and log-in (potentially logout text, too) are different on the home page and when the user views one of the Contributions.
Home page:

View Contribution page:

Once the user logs in, the nav bar alignment changes. Please see:

The top nav bar needs to use the available width and position content accordingly. Thanks.

converting to draft to fix header

I wasnt able to exactly reproduce the styling errors. But I updated some header stylings (flex divs) and pushed the latest. Let me know if the issue persists.

Need to add orange hover color for login/logout button.
Need to fix the spacing between different divs in the topnav bar.
Converting to draft.

Updated header styling.
The login/logout button does not have orange on hover. Let me know if that is needed. Also if I missed anything. @sandeep-ps
Ready for review.

Hi, Minu,
The style of the text "Contributions Catalog" in the nav bar still looks slightly different on the View Contribution and View Capability pages when compared with the Home page. The rest looks good to me. The login/logout button hover color change could wait for now. Thanks.

Hi, Minu,
The style of the text "Contributions Catalog" in the nav bar still looks slightly different on the View Contribution and View Capability pages when compared with the Home page. The rest looks good to me. The login/logout button hover color change could wait for now. Thanks.

Hi @sandeep-ps
I had to add the typekit.css style to every page. This made the "Contributions Catalog" title font same for all pages. Somehow to able to replicate the styling without adding the css style for each page.

Thanks, Minu. I think we can look at this again when doing the refactoring, but this looks good now.

Hi, @bingzhang, I'm going to ask Minu to merge this PR as it's been open for a while now. Please review the other PRs that are open in this repo. Thanks.
Hi, @minump, you can proceed with merging this. Thanks.