ummon CLI prints gibberish

It seems that this tool is affected by the issue reported at https://www.bleepingcomputer.com/news/security/dev-corrupts-npm-libs-colors-and-faker-breaking-thousands-of-apps/ - with a new install, any invocation of ummon just prints gibberish
https://github.com/Marak/colors.js/issues/285
It seems that the fix is to publish an npm-shrinkwrap.json instead of a package-lock.json:

The recommended use-case for npm-shrinkwrap.json is applications deployed through the publishing process on the registry: for example, ... command-line tools intended as global installs

Hi Michael, been a while! Hope all is well.
I'm not involved with ummon, but I just went through this with another
module. Thank you for the tip about npm shrinkwrap being more important for
CLI tools, I didn't realize that package-lock.json is never published (!)
whereas the shrinkwrap file is.
One good turn deserves another though: someday you're going to want to
update dependencies and you'll have forgotten about this. So you need to lock
down the version of the colors module to a specific release in
package.json too. That way "npm update" won't try to update it based on any
sort of semantic versioning rule.
(P.S. I think this issue may have already been handled upstream by the npm
team taking over the colors module. And if you were having this problem
with "faker", that guy's other module, my comments apply the same to that
one.)
On Mon, Jan 10, 2022 at 12:47 PM Michael Kopinsky @.***>
wrote:

Closed #25 https://github.com/ummon-server/ummon/issues/25 via b321ed1
https://github.com/ummon-server/ummon/commit/b321ed167856d57b64673b2215cbe7b2485e14df
.
—
Reply to this email directly, view it on GitHub
https://github.com/ummon-server/ummon/issues/25#event-5867752463, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAH27JUCCYJ2D26VHPIHXDUVMLS5ANCNFSM5LUICFAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
THOMAS BOUTELL | CHIEF TECHNOLOGY OFFICER
APOSTROPHECMS | apostrophecms.com | he/him/his

Thanks, Tom. Nice to hear from you - I'm surprised to see you're still watching this repo!
In subsequent commits I did pin the specific version of colors. Partly for the sake of future dependency updates, but mostly because yarn global add ummon (which we use in CI) doesn't respect npm-shrinkwrap.json, and it made more sense to lock down the version than to maintain two lock files.


[QUESTION] instruction token create

If you want to create a token, do you have to install all of these packages? I mean if in ERC20 ethereum creating a token is enough to use solidity code, compile, then send it to the RPC URL, while if SEI how? because the documentation is too difficult to understand.

Yes, you need to build your token contract locally before you deploy it, which requires you to install the toolchain.


how can I get current component from ngx-fromly  inner control

@Component({   selector: 'app-my-form',   template:
<form [formGroup]="form" (ngSubmit)="onSubmit()">
<formly-form [form]="form" [fields]="fields" [options]="options">
Submit

, }) export class MyFormComponent {   form = new FormGroup({});   model: any = {};   }
how can I get current component(MyFormComponent) from ngx-fromly inner control

The question is not clear, please elaborate.


reduce max mongo bulk ops to 1000

Prior to Mongo 3.5, the max bulk op size is 1000. In latest versions, it looks like it is configurable by the server. Future work could be to detect version and configured max bulk op size and set appropriately. (mgo should really be doing this...)
Commit which shows bulk op size: https://github.com/mongodb/mongo/commit/50328747a0f66362e95652822bb37ccf1fff67ad

Hello Everyone, I am syncing data between two mongodbs using transporter and my MongoDB version is 3.4.24. Since the max write batch size limit of that version is 1000, I am getting an error while using Transporter with details as:-
"flush error, Write batch sizes must be between 1 and 1000. Got 1398 operations".
Can anyone please help me in bypassing this error.


cleanup code getIssueFullPattern in modules/markup

fix #16415

I think this doesn't need to go in to 1.15 - the "race" is somewhat theoretical as we can assert that Init() gets called and completes before anything else would call the get* fn. There's likely to be another Once we can use to for setting these regexps too but I'll have to have a deeper look,

Codecov Report

Merging #16419 (8ab468b) into main (0393a57) will increase coverage by 1.18%.
The diff coverage is 49.05%.

@@            Coverage Diff             @@
##             main   #16419      +/-   ##
==========================================
+ Coverage   44.24%   45.42%   +1.18%
==========================================
  Files         695      717      +22
  Lines       82341    84108    +1767
==========================================
+ Hits        36429    38206    +1777
+ Misses      40014    39749     -265
- Partials     5898     6153     +255

Impacted Files
Coverage Δ

cmd/convert.go
0.00% <0.00%> (ø)

cmd/dump.go
0.91% <0.00%> (ø)

cmd/dump_repo.go
0.00% <0.00%> (ø)

cmd/generate.go
0.00% <0.00%> (ø)

cmd/hook.go
0.00% <0.00%> (ø)

cmd/migrate.go
0.00% <0.00%> (ø)

cmd/migrate_storage.go
0.00% <0.00%> (ø)

cmd/serv.go
2.36% <0.00%> (-0.22%)
:arrow_down:

cmd/web.go
0.00% <0.00%> (ø)

cmd/web_letsencrypt.go
0.00% <0.00%> (ø)

... and 307 more

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 febee86...8ab468b. Read the comment docs.

.


When will zoom be added on trackpad for macos?

Zoom on trackpad for macos (my 10.13.4) is very sharp, will there be a fix?
src/main/webapp/js/mxgraph/EditorUi.js
Line 2152

I have a similar issue - it is very difficult to precisely control the zoom level. Having support for  ⌘ + = and ⌘ + - shortcuts for zooming would be really useful.

Fixed in 12.4.2.

Fixed in 12.4.2.

