Bug: Vision5e overrides Limits

Foundry 11 build 315
D&D 3.1.2
Vision 5e: 1.14.3
Limits: 1.1.0
On a token, Darkvision is set as a racial sense to 60 feet as a trait of the Wood Elf race.
On the scene the token exists on, Darkvision is limited to 20 feet via the Limits feature.
The token has "Darkvision" (black/white) to 20 feet. It continues to have vision beyond that field to 60 feet.
Vision Range is set to 60 feet and not modifiable with a message "Automatically managed based on the Vision Mode and Detection Modes."
If I open the token and override the Darkvision value to 20 feet on the token, token's vision behaves as expected.

If it's a bug, it won't be fixed for Limits 1.x.x (V11). If this will still be an issue in Limits 2.0.0, feel free to reopen this issue.


docs: add documentation on how to use Jib with custom_build

Hello @maiamcc, @jazzdan,
Please review the following commits I made in branch nicks/jib:
5e5fb2f63666ad7054806c550783b2e208be42cc (2020-03-17 13:45:09 -0400)
docs: add documentation on how to use Jib with custom_build
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

I hacked this together while working on the Java guide. I'm not sure yet if it even makes sense to mention Jib in the Java guide, but it certainly makes sense to add notes about it in the custom_build guide.


Handler for restart makes the restart to be done late in the process

When using the role charleskorn.raspi-expanded-rootfs within a larger playbook, the handler for rebooting after changing the file system is done after all tasks has finished.
In my case that meant running out of free space before rebooting. I'd propose using a third task in
tasks/main.yml:
- name: Reboot after file system change
  reboot:
  when: unpartitioned.stdout != "0.00GB"

Then the reboot will be automatically handled directly after changing the file system but only if it was changed. Of course the notification for the handlers in the previous task also will be removed.
If you want to, I can create a pull request.

Hi @smuda, thanks for the feedback. Your suggestion makes sense to me, however I don't have any time in the near future to implement it, so a pull request would be great!

@charleskorn Thank you for your really quick response and your module as a whole.


Server streaming callable improvements

This is the cleaned up version of https://github.com/googleapis/gax-java/pull/403.  This is the full set of improvements I propose for ServerStreamingCallables. Each commit will be pulled out as a separate PR to review. This PR is meant to show the final goal and will be rebased as its parts get merged.
Currently, gax has rich support for UnaryCallable middleware that can be chained together to achieve complex chains of functionality like batched retries. This functionality is made possible by UnaryCallable's API of serializing all functionality into a single call() method. The streaming callable api, on the other hand, is a thin wrapper around gRpc primitives, which makes it very hard to build chainable middleware. The primary blocker is that ServerStreamingCallables requires the developer to implement 2 methods that have very different call semantics.
This proposal aims to:

improve the composability of ServerStreamingCallables
extend the api to make it more flexible (ie. allow it to be used with async back pressure)
add some middleware on top of the new api

The primary motivation for these changes is to allow the Cloud Bigtable client to migrate to the veneer ecosystem.
Changes:

minor prep changes: fix intellij gradle support & separate out the ServerStreaming tests into own files
add back pressure support:

add ResponseObserver that replaces ApiStreamObserver for ServerStreamingCallables
add a StreamController that allows the user to manually adjust flow control
update the GrpcDirectServerStreamingCallable to use the new interfaces

using the back pressure api, re-implement blocking iterator streaming in gax

add a ServerStream that replaces the old Iterator and duplicates the logic in grpc's ClientCalls

add a helper utility for clients to implement back pressure aware response merging logic
wrap all errors in ApiExceptions
add syntatic sugar to convert a ServerStreamingCallable into UnaryCallables: first() & all()
add watchdog middleware to prevent streams from hanging indefinitely
retries w. support for resuming streams

Codecov Report

Merging #433 into master will decrease coverage by 0.76%.
The diff coverage is 74.65%.

@@             Coverage Diff              @@
##             master     #433      +/-   ##
============================================
- Coverage     70.53%   69.77%   -0.77%
- Complexity      634      687      +53
============================================
  Files           136      147      +11
  Lines          2912     3050     +138
  Branches        220      254      +34
============================================
+ Hits           2054     2128      +74
- Misses          776      815      +39
- Partials         82      107      +25

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/com/google/api/gax/rpc/Callables.java
81.81% <0%> (-14.74%)
6 <0> (ø)

...oogle/api/gax/rpc/ServerStreamingCallSettings.java
0% <0%> (ø)
0 <0> (?)

.../grpc/GrpcServerStreamingRequestParamCallable.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...m/google/api/gax/grpc/GrpcApiExceptionFactory.java
100% <100%> (ø)
6 <6> (?)

...i/gax/grpc/GrpcExceptionBidiStreamingCallable.java
100% <100%> (ø)
2 <2> (?)

...gax/grpc/GrpcExceptionServerStreamingCallable.java
100% <100%> (ø)
2 <2> (?)

...a/com/google/api/gax/rpc/FirstElementCallable.java
100% <100%> (ø)
2 <2> (?)

.../com/google/api/gax/rpc/StreamingCallSettings.java
66.66% <100%> (ø)
2 <1> (ø)
:arrow_down:

.../java/com/google/api/gax/rpc/StreamController.java
100% <100%> (ø)
2 <2> (?)

...a/com/google/api/gax/grpc/GrpcCallableFactory.java
60.52% <40%> (-12.65%)
7 <1> (ø)

... and 77 more

Continue to review full report at Codecov.

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

Closing this now that the last part of this is out for review: #449


Implement advanced scenarios tests

mixed approach to OpenAPI document model generation (e.g.: static file + programming model + annotations)

Negative scenario about headers, see #1 (comment)

multiple and subsequent deployments/undeployments

WARN for missing HTTP listener

one test on JWT related Security annotations, see https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/quarkus-dev/QZgqrblsYEc/sgDcnHYlBAAJ

WARN for missing HTTP listener

Done, see security testing

one test on JWT related Security annotations, see https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/quarkus-dev/QZgqrblsYEc/sgDcnHYlBAAJ

Done PR to be sent once rebased after the last one (https://github.com/jboss-eap-qe/eap-microprofile-test-suite/pull/124) is merged.
Next Steps

Negative scenario about headers, see #1 (comment)

multiple and subsequent deployments/undeployments

WARN for missing HTTP listener

one test on JWT related Security annotations, see https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/quarkus-dev/QZgqrblsYEc/sgDcnHYlBAAJ

TBD PR to be sent once rebased after the last one (https://github.com/jboss-eap-qe/eap-microprofile-test-suite/pull/124) is merged. And there seems to be issues with single elements inside the SecuritySchema

Done, PR sent, see https://github.com/jboss-eap-qe/eap-microprofile-test-suite/pull/127

multiple and subsequent deployments/undeployments - left this aside and will file a proposal for memory pressure test


Fix checkoutv2 push trigger on dev

push trigger on dev

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@8085e93). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev    #2263   +/-   ##
======================================
  Coverage       ?   30.43%
  Complexity     ?     1663
======================================
  Files          ?      337
  Lines          ?    17830
  Branches       ?     2238
======================================
  Hits           ?     5426
  Misses         ?    11847
  Partials       ?      557

Continue to review full report at Codecov.

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


Presets not merging

Description
Thanks for this project, I love the idea and trying to use it in a pure TS monorepo (no intermediate bundling)...
Before being able to do this, I'm gonna need the type safety provided by using several presets from multiple packages, but I couldn't find a way to do it as the presets doesn't seem to merge...
When I use two presets in the preset field of panda.config.ts, it seem they are not merging.
Same if I use the @pandacss/dev/presets in the array with a custom preset...
Is there something I'm missing here?
Link to Reproduction
https://github.com/lalexdotcom/panda-presets
Steps to reproduce
Couldn't set it to stackblitz, the repo is a devcontainer for VSCode

Checkout
Select "Reopen in container" command
pnpm install
pnpm test

5 use cases are tested:

with a theme extension
with the default preset
with a custom preset
with the default presets and a custom one
with two custom presets

JS Framework
Not here yet (TS)
Panda CSS Version
0.6.0
Browser
No response
Operating System

[X] macOS
[ ] Windows
[X] Linux

Additional Information
My environment is a devcontainer (linux) with a macOS host...

@lalexdotcom I don't fully understand your issue, but one correction in your codebase, is that the presets should use the extend keyword
presets: [
    definePreset({
      theme: {
        extend:{
          tokens: {
            colors: { 'test.light': { value: '#112233' } }
          }
        }
      }
    }),
    definePreset({
      theme: {
        extend:{
          tokens: {
            colors: { 'test.dark': { value: '#112233' } },
            sizes: { '3xs': { value: '1px' } }
          }
        }
      }
    })],

for https://github.com/lalexdotcom/panda-presets/blob/d9f6ee07803bedb3c76259e2dc90e5087d5ab5f8/panda.config-with-two-presets.ts#L19

Yey, you got it! I totally didn't see the extend possibility for presets
I find that the subtle difference between theme and preset is a bit confusing...
Maybe there would be something to investigate here for a better DX (which is already the best I tried for many other aspects): maybe be a inherit boolean property in the theme property? It seems like there's no use of setting properties at the root of the theme object AND properties in the extend field...
In presets that can take an array, it can be confusing that an array element can cancel all the previous ones...
I don't have any clear answer for that, but maybe there's a way to make things more intuitive?
Thanks so much for your quick answer so I can go with panda!


Update 0036-valid-sudoku.java

Optimized 0036 valid sudoku (eliminated an unnecessary nested loop)

File(s) Modified: 0036-valid-sudoku.java
Language(s) Used: Java
Submission URL: _https://leetcode.com/problems/valid-sudoku/submissions/951584952/_

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

Thanks!


CE feedback slide

Add more details on how we identify CE feedback candidates based on estimates vs actual

Fixed slide to state queries that have already executed with incorrect assumptions


Wait for kind cluster to startup

There are some race conditions during startup, where if we apply config
before the cluster has started, it will never start. Best practice here
is to wait for the cluster to come up.

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
7879826ebbc42b9db50a4504298334f9e57f8e0e
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
7879826ebbc42b9db50a4504298334f9e57f8e0e
link
/test unit-tests_istio

integ-new-install-k8s-tests_istio
7879826ebbc42b9db50a4504298334f9e57f8e0e
link
/test integ-new-install-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio
7879826ebbc42b9db50a4504298334f9e57f8e0e
link
/test unit-tests_istio

integ-new-install-k8s-tests_istio
7879826ebbc42b9db50a4504298334f9e57f8e0e
link
/test integ-new-install-k8s-tests_istio

pilot-multicluster-e2e_istio
7879826ebbc42b9db50a4504298334f9e57f8e0e
link
/test pilot-multicluster-e2e_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest
https://github.com/istio/istio/issues/18258

/cherrypick release-1.4

@howardjohn: new pull request created: #18308

In response to this:

/cherrypick release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Snippet for 100% width of Dataview tables?

Hi, I'm trying to figure out which snippet of your theme is responsible for increasing the width of Dataview tables.
I'd love a way to increase the width of all Dataview tables to 100% without a specific cssclass, but regular text should retain its default width.
Any pointers?
Thanks!

Hi. Thanks for stopping by. You can have markdown tables and dataview tables with 100% width and normal note text with usual width (readable line length) but this feature currently works on reading mode only.
Add cssclasses property on the note you want to have the 100% tables width and write tw100 (tables -> width 100%) and/or tcw100 (tables and callouts -> width 100%). Tables will have 100% width in reading mode (markdown and dataview tables).
Note: if you are using MagicUser theme you should download theme version 20.8.2, because some adjustments were made for dataview tables (reading mode).
Maybe in the future I will add this feature for live preview and source modes.
All the best

I'm OK just using this in reading mode for now (though I'd love to be able to use the feature in Live Preview eventually), but would you be able to share which snippet does this?
I really like your theme, but I'd love to be able to use it with another one too. I switch sometimes.

