Interface missing when using Public Network

I tried to configure a public network attached to host devices.
Config:
rh1501.vm.network :public_network, :dev => "eth0", :mode => 'bridge'
Steps:

(I start with no virtual networks in libvirt)
Network vagrant-libvirt is automatically created in the net <IP_ADDRESS>/24
I tried this with both a real ethernet interface and a dummy interface.

If I use a real interface it gets stuck in Waiting for domain to get an IP address....  Guest created does not have any network interface but loopback.
If I use a dummy interface I get an error trace (see below)

Comments:
Is there any requirement about the state of the ethernet interface (address, up/down) before running vagrant up??
Actually, my first aim would be to create my own bridges and configure vagrant-libvirt to simply attach my guests to them. I understand that this is not supported. Please correct me if I'm wrong.
Versions:
Host running ubuntu 13.10
libvirt-bin 1.1.1-0ubuntu8.5
vagrant 1:1.4.3
vagrant-libvirt 0.0.15
Full trace when using dummy interface:
uninitialized constant VagrantPlugins::ProviderLibvirt::Action::CreateNetworkInterfaces::NetworkNotAvailableError

/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/create_network_interfaces.rb:160:in `interface_network'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/create_network_interfaces.rb:60:in `block in call'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/create_network_interfaces.rb:39:in `each'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/create_network_interfaces.rb:39:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/create_networks.rb:76:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/builtin/provision.rb:52:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/create_domain.rb:70:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/create_domain_volume.rb:51:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/handle_box_image.rb:38:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/builtin/handle_box_url.rb:24:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/handle_storage_pool.rb:21:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/set_name_of_domain.rb:36:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:95:in `block in finalize_action'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/builder.rb:116:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/runner.rb:69:in `block in run'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/util/busy.rb:19:in `busy'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/runner.rb:69:in `run'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/builtin/call.rb:51:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/home/user/.vagrant.d/gems/gems/vagrant-libvirt-0.0.15/lib/vagrant-libvirt/action/connect_libvirt.rb:84:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/builtin/config_validate.rb:25:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/warden.rb:34:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/builder.rb:116:in `call'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/runner.rb:69:in `block in run'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/util/busy.rb:19:in `busy'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/action/runner.rb:69:in `run'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/machine.rb:147:in `action'
/opt/vagrant/embedded/gems/gems/vagrant-1.4.3/lib/vagrant/batch_action.rb:63:in `block (2 levels) in run'

@qodeq @gnat42:

Actually, my first aim would be to create my own bridges and configure vagrant-libvirt to simply attach my guests to them. I understand that this is not supported. Please correct me if I'm wrong.

In order to connect a Vagrant VM to a pre-existing bridge, use the following syntax:
  config.vm.network "public_network", :dev => "br0", :mode => 'bridge', :type => "bridge"

This works fine under vagrant 1.8.1 and vagrant-libvirt 0.0.33; please reopen this issue if it is still a problem with current stable releases.


[AutoPR containerservice/resource-manager] Revert node public ip support from 2019-04-01 model.

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5841

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/4738


Support for progress bar updates at infrequent epochs

🚀 Feature
Right now there is a way to only update the progress bar after every XXX steps but

It doesn't work with custom callbacks according to the docs
Doing it every XXX epochs is often much more natural.

Motivation
For things that go through epochs and steps rapidly, the progress bar updates can cause an enormous amount of output.
This might slow things down with the flushing, but the bigger issue is on batch systems on the cloud where the logs are being captured to stdout.  those cannot do inplace changes to the text within a terminal, so the logs get enormous.
Alternatives
There also doesn't seem to be a way to hack it with a custom ProgressBar class.
Soemthing like
class InfrequentProgressBar(ProgressBar):

    def __init__(self,epoch_frequency):
        super().__init__()  # if you don't add refresh_rate = 0 then it still shows every time.
        self.epoch_frequency = epoch_frequency

    def on_train_epoch_start(self, trainer, *args):
        if trainer.current_epoch % self.epoch_frequency == 0:
            super().on_train_epoch_start(trainer, *args)

    def on_train_batch_end(self, trainer, *args):
        if trainer.current_epoch % self.epoch_frequency == 0:
            super().on_train_batch_end(trainer, *args)

    def on_validation_batch_end(self, trainer, *args):
        if trainer.current_epoch % self.epoch_frequency == 0:
            super().on_validation_batch_end(trainer, *args)

As a callback seemed to only show the epoch updating infrently for the progress bar itself, but the underlying stuff added to self.log still updates rapidly.
cc: @tchaton
cc @borda @awaelchli @ananthsub @carmocca

My current workaround is that I add the following to my config file:
  callbacks:
    - class_path: pytorch_lightning.callbacks.progress.TQDMProgressBar
      init_args:
        refresh_rate: 1 # set to 0 for no bar, then change epoch_heartbeat

add a constructor argument called epoch_heartbeat to my model, and then add
    def validation_epoch_end(self, outputs):
        # Only logs if the progress bar turned off
        if (
            (self.trainer.progress_bar_callback is None)
            and (self.hparams.epoch_heartbeat > 0)
            and (self.current_epoch % self.hparams.epoch_heartbeat == 0)
        ):
            print(f"Epoch {self.current_epoch}: {self.trainer.progress_bar_dict}")

If I don't do this, it wrecks the grid output.

I think your workaround is perfectly fine.

This might slow things down with the flushing, but the bigger issue is on batch systems on the cloud where the logs are being captured to stdout. those cannot do inplace changes to the text within a terminal, so the logs get enormous.

This is a big problem in the design of TQDM. https://github.com/PyTorchLightning/pytorch-lightning/pull/9000 attempted to fix it with no success.
@kaushikb11 is Rich better in this regard?

Sounds good.  If you think this is part of bigger redesign then maybe the stale-bot was right and the issue should be closed.


DO NOT MERGE: opened this PR to debug the CI

I need to look at the CI runs more closely to debug some bugs we've been having lately.
Maintainers will complete the following section

[ ] Commit messages are descriptive enough (hints)
[ ] Code coverage from testing does not decrease and new code is covered (check the PR coverage on codecov)
[ ] Controllers diagrams are updated when PR changes controllers code  (if applicable)

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/retest

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test integration-service-e2e

/test all

/retest

/test integration-service-e2e

/test integration-service-e2e

We were successfully able to identify and address the root cause of the bug: RHTAPBUGS-1051, which was failing our CI previously. So, I'm closing this PR.


Is it possible to single step the basic code?

HI,
Thanks  for the wonderful contribution.
Is there a mechanism to single step the basic code, if not can you please let me know would it be simple to create such a feature?

Thanks for the response!
Really appreciate it!


lib/cmsis_rtos_v1: Fix some Kconfig inconsistencies

Fixed some Kconfig inconsistencies around THREAD_CUSTOM_DATA,
POLL and NUM_PREEMPT_PRIORITIES.
Signed-off-by: Rajavardhan Gundi<EMAIL_ADDRESS>

Codecov Report

Merging #10281 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master   #10281      +/-   ##
==========================================
- Coverage   53.05%   53.05%   -0.01%
==========================================
  Files         214      214
  Lines       26201    26202       +1
  Branches     5645     5646       +1
==========================================
  Hits        13901    13901
  Misses      10028    10028
- Partials     2272     2273       +1

Impacted Files
Coverage Δ

lib/cmsis_rtos_v1/cmsis_thread.c
71.01% <0%> (-1.05%)
:arrow_down:

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 e2c590a...1454302. Read the comment docs.


Minimize image size even more

There has to be a ton of unneeded packages and probably other stuff that can be removed before creating the disk image.
get a list of all installed packages:
$ pacman -Q | more

Perhaps look at basing patchOS on Alpine Linux?

Remove all unnecessary drivers and modules from the kernel.
Disable module support and embed all needed drivers.


Mysql cdc duplicate synced data

sync mysql data to hive by mysql cdc connector.
1.initial synced all the data and it's right in hive table;
2.changed some data in original mysql table or remove some records;
3.the dest hive contains duplicate record when change the mysql existed data;

mysql cdc default format will generate 2 record when upstream data updated, one record is delete one record is insert.
and if you change the format to compatible_debezium_json, it will only generate 1 update record.
You can change the sink to Console then to check the result.
For your case, you use hive as destination, hive is not support update, delete operation. also cdc will generate a lots of small file. maybe it's not a good idea.


Add Natvis visualizations for core indexmap types

This change adds Natvis visualizations for some indexmap types to help improve the debugging experience on Windows.
Natvis is a framework that can be used to specify how types should be viewed under a supported debugger, such as the Windows debugger (WinDbg) and the Visual Studio debugger.
When debugging types such as IndexMap and/or IndexSet under a Windows debugger, the entries for these data structures are not always clear upon first glance. The internal data structure is shown and it's not presented in a user friendly way. The Rust compiler does have Natvis support for some types, but this is limited to some of the core libraries and not supported for external crates.
https://github.com/rust-lang/rfcs/pull/3191 proposes adding support for embedding debugging visualizations such as Natvis in a Rust crate. This RFC has been approved, merged and implemented.
This PR adds:
Natvis visualizations for some of the core types such as IndexMap and IndexSet.
Tests for testing visualizers embedded in the indexmap crate.
Updates to the CI pipeline to ensure tests for visualizers are run so they do not break silently.
A new debugger_visualizer feature for the indexmap crate to enable the unstable debugger_visualizer Rust feature.

The feature was stabilized in rust-lang/rust#108668, headed for Rust 1.71. Does this PR need updating for that? At the very least, we should not need the feature(debugger_visualizer) attribute anymore, but I don't know if there were any functional changes along the way.


chore: add full dataset

Changes

Add full dataset as bbc_full

@larwaa Suggestion: Rename the current 400-document dataset to bbc_400, and add the full dataset as just bbc, since that is what we use in the report, and if the course staff run our code, that is likely the dataset they will use.


docs: update crate, lib and readme links

Link to the contributing, changelog, and breaking changes docs at the
top of the page instead of just in in the main part of the doc. This
makes it easier to find them.
Rearrange the links to be in a more logical order.
Use link refs for all the links
Fix up the CI link to point to the right workflow

Based on https://discord.com/channels/1070692720437383208/1070692720437383211/1194100654344515594

I see there's an update to 0.25 version and it messed up few things I'm working on. Is there some "migration" guide ? Or changes in changelog ?

I'm hoping that by putting these right up front, they're harder to miss.


Unquoted dates don't parse correctly... Always return week #28

Any chance that we can have quotes added when they are missing?   Feels kind of silly putting quotes around date values in a column.

Week
Date

<!--FM WEEKNUM(B1)--> 3
"2022-01-10"
Ok

<!--FM WEEKNUM(B2)--> 28
2022-01-10
<-- Shouldn't be week 28

<!--FM WEEKNUM(B3)--> 24
"2022-06-07"
Ok

We need to make some kind of assumption here. For the typing purposes, parser tries to find out if the cell value is a number. Unfortunately, Number.parse works correctly with the strings like '2022-01-10', so this is why it is parsed unproperly.
To satisfy the requirement I need to recognize if a given string is a date or a number, maybe I can find some library for that, but still dates will have to be put in a proper format anyway.
Unfortunately again, Date.parse(1984) gives a proper result too.

Can we assume something like a trimmed value matching something like ^-? \n+(\.\n+)? is a number and everything else is a string?  Maybe it would be easier

There are multiple formats for a number, what about locale dependent or scientific notation?

If you look into the code, you certainly find out that I've implemented isNan check for Numbar.parseFloat long time ago.
And no, parseFloat is not returning isNan for 2022-01-10 (even if you put it as a string argument).

Meh, even if I figure out how to fix this number handling bug, it looks like my dream of using this for time calculations is toasted because the difference between how DATEVALUE as it is done with formula JS and how it's handled by this plugin.

Source
Method
Value
Note

Markdown Calc
<!--FM DATEVALUE('8/22/2011') * 1-->
 40778
This is an Excel Date (Days since Jan 1 1900) - Time info is lost 😢

Formula JS
DATEVALUE('8/22/2011') * 1
1313985600000
/1000 = Unix epoch time i.e. gdate -d @1313985600 = Mon Aug 22 00:00:00 EDT 2011

This formula works in Formula JS, but it's hopeless without time data...
((DATEVALUE(REGEXREPLACE(DATEVALUE("2022-09-07")+"","\\d\\d:\\d\\d:\\d\\d","5:30 PM")) - DATEVALUE(REGEXREPLACE(DATEVALUE("2022-09-07")+"","\\d\\d:\\d\\d:\\d\\d","8:00 AM")))/3600000)

(result is "9.5" hours)

I do not see any reasonable way to fulfill such a requirement.


增加garray和gset模块的单元测试功能

garray和gset模块的测试覆盖率达到90%以上。

Codecov Report

Merging #180 into master will increase coverage by 5.31%.
The diff coverage is 86.97%.

@@            Coverage Diff             @@
##           master     #180      +/-   ##
==========================================
+ Coverage   64.46%   69.78%   +5.31%
==========================================
  Files         197      197
  Lines       16861    16869       +8
==========================================
+ Hits        10870    11772     +902
+ Misses       5307     4408     -899
- Partials      684      689       +5

Impacted Files
Coverage Δ

g/container/garray/garray_sorted_int.go
87.7% <86.97%> (+71.32%)
:arrow_up:

g/os/gspath/gspath_cache.go
78.26% <0%> (-10.87%)
:arrow_down:

g/container/gtree/gtree_avltree.go
57.46% <0%> (-4.56%)
:arrow_down:

g/os/gcfg/gcfg.go
86.84% <0%> (+1.12%)
:arrow_up:

g/container/garray/garray_normal_interface.go
88.65% <0%> (+11.64%)
:arrow_up:

g/container/garray/garray_normal_int.go
87.71% <0%> (+16.66%)
:arrow_up:

g/container/gset/gset_int_set.go
86.44% <0%> (+17.51%)
:arrow_up:

g/container/garray/garray_normal_string.go
88.3% <0%> (+18.42%)
:arrow_up:

g/container/gset/gset.go
94.97% <0%> (+25.69%)
:arrow_up:

... and 3 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 548a0c4...b3a00be. Read the comment docs.


add-swift-support.js of v2.0 is missing on npm

fresh new version installed from npm, but without add-swift-support.js, therefore fail to install

Fixed with version 2.0.1


Error in heasarc.py

from gdt.core.heasarc import BrowseCatalog
Error Returned:
Traceback (most recent call last):
File "", line 1, in
File gdt-core/src/gdt/core/heasarc.py", line 379, in
class FileDownloader(BaseFinder):
File "gdt-core/src/gdt/core/heasarc.py", line 402, in FileDownloader
def bulk_download(self, urls: list[str], dest_dir: Union[str, Path], verbose: bool = True):
TypeError: 'type' object is not subscriptable

What version of python are you using?  I ran the import statement in 3.9.16 and it does not produce this error.

Also, make sure that you have the latest version (2.0.2).  I believe there was an update for this or a related issue.


minor tweaks npm v yarn

[ x ] I have read freeCodeCamp's contribution guidelines.
[ x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x ] My pull request targets the master branch of freeCodeCamp.
[ x ] None of my changes are plagiarized from another source without proper attribution.
[ x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@kenahrens Good add!
Congrats on your first-time contribution!


arrays cannot be accessed through any

Describe the bug
game.consoleOverlay.setVisible(true)
const arr = [1,2,3]
console.log(arr[1])
const x = arr as any
console.log(x[1]) // crashes here

Also x.length doesn't work.

This occurs when setting fields of sprite.data to arrays


refactor: upgrade project

:tada: This PR is included in version 0.113.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


No test suite found on windows for v0.28.5+

Describe the bug
Hey, I'm running vitest via npm scripts on Windows, which results in no test suite found for all test files.

It might be related to https://github.com/vitest-dev/vitest/issues/2020 since it's working for me if I run the command in the docker environment which is WSL (I guess, not sure). But I still want it to be runnable locally.
Downgrading to v0.28.4 solves the issue, which means the problem only exists for version v0.28.5 and later (I suspect some changes caused this in v0.28.5 release)
Since it's only happening on my local, so I don't think it's reproducible, but I would highly appreciate it if someone can inspect it.
Reproduction
Unfortunately no reproduction available :( reason described above.
System Info
System:
    OS: Windows 10 10.0.22000
    CPU: (8) x64 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz
    Memory: 16.28 GB / 31.69 GB
  Binaries:
    Node: 16.18.0 - C:\Program Files\nodejs\node.EXE
    npm: 8.19.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.22000.120.0), Chromium (110.0.1587.56)
    Internet Explorer: 11.0.22000.120
  npmPackages:
    @vitejs/plugin-react-swc: ^3.2.0 => 3.2.0
    @vitest/coverage-c8: ^0.29.1 => 0.29.1
    @vitest/coverage-istanbul: ^0.29.1 => 0.29.1
    @vitest/ui: ^0.29.1 => 0.29.1
    vite: ^4.1.4 => 4.1.4
    vitest: ^0.29.1 => 0.29.1

Used Package Manager
npm
Validations

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

@sheremet-va Hey, I'm sorry that it somehow works again.. even in my original case. I'm so confused😢.
I'll close the issue anyway, but in case it happens again I'll re-open the issue and attach a reproduction. Thanks for that.

@sheremet-va Hi, I've encountered the problem again, and here's the reproduction:
https://github.com/xsjcTony/vitest-bug-repord
Please clone and run
npm install
npm run test:unit

It will result in

Just FYI, Issue still exists in v0.29.2

Any updates?


THRIFT-5306: Rust code and generated code no longer has clippy warnings (Rust 1.40)

Client: rs
Specific changes (lib/check/tutorial/cross-test)

Update all clippy/rustfmt attributes to use new syntax
Use new numeric literal syntax
Use shorthand syntax for initializing structs
Use elided lifetimes for Formatter
Remove extern crate
Update all Cargo.toml files (except lib/rs) to remove direct references to ordered-float crate (use Thrift redefinition)
Update all Cargo.toml files (except lib/rs) to use updated, and a consistent set of crates

Specific changes (generator)

Remove unnecessary () return and else branch when writing fields
Use _f64 suffix for float literals during const initialization
Take ownership of self when writing a Thrift enum (represented by a Rust enum with integers) to an output protocol
Use shorthand syntax for initializing structs

NOTE: This does not fix any bugs. It simply updates the Rust code and generated code so that it no longer prints clippy warnings, and uses Rust 2018 idioms.
Specific changes (other)

Update tutorial README
Update min Rust version to reflect what we were actually testing with (Rust 1.40)

Appears to be erlang-related build issues. Merging.


x3d Customer ESPG Code Request

Customer X3d.sk - Expert for 3D Landscape Slovakia (Meixner distributor) is requesting support for following ESPG codes:
EPSG Codes requested:
epsg:102067
epsg:3046
epsg:4936
epsg:5514
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi @dwood777, are there models on our internal network using these zones so I can ensure they are working?


Prevent Right Click Bug

When you right click inside of a Riven graph it shows the context menu as intended. Clicking elsewhere hide the context menu would trigger the mousemove event and drag the graph, which is bad behavior.
This small commit fixes the bug so that you can right-click without unintentionally dragging the graph.

Thanks a lot, I will implement on Oscean as well.


Launches empty or internal server errors on endpoints (V3)

I'm retrieving empty arrays or server errors on V3 launches endpoints:

https://api.spacexdata.com/v3/launches/next (internal server error)
https://api.spacexdata.com/v3/launches/upcoming (empty results)

This is due to the V3 deprecation plan. The plan was after a certain date, V3/V2 would no longer receive new data updates, to gently push people over to V4. The links will remain active as long as the project exists, but no data will be updated.
It was originally slated for the end of September, but was pushed to November to give people adequate time to update to V4. The final cutoff time was Nov 7th, but I've waited a but past that  just to make sure no last minute needs came in.
I went and removed all the upcoming launches in V3, which is causing this issue. I can adjust the error presented, or add back one upcoming launch, but it's more of a band-aid fix.

@jakewmeyer I wasn't aware about the deprecation plan until yesterday. Would it be possible to extend this period for another week? This might give me some time on fixing this issue since I've to rewrite request logic. Currently, about 8k end users do not have launches feeds visible in my application.
I would very much appreciate if the deprecation period could be extended somehow.

Shortly after the release of the SpaceX Module for MagicMirror I also released a module based on the SpaceLaunchNow api. And personally I like it much better because it also shows other agencys. Is that also an option for you?
It's over here: https://spacelaunchnow.me/api/3.3.0

@jeroenboumans Sorry for the delay, I just backfilled the previous launches, and added two of the upcoming launches back.

@jakewmeyer no worries and thank you, I could prevent the impact on my side by switching the launches endpoints from V3 to V4. Since the structure in querying has changed this took me two evenings to rewrite this. I was planning to switch eventually but was't aware of the changes since I haven't got the time to update my app lately.
Is it possible to introduce a notification system where you can subscribe as a developer (by email for example) for being notified when API version change in structure or when endpoints will be deprecated soon?
@koxm thanks for the tip! Although that would take me some more time to fix the initial problem. ;) I just needed to fix this for my users in the shortest time possible.

Best solution currently would be to watch the repo to get notifications for releases. Going forward, all breaking changes including deprecations would be announced there.

Okay Jake, thanks!


FragPipe v22 worfklow file issue

Uploaded the output from the most recent version of FragPipe (v22) and it worked, but when the .workflow file with the parameters was uploaded => ERROR!

See #367

Fixed in #367


Feature/initial-config

Summary
This Pull Request introduces the initial configuration required to start working on the project. It includes setups for linting, testing, and documentation. It becomes essential to standardize development and ensure code quality.
Detailed Changes
Added Files

[ ] Add the Rubocop gem to improve code quality and enforce Ruby conventions.
[ ] Gemfile
[ ] Gemfile.lock
[ ] .rubocop.yml
[ ] .github/workflows/linters.yml

RSpec Installation for Testing

[ ] Add the RSpec gem for unit and integration tests.
[ ] Initialize RSpec from the terminal, creating the necessary setup.

Modified/Created Files:

[ ] Gemfile
[ ] Gemfile.lock
[ ] spec/spec_helper.rb
[ ] .rspec

License and README

[ ] Add the MIT.md file license to the project to clarify the app's usage rights.
[ ] Created a README.md file to document the project.

How to Test

Run bundle install to install all gems.
Run rubocop to execute the linter.
Run rspec to execute all tests.

Project Impact

[ ] Lays a solid foundation for future development.
[ ] Introduces mechanisms to ensure code quality.

Please, @PabloBona, I need you to review this PR and your feedback will be appreciated.
Greetings,
@ClaudiaRojasSoto

Thank you @PabloBona. See you tomorrow morning. 👍


Popover: integrate autoclosable with LayerManager

The popup closes when we hover over another layer, although it should not

The issue is not correct, the popovers don't close when rendered inside each other


confirm restores still work after force upgrade

Overview
If a user creates a backup on a version of automate before the force upgrade, and then restores will everything just work?
default restore path: should always work since we always restore to the version we took the backup with
question: do folks need to take a new backup after being force upgraded?

The upgrade path I would potentially be worried about here is the "upgrade restore" path; however, that path is never guaranteed to work.  Further, if the automate service knows how to upgrade the V1 data, that path should also work as expected.
The backup/restore tests cover most of what we care about, but I think it is probably good to do one manual test starting from a very old version doing a backup, upgrading, and then trying to restore the old backup via both the normal restore and the upgrade restore.

might be able to keep this super simple by adding the a2-iam-legacy-integration inspec suite to the post-upgrade set of tests run on https://github.com/chef/automate/blob/master/integration/tests/deep_upgrade_backup_restore.sh or https://github.com/chef/automate/blob/master/integration/tests/backup.sh

if upgrade restore does not work, document it and put it in the release notes

@stevendanna i think this is covered by https://github.com/chef/automate/blob/master/integration/tests/airgap_ow_upgrade.sh
if we break upgrade+restore, i think that test would break.

that being said, using a slightly newer version wouldn't be a horrible idea


PATH-swizzling: Support clang, gcc, etc.

Now that #20 is merged, we should also add shims for the following, at the minimum:

gcc
g++
clang
clang++
gas (GNU as)
gold (GNU ld)
lld (LLVM ld)

We should also talk about version shims at some point (e.g. clang-X), but this will provide the basis for that.


Ionic.io Dashboard Auth - only shows one page of users

@dwieeb @ericb what's going on? I'm sorry, but It's getting harder and harder to rely on this system when issues are being addressed...

Hey @sagivf, sorry for the delay here, I don't think it sent out a notification until you pinged us by name. We'll get this fixed up today.

thank you @ericb. Also there is this one https://github.com/driftyco/ionic-cloud/issues/83#issuecomment-260354112 which seems to be of interest to a few. It's ok if some things take there time, but please try to let us know what is the status on issues that are crucial. Thanks again on advance.

Definitely. Thank you! I'll have that deploy issue looked into as well.

@sagivf This should be all fixed up now. Let me know if you're still only seeing a single page. 👍


Improve unsuccessfull response handling

Proposal to address #12:

Allow it to specify a handler, if a request doesn't return a success status code.
Add a default handler, that only handles 404 responses, which returns an empty enumerable if an IEnumerable<> is expected and the default value for all other types (null for objects).
If the handler doesn't mark the response as handled, an Exception containing the status code is thrown, which inherits from HttpRequestException to ensure old users of the library still catch it, but also allows it to handle a specific status code if required.

This looks great 😃 Being able to pass custom handlers makes the client much more flexible to handle different responses. Thank you very much.


CB-9942 Normalize line endings in Camera plugin docs

Adds .gitattributes to ensure LF line endings in the repo
Jira issue

Merged in f5217bf02b9018f94ea7c3694a1172fd9a5b97ac


Upgrade CppToolsApi to use V2.

The following changes are proposed:
Implement canProvideBrowseConfiguration and provideBrowseConfiguration
And also use _cachedSourceFileConfiguration(the latest retrieved configuration) when _getConfiguration failed for specifc
file URI.
The purpose of this change
Currently,  code complete can not browse external include paths such as Qt and so on.

Other Notes/Information

@vector-of-bool This pull request would make vscode-cmake-tools' experience be much better.

@vector-of-bool Have a look, ready to merge


Fixed #24159 -- compilemessages runs across apps.

This updates the command to match the documentation, which states it runs over all .po files.
https://code.djangoproject.com/ticket/24159

buildbot, add to whitelist

Could you add a mention to the 1.9 release notes and also see if any docs in docs/ref/django-admin.txt or docs/topics/i18n/translation.txt need to be updated?

Done. The main docs did not need updating as they already assumed the change of this code had been made :-)
http://djangoci.com/view/PR Builders/job/pull-requests-trusty/1811/ passed, but gave an error trying to update the status on GitHub in http://djangoci.com/view/PR Builders/job/pull-requests-trusty/1811/console : "Could not update commit status of the Pull Request on GitHub."

It may be helpful to add "versionchanged" annotations near the docs that describe the new behavior (which you say already exist) so that people reading the dev docs but using an older version don't expect it.

Apart from the insert/append question, this looks great, thanks!

merged in dc395244d488d056974836fc1bca0f2e25bd5bed, thanks!


Add string features factory

Merging, as this allows us to continue adding more examples, and the string_features factory can very easily be grep replaced later once there is an appropriate factory.


Fixes import of pre-release version #138

PR Summary

Fixed import of pre-release version.

Fixes #138
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] Change is not breaking
[x] This PR is ready to merge and is not Work in Progress
Code changes

[x] Have unit tests created/ updated
[x] Link to a filed issue
[x] Change log has been updated with change under unreleased section

Tested these changes on dev branch and it works fine 🙂


[Console-IoT] Fix alignment issue in add a device page

Type of change

Refactoring

Description
 Fix alignment issue in add a device page
 https://marvelapp.com/prototype/5gbji22/screen/68586288

Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

Can one of the admins verify this patch?


conv int8 for armv7

Hi, I'm trying to use int8 convolution on odroid xu4, and its architecture is armv7a.
I've only modified the TARGET_NAME of "topi/recipe/conv/test_conv_int8_arm.py" as below
TARGET_NAME = 'llvm -device=arm_cpu -target=armv7l-linux-gnueabihf -mattr=+v7,+dotprod'

And message are:
INFO:test_conv_int8_intel:Workload, Kernel_size, FP32_time, INT8_time, Speedup
WARNING:autotvm:Cannot find config for target=llvm -device=arm_cpu -target=armv7l-linux-gnueabihf -mattr=neon, workload=('conv2d_NCHWc', (1, 4, 56, 56, 16, 'float32'), (4, 4, 3, 3, 16, 16, 'float32'), 1, 1, (1, 1), 'NCHWc', 'NCHWc', 'float32'). A fallback configuration is used, which may bring great performance regression.
WARNING:autotvm:Cannot find config for target=llvm -device=arm_cpu -target=armv7l-linux-gnueabihf -mattr=neon, workload=('conv2d_NCHWc_int8', (1, 4, 56, 56, 16, 'uint8'), (4, 4, 3, 3, 4, 16, 4, 'uint8'), 1, 1, (1, 1), 'NCHWc', 'NCHWc', 'uint32'). A fallback configuration is used, which may bring great performance regression.
LLVM ERROR: Cannot select: intrinsic %llvm.aarch64.neon.udot

It seems like tvm automatically chooses the target platform as aarch64... How should I change the code?
Thank you.

Thanks for bringing up the issue. please open a new thread on https://discuss.tvm.ai/


Parquet native reader fails to read an empty Parquet file

Reading an empty Parquet file throws the following error
VeloxRuntimeError: footerLength + 12 < fileLength_ (465 vs. 465)

The fix is to modify the check in velox/dwio/parquet/reader/ParquetReader.cpp:68 to VELOX_CHECK_LE(footerLength + 12, fileLength_);

CC: @Yuhta @kgpai

Thanks @majetideepak , can you change it while you are working on it?

@Yuhta will do!

Fixed and closing


Change Lock Pool to use MapMaker

The current LockPool uses a ConcurrentHashMap and keeps a reference count, then has a thread running to check to remove elements from the map when a certain size is exceeded. This was designed for smaller pool sizes, and becomes costly when having a large amount of locks, for example removing a large directory can cause the GC thread to run continually looking for locks to remove.
This PR replaces this design using the Guava MapMaker using values with weak references. This allows the java GC to collect unused locks when they are not referenced. Since the locks are small and quick to allocate, allowing the GC to happen quickly when not used can be beneficial as they do not get placed on the older generations.
Using a micro bench (added in the PR), which just randomly grabs locks in the range 1-2000000, we see the following:
Old:
Throughput: 4959813.495          ops/s
Time spent in GC: 4359.000              ms
New:
Throughput: 10310378.513          ops/s
Time spent in GC: 391.000              ms
Using the Alluxio StressBench using a 32 core master, and 32 core client, with 200 threads, and 1000000 files, using the GetStatus benchmark we see:
Old:
"throughput" : 155492.72
New:
"throughput" : 168153.16
To do a recursive removal of 2000000 files on a running cluster we see:
Old:
Time to do removal: 4m26.676s
New:
Time to do removal: 3m42.608s
In the below we see a large amount of the CPU work being done is just by the evictor thread during the removal:

In the below we see a large amount of memory being allocated by the evictor thread during the removal:

LGTM. just one concern: will this trigger more frequent system GC and result in other components in alluxio system running slower? Probably we can merge this and see if there's any regression on overmind stressbench tests.

In the micro bench, it seemed to reduce the GC costs even if it was allocating similar amounts of memory, since most the data was just being cleared in the young generation, of course this is just a microbenchmark so it might not be true in the real case, but the bench is doing most the work of the GetStatus operation, eg. traversing the tree. See below the GC rates, for the old version (POOL) vs the new version (MAPMAKER).
Benchmark                                                   (mDepth)  (mDistribution)  (mFileCount)  (mLockPoolType)  (mRocksConfig)  (mSingleFile)  (mType)  (mWidth)   Mode  Cnt       Score   Error   Units
InodeBenchRead.testMethod                                         10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt       455978.694           ops/s
InodeBenchRead.testMethod:·gc.alloc.rate                          10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt         2050.366          MB/sec
InodeBenchRead.testMethod:·gc.alloc.rate.norm                     10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt         5435.238            B/op
InodeBenchRead.testMethod:·gc.churn.G1_Eden_Space                 10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt         2041.299          MB/sec
InodeBenchRead.testMethod:·gc.churn.G1_Eden_Space.norm            10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt         5411.202            B/op
InodeBenchRead.testMethod:·gc.churn.G1_Old_Gen                    10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt           58.427          MB/sec
InodeBenchRead.testMethod:·gc.churn.G1_Old_Gen.norm               10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt          154.881            B/op
InodeBenchRead.testMethod:·gc.churn.G1_Survivor_Space             10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt           66.953          MB/sec
InodeBenchRead.testMethod:·gc.churn.G1_Survivor_Space.norm        10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt          177.484            B/op
InodeBenchRead.testMethod:·gc.count                               10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt           28.000          counts
InodeBenchRead.testMethod:·gc.time                                10          UNIFORM        200000             POOL      javaConfig           true     heap         0  thrpt         2038.000              ms
InodeBenchRead.testMethod                                         10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt       684004.386           ops/s
InodeBenchRead.testMethod:·gc.alloc.rate                          10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt         2989.872          MB/sec
InodeBenchRead.testMethod:·gc.alloc.rate.norm                     10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt         5281.564            B/op
InodeBenchRead.testMethod:·gc.churn.G1_Eden_Space                 10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt         3138.170          MB/sec
InodeBenchRead.testMethod:·gc.churn.G1_Eden_Space.norm            10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt         5543.531            B/op
InodeBenchRead.testMethod:·gc.churn.G1_Old_Gen                    10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt            0.193          MB/sec
InodeBenchRead.testMethod:·gc.churn.G1_Old_Gen.norm               10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt            0.341            B/op
InodeBenchRead.testMethod:·gc.churn.G1_Survivor_Space             10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt            0.173          MB/sec
InodeBenchRead.testMethod:·gc.churn.G1_Survivor_Space.norm        10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt            0.306            B/op
InodeBenchRead.testMethod:·gc.count                               10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt           20.000          counts
InodeBenchRead.testMethod:·gc.time                                10          UNIFORM        200000         MAPMAKER      javaConfig           true     heap         0  thrpt          985.000              ms

One thing I'd like to expand on is your point where the eviction thread is taking a lot of CPU. I think it's to some extent true, but hard to quantify by your illustrations.
The LockPool evictor thread is spinning non-stop and prone to be demanding on CPU. So if the other processes in your machine are not claiming CPU as hard, the spinning thread will see abundant CPU resource and just walk in and take many. In a more realistic Alluxio master, all threads are busy handling FileSystem requests, so the waste from a spinning thread should be much smaller than the illustration heat graph shows.
Despite that being said, I don't have a better way to measure the "spinning cost" directly. I think your microbenchmark is the only best way to indirectly do that, showing the end-to-end improvement by measuring the throughput doing the same command. We don't have a great way to measure exactly the spinning itself, but seeing the overall improvement is just as good.


fix: wrong trustyai env variable name

ref: https://github.com/opendatahub-io/opendatahub-operator/pull/665

/cherrypick main


Frontend to Backend port mapping

My question is similar to #59 and #75, in that I want frontend port 80 to proxy to service port :80, and frontend:443 to proxy to service :443. Even potentially by specifying the port mapping(as an envvar), similar to the docker --port option (-p 80:80 -p 443:443). Is this achievable with custom templates? Or any other way to accomplish this?

The release 1.123 adds this feature through the httpsProxy parameter. You'll need to put a certificate for SSH traffic to work.
Please try it out and let me know if you experience any problem.

This looks like it would work, but one problem: You've added a "mode https" to the https backend, but https option is invalid on "mode". Just make it http and I think this will work perfectly.
From proxy logs:
[ALERT] 353/161452 (109) : parsing [/cfg/haproxy.cfg:55] : unknown proxy mode 'https'.
[ALERT] 353/161452 (109) : parsing [/cfg/haproxy.cfg:63] : unknown proxy mode 'https'.
[ALERT] 353/161452 (109) : Error(s) found in configuration file : /cfg/haproxy.cfg
[ALERT] 353/161452 (109) : Fatal errors found in configuration.

Sometimes I am too confident in automated tests and do not check the feature manually. Sorry for that.
I just pushed the fix. Can you pull the latest version and let me know if it works as expected?

Yes, the mode http is fixed, the config looks good. I'll try and do more testing here before the holidays.
Thanks!!

After more testing,  I’ve discovered that if I start the proxy with -p 31000:80 (instead of 80:80) the dst_port value is now going to be 31000, which causes the acl to be false.
This isn’t exactly what I wanted. If haproxy receives on container port 80, proxy to port 80 backend (regardless of what the server mapping port is). I think using src_port instead would  be more accurate.  If you agree, can you make this change?
More background on what I’m doing: I have a set of services running on one overlay network and a second set on a different overlay network, so I’m starting 2 copies of the proxy-listener and the proxy. Since the 2nd proxy needs to be on some server ports other that 80 and 443, I want the container ports to control the BE ports for proxy.
Larry Henry
Lead Systems Analyst -  Digital Strategy Group Architecture Digital Thread GE
T<PHONE_NUMBER>
F<PHONE_NUMBER>
M<PHONE_NUMBER>
Larry.Henry<EMAIL_ADDRESS>Grace Lake Corporate Center
One Village Center Drive
Van Buren Twp, Michigan 48111
General Electric Company
GE imagination at work
From: Viktor Farcic<EMAIL_ADDRESS>Sent: Sunday, December 18, 2016 10:23 AM
To: vfarcic/docker-flow-proxy<EMAIL_ADDRESS>Cc: Henry, Larry (GE Digital<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: EXT: Re: [vfarcic/docker-flow-proxy] Frontend to Backend port mapping (#86)
The release 1.123https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vfarcic_docker-2Dflow-2Dproxy_releases_tag_1.123&d=DQMFaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=GSgjlded6A34H1Zzf6ZoPHdNBk0z6BdMZeSOPYZVcvE&m=SQBkx7wLz5Vizd-oRAiHENfkXZCjjx-ylnief4pBQgs&s=-KLQxkgt2qdgyl4mbJNeCsLMtkkK6p0nkTPGWIkPmtw&e= adds this feature through the httpsProxy parameter. You'll need to put a certificatehttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vfarcic_docker-2Dflow-2Dproxy-23put-2Dcertificate&d=DQMFaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=GSgjlded6A34H1Zzf6ZoPHdNBk0z6BdMZeSOPYZVcvE&m=SQBkx7wLz5Vizd-oRAiHENfkXZCjjx-ylnief4pBQgs&s=HljDJMRIxJ__jG2auCkYiv-F4VZ3q_4OBIDMF6C7XzY&e= for SSH traffic to work.
Please try it out and let me know if you experience any problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vfarcic_docker-2Dflow-2Dproxy_issues_86-23issuecomment-2D267827204&d=DQMFaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=GSgjlded6A34H1Zzf6ZoPHdNBk0z6BdMZeSOPYZVcvE&m=SQBkx7wLz5Vizd-oRAiHENfkXZCjjx-ylnief4pBQgs&s=_DTbDR9ySqqwXNLbtFMJCRWfviewxjJeA4r9tCX6omI&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWkCGBVThLxo3-2D-2DgZPwoTwW8qehBkOMiks5rJU-5FogaJpZM4LPSt5&d=DQMFaQ&c=IV_clAzoPDE253xZdHuilRgztyh_RiV3wUrLrDQYWSI&r=GSgjlded6A34H1Zzf6ZoPHdNBk0z6BdMZeSOPYZVcvE&m=SQBkx7wLz5Vizd-oRAiHENfkXZCjjx-ylnief4pBQgs&s=8-W_zywFJ508B0UhRHuii9TqBdvqADsnABMZTwIQI3c&e=.

You're right. That was an overlook on my part. It should be src_port.
New release with the fix will be available in a few minutes. Please try it out and let me know if it worked.


gNMI-1.14 enhancements

separate metadata checks for a more accurate representation of the steps described in README.md.

No unchanged relevant lines lost coverage.
Overall coverage remained the same at 55.448%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1949

Relevant Lines:
3515

💛  - Coveralls


switch to codecov for code coverage

Codecov Report

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

@@           Coverage Diff            @@
##             master      #5   +/-   ##
========================================
  Coverage          ?   96.4%
========================================
  Files             ?      11
  Lines             ?     473
  Branches          ?       0
========================================
  Hits              ?     456
  Misses            ?      17
  Partials          ?       0

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 4a3f8d2...07a68d7. Read the comment docs.


How do i make a wavy background stage with no source code?

Describe your problem here.
I used this:
function onCreate()
makeLuaSprite('chains', 'chains', -300, -200);
setLuaSpriteScrollFactor('chains', 0.6, 0.6);
addGlitchEffect('chains', 3, 4);
addLuaSprite('chains', false)
end
And it ended up crashing. Is there anyway to do this?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

it wasn't implemented yet, and I don't think it will anytime soon

sad :C


Protocol as inputs instead of settings

Targetting main but it's really a branch from #49

We should aim to merge this one after merging #49


Issue54 test api and v1 responses

Fixes #54

Addition of tests to ensure /api and /api/v1 are responding with expected values
Adjusted  /api/v1/user/password test expect values to not point at body[0] but rather body.

To Test:
$ npm test

  Requests to the root (/api) path
    ✓ GET: Returns a 200 status code (84ms)
    ✓ GET: Returns JSON format
    ✓ GET: Returns v1 route

  Requests to v1 root (/api/v1) path
    ✓ GET: Returns a 200 status code
    ✓ GET: Returns JSON format
    ✓ GET: Returns v1 route

  Requests to the root (/user/password) path with missing required body parameters.
    1) POST: Invalid password reset request with missing body parameters returns expected 422 - Unprocessable due to request validation error results.

  Requests to the root (/user/password) path
    2) POST: Valid password request with only user_id returns expected results.
    3) POST: Valid password request with only email returns expected results.
    4) POST: Valid password request with only mobile returns expected results.

  6 passing (5s)
  4 failing

👍


chore: check and warn if codeowners present but junit missing file paths

If CODEOWNERS is present but jUnit is missing file paths, print a suboptimal message.

Oh! Also, I forgot to ask for a test if we can as well 👍


@all-contributors bot

@all-contributors bot

@all-contributors please add firefly-cpp for code

@all-contributors please add @firefly-cpp for code

@all-contributors please add @StrajnarFilip for code, tests

@all-contributors please add @musicinmybrain for doc

@all-contributors please add @musicinmybrain for platform

@all-contributors please add @firefly-cpp for platform, example, ideas, mentoring, doc

@all-contributors please add @StrajnarFilip for ideas, example

@all-contributors please add @kloczek for bug


Default Request Data in Model Schema on Swagger UI using Swagger 6.0.0-rc1-final

Hi All,
Please help me out in implementing default request data to the controller action of the model schema on the Swagger UI with Swagger 6.0.0-rc1-final with C#.
i found relevant answer for this  but not the exact answer at https://mattfrear.com/2016/01/25/generating-swagger-example-requests-with-swashbuckle.
Above link works for Swagger 5.0, As i'm using Swagger 6.0.0-rc1-final,  the functions which are used at above link are not found in this swagger which i’m using.
Anyone worked with Swagger 6.0.0-rc1-final, Please post your answer?

For that version, the config settings you need are available via a separate calls ConfigureSwaggerSchema and ConfigureSwaggerDocument on IServiceCollection.
It's also worth noting that despite the names, 6.0.0-rc1-final is actually a couple of versions behind. The latest version is 6.0.0-beta902. Let's just say naming got a little messed up while ASP.NET Core was stabilizing itself. I'm trying to get a stable release of SB 6 out real soon and that should clear up all the confusion.


WRKLDS-649: Guard controller: set the readiness probe endpoint explicitly

KS and KCM does not publish /readyz endpoint, only /healthz. On the other hand KA publishes both /healthz and /readyz. The /readyz endpoint is more suitable for checking the readiness.
Thus, the guard controller needs to be allow to explicitly set the readiness probe endpoint.
/hold

/approve
so this can merge once someone is happy with the shape.

/hold cancel

Proof PRs:

https://github.com/openshift/cluster-kube-scheduler-operator/pull/459
https://github.com/openshift/cluster-kube-apiserver-operator/pull/1437

The KA guards are posting ready when reaching to the /readyz endpoint. KS stays unchanged.

/cherrypick release-4.12


Fix OnMiss not firing in all cases in Distributed Cache

In the current implementation of DistributedCacheAccessor (both Sync and Async) the only time _events.OnMiss fires is when

The de-serialized entry is null
The de-serialization fails

The bug is in case 1. The code currently checks that the value returned from the IDistributedCache is non null and returns early if it is null. This means we never start de-serialization and therefore never call _events.OnMiss. This leads to inaccurate distributed cache miss counters.
To fix this I added _events.OnMiss to the if block that checks the result from the IDistributedCache. This will cause the OnMiss event to also be triggered in the case of an exception from the IDistributedCache. If this behavior is not desired I can update the code to only fire the event when the result from the IDistributedCache is null.

Hi @ConMur and thanks for using FusionCache!
You seem to have a point: I'll look into it asap and will let you know.
Thanks!

I'm looking into this, will update soon.


[Unity][Transform] Raise error in FuseOpsByPattern for SSA violation

Internally, FuseOpsByPattern makes a mapping from relax variables to the fused group containing that variable.  If the input module violates SSA, this map may be ill-formed.  While not strictly necessary for FuseOps to handle ill-formed inputs, checking it at this level provides better error handling than propagating it to downstream passes.
This commit checks for ill-formed inputs that would produce invalid fused outputs and raises an error.

Rebased onto main to resolve conflicts


shenyu gateway how to use consul acl Usually companies use consul acl ？

shenyu gateway how to use consul acl Usually companies use consul acl ？

ACL is a security standard, do you mean any implement of this standard ?
If you mean, shenyu dose not support it currently.
Welcome you to participant shenyu contributor meeting and share your concrete usage scenario, we would like to discuss more detail.

Because the operation and maintenance consul has set access permissions（acl token），shenyu need read or write consul  but not support set acl token   The same goes for zookpeer You cannot access ， that already have access control. （ user: xxx, pwd:  yyy）

we will discuss it in recent contributor meeting, welcome you to participate to the discussion(we will send invite mail to dev subscribe). Or if you have any idea now, welcome to create a pull request to resolve it.

Hi @XiBeiWaQiang , The meeting is planned to hold on Thursday this week, please feel free to join us and share your idea.
In case you miss the meeting, please subscribe the dev mail list via:
https://shenyu.apache.org/community/subscribe-email/


Change in text fragment.

What article on docs.github.com is affected?
https://github.com/github/docs/blob/main/translations/es-XL/content/github/developing-online-with-codespaces/about-codespaces.md

What part(s) of the article would you like to see updated?

Additional information
The fragment of text "cad acodespace" needs to be changed to "cada codespace".

Thank you so much for opening an issue and PR for this fix @yair-rodriguez 💖
Unfortunately, we are not able to accept pull requests for translated content. Our translation process involves an integration with an external service at crowdin.com, where all translation activity happens. We hope to eventually open up the translation process to the open-source community, but we're not there yet.
I'll open an issue internally for our translators to look at.


error "Pass a string for project" or "Column created_at is not present in table my_project.dbt_elementary.data_monitoring_metrics" if upgrade dbt package version from 0.7.3 to 0.14.0

Describe the bug:

Currently I upgrade dbt elementary package version from 0.7.3 to 0.14.0
I got an error when run dbt test for elementary anomalies test:
dbt test
'''
08:19:43 Unhandled error while executing
Pass a string for project
08:19:43 4 of 26 ERROR elementary_source_table_anomalies_ileyshop_digdag_lobby_src_transaction_master_row_count [ERROR in 0.41s]

'''
If my command add flag
'''
dbt test --target prod
'''
the error will disappear and error will become
''' 08:41:26 on-run-end failed, error:
Column created_at is not present in table my_project.dbt_elementary.data_monitoring_metrics at [18:9]

'''
my dbt_project.yml setting for elementary:
'''
models:
elementary:
+database: my_project
+schema: dbt_elementary
+enabled: "{{ (target.name == 'prod') | as_bool }}"

'''

To Reproduce
Steps to reproduce the behavior:

1.upgrade dbt package from 0.7.3 to 0.14.0
2. run elementary anomalies dbt test or dbt test --target prod
Expected behavior
All tests pass and got No error

Environment (please complete the following information):

dbt package Version: 0.14.0
dbt core Version: 1.7.8

Additional context:
I don't change any config in dbt_project.yml.

Should I do any thing like drop  "data_monitoring_metrics" and other tmp tables for upgrading to v0.14.0?

To solve "Column created_at is not present " problem, I just have to delete previous temp table that elementary tests create.
And add a column "created_at" with timestamp type in data_monitoring_metrics table.
Then solve the problem in the future

To solve "Column created_at is not present " problem, I just have to delete previous temp table that elementary tests create.
And add a column "created_at" with timestamp type in data_monitoring_metrics table.
Then solve the problem in the future.


WIP: Adopt the proposed structure for formulas

Work In Progress

@abunashir this looks good!!

Some improvements to make:

rename ms_vista to cleartype
rename ms_system to ms_truetype_fonts

Can we migrate the information in Formulas from fontist/formulas?

We are going to remove the old one soon, and start using the formulas from the formulas repository.

Some improvements to make:

rename ms_vista to cleartype
rename ms_system to ms_truetype_fonts

Can we migrate the information in Formulas from fontist/formulas?


Error return nil

This code worked in 1.9.0:
func TestExpr(t *testing.T) {
	env := map[string]any{
		"semver": func(value string, cmp string) (bool, error) { return true, nil },
	}

	prog, err := expr.Compile(`semver("1.2.3","= 1.2.3")`, expr.Env(env), expr.AsBool(), expr.AllowUndefinedVariables())
	if err != nil {
		t.Fatalf("compile failed: %v", err)
	}

	res, err := expr.Run(prog, env)
	if err != nil {
		t.Fatalf("could not execute compound query: %s", err)
	}

	if res != true {
		t.Fatalf("expected true got: %v", res)
	}
}

Should the semver func return error then the expr.Run() would fail with that error, great and when its nil the bool would be used - also great.
Since 1.10.0 though when the error is nil the following happens:
compound_test.go:80: could not execute compound query: interface conversion: interface is nil, not error (1:1)
         | semver("1.2.3","= 1.2.3")
         | ^

So its not treating nil errors like nil
thanks for this amazing package!

I see, will try to understand the issues and find a solution later today.

The problem occurs in the following section:
// https://github.com/antonmedv/expr/blob/9009d4da3de69a1e5dcbb284a9140129939de6f1/vm/vm.go#L303
out := fn.Call(in)
if len(out) == 2 && !runtime.IsNil(out[1]) {
	panic(out[1].Interface().(error))
}

On the other hand, it works as expected after adjusting the If condition as follows:
// https://github.com/antonmedv/expr/blob/9009d4da3de69a1e5dcbb284a9140129939de6f1/optimizer/const_expr.go#L76
if len(out) == 2 && out[1].Type() == errorType && !out[1].IsNil() {

Fixed in b062494f3c863d0dfabf0eff469b42f995b0693e
Will create a new release.

I can confirm this is fixed, thanks a lot for this @antonmedv !


Jar file

Hi,
I couldn't locate the binary jar file MutationHotspotsDetection-1.2.1.jar. Would you please provide?
Thanks

Sorry I found it!

just uploaded: https://github.com/knowledgesystems/mutationhotspots/releases


Trouble running scaled down OC4 test case

Hi Everyone,
I trying to run OC4 test case with scaled down parameters. I used the scaling ratio of 1:50. I reduced the simulation time step to 0.0018 from 0.0125 which is used for full scale model. I get the following error.

I further reduced the time step to 0.0015. The simulation starts and it hangs. It doesn't show any error or proceeds. I'm not sure where I'm going wrong. Can anyone help me on this?

Dear @SudharsanKalidoss,
Are you applying scaling laws for mass, stiffness, geometry, time, etc.?  Do you derive the time step from the scaling law?  Are you sure you've applied the scaling laws correctly to all parameters?  Are you also changing constants within the controller?
In your first example, it sounds like the model is not balanced in the vertical direction (weight + vertical mooring pretension does not equal the buoyancy) or the model is numerically unstable.  I'm not sure I understand why OpenFAST would ever just hang (I've not heard of such an issue before), but there are many unexpected warnings before that anyway.  Both indicate a problem with the simulation set up.
Best regards.

Dear @jjonkman,
I used the scaling law as suggested in the following paper. I applied scaling law to all the parameters including mass, inertia, stiffness, geometry, and time.
FAST code verification of scaling laws for DeepCwind Floating Wind System Tests.pdf
Is it possible to avail the modeling parameters for DeepCwind floating wind system which is used for scaled down experiment?
Is there something else do I need to do to balance weight and vertical mooring pretension other than applying scaling law?
Thanks

Dear @SudharsanKalidoss,
The scaling laws presented in this paper, if properly applied to all variables, should keep the system stable.  In fact, as that paper shows, the results are identical if you run the simulation at model scale and scale the results back up to full scale versus running the simulation at full scale.  For this reason, we rarely build an OpenFAST model at model scale.
From the paper, time is scaled by lambda^(0.5), so for lambda = 1/50 and a full-scale DT of 0.0125 s, model-scale DT of 0.00177 s should be sufficient.  I would guess something in your model has not been consistently scaled.  Unfortunately, I'm not sure where the FAST model used in this paper is stored, so, I can't share it with you.  I would suggest simplifying the model to debug, e.g., does the model work as expected if you model the 6 degree of freedom rigid-body motion of the floater excited only be waves?
Best regards,

Dear @jjonkman,
I tried running model with only HydroDyn enable. It was working fine. Then enabled moorDyn I encountering the following error. It seems like there is a format error for the moorDyn input file.

Thanks

Dear @SudharsanKalidoss,
You say that you enabled MoorDyn, but the error message is related to MAP++.  Have you set CompMooring to 1 (MAP++) or 3 (MoorDyn) and is your mooring input file formatted for MAP++ or MoorDyn?
Best regards,

Dear @jjonkman,
I'm sorry. I didn't notice that. I changed it and MoorDyn solver ran. It shows a different error.

Thanks

Dear @SudharsanKalidoss,
Well, now it looks like your simulation has gone numerically unstable very quickly...within the first few time steps.  I would guess you still some scaled inputs set incorrectly.  Are you running the simulation now with only ElastoDyn, HydroDyn, and MoorDyn enabled and only the platform DOFs enbled in ElastoDyn?  Is the solution what you expect when you only have one platform DOF enabled?  Does a particular platform DOF bring about the instability?
Best regards,

Dear @jjonkman,
Yes, I'm running the simulation with only ElastoDyn, HydroDyn and MoorDyn enabled. I enabled only the platform DOFs. I tried running the setup by enabling the DOFs one by one. The simulation runs fine with 5 DOFs (except yaw DOF). Including yaw DOF diverges the simulation. Combination of yaw and sway DOFs works fine. COmbination of yaw with any other DOFs diverges.

Thanks

Dear @SudharsanKalidoss,
Does that give you any idea as to what the problem is?  Something set incorrectly with the yaw inertia or yaw stiffness?
Best regards,


make activation implicit ?

More practical and easy to use, but hides complexity.
Is it a good idea ??

We should probably have two ways t use this pacakge :

explicit rosimport.import_messages() that work similarly to libimport.import_module and does the samw thing as the implicit version
a way to activate the implicit version currently implemented, using importer hooks.

So the answer for this question is NO.


Update low-level graph spec to use any hashable for keys

xref https://github.com/dask/dask/pull/7142/files#r569620160
cc @eriknw @crusaderky

I think it would be a good idea to emphasize in custom_collections.rst that the definition of key there is stricter than in the general case


cs command not available after brew install

I feel like I’ve installed coursier before and this worked…
But when I do “brew install coursier”, it seems to install fine and symlink the file coursier but not cs as the docs would seem to imply
Should the brew formula be doing this or is something amiss??

I was mistaken. Apparently these two brew scripts are different
brew install coursier

brew install coursier/formulas/coursier

Looks like it install cs correctly now as recently tested.


Import is success, but missing on Index

When I do KlassIndex.import! it returns true
but I can't find my records on Kibana
if there is invalid data to one of my record, shouldn't it be fail?

Could you provide more details on this so we could reproduce it?


This video has content from UMG, WMG etc...

Recently i have this error:
with error: This video presents content from WMG. Watch it on youtube.
Domain: XCDYouTubeVideoErrorDomain
Code:   150
I Think that this are caused by an youtube Api change... Anyone know what i can do to bypass this situation? Thanks a lot!

same issue here too. Just started happening this morning onwards for various channels.

Same behavior here :(

Thanks @0xced, that was a quick fix! We owe you!

Thank you !! @0xced is there a donation page where we can help out?

@Pranoy1c There is a Donate button bellow the Psy video in the XCDYouTubeKit home screen here in git hub.

Thanks a lot @0xced!  Is there any way to avoid having to upload a new version of our apps to the app store when api changes like this occur?

@sajsanghvi This feature is planned, see #199.

@0xced I'm still seeing this issue with the latest version (2.5.3). I'm using this video identifier: wTMFpRwuSYg
"This video contains content from UMG. It is restricted from playback on certain sites or applications. Watch on YouTube";
NSURL = "https://www.youtube.com/get_video_info?el=embedded&hl=en&ps=default&video_id=wTMFpRwuSYg";

+1


[Feature Request] Limit number of rows displayed for large arrays/dictionaries

Hello,
I hope you're doing well. I would like to propose a feature enhancement that I believe could improve the user experience when dealing with large arrays or dictionaries. Currently, when using objprint to print an object with a large array or dictionary, it displays all the elements, making the output lengthy and overwhelming.
It would be great to have an optional argument that allows us to limit the number of rows displayed for arrays or dictionaries, similar to how pandas handles DataFrame display. By default, pandas shows 5 rows at the beginning and 5 at the end, with an ellipsis (…) in between to indicate that more rows are not shown. This approach gives users an idea of what's inside the data structure without cluttering the terminal output.
For instance, consider the following example:
from objprint import op, install

class Sample:
    def __init__(self, dict) -> None:
        self.data = dict

test_dict = {i:i for i in range(100)}

_ = op(Sample(test_dict))

In this case, objprint would print all 100 key-value pairs in self.data, making the output very long. It would be beneficial to have an option to display only the first and last few lines of the dictionary.
If you think this feature would be a valuable addition to the library, I would be more than happy to contribute to its implementation. Please let me know your thoughts on this suggestion.
Thank you for your time and consideration! Looking forward to hearing from you.
Best Regards,
Erwin

_ = op(Sample(test_dict), elements=10)?

Yes, or as a global config if the user wants to use the same element setup in the same file.

Yes, or as a global config if the user wants to use the same element setup in the same file.

I meant - try _ = op(Sample(test_dict), elements=10)

Or op.config(elements=10)


[Backport] Show the username as a fallback on feeds if full name is blank

A backport of #10438.

blocked until #10438 got merged


[core] Update release date for core packages

Set a date for the next core release.

@chradek do you want a different day for core-amqp?


fix: device "all" messaging

Codecov Report

Merging #66 (8fd724f) into master (69cf770) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master      #66   +/-   ##
=======================================
  Coverage   33.40%   33.40%
=======================================
  Files          19       19
  Lines         470      470
  Branches      180      180
=======================================
  Hits          157      157
  Misses        238      238
  Partials       75       75

Impacted Files
Coverage Δ

src/fetchAndWaitForLighthouseAudits.js
0.00% <ø> (ø)

src/postPrComment.js
1.49% <0.00%> (ø)

Continue to review full report at Codecov.

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


Possible memory leak with SqlCommand not being disposed

SQLCommand doesn't seem to be disposed, wrap the initialization in a using in non-transaction managed code to address

closed with #35


net: tcp: Fix possible race between TCP work items and context unref

Fix the possible race between TCP work items already scheduled for
execution, and tcp_conn_unref(), by moving the actual TCP context
releasing to the workqueue itself. That way we can be certain, that when
the work items are cancelled, they won't execute. It could be the case,
that the work item was already being processed by the work queue, so
clearing the context could lead to a crash.
Remove the comments around the mutex lock in the work handlers regarding
the race, as it's not the case anymore. I've kept the locks however, as
they do make sense in those places.
Fixes #59752

DNM till confirmed by the reporter.

Removing DNM as per https://github.com/zephyrproject-rtos/zephyr/issues/59752#issuecomment-1831536826


Remove all unnecessary files from npm package

Current version of Redux being installed has lots of unnecessary files:

With this update it looks like:

Tested by installing locally via npm install ~/nkbt/redux

:+1:


$cfg->addConnection unfriendly for DI containers like Aura DI 3.x

This is a great container but allows just one argument per parameter, meaning this mysql or whatever should go inside the array and addConnection should accept just an array. Of course, for compatibility reasons this form should also be kept, but we should allow just one array as an argument.
$cfg->addConnection('mysql', [
    'dbname' => 'mydb',
    'user' => 'user',
    'password' => 'secret',
    'host' => 'localhost',
    'driver' => 'pdo_mysql',
]);

becomes like:
$cfg->addConnection([
    'name' => 'mysql',
    'dbname' => 'mydb',
    'user' => 'user',
    'password' => 'secret',
    'host' => 'localhost',
    'driver' => 'pdo_mysql',
]);

and the contract should be like:
/**
* $name string|array
*/
public function addConnection($name, $dsn = null, $default = false) {}

In the future releases it should probably loose the other parameters and keep just the array (debatable).

I am pretty sure Aura.DI can handle method signatures with multiple parameters, and can support configuring Spot database connections as-is. Can you shed some light here, @pmjones?

Yes, some light here would be nice :) Sorry for posting this here, it is not related with Spot, but here is what I have tried:
<?php
use Aura\Di\ContainerBuilder;

require_once dirname(__DIR__) . '/vendor/autoload.php';

class Test
{
    public function setMe($param1, $param2)
    {
        echo "$param1 $param2";
    }
}

$builder = new ContainerBuilder();
$di = $builder->newInstance();
$di->setters['Test']['setMe'] = ['param1' => 1, 'param2' => 2];
$di->set('test', $di->newInstance('Test'));

$test = $di->get('test');

$di->setters['Test']['setMe']['param1'] = 1;
$di->setters['Test']['setMe']['param2'] = 2;
$di->set('test2', $di->newInstance('Test'));

$test2 = $di->get('test2');

You should be able to do it on of these two ways:
This one is untested:
<?php
$di->set('config', $di->lazyNew('\Spot\Config',
    [],
    [
        'addConnection' => [
            'name' => 'mysql',
            'dsn' => [
                'dbname' => 'mydb',
                'user' => 'user',
                'password' => 'secret',
                'host' => 'localhost',
                'driver' => 'pdo_mysql',
            ]
        ]
    ]
));

$di->set('db', $di->lazyNew('\Spot\Locator', [
    'config' => $di->lazyGet('config')
]));

This one is tested:
$di->set('db', $di->lazy(function () {
    $cfg = new \Spot\Config();
    $cfg->addConnection('mysql', [
        'dbname' => 'mydb',
        'user' => 'user',
        'password' => 'secret',
        'host' => 'localhost',
        'driver' => 'pdo_mysql',
    ]);
    return new \Spot\Locator($cfg);
}));

The first way sounds interesting, I will try it tonight.
But the second one is really just normal instantiating the Config object, although lazy load feature from the container has been used. I mean this is not bad, but nor as elegant as $di->setters.
I still think $di->setters should support more arguments, just like $di->params does.

Thanks @silentworks. I am closing this issue since this is not a problem with Spot.

Just to note, maybe someone else will have the same doubt. The first way from @silentworks does not work and triggers the same error as the setters array:
PHP Fatal error:  Uncaught exception 'ErrorException' with message 'Missing argument 2 for Spot\Config::addConnection(), called in /media/xxx/vendor/aura/di/src/Injection/Factory.php on line 109 and defined' in /media/xxx/vendor/vlucas/spot2/lib/Config.php:24
```

