Add Support for suggest Description and suggest Tags Permission

Is your feature request related to a problem? Please describe.
Currently, we have 4 permissions

Update Description/Tags
Update Lineage/Owner
As a user, if I want to only suggest a description or tag, I will not able to do that.

Describe the solution you'd like
Add Support for suggest description and suggest Tags Permission

@Sachin-chaurasiya  Is this related / blocking task for #3046

@Sachin-chaurasiya Is this related / blocking task for #3046?

Hey @mithmatt, not a blocker. but it would be good if we allow users to suggest/request descriptions based on permissions.

Thanks for the information
I need to check back with @vivekratnavel  on how the Request Description logic is implemented in the backend, so that we can surround it with the correct permissions.

Makes sense. So it looks like we just need to order our implementation plan in this way:

Thread based resolution (what we have now)
Task based resolution (may be similar to the discussion we had before on #2881)
Permission on who can open task (AuthZ for Suggest Description)

@vivekratnavel  Should 3 happen before 2? Or the above order is preferred?

For this workflow, we might not need 3. Here are my thoughts on this. Anyone should be able to suggest or request a description or tag just like anyone can open an issue or PR in Github. But, only the owner or users with permission to edit the description/tag should be allowed to accept the suggested description/tag.

For this workflow, we might not need 3. Here are my thoughts on this. Anyone should be able to suggest or request a description or tag just like anyone can open an issue or PR in Github. But, only the owner or users with permission to edit the description/tag should be allowed to accept the suggested description/tag.

So far our strategy has been implementing AuthZ as much as possible to all APIs that make changes in our system.
And support the case where anyone can do X by adding all users to a default global role (in our case DataConsumer) and provide it permission to do X.
This way we have access control in place technically, and the default use case works as is.
But at the same time, admins of OpenMetadata can leverage the fine-grained access control to limit permissions  if they choose to do so.
In other words, implement access control on task creation but ship with permissions to allow all to do so.
@vivekratnavel  What do you think about this strategy?

Sounds like a plan. We can allow data consumers to create conversations and reply in a conversation by default. I also wanted to discuss creating policies/rules that can allow certain operations dynamically. For example, I want to allow users to delete their own posts but not others' posts. An admin can delete anyone's posts. I would like to know your thoughts on this.

I want to allow users to delete their own posts but not others' posts. An admin can delete anyone's posts.

Admin being able to delete anyone's posts can be easily achieved today, with existing implementation.
The dynamic part - restrict users to do operation on their own entities (whether it is deleting their own posts, etc) - this is something we don't have yet. We have similar request, such as this one: owner of team should be able to manage that team  https://github.com/open-metadata/OpenMetadata/issues/2950 - we'll build this out soon - in a way we can reuse this here hopefully (haven't looked into all details yet, but I think it is possible.)

@Sachin-chaurasiya @vivekratnavel anything needs to be done here with new permissions


Sujoncham added a quote

Sujoncham added a quote with author's name and submitter

@sujoncham Awesome quote Sujocham. I am adding your quote. Keep it up!

@sujoncham You've been invited to the Hex Code Organization.


set http errorcode if an error occured; differend redis cachetimes for errorpages

Changed the http-status for RKIErrors to 503 and axios errors to 503 too. All other errors are set to 400.
Changed the redis caching time for errormessages too. Only succsessfull requests (http status 200) are cached for 1800 sec now.
Fix #321
I removed the parameter checks!

@marlon360 i have removed the parameter checks! Only the passing through of the RKI error to the http error remained, as well as different cache times for different http statuses. Successful requests (http status 200) are cached for 1800 seconds as before. All other http statuses are only cached for 180 seconds! please review!


Yabai occasionally fails to detect windows

In the last 2 to 3 months, I've started noticing that occasionally Yabai fails to recognize, and therefore manage, different apps. Closing and reopening the app always fixes the issue, but it is annoying. Specifically, I've noticed it with System Settings, Alacritty and IINA. I've been able to recently catch it when a terminal (Alacritty) was not recognized, so I could query Yabai without altering the window arrangement.
Setup
macOS 13.1.1, yabai 5.0.3, two native full-screen spaces, one normal with sketchybar at the top.

Alacritty is clearly not managed by Yabai, as I can't float or unfloat the window. The result of running yabai -m query --windows is below:
[{
	"id":362,
	"pid":10567,
	"app":"Code",
	"title":"battery.sh — Untitled (Workspace)",
	"frame":{
		"x":0.0000,
		"y":0.0000,
		"w":1440.0000,
		"h":900.0000
	},
	"role":"AXWindow",
	"subrole":"AXStandardWindow",
	"display":1,
	"space":2,
	"level":0,
	"opacity":1.0000,
	"split-type":"none",
	"split-child":"none",
	"stack-index":0,
	"can-move":false,
	"can-resize":true,
	"has-focus":true,
	"has-shadow":true,
	"has-border":true,
	"has-parent-zoom":false,
	"has-fullscreen-zoom":false,
	"is-native-fullscreen":true,
	"is-visible":false,
	"is-minimized":false,
	"is-hidden":false,
	"is-floating":true,
	"is-sticky":false,
	"is-topmost":false,
	"is-grabbed":false
},{
	"id":116,
	"pid":902,
	"app":"Arc",
	"title":"Applescript for specific System Preferences pane - Ask Different",
	"frame":{
		"x":0.0000,
		"y":0.0000,
		"w":1440.0000,
		"h":900.0000
	},
	"role":"AXWindow",
	"subrole":"AXStandardWindow",
	"display":1,
	"space":3,
	"level":0,
	"opacity":1.0000,
	"split-type":"none",
	"split-child":"none",
	"stack-index":0,
	"can-move":false,
	"can-resize":true,
	"has-focus":false,
	"has-shadow":true,
	"has-border":true,
	"has-parent-zoom":false,
	"has-fullscreen-zoom":false,
	"is-native-fullscreen":true,
	"is-visible":false,
	"is-minimized":false,
	"is-hidden":false,
	"is-floating":false,
	"is-sticky":false,
	"is-topmost":false,
	"is-grabbed":false
}]

It is evident that Yabai doesn't see the window. When querying the space in which the terminal is situated, I get an empty array.
> yabai -m query --windows --space 1
[]
> yabai -m query --windows --window
could not retrieve window details.

@mikemykhaylov Faced the same issue with Alacritty before. While other applications were working fine. I had to switch to other terminal emulator like kitty or wezterm, that fixed my workflow, but still unaware of the actual cause.

I see this in Alacritty as well and mainly only when I'm spawning it as the first window in an empty space.

Facing the same issue on mac apple silicon.
yabai-v5.0.6

Used to work fine on intel mac yabai-v4.0.1

This solved this issue for me: https://github.com/koekeishiya/yabai/issues/1250#issuecomment-1157252933

Maybe duplicate of #1833


How to get rid of "Element with id ... is already registered" error and get connections listed?

Describe the bug
I described it here:
https://stackoverflow.com/questions/74211903/error-element-with-id-already-registered-and-no-sql-connections-in-the-list
Expected behavior
I expect no pop-up with the error and MySQL connections to appear in the list.
Desktop (please complete the following information):

SQLTools Version 0.25.1
VS Code Version: 1.72.2
OS: Windows 10
Driver:

[ ] PostgreSQL/Redshift
[x] MySQL/MariaDB
[ ] MSSQL/Azure
[ ] SQLite
[ ] Other? Which...

Database version: MySQL Workbench 8.0

The suggestion in https://github.com/mtxr/vscode-sqltools/issues/668#issuecomment-993403972 solved the issue.

Starting with today's 0.27.1 release it should no longer be possible to define connections with duplicate IDs using the UI.


Added instructions for setup to README

Just a readme update to make it easier to set up

Thanks!


Python intelisense fails after latest insiders update

VSCode Version: 1.29.0-insider (Commit: 1358e903716cf77711493770d63dde1ea178a73e)
OS Version: Windows 10 1803

Steps to Reproduce:

Make sure you're running the latest VSCode insiders update with the Microsoft Python extension
Open a Python file and look at the developers terminal to see error

Does this issue occur when all extensions are disabled?: No, as the Python extension isn't loaded ever.
I believe something in the latest VSCode update broke the extension to the point it won't work outright.
Here's the VSCode's console log:
console.ts:134 [Extension Host] Activating extension 'ms-python.python' failed: Reflect.hasOwnMetadata is not a function. (at Object.showMessage (c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\node\extensionHostProcess.js:623:628))
t.log @ console.ts:134
extensionService.ts:994 Activating extension 'ms-python.python' failed: Reflect.hasOwnMetadata is not a function.
E._logMessageInConsole @ extensionService.ts:994
console.ts:134 [Extension Host] Activating extension `ms-python.python` failed:  Reflect.hasOwnMetadata is not a function (at c:\Program Files\Microsoft VS Code Insiders\resources\app\out\vs\workbench\node\extensionHostProcess.js:295:9)
t.log @ console.ts:134

[Extension Host] Here is the error stack:  TypeError: Reflect.hasOwnMetadata is not a function
    at _tagParameterOrProperty (C:\Users\User\.vscode-insiders\extensions\ms-python.python-2018.8.0\node_modules\inversify\lib\annotation\decorator_utils.js:22:17)
    at Object.tagParameter (C:\Users\User\.vscode-insiders\extensions\ms-python.python-2018.8.0\node_modules\inversify\lib\annotation\decorator_utils.js:7:5)
    at C:\Users\User\.vscode-insiders\extensions\ms-python.python-2018.8.0\node_modules\inversify\lib\annotation\inject.js:24:31
    at C:\Users\User\.vscode-insiders\extensions\ms-python.python-2018.8.0\out\client\languageServices\languageServerSurveyBanner.js:11:37
    ---SHORTENED---```

Oh, seems its been breaking more extensions they just Python. Will close then :)


Prevent from tapjacking flutter

Use case
anyone have any idea about how to prevent tapjacking in flutter,
val view = findViewById(android.R.id.content).rootView
view.filterTouchesWhenObscured = true
someone suggested this but nothing works
Proposal
anyone have any idea about how to prevent tapjacking in flutter,
val view = findViewById(android.R.id.content).rootView
view.filterTouchesWhenObscured = true
someone suggested this but nothing works

@Avijaygovinda
Your question is better suited for support channels like StackOverflow, so please ask the same there.
Closing from here, as this seems to be more of a support question rather than a bug report or feature request.


MinGW build fails in Appveyor

This MinGW build failed in Appveyor but I don't know why. Everything works fine until the package creation. Then it complains about virtual environment. I have never seen this problem until now. I guess it's related to to the fact conan-package-tools uses mingw_installer (in a virtual environment) but I don't know what to do to fix it.
Can you help me?

Hi, When I generate the contents for the activate.bat I look if the _CONAN_VENV env variable is defined. If it's defined I raise the error that you are seeing: "Deactivate the current virtual environment..." It's not ideal but we wanted to prevent the environment getting dirty with multiple executions of the virtual environment.
The problem is, I don't know why is keeping the variable _CONAN_VENV. It is set when you run the activate.bat, but I assume that a "self.run("activate.bat  && " invocation should not alter the system environment (just the current process one), but it seems that in some point is keeping it. Maybe keeping the python process one? We need to test it and see what happen.
If you want to help debugging maybe you can try crazy things like:
self.run("activate && %s %s && SET _CONAN_VENV=" % (os.sep.join([".", "bin", "helloworld"]), "conan"))

Hi, I think the problem is conan_package_tools is activating a virtualenv for compile with the custom MinGW and then it complains for your test method. I have to think the better fix to this situation, but now I understand it.

Hi. I've opened a PR in conan repository. We are going to remove those checks, I think they are not necessary anymore before some improvements in last releases. Will be fixed in release 0.16


Deep Link cannot be intercepted in method onShouldStartLoadWithRequest

Bug description:
Deep Link cannot be intercepted in method onShouldStartLoadWithRequest. Although it returned false, the application was called up.
<WebView
    ...
    onShouldStartLoadWithRequest = {this._onShouldStartLoadWithRequest.bind(this)}
    />

    _onShouldStartLoadWithRequest = (event) => {
        let url =event.url;
        if(url.indexOf("alipay://")>-1 || url.indexOf("alipays://")>-1) {
            Log.info("-----------------------------------_onShouldStartLoadWithRequest return false url = " + url);
            return false;
        }
        return true;
    }

To Reproduce:
Expected behavior:
I need to intercept deep link and deal with them myself.
Screenshots/Videos:
Environment:

OS: Android
OS version: Android 8.0 Oreo
react-native version: 0.59.8
react-native-webview version: 5.12.0

I might have the same problem.
onShouldStartLoadWithRequest and navigationStateChangedHandler are not called when click on links

@DonaldGuo @jannikbuschke have you two found any solutions for this issue? I'm trying to intercept a link such as plaidlink://hostname?params=value but none of the callbacks I try are called...

@DonaldGuo @jannikbuschke have you two found any solutions for this issue? I'm trying to intercept a link such as plaidlink://hostname?params=value but none of the callbacks I try are called...

I copy module file into project and change the code  on the Android platform. (I don't know how to change for iOS)
Judge url in com.reactnativecommunity.webview.RNCWebViewManager.RNCWebViewClient.shouldOverrideUrlLoading(WebView, String) function
And return true for target url

@DonaldGuo @jannikbuschke have you two found any solutions for this issue? I'm trying to intercept a link such as plaidlink://hostname?params=value but none of the callbacks I try are called...

@baugarten
I had the same problem and I solved it with this
	<WebView
		source={{ uri }}
                 onShouldStartLoadWithRequest={request => {
			return request.url.startsWith(request.url);
		}}
                 onNavigationStateChange={event => {
			if (event.url.indexOf("yourdeeplionk") > -1) {
		}
	/>

try to follow this
https://medium.com/@anibalajt/react-native-deep-linking-ios-android-4f24d762d0a8

Same problem here, anyone found a solution?

someone mentioned in other ticket that you should whitelist your scheme first, and then it will work
originWhitelist={['*']}

@Overtorment tried with your suggestion. It works, Thanks

@Overtorment tried with your suggestion. It works, Thanks

someone mentioned in other ticket that you should whitelist your scheme first, and then it will work
originWhitelist={['*']}

It works! Thank you!!!


Problem after upgrading to JQuery 3.5.1

I am not sure how and why this happens, but after upgrading to JQuery 3.51 (from 3.4.1), a lot of my libraries broke, including spectrum.
Every instance of <i /> or <span /> had to be replaced by the corresponding non-self-closing tag, like <i></i>.
Silly HTML language...
I had to fix your library by changing this self-closing tag:

Otherwise, the palette was broken.
I noticed that reverting JQuery back to the old version fixed the problem, but I might be wrong, so maybe try to verify this on your end too. It might have been a different library that caused this, because it was a whole "library-upgrade day" for me when it happened...
Update:
Ohh!!!
Actually, the 1.8.1 version published here has this fix, but if you download the 1.8.0 release ZIP, it doesn't!

I can confirm that v1.8.0 breaks with the new jQuery due to the self-closing tag issue, and that the issue is fixed in v1.8.1

It seems that there is bug in version 1.8.1 the problem is this code:
            function paletteElementClick(e) {
                if (e.data && e.data.ignore) {
                    set($(e.target).closest(".sp-thumb-el").data("color"));
                    move();
                }
                else {
                    set($(e.target).closest(".sp-thumb-el").data("color"));
                    move();

                    // If the picker is going to close immediately, a palette selection
                    // is a change.  Otherwise, it's a move only.
                    if (opts.hideAfterPaletteSelect) {
                        updateOriginalInput(true);
                        hide();
                    } else {
                        updateOriginalInput(); // here there should be true passed as argument
                    }
                }

                return false;
            }

When hideAfterPaletteSelect is not set it don't trigger the callback events. Which make ignore the pick of the palette.


Added sheenobu as maintainer

Things done
Now doesn't error:
nix-env -f $my_nixpkgs -iA spotify
Fixes #15131

Fixed in e843b35e398878da7b809f7a65a1decabcce1125.


Error: spawn rm ENOENT

Below error found while executing rm command using spawn in windows
Error: spawn rm ENOENT
at notFoundError

The rm executable does not exist on Windows. Use del instead.

Thank you 👍🏼


Update upgrade.md

added backticks to --manual-worker-upgrade

Deploy preview for docsdocker ready!
Built with commit 4c574a58d64786d86672f886029d4c447f0559c9
https://deploy-preview-9568--docsdocker.netlify.com


Send Multiple Tokens and NFT Builder, Mint, Burn

Name: Send Multiple Tokens and NFT Builder, Mint, Burn
Description: Users send multiple ADA addresses, tokens and create your own NFT, Mint, Burn with just a few clicks
Link: click here
Tags: minting, token, nftsupport

@DCOneCrypto so we can at least read this, why don't you please edit (or re-edit) showcases.js in this branch so it doesn't include all the quotation-mark, indentation and white space changes... showing just the thing(s) you are adding?
Note: unless showcase submissions are intimately related (I can't tell for sure if there are in fact 2 of them with all the noise in this diff), each PR should only for 1 addition.

@DCOneCryptođể ít nhất chúng ta có thể đọc được nội dung này, tại sao bạn không vui lòng chỉnh sửa (hoặc chỉnh sửa lại) nội dung của mình showcases.jsđể nó không bao gồm tất cả các thay đổi về dấu ngoặc kép, thụt đầu dòng và khoảng trắng thừa... chỉ hiển thị (các) nội dung bạn đang thêm?
Lưu ý: trừ khi các bài gửi giới thiệu có liên quan mật thiết với nhau (tôi không thể chắc chắn liệu thực tế có 2 bài trong số đó gây ồn ào ở điểm khác biệt này hay không), mỗi bài PR chỉ nên dành cho 1 bài bổ sung.

@DCOneCrypto so we can at least read this, why don't you please edit (or re-edit) your showcases.js so it doesn't include all the superfluous quotation-mark, indentation and white space changes... showing just the thing(s) you are adding?
Note: unless showcase submissions are intimately related (I can't tell for sure if there are in fact 2 of them with all the noise in this diff), each PR should only for 1 addition.

We have updated again as the information you mentioned. You review. Thank you 😁

@DCOneCrypto thanks for clearing up your submission to make it readable as a single change.  We can now see it points to a microsite (subdomained away from dconecrypto.finance where most of your credibility is established) with the 2 mentioned utilities on it.
There is no documentation link in the submission, with no documentation pages in the navigation bar on that microsite.  Only (if not connecting with a wallet) it displays the invitation "Connect your wallet and start exploring" which I could not imagine doing in a tool with no online documentation or vetting.
If anyone could make this tool suitable for the Dev Portal, I would suggest at least they include a page in the site navigation saying what this does that stock features of other wallets (like Eternl with its Transaction Builder) don't do.
We probably could figure that out by trial & error, but in our role as editors we are better suited to establish that submitted tools are trustworthy, well-documented and useful... if not unique.  In my opinion we are not seeing any of that here, so @DCOneCrypto if you are going to keep submitting tools here then please keep that in mind from both the editors' and your potential users' perspective.

@DCOneCrypto thanks for clearing up your submission to make it readable as a single change. We can now see it points to a microsite (subdomained away from dconecrypto.finance where most of your credibility is established) with the 2 mentioned utilities on it.
There is no documentation link in the submission, with no documentation pages in the navigation bar on that microsite. Only (if not connecting with a wallet) it displays the invitation "Connect your wallet and start exploring" which I could not imagine doing in a tool with no online documentation or vetting.
If anyone could make this tool suitable for the Dev Portal, I would suggest at least they include a page in the site navigation saying what this does that stock features of other wallets (like Eternl with its Transaction Builder) don't do.
We probably could figure that out by trial & error, but in our role as editors we are better suited to establish that submitted tools are trustworthy, well-documented and useful... if not unique. In my opinion we are not seeing any of that here, so @DCOneCrypto if you are going to keep submitting tools here then please keep that in mind from both the editors' and your potential users' perspective.

Hello @rphair
We have added tool information, function descriptions, how-to videos... Users can find the information in the "How It Work?" navigation?
We'll take note of the issue you raised when we have the next tool contribution
Thank you for your input & comments.

@DCOneCrypto I can see you've added an FAQ page to your navigation, which could be helpful although the answers are not very detailed. Still I don't think it matches the calibre of other tools in the NFT utility class here on the Developer Portal, but I will go along with the opinion of other reviewers if they think otherwise.

Thank you. We will wait for the votes of other reviewers. Great

Hii everyone!
After researching information in the Developer Portal, we found and summarized the comparison as follows:
Our team tool is now all-in-one: Send $ADA & tokens to many different wallet addresses in the same txid transaction (many applications currently do not support this function) and if there is support support (e.g. Eternl has support for sending native assets to multiple wallet addresses but they don't have an NFT, Token generator). If other developer tools have NFT, Token generation tools, most of the time they don't have built in functionality to send native assets to multiple wallet addresses.
In addition to creating tokens, NFT is currently most of the time through intermediaries and does not manage policy id by themselves, on the contrary, if the community uses our tool, they can automatically manage policy id and burn assets.
Here is our summary. Thank you to everyone who took the time to join and comment with us


Error message on wrong address stays

Describe the bug
When trying to use the Import data into LORIS toggle, missing one authentication prints an error message that stays even if the next tentative is successful. Not critical, but can confuse the user.

To Reproduce
Steps to reproduce the behavior (attach screenshots if applicable):

Use set2bids branch
toggle on the Import data into LORIS
enter a wrong address (I did not include the protocol used - https://) to connect to an existing LORIS instance
See error
Change the address to the right one (include the protocol - https://)
See the right connection and the error message

What did you expect to happen?
When the connection is established, no error message should be on-screen (maybe a temporary message).

this is a "one-shot login" bug :)


terser-webpack-plugin option extend?

What problem does this feature solve?
i want to set terser https://github.com/terser-js/terser#compress-options
drop_console
at<EMAIL_ADDRESS>can set option.uglify
https://github.com/nuxt/nuxt.js/blob/376dd35ba9465e1163a800bd0512654f68436b5f/lib/builder/webpack/client.config.js#L187-L205

This feature request is available on Nuxt community (#c7892)

This is possible with nuxt 2.1.0 via #4018.
Please refer to the docs: https://nuxtjs.org/api/configuration-build#terser


[New feature] Notification center

Righ now there is no way to see notifications in codabench. All good competitors have notification feature e.g. Kaggle.
It is difficult to manage things by just receiving emails
Notficiations could be about:

competition phase starting/ending soon
submission failure
new participant request
etc

shifted to discussion #1334


Option to change the port.

I have another service running on port 8000, so I've been editing the index.JS file manually, but a runtime parameter or at least a note on the readme might be easier.

I will look into this for the 0.0.6 release

There is already a parameter to change the port when executing it.  Look at the readme.md file in the Installation section.  The parameter is --port <port#> and has --port 8000 used in examples.

You are correct wolfie, Since I use docker and I can map ports there I didn’t know about the command line argument.
Closing issue

Sorry somehow missed that.


New version: MilosParipovic.OneCommander version <IP_ADDRESS>

Pull request has been created with komac v2.0.2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @BiosPlus,
This manifest adds Apps and Features entries that aren't present in previous PR versions. Should these entries also be added to the previous versions?
(Automated response - build 694.)


Revert "fix fix load cannot end when set exec_mem_limit (#…

…25090)"
This reverts commit 3076efcaff4096373886a848b86e295dc5b51516.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.22 seconds
stream load tsv:          567 seconds loaded<PHONE_NUMBER>9 Bytes, about 125 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.4 seconds inserted 10000000 Rows, about 340K ops/s
storage size:<PHONE_NUMBER>1 Bytes


Doubt Clarification

Hi shivakumar,
Your latest update works perfectly on create and update form.
I have a doubt i.e., how is the relationship being established between the lookup entity and the entity on which it is placed.
because when I see the API it inserts a dummy data , I want to check the actual data and using webAPI retrieve call I am unable to achieve it.
can you please help with the same
Regards,
Priyanka

Hi shivakumar,
Can you please help me with the above
Regards,
Priyanka


[QUESTION] How would I add CORS header Access-Control-Allow-Origin?

In developing a web frontend for busylight, I would like to allow the development machine (running a self-hosted dev version of the web UI) to make requests to the server where the busylight API is hosted.
Currently, browsers refuse to make those requests, claiming an Access-Control-Allow-Origin header is missing.
Where would I add that header in busylight_api.py? I don't think adding the header to every single GET endpoint is the right way -- is there some general/middleware layer in which I can add headers to all responses?
I'm willing to make the change and submit a PR -- just not too familiar with where in the API code to add things 😅

Update: I think I found the fix.
Now I need to find out whether to take in "accepted origins" from an environment variable (like how HTTP auth credentials are set) or try another approach (perhaps a "debug mode" that allows origins like localhost etc?)
What do you think -- debug mode modification vs. setting origins in an env var? Both?

Well, to be brutally honest, the web API is a proof of concept that I would not feel comfortable deploying in a production environment. I'm not happy with the current implementation since I essentially re-wrote the busylight.manager.LightManager class with FastAPI stuff bolted on. Also, I'm not a frontend guy so I have a distinct lack of experience when it comes to implementing web facing stuff.
After that confession, I am completely open to suggestions as to how to specify the cross origin control to achieve what you are looking for. I like the idea of debug mode being ultra-permissive to avoid configuration issues slowing down development and I am OK with using environment variables to control behavior instead of yet another configuration file.
I would love for you to submit a PR, I'll review it, have a little back in forth about style probably and then approve it.

No worries -- the FastAPI "bolt-on" stuff is pretty good (not that I would have chosen anything different -- I'm not a frontend guy either 😅 I'm just trying to hack together a way to control my lights from my phone)
The solution I found is to add an import for CORSMiddleware from FastAPI: https://fastapi.tiangolo.com/tutorial/cors/
I should have a PR out sometime today. 😄

@JnyJny Is the debug flag available in busylight_api.py?
I ask because I am trying to implement the CORS origin allowances for debug mode, but can't find a way to determine if debug mode is set in busylight_api.py 🤔

The command line -D debug flag is used to enable/disable logging and it's not immediately apparent to me how to pass an argument via the uvicorn.run interface.

I think the easiest way to implement this is adding something to the environment before calling uvicorn.run and then checking for it in the api object.

I think the easiest way to implement this is adding something to the environment before calling uvicorn.run and then checking for it in the api object.

As in adding an os.environ value to the runtime environment? (i.e. outside of the busylight "program"/executable/script itself -- an actual environment variable?)
Or do you mean storing something in the calling busylight class/object? (Not sure how to access the parent/calling object in Python but it seems like it should be possible... super? Not sure)

Update: I think I figured something out. Now to test my changes... :)

@JnyJny after wrestling with dependencies, I can't seem to get my code running. How do I run a development (non-pip-installed) version of the code (e.g. from a copy of the repo/folder on disk?)

I manage dependencies and packaging with the tool poetry. You can pip install it for convenience:
$ python3 -m pip install poetry
$ cd path/to/busylight
$ poetry shell
<creates a virtual environment with all the dependencies installed>
$ busyserve -D

I think the easiest way to implement this is adding something to the environment before calling uvicorn.run and then checking for it in the api object.

As in adding an os.environ value to the runtime environment? (i.e. outside of the busylight "program"/executable/script itself -- an actual environment variable?)
Or do you mean storing something in the calling busylight class/object? (Not sure how to access the parent/calling object in Python but it seems like it should be possible... super? Not sure)

Something like this in __main__:serve_http_api:
    os.environ["BUSYLIGHT_DISABLE_CORS"] = "True"
    try:
        uvicorn.run("busylight.api:busylightapi", ...)
    except:
        ...

And in api.busylight_api.BusylightAPI  in the __init__ check the environment for the existence and value of BUSYLIGHT_DISABLE_CORS and do the appropriate action.
It's gross since we are using the environment for "out-of-band" communications when we should just be passing a bool into the the init. When easy things are hard, it's an indication that tools are not being used as they were intended so it's points to some sort of architectural error in the web API code. That's on me of course :)  After the serial integration I'll have to revisit that code to make it less bad.

Thanks for the help! :) Yesterday I found a way to grab the GlobalOptions object from the "parent"(?) class -- if that doesn't work in my testing, I'll switch to setting separate env vars as you describe in https://github.com/JnyJny/busylight/issues/161#issuecomment-1208215547

I attempted to use poetry to install dependencies as you described in https://github.com/JnyJny/busylight/issues/161#issuecomment-1208202635 , however I can't seem to run busyserve or busylight in the poetry shell environment. Is there something else I need to configure?

I think I left off an important step in the previous directions:
$ python3 -m pip install -U poetry
$ cd path/to/busylight
$ poetry shell
$ poetry install
$ which busylight

Thanks! That got me further. Hitting a new error:
# busyserve -D -p 8888
2022-08-13 16:11:32.859 | INFO     | busylight.__main__:serve_http_api:378 - serving http api
2022-08-13 16:11:32.860 | ERROR    | busylight.__main__:serve_http_api:383 - import uvicorn failed: No module named 'uvicorn'
The package `uvicorn` is missing, unable to serve the busylight API.

I'm going to try python3 -m pip install -U uvicorn

Thanks! That got me further. Hitting a new error:
# busyserve -D -p 8888
2022-08-13 16:11:32.859 | INFO     | busylight.__main__:serve_http_api:378 - serving http api
2022-08-13 16:11:32.860 | ERROR    | busylight.__main__:serve_http_api:383 - import uvicorn failed: No module named 'uvicorn'
The package `uvicorn` is missing, unable to serve the busylight API.

I'm going to try python3 -m pip install -U uvicorn
Does the pyproject.toml file need to be modified?

Nope, I have helpfully omitted a step. :) The web API part is an optional install target and you can get poetry to install it thusly:
$ poetry shell
$ poetry install --extras webapi  # installs fastapi and uvicorn
$ busyserve -D ...

I've updated the top-level README.md to include development installation instructions using poetry. That way its harder for me to forget how it works and mislead people who ask.

Haha, thanks! :) I'm about to open a PR for the CORS changes.


Metadata fix

Basically just adds some checks for empty strings so that extract_metadata run properly for all sessions.

@pauladkisson Will I have to resync data from the Globus now?


update Activiti/activiti-build to master

UpdateBot pushed version changes from the source code in repository: Activiti/activiti-build ref: master

UpdateBot commands:
updatebot push --ref master https://github.com/Activiti/activiti-build.git

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-x-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


How do I modify the example below to display Streaming Video?

Has anyone created a version of the example below using streaming video?
https://github.com/googlesamples/mediapipe/tree/main/examples/pose_landmarker/python

@mcdonasd1212,
Please refer to below implementation of gesture recognition and modify according for pose landmarker.
import mediapipe as mp
import cv2

BaseOptions = mp.tasks.BaseOptions
GestureRecognizer = mp.tasks.vision.GestureRecognizer
GestureRecognizerOptions = mp.tasks.vision.GestureRecognizerOptions
GestureRecognizerResult = mp.tasks.vision.GestureRecognizerResult
VisionRunningMode = mp.tasks.vision.RunningMode

video = cv2.VideoCapture(0)

# Create a image segmenter instance with the live stream mode:
def print_result(result: GestureRecognizerResult, output_image: mp.Image, timestamp_ms: int):
    # cv2.imshow('Show', output_image.numpy_view())
    # imright = output_image.numpy_view()
    print(result.gestures)
    # cv2.imwrite('somefile.jpg', imright)

options = GestureRecognizerOptions(
    base_options=BaseOptions(model_asset_path='gesture_recognizer.task'),
    running_mode=VisionRunningMode.LIVE_STREAM,
    result_callback=print_result)

timestamp = 0
with GestureRecognizer.create_from_options(options) as recognizer:
  # The recognizer is initialized. Use it here.
    while video.isOpened():
        # Capture frame-by-frame
        ret, frame = video.read()

        if not ret:
            print("Ignoring empty frame")
            break

        timestamp += 1
        mp_image = mp.Image(image_format=mp.ImageFormat.SRGB, data=frame)
        # Send live image data to perform gesture recognition
        # The results are accessible via the `result_callback` provided in
        # the `GestureRecognizerOptions` object.
        # The gesture recognizer must be created with the live stream mode.
        recognizer.recognize_async(mp_image, timestamp)

        if cv2.waitKey(5) & 0xFF == 27:
            break

video.release()
cv2.destroyAllWindows(

Reference : How to implement gesture_recognizer.task for Live stream

This helps a lot!  One more thing, do you have an example of a function that returns the x,y,z coordinates for each Landmark?

Hello @mcdonasd1212,
This information is available in the documentation here as noted below:

The Pose Landmarker returns a poseLandmarkerResult object for each detection run. The result object contains coordinates for each pose landmark.
The following shows an example of the output data from this task:
PoseLandmarkerResult:
Landmarks:
Landmark #0:
x            : 0.638852
y            : 0.671197
z            : 0.129959
visibility   : 0.9999997615814209
presence     : 0.9999984502792358
Landmark #1:
x            : 0.634599
y            : 0.536441
z            : -0.06984
visibility   : 0.999909
presence     : 0.999958
... (33 landmarks per pose)

However, waitKey(5) is random value. You can give different value as per your requirements.

Is using an incrementing integer really a good practice for supplying the timestamps to this class?


[$1000] Return the error key rather than the translated string in all forms

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Description
This is a follow-up to the previous changes made in PR https://github.com/Expensify/App/pull/19324 and PR https://github.com/Expensify/App/pull/20093. While those PRs introduced initial updates to improve error handling by implementing key-based error storage, it was identified that there are still remaining form validation methods that require updates to align with the new mechanism.
The purpose of this follow-up PR is to comprehensively cover all form validation methods and ensure they return the error key rather than the translated string. By doing so, we aim to achieve consistent and streamlined error handling across the entire application, eliminating any remnants of the old translation-based approach.
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b859b012a0a1a383
Upwork Job ID:<PHONE_NUMBER>591188992
Last Price Increase: 2023-06-08

PR is here https://github.com/Expensify/App/pull/20228

@0xmiroslav yes, the speed bonus will still apply - pending no regressions.

Payments to be made on Jun 23:

Issue reported by: N/A
Selected Proposal for fix: @fedirjh [$1000] + [$500 PR bonus]
C+ Review: @0xmiroslav [$1000] + [$500 PR bonus]

@0xmiroslav can you complete the BugZero checklist above?

@fedirjh and @0xmiroslav Contract offers were sent!

@0xmiroslav payment sent and contract ended - Thank You! 🎉

Closing - thank you!


Consider to in addition to andThen

I just wrote a reddit comment on this, but perhaps a GitHub issue would be a home for it 😄 . To paraphrase:
Monocle 3 is an extremely exciting direction for the library to take. I love everything about it.
One tiny suggestion, though it's syntactic and I'm quite cool either way, would be to consider using to instead of (or in addition to) andThen. The composeVariants spread out and visually dominated the meaningful bits of the composition, those being the actual optics themselves. As composition is the primary operator, it could be nice to make it as syntactically diminutive as possible.
For comparison:
Group.users composeTraversal accounts composeTraversal orders composeOptional id
Group.users andThen accounts andThen orders andThen id
Group.users to accounts to orders to id
The last one, to me, is easiest to trace the important path of users -> accounts -> orders -> id.
Thanks for all of your hard work, Julien (and everyone!)!
P.S. If this is something you'd be into, I'd happily volunteer to open a PR for it.

Resolved here: https://reddit.com/r/scala/comments/l1ugt5/_/gkxgxu9/ 👍😎

Resolved here: https://reddit.com/r/scala/comments/l1ugt5/_/gkxgxu9/ 👍😎

Sorry @kitlangton, I swore I saw a comment from you but I couldn't remember where it was. I didn't check Monocle issues :(

Sorry @kitlangton, I swore I saw a comment from you but I couldn't remember where it was. I didn't check Monocle issues :(


nohash cache path and test

Codecov Report

Merging #2906 (7058df7) into master (0074ea5) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #2906      +/-   ##
==========================================
- Coverage   27.29%   27.29%   -0.01%
==========================================
  Files          99       99
  Lines       32316    32319       +3
==========================================
  Hits         8820     8820
- Misses      23496    23499       +3

Impacted Files
Coverage Δ

src/Cache/ParseCache.cpp
7.05% <0.00%> (-0.14%)
:arrow_down:

src/CommandLine/CommandLineParser.cpp
2.27% <ø> (ø)

Continue to review full report at Codecov.

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


NameError: name 'str2optimizer8bit_blockwise' is not defined

Hi, thanks for open-sourcing the project. I run the train_network.py module with the default settings and I get the following error.
Traceback (most recent call last):
File "train_network.py", line 773, in
train(args)
File "train_network.py", line 629, in train
optimizer.step()
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/accelerate/optimizer.py", line 134, in step
self.scaler.step(self.optimizer, closure)
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/torch/cuda/amp/grad_scaler.py", line 370, in step
retval = self._maybe_opt_step(optimizer, optimizer_state, *args, **kwargs)
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/torch/cuda/amp/grad_scaler.py", line 290, in _maybe_opt_step
retval = optimizer.step(*args, **kwargs)
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/torch/optim/lr_scheduler.py", line 69, in wrapper
return wrapped(*args, **kwargs)
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/torch/optim/optimizer.py", line 280, in wrapper
out = func(*args, **kwargs)
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/bitsandbytes/optim/optimizer.py", line 265, in step
self.update_step(group, p, gindex, pindex)
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/bitsandbytes/optim/optimizer.py", line 506, in update_step
F.optimizer_update_8bit_blockwise(
File "/home/ubuntu/rzamarefat/venv/lib/python3.8/site-packages/bitsandbytes/functional.py", line 858, in optimizer_update_8bit_blockwise
str2optimizer8bit_blockwise[optimizer_name][0](
NameError: name 'str2optimizer8bit_blockwise' is not defined
Any idea to solve it?

which version did u use on bitsandbytes?
if you want to use lion8bit maybe need to pip install bitsandbytes 0.38.1

Actually, I tested with 0.38.1, 0.38.0, 0.37.0, 0.36.0, and 0.35.0. The problem still exists.

Actually, I tested with 0.38.1, 0.38.0, 0.37.0, 0.36.0, and 0.35.0. The problem still exists.

i got it,it seems that u dont have supported cuda version.
it just support under cuda 11.8


109 review and test speech to text task

@900miles i move the discussion here from https://github.com/sensein/senselab/issues/109. let me know if you see any more changes to be done

Codecov Report
Attention: Patch coverage is 14.28571% with 60 lines in your changes missing coverage. Please review.

Project coverage is 61.37%. Comparing base (f3c595f) to head (5f7d9f1).
Report is 71 commits behind head on main.

Files
Patch %
Lines

src/tests/audio/tasks/speech_to_text_test.py
3.70%
26 Missing :warning:

...senselab/audio/tasks/speech_to_text/huggingface.py
0.00%
16 Missing :warning:

src/senselab/audio/tasks/speech_to_text/api.py
0.00%
7 Missing :warning:

src/tests/audio/conftest.py
41.66%
7 Missing :warning:

src/senselab/utils/data_structures/model.py
25.00%
3 Missing :warning:

...rc/senselab/audio/tasks/speech_to_text/__init__.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
- Coverage   63.04%   61.37%   -1.68%
==========================================
  Files          63       74      +11
  Lines        2073     2325     +252
==========================================
+ Hits         1307     1427     +120
- Misses        766      898     +132

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

I'm not sure if it's possible to re-open a merged pull request. Should this go on a new PR? Not sure of the procedure here.

I'm not sure if it's possible to re-open a merged pull request. Should this go on a new PR? Not sure of the procedure here.

Thank you @900miles. I think that by the time you commented, I had already merged the edits (at least to have something to show during tomorrow's meeting). i have replied to some of your comments and fixed most of them as part of a different PR. you can find all implemented changes in the main branch already


Make OrchestratorStorage a service

I want to make OrchestratorStorage a service. So we can get the OrchestratorStorage from the registry at a convenience.

I want to make the OrchestratorStorage a service, then the plugins and the skeleton can share the same OrchestratorStorage.
@jbonofre, thoughts? does it make sense?

OrchestratorStorage is actually already a service.
By default, the runtime can load one OrchestratorStorage service, this service can be used by several plugins. To do that, you just need to package OrchestratorStorage in one jar, and each plugin in other jars.
I designed the SPI to be decoupled, so, we can already have a single OrchestratorStorage shared by several plugins.

I am afraid you are missing something. The interface OrchestratorStorage on the branch main does not extend from the interface Service, or maybe I am on the wrong way?
I also see that you implement a SwiftOrchestratorStorage in OpenStack plugin, and make it a Service, I am working on the ObsOrchestratorStorage in Huawei plugin, so when all of these plugins are launched by the runtime, more than one OrchestratorStorage will be active.
In my view, I want to make the FileOrchestratorStorage the default OrchestratorStorage when no OrchestratorStorage is provided, and at the same time, move the SwiftOrchestratorStorage and the ObsOrchestratorStorage to separated modules. and package them into the different jars as you say.

My point is that we should have only one storage service. I don't think we will have the case where we will have both openstack and huaweicloud plugin in the same runtime. Today we have different runtimes : one per plugin

Ok, it makes sense by runtime with only one plugin, I see your point. thanks.
The ObsOrchestratorStorage  will be kept in the Huawei plugin module. And the "generic" storage sounds good to me.
By the way, the default FileOrchestratorStorage seems to only work for the skeleton, but not for the plugins.
Does this was designed to be? If so, I will close this issue.

The FileOrchestratorStorage is used as "fallback":
https://github.com/huaweicloud/osc/blob/main/services/orchestrator/src/main/java/org/eclipse/osc/orchestrator/OrchestratorService.java#L42
The orchestrator is looking for OrchestratorStorage service. If there's no storage service in the runtime, than, he falls back on FileOrchestratorStorage.
So, it's used by plugins if no storage service is defined.

So your point is that all storage actions must be limited in the skeleton?
For the plugins cannot get the fallback FileOrchestratorStorage instance from ServiceRegistry (FileOrchestratorStorage is not in the ServiceRegistry, or maybe we can pass through it by some other API?).
whatever,  A "generic" services/minio-storage would be better for now. let me follow up on this.
I will close this issue, thanks for your patience.

Maybe it's a good idea to have services/file-storage, services/minio-storage, etc.
I can do that.
Thoughts ?
Thanks !


KSCrashCachedData causes a crash inside ksthread_getQueueName

Thread 2 Crashed:
0   KSCrash                         0x000000010510eb4c ksthread_getQueueName (KSThread.c:87)
1   KSCrash                         0x000000010510eb30 ksthread_getQueueName (KSThread.c:80)
2   KSCrash                         0x00000001050e76d0 monitorCachedData (KSCrashCachedData.c:84)
3   libsystem_pthread.dylib         0x00000001823282b4 _pthread_body + 304
4   libsystem_pthread.dylib         0x0000000182328180 _pthread_start + 308

As you see we reported the same issue here - https://github.com/kstenerud/KSCrash/issues/207 waiting for an investigation from the author.

I want to know that did the author have resoveled the issue now ? because my project have seen many crash about this


Bug: Response code is 0 when it should be 301

Bug: Response code is 0 when it should be 301
This is happening on typhoeus v1.3.1
This is only happening for certain URL's, in this case: http://vereinintro.wordpress.com/
The following demonstrates the incorrect response code:
$ pry
[1] pry(main)> Net::HTTP.get_response URI('http://vereinintro.wordpress.com/')
=> #<Net::HTTPMovedPermanently 301 Moved Permanently readbody=true>
[2] pry(main)> require 'typhoeus'
=> true
[3] pry(main)> Typhoeus.get('http://vereinintro.wordpress.com/').code
=> 0

$ curl -I http://vereinintro.wordpress.com/
HTTP/1.1 301 Moved Permanently
Server: nginx
Date: Sat, 19 Oct 2019 05:44:00 GMT
Content-Type: text/html
Content-Length: 162
Connection: keep-alive
Location: https://vereinintro.wordpress.com/
X-ac: 1.syd _bur

The response code from Typhoeus.get should be 301 but is 0.

Another URL that shows the same problem in Typhoeus but not in other libs is: https://vereinintro.com

I can verify that behaviour but I think there is something weird with the server dropping requests based on User-Agent headers?
response = Typhoeus.get("https://vereinintro.wordpress.com")
response.code
=> 0
response.request.options
=> {:method=>:get, :headers=>{"User-Agent"=>"Typhoeus - https://github.com/typhoeus/typhoeus", "Expect"=>""}, :maxredirs=>50}

Now, let's add a silly User-Agent header
response = Typhoeus.get("https://vereinintro.wordpress.com", headers: { "User-Agent" => "My user agent" })
response.code
=> 301
response.request.options
=> {:headers=>{"User-Agent"=>"My user agent", "Expect"=>""}, :method=>:get, :maxredirs=>50}

I'm not sure what the User-Agent header (if any) is set with Net::HTTP.
If you want to not pass any User-Agent header set globally
``ruby
Typhoeus::Config.user_agent = ""

so then

```ruby
Typhoeus.get("https://vereinintro.wordpress.com").code
=> 301

I hope that helps

Yep, changing the User-Agent header from Typhoeus's default one fixed the issue for me. Strange how certain servers dislike the default header.
Thank you @PericlesTheo for the quick response and solution.


Job script example

We need an example job script. Currently the place holder is guides/job-script-example.rst  I kept it for now even though there is nothing there, because I think such an example is important.

Absolutely. I will self-assign it to me. Then I can learn something about the system
at the same time.


Client Allocation Request for: IPFS Found

Client Allocation Questions
Core Information

Name: IPFS Found
Website / Social Media: https://filecoin.in/
Region: Greater China
DataCap Requested: 5TiB
Addresses to be Notarized: f1jwiwiqfskfeiim5jrpjy65atp3ytzg4swpcijpi
Notary Requested: ozhtdong

Duplicate of #https://github.com/filecoin-project/filecoin-plus-client-onboarding/issues/10


[F2F-102] EU Driving Licence Expiry Screen

Proposed changes

What changed

Why did it change

Issue tracking

F2F-XXXX

Checklists
Environment variables or secrets

[ ] Documented in the README
[ ] Added screenshots to show the implementation is working
[ ] Ran cfn-lint on any SAM templates

Other considerations

please add screenshots


bip_handshake + bip_peer: Allow external users to define their own extension messages

Right now, we have Extensions in bip_handshake as well as built in extension messages in bip_peer.
However, we should be able to allow users to define their own extensions as part of the handshaking process, and subsequently send those messages using our PeerManager in bip_peer when the unioned Extensions struct indicates the peer supports such a message.

For now, we will proceed with the assumption that modifying Extensions requires a change to bip_handshake so that its easier for library components (in bip_peer) to know where to look, for the authoritative meaning for each of the bits that are set. This is important so that we can take action when a peer sends us a message that isn't specified in Extensions, we need to know what message we are(nt) looking for.
As for custom messages in bip_peer, we could have ExtensionType actually be ExtensionType<U>, where, if the users wants to define a single custom type, that can be U itself, otherwise, the user will have to multiplex their custom types within some enum they define, where that enum is what they set U to. This allows externally defined extensions without relying on runtime polymorphism using traits and heap allocation.

Closing as part of #88 as we allow users to define their own PeerProtocol, and layer that with the built in PeerWireProtocol, where we allow the custom ("extension") protocol to handle requests for which the PeerWireProtocol was unable to recognize.
In the future, we will implement both the extension protocol, as well as common extensions, in terms of PeerProtocol. This way we have a sort of batteries included approach to such common extension messages.


More docs required - storage key names / types

The toplevel README makes a brief mention of the AsyncStorage API that can be used to persist values. However, it doesn't go into any detail about, for example, what key names are used by the library and what types their values are expected to be.
By observation of my local copy, I have determined at least the following key names:
TODO: storage gets dc
TODO: storage gets dc2_auth_key
TODO: storage gets dc2_server_salt

I can likely guess that there may be other key names relating to other DC numbers also. But what else?
Additionally I suspect there is a bug in the documentation of the set method, which is currently documented as:

set(obj: any): Promise;

However, in my observation the method is invoked with two values - a string key, and an associated value; for example:
TODO: storage sets dc2_server_salt to c5cc98f309474772

I expect the docs should have said
set(key: string, obj: any): Promise<void>;

Tried to explain this thing more 7c5fec822becd82d9ec07ac1520d0be7b6905f49

The file-storage is not being exported to plugins

@wfjsw Yes, that's the problem, because if we do, then browser builds immediately fall due to lack of the fs. Now I'm looking for compromise solutions.

And it fails to write anything to disk, so far.

Proper realization going to be much larger, and right now module needs to have valid JSON object before the start.
CI testing works with that example

@zerobias I know that, but probably you should cat that file after finish processing. I can find only {} there.

@wfjsw this is because file-storage don't write anything to file...
https://github.com/zerobias/telegram-mtproto/blob/develop/src/plugins/file-storage/simple-file-storage.js#L67
/cc @zerobias lol

@goodmind No, its write data every 0.5s) But I agree that's awful :smile:

I invoked save() manually with no luck.

See also #45


is test case for resolve_tilde() in unset $HOME variable vallid?

when i run the test case i got
test/utils.cpp:272: FAILED:
  REQUIRE( utils::resolve_tilde("~") == "~" )

with expansion:
  "test" == "~"

Thank you for the vigilance, @ThangaAyyanar! :)
In the future, please don't ignore the "version" part of the issue template: if I haven't noticed this same issue before your report, I might've had trouble understanding what commit it refers to, and would've wasted time asking you about that and waiting for reply.

