ZWJ sequence emoji's: Warn when buying or flag address "fake" when sending

User is requesting a feature on the Hiro wallet such as the one below to avoid scams with "fake domains"
A. Warning when a user tries to buy unicode domain, that is a 'copy' of another emoij due to ZWJ Sequence.
Alien: xn--cr8h.stx
https://gamma.io/collections/bns/xn--cr8h.stx
Alien with other skin tone: xn--cr8hv5640c.stx
https://gamma.io/collections/bns/xn--cr8hv5640c.stx
B. A warning when sending to such an address.
Opensea and Metamask do this:

Preliminary analysis:
A. I am not sure where we would put such a warning on the wallet, on the sign screen where we show "allow mode warning"?. Perhaps it would be better to add something like that to a marketplace, as the example above (from opensea?).
Is this still an issue if the support was for all emoji's including the skin toned ones?
B.  Example if you add Alien emoji (👽.stx) on the send address field in Hiro Wallet it doesn't currently translate to a public address. I think that is a good protection against sending to the wrong address already, right?
See difference in address field
use of emoji:
(the mapped Stacks address is not shown, if you send "preview/confirm" an error is shown)

And use of unicode:
(the mapped Stacks address is shown)

Best practise would be to use the BNS name to send, instead of the emoji? When we add further support for unicode emoji's we should consider the consequences here.

Do I understand correctly that this is different with Metamask? Using 👽.eth the address field in the Metamask wallet: it would translate to xn--cr8h.eth so a user may (get scammed) buying the xn--cr8hv5640c.eth domain thinking it is 👽.eth and then later send or request funds to it going to the xn--cr8h.eth owner?
If we start warning for this, I think there are a lot more things that could be added in the future. For example this one (that includes persian and arabic numbers):

That looks a lot like (original, all arabic numbers)

This seems like a possible vulnerability to resolve as part of https://github.com/hirosystems/stacks-wallet-web/issues/2872 rather than with the current feature set, since as you mention, users can't enter emojis etc. into the send field currently in any case.

The wallet team cannot take on building a custom script to support emoji's. They are considered a fun addition for wallet users and the wallet team has other priorities.
That said we are open to integrating an alternative NPM package if someone can make a clear case that it conforms more correctly to the punycode standard.
We are currently using this NPM package for the translation of punycode to emoji's:
This is the lib,
https://www.npmjs.com/package/punycode
and using it's toUnicode method to do the conversion in this file,
https://github.com/hirosystems/stacks-wallet-web/blob/dev/src/app/common/hooks/account/use-account-names.ts

hi everyone, thank for topic, this is the problem what need to be solved ASAP. My proposal - Gamma team should add Single Punymoji Club to the Club category as they did with other Clubs. This will separate fake domains, as they have more then one punycode length and will separate other namespaces from .btc domains, so people will not be confused as it's continuously happen now.

Adding this here for discussion.

yes this must be clarified by Mechanism team

cc @hstove @jeffdomke re: BNSx above

Update from the BNS DAO discord:

Suppose hiro wallet input emoji domain Same as sendstx.com. If wallet does not use the warning tag. an example: visa.btc. Input and copy paste are different wallet addresses. How do you guys prevent this?

I think the wallet don’t has a warning :warning:. is not feasible

I think including zwj should give ⚠tag(Market/Wallet/Chain).The real zwj emoji in the library should not give ⚠tag.

Suggestion from the BNSDao discord:

Seems like a reasonable enhancement to help users check their punycodes maps to the emoji they think.

Converter problem. Real zwj emoji https://emojipedia.org/emoji-zwj-sequence/. Mint has been completed. All follow gamma/wallet/sendstx and project team devs' suggestion mint's .so switch converter?  What should real zwj emoji holders do?

It's worth noting that there are the emoji zwj sequences you link to above which alter the appearance of emojis (combining two or changing skin tone), but there are also a handful of specific emojis that make use of zwj without being classed as a sequence. The only ones I know of are the single digit set approved in Unicode 1.1 and added to Emoji 2.0 in 2015.
Example here: https://emojipedia.org/digit-one/

It's worth noting that there are the emoji zwj sequences you link to above which alter the appearance of emojis (combining two or changing skin tone), but there are also a handful of specific emojis that make use of zwj without being classed as a sequence. The only ones I know of are the single digit set approved in Unicode 1.1 and added to Emoji 2.0 in 2015.
Example here: https://emojipedia.org/digit-one/

that's why they don't want to use unicode visualisation as wallet address, instead will be used punycode as wallet address, but they will resolve it to unicode after you post punycode to double check receiver address

It's worth noting that there are the emoji zwj sequences you link to above which alter the appearance of emojis (combining two or changing skin tone), but there are also a handful of specific emojis that make use of zwj without being classed as a sequence. The only ones I know of are the single digit set approved in Unicode 1.1 and added to Emoji 2.0 in 2015.
Example here: https://emojipedia.org/digit-one/

punycode will be added back to wallets

Nat speaks very intelligently.  can temporarily accept this idea. It can be used on the testnet as soon as possible. But it will not accept changing the converter. You display your mint. don’t need to change anything for your bug mint

the

Nat speaks very intelligently. can temporarily accept this idea. It can be used on the testnet as soon as possible. But it will not accept changing the converter. You display your mint. don’t need to change anything for your bug mint

來自BNSDao不和諧的建議：
似乎是一個合理的增強功能，可以幫助使用者檢查他們的小程式地圖到他們認為的表情符號。

Suggestion from the BNSDao discord:
Seems like a reasonable enhancement to help users check their punycodes maps to the emoji they think.

Nat speaks very intelligently. can temporarily accept this idea. It can be used on the testnet as soon as possible. But it will not accept changing the converter. You display your mint. don’t need to change anything for your bug mint

Suggestion from the BNSDao discord:
Seems like a reasonable enhancement to help users check their punycodes maps to the emoji they think.

Nat speaks very intelligently. can temporarily accept this idea. It can be used on the testnet as soon as possible. But it will not accept changing the converter. You display your mint. don’t need to change anything for your bug mint

Suggestion from the BNSDao discord:
Seems like a reasonable enhancement to help users check their punycodes maps to the emoji they think.

yes, please! right now difficult to understand what emoji used as wallet address when adding the code only!

It looks good!  But fake mint they also show the same emoji.a long-term plan, it is best to have a ⚠tag for zwj. The official list of real zwj emojis does not require a warning tag

Even with punycode code use pay method. ⚠tag is required. Let fake mint know: fake is always fake

If you guys disagree has a ⚠tag.fake should be forever this

I received this today from a valued member of the community.

all this digits are fake
made with fe0f
this is not my wallet, people who minted posted in the chat
could you not show visualisation for fake digits at least?
we will have scammed buyers and a lot of FUD
https://adraffy.github.io/punycode.js/test/demo.html#u=1
code content digit+fe0f, nothing else
I have asked the team how they would feel about not rendering some of these codes such as a "fake" 21.btc.

Doing something about this is not going to be a priority anytime soon. https://github.com/hirosystems/wallet/issues/2962#issuecomment-1401658513
Although we can consider adding labeling punycode values that render in a confusing way. The risk profile is very different in the wallet vs. somewhere public like Gamma.
The wallet UI shows punycode->unicode conversions for a particular user who already bought / obtained these relevant BNS names. It does not show them for people who haven't yet bought them but might be deceived into doing so.
So, we could enhance this display, there's theoretically little to no surface area here for things to go wrong at the moment, as far as we can tell.
Although it may be frustrating, this is how punycode works. It is a legit use of the punycode protocol. Should we really be blocking part of that protocol because some users do not like the way it works?

As I have mentioned before our team can't prioritize work on this. Not even the suggestions above that could improve the user experience such as https://github.com/hirosystems/wallet/issues/2962#issuecomment-1467801838 but we are open to reviewing PR's send by the community to implement such an improvement.


AWS ECS agent support

Can we run this on AWS ECS? and get metrics for node, container and logs

Unfortunately, in the case of AWS ECS, it is not possible to collect metrics at the node level (because ECS is a serverless solution) and those based on eBPF (due to very limited privileges). Thus, node-agent cannot be used to monitor containers running on ECS. Perhaps we will implement the collection of ECS container metrics as part of aws-agent.

But, is possible to only do log analysis and turn off other features of agent like node metrics and network metrics?


Add Observability links

Description
Changes proposed in this pull request:

Add Observability section only if there's at leas one nav node to display
Allow user to provide the Title and VirtualService link via Config Params
Default to the Busola ConfigMap if the Observability is not configured via Config Params
Ignore the navigation node if the virtualService it's linked to doesn't exist (host value cannot be read)

Related issue(s)
Resolves #255

/retest

/retest

/test all


Solution added passing leetcode test

IMPORTANT: Please do not create a Pull Request without an issue.
All Submissions:

[x] I have read the CONTRIBUTING document.
[x] My code is written in Python3 and is ending with .py
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have checked that my submission does pass the test on LeetCode.com
[x] Does your filename follow the naming Conventions?
[x] Have you linked your PR to an Issue?

Closing issues
Closes #695

Is it supposed to pass that check? Because it is failing at a login function?


added call to load services before filtering

@lkysow @PeaceGeeks @cynthialee @Jareechang
Fix to make sure the services load before using a filter. Fixes page reload problems.

Thanks. Is this associated to an issue? If so, please close it.

Yes this was issue 65 i will close it.


How to configure function timeout

Just reading the documentation it states that a function runs has a10 seconds timeout (can be upgraded). What are the options, is this a code/config change to increase?
Thanks

If you are referring to the our cloud, you just need to upgrade your plan and you will get the new timeout automatically.


customizing fileIO dialog

🚀 Feature
Right now we just use the native fileIO dialog as bellow

but can can potentially customize our own fileIO dialog that would allow us to include extra toggle buttons, and drop-downs. I think this will be increasingly important as we try and deal with more complex layer and file types, and I'd rather us just have one pop-up menu where it all happens, instead of multiple pop-up menus, or many different open functions inside the file menu. The goal would be to unify as much as possible.
I imagine that we'd have one drop-down for layer type, and then depending on what layer type you had selected you would have different checkboxes / dropdowns available to you. We could expose all the keyword arguments for the layer pre-populated with their defaults, which I sort of like as people at least get to see them too, even if they don't edit them.
A common problem I saw introducing some biologist to napari the other day was they had jpegs of data that were rgb but they wanted to add using add_multichannel(data, axis=2) so they got the channels separated. Or they had multichannel tiff of different color channels which they wanted to add using add_multichannel(data, axis=0) but there was no way for them to do that from the fileOpen dialog, and it was quite cumbersome to do the conversion once added.
Thoughts @jni @royerloic?