The second way works, although it is just a hack. The only modification I did is to actually use anonymous to create a Config instance and not the Locator instance. So, something like:

```
use Spot\Config as SpotConfig;
use Spot\Locator;

        $di->set('spot_config', $di->lazy(function () use ($dbConfig) {
            $spotConfig = new SpotConfig();
            $spotConfig->addConnection('mysql', $dbConfig, true);
            return $spotConfig;
        }));
        $di->params[Locator::class]['config'] = $di->get('spot_config');
        $di->set('spot_orm', $di->lazyNew(Locator::class));
```

This is not a hack, it is actually recommended to do it this way if you cannot do it using any of the built in lazy* functions.
http://auraphp.com/packages/Aura.Di/lazy.html#1.6.5

Sorry for my English, I wanted to say a surrogate actually. Don't you think there should be a clean way?

There is nothing inherently wrong with a method signature having a string as the first parameter, and an array as the second. There is no issue here that requires any changes in Spot, and certainly not one that causes backwards compatibility breakages in future releases. I am not changing this method signature.

No Vance, I am not saying that you should change anything. This is obviously a problem of Aura, sorry for the inconvenience 😉 The problem is that there is no response from their side, but that's OK also, we are part of the OpenSouce comunnity anyway. Thank you for your great work, I like Spot very much and I am going to continue using it and try to help as much as I can.

