[Draft] [Codegen] fix array enum prop parsing for array enum types

Summary:
codegen generates type alias for  array enum props with uint32_t which cause wrong overloaded fromRawValue to call at runtime eventually app to terminate
more detailed info at issue #43821
Changelog:
[Internal] [Fixed] - Codegen for array enum props
Test Plan:
TODO

@dmytrorykun could you check this one?

@dmytrorykun could you help me find use case where the static inline std::string toString method on the props types are called ?

@netmaxt3r I'm looking into it right now.

Hey @netmaxt3r I've reviewed the PR, it looks really good.

could you help me find use case where the static inline std::string toString method on the props types are called ?

Could you please add more context to this question? Why is toString important? I couldn't find any usages of that in the generated code. I think in theory it can be used in product code.
As a followup to this, have you experienced any issues with event emitters? I think there might be similar problems there.

@dmytrorykun

Could you please add more context to this question? Why is toString important?

not important i couldn't get use case to test the function signature change , if its for product code then it is oky

As a followup to this, have you experienced any issues with event emitters? I think there might be similar problems there.

I couldn't see any type aliasing in GenerateEventEmitterCpp.js nor GenerateEventEmitterH.js is there any other place we need to check ?

@netmaxt3r I think the PR is in a good shape. Could you remove the "[Draft]" prefix from the title, and I will import it.


[BACKLOG-40099] - VFS support for Bowls

Major changes:

KettleVFS and ConnectionManager needed to be refactored to support having a Bowl context for most operations
KettleVFS has been refactored a bit, adding IKettleVFS and KettleVFSImpl as per-Bowl implementations. KettleVFS itself has been refactored to use an instance of an IKettleVFS for backwards compatibility.
Most KettleVFS static methods have been deprecated. Callers should now use KettleVFS.getInstance(Bowl) instead.
Some (sadly public) parts of ConnectionManager and KettleVFS need to be single-instance/global, such as the VFS FileManager or the ConnectionManager providers that get registered by plugins. These remain shared.
Because ConnectionManager instances do not share state, even with the same metastore, all ConnectionManagers should now be retrieved from a Bowl instance.
Backwards-compatible methods and default stub methods have been left in place to allow existing code, including possible customer-written code to continue working. Some code will not work with Bowls until they update, however.

Step Changes:

Two Steps, and two Job Entries have been updated to work as examples. NO UI work is ready for these steps in this PR, users need to hand-edit the step config to refer to a project connection by name (e.g. in a pvfs path).
Text File Input and Text File Output steps have been updated
Create File and Delete File job entries have been updated.

Knock-on effects:

ConnectionProvider API has new methods. All VFS providers that need to support Bowls will need to be updated.
Because VFS Filesystems need to be unique and to provide it to downstream usage, like by the Providers, the Bowl is now included in FileSystemOptions. Because of this, all Bowl implementations will need to implement equals() and hashcode()
There was a bug in Effective Java's double-checked locking idiom which blocked testing these changes. (MetaStoreConst, etc)
ResolvableResource seemed unused (in this or other repos), and was difficult to change, so I removed it from a couple places.

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 28m 2s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
core,engine,plugins/connections/ui,plugins/file-open-save-new/core,plugins/json/core,ui

:ok_hand: All tests passed!
Tests run: 4096, Failures: 0, Skipped: 11    Test Results

:information_source:  This is an automatic message


add feature for Forum post search

Description

Related Issue

Solves #

Type of change

[x] New feature (enhancement)
[ ] Bug fix
[ ] Refactor
[ ] Breaking change
[ ] Documentation

How Has This Been Tested?

[x] Automated testing
[ ] Manual testing

Checklist:

[x] Does a similar (open or closed) pull request not already exist?
[x] Is the pull request head repository a fork repository?
[x] Is the pull request compare branch a development branch?
[x] Is the code documented, particularly in hard-to-understand areas?
[x] Does the code build without new warnings?
[ ] Has testing been performed that proves changes are effective and work?
[ ] Has a self- and/or peer-review of the code been performed?
[x] Have dependent changes been merged and published in downstream modules?
[x] Does all new and existing automated testing pass?
[x] Is the person responsible for the repository assigned to the pull request?
[x] Is the pull request linked to a project?
[x] Is the pull request linked to a milestone?

For more information, refer to the Contributing Guidelines document.

Could you add a link in the PR description on what issue this PR is associated with, thanks.


Submodules should be groupable (or have their own group)

I've started grouping commands because that makes them easier to find and read. I'm also using submodules to group certain commands specific to e.g. database commands. However, when listing these using
# Show this help text
[group("General")]
help:
    @just --list --unsorted --justfile {{ justfile() }}

the submodules are simply appended to the grouped output, making it appear as if they are part of the last declared group:
Available recipes:
    [General]
    help                # Show this help text

    [Code Utils]
    format              # Reformat all source files
    lint                # Lint source files

    [Environment]
    chenv new_env       # CHange ENVironment
    genenv              # Generate a .env file based on secrets/{environment}. Set the `ENVIRONMENT` envvar to change environment.
    current_environment # Print the environment of the variables in the current .env file
    app ...             # Application commands
    auth ...            # Authentication module commands
    db ...              # Database management commands

At a minimum, I would expect the submodules to be grouped separately:
    [Environment]
    chenv new_env       # CHange ENVironment
    genenv              # Generate a .env file based on secrets/{environment}. Set the `ENVIRONMENT` envvar to change environment.
    current_environment # Print the environment of the variables in the current .env file

    app ...             # Application commands
    auth ...            # Authentication module commands
    db ...              # Database management commands

At best, I would like to be able to set custom groups for my submodules:
# Application commands
[group("General")]
mod app "src/app/.app.justfile"

# Authentication module commands
[group("Subcommands")]
mod auth "src/auth/.auth.justfile"

# Database management commands
[group("Subcommands")]
mod db "src/db/.db.justfile"

Available recipes:
    [General]
    help                # Show this help text
    app ...             # Application commands

    [Subcommands]
    auth ...            # Authentication module commands
    db ...              # Database management commands

    [Code Utils]
    format              # Reformat all source files
    lint                # Lint source files

    [Environment]
    chenv new_env       # CHange ENVironment
    genenv              # Generate a .env file based on secrets/{environment}. Set the `ENVIRONMENT` envvar to change environment.
    current_environment # Print the environment of the variables in the current .env file

Thanks for trying out modules!
I think these would be good changes. I added a space before submodules in --list when groups are present in #2244.
Putting modules into groups is a larger change, since it involves modifying the parser to accept attributes on modules, but it seems like a good idea, if anyone wants to tackle it.

Awesome, thanks - as always - for the quick fix @casey !

I'm working on a PR https://github.com/casey/just/pull/2247 that allows the doc attribute on modules; that has the parser changes necessary to make adding the group attribute on modules possible as well.

I have a potential fix for this in this PR: https://github.com/casey/just/pull/2263. Feel free to review whenever you get the chance!

@casey Is there any documentation I can update in the README for #2263? I was looking at updating the Recipe Groups and Modules sections with examples for grouping modules, but wasn't sure if that was the way to go. Let me know what you prefer and I'm happy to update the docs.

I just merged #2282, which adds some initial documentation to the attributes section, regarding which attributes apply to which types of items. Feel free to open another PR if you think further documentation would be helpful!


Handle BoM in Doc-Warden

Need to do some additional smoke-test on python 2.7. OR I drop python 2.7 support. I'm leaning that way right now.
@Azure/azure-sdk-eng

I ditched 2.7 support for my python scripts in the Java repo a while back due to unicode support and enums.

@JimSuplizio I did a verification across our repositories. We're definitely using Py36 any place we call warden. I feel safe with this update. Running this build to be certain.

Looks like I need to take a look at python specifically. Given that doc-warden is pinned to version 0.7.0 right now. I'm going to merge this PR and cut a new version, then work through why 0.7.0 has issues with azure-search-documents readme in THAT PR.


chore(deps): upgrade jenkins-x/jx3-pipeline-catalog to version 3.2.339

from: https://github.com/jenkins-x/jx

/retest

/retest

/retest


net: lwm2m: Correctly Support Bootstrap-Delete Operation

Summary

DELETE / is implemented as DELETE /0
DELETE /and DELETE /x for x != 0 is not supported/implemented

Background
Zephyr client should support bootstrap-delete according to the specification.
Currently the support is only implementing partially and that part not according to the specification:

If the client is in bootstrap mode and bootstrap server sends a DELETE request without Uri-Path option, then bootstrap_delete will be executed (wm2m_engine.c:3344)
bootstrap_delete deletes SECURITY instances > 0

Given bootstrap_delete is called when no uri option is set, the assumed contract of that method is that it should correspond to DELETE /  where all Instances of all Objects in the LwM2M Client MUST be removed (except for the two cases mentioned above) where The two exceptions are the LwM2M Bootstrap-Server Account, potentially including an associated Instance of an OSCORE Object ID:21, and the single Instance of the mandatory Device Object (ID:3), which are not affected by any Delete operation.
Proposal

Fix: Adjust current code to delete all other object instances as well
Enhancement: Support both no Uri-Path header and empty Uri-Path header, and treat them the same way
Enhancement: Support deletion of all instance of a particular object

Side Note
Sending a DELETE /x request to the client in bootstrap mode is executed as DELETE /x/0 (goes through regular delete request handler that does not check if uri path has length >=2 as it is not allowed by the spec outside of bootstrap, object and instance id must be set)