okay @Minoru


How to bundle all js, css and images files as one file using react-scripts?

Dear Sir!
I would like to bundle all assets files as one js file.
I am using react-scripts.
Please help me.
best Regards

Unfortunately, we do not allow tuning like this -- you'd need to eject.
What specific use case are you trying to accomplish?

Thanks for your reply. I want all assets files are combined as one file.

Yes, but for what reason? Why do you have this requirement?
Also, if you eject you can configure all assets to go through file-loader instead of url-loader, & disable extract css plugin.

What do you mean when you say “one file”? Which particular assets do you want to see combined and why?

Sorry to reopen this old thread, but I have the same question.
Has anything changed? Do I still need ejecting the app to have it all in one bundle.js file?
I'd like to use Zalando's Tailor to move into microfrontend world. This lib requests to serve a single file with js and css all together.
Thank you ;)

I've finally worked around this problem, without the need to eject, by just creating a webpack.config.js file at the root of my projec,t with:
const path = require("path")
const UglifyJsPlugin = require("uglifyjs-webpack-plugin")
const glob = require("glob")

module.exports = {
  entry: {
    "bundle.js": glob.sync("build/static/?(js|css)/main.*.?(js|css)").map(f => path.resolve(__dirname, f)),
  },
  output: {
    filename: "build/static/js/bundle.min.js",
  },
  module: {
    rules: [
      {
        test: /\.css$/,
        use: ["style-loader", "css-loader"],
      },
    ],
  },
  plugins: [new UglifyJsPlugin()],
}

