Changes connection error handling

The Next method for changes returns io.EOF even if the connection is interrupted (e.g. CouchDB is restarted/crashes).
This is because the More method on json.Decoder returns false when the connection is closed, and therefore Next method returns io.EOF.
See: https://github.com/go-kivik/couchdb/blob/master/changes.go#L59
if !r.dec.More() {
	return io.EOF
}

So if you have a goroutine that calls Next in a loop the connection error will go unnoticed and you will not get any more changes. Ideally Next would return io.ErrUnexpectedEOF when the connection is interrupted.

Here is a simple application that reproduces this issue:
package main

import (
	"context"
	"fmt"
	_ "github.com/go-kivik/couchdb" // CouchDB Driver for kivik
	"github.com/go-kivik/kivik"
)

func main() {
	cdb, err := kivik.New("couch", "http://admin:password@localhost:5984/")
	if err != nil {
		panic(err)
	}

	db := cdb.DB(context.TODO(), "mydatabase")
	if db.Err() != nil {
		panic(db.Err())
	}

	changes, err := db.Changes(context.TODO())
	if err != nil {
		panic(err)
	}

	fmt.Println("Connected to change feed. Please shutdown CouchDB to demonstrate the bug.")
	fmt.Println("Documents changed:")

	for {
		if !changes.Next() {
			if changes.Err() != nil {
				break
			}
		}

		fmt.Println(changes.ID())
	}

	fmt.Println("This will never be printed!")
}

Thank you for the error report, and the reproduction case. It's not immediately obvious to me how to fix it, but I'll work on it when I have time (hopefully this weekend). In the meantime, if you find a solution, a PR is always appreciated.

Thanks for looking into the issue as well as providing a great library!
Both More and Decode on json.Decoder are blocking so my initial idea was to remove the More call and add a special case to handle the io.EOF error. I am not certain on the full implication of this change though so was hesitant to do a PR.
The changed method would look something like this:
func (r *changesRows) Next(row *driver.Change) error {
	if r.dec == nil {
		r.dec = json.NewDecoder(r.body)
	}

	if err := r.dec.Decode(row); err != nil {
		if err == io.EOF {
			return io.EOF
		}

		return &kivik.Error{HTTPStatus: http.StatusBadGateway, Err: err}
	}
	return nil
}

I did a little experimentation with just using Decode(). That is promising, but there's a problem that it still seems to lose any read error, if the read error occurs at the end of a valid JSON object (which will be the typical case).
So I see two issues here:

Next() should not continue forever (as you observe). This is probably the easier of the two problems to fix.
Read errors should be returned to the caller.  I don't see an obvious way to handle this, based on my superficial testing thus far. But I should have more time to dig in tomorrow, and hopefully find something.

I believe I've found a solution. Stand by. :)

Please let me know if this solves your problem.

That solves the issue :) Thanks for the amazingly quick fix!


Should the caller deal with *retryableError

Hello Seth, thanks for the nice facility that I'm trying to use for the first time.
It seems that:

the Do signature says it returns an error, which is idiomatic (instead of a more specific and error-prone error type);
there is currently no promise about the exact form of the returned error;
on success (regardless the number of attempts), Do returns nil;
on failure, Do ignores the results of the first failed attempts, and returns the error of the very last failed attempt, wrapped in a *retryableError.

If my understanding (the above bullets) is correct, then it seems that it would be even nicer to return the "root cause" error to the caller of Do, i.e. not wrapped in a *retryableError.  Shall I try a PR?
Caveat: maybe I'm not familiar enough with errors.As and interface {Unwrap() error}, which means that it would be, in fact, more idiomatic to keep the *retryableError as a visible trace of the retry device, and let the caller unwrap/test the returned error.

Hey @Deleplace - thanks for opening an issue.

If my understanding (the above bullets) is correct, then it seems that it would be even nicer to return the "root cause" error to the caller of Do, i.e. not wrapped in a *retryableError. Shall I try a PR?

My thinking (and I'm open to being challenged here) is that the developer has control over whether a particular error should be "retryable" or not (which is why you have to explicitly wrap an error as retryable with retry.RetryableError - normal errors are returned immediately). Thus I assumed the developer would explicitly call errors.Unwrap on the result, if they wanted a specific error.
Now, it does get a little confusing when you have multiple possible returned errors:
retry.Do(ctx, b, func(ctx context.Context) error {
  if err := thing1(); err != nil {
    return err // don't retry
  }

  if err := thing2(); err != nil {
    return retry.RetryableError(err)
  }
})

In this case, I agree it's difficult for the caller to know whether it's "safe" to unwrap the error, since errors from thing1 should not be unwrapped, but errors from thing2 should be unwrapped. Is https://github.com/sethvargo/go-retry/pull/6 what you're thinking?

Thank you Seth for this quick action, yes #6 is exactly what I was thinking.
My code at call site was doing this safe but boilerplaty check:
	if rerr, ok := err.(interface{ Unwrap() error }); ok {
		// The non-nil error returned by retry.Do (as of 2021-12) may have type *retry.retryableError
		err = rerr.Unwrap()
	}

Indeed for a moment I forgot that my own client code was explicitly doing the wrapping retry.RetryableError(err). It's still nice to not having to do the unwrapping myself from v0.2.0 on.

There's something weird going on with the release v0.2.0, I'll open a distinct issue for this.


Let's not wait too long

I've realized a few were lagging behind, update dependencies.

Sorry, don't know how to satisfy clippy 😢


[request] Video file icons

Images have an nice image icon, but videos get the boring fallback.
Can we please add a generic video file icon covering:

.webm
.mp4

And potentially:

.ogv
.ogg

How's this look, mate? :-)

:+1:

Prepping a pull request, along with commits for #266 and #268.

The current image icon contains the outline of a file with a bent corner, should the video icon be consistent with that?
Presumably each video format needs a color. There are different colors used for image formats (.png, .jpg, etc.). How were those colors decided?

Actually, the video icon is supplied by the Octicons typeface. Which, funnily enough, appears to have changed:

https://octicons.github.com/icon/file-media/
https://octicons.github.com/icon/device-camera-video/

Either way, it's probably a lot greppier to see a camera for video and a sheet of paper for 2D images.
As for the colours, well, I'm assuming they were picked at random. I kinda drew the video icon colours based very loosely on the colours of their logos, if any.


Failed to connect: IoError(Custom { kind: TimedOut, error: "timed out while reading from pipe" })

i'm run code from examples but it fails.

OS: Windows 11
Library version: 0.5.9

Did you ever figure out what caused this issue?


chore(package): update rollup-plugin-commonjs to version 8.0.1

https://greenkeeper.io/

Codecov Report

Merging #64 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #64   +/-   ##
======================================
  Coverage    40.6%   40.6%
======================================
  Files           7       7
  Lines         133     133
======================================
  Hits           54      54
  Misses         79      79

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 b5762e7...936f61f. Read the comment docs.


feat: add openai_base_url config

Setting the  openai_base_url in yai  allows for more flexibility and customization.

@JessonChan Thank you! This is really nice feature and works great!

@ekkinox This would be a wonderful addition to this tool which is already great. Could you please find a moment to merge this? :)
@tarkh You probably used a local model? If yes, which works the best?


Search installed apps

Should suggest installed apps (using a pop up list above the query field) as the user is typing and they're matching.
Only activities allowed on the launcher and matching the query should be listed.

When this happens, could there be an option to disable the online searches?
I am looking for an assist app that only includes installed apps and saw JustSearch in F-Droid and checked this GitHub in case it was what I am looking for.

Hi Mikaela,
If/when this happens, it would probably cover your use case without adding
another option as it should show a list of only apps as you're typing, as
opposed to initiating a web search when submitting the query.

meanwhile I suggest looking/testing KISS launcher in non-launcher mode.


[CARBONDATA-2929][DataMap] Add block skipped info for explain command

This pr will add block skipped info by counting distinct file path from hit blocklet. It shows like below:
|== CarbonData Profiler ==
Table Scan on test
 - total: 125 blocks, 250 blocklets
 - filter: (l_partkey <> null and l_partkey = 1006)
 - pruned by Main DataMap
    - skipped: 119 blocks, 238 blocklets
 - pruned by CG DataMap
    - name: dm
    - provider: bloomfilter
    - skipped: 6 blocks, 12 blocklets

|== CarbonData Profiler ==
Table Scan on test
 - total: 125 blocks, 250 blocklets
 - filter: TEXT_MATCH('l_shipmode:AIR')
 - pruned by Main DataMap
    - skipped: 0 blocks, 0 blocklets
 - pruned by FG DataMap
    - name: dm
    - provider: lucene
    - skipped: 12 blocks, 80 blocklets

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/243/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/412/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.3/8482/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/249/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.3/8487/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/417/

I think the message can be optimized from
- total: 125 blocks, 250 blocklets
to
- total: 125 blocks containing 250 blocklets


YAML syntax documentation

It would be very helpful to include a link to the documentation for the YAML syntax allowed in these files. Where can I find it?

Dokumentinformation
⚠ Redigera inte det här avsnittet. Det krävs för docs.microsoft.com ➟ länkning till GitHub-problem.

ID: 38f7a23b-ebd4-af3e-68a2-275f7e45c33e
Version Independent ID: f552a18c-ba29-00d4-03b6-254d7556d319
Content: Create your first pipeline - Azure Pipelines & TFS
Content Source: docs/pipelines/get-started-yaml.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

https://docs.microsoft.com/en-us/azure/devops/pipelines/yaml-schema

Thank you, that's great! As I also pointed out, it would be great to include this in the documentation page for others to find.

fixed in a commit I just made to our internal master, should publish soon. thank you for the feedback.


Exports: KiCAD schematic roundtrip

<Prefix>: <Title>
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Checklist
Please read and execute the following:

[ ] My code follows the coding guidelines of this project
[ ] My PR title is following the contribution guidelines
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (if applicable)
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works (if applicable)
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I ran Black to format my code

Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct:

[ ] I agree to follow this project's Code of Conduct

Sweet. Thanks for the review. Can do on the last suggestion. Positional doesn't work because it at least appears to always be the last item, so I could maybe modify that to work
On Sat, Sep 14 2024 at 00:25, Ioannis Papamanoglou < @.*** > wrote:

@.**** commented on this pull request.
In src/faebryk/libs/sexp/dataclass_sexp.py (
https://github.com/atopile/faebryk/pull/64#discussion_r1759687099 ) :

@@ -181,6 +185,13 @@ def _decode[T](
if str(key) +
"s" in key_fields or str(key) in key_fields:

ungrouped_key_values.append(val)
continue
+
+
The only symbol this is currently used for is the hide symbol in
schematics
Unfortunately that doesn't generalize :(
If a token is parsed as parseMaybeAbsentBool a token without braces is
evaluated to default and not true. There is (at least) one hide you are
dealing with that is not a parseMaybeAbsentBool and thus behaves
differently. We need to give for that one a hint to the parser in some
way.
Sidenote: Isn't it fine to mark it positional?
If not I would suggest making a new class (like with SymEnum).
—
Reply to this email directly, view it on GitHub (
https://github.com/atopile/faebryk/pull/64#discussion_r1759687099 ) , or unsubscribe
(
https://github.com/notifications/unsubscribe-auth/ABALRKW5JN7O2B7M43UK2Y3ZWPQHXAVCNFSM6AAAAABOGCSBDCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGMBUGM4TQNBUGA
).
You are receiving this because you authored the thread. Message ID: <atopile/faebryk/pull/64/review/2304398440
@ github. com>


Fasta, typedefs and generics

According to @lrhn this
typedef void F<T>(T v);
void defaultF<T>(T v) { }

class X {
  final F f;
  const X({this.f: defaultF});
}

main() {
  var x = const X();
  x.f<dynamic>("hello1");
}

should be a compile time error because field (dynamic) → void f; doesn't take any type parameters but one is given.
On the other hand if it had been typedef F = void Function<T>(T); instead, it would have been <T extends core::Object>(T) → void f; which does take a single type argument and it would have been OK.
CL @ https://dart-review.googlesource.com/42949 is going to add a test for this and adds
artial_tearoff_instantiation_test/05: MissingCompileTimeError
partial_tearoff_instantiation_test/06: MissingCompileTimeError
partial_tearoff_instantiation_test/07: MissingCompileTimeError
partial_tearoff_instantiation_test/08: MissingCompileTimeError

in the $fasta section.

/cc @peter-ahe-google

Aske, this looks like a missing argument check of type arguments. I think you may have recently glanced in the direction of something related, so perhaps you could take a look?

I'm hitting this while working on https://github.com/dart-lang/sdk/issues/33308.  At some point I need to iterate over type arguments and check them against their respective bounds, and I expect both lists to be of the same length.

Fixed by 015840d31f8235ac0a3bd5c46e5ca16fe3314c06.


Update How-to-release.md

Please answer these questions before submitting pull request

Why submit this pull request?
[ ] Bug fix
[X] New feature provided
[ ] Improve performance

Modify the "release:perform" command, add maven profile which will execute git module update command.

Coverage remained the same at 19.191% when pulling e9448479c4bfb86503e7e6fd626bbfbf1ade4a3e on peng-yongsheng-patch-1 into afcd3314d13ca3c0ddaa70e81b8cdeda0e870536 on master.


Fix uninitialised variables in the BaseMaterial3D.

Fix uninitialised variables in the BaseMaterial3D, UBSan report:
scene/resources/material.cpp:1467:6: runtime error: load of value<PHONE_NUMBER>, which is not a valid value for type 'BaseMaterial3D::Transparency'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior scene/resources/material.cpp:1467:6 in

scene/resources/material.cpp:1481:6: runtime error: load of value<PHONE_NUMBER>, which is not a valid value for type 'BaseMaterial3D::AlphaAntiAliasing'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior scene/resources/material.cpp:1481:6 in

Thanks!


Record cluster scoped handlers

Problem: Cluster scoped handlers were not being cleaned up without
manually writing garbage collection functions.
Solution: All cluster scoped handlers will be recorded in a global map
in a generic format so that they can be removed by a generic function.

LGTM


app delete needs a way to wait until it's really gone

Right now, by default, az containerapp delete will wait a while before it returns to the user. And there's a --no-wait flag you can specify to return right away w/o waiting at all. However, in the mode where it waits it's not waiting long enough. Right now if I do a "delete" and then immediately do a "create" using the same app name I'll get an error due to a conflict. Either the default should wait until the app is fully gone so a create using the same name works, or we should add another flag to ask for that behavior. W/o this the user is forced to do their own polling mechanism to know when it's safe to reuse the name.
I'd prefer to have the default just wait until the name is safe to reuse since --no-wait should be sufficient for all other cases. Today's behavior is a bit undefined and I'm not sure who would want to have the command return to them in this unknown half-there state (it seems like it should be either "return immediately" or "return when really gone").

Closing this, please reopen if you feel strongly.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

AWS CodeBuild CI Report

CodeBuild project: DDKIntegrationTestsD8472609-5SCHWjldmWcf
Commit ID: 2ee49406cd113bff08167deaa91674fc5ab3fcea
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Weird behavior with git in VS Code

Env
Pure
2.5.2

Fish
fish, version 3.1.0

OS/distribution
Distributor ID:	Ubuntu

Description:	Ubuntu Focal Fossa (development branch)
Release:	20.04
Codename:	focal
Fish works fine on the terminal but when I open a project in the VS Code terminal I get the following:
Failed to execute process '/usr/bin/git'. Reason: The total size of the argument and environment lists 1.5MB exceeds the operating system limit of 2MB. Try running the command again with fewer arguments.
This gets printed out multiple times each time I run a command.

Do you still have the issue?
Error message is weird and I find nothing similar online.

I managed to fix it by removing a few exports that I was no longer using on .fishrc. Though, I must say, I ended up migrating to zsh as it is POSIX compliant and easier to work with.

Thanks @otaviomad :heart:


Installation Error

Hi There.
For the first time i tried your code and it is resulting in error (after running composer update)
Please help. Thanks.
Problem 1
- Conclusion: remove laravel/framework v5.0.33
- Conclusion: don't install laravel/framework v5.0.33
- Conclusion: don't install laravel/framework v5.0.32
- Conclusion: don't install laravel/framework v5.0.31
- Conclusion: don't install laravel/framework 5.0.30
- Conclusion: don't install laravel/framework v5.0.29
- Conclusion: don't install laravel/framework v5.0.28
- Conclusion: don't install laravel/framework v5.0.27
- Conclusion: don't install laravel/framework v5.0.26
- Conclusion: don't install laravel/framework v5.0.25
- Conclusion: don't install laravel/framework v5.0.24
- Conclusion: don't install laravel/framework v5.0.23
- Conclusion: don't install laravel/framework v5.0.22
- Conclusion: don't install laravel/framework v5.0.21
- Conclusion: don't install laravel/framework v5.0.20
- Conclusion: don't install laravel/framework v5.0.19
- Conclusion: don't install laravel/framework v5.0.18
- Conclusion: don't install laravel/framework v5.0.17
- Conclusion: don't install laravel/framework v5.0.16
- Conclusion: don't install laravel/framework v5.0.15
- Conclusion: don't install laravel/framework v5.0.14
- Conclusion: don't install laravel/framework v5.0.13
- Conclusion: don't install laravel/framework v5.0.12
- Conclusion: don't install laravel/framework v5.0.11
- Conclusion: don't install laravel/framework v5.0.10
- Conclusion: don't install laravel/framework v5.0.9
- Conclusion: don't install laravel/framework v5.0.8
- Conclusion: don't install laravel/framework v5.0.7
- Conclusion: don't install laravel/framework v5.0.6
- Conclusion: don't install laravel/framework v5.0.5
- Conclusion: don't install laravel/framework v5.0.4
- Conclusion: don't install laravel/framework v5.0.3
- Conclusion: don't install laravel/framework v5.0.2
- Conclusion: don't install laravel/framework v5.0.1
- lucadegasperi/oauth2-server-laravel 5.0.0 requires illuminate/config 5.1.* -> satisfiable by illuminate/config[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.6, v5.1.8].
- lucadegasperi/oauth2-server-laravel 5.0.1 requires illuminate/config 5.1.* -> satisfiable by illuminate/config[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.6, v5.1.8].
- lucadegasperi/oauth2-server-laravel 5.0.2 requires illuminate/config 5.1.* -> satisfiable by illuminate/config[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.6, v5.1.8].
- lucadegasperi/oauth2-server-laravel 5.0.3 requires illuminate/config 5.1.* -> satisfiable by illuminate/config[v5.1.1, v5.1.13, v5.1.16, v5.1.2, v5.1.20, v5.1.22, v5.1.6, v5.1.8].
- don't install illuminate/config v5.1.1|don't install laravel/framework v5.0.0
- don't install illuminate/config v5.1.13|don't install laravel/framework v5.0.0
- don't install illuminate/config v5.1.16|don't install laravel/framework v5.0.0
- don't install illuminate/config v5.1.2|don't install laravel/framework v5.0.0
- don't install illuminate/config v5.1.20|don't install laravel/framework v5.0.0
- don't install illuminate/config v5.1.22|don't install laravel/framework v5.0.0
- don't install illuminate/config v5.1.6|don't install laravel/framework v5.0.0
- don't install illuminate/config v5.1.8|don't install laravel/framework v5.0.0
- Installation request for laravel/framework 5.0.* -> satisfiable by laravel/framework[5.0.30, v5.0.0, v5.0.1, v5.0.10, v5.0.11, v5.0.12, v5.0.13, v5.0.14, v5.0.15, v5.0.16, v5.0.17, v5.0.18, v5.0.19, v5.0.2, v5.0.20, v5.0.21, v5.0.22, v5.0.23, v5.0.24, v5.0.25, v5.0.26, v5.0.27, v5.0.28, v5.0.29, v5.0.3, v5.0.31, v5.0.32, v5.0.33, v5.0.4, v5.0.5, v5.0.6, v5.0.7, v5.0.8, v5.0.9].
- Installation request for lucadegasperi/oauth2-server-laravel 5.0.* -> satisfiable by lucadegasperi/oauth2-server-laravel[5.0.0, 5.0.1, 5.0.2, 5.0.3].

What does your composer.json file look like that output isn't very helpful without that.  You probably are trying to install a version that is not compatible with the version of laravel you are using.

{
"name": "laravel/laravel",
"description": "The Laravel Framework.",
"keywords": ["framework", "laravel"],
"license": "MIT",
"type": "project",
"require": {
"laravel/framework": "5.0.",
"Illuminate/html": "^5.0",
"illuminate/html": "5.0.",
"doctrine/dbal": "^2.5",
"barryvdh/laravel-ide-helper": "^2.0",
"chrisbjr/api-guard": "^2.3",
"lucadegasperi/oauth2-server-laravel": "5.0.*"
},
"require-dev": {
	"phpunit/phpunit": "~4.0",
	"phpspec/phpspec": "~2.1"
},
"autoload": {
	"classmap": [
		"database"
	],
	"psr-4": {
		"App\\": "app/"
	}
},
"autoload-dev": {
	"classmap": [
		"tests/TestCase.php"
	]
},
"scripts": {
	"post-install-cmd": [
		"php artisan clear-compiled",
		"php artisan optimize"
	],
	"post-update-cmd": [
		"php artisan clear-compiled",
		"php artisan optimize"
	],
	"post-create-project-cmd": [
		"php -r \"copy('.env.example', '.env');\"",
		"php artisan key:generate"
	]
},
"config": {
	"preferred-install": "dist"
}

}

You need to check this.
You are trying to install the version for Laravel 5.1 you need
"lucadegasperi/oauth2-server-laravel": "4.0.*"

Thanks @ddelnano
It's working now. But i am bit confused. The table clearly says
Laravel 	OAuth Server 	PHP
5.1.x 	5.0.x 	>= 5.5.9
then why i need to use OAuth 4.0 with my Laravel 5.1?

You don't have Laravel 5.1 you have 5.0

Thanks for helping out @ddelnano!


Revert "Backporting NetBeans 23 patches"

7548 would be coming in NB-23.
Reverts oracle/javavscode#227

Not needed


'Select Azure DevOps Project' option gets stuck forever sometimes

'Select Azure DevOps Project' option gets stuck forever sometimes (snapshot below). The issue went away after restarting VSCode. Note that this has gone past the org selection - it was able to successfully list all orgs. So there are no issues with the VSCode instance.

There are 2 issues here:

Why the 'Select Azure DevOps Project' option was not listing the projects?
Even if for some reason the projects could not be listed, the UX should have timed out and showed some failure message to the user.

Closing as no repro


Issue Making the build work for Android (APK) - Docker for Windows issues.

Docker for Windows and Hyper V enabled
Currently I have Docker installed, have the git cloned and tried the commands.
I am getting various issues such as
`C:\Windows\system32>cd C:\UnityProjects\MediaPipeUnity
C:\UnityProjects\MediaPipeUnity>docker build -t mediapipe_unity:windows . -f docker/windows/x86_64/Dockerfile
[+] Building 0.2s (3/3) FINISHED
=> [internal] load build definition from Dockerfile                                                               0.0s
=> => transferring dockerfile: 3.57kB                                                                             0.0s
=> [internal] load .dockerignore                                                                                0.0s
=> => transferring context: 2B                                                                                0.0s
=> ERROR [internal] load metadata for mcr.microsoft.com/windows/servercore:ltsc2019                               0.2s

[internal] load metadata for mcr.microsoft.com/windows/servercore:ltsc2019:

failed to solve with frontend dockerfile.v0: failed to create LLB definition: no match for platform in manifest sha256:3fef9b0eac4a5d8b7d455dd07c3d8d3947657cc98bc48b0a02546ca5d782a4d5: not found`
Not exactly sure if I'm doing it correctly.

Please use a Linux image.
cf. https://github.com/homuler/MediaPipeUnityPlugin#android


Add section in readme to note it uses the aws-sdk-go and hence honors…

… the AWS_SDK_LOAD_CONFIG variable
Description of changes:
This is just adding mention to the readme about how to configure IAM roles for the daemon. This should be pretty straight forward to anyone looking at the repo and who has worked with the aws_sdk_go. This is mostly because other closed source documentation sources like this make mention of default credential resolutions, without making it clear that it is the go sdk default, not the broader default .
This method addresses the concerns in https://github.com/aws/aws-xray-daemon/pull/19
without changing the behavior of the daemon itself.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @defond0 ,
I have merged the PR. Since the Readme changes requested were nitpicks, I would do it myself.
Thanks.
Yogi


Update CONTRIBUTORS.md

Thanks for contributing to TVM!   Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from others in the community.

Sorry, you need to first contribute something then be added to the contributor list.


allow number format with leading zero

what i do:

i downloaded a section
i open VLC
i select "Advanced File open"
i selected the folder of a section

what i see:

when there are lectures with >9 numbers
then the lecture 10 start right after lection 1

what i expect

lectures in right order

how to fix:

allow lecture and section numbers formatte with leading zero:
1 --> 001
2 --> 002
10 --> 010

Yeah I have seen this numbering scheme implemented in some of the scripts which download Udemy courses. I never understood why they use leading zeros but now I know.
But if we generate a playlist per course and per section of course? Do you still need the leading zeros?
Thanks

i think some other apps (and the windows explorer ) not sort it right without a leeding zero.
so it is a low hanging fruit to make an option "add leading zeros" to get vlc work ;)

or maybe on windows by default on.
for #16 you should be aware that someone changed this setting while download sessions.

I think you are right, better to implement it as it wont hurt as anyway.

i totally agree with this naming schema =)

Maybe this issue can be joined with
[(https://github.com/FaisalUmair/udemy-downloader-gui/issues/317)]
In fact, my comment there is in the same line of what is written here.
[https://github.com/FaisalUmair/udemy-downloader-gui/issues/317#issuecomment-690767290]


JAMstack with Headless CMS is not supported

In the JAMstack build workflow, it is not possible to get dynamic content from an external API and generate a static site.
Specifically, if I set the repository_dispatch as the trigger for GitHub Actions<EMAIL_ADDRESS>will fail as shown in the screenshot. It's probably a JSON payload issue.

A typical scenario for JAMstack is to access an online Headless CMS like Contentful in the build workflow to generate static content.
Currently, Azure Static Web Apps doesn't exactly support JAMstack; GitHub Push or PR-based content updates are called SSG (Static Site Generate) rather than JAMstack.

Are you able to provide a GitHub repo as an example?

The repo that have validated this Issue is here,

https://github.com/k-miyake/vacd-jamstack

During Nuxt.js generate, it gets dynamic contents from Contentful API and generates static files.
As you can see in the workflow history below, it succeeds on the commit trigger, but fails on the repository dispatch(webhook) trigger.

https://github.com/k-miyake/vacd-jamstack/actions

Thank @k-miyake

@k-miyake I'll take a look at this, thanks for reporting it!

I have the same issue. When I trigger the GitHub Action workflow with the repository_dispatch event, it gets the error like the screenshot above. Both push and pull_request events are fine.

Yes this is a bug. The Build and Deploy Github Action looks for some event data provided by Github, but I believe the repository dispatch event does not provide this. We are working on removing this dependency but it will take some time. For now we can make this error clearer until we have the proper support in place.

@miwebst Thank you!
Many use cases rely on the repository_dispatch event such as manual deployment, scheduled deployment that is not covered by the schedule event, or approval process using the other approach - ChatOps or GitOps.
It would be great if this bug gets fixed sooner rather than later.

I'm also having the same problem with this. Would much appreciate for the fix.

This should be fixed now, please let us know if you continue to run into this issue!

@miwebst
The repository_dispatch trigger makes it possible for me to build and deploy successfully has been checked.
Here's the GitHub Actions log.
https://github.com/k-miyake/vacd-jamstack/runs/775400365?check_suite_focus=true
Thanks for the quick fix. Great job!


Varonisdsp: New App

Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[x] Testing of all the changes has been performed (for bug fixes / features)
[ ] The readme.html has been reviewed and added / updated if needed (for bug fixes / features)
[ ] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[ ] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[x] Verify all checks are passing.
[x] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[x] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[x] New App
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Security Considerations (REQUIRED)

[ ] If you are exposing any endpoints using a REST handler,
please document them in the readme.html.
[ ] If this is a new connector or you are adding new actions

[ ] Please document in the readme.html all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[ ] If any actions are unable to run on SOAR Cloud, please document this in the readme.html.

[NA] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[NA] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[NA] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)

Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.

What is the current behavior? (OPTIONAL)

Describe the current behavior that you are modifying.

What is the new behavior? (OPTIONAL)

Describe the behavior or changes that are being added by this PR.

Other information (OPTIONAL)

Any other information that is important to this PR such as screenshots of how the component looks before and after the change.

Pay close attention to (OPTIONAL)

Any specific code change or test case points which must be addressed/reviewed at the time of GA release.

Screenshots (if relevant)

Thanks for contributing!

Hello @vkorenkov-varonis , I saw you added min_phantom_version as <IP_ADDRESS>192, while according to our best practices, we are adding 5.4.0 as min_phantom_version. I hope you are ok with it.

@vkorenkov-varonis Is it fine for you if we change the min_phantom_version to 5.4.0 according to our standard practices?

@vkorenkov-varonis Is it fine for you if we change the min_phantom_version to 5.4.0 according to our standard practices?

it's ok for me


run_language_modeling.py

Hello, I am facing issues running the run_language_modeling.py script when running the example for pretraining Bio_clinicalbert using this line:
python3 run_language_modeling.py --output_dir ./models/clinicalBert-v1  --model_name_or_path  emilyalsentzer/Bio_ClinicalBERT  --mlm     --do_train     --learning_rate 5e-5     --max_steps 150000   --block_size 128   --save_steps 1000     --per_gpu_train_batch_size 32     --seed 42     --line_by_line      --train_data_file mimic_string.txt  --umls --config_name  config.json --med_document ./voc/vocab_updated.txt
issue 1 - it said the tokenizer did not have an argument called max_len. this was the error:
'AttributeError: 'BertTokenizerFast' object has no attribute 'max_len' '
Based on advice online, I updated It from 'tokenizer.max_len' to 'tokenizer.model_max_length' which seems to have resolved this issue
issue 2 - the current error message i am getting is:
'TypeError: init() got an unexpected keyword argument 'tui_ids''
When looking for answers to these online, I came across a comment on the huggingface transformers issue forum at https://github.com/huggingface/transformers/issues/8739
They said - 'It is actually due to #8604, where we removed several deprecated arguments. The run_language_modeling.py script is deprecated in favor of language-modeling/run_{clm, plm, mlm}.py.'
Does this apply to the scrpit for UmlsBERT as well? If so, how can I access the updated script? If not, how can I resolve the tui_ids issue?
I am running the scripts on google colab. This is the complete output I get when i run:
python3 run_language_modeling.py --output_dir ./models/clinicalBert-v1  --model_name_or_path  emilyalsentzer/Bio_ClinicalBERT  --mlm     --do_train     --learning_rate 5e-5     --max_steps 150000   --block_size 128   --save_steps 1000     --per_gpu_train_batch_size 32     --seed 42     --line_by_line      --train_data_file mimic_string.txt  --umls --config_name  config.json --med_document ./voc/vocab_updated.txt
output:
2021-07-05 09:47:55.207129: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcudart.so.11.0
07/05/2021 09:47:57 - WARNING - main -   Process rank: -1, device: cpu, n_gpu: 0, distributed training: False, 16-bits training: False
07/05/2021 09:47:57 - INFO - main -   Training/evaluation parameters TrainingArguments(
_n_gpu=0,
adafactor=False,
adam_beta1=0.9,
adam_beta2=0.999,
adam_epsilon=1e-08,
dataloader_drop_last=False,
dataloader_num_workers=0,
dataloader_pin_memory=True,
ddp_find_unused_parameters=None,
debug=[],
deepspeed=None,
disable_tqdm=False,
do_eval=False,
do_predict=False,
do_train=True,
eval_accumulation_steps=None,
eval_steps=500,
evaluation_strategy=IntervalStrategy.NO,
fp16=False,
fp16_backend=auto,
fp16_full_eval=False,
fp16_opt_level=O1,
gradient_accumulation_steps=1,
greater_is_better=None,
group_by_length=False,
ignore_data_skip=False,
label_names=None,
label_smoothing_factor=0.0,
learning_rate=5e-05,
length_column_name=length,
load_best_model_at_end=False,
local_rank=-1,
log_level=-1,
log_level_replica=-1,
log_on_each_node=True,
logging_dir=gdrive/My Drive/UmlsBERT-master/language-modeling/models/clinicalBert-v1/runs/Jul05_09-47-57_d7624bb0fdc5,
logging_first_step=False,
logging_steps=500,
logging_strategy=IntervalStrategy.STEPS,
lr_scheduler_type=SchedulerType.LINEAR,
max_grad_norm=1.0,
max_steps=150000,
metric_for_best_model=None,
mp_parameters=,
no_cuda=False,
num_train_epochs=3.0,
output_dir=gdrive/My Drive/UmlsBERT-master/language-modeling/models/clinicalBert-v1,
overwrite_output_dir=False,
past_index=-1,
per_device_eval_batch_size=8,
per_device_train_batch_size=8,
prediction_loss_only=False,
push_to_hub=False,
push_to_hub_model_id=clinicalBert-v1,
push_to_hub_organization=None,
push_to_hub_token=None,
remove_unused_columns=True,
report_to=['tensorboard'],
resume_from_checkpoint=None,
run_name=gdrive/My Drive/UmlsBERT-master/language-modeling/models/clinicalBert-v1,
save_on_each_node=False,
save_steps=1000,
save_strategy=IntervalStrategy.STEPS,
save_total_limit=None,
seed=42,
sharded_ddp=[],
skip_memory_metrics=True,
tpu_metrics_debug=False,
tpu_num_cores=None,
use_legacy_prediction_loop=False,
warmup_ratio=0.0,
warmup_steps=0,
weight_decay=0.0,
)
/usr/local/lib/python3.7/dist-packages/transformers/models/auto/modeling_auto.py:847: FutureWarning: The class AutoModelWithLMHead is deprecated and will be removed in a future version. Please use AutoModelForCausalLM for causal language models, AutoModelForMaskedLM for masked language models and AutoModelForSeq2SeqLM for encoder-decoder models.
FutureWarning,
Some weights of the model checkpoint at emilyalsentzer/Bio_ClinicalBERT were not used when initializing BertForMaskedLM: ['cls.seq_relationship.bias', 'cls.seq_relationship.weight']

This IS expected if you are initializing BertForMaskedLM from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).
This IS NOT expected if you are initializing BertForMaskedLM from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
Traceback (most recent call last):
File "gdrive/My Drive/UmlsBERT-master/language-modeling/run_language_modeling.py", line 355, in
main()
File "gdrive/My Drive/UmlsBERT-master/language-modeling/run_language_modeling.py", line 248, in main
tui_ids=tui_ids) if training_args.do_train else None
File "gdrive/My Drive/UmlsBERT-master/language-modeling/run_language_modeling.py", line 136, in get_dataset
tui_ids=tui_ids)
TypeError: init() got an unexpected keyword argument 'tui_ids'

