Improved indentation handling of IndentationHelper.GetIndentationSteps

Fixes #1036
Fixes #1037
For #1037 I've implemented a fourth option: rounding. For indentation size 4 this means: 0-1 no indent, 2-3 single indent. This is my personal preference, as it is forgiving when there is one space to many / little. I'll be happy to adapt it if wanted.

Current coverage is 79.44%

Merging #1751 into master will not affect coverage as of e3ab60a

@@            master   #1751   diff @@
======================================
  Files          546     548     +2
  Stmts        32764   32797    +33
  Branches      9078    9083     +5
  Methods
======================================
+ Hit          26029   26055    +26
- Partial       5273    5278     +5
- Missed        1462    1464     +2

Review entire Coverage Diff as of e3ab60a

Powered by Codecov. Updated on successful CI builds.

This seems fine.
I do wonder if we should instead simply use the previous indentation string unaltered, rather than trying to rewrite it using the currently configured indentation characters. But that is a separate issue.


Org switcher doesn't work

When you have >1 org the Create Org button disappears (probably a separate bug?) to be replaced by a dropdown. Choosing an org which is not the currently selected one simply refreshes the page without changing orgs..
Reproduction instructions

register and create org (call it orgA)
invite a second user
log in as second user (2nd user belongs to orgA as expected)
create an org (orgB)
both orgA and orgB exist and the create org button disappears. Org switcher at the  top shows both orgs but does not function

@benswinburne this has been fixes and was released as part of https://github.com/testrelay/testrelay/releases/tag/v0.1.12 . Let me know if this resolves your problems.


[FLINK-5865][state] Unwrapping exceptions and throw the original ones in state interfaces

What is the purpose of the change
Some implementations of state interfaces wrap the original exception and throw as FlinkRuntimeException. Since all the interfaces claim to throw Exception, it is unnecessary to wrap exceptions into FlinkRuntimeException, which is not intuitive to users in log.
Brief change log

Go through the implementation of state interfaces and unwrap the try...catch blocks in best effort.

Verifying this change
This change is a code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Hi @masteryhx , thanks for your comments! I have changed the implementation of getInternal and updateInternal accordingly.
Besides, I agree we could remove the exception from state interfaces and I'm preparing the FLIP, thanks.


Package could not verified with key

Description

Installed v0.2.0-alpha.2
Had the v0.2.0-alpha.2 + the corresponding .asc file in the input files folder
Clicked the software update from the application

The message said it could not verify the key.
Steps to reproduce
See above
Additional context

Hedera network
other
Version
0.2.0-alpha.2
Operating system
macOS

confirmed resolved


Generation of BC_PORTAL scripts does not longer work.

Two things:

it seems to hang but logging is not enough
bulk collect must use limits everywhere (ddl subcomponent).

Solved.


moved internal/pool/stats.Stats to internal/pool.Stats

Pull request type

Please check the type of change your PR introduces:

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

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Attention: Patch coverage is 16.66667% with 10 lines in your changes missing coverage. Please review.

Please upload report for BASE (master@43b57d6). Learn more about missing BASE report.

Files
Patch %
Lines

internal/query/client.go
0.00%
8 Missing :warning:

internal/pool/pool.go
66.66%
1 Missing :warning:

query/stats.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1338   +/-   ##
=========================================
  Coverage          ?   42.53%
=========================================
  Files             ?      326
  Lines             ?    33646
  Branches          ?        0
=========================================
  Hits              ?    14311
  Misses            ?    18865
  Partials          ?      470

Flag
Coverage Δ

go-1.20.x
44.06% <18.18%> (?)

go-1.21.x
44.64% <18.18%> (?)

go-1.22.x
41.69% <16.66%> (?)

macOS
41.70% <16.66%> (?)

ubuntu
41.71% <16.66%> (?)

unit
42.53% <16.66%> (?)

windows
44.58% <18.18%> (?)

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

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


Add student photo upload

Resize to 96x96, square

https://cloud.google.com/appengine/docs/python/images/usingimages?hl=en

Retrieving school photos from four11 database, photo uploads will no longer be necessary.


Topbar content update

Add tab for planned pages, following #1 :

[x] /activity : Create activity
[x] /activity/:activity_id : view activity (guideline, planning, rubric, review)
[x] /participants
[ ] /planning
[ ] /rubric
[ ] /review

Participants ok, topbar update done


tests: Reduce the number of organizations created

Change-type: patch
See: https://www.flowdock.com/app/rulemotion/resin-devops/threads/3YptdRi0qaO77P5D0PgcDLLAjTI
See: https://github.com/balena-io/balena-api/pull/3239
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>
Resolves: #
HQ:
See:
Depends-on:
Change-type: major|minor|patch

Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

@balena-ci I self-certify!


Cannot apply Filter to any Masked Sprite or Container

The issue from v2 now comes back to v4 which works fine on v3. What i did is,
Create Container
Create Background Sprite
Container Add the Background Sprite
Stage Add Container
Create Mask Sprite
Set Background Sprite Mask is Mask Sprite
Stage Add Mask Sprite
Create Displacement Sprite
Create Displacement Filter from that Displacement Sprite
Set Filter of the Background or Container to Displacement Filter
Stage Add the Displacement Sprite
=> pixi.min.js [sm]:17231 Uncaught TypeError: Cannot read property 'apply' of undefined
Is there anyone have the same problem with me ? Thank for the great beloved framework.

demo please? I will happy to fix that thing, i already fixed a bit for DisplacementFilter.

Wow, what a lightning response. Here the example i write on pixi example site. Thank you so much.
var renderer = PIXI.autoDetectRenderer(800, 600,{backgroundColor : 0x1099bb});
document.body.appendChild(renderer.view);
// create the root of the scene graph
var stage = new PIXI.Container();
// create a texture from an image path
var texture = PIXI.Texture.fromImage('_assets/basics/bunny.png');
// create a new Sprite using the texture
var bunny = new PIXI.Sprite(texture);
// center the sprite's anchor point
bunny.anchor.x = 0.5;
bunny.anchor.y = 0.5;
// move the sprite to the center of the screen
bunny.position.x = 200;
bunny.position.y = 150;
stage.addChild(bunny);
var maskBunny = new PIXI.Sprite.fromImage('_assets/basics/bunny.png');
stage.addChild(maskBunny);
// If this line is active, it will return error apply of undefined
bunny.mask = maskBunny;
// Filter
var displacementSprite = PIXI.Sprite.fromImage("http://i.imgur.com/2yYayZk.png");
var displacementFilter = new PIXI.filters.DisplacementFilter(displacementSprite);
// Apply it
bunny.filters = [displacementFilter];
stage.addChild(displacementSprite);
// start animating
animate();
function animate() {
requestAnimationFrame(animate);
// just for fun, let's rotate mr rabbit a little
bunny.rotation += 0.1;

displacementSprite.position.x += 10;

// render the container
renderer.render(stage);

}

Ok, confirmed: Two masks/filters inside each other are not working at all, FilterManager.pushFilter and popFilter are wrong.

good catch! will take a poke around

done! 👍


[Friends list] The friend is duplicated on the page 5 times after being added to the friends' list

Environment:
Windows 10 x64
Google Chrome 119.0.6045.105
Firefox 119.0 (64-bit)
Opera One(version: 103.0.4928.47)
Reproducible: always
Build found: 21.11.2023_22:00
Preconditions
There is one request for adding to the friends list and no friends in the friend list
Steps to reproduce

Go to the https://www.greencity.social/#/greenCity website
Log in
Click the "My Space" link
Click the '+' button in the 'Friends' block
Click the 'Requests' link
Confirm the request to add the friend
Pay attention to the list of friends

Actual result

Expected result
All added friends are shown on the page without duplicating after adding them
User story and test case links
E.g.: "User story #6423
Labels to be added
"Bug", Priority ("pri:РHigh "), Severity ("severity:Major"), Type ("Functional"), "Friend list"

'Create chat' button doesn't work. Bug had been in chat


Upgrade plugin button from within WHM

It would be nice if there was a button to upgrade the plugin from within the WHM interface, for simplicity purposes.

You can simply upgrade this by running upgrade script from the console. I don't think it is really needed.


README: change client service endpoint

The client service endpoints have changed since v0.2.5
[skip ci]
/cc @xiang90 @hongchaodeng

lgtm


Implement initial metrics to track.

Woah, this looks awesome! Pretty slick. I'll play around with it and see if I find any issues.
Good stuff :)

Thanks! :)


WORKSPACE snippet to use gcc-toolchain

It seems a WORKSPACE snippet to set up gcc-toolchain is missing from the current documentation. It would be nice to include the WORKSPACE snippet in documentation or the release page so that users can start using gcc-toolchain without troubles by copying the WORKSPACE snippet into their WORKSPACE files.
Based on this repo's WORKSPACE, below is an example of the WORKSPACE snippet and .bazelrc to use gcc-toolchain v0.4.2 to build binaries for x86_64. It would be great to confirm that there is something wrong or redundant with the snippet. I mentioned this because unusual configs, --action_env=BAZEL_DO_NOT_DETECT_CPP_TOOLCHAIN=1 and --incompatible_enable_cc_toolchain_resolution were required to add to .bazelrc so that Bazel can use the C/C++ toolchain managed by gcc-toolchain. If these configs are required indeed, I think it worth noting it in the documentation because these configs not so obvious for non-Bazel experts.
WORKSPACE:
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")

# Skylib
# gcc-toolchain depends on bazel-skylib
http_archive(
    name = "bazel_skylib",
    sha256 = "f7be3474d42aae265405a592bb7da8e171919d74c16f082a5457840f06054728",
    urls = [
        "https://mirror.bazel.build/github.com/bazelbuild/bazel-skylib/releases/download/1.2.1/bazel-skylib-1.2.1.tar.gz",
        "https://github.com/bazelbuild/bazel-skylib/releases/download/1.2.1/bazel-skylib-1.2.1.tar.gz",
    ],
)

