Looks to the wrong registry for scoped packages

Hey,
I'm using NPM's support for directing some specific scopes to different registries. Given the error below it seems to be npm-check is still looking for that package in the main registry.
                            �  NPM ERR!  https://registry.npmjs.org/ returned Not found : @some_scope

Cheers

Good find, I haven't used any scoped packages yet. I'll give it a look.

Cool.
You intentionally use String.prototype.replace with a string arg there? That's not global replace in that case. ;-)

Yeah, good code review comment. I was thinking there should only ever be one / in there, and if there are more than one then npm added a new feature that we might not support yet, but I should change it to be safe.

Actually that code is gone, replaced with Sindresorhus's package-json.
https://github.com/dylang/npm-check/blob/master/lib/npm/get-npm-info.js#L10


Change User Folder Name

Description of the new feature / enhancement
Changes the user folder name to the specified name in the User Folder Name textbox.
Scenario when this would be used?
People usually change their username using Control Panel or by adding a Microsoft Account on their Windows PC. The user folder does not change though. However, the user folder name changes on the desktop's user folder. People are annoyed by this so please add this feature. I am annoyed by the old user folder name and want to change it without any effort so please add it.
Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/5807


Consider making the first instantiated CMMCorePlus object  the global instance

Proposing putting the logic in instance():
        global _instance
        if _instance is None:
            _instance = cls()

into the init instead, so that the first CMMCorePlus is the global instance (you can still make others, for testing and stuff... but this just makes it a little easier to make the "first/main" one)
@ianhi . thoughts?

I'm definitely for this - at least for common use cases. In which case I think this is strictly a good thing to add some guard rails to make sure you always have the right thing. But need to continue to support the use case I've seen a few times on image.sc where people keep multiple core instances for when they have complicated custom setups/very high performance things.


Use a unified mapping for variable states.

This change is

crates/cairo-lang-sierra-generator/src/local_variables_test_data/e2e line 147 at r1 (raw file):

dup<felt252>([1]) -> ([1], [5])
store_temp<felt252>([5]) -> ([5])
store_local<felt252>([2], [1]) -> ([1])

I need to debug why this happens.
Code quote:
store_local<felt252>([2], [1]) -> ([1])

crates/cairo-lang-sierra-generator/src/store_variables/state.rs line 33 at r1 (raw file):
Previously, orizi wrote…
doc.
also - why no LocalVar?

I thought I'd add it in a future PR, do you think it should be done in the same PR?

crates/cairo-lang-sierra-generator/src/store_variables/state.rs line 37 at r2 (raw file):
Previously, orizi wrote…
Move the deleted doc to the new enum doc

I don't understand the comment.
What does '(for example, [ap - 1] + [ap - 2]) to [DeferredVariableInfo].' mean?

crates/cairo-lang-sierra-generator/src/store_variables/mod.rs line 381 at r1 (raw file):
Previously, ilyalesokhin-starkware wrote…
I'll try to merge this function with store_all_possibly_lost_variables.

done.


Improve ced pipeline

@bendichter @luiztauffer @alejoe91 The usual updates to the modern standard for the pipeline conversion, featuring quick_write and 'full save' for sorting extractors and lfp recordings from the notebook.
Need to fully test it, which I'll do so here in the next couple hours.
Points to add_ced_metadata for readability.

@bendichter Some small fixes were in order and are dependent on spikeextractors PR #531
Otherwise running smoothly, will have example files to share for next meeting

Closing this branch and opening new one per all the changes discussed from the last week, and moving the smaller more immediate changes to a new branch


Enable Prow + CI/CD

It would be great to have some automated testing in place to determine that changes do not break anything:
It might be good to start with:

Unit Tests
Linting

And eventually enable:

End-to-End tests

For e2e tests in Kubeflow, see  https://github.com/kubeflow/testing#setting-up-kubeflow-test-infrastructure

/assign @gsunner


Recently played songs (Working)

So I added a recently played icon button on the now playing page, I linked it to a tableview and got it working. I inputed the JSON recently played link in the station long description. Here is my code for the recently played tableviewcontroller:
import UIKit

//----------
//MARK: JSON
//----------

//The Initial Response From The JSON
struct Response: Codable {

    var playHistory: Album

}

//The Album Received Which Is An Array Of Song Data
struct Album: Codable {
    var song: [SongData]

}

//The SongData From The PlayHistory Album
struct SongData: Codable{

    var album: String
    var artist: String
    var cover: String
    var duration: String
    var programStartTS: String
    var title: String
}

class TableViewController: UITableViewController {

    //1. Create An Array To Store The SongData
    var songs = [SongData]()
    var currentStation: RadioStation!
    var downloadTask: URLSessionDownloadTask?

    override func viewDidLoad() { super.viewDidLoad()

        self.tableView.delegate = self
        self.tableView.dataSource = self

        //2. Load The JSON From The Main Bundle
        guard let urlText = URL (string: currentStation.longDesc)
            else { return }

        do{
            //a. Get The Data From The From The File
            let data = try Data(contentsOf: urlText)

            //b. Decode The Data To Our Structs
            let albumData = try JSONDecoder().decode(Response.self, from: data)

            //c. Append The Songs Array With The PlayHistory
            albumData.playHistory.song.forEach { songs.append($0) }

            //d. Test Some Data
            print("""
                **The First Album Details**
                Album = \(songs[0].album)
                Artist = \(songs[0].artist)
                Cover = \(songs[0].cover)
                Duration = \(songs[0].duration)
                Start = \(songs[0].programStartTS)
                Title = \(songs[0].title)
                """)

            //3. Load The Data
            DispatchQueue.main.async {
                self.tableView.reloadData()
            }
        }catch{

            print(error)
        }

    }

    //-----------------
    //MARK: UITableView
    //-----------------

    override func numberOfSections(in tableView: UITableView) -> Int {
        return 1
    }

    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return songs.count
    }

    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {

        //1. Create A Cell
        let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath) as! TableViewCell

        //2. Set It's Text
        cell.songTitle.text = songs[indexPath.row].title
        cell.artistLabel.text = songs[indexPath.row].artist

        //3. Get The Image
        //Start with placeholder image so it shows until image download completes
        cell.songCover.image = UIImage(named: "WhiteLogo.png")

        //Continue with your logic, no change really but could be shortened to:
        if let imageURL = URL(string: songs[indexPath.row].cover) {
            let request = URLSession.shared.dataTask(with: imageURL) { (imageData, response, error) in
                guard let imageData = imageData
                    else {
                        return
                }

                DispatchQueue.main.async {
                    cell.songCover.image = UIImage(data: imageData)
                }
            }

            request.resume()
        }
        return cell

    }

    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {

        print("""
            **Album \(indexPath.row) Selected**
            Album = \(songs[indexPath.row].album)
            Artist = \(songs[indexPath.row].artist)
            Cover = \(songs[indexPath.row].cover)
            Duration = \(songs[indexPath.row].duration)
            Start = \(songs[indexPath.row].programStartTS)
            Title = \(songs[indexPath.row].title)
            """)
    }

}

I got this working. I pushed the Current Station to my Recently Played View Controller. Then whatever station I am currently on it pulls its LongDesc link and uses that JSON link to parse my recently played tracks as shown above.

Thanks for share mannyd209!

Can you please share the completed project with recently played songs?

Hi there, I'd like to know if you can share the tutorial to add it ?
Thank you !

Hi,
Could you please share the completed project. I'm trying to implement this but I get nil crash.


Add Traffic data providers

Pull Request Template
Types of changes
What types of changes does your code introduce to Woosmap Geofencing?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Before you submit a pull request, please make sure you have to following:

[ ] I've read contributing guidelines
[ ] My code contain tests relevant for the problem I'm solving
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

The locationId parameter of calculateETA() must not be mandatory.
mPositionsManager.calculateETA(latOrigin,lngOrigin,listDestinationPoint);
                                     ^
  required: double,double,List<Pair<Double,Double>>,int
  found: Double,Double,List<Pair<Double,Double>>
  reason: actual and formal argument lists differ in length


Added API docs

Added API docs. Fixes #2.
Please check lines Lines #L59-69 before signing off.

merged manually. Thanks and sorry it took us forever to get this merged :)


Doesn't work at all

I've tried to implement from the CDN source and from the bower source also, no matter what I do, the input doesn't appear at all. Seems to be completely damaged.

I think this happened due to conflict with another plugin, I'm using ng-datepicker also, after I disabled it this plugin started to work. Not sure why.


RFC: namespaces: Allow container with agent PID namespace

This change adds a new field "agentPidns" to the CreateContainer
request. When this is set to true, the container instead of
creating and joining a new PID namespace will skip creating a PID
namespace and be in the same PID namespace as the agent/init process.
I am adding this functionality to support certain sidecar containers
such as debug/audit containers to be able to gather audit data/kernel
debug info.
Fixes #787
Signed-off-by: Archana Shinde<EMAIL_ADDRESS>

Will this bring  the security issues such as the users can access/kill the agent process?

@amshinde what's the complete plan (what else is needed?) to support debug/audit containers?
changing this to dnm

@amshinde what's the complete plan (what else is needed?) to support debug/audit containers?

@devimc I was hoping I could achieve this by specifying the container name for which agent pid sharing needs to be enabled, as a pod annotation. I see that the container name is passed down to the container in case of crio. But I do not see the same behaviour in case of containerd, containerd does not pass the container name that kubelet passes to it to the OCI runtime. In any case, since the kubelet annotation itself is not really documented API, I dont think we should be relying on it.
The only way to implement this would be to specify this as an environment variable for the container.
I think this should be ok to do, given that we plan to enable the agent pidns sharing behaviour by explicitly enabling this in the configuration file.
Wdyt @devimc @jodh-intel ?

While I agree this is a useful feature for debugging/auditing use case, we have decided to feature freeze for kata 1.x releases. Please re-submit it to https://github.com/kata-containers/kata-containers Thanks!
If it is strongly desired to include this in the 1.x kata releases like v1.12, please backport it from the 2.0-dev branch after it is merged there.

As we have discussed this in PTG and the conclusion is to allow new features to be in 1.x releases but they must be ported to 2.0-dev branch as well. For this PR, @amshinde will be the one to port it to 2.0-dev branch. Am I understanding it correctly?

Will this bring the security issues such as the users can access/kill the agent process?

@lifupan Yes, this is meant only as a debug feature and needs to be explicitly turned on by changing the configuration.toml file.
Have a look at the corresponding runtime PR:
https://github.com/kata-containers/runtime/pull/2723

As we have discussed this in PTG and the conclusion is to allow new features to be in 1.x releases but they must be ported to 2.0-dev branch as well. For this PR, @amshinde will be the one to port it to 2.0-dev branch. Am I understanding it correctly?

Yes, I shall backport this @bergwolf

@devimc @jodh-intel @lifupan @bergwolf This is ready for review, please take a look.

/test-ubuntu

We could now force-merge this (ignoring the known failing Zuul WIP check), but might be good to get final comments / approval from @devimc first.


Implement lazy-loading of child locales with missing prents