I'd like to keep fileIO native, as I always find it jarring to have a non-native fileIO dialog. The Mac's dialog in particular is excellent and people know exactly how to navigate it, it has spotlight built in, etc. I would prefer having a big custom menu with a single "select files" button that launches the native dialog (and lists the files once selected).


Correction for raw affiliation Behavior and Basal Ganglia Research Unit, CIC-IT, CIC1414, Pontchaillou University Hospital and University of Rennes, Rennes, France; France Développement Electronique (FDE), Monswiller, France; Institut des Neurosciences Cliniques de Rennes (INCR), Rennes, France

Correction needed for raw affiliation Behavior and Basal Ganglia Research Unit, CIC-IT, CIC1414, Pontchaillou University Hospital and University of Rennes, Rennes, France; France Développement Electronique (FDE), Monswiller, France; Institut des Neurosciences Cliniques de Rennes (INCR), Rennes, France
raw_affiliation_name: Behavior and Basal Ganglia Research Unit, CIC-IT, CIC1414, Pontchaillou University Hospital and University of Rennes, Rennes, France; France Développement Electronique (FDE), Monswiller, France; Institut des Neurosciences Cliniques de Rennes (INCR), Rennes, France
new_rors: 015m7wh34;01cmnv018;02baj6743
previous_rors: 015m7wh34;01cmnv018
works_examples: W4383998218
contact: 7f73714afb2daec1ed74b3265a1466bc:fcf977ea6e23ea18b9afb15dcc64bd4e @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Localization (English and Brazilian Portuguese language packs are included)

Wow, great pull request @andersondeoliveiramachado ! We will review it this week and merge it if all good.

Looking forward to the merge. Will translate it to german, after merging is done :) :+1:

I hope !


Word Counter Project

A simple JavaScript word counter project that will count the number of words, letters, and whitespaces present in a block of text.
I will be working on this project under GSSoC'22
Shubham Singh

Go for it @LiQuiD-404 👍


Support for batch of bbox

Hi,
It seems like the evaluation code does a for loop over all the N bbox and performs one predict call per bbox instead of doing a batch call to predict all masks from all bboxes at once.
eval_efficientvit_sam_model.py
for ann in anns:
            if ann["area"] < 1:
                continue

            sam_mask = ann_to_mask(ann, sam_image.shape[0], sam_image.shape[1])

            bbox = np.array(bbox_xywh_to_xyxy(ann["bbox"]))
            pre_mask = predict_mask_from_box(predictor, bbox)

Was this done on purpose?
From my understanding this makes EfficientViT-SAM much slower than other SAM implementations if we have more than 1 bbox.
Thanks

I just found out you can actually do :
transformed_bboxes = efficientvit_sam_predictor.apply_boxes(bbox_xyxy_unnormalized)
box_torch = torch.as_tensor(transformed_bboxes, dtype=torch.float, device=device)
masks, _, _ = efficientvit_sam_predictor.predict_torch(
        point_coords=None,
        point_labels=None,
        boxes=box_torch,
        multimask_output=False
)


Transform Framework

Introduce the transform framework, that will be used in the add mechanics in the topology class.
For this provide simple transforms :
Transforms to treat periodic boundary cases :  (including possible merging)

[ ] Period in x direction only (width depency)
[ ] Periodic in y direction      (height dependency)
[ ] Perdiodic X Y behavior (concatenation of x and y)

General transforms:

[ ] Rotations
[ ] Affine mappings
[ ] Diffeomorphism (function interface)

Class Periodic now involves all Periodic X,Y, XY in a single concepts.
Class Diffeomorphism involves Rotation, Affine mappings.


not recognized as an internal or external command, operable program or batch file.

i dont know how to fix this :(
https://gyazo.com/a854a4242ad96c0b012bea646668c811

You have to enter the quality together with the original command.
For example: livestreamer twitch.tv/kildemon best

what if we are doing it as stated above and it is still spitting out that error?


External Server Upload limit

Describe the bug
Usage of internet upload in external server is stuck on 50% causing network hitches and laggy voice in game
Expected behavior
Usage of 100% upload limit - mine is 500mb/s
Server Info:

Server Version: 5004
Client Version: canary
Client Build: b2372]
Game: FiveM

Additional context
screen
https://cdn.discordapp.com/attachments/814088945419812865/932043993963986994/unknown.png

Doesn't sound related to pma-voice, we don't handle any of that 🤔

Ok but how i can manage to rise upload if i have 500mb/s upload in speedtest and pma is using max 250?


Don't see the template ID of template name : Playlist

Dear Expert,
I need turn-off all template lists, but i can't see the ID template Playlist. And i don't know why i can't disable template Expense Tracker , although command ran it's okey.
Set-SPOTenant -DisableModernListTemplateIds '96D6DBE5-D7C3-430A-867A-0B72EB4065AB'
So please give me your solution to fix it. Thanks you!

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

ID: cf2d4570-0024-e5c0-035a-8827759aeabc
Version Independent ID: 42e2105b-5ede-3d22-139d-96cc61cf4ec5
Content: Control Microsoft Lists - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/control-lists.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Hello @ZICZAC1989
Looks like the correct id for Expense Tracker is '96d6dbe5-d7c3-030a-867a-0b72eb0ff1ce'
and for playlist '3a867b4a-7429-0e1a-b02e-bf4b240ff1ce'
I'll update the article

Dear,
Thank you so much!
Kind Regards,
@.***
From: Andres Mariano Gorzelany @.>
Sent: Tuesday, May 23, 2023 2:09 AM
To: MicrosoftDocs/OfficeDocs-SharePoint @.>
Cc: Cong Duong @.>; Mention @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SharePoint] Don't see the template ID of template name : Playlist (Issue #3840)
Hello @ZICZAC1989https://github.com/ZICZAC1989
Looks like the correct id for Expense Tracker is '96d6dbe5-d7c3-030a-867a-0b72eb0ff1ce'
and for playlist '3a867b4a-7429-0e1a-b02e-bf4b240ff1ce'
I'll update the article
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/3840#issuecomment-1557778520, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATPAYC5OFCIDGMHQPYDEYJDXHO2TLANCNFSM6AAAAAAYDLGDSQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Dear,
I hoping you are doing well,
You know when I try used 2 templated Id that you provide me, it’s okey with platform Web.
But I trying to use on Mobile, all template Id from MS still showing up. I don’t know why.
So please help me check a fix it.
Thank you so much!


[GHSA-w5m3-xh75-mp55] Vega has Cross-site Scripting vulnerability in lassoAppend function

Updates

Description

Comments
Remove references to unrelated product.

Hi there @azasypkin and @jheer! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.
This change will be reviewed by our highly-trained Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

UPDATE: We've updated advisory listed in Vega project repository with the changes proposed in the PR, would be great to reflect these changes in GitHub advisory as soon as possible as well. Thanks!


[BUG] Warning (fcitx.el): `fcitx-remote' is not avaiable.

After I start emacs.app from Alfred (MacOS), it shows the information below:

Warning (fcitx.el): `fcitx-remote' is not avaiable. Please check your fcitx installtion.

The fcitx.el was included in packages.el by (package! fcitx), and fcitx-remote-for-osx worked fine before. This warning information has been shown after updating. As a result, the function of fcitx-remote cannot be used.
But if I open emacs from terminal, there is no warning. I wonder if there is something wrong in doom emacs or my local configuration. I use zsh by the way.

System information

((emacs
  (version . "26.3")
  (features . "NOTIFY ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS NS MODULES THREADS")
  (build . "Sep 02, 2019")
  (buildopts "--with-ns '--enable-locallisppath=/Library/Application Support/Emacs/${version}/site-lisp:/Library/Application Support/Emacs/site-lisp' --with-modules"))
 (doom
  (version . "2.0.9")
  (build . "HEAD -> develop, origin/develop, origin/HEAD ac5168f8 2019-09-06 01:32:20 -0400"))
 (system
  (type . darwin)
  (config . "x86_64-apple-darwin18.2.0")
  (shell . "/bin/zsh")
  (uname . "Darwin 18.7.0 Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT 2019; root:xnu-4903.271.2~2/RELEASE_X86_64 x86_64")
  (path "/usr/local/opt/ruby/bin" "/usr/local/lib/ruby/gems/2.6.0/bin" "/usr/local/anaconda3/bin" "/usr/local/bin" "/usr/bin" "/bin" "/usr/sbin" "/sbin" "/Library/TeX/texbin" "/opt/X11/bin" "/Applications/Emacs.app/Contents/MacOS/bin-x86_64-10_14" "/Applications/Emacs.app/Contents/MacOS/libexec-x86_64-10_14" "/Applications/Emacs.app/Contents/MacOS/libexec" "/Applications/Emacs.app/Contents/MacOS/bin"))
 (config
  (envfile . envvar-file)
  (elc-files .  You can do so by running doom refresh (or doom env).

Have you refreshed your envvar file since installing fcitx-remote? You can do so by running doom refresh (or doom env).

Thanks for your reply.
Yes, I've done doom refresh every time after doom update. And I have installed fcitx-remote before, it worked fine with doom until the latest updating.
By the way, after recent updates, the highlights for modified lines of git files won't be shown correctly until I save the file twice. After saving the first time, the highlight bar on the left side of emacs won't change. I'm not sure is it worth to open another issue.

Have you refreshed your envvar file since installing fcitx-remote? You can do so by running doom refresh (or doom env).

Thanks for your reply.
Yes, I've done doom refresh every time after doom update. And I have installed fcitx-remote before, it worked fine with doom until the latest updating.
By the way, after recent updates, the highlights for modified lines of git files won't be shown correctly until I save the file twice. After saving the first time, the highlight bar on the left side of emacs won't change. I'm not sure is it worth to open another issue.

You can try to run doom env alone before you runing doom refresh.I found that sometimes the envvar file will be lost after doom update. At this time, only executing doom refresh will not generate a new envvar file. You must first execute doom env.

Thanks, it does happen what you said. And after updating doom today, the warning won't show up. However, the highlight of git still needs the second save to show. Fortunately, it won't bother too much.

Two days ago, in 280a2407a, I published a fix for doom env that may have caused your issue. For the time being, I'll consider the original issue resolved. Let me know if that isn't the case.
As for the double-save issue for git-gutter, I can't reproduce this. Can you reproduce it in vanilla Doom (see how to use the sandbox)? If so, please create a new issue for it.

Thanks hlissner, this issue was resolved.
As for the double-save issue for git-gutter, I'll try to reproduce it in vanilla Doom, and create another issue.


More linting

Make all crates use the same linting rules, and fixed a few.
Note that I fixed extra_unused_lifetimes for unsafe extern "C" fn chunk_collector(...) -- which looked a bit suspicious, but seems to be ok.  Need CI tests! :)

looks like it broke the build

thx, yeppii - we have working CI :)


Fix MCMC sampler in documentation

This is a fix for HMC sampler used in Example 3 of the MCMC documentation notebook. Currently, the predictive performance of the GPMC object is poor. This is likely due to the bad mixing of the HMC chain, in particular, those of the kernel lengthscale. By running the sampler for longer and increasing the length of the burn-in phase, it does appear that the problem can be rectified as good predictive performance is re-established, and good mixing is present in the chain.
I've also added two requirements to the list needed to locally compile documentation. After doing a clean install of GPFlow and the previously listed set of requirements, jupytext,  jupyter_client, and ipywidgets were still required to compile the notebooks.
Minimal working example
This is a very small PR, and all that has been changed is the hyperlinking to subsections within the MCMC example notebook, along with the number of samples and burn-in length used in the exponential likelihood example. An example would be:
-num_burnin_steps = ci_niter(300)
-num_samples = ci_niter(500)
+num_burnin_steps = ci_niter(600)
+num_samples = ci_niter(1000)

Increasing the number of data points to 100 does resolve this issue, although it is obviously now slower. I guess there are two possible resolutions to this issue then: increase the number of data points, or keep the data small and run the sampler for longer.


Issues with Safari (iOS and Mac) when in Private Browsing mode

It seems Private Browsing makes localStorage read only:
http://spin.atomicobject.com/2013/01/23/ios-private-browsing-localstorage/
When loading store.js, I get this error:
QUOTA_EXCEEDED_ERR: DOM Exception 22: An attempt was made to add something to storage that exceeded the quota.
I saw these lines in the lib:
try {
  store.set(namespace, namespace)
  if (store.get(namespace) != namespace) { store.disabled = true }
  store.remove(namespace)
} catch(e) {
  store.disabled = true
}

Maybe there's something to improve there…

@marcuswestin I see on the README and in this issue that basically you are recommending devs to ask users to switch off Private Browsing mode, otherwise store.js will not function.
May I recommend my project, memorystorage as an alternative to giving up? It's basically a polyfill for localStorage that's backed by regular memory. Sure, the data won't survive a browser reload, but there were never any guarantees anyway and at least the app will mostly work.
I'm currently using Rhaboo in my project and it too failed in Private Browsing mode, so I hacked on it to get it fixed and ended up developing MemoryStorage as a fallback and it's been working great for me.

Store.js v2.0 has been released with support for Safari private mode.
If you update to the latest version using the v1 backcompatible build you will automatically get this functionality plus a bunch more :)