load("@bazel_skylib//:workspace.bzl", "bazel_skylib_workspace")

bazel_skylib_workspace()

http_archive(
    name = "aspect_gcc_toolchain",
    sha256 = "3341394b1376fb96a87ac3ca01c582f7f18e7dc5e16e8cf40880a31dd7ac0e1e",
    strip_prefix = "gcc-toolchain-0.4.2",
    url = "https://github.com/aspect-build/gcc-toolchain/archive/refs/tags/0.4.2.tar.gz",
)

load("@aspect_gcc_toolchain//toolchain:defs.bzl", "ARCHS", "gcc_register_toolchain")

gcc_register_toolchain(
    name = "gcc_toolchain_x86_64",
    gcc_version = "10.3.0",
    sysroot_variant = "x86_64",
    target_arch = ARCHS.x86_64,
)

.bazelrc:
# Required to use the C/C++ toolchain managed by gcc-toolchain
build --action_env=BAZEL_DO_NOT_DETECT_CPP_TOOLCHAIN=1
build --incompatible_enable_cc_toolchain_resolution

# optional, but preferred for performance as per https://github.com/aspect-build/gcc-toolchain/issues/85
build --experimental_reuse_sandbox_directories

This is probably the missing piece of https://github.com/aspect-build/gcc-toolchain/issues/75 after adding https://github.com/aspect-build/gcc-toolchain/commit/bcc2a0407419b99459ca3aa48119ebbf3869c122 (which triggered the blog post https://blog.aspect.dev/hermetic-c-toolchain).
I'm tempted to just add this snippet as part of the releases with the top-level README.md pointing to the link of the latest release.