Well, I don't have the time to adapt the theme functinality to other themes. But you can find below a snippet to add this functionality without the cssclasses tw100 or tcw100 properties, adapted from this theme (MagicUser). I tested it on the default theme and it is working well (reading mode).
Let me know if it is working as expected on the theme you selected to use.
All the best
/* =====================================================
Reading Mode: Dataview and normal Markdown tables -> full width
- Other elements will have readable line width
- No CSS class per note needed
Adapted from MagicUser theme
https://github.com/drbap/magicuser-theme-for-obsidian
MIT License
Copyright (c) 2024 Bernardo Pires (@drbap)
=======================================================
*/

.markdown-reading-view .markdown-preview-sizer,
.block-language-dataview.node-insert-event {
  width: 100% !important;
  max-width: 100% !important;
}

.markdown-reading-view .markdown-preview-sizer>div>*:not(table) {
  margin-left: auto;
  margin-right: auto;
  min-width: var(--file-line-width);
  max-width: var(--file-line-width);
}

.markdown-preview-sizer>div>table {
  width: 100%;
  margin-top: 1em;
  margin-bottom: 1em;
}

that's perfect! that's all I wanted, I didn't mean for you to have to adapt it to other themes haha
thanks so much!

I hope it will be useful.
All the best


Got SIGTERM

Getting errors in the Homebridge Log, restarting
Not sure what causes this, see attached log
[5/12/2022, 12:32:15 PM] TypeError: device.currentState is not a function
    at sono.get_status (/usr/lib/node_modules/homebridge-display/dist/sonos-controller.js:18:20)
    at Socket.<anonymous> (/usr/lib/node_modules/homebridge-display/dist/index.js:563:19)
    at Socket.emit (node:events:527:28)
    at /usr/lib/node_modules/homebridge-display/node_modules/socket.io/dist/socket.js:435:32
    at processTicksAndRejections (node:internal/process/task_queues:78:11)
[5/12/2022, 12:32:15 PM] Got SIGTERM, shutting down Homebridge...
[5/12/2022, 12:32:15 PM] Got SIGTERM, shutting down child bridge process...
[5/12/2022, 12:32:20 PM] [HB Supervisor] Homebridge Process Ended. Code: 143, Signal: null
[5/12/2022, 12:32:25 PM] [HB Supervisor] Restarting Homebridge...
[5/12/2022, 12:32:25 PM] [HB Supervisor] Starting Homebridge with extra flags: -I
[5/12/2022, 12:32:25 PM] [HB Supervisor] Started Homebridge v1.4.1 with PID: 5088

Homebridge 1.4.1
Node JS v16.15.0
Raspbian GNU/Linux Buster (10)
EDIT:
Removed Sonos from the layout, this remains:
[5/12/2022, 1:21:21 PM] TypeError: Cannot read properties of undefined (reading 'get_status')
    at Socket.<anonymous> (/usr/lib/node_modules/homebridge-display/dist/index.js:563:19)
    at Socket.emit (node:events:527:28)
    at /usr/lib/node_modules/homebridge-display/node_modules/socket.io/dist/socket.js:435:32
    at processTicksAndRejections (node:internal/process/task_queues:78:11)

Still fatal

Sorry for the late response, I have been busy with work. So the first half of the issue is hard for me to debug without a Sonos device, so for the upcoming v3 release, I will not be including Sonos. The second half is due to the client device caching the Sonos widget and still trying to use the Sonos backend. This was an oversight on my part and is already fixed in the v3 release. Thanks for pointing this out!


Enable mypy

Some of the "fixes" applied in the second commit still need more work but this should get us going with mypy a little faster.

@miyunari PR #666 is spooky. 👻


issue after pull new tensorflow modification

hi,
i want to simply try your test but i pulled tensorflow version juste before and now i have this error
File "legacy/train.py", line 184, in
tf.app.run()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/platform/app.py", line 126, in run
_sys.exit(main(argv))
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/util/deprecation.py", line 250, in new_func
return func(*args, **kwargs)
File "legacy/train.py", line 93, in main
FLAGS.pipeline_config_path)
File "/home/tensorflow/models/research/object_detection/utils/config_util.py", line 99, in get_configs_from_pipeline_file
return create_configs_from_pipeline_proto(pipeline_config)
File "/home/tensorflow/models/research/object_detection/utils/config_util.py", line 123, in create_configs_from_pipeline_proto
configs["eval_input_config"] = configs["eval_input_configs"][0]
TypeError: 'InputReader' object does not support indexing
any idea ?

i deleted the [0] but i don't know if it is OK

Once I delete the [0], I get issue at next line, if anybody solved this issue, please help. I have followed the same steps from the github
if configs["eval_input_configs"]:
configs["eval_input_config"] = configs["eval_input_configs"][0]
if pipeline_config.HasField("graph_rewriter"):
configs["graph_rewriter_config"] = pipeline_config.graph_rewriter


Unable to compile latest crystal using brew upgrade --HEAD crystal_lang

Error in ./src/fiber.cr:26: this 'initialize' doesn't explicitly initialize instance variable '@stack_top' of Fiber, rendering it nilable

The instance variable '@stack_top' is initialized in other 'initialize' methods,
and by not initializing it here it's not clear if the variable is supposed
to be nilable or if this is a mistake.

To fix this error, either assign nil to it here:

  @stack_top = nil

Or declare it as nilable outside at the type level:

  @stack_top : (Pointer(Void))?

  def initialize(@name : String? = nil, &@proc : ->)
      ^~~~~~~~~~

make: *** [.build/crystal] Error 1

Further inspection shows that the crystal compiler (0.18.7) used by Homebrew to bootstrap the latest crystal compiler is incompatible.
I've been able to patch the issue and I'm running release tests now (the debug mode passed all tests)

Yeah, tests passed for make crystal and make crystal release=true (make spec succeeded on both counts)
diff --git a/src/fiber.cr b/src/fiber.cr
index 4983662..98d03f4 100644
--- a/src/fiber.cr
+++ b/src/fiber.cr
@@ -16,7 +16,6 @@ class Fiber

   @stack : Void*
   @resume_event : Event::Event?
-  @stack_top = uninitialized Void*
   protected property stack_top : Void*
   protected property stack_bottom : Void*
   protected property next_fiber : Fiber?
@@ -24,6 +23,7 @@ class Fiber
   property name : String?

   def initialize(@name : String? = nil, &@proc : ->)
+    @stack_top = uninitialized Void*
     @stack = Fiber.allocate_stack
     @stack_bottom = @stack + STACK_SIZE
     fiber_main = ->(f : Fiber) { f.run }

Well this hard to believe to not be a setup issue, given our CI builds the new compiler with the latest release on every commit.

I removed my pre-existing Crystal compilers before hand using brew uninstall --force crystal-lang.
I then installed 0.18.7 using brew install crystal-lang
Then I invoked brew upgrade -v --HEAD crystal-lang, which pulled the master branch from this repo and threw that error.
As this occurs on a vanilla, clean build, this could be a real problem.

I wonder if anyone was caught by this problem (whatever it was):
https://twitter.com/machomebrew/status/763686348041641984 (8/11/16 6:39 AM)
if you 'brew update'd in the last 21h you need to run
   cd $(brew --repo); git fetch && git reset --hard origin/master

Sorry and RT!

I was having trouble upgrading crystal-lang from brew-version 0.18.7 to brew-version 0.18.7_1 on one of my Macs, even though I've already made that upgrade without any problems on a different Mac.  The error message I saw was Error: autocode not installed.  But then I also noticed:
# sudo brew deps crystal
node

That's it, the entire list.  While on the Mac where everything worked fine, there was a somewhat longer and more sensible list:
# brew deps crystal-lang
bdw-gc
doxygen
libevent
libffi
libyaml
llvm
openssl
pcre

So I followed the above (git fetch...) instructions on the Mac which wasn't working, and brew got much farther into the upgrade of crystal-lang.  It still couldn't do the upgrade, but now it was having trouble downloading the archive for 0.18.7 from github.  So I assume something is wrong with github, or with my internet connection (which has been a little flakey at home), or with the brew-formula for crystal-lang.
I use brew for just a few packages, so for now my solution will be to 'tar'-up my /opt/homebrew on the good mac, and then extract that on the Mac where my /opt/homebrew is in some kind of confused state.

Resetting the repo did not help.
(cpython27) benjolitz-laptop:~/ $ cd $(brew --repo)
(cpython27) benjolitz-laptop:/usr/local [master]$ git st
On branch master
nothing to commit, working directory clean
(cpython27) benjolitz-laptop:/usr/local [master]$ git fetch
remote: Counting objects: 56694, done.
remote: Compressing objects: 100% (15143/15143), done.
remote: Total 56694 (delta 40337), reused 56350 (delta 40036), pack-reused 0
Receiving objects: 100% (56694/56694), 13.45 MiB | 2.28 MiB/s, done.
Resolving deltas: 100% (40337/40337), completed with 352 local objects.
From https://github.com/Homebrew/brew
   823505a..7395ff2  master     -> origin/master
 * [new tag]         0.2        -> 0.2
 * [new tag]         0.3        -> 0.3
 * [new tag]         0.4        -> 0.4
 * [new tag]         0.5        -> 0.5
 * [new tag]         0.6        -> 0.6
 * [new tag]         0.7        -> 0.7
 * [new tag]         0.7.1      -> 0.7.1
 * [new tag]         0.8        -> 0.8
 * [new tag]         0.8.1      -> 0.8.1
 * [new tag]         0.9        -> 0.9
 * [new tag]         0.9.1      -> 0.9.1
 * [new tag]         0.9.2      -> 0.9.2
 * [new tag]         0.9.3      -> 0.9.3
 * [new tag]         0.9.4      -> 0.9.4
 * [new tag]         0.9.5      -> 0.9.5
 * [new tag]         0.9.8      -> 0.9.8
 * [new tag]         0.9.9      -> 0.9.9
