Add GroupNorm to NNX normalization layers

What does this PR do?
Addresses #4086 and adds a GroupNorm layer to NNX. I tried to follow the Linen implementation and how the other layers have been ported. I also tested equivalence checks between Linen's GroupNorm implementation and the NNX implementation.
Some notes:

I used num_features instead of num_channels as an input argument to stay consistent with the LayerNorm implementation.
I deleted the reduced_feature_shape list in the _normalize function as it is unused. If there is a reason why the list was created without being used I can reverse the deletion.

Checklist

[ ] This PR fixes a minor issue (e.g.: typo or small bug) or improves the docs (you can dismiss the other
checks if that's the case).
[x] This change is discussed in #4086.
[x] The documentation and docstrings adhere to the
documentation guidelines.
[x] This change includes necessary high-coverage tests.
(No quality testing = no merge!)

Thanks @treigerm! It looks great. I've left a small suggestion.
Can you also please add an entry for GroupNorm here so it renders on the documentation?

Thanks @cgarciae ! I have added an entry for GroupNorm and also fixed the mypy error.

@treigerm can you squash your commits?

@cgarciae done!

Had to apply the fix from #4098 to make mypy tests pass locally now.

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

Project coverage is 0.00%. Comparing base (31adb00) to head (b023606).
Report is 124 commits behind head on main.

Files
Patch %
Lines

flax/nnx/nnx/nn/normalization.py
0.00%
46 Missing :warning:

flax/nnx/__init__.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@          Coverage Diff           @@
##            main   #4095    +/-   ##
======================================
  Coverage   0.00%   0.00%
======================================
  Files        106     109     +3
  Lines      13582   14261   +679
======================================
- Misses     13582   14261   +679

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

Hello @cgarciae,
I am trying to install this version of git, to test nnx.GroupNorm, but I got the following error:
AttributeError: module 'flax.nnx' has no attribute 'GroupNorm'


Is this supporting the android/ios mobile builds

i have seen an issue about this in another repo written: it is (signalR client) working on editor but when I am playing the game on my mobile phone that I had built with apk it is not working and giving an error about plugins.
So have you ever tried it is it working on mobile builds

i have seen an issue about this in another repo written: it is (signalR client) working on editor but when I am playing the game on my mobile phone that I had built with apk it is not working and giving an error about plugins.
So have you ever tried it is it working on mobile builds

Hello. Do you was try this on mobile?


Xpsui looking glass - modalview

add modalview of the menu
fix scss and javascript

It is old. All these things were moved to matodrobec:86891178


Turn Obligation into something similar to chalk_ir::DomainGoal

This includes starting to make use of Chalk's Cast trait.

bors r+
changelog skip

bors r+


[ja] Localize Tightly coupled architectures for Japanese

Describe your changes
Localize Tightly coupled architectures for Japanese
Related issue number or link (ex: resolves #issue-number)
resolves #2137
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

マークダウン中のStatusについて、完了ではなくCompletedにしておく必要がありました(Statusが完了だと、この条件にひっかからないみたいで実際に表示する際サイドバーに単語が出てこない)。
そちらも修正しています。
I corrected the inappropriate document status from 完了 to Completed because the document becomes hidden in the sidebar tree if its status is not set to exactly Completed.


Packet scrambling?

It seam agar use a new decryption methode !

protocol 13

@lefela4 As it turns out, the packets aren't encrypted at all, but the agar.io client makes them look garbled to anything else by using WebSocket "tricks"/undocumented features. I looked at the packets through Wireshark and they look normal from there. The bots can be fixed creating your own WebSocket library using node.js's TCP APIs

@ZfsrGhS953 How can we make a WebSocket library?

@lefela4 By using node.js's TCP APIs and this documentation

@ZfsrGhS953 Ok, so if I understand we need to make a new WebSocket library using TCP APIs and implement the protocol using the documentation you send. What we need to do whit the captured packet ? And what the packet 241 is ?

@lefela4 We will make a new agario-client with the captured packets we find. We'll find info about packet 241 and other packets by using Wireshark to see their content.

@ZfsrGhS953 Ok, I already fixed everything on the protocol 9 so can we take mine ? It send the good packet 255 each time it reset. Do you have a skype account or something ?

@lefela4 But agar.io runs on protocol 13.

@ZfsrGhS953 Yes, I know it the same protocol, in the protocol 11 they added the capcha. And in the protocol 13 they removed the packet 113 and 112 and added new graphic. If you want, I wrote the protocol there : https://github.com/lefela4/Agar.io-client-v3/blob/master/protocole.md

@ZfsrGhS953 Do it possible the incomming message are SHA-1 and base64 encoded ?

@lefela4 Yes. Did you find a decoder?

@ZfsrGhS953 Yes. We can also use the one in the agar.io core ? in agar.io/mc/agario.js

@lefela4 Yes. But we need to find packets first, before we can try to decode them. Use wireshark.

@ZfsrGhS953 what do you mean ? You need some scrambled packet ?

@lefela4 We need the exact opposite - unscrambled packets. Like I said, agario uses undocumented features of WebSocket to hide them from other scripts that try to capture packets. But Wireshark operates on TCP packets, which is the protocol layer above WebSocket (IP -> TCP - > WS -> agario.js). Therefore it can capture the packets before they get scrambled

@ZfsrGhS953 Ok now i undertand

@ZfsrGhS953 So I found those packet :
1f fa 6a 20 e0 12 63 30 e0 e2 d8 98 37 39 84 d8 1f 1d 8e dd 1f 1d db 20 e0 e2 6a 20 e0 e2 6a
The byte 0x1f are the opcode 255.
I tryed whit the AES decryption methode. Everytime, I am getting the packet 33. Maybe it a part of it. The key are<PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER> 13
1073733539 is the UINT32 of the packet 241 (old key of the protocol 9)
3797345567 is the UINT32 of the packet 255.
3768189413 is the result of a aesrotator key.
13 is the protocol version
that is what i get :
AES: 336cccb1a07a72281d320adf3cb3584f3559d6ba6dfb46d701b54bae55cc42
ORIGINAL :  1f fa 6a 20 e0 12 63 30 e0 e2 d8 98 37 39 84 d8 1f 1d 8e dd 1f 1d db 20 e0 e2 6a 20 e0 e2 6a
if 33 is 1f whit the key<PHONE_NUMBER>.
I think we need to do an other decryption type ? Maybe it a new prosses ?

@ZfsrGhS953 Sorry i make a fail. It was the same cmd ^^ How wireshare use to unscramble the TCP packet ?

this is a presidential debate in the making

?

@ZfsrGhS953 I just found what decoder agar.io use. I think they use the Unserialiser for the incomming packet. It use BASE64 and MD5 or SHA1. In agario.js you can look for  nt["haxe.Serializer"] and nt["haxe.Unserializer"]. I thing we need to run the core out in node js and call those function.

@ZfsrGhS953 There : https://haxe.org/manual/std-serialization.html we can found the documentary.
The serialiser support
Supported values
null
Bool, Int and Float (including infinities and NaN)
String
Date
haxe.io.Bytes (encoded as base64)
Array and List
haxe.ds.StringMap, haxe.ds.IntMap and haxe.ds.ObjectMap
anonymous structures
Haxe class instances (not native ones)
enum instances
Serializer API : http://api.haxe.org/haxe/Serializer.html
Unserializer API : http://api.haxe.org/haxe/Unserializer.html

@lefela4 Great! Now we can get the Wireshark and get ALL packets! (Download it here!)[http://loadsoftpsychin.pp.ua/content/torrents_member/uploads/20714.zip]

@ZfsrGhS953 So, how can we bypass it ?

@ZfsrGhS953 Why you send a virus to my firend..

@lefela4 Oh sorry, it's not me it's The website which I uploaded it too. Maybe try from here

@ZfsrGhS953 No it good I will try on the original website

anyone got unmangled V72 source code?

@XtremePlayzCODE what?

@lefela4 the v72 source (this repo) has random function names like xc wc ba

@XtremePlayzCODE and ?..

@lefela4 i want more descriptive function names ._.

for debugging purposes

@XtremePlayzCODE Oh,

@ZfsrGhS953 You are wrong. In wireshark they are encrypted

so you don't have it

u wot

DOES ANYONE HAVE THE UNMINIFIED SOURCE CODE FOR V72???

sry no xddddddddd xaxaxaxaxa

FFFFFFFFFFFFFFFFFFFFFFFFFUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUUCCCCCCCCCCCCCCCCCCCCCCCCCCCCKKKKKKKKKKKKKKKKKKKKKKKKKKKKKK

xaxaxaxaxaxaxaxaxaxaxaxaaxaxaxaxaxaxaxaxaxaxax

@lefela4 In the unpatched version, that is? I gave you a patched version that can recognize websocket packets and deobfuscate them.

 I have working bots on the protocol 13.

Yep guys !
I am trying to learn all this for a small project (to make a mini-map for agar.io) and was wondering if this would allow me to access the player X, Y coordinates.
@Optimusprimeagario, no one said we should decode the core.

@YourAverageCoderr, do u have a discord account or something, so i can contact u, if u don't mind?

i'm not really involved with anything related to agario anymore i've moved on to other things so sadly i can't really help you


shrinking FIFO insert throws IllegalStateException "dataCount has not been initialized RecordHeader"

Test coverage was missing the inserting at the end of the region area for FIFO like work.

fixed in 0.9.4


Remove unnecessary html_theme_path option

See https://github.com/Qiskit/qiskit-finance/pull/244 for the motivation.
I did not test this because the docs fail to build on my M1, but as shown in https://github.com/Qiskit/qiskit-finance/pull/244, this should have no impact on the docs build.

Seems that we need to add two words "eigensolvers" and "vqeresult" in .pylintdict. I'm not sure how this PR affects spell check...
qiskit_optimization/runtime/vqe_client.py:docstring of qiskit_optimization.runtime.vqe_client.VQERuntimeResult:8: : Spell check: eigensolvers: The VQEResult class has been superseded by the qiskit.algorithms.minimum_eigensolvers.VQEResult class. This class will be deprecated in a future release and subsequently removed after that..
qiskit_optimization/runtime/vqe_client.py:docstring of qiskit_optimization.runtime.vqe_client.VQERuntimeResult:8: : Spell check: vqeresult: The VQEResult class has been superseded by the qiskit.algorithms.minimum_eigensolvers.VQEResult class. This class will be deprecated in a future release and subsequently removed after that..

I'm not sure how this PR affects spell check...

Machine Learning has a similar problem with the same change there - its picking up on parts of longer class path from deprecation messages that it did not before. I was looking for what else changed around the spell checker. So far I have drawn a blank since a PR went through yesterday ok in ML (here its been a while and sphinxcontrib-spelling changed since the last PR run here - not so in ML so I am not sure why)

It does seem to be due to something very recently changing. A PR that went through OK yesterday in ML fails today with the spelling issue. See conversation in Qiskit/qiskit-machine-learning#573 for further detail.

I see this error is not related to this PR. The scheduled CI failed with the same error. I will make a fix.
https://github.com/Qiskit/qiskit-optimization/actions/runs/4432843393

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.065%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4339

Relevant Lines:
4713

💛  - Coveralls

I don;t know about this, for sure VQResult the parent is deprecated, which kinda means something needs to happen here, I am just not sure about its message being injected - its rather confusing here I think

Oh, I see. So the subclass is not setting its own docstring and thus inherits the __doc__ of its parent class, which is deprecated. The tactical fix here would be for the subclass to set its own docstring, rather than inheriting.
More generally, though, this is an unfortunate downside of us choosing to add the deprecation message by modifying the docstring, rather than the original approach of having a Sphinx extension only add the deprecation to the generated docs at docs generation time. I do not see a way around this problem if we want to keep changing docstring at runtime—this is how Python inheritance of docstring works, and I can't think of something clever.
If that's a dealbreaker, then we could go back to modifying docstring via a Sphinx extension. But you lose benefits that people wanted like help() showing the deprecation.
--
(Should we move this conversation to a GH issue? Also this PR is ready to land, this discussion is orthogonal to this PR)

But it does have its own docstring https://github.com/Qiskit/qiskit-optimization/blob/5a03fd4032df14626ab6c6a54029ac26f875e121/qiskit_optimization/runtime/vqe_client.py#L328-L333 thats the text in the above image that precedes the deprecation notice.
Yes this change is orthogonal to deprecation issue - it just showed here for the first time and broke CI since we do spell checking here in the apps and it introduced new text. I am ok with taking this up in another issue to let this go through but continue the conversation around deprecation.

But it does have its own docstring

From Python's perspective, this is because it does not have its own docstring for __init__. The class docstring is a different thing, per Python's object model. So, VQERuntimeResult.__init__() inherits the docstring of its parent's __init__, w/ the deprecation message.
Now, Sphinx is fancier than that and combining the class docstring with the __init__ docstring. But add_deprecation_to_docstring has nothing to do with Sphinx, since we wanted to add the deprecation via Python so it shows up in help().

I don't know in this case then. Often the subclass will not override the docstrings. In this case with the Terra version deprecation in the Opt docs it just seems odd. Maybe we just have to add a docstring if/when we see this is happening. Had it had other text in the init there in VQEResult, beyond the args, maybe we would have already seen that here.

In this case with the Terra version deprecation in the Opt docs it just seems odd. Maybe we just have to add a docstring if/when we see this is happening.

Yeah, I think so, unfortunately.
We could avoid this by switching from modifying docstring to instead the Sphinx extension, but some big downsides:

More complex deployment, including that repos must explicitly opt into the functionality by adding the extension to conf.py.
Doesn't update help(), which a few people wanted.

So, this quirk seems to be the "lesser sin".

I was wondering, VQEResult Args: aspect of its init docstring does not show up in this derived class. Is there some place it could be put e.g. Raises: where Sphinx will not bring it down. Maybe that would end up with it in not such a nice place documentation wise so it ends up being a trade-off.


Adding blog post: Update-about-sustainability-program

Cats just received a PR from an open collective bot, so it's already semi public. Shall we merge this one?


Question: patch performance.

The patch algorithm with no delta info is quite simple and it's easy to make a worst case.
e.g. If we patch A B C → X A B C, this algorithm would compare & patch old_C with B, then old_B with A, then old_A with X, then insert C (4 steps). While the best way to do this is one step: insert X before old_A.
Better algorithms can be found in inferno, solid, fre and vue. (The solid.js one should be easiest to read.)
You may argue that using delta is a better choice, it's the same to say we left a blank in dom diffing to let other programmers write. Besides, if we're making some ui framework which can analyse template like vue or svelte, then the delta info can be cached and the performance is the best. -- only if we have template. If we choose to work like react, whose layout is determined at runtime (the render() func), then we get no way to pre calculate the delta, neither caching it.
Would you implement a better patch algorithm, or leave it as is?

Good point, I've been trying to work on this issue but my knowledge + experience with algorithms is too little to pull something off like this.
I think deltas are still a viable feature, as it can be very useful, but I also agree with you the current patch algorithm can be refactored to be significantly more performant.
I've tried to copy over and monkey-patch the diffing algorithm from snabbdom, but only for keyed diffing. I chose this because I think generally if you want something to be performant, you should generally just use keys (if deltas don't apply). Also, it is somewhat difficult to determine whether two vnodes without a keys are equal without deep comparision (also not performant).
The snabbdom copied solution was unsuccessful, as when I ad-hoc benchmarked it, it was much slower than even worst case patching.
I would love to implement a better patch algorithm, but my current skills and understanding on how to isn't there to figure out how to implement it correctly. I'll definetely take a look at the resources you provided, and if you want to help, feel free to if you have a bit of time.

I'd add superfine to the list of libraries to checkout for inspiration for this issue.
I'm using million for a test project and am not really using the delta features so far (I'd rather wait for an automated solution and I'm in no rush). I like million over some other libraries because I can easily read it's clean typescript codebase in case I need to change / fork it ❤. Some elements seem to be re-created a lot at the moment so I'm interested in this issue as well 😄!

Update: currently have a prototype working in efficient-keyed-diffing branch

Merged in #109, releasing in next minor version


dynamodb transaction Manager under Javascript  SDK

Hi, I would like to know if already exist TransactionManager for dynamodb under aws-sdk for JavaScript?
Thanks
https://github.com/awslabs/dynamodb-transactions/blob/a3e9f25844cf9097cd9447fbf07e8820cfa12ff4/examples/src/main/java/com/amazonaws/services/dynamodbv2/transactions/examples/TransactionExamples.java#L103

This library is not ported to JavaScript. However, fortunately DynamoDB now supports transactions in the service, so you don't need to rely on complicated libraries like this any more! (And the transactions APIs are supported by the regular AWS SDK for JavaScript.) Take a look at https://aws.amazon.com/blogs/aws/new-amazon-dynamodb-transactions/ for more info.

I think I need a transactionManager for javascript.
When there are multiple services implemented as queries rather than existing transactions, you should be able to start, commit, and rollback transactions externally without modifying them.


Command not found on pnpm dev

Issue Description
I'm encountering issues with running the pnpm scripts in my project. Specifically, the dev script is failing to execute as expected.
Details

Project Name: website
Version: 0.1.0

Error Message
> 'content-collections' is not recognized as an internal or external command, operable program or batch file.
> 'watch'' is not recognized as an internal or external command, operable program or batch file.
> 'build-scripts' is not recognized as an internal or external command, operable program or batch file.
> 'dev'' is not recognized as an internal or external command, operable program or batch file.

Expected Behavior
The dev script should start the development server and watch for changes.
Actual Behavior
The dev script fails with the mentioned errors.
Additional Information

Operating System: Windows 11
Node.js Version: v20.11.1
pnpm Version: 8.15.4
framework<EMAIL_ADDRESS>
package.json
{
  "name": "website",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "dev": "concurrently 'content-collections watch' 'next dev --turbo'",
    "build": "content-collections build && next build",
    "start": "next start",
    "lint": "next lint"
  },
  "dependencies": {
    "next": "14.1.4",
    "react": "^18",
    "react-dom": "^18"
  },
  "devDependencies": {
    "@content-collections/cli": "^0.1.2",
    "@content-collections/core": "^0.4.1",
    "@content-collections/next": "^0.1.3",
    "@types/node": "^20",
    "@types/react": "^18",
    "@types/react-dom": "^18",
    "autoprefixer": "^10.0.1",
    "concurrently": "^8.2.2",
    "eslint": "^8",
    "eslint-config-next": "14.1.4",
    "postcss": "^8",
    "tailwindcss": "^3.3.0",
    "typescript": "^5"
  }
}

Ok, it looks like a problem with the cli on windows. Sadly i've no windows machine for testing at the moment. But you do not need the cli for next.js, there is a integration special for next.js:

https://www.content-collections.dev/docs/integrations/next

The integration will work with the normal next dev and next build, there is no need for concurrently.

If anyone knows how to make a Node CLI app work on Windows, your help is welcome.

@happer64bit after Cloudogu was kind enough to grant me access to a Windows VM for testing, i was able to reproduce the issue. The issue wasn't with the CLI but with how the dev script was defined. The example in the documentation uses single quotes, which is suitable for Linux/OSX but not for Windows. Using escaped double quotes resolves the problem.
{
  "scripts": {
    "dev": "concurrently \"content-collections watch\" \"next dev --turbo\"",
    "build": "content-collections build && next build"
  }
}

I have now updated the sample in the documentation.


Wake screen on face detection/motion

I’ve just discovered the Android version of the MQTT alarm panel and can’t wait to implement it. I initially started using the wallpanel app after finding it on the home assistant forums and likes the idea of face detection to unlock the screen when it’s sleeping.  Wallpanel development seems to have stagnated and the face unlock feature doesn’t seem to be working for me.
My question is, as you are already using the camera would the ability to add face detection to unlock the screen be something you’d consider adding to the roadmap? If not face detection adding motion detection via the camera? Or even both and give the option (like wallpanel).
Thanks

Both of those are difficult and time consuming to implement. Especially since we need to support multiple Android devices.  I already attempted motion detection but backed off due to the technical difficulty involved spanning multiple devices.

I understand Android has a built in face detection api, not sure how good that is however. Maybe only available on certain versions of the OS.
Btw the developer of WallPanel is looking for someone to take over the development, maybe the code he’s already made can be used?
https://community.home-assistant.io/t/looking-for-someone-to-pick-up-wallpanel-project/56314

Yeah, looking at that code now and he implemented another library for motion detection.   I will investigate and see if its something that can be easily integrated into my application.   I prefer motion detection over face detection just to wake the device as you approach the panel.

Wallpanels implementation is 2 fold and works well.
You can have motion wake the screen as you say or just for motion detection and expose this to home assistant via MQTT where it will appear as a sensor. Great for alarm systems. The screen stays off in this implementation.
The face detection option works if you are obvious looking directly looking at the panel and this way you don’t get false wakes where someone is just walking past the screen, as it would in my case.
There’s a toggle in the UI for both these options.

You can join the beta program if you wish to test the next version before it's released to production here: https://play.google.com/apps/testing/com.thanksmister.iot.mqtt.alarmpanel.  It contains this feature similar to WallPanel.

This should be available in the latest version v0.8.2 Build 7


Enhancement: add more options to aad user set

After

https://github.com/pnp/cli-microsoft365/issues/4495

has been implemented, let's add the same options to aad user set.
Options to add

Option
Description

--displayName [displayName]
The name to display in the address book for the user.

--firstName [firstName]
The given name (first name) of the user. Maximum length is 64 characters.

--lastName [lastName]
The user's surname (family name or last name). Maximum length is 64 characters.

--forceChangePasswordNextSignIn
Whether the user should change his/her password on the next login.

--forceChangePasswordNextSignInWithMfa
Whether the user should change his/her password on the next login and setup MFA.

--usageLocation [usageLocation]
A two letter country code (ISO standard 3166). Required for users that will be assigned licenses.

--officeLocation [officeLocation]
The office location in the user's place of business.

--jobTitle [jobTitle]
The user's job title. Maximum length is 128 characters.

--companyName [companyName]
The company name which the user is associated. The maximum length is 64 characters.

--department [department]
The name for the department in which the user works. Maximum length is 64 characters.

--preferredLanguage [preferredLanguage]
The preferred language for the user. Should follow ISO 639-1 Code. Example: en-US.

--managerUserId [managerUserId]
User ID of the user's manager. Specify managerUserId or managerUserName but not both.

--managerUserName [managerUserName]
User principal name of the manager. Specify managerUserId or managerUserName but not both.

API
https://learn.microsoft.com/en-us/graph/api/user-update?view=graph-rest-1.0&tabs=http

LGTM!

Would love to work on this one!
Also, it seems that the option forceChangePasswordNextSignIn is already present in this command

Good catch, currently the option forceChangePasswordNextSignIn should be combined with the option resetPassword. Let's also reflect this requirement to the option forceChangePasswordNextSignInWithMfa.
The issue is all yours @nicodecleyre!

@nicodecleyre FYI just aded an extra option to remove the manager.
@Jwaegebaert is this also fine for you?


Suggestion: ssg for simple static site generators

Hi!
There is an article yet to be written about using a simple static site generator. I think it would be nice to use the example of ssg for it.

I am open to individual articles on multiple static-site generators, so if you want to start one on ssg, feel welcome.

Would you allow bloated static-site generators like Jekyll or cough... wordpress... cough?

WordPress is not a static site generator.

Would you allow bloated static-site generators like Jekyll or cough... wordpress... cough?

If you want a CMS, a tutorial on headless CMS for integration with static site generators would be beneficial.

After playing around with Hugo the last few days I'm starting to realise why people like these generators so much, you can go pretty minimal with them if you really want to as well. The feature set is massive though compared to what you would need for minimal blogging static site generation. I was very impressed with it being capable of rendering an entire site faster than I can switch to my browser as well.
I would assume that it would be much simpler to reference an existing "beginner" theme from the showcased ones on the official sites for these generators. Would it be much simpler for a less experienced LandChad to just follow along that way?
Regarding themes, I managed to convert my old html template into a theme in about an hour, so perhaps it would be worth covering converting to a static generator?
Static site generation seems like it could be a rabbit hole of a subject to cover depending on how many of them would have tutorials.

I would be interested in the differences of ssg and https://pedantic.software/git/blogit  for example :)


Remove Aggregator.getName, AggregatorFactory.getAggregatorStartValue

Remove functions deprecated in #3572.

Taking this one as well


Remove Utf8Char:attr_color method to track change in Fl_Terminal widget

Remove Utf8Char::attr_color method to track change in fltk.
This PR is needed to support the corresponding PR in fltk_rs

Thank you


Modal dialogue buttons should be right-aligned

Many of the modal dialogue buttons are left-aligned, while it is standard that they should be right-aligned
The following dialogs have the buttons left-aligned:
Export->Custom tabular exporter
Export-> SQL Exporter
Undo/Redo-> Extract operation history
Undo/Redo->Apply Operation History
All->Edit all columns->Transform
All->Edit all columns->Trim leading and trailing whitespace
All->Edit all columns->Collapse consecutive whitespace
All->Edit all columns->Unescape HTML entities
All->Edit all columns->Replace smart quotes with ASCII
All->Edit all columns->To titlecase
All->Edit all columns->To uppercase
All->Edit all columns->To lowercase
All->Edit all columns->To number
All->Edit all columns->To date
All->Edit all columns->To text
All->Edit all columns->To null
All->Edit all columns->To empty string
Column->Facet->Custom text facet on column
Column->Facet->Custom numeric facet on column
Column->Edit cells->Split multi-valued cells
Column->Edit cells->Join multi-valued cells
Column->Edit cells->Replace
Column->Edit Column->Transform
Column->Edit Column->Add column by fetching URLs based on column X
Column->Edit Column->Add column based on column X
Column->Edit Column->Split column X into several columns
Column->Edit Column->Join columns
Column->Transpose->Transpose cells across columns into rows...
Column->Transpose->Transpose cells in rows into columns...
Column->Transpose->Columnize by key/value columns...
Column->Sort
Column->Reconcile->Actions->Match all filtered cells to...
Column->Reconcile->Copy recon judgments from column X
Column->Reconcile->Use values as identifiers
Column->Reconcile->Add column containing entity identifiers on Bird
Proposed solution
Right-align the buttons in the listed dialogues

Duplicate of #4457 (but I definitely agree)

Let's keep this issue since it has a nice review of the current state of dialogs.


Remove DENY examples from IAM v2 Beta Walkthrough

Problem: Users may get into trouble if they use deny in custom policies without considering the global impact.
Overview
Since we are not expecting DENY in policies to be a common use case, we would like to change up the IAMv2 walkthrough to not center on a DENY example.
Documentation Changes
Overview > New Policy Definition

Remove or modify the walkthrough examples that refer to DENY

User's Guide

Re-write custom policy walkthrough to not use a DENY example

also need to clarify and add more content to the global effects of DENY


the accurary is lower than 31.6%(kinetics datasets)

I have used the provided model, but why my test accuracy of the kinetics dataset is lower than yours, Could you please tell me the possible problem or reason, Thanks!

Hi, @Tylerjoe,
Please update to the latest version and download pre-trained models again.


Releasev1.4.1

Addressed #404 (for the record, #405 was raised as Issue).

@ykawashima Thanks for checking and editing! I will release v1.4.1


Build gvim on Linux - Instructions missing.

Cloned the repository and managed to compile easily vim under Linux (Ubuntu 16.04 LTS).  However, I read many web pages on the web and attempted to get gvim to be built, to no avail.  My latest attempt was the following:
./configure \
  --with-global-runtime="${SCRIPT_PATH_ABS}" \
  --with-features=huge \
  --enable-gui=gnome2 \
  --enable-rubyinterp=yes \
  --enable-pythoninterp=yes

The output does not indicate any gnome related issues, yet the 'gvim' binary is not being built.
The documentation provided in the repository should come with instructions on how to build gvim on Linux.

The gvim binary will not be created when you run make.
It is normally created (as a symlink) when you run make install.
If you just want to check if the GUI version works without installing, you can use ./vim -g.

To get the source, on any platform, see http://vim.wikia.com/wiki/Getting_the_Vim_source_with_Mercurial
To compile on Unix-like platforms including Linux, see http://users.skynet.be/antoine.mechelynck/vim/compunix.htm
There are several useful "make" targets:

make config configures (from a clean source) but doesn't compile
make compiles the current configuration; the compiled executable is left in the src/ directory of the source, it is not stripped of its symbols and not installed into your $PATH
make reconfig recomputes the configuration and compiles it. It is necessary in two cases:
** if you have changed your configuration
** if the libraries Vim needs have substantially changed, for instance after an upgrade of the OS
make install (compiles if the source has changed, and) installs the executable, the runtime files, everything -- in /usr/local/bin/ and (for version 8.0) in and under /usr/local/share/vim/vim80/ unless you specify something else (which I do not recommend unless you are a Linux distribution).
make installruntime installs only the runtime files
make installvimbin installs only the executable
and more

In all cases, I recommend to set the configuration by environment variables, not by running configure manually, because with environment variables you will get no bad surprises if make decides to recompute your configuration by running configure before the actual compile.
Best regards,
Tony.

It is mentioned briefly at :h gui-x11-start:
So if you call the executable "gvim", or make "gvim" a link to the executable,
then the GUI version will automatically be used.  Additional characters may be
added after "gvim", for example "gvim-5"

Ok.  Am I the first person who asked about this???  This is not intuitive.  May I suggest that the documentation within the source, a BUILD.md for instance, would explain this?  And 'make' would actually create in the src/ directory a symlink gvim -> vim to give a clue?  I had to open a bug report and bug all of you (thanks for responding!) because I could not find the information.
Should I open a new bug report to ask for better documentation?

Heh.  As a unix user (pretty much exclusively for the past 10 years) I'd argue that it isn't strange at all.  We are used to behaviour of a binary depending on the executable basename.  i.e. this:
[grochmal@haps ~]$ ls -l `which gvim`
lrwxrwxrwx 1 root root 3 Nov 16 02:06 /usr/bin/gvim -> vim

Is the same case as with sh:
[grochmal@haps ~]$ ls -l `which sh`
lrwxrwxrwx 1 root root 4 Dec 10 18:55 /usr/bin/sh -> bash

And bash does behave differently when run as sh (e.g. different default PS1).  And it is make install for bash that creates the soft link.
On the other hand I have no clue how the Vim installation works on MS Windows.

On Thu, Dec 22, 2016 at 11:31 PM, Michal Grochmal
<EMAIL_ADDRESS>wrote:

Heh. As a unix user (pretty much exclusively for the past 10 years) I'd argue that it isn't strange at all. We are used to behaviour of a binary depending on the executable basename. i.e. this:
[grochmal@haps ~]$ ls -l which gvim
lrwxrwxrwx 1 root root 3 Nov 16 02:06 /usr/bin/gvim -> vim
Is the same case as with sh:
[grochmal@haps ~]$ ls -l which sh
lrwxrwxrwx 1 root root 4 Dec 10 18:55 /usr/bin/sh -> bash
And bash does behave differently when run as sh (e.g. different default PS1). And it is make install for bash that creates the soft link.
On the other hand I have no clue how the Vim installation works on MS Windows.

On Windows, it is not possible to use a single executable to run as
both a GUI and a console utility, regardless of soft or even hard
links (IIUC, both kinds are possible in modern versions of Windows),
so vim.exe and gvim.exe must be different files. But the OP was asking
about Linux.
Best regards,
Tony.


fix(iceberg): fix map - arrow convert & fix sink nested types to iceberg

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

fix #18451
fix #16545
Note: source with nested type is not supported yet, because not implemented in iceberg-rs #18474
Note: create table in risingwave side is not supported yet https://github.com/risingwavelabs/risingwave/issues/18363#issuecomment-2340432750

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

ping @risingwavelabs/cargo-lock


Mouse Hover not triggering javascript

Which version of PhantomJS are you using? Tip: run phantomjs --version.
PhantomJS version 2.1.1

What steps will reproduce the problem?
I'm running a data extraction program to get pay data from a corporate web site.

Running selenium 3.4.3 using Webdriver option with Python

Using Firefox gecko driver, my program has no problem using the following sample code to hover over a link, triggering the link javascript, and then clicking on the link to get a popup window to open.
bb = driver.find_element_by_link_text('Activity Sheet')
ll = bb.location
act = ActionChains(driver)
need to hover over element to load javascript.
act.move_to_element(bb)
act.perform()
time.sleep(1)
print('Link is at location: ' + str(ll) + ' Enabled? ' + str(bb.is_enabled()) + " Displayed? " +
str(bb.is_displayed()))
driver.save_screenshot('scrshotHoverActivSht.png')
act = ActionChains(driver)
act.click(bb)
act.perform()

Running the same code as with the firefox driver, using the phantomjs driver, the "Activity Sheet"  link is not triggered. Clicking on the link has no effect. The lack of link trigger is seen in screen captures, as the link text never shows an underline associated with a mouse hover over the link. The move step and location check shows that the mouse 'is in the right place' in both driver configurations.
There is no error issued, the "click" with JS is done on a 'dead' link, IE, one with the javascript not triggered.

The link associated javascript is:
Activity Sheet

Which operating system are you using?
Windows 10

Did you use binary PhantomJS or did you compile it from source?
binary

Please provide any additional information below.
The problem appears to be that the mouse hover event never occurs. The link never becomes active, so the popup window expected never appears. Screen capture with firefox running shows an active underlined link, with phantomJS, no underline.

PhantomJS maintainer Vitaly is stepping down, he may be coming up with another PhantomJS-like project for headless Chrome. Until then, there might be only community support on issues.
In the meantime, you may want to check out CasperJS project, a well-documented navigation and testing tool based on PhantomJS, so using it is probably very familiar to a PhantomJS user. The main maintainer is working on adding headless + visible Chrome support now.
Also, I just finished developing TagUI tool base on CasperJS which can automate headless + visible Chrome. So users of the tool can transit to CasperJS easily once it officially supports Chrome.

Having the same issue here.
def hover(self):
        WebDriverWait(self.driver, 10).until(EC.visibility_of(self.root))
        ActionChains(self.driver).move_to_element(self.root).perform()

def test():
        assert not category.is_highlighted()
        category.hover()
        assert category.is_highlighted()

This will not change the class of the category element. I. e, JavaScript doesn't seem to be executed.
I'm still trying to figure it out though. Let me know if you have an answer or workaround.


V2 Question: How to fully minify/uglify CSS/JS ?

I would like to uglify and compress my JS and CSS as much as possible after every build. Should I use the old cordova-uglify method like how it's done in version 1? or is there a better way to do it now?
Ionic Version: 1.x / 2.x
2.x

Sorry but this question is better suited for the forum: https://forum.ionicframework.com/
However, it is on our roadmap to make it easy to minify and v2 builds, we should haven't completed it yet. Thanks


speech_recognition with ctc_loss failed

❓ Questions and Help
I wanted used "--criterion ctc_loss" instead of "--criterion cross_entropy_acc" and other parameters keep the same with examples/speech_recognition/README.md, I got a "TypeError: build_criterion() missing 1 required positional argument: 'task'" and I don't know how to fix this error

+1, i get a same error after replacing "cross_entropy_acc" with "ctc_loss", and i doubt it is caused by the override function "build_criterion", since i have never seen it in "cross_entropy_acc"'s module. How can i use ctc loss?

i rty to comment “build_criterion” in the CTC_loss.py, then i can build the model and criterion, However, fairseq raise another error;
'CTCCriterion' object has no attribute 'args'
which is similar as issue
https://github.com/pytorch/fairseq/issues/1802
So this is a bug?

This should be fixed now

Thanks a lot, BTW, there maybe anpther bug, when i use the infer.py in /examples/speech_recognition, when batchsize greater than 1, e.g., bsz=2, a Exception will be raised as follow:
memoryError: cuda runtime error(77):an illegal memory access was encountered at...
however, it will be fine if i use batchsize=1, while it is really slow when predict


[Bug]: poetry version incompatibility between llama-index and core

Bug Description
(poetry-test-py3.12) ➜  poetry-test poetry lock
Updating dependencies
Resolving dependencies... (0.6s)

Because llama-index (0.10.50) depends on llama-index-core (0.10.50)
 and no versions of llama-index match >0.10.50,<0.11.0, llama-index (>=0.10.50,<0.11.0) requires llama-index-core (0.10.50).
So, because poetry-test depends on both llama-index (^0.10.50) and llama-index-core (^0.10.50.post1), version solving failed.
(poetry-test-py3.12) ➜  poetry-test

Version
0.10.50.post1
Steps to Reproduce
Use this pyproject.toml (generated using poetry init)
(poetry-test-py3.12) ➜  poetry-test cat pyproject.toml
[tool.poetry]
name = "poetry-test"
version = "0.1.0"
description = ""
authors = ["Yi"]
readme = "README.md"
package-mode = false

[tool.poetry.dependencies]
python = "^3.12"
llama-index = "^0.10.50"
llama-index-core = "^0.10.50.post1"

[build-system]
requires = ["poetry-core"]
build-backend = "poetry.core.masonry.api"

Relevant Logs/Tracbacks
No response

Obviously if I remove llama-index-core from pyproject.toml poetry will be happy, but it'll refuse to install the latest bugfix release of core.

yea, llama-index is tied to specific releases of core to help with upgrades and downgrades at the llama-index level. The tradeoff here is what you've run into.
There will be a new release probably tomorrow, and then you'll get the latest version :)


join: improve performance

This adds a series of performance improvements to join:

Lock and buffer stdout, to save on costs in our single-thread use case and prevent overhead from things like writing single newlines.
Use faster techniques for finding and keeping track of fields.
Make an intelligent guess of how many fields there will be, rather than repeatedly growing every single line.

See the following benchmarks (each of the join executables has the cumulative improvements up to that commit, not just the respective one):
$ hyperfine -w 5 \
 "~/Documents/coreutils/join.main -i -t, -j14 kitch_bench.csv wr_bench.csv" \
 "~/Documents/coreutils/join.buffered -i -t, -j14 kitch_bench.csv wr_bench.csv" \
 "~/Documents/coreutils/join.field_parsing -i -t, -j14 kitch_bench.csv wr_bench.csv" \
 "~/Documents/coreutils/join.field_guess -i -t, -j14 kitch_bench.csv wr_bench.csv" \
 "join -i -t, -j14 kitch_bench.csv wr_bench.csv"
Benchmark 1: ~/Documents/coreutils/join.main -i -t, -j14 kitch_bench.csv wr_bench.csv
  Time (mean ± σ):     701.5 ms ±   8.5 ms    [User: 658.5 ms, System: 42.9 ms]
  Range (min … max):   690.4 ms … 716.4 ms    10 runs

Benchmark 2: ~/Documents/coreutils/join.buffered -i -t, -j14 kitch_bench.csv wr_bench.csv
  Time (mean ± σ):     471.3 ms ±  12.7 ms    [User: 444.6 ms, System: 26.6 ms]
  Range (min … max):   457.1 ms … 500.9 ms    10 runs

Benchmark 3: ~/Documents/coreutils/join.field_parsing -i -t, -j14 kitch_bench.csv wr_bench.csv
  Time (mean ± σ):     257.2 ms ±   4.4 ms    [User: 227.4 ms, System: 29.7 ms]
  Range (min … max):   252.2 ms … 266.7 ms    11 runs

Benchmark 4: ~/Documents/coreutils/join.field_guess -i -t, -j14 kitch_bench.csv wr_bench.csv
  Time (mean ± σ):     194.8 ms ±   4.1 ms    [User: 170.4 ms, System: 24.6 ms]
  Range (min … max):   188.3 ms … 202.9 ms    15 runs

Benchmark 5: join -i -t, -j14 kitch_bench.csv wr_bench.csv
  Time (mean ± σ):     240.0 ms ±   5.5 ms    [User: 212.6 ms, System: 27.5 ms]
  Range (min … max):   233.7 ms … 251.4 ms    12 runs

Summary
  '~/Documents/coreutils/join.field_guess -i -t, -j14 kitch_bench.csv wr_bench.csv' ran
    1.23 ± 0.04 times faster than 'join -i -t, -j14 kitch_bench.csv wr_bench.csv'
    1.32 ± 0.04 times faster than '~/Documents/coreutils/join.field_parsing -i -t, -j14 kitch_bench.csv wr_bench.csv'
    2.42 ± 0.08 times faster than '~/Documents/coreutils/join.buffered -i -t, -j14 kitch_bench.csv wr_bench.csv'
    3.60 ± 0.09 times faster than '~/Documents/coreutils/join.main -i -t, -j14 kitch_bench.csv wr_bench.csv'

The GNU join tested is from the GNU coreutils 8.32, from the default package shipped in Ubuntu 21.10. The kitch_bench.csv and wr_bench.csv files are made of public address data, with some minor post-processing (IIRC, just sorting the generated originals, but I could have forgotten another step or two). They are 14 MiB and 38 MiB, respectively, with many but not entirely common records. As you can see, all told, we're now 3.6x faster than the main branch currently, and 1.2x faster than GNU.
Also included is a commit adding some minor improvements to error handling (necessary to allow flushing stdout if we're about to terminate from unordered lines; without this, the GNU test will occasionally fail), and a commit adding documentation on benchmarking join.
The last of these optimizations, guessing how many fields there will be, has a minor worst-case scenario: we guess that the number of fields in a line is going to be the greatest number of fields we've seen in a line yet in this file, which in the typical case of "every line has the same number of fields" works perfectly, but in the unusual case of "a single line at the start with far more fields than any other line in the file", causes some memory overhead. Since each Line is only kept in memory long enough to find all its matches, and the process would need enough memory for the single long Line anyway, this shouldn't be much of an issue.

wahou, terrific. Is it possible to add some tests to cover the uncovered lines? (mostly error mgmt)

Hmm... I can definitely add a test for a failure on --check-order, that probably should have been there before this PR anyway, but most of the new untested lines are for handling the IOError case. Does the testing framework have a good way to consistently cause reading an open file or writing to stdout to fail? Off the top of my head, the only way I know to cause that in the real world (without some sort of file system failure or kernel bug) is to use network sockets, which seems a bit heavy-handed here.

Okay, I found some other tests that use /dev/full to break stdout, that should hopefully get us most of the way there. That and --check-order are now tested. I still don't know how to create bad reads on command.
I also added a commit to make sure when using --check-order that stdout gets flushed before printing the final error message.


pywebview 5.0 will give random errors > fix pywebview version to 4.3

Question
I just spent 2 days finding and battling random javascript errors (example below) when generating and running sequences of charts (using show_async). I finally tracked it down to an issue with pywebview version 5.x. As soon as I fixed pywebview to version 4.3 all issues disappeared.
I love this library though: While the documentation and examples could be better, with some effort it's definitely possible to create exactly the same charts as we see in TradingView, while using Python.
Code example
[pywebview] {'name': 'TypeError', 'message': "undefined is not an object (evaluating 'line.color')", 'line': 329, 'column': 28, 'sourceURL': 'about:blank', 'stack': 'makeLineRow@about:blank:329:28\neval code@\neval@[native code]\nglobal code@about:blank:4:33'}

Fixed in the latest version.
Louis


Configs get loaded more than once if parent body is updated again

What Happened?
If the primary body that a custom moon orbits is updated, the custom moon also gets updated again.
Problem with that is it will add everything that is already on it again.
Which can get quite extreme if you have multiple configs updating the primary body.
Here is one moon config with 6 planet configs. There should not be 6 ground spheres and orbit lines, but because it gets reupdated 6 times that happens.

Here's an example mod.
MegaPiggy.MultipleLoadBug.zip
What was supposed to happen?
Should only load a config once

whar


Error circle

I have a problem with circle.test test. My program is starting from (0,0,90) and is returning to (0,0,90). In the reference file, the robot is returning in (2,2,90), which is  not correct. Please repair the test.

The test is correct, do not round up numbers until you display them.
On Sat, Mar 21, 2020 at 12:52 PM Mariaoana<EMAIL_ADDRESS>wrote:

I have a problem with circle.test test. My program is starting from
(0,0,90) and is returning to (0,0,90). In the reference file, the robot is
returning in (2,2,90), which is not correct. Please repair the test.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/UPB-FILS/alf/issues/32, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVU5IKXSOFQMR7NSPSA5MDRISL7HANCNFSM4LQ34RUA
.

--
Alexandru RADOVICI
Universitatea "Politehnica" din Bucuresti
e-mail<EMAIL_ADDRESS>telefon:<PHONE_NUMBER>
www.ipworkshop.ro


How long support audio?

How long support audio  Or when do you have this plan  ？thanks

Audio annotation tool will be released around Q1 2023.
We will share the product roadmap soon!


Instructions seem to refer to index.html while TODO is in app.js

Setup Instructions
To run this example locally, either download the whole ReactFire repository or just this /commentsBox/ directory. Then replace the example Firebase URL with your own Firebase URL in the index.html file:
var firebaseApp = "https://my-firebase-app.firebaseio.com/"

// IMPORTANT: Replace below with your Firebase app URL
var firebaseUrl = "https://my-firebase-app.firebaseio.com/";

Thanks for reporting! I'll get this and #96 fixed soon.

Should be fixed by #96.


Decide on the guidelines for the type of vectors to be used when defining an interface

This issue is part of a broader discussion initiated with https://github.com/dic-iit/bipedal-locomotion-framework/issues/94
In particular, in case we have to develop an interface which requires a vector as input, we have to decide what to use. We have several possibilities:

eigen vectors, more specifically Eigen::Ref
GenericContainer::Vector
templates
...

In this issue, we can share thoughts and find an agreement.

Right now in the code, we have two approaches:

only Eigen in the header, see https://github.com/dic-iit/bipedal-locomotion-framework/blob/master/src/Planners/include/BipedalLocomotion/Planners/QuinticSpline.h
the use of GenericVector plus templates. The corresponding usage results in something like https://github.com/dic-iit/bipedal-locomotion-framework/blob/master/src/ParametersHandler/tests/ParametersHandlerTest.cpp#L72-L77

For the first solution we have:

Pros

the interface is directly usable with Eigen and Eigen-compatible objects (I am thinking about manif and cppAD)
the input objects can be used straight away for math operations without using toEigen

Cons

Resizing an Eigen::Ref can cause issues, so it should be used only when the input size is known beforehand
Requires using maps when the interface has to be used with std, iDynTree or yarp vectors.

For the second solution, we have:

Pros

It can be used with any type of vector usable with Span
Input vectors can be resized, provided that the original vector was resizable.

Cons

It requires defining an additional templated method, or to convert the input vector to GenericContainer::Vector manually.
Math operations require toEigen.

If I have to decide when to use one with respect to the other, I guess that I would distinguish the following two cases:

Math interface
I/O interface

For the first, I would go for Eigen objects, hence leaving the street open to future usages of automatic differentiation tools. I am not expecting the same need on libraries which take care of reading/writing outputs from/to other "systems". I call these, I/O interfaces, which include libraries to read parameters or to interface with the robot.
@GiulioRomualdi @prashanthr05 what do you think?

If I have to decide when to use one with respect to the other, I guess that I would distinguish the following two cases:
  - Math interface
  - I/O interface

For the first, I would go for Eigen objects, hence leaving the street open to future usages of automatic differentiation tools. I am not expecting the same need on libraries which take care of reading/writing outputs from/to other "systems". I call these, I/O interfaces, which include libraries to read parameters or to interface with the robot. in this case, I would go for GenericContainer::Vector since it allows dealing with different types of vectors of unknown dimensions natively.

I am totally in line with this.
I have a question regarding the Eigen::Ref though.

Resizing an Eigen::Ref can cause issues, so it should be used only when the input size is known beforehand

Is there any significant distinction between Eigen::Ref<Eigen::VectorXd> and Eigen::VectorXd&. Is it ok to resize a Eigen::VectorXd& type argument passed to a function?

Is there any significant distinction between Eigen::Ref<Eigen::VectorXd> and Eigen::VectorXd&. Is it ok to resize a Eigen::VectorXd& type argument passed to a function?

See https://eigen.tuxfamily.org/dox/TopicFunctionTakingEigenTypes.html. More specifically, if you set as input parameter an Eigen::VectorXd& you cannot use that method with an Eigen::map for example, at least not without allocating an Eigen::VectorXd&.

I would like to keep the interface Eigen only as much as possibile, and avoiding to create our custom types.
In the parameter handler, the GenericVector was required to handle different kinds of vectors (i.e. vectors of std::string).
My personal idea is:

In theory blf should contain yarp/iDynTree only in some small part of the code (communication with the robot,  kinDynComputation, etc)
Avoiding the toEigen() here and there is plus of eigen
The nice thing about the Ref is the fact that you can do.void foo(Eigen::Ref<Eigen::MatrixXd> m);

int main()
{
    Eigen::MatrixXd m(100,100);
    foo(m.block(10,10,23,41));
}

similarly, it can be done with the vectors.
Resizing an Eigen::Ref can be generally done, if the resize is not possible an error is thrown at runtime (in debug).
I would vote to keep the GenericVector in:

the parameter handler
Robot interface only

For point 5, I would add some template capability (I don't know how and what) to avoid to do this
foo(std::string, GenericContainer::Vector, GenericContainerVectorResizeMode)

It would be nice to have
foo(std::string, GenericContainer::Vector)

and then the GenericContainer::Vector is built to be resizable or not according to the initial vector. (I think that Eigen::Ref does something similar since you can build an Eigen::Ref<Eigen::Vector3d> )

For point 5, I would add some template capability (I don't know how and what) to avoid to do this
foo(std::string, GenericContainer::Vector, GenericContainerVectorResizeMode)

It would be nice to have
foo(std::string, GenericContainer::Vector)

and then the GenericContainer::Vector is built to be resizable or not according to the initial vector. (I think that Eigen::Ref does something similar since you can build an Eigen::Ref<Eigen::Vector3d> )

I thought about this. I later realized that the issue here would be that the choice of being resizable or not would be done by the developer, and not by the user. This may be undesirable, since the user may use fixed-size vectors with methods that have been conceived by the developer to work with resizable arrays. Right now this is possible, and eventually, a runtime error is thrown if there is an attempt of resizing the vector, but otherwise it works fine. Instead, if we set this property from template, it would be a strong requirement already a compilation time.
Notice that right now the interfaces using a GenericContainer::Vector are already like
foo(std::string, GenericContainer::Vector)

The resize mode is added to the template methods that do the conversion to a GenericContainer::Vector (see here).
So, the real problem is that it is not possible to convert a random vector to a GenericContainer::Vector reference.

So, the real problem is that it is not possible to convert a random vector to a GenericContainer::Vector reference.

This should be solved by #102. After this, the code that was like
void foo(const GenericContainer::Vector<double>& input);

can be turned in
void foo(const GenericContainer::Vector<const double>::Ref input);

while
void foo(GenericContainer::Vector<double>& input);

will become
void foo(GenericContainer::Vector<double>::Ref input);

The advantage of this, it that it can be used with all the vectors supported by GenericContainer::Vector. I have tried it also with block operations. The problem here is that this can be used only with the const version. Infact, block.data() returns a const pointer, since the memory may not be contiguous. The nice thing about Span though, is that it has a handy method called subspan. Hence, at least for vectors, we can achieve the same result with
Eigen::VectorXd test;
void foo(iDynTree::Span(test.data(), test.size()).subspan(5, 2))

Unfortunately, at the moment, it is not possible to use iDynTree::make_span with Eigen objects, but we can fix that.

Unfortunately, at the moment, it is not possible to use iDynTree::make_span with Eigen objects, but we can fix that.

I guess this has been fixed https://github.com/robotology/idyntree/pull/766

We are currently using Eigen::Vector Similar to https://github.com/ami-iit/bipedal-locomotion-framework/issues/96#issuecomment-2040044606 I'm going to close this


[RFC]: Add C implementation for @stdlib/math/base/special/tribonacci

Description
This RFC proposes adding C implementation for @stdlib/math/base/special/tribonacci.
int stdlib_math_tribonacci( const int x );

Related Issues
None.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

I would like to help out on this issue if able.

@Jordan-Gallivan That would be great! Let me review and get https://github.com/stdlib-js/stdlib/pull/783 merged first. Once merged, I'll circle back and you can begin work. https://github.com/stdlib-js/stdlib/pull/783 will be useful as a reference implementation you can use to help with the tribonacci work proposed here.

@kgryte Thanks! I'm new to the open source contribution so I wanna make sure I do this by the book.  Looking over the project and referencing other implementations, it looks like it should just reference the tribonocci value at an index of an array of tribonocci numbers.  That should be super easy to put to practice, I just wanted to make sure you don't want the implementation to do the actual recursion to generate the tribonocci value of n

Yes, that is correct. A lookup table is fine and follows the existing implementation strategy.

@Jordan-Gallivan In the meantime, you may want to consult our contributing guide and the linked development guide for help getting your local dev setup up and running. If you encounter any issues, LMK.

@Jordan-Gallivan #783 was merged. You can use fibonacci as a reference implementation which can be replicated for tribonacci.

@kgryte Just submitted my Pull Request.  Please let me know if that answers the mail.
I really appreciate you letting me contribute to this project.  Like I said, I'm new to open source and every rep helps me get better, and I hope I helped the project!
Feel free to pass along any feedback good/bad/otherwise so I can continue to improve.  I'd be happy to help again in the future if this is up to snuff.
Thanks!

@Jordan-Gallivan Awesome! Welcome to open source! Let's plan on continuing the feedback loop on the PR, which will hopefully be a good avenue for introducing you to more of stdlib. :)

Great work @Jordan-Gallivan! Congratulations for your first PR! 🎉

@Pranavchiku thanks! Happy to help


adding gitter tips and fb group

hold off until I can add the photos ;)  per usual.   Thanks!!


Function identifiers are allowed to be shadowed in their bodies

Both of the snippets below compile just fine, but should not, since variable shadowing is not allowed and functions in Tact are recursive by default:
fun rec(): Int {
    let rec: Int = 42;  // shadowing error
    return rec;
}

fun rec(): Int {
    let rec: Int = 42;  // shadowing error
    return rec();
}

@vitorpy since you've already opened a PR, can you leave a comment here so that I can assign you to this issue?

Sure thing - I'll see if I can find some time next week to get back to it.


Edit, delete icons for guides on profile page could use tooltips or other text hints

On the profile view, the edit and delete controls are icons. While these make sense from experience, less skilled users may not be able to identify them well.

The 'edit garden' control below is a lot more user friendly:

Mind if I tackle this?

Hey @AustinLMayes, thanks for offering, but there's already a PR by @speizerj for this in the works: https://github.com/openfarmcc/OpenFarm/pull/832

@simonv3 Think this should be closed? #832 resolved it.

@TanSA05 Feel free to close issues when you come across something you think should be closed!

Okay.


Rename citizen to user

It's only for the collection name, or we want to change "citizen" into "user" everywhere in the app (variable name etc...) ??

Everything. Under development, all the data tier has been renamed; we just need to complete it.

Can this renaming be the cause of notifications not working??
I get: [Fri Jan 16 2015 14:48:06 GMT+0100] ERROR: action resolve failed forgot-password (54b91694b29d7a0d3f983258) {"message":"user not found"}

@dalareo Check that your environment variables are set correctly. If you are using v0.9 verify that the mongoDB URL should be MONGOHQ_URL instead of MONGO_URL.

It is working now for me. It was "citizens" / "users" stuff that make notifications fail.

@dalareo check your configuration. If you stick to master branches, stuff should work. If you go with development stuff might not without some adjustments.


Please use Active/Inactive or True/False status instead of 0 and 1 values in the Attributes section.

Describe the Enhancement
When check the attributre section there we can see that the Required, Unique, Locale Based and Channel Based status values are appearing 0 and 1. It should be either Active/Inactive or True/False.
Steps To Reproduce

Go to Admin -> Catalog -> Attributes -> Check the Status value is appearing 0 and 1
Check Required, Unique, Locale Based and Channel Based

Expected Result
The values must appear as Active/Inactive or True/False.
Actual Result
Currently we can see that the boolean values are appearing.
Screenshots
Please find the screenshot here

Checked and Found that the issue has been fixed and PR have been merged and hence I am closing the issue. Please check the screenshot for the same.
https://prnt.sc/tkVNhGreo529
https://prnt.sc/qc0-hDu4ggxL


[WIP] *: upgrade grpc-go to v1.6.0, add watch with client keepalive test

grpc-go v1.6.0 has keepalive fix.
And add watch with client keepalive.

let's do the upgrade in one PR and adding test in another one?

@xiang90 Sure.

lgtm if tests are passed


Rebase and flatpickr

Rebase @bechir work to comply with 6.2 branch
Resolve #1183
Fix flatpickr as mandated by @bechir in https://github.com/symfony/demo/pull/1183#issuecomment-773693644

About Intl js date management, moment is not recommanded anymore, but they advice to use their new Intl Date wrapper lib Luxon or Temporal

Thanks for this contribution. Let's see if this time we can finish and merge this 🙏
I tested it locally and found some issues.
(1) The datepicker doesn't work for me:

And the error details:

(2) Some spacing and font sizes look too different. The new design doesn't have to be identical to before, but it should be closer.
See before/after comparisons:

In the above screenshot, let's tweak the top bar and the form fields to look like before, but let's keep the new bi buttons because they look better than before.

The language spotter in flatpicker.js is now fixed, just an oversight. At first I added the language as a data-attribute of the body, but I finally said to myself that it was already present in the html tag, so I had corrected too quickly...
You can now test the feature, I'll have a look for design adaptations.

These differences come from the change to Bootstrap 4 and conforms to the standard Bootstrap 4 interface standards.
Units are basically changed to rem rather than pixels.
They changed the size of the inputs, the functioning of the navbar...
They used a scaffolding script (_scaffolding.scss) which differs from the new reboot system (_reboot.scss) in many points.
I'm not sure I've mastered these concepts enough to act properly on the interface, but if someone reading this in the assembly is an UX/UI master, show up with pleasure!

It seems better ! Can you give a new try please ?

This looks fantastic now! Thanks a lot Arnaud.
However, there's two minor issues related to the datepicker. Please, tell me if I'm doing something wrong:
(1) There's a design issue with the form field icon:

(2) The datepicker works for me in all languages:

But in English it doesn't work 🤔

@javiereguiluz Fixed :)

Good job @atierant

Good job @atierant

Good job to you @bechir , the most is yours ! Thanks a lot for this !

This is now merged! Thanks Arnaud.


[0580] Amended form base

Context
The base of all models
Changes proposed in this pull request
BaseProviderForm had all the whistle and bells that is needed by any other thing that is not a provider.
Guidance to review
general refactor
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Migrate "Users" Section - Request Access Form


New version: Microsoft.Teams.Preview version <IP_ADDRESS>469

\n\nAuto-updated by vedantmgoyal2009/winget-pkgs-automation
Microsoft Reviewers: Open in CodeFlow

There are many versions of Teams that are there in both Stable and Preview channels.

The installer URLs:

According to the APIs mentioned by @itzlevvie at https://github.com/vedantmgoyal2009/winget-pkgs-automation/issues/157#issuecomment-962494234:

Preview: https://teams.microsoft.com/desktopclient/installer/windows/x64?ring=ring3_6 https://teams.microsoft.com/desktopclient/installer/windows/x32?ring=ring3_6 https://teams.microsoft.com/desktopclient/installer/windows/arm64?ring=ring3_6
To get MSI installers, replace .exe with .msi.
Production: https://teams.microsoft.com/desktopclient/installer/windows/x64?ring=general https://teams.microsoft.com/desktopclient/installer/windows/x32?ring=general https://teams.microsoft.com/desktopclient/installer/windows/arm64?ring=general
To get MSI installers, replace .exe with .msi.

They both return the same version, so my automation updated them to the same version.


Move templates from rice to packr

When changes are made to the event templates, they need to be recompiled. Right now that is only documented in source code at https://github.com/devopsdays/devopsdays-cli/blob/master/event/make_event_files.go#L23
	// to compile, cd to event directory and run `rice embed-go`

That's probably not amazing.
The idea is to migrate from using rice to instead use packr.
There are two main things that need to be done:

[ ] Refactor make_event_files.go and rice-box.go to use packr instead of rice
[ ] Update .goreleaser.yml to run packr as part of the build

@chriscoffee we might use this issue for the template re-working if you want to switch from rice; let me know and I can update the issue :)

If you look at #121 apparently, that's the ticket where I moved a bunch of stuff out of rice and put it into the .go files directly.
I'm going to rename this issue to be more about using packr instead - note that this will change how we release. The upshot of packr is that while doing local development, there is no need to run any commands to compile the templates; it is just something that happens at build time, so the build commands will need to be updated.

Of course, Mark Bates just announced a replacement for packr:
https://blog.gobuffalo.io/introducing-pkger-static-file-embedding-in-go-1ce76dc79c65


compile error

hi, my code need nms and dcn module. so i need from your code to obtian this module. then , i take out the modules from your code , then compile it. but the following error occurred .

By the way， Is there a tutorial for compiling individual components。
Thank you for your reply

The current message does not show the error traceback, so we do not know why it fails.

Please use the error template.


can't download from youtube (video not found)

rene@crow:~/data1/2020 music/2020 magic playlist$ youtube-dl --ignore-errors --extract-audio --audio-format mp3 --verbose https://www.youtube.com/playlist?list=PLN9FZ3ijwb1FZgDJhvjoOHO9S93Tg6Opy
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--ignore-errors', '--extract-audio', '--audio-format', 'mp3', '--verbose', 'https://www.youtube.com/playlist?list=PLN9FZ3ijwb1FZgDJhvjoOHO9S93Tg6Opy']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.03.14
[debug] Python version 3.6.9 (CPython) - Linux-5.3.0-42-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.6-0ubuntu<IP_ADDRESS>, ffprobe 3.4.6-0ubuntu<IP_ADDRESS>, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[youtube:playlist] PLN9FZ3ijwb1FZgDJhvjoOHO9S93Tg6Opy: Downloading webpage
[download] Downloading playlist: 2020 magic playlist
[youtube:playlist] PLN9FZ3ijwb1FZgDJhvjoOHO9S93Tg6Opy: Downloading page #1
[youtube:playlist] playlist 2020 magic playlist: Downloading 118 videos
[download] Downloading video 1 of 118
[youtube] NeQM1c-XCDc: Downloading webpage
[youtube] NeQM1c-XCDc: Downloading embed webpage
[youtube] NeQM1c-XCDc: Refetching age-gated info webpage
ERROR: NeQM1c-XCDc: "token" parameter not in video info for unknown reason; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 785, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 440, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/youtube.py", line 1627, in _real_extract
video_id=video_id)
youtube_dl.utils.ExtractorError: NeQM1c-XCDc: "token" parameter not in video info for unknown reason; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[download] Downloading video 2 of 118
[youtube] QObZCmp46eY: Downloading webpage
[youtube] QObZCmp46eY: Downloading video info webpage
ERROR: QObZCmp46eY: YouTube said: This video is unavailable.
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 785, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 440, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/youtube.py", line 1623, in _real_extract
expected=True, video_id=video_id)
youtube_dl.utils.ExtractorError: QObZCmp46eY: YouTube said: This video is unavailable.
[download] Downloading video 3 of 118
[youtube] LXPCR5JgJUg: Downloading webpage
[youtube] LXPCR5JgJUg: Downloading video info webpage
ERROR: LXPCR5JgJUg: YouTube said: This video is unavailable.
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 785, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 440, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/youtube.py", line 1623, in _real_extract
expected=True, video_id=video_id)
youtube_dl.utils.ExtractorError: LXPCR5JgJUg: YouTube said: This video is unavailable.
[download] Downloading video 4 of 118
[youtube] 0FiuDrfu2D8: Downloading webpage
[youtube] 0FiuDrfu2D8: Downloading video info webpage
ERROR: 0FiuDrfu2D8: YouTube said: This video is unavailable.
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 785, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 440, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/youtube.py", line 1623, in _real_extract
expected=True, video_id=video_id)
youtube_dl.utils.ExtractorError: 0FiuDrfu2D8: YouTube said: This video is unavailable.
[download] Downloading video 5 of 118
[youtube] jhdFe3evXpk: Downloading webpage
[youtube] jhdFe3evXpk: Downloading video info webpage
ERROR: jhdFe3evXpk: YouTube said: This video is unavailable.
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 785, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 440, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/youtube.py", line 1623, in _real_extract
expected=True, video_id=video_id)
youtube_dl.utils.ExtractorError: jhdFe3evXpk: YouTube said: This video is unavailable.
[download] Downloading video 6 of 118
[youtube] cF3OWCYLLVQ: Downloading webpage
[youtube] cF3OWCYLLVQ: Downloading video info webpage
^C
ERROR: Interrupted by user