I agree we should be clear about the extra flags required, they are indeed non-obvious. Before finding BAZEL_DO_NOT_DETECT_CPP_TOOLCHAIN buried somewhere deep into the source code, I used to set CC=/bin/false to ensure the toolchain resolution was being enforced, but it started to break some IDEs based on other users reporting directly to me (I haven't experienced it myself).

I'm tempted to just add this snippet as part of the releases with the top-level README.md pointing to the link of the latest release.

That sounds good to me.


Add version command

Is your feature request related to a problem? Please describe.
Add a version command.

Is your release process to create a Release on GitHub? (which then triggers an existing workflow)
I can modify an existing workflow so that the release's version number (tag) is passed in as -ldflags="-X" when building a Go program as output for the kyanos version.

Yes, I'm using Gorealser. You can find its configuration file in the root path of the project. And the workflow is release.yml


extend example HCI BLE controller on ESP32C3 (IDFGH-13756)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hello,
I would like to extend ESP-IDF UART HCI Controller example.
I would like to extend the controller commands with custom vendor specific commands, but I can't find a way to do this.
I will add a custom command on the host side (STM32), but I need to add it also on the ESP32C3 side, so I can make some action when parsing this commands.
Thank you for your help.

https://github.com/espressif/esp-idf/tree/master/examples/bluetooth/hci/controller_hci_uart_esp32c3_and_esp32s3

Thank you. I already enabled that, but I still don't understand how can I make a custom response to a vendor opcode.
When I send vendor specific code, stack automatically responds to it, and I don't know how to change the response.
Thank you for your help.

how can I make a custom response to a vendor opcode

Making a custom response would require altering Controller code, which we don't recommend.

are there are vendor specific commands that are already implemented?

We do have some of those. May I ask what sort of commands do you need? What specific functions are you looking for?
Thank you.

Hello,
I would like for example to read current firmware version on the module. And I would like to update the firmware through the same UART that it is used for HCI communication.
That is why I wanted to implement a few more commands that I would call as normal HCI commands
Thank you.

Sorry for the late reply. I have checked with my colleagues and unfortunately neither of these options are available at this time.

OK. Thank you.


Clean up unused tabs

Currently our old tabs are still stored inside the xml file despite unused.
Fix the code so that it cleans up unused tabs.

move to #142


Change "Return" to "Returns" where necessary in XML documentation

In many of the XML files it had been noted that when the documentation
refers to a return value, both "Return" and "Returns" are used. This
has now been fixed to only say "Returns".
Fixes #28867

Thanks, looks good.
FYI, the email you used to sign your commit (see header of https://github.com/godotengine/godot/commit/426bcc9941e98696c872b0f6421d2ef28b987e60.patch) is not affiliated to your GitHub account, which is why the commit doesn't appear as authored by your @seandheffernan account.
It's not a problem for Git itself, but if you want to claim that commit's authorship to your GitHub account, you can add that email as secondary email in your GitHub settings.

Thanks for the tip, I'll get right on that.

Cherry-picked for 3.1.2.


The example regular expression has a syntax typo on it

The regex code in the example does not work:
With(
    Match( "PT2H1M39S", "PT(?:<hours>\d+)H)?(?:(?<minutes>\d+)M)?(?:(?<seconds>\d+)S)?" ),
    Time( Value( hours ), Value( minutes ), Value( seconds ) )
)
// Result: 2:01 AM (as shown in a label control, use the Text function to see the seconds)

Verified within Power Apps:

The parenthesis is mismatched.
Changing PT(?:<hours>\d+)H) to PT(?:(?<hours>\d+)H) would fix this syntax error.
The final code should look like this:
With(
    Match( "PT2H1M39S", "PT(?:(?<hours>\d+)H)?(?:(?<minutes>\d+)M)?(?:(?<seconds>\d+)S)?" ),
    Time( Value( hours ), Value( minutes ), Value( seconds ) )
)
// Result: 2:01 AM (as shown in a label control, use the Text function to see the seconds)

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

ID: dfbafa61-3cfe-9895-2d63-26584e60df05
Version Independent ID: ac3bad74-443d-e429-4925-fe0eb6da8b32
Content: With function - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/functions/function-with.md
Service: powerapps
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Thanks for the report.  However, the code you suggest is already in the documentation and has been for six months.  Can you check again and see if you still see this?  If it is still there for you, let us know and which language you are using, it is possible something is broken in one of the localizations.

The regex syntax is still wrong. I am currently reading the en-US locale of the documentation.

I also checked with web.archive.org to see if the problem is only on my end, it isnt. You can take a look at the snapshot of the page.

Aha!  Got it, thanks for the reply.  There is the same code snippet in the Match function documentation, that's what I was checking, and in that article it was correct.  I'll update it for With shortly.  Thanks again!

Thanks again for the bug report.  The With function article has been fixed.


Add narrator voices for the rooms

Inspired by dakaguia world so we can add narrator voices for galleries

https://www.freecodecamp.org/news/a-primer-on-a-frame-audio-52dd56e54876/

It could work in a way that when the author mints NFT collection, it can include NFT audio which would be as voice-narrator for that particular collection which could offer people to drive their rooms in "customized" way


Revert de-including the examples

Sorry, got it wrong, we already depend on this package examples in Agents, so we need them like so

What's going on here? How can this package not depend on Agents,jl? If it exports agent.jl models the surely it must have it as a dependency?


[Bug]: panic during build phase

System Info
System Info
System:
OS: macOS 14.0
CPU: (10) arm64 Apple M1 Max
Shell: 5.9 - /bin/zsh
Binaries:
Node: 16.14.2
npm: 8.5.0
Details
Panic occurred at runtime. Please file an issue on GitHub with the backtrace below: https://github.com/web-infra-dev/rspack/issues
Panic occurred at runtime. Please file an issue on GitHub with the backtrace below: https://github.com/web-infra-dev/rspack/issues
Message:  should print diagnostics:   × an error occurred when formatting an argument
Location: crates/rspack_core/src/stats.rs:391
Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.
Run with RUST_BACKTRACE=full to include source snippets.
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
1: _napi_register_module_v1
at
2: _napi_register_module_v1
at
3: _napi_register_module_v1
at
4: _napi_register_module_v1
at
5: _napi_register_module_v1
at
6: _wasmer_vm_imported_memory32_atomic_notify
at
7: _wasmer_vm_imported_memory32_atomic_notify
at
8: _napi_register_module_v1
at
9: _napi_register_module_v1
at
10: _napi_register_module_v1
at
This happens when I connect css like this:
import 'semantic-ui-css/semantic.min.css';
And use CssExtractRspackPlugin as described in the instructions
module: {
   rules: [
      {
          test: /\.css$/i,
          use: [rspack.CssExtractRspackPlugin.loader, 'css-loader'],
          type: 'css',
       }
   ]
}

Reproduce link
No response
Reproduce Steps
run rspack

Try to create a minimal repro but fail, can you provide one?

module: {
   rules: [
      {
          test: /\.css$/i,
          use: [rspack.CssExtractRspackPlugin.loader, 'css-loader'],
-          type: 'css',
       }
   ]
}


nullcheck appMultiClient to avoid getAccounts nul

appMultipleClient is null when accountMode is SINGLE. When getAccounts() was called in SINGLE mode  the app throws:
java.lang.NullPointerException: Attempt to invoke interface method 'java.util.List com.microsoft.identity.client.IMultipleAccountPublicClientApplication.getAccounts()' on a null object reference.
found boolean is used within the loop to alert the user the requested account was not found in scope, so it can be instantiated only when appMultipleClient is a valid object

Thanks for your contribution! I'll release a revision with your fixes.


NFL Streams NFLStreams WEEK 3 REDDIT NFL STREAMS On Crackstreams

How to watch Week Three of the 2024-2025 NFL season live online for free—and without cable
Last weekend saw plenty of upsets as the Buccaneers, Falcons, and Vikings all shocked their opponents, but the big story of the 2024-2025 season so far might be the lackluster play of key parts of the Kansas City Chiefs offense.
Sure, the Super Bowl champs are 2-0 so far, but Patrick Mahomes threw just 13 passes in the first half and a pair of interceptions (and almost a third, if not for a penalty). Travis Kelce, meanwhile, had just one catch for five yards after completing just three catches in week one.
Will this be the weekend they return to form? If you’re tuning in to find out, you’ll have more than one way to catch the action. Here’s how best to do so.
Which NFL teams are playing this week? And what channels are airing the games?
The Chiefs will be center stage this week with Sunday night’s game. Here’s a look at who else is playing. As always, the home team is listed second.
Sunday, Sept. 22
New York Giants vs. Cleveland Browns, 1:00 p.m. ET on Fox
Philadelphia Eagles vs. New Orleans Saints, 1:00 p.m. ET on Fox
Houston Texans vs. Minnesota Vikings, 1:00 p.m. ET on CBS
Denver Broncos vs. Tampa Bay Buccaneers, 1:00 p.m. ET on Fox
Green Bay Packers vs. Tennessee Titans, 1:00 p.m. ET on Fox
Chicago Bears vs. Indianapolis Colts, 1:00 p.m. ET on CBS
Los Angeles Chargers vs. Pittsburgh Steelers, 1:00 p.m. ET on CBS
Miami Dolphins vs. Seattle Seahawks, 4:05 p.m. ET on CBS
Carolina Panthers vs. Las Vegas Raiders, 4:05 p.m. ET on CBS
Paid Content

Driving employees’ career growth is Ochsner Health’s key to success
From Ochsner Health System
San Francisco 49ers vs. Los Angeles Rams, 4:25 p.m. ET on Fox
Detroit Lions vs. Arizona Cardinals, 4:25 p.m. ET on Fox
Baltimore Ravens vs. Dallas Cowboys, 4:25 p.m. ET on Fox
Kansas City Chiefs vs. Atlanta Falcons, 8:20 p.m. ET on NBC
Monday, Sept. 23
Jacksonville Jaguars vs. Buffalo Bills, 7:30 p.m. ET on ESPN
Washington Commanders vs. Cincinnati Bengals, 8:15 p.m. ET on ABC
How can I watch NFL games for free—even if I am out of market?
The best way to watch any sort of network programming for free on a big screen is with a good HD antenna. To ensure you’re getting the most reliable signal, be sure to test the antenna in multiple locations in your home.
Can I stream 2024 NFL games live online if I don’t have a cable subscription?
Yes. Several, in fact.
Paramount+
CBS’s streaming service will give you a one-week free trial, followed by a $8 or $13 monthly charge. (An annual plan is as low as $30 right now.)
Peacock
NBC’s streaming service will give you access to several games, including all upcoming Sunday night matchups. It carries a $8 or $14 monthly charge—and the service has recently done away with its free trial. (The free version of Peacock does not include live sports.)
Disney+
Disney’s bundle of Disney+, Hulu and ESPN+ no longer has a free trial, so you’ll have to pay $17 per month for all three combined (or $30 per month for no ads on Hulu).
Including Live TV in the bundle bumps the price to $77 per month ($90 with no ads).
Hulu with Live TV
The free trial on this service lasts three days. Afterward, it will cost you $77 per month.
YouTubeTV
After a free trial, you can expect monthly charges of $73.
Sling TV
Dish Network’s Sling lower-tiered “Orange” plan will run you $40 per month. Adding the more comprehensive “Blue” plan bumps the cost to $55 per month. The seven-day free trial has disappeared, unfortunately.
DirecTV Stream
Formerly known as DirecTV Now, AT&T TVNow and AT&T TV, this oft-renamed streaming service will run you $80 per month and up after the free trial option. (The package that includes NFL Network will run you $99 per month.)
Fubo TV
This sports-focused cord-cutting service carries broadcast networks in most markets. There’s a seven-day free trial, followed by monthly charges of $80 and up, depending on the channels you choose.
Can I watch NFL games on Amazon?
You can, but not on Sunday. Amazon is the home of Thursday Night Football, with the exception of Thanksgiving (though it will air a game on Black Friday). The first matchup on Amazon this year will be the Dallas Cowboys facing the New York Giants on Sept. 26.
Does the NFL offer any viewing packages to watch the games I want?
It does!
NFL App
The NFL App will let you stream games that are being broadcast locally in your market on Sundays. If you want to watch an “out of market” game, you’ve got two choices.
NFL+
Watch live local and out of market games and (with the premium subscription) replays. You’re looking at a $50 charge per season. ($100 for premium.)
NFL Sunday Ticket
YouTube once again is the home for this channel. Prices this year start at $73 per month for Sunday Ticket and YouTubeTV or $449 for just Sunday Ticket (spanning the entirety of out-of-market games this season).

dfdf


include won't include strict implicits

Description
include now refuses to include strict implicit section variables.
Context
I ran into this on Mathlib's nightly-testing branch moving to nightly-2024-08-22 , in Mathlib/Analysis/Complex/TaylorSeries.lean.
Steps to Reproduce
section
variable {i : Nat}

include i in -- Works
example : True := by
  have _ := i
  trivial

end

section
variable ⦃i : Nat⦄

include i in -- Fails with "invalid 'include', variable 'i' has not been declared in the current scope"
example : True := by
  have _ := i
  trivial

end

Versions
nightly-2024-08-22, was working fine on nightly-2024-08-21
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

Was fixed by #5138


python3Packages.devolo-home-control-api: 0.17.0 -> 0.17.1

Motivation for this change
Update to latest upstream release 0.17.1
Change log: https://github.com/2Fake/devolo_home_control_api/releases/tag/v0.17.1
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118238 run on x86_64-linux 1

3 packages built:

home-assistant
python38Packages.devolo-home-control-api
python39Packages.devolo-home-control-api

Test failure on aarch64 is unrelated and flaky, and the log is not very helpful.


Readonly on radio and checkboxes #412

Reasons for making this change
While the controls themselves may not support the property directly it seemed to me that that "disabled" was closer than nothing at all. At least the user could update the data.
While disabled and readonly are subtly different, my suggestion is that using disabled is possibly better than doing nothing.
Reference:
Readonly on radio and checkboxes #412
Checklist

[ ] I'm updating documentation

[ ] I've checked the rendering of the Markdown text I've added
[ ] If I'm adding a new section, I've updated the Table of Content

[x] I'm adding or updating code

[ ] I've added and/or updated tests
[ ] I've updated docs if needed

[ ] I'm adding a new feature

[ ] I've updated the playground with an example use of the feature

While this pull request is still valid, I forgot the recent change for disabledCls.
It should also be:
const disabledCls = disabled || readonly ? "disabled" : "";
In both files.  But I'll wait for feedback first ... which I could find a way to kill it myself.  But happy to resubmit it with the above change as well.

Hi, sorry about the (unforgivably) long delay in review! I took a quick look at this and it basically looks OK, but it needs a rebase and it doesn't seem to cover CheckboxWidget (only CheckboxesWidget, which I think is only used for arrays). Could you update it please? I promise I'll review it faster this time  😞

We still want this feature. @crumblix, I feel really bad about the review delay, but would you be willing to keep working on this? Just say "no" and I'll take over the patch. Sorry again.

Bah I see that #470 is the current PR where work is happening. Closing this one.

Sure.  Probably still needs another rebase ... but I'd forgotten about the PR when you asked me last time and so created a new one.
It's more comprehensive.  I'd suggest we close this one and move discussion/review to that PR.
See:
https://github.com/mozilla-services/react-jsonschema-form/pull/470


Fix shapes in comments of nmt_with_attention.ipynb

It is a bit misleading and confusing that the output shape of decoder is currently commented as (batch_size * max_length, vocab). However the correct shape should be (batch_size * 1, vocab), since the input x of GRU layer has shape == (batch_size, 1, embedding_dim + hidden_size).

@googlebot I signed it

It also seems confusing to me that in the evaluate method, tf.multinomial(tf.exp(predictions), num_samples=1) is used to sample a prediction from the predictions logits. The usage of tf.exp seems redundant here, since predictions itself is the logits output of a dense layer in decoder.

Anyone of the reviewers I added know who wrote or knows about the RNN attention models?

@yashk2810

@SayHiRay Nice catch. Would you mind making the same change in the text generation notebook? Its in the generative examples folder. And also other notebooks?

tf.multinomial does expect un-normalized log-probabilities.
The Decode model is only ever called with input batch_size of 1.
So both of these changes look correct.

@yashk2810 @MarkDaoust Thanks for reviewing and mentioning about other notebooks. I have checked the usage of tf.multinomial in text_generation.ipynb and image_captioning_with_attention.ipynb, and found same mistakes. I have committed the changes. Please help take a look again. Thanks for help!

LGTM. Thanks for the fix 😀

@MarkDaoust I think this needs a "ready to pull" label

Yes!

Does this get merged automatically? I think it should get merged by the copybara bot right?

Yeah, its currently in the process of being merged. Some internal test flakes have slowed it down

I think the internal tests are failing because an owner hasn't approved it. It happened with me before.

Just looks like a test flake to me. Tests are re-running now

Sounds good 😀

Thanks a lot for helping with merging! 👏


[Feature request] Expanding matchmaker criteria on server

It would be useful to be able to expand criteria on server, for example:
extracted := nk.MatchmakerExtract()
// see which tickets were in matchmaker for a long time, modify their query and properties;
nk.MatchmakerInsert(extracted)

I haven’t tested that yet, but it seems that it is possible to expose this matchmaker functionality to the user code on server. LocalMatchmaker isn’t tied to websocket in a way that prevents using the matchmaker in the user code on the server. As far as I was able to tell.
We’re actually considering forking Nakama because we want to expand matchmaking criteria on the server and not on the client, because the criteria depends on the backend data, so it makes more sense to have the matchmaker expansion logic on the server.
One thing I can think of is clusterisation - multiple instances of Nakama. Here we may have a problem, but I'm not sure how Nakama's clusterisation works.

I think it would be good to understand the exact use case here:

can you tell us what information is available on the server that you'd like to use to enhance matchmaking criteria?
Also how can this be different to using a BeforeHook on Matchmaker request to add data for that user into the matchmaking criteria before it is processed by Nakama?
What kind of gameplay are you developing here?

Also since you've edited the original message,

One thing I can think of is clusterisation - multiple instances of Nakama. Here we may have a problem, but I'm not sure how Nakama's clusterisation works.

Are you currently using Nakama Enterprise?

Also since you've edited the original message,

One thing I can think of is clusterisation - multiple instances of Nakama. Here we may have a problem, but I'm not sure how Nakama's clusterisation works.

Are you currently using Nakama Enterprise?

No, we're not using Nakama enterprise

I think it would be good to understand the exact use case here:
* can you tell us what information is available on the server that you'd like to use to enhance matchmaking criteria?

* Also how can this be different to using a BeforeHook on Matchmaker request to add data for that user into the matchmaking criteria before it is processed by Nakama?

* What kind of gameplay are you developing here?

number of matches played, number of matches won, some kind of player's rank

How it is with the current situation: a player clicks Play and adds their matchmaker entry to the matchmaker, they sit there for X seconds and we know that we should expand the criteria. If we do it on the client, then the logic with timers and requesting matchmaker related data is on the client. We add another ticket with an expanded criteria, which is fine. But we're ideally thinking of adding themselves to the matchmaker once and then getting into the game while all the matchmaking expansion logic is done on the backend.
We also want to validate query and properties that the client has submitted on the backed, via hooks. We can do that but that means that Nakama on the server should also fetch matchmaker related data each time a user adds themselves to matchmaker.

a shooter similar to Overwatch

But we're ideally thinking of adding themselves to the matchmaker once and then getting into the game while all the matchmaking expansion logic is done on the backend.

Why can't this be done transparently on the client side whilst the player is in the game/match? The action of sending a matchmaker API call doesn't have to be explicitly driven by a player action.

How it is with the current situation: a player clicks Play and adds their matchmaker entry to the matchmaker, they sit there for X seconds and we know that we should expand the criteria. If we do it on the client, then the logic with timers and requesting matchmaker related data is on the client. We add another ticket with an expanded criteria, which is fine.

So if I read this, you'd like to mutate the matchmaker criteria after it's been submitted to the matchmaker, without cancelling/resubmitting? Are you sure you'd want to open a PR about this, this is non-trivial to get right.

Why can't this be done transparently on the client side whilst the player is in the game/match?
One problem with this is every time we want to update the matchmaker logic, we'll need to update the client, and push it to users. To mitigate that we'll need to do something like fetching matchmaking configuration from the server.

To mitigate that we'll need to do something like fetching matchmaking configuration from the server and passing it to .MatchmakerAdd(query, properties)

I think this is where a remote configuration system, like Satori comes in to play to be able to have these customisation available per-player without a game client update.
As I said originally, we can take a look at the PR, but this is non-trivial work to add and get right in a highly concurrent environment without significant overhead in performance. I suggest you take a different approach as we've spoken on this issue.

Thank you. Another related question: would that be a performance hit or some other problem, if we added localmatchmaker.Extract() method to the user code on the server? Right now matchmaker is like a black box for the server and we can't monitor its state. What, for example, if we want to see if no one is playing a certain game mode and we want to hide that in the client?
As far as I understand, extract just extracts the whole matchmaker state. Would that be fine if we extracted it once every few minutes to make a decision?

Alright, following your advice, we decided to not fork Nakama in case of matchmaking. Sorry for disturbance here and thanks for the project.

@dmitryrn It'd be good to carry this discussion on within the forum. Lets see if we can prepare a feature proposal which could be added to the game server.


Make radio jammer block suit sensors

About the PR

Radio jammers will now additionally block suit sensors.
When someone with suit sensors is in range of an active radio jammer, their suit sensor output will be jammed. This will lead to them appearing as frozen on any crew monitors for roughly 10 seconds before they completely disappear from it.
Why / Balance

This gives syndies more stealth options when doing their objectives, and provides an interesting use for the radio jammer.
Now on the implementation details:
With this current implementation you drop off suit sensors after roughly 10 seconds of being jammed. This means you get a 10 second period before anyone actively watching crew monitors can even suspect something's wrong. The reason I have it like this is:
a) It's the simplest way to implement this feature.
b) It's more realistic while still providing a pretty significant advantage.
Alternatively this could be changed to make the suit sensors submit outdated info indefinitely until the jammer has stopped.
Technical details