(cpython27) benjolitz-laptop:/usr/local [master]$ git reset --hard origin/master
HEAD is now at 7395ff2 Merge pull request #758 from reitermarkus/cask-dev-scripts
(cpython27) benjolitz-laptop:/usr/local [master]$ brew upgrade crystal-lang
Error: crystal-lang 0.18.7_1 already installed
(cpython27) benjolitz-laptop:/usr/local [master]$ brew upgrade --HEAD crystal-lang==> Upgrading 1 outdated package, with result:
crystal-lang HEAD_1
==> Upgrading crystal-lang
==> Using the sandbox
==> Cloning https://github.com/crystal-lang/crystal.git
Updating /Users/BenJolitz/Library/Caches/Homebrew/crystal-lang--git
==> Checking out branch master
==> Downloading https://github.com/crystal-lang/crystal/releases/download/0.18.6/crystal-0.18.6-1-darwin-x86_64.tar.gz
Already downloaded: /Users/BenJolitz/Library/Caches/Homebrew/crystal-lang--boot-0.18.6.tar.gz
==> make crystal release=true
Last 15 lines from /Users/BenJolitz/Library/Logs/Homebrew/crystal-lang/01.make:
and by not initializing it here it's not clear if the variable is supposed
to be nilable or if this is a mistake.

To fix this error, either assign nil to it here:

  @stack_top = nil

Or declare it as nilable outside at the type level:

  @stack_top : (Pointer(Void))?

  def initialize(@name : String? = nil, &@proc : ->)
      ^~~~~~~~~~

make: *** [.build/crystal] Error 1

READ THIS: https://git.io/brew-troubleshooting
If reporting this issue please do so at (not Homebrew/brew):
  https://github.com/Homebrew/homebrew-core/issues

(cpython27) benjolitz-laptop:/usr/local [master]$

I've done a pull and invoked a brew upgrade -vd --HEAD crystal-lang
It was successful : Crystal e9ab595 (2016-09-06) is returned from crystal -v
Considering it a closed issue.


staff pick label missing from homepage

Yellow staff pick label on homepage also seems to have disappeared. happening on staging and prod.
working properly on both staging and prod on /volunteer/
also working properly when i filter campaigns on the homepage

Hey, already worked out the fix.

@sergii-tkachenko - :speedboat:


Added exit button on the game

PR Description 📜
I have added exit button in the game, it'll be easy and user-friendly for the players to exit from the game.

Add a video or screenshot of your PR

Sure, this is the video showing the feature I have added for this website:
https://github.com/GameSphere-MultiPlayer/Dot-Box/assets/95140559/e2b1cb62-a2e2-421a-8f58-2876d665f5b0

This is a good idea. The button is too close to the edge. You should move it closer. Also, can you resolve the merge conflict, if yes please do that. I'd like you to add border-radius to the button of lets say 5 or 8 depending on which one looks better

there is a conflict with game.js, please resolve it


How can I implement redux persist ?

Hello I would like to add redux-persist on this starter, do you know how I can change src/redux/create.js to do so ?

+1


[flagd] add reconnect e2e tests

Implement the "reconnect e2e test suite" here by adding it to the existing e2e test suite.
How to:

start ghcr.io/open-feature/flagd-testbed-unstable and ghcr.io/open-feature/sync-testbed-unstable containers (these restart every 5s)
run the gherkin suite about twice (once for in-process and once for the RPC provider)

See the Java implementation PR here.

@toddbaert I think you have missed the link to the Java implementation PR. Can you please update it?

@toddbaert I think you have missed the link to the Java implementation PR. Can you please update it?

Updated! Thanks!

@toddbaert please update the link to the flagd-testbed to add flagd-reconnect.feature in the repo.

@toddbaert please update the link to the flagd-testbed to add flagd-reconnect.feature in the repo.

I'm sorry, I don't understand your request.
Do you mean update the git submodule? If so, you should be able to do that in a branch.

Okay, I'll update that in the PR.

Sure, just CD into the submodule and do a checkout (disregard if it gives you "detatched head" errors. If you have trouble, let me know and I can do it and then you can rebase on it.

@toddbaert I tried adding reconnect e2e tests but I'm having a trouble debugging the issue with the PROVIDER_ERROR callback. The ProviderErrorCallback is not being called when connection breaks. Here is my implementation if you have time please have a look into it.

@NeerajGartia21 sorry for the delay on checking back on this.
I checked your code and I think root cause is not defining providerErrorCallback function as a global function. Can you attempt that and see if this get resolved ?


Use atexit to call wb_supervisor_simulation_quit and cleanly quit Webots

As suggested in https://github.com/cyberbotics/webots/issues/2858

@Adman @dbscoach  I guess I should have come up with this myself a long time ago :)

Useless, sorry for bugging about this.


Tests fail on a recent rebuild in Debian

https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=841576
While I am the packager, I don't know enough ruby to diagnose this, so assistance is appreciated.

Hey there, the stack-trace is missing some information to fully reproduce the errors. I've released yell-2.0.7. Can you please give it a try and get back to me.

@rudionrails Looks like 2.0.7 built and the test ran successfully -- thanks for the new release!


360Player not working on WebGl

I'm trying the scene "360Player".
It work fine in Editor but publishing the WebGL build on my ftp I get the error:
Access to XMLHttpRequest at 'https://config.uca.cloud.unity3d.com/' from origin 'http://vr.alicefit.com' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: No 'Access-Control-Allow-Origin' header is present on the requested resource.

Hi @DanieleSuppo73, seems related to the fact your app is served through http but is requesting resources from https. Check this out if you are interested in learning more
Closing this issue for now, if the issue doesn't resolve or you find other difficulties feel free to re-open/open a new issue.


fix: update github workflow to use node>=16

GitHub workflow is giving off a warning about the node version used for
some of the actions (engineerd/setup-kind) being deprecated. This commit adds a
new action to ensure about installing a specific version of node.
Signed-off-by: Daniel Mellado<EMAIL_ADDRESS>

Doing the same in prometheus-operator: https://github.com/prometheus-operator/prometheus-operator/pull/5817

maybe we can switch to https://github.com/helm/kind-action which seems to see more updates?

Yes, I also submitted a fix to engineerd/setup-kind (https://github.com/engineerd/setup-kind/pull/247)  but doesn't seem to have any traction. I'll modify and update this PR to the helm action, thanks!


Add missing attributes

When trying to port code from Native C#, you sometimes encounter, a lot of attributes. Some of these attributes are just optimizations, debugger code or documentation. Sometimes, you have to remove these from the native c#. For example, this file or this file.
There are lots of attributes:
[DataContract]
[DebuggerDisplay("{DebugDisplayString,nq}")]
[CLSCompliant(false)]
[DataMember]
[Serializable]
[ComVisible(true)]
[ContractClass(typeof(StreamContract))]
[NonSerialized]
[Pure]
[HostProtection(ExternalThreading = true)]

See Also

#2600
#1455
#135

Related to #135.

Basic support added (marked with NonScriptableAttribute) - allows compilation of C# code using these new attributes but does not add new functionality like accessing them via reflection.


cannot logoin in with the account and password in the document?

I use docker-compose to install Tasking AI, but I cannot log in with the account and password in the document.The login interface keeps getting stuck and the verification cannot be successful.

@Oatelauser Please ensure the backend service is ready. You can use docker ps to check the container status.

I checked the running status of tasksingai's containers and found that they were all normal.

When I clicked to log in, I found that the request was not sent and the front-end reported an error and blocked it.

Thanks for providing these. It looks like you are trying to login from another LAN device.

Check whether the backend service is ready: GET http://YOUR_HOST_URL:8080/api/v1/health_check. If it does not return {"status":"success"}, restart docker.
Check if login works properly in a localhost environment. If so, check your firewall and security settings.

@jameszyao I tried the method you mentioned to troubleshoot the problem, but found several problems：
1.You suspect it is a firewall problem, so I accessed the health check interface on the current machine, and the result was a 500 response.

2.After I restarted docker, I found that some containers started by docker-compose were not started.

As shown in the screenshot above, 7 containers should be started.
You can use `docker-compose -p taskingai up -d` to start the remaining containers.


-opt:l:inline breaks incremental compilation

This was originally reported by @eparejatobes as https://github.com/sbt/sbt/issues/4125#issuecomment-385992117
Copying here the relevant portion of the link above; see ohnosequences/sbt4125 for more
steps
scalacOptions ++= Seq(
  "-unchecked",
  "-opt-warnings:_",
  "-opt:l:inline",
  "-opt-inline-from:**"
)

Open a shell and
git checkout "x=1"

Now run sbt on a different shell and do ~test. Once it runs for the first time,
git checkout "x=2"

Take a look at the test passing, and the definitions of x and y.
problem

Incremental compilation when used together with the optimizer generates incorrect bytecode: modifications in methods/vals are not propagated to wherever they are being inlined.

expectation

The incremental compiler should not generate incorrect bytecode.

I'd say we revise the expectation to be that change to x=2 is reflected in compilation and run.
notes
I am not sure how prevalent these optimization flags are, but we should probably print warning or invalidate all sources, similar to 50% threshold mechanism.

Basically, if a method is inlineable, its body is part of its API signature. We deal with that in dotty by storing the pretty-printed body of the method in a signature in ExtractAPI, though it'd be better to store a hash: https://github.com/lampepfl/dotty/blob/a3c4ec793545fd2b922eb469f7fc4d4a7d26b05f/compiler/src/dotty/tools/dotc/sbt/ExtractAPI.scala#L593-L602

Interesting. That would give better usability compared to invalidating all sources :)

Basically, if a method is inlineable, its body is part of its API signature (...)

What about def f = { g(0): @inline }?

Does that actually do anything in Scala 2.12 ?

Does that actually do anything in Scala 2.12 ?

per the docs it should, and I just checked it in the repo linked above; javap yields
with call=site @inline
public int z();
    Code:
       0: iconst_1
       1: getstatic     #19                 // Field issues/sbt4125/inlineFunTimes$.MODULE$:Lissues/sbt4125/inlineFunTimes$;
       4: ifnonnull     9
       7: aconst_null
       8: athrow
       9: iconst_1
      10: iadd
      11: ireturn