Well thats not a video. It is a playlist. Maybe try the individual videos.


https://github.com/mnaibei/Capstone-2.git

Hello we are a group of 2 students

Hi Team,
While you put in a lot of hard work in this project, unfortunately, I cannot proceed with reviewing your code.

Invalid Code Review Request 🚫

You should have sent a link to the GitHub issue that has a link to the GitHub project. Please edit the issue to reflect that or submit a new one. 👍🏽

Your Code Review Request will be marked as invalid in your Dashboard, but don't worry you can always try again! 💪
Cheers and Happy coding!👏👏👏
If you have any questions or need clarification on anything, feel free to leave a comment in the pull request thread and don't forget to tag me so I'll receive the notification.
Please note

To avoid confusion and delays, please do not open a new pull request for re-reviews. Use the same pull request submitted for the first review, unless otherwise instructed.

Invalid Code Review Request does not count towards the code reviews limit.


[Event Request] codeunit 5403 AddOnIntegrManagement - OnAfterTransferFromReqLineToPurchLine

In codeunit 5403 AddOnIntegrManagement please add the following event:
procedure TransferFromReqLineToPurchLine(var PurchOrderLine: Record "Purchase Line"; ReqLine: Record "Requisition Line")
    ...
                    end else
                        PurchOrderLine."Overhead Rate" := WorkCenter."Overhead Rate";
                    PurchOrderLine."Indirect Cost %" := WorkCenter."Indirect Cost %";
                    PurchOrderLine."Gen. Prod. Posting Group" := WorkCenter."Gen. Prod. Posting Group";
                    PurchOrderLine.Validate("Direct Unit Cost", "Direct Unit Cost");
                end;
            // new event
            OnAfterTransferFromReqLineToPurchLine(PurchOrderLine, ReqLine);
            // new event
        end;
    end;