P.S. I am not an expert programmer so do let me know if I should provide any further information as this is the first time i'm submitting an issue.
Thank you.
Best,
Jaya

Update: I have found the replacement files (run_clm.py, run_mlm.py, run_plm.py). When running the run_clm_no_trainer.py (since I'm using mimic data to train), I get this error:
ModuleNotFoundError: No module named 'datasets_modules.datasets.mimic_string'
Running on colab - This is the code:
!python3 'gdrive/My Drive/UmlsBERT-master/language-modeling/run_clm_no_trainer.py' --output_dir 'gdrive/My Drive/UmlsBERT-master/language-modeling/models/clinicalBert-v1'  --model_name_or_path  emilyalsentzer/Bio_ClinicalBERT   --learning_rate 5e-5     --block_size 128    --seed 42    --dataset_config_name  'gdrive/My Drive/UmlsBERT-master/language-modeling/config.json' --dataset_name 'gdrive/My Drive/UmlsBERT-master/language-modeling/mimic_string.txt'
Here is the full output:
2021-07-06 10:08:00.087779: I tensorflow/stream_executor/platform/default/dso_loader.cc:53] Successfully opened dynamic library libcudart.so.11.0
07/06/2021 10:08:01 - INFO - main - Distributed environment: NO
Num processes: 1
Process index: 0
Local process index: 0
Device: cuda
Use FP16 precision: False
Traceback (most recent call last):
File "gdrive/My Drive/UmlsBERT-master/language-modeling/run_clm_no_trainer.py", line 472, in
main()
File "gdrive/My Drive/UmlsBERT-master/language-modeling/run_clm_no_trainer.py", line 241, in main
raw_datasets = load_dataset(args.dataset_name, args.dataset_config_name)
File "/usr/local/lib/python3.7/dist-packages/datasets/load.py", line 838, in load_dataset
**config_kwargs,
File "/usr/local/lib/python3.7/dist-packages/datasets/load.py", line 687, in load_dataset_builder
builder_cls = import_main_class(module_path, dataset=True)
File "/usr/local/lib/python3.7/dist-packages/datasets/load.py", line 91, in import_main_class
module = importlib.import_module(module_path)
File "/usr/lib/python3.7/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 953, in _find_and_load_unlocked
File "", line 219, in _call_with_frames_removed
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 953, in _find_and_load_unlocked
File "", line 219, in _call_with_frames_removed
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 953, in _find_and_load_unlocked
File "", line 219, in _call_with_frames_removed
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 953, in _find_and_load_unlocked
File "", line 219, in _call_with_frames_removed
File "", line 1006, in _gcd_import
File "", line 983, in _find_and_load
File "", line 965, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'datasets_modules.datasets.mimic_string'