@acim I see. If that is the case, then I recommend you open a ticket/issue record in the Aura.DI project so you can bring it to their attention a bit better. You are likely to get a lot more help with the DI container there since it's their own project :)

I did that quite some time ago (before here) ;)


Responsive design issue/UI layout issue.

Using a Windows 10 PC, reproducible in both Google Chrome and Firefox, with my screen resolution 2560 x 1440 I get the following experience:

If I get time this weekend I'll submit a PR to fix it :)
Thanks for a great tool @jc21

Then can you please also fix the following use case:

For the main issue, it feels like there would be some errors in the browser console that would help

Also seeing this on a 4k display. There are no errors in the console, only the welcome messages.
The following temporarily fixes it for me:

Find the div with id=app. In Chrome, uncheck display:flex. Alternatively, locate .page in main.css and remove the same.

You should now see a fixed layout minus the footer.

To fix the footer, search for the  and add the highlighted CSS classes either in Chrome, or in main.css.
Unfortunately any reboots to the host or Docker container will result in having to redo these steps.

The UI is based entirely on Tabler, and it uses the layout and styles from there. It's likely that it's a problem with their project.

Confirmed that this is because tabler released a new version compatible with latest bootstrap. Have rolled back tabler and will be fixed in 2.0.13

Thanks a bunch!


