Base VHDS implementation

For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

@htuch, @brian-avery: This is the base (no on-demand updates) implementation of VHDS. Please note that RouteConfiguration.virtual_hosts and vhds can be used together now.

rebased.

LGTM

rebased
fixed formatting issues
added VHDS config validation tests

Merged details of RDS and VHDS updates and extracted resulting functionality into a dedicated class.
RouteConfigProviderManagerImpl, RdsRouteConfigProviderImpl, and RdsRouteConfigSubscription can be decoupled now (probably a job for a different PR though)
merged upstream changes

Docs/comments are still outstanding, but if these changes are ok, I'll get on that right away.

Oh, and I wonder if RouteConfigProvider::ConfigInfo should be moved out out of RouteConfigProvider. I can argue both ways...

@htuch: resolved a bunch of threads.

@htuch: added missing docs, merged latest changes.

/retest

@htuch: looks like coverage test passes (it core dumped on a previous run). release and ipv6 test are failing, and I think this isn't a flake, ads_integration_test seems to be passing everywhere else. For some reason rate limit test time outs when running on ipv6. Is there any way to run just this test suite on the CI with trace-level logging (tests pass when I run them on a local vm)?
[ RUN      ] IpVersionsClientType/AdsIntegrationTest.BasicWithRateLimiting/0
[2019-05-09 22:56:53.919][21433][critical][assert] [test/integration/http_integration.cc:313] assert failure: result. Details: Timed out waiting for new connection.
[2019-05-09 22:56:53.919][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:81] Caught Aborted, suspect faulting address 0x53b9
[2019-05-09 22:56:53.919][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:69] Backtrace (use tools/stack_decode.py to get line numbers):
[2019-05-09 22:56:53.926][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #0: Envoy::SignalAction::sigHandler() [0x242a8ec]
[2019-05-09 22:56:53.926][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #1: __restore_rt [0x7efc80e5b390]
[2019-05-09 22:56:53.933][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #2: Envoy::HttpIntegrationTest::waitForNextUpstreamRequest() [0xef49a7]
[2019-05-09 22:56:53.939][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #3: Envoy::HttpIntegrationTest::sendRequestAndWaitForResponse() [0xef485b]
[2019-05-09 22:56:53.946][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #4: Envoy::HttpIntegrationTest::makeHeaderOnlyRequest() [0xef677c]
[2019-05-09 22:56:53.952][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #5: Envoy::HttpIntegrationTest::testRouterHeaderOnlyRequestAndResponse() [0xef6987]
[2019-05-09 22:56:53.958][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #6: Envoy::(anonymous namespace)::AdsIntegrationTest::makeSingleRequest() [0xe932dd]
[2019-05-09 22:56:53.965][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #7: Envoy::(anonymous namespace)::AdsIntegrationTest::testBasicFlow() [0xe92150]
[2019-05-09 22:56:53.971][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #8: Envoy::(anonymous namespace)::AdsIntegrationTest_BasicWithRateLimiting_Test::TestBody() [0xe93d20]
[2019-05-09 22:56:53.978][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #9: testing::internal::HandleSehExceptionsInMethodIfSupported<>() [0x280f08e]
[2019-05-09 22:56:53.984][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #10: testing::internal::HandleExceptionsInMethodIfSupported<>() [0x280038e]
[2019-05-09 22:56:53.990][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #11: testing::Test::Run() [0x27eaf33]
[2019-05-09 22:56:53.997][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #12: testing::TestInfo::Run() [0x27ebb38]
[2019-05-09 22:56:54.003][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #13: testing::TestSuite::Run() [0x27ec213]
[2019-05-09 22:56:54.009][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #14: testing::internal::UnitTestImpl::RunAllTests() [0x27f965a]
[2019-05-09 22:56:54.016][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #15: testing::internal::HandleSehExceptionsInMethodIfSupported<>() [0x2811f1e]
[2019-05-09 22:56:54.022][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #16: testing::internal::HandleExceptionsInMethodIfSupported<>() [0x28027a4]
[2019-05-09 22:56:54.028][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #17: testing::UnitTest::Run() [0x27f9333]
[2019-05-09 22:56:54.034][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #18: RUN_ALL_TESTS() [0x1c8dab1]
[2019-05-09 22:56:54.041][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #19: Envoy::TestRunner::RunTests() [0x1c8d842]
[2019-05-09 22:56:54.047][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:73] #20: main [0x1c8c694]
[2019-05-09 22:56:54.047][21433][critical][backtrace] [bazel-out/k8-fastbuild/bin/source/server/_virtual_includes/backtrace_lib/server/backtrace.h:75] #21: [0x7efc801f3830]
external/bazel_tools/tools/test/test-setup.sh: line 310: 21433 Aborted                 (core dumped) "${TEST_PATH}" "$@" 2>&1

@dmitri-d coverage looks good now.
In terms of traces and running tests, you can hack https://github.com/envoyproxy/envoy/blob/master/ci/do_ci.sh to run whatever Bazel commands you want, including limiting the test filter and increasing trace level, for the CI targets you care about, on your branch and CI will run it.

/retest

rebased
squashed
signed-off
upstream/downstream connections get cleaned up in ads_integration_test teardown: https://github.com/envoyproxy/envoy/pull/6438/files#diff-158efafdef83dc24db33be613e278d33R69. Without it I was still seeing timeouts (weirdly, only when all tests were being run. When running just integration tests, ads_integration_test was passing without any changes).

@htuch: ci appears to be happy now.

Thanks for reviews, @htuch!


Memory leaks (potentially a Swift bug)

Describe the bug
Please read this thread.
To Reproduce
Everything mentioned in the linked thread.
Expected behavior
This should not leak.
Environment:

Device: iPhone 6 Plus
OS: iOS 11.3
Library version: 5.1.2
Swift version: Apple Swift version 4.1.2 (swiftlang-902.0.54 clang-9<IP_ADDRESS>)

I only used CentralManager but I've seen that the new PeripheralManager has similar API which would potentially leak as well.
Workaround:
Instead of nesting closures like:
.subscribe(onNext: { handler($0) })

We can pass the escaping handler closure directly to subscribe since the function types should already match up.
.subscribe(onNext: handler)

Hey, could you tell what is leaking? CentralManagerRestoredState? CentralManager?

Is it possible for you to make a screenshot of memory graph debugger?

Not at this moment. I'll setup a small project in the next few days where it's reproducible and do some screenshots of the debugger and save the memory graph. (To me it looks like a bug in Swift, but I also let you guys know.)

Great, looking forward for this 👍

Small update, found the same issue in our code base which is unrelated to RxBluetoothKit. Seems like it has an issue with nested functions.

@DevAndArtist do you have any update on this?

Thanks for discussion!
I will close this issue since there is no update and it looks like not related directly with RxBluetoothKit library.


Say README suggests to test -1 but the unit tests expect uint64

Further, the README says code should report any values that are out of range. But Say takes a uint64. So negatives are not possible. If the input is larger than max unsigned 64-bit the value will wrap before the function ever sees it. Input validation is a total red herring.

Thanks @shaleh

I believe since the instructions say any number from 0 to 999,999,999,999 and an uint64's max value is 18,446,744,073,709,551,615 that the request to report values out range refers to number from 1,000,000,000,000 to 18,446,744,073,709,551,615. A negative number would however cause an error in go. In rewriting this README would it be best to clarify the out of range testing or to just remove it along with the suggestion to test with -1?

Actually,since this is one of the earlier problems perhaps switch back to a standard signed int would be the right thing to do? This way the user gets to implement input validation. The current tests validate that ANY number that a uint64 can hold will work. Which directly contradicts the README since -1 can't be an input and numbers bigger than 1,000,000,000,000 are explicitly checked and expected to work.
Otherwise, document that no validation needs to be done and expect anything up to math.MaxUint64.

perhaps switch back to a standard signed int would be the right thing to do?

I agree with @shaleh here, I think say should use an int64. This would bring it in line with the canonical-data.json. I see no reason why this track should diverge from this. We should then update the test cases, and eventually implement a test generator, to reflect this.

Closed via 3e89b2c


Implementing 2 features, have 2 questions.

I'm trying to implement some features.

add 'total copied in gigabytes' section to 'copy log' file
add account changer and set 'MAX_RUN_TIME', 'MAX_RUN_TIME_PER_DAY' automatically

to do that, i have some questions.

Why did you revert commit for the "quota manger" ??

Where is code for creating google spreadsheet 'copy log~~' ??
-> i found code for writing copied status to log file
-> but couldn't find code for headers(= spreadsheet row with "progress, completed at, link to new folder, total files copied, donate bitcoin, app didn't work, please rate & review"

not sure, I wish I would have left a note in the PR. Most likely it broke something that I could immediately fix.
The sheet is actually just a regular Google sheet that I've made public in my account and the app copies the sheet when it initializes. Here's where the sheet ID is declared and here's where the file is copied


train.py error

I try to download the TUM VI dataset but i can't find the "mav0/mocap0/grad_velo.txt" in there (your code load TUM VI has read this file) so basically i can't train with this dataset. Could you help me with that

Since TUM don't have velocity, we calculate velocity based on first-order differentiation.
In /utils/deferentiate_vel.py

I noticed in TUMVI file config, the name dataset that you used in training is so small compare to total TUMVI dataset. Does it affect the result ?

If I remember correctly, only parts of the TUMVI dataset contain the ground truth within Vicon.
@minhlenhat21


Adds support for KHR material transparency

The transparency on KHR materials did not seem to be working.
After a little digging I found this part of the code to be the reason for that - please check if that's the appropriate fix for the issue.

Thanks again for the pull request and bringing this to my attention.
I looked in to this and eventually ended up needing to fix a number of other similar things at the same time in 9f9e6a5bdb081b6ffe2e4224cbc793a670424cc5. So I'm closing this PR as fixed by the commit.

Also please read commit message and see fix @ d26f16273ac020f82e15642c6cb56d70ae84695a.


Default behaviour of trailingSlash violated

Describe the bug
SvelteKit will remove trailing slashes from URLs by default according to the docs. This functionality is not fully provided anymore since https://github.com/sveltejs/kit/pull/10763 got merged.
Using the default value for paths.base the trailing slash will be added to any sub route like /home/foo/. However, this trailing slash will be removed after reloading the page. Moreover, in case of the default value for paths.base there is no trailing slash at root.
I would expect that the default behaviour of trailingSlash page option is still valid for default paths.base even after merging https://github.com/sveltejs/kit/pull/10763.
Reproduction
A minimal repro example of the issue can be found in this repo.
Logs
No response
System Info
System:
    OS: Windows 11 10.0.22621
    CPU: (16) x64 11th Gen Intel(R) Core(TM) i7-11850H @ 2.50GHz
    Memory: 16.05 GB / 31.73 GB
  Binaries:
    Node: 20.9.0 - ~\AppData\Local\Volta\tools\image\node\20.9.0\node.EXE
    Yarn: 1.22.18 - ~\AppData\Local\Volta\tools\image\yarn\1.22.18\bin\yarn.CMD
    npm: 10.1.0 - ~\AppData\Local\Volta\tools\image\npm\10.1.0\bin\npm.CMD
    pnpm: 8.4.0 - ~\AppData\Local\Volta\bin\pnpm.EXE
  Browsers:
    Edge: Chromium (120.0.2210.61)
    Internet Explorer: 11.0.22621.1
  npmPackages:
    @sveltejs/adapter-node: ^1.3.1 => 1.3.1
    @sveltejs/kit: ^1.30.3 => 1.30.3
    svelte: ^4.2.2 => 4.2.8
    vite: ^4.5.0 => 4.5.1

Severity
serious, but I can work around it
Additional Information
No response

same $page.url.pathname

Ran into this in production! Was wondering why I was suddenly getting reports of 404s.
Digging into this, I noticed several more things:

If the page you initially visit is root ("/"), all pages are treated as trailingSlash = "always" until you refresh the page
If the first page you visit is a subdirectory ("/"), trailingSlash is properly respected until you refresh the page
Using the browser back button at any point seems to cause trailingSlash become properly respected
Explicitly defining trailingSlash = "never" does not change this behavior
This issue exists in both SvelteKit 1.30.1 and 2.0.0

Downgrading to 1.30.0 or earlier is my current solution

For people still bumping on this, the latest 1.x version at the time of writing (1.30.4) still has the code that causes the trailing slash to always be added on navigations from the root page.
This was fixed in #11388 - but it seems that fix is only available in 2.x.
So your options on ^1.30.1 are either downgrading to 1.30.0, as mentioned above here, or upgrading to 2.x.

For people still bumping on this, the latest 1.x version at the time of writing (1.30.4) still has the code that causes the trailing slash to always be added on navigations from the root page.
This was fixed in #11388 - but it seems that fix is only available in 2.x.
So your options on ^1.30.1 are either downgrading to 1.30.0, as mentioned above here, or upgrading to 2.x.

I'm on kit v2.0.0 and have this bug. This:
on:click={() => goto('/enterprise')}
gives this in the address bar of the browser:
http://localhost:5173/enterprise/


How to enable the light mode in Ghost 3?

What do you mean exactly?
Maybe this helps: https://github.com/eddiesigner/liebling/issues/76#issuecomment-552467076


.coordDict contains coordinates for node ID's not included in the graph

I just ran into a situation where I made a graph with only a subset of the available nodes, with the idea that the rest of the nodes (they happen to be completely disconnected) would render below the post-layout graph. When running Dagre.runLayout [] graph |> .coordDict |> Dict.get someId, sometimes I get coordinates even if the someId was filtered out of the graph value (both from nodes and links). I assume these coordinates are useful as intermediate values, but I find they are confusing to be included in the final result. What do you think @goyalarchit?
@goyalarchit I'm continuing to have a great time using the library, thanks so much again for writing and maintaining.

@peterszerzo You're welcome! I'm happy to help in any way I can. I have one suggestion which may save you some time and effort: You may want to keep the dummy vertices as those vertices are the least number of vertices required to draw the drawing while also trying to minimize the number of edge crossings as computed by the Sugiyama algorithm. Since you are doing your own rendering, you could use these dummy vertices directly while building the bezier curves or can also generate your own control points by tweaking the coordinates of the dummy vertices.


Multi-GPU in-RAM

Reran notebooks to consider also in-ram data to assess (and report)
performance of pure model and identify bottleneck from IO

Also TF model now supports NCHW and test with fused=True for slim.batchnorm

Tensorflow is faster than Keras(TF) - as expected. But something is up with the data-loader -> huge bottlenecking for all but Keras.


kotlin value class "get property" is always showing as not called.

Describe the bug
value classes "getters" are never shown as covered when they are used in test cases
this happens in both jacoco and intellij  engines.
Errors
coverage shows 0% coverage
Expected behavior
coverage showing 100% when used.
Reproducer
create a value class:
@JvmInline
value class MyValueClass(val data: String)
in junit 5 test class:
@Test
fun myTest() {
val v = MyValueClass("ABC")
if(v.data != "ABC")
error("not abc")
}
Reports
If applicable, report files or screenshots.
Environment

Kover Gradle Plugin version: 0.6.2
Gradle version: 7.6
Kotlin project type: jvm

@tafypz Please try a fix with 1.0.707 engine version:
for Kotlin Gradle script
kover {
    engine.set(kotlinx.kover.api.IntellijEngine("1.0.707"))
}

for Groovy
kover {
    engine = new kotlinx.kover.api.IntellijEngine("1.0.707")
}

@zuevmaxi,m It did work with that version (well technically I see that the value class is omitted from the report)
Is that what is to be expected ?

@tafypz , yes. Unfortunately, we cannot say for sure whether getter and constructor have been called or not, so we exclude this lines. If such a class has some more methods, they will not be omitted

@zuevmaxim it looks like if the value class is formatted that way, it gets included in the report:
@JvmInline
 value class EntityFindByIdCommandParameter(
     val entityId: EntityId
 )

however formatted that way, it is omitted:
@JvmInline
value class EntityFindByIdCommandParameter(val entityId: EntityId)

Thanks, looks like we should take care of the constructor too

@tafypz Please try a fix with 1.0.708 engine version
Now only getter is ignored, but all the rest lines are covered as expected

@zuevmaxim 1.0.708 works as I expected.
Thanks !


Removing all uses of self.fail().

Replacing instead with assertRaises statements.

This is towards #212. Amazingly enough there are still lines in tests that never get executed / have branch misses:
tests/contrib/test_multistore_file.py : 108->110, 113->-112, 158->-138
tests/test_client.py                  : 311->-311, 329->-329
tests/test_file.py                    : 83->-82, 180->-180, 241->-226
tests/test_service_account.py         : 102->106

Though the test_service_account.py one is a bug in coverage.

Looks great; thank you for the clean-up and merge when you please.


Ubuntu 16.04.07 error trying to overwrite '/etc/dhcp/dhclient-exit-hooks.d/google_set_hostname

Hey,
I was trying to upgrade ubuntu but getting this while running command ( sudo apt-get update && sudo apt-get upgrade -f)
Do you want to continue? [Y/n] Y
Get:1 http://europe-west4-b.gce.clouds.archive.ubuntu.com/ubuntu xenial-updates/universe amd64 google-compute-engine all 20201222.00-0ubuntu2~16.04.0 [12.9 kB]
Fetched 12.9 kB in 0s (44.1 kB/s)
(Reading database ... 175234 files and directories currently installed.)
Preparing to unpack .../google-compute-engine_20201222.00-0ubuntu2~16.04.0_all.deb ...
Unpacking google-compute-engine (20201222.00-0ubuntu2~16.04.0) ...
dpkg: error processing archive /var/cache/apt/archives/google-compute-engine_20201222.00-0ubuntu2~16.04.0_all.deb (--unpack):
trying to overwrite '/etc/dhcp/dhclient-exit-hooks.d/google_set_hostname', which is also in package gce-compute-image-packages 20201222.00-0ubuntu2~16.04.0
Errors were encountered while processing:
/var/cache/apt/archives/google-compute-engine_20201222.00-0ubuntu2~16.04.0_all.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)

same as Ubuntu 18.04


Expanding Optimization Options

Custom optimizer training is now supported with a range of optimizers. Explore the details and utilize the new features through our updated script available at scripts/run_finetune_with_custom_optim.sh.

OK, Now I have updated all the optimizer test results.


Remove vehicle.memo field

Due to the changes in issue#159 the vehicle.memo field is redundant to vehicle.managementDetails field. Remove this memo field.
Dependancy:
Issues #158 and #157 (change in test_data.sql).

Fixed issue by renaming the vehicle.detail field and removing the vehicle.memo field:

Please note:
Old database backups were deleted because the database metadata has changed.


Technical Docs for our Server tests setup with TestNG are outdated

The download link in our docs for TestNG update URL is now out of date and some of the info in this section:  https://docs.openrefine.org/technical-reference/build-test-run#testing-in-eclipse
There's good info to get and reuse some of it from their README.md file in their repo here:
https://github.com/cbeust/testng-eclipse

Hello @thadguidry , can I get to work on this?

@elroykanye feel free to assign issues to yourself directly, you should have the required permissions to do so. Of course it is nice if you don't have to many issues assigned to yourself at once, to leave some for others. Thanks a million for all your contributions so far!

Oh! It's my pleasure. I think I will help others towards solving these issues while checking for others which should be closed or have persisted for too long.

One finding on windows is about the special characters. For the space character or possibly other characters, the TestNG will generate files in another location rather than the intended location.
To avoid this make sure the Eclipse workspace has no special characters including the space character.

Error: Unable to initialize main class org.testng.remote.RemoteTestNG
Caused by: java.lang.NoClassDefFoundError: org/testng/TestNGException
I encounters this error which seems to be related to the classpath settings for TestNG, according to the official doc https://www.testingdocs.com/questions/how-to-fix-the-error-unable-to-initialize-main-class-org-testng-remote-remotetestng/
However, I cannot add the classpath according to the instructions. The "Add Libray..." button is unselectable.

Which seems to be related with build path entry is missing: server/src/main/java

I don't have lots of experience with maven setup / plugins in Eclispe. Would appreciate any help from here. Thanks.

possible TODOs over the next few days:

check out the detailed steps here to see if I can get the TestNG work in Eclipse: https://testng.org/doc/eclipse.html
get and reuse some of it from their README.md file in their repo from the link @thadguidry provided above


Uncaught TypeError: this.action.disposalAction is not a function

[Enter steps to reproduce:]

...
...

Atom: 1.40.0 x64
Electron: 3.1.10
OS: Microsoft Windows 10 Pro
Thrown From: autoclose package 0.0.2
Stack Trace
Uncaught TypeError: this.action.disposalAction is not a function
At C:\Users\Prasanna\.atom\packages\autoclose\lib\autoclose.coffee:39

TypeError: this.action.disposalAction is not a function
    at Object._paneItemChanged (/packages/autoclose/lib/autoclose.coffee:39:29)
    at /packages/autoclose/lib/autoclose.coffee:26:8
    at Function.simpleDispatch (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:1209208)
    at Emitter.emit (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:1210649)
    at Workspace.didChangeActivePaneItemOnPaneContainer (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:424404)
    at WorkspaceCenter.paneContainer.onDidChangeActivePaneItem.t (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:14:44396)
    at Function.simpleDispatch (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:1209208)
    at Emitter.emit (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:1210649)
    at PaneContainer.didChangeActiveItemOnPane (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:446737)
    at Pane.setActiveItem (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:455581)
    at Pane.activateNextRecentlyUsedItem (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:456372)
    at HTMLElement.pane:show-next-recently-used-item (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:288983)
    at CommandRegistry.handleCommandEvent (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:349123)
    at KeymapManager.dispatchCommandEvent (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:1249835)
    at KeymapManager.handleKeyboardEvent (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:1245969)
    at WindowEventHandler.handleDocumentKeyEvent (~/AppData/Local/atom/app-1.40.0/resources/app/static/<embedded>:11:284329)

Commands
     -0:38.2.0 intentions:highlight (input.hidden-input)
     -0:38.1.0 pane:show-next-recently-used-item (input.hidden-input)
     -0:37.9.0 pane:move-active-item-to-top-of-stack (input.hidden-input)
     -0:37.6.0 intentions:highlight (input.hidden-input)
     -0:37.5.0 pane:show-next-recently-used-item (input.hidden-input)
     -0:37.4.0 pane:move-active-item-to-top-of-stack (input.hidden-input)
     -0:29.6.0 intentions:highlight (input.hidden-input)
     -0:29.3.0 command-palette:toggle (input.hidden-input)
     -0:25.9.0 core:move-down (input.hidden-input)
     -0:25.2.0 core:confirm (input.hidden-input)
     -0:25.2.0 settings-view:install-packages-and-themes (input.hidden-input)
     -0:16.7.0 pane:show-next-recently-used-item (button.btn.icon.enablement.icon-playback-play)
     -0:16.5.0 pane:move-active-item-to-top-of-stack (input.hidden-input)
     -0:13.2.0 intentions:highlight (input.hidden-input)
  2x -0:13 pane:show-next-recently-used-item (input.hidden-input)
     -0:12.5.0 pane:move-active-item-to-top-of-stack (input.hidden-input)

Non-Core Packages
atom-beautify 0.33.4
atom-file-icons 0.7.1
atom-ternjs 0.20.0
atom-transpose 0.3.5
atom-wrap-in-tag 0.6.0
autoclose 0.0.2
autocomplete-modules 2.3.0
busy-signal 2.0.1
change-case 0.6.5
docblockr 0.13.7
duplicate-line-or-selection 0.9.0
editorconfig 2.5.0
emmet 2.4.3
emmet-jsx-css-modules 1.0.0
highlight-selected 0.16.0
hyperclick 0.1.5
intentions 1.1.5
js-hyperclick 1.19.0
linter 2.3.0
linter-eslint 8.5.5
linter-ui-default 1.7.1
path-copy 0.14.0
pigments 0.40.2
prettier-atom 0.57.2
set-syntax 0.4.0
sync-settings 0.8.6
tab-foldername-index 3.3.1
toggle-quotes 1.1.4
tree-view-copy-relative-path 1.2.0

No Idea. Haven't checked out this repository yet. Waiting for the moderators/contributors to eye on this

I started experiencing this issue after I opened Atom with no Projects or files open, then selected File > Reopen Project > open a project from the project history here. Opening Atom with a Project folder already open does not replicate this issue.
This is using 1.40.1 for Windows 10.

How can this be resolved ?

Try using this fix
https://github.com/awaw00/autoclose/commit/4f3fe85121d6eceff49cff3cd85d19b2aa84439f?diff=split
resolved the issue for me!

So, it looks like this is the same issue as #12, and the fix @RohanAdhia mentions resolves them.

My pull request (https://github.com/awaw00/autoclose/pull/16) from over a year ago fixed this issue, but it has not been merged into master. This error is simply due to the deprecation of dispoalAction. Just open autoclose.coffee and replace @action.disposalAction() with @action.dispose().

Hello there. Check out the error code you got from your IDE..for me it was Atom. It told me this:TypeError:
this.action.disposalAction is not a function
at Object._paneItemChanged (file:///C:/Users/Ivan Lopez/.atom/packages/autoclose/lib/autoclose.coffee:39:29)
Copy the file path and paste it on your file explorer search bar. It should take you to the 'autoclose' file. Open it and make the changes you see above on line 29 and 39. Change from if @action then @action.disposalAction() to if @action then @action.dispose()
I hope this helps!

Copy the file path and paste it on your file explorer search bar. It should take you to the 'autoclose' file. Open it and make the changes listed below on your file's line 29 and 39. Change from:
if @action then @action.disposalAction()
to
if @action then @action.dispose()
I hope this helps!

This fix was mentioned multiple times in this thread already, and is already present as my commit from Oct. 2018 in https://github.com/awaw00/autoclose/commit/4f3fe85121d6eceff49cff3cd85d19b2aa84439f. OP should close this issue, as it has been solved for nearly two years.


Could not download from https://bcartifacts.azureedge.net/..., retrying from https://bcartifacts.blob.core.windows.net/...

Hello, it looks like artifact images are not available today, but I want to make sure nothing is wrong on my end.
My script:
$containerName = 'mycontainer'
$password = 'P@ssw0rd'
$securePassword = ConvertTo-SecureString -String $password -AsPlainText -Force
$credential = New-Object pscredential 'admin', $securePassword
$auth = 'UserPassword'
$artifactUrl = Get-BcArtifactUrl -type 'Sandbox' -country 'us' -select 'Latest'
$licenseFile = 'BC v17 Dev License.flf'
New-BcContainer `
    -accept_eula `
    -containerName $containerName `
    -credential $credential `
    -auth $auth `
    -artifactUrl $artifactUrl `
    -multitenant:$false `
    -licenseFile $licenseFile `
    -updateHosts

The output (this hangs at the end quite a while):
Could not download from https://bcartifacts.azureedge.net/..., retrying from https://bcartifacts.blob.core.windows.net/...
Then errors out:
`Download-File : Exception calling "DownloadFile" with "2" argument(s): "An exception occurred during a WebClient request."
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\1.0.19\Misc\Download-Artifacts.ps1:157 char:21

...             Download-File -sourceUrl $platformUrl -destinationFile $p ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Download-File], MethodInvocationException
FullyQualifiedErrorId : WebException,Download-File`

Could you share what $artifactUrl is?

Hi Freddy, thanks for the reply.  I ran Get-BcArtifactUrl -type 'Sandbox' -country 'us' -select 'Latest' and the output is: https://bcartifacts.azureedge.net/sandbox/17.5.22499.23364/us

I copied your script and ran it (only changing the license file) - works fine here.
could be something with the CDN, not sure.
Do you still get an error?

Note that you are running an old BcContainerHelper - not that I think that is the issue here.

Hi Freddy, it works now.  Sorry for the trouble - must have been a temporary hiccup somewhere

Yeah, storage accounts also had a problem after the big breakdown - maybe a follow up after that.


Craft 4 compatibility

Hi there, SpicyWeb
I was just wondering if you could confirm whether you have any plans to update this for Craft 4?
Thanks in advance, Andrew

Yep, will be starting work on that soon.

In the 2.x branch, I've updated it to the point of it being installable and selectable as a payment gateway. Although the actual Afterpay integration logic is largely untouched and I assume should still work, unfortunately I'm unable to test that at this point.


what is wrong? Who can help with the bug in href ??

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

Thank you for reporting your issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification.
Thank you, and happy coding.


prometheus-aruba-exporter: init at unstable-2023-01-18

Adds a prometheus exporter for Aruba devices. Module will be added in a seperate PR.
Things done

Built on platform(s)

[ ] x86_64-linux
[X] aarch64-linux
[ ] x86_64-darwin
[X] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

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

aarch64-darwin

:white_check_mark: 1 package built:

prometheus-aruba-exporter


hashpartition leak

Before submitting a bug report, please verify that you are using the most current version of Drill.
Describe the bug
hashpartition leak when allocate  memory exception with OutOfMemoryException
To Reproduce
Steps to reproduce the behavior:

prepare data for tpch 1s
20 concurrent for tpch sql8
set direct memory 5g
when it had OutOfMemoryException , stopped all sql.
finding memory leak

Expected behavior
(1)i set {DRILL_MAX_DIRECT_MEMORY:-"5G"}
(2) i run sql8 (sql detail as Additional context) with 20 concurrent
(3) it had OutOfMemoryException when create hashPartion
Error detail, log output or screenshots
Unable to allocate buffer of size 262144 (rounded from 262140) due to memory limit (41943040). Current allocation: 20447232
Drill version
drill 2.0
Additional context
Add any other context about the problem here.
select
o_year,
sum(case when nation = 'CHINA' then volume else 0 end) / sum(volume) as mkt_share
from (
select
extract(year from o_orderdate) as o_year,
l_extendedprice * 1.0 as volume,
n2.n_name as nation
from hive.tpch1s.part, hive.tpch1s.supplier, hive.tpch1s.lineitem, hive.tpch1s.orders, hive.tpch1s.customer, hive.tpch1s.nation n1, hive.tpch1s.nation n2, hive.tpch1s.region
where
p_partkey = l_partkey
and s_suppkey = l_suppkey
and l_orderkey = o_orderkey
and o_custkey = c_custkey
and c_nationkey = n1.n_nationkey
and n1.n_regionkey = r_regionkey
and r_name = 'ASIA'
and s_nationkey = n2.n_nationkey
and o_orderdate between date '1995-01-01'
and date '1996-12-31'
and p_type = 'LARGE BRUSHED BRASS') as all_nations
group by o_year
order by o_year

I fixed the bug see attachment
0001-bug-10227615-hashPartition-leak.patch

@shfshihuafeng  Can you please submit this as a pull request?

ok @cgivre


Add note on potential capa FP detection to documentation

          Actually, before the issue is closed, is there a false positive reporting process for Windows Defender and the other three engines that can be pursued to prevent this causing trouble for new users, or is this expected enough that it can be put perhaps into an FAQ for the program on this github project?

Originally posted by @RionEV in https://github.com/mandiant/capa/issues/2025#issuecomment-1988802299

Wow, great! Thanks for the extensive proposal and consideration!


Permission denied during persistence of crf component

rasa NLU version : 0.8.2
Used backend / pipeline : spacy_sklearn
Operating system (windows, osx, ...): Windows 10
Issue: Several
Am following instructions at http://rasa-nlu.readthedocs.io/en/latest/python.html
The first problem is with the training. It has a permission error on the temp directory. Since that is unlikely, I suspect that it is masking the real problem.
>>> trainer.train(training_data)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "D:\Tools\Anaconda\lib\site-packages\rasa_nlu\model.py", line 157, in train
    updates = component.train(*args)
  File "D:\Tools\Anaconda\lib\site-packages\rasa_nlu\extractors\crf_entity_extractor.py", line 80, in train
    self._train_model(dataset)
  File "D:\Tools\Anaconda\lib\site-packages\rasa_nlu\extractors\crf_entity_extractor.py", line 308, in _train_model
    self.ent_tagger.open(self.crf_file.name)
  File "pycrfsuite/_pycrfsuite.pyx", line 571, in pycrfsuite._pycrfsuite.Tagger.open (pycrfsuite/_pycrfsuite.cpp:7731)
  File "pycrfsuite/_pycrfsuite.pyx", line 717, in pycrfsuite._pycrfsuite.Tagger._check_model (pycrfsuite/_pycrfsuite.cpp:10037)
PermissionError: [Errno 13] Permission denied: 'R:\\LocalTmp\\tmp5rrxv8ce'

What is strange is that at some point it must have worked because I have a model
Which leads me to another problem when trying to load that model.
>>> interpreter = Interpreter.load(metadata, RasaNLUConfig("config.json"))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "D:\Tools\Anaconda\lib\site-packages\rasa_nlu\model.py", line 225, in load
    component = component_builder.load_component(component_name, context, model_config, meta)
  File "D:\Tools\Anaconda\lib\site-packages\rasa_nlu\components.py", line 340, in load_component
    component, cache_key = self.__get_cached_component(component_name, meta)
  File "D:\Tools\Anaconda\lib\site-packages\rasa_nlu\components.py", line 319, in __get_cached_component
    cache_key = component_class.cache_key(metadata)
  File "D:\Tools\Anaconda\lib\site-packages\rasa_nlu\utils\spacy_utils.py", line 64, in cache_key
    return cls.name + "-" + spacy_model_name
TypeError: must be str, not NoneType

Content of configuration file (if used & relevant):
{
  "pipeline": "spacy_sklearn",
  "path" : "./Rasa/models",
  "data" : "./Rasa/train/examples/demo-rasa.json",
  "emulate" : "luis"
}

Hi Vicharian,
I have faced the same issue and had posted an issue under :

It seems writing into temporary files in Windows is an issue and one way might be to implement sklearn crfsuite.
Not sure if the developers will agree.
There has been no comment from them as of now.

Looking at metadata.json in the model directory, I found that the model was an old one - from version 0.7.x. So the problem is loading the model is likely due to that.

for future reverence: had a chat with the creator of python-crfsuite and the best solution is to switch to sklearn-crfsuite (which he also maintains). this will allow us to avoid the temporary persistence of the model and directly use it after we trained it.

This is why I like ubuntu.It works perfectly in linux systems.
However,may I know how much time switching to sklearn might take.I need to implement rasa_nlu for my office work and the system is windows.

I'll take a look at it, as both libraries us the same underlying c library I guess it shouldn't be to big of a deal.

I replaced the implementation on the sklearn-crf branch - @shuvayan do you mind trying that out with your windows installation?

Hi @tmbo ,
I just tested the new version on windows. It seems to be working fine but I am getting only the last entity from the text and not the others.
{'entities': [{'end': 62,    'entity': 'size',    'extractor': 'ner_crf',    'start': 61,    'value': '9'}],  'intent': {'confidence': 0.0, 'name': ''},  'text': 'I want to buy shoes of brand Adidas and color brown and size 9'}
I have changed nothing in my code except replacing the crf_entity_extractor code as modified by you.
Not sure if this issue is because of the change in underlying library.

@shuvayan thanks a lot for trying it out. I'didn't notice that behaviour but I will try to reproduce the issue later today. Appreciate the feedback :+1:

@shuvayan I couldn'r reproduce the issue, it looks fine on the demo dataset (e.g. it finds two entities in i am looking for an italian restaurant in the north).
If you don't mind it would speed up things if you could send me your data + config to<EMAIL_ADDRESS>(will only be used for debugging this issue).

hi @tmbo ,
Sorry for the delay. I am attaching the data,config file and the python code used.
update.zip
`from rasa_nlu.converters import load_data
from rasa_nlu.config import RasaNLUConfig
from rasa_nlu.model import Trainer
from rasa_nlu.model import Metadata, Interpreter
training_data = load_data('data/testData.json')
trainer = Trainer(RasaNLUConfig("config_spacy.json"))
trainer.train(training_data)
model_directory = trainer.persist('./models/')
metadata = Metadata.load(model_directory)
interpreter = Interpreter.load(metadata, RasaNLUConfig("config_spacy.json"))
entities = interpreter.parse(u"I want to buy a brown pants of size 32")
`
Only the file corresponding to the crf_entity_extractor has to be changed,right? I did not change any other file in the source code.

Hi @tmbo ,
I would also like to request you to update the documentation with an example as to how to measure metrics like accuracy and f1-score now that sklearn is being used. It would be really helpful.

Hi @tmbo ,
Would like to know if you could reproduce the issue at your end??Or do I need to provide something more.

Sorry, @shuvayan Was at a conference last week and was involved in a client side project so not to much time there to look at this yet.
I will have a look at it today tough - promised 😉

Thanks a lot @tmbo . I really appreciate it. :)

I still can not reproduce your issue. Running your training data and using the generated model with the server results in the following output for the input I want to buy a brown pants of size 32:
{
  "entities": [
    {
      "end": 21,
      "entity": "color",
      "extractor": "ner_crf",
      "start": 16,
      "value": "brown"
    },
    {
      "end": 27,
      "entity": "product",
      "extractor": "ner_crf",
      "start": 22,
      "value": "pants"
    },
    {
      "end": 38,
      "entity": "size",
      "extractor": "ner_crf",
      "start": 36,
      "value": "32"
    }
  ],
  "intent": {
    "confidence": 0.0,
    "name": ""
  },
  "text": "I want to buy a brown pants of size 32"
}


Master to Live 1 - 4/17/19

Docs Build status updates of commit dd8ce6f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/perception-simulation.md
:warning:Warning
View
Details

mixed-reality-docs/using-the-windows-device-portal.md
:warning:Warning
View
Details

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

mixed-reality-docs/using-the-hololens-emulator.md
:white_check_mark:Succeeded
View

mixed-reality-docs/perception-simulation.md

Line 34: [Warning] Illegal link: toolbar -- missing bookmark. The file using-the-hololens-emulator.md doesn't contain a bookmark named anatomy-of-the-hololens-emulator.

mixed-reality-docs/using-the-windows-device-portal.md

Line 36: [Warning] Illegal link: generate a pairing PIN -- missing bookmark. The file using-visual-studio.md doesn't contain a bookmark named pairing-your-device-hololens.
Line 56: [Warning] Illegal link: toolbar -- missing bookmark. The file using-the-hololens-emulator.md doesn't contain a bookmark named anatomy-of-the-hololens-emulator.

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

Docs Build status updates of commit 9054c1b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/perception-simulation.md
:warning:Warning
View
Details

mixed-reality-docs/using-the-windows-device-portal.md
:warning:Warning
View
Details

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

mixed-reality-docs/realities-show.md
:white_check_mark:Succeeded
View

mixed-reality-docs/sf-academy-events.md
:white_check_mark:Succeeded
View

mixed-reality-docs/using-the-hololens-emulator.md
:white_check_mark:Succeeded
View

mixed-reality-docs/perception-simulation.md

Line 34: [Warning] Illegal link: toolbar -- missing bookmark. The file using-the-hololens-emulator.md doesn't contain a bookmark named anatomy-of-the-hololens-emulator.

mixed-reality-docs/using-the-windows-device-portal.md

Line 36: [Warning] Illegal link: generate a pairing PIN -- missing bookmark. The file using-visual-studio.md doesn't contain a bookmark named pairing-your-device-hololens.
Line 56: [Warning] Illegal link: toolbar -- missing bookmark. The file using-the-hololens-emulator.md doesn't contain a bookmark named anatomy-of-the-hololens-emulator.

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

Docs Build status updates of commit 36b4324:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/using-the-windows-device-portal.md
:warning:Warning
View
Details

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

mixed-reality-docs/perception-simulation.md
:white_check_mark:Succeeded
View

mixed-reality-docs/realities-show.md
:white_check_mark:Succeeded
View

mixed-reality-docs/sf-academy-events.md
:white_check_mark:Succeeded
View

mixed-reality-docs/using-the-hololens-emulator.md
:white_check_mark:Succeeded
View

mixed-reality-docs/using-the-windows-device-portal.md

Line 36: [Warning] Illegal link: generate a pairing PIN -- missing bookmark. The file using-visual-studio.md doesn't contain a bookmark named pairing-your-device-hololens.
Line 56: [Warning] Illegal link: toolbar -- missing bookmark. The file using-the-hololens-emulator.md doesn't contain a bookmark named anatomy-of-the-hololens-emulator.

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

Docs Build status updates of commit 59d44c0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/install-the-tools.md
:white_check_mark:Succeeded
View

mixed-reality-docs/perception-simulation.md
:white_check_mark:Succeeded
View

mixed-reality-docs/realities-show.md
:white_check_mark:Succeeded
View

mixed-reality-docs/sf-academy-events.md
:white_check_mark:Succeeded
View

mixed-reality-docs/using-the-hololens-emulator.md
:white_check_mark:Succeeded
View

mixed-reality-docs/using-the-windows-device-portal.md
:white_check_mark:Succeeded
View

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


Issue about training the supported methods

Hi. I have installed LibFewShot successfully, and got correct output by running test_install.yaml. However, the script gave the following error message when I tried to run other supported methods listed in ./config.

Could you give some suggestions about how to solve such problem?

Hi, we think it may be caused by uncertain behaviors of the OS or file system, we will never create the symlink file after
commit b050c06eb982defe57be7381e2877571ff561987, you can pull the newest codes to avoid this error.
Feel free to reopen this issue if you meet this error again.


Align have_relationship matcher with readme and add support for Symbol or String parameters

What is the current behavior?
The current README file provides an example usage for the matcher of

expect(document['data']).to have_relationships(:posts, :comments)
With the current implementation of have_relationships, this fails as it does not to_s the relationship array values provided.

PR #15 was rejected as it only added Symbol parameters to the relationships matcher without updating all matchers.
This PR includes support for symbol parameters in all matchers
What is the new behavior?
All matchers now accept strings or symbols (and string or symbol keyed hashes where applicable)
Relationships matcher now has a spec to test it.
Checklist
Please make sure the following requirements are complete:

[X] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been reviewed and added / updated if needed (for bug fixes /
features)
[X] All automated checks pass (CI/CD)

Superseded by #18

@sa73917 I went ahead and cherry-picked your work. Unfortunately I had to revert the changes you did on the tests, please review the relevant commit and let me know if you have questions why the rest of the work was not included b6085bc24ee0dd0d0db83387be4002f917c6973b
Thanks again for helping with this.

@stas - just looking at what eventually went into master - the aim of this PR was to allow all the
matchers to accept parameters that are strings or symbols.  (as in it's current state, some do (have_attribute, have_relationship) and some don't (have_type).  In my code I covered all the matchers but my main driver was allowing the have_type matcher to accept symbols.  It looks like after your changes before the merge this logic was removed.  Was this by design or an oversight?

@sa73917 I'm taking a second look and I don't think I follow.
Mainly, it does look like have_attributeand have_relationship are allowing strings/symbols and are not strict, which I agree should not be allowed. But I don't see how this PR solves this.
I'll be happy to merge the fix if you can open another PR that does not allow symbols and strings to match attributes and relationships in the same time though.
Sorry for missing out on the initial issue that was raised. :see_no_evil:

Sounds like your change to the code in my PR was intentional then (which is unfortunate IMO).
Whilst I understand a true JSON document can't have symbols in it, at the end of the day this is a Ruby gem that helps test JSON documents so (at least from my point of view) it should allow ruby developers to use it much like they would expect to interact with the vast majority of ruby gems (and in fact most standard ruby methods and objects)
Clearly personal opinion but I'd much prefer to read:
expect(document[:data]).to have_type(:users)

rather than:
expect(document['data']).to have_type('users')

My expectation when passing symbols into the matcher is (obviously) not that the type attribute will have a symbol as its value (as that would be impossible).  I'm just expecting the matcher will include a "to_s" on their expected type.  Eg.
module JSONAPI
  module RSpec
    module Type
      ::RSpec::Matchers.define :have_type do |expected|
        match do |actual|
          JSONAPI::RSpec.as_indifferent_hash(actual)['type'] == expected.to_s
        end
      end
    end
  end
end

Oh well, if you change your mind and would like a PR that ensures all the matchers will accept symbols, I'm more than happy to do so.  I'll respectfully decline the reverse however, as I'm loath to remove the symbol parameter functionality from the matchers that already accept them - I'd have to change a bunch of my specs into a format I much less prefer to read. :-)

I think we should make it clear that without jsonapi_indifferent_hash the matchers will be dumb and match documents only based on the user implementation. For example, if you are testing hashes with symbols as keys, and trying to match against a string key, tests won't pass.
And only if you set up jsonapi_indifferent_hash  = true the above will ignored, such that the hashes can match either with a string or a symbol.
I think the default behaviour is valid only for folks who really want to encourage a specific way of formatting the tests/jsonapi-documents, but for the most of the users, jsonapi_indifferent_hash = true should be more than enough.
Let me know if that makes sense. And I would definitely not mind updating those two matchers that force the string-based matching.
Thank you for clarifying this @sa73917!

No worries - always hard to understand intent in text isn't it! :-)
If we are to go down the path of the the jsonapi_indifferent_hash configuration flag being the driver for this, then we should update the existing matchers to reflect this change too..  So with it true - it's whatever combination of strings and symbols you feel like (in the source document and matcher parameters)..  with it false, it's strings only everywhere..
Is that what you're thinking?

Is that what you're thinking?

Nope. What I'm trying to say is that... If you're trying to use the matchers on a document like this with strings:
{
  data: {
    attributes: {
      id: "1",
      email<EMAIL_ADDRESS>    }
  }
}

it should fail, and succeed on symbols (since obviously the document uses symbols (likewise with the opposite: string keys, and trying to match with symbol names).
If you have enabled the jsonapi_indifferent_hash it will pass since we don't care what keys are being used.
Basically, I don't want us to alter the document/matcher arguments, unless we are explicitly told jsonapi_indifferent_hash = true.

I think we're in violent agreement on the true setting
with jsonapi_indifferent_hash set to true - it should work with whatever combination of symbols and strings you feel like throwing at it.  symbolized or string source document matching happily with symbolized or string based parameters.
On the false setting though - my thought was assume the source must be a valid JSON document and hence parameters must be strings.  Sounds like you'd like to leave source and parameters alone and just compare directly.  So if it's a symbolized source document, it should only match symbolized parameters.  If it's a string source document it should only match string parameters.
Do I have it now? :)

On the false setting though - my thought was assume the source must be a valid JSON document and hence parameters must be strings. Sounds like you'd like to leave source and parameters alone and just compare directly. So if it's a symbolized source document, it should only match symbolized parameters. If it's a string source document it should only match string parameters.

I'm afraid this can still be confusing since we'd have to convert to strings for have_meta as well (see #17 for example). So I really believe we should not touch/alter the original document or matcher arguments or folks won't like us at all! :see_no_evil:

Rather than go back and forth trying to describe this - here's a branch with my suggested change.  At a high level, it basically means each matcher calls as_indifferent_hash on the user supplied keys and (for the value matchers), on the user supplied values.
(I did think it might be worth having a jsonapi_indifferent_parameters configuration option (and related as_indifferent_param) method so the users can have control of whether source, parameters or both are processed to be indifferent but figured it's a minor change from where this branch is if you think its worth it..

@sa73917 I went ahead and made it clear that we will always convert matcher arguments where JSON:API spec dictates the format.
I'm considering this conversation resolved for now. Thank you for helping with it!


New Stationeers version incopability (sbn/lbn)

sbn and lbn Are not recognized. Also new preprocessing macros does not work. HASH("some-string") should be possible to insert as any number argument

HASH("name") doesn't accept name with space between words

Fixed HASH with space :)


override options poorly documented?

go-ftw and ftw before it both seem to assume the server being tested is at localhost:80.
The python ftw seems to have options to change that, e.g. --destaddr=domain.com --port 443 --protocol https, at least when run with test/test_default.py (I guess that was its equivalent of cloud mode).
So does go-ftw -- see applyInputOverride and config_test.go -- but the feature appears to be poorly documented,
with no example in README.md.

Good find, will add it to the docs.


(#185) Release deployment -- merge into main

#185

Please, check-up the functionality on the server.

It seems to be working fine. Accidentally I found #196. Not sure if it is a real problem or is just a result of a system not working properly for some hours yesterday.

Notifications service is not working.


AVRO-2645: Validate names in schemas in Ruby client

Make sure you have checked all steps below.
Jira

[x] My PR addresses the following Avro Jira issues and references them in the PR title. For example, "AVRO-1234: My Avro PR"

https://issues.apache.org/jira/browse/AVRO-2645
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Test that invalid names fail schema parsing

Commits

[x] My commits all reference Jira issues in their subject lines. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain Javadoc that explain what it does

Thanks for the change!  This regex passed on the mailing list recently and looks pretty rigorous!  https://github.com/heetch/cue-schema/blob/bb31c583dd09b87a4f1a206290bd57be39b6382f/avro/schema/schema.cue#L6
What do you think?  If it looks good to you, I'll start checking the other implementations -- I'm not convinced that they're all on the same page for fully qualified names like name="valid.9badPackage.GoodName"
(Disclaimer: I'm OK with regex, and the Avro spec to a degree, but this is my first exposure to ruby!)

Hey @RyanSkraba! That one looks great - I was trying to figure something else out with lookaheads and lookbehinds and was completely overthinking the problem. :p
https://image.slidesharecdn.com/regexpsecrets-130228042149-phpapp02/95/regexp-secrets-6-638.jpg?cb=1362026041
Updated the PR, let me know what you think!

I'm on board with enforcing the spec for names by default, but this will be a breaking change for anybody that is currently using non-conforming names.
I think we should include an option for people to opt out of the name validation. This is similar to what we did for enforcing the validation of field defaults: https://github.com/apache/avro/pull/497/files#diff-895d8c5301b553629a5f6d95d9bcc0c7R36-R43

Good idea - I'll add this in.

@RyanSkraba @tjwp added the option - I basically just copied word for word what we did with default validation 😄

Thanks! Looks good to me. I'm not seeing an option to review, since this was already approved -- but approved!

@RyanSkraba @tjwp any idea when this will be merged?

@dorner Thanks for the ping. As long as the CI is red, there is no chance to merge this. Could you rebase onto the master branch as well?

huh, it was green last I checked. 😮 sure, I'll rebase.

@Fokko all green now!

Woot woot! 🎉


Inline issue in docs and Need to add links related to admin console

Description:
I tried to test "A read-only mode for API Publisher"
(https://apim.docs.wso2.com/en/4.1.0/design/api-security/authorization/publisher-portal-in-read-only-mode/ ) ,
Related Issues:

Inline issue is found at the beginning
Also, the links for admin console related steps should be added to this.

fixed issue from https://github.com/wso2/docs-apim/pull/5421


Uptodate-ness compared to Glyphs.app 3.x?

Hi! Is there anything different in Glyphs.app 3.x that would warrant an update here?
Edit: or actually, which Glyphs.app version was it derived from?

I updated it from Glyphs 3.0.5-3112.

Hm? According to the blog, 3.0.5 isn't out yet? The data in master is from 14 Sep 2020?

That is the latest cutting edge version. Have a look at the glyphs3 branch.

Yes I saw, thanks :) but what version is the data in the master branch based on?

Not sure. It seems to be Glyphs 2.6.6.

Are there any plans to merge the Glyphs3 branche into the main branch? You could tag the current HEAD at master as (say) "glyphs-266", so it is still easily available as a reference.
I would be interested to have this data available as a Python package on PyPI. Would that be useful for others, too? I'd be willing to contribute such a thing, if we can agree on details.

glyphsLib already comes with a bundled GlyphData.xml resource file which is a mirror of this repository

Ah thanks, I didn't realize thay, I'll check it out. May be good enough for my purposes.


Exposing internals of SqlClient for Unittesting

The PR exposes the internals of SqlClient to a Unit Tests project.
I considered exposing the internals to FunctionalTests, but there were Type ambiguities found.
This would have required a significant refactoring of the client.
As a result, I went ahead with creating a new project.
Next steps: Get this project to run as part of the pipeline.

Checking in, is this still needed?

Maybe not


Change API keys for Google and Facebook social login

Feature
Why is this feature being added?
The team that created the social login shouldn't be in charge of the accounts that maintain the APIs for our social login. We also need to ensure that the API keys are only shared via server-side communication, not hard-coded into the client.
What should your feature do?
Note that keys are communicated via process.env. @hollomancer we need to create API keys on Operation Code's Google and Facebook accounts. Once we have those keys, we'll need to implement them into the back-end so that the front-end can securely grab them.
Example usage: Line 79ish

Instructions for generating keys:

Facebook
Google

Completed by @nellshamrell


[catnip] Implement drop for libOS for correct clean up

Description
Currently our libOSes do not have any code for cleaning up resources on exit, specifically for components that deal with external resources. For example, Catnip should make sure buffers are freed from DPDK and all DPDK data structures are cleaned up on exit.
How to Reproduce
Steps to reproduce the behavior:

Run Catnip
Crash on unit test
Cannot run again because of lingering state

Related Issues

#649
#651

Closing this because I think we've fixed all of these issues.


CircleCI: merge container builds

    Signed-off-by: Alexander Yustus<EMAIL_ADDRESS>
Signed-off-by: Alexander Yustus<EMAIL_ADDRESS>
Description

Motivation and Context

How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

duplicate of https://github.com/networkservicemesh/networkservicemesh/pull/1740

duplicate of #1740


2nd row of tabs sometimes disappears

Deployment type
(select the type below by replacing the space with an x)

[ ] Cloud Foundry Application (cf push)
[ ] Kubernetes, using a helm chart
[ ] Docker, using docker compose
[ ] Docker, single container deploying all components
[ ] Other (please provide below)

Expected behaviour
Actual behaviour
Steps to reproduce the behavior
Log output covering before error and any error statements
Insert log here

Detailed Description

Context

Possible Implementation

I believe this is fixed by removing the duplicate app header


using default kubeadmin (kube:admin) does not appear to work

Trying to test using kubeadmin token.
Creating context ck
Creating ServiceAccount kube:admin
Error from server (NotFound): serviceaccounts "kube:admin" not found
Logging in as ServiceAccount kube:admin
error: The token provided is invalid or expired.

Duplicate of #27


Bumping image version from 1.1.8 to 1.1.9

Signed-off-by: Mohammed Zeeshan Ahmed<EMAIL_ADDRESS>

/lgtm

/lgtm

/approve

/approve

/lgtm

/lgtm


[FEATURE REQUEST] User-level management of index credentials via uv

I read the documentation on providing credentials for private registries. (https://docs.astral.sh/uv/configuration/indexes/#providing-credentials).
This is generally fine to start with. However, I wonder if a further development of this is planned. E.g. similar to how poetry is handling these(https://python-poetry.org/docs/repositories/#private-repository-example):
poetry config http-basic.foo <username> <password>
would there be something like:

uv config uv_index.internal-proxy <username>
And in the next step a temporary text editor opens and the user enters the password  (which is a more secure practice than entering in the cli, due to logging)

Topic is also discussed in https://github.com/astral-sh/uv/issues/8810

I read the documentation on providing credentials for private registries. (https://docs.astral.sh/uv/configuration/indexes/#providing-credentials).
This is generally fine to start with. However, I wonder if a further development of this is planned. E.g. similar to how poetry is handling these(https://python-poetry.org/docs/repositories/#private-repository-example):
poetry config http-basic.foo <username> <password>
would there be something like:

uv config uv_index.internal-proxy <username>
And in the next step a temporary text editor opens and the user enters the password  (which is a more secure practice than entering in the cli, due to logging)

For pair programming or shared sessions, I don't want to enter my password in plain text in editor windows. Why not use the Credential Manager built into the operating system or a third-party keystore? Login information is stored encrypted, the password entry is hidden and the login information can also be backed up and restored (depending on the credential manager). This would be much more secure than storing credentials in plain text in personal toml files.
I would like to see a way to configure private registrations in pyproject.toml without username or password so that they can be shared between different teams. Since now uv only takes:
[[tool.uv.index]]
name = "private-python"
url =<EMAIL_ADDRESS>
and
[[tool.uv.index]]
name = "private-python"
url =<EMAIL_ADDRESS>
to work with keyring.
My preferred solution I described here: https://github.com/astral-sh/uv/issues/8810#issuecomment-2521703207
Same ability would be nice to have for upload repositorys.
This would really enable to use uv in company teams with on prem indexes and repos.


Avoid installing yanked chrono version v0.4.36

The latest version of crates chrono  (just released today) is broken.
Today we set chrono = "0.4" in Cargo.toml, which end up install the late version v0.4.36 (see https://crates.io/crates/chrono/0.4.36), which cause pull request check to fail (see https://github.com/apache/iceberg-rust/actions/runs/8440126307/job/23116321561?pr=306). I can also reproduce from my local with make check.
error[E0432]: unresolved import `chrono::LocalResult`
   --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/chrono-tz-0.8.6/src/timezone_impl.rs:318:21
    |
318 |         use chrono::LocalResult::*;
    |                     ^^^^^^^^^^^ `LocalResult` is a type alias, not a module
...

https://crates.io/crates/chrono/0.4.36 has been marked as yanked 2 hours ago. Verified locally after cargo update. Closing the issue.


[SuSo-I-002] Extract logic from components into services

This resolves https://github.com/julia-jordan-712/sudoku-solver-angular/issues/29


Error when using Motion v2 plugin

Hey!
I've animated a set of six icons, and I'm getting errors with two of them. The thing that they have in common is that they use the Burst effect from Mt. Mograph's Motion v2 plugin. It generates a shape layer with expressions tied to slider controls on that layer. I saw a previous response that said errors can occur with expressions tied to hidden layers, but as far as I can tell, that isn't happening here.
I can animate the burst without the plugin if needed, but I'd love to make it work if possible!
ERROR:
Uncaught TypeError: Cannot read property 'position' of undefined
ERROR:
jQuery.Deferred exception: Cannot read property 'position' of undefined TypeError: Cannot read property 'position' of undefined
at n.fn (eval at initiateExpression (http://URL/scripts.js:10:240), :3:52)
at n.execute [as getValue] (http://URL/scripts.js:9:31476)
at t [as getValue] (http://URL/scripts.js:5:28588)
at IShapeElement.BaseElement.prepareFrame (http://URL/scripts.js:6:29440)
at SVGRenderer.renderFrame (http://URL/scripts.js:6:10286)
at AnimationItem.renderFrame (http://URLscripts.js:9:16345)
at AnimationItem.gotoFrame (http://URL/scripts.js:9:16249)
at AnimationItem.checkLoaded (http://URL/scripts.js:9:15769)
at AnimationItem.t (http://URL/scripts.js:9:15275)
at AnimationItem.waitForFontsLoaded (http://URL/scripts.js:9:15348)

can you share the .ae project so I can take a look?

Here you go!
https://www.dropbox.com/s/v3k8aixuo0j8m8h/2016-09-22_COTM_Digital Portal Icons.aep?dl=0

Hi, there are a couple of things that are not yet supported by the plugin.
The position error is fixed and I will upload it to gihub this weekend.
But unfortunately repeaters and roving across time are not yet supported and it will take me some time to implement them.

Ah, I see! Thanks for checking on this!

On Oct 14, 2016, at 7:36 AM, hernan<EMAIL_ADDRESS>wrote:
Hi, there are a couple of things that are not yet supported by the plugin.
The position error is fixed and I will upload it to gihub this weekend.
But unfortunately repeaters and roving across time are not yet supported and it will take me some time to implement them.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/bodymovin/bodymovin/issues/157#issuecomment-253787636, or mute the thread https://github.com/notifications/unsubscribe-auth/AVw2aLbM8RcEV9YMqTSy76rrKsJEIcm8ks5qz3c1gaJpZM4KWFyO.


How to reference values from multiple prometheus queries

The dynamic text panel is awesome!  The only issue I have using the panel is when I have multiple prometheus queries and I need to reference values in each query (ex. {{A.Value}}, {{B.Value}}) where A and B are separate prometheus queries.   Steps take in grafana:

create two prometheus queries using table format with "instant" value
under the transformation tab, I first merge the queries and validate that both are returned as a single row
under the transformation tab, I use the Organize Fields option to rename Value #A and Value #B and then referenced those values in the Display Content section of the panel.

The end result is either an error or a blank panel
Grafana versions 7.1 - 7.5

Hey, and thanks for using the plugin! 👋
Following your steps, I wasn't able to reproduce the error you're getting. Since you've merged the queries, you should be able to reference them by field name. Something like this?:

You rock!  I tried a bunch of different things which over complicated my solution.  In the end, it turned out to be something as simple as putting the variable in double .  Thanks Marcus!

No worries! Glad you got it working! 💪


Black screen on IOS 8.1.2 Ipad

When the plugin for Cordova Camera Preview is ran on IOS 8.1.2. The app will load up fine, but when the start camera function is called. The screen goes black with the camera view displayed on the screen. Please see the screenshot below.

Any idea why this is? I have used the application demo and still get the same bug.

Thanks. For the iOS 8 Apple has changed some things for presenting transparent modal view controllers. The plugin and preview app has been updated. Please test in your device again.


HDDS-11639. Upgrade ozone-runner to Rocky Linux 9.3

What changes were proposed in this pull request?
Upgrade Rocky Linux from 8.9 to 9.3 in ozone-runner image.
https://issues.apache.org/jira/browse/HDDS-11639
How was this patch tested?
Pushed image to GitHub:
https://github.com/adoroszlai/ozone-docker-runner/pkgs/container/ozone-runner/299778774?tag=45de37522f47eb78f6cd5a4a125bd3f5b8d6f169
tested with Ozone CI:
https://github.com/adoroszlai/ozone/actions/runs/11662926001

Thanks @ChenSammi for the review.


No timeline in the devtools panel (an default project based on create-vue)

Describe the bug
There is no the Timeline tab in the devtools panel, that mentioned on the official site
Expected behavior
The Timeline tab that exists in the devtools panel
How to reproduce
Steps to reproduce the behavior:

Create a new Vue project with create-vue with these parameters:

Install dependencies with - npm install

Launch the project with - npm run dev

No any trace of the Timeline tab:

The config vite.config.js meets with the installation guide
import { fileURLToPath, URL } from 'node:url'

import { defineConfig } from 'vite'
import vue from '@vitejs/plugin-vue'
import vueDevTools from 'vite-plugin-vue-devtools'

// https://vitejs.dev/config/
export default defineConfig({
  plugins: [
    vue(),
    vueDevTools(),
  ],
  resolve: {
    alias: {
      '@': fileURLToPath(new URL('./src', import.meta.url))
    }
  }
})

Thanks for the feedback, In the new version, we merged the Timeline feature into the feature panel (e.g. pinia, vue-router). But we haven't updated the documentation, We'll update it to avoid confusion.

This seems to work for seeing timeline events for the router and store specifically, but where would one go to find timeline events for component events? This is where I found the timeline most useful, in order to see if an emit was actually sent correctly. Now it seems like this functionality has been removed and only router/store events are shown in timeline.


Invalid content /en/programming-languages/plutus/getting-started/

Where does the issue occur?
https://developers.cardano.org/en/programming-languages/plutus/getting-started/
Describe the issue
Two links under the "Getting Started" heading are dead.
Expected changes
Links should work.

Thank you! These links have been fixed.


Add file server

Description
Includes the file-server package in the install and setups up the service if requested.
Part of flowforge/flowforge#2686
Related Issue(s)

Checklist

[x] I have read the contribution guidelines
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

[ ] Upgrade instructions
[ ] Configuration details
[ ] Concepts

[ ] Changes flowforge.yml?

[ ] Issue/PR raised on flowforge/helm to update ConfigMap Template
[ ] Issue/PR raised on flowforge/CloudProject to update values for Staging/Production

Labels

[ ] Backport needed? -> add the backport label
[ ] Includes a DB migration? -> add the area:migration label

@hardillb I realise the merge conflict is due to timing (flowforge becoming flowfuse) but I just want to point out there are other references to "FlowForge" through out the scripts being output to the user:
Win

Linux

I did not block the PR as they can be addressed in a separate issue.


Add a performance fixme to OutputByteStream

CC @ddunbar This is the fixme that you asked about.

Ok, thanks!


Consider extending the Rules Engine code

It would be nice if you could extended the content a little more with a short-circuiting and a priority example.
Just an example that comes to my mind while writing this.
Let's say that a customer has referred another costumer and for that, you want to check if a ReferralRule applies before any other rule, the reason we wanted to be the first rule evaluated, is because in that case we wouldn't want to apply any other rule. Even the birthday rule would need to be omitted due to the 30% discount that ReferralRule  presents.

Sure, there are a couple of ways to approach this.

We want this rule to run first and short-circuit if it applies
This referral rule with its 30% discount is currently the best discount available, but that might not always be true. We want to make sure the customer gets the best discount they're eligible for

I think you're mainly interested in the first case; the second one just involves running all the rules and keeping the best/biggest discount.
I'll add an example showing the #1 approach and how you could implement it. Thanks!

Sure, there are a couple of ways to approach this.

We want this rule to run first and short-circuit if it applies
This referral rule with its 30% discount is currently the best discount available, but that might not always be true. We want to make sure the customer gets the best discount they're eligible for

I think you're mainly interested in the first case; the second one just involves running all the rules and keeping the best/biggest discount.
I'll add an example showing the #1 approach and how you could implement it. Thanks!

You are right, I'm interested in option 1. Thanks for the prompt response!

You are right, I'm interested in option 1. Thanks for the prompt response!

code for this is now here: https://github.com/ardalis/DesignPatternsInCSharp/tree/master/DesignPatternsInCSharp/RulesEngine/DiscountsShortCircuit

code for this is now here: https://github.com/ardalis/DesignPatternsInCSharp/tree/master/DesignPatternsInCSharp/RulesEngine/DiscountsShortCircuit


Quickly make dxf from autocad entities?

At first I saw the front page example and thought, cool, just add acad entities to the dxf doc. Then I realized you add netdxf EntityObjects, not acad entities.
I then thought the clone method would make it easy but realized netdxf has nothing to do with reading dwg items. There are no autocad references, which is half the whole point of the library (you don't need acad to use it).
So now I do want to use it to convert a dwg to dxf. Do I have to make code to fill in netdxf entitiyobjects with acad entity info for each type? Surely someone has already done this at least for testing.
I don't see it though, must I write my own?
thanks

Personally I use the API for cloudconvert.com for dwg to dxf and the do all dxf manipulation with netDXF.
If you want a good free library that does not rely on the web, then you will be hard pressed to find one, the best solution is simply to pay for one of the big ones.


Figure out the options passed through to _draw and _tearDown

Many PIXI subclasses have an unused options property that's used in their _draw and _tearDown functions. Figuring out what's actually called there would be good.

It looks like it's genuinely not called with any at all. Line 925 of board.js references await group.draw(); Similarly line 310 of placeables-layer.js calls return obj.draw();

According to shark;

Forward compatibility maybe, but speaking from pre-staff memories, they were often just a nicety for package developers.
the DataModel API has similar little cubby holes to pass along any extra data one might want.

We should just create an interface these reference if devs want to add to it

I am working on adding these and adding generics for them


.incr() docs incorret or unclear

Just trying to increment a value and the docs example didn't work, things that should be noted:

The property must already exist, so if it doesn't, you should first do a data.get() and then a data.set() if nothing exists
The docs show this example:

let table = 'rain'
let key = `Wai'ale'ale`
let averageInches = 450

// Increment
await data.incr({table, key, averageInches})

This gives an error where it says "prop" is not defined.
I had to do this:
await data.incr({table, key, prop: 'averageInches'})

Is this correct? I see you passing 450 int the example, is that a way to give it an initial value? If so, how do I do it?

docs bug! thx for filing this / def need to pass the prop to increment

Should be resolved with #159 thanks!


feat/new-lessons-on-js-promises

This adds four new lessons to teach js promises

[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] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes https://github.com/freeCodeCamp/freeCodeCamp/issues/36035

Not that there's anything wrong with opening a PR and marking it blocked while you are working on something, but GitHub has draft pull requests now. It's a nifty feature. It definitely cuts down on the notification noise. :)
Thanks for working on the curriculum.

Yea, sorry about that - I knew there was a draft mode PR, but I couldn't find how to create it (I know now) - and it also wouldn't let me remove the request for review - no need send out unnecessary notifications for this until it's closer to ready.

You could close this an open a fresh one in draft mode, if it's going to be a while? You don't have to delete the branch or anything. Just closing this and making a fresh one from the same branch should do the trick?

I think this is ready for some review - I've created tests that should work good enough, but if anyone has a better method - they could be better.

@moT01 I'm happy with all the changes!  I did add one tiny tweak, but I think everything's looking good now.  Could you commit the changes?  Github's being glitchy again.

okay, I think I've resolved everything here.

Everything LGTM now! :+1:

@ojeytonwilliams , @thecodingaviator , @Manish-Giri Everything looks good to me now.  Just need one more approval and this can be merged.
Great job and thanks for being patient through the review process!

@scissorsneedfoodtoo We are still waiting for one more QA review, but if you can go ahead and validate the changes to the meta.json file look good and approve, then we will merge once the final review has taken place.
Thanks.

Thanks for the heads up @RandellDawson. Great set of lessons here @moT01! Learners will get a lot out of them once master goes live.
Went through the lessons and they all seem to work. The only thing I noticed is that I think resolve and reject in "Complete a Promise with Resolve and Reject" should be lowercase. It's a bit of a nitpick, though, and I only noticed it because the other methods then and catch are both lowercase.

@scissorsneedfoodtoo In the context of this the Complete a Promise with Resolve and Reject challenge, it actually might make more sense to name it Resolve or Reject to Complete a Promise.  Note the first letter is upper case in both words "Resolve" and "Reject".  These are statuses and I think it makes sense to make them capitalized.  With the except of the actual Promise method named "resolve", a user can name the parameters of then Promise callback anything they want, because resolve and reject are not reserved words in JavaScript.  The following would be totally valid:
const makeServerRequest = new Promise((Res, Rej) => {
  // responseFromServer is set to false to represent an unsuccessful response from a server
  let responseFromServer = true;

  if(responseFromServer) {
    Res("We got the data");
  } else {
    Rej("Data not received");
  }
});
makeServerRequest.then(result => {
  console.log(result);
});

That is why I think it really did not matter that the titles of the challenges did not use Resolve and Reject.  This is my two cents worth.

Alright, I added all the suggestions.

@scissorsneedfoodtoo Just waiting on you now.

Thanks for your patience everyone.
You make a very good point about the title of that challenge, @RandellDawson, and your suggestions looks good to me. I'll leave it up to @moT01 if he wants to change it to Resolve or Reject to Complete a Promise or leave it as is. But I'll approve this as is for the time being.

Leave it.

Sounds good. Thank you for all of your hard work here, @moT01! :1st_place_medal: Can't wait to see these up on the site.

yay 🎉 - thanks for all the reviews everyone 🎉


Clean up Docs

The docs need to be better laid out. They are mostly some philosophy docs I have written. The new docs model needs to follow an easy quickstart method that gets people up and running fast while giving good philosophy about what this is about.
This makes 3 docs tracks:
Tutorial:
This needs to be a story track, one where the philosophy seeps into the verbiage of a tutorial presented in a story mode. The tutorial then leads into the reference material.
The tutorial starts with a quickstart, then examples around a few patterns, spine, beacon, etc.
Once the tutorial covers this, then the ref docs should be able to take over.
Philosophy
This presents a new overarching philosophy of programming. The origin of the philosophy, the structure of the philosophy and the concepts that create it need to be laid out.
Talk about how the modern world of development differs from where OOP came from. OOP is good, but we need a toolset that goes beyond OOP and creates software that works in the microservice/distributed world.
Reference
Good old reference docs.

I am going to say that this is done, the new approach of starting at the quickstart is in place.
Still needs some structural work though


No support for DG1

Trying to wrap my head around how I would add support for DG1.  Do you have or do you know where a full spec for HL7 can be had for free?  If not I'll have to pony up for the $50 version.
I have a partial spec for DG1, but its from a vendor and as such I'd rather know what the spec says without their translation/spin on it.
Example:
DG1|1|ICD|2700|Amino-acid transport disturbance
DG1|2|ICD|78060|Fever

Seeing this is 2024. Is this the latest version that you have ? New to HL7 and something like this could be very helpful


enhancing CI in .travis.yml

I'd like to continue to enhance the CI process if you find value in it. I noticed there were some tests in src/input/ that weren't getting run on push and thought it wouldn't hurt to run it.
I also created a ci folder in the root where scripts that travis will run can be stored. I've seen this pattern before and it's a good way to keep the travisfile readable and still organize your CI.
Also, curl pipe sh can lead to unexpected and unwanted results when there is a disconnect from the host. It's typical to redirect it into a file and then running the file so the remote server's connection will not be a concern.

Thanks!


Idempotency fixes for etcd certs and resolvconf tasks

ci check this

@bogdando are you free to check this out?


too match big requests to ldap on rolesmapping

Hello
We are have big domain with 1000+ people.
I made some domain groups with 5-20 persons.
I've set "resolve_nested_roles: false"
Now if I try to authenticate opendistro it too long time searching groups in domain ~ 20-30 sec.
if I disable rolesearch - authentication complete in 1s.
I made tcpdump and saw:
opendistro get list of all groups of my username
then it get all users by every group that were searched.
So there some groupes as "alluser", "officeusers" and etc has 1000 users and opendistro downloads  list of them every time.
~20 MB ldap traffic on 1 signing  on

we need ldap group filter.


respect convert_to_snake_case settings

fixes https://github.com/mirumee/ariadne-codegen/issues/275 by passing the convert_to_snake_case to various generators created via the get_package_generator function.
adds a test that attempts to test that all the generator class fields that are determined by data passed in from ClientSettings are respected.
tested both via unit tests and by regenerating code in which i experienced this issue myself

@rafalp would you be able to take a look at this whenever you get a chance? very minor change

Hi @Rosswell, @DamianCzajkowski is the man you want. I am no longer with Mirumee and I have no powers here anymore :D

Hi @Rosswell, could you add a changelog entry?

@DamianCzajkowski done!

I've checked it, and it looks fine. Just some linters failed, so after you resolve them, I will merge it and deploy a new version.

@DamianCzajkowski fixed :)

Appreciate this feature - it solved my problem!
I look forward to it being deployed. In the mean time I successfully got it from git.
I think the default feature is still broken though - Generated code works fine with the snake case. It breaks when trying to get nested properties.
Either way thanks!!


[Docs] Update the documentation to include the new GitHub issue workflow

Describe the enhancement requested
As we are moving towards GitHub issues for Arrow issue reports, the documentation needs to be updated so that the contributors have a place to look if they need the information about the workflow we are/will be having.
The issue will be divided into multiple subtasks (parts of the documentation) so that the review process will be easier:

[ ] https://github.com/apache/arrow README
[ ] https://github.com/apache/arrow/tree/master/r README
[ ] https://arrow.apache.org/community/
[ ] https://arrow.apache.org/docs/dev/developers/guide/step_by_step/finding_issues.html
[ ] https://arrow.apache.org/docs/dev/developers/guide/communication.html
[ ] https://arrow.apache.org/docs/dev/developers/bug_reports.html#bug-reports
[ ] Version specific project docs warnings https://arrow.apache.org/docs/
Tracked here: https://issues.apache.org/jira/browse/ARROW-18363
https://github.com/apache/arrow-site/pull/275
https://github.com/apache/arrow/pull/14687

Component(s)
Documentation

Anyone can now self-assign issues by commenting "take": https://github.com/apache/arrow/pull/14785


Add the fencing for the VR environment

On the katwalk version there was a steam vr fencing that warns the user that he is leaving the VR bounding box.
We should recover this to the git hub project.

Fencing is solved by Paulo Alcobia by placing physical objects limiting subject navigation in VR.
These new models are in the share point shared objects folder for this project.


Add greaterThanOrEqual (>=) resource, similar to an existing lessThan (<) resource

There is src/askvm/resources/math/lessThan.ts, which is very good but there should also be greaterThanOrEqual.
Dear new contributor,
If you have any questions on how to start, please write a comment with a tag: @czerwinskilukasz1 or @mhagmajer, or join our Discord Community and let us know there. We will be very happy to help you start! :)
Cheers,
Łukasz (@czerwinskilukasz1)
AskQL Core Developer

Try catch this issue :)

PR on https://github.com/xFAANG/askql/pull/141

@czerwinskilukasz1 @mhagmajer We can close this issue :)

Thanks @km4!


How use std::vector in C_ADD_VARIABLE_VECTOR

Hello! I`m 2 hours can``t solve this problem.
SS: https://i.imgur.com/baG19Kd.png

there's alot of exmple, next time please use search and activate atleast one of ur brain cells
https://github.com/rollraw/qo0-base/blob/5e95d973f68b52d6e6afb83e8309e2e81c04fe14/base/core/variables.h#L135
duplicate of #79

I  want add vectors in one vector

u cannot even look through other issues...


BTable sorting does not work

Describe the bug.
Btable-component, if sortable is set to true for a field (=column) then it should sort on it.
To Reproduce.

Open sandbox.io:  https://vc96sx-5173.csb.app/
Click on e.g. column "Name", "My age", "ID" or "salary"

Sandbox Code
https://codesandbox.io/p/sandbox/quiet-bash-vc96sx?file=%2Fsrc%2FApp.vue%3A36%2C21
Expected behavior.
It should be sorted on age or name.
Other info
Columns that have sortable === true get a pointer-cursor so that seems to work.
Environment info
"bootstrap": "^5.3.0",
"bootstrap-vue-next": "^0.9.20",

The issue is that you are not passing a v-model:sort-by, you can pass it for now or wait until PR https://github.com/bootstrap-vue-next/bootstrap-vue-next/pull/1262 gets published.

Ah, great. I must have missed that in the docs. I'm sorry. 🙈
I can confirm it works and I'll close the issue. 👍


Correction for raw affiliation University of Grenoble - Institute of Political Studies Grenoble (IEP Grenoble)

Correction needed for raw affiliation University of Grenoble - Institute of Political Studies Grenoble (IEP Grenoble)
raw_affiliation_name: University of Grenoble - Institute of Political Studies Grenoble (IEP Grenoble)
new_rors: 03c7zyj82;02rx3b187
previous_rors: 05tkv2291;02rx3b187
works_examples: W2123131520;W3200927441
contact: ffd78c3dc4cbe6c8ec308ed0ab358bd9:925a7aaa1a4ebb9b77edee6182c7d22fff459b5746 @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-18. The new affiliations should be visible within the next 7 days.


UI enhancement of Login, SignUp Screen

UI Issues:
1.Susi logo and email field spacing gets abrupt on smaller devices like iPhone 5s. - Login Screen
2.Password invalid texts gets overlapped with confirm password field. - Signup Screen.
3.Keyboard return does not hide the keyboard. - Login, SignUp Screen.
4.Tapping on empty space to hide keyboard - Login, SignUp Screen.
Solution:
1.Change constraints of the UI elements in Login, SignUp Controller
2.Add gesture for tap, and dismiss keyboard on tap gesture.
3.Add keyboard return button functionality to dismiss keyboard.
I would like to fix this issue.

@ishaan1995 close this issue as it got fixed by #30


Process team names from kaggle replays

Kaggle replays during production at the top level  of the replay json will have team names formatted like this:
"info": {"EpisodeId": 23575003, "LiveVideoPath": null, "TeamNames": ["Goose luck", "Goosebumps", "HandyRL", "NejuMixWATORI"]}

From: https://github.com/Kaggle/kaggle-environments/pull/134#discussion_r667132439
We will need to process that and add it to the visualizer (at the moment the visualizer just adds blanks)

closed by v1.2.2