@broglep-work Thanks for reporting. Indeed the current implementation oversimplified a lot. I've pushed a PR with proposed changes:
https://github.com/zephyrproject-rtos/zephyr/pull/30200
I've verified that DELETE / and DELETE /x works as expected, I wasn't able to verify DELETE /x/y though as the bootstrap serves I tried with did not issue such operation. I'm not expecting any surprises there though since the same bootstrap_delete handler shall be called then, which implements single resource removal in a straightforward way.
During the rework I've also realized that bootstrap DISCOVER operation is totally broken, but that's a story for a separate PR.


[UI] Missing link from results page back to Edit Project page

Can't quickly switch between Project Edit and Project Results views.

@edagan - do you have any suggestion on where to put it? How would it look?
Do we want to take it with the Dolly or wait with it post June?

Top right of page, Under "Go To Projects", works for me.
We should consult with Dolly for best placement, however.

hmmm, two 'go to' buttons on the same space can be confusing, what about 'divide and conquer' - one of the buttons can be on the bottom on the other side of the table. @edagan WDYT?

@soninob Whatever you fee looks best - just let me quickly navigate between the views, please :)

Guys, here is what I implemented:

What do you think?

I still find it confusing that we can jump to the results view for a project from the project list, I think that by first going to a Project View that allows you to switch between the Edit View and the Results View.
I think the navigation should follow a more-tree like path structure. This is design pattern that is heavily used and well known by users. Right now, if you do try to do this you can't:

Go to project list view
Click on magnifying glass (results)
Go to Project Edit Link
I am in Project Edit View
I want to look at results, but I can't

So, what I recommend is:

Remove magnifying glass from project list view
Instead make the list items linkable, to follow a more navigational feel
When you go to the Edit View it should look more like a Profile View that has a way to switch or drill down to the Results.
If you choose a drill down mechanism, then you can use a link like the one above. If you use more of a switch view (like tabs) then you don't need it because you will have the tabs available.

Hope that helps!

Thank you @rqc.
Removing magnifying glass will eliminate the option to show results page in one click, which I'm against that.
We have an item to allow jumping to results from the edit view.
@AndreeaDora - I would suggest that the label you added will be changed to a small button with only "Edit Project".

@soninob I did the changes that you requested:

As @rcq mentioned, we should have a tree like path for the navigation. Right now the user could get confused because he is able to go edit the project from the project results view, but he can not go back to the project results view directly from the edit view.
The action item we have in the edit view can be used to get to the project results view or to the test result view:

I would suggest to use it to get to the test result view.


English Please

What is the problem with Chinese README's
Firstly, we congratulate you for getting so much star by sharing this repository with humanity.
But it is very disappointing for non-Chinese speakers when one couldn't understand what a trending repository is about.
When we see such a repo on trending, our minds are blurring like Gollum's.

There is a way you can help to solve this disappointment which I believe is experienced by many people who want to know more about your valuable work and appreciate it.
What we want:

Please add English translation of your README so you are sharing your work and knowledge with more people.

How this will help you:

More feedback to fix and improve your project.
New ideas about your project.
Greater fame.

“Sharing knowledge is the most fundamental act of friendship. Because it is a way you can give something without loosing something.”
— Richard Stallman
Thank you!

The developers of repo will not be held responsible for problems that arise in this script.

This issue generated by us/english-please script. Please open issue on any error. Thank you!

It's very appreciate for  your advice, but the ppt resource of this repositories are written in Chinese, it's not very useful to translate only readme document to English version.


Update migration docs

Forgot to say it earlier, but thanks for the thorough review @fertapric ❤

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


UnicodeEncodeError: 'gbk' codec can't encode character

Hello, first of all thank you for your open source, I'm trying to learn to use this module recently. But today I got the following error when running the model, it seems like there are some errors when I follow the GIt commit? What should I do to fix it  ?

 Put the save_info  function in labml\internal\experiment\experiment_run.py :
    def save_info(self):
        ...
        if self.diff is not None:
            with open(str(self.diff_path), "w") as f:
                f.write(self.diff)

change into :