And adapting the build script in my package.json file like:
...
    "build": "npm run build:react && npm run build:bundle",
    "build:react": "react-scripts-ts build",
    "build:bundle": "webpack --config webpack.config.js",
...

Hope this helps others.

Not the most clean way but it absolutely works and I like it!! :D
Thank you very much for your share!
2018-03-27 16:30 GMT+02:00 Matias Surdi<EMAIL_ADDRESS>
I've finally worked around this problem, without the need to eject, by
just creating a webpack.config.js file at the root of my projec,t with:
const path = require("path")
const UglifyJsPlugin = require("uglifyjs-webpack-plugin")
const glob = require("glob")
module.exports = {
entry: {
"bundle.js": glob.sync("build/static/?(js|css)/main.*.?(js|css)").map(f => path.resolve(__dirname, f)),
},
output: {
filename: "build/static/js/bundle.min.js",
},
module: {
rules: [
{
test: /.css$/,
use: ["style-loader", "css-loader"],
},
],
},
plugins: [new UglifyJsPlugin()],
}
And adapting the build script in my package.json file like:
...
"build": "npm run build:react && npm run build:bundle",
"build:react": "react-scripts-ts build",
"build:bundle": "webpack --config webpack.config.js",
...
Hope this helps others.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/facebook/create-react-app/issues/3365#issuecomment-376546407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADFui640kEE-Ybe_4l8kTfFV5tPMnJ-Aks5tik0EgaJpZM4QMH82
.