Require all the things

Require all of the JSON spec files rather than dynamically loading them as intended.
Due to substack/node-browserify#377, we have to require all of the JSON spec files and then provide an interface for only returning the ones we want from getFileSpec and getEdits.

@doelleri I fixed the casing of the hmdaSyntactical* vars everywhere in 8b023e4

@poorgeek For maintainability, I would recommend this be updated with:

edits and specs merged into a single map called specs. Layout like:

var specs = {
        '2013': {
            'filespec': filespec2013,
            'hmda': {
                'macro': hmdaMacro2013,
                'syntactical': hmdaSyntactical2013
            },
            'lar': {
                'quality': larQuality2013,
                'syntactical': larSyntactical2013,
                'validity': larValidity2013
            },
            'ts': {
                'quality': tsQuality2013,
                'syntactical': tsSyntactical2013,
                'validity': tsValidity2013
            }
        },
        '2014': {
                   ...
        }
};

the vars for the file specs use filespecYYYY as the var name, not specYYYY
Update getFileSpec and getEdits to use the new combined specs map and variables
Remove VALID_YEARS
Update getValidYears() to iterate the top level properties of the specs map


fix(core): correctly delete backups for new entries

Summary
Fixes a bug where new entry backups weren't being deleted, and the CMS kept offering to restore when creating a new entry.
Closes #2259