⚠ Bofrost.de has degraded performance

In 5a083a6, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1158 ms

Resolved: Bofrost.de performance has improved in aec7bbc after 1 hour, 11 minutes.


? Alone in the Dark: The New Nightmare ?

International v1.3.7 & Tonyhax 1.4.5
Disc One SLUS-01201
Triggered at boot up and Title Loading
Starts at raining building scene, goes away once voice starts inside building, skips even with fresh burnt disk at 8x then starts up again every time navigator starts talking
Looks alot like mod chip skipping in LOD title screen looks just like every other anti piracy skipping ive seen.
I Burnt 2 Disc trying get it to go away

Is this an anti-piracy screen that shows up or something else? I'm not sure what you mean by anti-piracy skipping

Burned this game at 4x speed to CMC Pro CD-R, no issues with sound or skipping on my SCPH-101.
I recommend you buy Verbatim DataLifePlus or CMC Pro discs, the regular Verbatim CD-Rs not branded as DataLifePlus are horrible quality which would result in drop outs in sound during an FMV like this game has when you start a new game.

Awesome, thanks! This is great thank you so much
On Thu, Aug 31, 2023 at 4:04 AM Alex @.***> wrote:

Hope this helps mate https://alex-free.github.io/psx-cdr
—
Reply to this email directly, view it on GitHub
https://github.com/alex-free/tonyhax/issues/40#issuecomment-1700557287,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASVSOG4JUYP22Q4O5VBNYXTXYBAQBANCNFSM6AAAAAA4AG5UXU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I have memorex/data life + the new ones I was so happy the way it handled "hart of darkness" opening FMV.  I'm happy to try something else, The quality hack dose not run correctly & ive been putting off playing that game until I find better discs.
I did have to crank my laser up quite a bit to get hart of darkness to run good which will burn it up sooner from my understanding. But even an original HoD disc was skipping before I turned resistance down.

Just a heads up that the verbatim discs in your picture are not data life plus, I'm pretty sure anyways. The burner itself can also be really important to make games work as good as CD-ROMs.
Be really careful with changing laser resistance. If you measured with a multimeter while doing that you already damaged the laser because of the electricity from the multimeter itself.
Changing the laser resistance is the last thing you should do. Before that following the republication guide on my site should be one of the first as in most cases that is the real issue anyways.

Thanks for the tip! but 99.9% of these disc work perfect alone in the dark
& SON (patched) are only issues.
On Sat, Sep 2, 2023, 10:57 AM Alex @.***> wrote:

I have memorex/data life + the new ones I was so happy the way it handled
"hart of darkness" opening FMV.  I'm happy to try something else, The quality hack dose not run
correctly & ive been putting off playing that game until I find better
discs.
I did have to crank my laser up quite a bit to get hart of darkness to run
good which will burn it up sooner from my understanding. But even an
original HoD disc was skipping before I turned resistance down.
[image: 3 types]
https://user-images.githubusercontent.com/78325531/265131418-d7a1f90c-7ac3-4505-8622-5b227944838a.jpg
Just a heads up that the verbatim discs in your picture are not data life
plus, I'm pretty sure anyways. The burner itself can also be really
important to make games work as good as CD-ROMs.
Be really careful with changing laser resistance. If you measured with a
multimeter while doing that you already damaged the laser because of the
electricity from the multimeter itself.
Changing the laser resistance is the last thing you should do. Before that
following the republication guide on my site should be one of the first as
in most cases that is the real issue anyways.
—
Reply to this email directly, view it on GitHub
https://github.com/alex-free/tonyhax/issues/40#issuecomment-1703856465,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASVSOG6SKIN42K7YHNL7KXTXYNCPJANCNFSM6AAAAAA4AG5UXU
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for the tip! but 99.9% of these disc work perfect alone in the dark & SON (patched) are only issues.
…
On Sat, Sep 2, 2023, 10:57 AM Alex @.> wrote: I have memorex/data life + the new ones I was so happy the way it handled "hart of darkness" opening FMV.  I'm happy to try something else, The quality hack dose not run correctly & ive been putting off playing that game until I find better discs. I did have to crank my laser up quite a bit to get hart of darkness to run good which will burn it up sooner from my understanding. But even an original HoD disc was skipping before I turned resistance down. [image: 3 types] https://user-images.githubusercontent.com/78325531/265131418-d7a1f90c-7ac3-4505-8622-5b227944838a.jpg Just a heads up that the verbatim discs in your picture are not data life plus, I'm pretty sure anyways. The burner itself can also be really important to make games work as good as CD-ROMs. Be really careful with changing laser resistance. If you measured with a multimeter while doing that you already damaged the laser because of the electricity from the multimeter itself. Changing the laser resistance is the last thing you should do. Before that following the republication guide on my site should be one of the first as in most cases that is the real issue anyways. — Reply to this email directly, view it on GitHub <#40 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASVSOG6SKIN42K7YHNL7KXTXYNCPJANCNFSM6AAAAAA4AG5UXU . You are receiving this because you authored the thread.Message ID: @.>

Alone in the dark seems to have a very performance critical FMV (it seems quite high quality, probably one of the best settings) and SOTN has tons of XA audio streaming which is super particular on the PSX. SOTN unpatched should perform just as SOTN patched does too.

No the next set of disc I get will defiantly be your recommended I wish I would have had this information you posted before I ordered these... Im going to try to uses them on games with no FMV & im just going to order some now.


Chrisda to Main

Docs Build status updates of commit a77bfcc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-EXOMobileDeviceStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5c3bef1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Connect-ExchangeOnline.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Connect-IPPSSession.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Disconnect-ExchangeOnline.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXOCasMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXOMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXOMailboxFolderPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXOMailboxFolderStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXOMailboxPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXOMailboxStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXOMobileDeviceStatistics.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXORecipient.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Get-EXORecipientPermission.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Matrix classes

Added basic classes for Matrix and Vector representation and basic operation with Accelerate support

The Matrix class is good, I've merged it in. I'm not sure the Vector class is necessary, since Swift's standard Array performs the same function and the operators can easily be overloaded.
Are there any good reasons to keep a custom Vector class? I can't think of any at the moment, so I'll probably remove it soon if no other suggestions are made.

Yeah, at the moment the Vector class has little overhead over a simple [Float] array.
I think we want to keep it if we want to add some particular functionality, for example if you create a Vector from a Matrix using m.column(0)  and you use the matrixView on that it would be nice to get back a Matrix of 1 column and n rows, while if you create it as m.row(0) you will get back a matrix of 1 row and m columns.
Let me know what you think about it, changing it to a [Float] should be easy.
Do you also think we need a === operator to check if 2 Matrix are equal?

Is there a reason why classes were opted for instead of structs?


Tests unexpectedly fail in Node 6