--
Samuele Di Rito
Technical Consultant
Phone:<PHONE_NUMBER>
Phone:<PHONE_NUMBER>
Learned in:
ASP.NET | WPF | Silverlight | Java | Docker | Networking | Spring |
Hibernate | Batch | Scala | EntityFramework | Dapper | Influxdb | GraphQL |
Angular 1.x | Vue.js | React | REST | SOA | Mongo | Sql Server | IIS |
Nginx | Networking | Angular | Bash | Eclipse | VS Studio | Elasticsearch |
SMACK | LagomFramework

@msurdi good sir, you're a life saver

Just to give an use case for this is that I would like to use Cloudfront signed url to deliver access to just one file (index.html) and without the need to request others access, because that will require that I create a signed url for each one, or without using signed cookies, because right now API gateway does not allow to return multiple Set-Cookie headers.

Thanks
and for the best case we will need just one bundle.js file
when we use electron

Is there a way to inline everything (all css and js) into a single html file?


djhtml: init at 3.0.5

Description of changes
Adds djhtml, a utility to indent django or jinja templates
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

cc @RaitoBezarius

You should rename the prefix into python3Packages.djhtml


PTHREAD_STACK_MIN typo in POSIX/POSIX-NG OSAL

Both the POSIX and POSIX-NG OSALs contain the following in osapi.c:
{{{#!c
#ifndef PTHREAD_STACK_MIN
#define PTHREAD_STACK_MIN 8092
#endif
}}}
I believe this is supposed to be 8192 or 8*1024.