no @inline
  public int z();
    Code:
       0: iconst_1
       1: getstatic     #19                 // Field issues/sbt4125/inlineFunTimes$.MODULE$:Lissues/sbt4125/inlineFunTimes$;
       4: invokevirtual #22                 // Method issues/sbt4125/inlineFunTimes$.x:()I
       7: iadd
       8: ireturn

Huh. I would vote to deprecate that feature then :P

Could we deprecate all inlining?

It can be pretty important for performance

http://slides.com/cb372/inline-specialized-berlin-2016#/27 says it's slightly slower when used with HotSpot, in that particular benchmark.

im trying my hand as this for the spree, printing warnings.

Uf, I haven't looked at the details but fixing this it's quite complicated. We would need to teach Zinc:

To predict inlining that happens in genbcode (Zinc happens after pickler, bytecode optimizations happen at the end of the pipeling)
To understand the semantics of inlining (they can be complicated, not an easy task given how many heuristics they are involved)

I vote to:

Disable incremental compilation when inlining is involved.
Notify the user of this behaviour and advise him to not enable inlining by default in their build.

I still think it makes sense for the scalac bridge to implement what I said in https://github.com/sbt/zinc/issues/537#issuecomment-385999145

I still think it makes sense for the scalac bridge to implement what I said in #537 (comment)

Missed that. Can the scalac genbcode ever inline something that is not marked as @inline? If it can't, then I like your solution. This isn't a high priority for me, so help on this is welcome.

Let's start the petition for the call-site inline removal. Where do we start?

I suggest making a PR to scala/scala that drops the feature and put a warning in its place.

this is a terrible feature that should be removed :).

Would love to see both reasons and tests for this. I see https://github.com/sbt/zinc/issues/537#issuecomment-421637743 as a much more reasonable compromise.

Currently (2.13.12),
$ scalac '-opt:inline:<sources>' -d /tmp/sandbox src/main/scala/separateFile.scala src/main/scala/4125.scala
$ javap -c '/tmp/sandbox/issues/sbt4125/useInlinedStuff$.class'
Compiled from "separateFile.scala"
public final class issues.sbt4125.useInlinedStuff$ {
  public static final issues.sbt4125.useInlinedStuff$ MODULE$;

  public static {};
    Code:
       0: new           #2                  // class issues/sbt4125/useInlinedStuff$
       3: dup
       4: invokespecial #12                 // Method "<init>":()V
       7: putstatic     #14                 // Field MODULE$:Lissues/sbt4125/useInlinedStuff$;
      10: return

  public int z();
    Code:
       0: iconst_1
       1: getstatic     #21                 // Field issues/sbt4125/inlineFunTimes$.MODULE$:Lissues/sbt4125/inlineFunTimes$;
       4: pop
       5: iconst_1
       6: iadd
       7: ireturn
}

The inline annotation is not required.
Also, everyone is 5 years older since the previous comments.

The inline annotation is not required.

Yeah... I honestly thought that https://github.com/sbt/zinc/pull/1310 would cut it, until I tested the PR fix against 2.13.12...

Also, everyone is 5 years older since the previous comments.

Maybe I can advertise myself as a historian as I now have experience with digging up 5 year old archaic Github ticket.

c.c. https://github.com/scala/scala/pull/10633#issuecomment-2260107926


feat: add thrift language id support

I want to develop thrift lsp plugin, so thrift language id is needed here.


Enable JIT in workflows

Tested locally with act as much as I could, since Github wasn't running it.  Seems to be working.
edit: Of course the ARM tests that I couldn't run locally failed in CI.  Ah well.

Looks like everything runs, and luckily only one platform has a bug (that the current test suite discovers): Windows x64 with JIT.

It is stiil a progress. We should not enable Windows 64 then at the moment until we understand what is happening there.

I disabled the JIT tests on x64 Windows for now, plus added a message so we know it's disabled.


Can't summon Generic instance for T within T's companion object: error super constructor cannot be passed a self reference

As an example, the following code breaks:
import shapeless._

case class Abc(x: Int)

object Abc extends WithGeneric[Abc]

abstract class WithGeneric[T: Generic] {
  val generic: Generic[T] = implicitly[Generic[T]]
}

with error:
Error:(9, 21) super constructor cannot be passed a self reference unless parameter is declared by-name
object Abc extends WithGeneric[Abc]
                   ^

Note that changing the name of the object makes it work:
object NotAbc extends WithGeneric[Abc]

Our use case for this is that we want to be able to define helper traits that provide circe & pureconfig codec instances for case classes:
abstract class WithCirce[A: DerivedEncoder: DerivedDecoder] {
  implicit val enc: Encoder[A] = implicitly[DerivedEncoder[A]]
  implicit val dec: Decoder[A] = implicitly[DerivedDecoder[A]]
}

Both circe and pureconfig are based on shapeless, so right now we can't write this and have to workaround by using Cached[Lazy[_]]. Non-shapeless based derivations work as expected however.

The trouble is the materialized Generic[T] where T is instantiated at a case class will embed a reference to the case class companion. Hence your construct object Abc extends WithGeneric[Abc] is self-referential in a way that's not easily fixed.
It's just conceivable that byname implicits in 2.13.x will make it possible to make some progress, but for now I don't think there's a lot that can be done.

It's just conceivable that byname implicits in 2.13.x will make it possible to make some progress, but for now I don't think there's a lot that can be done.

No progress unfortunately

This could be mitigated if case classes use constructors and field access instead of the companion's apply / unapply but it's tricky to do. I remember attempting that at some point and it didn't work.


Link to old documentation

Hi, I know time goes on and versions too, but I think it would be useful If you store old docs somewhere in https://www.ember-bootstrap.com/number-of-version-here to make easier maintain projects based on old version that we can't migrate to new at the moment. Thanks a lot for all your work :)
BTW: if there is already the old documentation somewhere please just post the link here and close the issue. Thanks again
Thank you for taking the time to open an issue!
For general questions please use the #e-bootstrap channel on the Ember Discord server.
For bug reports please include the following section in your issue details.
If you include instructions on how to reproduce the bug or a failing test case
it will make it easier for us to track down the issue you're having.

Output from ember bootstrap:info:
[Replace this line with the output.]

Yes, I agree that would be very useful. But currently we don't have this set up unfortunately.

No problem. If you have an old documentation snapshot (markdown or html or
anything) I can host it somewhere then post here the link
Il giorno dom 11 ago 2019 alle 04:19 Simon Ihmig<EMAIL_ADDRESS>ha scritto:

Yes, I agree that would be very useful. But currently we don't have this
set up unfortunately.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kaliber5/ember-bootstrap/issues/869?email_source=notifications&email_token=AABC2KWXRB6NSSBQRBEECVDQD4WHVA5CNFSM4IKSMGM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4AVFOI#issuecomment-520180409,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABC2KV3UT72BAV6IKVNR3TQD4WHVANCNFSM4IKSMGMQ
.

--
Stefano Valicchia - Giardiniere Informatico
Computers are like air conditioners, they stop working properly if you open
Windows.

Hi @giano. I was having the same issue, being stuck on an older version of ember-bootstrap for now.
If you didn't figure this out already, you can check out the ember-bootstrap repository, switch to the gh-pages branch and then look through the history to find an entry where the CHANGELOG.md file contains the version you want to look at.
Check out that commit and then open index.html in the root directory in your browser.
In the image below you can see that I wanted to look at the 2.4.0 docs.

Thank you very much for this hint, I will download the according version
when I'll be back at work. Is there all old documentation there (API, etc)?
Thanks
Il giorno ven 16 ago 2019 alle 13:07 Jeff Cressman<EMAIL_ADDRESS>ha scritto:

Hi @giano https://github.com/giano. I was having the same issue, being
stuck on an older version of ember-bootstrap for now.
If you didn't figure this out already, you can check out the
ember-bootstrap repository, switch to the gh-pages branch and then look
through the history to find an entry where the CHANGELOG.md file contains
the version you want to look at.
Check out that commit and then open index.html in the root directory in
your browser.
In the image below you can see that I wanted to look at the 2.4.0 docs.
[image: Screen Shot 2019-08-16 at 3 31 46 pm]
https://user-images.githubusercontent.com/28325/63146873-9b24e280-c03b-11e9-8d84-6d39ac10f162.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kaliber5/ember-bootstrap/issues/869?email_source=notifications&email_token=AABC2KRYIBLWRWIOGQFIL63QEY72HA5CNFSM4IKSMGM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4NYFSI#issuecomment-521896649,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABC2KX6V4IR5KZYJNTAT5DQEY72HANCNFSM4IKSMGMQ
.

--
Stefano Valicchia - Giardiniere Informatico
Computers are like air conditioners, they stop working properly if you open
Windows.

Yeah @giano, it's the same as if you were looking at it on the current website. gh-pages is the branch used to build the docs website.


Add access token from camel context to the Default Webclient

I'm facing a problem where I could not inject access token retrieved in my context to the default WebClient bean of the GraphQL Client.
My application is a wrapper service which exposes a REST Service and it communicates to the backend GraphQL Server. At the server side the service is protected with OKTA OAUTH2.0, hence each request which I make should have a valid bearer token. At this moment the out of the box configuration provides all facility to generate a OAUTH token and call the backend server.
But, the trick here is in order to generate a OAUTH token successfully, I should retrieve the credentials (basically my REST service is exposed with Basic auth) and then add them to my token request to backend.
For instance, i would like to dynamically replace the value of "accessToken" here - https://github.com/Latsode/spring-graphql-shopify-client-sample/blob/master/src/main/java/com/client/shopify/graphql/util/ShopifyHeaderExchangeFilter.java
I tried creating my own Exchange filter but with no luck. Also, tried overwriting the GraphQlClient on MutationExecutor class etc..
My service is a spring-boot application built with Apache Camel. I use the service for Mutation with the Client code generated by by the plugin.
`
@Bean(name = "webClient")
public WebClient webClient(String graphqlEndpoint) {
try {
ExchangeStrategies exchangeStrategies = ExchangeStrategies.builder()
.codecs(configurer -> configurer.defaultCodecs().maxInMemorySize(-1)).build();
       String xsrfToken=java.util.UUID.randomUUID().toString();

        return WebClient
                .builder()
                .filter(addTokenFilter())
                .baseUrl(graphqlEndpoint)
                .defaultHeader(HttpHeaders.CONTENT_TYPE, MimeTypeUtils.APPLICATION_JSON_VALUE)
                .defaultHeader("ECM-CS-XSRF-Token",xsrfToken)
                .defaultCookie("ECM-CS-XSRF-Token",xsrfToken)
                .defaultUriVariables(Collections.singletonMap("url", graphqlEndpoint))
                .exchangeStrategies(exchangeStrategies)
                .build();

    } catch (Exception e) {
        throw new ExceptionInInitializerError(e);
    }
}

public ExchangeFilterFunction addTokenFilter() {
    return (request, next) -> {

        LOGGER.info("Creating renewTokenFilter");
        if (request.url().getPath().endsWith("/graphql")) {
            String token = request.body().toString();
            ClientRequest filtered = ClientRequest.from(request)
                    .header(HttpHeaders.AUTHORIZATION, accessToken) // I need to replace the token dynamically here.
                    .build();
            return next.exchange(filtered);
        } else {
            return next.exchange(request);
        }
    };
}`