Added a new event which the suit sensors system calls before actually constructing the message it will send to the crew monitoring server. If it is cancelled then the message is not sent. I also intentionally made it so this is checked after a new update time has been set, but if it's a problem it can just be moved above it.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

https://github.com/space-wizards/space-station-14/assets/32041239/3bc815ac-5ec1-4ef1-8594-b390da657d29
Changelog

:cl:

tweak: The radio jammer is now able to jam the signal of suit sensors.

Great change that turns the Radio Jammer from a somewhat useful to a very useful maints slasher item. Will definitely start using it more in the future if this gets implemented.

Does this work with tracking implants as well? Also looks like I'm gonna have to figure out merge conflicts for REAL this time 😆.
Lets hope I can actually manage to do it correctly this time... Great change!

Does this work with tracking implants as well?

Yes.

I just realized the event should just use CancellableEntityEventArgs however I'm dumb. I'll fix it tomorrow when I wake up.

me like


Start unnamed pools

Hello!
Would it be possible to remove the requirement for pools to be named?
Currently if we want to dynamically connect to a database (for example, using different databases for different uses in a multitenant system) we need to dynamically generate an atom and have some mechanism for recycling these atoms to avoid hitting the atom limit. This can be challenging to implement.
If we could reference the pool using a pid or a reference rather than a name we wouldn't have this issue.
Thanks,
Louis

Good point. I can't think of anything that would make this not possible. Only "difficulty" may be the API. I'll take a closer look and see what I can do.

Thank you!


Add Support for Additional Font & Text Style Properties

Description
The styles package needs support various font and text properties. Add them to here under "Font & Text Properties:" https://github.com/chasefleming/elem-go/blob/2b7e1d671ae12e5f272770d51c8801b28d970a94/styles/styles.go#L84
Properties to Add

font-style
text-shadow
vertical-align
word-spacing
word-break
text-indent

I would like to work on this, can you assign it to me

Assigned. Thanks @Emm-dev0 !

Working on it.

Please review the pull request, I am finished. Thanks


httpTrigger generate "Status: 502 Bad Gateway" when setting errors from mssql in res.body

Ran into 502 errors today implementing mssql for querying Azure MSSQL.  The feedback from the logs are pretty much non existent for these errors, but managed to figure it out and below are sample code that generated the error. Ran a query against a table that didn't exist.
const sql = require('mssql');
const config = require('../db.config');

module.exports = (context, req) => {
    new sql.connect(config).then(() => {
        new sql.Request()
        .query('SELECT * FROM table_dosent_exist').then((recordset) => {
            context.res = { body: recordset };
            context.done();
        }).catch((err) => {
            context.res = {
                status: 500,
                body: err
            };
            context.done();
        });
    }).catch((err) => {
        context.res = {
            status: 500,
            body: err
        };
        context.done();
    });
};

The response in the output window from within the function app portal is There was an error running function (getTest). Check logs output for the full error. If I run the same code in a node/express project the error I get is the following
{
  "name": "RequestError",
  "message": "Invalid object name 'table_dosent_exist'.",
  "code": "EREQUEST",
  "number": 208,
  "lineNumber": 1,
  "state": 1,
  "class": 16,
  "serverName": "xxxxxxxx",
  "procName": "",
  "precedingErrors": []
}

What solved it for me was to change body: err to body: JSON.parse(JSON.stringify(err)). Obviously this is not the optimal solution, but either I have misunderstood something or there's a possible improvment.
Cheers, Davud

//cc @mathewc
I think this might be a runtime issue - it looks like the runtime is barfing when given a object it can't serialize to JSON. @mathewc - I thought we had added some protections here, but I could be wrong. Maybe we just did it for context.log?

The err object returned from the mssql module gets serialized with 4 properties: code (string), message (string), (name) string, and constructor (function). We receive this error object as an ExpandoObject from Edge.js, and our conversion code blows up on it for some reason with a FatalExecutionEngineError.

I am seeing a similar issue with using mongoose/mongodb api with Cosmos on Azure function. Data that cant be serialized is throwing 502 - Web server received an invalid response while acting as a gateway or proxy server.
Only if I JSON.stringify the req body it works fine.


Adding ANSI_WARNINGS explainer, re: COUNT is int NULL

See https://github.com/MicrosoftDocs/sql-docs/issues/7088

Docs Build status updates of commit 50e4d20:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/functions/count-transact-sql.md
:white_check_mark:Succeeded

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

@markingmyname
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Learn Build status updates of commit a0cd29b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/functions/count-transact-sql.md
:white_check_mark:Succeeded

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

Learn Build status updates of commit 51e3c43:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/t-sql/functions/count-transact-sql.md
:white_check_mark:Succeeded

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

Replaced by internal PR. This PR will close automatically.


[bug] E2E controller testing job is failing: Error reading JToken from JsonReader