Imported from trac issue 209.  Created by tflemin1 on 2019-03-28T14:41:32, last modified: 2019-08-14T14:11:46


add unit-test for impulse fwd-dynamics [REPLACEMENT ISSUE]

The original issue
Id: 247
Title: add unit-test for impulse fwd-dynamics

could not be created.
This is a dummy issue, replacing the original one. It contains everything but the original issue description. In case the gitlab repository is still existing, visit the following link to show the original issue:
TODO

In GitLab by @cmastalli on Sep 19, 2019, 19:35
assigned to @MaximilienNaveau

In GitLab by @MaximilienNaveau on Oct 4, 2019, 17:04
mentioned in merge request !238

In GitLab by @MaximilienNaveau on Oct 25, 2019, 15:58
mentioned in merge request !252

In GitLab by @MaximilienNaveau on Oct 25, 2019, 16:05
Associated merge request:
This implement the unit-tests for the impulses.
https://gepgitlab.laas.fr/loco-3d/crocoddyl/merge_requests/252
To be done with other merge_request

Test the multiple-impulses
Test the costs
Test the cost-sum
Test the impulse-fwddyn

In GitLab by @MaximilienNaveau on Nov 5, 2019, 17:31
Associated merge request https://gepgitlab.laas.fr/loco-3d/crocoddyl/merge_requests/254
This https://gepgitlab.laas.fr/loco-3d/crocoddyl/merge_requests/254 implements the test_multiple_impusles
To be done with other merge_request