Hello,
This could fail for lots of reasons. So it's difficult to give a valid answer here.
And this is more a spring boot question than a question about this plugin.
Did you check pages like this one: https://www.baeldung.com/spring-boot-add-filter ?
Étienne


plot_calibration_purchases_vs_holdout_purchases taking for ever to run

I don't think the lambda function is necessary. It's looping each value to make a prediction.

Riiight, this looks like a big perf win. Are you comfortable submitting a PR, or would you like to leave it to me?

Sure, I would be happy to assist with the PR. Thanks for the awesome package @CamDavidsonPilon .

Riiight, this looks like a big perf win. Are you comfortable submitting a PR, or would you like to leave it to me?

Sure, I would be happy to assist with the PR. Thanks for the awesome package @CamDavidsonPilon .


Add generic scripts to aid in building the prognostic run environment

This PR generalizes some scripts used in building the prognostic run docker image so that they can also be used in building code on Gaea or other platforms.  I plan on making a followup PR with the code to actually create the environment on Gaea (this will involve updates to the fortran repo as well).
Coverage reports (updated automatically):

test_unit: 83%

I rebuilt and pushed the regular and GPU base images on my VM after 86e0855.


Broken Download link

Link  to download "Team Foundation Server Standalone Office Integration 2015 (free)" is broken

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

ID: 505849d3-01c8-a2cd-dc6c-271dd1120e2d
Version Independent ID: 577fc252-60e4-f670-bb92-6bdbcb7d5c69
Content: Bulk modify using Excel - Azure Boards and TFS
Content Source: docs/boards/backlogs/office/bulk-add-modify-work-items-excel.md
Product: devops
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@RodrigoGroener - thank you for the feedback. I'm updating the links as needed.


Add Arabic Restaurant Reviews

Checklist for adding MMTEB dataset

Reason for dataset addition:

This PR adds a dataset for Arabic restaurant reviews sentiment classification.

[x] I have tested that the dataset runs with the mteb package.
[x] I have run the following models on the task (adding the results to the pr). These can be run using the mteb run -m {model_name} -t {task_name} command.

[x] sentence-transformers/paraphrase-multilingual-MiniLM-L12-v2
[x] intfloat/multilingual-e5-small

[x] I have checked that the performance is neither trivial (both models gain close to perfect scores) nor random (both models gain close to random scores).
[x] I have considered the size of the dataset and reduced it if it is too big (2048 examples is typically large enough for most tasks)
[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.
[ ] I have added points for my submission to the POINTS.md file.

Hi @imenelydiaker @KennethEnevoldsen, I've updated the metadata eval split and reran evaluations. sorry for the misunderstanding.

Hi @imenelydiaker @KennethEnevoldsen, I've updated the metadata eval split and reran evaluations. sorry for the misunderstanding.

No worries!
Almost everything is looking good now. If you can address the two issues above I believe we should be there.

I'm not a fan of creating a test split out of a train split that is available on HF, imo it should be done on the source dataset. Else, the whole train split can be used to train models and then when evaluating with mteb on this same dataset we will encourage data leakage because the test set is created from the train one.

I agree. But a large number of datasets on the hub have only a train split and most of them have no activity. In my opinion, a possible solution is to host the dataset after creating the splits. This will also ensure stability.

I think using the "train" split is valid (multiple datasets use the train split), the best solution would naturally be to rehost it using only the desired splits (which could be new).
Rehosting with a train/test split which has the downside of seeming like we are testing something other than the original "train" split, which we naturally can't.
@imenelydiaker I agree with you that relabelled in the dataset_transform as a "test" split is problematic, but as long is it is stated as "train" I believe it is fine.

Okay let's keep the train split only.

Merging this into a branch to add the points, then I will merge it into main


Limit All is in the wrong place on the PowerShell command

It should be Get-SPWebApplication | Get-SPSite  -Limit All | Format-Table -Property URL,ContentDatabase
Otherwise you get an error because get-spwebapplication doesn't know what to do with -Limit All

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

ID: c6520bb3-81b9-3834-906a-5af3afc0ef1c
Version Independent ID: a588621a-ada7-847c-97ab-0ab741cf207c
Content: View all site collections in SharePoint Server
Content Source: SharePoint/SharePointServer/sites/view-all-site-collections.md
Product: sharepoint-server-itpro
GitHub Login: @bentoncity
Microsoft Alias: stevhord

@dubahu - Thank you for the catch and for taking the time to submit feedback. It is really appreciated!
I was able to get this into the backlog for the SharePoint team. The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better.
I made a note to request the team to update this when the work is complete.

@dubahu - I've revised this PowerShell command as per your suggestion and republished this topic.


Access Error to the PlayerBoard class of Netherboard Bukkit API

Hello,
I have an issue with the Netherboard Bukkit API. I added the dependency to my pom.xml file, and also added the .jar file to my project using my IDE's configuration options.
However, when I try to use the API's classes in my code, I get the error "cannot access fr.minuskube.netherboard.api.PlayerBoard".
I have checked that I am importing the necessary class correctly and that my dependency is correctly configured, but I am unable to resolve this issue.
Can you help me understand what is not working and how to solve this problem?
Thank you in advance for your help.
Pom.ylm :

fr.minuskube
netherboard-bukkit
1.2.3

ScoreboardManager.java :
public class ScoreboardManager {
public static void setScoreboard(Player player) {
    BPlayerBoard board = Netherboard.instance().createBoard(player, "Prison - Saison 1");

    LocalDate date = LocalDate.now();
    DateTimeFormatter formatter = DateTimeFormatter.ofPattern("dd/MM/yyyy");

    board.set("dsq", 1);
}

}
I am using IntelliJ IDEA Community Edition 2023.1 as my IDE and using Minecraft version 1.19.4.

Hello, I had the same issue when passing to 1.2.3
I think it's a mistake that the netherboard-bukkit:1.2.3 doesn't have the netherboard-core dependency.
What you need to do is adding this dependency :
<dependency>
    <groupId>fr.minuskube</groupId>
    <artifactId>netherboard-core</artifactId>
    <version>1.2.3</version>
    <scope>provided</scope>
</dependency>


Add new shape: ellipse

I added a new shape: ellipse.

Thank you for the PR. I'll have a look at this today.


Update TOC.md

Updated end user training video link

Docs Build status updates of commit 4de139a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Articles/TOC.md
:white_check_mark:Succeeded

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


Fix undefined string

Seems like we have less coverage without Rails...

Thanks! Yes, the rake pg task runs the unit specs without ActiveSupport loaded, and would have caught this bug, but I let a rogue activesupport require sneak into the unit specs a little while back (it was part of a PR, but I should have caught it). Sorry about that :frowning:
Fixed the specs, merged this, and released 0.11.2. Thanks again!

No problems and no worries, thank you! :D


Bump Requests Dependency Version to <= 2.25.99999

The Requests library has rolled out a few releases that are compatible with eventsourcing, so this PR simply brings the allowed versions of Requests up to date. One thing to note - version 2.25.x will be the last minor version to support Python 3.5, but this library requires 3.6 or greater anyway, so there's no issue there.

Added context behind the "why" of this PR - I am working on integrating eventsourcing with FastAPI which currently have incompatible version requirements for Requests. This PR will reconcile those differences and allow eventsourcing and FastAPI to be used in tandem to create an async, event-sourced HTTP API microservice.

Pull Request Test Coverage Report for Build 2921

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.2%) to 97.2%

Files with Coverage Reduction
New Missed Lines
%

eventsourcing/application/pipeline.py
1
96.3%

eventsourcing/domain/model/entity.py
1
99.23%

Totals

Change from base Build 2918:
0.2%

Covered Lines:
5381

Relevant Lines:
5536

💛  - Coveralls

Thanks a lot @bartboy011! I will release this change as v8.2.5 tomorrow.

@johnbywater of course! Thank you for getting this out so quickly, much appreciated!

Just released to PyPI @bartboy011. I also updated the versions of the other dependencies.
https://eventsourcing.readthedocs.io/en/stable/topics/release_notes.html#version-8-2-5-released-22-dec-2020


Enhance error code mapping

How to categorize this PR?

/area ops-productivity
/platform alicloud
/kind enhancement
/priority normal
What this PR does / why we need it:
Enhance the error code mapping for ErrorCode: InvalidCidrBlock.Overlapped in AliCloud.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

Updated PR to add one more mapping for azure
/platform azure

Can you elaborate under which circumstances the InvalidCidrBlock error occurs because at a fist glance it rather sounds like a configuration problem?

The error code returned from the AliCloud API was actually: InvalidCidrBlock.Overlapped because there was some other subnet with a conflicting CIDR. So I think, this fits into the infra dependencies error code, right?
Shall I rather put the full InvalidCidrBlock.Overlapped into the regex?

Okay, thanks for explaining. I'm fine with it and was just curious.


added allowEdit support for android

Changes

Allows user to crop selected image from gallery or captured from camera if allowEdit is set to true
Fallback if device does not support com.android.camera.action.CROP

there is still problem with cropping in android, when  choose to capture image.
I can׳t crop, why ?

Same as @roysG, I'm not able to edit the captured image.

Same as @roysG, I'm not able to edit the captured image. I have a Nexus 4.

Read the quirks
Android Quirks
Android also uses the Crop Activity for allowEdit, even though crop should work and actually pass the cropped image back to Cordova, the only one that works consistently is the one bundled with the Google Plus Photos application. Other crops may not work.
So, you have any app that can crop like google plus photos? in case you have it, does it open?


OCPCLOUD-2046: Manual rebase kubernetes 1.27 and go 1.20

This PR is based on the latest rebasebot rebase. I have squashed some of the commits and cherry-picked the ART go 1.20 commit on top.

/jira refresh

Hi @RadekManak , We also need to update https://github.com/openshift/cloud-provider-powervs repository similar to this,
cloud-provider-powervs repo shares most of the code with this repo with only few additional commits to support Power VS platform, Please let us know the plan or approach to update repo.

/approve
/lgtm

@Karthik-K-N We expect you to be taking care of powervs repo. There is open rebasebot PR https://github.com/openshift/cloud-provider-powervs/pull/24, It looks like it needs some manual fixes. You should be able to check out the rebasebot PR, update it and create PR with your fixes.
Looks like release-4.13 was not updated to use kubernetes 1.26 either. It is likely okay to not update release-4.13 and only update main, but if you want to fix it, you will have to run rebasebot manually or do the rebase yourself.
Feel free to message me on Slack if you need help.