This is a minimalistic redo of #3336 without the crazy test scaffolding.
@skfd what do you think.
I'd wait and see if users are happy, there are a few ways to improve it, but if that works I'm happy to leave it.

That's great, thank for fixing this!

Merged in e107f5067b340f921bdbcba3462ac9e8aca7c386


Don't add package to imports

See this commit:
https://github.com/japgolly/scalajs-benchmark/commit/e40b4025ef88bdc7a3dff942752688a8d19e7667#diff-fb17ec1fbc40fe04f706f6397be9b753
import japgolly.scalajs.benchmark.gui.`package`.CssSettings._

should actually be
import japgolly.scalajs.benchmark.gui.CssSettings._

@japgolly, thanks for reporting! PR #58 should fix this issue.

Actually, I just learned that having `package` in the import is still valid but unnecessary. So removed the "bug" label and marked it as an "enhancement".

Great! Thanks so much @liancheng !!


Basic documentation for OptionParser

If you find any errors, or something that I forgot but should be added, please write a comment in the relevant line :smile_cat:

Minor comments, looks good overall. Thanks Sergio! :)

Awesome, thanks!!
I'll merge and remove the docs about allowing command as an option. Then I'll fix this behaviour, but later...

@porras Thanks again for documenting this. Command line apps are one of the easiest things to get started with crystal so it's really nice that this class is now documented :-)