Disable 'use strict' to allow setting globals (clone)

Ive signed the Licence ... dont know why it hasn't updated ....


[Bug Report] color selector when the default value is white or white color value, and then click on the right side of the bar transform color type, no effect

Element UI version
1.3.4
OS/Browsers version
mac10.11.6    chrome55
Vue version
2.1.4
Reproduction Link
https://jsfiddle.net/pj71jkyw/246/
Steps to reproduce
打开https://jsfiddle.net/pj71jkyw/246/
点击打开颜色选择器
点击右侧变换颜色分类
What is Expected?
颜色选择框变换成其他颜色区间
What is actually happening?
没有变化

Looking into it.
Seems need to distinguish internal states from picker's value.

One quick fix:
watch color's "internal" _{hue, saturation, value, alpha} props
As far as current implementation goes, inputs are converted to HSV format, then HSV values are used to draw panels, so I don't think it's necessary to keep these props "internal" (and use one extra layer of getter/setter)
What do you think? @Leopoldthecoder

@wacky6 I'll take a look this weekend😂

@wacky6 It seems @Kingwl has helped fix this in #5184.

@Kingwl's fix should be ok.
Though I would still suggest that we improve color-picker afterwards (or as a long-term goal), since:

calling update() twice isn't very elegant, though it won't causes problems due to how Vue works
_hue feels like an internal prop, and should not be used outside of color.js

Maybe we could learn from React's one-way down design, reduce the use of watch/reactivity to produce an elegant (or "pure/functional" by React's ideas) solution?

@wacky6 I agree. Maybe some other time when we can have more energy on Element🌚.
Fixed in #5184


Tiktoken not published to cargo

It seems that the tiktoken package is not linkable from Rust using Cargo's default registry.
Are there plans to publish the tiktoken crate? Is it published on another registry?
Thanks for your work on this BPE encoder, I've already found it very useful!

Repro:
In a rust project, run
cargo add tiktoken

Expected behavior:
Cargo should find, download and add tiktoken to the available crates
Actual behavior:
$ cargo add tiktoken
    Updating crates.io index
error: the crate `tiktoken` could not be found in registry index.

In case helpful: https://github.com/dust-tt/dust/tree/main/core/src/providers/tiktoken

Very cool @spolu! I'd love to package this code as a separate crate for re-use in different rust projects.

For testing this out in other projects, I created and published a rust crate here: https://github.com/zurawiki/tiktoken-rs
Ideally, I hope we can integrate these changes back into the original project, so I'll leave this Issue open until we hear from a maintainer.

Nice!!

Thanks, I'm open to this, I just haven't spent the time to figure out Rust packaging yet :-)
I will get around to this at some point, thanks for the link to your repo!

Can you make also an alternative, pure python version of Tiktoken?
For those who cannot compile and run Rust binaries on their system (for various reasons: package managers support, company policy, intranet or local machine security, docking containers limitations, vm restrictions, environment virtualization, lack of Rust support in jupyter notebooks remote hosting, etc).

This is not my area of expertise, but if I may have a suggestion -
You can make a cargo workspace, create a tiktoken-lib or a tiktoken-core rust project, and then import it within the current lib.rs. That way it is housed within this repository itself.
https://crates.io/crates/cargo-workspaces is a helper which can allow you to publish individual projects within a workspace. I haven't used it myself though.

Can anyone figure out how to replace the python threading with rayon threading? On lines 140-141 of lib.rs there is a comment where the author mentions he tried threading with rayon but noticed it wasn't much faster than python threads.
I am still learning Rust so I am having a hard time with this...

Hi,
A question, why the mergeable_ranks is downloaded in runtime? why not to have it downloaded in the repo?
def gpt2():
    mergeable_ranks = data_gym_to_mergeable_bpe_ranks(
        vocab_bpe_file="https://openaipublic.blob.core.windows.net/gpt-2/encodings/main/vocab.bpe",
        encoder_json_file="https://openaipublic.blob.core.windows.net/gpt-2/encodings/main/encoder.json",
    )
    return {
        "name": "gpt2",
        "explicit_n_vocab": 50257,
        "pat_str": r"""'s|'t|'re|'ve|'m|'ll|'d| ?\p{L}+| ?\p{N}+| ?[^\s\p{L}\p{N}]+|\s+(?!\S)|\s+""",
        "mergeable_ranks": mergeable_ranks,
        "special_tokens": {"<|endoftext|>": 50256},
    }

Isn't this a waste of time in runtime? this should not change, and if it changes, it would no longer be that version totally valid for gtp2 or at least not the one with which the library was tested at the time, maybe have another more current version, tested and the other keep it but deprecated?


Mac restarts continuously with Flutter App