/retest

/label qe-approved

/label jira/valid-bug


Add latest tag by default to PullOptionsBuilder

What did you implement:
PullOptionsBuilder now adds a latest tag when creating it.

Closes: #179
How did you verify your change:
What (if anything) would need to be called out in the CHANGELOG for the next release:
I added necessary lines to CHANGELOG.

Would merge, but you'll need to rebase first :laughing:


Language Server Error

Does this occur consistently?  Yes
Repro steps:

Open ARM template in Vs code

Action: Language Server Error
Error type: Error
Error Message: The connection to the ARM Template Language Server got closed.
Version: 0.15.8
OS: darwin
OS Release: 22.3.0
Product: Visual Studio Code
Product Version: 1.75.1
Language: en

Call Stack
extension.bundle.js:209:204152
callWithTelemetryAndErrorHandlingSync extension.bundle.js:16:248270
WrappedErrorHandler.closed extension.bundle.js:209:204035
LanguageClient.handleConnectionClosed extension.bundle.js:16:170156
LanguageClient.handleConnectionClosed extension.bundle.js:209:71022
closeHandler extension.bundle.js:16:168118
CallbackList.invoke vscode-jsonrpc/events.js:55:39
Emitter.fire vscode-jsonrpc/events.js:117:36
closeHandler vscode-jsonrpc/connection.js:256:26
CallbackList.invoke extension.bundle.js:16:310204
Emitter.fire extension.bundle.js:16:310988
StreamMessageWriter.fireClose extension.bundle.js:209:146917
Socket.<anonymous> extension.bundle.js:209:147780

Same issue for me. July 2023 release of Visual Studio Code on MacOS.

Extension output
Using dotnet core executable at ***/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/6.0.20/dotnet
Starting ARM Template Language Server at ***/.vscode/extensions/msazurermtools.azurerm-vscode-tools-0.15.10/languageServer/Microsoft.ArmLanguageServer.dll
Language server nuget version: 3.0.0-preview.23156.3
Client options:
{
  "documentSelector": [
    {
      "language": "arm-template",
      "scheme": "file"
    },
    {
      "language": "arm-template",
      "scheme": "untitled"
    }
  ],
  "diagnosticCollectionName": "ARM Template Language Server diagnostics",
  "outputChannel": {
    "name": "Azure Resource Manager Tools",
    "extensionPrefix": "azureResourceManagerTools",
    "_outputChannel": {
      "name": "Azure Resource Manager Tools"
    }
  },
  "revealOutputChannelOn": 3,
  "synchronize": {
    "configurationSection": "azureResourceManagerTools"
  },
  "middleware": {}
}
Server options:
{
  "run": {
    "command": "***/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/6.0.20/dotnet",
    "args": [
      "***/.vscode/extensions/msazurermtools.azurerm-vscode-tools-0.15.10/languageServer/Microsoft.ArmLanguageServer.dll",
      "--logLevel",
      "Warning"
    ],
    "options": {
      "shell": false
    }
  },
  "debug": {
    "command": "***/Library/Application Support/Code/User/globalStorage/ms-dotnettools.vscode-dotnet-runtime/.dotnet/6.0.20/dotnet",
    "args": [
      "***/.vscode/extensions/msazurermtools.azurerm-vscode-tools-0.15.10/languageServer/Microsoft.ArmLanguageServer.dll",
      "--logLevel",
      "Warning"
    ],
    "options": {
      "shell": false
    }
  }
}
Failed to create CoreCLR, HRESULT: 0x80004005
Error: The connection to the ARM Template Language Server got closed.
[Info  - 9:35:18 AM] Connection to server got closed. Server will restart.
Failed to create CoreCLR, HRESULT: 0x80004005
Error: The connection to the ARM Template Language Server got closed.
[Info  - 9:35:18 AM] Connection to server got closed. Server will restart.
Failed to create CoreCLR, HRESULT: 0x80004005
Error: The connection to the ARM Template Language Server got closed.
[Info  - 9:35:18 AM] Connection to server got closed. Server will restart.
Failed to create CoreCLR, HRESULT: 0x80004005
Error: The connection to the ARM Template Language Server got closed.
[Info  - 9:35:18 AM] Connection to server got closed. Server will restart.
Failed to create CoreCLR, HRESULT: 0x80004005
Error: The connection to the ARM Template Language Server got closed.
[Error - 9:35:18 AM] Connection to server got closed. Server will not be restarted.
Error: ARM Template Language Server: An error occurred starting the language server.

Connection to server got closed. Server will not be restarted.
Language server failed on start-up: ARM Template Language Server: An error occurred starting the language server.

Connection to server got closed. Server will not be restarted.: Error: ARM Template Language Server: An error occurred starting the language server.

Connection to server got closed. Server will not be restarted.
	at ***/.vscode/extensions/msazurermtools.azurerm-vscode-tools-0.15.10/dist/extension.bundle.js:16:294550
	at Generator.throw (<anonymous>)
	at rejected (***/.vscode/extensions/msazurermtools.azurerm-vscode-tools-0.15.10/dist/extension.bundle.js:16:285678)


Enable setting FlightDescriptor on FlightDataEncoderBuilder

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
When using FlightDataEncoder to send arrow batches to a flight do_put endpoint, FlightDescriptor cannot easily be set on first message. https://arrow.apache.org/docs/format/Flight.html#uploading-data
Describe the solution you'd like
On FlightDataEncoderBuilder a with_flight_descriptor method would be nice. The FlightDescriptor would then be set on the first FlightData retrieved from the encoder.
Describe alternatives you've considered
Here is a workaround, I may be missing something simpler
let mut encoder = FlightDataEncoderBuilder::new()
    .with_schema(schema.clone())
    .build(stream);

let mut first_flight_data = encoder
    .next()
    .await
    .expect("no flight data")
    .expect("flight data was error");
first_flight_data.flight_descriptor = Some(FlightDescriptor {
    r#type: DescriptorType::Path.into(),
    path: vec!["table_name".to_string()],
    cmd: Bytes::default(),
});
let put_stream = stream::once(async { Ok(first_flight_data) }).chain(encoder);

Thanks for the suggestion @alexwilcoxson-rel  -- I agree this sounds like a nice addition to the API.
Given the request is well specified and the code is fairly easy to test, I think this would be a good first issue for someone to work on if they wanted, so marking it thusly

label_issue.py automatically added labels {'arrow'} from #4101


Fix arg typo and add sample_antecedent to the training process

It looks like there is a typo and a missed command in the README.md:

arg --base_model  should be --base
command python3 sample_antecedents.py --dataset <DATASET> --base <BASE_MODEL> --gpu <GPU> is required before pretraining the consequent estimator.

Please review if the sample antecedent command is rightful to get the desired result.

Thanks for your comments.
I merged your suggestion.


ME conduit compatibility

Compact Machines version: 1.12.2-3.0.13-b326
Forge version: 2800
I am not using Optifine: Optifine breaks everything, nope
Link to Crashlog: n/a
Preface: I know about the capability adapter, however this applies to the ME conduits within ender io (a different item) and that mod doesn't have a proven track record of updating, so I can't use it. I also know about quantum tunnels, however having that giant thing in my base rather defeats the point.
I would like to request compatibility be added for ME conduits and dense ME conduits so that they can connect through tunnels without additional blocks.

Sorry, no. I won't add custom mod compatibility. It's the mods job to do that.
Request them to support world-aware, sided forge capabilities so compact machines (and other mods like capability proxy) are automatically compatible to them.

Ah alright, I wasn't sure which side needed to add it. I will go make a report over there.

I went up the chain (as you can see) and was essentially told that it is up to you to implement the ae2 grid for this to work as compatibilities just won't work well for ae2. I was also told that you intentionally do not support anything other than capabilities. Why I do not know, but as this kind of behavior goes against what I want, I will be electing to remove the mod from my pack.

I indeed do not support other methods, because it is way to much work to keep up with all the mods that I would need to support - i had enough experience with that in cm1 that i nearly fully dropped supporting the mod at all.
Forge Capabilities is the API I'm using. It's an official forge api that is encouraged to be used by mod authors for mod interaction. I'm not sure why you think I need to provide another way or add custom integrations that are not using that system myself.
I fully understand the reasoning behind the AE devs decision (hey guys) not to use capabilities, but in the end that means it won't work with compact machines (or other mods proxying capabilities) directly. And that's fine. Especially because there is a fix with ruifungs capability adapter.

Sad, I had planned out my entire AE2 system into a set of Compact Machines (Crafting Processors, Storage, ME Controller, TD machines) and started building the machines, only to discover that while they show the cables inside and out they cannot connect unlike most other mods.
Are you sure you can't work with the other mod guys to make something happen?

It has nothing to with "not being able to work with other mod guys" or not having the technical know how. It was a feature in CM1 after all... It was a deliberate decision not to open these floodgates again. A design decision to go the way of least resistance and make as much as possible compatible out of the box.
This crying on the highest of levels is really annoying and unnecessary. There already is a way to solve this issue by placing a single block on the in- and outside of the tunnels. Does the capability adapter block not work for you or do you just don't know about it, @Sorbe?


[Question] WebApiClient intellisense

Please help me how I can get intellisense for WebApiClient in my Visual Studio 2017
Thanks

Hey @phuocle,
Are you using TypeScript or plain JS?
For TypeScript typings are shipped with the code, you'll just have to import 'xrm-webapi-client'.
For plain JS you'll need to add WebApiClient non-minified to your project and in all scripts where you need it, add a reference tag at the very top.
This might look like this:
XML /// <reference path="WebApiClient.js" />
This is a relative file path, you'll have to adjust it to the location where you saved it.
Kind Regards,
Florian

Hey @phuocle,
I guess you got it working?
I'll close this issue, please reopen if it did not work out for you.
Kind Regards,
Florian


Feature Request: Support for Conan packages

Request for support for Conan C++ packages.

Conan is getting more and more traction, and its support will be beneficial for both Conan and Dependabot.

Is it planned to add conan to dependabot?

Please, add Conan to this awesome service!

+1

Looks like they won't be adding new ecosystems
https://github.com/dependabot/dependabot-core/blob/main/CONTRIBUTING.md#why-have-we-paused-accepting-new-ecosystems

This is really sad... C/C++ are pretty much a MUST with Connan... :S Maybe if they provided a the underlying infrastructure to support the Github Enterprise...

This commit could reanimate the work on that topic.
https://github.com/dependabot/dependabot-core/commit/0f931f452e9e51a3996a010ca21c90115b7f80cf


[BUG] First kubectl command by an admin on an imported cluster produces an error

Rancher Server Setup