Test the costs
Test the cost-sum
Test the impulse-fwddyn


2D image to Frustum pointcloud

Where are the pointclouds being generated from the depth map?

sunrgbd: extract_rgbd_data.m
% Write point cloud in depth map
[rgb,points3d,depthInpaint,imsize]=read3dPoints(data);
rgb(isnan(points3d(:,1)),:) = [];
points3d(isnan(points3d(:,1)),:) = [];
points3d_rgb= [points3d, rgb];
filename = strcat(num2str(imageId,'%06d'), '.txt');%生成txt文件
dlmwrite(strcat(depth_folder, filename), points3d_rgb, 'delimiter', ' ');


Update Foucoco metadata

We need to update the metadata for Foucoco, now that we restarted the chain with a new genesis.

@pendulum-chain/product This change is necessary because, at the moment, our Foucoco Spacewalk vault is not able to connect to the new Foucoco chain because we changed some internal metadata parameters.
Afterward, we need to redeploy the nodes as described in this follow-up ticket.

@ebma since this also relates to the Foucoco work, should we add a label to reflect this was 'surprise' work?


Add bash kind action

This is a new bash kind action. Related issue is #2927.
bash action runtime is based on openwhisk/dockerskeleton and action proxy is overrided for bash script. also added jq to handle json in bash script
How to use
cat <<EOF > exec
#!/bin/bash
echo \$_test
EOF

It returns only the last line, supports auto boxing. (no need to make json object in script)
$ wsk -i action update test exec --kind bash
ok: updated action test
$ wsk -i action invoke hello -r -p test "Hello World!"
{
    "result": "Hello World!"
}

Using parameters easily
The parameters are easy to use as environment variables in bash scripts. access parameters using environment variables is more intuitive and easier to use.
sungjunyoung/openwhisk-bashAction, This is the repository in which it is implemented

Object

{
  "key":{
    "key1":"value1",
    "key2":"value2"
  }
}

#!/bin/bash
echo $_key_key1
echo $_key_key2

List

{
  "key":[
    {"first":"value1"},
    {"second":"value2"}
  ]
}

#!/bin/bash
echo $_key_0_first
echo $_key_1_second

also can be access theme using jq, jq is aleady installed.

#!/bin/bash
ARGS=$@
A=`echo "$ARGS" | jq '."a"'`
B=`echo "$ARGS" | jq '."b"'`
C=`echo "$ARGS" | jq '."c"[0]'`
RES=$(($B + $C))

Testing
./gradlew tests:test --tests actionContainers.BashActionContainerTests

Closes https://github.com/apache/incubator-openwhisk/issues/2927

Recent changes to dockerSkeleton runtime now allows for better support for bash actions.
Using --native with a simple script is the recommend way for bash actions.
For example:
bash.sh reads  a line from <STDIN>
$ cat bash.sh
#!/bin/bash
read ARGS
A=`echo "$ARGS" | jq '."a"'`
B=`echo "$ARGS" | jq '."b"'`
C=`echo "$ARGS" | jq '."c"[0]'`
RES=$(($B + $C))
echo A is $A

wsk action update bash bash.sh --native

ok: updated action bash

wsk action invoke bash -p a A -p b 123 -p c '[1,2,3]' -r

{
    "result": 124
}

wsk activation logs --last --strip

A is "A"

docs will update to reflect this new usage.


CSS color for hovered button

Hi,
default CSS contains definition for all elements in Cookie Consent div
.cc_div *, .cc_div :after, .cc_div :before {...}

however it doesn't cover some states, like hover, so if template of website contains code like this
button:hover {
    color: #fff;
}

block headers has white color on mouse hover.
This change helped:
.cc_div *, .cc_div *:hover, .cc_div :after, .cc_div :before {...}

Nice catch!


Revert "Log successful transaction proposal requests"

This reverts commit df6a65474ca5a52333418d9b589b249106669bb4.
This log is going to be reevaluated and until such reevaluation happens it will be removed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 91.209%

Files with Coverage Reduction
New Missed Lines
%

src/routes/transactions/entities/tests/human-description.builder.ts
1
90.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
4628

Relevant Lines:
4950

💛  - Coveralls


Turning on network debug message w/ LwM2M sample client will result in stack check failure

Test against master branch
commit ee107baad9c438d24c6fd584f27c7d133acbbedc
Author: Johan Hedberg<EMAIL_ADDRESS>Date:   Wed Oct 18 10:29:38 2017 +0300
Enable network debug info by adding CONFIG_NET_LOG_GLOBAL=y to file prj_qemu_x86.conf

diff --git a/samples/net/lwm2m_client/prj_qemu_x86.conf b/samples/net/lwm2m_client/prj_qemu_x86.conf
index 84fa4da..c2b82da 100644
--- a/samples/net/lwm2m_client/prj_qemu_x86.conf
+++ b/samples/net/lwm2m_client/prj_qemu_x86.conf
@@ -41,4 +41,6 @@ CONFIG_NET_APP_PEER_IPV6_ADDR="2001:db8::2"
 CONFIG_NET_APP_MY_IPV4_ADDR="<IP_ADDRESS>"
 CONFIG_NET_APP_PEER_IPV4_ADDR="<IP_ADDRESS>"

+CONFIG_NET_LOG_GLOBAL=y
+
 CONFIG_NET_SLIP_TAP=y

The error is met shortly after I execute command "make run"

[lib/lwm2m_engine] [DBG] lwm2m_engine_set: path:3303/0/5700, value:0x0042f3bc, len:8
[net/ctx] [DBG] net_context_bind: (0x00407f00): Context 0x004032a0 binding to UDP [2001:db8::1]:63532 iface 0x00436c40
[net/app] [DBG] _net_app_print_info: (0x00407f00): net app connect [2001:db8::1]:63532 -> [2001:db8::2]:5683
[net/conn] [DBG] net_conn_register: (0x00407f00): [0/10/17/0x33] remote 0x004032b4/2001:db8::2/5683 local 0x0042f2d8/2001:db8::1/63532 cb 0x00007910 ud 0x00400040
[lib/lwm2m_rd_client] [INF] lwm2m_rd_client_start: LWM2M Client: qemu_x86
[lib/lwm2m_rd_client] [INF] sm_do_init: RD Client started with endpoint 'qemu_x86' and client lifetime 0
[net/ipv6] [DBG] net_ipv6_prepare_for_send: (0x004080a0): No default route to 2001:db8::2
[net/ipv6] [DBG] net_ipv6_prepare_for_send: (0x004080a0): Neighbor lookup 0x00000000 (255) iface 0x00436c40 addr 2001:db8::2 state -
[net/nbr] [DBG] net_nbr_ref_debug: (0x004080a0): nbr 0x00436080 ref ***** Stack Check Fail! *****
Current thread ID = 0x004080a0
Faulting segment:address = 0x0008:0x00002caf
eax: 0x00436a48, ebx: 0x00000020, ecx: 0x00023740, edx: 0x00000031
esi: 0x00426094, edi: 0x00003f01, ebp: 0x00426004, esp: 0x00426000
eflags: 0x216
Fatal fault in thread 0x004080a0! Aborting.
QEMU: Terminated

Explicitly setting CONFIG_USERSPACE=n in prj_qemu_x86.conf and the error will be gone.

I guess this is caused by HW_STACK_PROTECTION and not a userspace issue, @andrewboie fyi

@nashif you are right. it's related to HW_STACK_PROTECTION instead USERSPACE.

It turns out I only have to expand the stack size of CONFIG_LWM2M_RD_CLIENT_STACK_SIZE when enabling the debug info.


Query regarding calculation of iteration_steps and role of global_batch_size.

I have 100000 observations(num_train_examples ), train_batch_size  = 4, num_gpu_cores  = 8,
I want to train with num_train_epochs = 1,
Could you please suggest do I need to go for
situation -1 iteration_steps  = 100000/32 or
situation -2 iteration_steps = 100000/4
If I go with situation-2 what is the role of global_batch_size ?

I have 100000 observations(num_train_examples ), train_batch_size = 4, num_gpu_cores = 8,
I want to train with num_train_epochs = 1,
Could you please suggest do I need to go for
situation -1 iteration_steps = 100000/32 or
situation -2 iteration_steps = 100000/4
If I go with situation-2 what is the role of global_batch_size ?