I am using IntelliJ for development. When I work on that for Flutter development it continuously get crashing my Mac machine.
I am using following version of Flutter:
chandra@Chandras-MacBook-Pro ios % flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.0.5, on macOS 11.5 20G71 darwin-x64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
[✓] Xcode - develop for iOS and macOS
[✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[✓] Android Studio (version 4.2)
[✓] IntelliJ IDEA Ultimate Edition (version 2020.2.4)
[✓] VS Code (version 1.58.2)
[✓] Connected device (1 available)
! Doctor found issues in 1 category.
Here are further Crash log from Mac:
Process:               watchman [946]
Path:                  /usr/local/Cellar/watchman/4.9.0/libexec/bin/watchman
Identifier:            watchman
Version:               0
Code Type:             X86-64 (Native)
Parent Process:        ??? [1]
Responsible:           watchman [946]
User ID:               501
Date/Time:             2021-08-04 11:52:02.176 +0530
OS Version:            macOS 11.5 (20G71)
Report Version:        12
Anonymous UUID:        DA1381AA-B996-D4EE-28A6-3EE885872785
Time Awake Since Boot: 400 seconds
System Integrity Protection: enabled
Crashed Thread:        0
Exception Type:        EXC_CRASH (SIGABRT)
Exception Codes:       0x0000000000000000, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY
Termination Reason:    DYLD, [0x1] Library missing
Application Specific Information:
dyld: launch, loading dependent libraries
Dyld Error Message:
dyld: Using shared cache: F2DC58FA-E869-380A-8C1E-53D831FB3F9E
Library not loaded: /usr/local/opt/openssl/lib/libcrypto.1.0.0.dylib
Referenced from: /usr/local/Cellar/watchman/4.9.0/libexec/bin/watchman
Reason: image not found
Binary Images:
0x103db1000 -        0x103e6afff +watchman (0)  /usr/local/Cellar/watchman/4.9.0/libexec/bin/watchman
0x108c74000 -        0x108d0ffff  dyld (852.2) <57DB2053-BFD5-3683-97C6-F1DB2A1F1D09> /usr/lib/dyld

if VsCode and Android Studio work well maybe is an IntelliJ issue.

It might not related on IntelliJ issue and definitely not flutter issue i think.
It looks like problem on watchman installation using homebrew cannot find libcrypto.
You could try to install libcrypto. Maybe it will solve it.
Watchman is used to detect change of file. I think i remember some setting on IntelliJ on that, we could use other detection app.

Hi @ChandraPrakashJangid

Here are further Crash log from Mac:
Process: watchman [946]
Path: /usr/local/Cellar/watchman/4.9.0/libexec/bin/watchman

This crash is not related to Flutter or Flutter IntelliJ plugin
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Review of main.cf and master.cf by a Postfix expert

It works, but can be improved.
If a Postfix expert could have a look, it would be nice.
Thank you.

Tom, in response. Your main.cf file works well but there is a more robust method to assign configuration settings. Using postconf -e you will be able to assign the needed variables. Also you don't need to manage the file location and placement of said file directly in the build. Your main.cf can be removed and the needed config can be directly applied within start-mailserver.sh
Test this:
postconf -e smtpd_relay_restrictions=permit_mynetworks, permit_sasl_authenticated, defer_unauth_destination

I have been building and continuing to improve a Docker Postfix Project: https://github.com/htmlgraphic/Postfix

Hope this was helpful. :+1:


Broadcast not work

I'm trying a application with spring-cloud-stream. The application has two consumers, one is used to receive broadcast messages(all instances receive the same message), and the other one is not(only one instance receive messages in the same consumer group.)
I have two instances to run the application and the application.yml files are as below
# for instance 1
spring:
  cloud:
    stream:
      kafka:
        binder:
          brokers: myhost01:4443,myhost02:4443,myhost03:4443
          zkNodes: myhost01:2181,myhost02:2181,myhost03:2181
          autoCreateTopics: true
          autoAddPartitions: true
      bindings:
        channel1:
          consumer:
            startOffset: earliest
            partitioned: true
            instanceIndex: 0
            instanceCount: 2
          destination: topic1
          group: topic1-group
        channel2:
          consumer:
            startOffset: earliest
            instanceIndex: 0
            instanceCount: 1
          destination: topic2
          group: topic2-ins1

# for instance 2
spring:
  cloud:
    stream:
      kafka:
        binder:
          brokers: myhost01:4443,myhost02:4443,myhost03:4443
          zkNodes: myhost01:2181,myhost02:2181,myhost03:2181
          autoCreateTopics: true
          autoAddPartitions: true
      bindings:
        channel1:
          consumer:
            startOffset: earliest
            partitioned: true
            instanceIndex: 1
            instanceCount: 2
          destination: topic1
          group: topic1-group
        channel2:
          consumer:
            startOffset: earliest
            instanceIndex: 0
            instanceCount: 1
          destination: topic2
          group: topic2-ins2

The consumers binded to channel1 works well on two instances (only one consumer receive messages one time).
But conumsers binded to channel2 seemed not. I want the messages sent to topic2 broadcasted to all the consumers, but only one received.
Did I miss any other configurations?
kafka version: <IP_ADDRESS>
dependency in my pom
            <dependency>
                <groupId>org.springframework.cloud</groupId>
                <artifactId>spring-cloud-stream-dependencies</artifactId>
                <version>1.0.2.RELEASE</version>
                <type>pom</type>
                <scope>import</scope>
            </dependency>

Thanks!!

hi @beenryuu,
This has only been fixed since v/ 1.1.0.RELEASE and is an open issue v 1.0.2.RELEASE .Use Spring Cloud Camden or Spring Cloud Stream Dependency Train Brooklyn.RELEASE.
See https://github.com/spring-cloud/spring-cloud-stream/issues/654 for details.

@mbogoevici
Thanks for your advice.

Use Spring Cloud Camden or Spring Cloud Stream Dependency Train Brooklyn.RELEASE.

It works just like I've imaged.
Thanks a lot. You've been a big help.

Thanks Bin. Closing this.


Renderer not working without d.ts files

Describe the bug
I noticed that the vite renderer doesnt works without the generated d.ts files. My problem is that vite always refreshes multiple times. One time for the file it self an one time for the d.ts file:
15:33:49 [vite] hmr update /src/components/Headbar.js, /src/main.scss
15:33:49 [vite] page reload src/components/Headbar.d.ts

Its really annoying in developement, how can I get around this. When I use the plain vite for my frontend, I dont have this issue.
Electron-Vite Version
2.0.0
Electron Version
28.2.0
Vite Version
5.0.12
Validations

[X] Follow the Code of Conduct.
[X] Read the Contributing Guidelines.
[x] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

My fault, can be ignored


fix --no-cache support

This change disables prestissimo when --no-cache flag is passed.
Regex is taken from https://github.com/composer/composer/blob/f154d5c09d69a6338d6a955bf51cc264d0448b3d/src/Composer/Cache.php#L62-L65
fixes #199

Coverage decreased (-0.1%) to 80.145% when pulling 9673bc28e8bcb6905184e28452bd91615c9de555 on IonBazan:bugfix/no-cache into fb6724d5ee81c96bebd2255175d8c364ac9301e3 on hirak:master.

@kubawerlos Tests added :)


fix highlight error of chainer/functions/array/reshape.py

related to #3242.
The example code of reshape is not highlighted. It may be occurred by the parse error.
This page said following: http://www.sphinx-doc.org/en/stable/markup/code.html

Within Python highlighting mode, interactive sessions are recognized automatically and highlighted appropriately. Normal Python code is only highlighted if it is parseable (so you can use Python as the default, but interspersed snippets of shell commands or other code blocks will not be highlighted as Python).

And this PR, I just remove the some strings in order not to make parse error, just delete the line chainer.utils.type_check.InvalidType:.

some error in AppVeyor build

Jenkins, test this please

LGTM


introduce env variables for common commandline parameters

it would be useful to be possible to give command line parameters as env variables
this would make the command line shorter.
if command line is specified it should override env var and default settings:
default->env-var->command line (override sequence)
in case of enable/disable checker list: the default should be extended by either the env var or the command line param
these params should have env var equivalent:
-workspace
-dbport
-dbhost
-dbuser
-dbname
-enable/disable checker list
-viewport
-suppress file path
if the env var is speficied then the CodeChecker script should list their values when started.

It might be just my preference, but I do not like when an application relies on many environment variable. Wouldn't it be more user friendly to have a config file and one environment variable which is the path to the config? (or define a heuritic to find the config file)

Configuring environment variables is only an option, not required. The same settings can be configured in the command line right now. The environment variables can be a flexible option for different build environments.
You can always write a simple setup script for your environment variables or command line options.

Implicitly reading env vars could result frustrating debugging situations, especially when there are so many of them. Even with a switch (like '--env-var-conf' ) users may unintentionally use env vars that were set the week before (with tmux, screen it is common to have terminal sessions as old as the uptime).
I think users, whom you aim this feature for, are quite happy with their current solution:
alias mycheck='Codechecker check --dbport 5432 --dbusername jack --dbname ccdb -l myproject/log'
mycheck -n third_check_of_myproject

It is flexible, simpler than setting an env var for each config option, and doesn't limit you to one config per terminal session.

We finally decided to use config files and a single environment variable CODECHECKER_CONFIG containing the location of the config file.
by default CodeChecker will take the config files from ~/.codechecker.conf
if not found it will take it from $CODECHECKER_CONFIG
The config file will contain paramaters as key value pairs in json format
if any parameter is defined in the command line it overrides the config file.

by default CodeChecker will take the config files from ~/.codechecker.conf
if not found it will take it from $CODECHECKER_CONFIG

Why not $CODECHECKER_CONFIG by default, if the variable is unset then ~/.codechecker.conf?

makes sense. So from $CODECHECKER_CONFIG by default and if not set ~/.codechecker.conf.

This issue is not relevant any more, because we don't have such a config file.


Java Syntax Highlighting Inferior in macOS than Windows

Notice the syntax highlighting for the same file is better on Windows than Mac. Can this be fixed?

OS: Windows 10 Enterprise; VSCode: Version 1.15.1

OS: macOS Sierra 10.12.6; VSCode: Version 1.15.1

Reproduces without extensions: Yes

@janjulian It looks like on the Mac you are using the Dark (Visual Studio) color theme, but on Windows Dark+.

Consider this fixed thanks to @aeschli!


Replace deprecated apis from scipy.misc with their official substitutions; Add align_corners=False to get rid of a warning.

Every use of scipy.misc.imresize and scipy.misc.imrotate from lib/utils/transform.py is replaced by their official recommended substitutions:

https://docs.scipy.org/doc/scipy-1.2.1/reference/generated/scipy.misc.imresize.html:

imresize is deprecated! imresize is deprecated in SciPy 1.0.0, and will be removed in 1.3.0. Use Pillow instead: numpy.array(Image.fromarray(arr).resize()).

https://docs.scipy.org/doc/scipy-1.1.0/reference/generated/scipy.misc.imrotate.html:

imrotate is deprecated! imrotate is deprecated in SciPy 1.0.0, and will be removed in 1.2.0. Use skimage.transform.rotate instead.

Therefore, add scikit-image and pillow to requirements.txt to use the new apis.

Add align_corners=False to get rid of the warning from torch.nn.functional.interpolate since torch>=0.4.0.

I was hit by this bug https://github.com/scikit-image/scikit-image/issues/4241#issuecomment-542162591
I had to import transform explicitly using "import skimage.transform"


Fix Log Replication Deletions

Explain the changes
This PR ensures that even if multiple actions are performed on an object, we process all of the actions instead of processing only the last one. Processing only the last action causes issues in the scenario when the logs are not chronologically ordered, for example: DELETE /cat.jpeg then PUT /cat.jpeg, when these 2 entries are part of the same candidate batch, NooBaa would just process the last PUT which means that the DELETE will never be processed.
PS: Added JS Docs to have the static analysis "safety net".
Issues: Fixed #xxx / Gap #xxx
This issue was discovered by Sagi (QE) during happy path testing.

[ ] Doc added/updated
[ ] Tests added

@utkarsh-pro https://github.com/noobaa/noobaa-core/pull/7235 is this PR will verify these changes?


Confused about the "increOdomAffine" in mapOptmization.cpp

Hi, Tixiao Shan:
Thanks for sharing your code. It's very useful.
During learning your paper and code, I'm wonder the meaning of the code below.
https://github.com/TixiaoShan/LIO-SAM/blob/9991daa47eb76124659570daab299bf877dde8bb/src/mapOptmization.cpp#L1663-L1664
In my view, the "increOdomAffine" and the "incrementalOdometryAffineFront" are the same before program go through the line 1663 and 1664.
At the first time code reach the line 1663, the "increOdomAffine" is equal to the previous frame's "transformTobeMapped", at the same time, the "incrementalOdometryAffineFront" is on record as previous frame's "transformTobeMapped".
https://github.com/TixiaoShan/LIO-SAM/blob/9991daa47eb76124659570daab299bf877dde8bb/src/mapOptmization.cpp#L788-L789
And also, we can simplify the code at line 1663 - 1664 like:
increOdomAffine = increOdomAffine * incrementalOdometryAffineFront.inverse() * incrementalOdometryAffineBack;
According to previous analysis, the "increOdomAffine" is equal to the "incrementalOdometryAffineFront", so we can write as:
increOdomAffine = incrementalOdometryAffineFront * incrementalOdometryAffineFront.inverse() * incrementalOdometryAffineBack;
As all known the "incrementalOdometryAffineFront * incrementalOdometryAffineFront.inverse()" is identity. So the
"increOdomAffine" is equal to the "incrementalOdometryAffineBack" which is equal to the "transformTobeMapped" that computed at the scan2MapOptimization function.
So, I think we can just simplify the code at line 1663 - 1664 to:
increOdomAffine = trans2Affine3f(transformTobeMapped);
Previous code will bring some error of calculation and this will be more clear and more accurate, isn't it?
Thanks in advantage and kind regards,
Liuyang Li

@Lurvelly
increOdomAffine = trans2Affine3f(transformTobeMapped)
is totally different from
Eigen::Affine3f affineIncre = incrementalOdometryAffineFront.inverse() * incrementalOdometryAffineBack;
increOdomAffine = increOdomAffine * affineIncre;