Rancher version: v2.7.2
Installation option (Docker install/Helm Chart):

local

Information about the Cluster

Kubernetes version: v1.26.3
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):
custom: k3d cluster

k3s: v1.25.7
k3d: v5.4.9

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

Admin

Describe the bug

The first kubectl command issued after importing a cluster by an admin user produces an error message. Subsequent kubectl commands on the same cluster are successful.
To Reproduce

I was able to reproduce this using the following steps in a local instance of rancher:

Create an admin user (Hamburger Menu -> Users & Authentication -> Create). Set the global permission to Administrator
Create another local cluster k3d create cluster import-test
In rancher, import a generic cluster (Hamburger Menu -> Cluster Management -> Import Existing -> Generic)
It gives you a registration command to run to import it into rancher. Run that
In rancher, wait for the cluster to become active
Log out
Log in as the admin user you created
From the home page, select the new cluster and get the kubeconfig (top right)
Paste the kubeconfig in a terminal into a yaml file
Run any kubectl command using the kubeconfig kubectl --kubeconfig test-config.yaml get pods -A

Result
I0414 16:04:58.148389   45660 versioner.go:58] an error on the server ("unable to create impersonator account: error setting up impersonation for user u-glkpr: failed to get secret for service account: cattle-impersonation-system/cattle-impersonation-u-glkpr, error: resource name may not be empty") has prevented the request from succeeding

Expected Result

When the admin user issues the kubectl command it should succeed with no error message.
Screenshots

Additional context

Point 2 of this answer has some details:
https://stackoverflow.com/questions/60109761/error-while-running-pod-defination-file-in-kubernetes

duplicate #43293 maybe?

Assigning to team/collie since it is impersonation related.

Taking this temporarily for triage

I was unable to reproduce this on 2.9-head.
I will try with previous Rancher releases.

I was also unable to reproduce this on the latest 2.8 release.
Strangely enough, I could not repro on the reported version 2.7.2 either.
@JonCrowther I know it's been awhile since you originally reported this, but do you have any additional tips for me to repro this?

I can no longer reproduce the issue either. Closing!


Issue #8921 fixed.

Issue Reference

Issue #8921 fixed.

@suraj-webkul
As I have checked and found that the issue related to the response of the status button is working fine. Please check the video for the reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/5957904f-212b-4db8-9783-d7856127279a


Implement kernel for casting float to decimal

copilot:summary

/prbot review

Related: https://github.com/NVIDIA/spark-rapids/issues/9847

build

build

build

build

build

build

build

Minor nits on comments. Do we have any performance measurements for the new kernel? Wondering if it's worth making a benchmark for it.

Previously @thirtiseven helped with benchmarking, like this https://github.com/NVIDIA/spark-rapids/pull/10917#issuecomment-2134569776. @thirtiseven can you run such benchmark again on this please? Thanks.

build

New perf numbers:

Type
CPU
24.08
gluten
This PR

Double
147,121.5
524.25
7,105.5
520.5

Float
89,442
440.75
8,007.5
657.75

Float type is a little slower than current code, but it runs much faster than cpu and gluten.


doc: order autosummary to respect inter-module dependencies.

Increase version to 0.0.12.dev0

Current coverage is 100.00%

Merging #91 into master will not affect coverage as of 6de4f0a

@@            master     #91   diff @@
======================================
  Files           29      29
  Stmts          952     952
  Branches         0       0
  Methods          0       0
======================================
  Hit            952     952
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 6de4f0a

Powered by Codecov. Updated on successful CI builds.


conda-env-deactivate is broken by recent changes to conda-env-stripped-path

I just updated (;; Package-Version: 20190528.421) and can't activate an environment now.
conda-env-stripped-path now tries to split a PATH string into a list of strings, but it is being fed exec-path in conda-env-deactivate, which is already a list of strings.
Debugger entered--Lisp error: (wrong-type-argument stringp ("/home/james/bin" "/usr/local/sbin" "/usr/local/bin" "/usr/sbin" "/usr/bin" "/sbin" "/bin" "/usr/games" "/usr/local/games" "/snap/bin" "/usr/local/libexec/emacs/26.2/x86_64-pc-linux-gnu"))
  string-match(":" ("/home/james/bin" "/usr/local/sbin" "/usr/local/bin" "/usr/sbin" "/usr/bin" "/sbin" "/bin" "/usr/games" "/usr/local/games" "/snap/bin" "/usr/local/libexec/emacs/26.2/x86_64-pc-linux-gnu") 0)
  split-string(("/home/james/bin" "/usr/local/sbin" "/usr/local/bin" "/usr/sbin" "/usr/bin" "/sbin" "/bin" "/usr/games" "/usr/local/games" "/snap/bin" "/usr/local/libexec/emacs/26.2/x86_64-pc-linux-gnu") ":" nil)
  (progn (split-string s separator omit-nulls))
  (unwind-protect (progn (split-string s separator omit-nulls)) (set-match-data save-match-data-internal (quote evaporate)))
  (let ((save-match-data-internal (match-data))) (unwind-protect (progn (split-string s separator omit-nulls)) (set-match-data save-match-data-internal (quote evaporate))))
  s-split(":" ("/home/james/bin" "/usr/local/sbin" "/usr/local/bin" "/usr/sbin" "/usr/bin" "/sbin" "/bin" "/usr/games" "/usr/local/games" "/snap/bin" "/usr/local/libexec/emacs/26.2/x86_64-pc-linux-gnu"))
  conda-env-stripped-path(("/home/james/bin" "/usr/local/sbin" "/usr/local/bin" "/usr/sbin" "/usr/bin" "/sbin" "/bin" "/usr/games" "/usr/local/games" "/snap/bin" "/usr/local/libexec/emacs/26.2/x86_64-pc-linux-gnu"))
  conda-env-deactivate()
  conda-env-activate()
  funcall-interactively(conda-env-activate)
  call-interactively(conda-env-activate record nil)
  command-execute(conda-env-activate record)
  #f(compiled-function (cmd) #<bytecode 0x1b4de25>)("conda-env-activate")

@WJCFerguson great catch, I'll try and get a fix out today, thank you for the report!

@WJCFerguson this should be closed by #33 and the updated build should be on MELPA in a few hours, please let me know if things didn't work as expected!

@necaris Thanks for the quick fix, but unfortunately with the 20190531.1807 release I am still not able to activate an environment. conda-env-activate now gives a Wrong type argument: listp error.

I hit that one too. conda-env-stripped-path now expects a list of strings, not exec-path and fails in https://github.com/necaris/conda.el/blob/master/conda.el#L265


Upgrade Spring Platform IO to 2.0.5 and Spring Boot to 1.3.5

Coverage remained the same at 89.095% when pulling 263f807e93e284066cbce9c7cd46a57677e95526 on tgianos:develop into 1a365f9ab0792ded04df65e88b8ab5d48c067adc on Netflix:develop.

Coverage remained the same at 89.095% when pulling 263f807e93e284066cbce9c7cd46a57677e95526 on tgianos:develop into 1a365f9ab0792ded04df65e88b8ab5d48c067adc on Netflix:develop.


Serialization: Break the cycle between Serialization and SymbolGraphGen

Push the top level logic for writing out swiftmodules and associated files into the frontend library which has access to all the necessary dependencies.
Fixes the circular dependency between Serialization and SymbolGraphGen that was introduced with https://github.com/apple/swift/pull/35110.

@swift-ci please test

@swift-ci please test

This depends on https://github.com/apple/swift/pull/61792 since it removes the last circular dependency that was bringing in ConstExtract everywhere.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test


Fancy Zones don't stick

Microsoft PowerToys version
0.60.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones Editor
Steps to reproduce
Edit Fancy Zones using layout editor. Observe that the layouts chosen for each of the monitors does not stick. Zone layouts seem to change at random or always default to grid layout.

PowerToysReport_2022-07-07-23-51-22.zip
.
✔ Expected Behavior
Chosen layouts stick to chosen monitors
❌ Actual Behavior
Layouts are assigned to monitors at random. Sometimes all monitors take on grid layout even though I only chose that layout for 1 monitor.
Other Software
No response

/dup of https://github.com/microsoft/PowerToys/issues/19203
Working on a fix. Thank you.


Removal of execution strategies

This pull request is about removing the execution strategies. The execution strategy was determining how a runtime call would be executed, e.g. only use the wasm runtime, only use the native or a combination of both. Based on the context of the runtime call (block import, block building etc) the execution strategy could be configured. So, a node could for example use the native runtime for building a block and then use the wasm runtime to import blocks. Just because the execution strategy was saying to use the native runtime, it didn't mean that we would use the native runtime all the time. For example when the native runtime doesn't match the wasm runtime a runtime call could not use the native runtime as this could lead to nondeterministic behavior. If not changed by the user via CLI the default execution strategy for all contexts is already set to wasm only.
In the light of removing the native runtime choosing the execution strategy will be irrelevant as the only strategy to choose would be "wasm only". This pull requests should be seen as preparation for making sp-api not compile time dependent on the native runtime.
Behavior changes of the runtime api
When calling into the runtime using the runtime api (sp-api) there was always the possibility to register execution extensions. These execution extensions would then be accessible from any host function. Before this pull request based on the context given to the runtime call there could have been passed custom "capabilities". These capabilities determined which kind of execution extensions the runtime call could access. The default context for every runtime call was OffchainCall with no explicit capabilities. This meant that the runtime call had access to the read from the offchain database, use the keystore and to submit a transaction to the transaction pool. All of this isn't available by default anymore. This means when you have some code that wants to submit a transaction from a runtime call, the transaction pool needs to be registered manually as execution extension for the runtime api instance. The runtime api instance now provides the register_extension function for registering these execution extensions. These execution extensions will then be available for the entire lifetime of the runtime api instance, aka for all calls done via this instance. This is another behavior change which now also keeps the execution extensions "alive" while building a block (because this keeps internally the runtime api instance alive). The "global" execution extensions factory also still exists, but should only be used to initialize execution extensions that need to be available for every runtime call. For simplifying the creation of a transaction pool execution extension this pull requests introduces OffchainTransactionPoolFactory. This factory can be used to instantiate instances of the transaction pool extension at given blocks. The pull requests shows how to use them in the BABE and Grandpa client crates.
Removal of build_offchain_workers
To spawn offchain workers the sc-service crate provided the function build_offchain_workers. This function was any reference to sc-offchain was removed from the sc-service crate. First, this makes the sc-service crate less "god-like" and secondly makes the initialization of the offchain worker more "obvious" to the user. To spawn the offchain worker, the following code needs to be added to the service file:
if enable_offchain_worker {
	task_manager.spawn_handle().spawn(
		"offchain-workers-runner",
		"offchain-work",
		sc_offchain::OffchainWorkers::new(sc_offchain::OffchainWorkerOptions {
			runtime_api_provider: client.clone(),
			keystore: Some(keystore_container.keystore()),
			offchain_db: backend.offchain_storage(),
			transaction_pool: Some(OffchainTransactionPoolFactory::new(
				transaction_pool.clone(),
			)),
			network_provider: network.clone(),
			is_validator: role.is_authority(),
			enable_http_requests: true,
			custom_extensions: move |_| {
				vec![]
			},
		})
		.run(client.clone(), task_manager.spawn_handle())
		.boxed(),
	);
}