Assume that fixed num_train_epochs = 1 and train_batch_size = 4. If use multi-gpu and num_gpu_cores = 8 (situation 1), then global_batch_size = 32 and iteration step =  100000/4. If num_gpu_cores = 1 only (situation 2), then global_batch_size = 4 and iteration step = 100000/4 also. Now, you can see the difference that situation 1 could provide larger effective batch_size in training (32 vs. 4) with same iteration step and it is more stable.


Add FlagwiseNot().

Convenience to invert all flags on an enum.
Fixes #10.

Oh, this should probably be against InDev branch to ensure that master is always the latest release.

Oh, this should probably be against InDev branch to ensure that master is always the latest release.

I cannot figure out how to change the target branch of a PR (maybe that is intentionally impossible). So I have created PR #12 in order to target InDev.


Assignment5 tommy

Added dataset and python notebook for Assignment 5

Added dataset and python notebook to assignment 5

Thanks @tommyho973 - I wish you had split the notebook into cells so we can see code and visualizations together.
Overall good work-
Score - 90/100


vasp custodian python call might not follow running python interpreter

Details about the quacc environment

quacc version: latest
Python version: 3.11

What is the issue?
The vasp w/ custodian calculator has the ASE calculator make a subprocess that calls custodian with python:
https://github.com/Quantum-Accelerators/quacc/blob/main/src/quacc/calculators/vasp/vasp.py#L176
This works if you are in an environment where python always refers the current running python. This is not necessarily the case in a jupyter notebook that is using a loaded conda kernel.
A safer way to do this would be
return f"{sys.executable} {run_vasp_custodian_file}"
This works because explicitly calling the python from the appropriate conda environment will handle imports/etc cleanly.
How can we easily reproduce the issue?
Any vasp calculator call with custodian in jupyter if the process that launched jupyter does not have custodian installed.

Thanks for the report. Indeed, I have never tried doing that. Will be closed by #1522.


👨🚀 Compatibility with Astro

Prerequisites

Version: 6.1.1
Using: Astro + React + Typescript
Operating system: Windows 11

Step to reproduce

Create a new component with Iconoir icon:

import { GitHub } from "iconoir-react";

const WithIconoirIcon = () => {
  return (
    <GitHub width={23} height={23} />
  )
}

export default WithIconoirIcon

Import in a Astro file:

---
import WithIconoirIcon from "../components/withIconoiricon";
---

<WithIconoirIcon />

Running npm run dev works but if you run npm run build it shows the following error:

 generating static routes
 error   Named export 'GitHub' not found. The requested module 'iconoir-react' is a CommonJS module, which may not support all module.exports as named exports.
  CommonJS modules can always be imported via the default export, for example using:

  import pkg from 'iconoir-react';
  const { Twitter, GitHub, LinkedIn, Heart, OpenNewWindow, Npm, Puzzle } = pkg;

Additional info or screenshots:

🎉 Fixed in v6.2.0


in order to reduce writes we could keep file content in chunks

this would reduce the actual data we need to rewrite when we change just a small portion of the file
we would just re-encrypt just the chunks that overlaps with changed data

we don't implement this because we will use WAL to write diffs also and recreate file from that on crash


Patch 2

Please ignore.
I tried to suggest 2 changes to make things clearer in
https://github.com/MiczFlor/RPi-Jukebox-RFID/blob/future3/main/documentation/builders/installation.md#install-phoniebox-software
and
https://github.com/MiczFlor/RPi-Jukebox-RFID/blob/future3/main/documentation/builders/audio.md
not 787 changes !

You just chose the wrong target branch. I changed it to future3/main.
But could you please do your changes on a future3/develop based branch and also choose this as the target branch?
You could reset your branch on it and cherry pick your changes if you are familiar enough with git.

@AlvinSchiller its now on future3/main, did you want it on future3/develop instead?

@AlvinSchiller its now on future3/main, did you want it on future3/develop instead?

Not yet, as the PR branch is based on future3/main. This way only the real changes are shown.
If the branch is corrected it can be changed to future3/develop.

@AlvinSchiller we need that on dev from my point of view

Just changed it to point to future3/develop

Just changed it to point to future3/develop

thats not enough, as the head branch is based on main. Thats why i left the base for now.
I rebase the branch to develop now

It worked for me without sudo btw

It worked for me without sudo btw. tested on bullseye and bookworm.

so we close the pr?

@SKHSKHilber I added your second suggestion about the 64-bit kernel fix in https://github.com/MiczFlor/RPi-Jukebox-RFID/pull/2216


qwant-api error: The API returned the error code '7'

Hello,
Thank you for the great package. I use the api to search for images. It has been working for a couple of months, until recently. Now it always gets "qwant-api error: The API returned the error code '7'".
Could you please help to look into it? Thank you.

Unfortunately those error codes are returned by qwant itself and are not documented. I currently do not have time to look into this due to military service

Thank you for your reply. Good luck in the military service.

This is an old issue but just in case someone encounters the same error, this error means that the locale parameter in the request is incorrect, so either remove it or use a correct locale (for instance "en_us")


Resolve warnings when running CodeQL-Build

Addressed the following warning annotated when running CodeQL-Build:

1 issue was detected with this workflow: git checkout HEAD^2 is no longer necessary. Please remove this step as Code Scanning recommends analyzing the merge commit for best results.

Sample:
https://github.com/awslabs/autogluon/actions/runs/499961099

This is the running result:
https://github.com/awslabs/autogluon/actions/runs/501473878

This is the running result:
https://github.com/awslabs/autogluon/actions/runs/501473878

Job PR-879-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-879/1/index.html

Job PR-879-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-879/1/index.html


add smartphone crash icon

Preview

any suggestion on improving this icon

I'd just put an x in the phone.

I tried it conflict with the dot the bottom.

I think you can try to remove the dot.

Will it even look like phone if we remove the dot?

How many phones still have a physical home button nowadays?

How about now?

Preview

Feels like the phone is charging now :P

@ericfennis Created the PR https://github.com/lucide-icons/lucide/pull/316

@mittalyashu What should we do with this one?

How about something like this?

IK, it's a vague idea, tried to think of a different way. As we're trying to show the phone is crash not the actual phone.

And what about just an warning sign with the phone?

Like this https://github.com/lucide-icons/lucide/pull/306#issuecomment-835792843


Intersection exception

I use the library to calculate intersections of UIBezierPaths. In some cases I run into the BezierPathIntersectionException (mismatched intersection length) when intersecting paths with almost identical path segments.
I've added a test case that reproduces the exception. From briefly looking into the algorithm I don't understand what's going wrong here. I would be glad if you could take a look into it.
It might be the same issue as what's causing #9.
Thanks!

Big thanks for the test case! I don't have time in the near-term to look into this, but when I finally do get time that test will be a big help. I think you're right that this is #9 as well.

This is big bug in my app. When can you see this bug and resolve problem?

Hi @TianLong1989, unfortunately I wont have time to fix this in the near term. Depending on your paths, you might be able to scale them up by 10 or 100, run the clipping, and scale the results back down. i know that's also not ideal, but could possibly resolve the issue for you in some cases.

test case from the Arclite referenced above:
(lldb) po textPath
path = UIBezierPath()
path.move(to: CGPoint(x: 148, y: 1422))
path.addLine(to: CGPoint(x: 148, y: 1456))
path.addLine(to: CGPoint(x: 153, y: 1456))
path.addLine(to: CGPoint(x: 153, y: 1422))
path.close()

(lldb) po wordPath
path = UIBezierPath()
path.move(to: CGPoint(x: 93.3019079008692, y: 1422.00000072504))
path.addLine(to: CGPoint(x: 93.3019079008692, y: 1467.087208519229))
path.addLine(to: CGPoint(x: 1164.245267275869, y: 1467.087208519229))
path.addLine(to: CGPoint(x: 1164.245267275869, y: 1422.00000072504))
path.close()


User reported that ledger wallet isn't working in the app

Only info we got:
"When I select a Ledger wallet here http://app.bancor.network page - nothing happens".
No idea if we ever tested it but it should be tested and confirmed working.

I have a Ledger, just need to find it and set it up.

tested, and works fine.
Login, loads balances, can confirm transaction.
Only worked on mainnet though. Ropsten didnt load the balances.


Add RDF* support

The RDF* proposal is getting traction in the RDF community by allowing easy triple annotation. It would be nice to support it in Oxigraph.
Relevant documents:
Specification draft: https://w3c.github.io/rdf-star/#
SPARQL* query: https://arxiv.org/pdf/1406.3399.pdf
SPARQL* update: https://blog.liu.se/olafhartig/documents/sparql-update/

This is now done in the v0.3 branch.

I have just released v0.3.0-beta.1 that provides a full implementation of the RDF-star community group final report.

I have just released v0.3.0-beta.1 that provides a full implementation of the RDF-star community group final report.


Deduplicate Stronghold Nodejs wrapper code

Description of change
Uses a declarative macro to deduplicate wrapper struct code in the Stronghold Nodejs napi.rs bindings and simplifies the project layout.
Also updates to the latest napi.rs version, which necessitates the change from NapiDID to NapiDid (former causes problems and the js_name workaround no longer works). This allows us to unpin the version.
Links to any relevant issues
Fixes sub-task of #669.
Type of change