Estimation of biases of imu need lidar odometry to help constraint it. However, pose graph optimization using GPS would cause transformTobeMapped to jump hugely and suddenly, which will cause failure of estimation of biases of imu. So it needs to use a version of lidar odometry without being affected by pose graph optimization, that is the laserOdomIncremental.

@JaySlamer
Dear Jay,
Thanks for your reply.
Yes, you are right.
That's my mistake, I haven't noticed that pose graph optimization would change the transformTobeMapped, so I was simple enough to think this question.
I will close this issue, thanks again.

@Lurvelly increOdomAffine = trans2Affine3f(transformTobeMapped) is totally different from
Eigen::Affine3f affineIncre = incrementalOdometryAffineFront.inverse() * incrementalOdometryAffineBack;
increOdomAffine = increOdomAffine * affineIncre;

Estimation of biases of imu need lidar odometry to help constraint it. However, pose graph optimization using GPS would cause transformTobeMapped to jump hugely and suddenly, which will cause failure of estimation of biases of imu. So it needs to use a version of lidar odometry without being affected by pose graph optimization, that is the laserOdomIncremental.


I download  "GHOST " it is an error

Tips:  Error: Status 403 trying to pull repository library/ghost: "Account is not Active"

Please login to http://hub.docker.com and activate your account.


Update Dependencies

build: upgrade build-tools to v0.9.0
build: upgrade pre-commit-hooks to v0.2.2
build: trigger ci pipeline on build and pull_request events
chore: prepare v0.5.1 release

closed in favor of #36


Service constructor default value not allowed

Informations

Type
Version

Bug
3.10.x

Description
Cannot define param with default value in constructor of a Service. E.g:
@Service() class MyService {  constructor(myValue = 'default value'){ ... } }
This fails because the injector service tries to inject the myValue as another service instance. Is it possible that the injector sees those params that have default values and skip them?

It's strange... When you declare a class as a Service, it mean you want to delegate the class building to the InjectorService and use the Singleton pattern. And you will never build your class from another class (excepted in the unit test maybe).
In which case would you need this?

I have service hierarchy that needs some other details during construction of the instances. E.g. I have a BaseService that sets-up basic CRUD ops for Base model class. Then I have an ItemService extends BaseService. The Item extends Base. This service add bit more specific store ops for the Items. The Item has subclasses, Document, Layout and Widget. This is a REST for a component-based solution I am building. The Item has discriminator prop "type" that is persisted in nosql, but when I map back I am instantiating the correct subclass based on the type prop.
Because I have only single collection of Items with their "type" I have created subservices for Document, Layout and Widget models, that is.. DocumentService, LayoutService, WidgetService respectively. In the ItemService I extend the BaseService find (Read) method and add additional query params so that I filter only the types of Items I need. For e.g. if I execute the find from the ItemService it will execute this query:
{ type: { $in: types} } where "types" is protected types: string[] = ['document', 'layout', 'widget'] in the ItemService constructor.
If I execute the find from the DocumentService, I need to scope down the types to only 'document', so through the constructor of the DocumentService I have this:
`
@Service()
ItemService extends BaseService {
constructor(protected types: ['document', 'layout', 'widget']) { ... }
}
find(query?) {
return this.collection({... query, { type: { $in: this.types} }});
}
`
@Service() DocumentService extends ItemService<Document> { constructor() {  super(['document']); } }
Now if I execute find from the ItemService instance I will get all the items with types 'document', 'layout', 'widget'. But if I execute the find from the DocumentService instance I will get only items with type 'document'.
I hope you understand the use case?
Thanks!

I have found a workaround for this issue but I think it would be helpful if it's possible the InjectorService to skip props with default values if it cannot find the appropriate instance to be inejected in the registry. The current workaround I have is to add additional method init() in the service hierarchy which internally sets the types prop, and override this in the other services to redefine the type prop:
class ItemService {
protected types: string[];
constructor() {
...
this.init();
}
init() {
this.types = ['document', 'layout', 'widget'];
}
}
class DocumentService extends ItemService{
init() {
this.types = ['document'];
}
}
This works but is a bit more code for the job.

Well, You can write this, without mode code:
@Service()
class ItemService {
    protected types: string[] = ['document', 'layout', 'widget'];
    constructor() {}
}

@Service()
class DocumentService extends ItemService{
    protected types: string[] = ['document']
}

Then to answer you, default value isn't detectable. Only the type of the parameters is given. And the purpose of the Injector is to emit an exception when the Service is unknown. Change this rule is not possible. Instead of, use a decorator is possible to avoid this behavior.
function SkipInject(target: Type<T>, targetKey: string, descriptor: TypedPropertyDescriptor<Function> | number) {
      const paramTypes = Metadata.getParamTypes(target, targetKey);
      paramTypes[descriptor] = SkipInject;
     Metadata.setParamTypes(target, targetKey, paramTypes);
}

Then patch the InjectorService.ts L245:

    static invoke<T>(target: any, locals: Map<string | Function, any> = new Map<Function, any>(), designParamTypes?: any[], requiredScope: boolean = false): T {

        if (!designParamTypes) {
            designParamTypes = Metadata.getParamTypes(target);
        }

        const parentScope = Store.from(target).get("scope");

        const services = designParamTypes
            .map((serviceType: any) => {

                if (serviceType === SkipInject) {
                     return undefined;
                }

                const serviceName = typeof serviceType === "function" ? nameOf(serviceType) : serviceType;

                /* istanbul ignore next */
                if (locals.has(serviceName)) {
                    return locals.get(serviceName);
                }
               // ....
          })
       //...
   }

You can try this solution and send me PR, if it works. (but I prefer the first the solution :p).
See you,
Romain

@Romakita actually your solution is much more elegant, I did complex it a bit myself. I guess it's not needed to patch with SkipInject because it will create more confusion. It should be only used in conjunction with default value params, if omitted it will create issues.
Thanks, you can close this ticket!


Support nested arrays

[x] lowercasesIn() nested arrays support.
[ ] duplicatesIn() nested arrays support.
[ ] singleWordsIn() nested arrays support.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (c5a2bf7) to head (4d63473).

Additional details and impacted files
@@             Coverage Diff             @@
##                main       #12   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity        18        19    +1
===========================================
  Files              4         4
  Lines             68        71    +3
===========================================
+ Hits              68        71    +3

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


python312Packages.httpie: 3.2.3 -> 3.2.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.httpie is: Command line HTTP client whose goal is to make CLI human-friendly
meta.homepage for python312Packages.httpie is: https://httpie.org/
meta.changelog for python312Packages.httpie is: https://github.com/httpie/httpie/blob/3.2.4/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.2.4 with grep in /nix/store/qh4s8f6pkg9248c9gj4myz81c128if5b-python3.12-httpie-3.2.4
found 3.2.4 in filename of file in /nix/store/qh4s8f6pkg9248c9gj4myz81c128if5b-python3.12-httpie-3.2.4

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

7 total rebuild path(s)

7 package rebuild(s)

First fifty rebuilds by attrpath

http-prompt
httpie
python311Packages.httpie
python311Packages.httpie-ntlm
python312Packages.httpie
python312Packages.httpie-ntlm

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/c01621586cb27a845bc6a41f856037588bdab894#python312Packages.httpie

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qh4s8f6pkg9248c9gj4myz81c128if5b-python3.12-httpie-3.2.4
ls -la /nix/store/qh4s8f6pkg9248c9gj4myz81c128if5b-python3.12-httpie-3.2.4/bin

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

x86_64-linux

:white_check_mark: 10 packages built:

http-prompt
http-prompt.dist
httpie (python312Packages.httpie)
httpie.dist (python312Packages.httpie.dist)
python311Packages.httpie
python311Packages.httpie-ntlm
python311Packages.httpie-ntlm.dist
python311Packages.httpie.dist
python312Packages.httpie-ntlm
python312Packages.httpie-ntlm.dist

Maintainer pings
cc @antono @relrod @schneefux for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

Review per https://github.com/NixOS/nixpkgs/tree/7ffd9ae656aec493492b44d0ddfb28e79a1ea25d/pkgs#package-updates
Reviewed points

[x] package name fits guidelines
[x] package version fits guidelines
[x] package builds on x86-64
[x] executables tested on x86-64 (not httpie program)
[ ] all depending packages build
[ ] patches have a comment describing either the upstream URL or a reason why the patch wasn't upstreamed
[ ] patches that are remotely available are fetched rather than vendored

Possible improvements
Repo name changed from httpie to cli, here is what http (built with this this pull request, after cherry-picking onto current nixos-unstable) says:
> result/http https://github.com/httpie/httpie
HTTP/1.1 301 Moved Permanently
[...]
Location: https://github.com/httpie/cli

That should probably be adapted in the package file (in src and in changelog), but that should not block the pull request at hand.
Comments
Works fine! I approve, can be merged!


net::Address::Inet may contain an IPv6 address

This seems in violation of the docs of this enum variant, which states...

IPv4 Internet protocols

...and also in violation of the design goal of having both an Inet and an Inet6 variant.
Personally, I think that net::Address::Inet should contain an Ipv4Addr and net::Address::Inet6 should contain an Ipv6Addr. The SocketAddr type that is currently put in those enum variants seems much more general than it needs to be.

Yeah, you are right, this seems incorrect. I agree that different enum variants should work better


Add Brazilian Portuguese translation

@Yogotami awesome thanks!

Wanted to ask you about the cardinal points. Those are not the standard names, are they? I saw in the code that it automatically gets the cardinal points and writes its abbreviation. Is the purpose of this .json just to change what the code writes? If it is, then we'll need to fix all the cardinal points from the translations.
ps: I'm not very savvy, just very interested.

@Yogotami i think they are standard for english, arent they? And yes the app use the abbreviation to show the moon position

According to this, some are different from the code (which cause a bit of confusion for me):

Ok i LL check thanks


Deploy multiple pipelines

The KPops deploy, destroy, delete, and clean should be able to take multiple pipeline files. Improves performance because files are loaded only once and helm repo commands are only run once

I gave this some thought and I believe this kind of functionality adds unnecessary complexity and doesn't follow the paradigm for scriptable CLIs.
Especially since it can be solved quite easily with existing tools
e.g.
fd pipeline.yaml pipelines/examples | xargs -L1 kpops generate

will run
kpops generate pipelines/examples/example1/pipeline.yaml
kpops generate pipelines/examples/example2/pipeline.yaml
kpops generate pipelines/examples/example3/pipeline.yaml
...

Especially in CI we already use matrix jobs to parallelize execution. Are there any other use cases where you would really need this?

I think there would be multiple advantages when allowing this:

caching of defaults can speed up execution
inter-pipeline dependencies. Currently you can only reference components from the same pipeline but referencing other pipelines would be easier to do. Especially wrt execution order and parallelization
this issue was actually brought back to the table because we are using matrix jobs. There were too many matrix jobs and GitHub actions is really bad at handling those. Therefore, related pipelines can be grouped by specifying a top-folder or a list of pipelines and the KPOps action can still be used without needing to write a custom bash wrapper

caching of defaults can speed up execution

That's true, however we're talking about shaving off a couple of ms at best. A deploy operation usually takes several minutes so that's pretty negligible. Now that defaults can be distributed, I'd argue it's even less of a factor, since only the shared defaults of multiple pipelines (i.e. top level) would be cached.
In terms of performance you would lose parallelism which is a much more important speed indicator imo.

inter-pipeline dependencies. Currently you can only reference components from the same pipeline but referencing other pipelines would be easier to do. Especially wrt execution order and parallelization

This is a separate feature and out of scope for this ticket. This is about passing multiple pipelines into a KPOps operation.

this issue was actually brought back to the table because we are using matrix jobs. There were too many matrix jobs and GitHub actions is really bad at handling those. Therefore, related pipelines can be grouped by specifying a top-folder or a list of pipelines and the KPOps action can still be used without needing to write a custom bash wrapper

Hm, I know that GitHub has some weaknesses and their CI is not fully where we'd want it to be. Are there any tickets about this on their issue tracker? It seems more of a UI/UX issue of their web frontend than a KPOps concern.


preserve scroll position without controller/QP?

Hi!
I have a pagination component for which I use QPs to track which page of pagination the user is on.
However, this means that this triggers scroll to top when paginating... so I took a look at the guides and found the preserveScrollPosition feature - however it seems that this requires access to the controller to enable/disable this feature.
I'm wondering how to preserve scroll position from my component without having to anticipate which routes the pagination may be used on and pre-set preserveScrollPosition on those route's controllers.
Seems that a service would be the ideal way to handle that... however, because it seems that service doesn't exist, I'm instead considering injecting the controller for the current route into the component and disabling preserve scroll when the pagination QPs are activated.
Any thoughts on other ways to achieve this or downsides to controller injecting?
Thanks!

Actually, it seems that injecting a controller into a component isn't a thing. But it also looks managing preserveScrollPosition through a service wouldn't be too difficult either.
I'll try to create a RouterScroll service PR over the next few days...


Acceptance make commands not consistent

The acceptance make commands are not consistent between the project root and backend.
root
https://github.com/plone/cookieplone-templates/blob/f513c962b1bd517460e727cdcea9cfc8c848bb1d/project/{{ cookiecutter.__folder_name }}/Makefile#L155-L164
As you can see, the command to start the backend from the root is acceptance-backend-dev-start, which is different from that while in the backend directory of acceptance-backend-start.
I can submit a PR to fix this inconsistency, by inserting -dev as needed, and updating documentation that refers to it.
However, we need to be careful about the context of the documentation, specifically whether you used Cookieplone or cloned Volto to contribute. In Volto, the backend command omits -dev. Maybe we should change these commands in Volto core as well?
https://github.com/plone/volto/blob/cfea4d988d25e41779b0742606bf43fa43d2bbee/packages/volto/Makefile#L116-L129
##### Acceptance tests (Cypress)
######### Dev mode Acceptance tests

.PHONY: acceptance-frontend-dev-start
acceptance-frontend-dev-start: ## Start acceptance frontend in development mode
	RAZZLE_API_PATH=http://<IP_ADDRESS>:55001/plone pnpm start

######### Core Acceptance tests

.PHONY: acceptance-backend-start
acceptance-backend-start: ## Start backend acceptance server
	#docker run -it --rm -p 55001:55001 $(DOCKER_IMAGE_ACCEPTANCE)
	# Uncomment next line and comment line above to use a custom image with the acceptance server (in case you need to use an specific backend add-on or version)
	docker run -it --rm -e ZSERVER_HOST=<IP_ADDRESS> -e ZSERVER_PORT=55001 -p 55001:55001 -e ADDONS='$(KGS) $(TESTING_ADDONS)' -e APPLY_PROFILES=plone.app.contenttypes:plone-content,plone.restapi:default,plone.volto:default-homepage -e CONFIGURE_PACKAGES=plone.app.contenttypes,plone.restapi,plone.volto,plone.volto.cors $(DOCKER_IMAGE) ./bin/robot-server plone.app.robotframework.testing.VOLTO_ROBOT_TESTING

The frontend has a distinction between dev and production modes (in dev mode it watches for file changes and does hot module reloading in the browser). The backend doesn't have this distinction. So I would remove "dev" from all backend commands, but keep it for the frontend commands that use pnpm start rather than pnpm start:prod.


Abl 10 fill db for perftests tool - intermediate

Readiness checklist

[x] I added/updated unit tests.
[x] I added/updated integration tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

hm, I'm a bit confused why my review got requested .. is the base branch outdated?

hm, I'm a bit confused why my review got requested .. is the base branch outdated?

Hm. I didn't request it. I just rebase my abl-10 on main and that pr branch abl-10-fill-db-for-perftests-tool on abl-10 to get the updated Directory.Build.props
https://github.com/nmshd/backbone/pull/921#discussion_r1814280770 into it.


Documentation: add a "tuning for production" page

Add limits of Marathon (2K apps, 50K Tasks)

where is start?

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-3676. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


trip-simulator lib API

I would like to break out the path simulation code into a library API called by the CLI. This will allow for tests around core functions, as well as the ability to use the GPS physical simulation functionality in custom use cases, such as creating paths from a pre-existing OD pair dataset.

HI Morgan, would like to help with this, especially for creating paths from a pre-existing OD pair part.
Where would you recommend to start with?

@eth4io Good question, I've been trying to think about a reasonable architecture here. In v1, this tool was designed to work with a pre-existing OD pair dataset. The new implementation in v2 works a bit differently and progressively builds trips as the simulation ticks forward. Additionally, we moved to a model that simulates the activity between trips as well, as driver agents search for a new passenger. Maybe we could generate a new driver as the simulation ticks past each trip, then delete the agent as their trips end?

Dismissing agents with some probability and generating new agents would be good idea as in reality not all drivers drives all day and some may also just do few trips. They can also have preferred start and end point as drivers most likely start / end at somewhere near home

I've decided against implementing this. v2 and v3 really settled on a form factor that feels clunky in a library format, and I don't see this changing in v4.


[BTControl] Ensure bonded devices automatically reconnect

https://jenkins.code.rdkcentral.com/job/github-triggered/426/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/427/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/429/ started


Replay #41 against latest master

This is the patch from #41 replayed against latest master (just to stay up to date). See https://github.com/mapbox/node-tilejson/pull/41#issuecomment-175828869 for the motivation behind this new branch.
Test failures are unrelated to this branch - impacting all branches including master: https://github.com/mapbox/node-tilejson/issues/44

per https://github.com/mapbox/node-tilejson/pull/41#issuecomment-443566750 closing


[bug] CUDA messes up after improperly closing Tensorflow session

Whenever I run Keras sessions on PyCharm, and I "forget" to properly close the session (example: I log off the computer, or close session). It seems like the CUDA driver stops working. I can still run $nvcc -V and $nvidia-smi successfully but running the ./deviceQuery sanity check from the CUDA samples fails, which means the system has stopped detecting the graphics card.
Has anyone else had this problem? I definitely recall that this is not the first time this has happened to me using tensorflow vs say using torch, pytorch that also use the graphics card and "forgetting" to close my session.
Perhaps I am just being spoiled? Note that this has happened both using tensorflow directly and using keras with tensorflow as backend (where I do not have direct control over closing sessions directly as far as I know).

@ArturoDeza does this also occur if you kill the TensorFlow process, or just when you log out?
@zheng-xq is this a known problem? Since it happens in other ML libraries, it sounds like a bug in Cuda or the Nvidia drivers.

@reedwm I have never had this issue happen to me on PyTorch or Torch, either by me not closing the session properly or just killing the process. So I think this is more of a bug on the tensorflow side.

Whoops sorry, I misinterpreted your comment. If it only occurs in TensorFlow, it is less likely to be a bug in Cuda.
I cannot reproduce by creating a TensorFlow tf.Session then closing it or killing the process. I haven't tried logging out yet since that will close all my windows which is inconvenient, but I'll try that later.
@zheng-xq do you know what the issue is?

Thank you @reedwm for looking into this. Once I find the exact cause of this issue again, I will repost it. I guess I felt a quite frustrated since I did a mix of these things, and had to re-install the CUDA driver three times in the last 2 days because of running a process on keras with tensorflow in the backend, and all of a sudden the next day (or hour) the driver stopped working. But this has definitely happened to me before with pure tensorflow processes on other computers.

@reedwm @zheng-xq Bug Reproduced: I started an interactive python keras program with tensorflow in the backend, and "closed my laptop". I left it closed overnight at work, and got back in the morning, and when I still had the interactive python console on, the GPU didn't seem to respond. I could still get a positive response by typing in $nvcc -V and $nvidia-smi , but couldn't run the ./deviceQuery and ./bandwidthTest samples from the NVIDIA CUDA Samples folder.
Solution: Restart computer. It seems like the Driver gets messed up when you leave a process hanging in an interactive console and don't close the console before logging out of the session or "making the computer sleep" (the case of the laptop when you close it). It works fine now as if nothing has happened.


use /opt/cockpit as AYS repo folder instead of /opt/cockpit/project

done in :https://github.com/Jumpscale/jscockpit/commit/732831360a9347680723967d6f62f358590743eb


With WinAppSDK 1.4 instantiating Multiple Windows crashes the application

Describe the bug
After updating to WinAppSDK 1.4, when instantiating multiple windows in OnLaunched of the Application class, the app crashes.
Steps to reproduce the bug

In Visual Studio create a new blank WinUI3 C# app
Open the package manager and update both BuildTools (10.0.22621.756) and WindowsAppSDK (1.4.230913002) to those versions
In the OnLaunched event add an additional var window = new MainWindow()
When running it crashes on that line.

It doesn't matter if you try to store the instance in a member variable or if you instantiate a completely different Window.
Expected behavior
On Start the application opens two windows.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.4.0: 1.4.230822000
Windows version
Windows 10 (21H2): Build 19044
Additional context
I actually use AppSDK version 1.4.230913002 and windows version 22H2 19045, but the GitHub issue doesn't show those as selections.

Probably dupe of #8806

Closing as Dup of #8806


Handle webcam when trying this code on laptop

Sometimes, people don't have an available raspi (group projects with only 1 raspi for instance...)
So, they need to test the code on their laptop, but the only camera that is handled is the PiCamera...

