youtube_dl is not part of the bundle. Add it to Gemfile. (Gem::LoadError)

When calling:
YoutubeDL.download "https://www.youtube.com/watch?v=gvdf5n-zI14", output: 'some_file.mp4'
I get this error:
/Users/jackson/.rbenv/versions/2.2.4/lib/ruby/gems/2.2.0/gems/bundler-1.11.2/lib/bundler/rubygems_integration.rb:304:in block in replace_gem: youtube_dl is not part of the bundle. Add it to Gemfile. (Gem::LoadError)
I previously had 'youtube_dl' in my gemfile, but replaced with 'youtube-dl.rb' and bundle install, bundle update.
Any idea why I'm getting this error message?

You have an outdated Gemfile.lock. You can just delete Gemfile.lock and regenerate using bundle install. Try that and see if that fixes anything.

Hmm.. No luck. Here's the trace:
from /Users/jackson/.rbenv/versions/2.2.4/bin/youtube-dl:22:in `<main>'
from /Users/jackson/.rbenv/versions/2.2.4/lib/ruby/gems/2.2.0/gems/cocaine-0.5.8/lib/cocaine/command_line.rb:91:in `run'

Ahh, I see the problem: youtube_dl is still installed, which causes a conflict with this gem when searching for a usable youtube-dl binary. See Issue #24.

Oh, weird. Any idea on how to uninstall it? I thought removing it from gemfile was enough.. no?

gem uninstall youtube_dl or use a different gemset.
This gem, youtube-dl.rb, searches for a youtube-dl binary in your PATH before falling back to the built in youtube-dl. But the youtube_dl gem includes a binary named youtube-dl which from what I can tell is basically a really old version of youtube-dl. When youtube_dl (the gem) is installed, rbenv will make a Ruby file called youtube-dl to correspond with the executable youtube_dl (the gem) includes, which basically figures out the Ruby environment you are using at the moment and redirects it to the appropriate directory. That's where the Gem::LoadError is coming from.
It's really complicated, and it really sucks, but there's nothing I can really do about it.