[x] Chore/Bug fix (a non-breaking change which fixes an issue)
[ ] Enhancement (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Fix

How the change has been tested
Stronghold Nodejs tests pass locally.
Change checklist

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Awesome, that is a huge improvement!


Moved arquillian classes from core to arquillian-extension. Cleanup dependencies.

Thanks!
I really like that the arquillian classes are now in their own project and no longer mixed with the core tests.


Simplify and speed up common prefix searcher

This PR simplifies the implementation of common prefix search and avoids storing mapped characters in an auxiliary data structure. With this modification, the search speed was improved as follows.
unidic/cps/crawdad/trie time:   [6.9536 ms 6.9676 ms 6.9839 ms]
                        change: [-14.553% -13.982% -13.439%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 10 measurements (20.00%)
  2 (20.00%) high mild
unidic/cps/crawdad/mptrie
                        time:   [8.1081 ms 8.1424 ms 8.1795 ms]
                        change: [-18.109% -17.562% -16.983%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 1 outliers among 10 measurements (10.00%)
  1 (10.00%) high mild

Discussion
This modification contains the following trade off.

Pros.

Make the API simpler.
Remove the auxiliary data structure to store mapped characters.

Cons.

Map the same characters in a haystack repeatedly.
Omit byte positions from matched results.

The experimental results indicate that storing and accessing mapped characters in an array is more costly than mapping characters repeatedly. The new simpler API would be easier to use, since it is not necessary to create an instance to store characters. Byte positions are not a necessary result for all users, so there would be no need to provide them in this library.

@kampersanda
Thank you for the improvements.
I think for most users, byte positions are more valuable than code point positions because they store text as a byte array in most cases.
However, keeping in mind that the argument takes an iterator of chars, I do not think it is a bad idea to keep only code point positions.
I want to give some thought to which would be better.


docs - creating new identical backup schedule

When creating a new backup schedule for a volume that was restored, will existing backups be taken into account by the Retain policy?
For example:
Old schedule Schedule: 00:00 every day
New schedule Schedule: 00:00 every day

Not likely, since the retain policy is determined by the recurring job name. The new volume likely has a new recurring job name.

Not likely, since the retain policy is determined by the recurring job name. The new volume likely has a new recurring job name.

I've kept the volume name exactly the same. Are the recurring job names seeded from some some sort of volume information, or randomly generated?

The recurring job name is random since you can have multiple jobs for one volume.

The recurring job name is random since you can have multiple jobs for one volume.

We can see this creating extra UX effort to clean out the old backups created by the old recurring schedule

We can provide an option to accept the predefined recurring job name, so you can put the old recurring job name there.
But if you're concerned about the storage space used in the backup store, since it's incremental, the extra space used by the old backups are minimal.

We can provide an option to accept the predefined recurring job name, so you can put the old recurring job name there.
But if you're concerned about the storage space used in the backup store, since it's incremental, the extra space used by the old backups are minimal.

Manually inputting the old recurring name might bring us back to the same to this issue https://github.com/longhorn/longhorn/issues/946 and https://github.com/longhorn/longhorn/issues/883 and https://github.com/longhorn/longhorn/issues/935
Where manual inputting works when there are edge cases. But where it might not do so well, is when we have a few hundred volumes we'll have to individually restore their schedules


Trouble deriving new DbContext

I want to derive a new dbcontext, in order to use multiple migration sets.
I use the following code from the 'Two migration sets' sample code above:
    public class MariaDBDbContext : ApplicationDbContext
    {
        public MariaDBDbContext(DbContextOptions<MariaDBDbContext> options)
            : base(options)
        {
        }

        protected override void OnConfiguring(DbContextOptionsBuilder options)
            => options.UseSqlite("Data Source=my.db");
    }

If i don't use the constructor, i get an error about not supplying the 'options' constructor param, and if use it, i get an error about not being able to convert it.
Please provide the correct code.

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

ID: 7b011c7a-3534-78e3-b491-addd96fb5ba1
Version Independent ID: 371826c5-3ef2-4eef-32e9-d55c777cd9d3
Content: Migrations with Multiple Providers - EF Core
Content Source: entity-framework/core/managing-schemas/migrations/providers.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @bricelam
Microsoft Alias: bricelam

@derth There's nothing obviously wrong in the code you're posting. Please post a small project that reproduces the error you are seeing so that we can investigate.

Just create a new project with auth (to include a dbcontext). Try to add a new dbcontext like the docs say.
It can't convert from:
DbContextOptions options
to:
DbContextOptions options
error:
Error	CS1503	Argument 1: cannot convert from 'Microsoft.EntityFrameworkCore.DbContextOptions<WebApplication2.Data.MySqliteDbContext>' to 'Microsoft.EntityFrameworkCore.DbContextOptions<WebApplication2.Data.ApplicationDbContext>'

@derth See #594. What does your ApplicationDbContext class look like? It should have a protected constructor that takes a non-generic DbContextOptions parameter. This is the constructor that should be called from the public constructor of your class.


Use profile picture in Social Auth

add to SOCIAL_AUTH_PIPELINE
    'saleor.pipeline.save_profile_picture',
declare function (i do only for Google)

def save_profile_picture(backend, user, response, details, is_new=False, *args, **kwargs):
    if backend.__class__.__name__ == 'CustomGoogleOAuth2':
        # user_object = User.objects.get_or_create(user=user)  # RETURNS TUPLE (instance, created(boolean))
        image = response['image'].get('url')
        display_name = response['displayName']

        user.image = image
        user.display_name = display_name
        user.save()

now we can use user avatar in templates:
                    <img src="{{ user.image }}" alt="Avatar" class="img-rounded" style="width:30px;">

I don’t feel like we should use pictures for the customers. Can’t find the reason why the user appearance would help with anything that happens in the shop.

I agree with @Pacu2. We're not planning to introduce avatars in the current storefront. It's the type of customization that people can do themselves in their forks of Saleor. This issue is a good reference for anyone looking for a possible solution.


Use the Angular Package Format

There's been some development in publishing libraries since our build system was last time revised. We now have an official document called Angular Package Format (APF), which describes the what and why.
As for how... There's been a lot of discussion everywhere. In the CLI repo, it started here and after more than a year continued here. Here, we have a very informant comment by Rob, which explains problems with packaging and why CLI cannot do it the way we create apps with it.
The important bit in the comment is angular-quickstart-lib repo by Filipe, Angular CLI core member which serves as an APF starter. There's a PR to make the readme of that repo an official Angular guide, but looks like it got stalled for a month now; so I'm not sure where we're at from the standpoint of "this is official and this is how it should be done" by the Angular core team.
There's also an ongoing attempt to create a CLI for creating libraries in APF, called ng-packagr, but it's rather new and not used much yet. So I'd rather use the aforementioned APF boilerplate as a reference point and re-organize the repo.

@lazarljubenovic 100% agree. I didn't know that there's a doc already. I created this issue #974 a while ago. This would also fix the several Universal issues.

Merging such a change would pretty much break all pending PRs because everything would be moved around and renamed, so it might be a good idea to take a few weeks going through some larger PRs first?

@lazarljubenovic no, this should be independent from the code changes, as we have to change the structure of the package content but not the source code itself.

Hey guys, I wrote this blog post about using ng-packagr to build out a component library using the CLI, https://medium.com/@ngl817/building-an-angular-4-component-library-with-the-angular-cli-and-ng-packagr-53b2ade0701e - having worked on a few component libraries myself in the last year and a half, this is by far the easiest process I've come across. Would recommend you check it out, and if you see holes in the process would very much appreciate feedback. Thanks.

how about this option ?
https://github.com/jvandemo/generator-angular2-library

Hey, any update on this?

They have a lib part in case it cab usefull for you guys https://blog.nrwl.io/nrwl-nx-an-open-source-toolkit-for-enterprise-angular-applications-38698e94d65

Not solved. Should probably be re-opened (:thinking: ?)


Make extensionHostProcess.js more accessible

Does this issue occur when all extensions are disabled?: well it's about developing extensions

VS Code Version: latest insiders
OS Version: Manjaro

Every time I'm debugging extension code I'm battling against extensionHostProcess.js. There was an issue once 5 years ago #74845 where the suggestion was to exclude it from the debugger. But I don't want to do that - often this very file includes some hints to what went wrong.
Here's just one example: Today, I was presented with this (no stack trace):

("Canceled" thousands of times, in extensionHostProcess:147). I have no idea where it comes from nor what it does. "Canceled" doesn't occur in my workspace. It keeps logging this hundreds of times per second, and eventually the entire editor freezes up. Might be my fault or some log output loop, idk, doesn't matter, this would be okay so far, it it were possible to debug.
I could click on the file and after 30 seconds of unresponsiveness, I am eventually facing this:

So I put a breakpoint into line 147, but the "Canceled" message just kept logging as before, so I guess it pointed to the wrong line.
After restarting the extension, the file became invalid, as now a new extensionHostProcess.js tab opens up. Apparently every runner gets its own.
After restarting, the file is gone.

 Development is always a debugging process, and debugging is a constant frustrating pain. I find myself battling against the horrible usability of extensionHostProcess.js just way too many times, and it really doesn't have to be like this.
Please,

at least configure your minifier to keep line breaks, or better,
disable aggressive minification for this file
split this file into multiple chunks so it doesn't take ages to load
add columns to the loggings, e.g. it should be extensionHostProcess.js:147:40205, not just :147
if possible, reuse it between extensions
if possible, keep it open after restart

We have no plans to disable minification for the extension host file, as there are strong benefits to keeping it there. Note that if have extensionHostProcess.js in your call stack you can hit the curly braces {} in the top right editor actions to deminify the code.
We don't do in-process code splitting at all for VS Code yet, but might in the future. That's tracked here: https://github.com/microsoft/vscode/issues/164068
Currently the logging only shows the line in the REPL for cleanliness, but clicking on the log location will bring you to the right column in the file. Are there cases where you'd want to instead only see the column number but not jump to that location in a file?
Keeping/reusing the editor is tracked in https://github.com/microsoft/vscode/issues/131502
"So I put a breakpoint into line 147, but the "Canceled" message just kept logging as before, so I guess it pointed to the wrong line." -- note that you can place a breakpoint at a specific column in a line via the Debug: Inline Breakpoint command (shift F9)


fix: make tekton controller image configurable

Signed-off-by: ankitm123<EMAIL_ADDRESS>

may be we can make all images configurable in a subsequent PR, there is this PR: #30, need to check if that works ...

will fix merge conflicts after #36 is merged

may be we can make all images configurable in a subsequent PR, there is this PR: #30, need to check if that works ...

Well, no, since #30 includes hand made changes. But the approach in this PR could be expanded.

Agreed, I will modify this pr to make all the images configurable ..


Incorrect message "Resource has unsaved changes. Are you sure you want to close?" displaying

Closing a saved description in the University of Washington's resource templates results in the message "Resource has unsaved changes. Are you sure you want to close?" when there are in fact no unsaved changes. This happens with both new descriptions and after editing previously-saved descriptions and re-saving.

@CECSpecialistI Thanks for reporting! I can't reproduce the problem. Can you please provide exact steps to reproduce?

Sure! The problem is in production, if that makes a difference.
Either:
a) Open any WAU resource template, fill out the required fields, save, and hit close
or,
b) Open a WAU description at any level (the one above, "How to train your boxer", works), make some change, save it, and hit close
If that doesn't reproduce the problem, maybe it's an issue with our "group" specifically?

Afraid I still can't reproduce. Can you try a hard refresh and see if it can be replicated?
Assuming that doesn't fix it, what OS are you on? What browser and version?

I am having the same issue.  I am using Firefox 71.0 64 bit on Windows 10 education.  I'm at Stanford,  Here's a screenshot of an instance i created today, apparently successfully saved despite the message, closed and reopened without making changes, then tried to close:

And here's a work that i created on Nov. 12.  I just opened it, didn't make any changes, and tried to close it.

This can be reproduced with https://trellis.stage.sinopia.io/repository/stanford/3b9c61b0-0279-4d17-915c-1f1f725ed2ca in Firefox on Mac and Windows.

Sorry for the delay, my kid got sick yesterday. I'm using Firefox Version 71.0 (64-bit) on Windows 10. Hard refresh does not affect the problem.