```python
    def save_info(self):
        ...
        if self.diff is not None:
            with open(str(self.diff_path), "w",encoding='utf-8') as f:
                f.write(self.diff)


Not sure if bug or feature - eggs don't hatch when dispensed

Describe the bug
I'm not sure if this is a bug, or a feature of the mod, or a limitation of the way mods work, but;
I have a world I'm playing where we've got a big flock of ducks from your mod - not purposefully, but there's LOADS of them, and I haven't seen any chickens, so we just decided to farm the ducks for meat.
Earlier today I wanted to make an automated egg-hatching area, because doing it by hand takes a while, and eventually I might build a cooker, so these blocks can be integrated into that. However, I realised that when fired by a dispenser, your eggs just drop as items. At first, I thought this was being caused by my 'better dropped items' mod, but once I had the chance to test a world with that mod removed, I realised it's just the behaviour of the duck and goose eggs from this mod in particular.
TL;DR: Duck and Goose Eggs do not hatch when ejected from a dispenser.
To Reproduce
Steps to reproduce the behavior:

Build your preferred redstone ticker/clock (I used a V of blocks, two torches, a single bit of redstone dust) and dispenser setup
Place eggs (duck or goose) from this mod in the dispenser
Observe that they don't pop
Maybe test chicken eggs if you like

Expected behavior
I expected the eggs to be fired and hatch/crack/pop in the same manner as vanilla chicken eggs.
Version
I am running a Fabric launcher, Minecraft 1.17 pack I'm gradually putting together myself, through curseforge.
Error Log
There's no game-crashing error, nor anything barring the usual data fixer errors, which aren't related to this issue as far as I understand it.

Oh, I didn't notice that. Thanks for bringing this up.
Yeah the eggs should behave just like chicken eggs, this is an oversight at the moment and should be easy to fix.
I'll try to get a new release out next weekend.

any update on this?


error in committees/CommitteeAgendaSearch: failed to get meetings for committee 571

This issue was automatically created, please do not modify the title or remove any labels
Unexpected exception received while trying to get meetings of committee 571: ('Connection aborted.', error(110, 'Connection timed out'))

Encountered the error again
Unexpected exception received while trying to get meetings of committee 571: ('Connection aborted.', error(110, 'Connection timed out'))

Encountered the error again
Unexpected exception received while trying to get meetings of committee 571: ('Connection aborted.', error(110, 'Connection timed out'))

Encountered the error again
Unexpected exception received while trying to get meetings of committee 571: ('Connection aborted.', error(110, 'Connection timed out'))

Encountered the error again
Unexpected exception received while trying to get meetings of committee 571: ('Connection aborted.', error(110, 'Connection timed out'))


Are there related documents about function address offset?

https://github.com/devxoul/Stubber/blob/9f1833587f2ec1b590e0047b00ab11c85d3ae57b/Sources/Stubber/Stubber.swift#L102

I did refer https://github.com/rodionovd/SWRoute/wiki/Function-hooking-in-Swift . But the memory address keeps changing in Xcode 12 everytime I access to the function :(


Implement linear back-off for Shopify API requests

This CI script runs several API requests in sequence and can result in the rate limit being exhausted on the store. This can be quite disruptive to the workflow when validating and merging several PRs against a single store's development (or production) shop.
This PR adds a new parameter, shop_app_api_frequency which represents the frequency at which requests will be made to the Shopify API. The default value is 2, which is the maximum sustained frequency for non-Shopify Plus shops. The frequency can be overridden for Shopify Plus stores by providing a value of 4.
Since we must compute the period (i.e., the amount of time to sleep) by solving (1 / api_frequency), the non-integer math is delegated to bc.
While this rate-limiting behaviour adds a few seconds to the execution time of the script, it prevents the issue of exceeding the rate limit (thus, having to cold boot the whole container and script again which takes a rather long amount of time).

I've signed the CLA but my check still failed due to a bad email configuration on my account, is it possible to have it re-run?

Am running into this rate-limiting issue. Anything I can do to help get this across the finish line?


Enable more entries and column in stress legend

LGTM

Merged build finished. Test PASSed.

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

alluxio-bot, merge this please


Add alpine

Currently fails to build with the following error:
Error: Invalid formula: /home/linuxbrew/.linuxbrew/Library/Taps/homebrew/homebrew-core/Formula/gcc.rb
Error: Invalid formula: /home/linuxbrew/.linuxbrew/Library/Taps/homebrew/homebrew-core/Formula/llvm.rb
Error: Invalid formula<EMAIL_ADDRESS>Error: Cannot tap homebrew/core: invalid syntax in tap!
Unable to locate the system's glibc
Unable to locate the system's glibc
Unable to locate the system's glibc
Error: Failure while executing: /home/linuxbrew/.linuxbrew/bin/brew tap homebrew/core -q

Alpine Linux does not include glibc. It uses musl libc instead, so this error is not surprising. Rather than an error, perhaps not being able to find the host glibc should be a warning.

Alpinne Linux has a glibc compatibility package libc6-compat.


Final Group Capstone - Mentor Me Now - Front-end: Implemented the required changes

This branch has already undergone review in this PR. It was established to address the modifications suggested by the previous reviewer.
Modifications Implemented:

Corrected the link for the reserve button on the Mentor details page.
Included a link to the Kanban Board and an image of the initial state.

Here is the link of the Back end PR

Thank you, @Meltrust, for your dedication and for approving the changes.


Top Categories Section [Home Page]

Add a "Top Categories" section below the Home Page
It should have six cards (2X3):
- Mithai Masti (Indian Sweets)
- Nashta ( Indian Breakfast)
- Chaat and Snacks
- Fasting Food
- Drinks
- Main Course

Each card should have an image ( add some box-shadow to the card so that it is visible separately from the background-color) and on hover effect. Check Sample.
On Hover, there should be a button "Check Out" which would redirect to the particular page (to be created later. Keep # for now)

Can i work on this issue ?

Can I work on this issue?

Any update?

I want to work on this issue under JWOC

@smaranjitghose Am sorry if I am late. Have been working on it. A little newbie to work efficiently as learning takes time.

@smaranjitghose Shall I put these cards in a carousal with three dots at the end or just display them in a 2 x 3 fashion. Am thinking that for now i will just display them in grid and in the next issue(if created for adding carousal), i shall work on adding theses cards in carousal. How does it sound?

You can simply display them in 2 X 3 grid.
Try to implement the hover effect suggested in the reference above
For the categories use: the three titles mentioned in the design wireframe (in the description) + Vegetarian Vendetta + Snack Satisfaction + Beverages
No description text is required.
Make sure that the text and the images can be defined inside a simple JSON object for easier modification

@smaranjitghose

I should create only this page right?

If i put the below in six categories as card title

Mithai Masti (Indian Sweets)
Nashta ( Indian Breakfast)
Chaat and Snacks
Fasting Food
Drinks
Main Course
Then where should i put Vegetarian Vendetta + Snack Satisfaction + Beverages

My bad 😅, I forgot that I had already given 6 categories. Go ahead with those. Avoid the last three

ok thank you

Am doing so now

I want to work on this issue under JWOC


How can I update inline_models's form_choices

class People(db.Model):
     ....
class PeopleCity(db.Model):
     ...
     people = db.relationship('People', backref='t_people_city')

class PeopleModelView(ModelView):
      inline_models = (PeopleCity, dict(
            form_choices={'city_id': get_city_choices()},
            form_args={'city_id': {'coerce': int}},
      )
      ...
      def create_form(self, obj=None):
             form = super(PeopleModelView, self).create_form(obj=obj)
             form.t_people_city.form.city_id.choices = [....]  # not working
             return form

and i hope change city_id value in create_form,edit_form
but, I don't know what I should do

using QuerySelectField can slove this issue
inline_models = (PeopleCity, dict(
           form_overrides= {'city_id':QuerySelectField },
            form_choices={'city_id': get_city_choices()},
            form_args': {'city_id': {'query_factory': lambda: Location.query.filter(Location.type == 'city'),
                                    'widget': Select2Widget()}
      )


Simplification of workflow?

review for JOSS
I find it a bit strange that the project contains 3 different executables and it is not completely clear to me how the interoperate and which data that is output from one program should be fed to the other. To simply this I would suggest to restructure the workflow as follows:
Instead of shipping three separate executables, ship a single one that internally dispatches to functions doing the work for each stage of the calculation. In order to decide which steps should be executed one can use for example input arguments / commandline flags. If the data required for a particular step has not yet been computed it can be computed on the fly and passed from one stage to the other inside the executable without worrying the user to build a new input file manually for each stage of the calculation.
If the aforementioned is not possible for some reason than at the very least the authors should provide a set of script that execute the programs in the right order and parse / drop input / output files automatically to some extent to reduce on the required manual labour.

Considering that the execution of suscep requires additional information from the user, and that the calculation of both J-values and susceptibility is not necessarily required every time, we decided not to create a single executable.  Clear instructions to execute each executable have been provided in the wiki. We have also included a basic script (in the bin directory) for executing both the ej_calc executables.

The script in the bin directory is basically useless. It's not fair to assume J2suscep will be placed in the home directory (or even named as J2suscep).
Why not use command-line arguments to merge the binaries? For example, running "ej_calc -f" would run the formal version, and "ej_calc -s" would be the spin_dens version. Maybe even just one binary for all "j2suscep" with -f and -s above, and -S for suscep or something along those lines.

I agree with @mturiansky. The script is not useful for me either and I don't see my original points fully addressed in the current version of the code.

@mfherbst @mturiansky
The script has been modified for better functionality. Once the installation is complete, one needs to either copy/move the contents of the bin folder to a directory in their pathway (the 'PATH' environment variable) or add the bin to the pathway (see wiki). The script also solves the problem of executing multiple executables for ej_calc using the same input file and it can also be used to execute the suscep code. Furthermore, it can be used to execute all the codes sequentially passing the output on one as the input of the next and hence go all the way from the initial input data, through the coupling constants to the susceptibility data output.
The input file for suscep has been broken down into three files in a way such that the input file of ej_calc containing the Hamiltonian can be directly used as one of the three input files that are required. The second file is generated by the script if one tries to execute all codes sequentially. The third input file contains additional information that is needed from the user end and has to be created manually.
One can use the example input files as templates to edit as needed.

This seems like an improvement. It would be nice if the script would print out usage information when invalid input is entered instead of just exiting with no message. (Just need to add an else block to the script.)

I agree. In a similar vein I would consider adding a -h flag or similar, which prints usage.

@mturiansky @mfherbst The script has been modified such that it prints out usage information when invalid input is provided or when –h tag is used.

It's a bit repetitive (for the error section and the -h section of the script). Perhaps you can refactor that. Otherwise I'm good.

@mfherbst The script has been updated to avoid repetition now.


user.User.Uid should be int instead of string

What version of Go are you using (go version)?

$ go version
go version go1.11 linux/amd64

Does this issue reproduce with the latest release?
Yes!
$ go doc user.User
package user // import "os/user"

type User struct {
	// Uid is the user ID.
	// On POSIX systems, this is a decimal number representing the uid.
	// On Windows, this is a security identifier (SID) in a string format.
	// On Plan 9, this is the contents of /dev/user.
	Uid string
	// Gid is the primary group ID.
	// On POSIX systems, this is a decimal number representing the gid.
	// On Windows, this is a SID in a string format.
	// On Plan 9, this is the contents of /dev/user.
	Gid string
	// Username is the login name.
	Username string
	// Name is the user's real or display name.
	// It might be blank.
	// On POSIX systems, this is the first (or only) entry in the GECOS field
	// list.
	// On Windows, this is the user's display name.
	// On Plan 9, this is the contents of /dev/user.
	Name string
	// HomeDir is the path to the user's home directory (if they have one).
	HomeDir string
}
    User represents a user account.

What did you expect to see?
$ go doc user.User
package user // import "os/user"

type User struct {
	Uid int
	Gid int
	Username string
	Name string
	HomeDir string
}

What did you see instead?
$ go doc user.User
package user // import "os/user"

type User struct {
	Uid string
	Gid string
	Username string
	Name string
	HomeDir string
}

The documentation for Uid points out that it is only numeric on POSIX systems. On other systems, they're string values. It therefore cannot be an integer.

See also #6495.


The windows files needed for builds.

here you go, after several days. :P

i eventually did it through the web interface. 🙁

Thank you for your work! But why is there some npm command in https://github.com/stefansl/image-shrinker/pull/44/commits/718c856fb46974dcd2c36af100ee2eb0759f96bc

yep! sorry about that. i'm just gonna push my fixes. editing on the web ui is trash.

oops.

i'm just gonna delete this. bad stuff is happening here.

fyi, that's what the accidental commit was.

i'm making a new pr and i'm gonna use branches this time. 😉

https://github.com/stefansl/image-shrinker/pull/45. this pr can go and disappear in the hole called closed pull requests.


Camera Roll, User denied access, -3311

This is how my code looks like
 componentDidMount() {
    var fetchParams = {
      first: 25
    };

    CameraRoll.getPhotos(fetchParams, this.storeImages.bind(this), this.logError);
  }

  storeImages (data) {
    var assets = data.edges;
    var images = assets.map((asset) => asset.node.image);
    this.setState({
      images: images
    });
  }

  logError (error) {
    console.warn(error);
  }

Once I reached this point in the app, I got the following error:
Object {code: -3311, message: "User denied access", nativeStackIOS: Array[9], domain: "ALAssetsLibraryErrorDomain"}

Any clue about how to fix it?

+1, was fine before

@facebook-github-bot stackoverflow

This issue looks like a question that would be best asked on StackOverflow. StackOverflow is amazing for Q&A: it has a reputation system, voting, the ability to mark a question as answered. Because of the reputation system it is likely the community will see and answer your question there. This also helps us use the GitHub bug tracker for bugs only. Will close this as this is really a question that should be asked on SO.
PS. iOS-wise, once you have dismissed permission box, you can go to settings -> choose your app and enable access back again. Given the error, you can prompt your users to do so.


[BEAM-2939, BEAM-5600, BEAM-6868, BEAM-7233] Expose residual roots and bundle finalization callbacks to portable runners.

This is towards supporting bundle finalization and SDK initiated self checkpointing of bundles for portable runners.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @boyuanzz
CC: @mxm @tweise @iemejia

Run Java PreCommit


devices: implement org_id support.

Fixes #THEEDGE-1980
Type of change
What is it?

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[X] Tests update
[X] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] I run go fmt ./... to check that my code is properly formatted
[X] I run go vet ./... to check that my code is free of common Go style mistakes
[X] I run make lint to prints out coding style mistakes and fix them
[X] I run make test-no-fdo to check all tests passed (beside fdo)

Test results are available in Ibutsu. The test run IDs are:
79d0b897-32ae-4c86-9913-fe0318116a59
c02c1e11-8bbd-4a43-8b77-6a2e954eb626


Add aitree.io

discovered this site accidentally when testing if the filter worked haha
might also be a good place to find other sites to block since it is a list of a bunch of ai "tools"

yeah i have quite a few of these "tool repositories" in the backlog, but an additional one is good too! 100% will be get around to putting all of these sites in the list, rn i'm more focused on getting AI generated images out of practical searches.


"Oops, could not connect"

i have installed a new rds 2016 farm. configured html 5 like described above but cant connect a remote app through html5. error: "oops, could not connect to "remoteappname""
same remoteapp can be opend without errors on rdweb as mstsc.

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

ID: 38213294-2784-2341-ac3e-77d2b017e2a2
Version Independent ID: 8d30b4c5-68df-5fa2-d7e9-ede9b406527f
Content: Set up the Remote Desktop web client for your users
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/remote-desktop-web-client-admin.md
Product: windows-server-threshold
GitHub Login: @Heidilohr
Microsoft Alias: helohr

@florianschleifer I recommend you get in touch with the dev team by email.

We are also having the same issue. Were you able to resolve this?

First step is to register the NPS in the domain. Second is to start the NPS service (!).
https://docs.microsoft.com/en-us/windows-server/networking/technologies/nps/nps-manage-register
I found the second step only while I clicked around in the NPS console, more randomly than systematically. After both steps the HTML5 access worked. Have RDS 2019 in use.

I have tried all recommendations.  I cannot get past this error for the web client.  The RDP launches correctly for the RemoteApps.  Any help would be greatly appreciated

I am using a Third party wild card cert.  Would this have any affect on this problem?

@BryanAdvisicon : If you have tried all recommendations already, I would suggest testing with a Self-signed Certificate from the server, if possible. (I don't have access to this kind of scenario, so I am unable to provide specific help.)

This is the event log when this happens The user "DOMAIN\user1", on client computer "<IP_ADDRESS>", met connection authorization policy and resource authorization policy requirements, but could not connect to resource "rds.mydomain.com". Connection protocol used: "HTTP". The following error occurred: "23005".  Do you have any advice of the fix?

Sorry, I don't have the required experience in this field. I am merely an IT tech guy trying to help amend the documentation when the documentation get reports on incorrect content. This site is primarily for reporting and improving the docs, so most help requests for issues not directly targeting improvement of the docs need to be directed to ordinary help lines and other support instances or community forums like TechNet. Actual solutions may only randomly appear, depending on if an external user or Microsoft team member posts a reply with a known solution. We can not guarantee that a solution will be provided on this site, unless it is part of an open ticket to amend existing documentation.
If you find your issue to be caused by incorrect documentation, you may want to open a new issue ticket, based on your results, available documentation and possible solutions from other known good sources.


Portuguese Translation: Specifics Lessons 🇧🇷

Specifics
[x]  ecto.md
[x]  eex.md
[ ]  ets.md
[ ]  mnesia.md
[ ]  plug.md

ecto.md translation in progress

ecto.md translation in done!
eex.md translation in progress...

@psantos10 super thanks for these translations! You're awesome :smile:
There was a small but important change in the Mnesia (Specifics) lesson last night just incase you have copied any english files.

mnesia.md in progress.

@pjhampton Mnesia is updated and in Portuguese now. #done

Great job guys. Thanks very much

plug.md in progress.

ets.md in progress

Done! o/

:boom:  I'll get this tweeted out today.  Thank you everyone!  :+1:


How do I extend drawio for collaborative editing?

I'd like to add some functionality to allow for collaborative editing of flowcharts.  I'm not familiar enough with the technologies in drawio to be efficient, so where do I start?

Honestly, if you're not familiar with a large scale JS application, and with collaboration technologies, you have no literally no chance of adding it, I wouldn't waste your time.

Please don't shut me down like this.  I've got all the theory, I just need a few of the tools.  Let me waste my time.

OK, there :)

Thank you.  So I need to read up on electron, and mxGraph still seems to be the core of the model.  Where else should I read?


Radius Error on form submit

Hi, i think there is a problem with the database insert.
When i click to the connect button, the registration form is back
In the system log of pfsense i can see this error :
(6) Invalid user (sql1 : Failed to create the pair: Invalid vendor name in attribute name "Password"<EMAIL_ADDRESS>(from client tester port 2000 cli xx:xx:xx:xx:xx:xx)
where xx is the mac address
And in the user database nothin is recorded
I use pfsense 2.4.3 so maybe your script isn't compatible.

After research i juste find what is wrong in the code.
In the ozy-captive.php on the line 281
if (!$statement = $db->prepare("INSERT INTO radcheck (username, attribute,  value) VALUES (?, 'Password', ?)"))
The password value is wrong, now, freeradius use Cleartext-Password. And now freeradius use the value := in the op column.
So i replace this line by
if (!$statement = $db->prepare("INSERT INTO radcheck (username, attribute, op, value) VALUES (?, 'Cleartext-Password', ':=', ?)"))
and it's work !

Thank you for your help.
Did you have to modify anything else ? Also, what FreeRADIUS version do you run so I can update the script to run with newer pfSense builds.

No i dont have modify anything else.
For FreeRadius, I use the freeradius3-3.0.17 version.
And after some checks, i think it will be better to modify the sql script to change the attribute and op default value than the php script itself ;)

Thank you. I will update the docs accordingly.


Static Background File Viewer does not show images or files

The file viewer does not show images or any files. Even after given the files an folders full permissions.
I am on
CSS Version: 1.7.1
stable version of steamOS: 3.4.8

This is not an issue with the theme or cssloader. This is an issue with decky. Please use decky pre-release to fix this.


can u make a guide on how to install and use

i tried everything but when i put last command, it doesnt work. please help with it.

Hello,
When you say "last command", I assume you mean this one:
python generate_accounts.py [-h] [-n NUMBER_OF_ACCOUNTS] [-p PASSWORD]

You're most likely missing the MegaTools software.
This is necessary for the script to work!
If you're on Windows, you can download the binary from https://megatools.megous.com/builds/builds/.
This link will download the x64 version for Windows.
You need to download that Zip-file, and extract the megatools.exe file inside.
You then need to place megatools.exe within a folder that's in your PATH.
Typically something like C:/Windows would work.
If you're on Linux you can get it from your package manager.
sudo apt-get install megatools

When you've installed MegaTools, and installed the Python requirements in requirements.txt, you can simply run the following command, which will make 3 accounts for you:
python generate_accounts.py

I hope this helps.
If not, feel free to send me some screenshots to further help you along!

with your instructions the generator worked very well

I'm glad you got the generator to work!
Maybe I will include some more information on how to install MegaTools.
It's pretty straight forward on Windows, but I remember having some issues with it on Linux before, trying to find the right binary.

I used the command as written by you in readme but it gave an error(screenshot below).
I see.

This error is caused by you having an extra line-break in your accounts.txt file.
I will push a new commit to fix this.

also I waana know can I sign to accounts where I just have its email and pass?. I have few accounts like that but I guess its not possible as the signin command takes up email,its password and then mega account id pass if I am right.

You're right.
The accounts.txt is a CSV (Comma Separated Values) file, in the following format:
email | email_id | email_password | name | email:password | purpose

As it is right now, you can add more accounts by simply adding the following line to your accounts.txt file:
e@mail.com,-,-,-,e@mail.com:P@ssw0rd,-
Remember to include 2 line-breaks after the new account!

Again, I will try to make some changes to the script, to make this easier to work with.
I am not pleased with how it is right now either.

Thank you for your help and giving us this awesome generator.

I'm glad you're enjoying this script!
If you want to help me out, you should consider "starring" this repo, to help others find it on GitHub as well.
You can do this by clicking the "star" button on the top-right of this page.
Thanks!

hello. I am going to try and follow everything you added and respond back to you in a day or two. Also I am very pleased on how you responded and helped me while some creators here just make a script and doesnt solve and help with any errors. So i appreciate your help in this thing.
I have "Starring/star" this repo as I am very well pleased with its working and your help.
as u said u are going to make this changes so  As a suggestion I would like If you can add a function to check the storage left in the accounts and print it out :).
Again Thank you for your help.

| This error is caused by you having an extra line-break in your accounts.txt file.|
_well i had just one line break (automatically) in the accounts file but when I removed that one break line , the signin worked I guess. Shared a ss ,do let me know do this indicate the function of "signin" is working.
_

well  u said the accounts file be  "accounts.txt" but its its excel format. hope its oky though.

|As it is right now, you can add more accounts by simply adding the following line to your accounts.txt file:
e@mail.com,-,-,-,e@mail.com:P@ssw0rd,-|
_I can't add "-"_as its a excel file and it takes it as a negative value .

Do clear these doubts.
Thank You

hello. I know I can be quite frustating to deal with, So I am sorry about that.
Well I am quite shocked how It logged in flawlessly into the accounts with a list of around 55acc. just got the below error with 2 accounts. What does it indicate?

I have shared a ss , does the "logged in" means it logged in the account and logged out?
And after how much time should I login into these accounts to keep them alive?

Thank you.

@f-o am getting this

@f-o am getting this

Same behavior here. This isn't working.

@f-o am getting this

Thanks for letting me know. I've fixed the issue just now.

@f-o am getting this

Thanks for letting me know. I've fixed the issue just now. In the future, please create a new issue about problems with the script, instead of writing in existing ones.

will do, thanks

I have shared a ss , does the "logged in" means it logged in the account and logged out? And after how much time should I login into these accounts to keep them alive?
Thank you.

After a very long time, I have finally made some new updates to this script.
These changes will namely;

Make it easier to add new accounts manually to the CSV
Better display whether the account encountered any errors when trying to sign in

To answer your question above: If the script automatically moves on to the next account, the sign in was successful. It will only print an error if it could not sign in to the account.
I will close the issue now, since I kept it open while I was working on the new CSV format.
If you still have issues with running the script, please open a new issue and explain it like you did above.


[Bug report] Can't save GIF on iOS

Version
2.0.1
Platforms
iOS
Device Model
Iphone 14 pro Max
flutter info
FINE: Pub 2.19.5
[✓] Flutter (Channel stable, 3.7.8, on macOS 13.4.1 22F770820d darwin-arm64, locale fr-FR)
    • Flutter version 3.7.8 on channel stable at /Users/pierreb/fvm/versions/3.7.8
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 90c64ed42b (il y a 4 mois), 2023-03-21 11:27:08 -0500
    • Engine revision 9aa7816315
    • Dart version 2.19.5
    • DevTools version 2.20.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/pierreb/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.0

[✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.80.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.68.0

[✓] Connected device (2 available)
    • iPhone 14 Pro Max (mobile) • 293BBAC4-0447-48FB-83A6-F993377F65BB • ios          • com.apple.CoreSimulator.SimRuntime.iOS-16-4 (simulator)

How to reproduce?
I'm using your package to save images in .jpg or .gif format. Unfortunately I'm having problems on the iOS side when I want to save a .gif the saved file appears as a .jpg image.
To reproduce this error, simply display a .gif in your application and save it using the .saveImage() method. The saving will be completed but the result will be an image and not a GIF.
Logs
No response
Example code (optional)
No response
Contact
No response

Please post your code, and also send the image or image link

I've edited the post. Sorry for the missing informations

since it is already night in our country, I will solve this problem tomorrow

Ok seems good on my side, might be an error or I was exhausted idk.
Sorry to have disturbed you.

Ok seems good on my side, might be an error or I was exhausted idk.
Sorry to have disturbed you.


Latest Softalacious OC <IP_ADDRESS> Errors

				$network_title = 'Customnet(' . $this->setting('network')['url'] . ')';

this line gives Parse error: syntax error, unexpected '[', expecting ')' in various places in the source file. opencart <IP_ADDRESS> using ssl

https://github.com/bitpay/opencart-plugin/issues/31#issuecomment-94746249


Memory leak in Join method

Just for test take first test method (Join()), and run it several times (10 000, 100 000, etc.), like this. And every next iteration will take a little bit longer to pass through + we'll get memory leaks. For example, first time it will take about 00:00:00.003, And on 12000 iteration it will take 00:00:00.101 already. On iteration 27400 it will take 00:00:00.214 and so on..
But it's on a small data. In a real project (in my case), for example every next Join gets about 5sec longer:
1 iteration = 3.7 sec
2 iteration = 8 sec
3 iteration = 13 sec
4 iteration = 19 sec.
And now it's a critical bug for me. I think that there is some problems with static cache.

Maybe related to https://github.com/StefH/System.Linq.Dynamic.Core/issues/84 ?

Yep, I thought about it, but I'm not sure. Anyway, please, fix this ASAP :)) Thank you!

Please test latest NuGet version<IP_ADDRESS>

Thank you, I've tested it now. Now it soo much better, but still leaks:
1 iteration = 0.95 sec (fantastic!)
2 iteration = 1.4 sec
3 iteration = 1.9 sec
4 iteration = 2.4 sec
5 iteration = 2.98 sec
Well, here in every iteration I work wit exactly the same classes, that connected together through lots of JOINs, it's all the same, but I change just a condition of WHERE statement from iteration to iteration. Hope, this will help.

Can you add this example project on github? That makes it easier for me to analyse and debug.

The problem is that it's not an example project, I tell you timers from real app and it uses big data with lot's of JOINS, and of course, I can't upload it. But ok, I'll try to make some demo test for this case

Hi! I've made some demo, try to download source from my repo on github, dunno if I done everything right. There is a simple console app, and you could see the growth of time and memory :) Looking forward to your answer!

Are you sure you have tested the latest version?
Because when I run your example, I see this:
press key to run
 0 - 251
10 - 6
20 - 8
30 - 4
40 - 3
50 - 9
60 - 2
70 - 7
80 - 2
90 - 4
100 - 2
110 - 12
120 - 2
130 - 2
140 - 3

Ok... very strange, on <IP_ADDRESS> it really works. I'll check this on a real project today and let you know how it goes

Ok, finally I got it :) Try out my last commit, but not sure that you'll think it's a bug :) The problem was that I've generated random names every time for same types - I mean different names for inner and outter fields. And everytime of course you can't find it in your cache dictionary, you save new record. If it's possible to fix - that'll be cool.

I will check your project.
However you can also fork this project and make a pull request on my source code : MemoryLeakTest project in scr-console folder.

Ok, will do

I did quickly check your test code.
But this is not a bug, because you are creating different anonymous types (based on a random name)
See this example:
var list = new [] { 1, 2, 3 };
var q = list.AsQueryable();
var resultA = q.Select(x => new { A = x });
var resultB = q.Select(x => new { B = x });

The code above creates 2 different types.
System.Linq.Dynamic.Core will do the same, except it's dynamic.

Yep, that's why I said that probably it's not a bug. But actually I can use same type in different parts of program, use different names and get this memory leak. And it's not a good idea. Maybe will be better to remove this string name from dictionary's key? Just leave only type names?

This sounds possible, I will take a look at the code and do some tests.

Sorry, this is not possible. Because the dynamic generated types do not only define the Type for each property, but off course the property-name itself is also generated.


🐛 BUG: Miniflare service bindings don't work correctly with RPC

Which Cloudflare product(s) does this pertain to?
Miniflare
What version(s) of the tool(s) are you using?
3.20240701.0 [Miniflare]
What version of Node are you using?
22.4.1
What operating system and version are you using?
Mac Sonoma 14.3
Describe the Bug
Observed behavior
Returning an object from an RPC call in Miniflare throws an error DevalueError: Cannot stringify POJOs with symbolic keys.
Returning an RpcTarget subclass from a getter doesn't work correctly, but does with a method.
Expected behavior
Returning an object through RPC works in Miniflare service bindings
Returning an RpcTarget subclass from a getter works in Miniflare service bindings
Please provide a link to a minimal reproduction
https://github.com/katis/miniflare-pojo-bug-demo
Please provide any relevant error logs
No response

#6285 is related

Thanks so much for the issue @katis and for the very nice minimal reproduction 🙂

Returning an object from an RPC call in Miniflare throws an error DevalueError: Cannot stringify POJOs with symbolic keys.

This should be getting fixed in #6301 please give the PR's prerelease a go if you want 🙂

Returning an RpcTarget subclass from a getter doesn't work correctly, but does with a method.

I'll look into this in the next few days 🙂
(PS: unfortunately #6301 is regressing the "but does with a method" 😓)

The solution in #6301 has been refactored and there is no regression at all anymore 🙂

(so this leaves only the getter issue to address)
(@katis thanks a lot again for the valuable repro and tests!)


Expected results data disappear after refresh on scorelab website

This resolves #174
The issue is on scorelab website Gsoc2021 page in idealist the expected results data disappear after refresh which is because the dangerouslySetInnerHTML on p tag.
I have added div tag instead of p tag which may solve this issue in production.
I google the issue for sometime and found that this is due to the nested p tag.I am leaving a reference link of similar react and gatsby issue.
Reference link:-
React Issue
Gatsby Issue

I would request @charithccmc and @wdevon99 to please review this PR.


How to pass editable state into column formatters

I am trying to implement a toggle button which has its own jsx inside one of the boostrap table's columns (for leftmost cell each row).
My issue is I can't seem to find a way to pass the parent's callback function from the react-bootstrap-table itself into a column formatter (i.e. there is no bind this allowed) such that a callback gets called from the parent that can change state based on whether someone hits the toggle button in each cell or not.
Basically my overall goal is to have some "parent rows" with expander button on left-hand most column and if a person hits that expander button it will change the state of a hiderow[] array of booleans and redraw the table data showing the once hidden child rows based on parent expansion.  (These are basically the same display style as the parents, it's just that the parent expander button "uncoils them into view".)
Every other bit of the logic works in this pipeline except how to get a state-changing function passed to an external .jsx as a callback.  Because as opposed to normal React parent-child interaction I have to somehow feed the callback function through the column formatter function, which seems to reside outside the React component that uses it (hence passing state functions not easy).
I have to go through the column formatter obviously because this toggle button appears on every single row and so has to be replicated, calling different ids, etc...  I tried to use a generic column click event but lol can't get rowID from that nor grab the inner toggle imagebutton except for maybe low-level js.  Similarly the row click event.  (There does not appear to be a cell-click event but this too would need a further "zoom in" on the image button inside it).

I'm also not able to update columns with dynamic data in the formatter: functions of column definition.
Each time state gets updated the column formatter: functions were never run

@JohnVLinton  It is possible to do so.. formatters: are pure functions and you need to pass external state data using formatExtraData along with formatter
please check doc below
https://react-bootstrap-table.github.io/react-bootstrap-table2/docs/column-props.html#columnformatextradata-any

Same here, passing this.state.something into formatExtraData does not seem to re-run the formatter

Did any of your find a workaround for this?  I'm hitting the same issue right now.

Yes I believe there is a way we found (not right in front of me).
Key is for you to console.log() inside the formatter the objects and you should be able to find how to pass an additional property.  You basically walk it out until you use up all the fields the library needs, (p1, p2, p3, ...) then the next space is where you can add your own, I think, if I recall right...

@jvlinton ,Could you please provide an example for this..i need to show a tooltip which on hover of a cell..stuck with the same issue,state is updated but formatter is not being invoked.
 `formatter: (cell, row, rowIndex, { showTooltip }) => {
      console.log('in cel', cell)
      console.log('tooltipOpen in formatter', showTooltip)
      const tags = cell.split(',').filter(a => a)
      console.log('tags', tags)
      if (tags.length <= 2) {
        console.log('in first')
        return
{cell}

      } else {
        return (

            <div id={`TooltipExample${row.id.replace(/\//g, '')}`}>{`${
              tags[0]
            },${tags[1]} + ${tags.length - 2} more`}

            <Tooltip
              placement="right"
              isOpen={showTooltip}
              target={`TooltipExample${row.id.replace(/\//g, '')}`}
              toggle={this.toggleTooltip}
            >
              {`${tags.slice(2).join(',')}`}
            </Tooltip>

        )
      }
    },
    formatExtraData: {
      showTooltip
    }`

toggleTooltip = () => {
console.log('in toggle')
this.setState(prevState => ({ showTooltip: !prevState.showTooltip }))

}

I had to re-render the component with a show/hide react condition after trying to solve it for hours.
It used to work in previous version(^1.1) but after 2, i believe performance checks inside the component has made this update to not happen.

The same issue for me unfortunately :/

@DorienMay
Do you have a working example? I still can't get this to work even on 4.0.1...

I'll try to find an example of the code...  The secret to figuring out how is to console.log each of the things coming into the formatter, if I remember rightly, and keep adding params until you can add something new that the library isn't already sending in...  Or else I think you can hack/overwrite the row object and return extra param or something like that

Try: static keyword

Bind the static method in constructor
MyClass.showModal = MyClass.showModal.bind(this);

Define the method
static showModal() {
this.setState({
show: true
})
};

Call the static method from Formatter method

const cellFormatter = () => {
return (
<Button className={'btn btn-primary'} onClick={MyClass.showModal}>Attend
);
};

First set a key on the bootstrap table.  When you update the data which you're not seeing reformatted, also change the key name of the bootstrap table.  That will force React to re-rendered the table and your formatter method will, just like magic, do its thing.

did you find any solution for this ?


fix(row): filtered row with filterRowComparator give runtime error

on filter function this.gantt is udedined
and the array contains data not row then do not need to actual.model

Coverage decreased (-0.02%) to 59.79% when pulling eef356c4c522158d2c33127d4f611cdb2b3f78f9 on henkensm:master into ffa55de212ed35c96f44e54cd8e928fe9d370fb9 on angular-gantt:master.

Coverage increased (+0.22%) to 60.03% when pulling e2b5f6dd75322a2f458f65801455ec517929101d on henkensm:master into ffa55de212ed35c96f44e54cd8e928fe9d370fb9 on angular-gantt:master.

Thanks for this pull request ! Could you rebase the 3 commits into a single one ?

replace by one pull request


titles are on top now?

@kissane I've noticed that you've moved titles to pages above images on some pages where that was not previously the case. Is that intentional? I can simplify that if so, because right now we have post_notitle for pages with images vs normal post pages. could stop calling the _notitle template if so.

@sinker: They're currently wildly inconsistent. I think this is a visual design call, so LMK what you think is best and I'll go through and standardize it.

my preference is the title below an image because title > BIG IMAGE > text feels weird to me

OK, I'll change that where they're the other way now

Done, will close after confirming once the PR works

yep, all done (and boy Github loved that merge for some reason). closing.


Implement compression properly

This PR fixes the creation and parsing of compressed message sets, making the library work follow the Kafka protocol regarding that functionality.
@nmred some other small fixes are included I can extract them to a different PR to make it easier to review. I really recommend you to review this commit by commit

some other small fixes are included I can extract them to a different PR to make it easier to review.

Good method, I will use this method in the future 👍

Good method, I will use this method in the future 👍

@nmred I try to have atomic commits and atomic PRs as much as possible. git rebase and git add -p are great tools to help on that =)

@nmred @lcobucci  https://github.com/weiboad/kafka-php/pull/163/files#diff-3e73c513e41712529b1b24d133cbe487R134 original message offset is getting set to zero after decompression. This bubbles to Assignment::consumerOffsets, gets commited and then fetched. Result of this, you constantly keep reprocessing same batch of message starting from offset=0 over and over again.
This makes lib unusable with compressed messages. Could you check this behaviour?
As workaround I have passed $ret['data']['offset'] into recursive call and then writing into decompressed message.

@Im0rtality that's quite unexpected, specially considering that the integration test suite passes... could you try to reproduce that behaviour in a test? You can use https://github.com/weiboad/kafka-php/blob/master/tests/Functional/Ticket/GH181Test.php as example


Subtyping and type equivalence

Implement subtyping (A <:< B) and its corollary type equivalence (A =:= B).

Done in #163.


Bottleneck distance between identical diagrams is non-zero.

bottleneck function repeatedly outputs non-zero distance between identical persistence diagram inputs. For example:

data = np.random.random((5,2))
pd = rips.fit_transform(data)
d = bottleneck(pd[0], pd[0])
d
0.05305507779121399

Note, at first pass it seems like d gets smaller as the number of points in data increases?

Hello Melissa, thank you so much for bringing this to our attention.  I will have a look at this as soon as I can.  Sorry for the trouble!

Okay, finally getting back to this.  I'm running the following code here
    from ripser import ripser
    from persim import bottleneck, bottleneck_matching
    import matplotlib.pyplot as plt
    import numpy as np
    np.random.seed(5) #3
    data = np.random.randn(5, 4)
    pd = ripser(data)['dgms'][0]
    dist, (matchidx, D) = bottleneck(pd, pd, matching=True)
    print(dist)
    bottleneck_matching(pd, pd, matchidx, D)
    plt.show()

Visually, the bug is apparent.

A very small error is made

I'll try to see if I can figure out why...


PERF: Add ThreadedGenerateData to ImageGridSampler

Uses multi-threading (by default) when doing grid-sampling.

Superseded by:

pull request #1017


vcpkg-tool, in Classic mode, adds default-features by itself

Describe the bug
If I specify one or more port feature explicitly, I expect that only these features are installed. Neither core nor other default features. But the default-features seem to be always added.
Environment
Windows11 latest vcpkg-tool 2023-09-15-ac02a9f660977426b8ec6392919fbb1d51b10998
To Reproduce
Steps to reproduce the behavior:

./vcpkg depend-info qt5compat[qml]  --format=tree
Results in:

PS C:\Users\Joerg.WORLDWARTWEB\source\repos\JoergAtGithub\vcpkg> ./vcpkg depend-info qt5compat[qml]  --format=tree
warning: In the September 2023 release, the default triplet for vcpkg libraries changed from x86-windows to the detected host triplet (x64-windows). For the old behavior, add --triplet x86-windows . To suppress this message, add --triplet x64-windows .
qt5compat[big-codecs, codecs, default-features, qml, textcodec]

Expected behavior
PS C:\Users\Joerg.WORLDWARTWEB\source\repos\JoergAtGithub\vcpkg> ./vcpkg depend-info qt5compat[qml]  --format=tree
warning: In the September 2023 release, the default triplet for vcpkg libraries changed from x86-windows to the detected host triplet (x64-windows). For the old behavior, add --triplet x86-windows . To suppress this message, add --triplet x64-windows .
qt5compat[qml]

Thanks for the explanation!


Need tests for elastic wave PDEs

Elastic and disco now have norm-check tests in the examples and notebook that are run in travis and pipeline. Closing?

I think we can close this yes, though having some MMS-based tests wouldn't be bad

Closing, MMS PR open for further improvement.


Further split up Gemfile into groups

A full brew install-bundler-gems is somewhat slow. This PR splits up the gem groups further to allow faster runs in CI etc, as well as allow a base brew install-bundler-gems --groups= only install what we already vendor (roughly). This could be used in future PRs to bootstrap a new Ruby.
Draft because this requires #15952.

A full brew install-bundler-gems is somewhat slow. This PR splits up the gem groups further to allow faster runs in CI etc, as well as allow a base brew install-bundler-gems --groups= only install what we already vendor (roughly). This could be used in future PRs to bootstrap a new Ruby.

Can we see some benchmarks of the speed differences here? This seems to add quite a lot of complexity so want to see the differences here.

A full brew install-bundler-gems is somewhat slow. This PR splits up the gem groups further to allow faster runs in CI etc, as well as allow a base brew install-bundler-gems --groups= only install what we already vendor (roughly). This could be used in future PRs to bootstrap a new Ruby.

Can we see some benchmarks of the speed differences here? This seems to add quite a lot of complexity so want to see the differences here.

Will test when I test #15955, but it currently takes 1 minute in our CI and also requires a fully functional compiler.

Changes here should hopefully reduce that down to a few seconds with zero native builds required.

Cool, if it works: great, decent speedup.
No longer native builds required because they are cached or something else?

Thanks again @Bo98!


Fixed #35867 -- Allowed links in admindocs view details summary.

Trac ticket number

ticket-35867
Branch description
This branch adds tests and bug fixes for ticket-35867
Checklist

[x] This PR targets the main branch.
[x] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[x] I have added or updated relevant tests.
[x] I have added or updated relevant docs, including release notes if applicable.
[x] I have attached screenshots in both light and dark modes for any UI changes.

You propose to revert part of 87315cc22d3 ticket #2411. Unless HTML5 differs from the XHTML days of that patch, It appears that tags are stripped because
 is not valid within headings.

@timgraham
Is this patch fine, or should i strip only the  tags?

You propose to revert part of 87315cc ticket #2411. Unless HTML5 differs from the XHTML days of that patch, It appears that tags are stripped because
 is not valid within headings.

Yes good point, tested with https://validator.w3.org/nu/#textarea and it's not valid
It appears in 87315cc it was only partially implemented, and didn't striptags from the summary for models (see admin_docs.test_views.TestModelDetailView.test_model_docstring_renders_correctly)
Having a util like strip_p_tags might work
--- a/django/contrib/admindocs/utils.py
+++ b/django/contrib/admindocs/utils.py
@@ -242,3 +242,7 @@ def remove_non_capturing_groups(pattern):
         final_pattern += pattern[prev_end:start]
         prev_end = end
     return final_pattern + pattern[prev_end:]
+
+
+def strip_p_tags(value):
+    return mark_safe(value.replace("
", "").replace("
", ""))
diff --git a/django/contrib/admindocs/views.py b/django/contrib/admindocs/views.py
index 38a2bb9286..3fdb34e0d1 100644
--- a/django/contrib/admindocs/views.py
+++ b/django/contrib/admindocs/views.py
@@ -30,7 +30,7 @@ from django.utils.inspect import (
 from django.utils.translation import gettext as _
 from django.views.generic import TemplateView

-from .utils import get_view_name
+from .utils import get_view_name, strip_p_tags

 # Exclude methods starting with these strings from documentation
 MODEL_METHODS_EXCLUDE = ("_", "add_", "delete", "save", "set_")
@@ -195,7 +195,7 @@ class ViewDetailView(BaseAdminDocsView):
             **{
                 **kwargs,
                 "name": view,
-                "summary": title,
+                "summary": strip_p_tags(title),
                 "body": body,
                 "meta": metadata,
             }
@@ -384,7 +384,7 @@ class ModelDetailView(BaseAdminDocsView):
             **{
                 **kwargs,
                 "name": opts.label,
-                "summary": title,
+                "summary": strip_p_tags(title),
                 "description": body,
                 "fields": fields,
                 "methods": methods,
diff --git a/tests/admin_docs/test_views.py b/tests/admin_docs/test_views.py
index cd68ebb10b..c48a89a1b0 100644
--- a/tests/admin_docs/test_views.py
+++ b/tests/admin_docs/test_views.py
@@ -91,9 +91,9 @@ class AdminDocViewTests(TestDataMixin, AdminDocsTestCase):

     def testview_docstring_links(self):
         summary = (
-            '

This is a view for '
+            '
This is a view for '
             ''
-            "myapp.Company

"
+            "myapp.Company
"
         )
         url = reverse(
             "django-admindocs-views-detail", args=["admin_docs.views.CompanyView"]
@@ -420,9 +420,9 @@ class TestModelDetailView(TestDataMixin, AdminDocsTestCase):

     def test_model_docstring_renders_correctly(self):
         summary = (
-            '

Stores information about a person, related to '
+            '
Stores information about a person, related to '
             ''
-            "myapp.Company.

"
+            "myapp.Company.
"
         )
         subheading = "
Notes
"
         body = (

Added functionality to strip only p tags
@sarahboyce @timgraham

Thanks for fixing this


Change the name to something more snappy

Uhhh... no.

Do you mean the name of "neurosynth" itself??

so this is what a hackathon is good for?

On Oct 23, 2015, at 2:34 PM, Chris Filo Gorgolewski<EMAIL_ADDRESS>wrote:
—
Reply to this email directly or view it on GitHub https://github.com/neurosynth/neurosynth/issues/56.

—
Russell A. Poldrack
Professor of Psychology
Bldg. 420, Jordan Hall
Stanford University
Stanford, CA 94305
<EMAIL_ADDRESS>http://www.poldracklab.org/

What do you think @margulies?

Closing this issue and recommending that @chrisfilo seek professional help. And @poldrack : yes :)

This is probably one of my favorite issues of all time... all in under 5 minutes!

Well are you just gonna compliment us, @vsoch, or are you actually going to suggest something snappy?

Neuromoog?

Reopening this issue so we can consider changing the name of my life's work to something snappier.

I like it. Closing issue but will rebrand whole project (and life) in coming weeks.

Now I am jealous I will not witness in my own eyes this event of the century.
On October 23, 2015 6:11:17 PM EDT, Tal Yarkoni<EMAIL_ADDRESS>wrote:

Closed #56.

Reply to this email directly or view it on GitHub:
https://github.com/neurosynth/neurosynth/issues/56#event-444304329

--
Sent from a phone which beats iPhone.

You guys are really silly. FYI


Floating point calling convention is not applied when the called function is an extern one

For the following source code
#include <stdio.h>
#include <math.h>

int main(int argc, char** argv)
{
    double a = argc;
    double b = argc + 5;
    double c = pow(a, b);
    return 0;
}

compiled with gcc -o floating_point_test floating_point_test.c -lm, we can see the HLIL of pow is screwed up:
00001149  int32_t main(int32_t argc, char** argv, char** envp)

00001158      char** var_38 = argv
00001178      int64_t var_18 = float.d(argc + 5)
0000118f      pow(x: argc, y: argv)
00001199      int64_t var_10 = float.d(argc)
000011a3      return 0

The call to pow does not use the two doubles as its parameters. Instead, it uses argc and argv, which I believe happens to be the first two things in the calling convention if floating point convention is ignored.
Binary: floating_point_test.zip
However, if the pow function is a function inside of the binary, then everything works fine:
#include <stdio.h>
#include <math.h>

double sum(double a, double b)
{
    return a + b;
}

int main(int argc, char** argv)
{
    double a = argc;
    double b = argc + 5;
    double c = sum(a, b);
    return 0;
}

00001129  double sum(double arg1 @ zmm0, double arg2 @ zmm1)

00001150      return arg1 + arg2

00001151  int32_t main(int32_t argc, char** argv, char** envp)

00001160      char** var_38 = argv
000011a1      int64_t var_10 = sum(float.d(argc), float.d(argc + 5))
000011ab      return 0

P.S.: I noticed this while working with https://crackmes.one/crackme/641bb9a533c5d447bc761b6f, it triggers the same bug at 0x177c
sub_177c:
   0 @ 0000178b  int32_t var_10 = arg2.d
   1 @ 000017ae  pow(x: arg1, y: arg2)
   2 @ 000017be  return float.d(arg1.d)

Fixed in 3.6.4705


Support Firebase Cloud Messaging (FCM)

See https://firebase.google.com/docs/cloud-messaging/send-message

+1

+1

@jeromegamez Hi, It's me again :-)
Is there any plan for this issue?
I have an immediate requirement on FCM, I started evaluating possible implementations and I may contribute back to this lovely repo if you like and I can accomplish :-)

@lashae @webleaf @aloisejulian @spitcat @gyimgabor @JulienSopheos, @michael-wolfram Please have a look at #189 - it's really early progress, and I currently can't test if and how the messages arrive at the clients (I only see that the API doesn't return an error), but if you test and engage, I am sure we can bring the topic forward rather sooner than later :)

@jeromegamez I would like to give it a shot, as soon as I have an available time slot ;-) By the way, thank you for your efforts for keeping things going ;-)

Released with 4.5.0, Documentation can be found at https://firebase-php.readthedocs.io/en/latest/cloud-messaging.html 🎉


ANY -> Any -> T where T

@JeffBezanson @andreasnoack I have a feeling this failure (which I'm realizing was in the other PR that I hastily merged) seems like it may be related to not using ANY?
ERROR: LoadError: return type Array{Float64,2} does not match inferred return type Any
Stacktrace:
 [1] macro expansion at /home/travis/.julia/v0.7/FFTW/test/runtests.jl:335 [inlined]
 [2] anonymous at ./<missing>:?
 [3] include_relative(::Module, ::String) at ./loading.jl:464
 [4] include(::Module, ::String) at ./sysimg.jl:14
 [5] process_options(::Base.JLOptions) at ./client.jl:315
 [6] _start() at ./client.jl:383
while loading /home/travis/.julia/v0.7/FFTW/test/runtests.jl, in expression starting on line 331

You could try moving where T outside, putting T with the rest of the method parameters {inplace,N}. See if that helps.

Doesn't seem to have made a difference, though that's assuming I've correctly identified the current inference failure as having to do with the switch from ANY.

Travis pr passes now.

This is actually no longer necessary, as I incorporated these changes in a different PR.


Content shift on hovering audio with type in mixed results

Description

On hovering over an the box audio player, the content shifts out of the box.
Reproduction
The audio controls shift when the CC license icons are shown, when these conditions are met:

the audio title is long as spans more than one line
the type of the audio is known, adding another line of content

Screenshots
https://user-images.githubusercontent.com/16580576/216284178-8ee5faf6-078e-4924-96b7-ff0551d1abf1.mov
Environment

Device: MacBook Pro M1
OS: macOS Ventura 13.1
Browser: Safari

This does not seem like a browser and OS specific problem.

@dhruvkb Are you still able to reproduce this? I'm not able to reproduce on any of the results in this search: https://openverse.org/search/?q=LL-Q7026

The fix for this issue is in #455


 Yes/No
Yes

VS Code Version: 1.74.0
OS Version: WIndows 10

Steps to Reproduce:

Install Microsoft's Python extension (https://marketplace.visualstudio.com/items?itemName=ms-python.python)

There doesn't seem to be a command in the palette to disable this auto-installation of related extensions.
Installing unwanted software without being able to opt out of it is a serious defect in my book. I for one only install vsix files which I have had scanned for malware before, but this auto-installation makes this approach impossible. Doesn't this open the door for extension authors to install malware along with their extensions without the users' knowing?

/extPython

It looks like this is caused by the Python extension. Please file the issue to the Python extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!

Deferring to @sandy081 who may decide to reopen this.

CC @isidorn


Rename TN provider label

Renamed provider label as requested in (https://issues.dp.la/issues/8328) and trimmed whitespace.

I think that's pretty safe to merge, so I will do so.


[Hub Generated] Review request for Microsoft.Batch to add version stable/2023-05-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki. Note that this doesn't apply if you are trying to merge a PR that was previously in the private repository.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

/azp run unifiedPipeline

        "CriCompatible"

Do you want to name this enum value as "CRICompatible"?

Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2023-05-01/BatchManagement.json:4764 in 0c48669. [](commit_id = 0c486695dace88245e55559efa0a2a93e26228f8, deletion_comment = False)

        "CriCompatible"

Do you want to name this enum value as "CRICompatible"?

Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2023-05-01/BatchManagement.json:4764 in 0c48669. [](commit_id = 0c486695dace88245e55559efa0a2a93e26228f8, deletion_comment = False)

    "enableAutomaticUpgrade": {

Should this be enum here? Something like this:
upgradeMode with values ["Automatic", "Manual"] etc.

Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2023-05-01/BatchManagement.json:4865 in 0c48669. [](commit_id = 0c486695dace88245e55559efa0a2a93e26228f8, deletion_comment = False)

    "enableAcceleratedNetworking": {

See if this can be an enum type.

Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2023-05-01/BatchManagement.json:5519 in 0c48669. [](commit_id = 0c486695dace88245e55559efa0a2a93e26228f8, deletion_comment = False)

CRICompatible is a widely recognized term in the container world, where CRI stands for Container Runtime Interface. This term is commonly used in Microsoft's published documentation as well, allowing for direct and consistent communication within the industry.
CRI
doc link

    "enableAutomaticUpgrade": {

Should this be enum here? Something like this:
upgradeMode with values ["Automatic", "Manual"] etc.
Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2023-05-01/BatchManagement.json:4865 in 0c48669. [](commit_id = 0c48669, deletion_comment = False)

The enableAutomaticUpgrade parameter should be of type bool, as per the design followed by the VM team for this feature of the VM extension. REST API for Virtual Machine
{
    "name": "extensionName",
    "type": "Microsoft.Compute/virtualMachines/extensions",
    "location": "",
    "properties": {
        "autoUpgradeMinorVersion": true,
        "enableAutomaticUpgrade": true,
        "publisher": "Microsoft.Azure.Monitoring.DependencyAgent",
        "type": "DependencyAgentWindows",
        "typeHandlerVersion": "9.5"
        }
}

    "enableAcceleratedNetworking": {

See if this can be an enum type.
Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2023-05-01/BatchManagement.json:5519 in 0c48669. [](commit_id = 0c48669, deletion_comment = False)

Same reason for enableAcceleratedNetworking be bool type, we followed the VMSS design. Accelerated Networking
"networkProfile": {
"networkInterfaceConfigurations": [
{
"name": "niconfig1",
"properties": {
"primary": true,
"enableAcceleratedNetworking" : true,
"ipConfigurations": [
...
]
}
}
]
}

@qiaozha please review the JS breaking change
@raych1 please review the GO breaking change

@qiaozha please review the JS breaking change


This appears to run on one node on the server but not on multiple nodes. Error was
<IP_ADDRESS>:   File "/app/gpt_neox/gpt_neox.py", line 211, in __init__
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:     self.token_emb = nn.Embedding(num_tokens, dim)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/sparse.py", line 109, in __init__
<IP_ADDRESS>:     self.weight = Parameter(torch.Tensor(num_embeddings, embedding_dim))
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:     return self.typename(*self.module_args, **self.module_kwargs)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:   File "/app/gpt_neox/gpt_neox.py", line 211, in __init__
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>:     self.token_emb = nn.Embedding(num_tokens, dim)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/sparse.py", line 109, in __init__
<IP_ADDRESS>:     super().__init__(layers=spec, loss_fn=loss_fn, num_stages=num_stages, **kwargs)
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/deepspeed/runtime/pipe/module.py", line 177, in __init__
<IP_ADDRESS>:     self.token_emb = nn.Embedding(num_tokens, dim)
<IP_ADDRESS>:     self.weight = Parameter(torch.Tensor(num_embeddings, embedding_dim))  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/sparse.py", line 109, in __init__
<IP_ADDRESS>:
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:     self.weight = Parameter(torch.Tensor(num_embeddings, embedding_dim))
<IP_ADDRESS>:     self._partition_layers(method=partition_method)
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/deepspeed/runtime/pipe/module.py", line 363, in _partition_layers
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:     param_counts = self._count_layer_params()
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/deepspeed/runtime/pipe/module.py", line 262, in _count_layer_params
<IP_ADDRESS>:     l = layer.build()
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/deepspeed/runtime/pipe/module.py", line 68, in build
<IP_ADDRESS>:     return self.typename(*self.module_args, **self.module_kwargs)
<IP_ADDRESS>:   File "/app/gpt_neox/gpt_neox.py", line 211, in __init__
<IP_ADDRESS>:     self.token_emb = nn.Embedding(num_tokens, dim)
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/sparse.py", line 109, in __init__
<IP_ADDRESS>:     self.weight = Parameter(torch.Tensor(num_embeddings, embedding_dim))
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)

This appears to run on one node on the server but not on multiple nodes. Error was
<IP_ADDRESS>:   File "/app/gpt_neox/gpt_neox.py", line 211, in __init__
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:     self.token_emb = nn.Embedding(num_tokens, dim)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/sparse.py", line 109, in __init__
<IP_ADDRESS>:     self.weight = Parameter(torch.Tensor(num_embeddings, embedding_dim))
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:     return self.typename(*self.module_args, **self.module_kwargs)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:   File "/app/gpt_neox/gpt_neox.py", line 211, in __init__
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>:     self.token_emb = nn.Embedding(num_tokens, dim)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/sparse.py", line 109, in __init__
<IP_ADDRESS>:     super().__init__(layers=spec, loss_fn=loss_fn, num_stages=num_stages, **kwargs)
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/deepspeed/runtime/pipe/module.py", line 177, in __init__
<IP_ADDRESS>:     self.token_emb = nn.Embedding(num_tokens, dim)
<IP_ADDRESS>:     self.weight = Parameter(torch.Tensor(num_embeddings, embedding_dim))  File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/sparse.py", line 109, in __init__
<IP_ADDRESS>:
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:     self.weight = Parameter(torch.Tensor(num_embeddings, embedding_dim))
<IP_ADDRESS>:     self._partition_layers(method=partition_method)
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/deepspeed/runtime/pipe/module.py", line 363, in _partition_layers
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)
<IP_ADDRESS>:  * (torch.Storage storage)
<IP_ADDRESS>:  * (Tensor other)
<IP_ADDRESS>:  * (tuple of ints size, *, torch.device device)
<IP_ADDRESS>:  * (object data, *, torch.device device)
<IP_ADDRESS>:
<IP_ADDRESS>:     param_counts = self._count_layer_params()
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/deepspeed/runtime/pipe/module.py", line 262, in _count_layer_params
<IP_ADDRESS>:     l = layer.build()
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/deepspeed/runtime/pipe/module.py", line 68, in build
<IP_ADDRESS>:     return self.typename(*self.module_args, **self.module_kwargs)
<IP_ADDRESS>:   File "/app/gpt_neox/gpt_neox.py", line 211, in __init__
<IP_ADDRESS>:     self.token_emb = nn.Embedding(num_tokens, dim)
<IP_ADDRESS>:   File "/usr/local/lib/python3.8/dist-packages/torch/nn/modules/sparse.py", line 109, in __init__
<IP_ADDRESS>:     self.weight = Parameter(torch.Tensor(num_embeddings, embedding_dim))
<IP_ADDRESS>: TypeError: new() received an invalid combination of arguments - got (NoneType, int), but expected one of:
<IP_ADDRESS>:  * (*, torch.device device)
<IP_ADDRESS>:       didn't match because some of the arguments have invalid types: (!NoneType!, !int!)

Further testing revealed the following:
sh scripts/train_enwiki8.sh throws the error Check batch related parameters. train_batch_size is not equal to micro_batch_per_gpu * gradient_acc_step * world_size1028 != 128 * 1 * 8 when run on one node.
sh scripts/train_enwiki8.sh throws the error AssertionError: Check batch related parameters. train_batch_size is not equal to micro_batch_per_gpu * gradient_acc_step * world_size1028 != 32 * 1 * 32 when run on multiple nodes.
sh scripts/train_enwiki8_pipeline.sh runs on one node.
sh scripts/train_enwik8_pipeline.sh throws the error RuntimeError: CUDA error: invalid device ordinal when run on multiple nodes.
sh scripts/train_gpt3small.sh requires the instilation of  thecupy (not a problem) and mpi4py (a problem) packages when run on one node.
sh scripts/train_gpt3small.sh hangs when run on multiple nodes.
sh scripts/train_gpt3small_pipeline.sh runs on one node.
sh scripts/train_gpt3small_pipeline.sh hangs on multiple nodes.

Further testing revealed the following:
sh scripts/train_enwiki8.sh throws the error Check batch related parameters. train_batch_size is not equal to micro_batch_per_gpu * gradient_acc_step * world_size1028 != 128 * 1 * 8 when run on one node.
sh scripts/train_enwiki8.sh throws the error AssertionError: Check batch related parameters. train_batch_size is not equal to micro_batch_per_gpu * gradient_acc_step * world_size1028 != 32 * 1 * 32 when run on multiple nodes.
sh scripts/train_enwiki8_pipeline.sh runs on one node.
sh scripts/train_enwik8_pipeline.sh throws the error RuntimeError: CUDA error: invalid device ordinal when run on multiple nodes.
sh scripts/train_gpt3small.sh requires the instilation of  thecupy (not a problem) and mpi4py (a problem) packages when run on one node.
sh scripts/train_gpt3small.sh hangs when run on multiple nodes.
sh scripts/train_gpt3small_pipeline.sh runs on one node.
sh scripts/train_gpt3small_pipeline.sh hangs on multiple nodes.

The above is with ZeRO 1. While I haven't rigorously tested with ZeRO 2, gpt3small OOM'd on one node with ZeRO 2.

The above is with ZeRO 1. While I haven't rigorously tested with ZeRO 2, gpt3small OOM'd on one node with ZeRO 2.