Fair enough! But crul will break with that release (no worries, I'll fix it :smile:)


Safer check for active caption

Some browsers don't seem to support className as a string that can be used with indexOf so they throw when using captions like this:
Uncaught TypeError: a[b].target.className.indexOf is not a function
Summary of proposed changes
I've just swapped that by classList.contains which should be safer and also (IMHO) clearer.
Task list

[ ] Tested on supported browsers
[ ] Gulp build completed

Thanks! 👍


ModuleNotFoundError: No module named 'pyqt_openai'

Hi Gyu Yoon,
hope you are doing well!
I'd like to draw you attention to still open an old issue.
You cannot run your code without going through setup process, just as mentioned here.
I believe you went through setup process (python setup.py install) at least once and the modules are on the path. If you try to follow the steps from above link  in a fresh environment you will get this:
peter@alfa:~/Development/AI-and-chatGPT/pyqt-openai$ ll
total 44
drwxrwxr-x  5 peter peter 4096 Mai 28 19:24 ./
drwxrwxr-x 15 peter peter 4096 Mai 28 19:24 ../
drwxrwxr-x  8 peter peter 4096 Mai 28 19:24 .git/
-rw-rw-r--  1 peter peter 1860 Mai 28 19:24 .gitignore
drwxrwxr-x  3 peter peter 4096 Mai 28 19:24 .idea/
-rw-rw-r--  1 peter peter 1070 Mai 28 19:24 LICENSE
drwxrwxr-x  8 peter peter 4096 Mai 28 19:24 pyqt_openai/
-rw-rw-r--  1 peter peter 5946 Mai 28 19:24 README.md
-rw-rw-r--  1 peter peter   66 Mai 28 19:24 requirements.txt
-rw-rw-r--  1 peter peter 1165 Mai 28 19:24 setup.py
peter@alfa:~/Development/AI-and-chatGPT/pyqt-openai$ cd pyqt_openai/
peter@alfa:~/Development/AI-and-chatGPT/pyqt-openai/pyqt_openai$ python3 main.py
Traceback (most recent call last):
  File "/home/peter/Development/AI-and-chatGPT/pyqt-openai/pyqt_openai/main.py", line 10, in <module>
    from pyqt_openai.aboutDialog import AboutDialog
ModuleNotFoundError: No module named 'pyqt_openai'
peter@alfa:~/Development/AI-and-chatGPT/pyqt-openai/pyqt_openai$

I have to admit that i'm not familiar with Linux so i've installed the Ubuntu in VirtualBox to use pyqt-openai
I didn't go through python setup.py install because in Windows 11 it works even if i don't do it. Python in Windows adds current path to env path to make it work, unlike Linux.
So i had to consider Linux user.. 😥 very sorry. Thanks for bringing up this
After run python setup.py install, it works and i've just added it in README

After run python setup.py install, it works and i've just added it in README
Hi thanks for the efforts in checking this in details! I looked into the details as well. If someone does not want to install it and still test it in a local folder, it is enough to add to the PYTHONPATH variable path to the "pyqt-openai" folder.

So a quick workaround is to run the command from the folder where you clone the repo
peter@alfa:~/Development/AI-and-chatGPT/pyqt-openai$ export PYTHONPATH=$(pwd):$PYTHONPATH
or add to your .profile or .bashrc file a fixed path:
export PYTHONPATH=~/Development/AI-and-chatGPT/pyqt-openai:$PYTHONPATH
This helps python to find embedded modules in the pyqt-openai folder.


AttributeError: 'Chart' object has no attribute 'chart_title'

I am trying to add a title to my chart. On running below , I don't get any error but no titles appear on my chart.
My specs are: python-pptx version = 0.6.1 , python 2.7, mac osx, powerpoint for mac 15.26
df = pd.read_excel("/Users/vagabond/Documents/python_pptx_3/Bar_Charts.xlsx", sheetname=None)
f = open('/Users/vagabond/Documents/python_pptx_3/Presentation1.pptx')
prs = ppt.Presentation(f)
for sheetname, data in df.iteritems():
slide_layout = prs.slide_layouts[9]
slide = prs.slides.add_slide(slide_layout)

chart_data = ppt.chart.data.XyChartData()

series_1 = chart_data.add_series('Series 1')

## iterate through rows of the data frame for desired columns and add data points
for index, row in data.iterrows():
    series_1.add_data_point(row['Share_of_apples'], row['Share_of_oranges'])

## define co-ordinates on the slide
x, y, cx, cy = Inches(1), Inches(3), Inches(7), Inches(4)

chart = slide.shapes.add_chart(
XL_CHART_TYPE.XY_SCATTER, x, y, cx, cy, chart_data).chart

chart.has_title = True
chart.title = sheetname

prs.save('scatter_charts.pptx')

When I try this : chart.has_title , I get an error:
AttributeError                            Traceback (most recent call last)
 in ()
----> 1 chart.has_title
AttributeError: 'Chart' object has no attribute 'has_title'
posted on SO as question also : http://stackoverflow.com/questions/40848830/chart-title-for-chart-using-python-pptx

I answered on SO. Short version is that Chart.title is not in the API yet.

Chart.chart_title was added in v0.6.4 today.


Improvements for comparison documentation

FWIW some of the delight examples for use-package are unnecessarily verbose.
In particular, the library name shouldn't ever be needed for minor modes, because that is already implicit when use-package is being used (for major modes you would still want to pass the :major argument).  You can simplify these examples further still, though:
To hide a lighter:
(use-package rainbow-mode
  :delight)

To rename a lighter:
(use-package rainbow-mode
  :delight " Bar")

These expand to (delight '((rainbow-mode nil rainbow-mode))) and (delight '((rainbow-mode " Bar" rainbow-mode))) respectively.
You only need to pass the mode name if that did not match the package/feature name.
Your final example is also passing in too many arguments -- I think you've accidentally kept the nil when adding the " Foo":
(use-package quux
  :delight
  (foo-mode " Foo" nil "quux")
  (bar-mode " Bar" nil "quux"))

That should be:
(use-package quux
  :delight
  (foo-mode " Foo")
  (bar-mode " Bar"))

These should be the correct equivalents:

;; auto-fill-mode (standard minor mode)
(blackout 'auto-fill-mode)
(delight 'auto-fill-function nil t)

(blackout 'auto-fill-mode " Auto-Fill")
(delight 'auto-fill-function " Auto-Fill" t)

;; emacs-lisp-mode (major mode)
(blackout 'emacs-lisp-mode "Elisp")
(delight 'emacs-lisp-mode "Elisp" :major)

;; ivy-mode in ivy.el
(with-eval-after-load 'ivy
  (blackout 'ivy-mode))
(delight 'ivy-mode nil "ivy")

;; foo-mode in foo-mode.el
(use-package foo-mode
  :blackout t)
(use-package foo-mode
  :delight)

(use-package foo-mode
  :blackout " Foo")
(use-package foo-mode
  :delight " Foo")

;; foo-mode in foo.el
(use-package foo
  :blackout t)
(use-package foo
  :delight foo-mode)

(use-package foo
  :blackout " Foo")
(use-package foo
  :delight foo-mode " Foo")

;; foo-mode and bar-mode in quux.el
(use-package quux
  :blackout ((foo-mode . " Foo")
             (bar-mode . " Bar")))
(use-package quux
  :delight (foo-mode " Foo")
           (bar-mode " Bar"))

Thanks for the corrections, would you mind submitting a pull request fixing the README?

Merged, thanks for taking the time!


how do i start it, i already installed it?

how do i start it, i already installed it!

RTFM https://github.com/dz0ny/leapcast
;)

TFM is missing a step after 'python setup install' 'leapcast'

run
leapcast

and
leapcast -h

to get a list possible options
TFM is not missing a step, have a look into the last box of code on
https://github.com/dz0ny/leapcast

The step is not missing, but at the first glance, we can think the box of code is for windows only so i didn't read it. Missing a title that clearly indicate that for running.


Jumpstart HCIBox - don't use .local domain

Hi,
first thank you for this jumpstart, that's really cool to put together.
Domain .local is not recommended. I had problem with integrating linux into this domain in past.
Check this article https://social.technet.microsoft.com/wiki/contents/articles/34981.active-directory-best-practices-for-internal-domain-and-network-names.aspx

Thanks @jjindrich. The domain can be configured by modifying the HCIBox-Config.psd1 file in your own fork of the Jumpstart repo. You can point to your fork at deployment by providing the githubAccount and githubBranch parameters to the Bicep template and point to your fork.
Did you have any specific issues with HCIBox capability because of the domain name?

Currently not, but it can come if somebody will deploy additional machines into dns.


trying again

ddd

👷 Deploy Preview for sparkly-mousse-72daf9 processing.

Name
Link

🔨 Latest commit
baf7067ddddadab492eb2da8c64b69294f66c94e

🔍 Latest deploy log
https://app.netlify.com/sites/sparkly-mousse-72daf9/deploys/62a073391da66a0008d59021


deployment: investigate how users will be able to deploy their instances when adding new custom fields

[ ] How do we create mappings in advance when deploying a new instance e.g containers

Possible thoughts

OC Jobs (or initContainer)? we need a new container with the new image to run the mapping creation but not deploy it
Use an application flag i.e to mark fields as active/inactive i.e they are in mapping but not avaialbe to users until the flag is set to True.
Spend some time to see how they solve that in containerized environment

will be included in services setup, ad-hoc new custom fields will need to be run through the custom-fields command


FromBody valuetuple to have both stream and formal-binding

Currently the [FromBody] can be used just one time and that is correct for various reasons.
In my scenario (WebHook callback) I have a "formal" model (the payload) and also the need to read the request-body as a stream to validate the hash.
What I'm thinking is something like this:
[FromBody] (MyModel, byte[])
[FromBody] (MyModel, string)
[FromBody(Encoding = Encoding.UTF8)] (MyModel, string[])
The implementation should bind the first value of the tuple as normal (current behavior). The second tuple value can be just of type byte[] or string and its value represents the "naked-body".

Not clear what your intentions are. However, if the request body contains various parts, since body is not organized as name-value pairs, the onlly way to recognize the various parts is writing a custom formatter. Otherwise you have to process everything as a sequence of bytes in the Controller itself or in an ActionFilter

The case is the a receiver of a webhook.
The webhook (a POST to my app) has a payload (a model) and some headers.
One of the headers is the "Notification-Signature" that is basically an hash (HMACSHA256) calculated on the whole request-content plus some headers.
Inside my action I need the payload, which contains a JSON, with all needed information about the notification but, before trust the POST, I have to check (recalculate) the "Notification-Signature".
For sure I can implement a custom binder, a middleware, maybe an ActionFilter... the quick-dirty solution is avoid to specify any kind of binding, for the Request.Body, and manage everything from inside the action.
Would be useful to have an OOTB solution for something that is not so strange nowadays.

I know... its a shame!! but, so far, it pay the bill
		private (T, byte[]) FromRequestBody<T>() where T: class
		{
			if (Request?.Body == null)
			{
				return (null, null);
			}
			byte[] content;
			using (var contentStream = new MemoryStream())
			{
				Request.Body.CopyTo(contentStream);
				contentStream.Flush();
				contentStream.Seek(0, SeekOrigin.Begin);
				content = contentStream.ToArray();
			}
			var data = JsonConvert.DeserializeObject<T>(Encoding.UTF8.GetString(content));
			ModelState.Clear();
			TryValidateModel(data);
			return (data, content);
		}

If the pronlem is "validating the payload" I think the right place is a Mddleware that check for the specific headers, and when it recognize a request for valifation performs the hash verification (or any other kind of verification requested by the header). In case validation fails it should immediately return a client error result breaking the pipeline.
Controllers don't need to know anything about body validation, they must just process the payload

Perhaps.... even if the "normal" usage of validation is inside the action :
if(!ModelState.IsValid)
{
return BadRequest(ModelState);
}

In my case, even if it is a webhook, I have to validate the payload, the headers managed by a specific action and the signature.
For sure I can distribute the validation in various places but I prefer to have an all-in-one without use something that sound automagical to my colleagues,
Grazie per la collaborazione Franco.

It is something similar to authentication, or to Antiforgery token. You should validate it either in a controller filter or in a Middleware. The easiest place where to process the Raw Request is a Middleware but you may process it also in an Action Filters or in the Action Method itself.
I don't like the validation code be in the action method...Consider you might need the same validation logic in other controllers. So you should factor out the whole validation routine.
If you dont like also Action Filters consider using a generic controller that implements this protocol, other controllers might inherit from.
I suggest the following path:
1, Your action method uses [FromBody] and if you don want to use ActionFilters of Miffleware, has also parameters for your headers (using FromHeader).
2. Then call a protected method that performs the validation (possibly inherited from a base controller), and just pass it all needed headers. The validation routine may take the body from he Controller and use both raw body and header to perform validation. In case of error the routine add error nessages to the ModelState.
3. After the call to the validation routine, the action method either uses the model contained in the [FromBody] parameter, or returns the error.

Franco, have you tried to read the Request.Body before or after Action binding [FromBody] in the current release of aspnetcore mvc ?
I'm asking because the reason of my request is not about where put the code (13 code lines) I have already working.
BTW, this is the signature of the action
		[HttpPost()]
		public async Task<IActionResult> DayChanged(SellerDayStatsChanged data
			, [FromHeader(Name = "Notification-Topic")] string topic
			, [FromHeader(Name = "Notification-Type")] string notificationType
			, [FromHeader(Name = "Notification-Hook-ID")] string hookIdtopic
			, [FromHeader(Name = "Notification-URI")] string acknowledgedUri
			, [FromHeader(Name = "Notification-Signature")] string signature
			)
		{
			(SellerDayStatsChanged data, byte[] content) fromRequestBody = InfoFromRequest<SellerDayStatsChanged>();
			if(fromRequestBody.data != null)
			{
				data = fromRequestBody.data;
			}

This Action, with [FromBody] in the signature, works perfectly in tests and even in "integration" tests using the TestServer; but when you try to use it in production it fails because you can't read the Request.Body multiple times and you can't Seek it. To make it work you need a very ugly, and IMO dangerous, trick using a middleware to swap the Request.Body stream with a MemoryStream... or something like that.
If there is no intention to implement this feature OOTB there is no problem, as proposal is just a proposal.
Nuovamente grazie per la collaborazione Franco.

@fabiomaulo ,
If the problem is Request.Body being not stored, but read directly from the request stream so it can be read just once, then allowing several parameters match the same content would not solve the problem. Since a similar feature would nead the request body be read several times, in any case, which in turns implies the request body must be stored somehow.
So I see just 3 possible solutions:

Writing a new formatter for the content type used by the WebHook. That is a formatter, that is able to do the whole job by processing both headers and body when this content type is encountered (May you recognize your WebHook pattern from its content-type? If yes a new formatter for that content type may be written ).
Writing a middleware that enables a storing logic for the body when some conditions are met. This way one may read Request body several time. Stream rewind may be enabled by calling Request.EnableRewind(), BEFORE its first read. Thus you have to implement a middleware that performs this call when your WebHook is recognized somehow. You cant call it all the times, otherwise you might experience performance problems when you send big files to the servers.(images, videos, etc.)
Wait the WebHooks library is implemented for Asp.Net core too (see here).

Franco,

If the problem is Request.Body being not stored, but read directly from the request stream so it can be read just once, then allowing several parameters match the same content would not solve the problem.

As you can see in my proposal, and in my solution, because the parameter is just one (the ValueTuple) there is no need to read the Request.Body more than one. In that 13 code-lines there is no magic.
Grazie molte per la collaborazione Franco.
P.S. I hope you will need and you will use that library to produce webhooks.

Closing because there are no plans to implement this.


Add create-message workflow to further automate the microblog

Adds a create-message workflow to make it easier for other repos to automate with microblog
Couple things here:

Adds workflow_dispatch so the create-message workflow workflow can be triggered via the Actions UI or via API.
Adds workflow_call  so both this repository and other repositories can delegate to create-message as a job in their own workflows. This is pretty neat, there are lots more docs on Reusing worfklows.
Updates the existing announce-calls workflow to use the new create-message workflow

I've adapted my fork to test this out (only echo messages, don't tweet/toot): https://github.com/paul121/farmOS-microblog

[x] Create message via UI https://github.com/paul121/farmOS-microblog/commit/e0d177736969f30a87958ea4adfb1404f3300c81
[x] Create message from another repo using this workflow. https://github.com/paul121/farmOS-microblog/commit/efc75c03132a6504f7eb0afcede2dadc3d204c92

Using a test in my fork of farmOS/farmOS-community-blog to announce changes to blog posts: https://github.com/paul121/farmOS-community-blog/commit/e9d9b908d9cb8148299089b856daf8ab1ea420b2

[ ] Test that announce-calls still works...

For the farmOS organization we should be able to use secrets: inherit in the "caller" workflow from other repositories. This should make things easier because secrets.MICROBLOG_DEPLOY_KEY only needs to be configured in one place, not each repository. https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idsecretsinherit
I tried using secrets: inherit in my own testing but it seems it doesn't work across repos owned by the same personal account, needs to be an organization: https://github.com/orgs/community/discussions/23107

Force push so announce-calls uses secrets: inherit (there was actually a typo before)

This is great @paul121! I love how you've reworked this.
Once this gets merged we can simplify the code I just added to farmOS for announcing new releases: https://github.com/farmOS/farmOS/commit/9a968c5a2dcf49b2a2c290a59bc52b61cd73df3c

For the farmOS organization we should be able to use secrets: inherit in the "caller" workflow from other repositories. This should make things easier because secrets.MICROBLOG_DEPLOY_KEY only needs to be configured in one place, not each repository. https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idsecretsinherit

This is cool! I didn't know about that option.
One (very mild) concern comes to mind: is there a risk of exposing secrets we don't need/want to be exposing? Is it better to be explicit in this case?
The only secret we need for this is MICROBLOG_DEPLOY_KEY, which is currently an organization secret. But some of our repos have repo-specific secrets (eg: the farmOS/farmOS repo has secrets for pushing Docker images to Docker Hub). If we reuse this secrete: inherit pattern in other repos, are we potentially creating a future vulnerability?
It sounds like secrets: inherit would also send repository-specific secrets to the farmOS-microblog workflow. It's hard to imagine how this could be exploited, but the fact that the create-message workflow is specifically for broadcasting messages to the public it seems like something we should be extra careful with.
Or actually... now that I'm reading the docs a bit more, I wonder if it would work at all, in its current implementation:

If a caller workflow passes a secret that is not specified in the called workflow, this results in an error.

https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#onworkflow_callsecrets
We have two organization secrets currently, and farmOS/farmOS has repo secrets. It sounds like secrets: inherit would pas all four. But since our inputs.secrets ONLY lists MICROBLOG_DEPLOY_KEY would the workflow result in an error? Curious if you didn't see this in your tests because you only had one secret?
Seems like listing just the MICROBLOG_DEPLOY_KEY secret in each "caller" workflow would be fine, and more explicit/secure. What do you think?

Oh! One other thing... let's add a bit of validation to ensure the microblog post message doesn't have any potential issues with it.
I already ran into one, where a post was over 280 characters - went out to Mastodon just fine, but failed sending to Twitter. Not surprising, and something we can easily check for. The interesting thing, though, was that the post had a long URL in it, which Twitter usually converts automatically to a shortened URL, so it doesn't count against the number of characters - that does not happen via API posts apparently. I was able to simply copy and paste the message into the Twitter website and it accepted it no problem. So we are a bit more strictly limited to 280 characters in this microblog. But that's OK.
The other validation that comes to mind is single/double quotes in the message. Currently the commit command looks like this:
git commit --allow-empty --cleanup=whitespace -m '${{ inputs.message }}'
I think that means a message with a single quote wouldn't work.
I think we should change that line to use double-quotes (which will be less-common in messages than apostrophes), and add a validation check that fails the workflow if the message contains a double-quote.
Or... maybe we could use a Heredoc? https://www.tecmint.com/use-heredoc-in-shell-scripting/
I seem to recall some security considerations there as well though... auto-expanding environment variables maybe? Things to think about...

It sounds like secrets: inherit would also send repository-specific secrets to the farmOS-microblog workflow.
Or actually... now that I'm reading the docs a bit more, I wonder if it would work at all, in its current implementation:

It's a good question, but I imagine they would only inherit the necessary secrets defined in the "called workflow"... I am making an assumption, but I bet the docs are just poorly worded. Will try and look into this though

I think that means a message with a single quote wouldn't work.

Shoot, yea Testin' a single quote failed: https://github.com/paul121/farmOS-microblog/actions/runs/3516602692/jobs/5893371659

@paul121 I did some more work on this...
I rebased it onto the current origin/main, and added a commit that uses a heredoc block to wrap the commit message, which allows quotes to be included in the message.
I also added a commit that prevents the Mastodon step from running unless its running on the official farmOS/farmOS-microblog repo, and I added an "Echo message" step before that for debugging purposes (copied form one of your other commits).

let's add a bit of validation to ensure the microblog post message doesn't have any potential issues with it.
I already ran into one, where a post was over 280 characters - went out to Mastodon just fine, but failed sending to Twitter.

I don't think we need to worry about this anymore. I just removed the step that posts to Twitter because they are no longer a free and publicly available service, and the API calls stopped working. So we are only limited to 500 characters now for Mastodon, and I think it will be hard for us to hit that limit. And if we do, the post to Mastodon will fail and we can fix it manually if we need to.

One (very mild) concern comes to mind: is there a risk of exposing secrets we don't need/want to be exposing? Is it better to be explicit in this case?

I did a little testing of this myself and it doesn't seem like there's any risk here. I wasn't able to get any environment variables or secrets to be included in a message. So I think we're good (and the risk is so low anyway - we would know if anyone tried to exploit it).
So if these extra commits look good to you I think we can merge this!
I'd like to do that as a merge commit with a message of some kind, so we can decide what that should be when we're ready. Maybe we use it to link to this new forum topic: https://farmos.discourse.group/t/accepting-suggestions-for-the-microblog/1716


Re-organize website and add more bpfd-operator docs

Works on https://github.com/bpfd-dev/bpfd/pull/383

I would at the very least like #481 and #476 To go in before this so they don't have any rebase confusions.

Thanks for doing this @astoycos.  I think it’s time for more organization.  However, as I was looking through the changes, I was thinking of another alternative that’s probably closer to what we had originally.  What do you think about the following:

Home

Pull in the info from the README. (It would be nice to be able to document it one time, and then #include it both places, but I’d need to look into how to do this with md files).

Community

The Project Governance docs from Dave

Users
(Info needed by application developers who want to use bpfd for their application)

K8S Quick Start
API Reference
EBPF Bytecode Image Specifications
Examples

Developers
(Info needed by people who want to write bpfd code)

Developing the bpfd-operator
TODO: Other bpfd design docs
Info on contributing

@Billy99 Thanks for the help here some nits

Move the bpfd-operator Quick Start to the "Getting Started" section
Link the "getting Started" section to the Home page
Move "Community" to the last in the line so Home, Getting Started, Developers, Community
Maybe change Developers to Developing?
Add a Note to the home page that we're listed as an ebpf.io "emerging project"

Hrm now for some reason the landing page isn't "home" it's


Ensure volume is within valid range

Closes https://github.com/ambientsound/pms/issues/99

Codecov Report

Merging #102 into master will increase coverage by 0.15%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
+ Coverage   54.01%   54.17%   +0.15%
==========================================
  Files          51       51
  Lines        1779     1785       +6
==========================================
+ Hits          961      967       +6
  Misses        796      796
  Partials       22       22

Impacted Files
Coverage Δ

commands/volume.go
69.23% <100%> (+5.59%)
:arrow_up:

Continue to review full report at Codecov.

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


Reading /proc files line by line does not go into infinitive loop

RT #127370

PR associated with this test was merged https://github.com/MoarVM/MoarVM/pull/388


Fixing and updating script for TWE DI-435

Added matching dates based on b37 projects instead of b38.
Fixed script to only add samples to files_to_merge if QC status present.
Add printing for ids of failed samples.
Added output for projects missing QC status files.

This change is

Hello @RSWilson1! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file DI-435/find_vcfs_to_merge.py:

Line 206:27: E261 at least two spaces before inline comment
Line 208:80: E501 line too long (105 > 79 characters)
Line 234:80: E501 line too long (111 > 79 characters)
Line 235:80: E501 line too long (142 > 79 characters)
Line 403:80: E501 line too long (105 > 79 characters)
Line 414:80: E501 line too long (85 > 79 characters)
Line 427:80: E501 line too long (97 > 79 characters)


KZG verifier Solidity generator

Compatible with the proofs generated by https://github.com/arkworks-rs/poly-commit/blob/master/poly-commit/src/kzg10/mod.rs .
This is for the onchain (Ethereum's EVM) decider case, where the CommittedInstance's commitments use KZG for the instances of the main Nova circuit (and Pedersen for the CycleFold circuit instances).
The idea is to end up having a library method that given a trustedsetup of the kzg outputs the solidity code that verifies the commitments.
An implementation of a KZG Solidity verifier (non arkworks compatible) can be found at https://github.com/weijiekoh/libkzg/blob/master/sol/KZGVerifier.sol
Note that this is not yet the CLI, this is a library method. The CLI will be done once the KZG & Groth16 verifier generators are ready, the issue to track the CLI task is https://github.com/privacy-scaling-explorations/folding-schemes/issues/57

Will be tackling this as discussed! Can I/you update the title to reflect that this is the actual solidity contract library and not the CLI generator? Also, let's start a workspace where we can have an another repo for hosting this?


fix progress bar label for batch gui in APOC widget

Says 'Training on %s 8 Images' and then hidden by the bar

Closed with typo fix on main


Chrome driver.

I am currently having problems with the Chrome driver.
In the readme I read that downgrading Chrome would help. However, I have to say that from a security point of view this is not an option for me and on the other hand my company forbids version downgrade.
Is there a possibility to consider other fixes? Maybe in the Docker container itself?
gpt-researcher-gpt-researcher-1  | ERROR:    Exception in ASGI application
gpt-researcher-gpt-researcher-1  | Traceback (most recent call last):
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/uvicorn/protocols/websockets/wsproto_impl.py", line 234, in run_asgi
gpt-researcher-gpt-researcher-1  |     result = await self.app(self.scope, self.receive, self.send)
gpt-researcher-gpt-researcher-1  |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/uvicorn/middleware/proxy_headers.py", line 84, in __call__
gpt-researcher-gpt-researcher-1  |     return await self.app(scope, receive, send)
gpt-researcher-gpt-researcher-1  |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/fastapi/applications.py", line 289, in __call__
gpt-researcher-gpt-researcher-1  |     await super().__call__(scope, receive, send)
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/starlette/applications.py", line 122, in __call__
gpt-researcher-gpt-researcher-1  |     await self.middleware_stack(scope, receive, send)
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/errors.py", line 149, in __call__
gpt-researcher-gpt-researcher-1  |     await self.app(scope, receive, send)
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 79, in __call__
gpt-researcher-gpt-researcher-1  |     raise exc
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 68, in __call__
gpt-researcher-gpt-researcher-1  |     await self.app(scope, receive, sender)
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/fastapi/middleware/asyncexitstack.py", line 20, in __call__
gpt-researcher-gpt-researcher-1  |     raise e
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/fastapi/middleware/asyncexitstack.py", line 17, in __call__
gpt-researcher-gpt-researcher-1  |     await self.app(scope, receive, send)
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 718, in __call__
gpt-researcher-gpt-researcher-1  |     await route.handle(scope, receive, send)
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 341, in handle
gpt-researcher-gpt-researcher-1  |     await self.app(scope, receive, send)
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 82, in app
gpt-researcher-gpt-researcher-1  |     await func(session)
gpt-researcher-gpt-researcher-1  |   File "/usr/local/lib/python3.11/site-packages/fastapi/routing.py", line 324, in app
gpt-researcher-gpt-researcher-1  |     await dependant.call(**values)
gpt-researcher-gpt-researcher-1  |   File "/usr/src/app/main.py", line 60, in websocket_endpoint
gpt-researcher-gpt-researcher-1  |     await manager.start_streaming(task, report_type, agent, agent_role_prompt, websocket)
gpt-researcher-gpt-researcher-1  |   File "/usr/src/app/agent/run.py", line 38, in start_streaming
gpt-researcher-gpt-researcher-1  |     report, path = await run_agent(task, report_type, agent, agent_role_prompt, websocket)
gpt-researcher-gpt-researcher-1  |                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
gpt-researcher-gpt-researcher-1  |   File "/usr/src/app/agent/run.py", line 50, in run_agent
gpt-researcher-gpt-researcher-1  |     await assistant.conduct_research()
gpt-researcher-gpt-researcher-1  |   File "/usr/src/app/agent/research_agent.py", line 143, in conduct_research
gpt-researcher-gpt-researcher-1  |     research_result = await self.run_search_summary(query)
gpt-researcher-gpt-researcher-1  |                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
gpt-researcher-gpt-researcher-1  |   File "/usr/src/app/agent/research_agent.py", line 128, in run_search_summary
gpt-researcher-gpt-researcher-1  |     os.makedirs(os.path.dirname(f"./outputs/{self.directory_name}/research-{query}.txt"), exist_ok=True)
gpt-researcher-gpt-researcher-1  |   File "<frozen os>", line 225, in makedirs
gpt-researcher-gpt-researcher-1  | PermissionError: [Errno 13] Permission denied: './outputs/267a4451-ea58-4684-a385-ed11bd8ef6f7'
gpt-researcher-gpt-researcher-1  | INFO:     ('<IP_ADDRESS>', 38550) - "WebSocket /ws" [accepted]
gpt-researcher-gpt-researcher-1  | An error occurred while processing the url https://digital-strategy.ec.europa.eu/de/policies/european-approach-artificial-intelligence: Message: unknown error: Chrome failed to start: exited abnormally.
gpt-researcher-gpt-researcher-1  |   (chrome not reachable)
gpt-researcher-gpt-researcher-1  |   (The process started from chrome location /home/gpt-researcher/.cache/selenium/chrome/linux64/115.0.5790.170/chrome is no longer running, so ChromeDriver is assuming that Chrome has crashed.)
gpt-researcher-gpt-researcher-1  | Stacktrace:
gpt-researcher-gpt-researcher-1  | #0 0xaaaad8ab4e6c <unknown>
gpt-researcher-gpt-researcher-1  | #1 0xaaaad88095b4 <unknown>
gpt-researcher-gpt-researcher-1  | #2 0xaaaad882f800 <unknown>
gpt-researcher-gpt-researcher-1  | #3 0xaaaad882c608 <unknown>
gpt-researcher-gpt-researcher-1  | #4 0xaaaad886da74 <unknown>
gpt-researcher-gpt-researcher-1  | #5 0xaaaad8835950 <unknown>
gpt-researcher-gpt-researcher-1  | #6 0xaaaad8836bdc <unknown>
gpt-researcher-gpt-researcher-1  | #7 0xaaaad8a81c48 <unknown>
gpt-researcher-gpt-researcher-1  | #8 0xaaaad8a852e4 <unknown>
gpt-researcher-gpt-researcher-1  | #9 0xaaaad8a84e80 <unknown>
gpt-researcher-gpt-researcher-1  | #10 0xaaaad8a8be90 <unknown>
gpt-researcher-gpt-researcher-1  | #11 0xaaaad8a859ec <unknown>
gpt-researcher-gpt-researcher-1  | #12 0xaaaad8a5f7e4 <unknown>
gpt-researcher-gpt-researcher-1  | #13 0xaaaad8a9f114 <unknown>
gpt-researcher-gpt-researcher-1  | #14 0xaaaad8a9f2dc <unknown>
gpt-researcher-gpt-researcher-1  | #15 0xaaaad8aad468 <unknown>
gpt-researcher-gpt-researcher-1  | #16 0xffffac6fb648 start_thread
gpt-researcher-gpt-researcher-1  | #17 0xffffabea4fdc <unknown>
gpt-researcher-gpt-researcher-1  |
gpt-researcher-gpt-researcher-1  | An error occurred while processing the url https://www.europarl.europa.eu/news/en/headlines/society/20230601STO93804/eu-ai-act-first-regulation-on-artificial-intelligence: Message: unknown error: Chrome failed to start: exited abnormally.
gpt-researcher-gpt-researcher-1  |   (chrome not reachable)
gpt-researcher-gpt-researcher-1  |   (The process started from chrome location /home/gpt-researcher/.cache/selenium/chrome/linux64/115.0.5790.170/chrome is no longer running, so ChromeDriver is assuming that Chrome has crashed.)
gpt-researcher-gpt-researcher-1  | Stacktrace:
gpt-researcher-gpt-researcher-1  | #0 0xaaaac5d84e6c <unknown>
gpt-researcher-gpt-researcher-1  | #1 0xaaaac5ad95b4 <unknown>

It works fine with Firefox. That seems a better idea than downgrading an existing version of Chrome for anyone who has the option of also using Firefox.

How did u set Firefox as browser for this?

Fixing the user-agent param worked for me I think. Maybe a windows formating issue.
options.add_argument(CFG.user_agent)
options.add_argument(f"user-agent={CFG.user_agent}")

@ill-yes You should be able to switch your default browser from Chrome to Firefox in your computer's settings. I didn't want to do that, I just copied and pasted the URL from Chrome into Firefox.

@tomseidel thank you for the suggestion we've just merged that #159


How to set batch size when training AllSet?

Hi, this is a nice work. I read the code:
https://github.com/jianhao2016/AllSet/blob/0d0e399a9168829fa898dd56f3d32bee36953b04/src/train.py#L325-L327
https://github.com/jianhao2016/AllSet/blob/0d0e399a9168829fa898dd56f3d32bee36953b04/src/train.py#L345
https://github.com/jianhao2016/AllSet/blob/0d0e399a9168829fa898dd56f3d32bee36953b04/src/train.py#L478
It seems that you train the model with all the data in a batch. Thus I have a question about how to set batch size when I train the AllSet model. Thank you.

Hi @sakuraiiiii ,
This is a great question! Unfortunately, mini-batch training in AllSet is currently an open problem, where we stated in our paper (Appendix A). We conjecture one can develop a GraphSAINT or ClusterGCN type of methods to allow mini-batch training. There are also many open directions stated in Appendix A, which we conjecture that one can leverage the similar idea in the GNN literature.
Please let me know if you manage to develop mini-batch training! It would be a great improvement to AllSet and to general hypergraph neural networks!
Best,
Eli


docs Fix some links in docs is broken

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

./run buildall

./run buildall

run p0


Advice on IThemeSelector

I think we can modify the definition of IThemeSelector as follows,
public interface IThemeSelector
{
///
/// Returns a  representing a specific theme
/// and a priority order. The highest priority value will be used.
///
///
Task GetThemeAsync();
///
/// if CanSet is true,this will be used to set the theme
///
///
void SetTheme(string themeName);
///
/// it is a flag to tell the stysem,the selector either can be seted or not.
///
bool CanSet { get; }
///
/// this will tell the system,the theme.txt need contains this Tag
///
string Tag { get; }
///
/// a Display name,that will be used in Orchard.Themes.AdminMenu to build a menu
///
LocalizedString DisplayName { get; }
///
/// this like a parameter to build a menu
///
string Name { get; }
}
This enables the implementation of the required template for different scenarios
For example, we can set up a website template, the Admin template, or you can also set the mobile terminal templates, etc.
it will be like this:

Click setSite maybe like this:

Click setAdmin maybe like this:

and if do this ,we must modify AdminThemeSelector  like this:
public class AdminThemeSelector : IThemeSelector
{
private readonly IAdminThemeService _adminThemeService;
private readonly IHttpContextAccessor _httpContextAccessor;
    public AdminThemeSelector(
        IStringLocalizer<AdminThemeSelector>s,
        IAdminThemeService adminThemeService,
        IHttpContextAccessor httpContextAccessor
        )
    {
        _adminThemeService = adminThemeService;
        _httpContextAccessor = httpContextAccessor;
        T = s;
    }

    public async Task<ThemeSelectorResult> GetThemeAsync()
    {
        if (AdminAttribute.IsApplied(_httpContextAccessor.HttpContext))
        {
            string adminThemeName = await _adminThemeService.GetAdminThemeNameAsync();
            if (String.IsNullOrEmpty(adminThemeName))
            {
                return null;
            }

            return new ThemeSelectorResult
            {
                Priority = 100,
                ThemeName = adminThemeName
            };
        }

        return null;
    }
    IStringLocalizer T { get; set; }
    public bool CanSet { get { return true; } }
    public void SetTheme(string themeName)
    {
        _adminThemeService.SetAdminThemeAsync(themeName);
    }

    public string Tag { get { return "admin"; } }

    public LocalizedString DisplayName { get { return T["AdminTheme"]; } }

    public string Name { get { return "adminTheme"; } }
}

and so on.

Hope to be adopted and I will also modify the implementation of O1

I have copied some of the antoine and sipke code, I am very sorry about this point, because I am a new git user, will not be applied to merge, pull, etc., can only manually copy some of the necessary code.

We will definitely merge their PR first before accepting your work.
Also, as a general rule please file an issue with your suggestions, they look interesting but we'd like to be able to talk about it before having you start a huge piece work.

Closing the PR because there are too many conflicts, but we are keeping the issue targetting RC as it's a good suggestion. We'll revisit it soon.


❓ [Question] how to specify dynamic shape when using torch_tensorrt.save

❓ Question

I was following the documentation on compiling a model with dynamic input shape. When saving the compiled graph module (following this), the new torch_tensorrt.save(module, path, inputs) API requires inputs to be all tensors. How do I pass dynamic shapes to torch_tensorrt.save? Error: https://github.com/pytorch/TensorRT/blob/77278fe395d6ffdd456fd7a8a94852cd27ee63a9/py/torch_tensorrt/_compile.py#L420
import torch
import torch_tensorrt

model = torch.hub.load('pytorch/vision:v0.10.0', 'resnet50', pretrained=True)
model.eval().cuda()
inputs = [torch_tensorrt.Input(min_shape=[1, 3, 224, 224],
                              opt_shape=[4, 3, 224, 224],
                              max_shape=[8, 3, 224, 224],
                              dtype=torch.float32)]
trt_gm = torch_tensorrt.compile(model, ir="dynamo", inputs=inputs)
torch_tensorrt.save(trt_gm, "trt_gm.ep", inputs=inputs)

WARNING:torch_tensorrt.dynamo.conversion.aten_ops_converters:Unable to import quantization op. Please install modelopt library (https://github.com/NVIDIA/TensorRT-Model-Optimizer?tab=readme-ov-file#installation) to add support for compiling quantized models
INFO:torch_tensorrt.dynamo._compiler:Compilation Settings: CompilationSettings(enabled_precisions={<dtype.f32: 7>}, debug=False, workspace_size=0, min_block_size=5, torch_executed_ops=set(), pass_through_build_failures=False, max_aux_streams=None, version_compatible=False, optimization_level=None, use_python_runtime=False, truncate_double=False, use_fast_partitioner=True, enable_experimental_decompositions=False, device=Device(type=DeviceType.GPU, gpu_id=0), require_full_compilation=False, disable_tf32=False, assume_dynamic_shape_support=False, sparse_weights=False, refit=False, engine_capability=<EngineCapability.STANDARD: 1>, num_avg_timing_iters=1, dla_sram_size=1048576, dla_local_dram_size=1073741824, dla_global_dram_size=536870912, dryrun=False, hardware_compatible=False, timing_cache_path='/tmp/timing_cache.bin')

INFO:torch_tensorrt.dynamo._compiler:Partitioning the graph via the fast partitioner
INFO:torch_tensorrt [TensorRT Conversion Context]:[MemUsageChange] Init CUDA: CPU +1, GPU +0, now: CPU 449, GPU 1622 (MiB)
INFO:torch_tensorrt [TensorRT Conversion Context]:[MemUsageChange] Init builder kernel library: CPU +1622, GPU +288, now: CPU 2218, GPU 1910 (MiB)
WARNING:torch_tensorrt.dynamo.conversion.converter_utils:Detected unparseable type in node formatting: <class 'torch.SymInt'>
INFO:torch_tensorrt.dynamo.conversion._TRTInterpreter:TRT INetwork construction elapsed time: 0:00:00.609398
INFO:torch_tensorrt [TensorRT Conversion Context]:Global timing cache in use. Profiling results in this builder pass will be stored.
INFO:torch_tensorrt [TensorRT Conversion Context]:Detected 1 inputs and 1 output network tensors.
INFO:torch_tensorrt [TensorRT Conversion Context]:Total Host Persistent Memory: 343968
INFO:torch_tensorrt [TensorRT Conversion Context]:Total Device Persistent Memory: 7168
INFO:torch_tensorrt [TensorRT Conversion Context]:Total Scratch Memory: 6424576
INFO:torch_tensorrt [TensorRT Conversion Context]:[BlockAssignment] Started assigning block shifts. This will take 86 steps to complete.
INFO:torch_tensorrt [TensorRT Conversion Context]:[BlockAssignment] Algorithm ShiftNTopDown took 0.644934ms to assign 4 blocks to 86 nodes requiring 65830912 bytes.
INFO:torch_tensorrt [TensorRT Conversion Context]:Total Activation Memory: 65830912
INFO:torch_tensorrt [TensorRT Conversion Context]:Total Weights Memory: 127383968
INFO:torch_tensorrt [TensorRT Conversion Context]:Engine generation completed in 0.553365 seconds.
INFO:torch_tensorrt [TensorRT Conversion Context]:[MemUsageStats] Peak memory usage of TRT CPU/GPU memory allocators: CPU 16 MiB, GPU 129 MiB
INFO:torch_tensorrt [TensorRT Conversion Context]:[MemUsageStats] Peak memory usage during Engine building and serialization: CPU: 4064 MiB
INFO:torch_tensorrt.dynamo.conversion._TRTInterpreter:Build TRT engine elapsed time: 0:00:00.649827
INFO:torch_tensorrt.dynamo.conversion._TRTInterpreter:TRT Engine uses: 129675836 bytes of Memory
INFO:torch_tensorrt [TensorRT Conversion Context]:Serialized 26 bytes of code generator cache.
INFO:torch_tensorrt [TensorRT Conversion Context]:Serialized 292352 bytes of compilation cache.
INFO:torch_tensorrt [TensorRT Conversion Context]:Serialized 3744 timing cache entries
WARNING: [Torch-TensorRT] - Detected this engine is being instantitated in a multi-GPU system with multi-device safe mode disabled. For more on the implications of this as well as workarounds, see the linked documentation (https://pytorch.org/TensorRT/user_guide/runtime.html#multi-device-safe-mode)
Traceback (most recent call last):
  File "test.py", line 11, in <module>
    torch_tensorrt.save(trt_gm, "trt_gm.ep", inputs=inputs)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "path/.venv/lib/python3.11/site-packages/torch_tensorrt/_compile.py", line 420, in save
    raise ValueError(
ValueError: Not all inputs provided are torch.tensors. Please provide torch.tensors as inputs

What you have already tried

Environment

Build information about Torch-TensorRT can be found by turning on debug messages

PyTorch Version (e.g., 1.0): 2.4
CPU Architecture:
OS (e.g., Linux): Linux
How you installed PyTorch (conda, pip, libtorch, source):
Build command you used (if compiling from source):
Are you using local sources or building from archives:
Python version: 3.11
CUDA version: 12.1
GPU models and configuration:
Any other relevant information:

Additional context

Hello @Qi-Zha0 ,
You should pass torch tensor inputs to the save API. These inputs should be in the range (min_shape, opt_shape, max_shape). eg: torch_tensorrt.save(trt_gm, "trt_gm.ep", inputs=[torch.randn(4, 3, 224, 224).cuda()]). We shall update the documentation to make this more clear. Thanks !!

@peri044 Thank you for clarifying!


Unable to implement Origins: Classes as a dependency using Jitpack.

As the title suggests, implementing a Jitpack repository that points to any recent version of Origins Classes will fail to download the mod as a dependency.
gradle.properties relevant content:
origins_version=v1.6.2
classes_version=v1.4.0

(does not work with 1.4.0 as the parameter either)
build.gradle relevant content:
repositories {
       minecraft "com.mojang:minecraft:${project.minecraft_version}"
       maven {
		name = "Ladysnake Libs"
		url = 'https://ladysnake.jfrog.io/artifactory/mods'
	}
	maven {
		url = 'https://maven.cafeteria.dev'
		content {
			includeGroup 'net.adriantodt.fabricmc'
		}
	}
	maven {
		url "https://maven.jamieswhiteshirt.com/libs-release"
		content {
			includeGroup "com.jamieswhiteshirt"
		}
	}
	maven {
		url "https://jitpack.io"
	}
	maven {
		url "https://maven.shedaniel.me/"
	}
	maven {
		url "https://maven.terraformersmc.com/"
	}
}
dependencies {
       minecraft "com.mojang:minecraft:${project.minecraft_version}"
       mappings "net.fabricmc:yarn:${project.yarn_mappings}:v2"
       modImplementation "net.fabricmc:fabric-loader:${project.loader_version}"
       modImplementation "net.fabricmc.fabric-api:fabric-api:${project.fabric_version}"
       modImplementation "com.github.apace100:origins-fabric:${project.origins_version}"
       modImplementation "com.github.apace100:origins-classes:${project.classes_version}"
}

Expected:
Origins: Classes is implemented as an external library for the mod.
Actual:
An error log is produced:
A problem occurred configuring root project 'Project'.
> Could not resolve all dependencies for configuration ':modImplementation'.
   > Could not find com.github.apace100:origins-classes:v1.4.0.
     Searched in the following locations:
       - file:/C:/Users/user/Desktop/Project/.gradle/loom-cache/remapped_mods/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.pom
       - https://maven.fabricmc.net/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.pom
       - https://libraries.minecraft.net/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.pom
       - https://libraries.minecraft.net/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.jar
       - https://repo.maven.apache.org/maven2/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.pom
       - file:/C:/Users/user/.gradle/caches/fabric-loom/v1.4.0/origins-classes.jar
       - file:/C:/Users/user/Desktop/Project/.gradle/loom-cache/v1.4.0/origins-classes.jar
       - https://ladysnake.jfrog.io/artifactory/mods/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.pom
       - https://jitpack.io/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.pom
       - https://maven.shedaniel.me/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.pom
       - https://maven.terraformersmc.com/com/github/apace100/origins-classes/v1.4.0/origins-classes-v1.4.0.pom
     Required by:
         project :

Jitpack Latest Files (https://jitpack.io/#apace100/origins-classes):

Jitpack Build Log (https://jitpack.io/com/github/apace100/origins-classes/v1.4.0/build.log)
I believe I've taken all the necessary steps to implement the mod as a library here. If I've done anything wrong please correct me so I can fix it. Cheers.

Hi there!
I hadn't yet added a jitpack.yml file to this repository, which is necessary to allow JitPack to build the source with the proper Java version.
I did this now in this commit: d27b9c9
You should be able to use the short commit hash (d27b9c9) as the version for Origins: Classes in your build.gradle / gradle.properties. Keep in mind doing this the first time requests JitPack to start building that commit, so it will likely time out and then only work on your second or third try.
Let me know if it works so I can close the issue. :)

Still unable to download, with the same errors as before (even after multiple attempts to build, this is visible on the Jitpack page also). I have tried both d27b9c9 and d27b9c9d6f42bda93ec5d8e0d01f40e9f48056aa as hashes for the version number.

Still no dice. Is there a specific version of Origins I should be adding as a dependency as well? I'm using v1.6.2 as the current version.

I'm really sorry about this!
As a temporary work-around, you can change the following dependency line in your build.gradle:
       modImplementation "com.github.apace100:origins-fabric:${project.origins_version}"

To:
       modImplementation "com.github.eggohito:origins-fabric:${project.origins_version}"

And it should be able to fetch the dependencies then.


Removed custom javascript formatter

will use basic formatter instead; addresses #42, #39, #23

LGTM
Will check on top 10 repos.


Align Hook Naming

useCall vs useContractCall
useTx vs useContractTx
useDryRun vs useContractDryRun

As of now we prefer the simpler version, it could clash with potential substrate/runtime hooks down the line but maybe these are handled differently anyways and are more advanced.

We can use other naming conventions such as useChainCall or useRuntimeCall for calling the runtime vs a contract. I like that we can make contract dev-ex as ergonomic as possible.

I think that useEvents should be the subscription version. Then we add useEventsAt as mentioned here

We can use other naming conventions such as useChainCall or useRuntimeCall for calling the runtime vs a contract. I like that we can make contract dev-ex as ergonomic as possible.

I like that! Maybe even back to useContractCall as well? So I guess useChainCall will be more for generic extrinsics and useContractCall a more opinionated version for convenient use?


Fixed Maven URLs to be HTTPS

Maven repository now requires HTTPS.  Fixed Maven URLs to be HTTPS

Thanks for the fix!


Roll engine 5b7c0089e3ca..ea4654a30b06 (1 commits)

https://github.com/flutter/engine/compare/5b7c0089e3ca...ea4654a30b06
git log 5b7c0089e3caeaa5d118530c2cf17470214bf4c0..ea4654a30b0625d701fd42bbcc2743794d7a47ce --no-merges --oneline
ea4654a30 Update GN to git_revision:152c5144ceed9592c20f0c8fd55769646077569b (flutter/engine#9987)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (aaclarke@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


ODS-5246 - Update SDKGen to target net6

Also updated nuget packages to match versions they were updated to in other projects. Latest log4net is now 2.0.14, but since other projects were updated to 2.0.13, it was updated only to that version.
No updates were needed to the implementation repo since all the relevant packages pull "latest"

Invoke-SdkGen currently fails because it tries to find the executable in the netcoreapp3.1 folder, please rename it to net6.0 here:
https://github.com/Ed-Fi-Alliance-OSS/Ed-Fi-ODS-Implementation/blob/main/logistics/scripts/modules/TestHarness.psm1#L71

This has been updated now to no longer look for the hard coded 3.1 path and instead look for it in a similar fashion to what we do for the test harness executable.


Mobile pinning touch menu layout/meny styling

@ryantmer There are certainly a few things that will change based on rebasing once #37 is merged, so I'll do that once that's done.

Few little things, but LGTM :shipit: I feel like the position logic could be simpler, since it's kinda hard to follow at the moment, but a) that could just be Friday brain now allowing me to math, and b) it works, so all good!

@ryantmer It we don't care as much about displaying above the touch point, and make horizontal layout symmetric, definitely. But, I think layout will be changed once we add more items, so I'll revisit then =)

Hm, yeah fair point. That'll complicate things fo sho.


Gray background can be seen through row animation

For grouped tables, the default gray background can be seen blinking in the animation area while row animation is being performed. The end result is all white, as expected, but the animation looks glitchy.

But when I animate via row hight the glitch is not present, so it's fine. Thanks for the good work.


[DOCS] Slack channel #cdk-cicd-wrapper not found

Describe the issue
The README file mentions the #cdk-cicd-wrapper Slack channel in cdk.dev. Have you created and joined this channel yet? I can't find it in the cdk.dev Slack workspace.

Links
https://github.com/cdklabs/cdk-cicd-wrapper?tab=readme-ov-file#community

Thanks for reporting this, we are still working on having a dedicated public channel for this repository, stay tuned. In case you have any open topics please feel free to submit a Github issue for now.


Feature request: Access sensu logs

It would be nice to be able to access the sensu (server) logs with uchiwa in case there is a problem with a check not working properly. For example, if you write a handler and your json is formatted wrong, then that handler is just ignored by sensu and one needs to read in the sensu server log to see that it was in fact a json problem. I would guess it is the same with checks, if they are not properly set up there is only evidence in the server logs about a problem.
(I realise that it might be out of scope for uchiwa)

Hi,
Unfortunately this is currently impossible, considering that Uchiwa only interacts with the Sensu API, for simplicity and scaling concerns, which means that this information would need to be accessible from the API.
Thanks


Share mistake

Make it possible to link to a specific mistake. https://mistakes.computer/1? Support JSON too? https://mistakes.computer/1.json. Add a "Share this mistake" link in the footer.

One request for this feature was to use a slug for the URL:

I really wanna be able to link people to specific mistakes like https://mistakes.computer/dns


Desktop sharing with desktop sound

Hi,
When i play a video, the other ones cannot hear the video audio.
I think your code does not support hat yet.
Is this something you have planned for the future?

Yeah currently no audio is being sent. This is to prevent feedback. For example it might transfer back the audio of all speakers and thus cause feedback.
I'm wondering what's your use case for sending computer audio?

Thanks for you answer.
I would like to preview a video production with a customer. (without uploading it to the cloud, because it is not finished yet)

Interesting! I'm going to try to add an option to share the computer audio as well. I'm just kind of swamped with other projects at the moment so it might take a few weeks.

Yes, we miss that in other applications like teamviewer. That is why we tried your solutions. Works well!
When you add this thing, you would make your app unique (in my opinion).

Hi jeremija.
Did you already have time to look into this feature request?

Hi @alainvanderbroeck, I just pushed version 4.0.9 which has support for this. I'm waiting for the build to complete and then I'm going to push it to peercalls.com.

However, I noticed that in Chrome it only works when choosing a Chrome tab to share (and you have to select the Share audio checkbox at the bottom of the picker provided by Chrome UI) - it doesn't work for sharing the whole desktop. This is  a browser restriction, and there doesn't seem to be anything I can do about it.
I couldn't get it to work in Firefox and I haven't tried Safari yet. Please let me know what your experience is.

Thank you for your quick response.
We are looking forward to this feature. Many thanks!
We are going to try it this week. We'll let you know what our experience is.

Great - I'd recommend you to take a look at the Browser Compatibility table here: https://developer.mozilla.org/en-US/docs/Web/API/MediaDevices/getDisplayMedia#Browser_compatibility
Looks like Chrome and Edge on Windows and Chrome OS/Edge OS can capture audio of the whole machine, whereas on Mac and Linux it's limited to selecting a single Chrome tab only.


Encountered invalid end-to-end format

Rasa version: 1.9.6
Python version: 3.6.9
Operating system (windows, osx, ...): windows
Issue: Cannot test my model because of an error in my stories, but they look good, and I've been using the same stories with no problems.
Error (including full traceback):
Error in line 2: Encountered invalid end-to-end format for message time_remaining. Please visit the documentation page on end-to-end testing at https://rasa.com/docs/rasa/user-guide/testing-your-assitant/#end-to-end-testing/

Command or request that led to error:
rasa test

stories.md:
## time_remaining
* time_remaining
  - action_time_remaining

## current_dose
* current_dose
  - action_current_dose

## treatment_end
* treatment_end
  - action_treatment_end

## required_dose
* required_dose
  - action_required_dose

Hi , look at your e2e stories file not the stories.md file , and read the doc on how to test the model .

Hi, where do I find my e2e stories? I read the docs, I followed it for testing my model

I believe they reside in the tests folder that comes with rasa init ?

Thanks for raising this issue, @alwx will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗


BUG - evaluate_result breaks down when it accepts a single dict argument

After #576,
It happens that the depreciation cycle doesn't work for benchmarks that anticipated passing in solver_result as a dictionary.
Indeed, this check destruct the input dict
https://github.com/benchopt/benchopt/blob/c689d242ac7910d9f1b4dbf9b0dff915925559e2/benchopt/base.py#L439-L446
resulting in
TypeError: compute() got an unexpected keyword argument 'x'

as illustrated in the PR change.

Pinning @agonon for flagging that

This issue should be fixed by #631

This should have been resolved in #631 . Feel free to reopen if this is not the case.


Deduplicate common conversions

This PR is attacking the high code duplication of 31% that Codacy is showing us. This makes the code certainly hard to fix and read.
The common conversion methods have been moved into utils.go.

So although the coverage is reducing the deduplication is -54. I think it would be appropriate to add tests at a later stage as we will have to add new fixtures and add new test methods.


Memory leak after disposing SqliteCommand

aspnet/Microsoft.Data.Sqlite#388 this pull request introduced a list of weak references to commands inside SqliteConnection. Commands are disposed and the list are cleared at Connection closing. But if the connection lifetime is long, this list keeps growing. I think the command should be removed from this list on SqliteCommand.Dispose.
Microsoft.Data.Sqlite version: 2.1.0

The problem is not with weak references but with the Connection._commands list itself. Every new command adds an item to this list, but then items are never removed from the list, only when Connection is closed.
For example:
using Microsoft.Data.Sqlite;

namespace MemoryLeak
{
    class Program
    {
        static void Main(string[] args)
        {
            var connection = new SqliteConnection("Data Source=:memory:");
            connection.Open();
            while(true)
            {
                using (var cmd = new SqliteCommand("SELECT 1", connection))
                {
                }
            }
        }
    }
}

Let it run for a few seconds and then put a breakpoint inside the while loop and check the _commands field of the connection. It will contain a lot of items.
I tried to fix it in https://github.com/v1rusw0rm/EntityFrameworkCore/commit/5911ace285db6a045f0ce976472872ef6ce5f21c but can't figure out how to do it better without copying the list.

Getting out of memory leak in .NET core 2.2 because of this. Is there no way to remove it from internal list? Happy to use reflection or any hacks...


feat: added cookies API definitions

Purpose
Add definitions of the Cookies API functionality.
Approach

Add interfaces for the Cookies API functionality.
Fix test
Refactor errors messages
actionRequiredCookieArguments
actionCookieArgumentError
actionCookieArgumentsError
actionUrlCookieArgumentError
actionUrlsCookieArgumentError
actionStringOptionError
actionDateOptionError
actionNumberOptionError

References
Firt part
Second part
Pre-Merge TODO

[ ] Write tests for your proposed changes
[ ] Make sure that existing tests do not fail
[ ] Write documentation

error messages (with some further questions):

Cookies argument is required.
The mandatory "Cookies" argument is missing.

The "cookie" is expected to be an object, a string or array of objects or strings.
The value of the "cookie" [why do we use the singular 'cookie' here, and the plural above?] argument does not belong to an acceptable data type: Object, String, or Array of objects or strings.

The "cookie" at number ${err.index + 1} is expected to be an object or a string.
The value of the cookie number ${err.index + 1} does not belong to an acceptable data type: Object or String.

The "urls" argument isn't valid.
I don't know what to replace this error message with because the word 'invalid' can mean lots of different things.

Elements at index ${err.index + 1} of the "urls" argument isn't valid.
The 'urls' array contains an element with an invalid value. Element index: ${err.index + 1}   [+ see my earlier comment]

The "${err.optionName}" option is expected to be a string value, but it was ${err.actualValue}.
The value of the "${err.optionName}" option belongs to an unsupported data type (${err.actualValue}). The "${err.optionName}" option only accepts String values.

The "${err.optionName}" option is expected to be a Date value, but it was ${err.actualValue}.
The value of the "${err.optionName}" option belongs to an unsupported data type (${err.actualValue}). The "${err.optionName}" option only accepts Date values.

The "${err.optionName}" option is expected to be a number, but it was ${err.actualValue}.
The value of the "${err.optionName}" option belongs to an unsupported data type (${err.actualValue}). The "${err.optionName}" option only accepts Number values.


[DOC]: Wrong information - AlexNet 2012

📚 The doc issue

Back in 2012, AlexNet won the champion of the ImageNet competition, and it was trained on a single GPU.

Two GTX 580 GPUs with 3GB.
Ref: https://www.colossalai.org/docs/concepts/distributed_training

Hi @INF800  Thank you for pointing it out, we will fix it as soon as possible 🔥

Fixed and will be updated to the website soon : )


Update Microsoft.EntityFrameworkCore.rd.xml

The test failure is interesting.

It's not related. Probably part of https://github.com/aspnet/EntityFrameworkCore/issues/16342


bazel: Test with 6.0.0-pre.20220414.2

To get ahead of any issues with https://github.com/envoyproxy/envoy/pull/21070

needs https://github.com/envoyproxy/envoy-mobile/pull/2227


tab not visible

Hi mike when i implement your library, i always get this error on my console
[ngTransclude:orphan] http://errors.angularjs.org/1.4.8/ngTransclude/orphan?p0=<div class%3D"scrtabs-tabs-movable-container" ng-transclude%3D"">
and my tab not visible. can you give me solution for this issue? thanks

I'd need some information about how you're using it. I haven't seen that error before.
If you can put together a plunk that shows the error occurring, that would be the best. If you can't put it in a plunk, I'd at least need to see your markup and, if your tabs are data-driven, the underlying tabs data.

Hi mike i try to upload plnkr.co. but still not working, and i try it until working.
FYI i will use your library with combine ui-grid, as like a picture bellow

so there is a tab with ui-grid in every tabs.
i hope this will help you to explain it.


I often meet an error called "REGDB_E_CLASSNOTREG Class not registered"

this is my model IDL
namespace BlankApp1
{
    runtimeclass  Model : Windows.UI.Xaml.DependencyObject, Windows.UI.Xaml.Data.INotifyPropertyChanged
    {
        Model();
        String Title;
    }
}

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

ID: a4db0aab-b906-9dec-381c-ceccc1e90c45
Version Independent ID: 493a4c50-ffa1-5e32-c757-92306c952887
Content: XAML controls; bind to a C++/WinRT property - UWP app developer
Content Source: windows-apps-src/cpp-and-winrt-apis/binding-property.md
Product: windows
GitHub Login: @stevewhims
Microsoft Alias: stwhi

Hi, @cjw1115, that error can happen if you instantiate a locally-implemented runtime class via the projected type's default constructor. For example, if you declare a field like this:
private:
    BlankApp1::Model m_model; // This attempts to call the projected type's default constructor, which won't work if you've declared BlankApp1::Model in the same project, because that type is not registered.

Instead, do this:
private:
    BlankApp1::Model m_model{ nullptr }; // This calls a special constructor on the projected type, which doesn't attempt to activate the runtime class (hence it needn't be registered).

And then assign a value to that field later like this:
m_model = winrt::make<BlankApp1::implementation::Model>(); // BlankApp1::implementation::Model is the implementation type, but winrt::make returns the projected type. Again, this succeeds without the type needing to be registered.

The topic in question shows this same pattern. And the Troubleshooting topic indicates other reasons you might be seeing REGDB_E_CLASSNOTREGISTERED.

It works well after I correct my code as you described. Thanks a lot~

Great! Thanks, @cjw1115. ~Steve


Update iteration_5_spec.rb

@applegrain: 263519844 when filtered for valid invoices and one time buyers returns a quantity of 15, while 263518806 has a quantity of 24
This is the last pull request from me!

This is the last pull request from me!

Does this mean you're done with the project? Congrats!! :trophy:


webgl Video performance issue in V4.4.0

Hello,
I want to make the switch from v3.0.9 to v4.4.0 and everything is going smooth but i'm getting quite a frame drop if i'm playing a fullhd video. I'm using the webgl renderer. The machines that i'm testing this on is an old laptop and a NUC machine with only integrated intel graphics card (Iris 540). The code is quite simple, create a renderer, create a video, render the video.
In V3.0.9 i am getting 60fps without any issues and in V4.4.0 i'm getting around 40 fps. If i switch to software rendering i get 60fps.
I looked a bit into the implementation and it seems that the VideoBaseTexture changed to some extent so i am wondering if someone experienced this before.
The code is (disregard the file name as it is only 1920x1080):
<!DOCTYPE html>
<html>
  <head>
    <meta charset="UTF-8">
    <title>Hello World!</title>
    <style>
      body {
          overflow:hidden;
          padding:0px;
          margin:0px;
      }
    </style>
    <script src="lib/pixi4/pixi.js"> </script>
    <script src="lib/fpsmeter/fpsmeter.js"> </script>
  </head>
  <body bgcolor="#AAAAAA">
    <script>
      var width =  window.screen.width;
      var height =  window.screen.height;
      var renderer = PIXI.autoDetectRenderer(width, height - 0, { transparent: true, antialias: true });
      document.body.appendChild(renderer.view);

      var texture = PIXI.Texture.fromVideo("media/4k1.mp4");
      var videoSprite = new PIXI.Sprite(texture);
      videoSprite.width = renderer.width;
      videoSprite.height = renderer.height;

      var meter = new FPSMeter();
      animate();
      function animate(){
          meter.tick();
          renderer.render(videoSprite);
          requestAnimationFrame(animate);
      }
    </script>
  </body>
</html>

and, of course, Thank you!

Hi there! can you try:
var renderer = PIXI.autoDetectRenderer(width, height - 0, { transparent: true, antialias: true, legacy:true });

cheers!

Thanks for the quick response. Gave the legacy: true  a shot but i get the same results.

Ok, so i looked into the code and i got to the point where the VideoBaseTexture is updated every frame while playing. It seems that in V3 this was done with window.requestAnimationFrame(this._onUpdate); and now, in V4.4.0,  it is done by subscribing to a "ticker" ticker.shared.add(this.update, this);. Can not say if this is somehow messing with the frames timing or if the issues goes even further when the textures are uploaded to the graphics memory.
Will look some more into it but i would really appreciate any (constructive :) input.

I was kind of hoping that the issue would be with the ticker but it is not. Digging deeper.

@GoodBoyDigital  , transparent: true will take a better performance for WebGLRenderer ?

@finscn removing the transparent : true does improve the framerate a bit but only up to around 50 fps.
So i changed the example to add 103 rotating text on top of the video. The V3 keeps running at 60fps (this is how i got to 103) but V4.4.0 drops to something like 30fps.
I also tried with only rotating texts (no video) and i can get up to 156 of them on V3 to keep the fps to 60. V4.4.0 also runs at 60fps with the legacy: true. Without it the fps drops to around 45.
From all this i can deduce that the changes to the renderer between V3 and V4.4.0 are somehow hitting the performance and the legacy option is solving that but not for the videos.
So i looked a bit more into the code and it seems that the legacy parameter is used in:

WebGLRenderer constructor: _pixiGlCore2.default.VertexArrayObject.FORCE_NATIVE = true;
SpriteRenderer.prototype.onContextChange to this.MAX_TEXTURES = 1;
There is also a legacy renderer used in RenderTexture but i don't think that this one is used in my example.

Not sure what the VAO alternative for the non legacy option is and i have very limited understanding of the way PIXI maps to webgl so i will stick to V3.0.9 for now i guess the performance issue is not present only on my machine so here is the new code:
`

Hello World!

I have the same issue 👎
If I force the PIXI.Application to Canvas 2D => 60fps.
else WebGL => 45/40 fps.
I however need the WebGL renderer for filters.
Any solution ?

@antoninlanglade is your case also about VideoSprite?
Guys, we cant optimize that, its WebGL and browser issue how it uploads those frames. Especial on older Intel HD. I know optimizations for older Intel HD webgl but they are certainly not about videos.

For me it's a VideoBaseTexture in full screen
 this.baseTexture = PIXI.VideoBaseTexture.fromUrl(url);     this.baseTexture.autoPlay = false;     this.texture = new PIXI.Texture(this.baseTexture);     this.videoSprite = new PIXI.Sprite(this.texture);
Chrome : 40fps
Safari : 58/60fps
Mozilla : 60fps

I'm doing tests with PIXI-Video in mobiles. If I load only one video is good, but if I load two or more the performance low.
/// GOOD ///
let videoTexture = PIXI.Texture.fromVideo('A.mp4');
let vs = new PIXI.Sprite(videoTexture);
root.addChild(vs);
/// LOW PERFORMANCE IN MOBILE ///
let videoTextureA = PIXI.Texture.fromVideo('A.mp4');
let videoTextureB = PIXI.Texture.fromVideo('B.mp4');
let videoTextureC = PIXI.Texture.fromVideo('C.mp4');
let vsA = new PIXI.Sprite(videoTextureA);
let vsB = new PIXI.Sprite(videoTextureB);
let vsC = new PIXI.Sprite(videoTextureC);
root.addChild(vsA);
root.addChild(vsB);
root.addChild(vsC);

There's an idea that we shouldnt use "texSubImage2D" for videos in GLTexture.upload
@rtalon83 more than one video? on mobiles? why do you need it?

AnimatedSprite was a amazing idea for run several animations in mobile ( 10 animations 240x240 ) with fantastic results. The problem is the size of resources ( 10 atlas 2048x2048 2MB ). My next step is Video. I want to test the performance, size, fps. Then, in this moment,  I'm loading four different videos in my mobile but I have low performance.


jpegpy error

hi，I got some issue. Python 3.10.6. When I imported _jpegpy, i got an error
Traceback (most recent call last):
  File "/home/ap/jpegpy/jpegpy.py", line 6, in <module>
    import _jpegpy
ImportError: /home/ap/jpegpy/_jpegpy.so: undefined symbol: `jpeg_mem_src``

For the make，i can compile the generated so file, but there are warnings

make: pkg-config: No such file or directory
make: mdl: No such file or directory
make: pkg-config: No such file or directory
make: mdl: No such file or directory
g++ _jpegpy.cc -o _jpegpy.so -std=c++11 -O3   -UNDEBUG   --shared -fPIC
In file included from pybind11/pytypes.h:12,
from pybind11/cast.h:13,
from pybind11/attr.h:13,
from pybind11/pybind11.h:36,
from _jpegpy.cc:6:
pybind11/common.h:393:33: warning: ‘int PyThread_create_key()’ is deprecated [-Wdeprecated-declarations]
393 |     decltype(PyThread_create_key()) tstate = 0; // Usually an int but a long on Cygwin64 with Python 3.x
|              ~^~
In file included from /usr/include/python3.10/Python.h:122,
from pybind11/common.h:99,
from pybind11/pytypes.h:12,
from pybind11/cast.h:13,
from pybind11/attr.h:13,
from pybind11/pybind11.h:36,
from _jpegpy.cc:6:
/usr/include/python3.10/pythread.h:108:36: note: declared here
108 | Py_DEPRECATED(3.7) PyAPI_FUNC(int) PyThread_create_key(void);
|                                    ^

Hi, @anjunzi .
It seems that you do not have pkg-config.
Try apt-get install pkg-config first. See if it works.
Also, we have updated the Makefile for _jpegpy to reduce some warnings.
You may update it and make again.


Failing on Kotlin DSL script compilation warnings should produce a message with a hint on the root cause

Expected Behavior
The error message should hint that the build failed because the Kotlin compiler produced a warning and the org.gradle.kotlin.dsl.allWarningsAsErrors was set to true.
Current Behavior
The current error message only mentions a compiler warning.

> Configure project :
w: file:///private/tmp/gradle-init-2/foo.gradle.kts:3:1: 'SomeDeprecatedType' is deprecated. BECAUSE

FAILURE: Build failed with an exception.

* Where:
Script '/private/tmp/gradle-init-2/foo.gradle.kts' line: 3

* What went wrong:
Script compilation error:

  Line 3: SomeDeprecatedType()
          ^ 'SomeDeprecatedType' is deprecated. BECAUSE

1 error

* Try:
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

* Exception is:
ScriptCompilationException(scriptCompilationErrors=[ScriptCompilationError(message='SomeDeprecatedType' is deprecated. BECAUSE, location=/Users/donat/.gradle/.tmp/gradle-kotlin-dsl-6035774312227215926.tmp/foo.gradle.kts (3:1))])
	at org.gradle.kotlin.dsl.support.KotlinCompilerKt.compileKotlinScriptModuleTo(KotlinCompiler.kt:205)
...

Context (optional)
As a build user, it can be puzzling to figure out why the build (or the IDE sync) start to fail.
Steps to Reproduce

generate an empty project with the init task
define and invoke a deprecated method in the build script
import the project into IntelliJ and observe the error output

Gradle version
8.2-20230427234156+0000
Build scan URL (optional)
No response
Your Environment (optional)
No response

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team and is prioritized by them.


Wrong line number on web

Environment
How do you use Sentry?
Sentry SaaS (sentry.io)
Which SDK and version?
latest
Steps to Reproduce
use this sample https://github.com/lucas-zimerman/sentry-capacitor/tree/ref/docs-m1/example/ionic-angular-v3
yarn install
ionic build
npx cap sync
http-server ./www
and just open the app and sentry will receive an error.
Expected Result
Correct Line numbers and correct in app
Actual Result
https://sentry.io/organizations/sentry-sdks/issues/3083897173/?project=5522756&query=is%3Aunresolved&statsPeriod=24h

The error happened on line 43 on   Sentry.captureException(new Error("custom error"));
instead it points to expor class AppModule {} (line 56)
InApp issue is reported here: #43
And I need to validate if the same happens on Android/iOS

Wrong line number is due to different codebases using the same release number


Transaction rollbacked back issue.

it will cause the transaction was rolled back more times than it was started for the following script.

execute the two following statements in "parallel".
a. place.update();
b. place.update();
each of a and b will trigger the following statement.
Begin
Let place = 'insert....'
Let live = 'create.....'
Commit
return $place

We fixed this in 2.0.1. Please could you try it?

it doesn't work for 2.0.1. but i put these following two methods in async.parallel() then it work fine.
a. place.update();
b. place.update();
async.parallel([
function(){ place.update();  },
function(){place.update();}
], callback);

Hi @martingg88 ,
We are not experts in Node.js could you provide us detailed steps which we need to do in your code or could you provide us separate test case to reproduce this issue ?

seems like this issue is already fixed.

Cool ! Thank you for feedback.


Check that pod starting not working as expected

Description
Check that pod starting in the deployment process is not working as expected, it checks the status not Pending, but sometimes it is running but still not ready.
https://github.com/threefoldtech/js-sdk/blob/9c0c560b16761920d5657ecae0601cd7a18a202f/jumpscale/packages/vdc_dashboard/sals/solutions_chatflow.py#L417

tested with mattermost worked ok


Document RISC-V following Itanium C++ ABI for C++ mangling rule.

Address #175.

Changes:

Apply Jim's suggestion.

Changes:

Apply Jessica's suggestion.

Changes:

Rebase to master.

@jrtc27 let me know if OK to you :)

Either you never applied my suggestions or you lost them in a rebase

Either you never applied my suggestions or you lost them in a rebase

I guess those changes might lost during rebase...updated and included those suggestion now.


Offset and inset not working

When I call this snippet of code:
myView.snp.updateConstraints { make in
make.centerY.equalToSuperview().inset(offset)
}
or
myView.snp.updateConstraints { make in
make.centerY.equalToSuperview().offset(offset)
}
The inset and the offset methods are not taken into account

This issue does not follow the issue template and has been closed.


Refactor flow aggregation

Flows are identified with the following fields:
protocol
address_a
port_a
address_b
port_b

The first address is whichever one is first when ordered byte-wise. This is good for identifying flows (which can go in either direction), but the statistics aggregation should happen separately.
The last thing is that we should probably ignore "flows" with only one entry. We can't calculate derivatives with only one entry, so it doesn't make sense to rank them either.

This is done.


chore: Cherry picked OPTIC-353 into release 1.11.0

Hi @juliosgarbi!
This PR was created in a response to PRs in upstream repos:

https://github.com/HumanSignal/label-studio-frontend/pull/1679

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

This PR was closed because it has been stalled for 10 days with no activity.