without opencv


Critical keyboard bug

When the sidebar is opened the keyboard slides up and down as it is constantly reactivated. Even when a new chat is started once one character is written the keyboard slides down and you cannot chat.
https://user-images.githubusercontent.com/71072643/222993810-475e7ba9-0b11-4c7a-961e-634439e99095.mp4
This unfortunately breaks the entiere app.
Edit - I've tested v1.0.3 pre-release. and the issue still occurs.

Same problem.

Same but worse and only when tapping on the date box  https://github.com/user-attachments/assets/71c6c4ff-46b0-4cde-8152-e9537f9d772e


Adding a diff viewer wrapper over merge module

Parameters:
DOM component,
Left text,
Right text
Takes care of avoiding re-initialization of diff same DOM model, just replaces the text if the diff DOM model already exists

this repo is just for distribution on bower.
it's not the real codemirror github repo. ;)


[SPARK-11819][SQL][follow-up] fix scala 2.11 build

seems scala 2.11 doesn't support: define private methods in trait xxx and use it in object xxx extend xxx.

cc @JoshRosen

Test build #46469 has started for PR 9879 at commit 6d67437.

Test build #46469 has finished for PR 9879 at commit 6d67437.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46469/
Test PASSed.

Merged build finished. Test PASSed.

Thanks - merging this in.


(vb.net) a smaller than sign in a comment crashes all following highlighting

Describe the issue
A simple comment including a smaller than sign < will make all following code lines formatted as comments.
Which language seems to have the issue?
The issue is with VB.NET (probably but not tested the same issue with vbscript).
Are you using highlight or highlightAuto?
Using highlight (via tag associations on SO)
Sample Code to Reproduce
'rule 1: column A allows only text length < 9
Set AffectedCells = Intersect(Target, Target.Parent.Range("A:A"))

as shown in this filddle: https://jsfiddle.net/pcrcoding/4bcxaot0/
Expected behavior
I expect lines that start with a ' or REM  to handle the entire line until line break as coment and end comment formatting at the line break. The expected behaviour would be the one seen in this post at "Sample Code to Reproduce"
Additional context
Also check https://github.com/highlightjs/highlight.js/issues/2851 for the REM .

Confirm. It’s seen as the opening of a doctag which is never closed.  Probably still should terminate at line end either way.

Right now at a glance I'd say we're doing comments all wrong in VB.net.  Since doctags really only exist inside ''' comments, correct?

@joshgoebel If you are talking about the XML documentation comments, yes they all start with ''' and end with the line break.
I linked the documentation below:
https://docs.microsoft.com/en-us/dotnet/visual-basic/programming-guide/program-structure/documenting-your-code-with-xml


Exclude Rails 4.2 + ruby-head (2.6) build

Summary
Exclude Rails 4.2 + ruby-head (2.6) build matrix from Travis CI.
Details
This PR excludes the following error matrix from Travis CI.
NoMethodError:
  undefined method `new' for BigDecimal:Class
# /home/travis/.rvm/gems/ruby-head/gems/activesupport-4.2.11/lib/active_support/core_ext/object/duplicable.rb:111:in `<class:BigDecimal>'

https://travis-ci.org/cucumber/cucumber-rails/jobs/462430088#L1569-L1594
Motivation and Context
Rails 4.2 is using BigDecimal.new. However BigDecimal.new was removed from Ruby 2.6.
https://github.com/ruby/ruby/commit/2810c12a99fed7297d558d8b1aaaf755eb8b8a2d#diff-64cf76d72ee12a82d3093efe6c2f83f7
% ruby -rbigdecimal -ve 'BigDecimal.new(5.0)'
ruby 2.6.0dev (2018-12-10 trunk 66302) [x86_64-darwin17]
/Users/koic/.rbenv/versions/2.6.0-dev/lib/ruby/site_ruby/2.6.0/rubygems/version.rb:216:
warning: deprecated Object#=~ is called on Integer; it always returns
nil
Traceback (most recent call last):
-e:1:in `<main>': undefined method `new' for BigDecimal:Class (NoMethodError)

Checklist:

[ ] I've added tests for my code.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I decided to revert removing BigDecimal.new for the next version of bigdecimal.
Anyway, I think you can stick the version of bigdecimal to 1.3.5.

I reverted this change by #402. Thanks @mrkn!


Downloaded libs are deleted and dowloaded all over again when deleting "build" directory

The way the libraries get downloaded from github seems super useful, however I do not understand why removing the build directory (the one created when compiling) appears to have cmake download the library all over again. To reproduce:

compile
remove the "build" directory: the libs/JUCE is still there
the next run of cmake will scratch the libs/JUCE and redownload it

I agree but this is a CPM issue, please, consider adding an issue there. Thanks!


Changes Unknown when pulling 085aec6088f2b9bf9bdeaffa684c600ed6c53370 on fav-button into ** on development**.


add-ons menu

I tried with v6.0.3 and see the com.siemens.bt.jazz.services.WorkItemBulkMover.IWorkItemBulkMoverService active but there is no menu called Add-ons, did I miss anything when deploying?
From:
Download the Client Extension from the Releases page.
Download the Menu Provider from the Releases page.
Download the Web Service from the Bulk Mover Service Releases page.
The first two links point to the same files:
com.siemens.bt.jazz.ui.WorkItemBulkMover.menuProvider_1.2.2.zip
the 3rd one goes to
com.siemens.bt.jazz.services.WorkItemBulkMover.updatesite-1.2.1.zip
I downloaded both of them and followed the Installation guide to add both them.

You need to download both plugins:

com.siemens.bt.jazz.ui.WorkItemBulkMover.menuProvider_1.2.2.zip
com.siemens.bt.jazz.ui.WorkItemBulkMover_1.2.2.zip
One is the User Interface itself and the other is the Add-On Menu

Thanks for the info.
I am able to see Add-ons menu and did a move now in a simple test.


Fixed test engine reporting when there is an exception in either the …

…init block, the beforeSpec method or the afterSpec method #771

Mocha console is coming with this commit.
Is that correct? I don't think I really mind, just want to know if you're ready with it

The mocha console commit is a mistake on this branch but it needs to go in master anyway.


Line numbers sorted lexicographically instead of numerically.

 Would you be interested in taking a crack at a pull request, or would you like me to fix it?


Number of keywords not highlighting

Looks like a lot of key words are not highlighting:

Took a look at the language file and I can't see anything obviously wrong - the keywords are in there, but for some reason they are not highlighting properly. Need to take a closer look when I have more time, but i suspect there's a broken pipe somewhere that breaks the long keyword strings.

Hi Rick.
I did this program last year with version 0.1 of Visual Studio Code and it was ok.
Now with the latest version there are problems : I'll take a look at that

Hi Rick,
I just take time to look at your issue :
a) Visual Studio Code without my Langage extension and my Theme extension
Showing that:

b) adding my Langage extension by running : "vfp_tmlanguage_generator.prg"
Showing that:

it seem like you have
c) adding my theme extension  by running : "vfp_tmtheme_generator.prg"
Showing that:

That work,
I think I can update: vfp_theme_generator.prg at line 181 -> 183
from
"engines": {
"vscode": "^0.10.1"
}
to vscode:"^1"
again sorry for my badly english
With best regards

Hi Rick,
I just pushed small updates for use with default theme of vscode.
that should be ok now: if you want test?
best regards
Francis


Preload Steam Overlay

Since we're using steam anyway we can also use it's overlay.
diff --git a/truckersmp-cli b/truckersmp-cli
index 059995d..f518f59 100755
--- a/truckersmp-cli
+++ b/truckersmp-cli
@@ -149,6 +149,7 @@ start_with_proton() {

     SteamGameId="$steamid" \
     SteamAppId="$steamid" \
+    LD_PRELOAD="${LD_PRELOAD}${LD_PRELOAD:+:}$steamdir/ubuntu12_64/gameoverlayrenderer.so" \
     STEAM_COMPAT_DATA_PATH="$prefixdir" \
     STEAM_COMPAT_CLIENT_INSTALL_PATH="$steamdir" \
     /usr/bin/env python3    "$protondir/proton" \

Has this been merged into the main code? I'm asking this as currently Steam Overlay doesn't work for me. If not do you have any plans to do so? Thank you.

Can you tell me how to use it? I read the source code for a day.

$ patch -i difffile.patch
patching file truckersmp-cli
Hunk #1 succeeded at 148 with fuzz 2 (offset -1 lines).

Do I need to modify the source code or write another C file?

Just create a difffile.patch file with the content above.

Implemented in #31.


Support for xibs and row height

Support for Row height, default is 44.
Support for Xibs. A Custom class is required when specifying a xib.
Examples provided.

Awesome work! In the future, I think breaking out the xib work and the cell height work into separate PRs would be helpful around discussing the changes.
Gave some feedback, but would love if someone else gave this a second set of :eyes: too!

Done :)

Updated

Hi! I left a few comments but otherwise this looks good. Thanks for the PR. :smile_cat:

Updated

I fixed the where clause mistake.
I don't know I did not noticed it earlier :confounded:

Thanks for fixing! Alright I promise one more thing, then :green_apple: so we can merge. :dancers:

Done :smile:

Merged! Thanks again. 🎉


[orc] update to 1.7.2 or higher