[IntegrationEvent(false, false)]
local procedure OnAfterTransferFromReqLineToPurchLine(var PurchOrderLine: Record "Purchase Line"; ReqLine: Record "Requisition Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


BUG: Footer of Schemes page is too large and not appealing.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The footer of the Schemes page is not matching with that of the home page.

Footer of Schemes Page

Footer of Home Page

Record

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

Thank You!!


Concurrent use, running multiple backups at the same time

Hi,
Can multiple instances of restic use the same repository at the same time?
I looked at the docs but I couldn't find an answer to this...
Thanks!
Colin

Yes, you can, but you won't get the optimal deduplication, so the backups will take a bit more space. A run of restic optimize should clear that up, though.
Please close the issue if your question was answered.

Thanks fd0.


Documentation about gathering and displaying results of existing scenarios

The Core tutorial is actually pretty good for teaching the basics:
https://docs.taipy.io/en/latest/tutorials/fundamentals/2_scenario_management_overview/
The problem is as soon as you want something a bit out of the bounds of this tutorial, you're lost. We need more tutorials and demos on advanced core concepts. One of them is how do I gather and display the results of existing scenarios. A good example is in this app:
https://github.com/Avaiga/demo-mlp-regularization

How do I gather the accuracies of my scenarios and display them in a bar chart?

About gathering results.
A scenario comparison feature allows the user to define comparison functions in scenario_configurations.
https://docs.taipy.io/en/develop/manuals/core/config/scenario-config/#using-scenario-comparators
Did you check that?
There is also a tutorial for that:
https://docs.taipy.io/en/develop/tutorials/scenario_management/6_scenario_comparison/
That answers the question of how to gather results. Does it?

About the display:
The display part depends on the comparison results defined by the user, and you can use taipy GUI controls to display the result.

If you want more automation, like not having to define the comparison function, please be aware that we are currently working on designing a scenario_comparator visual element that would fit most of the use cases.
Can you check the two documentation link and close the ticket if it does answer it?

When I was creating the app, I looked through the docs using the search bar. I saw the tutorial
https://docs.taipy.io/en/develop/tutorials/scenario_management/6_scenario_comparison/
But it does not address what I need. I just need a:
existing_scenarios = tp.get_scenarios()
accuracies = [scenario.accuracy.read() for scenario in existing_scenarios]

Then I'll display the accuracies using a chart.
I could not find this within the doc. I don't understand the scenario comparison thing, it looks like a heavy tool for what it does and the tutorial does not show images or anything impressive

I believe the scenario comparison is exactly made for what you need. I agree and confess this API is challenging to understand.
It consists of defining a custom function to provide to your scenario config, taking a list of data nodes as input, and computing the comparison output in the suitable format. Then, you can use standard taipy GUI controls to display the formatted data.
Something like:
existing_scenarios = tp.get_scenarios()
accuracies = tp.compare_scenarios(existing_scenarios, data_node_config_id="accuracy")

The data variable would have the content/format of your choice defined by your custom function. You can use any GUI component to display it.
How would you make it clearer or easier to understand?
What would you expect to be an ideal solution for comparing your scenarios?

The example you show would be perfect:
existing_scenarios = tp.get_scenarios()
accuracies = tp.compare_scenarios(existing_scenarios, data_node_config_id="accuracy")

chart_md = "<| {accuracies} | chart | ... |>"

We would only need to change the scenario comparison tutorial to use this code on an actual use case with an image (like comparing accuracies of different models on a bar chart)

This is anissue about documentation.


fix(sagaInjectors) Prevent sagas to be injected multiple times

This PR prevents the sagas to be injected if its key is duplicate with the key inside the injectedSagas object.
When a duplicate key passed to the injectSaga() function, the current saga in the injectedSagas object will be replaced with the newly created one. So, there is no way we can refer to the old saga and cancel it. Therefore, we need to be careful to not inject a saga that has a duplicate "key".
Related to #1983 and #2061.
Before opening a pull request, please ensure:

[x] You have followed our contributing guidelines
[x] double-check your branch is based on dev and targets dev
[x] Pull request has tests (we are going for 100% coverage!)
[x] Code is well-commented, linted and follows project conventions
[x] Documentation is updated (if necessary)
[x] Internal code generators and templates are updated (if necessary)
[x] Description explains the issue/use-case resolved and auto-closes related issues

Coverage remained the same at 100.0% when pulling 52cb86913ddfc8b7a1ede3733b03c4273a73333e on lvarayut:fix_duplicate-sagas into 9129f21f2c4199a310b105173d45a0bf9475eb30 on react-boilerplate:dev.

Coverage remained the same at 100.0% when pulling 52cb86913ddfc8b7a1ede3733b03c4273a73333e on lvarayut:fix_duplicate-sagas into 9129f21f2c4199a310b105173d45a0bf9475eb30 on react-boilerplate:dev.

Coverage remained the same at 100.0% when pulling 52cb86913ddfc8b7a1ede3733b03c4273a73333e on lvarayut:fix_duplicate-sagas into 9129f21f2c4199a310b105173d45a0bf9475eb30 on react-boilerplate:dev.

Deploy preview failed.
Built with commit a1eabe0b9842fe26f732a6cd0e9cd54e112e56b2
https://app.netlify.com/sites/react-boilerplate/deploys/5a315f45a1147764782632d0

Coverage remained the same at 100.0% when pulling a1eabe0b9842fe26f732a6cd0e9cd54e112e56b2 on lvarayut:fix_duplicate-sagas into 9129f21f2c4199a310b105173d45a0bf9475eb30 on react-boilerplate:dev.

Coverage remained the same at 100.0% when pulling 8cd81a4e3011ca9494d720b84656524e75d94cae on lvarayut:fix_duplicate-sagas into c21184fa8ea06ab478d0e5293189697fca49ff6b on react-boilerplate:dev.

Coverage remained the same at 100.0% when pulling 8cd81a4e3011ca9494d720b84656524e75d94cae on lvarayut:fix_duplicate-sagas into c21184fa8ea06ab478d0e5293189697fca49ff6b on react-boilerplate:dev.

cc @Dattaya


[feature request] Remove Copy bound from Real

Basically the reverse request of #243 - Currently, Real implies Copy, which restricts it from being implemented for real-value types that use non-copy internals (for example, a type that represents values as whole fractions backed by a BigInt). The Copy bound doesn't seem to be actually useful to the trait, and more like a holdover from when it was split out from Float.
This would be a breaking change unfortunately.

Just an added thought - If this change is made, it may be a good time to add Bounded as a super-trait and remove the min_value and max_value methods, as they're documented to do the same things


As a user, I can view the list of doctor sorted by name

So that I can look through the list of doctors logically.

Feature discarded.


tab fix

Author can change the title now
Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #91
Test URLs:

Before: https://main--icicidirect--aemsites.hlx.live/draft/ravverma/videos-podcasts
After: https://fix-tab--icicidirect--aemsites.hlx.live/draft/ravverma/videos-podcasts

Previous authoring in which author can not change the title as it was taking as key to identify api end point

now, in this new authoring , title can be changed

PS: Number of tabs can be added more, it was already there.

Looks like the cards are not showing, do we need to add something to the doc?

As discussed in the meeting block would be like


expert area update

I believe I was close to finishing the expert area logic before this  week's big PR change, but it needs to be integrated. This would also involve an update of the expert logic for items.

began some work on expert logic this morning, found a few things to update in casual too, but it needs to update from today's version

Still working through the expert update and occasionally finding problems to fix in casual. This weekend is not looking too busy so I might get to do some of it and maybe post an update.


getLocation response does not get sent back to MQTT

It appears that the getLocation command, after being returned, does not get sent to the MQTT broker to be send to device_tracker.
Logging output from the docker container:
info: Command sent {"command":"getLocation"} info: Command completed {"command":"getLocation"} info: Command response data {"data":{"commandResponse":{"body":{"direction":{"uom":"degree","value":"0.0"},"location":{"lat":"42.0000000","long":"-71.0000000"},"speed":{"uom":"kmph","value":"0.0"}},"completionTime":"2023-01-26T05:44:53.336Z","requestTime":"2023-01-26T05:44:50.577Z","status":"success","type":"location","url":"https://api.gm.com:443/api/v1/account/vehicles/MY_VIN/requests/location12345678901234567890"}}} info: Requesting diagnostics

It looks like this bug was patched by somebody else -- Perhaps you can merge their change?
https://github.com/BigThunderSR/onstar2mqtt/commit/824831b7967031955a02a324ec8b543d02a28b14

Merged the above fix and pushed 1.5.4. Please confirm, I no longer have the OnStar package for commands on my car.


Incorrect second sample (documentation)

Hey,
regarding the second sample with text "The Australian platypus is seemingly a hybrid of mammal and reptilian creature.", it contains 12 words (not 13) and 23 syllables (not 26).
Otherwise, thanks a lot for the module!


Android Nougat

How should be the behavior in the API 24? Inside my cell phone (Moto G4 plus) didn't take a picture, button didn't works.

@brunozaranza If it's the same problem on the Moto G5+ then it is caused by the fact that the device has a passive Auto focus: when you press the button it actual tries to shoot a photo but the procedure stop because of an unexpected state of the variable afState.
Sent from my Motorola Moto G (5) Plus using FastHub

The app freezes at the line mCameraOpenCloseLock.acquire(); for Android 7.
Any fixes?

This sample has been migrated to a new location where we can accept Pull Requests (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


fatal error: boost/utility/string_view.hpp: No such file or directory

Catch compilation error:
/home/ksergey/dev/gdax_feed_handler/deps/Beast/include/beast/core/string.hpp:11:41: fatal error: boost/utility/string_view.hpp: No such file or directory

In my system installed boost 1.58. Which version of boost is required now?
Will be good to see in Beast/CMakeLists.txt something like find_package(Boost 1.60.0 REQUIRED COMPONENTS  ${BOOST_COMPONENTS})

You need a more recent Boost. I think 1.63, or 1.64.

Thank you!

Keeping this open since there's an actionable request here:
find_package(Boost 1.60.0 REQUIRED COMPONENTS ${BOOST_COMPONENTS})

I'm running Fedora 25 (the latest stable release), and it only has Boost 1.60 available in its main repositories - which means I now can't compile Beast.  Boost 1.60 was only released 6 months ago.
Is the thinking behind this requirement that Beast will be released soon within Boost, so it only needs to support the Boost version it ships in and nothing earlier?

Is the thinking behind this requirement that Beast will be released soon within Boost, so it only needs to support the Boost version it ships in and nothing earlier?

Yes. Although, maybe to get you going I could #ifdef the string_view depending on the Boost version, would you like to make that change? Its in <beast/core/string.hpp>. Something like this:
#include <boost/version.hpp>
#if BOOST_VERSION < 106400
#include <boost/utility/string_ref.hpp>
using string_view = boost::string_ref
#else
#include <boost/utility/string_view.hpp>
using string_view = boost::string_view
#endif

That's fine, thank you!


Homepage

added index.html
added style.css
added linters
checked error in formating

Hi @Kaiserabbas ,

Good job so far! 👏. Your Code is Awesome! 💪
There are some few issues 🐛 that you still need to work 🔧 on though to go to the next project ⏭ but you are almost there! 🏃
Don't Worry 🙏
It's ok to have bugs in your code 🐛. Please, see this code review as a chance for you to learn practical coding skills and learn new tips that can improve your code quality 💯 and help you **get a better ** 💼.
I am going to guide you on how to fix them down below. 🤓
Highlights 🎯

✔ Descriptive PR title and description
✔ Good job using inline comments
The code is top-notch! 💯 and has a huge potential.

Required Changes ♻
Check the comments under the review.
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Important Notes 📝

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Note: Please, don't Merge This Pull Request yet. Only Merge when it is Approved ✅.

Thank you for your time 🙏 and call me if you need help or clarification ☎

This is My GitHub:
This is My Twitter:
This is My linkedIn.
This is My Slack:

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank for your comments, I have added new comments Kindly check and approve thank You


Include code snippet in readme

Can include the key snippet from function_app.py so we can emphasize functions being about code?  Please see C# and JS readme examples

/assign @s-batra

Good point, included!


probleme with mysql

Hello, thank you for this docker.
just a small problem I migrated the docker from windows (docker desktop) to synology I had a problem with the ips but that's fixed so my dockers run on different ips except mysql. so I don't know connect to the database can you help me please? instead of having database for the connection I would like to have an ip address.
I am with the basic files.
phh-8.1
i have this error on docker mysql:
mysqld failed while attempting to check config

command was: mysqld --verbose --help --log-bin-index=/tmp/tmp.Tcr1VBKHVn

mysqld: [ERROR] Found option without preceding group in config file /etc/mysql/my.cnf at line 1.

Hi  @SnoxiK
inside the docker-compose.yml there is a section inside the database area at the end where it says <IP_ADDRESS> .... . You need to change/remove this <IP_ADDRESS> part,  then the container should listen on your bridged network address like the webserver (your NAS adress).

Can't get docker to go following command because there is version error with synology docker but it seems the right thing for normal dockerdesktop :)

My quick fix was to uncomment the default-authentication-plugin section in /etc/my.cnf. I replace the RUN command with
RUN sed -i 's/^# default-authentication-plugin/default-authentication-plugin/g' /etc/my.cnf

Did not work for me.

I will check this today and if necessary I will provide a fix asap.

Please rebuild your stack with docker-compose up -d --build. Should work now. Let me know of you still encounter any issues with the db container.

Please rebuild your stack with docker-compose up -d --build. Should work now. Looks indeed like a change in the upstream image. Let me know if you still encounter any issues with the db container.

Works like a charm. Tested on 20.04.4 LTS (Focal Fossa) in WSL.
thanks!

I've got the same thing. Don't know what caused it. Moved the docker stack from Ubuntu to Arch Linux though. docker-compose up -d --build did not work for me.
dennis@server2:~/helloworld/website/docker-compose-lamp$ sudo docker-compose up
[+] Running 4/4
⠿ Network helloworld_default       Created                                                                                0.1s
⠿ Container helloworld-mysql8      Created                                                                                0.2s
⠿ Container helloworld-phpmyadmin  Created                                                                                0.2s
⠿ Container helloworld-php8        Created                                                                                0.2s
Attaching to helloworld-mysql8, helloworld-php8, helloworld-phpmyadmin
helloworld-mysql8      | 2022-07-29 11:51:24+00:00 [Note] [Entrypoint]: Entrypoint script for MySQL Server 8.0.30-1.el8 started.
helloworld-mysql8      | 2022-07-29 11:51:24+00:00 [ERROR] [Entrypoint]: mysqld failed while attempting to check config
helloworld-mysql8      |   command was: mysqld --verbose --help --log-bin-index=/tmp/tmp.FlTHrnOyM6
helloworld-mysql8      |   mysqld: [ERROR] Found option without preceding group in config file /etc/mysql/my.cnf at line 1.
helloworld-mysql8      | mysqld: [ERROR] Fatal error in defaults handling. Program aborted!
helloworld-mysql8 exited with code 1
helloworld-phpmyadmin  | AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
helloworld-phpmyadmin  | AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
helloworld-phpmyadmin  | [Fri Jul 29 11:51:25.820068 2022] [mpm_prefork:notice] [pid 1] AH00163: Apache/2.4.53 (Debian) PHP/8.0.19 configured -- resuming normal operations
helloworld-phpmyadmin  | [Fri Jul 29 11:51:25.820141 2022] [core:notice] [pid 1] AH00094: Command line: 'apache2 -D FOREGROUND'
helloworld-php8        | AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
helloworld-mysql8      | 2022-07-29 11:51:26+00:00 [ERROR] [Entrypoint]: mysqld failed while attempting to check config
helloworld-mysql8      |   command was: mysqld --verbose --help --log-bin-index=/tmp/tmp.v84hTe8SLz
helloworld-mysql8      |   mysqld: [ERROR] Found option without preceding group in config file /etc/mysql/my.cnf at line 1.
helloworld-mysql8      | mysqld: [ERROR] Fatal error in defaults handling. Program aborted!

Got my issue fixed via a work-around. My issue was caused because I moved my containers over from an Ubuntu based server to a new Arch based server and thus pulled in the new docker images. The newer version has caused the issue.
Resolved the issue by:

Copying the /etc/mysql/my.cnf from the container to my docker host via docker cp <containername>:/etc/mysql/my.cnf ./config/mysql/my.cnf
Adding an additional bind mount to the docker-compose.yml file for the config file: - ./config/mysql/my.cnf:/etc/mysql/my.cnf
Adding the [mysqld] to the config file, so the file now contains:
[mysqld] default-authentication-plugin=mysql_native_password

Thanks @denniskorbginski for the hint.

@dennisvanderpool I assume you have git pulled before the rebuild ?
You could try to delete you local docker image which is tagged with $COMPOSE_PROJECT-$databaseversion name to force a build.

No update from issue creator - close


Update Moebius to work with latest Elixir.

Fix several warnings
Replace Poison with Jason
Replace Poolboy with ConnectionPool

Note: the Jason lib can't handle Structs out of the box so I had to create a workaround here: https://github.com/cpursley/moebius/commit/3bb49848dfcfef21013f55f4ec5c1aa8d0714d37#diff-1c2325d2ae886d450e703e512eb50de0d7382259c4d5098103f6321dd19b3a40R125

Thank you!


Updating-FAQ-Link(EXPOSUREAPP-8216)

A new FAQ link target for EN.

Please only change values-de, everything has to be delivered via the translation system. so maybe give the translators a heads up to change it accordingly.