Not work :(

Which browser and version have you tried? Did you verify the draw.io version in Help, About?

Which browser and version have you tried? Did you verify the draw.io version in Help, About?

chrome 78
https://drive.google.com/file/d/15jwgjROmugiOOd6j6tCZ_51BMqa60H06/view

I also still have this issue. I don't know what it was like before and perhaps it's gotten better, but it's still too aggressive to be usable for me.
Browser: Firefox 74.0
Diagrams.net version: 12.9.1

Trackpad zoom is still way too aggressive.
Issue occurs using native Mac application (13.0.3) with "pinch to zoom" gesture or with ALT+Scroll.

I'm having this issue as well. zoom is extremely aggressive, both for pinch & ALT+Scroll.
Mac App 13.3.9 and Browser app 14.4.2 (Chromium: 83.0.4103.116).
Look forward to a fix. Let me know if I can help out in any way.

I'm trying to fix this problem.
I think changing the zoomFactor property is effective as shown below.
https://github.com/tanabe/drawio/commit/84f9c5ecb2302291381319352b21004f7dcb53cb#diff-0b587672dc918d3ce1dfb8c608cb4eb9R2548
Here is the live sample.
https://tanabe.github.io/drawio/src/main/webapp/index.html

I think the zoom could be a tad more aggressive, but I'd be happy with this
as it is - it's still very much more usable.
On Wed, Sep 30, 2020 at 12:03 PM Hideaki Tanabe<EMAIL_ADDRESS>wrote:

I'm trying to fix this problem.
I think changing the zoomFactor property is effective as shown below.
tanabe@84f9c5e#diff-0b587672dc918d3ce1dfb8c608cb4eb9R2548
https://github.com/tanabe/drawio/commit/84f9c5ecb2302291381319352b21004f7dcb53cb#diff-0b587672dc918d3ce1dfb8c608cb4eb9R2548
Here is the live sample.
https://tanabe.github.io/drawio/src/main/webapp/index.html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jgraph/drawio/issues/638#issuecomment-701519993, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF4VUQEDKKOGG7QLXH4KR3SINQFPANCNFSM4JISP34Q
.

--
^[:wq^M

The problem is that this changes wheel mouse zooming speed as well. We'd prefer it if only the trackpad was affected.

I'm trying to fix this problem.
I think changing the zoomFactor property is effective as shown below.
tanabe@84f9c5e#diff-0b587672dc918d3ce1dfb8c608cb4eb9R2548
Here is the live sample.
https://tanabe.github.io/drawio/src/main/webapp/index.html

I don't see any difference between production or the example on trackpad or wheel mouse.

Was testing this right now as well. We have to consider the browser used...
Safari 14.0: Both, production and @tanabe sample have "steps" while zooming, this decreases UX massively. Responsiveness became even worse in @tanabe live sample. It takes more "pinch-distance" until a zoom step is done.
Chrome 85.0.xx: Here we have a Mac OS fashioned smooth zoom in production and live sample. While production is way to fast, @tanabe sample has perfect speed in my opinion.
Firefox 81.0.1: Has steps and is fast as f***.. unusable in production and live sample.
draw.io Mac application: Has steps and is too fast as well in production. Don't know how to test the live sample with the application.
Hope this helps!

That's useful, thanks for testing them all. Targeting Chrome makes sense given the user base and that the desktop build uses Chrome. Firefox, would be nice, but is dying. Safari would be good if we didn't make it any worse, given this is a Mac specific function.

Sounds reasonable. I was already suspecting the desktop build to be based upon Chrome, glad to hear there is a chance to have a proper Zoom there.
Regarding Safari: I don't think the mentioned deterioration is relevant. Pinch-to-Zoom is already unusable in Safari and the proposed enhancement of @tanabe doesn't change this. I think Safari users will just continue to use the Zoom buttons... However, his solution is providing a solution at all for Pinch-to-Zoom on Mac OS (either in Chrome or desktop build).

That's useful, thanks for testing them all. Targeting Chrome makes sense given the user base and that the desktop build uses Chrome. Firefox, would be nice, but is dying. Safari would be good if we didn't make it any worse, given this is a Mac specific function.

Part of the reason why Firefox is "dying" is because web developers are starting to target Chrome only, with Chrome slowly but surely becoming the next IE.
I implore organizations like Draw.io to support the latest versions of ALL the major browsers, including Firefox, so that we can avoid that path. I really don't want web standards to be exclusively in Google's hands.

I only work in FF (someone on the team has to), it's FF that's the problem at a high level. It's quality has been on the decline for years, well over 50% of our browser specific fixes in draw are for FF alone, more than the other supported browser put together.
Currently, I have to restart FF several times a week as text areas get laggy. I wish exactly as you do, but it's just typical that's if FF that's the worst behaving here.

I updated sample.
https://tanabe.github.io/drawio/src/main/webapp/index2.html
I checked the behavior of the trackpad in Firefox 81.0 and Safari 14.0 and Chrome 85.0.4183.121.
Let me know what you think about this.

Thanks - tested w/ Chrome 85.0.4183.121 and it became worse. Zooming falls asleep after a few cm of pinching and one has to start pinching again on the trackpad. Apart from that there are "steps" again.
I did not notice any change while using FF.

Thanks for your efforts. The key problem for me here is that the new example makes zooming with the mouse wheel somewhat worse. Zooming in isn't usable anymore.
Is there a way to limit the new behaviour only to the trackpad and not the mouse wheel?
I also find the zooming in too fast in the new example in both FF and Chrome, anyone else getting that?

I'm trying to fix this problem.
I think changing the zoomFactor property is effective as shown below.
tanabe@84f9c5e#diff-0b587672dc918d3ce1dfb8c608cb4eb9R2548
Here is the live sample.
https://tanabe.github.io/drawio/src/main/webapp/index.html

The demo on the above link is working for me. I have been facing this issue for a long time and decided to look for a solution.
I am using Macbook Pro with its built-in trackpad. In fact above demo handles zoom-in so nicely I started copying my diagram to it and working there.

We'll add an option to configure this in the next release as follows:
{
  "zoomFactor": 1.05
}

Note that this affects mouse wheel and trackpad zoom. Unfortunately, there is no way to configure them separately.

Sorry, I am a rookie compared to all the contributors on this thread but tho I wonder, can we assign separate variables for the 2 different zoom-factors, one for trackpad and another for mouse-wheel. I expect for many users may have problems when considering accidental touches on their trackpad using mouse but I am giving an idea.
I also wonder, will this increase the job for developer when adding features. I am pro at using trackpad and trackball but I hate mouse now because of the different machines having different accuracy for tracking. eg. my mx master mouse stutters so much on my windows and mac due to faulty updates from logitech. The brand is good but the quality in their software resolution needs work. :( if this comment bothers you TT

We'll add an option to configure this in the next release as follows:
{
  "zoomFactor": 1.05
}

Note that this affects mouse wheel and trackpad zoom. Unfortunately, there is no way to configure them separately.

Thanks for this option.
I don't use a mouse. Trackpad scroll is not affected but the zoom is smoother now.
Overall, It's working as expected. 🙏

@rahul286 how to install this setting in the project

@dharnil please go to Extra ➞ Configuration (screenshot below)

Paste the config in the text area and click "Apply"

Please note that you will need to reload the app in your web browser to change the effect.
I used the value 1.08, the answer from @alderg had a value of 1.05, you can use anything else as per your comfort.


Checksum Fails

I've had the checksum fail on some valid GST numbers. Please check your algo with well known GST numbers

on js or py

It will fail for 32AADCK5978M1Z0

@EFFICIENTCORPORATES Please share the details or link to understand  the GSTIN

The Break Up of GSTIN is very well explained by you in your README file... But one thing which we need to understand is that there are different types of GSTIN. A single entity can have its TDS GSTIN, Normal GSTIN and an ISD GSTIN for complying with different rules of GST.
Similarly, the GST Number allocated to Govt Department or a United Nations body is different.  Also, a Non-Resident Taxable Person (NRTP) has a different GSTIN. Companies like Amazon Prime, Hotstar, Netflix come under the category of OIDAR and have different GSTIN..
Regex for each one of these GSTIN will be different... For further details you can visit the GST developers site:
https://developer.gst.gov.in/apiportal/taxpayer/returns

The checksum validation is failing because there is an edge condition that needs to handled i.e. when the final checksum code is 36 we need to attribute it to 0.
def checksum(gst):
    check = gst[-1]
    gst = gst[:-1]
    l = [int(c) if c.isdigit() else ord(c)-55 for c in gst]
    l = [val*(ind % 2 + 1) for (ind, val) in list(enumerate(l))]
    l = [(int(x/36) + x%36) for x in l]
    csum = (36 - sum(l)%36)
    if(csum == 36):
        csum = "0"
    else:
        csum = str(csum) if (csum < 10) else chr(csum + 55)
    return True if (check == csum) else False


Add the concept of a "readiness" similar to k8s, etc.

@mistermoe and I chatted about this
Could be something like this:
//ftl:export
func Readiness(ctx context.Context) error {
	_, err := bearerDID.Get(ctx)
	return err
}

Leaving some quick questions here for whoever ends up picking this up:

Should we require the function to be named Readiness or should we rely on an //ftl:readiness directive instead? If so, do we want Readiness to be a reserved name? i.e. no other functions can be named that.
Do we want to perform the readiness check (i.e. call each exported Readiness func) in Service.Status() in controller.go?
Should readiness effectively be a specialty verb? i.e. if a function is exported with the readiness directive, and only that directive, then it also automatically gets treated as a verb in all the ways a verb would be; it's just also invoked for the readiness checks at startup? It's a bit odd because by the time the module is live, any Readiness function would presumably be returning nil persistently, so there wouldn't really be a point to calling it.

Related: is there any reason to make these exportable? For module-to-module calls, it seems like exporting is just not useful. For calling it from the controller, we could argue that's a public/external access, in which case the visibility should technically be public, but that's basically implied already by the nature of readiness.
tl;dr: I'm leaning towards: Readiness is not a verb. It needs directive //ftl:readiness precisely. No explicit export in the directive. No module-to-module calls. The controller will call this function at startup.

I agree re. not being exported, I think //ftl:export in the example predated visibility controls.
I think it should be a verb for consistency though, as that is our unit of work for everything in FTL. I'm fine with either a well known name or //ftl:readiness, though the latter is more explicit.

Sounds good! I'll pick this up after I finish my other issues if no one else beats me to it (but everyone definitely please feel free to take this!)


URL parsing error

Now that the bot is online, and I made my own server with the bot to use it again after not using it in months, whenever I try downloading a video, whether i use a search term or a video link, I just get a message reading "URL parsing error"

Noticed the bot is offline again, hopefully its just maintenance so that this issue gets fixed!


chore(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2503

Questions ?
Please refer to the Backport tool documentation


Fix: use mongodb transactions

Introduction
We weren't using transactions when saving data so if one call saved data could be in a bad state
Resolution
Use mongodb transactions and add some misc. fixes/logging

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

:exclamation: No coverage uploaded for pull request base (main-stbr@159718e). Click here to learn what that means.

Additional details and impacted files
@@             Coverage Diff              @@
##             main-stbr    #2331   +/-   ##
============================================
  Coverage             ?   23.74%
============================================
  Files                ?     1089
  Lines                ?    26289
  Branches             ?     4191
============================================
  Hits                 ?     6243
  Misses               ?    17491
  Partials             ?     2555

Flag
Coverage Δ

ui-test
64.15% <0.00%> (?)

unit-test
4.72% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


The content of a system backup for D365 CE

[Having in mind that we cannot download (at least not directly) the backup to check the actual files, we need to point the partners/customers to this statement from the article:
"System backups occur continuously; this is different from previous versions where backups were once a day. Because the underlying technology used is Azure SQL Database, see [Automated backups] (https://docs.microsoft.com/en-us/azure/sql-database/sql-database-automated-backups) for details."]
Actually, one of my partners wants to know what type of data is backed up during this process, mentioned in this article:  https://docs.microsoft.com/en-us/power-platform/admin/backup-restore-environments
Thank you,
AncaR

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

ID: d9c58fb1-e6a2-5210-3774-4ea4bbff4572
Version Independent ID: 57c1780c-b097-af8c-82fa-47e38a63995c
Content: Back up and restore environments - Power Platform
Content Source: power-platform/admin/backup-restore-environments.md
Service: power-platform
Sub-service: admin
GitHub Login: @ChrisGarty
Microsoft Alias: cgarty

Thanks for your comment, @AncaRadu. This issue has been assigned to the doc author for review.

@AncaRadu
Sorry for the delayed response. Taking to internal thread.


Typing issue with filter queries and generics

Describe the bug
Cannot explain it better than the reproduction: countWaiting used to work in v5, the other two also did not

Reproduction
https://github.com/mikro-orm/reproduction/pull/3
What driver are you using?
None
MikroORM version
6.0.0-rc.2
Node.js version
20
Operating system
osx
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Slack.
[X] The provided reproduction is a minimal reproducible example of the bug.

I will need to check, but if this worked in v5, it might have been because the method was not type-safe before, as what you are trying to do is a TS limitation IMHO.
https://stackoverflow.com/questions/65490587/generics-doesnt-seem-to-fit-well-with-queryfilter/65523491#65523491

Oh I see, it does not work even with the type assertion.


Add license to QCommandPalette

Hey there!
Thank you for the project :)
It seems like currently, QCommandPalette does not specify any LICENSE, in comments or in a LICENSE file. This prevents the community from using the project.
Please add the license which you feel that best describes the way you want the community to use and modify this project.
I suggest going to the website GitHub created for this subject -> https://choosealicense.com
We would be happy to use this project and modify it to our needs. Thank you!

Thanks for the interest. I licensed it with MIT license. Have fun with it!

I made this more as a personal project. I didn't know how to do the popup close correctly with Qt. Please let me know if you a better way how to do it!

I made this more as a personal project

That's great! We need more people to do "personal projects" in Cutter codebase :) If you have some spare time to do some Qt open-source in a highly popular project, we will be happy to have you :)


copy and past of blocks BETWEEN different programs

I use a function to explain what I want. I have written a function and want to use it in another program.
Many solutions are possible, but at least one should be available:

copy in one program, open another program, paste and get the function.
save the function in the lab using name "myFunction" and have a new menu entry "import fragment"
drag and drop between different browser instances.
I prefer the first solution.

seems to be only related to #104

item 1 (copy in one program, open another program, paste and get the function) is already available!
+1 for item 2 as an addition

Sorry for asking, but is it possible that item 1 is no longer possible? I knew that I once tried it successfully but either I do not remember how or it does not work any longer.
If it still works, it would be nice to hav some instructions in openrobertalab.
Cheers corvus-albus


Would this work for me?

I have a form with 10 input fields, 3 of those being file upload fields. Would this work for that or one file upload form only?
Thanks

It do works for your case.

ah right. I am using php, right now u click the submit button, it then uploads all files and posts the text data. I am not using AJAX so just posts it normally. How would you recommend i do initial setup of your js script to just test it out? Thanks! You have developed some amaxing tools!

This is my php code (ignore all bad habits, just me experimenting with some new techinques)
	public function testControllerMethod()
	{
        if($_SERVER['REQUEST_METHOD'] == 'POST')
        {
            $this->csrf->CSRFTokenVerify();

            $files = [
                'icon_img' => $_FILES['icon_img']['name'],
                'iname' => generateRandomString(25),
                'preview_img' => $_FILES['preview_img']['name'],
                'pname' => generateRandomString(25),
                'item_file' => $_FILES['item_file']['name'],
                'fname' => generateRandomString(35),
            ];
            $formData = [
                'name' => trim($_POST['name']),
				'sdescription' => trim($_POST['sdescription']),
				'content' => trim($_POST['content']),
				'features' => trim($_POST['features']),
				'price' => trim($_POST['price']),
				'tags' => trim($_POST['tags']),
				'cat_id' => trim($_POST['cat_id']),
				'demo' => trim($_POST['demo']),
				'video' => trim($_POST['video']),
                'icon_img' => (empty($files['icon_img']) === TRUE ? '' : $files['iname'].'.'.pathinfo($files['icon_img'], PATHINFO_EXTENSION)),
                'preview_img' => (empty($files['preview_img']) === TRUE ? '' : $files['pname'].'.'.pathinfo($files['preview_img'], PATHINFO_EXTENSION)),
                'item_file' => (empty($files['item_file']) === TRUE ? '' : $files['fname'].'.'.pathinfo($files['item_file'], PATHINFO_EXTENSION)),
                'user_pin' => $_SESSION['upin'],
                'name_err' => '',
				'sdescription_err' => '',
				'content_err' => '',
				'features_err' => '',
                'support_err' => '',
				'price_err' => '',
				'cat_id_err' => '',
				'tags_err' => '',
				'right_to_sell_err' => '',
                'icon_img_err' => '',
                'preview_img_err' => '',
                'item_file_err' => '',
            ];
            $data = [
                'formData' => $formData,
                'title' => 'Upload a product test',
                'description' => '',
            ];

			if(empty($data['formData']['name'])) {
				$data['formData']['name_err'] = 'Item name is required!';
			} elseif(strlen($data['formData']['name']) > 50) {
                $data['formData']['name_err'] = 'Item name cannot be more than 50 characters!';
            }
			if(empty($data['formData']['sdescription'])) {
				$data['formData']['sdescription_err'] = 'Short description is required!';
			} elseif(strlen($data['formData']['name']) > 130) {
                $data['formData']['sdescription_err'] = 'Short description cannot be more than 50 characters!';
            }
			if(empty($data['formData']['content'])) {
				$data['formData']['content_err'] = 'Content box is required!';
			}
			if(empty($data['formData']['features'])) {
				$data['formData']['features_err'] = 'Features box is required!';
			}
			if(empty($data['formData']['price'])) {
				$data['formData']['price_err'] = 'Item price is required!';
			} elseif($data['formData']['price'] < $getSettings[0]['payment_minprice']) {
                $data['formData']['price_err'] = 'Item price needs to be more than ' . $getSettings[0]['payment_currency_sym'].$getSettings[0]['payment_minprice'];
            }
            if(empty($data['formData']['tags'])) {
				$data['formData']['tags_err'] = 'Tags are required!';
			}
			if($data['formData']['right_to_sell'] == 0) {
				$data['formData']['right_to_sell_err'] = 'You have not checked whether you have the right to sell this item!';
			}

            $categoryId = array_column($marketplace['getCategories'], 'id');
            if(empty($data['formData']['cat_id'])) {
                $data['formData']['cat_id_err'] = 'You need to select a category!';
            } elseif (!in_array($data['formData']['cat_id'], $categoryId)) {
                $data['formData']['cat_id_err'] = 'We do not tolerate malicious behaviour on our site! You have been logged!';
            }

			if(empty($_FILES['icon_img']['name']) === TRUE){
				$data['formData']['icon_img_err'] = 'A thumbnail image is required!';
			}
			if(empty($_FILES['preview_img']['name']) === TRUE){
				$data['formData']['preview_img_err'] = 'A preview image is required!';
			}
			if(empty($_FILES['item_file']['name']) === TRUE){
				$data['formData']['item_file_err'] = 'A main file is required!';
			}

            if(empty($data['formData']['name_err']) && empty($data['formData']['sdescription_err']) && empty($data['formData']['content_err']) && empty($data['formData']['features_err']) && empty($data['formData']['price_err']) && empty($data['formData']['tags_err']) && empty($data['formData']['stock_amount_err']) && empty($data['formData']['right_to_sell_err']) && empty($data['formData']['category_err']) && empty($data['formData']['support_err']) && empty($data['formData']['icon_img_err']) && empty($data['formData']['preview_img_err']) && empty($data['formData']['item_file_err'])) {

                    $upload_errors = [];
                    $uploaded_files = [];

                    foreach(array("icon_img", "preview_img", "item_file") as $file_id) {
                        if(isset($_FILES[$file_id]) && isset($data['formData'][$file_id]) && $data['formData'][$file_id]) {
                            $upload = new UploadMe($_FILES[$file_id]);

                            switch ($file_id) {
                                case "icon_img":
                                    $upload_id = "Thumbnail Image";
                                    $allowed_extensions = array('png', 'jpg', 'jpeg', 'gif');
                                    $allowed_types = array('image/jpeg','image/png','image/gif');
                                    $max_size = 2;
                                    $max_dimensions = array('200', '200');
                                    $upload_dir = "thumbnail";
                                    $file_name = $files['iname'];
                                    break;
                                case "preview_img":
                                    $upload_id = "Preview Image";
                                    $allowed_extensions = array('png', 'jpg', 'jpeg', 'gif');
                                    $allowed_types = array('image/jpeg','image/png','image/gif');
                                    $max_size = 3;
                                    $max_dimensions = array('1600', '800');
                                    $upload_dir = "preview";
                                    $file_name = $files['pname'];
                                    break;
                                default:
                                    $upload_id = "File";
                                    $allowed_extensions = array('zip', 'rar');
                                    $allowed_types = array('application/zip', 'application/octet-stream', 'application/x-zip-compressed', 'multipart/x-zip', 'application/x-rar-compressed');
                                    $max_size = 96;
                                    $max_dimensions = false;
                                    $upload_dir = "files/" . SECURE_FOLDER;
                                    $file_name = $files['fname'];
                            }

                            if ($allowed_extensions)
                                $upload->allowed_extensions($allowed_extensions);

                            if ($allowed_types)
                                $upload->allowed_types($allowed_types);

                            if ($max_size)
                                $upload->max_size($max_size);

                            if ($max_dimensions)
                                $upload->max_image_dimensions($max_dimensions); // in pixels

                            if ($upload_dir)
                                $upload->path("uploads/marketplace/items/" . $upload_dir);

                            $upload->name($file_name);
                            if ($upload->upload()) {
                                $uploaded_files[] = $upload->get_path($upload->get_error());
                            } else {
                                $upload_errors[] = $upload_id . " Upload: " . $upload->get_error();
                            }
                        }
                    }

                    if (count($upload_errors) > 0) {
                        foreach ($upload_errors as $error) {
                            echo $error . "
";
                        }
                        foreach ($uploaded_files as $file) {
                            @unlink($file);
                        }
                } else {
                    if($this->m->addItem($formData)) {
                        $notification = [
                            'notification_userpin' => $_SESSION['upin'],
                            'notification_title' => '' . $data['formData']['name'] . ' was successfully submitted & pending review!',
                            'notification_type' => '2',
                            'notification_location' => FULL_ROOT . '/dashboard',
                        ];
                        $this->start->insertNotification($notification);
                        flash('message', 'Your test prod has been submitted and is pending review!', 'alert alert-success mb-0');
                        redirect('dashboard');
                    } else{
                        die(LANG['something-went-wrong']);
                    }
                }
            } else {
                $this->view('test/upload', $data);
            }
        } else {
            $formData = [
                'name' => '',
				'sdescription' => '',
				'content' => '',
				'features' => '',
				'price' => '',
				'tags' => '',
				'cat_id' => '',
				'demo' => '',
				'video' => '',
				'support' => '',
				'right_to_sell' => '0',
				'review_message' => '',
                'icon_img' => '',
                'preview_img' => '',
                'item_file' => '',
                'name_err' => '',
				'sdescription_err' => '',
				'content_err' => '',
				'features_err' => '',
				'price_err' => '',
				'cat_id_err' => '',
				'tags_err' => '',
				'support_err' => '',
				'right_to_sell_err' => '',
                'icon_img_err' => '',
                'preview_img_err' => '',
                'item_file_err' => '',
            ];
            $data = [
                'getSettings' => $getSettings,
                'getNav' => $getNav,
                'getConstantUser' => $getConstantUser,
                'marketplace' => $marketplace,
                'formData' => $formData,
				'title' => 'Upload an prod',
				'description' => ''
            ];
            $this->view('test/upload', $data);

        }
    }

I'm afraid that you could not use my tool now. For you have multiple file uploaders and my plugin only supports one uploader per form currently. Please leave this issue open and give me some time to upgrade this plugin.
Your requirement will be satisfied in the next upgrade and I will give you a solution. Thanks for your advice.

Oh wow! Thankyou very much! You are an excellent developer! 👍 🥇

@jackttcoms

Hello, sorry for slow reply. That would be a much better alternative to my method! Just to make sure I have understood, it basically uploads as soon as you select the file instead of pressing the submit button and then once it is uploaded then they click the submit button it just uploads the name for database etc. Thanks again for your amazing help!

Ok. I will give you a demo later.

@jackttcoms I have created an example page for you.
In this page, I achieved your requirements.
Please try it here.

This issue will be closed after seven days of no response.

Wow thankyou thankyou! May I ask tho where I can find the php code? Thankyou very much Anduin.

There was no PHP code because I really can't write PHP. But reading the code above wasn't difficult I guess. You can try to imitate my design and achieve that in PHP.
The server-side code was also very simple. I have just created two server-side handlers and the source code is here:

https://github.com/AiursoftWeb/Nexus/blob/master/Developer/Controllers/SamplesController.cs#L38-L68

Wow thankyou again very much! You are one of the best developers I have come across. Ever. I am currently learning C sharp myself is that the language u usually code in?

I usually write the following languages: (Ordered by the skill levels)

C#
TypeScript
JavaScript
Bash
C++

BTW, can you successfully open the example form and submit the form to see the uploader progress here?
https://developer.aiursoft.com/samples/formsample

Hi sorry for delayed reply. Would you be able to add jpg and then I will be able to test but from what I see so far it works perfectly

Sorry jpeg. I am uploading from an iPhone

I will go on my computer now dw

only files with .png, .jpg and .bmp is allowed.

Yep it uploads perfectly but could we add a javascript check where onComplete it closes the modal or says DONE because currently it just goes back to 0%. (only minor not important)
Thanks

If you do need to protect your uploader, the best way is to add a recaptcha and only passed user have the right to ask the server to save the file.
Hidding the URL is impossible because the user can always get it from your network tab in the F12 developer tool.

Oh ok, thanks very much. Let me know when I can see the PHP. Thanks again Anduin.

I have created another example form for you. In this form, I achieved more features like disable the upload button and show notifications.
Try it here: https://developer.aiursoft.com/Samples/FormSample


Latest version not available on F-Droid

It looks like F-Droid is still picking up v3.0.14 which doesn't seem to be the latest version (v3.1.0).
Link: https://f-droid.org/repository/browse/?fdfilter=duckduckgo&fdid=com.duckduckgo.mobile.android

Closing because moved to the Search & Stories repo: https://github.com/duckduckgo/android-search-and-stories/issues/307


N27 - godot-rust

Part of #818

@17cupsofcoffee Thank you for the review. Apologies for the missing links, I didn't see those when I was initially previewing the file. It is is fixed in my latest commit.


Add eslint

What: add lint rules to our repo and set up yarn lint command
Why: automate linting
Implementation details: copy the eslint rules from the main repo and add them to our repo

Merged


Push notification settings

All push notifications in Rocketchat with ``rocketchat-push-gateway''
What is the way to make it work with ?
I can build the Apprise GUI with Docker and display the UI
And you can see Mongo's Raix Push App Tokens in the metabase
Looks like you're using GCM now
I'm looking for a way to make push notifications work in a completely ”closed network”

There is absolutely no GCM involved in the push gateway, it just uses the GCM token from the Android app to identify the push targets. The push service is provided by Gotify which uses websockets as communication channel to display the push notifications. If the network diagram in the readme is completely isolated on a private LAN, it will still work. I think the answers you are looking for are in understanding Gotify more: https://gotify.net/
                                   +------------+
                                   |            |
                      +----------->+ RocketChat |
                      |            |            |
                      |            +-----+------+
                      |                  |
         +------------+--+               v
   SSL   |               |        +------+-------+
+------->+ Reverse proxy |        |              |
         |               |        | Push gateway |
         +------------+--+        |              |
                      |           +-----------+--+
                      |                       |
                      |                       v
                      |                 +-----+----+
                      |                 |          |
                      +---------------->+  Gotify  |
                                        |          |
                                        +----------+


Nested samplers notebook

New Jupyter notebook detailing each of the Nested Sampling (NS) classes in Gleipnir. Also, a few additions to docstrings as well the addition of file root properties to the wrapper NS classes.

Upgrade version from 0.16.0 to 0.17.0


[DO NOT MERGE] SignUrl for Cloud CDN

Sign URL sample for Cloud CDN using .NET Core

Re: When you pass a string to a function, and modify its value, its value
in the outer scope remains the same.
By default, parameters are passed by value in C# - unless you use "out" or
"ref" parameters. string is a reference type but it's immutable type, so
every change you make to original string will create a new string with
changed value, point existing reference to that string and let GC discard
the old value. So when passing a string as input to a method, you pass a
copy of a reference to that string (or you can say that the reference is
being passed by value). In other words, when passing a string to a method,
you pass a copy of a reference to that string so when you later on do
re-assignment to that variable, you in fact re-reference to a new place in
memory holding a new value but you are not changing original string. This
what explains your experiment. Even though technically, I can use the same
variable passed to the method to make necessary modifications, it's
considered a bad practice to do so - unless your method is explicitly
designed to mutate input parameters. This is not the case in my method, so
I prefer to stick with best practices here.
On Wed, Feb 14, 2018 at 2:30 PM Ahmet Alp Balkan<EMAIL_ADDRESS>wrote:

@ahmetb commented on this pull request.
In signUrl/SignUrlSample/Program.cs
https://github.com/GoogleCloudPlatform/dotnet-docs-samples/pull/448#discussion_r168331313
:

       byte[] hash = ComputeHash(decodedKey, builder.Uri.AbsoluteUri);

       string encodedHash = Base64UrlEncode(hash);

       builder.Query += $"&Signature={encodedHash}";

       return builder.Uri.AbsoluteUri;

   }

   private static long ToUnixTime(DateTime date)

   {

       var epoch = new DateTime(1970, 1, 1, 0, 0, 0, DateTimeKind.Utc);

       return Convert.ToInt64((date - epoch).TotalSeconds);

   }

   private static byte[] Base64UrlDecode(string arg)

   {

       string s = arg;

Maybe I'm recalling this wrong about C#, but here's an experiment:
https://dotnetfiddle.net/o9EV5F
When you pass a string to a function, and modify its value, its value in
the outer scope remains the same.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/dotnet-docs-samples/pull/448#discussion_r168331313,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfIRAnWYRzVoVDo_Snf7xIxmB90fQY-Vks5tU16OgaJpZM4SEnxk
.

Understood. My problem with the current code is that some of the declarations seem unnecessary as we're often just processing and returning a string, the whole method can be a single return stmt, or even can be inlined in SignUrl() method.
I'll leave it to you and the language experts on this repo, to decide on code cleanliness. I think our Style/Readability Guide probably offers a solution for these sort of "preference" matters. I recommend consulting to the Style Guide.

LGTM

This is now OK to merge. If there are no outstanding comments, can someone merge it?

Yeah, will do
On Wed, Feb 28, 2018 at 1:09 PM Ahmet Alp Balkan<EMAIL_ADDRESS>wrote:

This is now OK to merge. If there are no outstanding comments, can someone
merge it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/dotnet-docs-samples/pull/448#issuecomment-369383603,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfIRAv9wbksMQTYUiakiYUcfzFJQU03Oks5tZcCQgaJpZM4SEnxk
.


Add tests for font case sensivity of hex keys

Signed-off-by: Igor Egorov<EMAIL_ADDRESS>This pr replaces #25
Description of the Change
Some time ago the format of the way keys are passed to commands was changed from bytes arrays to strings.
This change introduces a set of tests for font case sensitivity for keys fields.
Benefits
Increased test coverage.
Possible Drawbacks
Potentially the test can be written without ITF usage.
Usage Examples or Tests
hex_keys_tests

CI was not able notify github about build status.
All the tests have been passed.


New User: Cloud6

Please fill the following out:

Name: Cloud6
Homepage url: http://cloud6.io/
Brand Guidelines/Licensing: N/A
Logo: http://cl.ly/image/3W1w2W1x3x1C

Nice, wanna open a PR for it? Here's some examples


Creating tables/feature classes with protected keywords

Me again! Found one last issue. If someone creates a feature class or table that uses one of the SQLite keywords (see https://www.sqlite.org/lang_keywords.html for the full list), the following error is thrown (in this case using or as the feature class name):
sqlite3.OperationalError: near "or": syntax error

While it's probably not a good idea to use those keywords, users will find a way to break that! In fact, it is possible to create tables with those keywords - you just need to quote the table name in the CREATE TABLE statements. For example, you can change the CREATE_FEATURE_TABLE SQL in fudgeo (https://github.com/realiii/fudgeo/blob/def68ecbbe25c63e9ddca72b15b3719006a81bf8/fudgeo/sql.py#L61) to this:
CREATE_FEATURE_TABLE = """
    CREATE TABLE '{name}' (
        fid INTEGER not null primary key autoincrement,
        SHAPE {feature_type}{other_fields})
"""

(just quoted the table name)
Either that or find a way of checking for those keywords beforehand, but I can't seem to find a way to do that from the sqlite3 library. There are routines that can do that in sqlite (see https://www.sqlite.org/c3ref/keyword_check.html), but I have no clue how to access them from Python.

Marking as won't implement, you can protect against this in your code to avoid creation of such tables or column names


newsletter-259: translate into Chinese

Newsletter-259: translate into Chinese

Newsletter-259: translate into Chinese


Cache json deserialization for performance of plugins

Cache json deserialization for performance of plugins

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #1317 (371f2fb) into master (902445b) will decrease coverage by 1.09%.
The diff coverage is 44.92%.

@@             Coverage Diff              @@
##             master    #1317      +/-   ##
============================================
- Coverage     69.52%   68.42%   -1.10%
- Complexity     2057     2079      +22
============================================
  Files           380      391      +11
  Lines          8153     8273     +120
  Branches        827      827
============================================
- Hits           5668     5661       -7
- Misses         2070     2195     +125
- Partials        415      417       +2

Impacted Files
Coverage Δ
Complexity Δ

...h/handler/ContextPathMappingPluginDataHandler.java
8.33% <0.00%> (ø)
1.00 <0.00> (ø)

...ugin/resilience4j/handler/Resilience4JHandler.java
7.14% <0.00%> (-7.15%)
1.00 <0.00> (ø)

...ingcloud/handler/SpringCloudPluginDataHandler.java
4.76% <4.76%> (ø)
1.00 <1.00> (?)

...in/redirect/handler/RedirectPluginDataHandler.java
8.33% <8.33%> (ø)
1.00 <1.00> (?)

...ugin/rewrite/handler/RewritePluginDataHandler.java
8.33% <8.33%> (ø)
1.00 <1.00> (?)

...elimiter/handler/RateLimiterPluginDataHandler.java
67.64% <10.00%> (-9.94%)
15.00 <1.00> (+1.00)
:arrow_down:

...ugin/hystrix/handler/HystrixPluginDataHandler.java
46.15% <30.00%> (-53.85%)
4.00 <1.00> (+1.00)
:arrow_down:

.../contextpath/cache/ContextPathRuleHandleCache.java
75.00% <66.66%> (ø)
2.00 <1.00> (?)

...l/plugin/hystrix/cache/HystrixRuleHandleCache.java
75.00% <75.00%> (ø)
2.00 <2.00> (?)

.../ratelimiter/cache/RatelimiterRuleHandleCache.java
75.00% <75.00%> (ø)
2.00 <2.00> (?)

... and 32 more

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 902445b...371f2fb. Read the comment docs.


Added PR and Issue Template

@adityaslyf It does not contains a PR template please add it.

@Mr-Sunglasses Thank you for getting back to me. I apologize for not including the PR template in my initial pull request. I appreciate your guidance, and I will add the required PR template to my pull request as soon as possible.


Model scope call after relation returns Builder

Currently code like this $this->hasOne(User::class)->active(); results in error saying Call to an undefined method Illuminate\Database\Eloquent\Relations\HasOne::active(). This PR fixes this.
Relation class in Laravel forwards unknown calls to Builder. And Builder can call scope methods on the model. So it is safe to call model scopes from the relation, like above.
There was one issue that I couldn't solve. As far as my understanding goes for the Reflection and PHPStan, there is no way to know the calling context from the reflection.
So in this implementation, I'm treating every unknown call as a scope method call, if the method doesn't also exist in Builder
I'm open to suggestions if anyone knows how to get the actual scope method reflection from inside the ModelScopeAfterRelations pipe.

Thanks!

@canvural Let me know if you would like to maintain this project.

@nunomaduro I don't have too much experience with writing PHPStan extensions. I don't know if I'd be the right person.
I'm just fixing the errors that I found when running Larastan on my project. And I have some free time over the next 2 or 3 weeks. I'll continue to fix the issues I found. As for the maintaining let's see after this time.


1.1.6 wmagent patch (#8311, #8265, #8344)

#8311 bump worker thread heartbeat to 2h
#8265 fix acdc parentage
#8344 Skip MCFake file parentage

Do not apply to production agent until new agent test is done.

@amaltaro, Alan I applied this to production agents and update the twiki and trello. I will monitor the agents start to fixing the parentage tomorrow.

Thanks!


The Seastone Chair is not unique

The Seastone Chair currently is not unique (but it is).

Street of the Sisters too.


Issue with Chrome - Not working properly

Hello,
It seems the livedemo is not working properly with Chrome.
Google Chrome is up to date
Version 91.0.4472.124 (Official Build) (64-bit)

Another user reported it a few month ago here:
https://datatables.net/forums/discussion/65799/datatable-column-resize-plugin

ColReorderWithResize.js file seems to be having issues after chrome 91 update

Console is saying "resizing" then "I have been resized!" but nothing change on the DataTable.
Thanks!

Hmm sorry about the late reply. The plugin should still work as of now. Could you try the updated live demo URL?
https://dhobi.github.io/datatables.colResize/


Parse error in .joker file

I added a .joker file to avoid a warning about unused namespace:
{:ignored-unused-namespaces [my.namespace]}

The warning disappeared as expected but I get a Parse error in the .joker file itself: Unable to resolve symbol: my.namespace.
Is there any way to avoid it? Like disable linting of the .joker file? https://github.com/rgdelato/linter-joker/blob/master/index.js mentions the .joke extension. Should that be .joker?
Regards,
Erik Severinsen

I just created a PR #6 with a simple fix for this issue.


[ConfidentialLedger] Generate SDK based on preview-2024-01-26 spec

Packages impacted by this PR
@azure-rest/confidential-ledger
Issues associated with this PR
Add a single API - the ability to list users from the "app/users" endpoint. Please see the doc here.
Describe the problem that is addressed by this PR
Customers can GET details of a user from the app/users/{userId} endpoint, but needed to know the userId beforehand.  The API was amended in preview-2024-01-26 to include the ability to list all users from the app/users endpoint.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Yes
Provide a list of related PRs (if any)
Spec API Change
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[X] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[ ] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure-rest/confidential-ledger


Dropping timestamp_to_datetime?

So, I was working with the API and realized that the Models convert the timestamp to a datetime object; I was wondering if this 'helper' should actually be dropped from being enforced on the Model and just given to the user incase they wanted to use it, because it's a lot harder to reverse a datetime to timestamp than it is converting a timestamp to datetime.
So, the gist of my idea: Remove enforcing the created_time from being turned into a datetime object to just returning it to the user as an int, but still give them the helper function 'timestamp_to_datetime' in case they wanted to turn it into a datetime object.

??

import datetime
time time
time.mktime( datetime.datetime.utcnow().timetuple() )

Does that work for you?

Hm, that does work. The example I seen on StackOverflow was much more complicated. Maybe, keep the created_time a datetime object but add datetime_to_timestamp() in the helper?

Actually, that doesn't work.
from datetime import datetime
import time

ts =<PHONE_NUMBER>
dt = datetime.utcfromtimestamp(float(ts))  # datetime.datetime(2010, 7, 17, 2, 0, 30)

int(time.mktime(dt.timetuple()))  #<PHONE_NUMBER>

# original ts<PHONE_NUMBER>) !=
# new     ts<PHONE_NUMBER>)

Hey Michael,
Welcome to the wild world of Python datetime :)
To use mktime, you actually need a non-UTC timestamp, so you would do .fromtimestamp in your example instead of utcfromtimestamp.
If you want to pull a timestamp out of a datetime that came from utcfromtimestamp, you can do (continuing your example):
import calendar
calendar.timegm(dt.timetuple())

I think the current behavior is nice because time comparison and formatting will be much better with a datetime object, but I'd consider a patch that adds a created_time_as_timestamp or such property on media.

Yeah, I've been using something like that calendar method to convert all time formats to a nice timestamp, a "strtotime" if you will.
Anyways, I'll add that func sometime today! :)


X509 proxy and uk e science ca update

A small edit to the getting started documentation, as the UK eScience certificate information, link, and adding it to the container was missed by a user.
Also made small change to config to use the x509_proxy rather than x509

Not related to the changes, but I think you need to adapt your GitHub workflow, this PR is again including some commits that, while at least they don't change unintended files, should probably not be there:

This may help: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork
@enolfc , @andrea-manzi letting you review the PR.

Thanks for that link, I will take a look, and I did notice that list of commits that weren't helpful

Thanks for that link, I will take a look, and I did notice that list of commits that weren't helpful

No worries, cf https://xkcd.com/1597/ ^^ :)


⚠ Fortnite has degraded performance

In fe431a6, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1723 ms

Resolved: Fortnite performance has improved in 8358b80.


Investigate cancelable promises

Some torrents take a long time to load and mess up the state. This race condition situation can be fixed with cancelable promises

Investigate RX and redux-obeservable instead of this


Setting User-Agent header doesn't work

<EMAIL_ADDRESS>commented:

To reproduce:
page.open('...', { method: 'get', headers: { 'User-Agent': 'foo' } })
As a workaround, the user agent can be set via page.settings.userAgent, but the above should really work too.

Disclaimer:
This issue was migrated on 2013-03-15 from the project's former issue tracker on Google Code, Issue #745.
:star2:   3 people had starred this issue at the time of migration.

is there any update on this? Because I'm running into the same problem right now, that our web-application recognises the Capybara-Poltegeist test as invalid browser due the header not set properly.

This issue still seems to be present. I'm using version 2.1.1.


Typing library in Python causing formatting / intellisense fail

Issue Type: Bug
Using Any, Dict, List or Tuple from the typing library removes syntax highlighting and intellisense from my code.
VS Code version: Code 1.21.1 (79b44aa704ce542d8ca4a3cc44cfca566e7720f1, 2018-03-14T14:46:47.128Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-1620 v3 @ 3.50GHz (8 x 3492)

Memory (System)
15.93GB (6.22GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (14)

Extension
Author (truncated)
Version

code-gnu-global
aus
0.2.2

vscode-markdownlint
Dav
0.14.1

code-runner
for
0.9.3

vscode-codemetrics
kis
1.15.0

MagicPython
mag
1.0.12

python
ms-
2018.3.1

vscode-ai
ms-
0.1.7

cpptools
ms-
0.16.1

csharp
ms-
1.14.0

PowerShell
ms-
1.6.0

team
ms-
1.122.0

vscode-docker
Pet
0.0.26

code-settings-sync
Sha
2.9.0

python
tht
0.2.3

Reproduces only with extensions

Can you provide a code sample?

Sure.
Code sample:
`from typing import Any, Dict, List, Tuple
class Foo(object):
@property
def bar(self) -> int:
    return 1

@property
def baz(self) -> None:
    return

def hello_world(self):
    print("Hello world")
    return -1

def broken_hello(self) -> int:
    print("Hello world")
    this_is_not_a_method()
    return -1
`

Attached a screenshot of the lack of syntax highlighting (or error!)

This issue was moved to MagicStack/MagicPython#131


Semver syntax is poorly chosen

OK, it is inevitable that opinions will differ on the semver syntax. But it seems to me that the syntax that has been adopted is a bit confusing and not sufficiently self-evident.
I'd like to offer my suggestions on how to improve this.
Referring to https://github.com/golang/dep#semantic-versioning,

The inequality operators = != > < >= <= are necessary and satisfactory.
The wildcards are obvious and therefore satisfactory. It should be clear that the wildcard must be on the right, so 1.x.2 is not a valid use though. The parser should be lax about 1.x and 1.x.x having the same meaning because it is unambiguous.
The literal range is necessary and satisfactory.
The minor range and major range are not satisfactory because they duplicate what wildcards provide - but they do it in an obscure way. You Just Have To Know (TM).
When you specify a version without an operator, dep automatically uses the major range operator by default. This is not obvious and will cause confusion for some people.

So I suggest two changes:

Drop the minor and major ranges. So wildcards will be necessary when there is a preference to use as high version as possible.
The default behaviour should be = instead of ^. That is, if you don't specify wildcards, you get the version you did specify and nothing else.

Generally, the parser ought to be lax about whitespace.

sorry, but this is far past the point of no return. changing the interpretation of semver would break the Gopkg.toml backwards compatibility guarantee.
this can be revisited when it comes time to move into the toolchain, when we may have a chance to make breaking changes to the file format.

When you specify a version without an operator, dep automatically uses the major range operator by default. This is not obvious and will cause confusion for some people.
The default behaviour should be = instead of ^. That is, if you don't specify wildcards, you get the version you did specify and nothing else.

this, in particular, has been hashed through elsewhere - #929. we were aware of the costs going in. it's another thing we could revisit later.
to be clear, it's not that i'm entirely happy with our semver parsing, either. but the ship's sailed.


example

Hello @cayter !
Can you add some example?

Ok. Let me add in some later.

Hi, @mibamur!
I've added some examples here. Please let me know if there's anything else that I can help with. Thanks!


build: fix check_set_compiler_property

check_set_compiler_property() was calling zephyr_check_compiler_flag without first having set 'check'. This is required since https://github.com/zephyrproject-rtos/zephyr/commit/c32e4a71bf867029b6e828e1c1e49865c48b1c52 added a second dereferencing for the 'check' argument.
Issue #53124

@yperess thanks for the description in #55490, was able to reproduce.
Proper fix posted here: #55516


Add totp.verify examples with custom time.

#85
#81
Add examples of totp.verify in README.
If time is specified in speakeasy.totp, it should be present in speakeasy.totp.verify and speakeasy.totp.verifyDelta respectively.

Coverage remained the same at 100.0% when pulling a11d8764281e479e9c5bcb3edd8d305b7a9e3885 on railsstudent:update-totp-verify-documentation into e63b936d562fa16c5970d0821a221cf1a59f559b on speakeasyjs:master.

Coverage remained the same at 100.0% when pulling a11d8764281e479e9c5bcb3edd8d305b7a9e3885 on railsstudent:update-totp-verify-documentation into e63b936d562fa16c5970d0821a221cf1a59f559b on speakeasyjs:master.

Coverage remained the same at 100.0% when pulling a11d8764281e479e9c5bcb3edd8d305b7a9e3885 on railsstudent:update-totp-verify-documentation into e63b936d562fa16c5970d0821a221cf1a59f559b on speakeasyjs:master.

Coverage remained the same at 100.0% when pulling a11d8764281e479e9c5bcb3edd8d305b7a9e3885 on railsstudent:update-totp-verify-documentation into e63b936d562fa16c5970d0821a221cf1a59f559b on speakeasyjs:master.

Coverage remained the same at 100.0% when pulling a11d8764281e479e9c5bcb3edd8d305b7a9e3885 on railsstudent:update-totp-verify-documentation into e63b936d562fa16c5970d0821a221cf1a59f559b on speakeasyjs:master.

LGTM. @mikepb cross-check?


Unsupported Postgres type jsonb

trying to use postgres_scan on a table with jsonb fields and getting the following exception: Unsupported Postgres type jsonb

This should be fixed in the latest version though?

yes, you are correct. its fixed in the latest version.
I had the old version when I was running the test with jsonb
On Thu, Dec 1, 2022 at 1:51 AM Hannes Mühleisen @.***>
wrote:

This should be fixed in the latest version though?
—
Reply to this email directly, view it on GitHub
https://github.com/duckdblabs/postgres_scanner/issues/62#issuecomment-1333290520,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADUS6DCJZZ4LSSCMVRGQULWLBDFLANCNFSM6AAAAAASQBPNK4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Added more Verbose Input description for  splitters

Fixes #1368
Describe the changes you have made in this PR -
I have added more verbose input descriptions for splitters as described in issue#1368
Screenshots of the changes (If any) -
BEFORE->

AFTER->

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 82.63% when pulling 74e2a8c59088209b33da4d8189f1d37c06032e01 on Prerna-0202:Verbose#1368 into a420d34082e1bb4c481839f1cf1b9eba364c3af0 on CircuitVerse:master.

@gr455 @YashKumarVerma, Can you please review my pr ...

Second argument for prompt is a sample input, not a proper placeholder like placeholder from  field. I think we can hold this as we need to remove usages of prompt and alert from the codebase anyway


Unzip Feature

Unzip feature should be added next update since File util didnt work anymore in Android11

Added in 1.0.0


Fix bug #20 (Cleanup of #24)

This is the changes from #24, cherry-picked by hand to remove the extra LED logic and reformatted to the original code style.

Thanks for the PR, I tested and it works fine 👍


Target build through PR same as to branch.

Don't run both builds on PR, only the relevant one (and only if the PR targets a relevant branch).

The non-terraform build is currently a required status check, so PRs targeting terraform only couldn't be merged with this change.


♻ Refactor SSW Rules Widget

Cc: @bradystroud @babakamyljanovssw @adamcogan
Hi Team,
Pain
Our current SSW Rules Widget relies on Azure Functions and a CosmosDB table, which adds unnecessary complexity and maintenance overhead to our system. The current issues we are facing is due to the builds and Azure Functions being out of sync and not storing all the data required.
What code could be improved?

The SSW Rules Widget codebase, specifically the parts that interact with Azure Functions to fetch rules.
A Single consistent retrieval method(s) that return the latest Rules information. Function is used and exported from the widget and used inside Rules + Widget.

Acceptance Criteria

[x] Add new fields title, uri, isArchived to history.json
[ ] Implement GitHub API integration to fetch GitHub user information
[ ] Refactor to use the history.json api instead of the Azure Functions
[ ] Remove all Azure Functions previously used for fetching rules
[ ] Remove the unnecessary table from CosmosDB
[ ] Ensure all existing functionality is maintained
[ ] Implement caching mechanism to optimise GitHub API calls (optional - might not be needed)
[ ] Update documentation to reflect new architecture
[ ] All tests pass, including new tests for GitHub API integration
[ ] Performance benchmarks show no degradation

More Information

We'll need GitHub API credentials or tokens for authentication
Consider implementing a fallback mechanism in case of GitHub API unavailability
Ensure rate limiting considerations are accounted for in the new implementation

Screenshots

Figure: Current Widget Architecture Flow

Figure: Planned Widget Architecture Flow
Thanks!

Update
PRs drafted to add new fields to history.json
https://github.com/SSWConsulting/SSW.Rules.Functions/pull/59
https://github.com/SSWConsulting/SSW.Rules/pull/1440

Update:
There were some blockers due workflows were failing for SSW.Rules and SSW.Rules.Functions projects.
New Issues are created and PRs to resolve these blockers:
Issues:

https://github.com/SSWConsulting/SSW.Rules/issues/1448
https://github.com/SSWConsulting/SSW.Rules.Functions/issues/65

PRs:

https://github.com/SSWConsulting/SSW.Rules.Content/pull/9142
https://github.com/SSWConsulting/SSW.Rules.Functions/pull/61
https://github.com/SSWConsulting/SSW.Rules.Functions/pull/62
https://github.com/SSWConsulting/SSW.Rules.Functions/pull/63
https://github.com/SSWConsulting/SSW.Rules.Functions/pull/64

Update:
Drafted PR on SSW.Rules.Widget repo to refactor so it uses history.json and commits.json
https://github.com/SSWConsulting/SSW.Rules.Widget/pull/55

Update:
PRs to update widget version for SSW.Rules and SSW.People projects

https://github.com/SSWConsulting/SSW.Rules/pull/1449
https://github.com/SSWConsulting/SSW.People/pull/677

cc: @bradystroud @JackDevAU
Update:
We are going to wait for 1 week and then proceed with removing unnecessary tables from CosmosDB

Update:
The UpdateLatestRules functions wasn't being used after last time I checked which is good and can be removed safely, but the GetLatestRules function was called a few times.

Figure: A few calls made to function after last check
Not sure where it is being called from, this function was being used in widget, but we have updated widget to new version to use history.json.
I checked the request logs for this function before widget upgrade and after widget upgrade for the 7 day period:

1400 logs are returned before widget upgrade
7 logs are returned after widget upgrade

Currently investigating where these calls are made from

Update
I couldn't find from where the GetLatestRules are being called yet. The calls are not frequent as it is used to be after recent changes to widget's data source. Currently I'm working on https://github.com/SSWConsulting/SSW.Rules/issues/1448 as top priority.
This work still in progress

✅ GetLatestRules and UpdateLatestRules functions were removed

Implement GitHub API integration to fetch GitHub user information

All tests pass, including new tests for GitHub API integration

Implement caching mechanism to optimize GitHub API calls (optional - might not be needed)

All above tasks are not required, as we don't need to call GitHub API to get user info as commits.json already has user information and widget is using it to get latest rules updated by specified user.
For more info look at this PR - https://github.com/SSWConsulting/SSW.Rules.Widget/pull/55

✅ Done - removed LatestRules container from cosmos db

Figure: current cosmos db containers


Release/1.2.24

Can you also check why tests are failing>

@sacOO7 sure, it looked like github actions wasn't able to find last java version, but it's all good now


Implement a LazyExchange.

Declared upon first use. Allows for retrieval of name / options without declaration. Can be returned by DeliveredMessage instead of just a string name.

It's probably not possible to get the type/options of a declared exchange (unless it's one of the known amq.* ones).  So if this goes ahead at all it should probably be called something like UnknownExchange. The type() and options() methods would need to throw in this case.


Enable preemption

Procedure:

Remove put object from server
Record time put object spent in the server
Move put object back to the queue (state = idle, repeat steps in Put function without calling it so that you can use the same original Put object)

Changes required

[ ] Server (resource) should store the object that is currently being processed (e.g., Put object that was dequeued from the put_queue)
[ ] Put objects should have an age field to denote the amount of work done on the object

It looks like you should use a Store instead of a Container to accomplish this.
You could pass a job object to the Store where you can log the age of the job.

closed by #28c010f


Fix incorrect debug message

#1556 Fixes a panic when sandbox.osSbox is nil. But the error message returned is not getting displayed correctly. Fixed it and also changed it to a warning so that such errors won't go unnoticed.
Signed-off-by: Santhosh Manohar<EMAIL_ADDRESS>

LGTM


mozcdic-ut-personal-names: 0-unstable-2024-09-21 -> 0-unstable-2024-10-14

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mozcdic-ut-personal-names is: Mozc UT Personal Name Dictionary is a dictionary for Mozc.
meta.homepage for mozcdic-ut-personal-names is: https://github.com/utuhiro78/mozcdic-ut-personal-names
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0-unstable-2024-10-14 in filename of file in /nix/store/knk715g0fb1rhwjvcl782893ff5a3zxp-mozcdic-ut-personal-names-0-unstable-2024-10-14

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

ibus-engines.mozc-ut

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/knk715g0fb1rhwjvcl782893ff5a3zxp-mozcdic-ut-personal-names-0-unstable-2024-10-14 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/19742067f4694a18c827aceaebdb0221bb7ab818#mozcdic-ut-personal-names

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/knk715g0fb1rhwjvcl782893ff5a3zxp-mozcdic-ut-personal-names-0-unstable-2024-10-14
ls -la /nix/store/knk715g0fb1rhwjvcl782893ff5a3zxp-mozcdic-ut-personal-names-0-unstable-2024-10-14/bin

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

x86_64-linux

:white_check_mark: 3 packages built:

ibus-engines.mozc-ut
merge-ut-dictionaries
mozcdic-ut-personal-names

Maintainer pings
cc @pineapplehunter for testing.

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

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

@NixOS/nixpkgs-merge-bot merge


Require data directory to be set

Right now, the data directory is automatically derived from the mount directory. However, this can be confusing so it's probably better to explicitly set the mount and data directory.
Originally mentioned by David Cameron on Twitter: https://twitter.com/dave_cameron/status/1571514794989191169

Closed via #109


Cannot install OmniSharp server in neovim E684: list index out of range: -1

I'm trying to install this plugin in neovim by using vim-plug. The plugin omnisharp-vim installed correctly. Whenever I try to open a .cs file however, it asks me to install the OmniSharp server. I type Y and after a few seconds, it presents me with the file. I get no error message, but it seems like the OmniSharp server hasn't been installed, as I cannot use any of the mappings. On top of that, when I want to edit a new file with :edit, it tells me that the OmnSharp server isn't installed and asks me again to install it. When I press Y this time, I do get an error:
E684: list index out of range: -1

It also keeps asking me to install the server when I close all files and try to open a .cs file again.
Furthermore, when I try to install the server with :OmniSharpInstall, I get the following error message:
Failed to install the OmniSharp-Roslyn server
Error detected while processing function OmniSharp#Install:
line 53:
E684: list index out of range: -1

I have had this problem on 3 devices:

Raspberry Pi 4 with Raspbian (lite: no GUI) installed. This is the main device that I want OmniSharp to work on
ThinkPad X220 with Void Linux (window manager: dwm)
Debian (desktop environment: KDE) Virtual Machine in VirtualBox on a Windows 10 host

Neovim version: 0.6.1
Plugin manager: vim-plug
OS: Linux 64-bit (x64) and Linux 32-bit ARM (Raspbian)

I'll have a look later on and see if I have any problems with the installer from neovim.
In the meantime, some things you could try are:

Install the server from vim instead of neovim. Once it's installed (which just means downloaded from a GitHub release and extracted) it will be usable from neovim.
Use the install script directly, see https://github.com/OmniSharp/omnisharp-vim/tree/master/installer

The installer is working correctly for me in archlinux with nvim 0.6.1.
I've just pushed commit 70810cf which should avoid the E684: list index out of range: -1 error you're seeing, but I don't understand how that situation is arising in the first place. Your neovim is running let l:error_msgs = systemlist(l:command) where l:command is the installer script, the system call is failing somehow, and l:error_msgs is being populated with an empty list. That's a bit peculiar, I would at least have expected an error message to be returned.
I suspect there is some kind of permissions issue here.

Closing, as unable to reproduce, and no feedback from OP


Add support for pprof

It can help us to get more input from the users and solve issues like #137
I'll do another PR with some instructions on how to use the tool

Thannks @gianrubio. Can we add the docs here?

@simonswine please check if the docs are acceptable


Twitter Session Even After Logout are Persistent

My environment

Android device: Moto G3
Android OS version: Nougat 7.1.1
Google Play Services version: 10.2.98
Firebase/Play Services SDK version: 1.2.0
FirebaseUI version: 1.2.0

Describe the problem:
Once a user logs in using twitter as a provider. Its twitter session is persistent even after he logs out of the firebase account.
Changes required in
com.firebase.ui.auth.provider.TwitterProvider
Proposed solution:
It can be removed by flushing the cookies everytime we call TwitterProvider.startLogin function
This reduces the effort for logging out every time you want to login using different user account
This is similar to what it is there in GoogleProvider
Relevant Code:
Problem starts here
@Override
public void startLogin(Activity activity) {
mTwitterAuthClient.authorize(activity, this);
}
Solution
@SuppressWarnings("deprecation")
private static void clearCookies(Context context) {
    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP_MR1) {
        Log.d(TAG, "Using clearCookies code for API >=" + String.valueOf(Build.VERSION_CODES
                .LOLLIPOP_MR1));
        CookieManager.getInstance().removeAllCookies(null);
        CookieManager.getInstance().flush();
    } else {
        Log.d(TAG, "Using clearCookies code for API <" + String.valueOf(Build.VERSION_CODES
                .LOLLIPOP_MR1));
        CookieSyncManager cookieSyncMngr = CookieSyncManager.createInstance(context);
        cookieSyncMngr.startSync();
        CookieManager cookieManager = CookieManager.getInstance();
        cookieManager.removeAllCookie();
        cookieManager.removeSessionCookie();
        cookieSyncMngr.stopSync();
        cookieSyncMngr.sync();
    }
}

@Override
public void startLogin(Activity activity) {
clearCookies(activity);
mTwitterAuthClient.authorize(activity, this);
}

@chiragbhatia94 yikes, this is a pretty obvious miss by us.  Can't believe we went so long with this behavior!
In the signOut function, looks like we're only calling Facebook, Google, and Firebase sign out:
https://github.com/firebase/FirebaseUI-Android/blob/master/auth/src/main/java/com/firebase/ui/auth/AuthUI.java#L165

@samtstern I noticed this right now, similar issue is with facebook login..

@chiragbhatia94 do you have the Facebook app installed?  If you don't then Facebook login uses the Chrome webview implementation which has cookies, so we can't initiate a full sign out of Facebook in that case.  We can only clear the app session.

@samtstern sir then this is a security issue. Instead of using the chromes web view we should use android's native webview that is usually used with facebook login (in case one is not using firebase)

this is from one of my other projects in which I used facebook login using firebase without authui... and this does not store the session details

@chiragbhatia94 I don't think we don't want that because it provides poor user experience. Using Chrome Custom Tabs allows the user to retrieve a stored session thus removing the need for them to sign in yet again. However, I think you can disable CCT by adding this to your manifest:
<activity
    android:name="com.facebook.CustomTabActivity"
    android:exported="true"
    tools:node="remove"/>

@chiragbhatia94 See #645, it should fix this issue!

I'm seeing strange issues with facebook login in FirebaseUi v1.2.0:

On logging in with facebook
logging out with (verifying it works and CompleteListener is called)
Uninstalling and reinstalling
On launch, the first thing I do is call FirebaseAuth.getInstance().getCurrentUser() which returns the user that I had logged out before reinstalling

Before I do any more digging/raising an issue, is this issue or PR related?

@ahaverty please raise a new issue for what you're seeing, definitely seems strange but not quite sure yet.

@samtstern @ahaverty I've seen this before and I believe it is due to the auto app backup introduced in M. Try going to Google Drive and deleting the backup and you should hopefully see that the user isn't logged in when you reinstall.

@SUPERCILEX Yeh, setting backups to false stopped it from happening. Sounds like a serious security issue though right? Thanks @samtstern I'll open one now, although I don't have much to add as I think setting backup false is something I wanted anyway.

@ahaverty I don't believe this is a security risk, see my explanation: https://github.com/firebase/FirebaseUI-Android/issues/688#issuecomment-299246794.

We've hit the same weird behavior using oauth with LinkedIn.
Even after reading @ahaverty comment on #688 I am still under the impression this is a security issue for a user. I will detail the user flow:

User A is borrowing user B device for a one-time session
User A is installing my App, authenticating using his LinkedIn oauth
User A is uninstalling my App consciously.
==> User A expectation is that there is no trace of his activity on user B device.
User B in installing my App, and his directly authenticated as user A!!!!

What am I missing?


Unremovable Pairing

Describe the bug
Ran into an issue with a web based dApp that seemed to forget its pairing details. It showed up as connected on the dApp, my mobile wallet also showed up as connected. When I tried to interact with the dApp, nothing was received by the wallet. I signed out of the dApp, connected again and everything worked. Later I discovered that the mobile wallet had 2 pairings with with same dApp active and unexpired. Trying to remove both, one of them failed with an error
Serializer.symmetricKeyForTopicNotFound

And I am now unable to remove the pairing on my side. It seems like the dApp removed something but mobile never received notice and now its stuck in local cache. It has almost a month left on the expiration date so it will remain on device for a very long time
This is somewhat related to this other issue: https://github.com/WalletConnect/WalletConnectSwiftV2/issues/1344. In general, web versions of the SDKs seem to modify the local cache, and then trigger the networking (or always update the cache), whereas the Swift SDK seems to do the networking and only touch the local cache if it succeeds. Often leaving the cache in a broken state.
SDK Version

Client: Swift
Version: 1.18.8

To Reproduce
Steps to reproduce the behavior:
Unable to reproduce
Expected behavior
When calling disconnect, i'd expect the item to be removed no matter what happens. If dApp never receives it, user can at least recover by signing out on that side also. In the current state the user is unable to recover and is stuck with identical pairings for ~30 days
Device (please complete the following information):

pairing topic in case its useful: f735578eb756d3b908d19eef71f675da8832dc904d67c0b1740d04bbb6b16e58

Update: this is actually now becoming very common. In at least one case I know a dApp updated multiple dependencies and lost its WC2 session somehow (I suspect this is likely a bug in wrappers around WC2 being used). And now i'm stuck with multiple copies of that connection. Others are dApps i've not used in sometime, the connection appears to be up on both sides, but its not working and I can't disconnect on mobile side
@llbartekll any chance I can request the priority of this one be moved up? Were hoping to submit our app to app store next week, and this is likely a blocker for us

Hey, I will look into it

@simonmcl could you try to upgrade your dependency to this branch and check if you can remove the now?
#1376

@llbartekll Almost. It is deleting the records now locally, but still throws the error. So my screen is telling the user that the disconnect failed, and not refreshing. Then when they back out and return to the screen its gone

hmm, it shouldn't throw anymore 🤔 can you upgrade to the last commit? do you still have stale pairing to reproduce?

@llbartekll I can't find another stale one. The last dev build that was really bad I deleted as it was causing issues with automated UI tests. Either way it worked, if the throw can't be verified i'd prefer to just catch that error in this one case on my side and suppress it for now, rather than leaving users stuck

tbh I didn't experience it myself, so it's difficult to find the cause. In the pr, in case the issue appears, the pairing will be removed anyway on your client and the func won't throw, so user won't get stuck. We are also planning to implement some simplification to pairing, so I think this should be fine for now.
please confirm if that works

@llbartekll yes thats fine for me/this ticket. Thank you for prioritising this, much appreciated

cool, i will include it in 1.19.4


pull request for e2e test at 11/21/2016 16:26:05

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Customisation of speaker select message and prompt in GroupChat

Why are these changes needed?
During a Group Chat the Select Speaker Message (first in the context) and Select Speaker Prompt (last in the context) are hardcoded f-strings based on a role-playing prompt. To provide greater flexibility in the final content (for both OpenAI and especially non-OpenAI models), I'm proposing the ability to customise both messages (while including the ability to include the role descriptions and list).
From my testing with open source/weight LLMs I have found that changing these can influence the LLM noticeably.
Changes
I'm proposing the addition of two new Optional[str] (default is None) attributes of the GroupChat class:
custom_select_speaker_msg
custom_select_speaker_prompt
These will be used in the GroupChat select_speaker_msg and select_speaker_prompt functions, which will return the current prompt if the respective attributes are None.
custom_select_speaker_msg can include {roles} in its string, which will be replaced with self._participant_roles(agents), and it can include {agentlist} which will be replaced with {[agent.name for agent in agents]}.
custom_select_speaker_prompt can include {agentlist} which will be replaced with {[agent.name for agent in agents]}.
This enables you to replicate the insertion of role descriptions and role lists as per the current prompts.
I have not added a documentation example but will do if we are pushing forward with this change.
Related issue number
This is related to my Draft PR #1746.
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[X] I've made sure all auto checks have passed.

Codecov Report
Attention: Patch coverage is 53.84615% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 44.70%. Comparing base (dd61eaa) to head (633bd2a).

Files
Patch %
Lines

autogen/agentchat/groupchat.py
53.84%
4 Missing and 2 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2182      +/-   ##
==========================================
+ Coverage   37.97%   44.70%   +6.73%
==========================================
  Files          75       75
  Lines        7587     7598      +11
  Branches     1634     1770     +136
==========================================
+ Hits         2881     3397     +516
+ Misses       4463     3919     -544
- Partials      243      282      +39

Flag
Coverage Δ

unittests
44.60% <53.84%> (+6.64%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Some test errors in CI, could you check the test. It looks like the whitespaces in the expected string doesn't match the output.
>       assert (
            groupchat.select_speaker_msg()
            == """You are the CEO of a niche organisation creating small software tools for the healthcare sector with a small team of specialists. Call them in sequence.
    The job roles and responsibilities are:
    Alice: A wonderful employee named Alice.
    Bob: An amazing employee named Bob.

    You must select only from ['Alice', 'Bob']."""
        )
E       assert "You are the ...ice', 'Bob']." == "You are the ...ice', 'Bob']."
E
E         Skipping [143](https://github.com/microsoft/autogen/actions/runs/8485511517/job/23250435999?pr=2182#step:7:144) identical leading characters in diff, use -v to show
E           sequence.
E         - The job roles and responsibilities are:
E         +     The job roles and responsibilities are:
E         ? ++++
E         - Alice: A wonderful employee named Alice.
E         +     Alice: A wonderful employee named Alice.
E         ? ++++
E           Bob: An amazing employee named Bob.
E
E         - You must select only from ['Alice', 'Bob'].
E         +     You must select only from ['Alice', 'Bob'].
E         ? ++++

test/agentchat/test_groupchat.py:1222: AssertionError

Some test errors in CI, could you check the test. It looks like the whitespaces in the expected string doesn't match the output.
>       assert (
            groupchat.select_speaker_msg()
            == """You are the CEO of a niche organisation creating small software tools for the healthcare sector with a small team of specialists. Call them in sequence.
    The job roles and responsibilities are:
    Alice: A wonderful employee named Alice.
    Bob: An amazing employee named Bob.

    You must select only from ['Alice', 'Bob']."""
        )
E       assert "You are the ...ice', 'Bob']." == "You are the ...ice', 'Bob']."
E
E         Skipping [143](https://github.com/microsoft/autogen/actions/runs/8485511517/job/23250435999?pr=2182#step:7:144) identical leading characters in diff, use -v to show
E           sequence.
E         - The job roles and responsibilities are:
E         +     The job roles and responsibilities are:
E         ? ++++
E         - Alice: A wonderful employee named Alice.
E         +     Alice: A wonderful employee named Alice.
E         ? ++++
E           Bob: An amazing employee named Bob.
E
E         - You must select only from ['Alice', 'Bob'].
E         +     You must select only from ['Alice', 'Bob'].
E         ? ++++

test/agentchat/test_groupchat.py:1222: AssertionError

I'm not sure why that is failing, it should match, let me try running tests again.

We can't run Open AI tests in CI from branches outside of the Microsoft repo.
Can you push your code to a new branch in the Microsoft repo, and create a new PR referencing this one? So we can run the OpenAi test

We can't run Open AI tests in CI from branches outside of the Microsoft repo.
Can you push your code to a new branch in the Microsoft repo, and create a new PR referencing this one? So we can run the OpenAi test

Thanks, I'm working to do this now.


dock run --user parameter should override container's redmine user

When I run this container definif a user, the container won't get up.
I defined this user: www-data inside my docker-compose.yml.
# docker-compose up
Creating redmine_app_1
Attaching to redmine_app_1
app_1 | Initializing logdir...
app_1 | mkdir: cannot create directory '/var/log/redmine/supervisor': Permission denied
redmine_app_1 exited with code 0

I think the container should respect the parameters from the docker run, letting them override the default parameters from the container's initialization scripts.

@endersonmaia there is a setup taking place in the container that require root privileges. This includes but is not limited to:

correcting file/directory permissions
starting supervisor

Specifying --user while starting the container means that the specified user unless that is root will not be able to correctly setup the container for launch.
What is your use case here? Why do you want to launch the container with --user www-data? Maybe you need to look at https://github.com/sameersbn/docker-redmine#mapping-host-user-and-group if thats what you are interested in.
Thank you.

I get an error when define the www-data UID 'cause it already exists (33) on the host system and on the container.

existence of the user on the host is fine. but i am not sure how to handle the case where a user with the requested UID already exists in the container (www-data in this case). Possible solution is to remove the existing user in this case before performing the uid mapping. However I am not sure whether this could have any negative effects.

Closing due to old age


test(filter): add subscription marble tests for filter

Related to #390.

Change looks good for me.

:+1:

Merged with 1e9769d, thanks @staltz


Site Request:  Comcast Sportsnet

Comcast Sports uses "ThePlatform"
NOTE: Comcast Sports are affiliated with (co-owned by?) NBCSports, which also use "ThePlatform"
URL: http://www.csnne.com/blog/bruins-talk/charlie-jacobs-disgusted-bs-unacceptable-play
youtube-dl.py -v "http://www.csnne.com/blog/bruins-talk/charlie-jacobs-disgusted-bs-unacceptable-play"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.csnne.com/blog/bruins-talk/charlie-jacobs-disgusted-bs-unacceptable-play']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2015.04.09
[debug] Python version 2.7.6 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-40824-g31dfe20, rtmpdump 2.4
[debug] Proxy map: {}
[generic] charlie-jacobs-disgusted-bs-unacceptable-play: Requesting header
WARNING: Falling back on generic information extractor.
[generic] charlie-jacobs-disgusted-bs-unacceptable-play: Downloading webpage
[generic] charlie-jacobs-disgusted-bs-unacceptable-play: Extracting information
ERROR: Unsupported URL: http://www.csnne.com/blog/bruins-talk/charlie-jacobs-disgusted-bs-unacceptable-play
Traceback (most recent call last):
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\extractor\generic.py", line 858, in _real_extract
doc = parse_xml(webpage)
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\utils.py", line 1531, in parse_xml
tree = xml.etree.ElementTree.XML(s.encode('utf-8'), **kwargs)
File "C:\Python27\lib\xml\etree\ElementTree.py", line 1300, in XML    parser.feed(text)
File "C:\Python27\lib\xml\etree\ElementTree.py", line 1642, in feed    self._raiseerror(v)
File "C:\Python27\lib\xml\etree\ElementTree.py", line 1506, in _raiseerror    raise err
ParseError: mismatched tag: line 117, column 2
Traceback (most recent call last):
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\YoutubeDL.py", line 651, in extract_info
ie_result = ie.extract(url)
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\extractor\common.py", line 275, in extract
return self._real_extract(url)
File "C:\Transmogrifier\youtube-dl.py\youtube_dl\extractor\generic.py", line 1364, in _real_extract
raise UnsupportedError(url)
UnsupportedError: Unsupported URL: http://www.csnne.com/blog/bruins-talk/charlie-jacobs-disgusted-bs-unacceptable-play
Thanks
Ringo

Not sure if these can be added to (or a generic?):
[Yahoo/NBCSports] Fix 5226 and add support for NBC sports #5328
but it seems NBC Sports has number of regional sports channels, in addition to the original post csne.com.
Here is the list :  http://www.comcastsportsnet.com/
(NOTE: the NY channel seems different from the rest)
Example URLS.
http://www.csnbaltimore.com/blog/orioles-talk/orioles-lose-fifth-straight-mccullers-dominates
http://www.csnchicago.com/blackhawks/hulk-hogan-believes-blackhawks-are-lot-trouble
http://www.csnbayarea.com/warriors/who-has-edge-nba-finals-warriors-or-cavs
http://www.csncalifornia.com/warriors/who-has-edge-nba-finals-warriors-or-cavs
http://www.csnchicago.com/blackhawks/hulk-hogan-believes-blackhawks-are-lot-trouble
http://www.csnnw.com/blazers/nba-scouts-take-montrezl-harrell-and-christian-wood
http://www.csnphilly.com/football-philadelphia-eagles/scheme-change-byron-maxwell-could-match-no-1-wrs
http://www.csnwashington.com/baseball-washington-nationals/talk/buehrle-pitches-complete-game-shutout-downs-nationals-8-0
Thanks..and amazing work as always!
Ringo

all URLs redirect now to NBC Sports website.


Remove redundant 'reduce' overloads

Fixes #25454

The contextual type would be incorrect anyway in the case that U !== T, wouldn't it? And I don't think   initialValue will commonly be context sensitive. Except in the case of tuples, maybe.

I did come up with a test case that's broken by this, see the bottom of arrayReduce.ts. Probably not a common situation though... I don't think there's a way we can support initialValue having an arbitrary type, and also give it a contextual type based on the assumption that it's the same type as the array elements (which may be a supertype of what you wanted initialValue to be, which caused the original issue).

@mhegazy @sandersn @rbuckton @DanielRosenwasser @weswigham Could you review?

@mhegazy @sandersn @rbuckton @DanielRosenwasser @weswigham Could you review?

@andy-ms let's get this up to date and then see what the RWC run turns up

@typescript-bot test this

Nontrivial RWC breaks occurred