I'm on netlify-cms-app v 2.12.12 and still getting this prompt every time?


LONG columns in Oracle dictionary tables causing “Stream has already been closed” Exception

Environment
Liquibase Version: 3.8.0 (probably 4.8.0 as well)
Liquibase Integration & Version: java
Liquibase Extension(s) & Version:
Database Vendor & Version: Oracle
Operating System Type & Version: Linux Ubuntu & Windows
Description
While executing some changeSets an SQLexception might get thrown.
Steps To Reproduce
Behaviour is not reproducable, but does happen regularly.
Actual Behavior
Caused by: java.sql.SQLException: Stream has already been closed     at oracle.jdbc.driver.LongAccessor.getBytesInternal(LongAccessor.java:129)     at oracle.jdbc.driver.Accessor.getBytes(Accessor.java:940)     at oracle.jdbc.driver.LongAccessor.getString(LongAccessor.java:156)     at oracle.jdbc.driver.CharCommonAccessor.getObject(CharCommonAccessor.java:616)     at oracle.jdbc.driver.GeneratedStatement.getObject(GeneratedStatement.java:235)     at oracle.jdbc.driver.GeneratedScrollableResultSet.getObject(GeneratedScrollableResultSet.java:306)     at org.apache.commons.dbcp2.DelegatingResultSet.getObject(DelegatingResultSet.java:351)     at org.apache.commons.dbcp2.DelegatingResultSet.getObject(DelegatingResultSet.java:351)     at liquibase.util.JdbcUtils.getResultSetValue(JdbcUtils.java:80)     at liquibase.executor.jvm.ColumnMapRowMapper.getColumnValue(ColumnMapRowMapper.java:73)     at liquibase.snapshot.JdbcDatabaseSnapshot$CachingDatabaseMetaData$GetNotNullConstraintsResultSetCache$1.getColumnValue(JdbcDatabaseSnapshot.java:934)     at liquibase.executor.jvm.ColumnMapRowMapper.mapRow(ColumnMapRowMapper.java:35)     at liquibase.executor.jvm.RowMapperNotNullConstraintsResultSetExtractor.extractData(RowMapperNotNullConstraintsResultSetExtractor.java:43)     at liquibase.snapshot.JdbcDatabaseSnapshot$CachingDatabaseMetaData$GetNotNullConstraintsResultSetCache.extract(JdbcDatabaseSnapshot.java:940)     at liquibase.snapshot.ResultSetCache$ResultSetExtractor.cachedExecuteAndExtract(ResultSetCache.java:295)     at liquibase.snapshot.ResultSetCache$ResultSetExtractor.executeAndExtract(ResultSetCache.java:274)     at liquibase.snapshot.ResultSetCache$ResultSetExtractor.executeAndExtract(ResultSetCache.java:266)     at liquibase.snapshot.JdbcDatabaseSnapshot$CachingDatabaseMetaData$GetNotNullConstraintsResultSetCache.oracleQuery(JdbcDatabaseSnapshot.java:917)     at liquibase.snapshot.JdbcDatabaseSnapshot$CachingDatabaseMetaData$GetNotNullConstraintsResultSetCache.fastFetchQuery(JdbcDatabaseSnapshot.java:872)     at liquibase.snapshot.ResultSetCache$SingleResultSetExtractor.fastFetch(ResultSetCache.java:393)     at liquibase.snapshot.ResultSetCache.get(ResultSetCache.java:76)
Expected/Desired Behavior
No more exception.
Additional Context
Some resources on the web