This provides a way to provide all the required objects that will then be available as execution extension from the offchain_worker call in the runtime. The custom_extensions is a closure that will be called every time the offchain_worker should be called and can return custom execution extensions that should be made available.
Changes to Grandpa and BABE client crates
The sc-consensus-grandpa crate now requires the parameter offchain_tx_pool_factory in GrandpaParams. This will be used by grandpa when there is an equivocation report to submit the transaction from the runtime.
The sc-consensus-babe crate changes the signature of import_queue to receive the dedicated ImportQueueParams. These parameters have named args to make it more obvious what each of them is doing. It also gets the offchain tx pool factory passed which it will use when sending an equivocation report.
CLI args deprecation
The following CLI args are now deprecated:

--execution-syncing
--execution-import-block
--execution-block-construction
--execution-offchain-worker
--execution-other
--execution

All of these CLI args are still present, but the node will print an error if the user passes them. In the future these will be removed.

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2992646

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3000750

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3001151

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3004539

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3006790

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3007659

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3099329

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3135339

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3160574

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/3160869

bot merge

bot merge

bot merge


add test to verify error response for invalid xml

Issue #13 assumes that the parser returns a promise. The parse() function calls the passed in callback with an error and response params. This pull request contains a unit tests that demonstrates the graceful error/failure case for invalid xml (comment containing '--').
I would appreciate this case/pr being tagged with #hacktoberfest

Coverage increased (+6.5%) to 93.548% when pulling bdefa36309f3c6e1d9bb3529854647249d77453e on rswhiting:master into d59c365fb15a9993b0db19d87da407dccf8f744d on intuit:master.

TypeError: util.promisify requires a true ES5 environment
This seems like a build configuration problem. Help?

@rswhiting We have a new version using promises... Could you rebase your branch and verify if this problem? Please re-open this PR if it is still an issue!


[DCOS-56669] Bump SDK to version 0.57.0.

What changes were proposed in this pull request?
Upgrade sdk version to 0.57.0
For detailed description on changes refer : https://github.com/mesosphere/dcos-commons/releases/tag/0.57.0
Resolves Bump SDK to version 0.57.0
How were these changes tested?
Build code locally and tried running service

I approve but we don't want to merge this PR into master. It can serve as the release branch and if we discover issues in the release process, we'll add commits to this branch to fix those issues.

I approve but we don't want to merge this PR into master. It can serve as the release branch and if we discover issues in the release process, we'll add commits to this branch to fix those issues.

Thanks Sam @samvantran , Please re-review when you get time

HDFS 2.7.0-3.2.1 released, closing this PR: https://docs.d2iq.com/mesosphere/dcos/services/hdfs/2.7.0-3.2.1/release-notes/


Added component to list products on the store page

Used the getShirts function to create a component that lists all available products in cards.

btw someone will need to fix the css. I checked globals and messed around with tailwinds and found nothing

what exactly is broken with the css? I can go fix it later today or tmrw

I want them to be evenly spaced apart but they appear all right next to each other like padding is being ignored

Please do not make a fork and then make changes
we have a dev branch for a reason (otherwise the Vercel deployment doesn't work)
please use the dev branch moving forward but right now this is fine


SyntaxError: Unexpected token ) in /routes/index.js

I'm encountering this error when attempting to use this package in the context of a firebase cloud fuction, which refuses to deploy given a small syntax error:
Detailed stack trace: /user_code/node_modules/@shopify/shopify-express/routes/index.js:19   );   ^SyntaxError: Unexpected token )     at createScript (vm.js:56:10)     at Object.runInThisContext (vm.js:97:10)     at Module._compile (module.js:549:28)     at Object.Module._extensions..js (module.js:586:10)     at Module.load (module.js:494:32)     at tryModuleLoad (module.js:453:12)     at Function.Module._load (module.js:445:3)     at Module.require (module.js:504:17)     at require (internal/module.js:20:19)     at Object.<anonymous> (/user_code/node_modules/@shopify/shopify-express/index.js:2:22)
It appears to be due to a stray comma on the listed line, before the ending ')'. Looks like this:
  router.use(     '/api',     rawParser,     verifyApiCall,     shopifyApiProxy,   );
Easy fix.

On closer inspection it looks like this is intentional (as evidenced by Prettifier config), and just an incompatibility with Firebase. I've forked the repo and make the necessary modifications instead.


backport: Guix related backports

Issue being fixed or feature implemented
Relatively small batch of guix backports
What was done?
How Has This Been Tested?
GitHub guix builds tbd
Breaking Changes
should be none
Checklist:
Go over all the following points, and put an x in all the boxes that apply.

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

Guix Automation has began to build this PR tagged as v20.1.0-devpr5822.be1aba90. A new comment will be made when the image is pushed.

Guix Automation has completed; a release should be present here: https://github.com/dashpay/dash-dev-branches/releases/tag/v20.1.0-devpr5822.be1aba90. The image should be on dockerhub soon.

Looks like it works :) waiting for GitHub based guix now

Guix Automation has began to build this PR tagged as v20.1.0-devpr5822.196ea08b. A new comment will be made when the image is pushed.

Guix Automation has completed; a release should be present here: https://github.com/dashpay/dash-dev-branches/releases/tag/v20.1.0-devpr5822.196ea08b. The image should be on dockerhub soon.

Alright let's try again :)

Guix Automation has began to build this PR tagged as v20.1.0-devpr5822.74bc4461. A new comment will be made when the image is pushed.

Guix Automation has completed; a release should be present here: https://github.com/dashpay/dash-dev-branches/releases/tag/v20.1.0-devpr5822.74bc4461. The image should be on dockerhub soon.


Add instruction for BWC tests on distribution level

Signed-off-by: Zelin Hao<EMAIL_ADDRESS>Description
Adding onboarding instruction and documents for BWC tests on distribution level for plugins.
Issues Resolved
Part of https://github.com/opensearch-project/opensearch-build/issues/2870
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock @saratvemulapalli Could you help review another times. Need one more approval. Thanks.

Merging this for now.


fix(iam-user): Direct policy attachment of iam-policy-created resources

Description
Followed @antonbabenko's steer from https://github.com/terraform-aws-modules/terraform-aws-iam/pull/421#pullrequestreview-1645771068 on how to fix attaching policies created by iam-policy to iam-user.
Motivation and Context
Fixes https://github.com/terraform-aws-modules/terraform-aws-iam/issues/408
Breaking Changes
None.
How Has This Been Tested?

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

[x] I have executed pre-commit run -a on my pull request
[x] I have performed a terraform plan and terraform apply locally

@antonbabenko could you please review this PR? Thanks!

This PR is included in version 5.30.1 :tada:


Installation error. Can't find the specified version of tensorflow.

I am using python 3.8 with pip 20.0.2
ERROR: Could not find a version that satisfies the requirement tensorflow<=1.15.2,>=1.13.1 (from deephyper==0.1.7) (from versions: 2.2.0rc1, 2.2.0rc2, 2.2.0rc3) ERROR: No matching distribution found for tensorflow<=1.15.2,>=1.13.1 (from deephyper==0.1.7)

Hello @Viren-C could I have more information about the way you tried to install DeepHyper:

what environment where you using?
what procedure did you follow to install DeepHyper?
did you have an already existing tensorflow version?
does your current environment allow for installing tensorflow between 1.13 and 1.15?

I am using fresh empty environment.
I tried installing using pip and through github and still got the same error.
No, the environment in which i am working has no version of tensorflow.
My current environment has no restrictions for any installations

I managed to reproduce your error, and apparently Tensorflow 1.13 to 1.15 are not available with python 3.8. A possible fixe is to use conda to create a virtual environment under python 3.7 such as:
$ conda create -n dh-env python=3.7


[Question] How is the cursor encoded?

I need to create cursors on the fly for each document from the frontend itself so that I could do infinite loading.
Let's assume I have a document in my frontend
{
  id: string,
  likesCount: Number
  //...rest of the fields
}

I want to create the cursor for this item from id and likesCount on the frontend itself to paginate from there.

@cryptrr Apologies for the delayed reply here. The default cursor encoder is here https://github.com/charlie-hadden/kysely-paginate/blob/ed37a87ad9c940596fc44e6b590d6d9ba6289b8d/src/cursor.ts#L262-L295
Depending on your exact use case, the cursorPerRow option may also be useful: https://github.com/charlie-hadden/kysely-paginate/blob/ed37a87ad9c940596fc44e6b590d6d9ba6289b8d/test/cursor.test.ts#L52-L59


Update GNU Release Flags

Testing at NAS seems to show that moving from -O3 to -O2 fixes issues with TOSS4.
cc: @gmao-rreichle that GNU results might change in the pursuit of portability between centers...

Closing as we can get away with degrading Moist, see https://github.com/GEOS-ESM/GEOSgcm_GridComp/pull/762


LiveLambda: Large WebSocket response throw IllegalLocationConstraintException in new regions

S3 is used to process large WebSocket responses (e.g. in ws.ts), but the region option is not explicitly set const s3 = new S3();.
Therefore IllegalLocationConstraintException was thrown, as that is required for those new regions.
Please add an option to allow setting the region option, or pass the region of the stack to the handler.

Hi @TerenzChan, I'm taking a look at this.

const s3 = new S3();
Are you referencing the stub function here https://github.com/serverless-stack/serverless-stack/blob/master/packages/resources/assets/stub/index.js#L17?

Btw which region did you try with?

Hi @TerenzChan, I'm taking a look at this.

const s3 = new S3();
Are you referencing the stub function here https://github.com/serverless-stack/serverless-stack/blob/master/packages/resources/assets/stub/index.js#L17?

Seems like that will exist in those files that used S3.
I found it in the stub function you posted and
https://github.com/serverless-stack/serverless-stack/blob/master/packages/core/src/runtime/ws.ts#L7

Btw which region did you try with?

I used ap-east-1.
It looks like the global endpoint of S3 will be used if the region option is not set. And the request will not be redirected to the correct region automatically for those buckets in opt-in regions, which were launched after March 20, 2019.

Hi @TerenzChan, this is fixed in v0.66.1. Give it a try and let me know if it works for you.

Hi @fwang, It works now. Thanks!