Library name: orc
New version number: 1.7.2 or higher
This is required to enable the orc feature of the arrow port in version 7.0.0 or higher (https://github.com/apache/arrow/pull/12006, https://github.com/microsoft/vcpkg/pull/23032).

Hello, is there a reason why this is still active ? I needed this to compile last version of Arrow (8.0.0) with ORC support.
I did this "by hand" by applying the pull request proposed by Thomas1664 on my local files. It then compiled successfully.

Any news on this matter ?

@LouisClt, I have committed pr update orc to latest version 1.7.6 and ci test is successful, please wait for this pr merge


Move generation from BlobInfo to BlobId

This PR follows from #363. Main changes are:

Add generation to BlobId and remove from BlobInfo, update corresponding tests
Add generationMatch() and generationNotMatch() methods to (BlobSource/BlobGet)Option
Add setGeneration method to (BlobSource/BlobGet)Option to set generation value in empty generation options from BlobId
Add support for empty generation options in storage.get
Add support for empty generation options in storage.readAllBytes
Add support for empty generation options in storage.reader
Add support for empty generation options in storage.delete
Add support for empty generation options in BatchRequest
Add support for empty generation options in CopyRequest
Update/and unit and integration tests

Remarks: I left ComposeRequest unchanged as right now we set source blobs as a pair name, generation, this is due to the fact that in compose operation all blobs must share the same bucket. To support specifying sources as BlobId we need to document that the bucket information is dropped or check that all blobIds are in the same bucket. If you see this feature as necessary I would rather make it in a different PR.

I rebased and added comments on top of both BlobId class and BlobId.of method.

Remarks: I left ComposeRequest unchanged as right now we set source blobs as a pair name, generation

Sounds good to me.
Do you plan to change DefaultStorageRpc to use the generation value in a separate PR?

Do you plan to change DefaultStorageRpc to use the generation value in a separate PR?

I'm missing something again, sorry:) what do you mean with this?

From #363 "And we should use that generation (if supplied) in applicable operations such as delete, compose, copy".
E.g. in DefaultStorageRpc.deleteRequest we should call  .setGeneration(blob.generation).

Uh now I see. I will make it part of this PR!

Comments addressed


Productize Task/ValueTask/Async.ignore

I've long felt icky about using Async.Ignore. Seeing the Async.ignore that's presently hiding in plain sight here makes me realise the absence of a Task.ignore and ValueTask.ignore from FSharp.Core is something I've also been grinning and bearing (sprinkling :> Task and all sorts of other such mindless hacks). Utils.fs presently has helpers that feel very tempting to use more broadly. However, binding  to a set of support helpers on the fringe of this library's core feature set is obviously debatable.
(The other pair from here that I feel deserves a canonical implementation in a library that doesn't include a jungle of other less-related helpers is Async.ofTask/Task.toAsync, but with corrected AggregateException semantics, and honoring of Async.Cancellation when compared to Async.AwaitTask as per FSharp.Core. Will likely calve off a separate issue from this if placeholder FSharp.Core.TaskShims package/repo is deemed to be the best home for a set of ignore helpers)
related:

precursor discussion in PR comments
add AwaitTaskCorrect to FSharp.Core suggestion

/cc @dsyme, we should discuss this. Some of this effort may ultimately land back into F# Core, but agreeing on a TaskEx library (name tbd) and adding it to the FsProjects realm may be a good idea.
This is related to #78

Yeah I have a lot of the same functions floating around in IcedTasks (example1, example2) and FsToolkit (example1, example2)
I even have a tool that writes out all the functions (like map, bind, apply, zip, etc) and which libraries contain them because it's hard to know who implementing what these days and where the gaps are in functionality.

Closing in favor of #139 (general high level discussion about shimming and/or libraries/projects) and #140 (ignore helpers only)

Switching close reason; work is not completed; sorry!

Thanks for writing out these covering issues! Much appreciated!


MAHOUT-1978  Implement Assumption-Free K-MC^2

Purpose of PR:
Implement Assumption-Free K-MC^2
Important ToDos
Please mark each with an "x"

[x] A JIRA ticket exists (if not, please create this first)[https://issues.apache.org/jira/browse/ZEPPELIN/]
[x] Title of PR is "MAHOUT-XXXX Brief Description of Changes" where XXXX is the JIRA number.
[x] Created unit tests where appropriate
[x] Added licenses correct on newly added files
[ ] Assigned JIRA to self
[x] Added documentation in scala docs/java docs, and to website
[x] Successfully built and ran all unit tests, verified that all tests pass locally.

If all of these things aren't complete, but you still feel it is
appropriate to open a PR, please add [WIP] after MAHOUT-XXXX before the
descriptions- e.g. "MAHOUT-XXXX [WIP] Description of Change"
Does this change break earlier versions?
Is this the beginning of a larger project for which a feature branch should be made?

hey @meftim0va we changed master to trunk guessing this was never merged bc of a moratorium on MR patches/algos- however, I'm ok adding this after the code freeze/release if you want to reopen against trunk


Fix geometry for one case where two roads go between the same pair of…

… intersections. #136

In this one case, the problem was the hit point between the two roads being detected at the wrong end:

As the comment says, this is a bug in polyline. And apparently it's not the first weird one: https://github.com/a-b-street/abstreet/blob/aa40be4e82acd390d3532eb13e61fdce0d0a0fd8/geom/src/polyline.rs#L790
I'm going to see if I can rip out this scary old line intersection code I wrote myself years ago and switch to georust now. :)

Also, loop roads in leeds_cycleway and roosevelt_cycletrack are still broken for some other reason, I'll look at those too


Log cache reference update

Update references to archived log-cache to use go-log-cache instead

@Benjamintf1 are you able to merge this? I don't have write access

🤔 @Benjamintf1 you should have write access according to the org automation yaml.
🙏 @ameowlia do you have admin access here? Is there any setting that might prevent approvers with write access from merging PRs?


Propagate arbitrary/configurable http headers from publisher to subscribers

In what area(s)?
/area runtime
/area docs
HTTP request header propagation
It would be great if dapr sidecars used for pub/sub could be configured to forward certain headers from the publisher request to the subscriber request.
Example Use Case:
For collection of distributed traces in a microservice environment we use Datadog and for instrumentation of our Java-based micro services the Datadog Java agent. This Java agent does not support the W3C traceparent header that is currently forwarded by dapr. Instead it propagates trace context using the headers:

x-datadog-trace-id
x-datadog-parent-id
x-datadog-sampling-priority
x-datadog-origin

So to achieve correct trace context propagation we would like to configure dapr in such a way that the header values the Java agent sets on publishing requests are also present on the subscription's imcoming http requests where they can be picked up by the Java agent of the receiving service.

Dapr supports arbitrary fields in a custom CloudEvents envelope, so theoretically the publisher can just add all those fields to the CloudEvent message and these would be visible to subscribers.
Does that meet your needs? https://docs.dapr.io/developing-applications/building-blocks/pubsub/howto-publish-subscribe/#sending-a-custom-cloudevent

When I add custom fields to the envelope do they arrive as HTTP request headers in the HTTP call from dapr sidecar to the subscriber? I am aware that I can manually read data from the envelope in the Java code of the subscriber but the Datadog Java agent instrumentation expects the tracing information in HTTP request headers. (Just the same way the OpenTelemetry Java agent does too, only other headers than "traceparent")
I was thinking of something already implemented like

dapr sidecars propagate the "traceparent" HTTP request header from publisher to the subscriber
dapr sidecars propagate arbitrary HTTP headers from caller to service when using dapr service invocation

I need the same function

@yaron2 I also need this.

We are also looking for this feature, where we can pass custom headers while invoking

invokeBinding when kafka is configured (kafka.binding). This works fine when using with http binding. But does not work as expected while using with kafka binding.
publishEvent when using with kafka.pubsub
Above using java SDK.

@yaron2  please Is there any plan to support x-b3 tracking protocol? thanks

Any updates on that?

+1 on this.
We have a fundamental architectural use-case which relies on this capability becoming available soon, ideally in 1.9.

FYI @mukundansundar
And somewhat related: There is also the request for arbitrary fields / properties on the message data itself. This is referred to as cloud event extensions generally.
I have a draft PR for the cloud event extensions, but it seems we additionally would need a field on the cloud event envelope proto to store arbitrary headers in some sort of map.

FYI @mukundansundar
And somewhat related: There is also the request for arbitrary fields / properties on the message data itself. This is referred to as cloud event extensions generally.
I have a draft PR for the cloud event extensions, but it seems we additionally would need a field on the cloud event envelope proto to store arbitrary headers in some sort of map.
@berndverst
For a cloudEvent we can have an additional field called metadata, but for raw payload we will not be able to store any metadata with the payload itself.

The main need is to be able to set some application defined properties along with the message.
Most of the brokers and their SDKs do support storing storing application defined properties/metadata along with the message.
For example:
EventHub has a Properties map[string]interface{} field for every Event.
ServiceBus message has an ApplicationProperties map[string]interface{} field to store custom metadata for a message.
GCP Pubsub has an Attributes map[string]string field to store custom key-value pairs the message should be labelled with.
Hazelcast has no support for metadata storing along with message.
NATS Jetstream has support for Header map[string]string field through the Msg struct which can be used to publish message to Jetstream.
Kafka has the Headers []RecordHeader field to store metadata along with message.
MQTT has no support for metadata storing along with message.
Pulsar has a Properties map[string]string for application defined properties on the message.
RabbitMQ has a Headers Table field (Table is map[string]interface{}) for Application/exchange specific fields.
Redis has no support directly for storing the metadata.
RocketMQ has a properties map[string]string field storing metadata.
In this scenario except for Redis, Hazelcast and MQTT, all brokers do seem to support metadata to be stored along with the event.
Should we target storing these headers as metadata in these brokers for each message? @dapr/maintainers-dapr thoughts?

In addition to @mukundansundar list :

AWS SQS also supports up to 10 custom properties on each message. Although each property must be typed.

Can +1 saying that it's preventing me from working with e.g. Azure Digital Twins since it sends to Event Hub and they utilize source as the twinId thus preventing twinId to be accessible as the property is not being mapped correctly.

Any progress?

Is it possible to get Application Properties (enriched messages) from an Event hub via a Dapr binding now?
Currently using a bindings.azure.eventhubs and not a pub-sub, not sure if I need to switch to a pub-sub to get the additional properties.


How do we set Docker Proxy?

I could install hypriot into Raspberry-pi. But I could pull docker image , I am sure that my company's proxy makes this error.
How do we set Docker proxy?  Do we have to set proxy at /etc/default/docker ?

@yoshihingis I cannot test it, but the official docs https://docs.docker.com/engine/admin/systemd/#http-proxy should show you the way how to do it.

Thank you for your advice.
I tried to the way which you suggestd, this problem was solved.
Thank you again!


Generalize BaseProblem to capture sampling problems.

The current BaseProblem interface should be renamed to BaseSecQuantProblem (or similar) since it contains logic specific to second quantized problems that does not appear in sampling problems.
There is need for a new interface, one level higher, called BaseProblem which both BaseSecQuantProblem  and BaseSamplingProblem would implement. Then, ground state eigensolvers present in Nature should operate on the new BaseProblem, not the current one.

Hi @dlasecki ,
can I have a look at this one?
Best,
Martin

Hello @MartinBeseda,
After internal discussions we decided to take a different route for now than generalizing the existing BaseProblem class. We still need to figure out some details about it internally. Thanks for offering help though. For now, I'm closing this issue.


[BakaUpdatesMangaReleasesBridge] Added option to display releases based on a user list

Given the last part to a user's public list, returns just the manga from the Releases page that were on the given list.

Same here. Please fix linting.

Thank you for the PR !


Firefox produces warning "Will-change memory consumption too high"

Warning produced in Firefox 67:
Will-change memory consumption is too high. Budget limit is the document surface area multiplied by 3 (960614 px). Occurrences of will-change over the budget will be ignored.
I assume this has something to do with the way the project is bundled. Do you have any suggestions for optimizing will-change usage?

Please open an issue on the react-spring repo (as this project uses this): https://github.com/react-spring/react-spring
The starter doesn't use the "will-change" property. Thanks!