https://blog.jooq.org/oracle-long-and-long-raw-causing-stream-has-already-been-closed-exception/
https://stackoverflow.com/questions/10174951/hibernate-custom-type-to-avoid-caused-by-java-sql-sqlexception-stream-has-al

The column SEARCH_CONDITION from table USER_CONSTRAINTS and the column DATA_DEFAULT from ALL_TAB_COLS are both LONG columns.
After I removed DATA_DEFAULT from the code and replaced the value of SEARCH_CONDITION by NULL, the problem has gone away. However I assume that breaks some other functionality.

Thanks for the details @ponziani we will take a look.

@kataggart any feedback on this ?

@ponziani it's in the queue for development, but we unfortunately have not been able to get to it yet -- we, of course, welcome any PRs from the Community to address the issue as well. That always helps things move along quicker. Thanks!


Add CoreIssuance/issueTo

Includes adding of a recipient parameter in issueInternal

Pull Request Test Coverage Report for Build 3249

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 3246:
0.0%

Covered Lines:
698

Relevant Lines:
698

💛  - Coveralls


fixup for Optic concat expression test

Because the enode Optic API is no longer sniffing for and enforcing data types but, instead, allowing implicit casts to occur, a string numeric in the concat test is turning into a numeric, which is causing an equality test to fail.