Do you want to request a feature or report a bug?
This might be a bug.
What is the current behavior?
I have a small set of tests that pass on Node 8, but fail on Node 6.
The tests have a beforeEach() that resets the state of two global properties. Each test then sets the properties to different values before making assertions. in Node 6 it seems that the beforeEach reset isn't being applied.
There's a basic demo repo here: https://github.com/m-allanson/jest-global-test
Which is based on the behaviour found here: https://github.com/gatsbyjs/gatsby/pull/3233
What is the expected behavior?
The tests should pass in both Node 8 and Node 6.
Jest version: 22.0.3
Yarn version: 1.3.2
Node 6 version: 6.12.2
Node 8 version: 8.9.3
OS: MacOS Sierra 10.12.6 (also observed in Travis CI's Trusty environment https://travis-ci.org/gatsbyjs/gatsby/builds/319991766)

IIRC this was a bug in Node 6 fixed in Node 8 while rewriting some VM specific logic. I can’t search for it currently because I’m on a shitty mobile connection. You can usually bypass it by setting these values to undefined

That's great 👍, thanks for the swift reply.


reCache example

From the docs it doesn't explain the fields necessary to recache a credit card. Do you have to include the card number or just the cvv? An example would be helpful.
Barring that, updated docs

@pixeldrew just the CVV! Spreedly already has the number stored so no need to recache that value. We'll see what we can do to make the docs clearer.


Updated docs and added functional tests.

Added group_vars symlink functional test along with docs.


need default nginx page with links to default resources

Default nginx index.html page with static links and information about each resource and how it is used:
jenkins
zafira
sonarqube
rabbitmq
STF
etc

Created #12


Proxy: Intermittent inbound_tcp test failure

running 5 tests
test outbound_times_out ... ignored
server h1 error: invalid HTTP version specified
ERROR:conduit_proxy::map_err: turning service error into 500: Inner(Upstream(Inner(Inner(Inner(Error { kind: Inner(Error { kind: Proto(FRAME_SIZE_ERROR) }) })))))
test outbound_uses_orig_dst_if_not_local_svc ... ok
test outbound_reconnects_if_controller_stream_ends ... ok
test outbound_asks_controller_api ... ok
test outbound_updates_newer_services ... ok

test result: ok. 4 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/telemetry-aa22a33a1ddc7e5f

running 5 tests
test records_latency_statistics ... ignored
test telemetry_report_errors_are_ignored ... ok
test inbound_aggregates_telemetry_over_several_requests ... ok
test http1_inbound_sends_telemetry ... ok
test inbound_sends_telemetry ... ok

test result: ok. 4 passed; 0 failed; 1 ignored; 0 measured; 0 filtered out

     Running target/debug/deps/transparency-ea2372ecb1a50674

running 12 tests
thread 'support server' panicked at 'assertion failed: `(left == right)`
  left: `[]`,
 right: `[99, 117, 115, 116, 111, 109, 32, 116, 99, 112, 32, 104, 101, 108, 108, 111]`', proxy/tests/transparency.rs:208:13
test http1_connect_not_supported ... ok
test inbound_tcp ... FAILED
ERROR:conduit_proxy::map_err: turning service error into 500: Inner(Upstream(Inner(Inner(Error { kind: Inner(Error { kind: Proto(INTERNAL_ERROR) }) }))))
test outbound_tcp ... ok
test http11_upgrade_not_supported ... ok
test tcp_with_no_orig_dst ... ok
test http10_with_host ... ok
test http11_absolute_uri_differs_from_host ... ok
test inbound_http1 ... ok
test http1_get_doesnt_add_transfer_encoding ... ok
test http10_without_host ... ok
test http1_removes_connection_headers ... ok
test outbound_http1 ... ok

failures:

---- inbound_tcp stdout ----
        thread 'inbound_tcp' panicked at 'read: Error { repr: Os { code: 104, message: "Connection reset by peer" } }', /checkout/src/libcore/result.rs:916:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.

failures:
    inbound_tcp

test result: FAILED. 11 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out

This was commit 381fb3800e1999912ebe2454f82bc1da5c03ba9f.

@briansmith I haven't seen this test fail, but I wonder what happens if you adjust the private connect timeout value?

@hawkw to reproduce this, i might try editing travis.yml (on your branch only) so that tests are run with for i in $(jot 100) ; do RUST_LOG=... cargo test -- inbound_tcp ; done (or something like that).  It seems much easier to reproduce these sorts of failures on CI hosts (presumably, because their hosts are frequently oversubscribed)

I believe @seanmonstar is currently looking into this and related issues.


Please update to Swift 5

Please update the project to Swift5 for github and cocoapods... getting these warnings:

Starscream/Compression.swift:59:25: 'withUnsafeBytes' is deprecated: use `withUnsafeBytes<R>(_: (UnsafeRawBufferPointer) throws -> R) rethrows -> R` instead

Starscream/Compression.swift:140:14: 'withUnsafeBytes' is deprecated: use `withUnsafeBytes<R>(_: (UnsafeRawBufferPointer) throws -> R) rethrows -> R` instead

Starscream/WebSocket.swift:282:30: 'withUnsafeMutableBytes' is deprecated: use `withUnsafeMutableBytes<R>(_: (UnsafeMutableRawBufferPointer) throws -> R) rethrows -> R` instead

Starscream/WebSocket.swift:1326:14: 'withUnsafeBytes' is deprecated: use `withUnsafeBytes<R>(_: (UnsafeRawBufferPointer) throws -> R) rethrows -> R` instead

Starscream/WebSocket.swift:1327:16: 'init(bytes:)' is deprecated: use `init(_:)` instead

This is resolved #653


firmware-updater: 0-unstable-2024-10-03 -> 0-unstable-2024-20-11

Things done
updated

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 364084

x86_64-linux

:white_check_mark: 3 packages built:

firmware-updater
firmware-updater.debug
firmware-updater.pubcache


Hopefully corrects/fixes WEBPA-2459

The file for this exists in etc/ directory.  It should be used in place and not built int.


Confused about auth code

I recently got td working with node through tdl, it asks me for phone number and then auth code.
I entered the phone number and it passed but then the auth code said something about SMS, I checked my phone for 5 minutes no such thing. Turns out I had to run the normal telegram desktop app during the state where my node app was between connect() and login().
Is this normal behavior? can I run the desktop app and my node app same time? do I have to do this auth code often? like for example on relaunching my node app or disconnects?
Also a bit confused on bots vs users, lets say I am a user member of 5 channels. I want to monitor those channels in my node app for new messages and simply log them. I want to run this on a server with minimal human interactions for example, should I be using bots or user mode? I am not sure if bots can even join channels?

You can do through TDLib exactly the same that you can do through any other Telegram client. You don't need to have access to other logged in devices to be able to log in or log in again after every restart.
In authorizationStateWaitCode you receive authenticationCodeInfo with exact information about the way the code was sent. Usually the code is sent to another recently active client first, in which case code_info.type is of type authenticationCodeTypeTelegramMessage. The field code_info.next_type contains information about the next method, which will be used for code delivery. It is usually of the type authenticationCodeTypeSms.
Bots can be added to channels only by their administrators. If you are an administrator of the channels, then you can use a bot, but it would be simpler to control the bot through the Telegram Bot API. Otherwise you can join them only as a user and need to handle updateNewMessage updates from the channels.

Thanks for the explanation! I just wanted to ask one more thing without opening a new issue. These are the updates I need to handle for my task I think?
updateNewMessage //New message comes even when not having a channel selected?
updateMessageContent //Do I need to handle this in the rare case when a channel admin edits the message?
updateFile //Do I need to handle this as well? or do I simply parse photo info from updateNewMessage?

Handling of updateNewMessage and updateMessageEdited+getMessage` should be enough for your needs.

Fantastic thanks :)


chore(deps): Update dependencies

I've collected the following renovate (secops) PRs into a single PR:

https://github.com/apollographql/apollo-ios-dev/pull/141
https://github.com/apollographql/apollo-ios-dev/pull/143
https://github.com/apollographql/apollo-ios-dev/pull/144
https://github.com/apollographql/apollo-ios-dev/pull/148
https://github.com/apollographql/apollo-ios-dev/pull/149
https://github.com/apollographql/apollo-ios-dev/pull/145
https://github.com/apollographql/apollo-ios-dev/pull/150
https://github.com/apollographql/apollo-ios-dev/pull/142

Merging this without an approval because I've approved all the PRs that went into this.


Add DatePiece class

DatePiece is a way to construct Date values when only one of [year,
month, day] is specified. The big-end non-supplied values get set
to the present and the little-end non-supplied values get set to the
smallest possible. For example: DatePiece('month', 3) gets compiled to
the Date corresponding to the first of April in the present year
(January is month 0). This is useful for disambiguating a reference
such as "the 11th" in the context of dates.
TODO: parser functionalities for DatePiece

Pull Request Test Coverage Report for Build 973

23 of 39 (58.97%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 80.284%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ast/values.js
7
8
87.5%

lib/nn-syntax/tonn_converter.js
1
2
50.0%

lib/prettyprint.js
1
2
50.0%

lib/describe.js
1
14
7.14%

Totals

Change from base Build 972:
-0.08%

Covered Lines:
8548

Relevant Lines:
10254

💛  - Coveralls

Also, currently the tokenizer recognizes "year + month" and "month + day" as a full date. What's the plan to address that? Are we changing the tokenizer instead?

, currently the tokenizer recognizes "year + month" and "month + day" as a full date. What's the plan to address that? Are we changing the tokenizer instead?

I think that'd make sense. For month+day I imagine the values shouldn't be dependent on the year the tokenizer is run. For year + month, I think it'd be nice to preserve the information that the utterance did not in fact contain a full date for as long as possible i.e. up until compilation to full date.
Thanks for all the comments! I'll get to work on those.


Implement TLS Channel ID protection

As the FIDO spec author pointed out in https://github.com/google/u2f-ref-code/issues/26#issuecomment-70691479, TLS Channel IDs can be used with U2F to protect against HTTPS MITMs, even where the MITM is using a valid certificate.
From my research, TLS Channel IDs need the server to care, and do the checking to detect that the channel ID that the U2F signed is the same one on the current TLS connection.
The problem for me is that I'm terminating HTTPS on nginx, not in the app where I have the U2F signature on hand. So it's incumbent on nginx to pass the TLS channel ID value back to the application, in an HTTP header or something. But searching around, nginx has no support for this that I can tell.
But Python 3 does, as of 3.3, so what I could potentially do is set up a tiny Python-based HTTP proxy, and have it receive traffic to /admin instead of nginx. But that's tricky too:

What protections am I losing by dropping nginx in front of /admin and using some tiny Python proxy?
I can't hand off just /admin to a different direction unless I perform TLS termination to discern the route. So I either put the Python proxy in front of everything, or I run it at some other port, or....I don't know.

I'm honestly not sure how to proceed. But I wish to proceed.

Apache implementation: http://code.metager.de/source/history/apache/httpd/modules/ssl/mod_ssl.h
Python 3 implementation: http://bugs.python.org/issue12551

The Apache implementation is at a low level, I'm not sure how I'd actually use it in an Apache config. No others in relevant languages/systems I can find. :(

Go implementation: https://codereview.appspot.com/117100043/

Though agl warns that Channel Bindings are broken (by the Triple Handshake Attack until the TLS Session Hash extension gets finalized with the IETF.
It appears Go proceeded without the extension for the time being.

HAProxy implementation, April 2014: http://git.1wt.eu/web?p=haproxy.git;a=commitdiff;h=c1ad52e8f781c301f6308d27f788354b77da1610

From the discussion, this is an example in HAProxy of using it:
http-request set-header X-TLS-UNIQUE-ID %{+Q}[ssl_fc_unique_id]

I don't know if I'm up for moving my whole server over to HAProxy, but this is an enticing reason to give it a shot.


Correct minor typo

@adrianwells : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@adrianwells - thanks for catching and fixing this typo!


JSON response for disabled parkings is incomplete

Found today that the response for disabled parkings is incomplete and returns the following data attributes incorrectly or not at all:

Longitude
Latitude
Strasse (is now strae)

Still a problem at the moment!


notes need to sync with notes when switching tours

Switch from one tour to another. The notes remain from the previous one.
Need to put obx on the list value?

Yup, making the notes as array of obx solved it


WIP: Migrate patient details to own table

What
Create a new table for patient details (patient_details) and migrate the the patient data from the scheduled_calls_table
to the new patient_details table. The scheduled_calls_table references the corresponding entry in the patient_details table via a foreign key to the id column of patient_details.
Why
The visits (scheduled_calls_table) table needs to be anonymous for the upcoming user journey tracking work.
Notes
As this db migration involved migration of data, migration tests have been added to check the data is moved across correctly. These tests run in their own job in the CI pipeline.

closing and re-opening to test heroku review app


Bug 2139479: virtualization overview crashes for non-priv user

Signed-off-by: Aviv Turgeman<EMAIL_ADDRESS>📝 Description

NamespaceBar component is causing issues for non-priv users, removed NamespaceBar component and injected via console-extensions

🎥 Demo
Before

After

/lgtm

/retest


WARNING: Cannot update utime of file

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
hello, my downloads retry a warning:
[Merger] Merging formats into "/downloads/TV Mundo Maior - Seu canal espírita #/[2017-11-21] - [Saúde no Terceiro Milênio ｜ RBN (21⧸11⧸2017)]/y-hy2_aogKA.mp4"
WARNING: Cannot update utime of file
Deleting original file /downloads/TV Mundo Maior - Seu canal espírita #/[2017-11-21] - [Saúde no Terceiro Milênio ｜ RBN (21⧸11⧸2017)]/y-hy2_aogKA.f247.webm (pass -k to keep)
Deleting original file /downloads/TV Mundo Maior - Seu canal espírita #/[2017-11-21] - [Saúde no Terceiro Milênio ｜ RBN (21⧸11⧸2017)]/y-hy2_aogKA.f251.webm (pass -k to keep)
[EmbedSubtitle] There aren't any subtitles to embed
[Metadata] Adding metadata to "/downloads/TV Mundo Maior - Seu canal espírita #/[2017-11-21] - [Saúde no Terceiro Milênio ｜ RBN (21⧸11⧸2017)]/y-hy2_aogKA.mp4"
WARNING: Cannot update utime of file
[ThumbnailsConvertor] Converting thumbnail "/downloads/TV Mundo Maior - Seu canal espírita #/[2017-11-21] - [Saúde no Terceiro Milênio ｜ RBN (21⧸11⧸2017)]/y-hy2_aogKA.webp" to png
WARNING: Cannot update utime of file
[EmbedThumbnail] mutagen: Adding thumbnail to "/downloads/TV Mundo Maior - Seu canal espírita #/[2017-11-21] - [Saúde no Terceiro Milênio ｜ RBN (21⧸11⧸2017)]/y-hy2_aogKA.mp4"
WARNING: Cannot update utime of file
How i can solve this warning?
Thank you.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

please add a full verbose log

Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
If using API, add 'verbose': True to YoutubeDL params instead
Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Make sure you are only asking a question and not reporting a bug or requesting a feature.
If your question contains "isn't working" or "can you add", this is most likely the wrong template.
If you are in doubt whether this is the right template, USE ANOTHER TEMPLATE!

Do not abuse the "Question" template to post incomplete issues/feature requests. At bae minimum, post a verbose log! Repeated abuse will be treated as spam


Feature Request - Route53 Integration ++ Security Improvments

Hello Guys.
Opening this Feature Request to ask you some stuff that help us to management the cluster and keep the stuffs safe.

It would be very interesting if LAMBDA that updates DynamoDB also updated/created a DNS entry in Route53, keeping the name of the Firewall Master always with the IP (private and public) updated. It would help us with cluster management.
The permission created for LAMBDA is S3 Full for all buckets. This generates several security issues in several cloud systems. It would be good to limit the permissions used by LAMBDA and limit the policy resource with the Bucket name/path.
The ENV VAR with the Firewall password is in plain text. It would be good to use AWS password vault for this.

Thanks in Advanced.

Hi @lucasaboud0,
Thank you for raising this issue. We will investigate and improve it.
Thanks,
Xing


[Sign Up Page] No error messages when entering too long First Name and Last Name

Environment: Windows 10 Pro Google Chrome Version 119.0.6045.201 (Official Build) (64-bit)
Reproducible: always
Preconditions
Site https://zero-waste-staging.onrender.com/en is opened
Steps to reproduce

Go to 'Sign Up' tab
Click on 'First Name' field and enter the following data: dddddddddAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAdvgggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggg
Click on 'Last Name' field and enter the following data: dddddddddAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAdvggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggt
Click 'Sign Up' button

Actual result
There are no error messages in regards of 'First Name' and 'Last Name' fields. The system shows that entered data is correct and text fields are highlighted in green color.
Expected result

An error mesasage 'First name is too long' appears in regards of 'First Name' field.
An error mesasage 'Last name is too long' appears in regards of 'Last Name' field.

User story and test case links
User story https://github.com/ita-social-projects/ZeroWaste/issues/626
Test case https://jira.softserve.academy/browse/SWQ-462

Developers Team and Tech Expert agreed the maximum for entering - 50 characters for First and Last Name fields.

https://github.com/ita-social-projects/ZeroWaste/issues/660


self.log is not a function

Please tell us about your environment:

winston version?

[ ] winston@2
[x] winston@3

node -v outputs: 14 and 15
Operating System? (Windows, macOS, or Linux) Linux and macOS
Language? (all | TypeScript X.X | ES6/7 | ES5 | Dart) TypeScript ES2019+

What is the problem?

self.log is not a function. I can only reproduce this occasionally in specific circumstances:

Using koa2-winston in a Koa app
Using several sets of proxies with the http-proxy library

What do you expect to happen instead?

self.log should always be defined
Other information

We've maintained a fork checking if self.log is defined here. I think this has something to do with lots of event listeners/emitters but I haven't been able to track that down. Would a PR be welcome to bring that change into upstream? Or, if this doesn't seem to be an issue with Winston, does anyone have any pointers?

Closing as this seems to be an issue with a different library.


Best way to record data

I am looking to record input and output to the vLLM. I could put an HTTP proxy in front and capture the traffic, or modify your handler.
Rather than make changes to the code, I was wondering if you might have a better way to do this?

We currently do not have a way to capture token usage across all workers, but it is a future item we will implement


Cannot start webgme on node6.11

webgme start
module.js:529
throw err;
^
Error: Cannot find module '../lib/BinUtils'
at Function.Module._resolveFilename (module.js:527:15)
at Function.Module._load (module.js:476:23)
at Module.require (module.js:568:17)
at require (internal/module.js:11:18)
at Object. (/usr/lib/node_modules/webgme-cli/bin/webgme:29:1)
at Module._compile (module.js:624:30)
at Object.Module._extensions..js (module.js:635:10)
at Module.load (module.js:545:32)
at tryModuleLoad (module.js:508:12)
at Function.Module._load (module.js:500:3)

How did you install webgme-cli? Did you use npm install -g webgme-cli?
On Sep 13, 2017 16:35, "Anirban Bhattacharjee"<EMAIL_ADDRESS>wrote:

webgme start
module.js:529
throw err;
^
Error: Cannot find module '../lib/BinUtils'
at Function.Module._resolveFilename (module.js:527:15)
at Function.Module._load (module.js:476:23)
at Module.require (module.js:568:17)
at require (internal/module.js:11:18)
at Object. (/usr/lib/node_modules/webgme-cli/bin/webgme:29:1)
at Module._compile (module.js:624:30)
at Object.Module._extensions..js (module.js:635:10)
at Module.load (module.js:545:32)
at tryModuleLoad (module.js:508:12)
at Function.Module._load (module.js:500:3)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fwebgme%2Fwebgme-cli%2Fissues%2F214&data=02|01|brian.broll%40vanderbilt.edu|fc895f1b695b4325d0d508d4fb19370e|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636409533100345135&sdata=4ZjF%2FIqVqOErp9f2AXhyGvwq9g%2Bg6HALbZphX9Rp%2FMk%3D&reserved=0,
or mute the thread
https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEwIBdBw3eJkclCMcDF9x-bUoLuLu7V6ks5siJDZgaJpZM4PW9_z&data=02|01|brian.broll%40vanderbilt.edu|fc895f1b695b4325d0d508d4fb19370e|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636409533100345135&sdata=a3kI%2Bw0WYLXqP1k4bX%2B9pBjdxTHzlNDSkTJejD0T5qg%3D&reserved=0
.

Yes..I did use npm install -g webgme-cli. npm start is working fine though.
But, I installed node differently.
curl -sL https://deb.nodesource.com/setup_6.x | sudo -E bash -
sudo apt-get install -y nodejs

Can you check the version with node -v and the version of npm with which npm? If you are installing node from source, you shouldn't need to install it with apt-get also (it could be causing problems).

I didn't do any apt-get install.
Command sequence:
curl -sL https://deb.nodesource.com/setup_6.x | sudo -E bash -
sudo apt-get install -y nodejs
npm install -g webgme-cli
webgme init
and got that error.

So, basically, the missing files are generated from the source files as the source for webgme-cli includes some ES6 which needs to be transpiled to ES5. They could be missing because the postinstall build failed (at least that is my suspicion) due to an old version of NodeJS being used to install it.
I think the problem is with the apt-get install in the second command. If you are installing nodejs from source by downloading the package, you shouldn't need to install it using apt-get as well. I suspect that the version of npm that you are using with npm could be using an old version of NodeJS.
Regardless, I would recommend removing the version of nodejs installed using apt-get and then removing and reinstalling webgme-cli (with only one version of nodejs installed). If you still have problems, can you post the output of the following commands with the next comment?
which node
which npm
which webgme

node -v
npm -v
webgme -v

And, of course, if there are any build errors when installing webgme-cli, those would be good to include, too!

I installed nodeJS as described in https://nodejs.org/en/download/package-manager/#debian-and-ubuntu-based-linux-distributions.
curl -sL https://deb.nodesource.com/setup_6.x | sudo -E bash -
This script run to prepare the nodeJS package and finally give the output ## Run apt-get install nodejs(as root) to install Node.js v6.x and npm
Then sudo apt-get install -y nodejs installs nodeJS and npm package.
And this is not nodejs/npm issue, because other things are working. Even npm start is working to start webgme server without any error.
I didn't get any build error, pasting below"
 npm install -g webgme-cli
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>..psst! While Bower is maintained, we recommend Yarn and Webpack for *new* front-end projects! Yarn's advantage is security and reliability, and Webpack's is support for both CommonJS and AMD projects. Currently there's no migration path, but please help to create it: https://github.com/bower/bower/issues/2467
npm WARN deprecated<EMAIL_ADDRESS>Use uuid module instead
/usr/bin/webgme -> /usr/lib/node_modules/webgme-cli/bin/webgme

><EMAIL_ADDRESS>postinstall /usr/lib/node_modules/webgme-cli
> node bin/install.js

Note: Autocomplete for bash can be enabled with:

        echo ". /usr/lib/node_modules/webgme-cli/extra/webgme.completion.bash" >> ~/.bashrc

npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/webgme-cli/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

+<EMAIL_ADDRESS>added 115 packages and updated 1 package in 32.799s

when I do webgme start/init/new etc I get that error.

# which node
/usr/bin/node
# which npm
/usr/bin/npm
# which webgme
/usr/bin/webgme
# node -v
v6.11.3
# npm -v
5.4.2
# webgme -v
module.js:471
    throw err;
    ^

Error: Cannot find module '../lib/BinUtils'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/usr/lib/node_modules/webgme-cli/bin/webgme:29:1)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)

@Anirban2404: We took care of this in person, right? If I remember correctly, we uninstalled the webgme-cli then reinstalled it making sure only node v6 was on the system, right?

Yes, I forgot to close the issue. Sorry.


[master]: Do not replace https with httpss

First of all, thank you for this useful little extension. It saved me a lot of time:)
Recently cloudflare has become https based, therefore the following happens:

console.inject('lodash');
undefined
VM3828:33 XHR finished loading: GET "https://api.cdnjs.com/libraries?search=lodash".getURLs @ VM3828:33console.inject @ VM3828:36(anonymous function) @ VM4021:2InjectedScript._evaluateOn @ VM3711:883InjectedScript._evaluateAndWrap @ VM3711:816InjectedScript.evaluate @ VM3711:682
VM3828:22 library injected from httpss://cdnjs.cloudflare.com/ajax/libs/lodash.js/3.9.3/lodash.min.js
VM3828:21 GET httpss://cdnjs.cloudflare.com/ajax/libs/lodash.js/3.9.3/lodash.min.js net::ERR_UNKNOWN_URL_SCHEME

I made a little fix for that.
Thanks,
D.

Thanks!
I only use this tool every now and then, really glad to see it's helping others out as well!


Remove tianon/true as the "nop" image

Expected Behavior
The "nop" image that Pipelines uses to stop Sidecars should be available.
Actual Behavior
It appears that the SHA for tianon/true, our current "nop" image, has changed in Docker Hub and results in errors like this in TaskRun Pod logs:
  Normal   Pulling    2s    kubelet, gke-sidecars-default-pool-2579a152-cb5d  Pulling image "tianon/true@sha256:183cb5fd54142948ad88cc986a217465cfe8786cfdd89b1ed1fc49825da413a7"
  Warning  Failed     2s    kubelet, gke-sidecars-default-pool-2579a152-cb5d  Failed to pull image "tianon/true@sha256:183cb5fd54142948ad88cc986a217465cfe8786cfdd89b1ed1fc49825da413a7": rpc error: code = Unknown desc = Error response from daemon: received unexpected HTTP status: 500 Internal Server Error
  Warning  Failed     2s    kubelet, gke-sidecars-default-pool-2579a152-cb5d  Error: ErrImagePull

Steps to Reproduce the Problem

Create a fresh cluster to make sure that the tianon/true image isn't cached on a Node
Deploy Tekton Pipelines
kubectl create -f ./examples/v1beta1/taskruns/sidecar-interp.yaml

Notice that the Pod underlying the TaskRun will fail with a ErrImagePull
Additional Info

Kubernetes version:

1.16.8

Tekton Pipeline version:

master branch, but also affects 0.12 and 0.11 releases.

Fixed by #2713 should be closed

I guess so too but, @sbwsg if not using tianon/true, which image would we use ? :thinking:

we could either bring back our pipeline-owned nop image, or go with a different strategy. i have been wondering what would happen on various platforms if we just replaced the image with a non-existent one, for example.


Add preset for street cabinets

Hi. It would be nice if there was a preset for man_made=street_cabinet. Plus, the street_cabinet=* tag with it. Another option would be adding a street_cabinet selector to the substation preset. That's the main usage for why I'd like to see it added.
Thanks

Looks like @quincylvania already added this in c9a579be


Use YouCompleteMe for vim and neovim

Hi!
I want to open this issue to propose we use YouCompleteMe for vim and neovim.
Why ?
In the last month I resolved to use deoplete after use YouCompleteMe for a lot of months, and I found a few points that YouCompleteMe is much better.
Folder Completion
YouCompleteMe uses the opened file path as base to propose the completion and deoplete use the  path that you started vim/neovim as base, that is a big difference between you import something wrong.
Small Docs
YouCompleteMe gives to you an small explanation about the function that you will use.
Performance ?
In one month using deoplete I didn't feel any difference in performance between it and YouCompleteMe.

Hey @renatoagds would you mind making a PR with this? I'd like to see what the change looks like.

@eanplatter sure! I'll do that asap. 👍

ternjs is what provides the signature in completion (screenshot of my setup with deoplete: http://screencast.com/t/I1sA4WRxHWQ)

deoplete has an option to set basepath for where file completion starts from: https://github.com/Shougo/deoplete.nvim/blob/master/doc/deoplete.txt#L324
this should really be a personal preference, you can always :cd to where you want to be

@davidosomething nice! so, I think we can set deoplete#file#enable_buffer_path as 1 by default, IMO.


Rename references to CFME

This expands on #63.  There are a number of files named cfme.  In addition, the sample postgres conf files are named Evm_V5...
@abellotti can we rename the external auth files?

Sure, See concerns in https://github.com/ManageIQ/manageiq-appliance/pull/62 comment. Thanks.

Looks like they are all gone :tada:


Some 2.11.0 URLs redirect unhelpfully

Summary
I upgraded from 2.9/4 to 2.11.0. I selected an old URL from my browser autosuggest (localhost:2746/cron-workflows) and it was redirected to localhost:2746/cron-workflows/undefined, and showed no Cron Workflows.
Same issue affects localhost:2746/workflow-templates
Diagnostics
Argo v 2.11.0

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Maybe fixed by Fixes #4019 (#4021)

@brabster to understand the cause of this issue, can you please tell me:

Are you using a namespace of cluster scope install?
What happens if you open the page in "private browsing" mode?
What is in localStorage.getItem("current_namespace") return.
What happens if you clear local storage?

You should be able to work around this by:

Changing the URL.
Entering and clearing the namespace control.

Hi Alex.
Yep, we can work around it easily thanks.

Are you using a namespace of cluster scope install?

I install to the argo namespace - if that's what you mean?

What happens if you open the page in "private browsing" mode?

Same thing happens

What is in localStorage.getItem("current_namespace") return

null

What happens if you clear local storage?

there's nothing in local storage to clear

Are you using a namespace of cluster scope instal

Do you run the server using --namespaced?

I'm just using the install manifest as-is https://raw.githubusercontent.com/argoproj/argo/v2.11.0/manifests/install.yaml, so not using the --namespaced flag as far as I can tell

You can try argoproj/argocli:latest to see if the fix works for you.

Fixed AFAIK.

Same issue for workflow-template page in v2.11.

Issue still present in v2.11.3


Is this library still required?

referring to your comment:
https://github.com/vuejs/vue-class-component/issues/49

Please use the official ts-loader. Vue support has been merged. https://github.com/TypeStrong/ts-loader#appendtssuffixto-regexp-default

However, this library is still included in your example:
https://github.com/HerringtonDarkholme/vue-ts-example
Thanks

Is this library still required?

No, you just need ts-loader.

thanks, so in terms of setup, very close to vue-class-component, basically the same libraries with av-ts or vue-class-component.


pybamm fails when running script a second time in spyder on windows

Describe the bug
Running any script more than once seems to result in an import error because of the recent changes to parameters
File "", line 908, in _find_spec
AttributeError: 'ImportDenier' object has no attribute 'find_spec'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\tom\code\PyBaMM\examples\scripts\drive_cycle.py", line 14, in
param = model.default_parameter_values
File "c:\users\tom\code\pybamm\pybamm\models\full_battery_models\base_battery_model.py", line 346, in default_parameter_values
return pybamm.ParameterValues(chemistry=pybamm.parameter_sets.Marquis2019)
File "c:\users\tom\code\pybamm\pybamm\parameters\parameter_values.py", line 73, in init
self.update_from_chemistry(chemistry)
File "c:\users\tom\code\pybamm\pybamm\parameters\parameter_values.py", line 216, in update_from_chemistry
self.update(
File "c:\users\tom\code\pybamm\pybamm\parameters\parameter_values.py", line 301, in update
loaded_value = pybamm.load_function(os.path.join(path, value[10:]))
File "c:\users\tom\code\pybamm\pybamm\util.py", line 273, in load_function
module_object = importlib.import_module(path)
File "C:\Users\tom\anaconda3\lib\importlib_init_.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 961, in _find_and_load_unlocked
File "", line 219, in _call_with_frames_removed
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 971, in _find_and_load_unlocked
File "", line 910, in _find_spec
File "", line 884, in _find_spec_legacy
File "C:\Users\tom\anaconda3\lib\site-packages\IPython\external\qt_loaders.py", line 48, in find_module
if path:
File "", line 1183, in len
File "", line 1162, in _recalculate
File "", line 1158, in _get_parent_path
KeyError: 'pybamm.input.parameters.lithium_ion'

I'm using spyder on windows - running any script twice without restarting the kernel results in the above error. Still happening - any ideas?

Seems to be ok when I run example scripts from the pybamm directory but not scripts from another folder

Think I fixed it by putting empty init files into all the parameter subfolders


Adding Walkthrough on NodeJS for PBIE

NodeJS Walkthrough for PBIE

@romitgirdhar,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Added code samples for Java & .NET versions 4.0 and below.


Carriage returns in the text output

I have the recommended option implemented to include linefeeds but in sacredboard I still see very verbose logging for progress bars in the captured output.
I'm using tqdm if anyone else has run into this problem before.

Hi, I believe this may be rather an issue with Sacred.
May I ask you to give me an example of what you expect and what you actually get?
Also, an export of the run from your Mongo database would be helpful. Thanks.

Thanks for getting back so quick. I will get back to you once i figure out more about the issue and if it's an underlying sacred thing or sacredboard. You might be right that it's a problem with sacred.

Closing as probably not related to sacredboard itself.


VZ-10530: Build Dex v2.37.0 from source using OEL8  base image

Builds Dex v2.37.0 from source using OEL8 base image

@lake-of-dreams The dependency is there in the upstream Dockerfile, for the templated configuration file used by the entry point. I have removed that dependency now, by including config.dev.yaml.


Link in TOC does not work

Description
I click on section 6 in the TOC and get to https://arxiv.org/html/2401.07564v3/
This time additional data autfilled. Hence resubmitting instead of https://github.com/arXiv/html_feedback/issues/671 .
I also reported the autofill that sometimes works and sometimes not. https://arxiv-org.atlassian.net/servicedesk/customer/portal/1/AH-92007
(Optional:) Please add any files, screenshots, or other information here.
No response
(Required) What is this issue most closely related to? Select one.
Other
Internal issue ID
c04c2c0f-508b-48b3-858a-e290b428b030
Paper URL
https://arxiv.org/html/2401.07564v3
Browser
Firefox/113.0
Device Type
Desktop

Just to say I am seeing the same in https://arxiv.org/html/2312.04273v3. The links add a / so instead of just adding the URI fragment it is a new page, e.g. https://arxiv.org/html/2312.04273v3/#S2

Oh, right, @ericphanson . It's as simple as removing the / . I hope they quickly find where it's added by the code.


So twitter is still useing this lib?

or not?

We just open sourced this last week.  Absolutely Twitter is using TNL.  It is core to the Twitter for iOS app and will be for the foreseeable future.


Added SQLite package into depends

NOTE: Depends builds just needs to pass, the sqlite package is not actually used yet, this is just prep for my next PR for the wallet SQLite support

Build for this seems to be done, but link not posted by bot: https://build.nav.community/binaries/mxaddict%3Aupstream-depends-osx-sdk-12.1/


Change processing of the field 'second name' in the endpoint 'update party'

I
as a Role = OWNER or Role = HR
want to change second name in the Object = party
When
endpoint update party is required.
If update party.second name is null set party. second name as null
Example:
"party": {
"id": "d290f1ee",
"first_name": "Гохар",
"last_name": "Алі-Шах",
"second_name": null,
"birth_date": "1991-08-19T00:00:00.000Z",
"gender": "MALE",
"no_tax_id": false,
"tax_id": "3126509816",
"documents": [
{
"type": "PASSPORT",
"number": "АА120518",
"issued_by": "Рокитнянським РВ ГУ МВС Київської області",
"issued_at": "2017-02-28",
"expiration_date": "2027-02-28"
}
],
Attentin: this change must be included into next release and implemented into prod instance in January, 2019

[ ] error on create legal_entity.
On create LE we also create an employee request, so we also should have a possibility to reset second name for a OWNER

"error": {
        "invalid": [
            {
                "entry": "$.owner.second_name",
                "entry_type": "json_data_property",
                "rules": [
                    {
                        "description": "type mismatch. Expected String but got Null",
                        "params": [
                            "string"
                        ],
                        "rule": "cast"
                    }
                ]
            }
        ]


[networking] Implement hostname resolution + Multiaddress support

We currently parse socketaddrs out of our configuration.
We have plans to:

support table stakes host name resolution (akin to https://github.com/MystenLabs/sui/pull/1595)
support  the multiaddr format for

contacting a Narwhal node (i.e. this is what should be in the configuration file)
keeping the address of a node around  (rather than the SocketAddr elements we have now).

the whole point of multiaddr is to support multiple network endpoints, but for now we're not trying to exploit that capability (except maybe in retries). Any reasonable retry policy (in order, at random) should be ok.

A byproduct of multiaddresses is that for a single http address, this is backwards-compatible with an http endpoint.

Addressed in https://github.com/MystenLabs/narwhal/pull/220


feature: add join reorder rule.

Proposed changes
Issue Number: close #xxx
Problem summary
Add join reorder rule.

JoinExchange
JoinLeftAssociate
JoinRightAssociate
SemiJoinTranspose

Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[x] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

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

out of date


migrate dev errors with The underlying table for model _prisma_migrations does not exist. when using a non default PostgreSQL schema

Bug description
Here the project is using a non default PostgreSQL schema error-handling-prod (i.e. not public)
npx prisma migrate dev errors with
Environment variables loaded from prisma/.env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database "mydb123", schema "error-handling-prod" at "localhost:5432"

PostgreSQL database mydb123 created at localhost:5432

Applying migration `20220510114355_baseline`
Error: P1014

The underlying table for model `_prisma_migrations` does not exist.

Screenshots from TablePlus, it seems the _prisma_migrations was created in the error-handling-prod schema but Prisma Migrate tries to find it in the public schema where it's missing maybe?

How to reproduce
Clone https://github.com/Jolg42/repro-13269

yarn
check .env and add a valid connection string
yarn prisma migrate dev

Expected behavior
Should work
Environment & setup

OS: macOS
Database: PostgreSQL
Node.js version: 16.10

Prisma Version
3.14.0-dev.60

Can you share the contents of the migrations directory?

@tomhoule see full repro at https://github.com/Jolg42/repro-13269

The baselining using pg_dumo was successful after commenting out the following line from the migration.sql file
SELECT pg_catalog.set_config('search_path', '', false);