Describe the bug
E2E controller testing job triggered on a Pull Request is failing on on the Post Checkout Repository step:
Error: The template is not valid. .github/workflows/build_test.yml (Line: 109, Col: 16): Error reading JToken from JsonReader. Path '', line 0, position 0.

Build example
To Reproduce
Create a PR
Expected behavior
The job is succeed

The issue is no longer reproducible and has resolved itself.


Suggestion: Making the plugin scan for .feature files in java and resource folder instead of only the resource folder

Currently the .feature files are put in the resource files and the step definition files are added to the java folder.
It seems a little disseparate when they are connected other and I was thinking if it would be better to have them together. I was thinkg about making it possible to have feature files in the java folder.
And then scanning the allSource folder instead of the resource folder when looking for feature files:
The change I had in mind is the following in CucumberRunner
Instead of following line:
    def features = sourceSet.resources.matching {

Then it should be
    def features = sourceSet.allSource.matching {

I will be happy to make the change if it will be accepted
But let me know

By convention, the java folder in a source set should only contain java source files. Other resources the program depends on (such as feature files) should be in the resources folder.
That being said, the plugin should probably be scanning the output dir for the source set in case the need were to ever arise for their to be a processing step, such as token substitution, on the feature files. That change would effectively give you the (unrecommended) option to commingle feature files with the java source files as they all end up in the same output directories.

You are right that strictly speaking .feature should be in resources and it is merely a convenience when using and IDE that .feature files and step files are in the same folder structure
However if #15 is accepted I can work around it by using inheritance, and this one can be closed then

Closing this issue as the referenced Pull request was merged.


There is no need for jasmine-node to run tests

According to official Jasmine documentation, there is a npm package called jasmine, that includes the Jasmine testing framework and can run your JavaScript tests.
It would be as easy as:
npm install jasmine
jasmine init
jasmine

It would be a good idea to update this track's documentation to reference jasmine instead of jasmine-node to run the tests.
Thanks @joelwallis for pointing this out on #315
@exercism/javascript Any thoughts?

Hey!
Good to see it's in progress. I'm available to implement this change if you're willing to review my work :)
@exercism/javascript take a second to check if it's a good idea, and gimme a green flag so I can make a PR for this change.
PS: I've just npm i -g jasmine, then jasmine ./my-solution-file.js. There was no need to jasmine init.

@rchavarria @joelwallis I think this is great.
I think that a global install is necessary to use the command line jasmine, and that we would need to change the directory structure.  So, I would propose we have two choices:

Tell the users to install globally, and use @joelwallis' approach, which requires a global install but no directory changes.  This is simple, but has the disadvantage that our dependencies are not all encapsulated in each exercise.

Add a package.json file to each exercise, similar to what we have in the Ecmascript track.  This would allow the users to change node versions easily and have their dependencies spelled out for each exercise.  It would also mean we could set up an npm test script, so the user doesn't need to type the filename each time.

@exercism/javascript any feeling about these two different approaches?

and that we would need to change the directory structure. So, I would propose we have two choices:

I did not get what change you are referring here @matthewmorgan

Option 2 (add package.json) seems desirable, because it could simplify the life to the users a little bit. But I don't like how we manage package.json files in xecmascript (but I don't have a better approach neither :sad: ). It needs to copy the same version of the file on each exercise, or manage a lot of almost identicall package.json, and I don't like those approaches.
Right now, in the documentation, we're suggesting to install jasmine-node globally, so as a first step, I'd do the same with jasmine.
I was a bit worried about jasmine init command. I don't know what that command does. Any clue? Anyway, @joelwallis says it works without executing that command.

I was a bit worried about jasmine init command. I don't know what that command does. Any clue?

jasmine init creates this file in spec/support/jasmine.json:
{
  "spec_dir": "spec",
  "spec_files": [
    "**/*[sS]pec.js"
  ],
  "helpers": [
    "helpers/**/*.js"
  ],
  "stopSpecOnExpectationFailure": false,
  "random": false
}

Since we have exercise.js and exercise.spec.js in the same directory, we don't need this config.

Right @tejasbubane this is related to my comment about needing to change the directory structure if we want to just use the jasmine command without specifying a target.  It expects the spec files to be in /spec I think.

It expects the spec files to be in /spec I think.

@matthewmorgan Nope. It does not.
With the current directory structure where:
$  ls
HINTS.md            example.js          hello-world.js      hello-world.spec.js

I can run jasmine hello-world.spec.js from the hello-world directory. Or I can even run jasmine hello-world/hello-world.spec.js from outside the hello-world directory.

@tejasbubane I think that @matthewmorgan was trying to say that, if we want students to test the lesson by just running jasmine - without arguments, the program itself would expected tests to be inside a /spec folder.
I believe it's kinda overwhelming for now. From an user perspective, executing jasmine against a test file makes more sense that just executing jasmine and having some magic happening, without any clue about how it really happens. For me it makes me feel quite lost.
Btw, if we are going to support package.json files in exercise files we must embrace Node.js and its industry standards. I must:

Provide a package.json for each lesson;
Use npm scripts to provide a standard way to test code (npm test);
Perhaps running npm install right after exercism fetch javascript (is that possible?)

We could even use tools like gulp to provide file watching functionalities for students having their solution tested right after saving it, but this would only be possible if we support exercise-based package.json files.

We could even use tools like gulp to provide file watching functionalities for students having their solution tested right after saving it, but this would only be possible if we support exercise-based package.json files.

I am of the opinion of keeping it as simple as possible and not bring in build tools. Reasons being there is no standard one there, and they get outdated eventually.
Running against files seems more natural to me (maybe because I come from ruby where I run rspec file-name.rb).

I'm with you @tejasbubane. KISS.

@joelwallis has offered to work on this, I'm inclined to say "do it!"
@exercism/javascript unless there are any objections?  @joelwallis you will want to open a PR for 'dibs' on implementing this change.  It's simple, but touches a lot of files.  Please let me know if you want any help.

I agree with @tejasbubane: I prefer not having package.json on each exercise (it's causing lots of work on xecmascript, but it is needed in that track) and I prefer running tests specifying the spec file (jasmine hello-world.spec.js)
@joelwallis, it's all yours! We'll be happy to help on anything you need.

Awesome!
But, @matthewmorgan what did you mean with 'dibs'?

@joelwallis A claim to the right to use or enjoy something exclusively or before anyone else.

Roger that.

@joelwallis yeah it's an attempt to prevent what happened to you when you made the "duplicate" pr a few days ago.  :)

@matthewmorgan and @joelwallis sorry for bring this issue back, but I'm getting an error when I use the jasmine . command within the /hello-word directory.
When I run jasmine hello-world.spec.js the test runs as expect.
However, when I run jasmine . as the README.md instruction says I'm getting this error:
module.js:471
    throw err;
    ^

Error: Cannot find module '/Users/*****/Desktop/exercism/javascript/hello-world'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at /usr/local/lib/node_modules/jasmine/lib/jasmine.js:93:5
    at Array.forEach (native)
    at Jasmine.loadSpecs (/usr/local/lib/node_modules/jasmine/lib/jasmine.js:92:18)
    at Jasmine.execute (/usr/local/lib/node_modules/jasmine/lib/jasmine.js:197:8)
    at runJasmine (/usr/local/lib/node_modules/jasmine/lib/command.js:125:11)
    at Command.run (/usr/local/lib/node_modules/jasmine/lib/command.js:55:9)

Hi, Ronalson.
Did you tried just running jasmine, without the forwarding dot?
Joel Wallis Jucá
joelwallis.com
On May 6, 2017 6:10 PM, "Ronalson Filho"<EMAIL_ADDRESS>wrote:

@matthewmorgan https://github.com/matthewmorgan and @joelwallis
https://github.com/joelwallis sorry for bring this issue back, but I'm
getting an error when I use the jasmine . command within the /hello-word
directory.
When I run jasmine hello-world.spec.js the test runs as expect.
However, when I run jasmine . as the README.md instruction says I'm
getting this error:
module.js:471
throw err;
^
Error: Cannot find module '/Users/*****/Desktop/exercism/javascript/hello-world'
at Function.Module._resolveFilename (module.js:469:15)
at Function.Module._load (module.js:417:25)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at /usr/local/lib/node_modules/jasmine/lib/jasmine.js:93:5
at Array.forEach (native)
at Jasmine.loadSpecs (/usr/local/lib/node_modules/jasmine/lib/jasmine.js:92:18)
at Jasmine.execute (/usr/local/lib/node_modules/jasmine/lib/jasmine.js:197:8)
at runJasmine (/usr/local/lib/node_modules/jasmine/lib/command.js:125:11)
at Command.run (/usr/local/lib/node_modules/jasmine/lib/command.js:55:9)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xjavascript/issues/341#issuecomment-299666156,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApIXGhW8mHxXxFEO59j38ygxzwzY06-ks5r3OHEgaJpZM4M_iJv
.

Hi Joel,
Nope, I haven't done it before. But I did it now and this is what I got:
Started

No specs found
Finished in 0.004 seconds

Thanks for reporting! I'll take a closer look on it tomorrow.
If you don't mind, open a new issue describing the problem you're facing,
alongside outputs you're getting from your errors, and the version of
Jasmine you're using.
Also, please mention me on a comment so I can be notified.
Joel Wallis Jucá
joelwallis.com
On May 6, 2017 6:39 PM, "Ronalson Filho"<EMAIL_ADDRESS>wrote:

Hi Joel,
Nope, I haven't done it before. But I did it now and this is what I got:
Started
No specs found
Finished in 0.004 seconds
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xjavascript/issues/341#issuecomment-299667517,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApIXKLVXuxi3HYQkjQ5bVsf7dcVj2uaks5r3OiogaJpZM4M_iJv
.


Are all TokenKind.Generic tokens actually StringToken?

The following definition of TokenKind.Generic would indicate that all tokens of TokenKind.Generic should be a StringToken. (or specifically either StringLiteral or StringExpandable)
https://github.com/PowerShell/PowerShell/blob/22c6fb4a8753ec2d78b2fb1720e73366805c5e6f/src/System.Management.Automation/engine/parser/token.cs#L65-L72
However, the method Token.SetIsCommandArgument does not guarantee that.  This method is used to revert the TokenKind of special tokens when they appear as command names or arguments.  Most all the special tokens that receive this treatment are not StringTokens.
https://github.com/PowerShell/PowerShell/blob/22c6fb4a8753ec2d78b2fb1720e73366805c5e6f/src/System.Management.Automation/engine/parser/token.cs#L1200-L1208
Should the documentation above TokenKind.Generic be reworded?
Reference PR #10295.

/cc @rjmholt Could you please comment?

/cc @rjmholt Could you please comment?


feat: Add exports.types to package.json

This allows TypeScript to correctly resolve type imports when using Node's module resolution with:

"moduleResolution": "NodeNext", or
"moduleResolution": "Node16"

Refs:

Typescript docs
Node docs

Hi @marijnh, I just added a description to the PR. Apologies for not doing so when submitting.

The rest of the specifications were correct, and this change won't affect CJS and ESM resolution, so all resolution paths will work correctly. I like to run npx -y publint (see also the web tool) to make sure there aren't any errors in the exports specs, however this won't catch missing specifications like in this case. Getting these right is quite difficult! (ref).

Hi @marijnh, anything I can do to help get this PR plus this and this merged? Thank you!

I'm not seeing any problems when I import the CodeMirror modules from a TypeScript project that has moduleResolution set to node16, so I am still confused about what this adds.

See also https://github.com/ProseMirror/prosemirror/issues/1305#issuecomment-1208358246


Make the payment reccurence dropdown menu more visible

Let's add a well-known ▼ to make it clear users can select payment recurrence.

done with ac970eb50ec7b1114c12557d925d171d55df0e78


When I put "flingContainer.setAdapter(arrayAdapter);" into the OnCreate(), it can be seen when the activity start! But when I put "flingContainer.setAdapter(arrayAdapter);" into the OnClickListener of a button , the view cannot be seen!

I have same issues can you help me I want show my view after response but my view not show. My adapter constructor call but adapter getView and getCount method not call.

Have you called the method notifyDataSetChanged() of the adapter?

yes I have called notifyDataSetChanged() after get response from retrofit library but it's not working.

Can you pack the code and send it to<EMAIL_ADDRESS>If solved the problem, I would feedback to you as soon as possible.

I got the same problem using setAdapter in other function on my Fragment, getCount() and getView() were never called on adapter, but using notifyDataSetChanged() in this way
/*arrayAdapter = new CustomBaseAdapter(getActivity(), list);
flingContainer.setAdapter(arrayAdapter);
arrayAdapter.notifyDataSetChanged();
*/
it works


Add base, scroll and ZkSync

Description
Fixes #58
TODO:
We need to add ENV variables :
NEXT_PUBLIC_SCROLL_ETHERSCAN_API_KEY=
NEXT_PUBLIC_BASE_ETHERSCAN_API_KEY=

think maybe we should add api keys for all just in case anyAbi fails.
Scroll : https://scrollscan.com/myapikey
Base : https://basescan.org/myapikey
ZKSync: No need, its open

Anytime we need to add a new network in the future we just need to tweak :

update targetNetworks
Update NETWORK_DATA (inside networks.ts)
Add env variables

Thanks @technophile-04 !
Created some API Keys and added them as env vars on Vercel.


gRPC: Basic environment to work on gRPC APIs

#48 needs to be included into master to proceed further on gRPC activities.

Done in #48.


Fixes to minimal new.target implementation

Current limitations, see tests/ecmascript/test-expr-newtarget-eval-code.js:

[ ] Fix handling of eval() in program code
[ ] Fix handling of indirect eval in function code

Other limitations / to be checked:

[ ] new.target as LHS, runtime or compile time ReferenceError (current: runtime, Firefox: compile time)

Other than explicitly via Reflect.construct(), is there ever any way that new.target can be anything other than undefined or (in case of new) the same constructor it's being used in?

It would be different in super() calls but since they're not yet supported, there's currently no way for there to be a difference between the current constructor and new.target. The current implementation also takes advantage of this, and will need to be reworked when super() is implemented.

My main motivation for adding new.target is to provide a (standard) mechanism for detecting constructor calls for Ecmascript functions, which allows writing more robust code. That's one function of new.target, and the other (perhaps main function) is the ES2015 class stuff.

if (!new.target) throw Error() seems to be a common idiom, so your reasoning makes sense. :)


Integrate docsite

Resolves #266, related to https://github.com/OpenZeppelin/docs.openzeppelin.com/pull/324.

Pending items:

[ ] fix links: internal (within docs) and external (usually github)
[ ] fix API formatting

Future work, on other PRs:

splitting API docs into their own pages (opening the door to autogenerating them from natspec in the future)
automating version updates into antora.yml

Decided not to change the API part:

the None. issue is not an issue since the docsite's already inlines it next to Parameters: or Returns:. Having it in separate lines on code makes it easier to follow/edit.
unsure of mixing light with dark themes, since the dark one is better for code snippets surrounded by text. we can maybe update the API part but I guess it's better for a designer to look at it (along with the rest of the docsite, since this changes are being rushed into the docs without an actual design. It will also be easier to modify once we split the API from the rest of the docs, as with the Solidity docs.

This is how it the light theme looks in code blocks surrounded by text (ugly):


[Q] Detecting Dropped/Added Component PID with --error-analyze

Hello @lelegard
I hope you are doing well.
Before main subject, We are so appreciative of your generous project.
We are working on some TS stream that will received on IP and we need to detect errors according to ETR 101 290 standard.
Actually we need to detect which PMT's Component (audio,video and etc) would be dropped or added on real-time stream.
We could detect same Component PID that was dropped and added some seconds later (for example... according below image, we have dropped PID = 0x257 audio component and catch PAT/SDT actual error. after some seconds, we have added it again and these two errors was fixed)

But when we have dropped some components and added new components with different PIDs , tsduck can not detect any related errors. According to below images, we have dropped pid = 0x0046 and addedpid = 0x0047... and could detect these modifications with your --service-analysis tag that is a kind of polling based request and we should compare them manually. Actually we need notification based solution like your --error-analysis tag and related error (ETR 101 290 standard)

We are working with tsp command according to below script. so we need to detect these kind of add/drop actions that will have different PIDs
tsp -v -m --synchronous-log -I ip -r <IP_ADDRESS>:2000 -P analyze -i8 --error-analysis -O drop
We will appreciated if you could help us about the point.
Thank you in advance
Taha

Hi @tahaarian
Thanks for your feedback.
The need for real-time analysis is not addressed by the analyze plugin. This plugin and its command counter-part tsanalyze are offline analyzers. It only reports a global analysis of a TS over a certain period. For the record, tsanalyze was the first TSDuck tool still alive. It existed before tsp and I developed it to understand the structure of TS files.
When used with the option --interval, the analyze plugin is just some kind of offline analyzer over repetitive periods of time, giving a snapshot of the TS during each interval.
Real-time events such as new or deleted PID's and others such as table renewal or crypto-periods can be detected with the history plugin. It is less synthetic than analyze but can report individual events. The idea was to precisely analyze sequences such as changes in service structure or clear/scrambled transitions. Some events such as deleted PID's are reported out-of-sequence because there is no such event as 'this PID is deleted from now on'. You just realize later that you don't see any more packets from a PID. The user's guide contains basic examples to sort the report in a strict chronological way.

Hi @lelegard
Thank you for your quick response.
i have used history tag as you said ... it was so great and we could catch changes on PIDs
Thank you in advance
Taha

Hi again @lelegard
sorry , i had another question.
could you explain me what kind of events which tsduck will catch by history tag ?
actually we have some other points that we need to detect
maybe you could share event type list that its support them
Thank you in advance
Taha

maybe you could share event type list that its support them

Just added in the user's guide, in the documentation of the plugin history.
Version 3.25-2145, in the repo right now and will be automatically updated on the web site tonight.


Generate gRPC.podspec from build.json

This will help avoid future breakages by ensuring that the podspec is consistent with other build scripts.

This is fixed by #2064.

\o/
On Wed, Jun 17, 2015 at 11:22 AM Michael Lumish<EMAIL_ADDRESS>wrote:

This is fixed by #2064 https://github.com/grpc/grpc/pull/2064.
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/1467#issuecomment-112903271.


Member updates 2

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you added your SVG to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
No cached entry, and Transposit (member) has issues with logo: transposit.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/transposit.svg'
No cached entry, and Valve Software (member) has issues with twitter: https://twitter.com/valveoficial, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and Vorteil (member) has issues with twitter: https://twitter.com/vorteilOS, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and Sosivio (member) has issues with twitter: https://twitter.com/SosivioLtd, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Empty twitter for Rancher Federal (member): https://twitter.com/rancherfederal
No cached entry, and Kingsoft (member) has issues with twitter: https://twitter.com/KingsoftC, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
No cached entry, and Kingsoft Cloud (KCSP) has issues with twitter: https://twitter.com/KingsoftC, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
No cached entry, and Kingsoft Cloud has issues with twitter: https://twitter.com/KingsoftC, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for Standard Library: https://twitter.com/AutocodeHQ
Empty twitter for StackStorm: https://twitter.com/Stack_Storm
Item Transposit (member) has no image_data

Build failed because of:
No cached entry, and Saaras (member) has issues with logo: saaras.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/saaras.svg'
No cached entry, and Sosivio (member) has issues with twitter: https://twitter.com/SosivioLtd, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and Valve Software (member) has issues with twitter: https://twitter.com/valveoficial, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Empty twitter for Rancher Federal (member): https://twitter.com/rancherfederal
No cached entry, and Kingsoft (member) has issues with twitter: https://twitter.com/KingsoftC, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
No cached entry, and Kingsoft Cloud (KCSP) has issues with twitter: https://twitter.com/KingsoftC, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
No cached entry, and Kingsoft Cloud has issues with twitter: https://twitter.com/KingsoftC, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for Standard Library: https://twitter.com/AutocodeHQ
Empty twitter for StackStorm: https://twitter.com/Stack_Storm
Item Saaras (member) has no image_data

LGTM

@caniszczyk please merge, thanks!


com.google.api.client.googleapis.extensions.android.gms.auth.GoogleAuthIOException

I am doing the following after you guys granted the access to me. However I am still getting this exception:
com.google.api.client.googleapis.extensions.android.gms.auth.GoogleAuthIOException
CallService.getDefaultInstance().register(userId, "holaras1424", GOOGLE_CLOUD_PROJECT_ID, MainActivity.this, new CallService.Callback() {});
In GOOGLE_CLOUD_PROJECT_ID I have passed Google Project ID. I have even tried by passing Google Project Number in place of GOOGLE_CLOUD_PROJECT_ID but still I am getting the exception mentioned above.
What am I doing wrong? Am I missing something?

Resend your client details, will have it checked.


Some minor corrections

Modifications made:

Some grammar corrections
I didn't modify the input/output convention (it seems like yours is correct, while in the $\pi$-calculus the opposite is usually adopted). I just adjusted the text of the first exercise according to the convention
I applied your suggested update to the first exercise of the Ex.1 sheet
In Ex.1, I put some parentheses in (a.P + Q)\b so that your reasoning is correct
in Ex.3, corrected the definition of bisimulation

Some possible improvements (not applied in this pull request)

(probably the most important) I think the $+_L$ rule in the LTS semantics of CCS (in Ex.1) is wrong: you should have P+Q in the conclusions, instead of Q+P. I can't modify it because it's an image
You should probably take a decision about your desired interpretation of the "little.collect.collect" issue and correct the exercise accordingly (also observe that, in your last email, you didn't put any parenthesis)
In Ex.1, in the exercise "list all the different reductions that c.(\overline{a} | (b. a | d )) can perform to reach 0 | (0 | 0)": the exercise is good, because there are sequences, parallel composition and synchronization. However, asking to list all of the transitions (which are quite a few) might be too much, for someone who has never seen such transitions.
In Ex.1, you might add something easy to deal with relabeling (only if you consider this construct important enough): for example, write the derivation tree of $(b.P)[a/b] \xrightarrow{a} P$
In Ex.2, you ask to list all of the forward and backward transitions of some processes (which is a very useful exercise). You might add some easy process which has more than 1 possible backward transition, like a[k].c | b[k']. I suggest this because, after solving this exercise, I wondered "is there always only one backward transition maximum?"

That is excellent -- thanks a lot.


how to install package to the seperate directories

I  install several libraries , and they was instlled together in the dir vcpkg\installed\x86-windows\include ,can we install them in seperate dir such as \include\ffmpeg \include\sdl2.
thank you

Currently vcpkg allows each package to decide if it installs files into the top level include directory or into a subdirectory.  There is not a way to force each of them into their own directory (without modifying every portfile).
However this is a common question/request and we are interested in adding this feature sometime in the future.

thanks very much.


Does it support ANSI sequences?

something like https://github.com/binondord/AckTerm
How Ansi, VT100, ECMA-48 color displaying is implemented ?

TerminalControl is derived from CharacterDocumentViewer:
https://github.com/poderosaproject/poderosa/blob/master/TerminalEmulator/TerminalControl.cs#L42
which in turn is derived from System.Windows.Forms.Control:
https://github.com/poderosaproject/poderosa/blob/master/Core/CharacterDocumentViewer.cs#L41


Incorrect Import in ServiceProvider

Caffeinated\Modules\Support\ServiceProvider

Imports a Factory class which does not exist.
use Illuminate\Database\Eloquent\Factory;

Should be
use Illuminate\Database\Eloquent\Factories\Factory;

I would submit a pull request, but you haven't processed any of them in almost a year so I don't imagine that would get any results.

Actually I spoke too soon ... that Factory class is an abstract.
I'll leave it up to you guys to figure this out. Too many bugs for me.

Same problem with my project. Do you have any solution?
Thanks

Same problem with my project. Do you have any solution?
Thanks

Same problem with my project. Do you have any solution?
Thanks

The official update extension named  laravel/legacy-factories can fixed it,I've already tried. Look here


Add Kali-specific tasks to work around a bug in TigerVNC

🗣 Description
This pull request:

Adds some Kali-specific tasks to work around a bug in the the Debian Testing tigervnc-standalone-server package.
Makes a few changes to slightly modernize this Ansible role.

The Kali-specific changes should be reverted once tigervnc-standalone-server version 1.11.0+dfsg-2 becomes available for Debian Testing (codenamed bullseye).  I created #15 as a reminder to do this.
💭 Motivation and context
Resolves cisagov/cool-system#183.
🧪 Testing
All pre-commit hooks and molecule tests pass.  I also built a new Kali AMI with these changes and deployed it to our staging COOL environment to verify that these changes work as expected.
✅ Checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[ ] All new and existing tests pass.

Unsure why the dependabot check is dangling.


Rename package io.github.jhipster -> tech.jhipster

Related with https://github.com/jhipster/generator-jhipster/issues/12854

Superseded by #26


How enable multi line subtitle

Is there a way to display subtitles with multiple line?

I want to learn it, too.


List of expressions could be null resulting in a crash

Error message:
for (let expr of expressions) {
^
TypeError: Cannot read property 'Symbol(Symbol.iterator)' of null
addressed by adding an additional null check.

Wow, thank you!


XRT Balance not shown on staking page

When trying to stake XRT, balances are not showing properly on the dapp page (https://dapp.robonomics.network/#/staking/)
See below screenshot.

This issue is no relevant. Please open a new issue if you think you have a related issue.


less import from NODE_PATH or local node_modules. Fixes jedireza/aqua#67

Fixes #67

Thanks for creating a PR. I don't think we should do this by default, seems like something to be customized later.
Also, I grep'd the gulp-less and less.js repos but didn't see either of them using NODE_PATH, so I'm not sure how this is being affected by using the environment variable.
From: https://nodejs.org/api/modules.html#modules_loading_from_the_global_folders

NODE_PATH was originally created to support loading modules from varying paths before the current module resolution algorithm was frozen.
NODE_PATH is still supported, but is less necessary now that the Node.js ecosystem has settled on a convention for locating dependent modules. Sometimes deployments that rely on NODE_PATH show surprising behavior when people are unaware that NODE_PATH must be set. Sometimes a module's dependencies change, causing a different version (or even a different module) to be loaded as the NODE_PATH is searched.

After reading those bits, using NODE_PATH seems like a code smell.

Also, I grep'd the gulp-less and less.js repos but didn't see either of them using NODE_PATH, so I'm not sure how this is being affected by using the environment variable.

Here's how you would do it:
.pipe(Less({
          compress: true,
          paths: [ process.env.NODE_PATH || './node_modules' ]
        }))

Sometimes deployments that rely on NODE_PATH show surprising behavior when people are unaware that NODE_PATH must be set.

That's because library authors assume ./node_modules, and don't honour NODE_PATH as a first-class citizen of the Node ecosystem. I don't know why the official documentation would cede to such disuse.


Upgrade crate version to 0.2.111

r? @Amanieu

@bors r+

:pushpin: Commit 3608e3126482440e72823283a76e2e106bf6b73c has been approved by Amanieu

:hourglass: Testing commit 3608e3126482440e72823283a76e2e106bf6b73c with merge f42fe9a349300d4a07ba2ee8df4475005e1a5ef7...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing f42fe9a349300d4a07ba2ee8df4475005e1a5ef7 to master...


[microTVM][Zephyr] Add 'serial_number' option

This PR does these changes:

add serial number to specify device serial number in Zephyr testing
change Zephyr server API to run tests with specific serial number
remove west_cmd in test arguments as it is not used
refactor Zephyr server API to specify required/optional project options at the beginning of each API call.

cc @areusch @guberti for review

@gromero the change from zephyr_board to board happened in previous PRs to make it standard across the API servers. It is not part of this PR.

@mehrdadh I changed the title to add a space after the tags and tweak it a bit the wording to emphasize the new option 'serial_number' since the tweak on the tests would be a merely a consequence of it, although I do guess you're adding it for the internal test fleet.

@guberti in cortex_m docker image, the west_cmd is alos '/venv/apache-tvm-py3.7/bin/python3 -m west'
and it works without issue. Not sure what happened in that case

@guberti I added details about using serial_number in testing.

@gromero thanks for catching those! PTAL.

To whom / Committer / Maintainer that will squash & merge this PR, please consider copying the text from this PR body/description and pasting it as the commit message, avoiding the commit message prepared by GH automatically that brings the review comments, it just takes 1 second and produces a clearer git log. Also in consonance with the Commit Message Guideline. Thanks!

I rebased because of this commit: https://github.com/apache/tvm/commit/8cccc253dae1474e17eca47ce67b7a34e0c41eba

Thanks @mehrdadh @guberti @alanmacd! It's merged.


Mention this wrapper to the Ghost team

https://forum.ghost.org/t/python-sdk-library-unofficial/6504/3

If the library was updated to work with the new APIs and matched the functionality of our official JS libraries as much as possible we’d be happy to link to it from the docs

https://forum.ghost.org/t/i-wrote-a-c-wrapper-for-api-v2-0/7635


premissions

Please tick all applicable boxes.

[x] I am using Python 3.5 or higher (run python --version on the command line)
[x] I have followed the official guides to install the bot for my system
[x] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[x] The latest master version (1.9.8)
[ ] The latest review version

What type of issue are you creating?

[ ] Bug
[ ] Feature request
[x] Question

Description of issue
Hello, i think my bot cant read premission.ini
in options.ini    - OwnerID set my id
in premission file (use standart groups) i want give all commands to some users(Not role):
[MusicMaster]
UserList = MyID Friend1ID Friend2ID
MaxSongLength = 0
MaxSongs = 0
MaxPlaylistLength = 0
MaxSearchItems = 20
AllowPlaylists = yes
InstaSkip = yes
Remove = yes
SkipWhenAbsent = no
BypassKaraokeMode = yes
Extractors =
restart bot, and if Friend1ID try  "skip"   he got msg "need one more user for skip song"
Platform: Windows Server, Ubuntu 16.04

Change logs are cool.
Use skip f


[JENKINS-16365] load all available permissions from Jenkins

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests


broken output when adding a certificate fails

galaxy -env prod -pool rfmweb stack:pool_update --ssl-cert star_roomformore_com
Could not find certificate ''

should at least show what the requested cert was

Also can't add a cert if the ELB doesn't already have an HTTP listener


Multi Period Drm in DASH

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
Custom version based on version 4.7.11
What browser and OS are you using?
Chrome, Windows 10
Please ask your question
I'm looking into playing streams with multi period drm in DASH in shaka player, specifically around a use case where different periods have differing license urls(and can change over time via a dynamic manifest).
For a trimmed down example(widevine) of a manifest update where this is relevant:
<MPD>
	<Period>
		<AdaptationSet id="0" contentType="video" mimeType="video/mp4">
			<ContentProtection schemeIdUri="urn:uuid:edef8ba9-79d6-4ace-a3c8-27dcd51d21ed">
				<dashif:Laurl>license1</dashif:Laurl>
			</ContentProtection>
		</AdaptationSet>
	</Period>
	<Period>
		<AdaptationSet id="0" contentType="video" mimeType="video/mp4">
			<ContentProtection schemeIdUri="urn:uuid:edef8ba9-79d6-4ace-a3c8-27dcd51d21ed">
				<dashif:Laurl>license2</dashif:Laurl>
			</ContentProtection>
		</AdaptationSet>
	</Period>
</MPD>

The expectation would be that shaka player would retrieve license1 for the first period and license2 for the second period(you can use a request filter to do any custom context wrapping for security, but the uri in the filter should be the relevant license from the period). In practice, it seems to always request license1.
Investigation of source indicates the following:
The drm_engine maintains it's own instance of drmInfo(currentDrmInfo_), which is what gets used when the drm engine requests a new license via drm_engine.newInitData(initDataType, initData) evaluating that initData requires a new drm session. currentDrmInfo_ is not updated when encountering a new period in the manifest with different drm, and will contain the licenseServerUri of the first period.
Further digging indicates that when periods are combined via periods.combinePeriods, drm from multiple periods is flattened via getCommonDrmInfos. This process combines the initData and keyIds from both periods(into entries in an array / set for each), but the licenseServerUri is flattened to either the license from the first or second source.
I managed to hack something together locally to request the license I expected - I added a custom array to drmInfo that preserved the licenseUris from each drm when merging via getCommonDrmInfos, tweaked player.processDrmInfos to pass in the equivalent license server url to drm_engine.newInitData, tweaked drm_engine.createSession to add the license information to the metadata of the session, and finally tweaked drm_engine.sendLicenseRequest to use the license on the sessions metadata if present over the currentDrmInfo.
While it works in my narrow case, I don't think this would be a sound approach in general because it ends up mirroring data and is in essence working around some things that the code is based around, namely that there are optimizations that flatten out drm information in varying parts of the manifest and supplied configuration, and the drm engine seems designed to work with a single drmInfo(currentDrmInfo_) object that is being contextually overridden.
My limited perspective on it would be that it might be better to either have more license request information on the session metadata and use that rather than currentDrmInfo_ when sending licenses(basically a cleaner version of what I hacked together, but maybe using the full drmInfo), or to somehow have the drmEngines currentDrmInfo object track the association of licenseUrls for different periods.
Either approach seems like a decent amount of work and may have implications not readily apparent to me, so I figured I would reach out get the devs thoughts on this(and whether multi period drm is something on the radar in general).

This is currently not supported in Shaka, please create an feature request issue so we can study it. Thanks!

I thought this might be simpler than it evidently is, so I'll spend some more time to collecting things to do a proper feature request. Thanks for your time.


tag balancing problem in php

Type: Bug
Sometime adding a closing html tag changes the opening <?php tag. I've attempted to create the smallest case the reproduces this error but it seems to be dependend on timing as well as amount of code. See the video. Sometimes the <?php tag is  extended with the complete tagname (<tr?php), sometimes only the first couple off characters.
 You can start vscode with --disable-extensions or run the command "Start Extension Bisect" to try to figure out which one it is

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

To anyone with the same issue - I have a feeling that Prettier extension is the bad guy here. It seems that disabling it fixes the problem, at least it worked for me.


Add CI/CD for LIBSPDM_ENABLE_CAPABILITY_MUT_AUTH_CAP and LIBSPDM_ENAB…

Fix : #1369
Signed-off-by: Xiaohanjlll<EMAIL_ADDRESS>

You can add this to https://github.com/DMTF/libspdm/blob/main/.github/workflows/codeql-analysis.yml as well.


python310Packages.heatzypy: set version

Description of changes
Version is not set by workflow: https://github.com/Cyr-ius/heatzypy/issues/7

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 200859 run on x86_64-linux 1

2 packages built:

python310Packages.heatzypy
python39Packages.heatzypy


S3ToRedshift Implementation

For Design, Please refer:
https://wiki.cask.co/display/CE/S3toRedshift+Action

The S3ToRedshift Action plugin is in testing phase.
Following points still need to be addressed for this plugin:
Unit Testing
UI Testing

LGTM


scripts get-hepl-3 stop working

Hello All,
I want to say that i think all of you are doing great and important work here.
Since last night the get-helm-3 script stop working, when using it like so get-helm-3.sh --version v3.3.0 i get an error
get-helm-3.sh: line 309: syntax error near unexpected token 11213'`
using the script on ubuntu 18.04 (docker)
please advice

Never mind, I was able to solve this.
Thank you


same genotype call for all individuals

Kill the process with a warning message for the user when a marker is homogenous (all 0 for example).

In such situation, the marker as a sigma of 0.0. The inverse is nan.
We need to decide how this should be handled. Note that is very unlikely to happen with a normal dataset. The case appeared in the synthetic dataset with 200 individuals.


[feat] add script..

忽略 README 文件...

大哥再辛苦下，解决下冲突

你发错了吧。。
---Original---
From<EMAIL_ADDRESS>Date: Fri, Aug 21, 2020 10:14 AM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: Re: [imfht/fanhaodaquan] [feat] add script.. (#52)
大哥再辛苦下，解决下冲突
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


EarlyBinder as default

Proposal
Change all queries which currently have a bound_X variant to return EarlyBinder<T> by default and remove the bound_X version. Add a method fn EarlyBinder::<T>::subst_identity(self) -> T if these queries are used in the identity context and don't need to actually substitute anything.
Not having EarlyBinder be the default can very easily result in incorrect uses of these queries, e.g. https://github.com/rust-lang/rust/blob/736c675d2ab65bcde6554e1b73340c2dbc27c85a/compiler/rustc_hir_analysis/src/astconv/mod.rs#L2720 which I found while reviewing https://github.com/rust-lang/rust/pull/101947. The normalize_ty normalizes type_of(def_id) in the wrong environment as we only call subst afterwards.
Mentors or Reviewers
I am available to mentor and review this work.
Process
The main points of the Major Change Process are as follows:

[x] File an issue describing the proposal.
[ ] A types team member or contributor who is knowledgeable in the area can second by writing @rustbot second.

Finding a "second" suffices for internal changes. If however, you are proposing a new public-facing feature, such as a -C flag, then full team check-off is required.
Types team members can initiate a check-off via @rfcbot fcp merge on either the MCP or the PR.

[ ] Once an MCP is seconded, the Final Comment Period begins. If no objections are raised after 10 days, the MCP is considered approved.

You can read more about Major Change Proposals on forge.
Comments
This issue is not meant to be used for technical discussion. There is a Zulip stream for that. Use this issue to leave procedural comments, such as volunteering to review, indicating that you second the proposal (or third, etc), or raising a concern that you would like to be addressed.

@rustbot second

closing as accepted


feat: add rw_rate_limit system catalog

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
It's not easy to know what jobs are rate limited, so add this.
I want to avoid iterating over all fragments, so I introduced a new fragment flag for FsFetch. This means rw_rate_limit won't show fs fragments created before this PR, and I think it's acceptable.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#19466  👈 (View in Graphite)
#19445
#19443
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Oops. Seems system catalog is not isolated between databases 🤔

Oops. Seems this system catalog is not isolated between databases 🤔

Some catalogs support this, but some don't: e.g,

rw_tables is isolated
rw_fragments is not

System catalogs about metadata of the streaming job are all not isolated between databases. They are used solely for debugging.

Yes, I was also a little hesitant to add isolation which might hurt debugging. 🤪

Merge activity

Nov 20, 3:57 PM GMT+8: A user started a stack merge that includes this pull request via Graphite.