Assigning for awareness and not because additional coverage is needed.
The API didn't change, only a unit test.
The error in the "expressions concat" unit test should be gone now.
If so, this issue can be closed.

No regressions seen. Verified on develop branch with 10.0.1 ML server build dated 07022019.


HttpContext not working in Kestrel in Visual Studio Code

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
I have set up Windows authentication with the Negotiate package but in Kestrel I am noticing that the HttpContext gets dropped after the initial ClaimsTransformation works.  It's later down the pipeline where I'm not getting the context.User in the AuthorizationRoleHandler.  This worked in ASP.NET Core 5/IIS with essentially the same code, except Program was split into Startup, etc.
I'm noticing that the AuthenticationType is showing as NTLM.  My understanding is that Kerberos will try Negotiate first and will fall back to NTLM.  This seems to be occuring as I see NTLM listed as the AuthenticationType when I'm debugging.
builder.Services.AddAuthentication(NegotiateDefaults.AuthenticationScheme)    .AddNegotiate();
I'm also wiring up a Claims Transformer.
builder.Services.AddSingleton<IClaimsTransformation, MyClaimsTransformer>();
Within the Claims transformer everything is working fine.  It picks up the user identity and assigns the claims;
public class MyClaimsTransformer : IClaimsTransformation
{
    public Task<ClaimsPrincipal> TransformAsync(ClaimsPrincipal principal)
    {
         ClaimsIdentity identity = principal.Identity as ClaimsIdentity;
         //add claims
        return Task.FromResult(principal);
    }
}