Ah, confusing but makes sense. Thanks for the explanation. Successfully removed youtube_dl, uninstalled youtube-dl.rb and re-installed from scratch.
Same use case, I'm getting this error:
Cocaine::CommandNotFoundError: Cocaine::CommandNotFoundError
from /Users/jackson/.rbenv/versions/2.2.4/lib/ruby/gems/2.2.0/gems/cocaine-0.5.8/lib/cocaine/command_line.rb:82:in `run'
Sorry to keep beating at this, really appreciate your help.

I've traced my error back to line 39 in youtube-dl.rb/lib/youtube-dl/video.rb
set_information_from_json(YoutubeDL::Runner.new(url, @download_options).run)
causes the error:

Cocaine::CommandNotFoundError: Cocaine::CommandNotFoundError

In support.rb line 30:
line = Cocaine::CommandLine.new(executable_path, command)
executable_path == "/Users/birchlore/.rbenv/shims/youtube-dl"
so my line.command looks like  ""/Users/birchlore/.rbenv/shims/youtube-dl --no-color --no-progress --print-json --format :format --prefer-ffmpeg --extract-audio --audio-format :audio_format \"https://www.youtube.com/watch?v=VA6kCaxaTlk\""
and then line.run throws the error.

Update. Issue resolved by running brew install youtube-dl. Not sure why I had to install it separately.

This is a very weird situation, but I applaud your persistence. I'll keep this issue open since it is technically a bug, and we might have to call in the rbenv guys to figure it out.
Anyway, thanks for bringing my attention to an issue!


[Backport 2.x] [Segment Replication] Fix bug where replica shows stale doc count during engine reset. (#9495)

Manual backport of (#9495).

Gradle Check (Jenkins) Run Completed with:

RESULT: FAILURE ❌
URL: https://build.ci.opensearch.org/job/gradle-check/23672/
CommitID: 1a7df60
Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
Is the failure a flaky test unrelated to your change?

https://github.com/opensearch-project/OpenSearch/issues/9580


fix help error

Codecov Report

Merging #73 into release-rc4 will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@                Coverage Diff                @@
##             release-rc4      #73      +/-   ##
=================================================
- Coverage          59.24%   59.22%   -0.02%
  Complexity           347      347
=================================================
  Files                 37       37
  Lines               3050     3051       +1
  Branches             228      228
=================================================
  Hits                1807     1807
- Misses              1077     1078       +1
  Partials             166      166

Impacted Files
Coverage Δ
Complexity Δ

...ss/stub/fabric/proxy/ProxyChaincodeDeployment.java
44.55% <0.00%> (-0.24%)
5.00 <0.00> (ø)

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 3883983...f98490f. Read the comment docs.


fix(Remindme): removed NLP and use Klasa's argument parser instead

can you also add a show action to reminders to show the full reminders text?

Done in d453c77.


Use the title text style for the back button text.

The back text should match the Header title. While it was possible to set custom styles typeface (i.e. typeface) for the title, the back button text was impossible to style.
Test plan (required)
Add the following after line 70 of examples/NavigationPlayground/js/SimpleStack.js:
    headerTitleStyle: {
      textDecorationLine: 'line-through',
      letterSpacing: 2,
      fontWeight: '500',
    },

You should then have something a little like this:

This is already being tracked in #1083


Added first version of normalize_path_middleware

What do these changes do?
Adds a normalize_path_middleware middleware that deals with trailing slashes and double slashes in path.
As a side effect, also Request.clone() was implemented.
Are there changes in behavior for the user?
Users will now be able to do:
from aiohttp import web
from aiohttp.middlewares import normalize_path_middleware
app = web.Application(middlewares=[normalize_path_middleware])

Related issue number
#355
Checklist

[x] I think the code is well written
[x] Unit tests for the changes exist
[x] Documentation reflects the changes

Extra checklist

[ ] Tests for Request.clone
[x] Add args to middleware for append_slash and merge_slashes

I'm adding the changes to look for feedback. Right now the test for the append_slash is working. I'll start now with the double slashes one.

No, request.clone() shouldn't accept RawRequestMessage -- the later lies on very low abstraction level, much lower than all aiohttp.web operations.
The signature should be like this:
request.clone(*, method=_sentinel, path=_sentinel, ...)
Not needed to provide changing all request's attrs, let's limit ourself to obvious and most useful ones.
Overrides should be keyword-only.

Current coverage is 98.30% (diff: 100%)

Merging #1118 into master will increase coverage by 0.01%

@@             master      #1118   diff @@
==========================================
  Files            28         29     +1
  Lines          6474       6491    +17
  Methods           0          0
  Messages          0          0
  Branches       1085       1087     +2
==========================================
+ Hits           6363       6381    +18
  Misses           61         61
+ Partials         50         49     -1

Powered by Codecov. Last update 671617d...8f8b974

Looks pretty good but please convert middlewares.normalize_path_middleware into a factory.
I have a guts feeling: sometimes we'll need to add some parameters to the middleware, e.g.:
app.middlewares.append(middlewares.normalize_path_middleware(add_trailing=True, remove_trailing=False))
All params should be keyword-only arguments.
Right now it's not an issue but please respect my guts :)
Also middlewares.normalize_path_middleware looks redundant, just middlewares.normalize_path should be ok.

Yeah! In my current branch I have it as a factory but haven't commited it yet until I have it working with both args (merge_slashes and append_slash).
Ok with keyword only and Ok with renaming :).

With this last commit functionality is covered. Anyway, in case it is ready to merge in your opinion, I would like to keep it until tomorrow night to see if I can find a more readable way to express it.
A bit off topic but that came while working with this merge request:

Would it make sense to open a new issue for implementing an alternative router.resolves? Current implementation passing the Request object and then when return having to check with isinstance IMHO is not clean. It would be better to just send method and path (as you already started in your pull request) and then agree on some syntax to make it easier to check if resolved or not.

Every time a request is resolved, the request._match_info has to be filled automatically. Would it make sense to do that inside the resolve method?

Finished. We still have a caveat that I don't know how (or if) you want to address it.
Suppose the user has the following route defined: /resource.
If the user tries to access to /resource/, it will return a 404 no matter if the middleware is active or not. Right now we only deal with cases where we want to append trailing slash.

I think this would also solve #682

Rename the module to web_middlewares,  import it in web.py and make accessible in web.py's __all__.
The following should work:
from aiohttp import web
web.normalize_path_middleware

Is the clone function really necessary? As it may impose a performance issue if the server is under a huge amount of load. In my opinion a better approach would be to provide an overload function for UrlDispatcher::resolve, so it basically accepts (method, raw_path) which can be obtained by calling the appropriate member functions on a request object (Or we have to manually iterate over resources and call resolve which is again better than actually cloning the whole request object).
Also I am wondering if it's actually better to directly integrate this feature in aiohttp's core (One can enable this feature by a flag), so we can actually drop returning HTTPMovedPermanently.
For example the expected behavior for this would be:
A request something like this Req { method="GET", path="///account//info/" } would be normalized to Req { method="GET", path="/account/info/" }, then we will actually try to find an appropriate handler, this way we are avoiding returning a HTTPMovedPermanently.
Again, I think we we should do the normalization as early as possible to elimination possible performance (More like we are being repetitive).

I was working on an temporary approach (More like a hack) where I utilize a middleware which does the normalization then crafts a temporary object where raw_path and method are stored. Instead of returning a HTTPMovedPermanently, the middleware is actually trying to call the appropriate handler. This is the time where I noticed something, this approach might not work with expect_handlers because middlewares aren't applied here (https://github.com/KeepSafe/aiohttp/blob/master/aiohttp/web.py#L81), so I guess this won't work without actually changing internal code.

@cynecx I have this MR a bit abandoned because Im spending my time with other stuff right now. About what you comment, I remember my first approach trying what you said about not using the HTTPMovedPermanently but I found out something (maybe same as you) that made me go with the redirect.
On the other hand, I totally agree that an alternative resolve method where only the method and path are sent should exist.

.clone() is implemented by #1361

Wow looks interresting be nice if this PR could be updated so it can be Merged.

This is a useful middleware. And if this gets integrated, I could get rid of my currently-broken implementation in aiohttp_utils. 👍

I've upgraded the branch to resolve conflicts and apply some comments from @asvetlov. I'm having some problems from 5 test cases that didn't have before:
self = <aiohttp.test_utils.TestServer object at 0x7f1108c474a8>, path = '//resource2//a//b/'

    def make_url(self, path):
        url = URL(path)
>       assert not url.is_absolute()
E       assert not True
E        +  where True = <bound method URL.is_absolute of URL('//resource2//a//b/')>()
E        +    where <bound method URL.is_absolute of URL('//resource2//a//b/')> = URL('//resource2//a//b/').is_absolute

I can change the test cases and instead of //resource2//a//b/ use ///resource2//a//b/ but feels like cheating.
Also, I'm missing the documentation in web_reference.rst. Do you want me to copy paste the docstring there or include it or?

Create new section in web_reference "Middlewares", and add you doctoring.

why '//resource2//a//b/' this test fails?

 This is a behavior that was introduced with yarl because I don't remember this happening before.

i see. is it possible to skip make_url call with some param and construct it manually?

I have no idea, the code for the make_url is
    def make_url(self, path):
        url = URL(path)
        assert not url.is_absolute()
        return self._root.join(url)

which doesn't accept extra parameter. I believe it could be a yarl bug (or feature). Out of scope of this MR IMO.

ok, i will fix that

@argaen i added tests that you deleted. some of them actually failing.
https://github.com/KeepSafe/aiohttp/tree/normalize-middleware-fixes

@argaen i need you to review failing tests

#1583


CMake Error at src/CMakeLists.txt:136 (target_link_libraries):

请教 这个是我环境有问题还是？
Build FluentUI demo applications.: ON
Build static library.: OFF
Path to FluentUI plugin.: D:/Qt/6.6.0/msvc2019_64/qml/FluentUI
-- Configuring done
CMake Error at src/CMakeLists.txt:136 (target_link_libraries):
Target "fluentuiplugin" links to:
FramelessHelper::Core

but the target was not found.  Possible reasons include:
* There is a typo in the target name.
* A find_package call is missing for an IMPORTED target.
* An ALIAS target is missing.

上面那个是用命令的
我用qt creator 11 打开CMakeLists.txt 就变成这样了

最新git下 拉下来  framelesshelper中没有内容

git clone --recursive https://github.com/zhuzichu520/FluentUI.git  克隆代码的时候加个参数--recursive

大意了， 没看到git中还有另两个git仓库

多谢多谢哈


Add GitHub Actions workflow for release builds

Better late than never (#4490), I finally worked on a GitHub Actions workflow to build scrcpy releases.
I had to refactor the release.mk makefile to parallelize server and client build jobs. It is only run manually to build a release (not on push or pull request, I don't need it for now).

In the end, it produces a zip artifact containing:

scrcpy-server-XXX
scrcpy-win32-XXX.zip
scrcpy-win64-XXX.zip
SHA256SUMS.txt

where XXX is the input name you give when you manually start the workflow (if empty, it uses the ref name, for example the branch or tag name).
To test it:

fork the repo to your own account
checkout the branch from this PR, and push it to your master branch
enable GitHub Actions in your repo settings
go to the Actions tab
select "Build"
select "Run workflow"
keep master branch, choose a version name (for example v42)
click on Run workflow

For now, it generates the same targets as ./release.sh did locally (scrcpy-server, release win32, release win64).
The goal now is to add more jobs, especially:

macOS releases
.deb packages for Ubuntu and Debian recent versions (#4427)

Your help is welcome! :heart:
Now that scrcpy-server is generated separately once and for all, it will be easier to write the jobs for building the scrcpy client for different targets (Android SDK is not needed).
As an important constraint, the jobs MUST only reference "official" github actions (or dockers images), not actions/images from random users. Keep it as simple as possible.

Fixes #4490
Refs https://github.com/Genymobile/scrcpy/issues/2256#issuecomment-1218375121 and next comments (cc @dur-randir @LeeBinder @Coool)
Refs #1709 (cc @dylanmtaylor)
Refs #3721 (cc @qmfrederik)
Refs #4427 (cc @hacksysteam)
Refs #4489 (cc @Gary-Cod)

Awesome, great work, Romain. I'm recovering health-wise right now which might take a while so will be mostly offline for the near future :/

I've added macos build job + modified makefile in my fork at https://github.com/dur-randir/scrcpy/commit/6c4c2c33b34d028de4227f6e5c93b6ad57b610fe. The next question is, this generates unsigned binaries - e.g. you either have toprovide a signing key (see https://docs.github.com/en/actions/use-cases-and-examples/deploying/installing-an-apple-certificate-on-macos-runners-for-xcode-development) or a user have to perform additional steps after download (like brew does automatically when installing from casks).

I've added macos build job + modified makefile in my fork at dur-randir@6c4c2c3. The next question is, this generates unsigned binaries - e.g. you either have to provide a signing key (see https://docs.github.com/en/actions/use-cases-and-examples/deploying/installing-an-apple-certificate-on-macos-runners-for-xcode-development) or a user have to perform additional steps after download (like brew does automatically when installing from casks).

amd64 aka x86_64 build actions config ? I can't see them there.

@dur-randir Thank you very much!
A few questions/remarks:

Could app/deps/ffmpeg-macos.sh be "merged" into app/defs/ffmpeg.sh to avoid duplication (a common file for all platforms)?
It seems only FFmpeg is built for macOS, not the other dependencies (SDL and libusb)? (and an adb binary for macOS)
Is macpack absolutely necessary? Can we build a release without this tool?

I'll read about signing keys later.

Could app/deps/ffmpeg-macos.sh be "merged" into app/defs/ffmpeg.sh to avoid duplication (a common file for all platforms)?

Sure, I've done it separately because of didn't know in advance what'd be needed to be changed. There's dievergence just in two places - additional args list for configure disable-debug..enable-videotoolbox and export PATH, which can be put under if $HOST in deps/common.

It seems only FFmpeg is built for macOS, not the other dependencies (SDL and libusb)? (and an adb binary for macOS)

They're taken from brew install in binary form.

Is macpack absolutely necessary? Can we build a release without this tool?

Yes and no. I've tried (again!) to make a fully static build (link ffmpeg + libusb + libsdl) but I don't know meson at all and it broke somewhere down the road while double-linking ssh$whatever while I was trying to shove them all together and then I ceased trying. Without static linking with those dependencies, one must relocate all libraries to be searched at relative paths by DYLD (they're baked by absolute paths by default on macos). IIRC this can be done with native instruments, but they're non-recursive, so I've picked one tool that works recursively out-of-the box. Another one tool is https://github.com/auriamg/macdylibbundler which can be installed from brew. Or create this recursive hand-written loop for path change. Or ideally make a fully static build, but I've given up. If you know meson, maybe you can create a recipe for at least linux, so I can tweak it for macos?

Sure, I've done it separately because of I didn't know in advance what'd be needed to be changed.

:+1:

They're taken from brew install in binary form.

OK :+1: In the end, I think we want to build them without depending on brew.

IIRC this can be done with native instruments, but they're non-recursive, so I've picked one tool that works recursively out-of-the box.

Indeed, the native tool seems to be install_name_tool (which is used by macpack).
For windows, the scrcpy builds have 6 dll (so these libs are linked dynamically):

avcodec-61.dll
avformat-61.dll
avutil-59.dll
swresample-5.dll
libusb-1.0.dll
SDL2.dll

I think the macOS build could do the same, there would be no need for finding dependencies recursively.

Here, i added for MacOS x86-64 thanks to @dur-randir db01900
artifacts

Awesome, you ROCK, @Genxster1998, thank you 👍
These two script files come in handy 😉:
assign icon.png to scrcpy binary.zip
scrcpy-launcher.zip
Unpack them into the same dir as scrcpy, scrcpy-server etc.
run 'assign icon.png to scrcpy binary' first
run 'scrcpy-launcher' to launch scrcpy

.. and for those who like it as a native macOS app 😀:
scrcpy 2.7 (Intel) app.zip

@LeeBinder , you won't need a launcher to furnish path to scrcpy-server as it has been set default to executable_path already rather than /opt/... so no need envt. just adb,scrcpy,libs,server in same dir after patching binary for icon. i think *.dylib(s), adb and scrcpy-server can be moved to Frameworks in proper macOS app.

run 'scrcpy-launcher' to launch scrcpy

@Genxster1998 ideally, but it's not working (yet). Because when I simply run scrcpy even if scrcpy-server IS present in the same directory, I get:
ERROR: Could not get local file path, using scrcpy-server from current directory
stat: No such file or directory
ERROR: 'scrcpy-server' does not exist or is not a regular file
ERROR: Server connection failed

Try it yourself, please.
Looks as if you need to add something in the compile/ build script so that scrcpy indeed DOES look in the same directory. Something is still missing.

Looks as if you need to add something in the compile/ build script so that scrcpy indeed DOES look in the same directory.

The option is build_portable: https://github.com/Genymobile/scrcpy/blob/665ccb32f5306ebd866dc0d99f4d08ed2aeb91c3/meson_options.txt#L4
(add -Dbuild_portable=true to the meson command)
However, it the code to find the executable path is not implemented for macos:
https://github.com/Genymobile/scrcpy/blob/665ccb32f5306ebd866dc0d99f4d08ed2aeb91c3/app/src/sys/unix/file.c#L64-L67

@LeeBinder Thats true indeed , Launching app from launchpad or Finder is not working. I tried adding LSEnvironment dict keys in info.plist which is specifically made for same purpose. yet it still fails to acknowledge server path from there. btw it works with open -a scrcpy.app in terminal. @rom1v Yeah, it is compiling with portable boolean true for meson, i guess a static path could be assigned for MacOS GUI app in ../Resources or ../Libraries Directory.

I see. Agreed, should eventually be in ../Resources (neither in ../Libraries nor in ../Frameworks because it's neither of those but a Android executable binary)

The scripts to build dependencies are currently specific to cross-compile Windows from Linux. I'll have to adapt them to also support native build (will be useful for macOS). Meanwhile, I can merge this PR.

See #5515.


Broken public API in tsserver

Hi. This commit broke our integration with tsserver. Now the commands are 'phantom' and we cannot reference them in our code. For example we use it here:
https://github.com/JetBrains/intellij-plugins/blob/master/AngularJS/src/ngLanguageService/angular-session.ts#L46
We use own abstraction under tsserver (in fact tsserverlibrary.js) in IDEA / WebStorm / other jetbrains products so a lot of users will be affected.
Is it expected breaking change?
//cc @andy-ms

rel. https://youtrack.jetbrains.com/issue/WEB-27249

can you build against lib\protocol.d.ts instead?

@mhegazy yes, I can. But it requires to rebuild the app with ts2.4 and unfortunately the version will be broken for all our users WS2017.1 / IDEA2017.1 etc.

The file is built with preserveConstEnums, so the enum should still be there. i am not sure i understand the issue.. do you have tsserverlibrary in the same process? if so, after this change is ts.server.protocol.CommandTypes non-existent?

@mhegazy sorry, I will clarify.
In our javascript (typescript) code we use ts.server.protocol.CommandNames. After the
121b46a8c0f71b73817f429b57e27e61R6 and e3155e304f4c857fb1d9df7fc74154e9L115
ts.server.protocol.CommandNames doesn't exist in runtime.

I see the issue now, can you give https://github.com/Microsoft/TypeScript/pull/16409 a try?

@mhegazy Yes, it works. The version typescript@rc doesn't contain the changes. Is it ok?

the next one<EMAIL_ADDRESS>will be out in 2 weeks.

same issue here

same issue like
Process: Error processing message: Cannot read property 'getOrCreateScriptInfo' of undefined TypeError: Cannot read property 'getOrCreateScriptInfo' of undefined

@eclimoom Could you create a new issue for that and provide instructions to reproduce the error?

thank you,
after update my webstorm no this error.


No option to enable

I get no option to enable:

I installed into Violentmonkey from Github:

g, 0  && g, - have no effect that I can see.
Window 10 x64 1803
Firefox x64 61.0.1

Hi @Merrit!
Please make sure the userscript is enabled, you can verify it is by opening the developer tools (F12) console tab. You should see something like this:

If not, see if there are any errors reported in the console and share them here.

Hi @Mottie!
Thanks for the reply.
It is definitely enabled as a userscript.
It is throwing errors it seems. To be safe I disabled adblocker & Privacy Badger, but that made no difference.
Content Security Policy: The page’s settings blocked the loading of a resource at self (“script-src”). Source: !function e(a){var b=document.querySelec.... 50:1
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/GothamSSm-Medium-Italic.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/GothamSSm-Medium.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/GothamSSm-Bold-Italic.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/GothamSSm-Bold.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/caecilialtstd-bold-webfont.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/caecilialtstd-roman-webfont.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/GothamSSm-Light-Italic.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/GothamSSm-Light.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/GothamSSm-Book-Italic.woff (“font-src”).
Content Security Policy: The page’s settings blocked the loading of a resource at chrome-extension://17216d91-8546-48a0-b10f-69cca74b9e62/fonts/GothamSSm-Book.woff (“font-src”).

Those errors are not related to this userscript. Try disabling that chrome extension that is attempting to load a custom font on GitHub and see if that was the cause.

I disabled all extensions except for Violentmonkey, still wasn't working.
So I tried disabling Violentmonkey and installed Greasemonkey & the script again. That worked.
So it just seems like Violentmonkey isn't playing nicely.
Thanks for the help! 😃 🎆


http://passguardian.com/ -> 404

Site temporarily disabled

This page has been temporarily disabled due to a misconfigured custom domain.

Alternatively, http://amper5and.github.io/secrets.js/

DNS updated


Split server into frontend and backend

Proposal to make it easier to start building backend things on top. E.g graphQL, etc could be added into server/backend instead of cluttering server/index.js more.

Coverage remained the same at 98.953% when pulling 4e02e9ddde9831f79a091017b1221ecfd035550e on belongapp:server-structure into 148da8c3f4d4604f71e65a29fb409c9544005909 on mxstbr:dev.

I like it! 👍 Makes things a lot clearer.

Coverage remained the same at 98.953% when pulling abab1a9e9240e367b318c52fd9f2cd4f4a1a3d75 on sedubois:server-structure into 96bed78dc3fce1c093e76c900cfe72a183db8495 on mxstbr:dev.

Can I put my 2cents in?
I see you have this concept in the code already, and this is an improvement to that.
For a newbie though, the idea of server frontend/backend is a bit weird as the server is only backend.  In this context though it looks like you are talking about routers.  So i'd just have a server/api and server/routes or something similar

Hmm I'm not entirely familiar with web parlance, but in my understanding, server and backend are different things. The server (express in our case) "serves" both the code which will run both remotely in the browser client (frontend) and the code which will run within the server itself (backend).
I wouldn't talk about routes either, because not all servers have routes (e.g GraphQL).

fair enough - sounds like we're thinking about the concepts from completely different viewpoints!
Once I read through the code more thoroughly I do see the distinction (serving the backend api and serving the frontend assets), but it was just that extra bit of effort which can sometimes be avoided completely.
no worries, as I say, just my 2cents :)

Coverage remained the same at 98.953% when pulling 1f01fd5e904578744e107dd5db94a3d3251f7aaa on sedubois:server-structure into 96bed78dc3fce1c093e76c900cfe72a183db8495 on mxstbr:dev.

What about calling it app and api instead of frontend and backend? I think that'd make the distinction very clear!

 "Backend API" would be more correct (in full words: "the application programming interface of the backend").
Second, personally I don't see why using a term as api in the code: the API you refer to is just the entry point to access the backend code, it's not all of the backend code (i.e it would be incorrect to say that persistence-related code, or business-logic code, is part of an API).
IMHO

While I agree with you, I don't think having a folder called server/api/ is confusing, it's pretty clear that that's the backend API, or not?

Well, the server also serves the front-end code. Server and back-end are not the same thing.
To make a digression, it's true that we abuse the language anyway with the word server itself. Technically server is short-hand for HTTP server, but actually what we put under that directory is the Javascript code called by the HTTP server.
To come back to the term API, backend/api would be more accurate, then devs could add siblings backend/services and backend/persistence as needed.
IMHO 😄 I'm probably just pedantic. Naming things properly is one of the hard things in software dev.
But feel free to just close the issue, I don't need this any more anyway. And then feel free to rename things as you want.

But feel free to just close the issue, I don't need this any more anyway. And then feel free to rename things as you want.

No, I actually like this change, I'm just bikeshedding over the naming! Maybe you're right, I'm not a backend dev. I'll have a think about it, and if I can't come up with anything better I'll merge it tomorrow.

I'm just bikeshedding over the naming

kudos for teaching me new vocab 😉

I do come from a backend (Java/Spring) environment. IMHO app/api is fine.
May not be the most descriptive, but not hard to figure out looking at the code, could (should) even be documented.
One of things that really annoyed me in the Java/Spring world was the extremely verbose path and naming conventions:
/Project/src/main/java/com/project/project/web/ApplicationConversionServiceFactoryBean.java
Yes, that's from a real, live on the web project (Project is not the real name).

Keep in mind that tab completion will be slightly less useful if you choose app and api.

Coverage remained the same at 98.605% when pulling 99e4b876203cd6898653f562e146ccfc7ccfc8ce on sedubois:server-structure into 33485f41053caa6e90b9cbede8b51c168b38bda4 on mxstbr:dev.

Hi @sedubois, this branch looks strikingly similar to #1556. Would you be able to take a quick look at that PR and provide any feedback? Im going to close this PR in favour of the other only because this branch getting stale.

Hi @sedubois @samit4me I don't understand why would server/index.js use the same process for both the backend and the frontend server? why not 2 different express apps?

@xavifuefer To my understanding, this server is purely an application server, so I am unsure on the need to have two servers running. They would need to have different ports.

As discussed in this #194 Comment, the currently provided express server:

solely exist to make npm run serve work and tunnels your local connection with ngrok.


refactor(shared): isDefined util

It won't work. If, for example, globalThis is not defined, it will throw a ReferenceError.


NX 2.0 Eulerian Circuit now supports MultiGraph with multiple keys

New functionality availability w:
https://github.com/networkx/networkx/pull/2359
This can simplify create_eulerian_circuit
https://github.com/brooksandrew/postman_problems/blob/master/postman_problems/graph.py#L178

implemented w https://github.com/brooksandrew/postman_problems/pull/45


create and add LICENSE

Add MIT license in the repository
Refer the below link for more information.
Read about how to add license

I am on it

@Himanshu664  thanks for contributing.


How to disable spell check?

Hi, thanks for the library.
I want to disable the spell check, how can I makes it work?
Just found for quill config here, but don't know how to implement it here.
Thanks in advance

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 7 days since being marked as stale.


Linting

This will most likely create merge conflicts for the other open PRs. Can we pull in changes and lint in one of the others instead? In the future we won't merge a PR unless it meets all of the checks to avoid situations like this. Closing for now.


Farming Stations creating multiple fake worlds resulting in performance decrease

Issue Description:
https://hastebin.com/sapofozevo.vbs Sponge is emitting this stacktrace several times per ten minutes (about 3 times, sometimes more), I'm worried that multiple fake worlds are being creates at the same time, causing lag.
Everytime when this stacktrace is emitted, a fake world is created, resulting in performance decrease.
What happens:
Farming Stations creating multiple fake worlds, I think this is unintended
What you expected to happen:
I expect that there would only be 1 fake world for all Farming stations.
Steps to reproduce:

Set up SpongeForge
Set up EnderIO
Set up Farming Station
Wait?

...

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.10.2-3.1.190
EnderCore: EnderCore-1.10.2-<IP_ADDRESS>-beta
Minecraft: 1.10.2
Forge: <IP_ADDRESS>6

Your most recent log file where the issue was present:
https://hastebin.com/sapofozevo.vbs

Ender IO is not creating worlds, it only is wrapping the real world. And it is only doing this once per Farming Station. So (a) Sponge mixes up World with WorldServer/WorldClient and (b) your farming station is in a chunk that get loaded and unloaded quite often.


Not working

This plug in not working

@topasi Details on particular issue(s) you're having will be necessary for us to help you. Did you look at the source code for the demo to see how it works there?

@adjohnson916 Thank for the great plugin. I already solve the problem. I downloaded the package and view the demo folder but it seems not working because the resources files cant locate the bower_component not found.

@topasi, that makes sense. You'll need the Bower dependencies installed. Let me know if there's a different issue. Thanks!

@topasi, that makes sense. You'll need the Bower dependencies installed. Let me know if there's a different issue. Thanks!


mpsc: try_send() erroneously returns error “Full”

Documentation for mpsc::channel(buffer: usize) says: “The channel's capacity is equal to buffer + num-senders.” If I have two senders and specified that I want to have one additional place in my buffer, both senders should be able to send one element, and thereafter, one of both should be able to send an additional one.
use futures::channel::mpsc;

fn main() {
    let (mut sender_1, mut receiver) = mpsc::channel(1);
    let mut sender_2 = sender_1.clone();

    // Let both senders occupy their guaranteed slot. This works.
    sender_1.try_send(1).unwrap();
    sender_2.try_send(2).unwrap();

    // There is now one slot left, but if sender_2 tries to send,
    // it encounters an "TrySendError { kind: Full }":
    if let Err(err) = sender_2.try_send(3) {
        println!("sender_2 wasn't able to send: {:?}", err);
    }

    // But sender_1 is able to do so:
    sender_1.try_send(4).unwrap();
}

I‘m using futures-preview = "0.3.0-alpha.11 and rustc 1.33.0-nightly (a2b0f247b 2018-12-30).

This seems correct to me, the first time you send through sender_1 it occupies the shared slot, then sender_2 uses up its dedicated slot, you try to send through sender_2 but the shared slot and its dedicated slot are both in use so it can't send, but then sender_1 can send because its dedicated slot is still available. The key is that all the capacity is not equal, the senders dedicated slots can't be used by any other senders, as the next sentence says: 'In other words, each sender gets a guaranteed slot in the channel capacity, and on top of that there are buffer "first come, first serve" slots available to all senders'. I think the documentation here could be probably be rewritten a little to make it clearer that the total capacity is not available to all senders.

Oh, okay, I thought each sender uses its dedicated slot first (that‘s how I understood that sentence). Maybe I‘ll make a pull request to improve the documentation here.

Closing, but @gralpli I'd love to see a docs PR if you're interested in writing one! :)


Initial work on Macro Polo

This is the initial bulk of Macro Polo. The README should explain most everything contained in this PR.
For an example of usage, see cfgov-refresh PR#338

LTGM :+1:

I can't help with the Python, but made a few minor comments on the README.

Looks great!


changing $http default headers definition

pull request for issue #5855

Hi, you have missed the indexer (line 1400) and I think the type can be made more precise.

I couldn't find what the requestType was defined as in the source.  After doing some debugging on a project I'm working on, I don't even see a default value for requestType getting set.  The only place in angular I can find a reference to a property called requestType is in the comments of resource.js line 106.  I'm not sure it has anything to do with $http defaults, thats why I didn't change it.

Hi, [requestType: string]: string|(() => string) is an indexer. The name was arbitrary: it was only there to make the syntax valid. It allows the headers for more request types to be defined without TS1.6 compiler screaming about unknown properties.
However, the actual problem is somewhere else. Seem the update to my comment:
https://github.com/borisyankov/DefinitelyTyped/issues/5855#issuecomment-141118180

angularjs/angular.d.ts
to author(@diegovilar). could you review this PR?
:+1: or :-1:?
check list

[ ] pass the Travic-CI test?

Does this look right to you @diegovilar? Looking to get this thing into my project asap.  Do I need to do anything else?

I think any is too much loose.
please add some test code.
I think string | (() => string) | {[name: string]: string; } is more better typings. isn't it?

Hi, as discussed in the bug (#5855), the interface needs different types when used in different APIs. A proper fix would require changing other stuff as well.
Typing it as any hopefully signals that the type definition stops there and the user should read the docs. If we use a very specific type, users might be mislead into thinking all these types are valid with both APIs, which is not the case.

Hello and thank you for your contribution!
Due to an excessively long queue of pull requests that have become stale, we are "declaring bankruptcy" and closing all PRs opened before May 1, 2016. If you'd still like to merge this code in, please open a new PR that has been merged and rebased with the master branch.
Going forward, we are committing to review or merge all PRs on a regular basis so this bankruptcy will not occur again. We apologize for the incovenience and hope you will continue to contribute to DefinitelyTyped in the future.
Thanks
The TypeScript and DefinitelyTyped teams


Expand the docstring for jax.checkpoint.

I found the old docstring to be a bit inaccurate, but mostly
not explanatory enough. I think that this feature deserves
a tutorial, in addition to an expanded docstring.

@jheek PTAL


Sanitize RPC commands

It's more of a low priority feature request and maybe a good first issue, but something I noticed is that we could validate the parameters in a better way.
Some of our methods doesn't check for validity of claim id or how the parameters comes: https://github.com/lbryio/lbrycrd/blob/master/src/rpc/claimtrie.cpp#L498
In opposition, some of Bitcoin commands does the check and returns an error if it's not a proper hash. For instance getrawtransaction calls ParseHashV for checking it: https://github.com/lbryio/lbrycrd/blob/270307a29070a4c10095528e232ad0f8e251ec5a/src/rpc/rawtransaction.cpp#L183
Which is implemented here, where we could also add our validators: https://github.com/lbryio/lbrycrd/blob/270307a29070a4c10095528e232ad0f8e251ec5a/src/rpc/server.cpp#L144
Aside from the usability of the cli for people giving the wrong input to commands, this should avoid calling methods over bogus data with sometimes unpredictable outcomes.

Thank you @shyba! And yes, this is a good first issue.
If it's not too much work, could you add a list of methods that need this, or a way someone else could identify the ones that do?

Sure!
Commands are implemented here: https://github.com/lbryio/lbrycrd/blob/master/src/rpc/claimtrie.cpp
The ones that needs sanitizing are:

[ ] - getnameproof (blockhash)
[ ] - getclaimsfortx (txid)
[ ] - getclaimbyid (claim id)

See how it's done in here: https://github.com/lbryio/lbrycrd/blob/270307a29070a4c10095528e232ad0f8e251ec5a/src/rpc/rawtransaction.cpp#L183
For the name parameter in getclaimsforname and getvalueforname no sanitization is necessary since they are expected to be strings in any format.

Fixed via https://github.com/lbryio/lbrycrd/commit/e3813362815556634046970d978ba66cef84fea9


Release the package on PyPI

depends on #10

Hi @yakutovicha - Given that CLEED will be compiled to platform-native shared libraries (via CMake as per #40) , then I recommend targeting multiple operating systems and CPU architectures via cibuildwheel and then publishing with the official pypa action. cibuildwheel also provides hooks for testing, which could prove incredibly useful when targeting different architectures.
There are probably plenty of gotchas (here be dragons :dragon::dragon: issues as found with the cleed_sym historically) but this will maximise availability to all potential users and allow a way to spread testing as wide as possible.

then I recommend targeting multiple operating systems and CPU architectures via cibuildwheel and then publishing with the official pypa action.

Thanks for the suggestion @Liam-Deacon, I agree. We have some experience using cibuildwheel for another project and are quite satisfied with it 👍 .


Security Vulnerabilities in PHP <IP_ADDRESS> in Windows Server 2022 Image Version: 20240603.1.0

Description
Windows Server 2022
OS Version: 10.0.20348 Build 2461
Image Version: 20240603.1.0
The vulnerability was found in the National Vulnerability Database (NVD) based on the CPE cpe:2.3:a:php:php with NVD severity: Critical.
CVV: CVE-2024-4577
Severity : Critical
Website Link :nvd.nist.gov/vuln/detail/CVE-2024-4577
Current Version
<IP_ADDRESS>
Fixed Version
8.3.8
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
Windows Server 2022
OS Version: 10.0.20348 Build 2461
Image Version: 20240603.1.0
The vulnerability was found in the National Vulnerability Database (NVD) based on the CPE cpe:2.3:a:php:php with NVD severity: Critical.
CVV: CVE-2024-4577
Severity : Critical
Website Link :nvd.nist.gov/vuln/detail/CVE-2024-4577
Current Version
<IP_ADDRESS>
Fixed Version
8.3.8
The vulnerability can be remediated by updating PHP to 8.3.8 or higher.
Need to patch the vulnerability Php Verison
Is it regression?
8.3.8
Expected behavior
In PHP versions 8.1.* before 8.1.29, 8.2.* before 8.2.20, 8.3.* before 8.3.8, when using Apache and PHP-CGI on Windows, if the system is set up to use certain code pages, Windows may use "Best-Fit" behavior to replace characters in command line given to Win32 API functions. PHP CGI module may misinterpret those characters as PHP options, which may allow a malicious user to pass options to PHP binary being run, and thus reveal the source code of scripts, run arbitrary PHP code on the server, etc.
Actual behavior
Security Breach
Repro steps
Security Vulnerabilities

Hi @ihari277  , the image has been rolled, and new version of PHP 8.3.8 exists. Hope Your issue will be resolved. thanks.


Feature: Adds support for the Redis cluster

Is your feature request related to a problem? Please describe.
We use redis cluster at our company, but there is no cluster mode support in faststream. When trying to connect to one of the nodes and send a message, faststream displays an error
redis.exceptions.ResponseError: MOVED 7304 <IP_ADDRESS>:1234

Describe the solution you'd like
I would like to get support for redis cluster mode. And specifically:

I am most interested in working with redis streams from cluster mode, so it would be cool if you added a RedisClusterBroker that would take as an argument a list of connections to clusters
The method of sending a message must correctly distribute messages across clusters (you can read about this in the redis documentation) and have a parameter for the name of the consumer group when the "stream" parameter is specified
The consumer must create a stream in the cluster and correctly receive messages from different nodes

I experimented a bit and came to the conclusion that the following solution will solve the problem
from typing import Any

from faststream.broker.core.asynchronous import BrokerAsyncUsecase
from faststream.redis.broker import RedisBroker
from faststream.redis.message import AnyRedisDict
from faststream.redis.producer import RedisFastProducer
from faststream.redis.security import parse_security
from faststream.redis.shared.logging import RedisLoggingMixin
from faststream.types import AnyDict
from redis.asyncio.cluster import RedisCluster as Redis
from redis.asyncio.connection import parse_url
from typing_extensions import override

class RedisClusterBroker(RedisBroker, RedisLoggingMixin, BrokerAsyncUsecase[AnyRedisDict, 'Redis[bytes]']):
    @override
    async def _connect(  # type: ignore[override]
        self,
        url: str,
        **kwargs: Any,
    ) -> 'Redis[bytes]':
        url_options: AnyDict = parse_url(url)
        url_options.update(kwargs)
        url_options.update(parse_security(self.security))

        client = Redis.from_url(url=self.url)
        self._producer = RedisFastProducer(
            connection=client,
            parser=self._global_parser,  # type: ignore[arg-type]
            decoder=self._global_parser,  # type: ignore[arg-type]
        )
        return client

But in order to be able to fully use the arguments of the RedisCluster class, you need to make a few more changes. Maybe one of these days I'll create an MR with everything I need.

UPD:
from collections.abc import Mapping
from typing import Any

from faststream.__about__ import __version__
from faststream.redis import RedisBroker
from faststream.redis.publisher.producer import RedisFastProducer
from faststream.redis.security import parse_security
from faststream.types import AnyDict
from redis.asyncio.cluster import RedisCluster
from redis.asyncio.connection import parse_url
from typing_extensions import override

class RedisClusterBroker(RedisBroker):
    @override
    async def _connect(  # type: ignore[override]
        self,
        url: str,
        *,
        host: str,
        port: str | int,
        db: str | int,
        connection_class: Any,
        client_name: str | None,
        health_check_interval: float,
        max_connections: int | None,
        socket_timeout: float | None,
        socket_connect_timeout: float | None,
        socket_read_size: int,
        socket_keepalive: bool,
        socket_keepalive_options: Mapping[int, int | bytes] | None,
        socket_type: int,
        retry_on_timeout: bool,
        encoding: str,
        encoding_errors: str,
        decode_responses: bool,
        parser_class: Any,
        encoder_class: Any,
    ) -> 'RedisCluster[bytes]':
        url_options: AnyDict = {
            **dict(parse_url(url)),
            **parse_security(self.security),
            'client_name': client_name,
            'health_check_interval': health_check_interval,
            'max_connections': max_connections,
            'socket_timeout': socket_timeout,
            'socket_connect_timeout': socket_connect_timeout,
            'encoding': encoding,
            'encoding_errors': encoding_errors,
            'decode_responses': decode_responses,
        }

        client: RedisCluster[bytes] = RedisCluster(
            **url_options,
            lib_name='faststream',
            lib_version=__version__,
        )
        self._producer = RedisFastProducer(
            connection=client,
            parser=self._parser,
            decoder=self._decoder,
        )
        return client


Fix assembly

Make apache-activemq assembly work again.

That's what we did here:

Bump ant plugin to 1.8
Fix run.jar creation
Fix surefire plugin config for assembly
Delete bundle plugin
Add unpack-console assembly portion
Set tar long file mode to 'gnu'
Bump assembly plugin version to 3.3.0
Remove maven one plugin


New Version Bug: Room scrolls to bottom when users join/leave

https://user-images.githubusercontent.com/57961822/113359980-29e98e80-9349-11eb-8025-4d8ceb78363d.mov

can reproduce but for me it just happens randomly. even when there's no new message or anyone leaving/joining

Both of you are using an older version of DogeHouse, I can identify this because I fixed this exact bug and overhaled the scrollbars. it is live on https://next.dogehouse.tv

Apparently this happens only on Firefox

Both of you are using an older version of DogeHouse, I can identify this because I fixed this exact bug and overhaled the scrollbars. it is live on https://next.dogehouse.tv

I was checking on this link itself...

Apparently this happens only on Firefox
There are a lot of compatibility issues with firefox.

oh ok, yea even im using firefox

putting the same message in here in case somebody wants to take it.

Yeah forgot to mention that part. I’m on Firefox

Fixed, but now there's a massive margin on the bottom.


vulnsrc_oracle: one vulnerability per CVE

Get one vulnerability per CVE for Oracle instead of one per ELSA so we can have NVD metadata added to the vulnerabilities.
Related: #495, #499.

@jzelinskie tagging you since you reviewed the RHEL version of this ...

Woohoo! Thank you @jzelinskie!


Black list for strings(not for char)

Hello,
I am wondering if there is an option to not only use char as a blacklist but also specific words? For instance, if tesseract converts the image to string not according to our requirements, we would like to add that string to a blacklist file. This would prevent the tesseract to do the same error twice and most probably easen the task of the tesseract.

There is no option to blacklist words/strings.


fix: App crashes when value in quantity textview is erased (#624)

Fixes #624
Changes: makes the default quantity zero if the quantity field in create new ticket form is left blank
which prevents the app from crashing when the value is erased.

@iamareebjamal sir can you help me out why my travis-ci check is failing

thanks for the advice @Masquerade0097 , but now my Circle ci test failed, can you help again

@AkshayCHD CircleCI is optional. It is also sometimes unreliable. I'm trying to fix it for a few days. Only check travis

@iamareebjamal sir please review

@AkshayCHD Please fix travis build

@iamareebjamal how can i track why my travis ci test is failing

https://travis-ci.org/fossasia/open-event-orga-app/builds/337004381?utm_source=github_status&utm_medium=notification

@AkshayCHD There are PMD violations in your build. Please check and correct them and then send PR. Thanks

@iamareebjamal removed the travis issue

@iamareebjamal when I use StringUtils.isEmpty(value) ? null : Long.parseLong(value) i get PMD violation
Assigning an Object to null is a code smell.  Consider refactoring
but when i do it without ternary operator it shows no such issue.

You should directly return and not assign

@iamareebjamal i placed the code directly in the return statement, i didn't assign

Then you may add a @SuppressWarnings annotation for PMD violation

@iamareebjamal please review, hopefully the last time :sweat_smile:

@iamareebjamal please review

@iamareebjamal please review

@iamareebjamal please review


tests: add sav/Darwin/test_loader_args5.res

because /lib does not exists
Signed-off-by: Jean Privat<EMAIL_ADDRESS>

test osx please

+1


Fixes #1843 Launch and kill a worker or master

[error] /home/travis/build/gearpump/gearpump/services/jvm/src/test/scala/io/gearpump/services/StaticServiceSpec.scala:34: not enough arguments for constructor StaticService: (system: akka.actor.ActorSystem, supervisorPath: String)io.gearpump.services.StaticService.
[error] Unspecified value parameter supervisorPath.
[error]   def route = new StaticService(system).route
[error]

+1 for the code after getting the UT fixed.

Compile failed.

[error] (gearpump-experiments-yarn/test:compileIncremental) Compilation failed
[error] (gearpump-streaming/test:test) sbt.TestsFailedException: Tests unsuccessful
[error] Total time: 426 s, completed Jan 20, 2016 8:13:12 AM

[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/appmaster/YarnAppMasterSpec.scala:163: not found: value AddWorker
[error] Error occurred in an application involving default arguments.
[error]     appMaster.tell(AddWorker(newWorkerCount), client.ref)
[error]                    ^
[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/appmaster/YarnAppMasterSpec.scala:164: not found: type CommandResult
[error] Error occurred in an application involving default arguments.
[error]     client.expectMsgType[CommandResult]
[error]                          ^
[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/appmaster/YarnAppMasterSpec.scala:181: not found: value RemoveWorker
[error] Error occurred in an application involving default arguments.
[error]     appMaster.tell(RemoveWorker(workerContainerId.toString), client.ref)
[error]                    ^
[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/appmaster/YarnAppMasterSpec.scala:182: not found: type CommandResult
[error] Error occurred in an application involving default arguments.
[error]     client.expectMsgType[CommandResult]
[error]                          ^
[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/appmaster/YarnAppMasterSpec.scala:193: not found: type CommandResult
[error]     client.expectMsgType[CommandResult]
[error]                          ^
[INFO] [01/20/2016 08:11:48.500] [SessionTrackerImpl] SessionTrackerImpl exited loop!
[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/client/ManageClusterSpec.scala:26: value RemoveWorker is not a member of object io.gearpump.experiments.yarn.appmaster.YarnAppMaster
[error] import io.gearpump.experiments.yarn.appmaster.YarnAppMaster.{Kill, ClusterInfo, QueryClusterInfo, Version, QueryVersion, RemoveWorker, CommandResult, AddWorker, ActiveConfig, GetActiveConfig}
[error]        ^
[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/client/ManageClusterSpec.scala:71: not found: value AddWorker
[error]     appMaster.expectMsg(AddWorker(1))
[error]                         ^
[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/client/ManageClusterSpec.scala:84: not found: value RemoveWorker
[error]     appMaster.expectMsg(RemoveWorker("1"))
[error]                         ^
[INFO] [01/20/2016 08:11:48.983] [Slf4jLogger] Slf4jLogger started
[error] /home/travis/build/gearpump/gearpump/experiments/yarn/src/test/scala/io/gearpump/experiments/yarn/client/ManageClusterSpec.scala:128: not found: value CommandResult
[error]     appMaster.reply(CommandResult(true))
[error]                     ^
[error] 10 errors found

Current coverage is 62.42%

Merging #1894 into master will decrease coverage by -2.49% as of f9844fa

@@            master   #1894   diff @@
======================================
  Files          172     173     +1
  Stmts         5176    5216    +40
  Branches       574     578     +4
  Methods
======================================
- Hit           3360    3256   -104
  Partial          0       0
- Missed        1816    1960   +144

Review entire Coverage Diff as of f9844fa

Uncovered Suggestions

+0.27% via ...cutor/Executor.scala#271...284
+0.27% via ...MetricsService.scala#118...131
+0.25% via .../master/Master.scala#124...136
See 7 more...

Powered by Codecov. Updated on successful CI builds.


add "mark all as read" button

for those times when you want to declare channel bankruptcy

Suggestion: it should be in the top right of the title bar on desktop, and be a drop down of the channel title on mobile.

I've just got a basic version of "mark all as read" working in my local development copy. Once I've got the UI bit looking and working nice (particularly for mobile) I will submit a PR for your review 🙂
Do we want to have a confirmation alert before actually marking everything as read?

I would be very much in favour of a confirmation button, otherwise one accidental click and you have no idea where you are :)

That's awesome!
In general I'm not a fan of confirmation dialogs. I would rather the button be hidden behind a menu first, but then clicking it takes the action immediately. That still provides protection against accidentally clicking it, but avoids the annoyance of a popup.

I've currently got this for desktop and above:

At smaller resolutions, I've gone for @RosemaryOrchard's suggestion of a combo title + dropdown:

I'm having some trouble with iOS continuing to display the combo title as a button, complete with border and regular text size, despite my additional CSS. I'm not sure if that's an issue with how I'm overriding Bulma (I've never used it before now), a caching issue, or a quirk of iOS. There's also some issues with top-bar layout on iPad in landscape. I'll work on it a bit more.

I haven't got an Android device to hand at the moment, so haven't been able to test there.

It turns out the iOS issues were caused by aggressive caching directives sent by my server, meaning the updated CSS wasn't loaded. I've since deleted the device cache and retested successfully.


Release 0.103.1

By the moment of PR opening:
Mainnet states match:
anna@kiwi:~/Documents/GitProjects/nspcc-dev/neo-go$ go run scripts/compare-states/compare-states.go http://seed1.neo.org:10332 http://localhost:10332
at 0: d34893828a2ddb99c87226a92f764ae2dfadd0d2dc53fb33ae425be83b8d30b4 vs d34893828a2ddb99c87226a92f764ae2dfadd0d2dc53fb33ae425be83b8d30b4
at 4380541:<PHONE_NUMBER>885577433845bfd317ea125e81e32cec4fc7290943cd111d44a1b2 vs<PHONE_NUMBER>885577433845bfd317ea125e81e32cec4fc7290943cd111d44a1b2

Testnet states match:
anna@kiwi:~/Documents/GitProjects/nspcc-dev/neo-go$ go run scripts/compare-states/compare-states.go http://seed1t5.neo.org:20332 http://localhost:20332
at 0: ed359c92fd2e5d3fb6cc57c1599481b223b9eea82bcf940dd1a3db354a9b99d6 vs ed359c92fd2e5d3fb6cc57c1599481b223b9eea82bcf940dd1a3db354a9b99d6
at 2976087: c6ee810f3973d29b60787d47d01f2dc4d6de535828b93c4791f490d17f59673c vs c6ee810f3973d29b60787d47d01f2dc4d6de535828b93c4791f490d17f59673c

Mixed NeoBench setup is OK:
anna@kiwi:~/Documents/GitProjects/nspcc-dev/neo-bench$ make start.MixedFourNodesGoRPC50rate
./runner.sh --nodes mixed -d "MixedGoRPC4x1" -m rate -q 50 -z 5m -t 30s
make[1]: Entering directory '/home/anna/Documents/GitProjects/nspcc-dev/neo-bench'
=> Stop environment
=> Generate configurations for single-node and four-nodes networks from templates
+ cd ./cmd
+ go run ./config/ --go-template go.protocol.template.yml --go-db leveldb --sharp-template sharp.protocol.template.yml --sharp-db LevelDBStore
creating: 813130921/go.protocol.template.yml
creating: 813130921/go.protocol.template.yml
creating: 813130921/sharp.protocol.template.yml
creating: 813130921/sharp.protocol.template.yml
make[1]: Leaving directory '/home/anna/Documents/GitProjects/nspcc-dev/neo-bench'
Creating network "neo_go_network" with the default driver
Creating neo_go_node_four  ... done
Creating neo-cli-node-one  ... done
Creating neo_go_node_three ... done
Creating neo-cli-node-two  ... done
Creating ir_healthy_1      ... done
Creating go-node           ... done
Creating ir_node_healthy_1 ... done
2023/11/08 08:31:23 Used [go-node:20331] rpc addresses
2023/11/08 08:31:23 Run benchmark for MixedGoRPC4x1 :: NEO-GO
2023/11/08 08:31:32 Read 3000000 txs from /dump.txs
2023/11/08 08:31:40 Done 7.26214406s
2023/11/08 08:31:40 Init worker with 50 QPS / 5m0s time limit (3000000 txs will try to send)
2023/11/08 08:31:40 Prepare chain for benchmark
2023/11/08 08:31:40 Determined validators count: 4
2023/11/08 08:31:40 Sending NEO and GAS transfer tx
2023/11/08 08:31:42 CPU: 5.430%, Mem: 372.309MB
2023/11/08 08:31:44 Contract hash: ceb508fc02abc2dc27228e21976699047bbbcce0
2023/11/08 08:31:44 Sending contract deploy tx
2023/11/08 08:31:44 Contract was persisted: false
2023/11/08 08:31:45 Contract was persisted: false
2023/11/08 08:31:45 Contract was persisted: false
2023/11/08 08:31:46 Contract was persisted: false
2023/11/08 08:31:46 Contract was persisted: false
2023/11/08 08:31:47 Contract was persisted: false
2023/11/08 08:31:47 Contract was persisted: false
2023/11/08 08:31:48 Contract was persisted: false
2023/11/08 08:31:48 Contract was persisted: false
2023/11/08 08:31:49 Contract was persisted: false
2023/11/08 08:31:49 Contract was persisted: true
2023/11/08 08:31:49 fetch current block count
2023/11/08 08:31:49 Waiting for an empty block to be processed
2023/11/08 08:31:52 CPU: 1.513%, Mem: 383.984MB
2023/11/08 08:31:54 Started test from block = 9 at unix time =<PHONE_NUMBER>368
2023/11/08 08:31:55 empty block: 9
2023/11/08 08:31:59 #10: 231 transactions in 5058 ms - 45.670225 tps
2023/11/08 08:32:02 CPU: 7.849%, Mem: 367.332MB
2023/11/08 08:32:04 #11: 252 transactions in 5038 ms - 50.019849 tps
2023/11/08 08:32:09 #12: 252 transactions in 5034 ms - 50.059595 tps
2023/11/08 08:32:13 CPU: 12.019%, Mem: 377.520MB

Fixed.


[ADD]: APPAREL COMING SOON PAGE ✨

Close #2361
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

Please review this PR @MohitGupta121


pull request for e2e test at 11/20/2016 10:56:06

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

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


Potential error with algorithm used to check if a point is inside a triangle.

Hi there,
I believe there is an error with the algorithm you have given us to determine if a point is within a triangle in lecture 5, more specifically with how the normal vector is calculated. If we use (the latex is not rendering correctly, the below should be a 2x1 vector)
$$\mathbf{n} = \begin{pmatrix} y_2 - y_1 \ x_1 - x_2 \end{pmatrix},$$
we end up with normal vectors that face outside of the triangle. This means that if a point is inside the triangle, the dot product between this normal and the vector from one of the vertices of the edge to the point will be negative as the angle will be greater than 90 degrees. I think we are either meant to compute the normal vector using
$$\mathbf{n} = \begin{pmatrix} y_1 - y_2 \ x_2 - x_1 \end{pmatrix}$$
and then proceed as stated, or we should take a negative dot product for each edge as a sign that the point is inside the triangle. Please do let me know if I am wrong.
All the best,
Ben

Hi @esemsc-fe7bf37d, thank you for spotting this. Yes, you can either change the way you calculate the normal vector or use the negative of the dot product.
We will consider your suggestion and attempt to introduce this change for next year.


Device storage

Allows Devices to be used like dictionaries, allowing you to store state information inside them:
device = Device(...)
device['thing1'] = 'Hello world'
device['thing2'] = 42

print(device['thing1'])
print(device['thing2'])

I found out that it is also quite easy to implement the following syntax with __setattr__ and __getattr__:
device.thing1 = 'Hello'
device.thing2 = 42

print(device.thing1, device.thing2)

Any opinions on which is more desireable? Main problem with the latter attribute method is that it also applies to access internal to the class (self._internal_variable = ...), where as the former item method is separate from regular attribute access methods. I think the latter attribute method is easier to use, but opens the possibility for name collisions, though if all internal variables start with an underscore, it shouldn't be a problem.
I might switch to the attribute method in that case.

By internal dictionary, are you referring to __dict__ or the _storage that's currently implemented?
The other option I was thinking of was:
    def __getattr__(self, key):
        return self.__dict__[key]

    def __setattr__(self, key, value):
        self.__dict__[key] = value

Essentially just allows you to set new attributes from the outside.

I meant having a separate class for attribute access to a dict.
This should be all you'd need. Since it inherits from dict it works exactly like one in every other way.
class AttributeDict(dict):
    def __getattr__(self, name):
        return self[name]

    def __setattr__(self, name, value):
        self[name] = value

    def __delattr__(self, name):
        del self[name]

So would Device instanciate one of these AttributeDicts, or inherit from it? I don't see exactly how that would fit in Device, or the advantage of creating a separate class for this.
All this needs to do is prevent us from having to use the global or nonlocal keywords in our coroutines we attach to a Device.
It would also be nice to have @tyrelkostyk's thoughts on what syntax is most intuitive.

A device would have an instance of an AttributeDict, having a separate class means we don't have worry about adding new functionality to the device without potentially having a name clash with someone's code that uses device.some_storage_name.
If we have this code in the Device itself, I feel like we're going to be making it responsible for too many things.

So from the user's perspective you'd have to do this:
device = Device('test', 'rover')
device.storage.message = 'Initial msg'

@device.every('10ms')
async def loop():
    print(device.storage.message)

@device.on('other/newmsg')
async def update_msg(event, data):
    device.storage.message = data

Now that I type that out, it isn't too bad, but I feel it would be more intuitive to be able to do device.message. I agree with the reluctance to add too much responsibility to Device, but I also feel that a Device's state should be the responsibility of the Device.
Another option to use Devices:
class myDevice(Device):
    def __init__(self, *args):
        super().__init__(*args)
        self.message = ''

device = myDevice('test', 'rover')
device.message = 'Initial'

@device.every('10ms')
async def loop():
    print(device.message)

@device.on('other/newmsg')
async def update_msg(event, data):
    device.message = data

This is more how Roveberrypy did things, not saying its necessarily the best way to do it, but another option that should be intuitive if you're confortable with OO programming.
I have an idea for refactoring Device (and SerialDevice) that would make having a separate AttributeDict make more sense to me, I'll create an issue for it this evening.

From my understanding, I don't think it'd be confusing to define an AttributeDict within device.py. It still allows for easy variable definitions within each device instance, and having that ".storage" attribute in the variable declaration makes sense to me, it even adds clarity to someone reading the code as they can then know immediately that that's part of that device's storage. The only possibly confusing part is the use of getattr, setattr, etc but even those aren't too complicated from a high level perspective.

Cool, I'll just implement the AttributeDict, and instanciate it in Device then.


Trainer

It's happening!

Absolutely! 🚀

Any updates? Is it usable?

Hey @yukiarimo
It's almost done.
I just want to run some extra tests (QLoRA, full finetune) and finish Qwen2-VL to release it.

is there a dataset/ script that I can test this with my mac studio?


Fix issue #45: -F flag doesn't work with vhdlan

ModelSim does accept the flag and also accepts directives such as
+define.  We now use -f for VCS and -F for ModelSim.  In addition,
ModelSim accepts directives like +define and VCS does not.

I did it by hand. I don't think any of the standard test suites have VHDL, since we would have found the bug.


Local pull request success status testing

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

File
Status
Preview URL
Details

E2E_DocsBranch_Dynamic/index.md
:white_check_mark:Succeeded
View

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 in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


test and support node 18 (no longer node 16)

My recommendation is that we should stop testing and supporting Node 16, and instead bump this to Node 18.
We had been supporting Node 16 because the July 2023 VS Code release shipped with a version of Electron that used Node 16. That is now well in the past, and:

VS Code's Aug 2023 release shipped with Node 18 (up from Node 16): https://code.visualstudio.com/updates/v1_82#_engineering
Node 16 was end-of-life'd on 2023-09-11: https://nodejs.org/en/blog/announcements/nodejs16-eol
For JetBrains, we use the agent binary that bundles Node (20), so we don't depend on Node 16 even if that's what is installed on the system.

An alternative is to start testing Node 16, 18, AND 20 (ie add tests for Node 18 since it's the version in use by VS Code). This would be safer in case any customers are still using an older version of VS Code that requires Node 16. I'll confirm this before merging: https://sourcegraph.slack.com/archives/C04MSD3DP5L/p1713845081115149.
Test plan
CI

@aramaraju I'm going to merge this optimistically, but let me know if you see any evidence that would make us want to continue supporting Node 16.


Login Window Disappears

Hi,
I followed the steps for the TEOC Deployment Guide and was able to added the app to Teams. When I attempt to login, a window appears and redirects then closes. The window is blank. I checked the other submitted issues but did not see a resolution.

@ma-dockins, Thank you for reaching out to us.
Can you please reproduce the issue again in browser version of teams and share the console logs for further troubleshooting on this issue?

I don't see those options. Below is what I have when attempting to access console logs for TEOC:

@ma-dockins ,
Please open TEOC app in web version of teams and follow below steps for generating console logs.

Open Developer Tools.

Select console and all levels.

Refresh the browser and share logs with us.

Thank you! Attached are the logs
teams.microsoft.com-1695140881634.log

@ma-dockins,
From the logs you have shared, the issue is related to CORS policy.
Refer similar issue #111. Please let us know if that helps.

Hi,
I followed the link you provided and verified all information is provided as instructed. The issue still persists.

Hi,
teams.microsoft.com-1696449006735.log
I was able to resolve the CORS error. However, I am not encountering an internal 500 server error. I have attached the console logs.

@ma-dockins,
Thank you for sharing the logs,
Can you please share manifest file with us for further investigation.
Also, please confirm if deployment was successful.

Hello,
I am still experiencing the 500 internal server error. I've attached the manifest.
teoc.zip.zip

Hi,
All 7 Deployments succeeded.
TEOCHCPHeadquarters.txt

@ma-dockins, We have verified the manifest files shared.
We have found that along with WebSiteURL placeholder you have also updated the value for WebSiteURL under developers.

Which is causing an issue. Replace that with below value and let us know if that work.

@ma-dockins, Can you please navigate to Resource Group > Deployments and share screenshot with us.
We know you have confirmed that deployment was successful, but just for validation we will need details if simpleAuthProvision deployment is successful or not as its throwing 500 error.

Also request you to kindly refer similar issue reported by user #94

@ma-dockins,
Can you please try to restart the simple auth app service once and check if its working.
If that is not working, you will have to do the deployment from scratch with new resource group by deleting the existing resource group.


metrics endpoint

Currently the metrics endpoint returns:
http://localhost:18887/metrics
# HELP internal_events Count of internal events
# TYPE internal_events counter
internal_events{category="main_inbound_sse_data",description="db_save_end"} 615
internal_events{category="main_inbound_sse_data",description="db_save_start"} 615
internal_events{category="main_inbound_sse_data",description="event_received_end"} 615
internal_events{category="main_inbound_sse_data",description="event_received_start"} 615
internal_events{category="main_inbound_sse_data",description="outbound_sse_data_send_end"} 615
internal_events{category="main_inbound_sse_data",description="outbound_sse_data_send_start"} 615
# HELP node_statuses Current status of node to which sidecar is connected. Numbers mean: 0 - preparing; 1 - connecting; 2 - connected; 3 - reconnecting; -1 - defunct (used up all connection attempts)
# TYPE node_statuses gauge
node_statuses{node="<IP_ADDRESS>:9999"} 2
# HELP received_bytes Received bytes
# TYPE received_bytes histogram
received_bytes_bucket{filter="events/deploys",le="500"} 1
received_bytes_bucket{filter="events/deploys",le="1000"} 1
received_bytes_bucket{filter="events/deploys",le="2000"} 6
received_bytes_bucket{filter="events/deploys",le="5000"} 6
received_bytes_bucket{filter="events/deploys",le="50000"} 6
received_bytes_bucket{filter="events/deploys",le="500000"} 6
received_bytes_bucket{filter="events/deploys",le="5000000"} 6
received_bytes_bucket{filter="events/deploys",le="50000000"} 6
received_bytes_bucket{filter="events/deploys",le="+Inf"} 6
received_bytes_sum{filter="events/deploys"} 5800
received_bytes_count{filter="events/deploys"} 6
received_bytes_bucket{filter="events/main",le="500"} 1
received_bytes_bucket{filter="events/main",le="1000"} 7
received_bytes_bucket{filter="events/main",le="2000"} 7
received_bytes_bucket{filter="events/main",le="5000"} 7
received_bytes_bucket{filter="events/main",le="50000"} 11
received_bytes_bucket{filter="events/main",le="500000"} 11
received_bytes_bucket{filter="events/main",le="5000000"} 11
received_bytes_bucket{filter="events/main",le="50000000"} 11
received_bytes_bucket{filter="events/main",le="+Inf"} 11
received_bytes_sum{filter="events/main"} 35733
received_bytes_count{filter="events/main"} 11
received_bytes_bucket{filter="events/sigs",le="500"} 601
received_bytes_bucket{filter="events/sigs",le="1000"} 601
received_bytes_bucket{filter="events/sigs",le="2000"} 601
received_bytes_bucket{filter="events/sigs",le="5000"} 601
received_bytes_bucket{filter="events/sigs",le="50000"} 601
received_bytes_bucket{filter="events/sigs",le="500000"} 601
received_bytes_bucket{filter="events/sigs",le="5000000"} 601
received_bytes_bucket{filter="events/sigs",le="50000000"} 601
received_bytes_bucket{filter="events/sigs",le="+Inf"} 601
received_bytes_sum{filter="events/sigs"} 207034
received_bytes_count{filter="events/sigs"} 601
# HELP process_cpu_seconds_total Total user and system CPU time spent in seconds.
# TYPE process_cpu_seconds_total counter
process_cpu_seconds_total 1
# HELP process_max_fds Maximum number of open file descriptors.
# TYPE process_max_fds gauge
process_max_fds 1048576
# HELP process_open_fds Number of open file descriptors.
# TYPE process_open_fds gauge
process_open_fds 31
# HELP process_resident_memory_bytes Resident memory size in bytes.
# TYPE process_resident_memory_bytes gauge
process_resident_memory_bytes 116027392
# HELP process_start_time_seconds Start time of the process since unix epoch in seconds.
# TYPE process_start_time_seconds gauge
process_start_time_seconds<PHONE_NUMBER>
# HELP process_threads Number of OS threads in the process.
# TYPE process_threads gauge
process_threads 27
# HELP process_virtual_memory_bytes Virtual memory size in bytes.
# TYPE process_virtual_memory_bytes gauge
process_virtual_memory_bytes<PHONE_NUMBER>

Need metrics should include:

TBD

Closing as the ticket is incomplete.


Property/Method not supported error.

I am getting an error when i included  Bootstrap in my project that says " Object doesn't support property or method 'on' "
----START-----
$('body').on('click.modal.data-api', '[data-toggle="modal"]', function ( e ) {
var $this = $(this), href
, $target = $($this.attr('data-target') || (href = $this.attr('href')) && href.replace(/.*(?=#[^\s]+$)/, '')) //strip for ie7
, option = $target.data('modal') ? 'toggle' : $.extend({}, $target.data(), $this.data())
e.preventDefault()
$target.modal(option)
})
---- END-----
I am using bootstrap with 960.gs and jQuery 1.6 and jQuery UI.

Hello...brand new here, looks pretty cool.  Have you tried to upgrade your jQuery version to 1.7?  I believe the 'on' event was added to the API in version 1.7:
http://blog.jquery.com/2011/11/03/jquery-1-7-released/
Hope that helps,
Randy

Yep - bootstrap relies on 1.7 - thanks!


Compilation warning (version 2.1.4)

Line 7 of "dist/enquire.js" contains minified js code.  I'm using create-react-app template and my app compiled with warning about this line of code.

Why does that cause a warning?
On 8 Mar 2017 8:29 pm, "seralexwin"<EMAIL_ADDRESS>wrote:

Line 7 of "dist/enquire.js" contains minified js code. I'm using
create-react-app template and my app compiled with warning about this line
of code.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/WickyNilliams/enquire.js/issues/160, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABCnPsWJAlaE5iI9GljGWNF1DO6pFYMCks5rjw_CgaJpZM4MXRUm
.

Because you include minified js code inside not minified version of enquire.js

It's a bizarre thing to complain about. It shouldn't cause any issues
having a line of minified code, it's simply the browserify boilerplate.
What is the exact message? Is there anywhere in CRA docs that mentions this
behaviour?
On 8 Mar 2017 8:37 pm, "seralexwin"<EMAIL_ADDRESS>wrote:
Because you include minified js code inside not minified version of
enquire.js
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/WickyNilliams/enquire.js/issues/160#issuecomment-285161417,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABCnPgNb29cqkcOjRFcbBSTBEySQRvHxks5rjxGAgaJpZM4MXRUm
.

I'm assuming that this is a similar problem if not 100% the same.  During our browerify run, I'm seeing the following.
[14:55:08] Starting 'js'...
events.js:141
      throw er; // Unhandled 'error' event
      ^

Error: Cannot find module '1' from '/Users/szeller/dev/foo/node_modules/react-slick/node_modules/react-responsive-mixin/node_modules/enquire.js/dist'

OK how about this... I point the main field of package.json to src/index.js and create a separate UMD build for people that want to consume it that way. That should do the trick, right?

yeah, my guess is that will work fine.

I just did that and published as v2.1.5. I also deprecated v2.1.4 on npm. Please try and report back

yep.  that fixed the problem I was having.

Awesome! @seralexwin how about for you?

I can't reproduce it anymore

Good to hear!


Playback start broken on v3

Hi,
I found the following issue.
Background:
Version 2 and Version 3 of fluid player use exactly the same source files for video playback.
Expected behavior:
Clicking on play starts video playback. This is true for v2, but not for v3.
Actual behavior:
v3 doesn't start the playback at all, instead the following issues are reported in the web console:
[SourceBufferSink] SourceBuffer append failed "InvalidStateError: Failed to execute 'appendBuffer' on 'SourceBuffer': This SourceBuffer has been removed from the parent media source."
Can be tested here:

Working v2: https://majamee.000webhostapp.com/player/fluid-player.html
Not-working v3: https://majamee.000webhostapp.com/player/fluid-player-v3.html

I would appreciate, if you could have a look at it, especially as the video files are untouched and shared between the two versions.
Thanks & BR, majamee

This looks like coming from dash.js - we did bump a couple of minor versions with V3 - from 3.0.0 to 3.0.3.

Reproduced and affects both v2 and v3, with some differences. The issue is intermittent - sometimes happens, sometimes does not. I suspect due to buffer states of the stream. It also behaves differently depending on whether it's VOD or live stream.
This definitely has something to do with dashjs.

Gets better - it does not seem the above error is actually related. Sometimes playback continues just fine with the error there and sometimes it does not. What is more interesting is that dash.js triggers playbackPlaying event regardless if the there is actual video playing or not.

The issue has been identified - dash.js comes with a "feature" called autoCreate - it would attach to all video sources automatically once loaded to a page. Incidentally, Fluid Player always attaches it's own instances of dash.js to the sources when necessary. This double loading of dash causes race conditions and thus the player breaks.
Fix will be released with v3.0.1.

Awesome, thanks for checking. Looking forward to your next release :)

This should now be fixed with the latest release.


Issue 233: Fixing pipeline expressions for objects produce strange results in the UI wizard

+1 after comments addressed


Improved Event List UI

👋
UI isn't really my thing - but I figured I can make some minor tweaks to events page.
I'm not married to the design, if you don't think its a good idea, feel free to modify or just close the pr.
Proposed Design

Original Design

@bogdankharchenko Thanks, this certainly addresses a few things in the current UI that are not great.
For instance, not showing "AM/PM" instead of "24 hour" times is nice.
Overall, I think it's easier on the eyes.

We have the "Add to Calendar" feature that's not represented in the PR. Are you proposing removing that feature?
It would be nice to make the AM / PM all CAPS
I'd either remove the '22 year part or make it the full year, 2022

We've never put adequate thought into testing the site for mobile responsive, accessibility, performance. These are all areas where I'm sure we could do a better job. This PR could be a good opportunity to do a bit of extra testing and probing to cover these other areas.

@allella I will push these suggestions.
The Add to Calendar does exist, if you click on the date, definitely not as obvious.
My next PR, I want to add a iCal feed which can just stream events to your iphone/google calendar instead of adding an individual event - this way you can always be informed without checking the site.
I think that would be a nice feature.

Alright.
We do have #109 still being an issue and I believe it's to do with the caching Zach put in place on the Laravel side because the API from which it sources the data doesn't have the duplication.
We'd want to get that buttoned up before publishing something to people's calendars and risking having bad info.  I haven't gotten into the guts of the Laravel app and the caching, since Zach has mostly responded to those requests, but it would be great to have someone to supplement what he's been able to volunteer.
Also, I've been part of a related freeCodeCamp project, called Chapter, which we hope will offer a free self-hosted alternative to Meetup.com.  One of the things that's part of the MVP for it is to use a Google Calendar API approach for updating individual events to which people have RSVPed.  I'm not even clear how that API approach will work, but I did put other notes related to the more traditional iCal files that Meetup and Eventbrite send with RSVPs and it's very limited in that they will never update an event if something is changed about it.
I believe you're talking about a feed URL of all events, instead of tracking individual events. I know you can easily add a new toggle on / off calendar to apps like Google Calendar, and that it would refresh itself, but I'm not sure many consumers of the HackGreenville app will want to see dozens of random tech events on their personal calendar.

Pushed new changes to UI, also added mobile screenshot - this is optimized to be responsive across all devices.

Not sure if this is part of the free font-awesome icons, but perhaps there's something like https://fontawesome.com/icons/calendar-plus?s=solid with a mouseover / span tag that says "Add {Event Name} {Event Data} to Google Calendar" so it works on mouse over and probably would be good for accessibility.

The calendar icons on the homepage have a mouseover text that's hidden, so we might be able to reuse that pattern for a calendar-plus font icon.

@allella does this look any better?

Also, just a reminder, since I just used it myself for the Slack app background color, that Zach's wife created this style guide
https://hackgreenville.com/styles

@allella These are all the default bootstrap theme styles, which are represented in the project.

Looks good. Does it have a mouseover for text to well people and screenreaders what that icon does / means?

@all-contributors add @bogdankharchenko for code and design

@allella added screen reader info, and hover title.

I'm not sure of the best practices on this, but my guess is that since we'll have a lot of "Add to Calendar" icons, and other text for that matter, that we'll want to be more verbose in some places and less in others.
For instance, on a screen reader it may be good to have "Add {something unique about the event} to Calendar". Otherwise, it may be hard to differentiate what's about to be added.  I know it's an edge case, but it's probably the sort of thing that doesn't hurt to be more verbose on a mouseover / tooltip, but is more useful for the accessibility case.

@allella just turn on screen reader in safari and see how it looks

I don't have my dev system setup because I just upgraded my OS and it blew away my local PHP and extensions.
If you're happy with it, then I'll give it another look when it makes it to the staging server.

Got it. I think this is moving in the right direction. We can certainly iron this out as we get feedback in staging/prod.
I say merge and then we deal with improvements later

This is merged and staged. Though, I need to do some server upgrades so I'll do that before pushing this live.
https://stage.hackgreenville.com/events

Thanks again.  This is now live.  And, the server is now using PHP 8.1, so now there are less hassle trying to stage and deploy things.
https://hackgreenville.com/events


ML 6.6: Feedforward Neural Networks (D) #6623

Description 📜
ML 6.6: Feedforward Neural Networks (D) #6623

Type of change 📝

[x ] Doc#umentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x ] I follow Contributing Guidelines & Code of conduct of this project.
[x ] My changes generates no new warnings.
[ x] I'm GWOC'21 contributor

@ASLManasa do the changes ASAP

will do it positively by  EOD


Migrate Admin APIs from Ballerina to Node

Description
As per the architectural change, the backend logic of admin APIs needs to be moved to the node application. This task should include

Organization CRUD operation
Organization content/ layout CRUD operation (including zip uploading logic)
Web APP file retrieval

Affected Component
APIM
Version
4.5.0
Related Issues
No response
Suggested Labels
No response

Moved to: https://github.com/wso2/api-manager/issues/3346


⛏ Write a test to add server specific signatures

💭 Introduction:
add tests to find signatures for common servers such as (Apache, Django Flask, Struts, Springboot etc)
📚 Reading
You can find a detailed documentation of test editor rules here
Find 100+ examples of YAML tests here
✅ Task summary:

[ ] Ask to be assigned to the issue.
[ ] Wait to be assigned. We will try to assign in less than 2 hours.
[ ] Fork the tests-library repository, create a new branch and commit the yaml file which will be called in your test.
[ ] Signup for Akto
[ ] Check in the Attempt tab, if the payload changes, then task is done.
[ ] Submit the PR here.

✌🏻 Hints:
You can build the yaml template by referring this link
🙋🏼♂ Questions:
If you have questions, need any help, or just want to hang out, make sure to join us on our Discord server.

@Ankita28g
i want to work on this issue.can u please assign me this issue.
Thank you.

I have assigned it to you @Anurag-space. Happy hacking 🚀 . Feel free to join our Discord


VSCode integrated terminal is not inheriting the virtualenv created by pipenv

Environment data
VS Code version:  1.19.3
Python Extension version:  3.6.4
Python Version: 3
OS and version: MacOS High Sierra 10.13.3
Pipenv version 9.0.1 installed via brew install pipenv
Actual behavior
When I create a virtualenv via pipenv (using pipenv shell) everything works in Terminal. Specifically, which python returns a path to python in the virtual env, e.g. /Users/tjaffri/.local/share/virtualenvs/hello-world-AxAtxCb5/bin/python.
Next, I launch vscode from inside the virtualenv using code .. I bring up the integrated terminal inside vscode and now it seems that vscode is not inheriting the virtualenv correctly. Specifically, which python returns a path to the system python /usr/bin/python.
Note that my vscode settings are completely empty (both workspace and user), so they are using default values for pythonPath etc.
When I try to do pipenv shell inside the vscode integrated terminal, I get the following output indicating that the virtualenv is active (but for some reason the python being used is still the system python)
Taqis-MacBook-Pro:hello-world tjaffri$ pipenv shell
Shell for /Users/tjaffri/.local/share/virtualenvs/hello-world-AxAtxCb5 already activated.
No action taken to avoid nested environments.

When I run the script in the integrated terminal via python3 hello.py (I have to use python3 rather than python due to the aforementioned issue) then there are problems with imported modules. Since the python3 being used is also the system python3 inside the integrated terminal, any modules installed via pipenv e.g. pipenv install beautifulsoup4 are not found and cannot be imported.
Note that when I run the same python3 hello.py outside the integrated terminal, i.e. in the pipenv shell in Terminal everything works correctly.
Expected  behavior
Pipenv created virtual environment should inherit into the vscode integrated terminal, when vscode is launched from inside a virtual environment.
Specifically, the python from inside the virtual environment should be used, so that imported modules in the virtual environment resolve correctly.
Steps to reproduce:

Open Terminal
mkdir hello-world
cd hello-world
pipenv install
pipenv shell
which python (note the result, this will be python inside the virtualenv)
code .
launch integrated terminal
which python (note the result, this will be the system python)

@tjaffri I was having the same issue that you originally described above, but couldn't decipher the answer for a long time. Posting here in case others are still struggling with this.
My interpretation (and the only way I have made pipenv work with VS Code) is that you must make sure that you have not already activated the virtual environment in your Terminal with pipenv shell before you type code .  As long as this is the case, if you type pipenv shell in the integrated terminal after VS Code launches,  it works perfectly and as expected.
This may not be obvious if:

you pipenv shell by habit whenever you move into a project directory in Terminal (or, like me, you have autoenv trigger this automatically)
you are migrating to pipenv from conda. In conda, if you have already activated a virtual environment in Terminal, then type code ., then activate the virtual environment again in the integrated terminal, you do not get any errors

@ptiger10 mind opening a new issue about trying to detect when the user ran pipenv shell already? At worst we can leave it open for someone to contribute code to handle this case.


Can traditional parsing work?

Thanks, @elronbandel ,  you mean the ":" is forbidden even it does not go to inside any name, only between names?
Is it OK to use ";" instead?


Distillery plugin - location of rel has changed

In version 1.0 of distillery the rel map has moved to _build/env.
This causes the data-step to fail on the non-existing rel/project map

I created a PR to solve this.


Added games to datajson file

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #4988
My games were not included in gamedata.json file, I have added my games into that file

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in GameZone's README.md
[x] I have added README.md in my Game folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.
[x] I have added my Game to gamesData.json file and it is loading properly on main website as intended.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @vansh-codes ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Please review @ishita-43

Thank you  @vansh-codes , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


fix: Modify to not use sg submodule do to a failure

Description

Do not use terraform-aws-modules/terraform-aws-security-group submodule with the ingress_with_source_security_group_id.
Motivation and Context

Fix #320
The issues have been reported for the terraform-aws-modules/terraform-aws-security-group when used with the ingress_with_source_security_group_id.
It is reported that this can be avoided by not using submodules.

https://github.com/terraform-aws-modules/terraform-aws-security-group/issues/193#issuecomment-874081213

Breaking Changes

No.
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

Unfortunately, this works with aws provider v4, but seems to cause an error with v3. It was my environmental issue.


transpose exercise

Closes #241

Good first iteration, let's polish it a bit!

Thanks for the review. I'd address them.


Move atleast_1d, atleast_2d, atleast_3d

What do these changes do?

Merge: atleast_1d, atleast_2d, atleast_3d.
Merge tensor/expression/core.py into tensor/core.py
Add executor in: tensor/datasource/ones.py,tensor/datasource/scalar.py
Test pass: test_base, test_base_execute
Related issue number
#453

Should add tests directory in each module.
|
|- tensor
   |- arithmetic
   |- base
      |- tests
         |- test_base.py
         |- test_base_execution.py
   |- ...


Fix target name handling in InitializeSecurityContextA

@RRRadicalEdward just so you, it was missing the code required in the "A" function to pass the target name properly ;)