I've also wired up an HttpContextAccessor
builder.Services.AddHttpContextAccessor();
builder.Services.AddAuthorization(options =>
{
    options.FallbackPolicy = options.DefaultPolicy;
    options.AddPolicy("AuthenticatedOnly", policy => {
          policy.Requirements.Add(new AuthenticatedRequirement(true));
    ]);
]);

builder.Services.AddSingleton<IAuthorizationHandler,AuthenticatedRoleHandler>();`
The problem comes in where I'm using Policies with Requirements.  The AuthorizationRoleHandler is not picking up the user from the context.  This behavior I notice in Visual Studio Code with ASP.NET Core 6.  In a similar project in Visual Studio 2019, ASP.NET Core 5 the basically same code works as expected.  I'm also noticing this behavior when I deploy the application in a Docker container with Kestrel.
public class AuthenticatedRoleHandler:AuthorizationHandler<AuthenticatedRequirement>
{
     protected override Task HandleRequirementAsync(AuthorizationContext context, AuthenticatedRequirement requirement)
    {
             **ClaimsPrincipal user = context.User;//Here it is null**
             // other code
     }
}

Expected Behavior
The role handler should be picking up the AuthorizationHandlerContext user, but it is null.
public class AuthenticatedRoleHandler:AuthorizationHandler<AuthenticatedRequirement>
{
     protected override Task HandleRequirementAsync(AuthorizationContext context, AuthenticatedRequirement requirement)
    {
             **ClaimsPrincipal user = context.User;//Here it is null**
             // other code
     }
}

Steps To Reproduce
No response
Exceptions (if any)
I get a 401 status code Unauthorized on an annotated controller method.
.NET Version
No response
Anything else?
6.0.13 runtime
6.0.402 sdk
**EDIT by @Rick-Anderson

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

ID: f22dcddb-0c25-be1a-9a02-9e71dce86bdb
Version Independent ID: 3e0e12c2-e544-5655-e001-d2ec180b11a8
Content: Kestrel web server implementation in ASP.NET Core
Content Source: aspnetcore/fundamentals/servers/kestrel.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@Ninja4Code feel free to suggest a doc change.

@Ninja4Code can you suggest a change to Kestrel web server implementation in ASP.NET Core?

Sure, I'm putting something together right now.  It will probably be more than you need but you can edit it if you like.  Give me about an hour or so.

Sure, I'm putting something together right now. It will probably be more than you need but you can edit it if you like. Give me about an hour or so.

Awesome, no hurry.

windows-auth-kestrel.docx

This is what I view as a separate issue but is somewhat related to what I posted here.  I am now having an issue with Negotiate working on the http port 80 but not on the https port 443.  Should I create a new issue?

@Ninja4Code Create an issue in https://github.com/dotnet/AspNetDocs/issues/809
Provide a link to a public repo that contains the simplest possible sample that demonstrates the problem.
cc @Tratcher  in case he has other ideas.

That link is for documentation though?

But I think I discovered something in the logging.  It's creating an HTTP/2 request when I use https and I read that Kerberos does not support that.  If I launch an http request on 80 that's an HTTP/1.1 and that goes through.

You're correct, Kerberos doesn't work over HTTP/2. You should disable HTTP/2 in Kestrel.
https://learn.microsoft.com/en-us/aspnet/core/fundamentals/servers/kestrel/endpoints?view=aspnetcore-7.0#listenoptionsprotocols

@Tratcher is this in our docs?

@Tratcher is this in our docs?

No it doesn't. #28408 I'll have @Tratcher  review my update

The one difference I would make in the documentation is that for the setspn statement that it needs to be the user account either running the application, app pool or container.  I think it mentions in the ASP.NET Core documentation that Kerberos is a user-based authentication, rather than kernel-based.

I was able to completely resolve the Kerberos issue in the Docker container with a registered SPN to the docker container host  machine with a GMSA account.  I can confirm this.


Slideshow Function

It should be possible to play the pictures in a folder as a slideshow. Maybe in the view Menu.

An selection of Javascript Libs for slideshow:

Facybox https://fancyapps.com/fancybox/3/
Lightbox by Lokesh Dhakar http://lokeshdhakar.com/projects/lightbox2/images/thumb-2.jpg

By adding the config parameter in de programm settings (https://wiki.openmediadesk.net/de/Konfigurationsparameter#Konfigurationsparameter) the slideshow function can now be tested


Not working

[2020-04-13 14:29:21][INFO] -> VERBOSE: ON [2020-04-13 14:29:21][INFO] -> CFCurl 1. handle checked and validated as protected. [2020-04-13 14:29:21][INFO] -> CFCurl 2. got cloudflare handle. [2020-04-13 14:29:21][INFO] -> CFCurl 3.1. (try: 0 first_attempt: 1) uam page requested with new cURL handle. request headers: GET /?pg=1 HTTP/2 Host: www..com User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36 Accept: / response headers: HTTP/2 503 date: Mon, 13 Apr 2020 12:29:21 GMT content-type: text/html; charset=UTF-8 x-frame-options: SAMEORIGIN set-cookie: __cfduid=d5633ae2ee96f108bd0e1ee30545d4b931586780961; expires=Wed, 13-May-20 12:29:21 GMT; path=/; domain=..com; HttpOnly; SameSite=Lax; Secure cache-control: no-cache expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct" server: cloudflare cf-ray:<PHONE_NUMBER>bbf2d4-WAW [2020-04-13 14:29:21][INFO] -> CFCurl 3.2. (try: 0 first_attempt: 1) _cfduid cookie is: Hostname=#HttpOnly..com; Subdomain=TRUE; Path=/; Secure=TRUE; Expiry=1589372961; Name=__cfduid; Value=d5633ae2ee96f108bd0e1ee30545d4b931586780961; [2020-04-13 14:29:26][INFO] -> CFBypass 1. Waiting for 4 seconds... [2020-04-13 14:29:26][INFO] -> CFBypass 2. Fetching parameters... [2020-04-13 14:29:26][INFO] -> jschl_vc: 45577e3db2b55c884fd3d1a227fa9ad3 [2020-04-13 14:29:26][INFO] -> pass:<PHONE_NUMBER>.488-u3P4v1qeqs Parse error: syntax error, unexpected 'p' (T_STRING), expecting ';' in /usr/home//domains/***/public_html/cron/cloudflare-bypass/CloudflareBypass/CFBypass.php(241) : eval()'d code on line 1

yes i even worry it doesn't work anymore

it doesn't v4 is coming soon :)
Readme has updated and will update again when v4 is ready for beta test. If you want to be added to alpha testing, let me know and I will see if I can add you guys in.

I can be a tester.

@KyranRana can be a tester if you need extra hands

I would like to participate in alpha test aswell

Hi @KyranRana,
If it's still possible I'm interested in participating in the alpha/beta test(s) of the next major version !
I have been using your script since it existed.
Thank you.

as the script doesn't work anymore at all i think everybody will be interested for a new version

If you want to be added to alpha testing, let me know and I will see if I can add you guys in.

Hi @KyranRana, I want to participate in alpha testing, can you add me in?

@KyranRana by using puppeteer i am able to access sites behind cf, however extracting the cookies from it and passing it to the PHP script the request still fails. here what i have so far.
docker run --rm  ${PWD}/test.js:/app/index.js alekzonder/puppeteer:latest

// test.js
const puppeteer = require('puppeteer');

(async () => {

    const browser = await puppeteer.launch({
        args: [
            '--no-sandbox',
            '--disable-setuid-sandbox'
        ]
    });

    try {
        const page = await browser.newPage();

        await page.setUserAgent('Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.122 Safari/537.36');

        await page.goto('https://www.site-behind-cf.com', {waitUntil: 'networkidle2'});

        await page.waitForNavigation({'waitUntil': 'networkidle0'});

        const cookies = await page.cookies();

        console.log(JSON.stringify(cookies));
    } catch (e) {
        console.log('Error exiting');
        console.error(e);
        browser.close();
    } finally {
        browser.close();
    }

})();

any guidance would be appreciated.

@ArabCoders How do you do that with puppeteer

@ArabCoders How do you do that with puppeteer

by visiting the site that is gated by cf then extracting the token and passing that to the php app to use.


Implement gRPC benchmark test

What this PR does / why we need it:
Implement gRPC benchmark test for checking overhead of gRPC, especially marshaling and unmarshaling.
But pprof which is made by this benchmark does not show node about marshaling

[X] client ↔ sever test
[X] client ↔ server ↔ client (use go routines) test
[X] admin client ↔ server test
[ ] the test that attach large document (about 10Mib) to multiple clients

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation:

Checklist:

[ ] Added relevant tests or not required
[ ] Didn't break anything

@hackerwins I find logging.SetLogLevel( ) in logging.go which change log level. thanks for comments 👍

In test:attach large document,
When the size of the document is larger than a certain amount, the error will occur, so I tested it with a size that does not cause any errors.
I attach a picture of the error that occurred.

@DongjinS This error seems to be related to MaxRequestBytes.
https://github.com/yorkie-team/yorkie/blob/147cd1fcf2d320d01bb7ab95eb3d2643f3c4875d/server/rpc/server.go#L75

@hackerwins
Even after changing to MaxRequestBytes 10Mib, the "ResourceExhausted" error still occurs depending on the "builder" size of the "attach large document test".
I am attaching pictures below.

Error condition: MaxRequestBytes = 10Mib, builder size = 10485760

Error condition:  MaxRequestBytes = 10Mib, builder size = 10000000