Does not Build

We are testing/trying this but it does not work.
gyp info it worked if it ends with ok
gyp verb cli [
gyp verb cli   'C:\\Program Files\\nodejs\\node.exe',
gyp verb cli   'C:\\1_Code\\testbed_Frontend\\ngx-admin\\frontend\\node_modules\\node-gyp\\bin\\node-gyp.js',
gyp verb cli   'rebuild',
gyp verb cli   '--verbose',
gyp verb cli   '--libsass_ext=',
gyp verb cli   '--libsass_cflags=',
gyp verb cli   '--libsass_ldflags=',
gyp verb cli   '--libsass_library='
gyp verb cli ]
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| win32 | x64
gyp verb command rebuild []
gyp verb command clean []
gyp verb clean removing "build" directory
gyp verb command configure []
gyp verb check python checking for Python executable "python2" in the PATH
gyp verb `which` failed Error: not found: python2
gyp verb `which` failed     at getNotFoundError (C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\which\which.js:13:12)
gyp verb `which` failed     at F (C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\which\which.js:68:19)
gyp verb `which` failed     at E (C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\which\which.js:80:29)
gyp verb `which` failed     at C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\which\which.js:89:16
gyp verb `which` failed     at C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\isexe\index.js:42:5
gyp verb `which` failed     at C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\isexe\windows.js:36:5
gyp verb `which` failed     at FSReqCallback.oncomplete (fs.js:183:21)
gyp verb `which` failed  python2 Error: not found: python2
gyp verb `which` failed     at getNotFoundError (C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\which\which.js:13:12)
gyp verb `which` failed     at F (C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\which\which.js:68:19)
gyp verb `which` failed     at E (C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\which\which.js:80:29)
gyp verb `which` failed     at C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\which\which.js:89:16
gyp verb `which` failed     at C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\isexe\index.js:42:5
gyp verb `which` failed     at C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\isexe\windows.js:36:5
gyp verb `which` failed     at FSReqCallback.oncomplete (fs.js:183:21) {
gyp verb `which` failed   code: 'ENOENT'
gyp verb `which` failed }
gyp verb check python checking for Python executable "python" in the PATH
gyp verb `which` succeeded python C:\Python39\python.EXE
gyp ERR! configure error
gyp ERR! stack Error: Command failed: C:\Python39\python.EXE -c import sys; print "%s.%s.%s" % sys.version_info[:3];
gyp ERR! stack   File "<string>", line 1
gyp ERR! stack     import sys; print "%s.%s.%s" % sys.version_info[:3];
gyp ERR! stack                       ^
gyp ERR! stack SyntaxError: invalid syntax
gyp ERR! stack
gyp ERR! stack     at ChildProcess.exithandler (child_process.js:308:12)
gyp ERR! stack     at ChildProcess.emit (events.js:315:20)
gyp ERR! stack     at maybeClose (internal/child_process.js:1048:16)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:288:5)
gyp ERR! System Windows_NT 10.0.18363
gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "C:\\1_Code\\testbed_Frontend\\ngx-admin\\frontend\\node_modules\\node-gyp\\bin\\node-gyp.js" "rebuild" "--verbose" "--libsass_ext=" "--libsass_cflags=" "--libsass_ldflags=" "--libsass_library="
gyp ERR! cwd C:\1_Code\testbed_Frontend\ngx-admin\frontend\node_modules\node-sass
gyp ERR! node -v v14.15.4
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
Build failed with error code: 1
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>- 3 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of typescript@>= 2.3.1 < 3 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>license should be a valid SPDX license expression
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})

Can you also update to .Net 5

Hi, have you had any joy?

Hi,
Little late but:
Download and install python 2.7 and then
npm config set python C:\Python27\python.exe


[Keras tensorflow] float16 doesn't work with conv2d ?

Hello,
I have a rtx card to use RT cores (dedicated for NN, uses half-precision to my understanding) I'd like using float16 so I :
from tensorflow.keras import backend
os.environ['KERAS_FLOATX'] = 'float16'
os.environ['TF_FP16_CONV_USE_FP32_COMPUTE'] = '0'
backend.set_floatx('float16')

but this triggers error :
model = Sequential([
Conv2D(32, kernel_size=(5, 5), input_shape=(32, 32, 3), strides=(1, 1))
])
model.compile(optimizer=Adam(), loss='mse')
Traceback (most recent call last):
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 510, in _apply_op_helper
preferred_dtype=default_dtype)
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\framework\ops.py", line 1107, in internal_convert_to_tensor
ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\framework\ops.py", line 944, in _TensorTensorConversionFunction
(dtype.name, t.dtype.name, str(t)))
ValueError: Tensor conversion requested dtype float16 for Tensor with dtype float32: 'Tensor("conv2d_sample_weights:0", shape=(?,), dtype=float32)'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "float16.py", line 17, in
init()
File "float16.py", line 15, in init
model.compile(optimizer=Adam(), loss='mse')
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\keras\engine\training.py", line 442, in compile
output_loss = weighted_loss(y_true, y_pred, sample_weight, mask)
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\keras\engine\training_utils.py", line 453, in weighted
score_array *= weights
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\ops\math_ops.py", line 847, in binary_op_wrapper
return func(x, y, name=name)
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\ops\math_ops.py", line 1091, in _mul_dispatch
return gen_math_ops.mul(x, y, name=name)
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\ops\gen_math_ops.py", line 4758, in mul
"Mul", x=x, y=y, name=name)
File "C:\Users\xd111\Miniconda3\envs\evo\lib\site-packages\tensorflow\python\framework\op_def_library.py", line 546, in _apply_op_helper
inferred_from[input_arg.type_attr]))
TypeError: Input 'y' of 'Mul' Op has type float32 that does not match type float16 of argument 'x'.
I'm not a professional I'm a newbie to NN so excuse me if my question is not relevant.

Nagging Assignee @fchollet: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @fchollet: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Closing this as it is in "awaiting response" status for more than 7 days. Feel free to add your comments and we will reopen(if required).


Exercise 2: Improve steps for calculating fees and ttl

This exercise could be improved by including the steps for calculating the minimum fee for the transaction instead of a hard-coded number. When sending real transactions, people will want to know how to do this.
Also, the TTL is just set to a hardcoded high value. I think it's important to query the tip and add 1000 or so to the slot number.

We explain how to calculate minimal fees in the tutorials. We also at least mention how --ttl should be picked. So maybe it would be enough to no replicate the explanation in the exercise, but instead just point to the tutorial? @AndrewWestberg


Actual Normal Health Bar Colors and BF-Old trigger.

What feature do you want to get added on the base engine?
An option for normal health bar colors (and maybe the 9 press for bf-old icon trigger as well).
Example: Kade Engine has an option for normal health bar colors, and it has kept the "Press 9 to change the icon to BF-Old."
Or is it possible to do that with a LUA script? (Haven't done FNF'PE LUA before.)

https://cdn.discordapp.com/attachments/922851578996744252/978068986401009714/vanillapatch2.zip try this

Pressing 9 is possible with Lua...
But the health bar could be added...
For now youd need to change the JSONs to get the colors...

I should make a pull request that ties vanilla easter eggs to the "engine watermarks" option. If Psych watermarks are enabled, the easter eggs can't be activated, vice versa. (Shadow removed them to improve lua functionality, and for some other reasons)


Is it possible to make injector healthz port configurable ?

In what area(s)?
/area operator
Ask your question here
Following up from PR https://github.com/dapr/dapr/pull/4120 if the setting hostNetworks is applied then it will use host network namespace which also includes the health check port, 8080 in this case.
I found it's currently hard-coded in this line:
https://github.com/dapr/dapr/blob/468638abdd3476d74628bb920bf7091a83a7fc33/cmd/injector/main.go#L35
Would it be possible to add that port configuration to the command line arguments ?
This way, it will allow to use a separate port for health checks since 8080 is a very popular choice across the industry. As an example, I had to do the very same configuration for AWS ALB Controller, it was using the port 8080 for metrics, which I updated to 8081 so they can both (dapr and alb-controller) coexist in the same cluster under the same EKS limitations.
I'm not too familiar with the rest of the codebase and not sure whether there are side effects or other implications of this change.

@abelperezok yep, you can do it!

Closed with #4155


Problems building webpack examples

First, thanks for these great examples,
So both the webpack examples are using the older opencv-electron so the base "opencv4nodejs".
---webpack-reactjs-typescript
So that's a small update, but when i try and run the webpack-reactjs-typescript example i get this problem
ERROR in<EMAIL_ADDRESS>Module parse failed: Unexpected token (1:17)
You may need an appropriate loader to handle this file type.
| const isFn = (obj: unknown) => typeof obj === 'function';
| const isAsyncFn = (fn: Function) => fn.prototype.constructor.name.endsWith('Async');
|
<EMAIL_ADDRESS>5:36-58
@ ./app/Root.tsx
@ ./app/index.tsx
@ multi (webpack)-dev-server/client?http://localhost:3000 webpack/hot/dev-server react-hot-loader/patch webpack-dev-server/client?http://localhost:3000 webpack/hot/only-dev-server ./app/index.tsx
It seems the problem is has something to do with webpack loading a typescript file from a javascript file or something a long that way.
I'm guessing it a wepack problem.

I second this, in general the examples need to be bumped in dependencies.
I had to update the nodejs4opencv dependency so the auto build works correctly,
Then I had to update webpack so it could deal with all files the right way.
Your problem @brendena is probably solved with
            {
                test: /\.ts$/,
                use: 'ts-loader?configFile=tsconfig.json'
            },

But one thing I couldn't get over, that probably needs @UrielCh s knowledge, is stemming from cvloader's getOpenCV.
Essentially, __dirname defaults to node_modules/electron/resources/electron.asar or something, that isn't even available.
If you then tell webpack to use it the right way, i.e.
node: {
    __dirname: true
}

It tries to get them from<EMAIL_ADDRESS>which is more correct, however webpack only keeps the lib/cjs folder, not the binaries, because they aren't statically required anywhere....
A further complication for me was that I had to use OPENCV_BUILD_ROOT because the opencv test files had some extra long file names, exceeding MAX_PATH at the place it was stored.
Now neither have I gotten this to work with the webpack-dev-server yet (where I can still pass the environment variable) nor do I trust this to work well in a packaged scenario, so every help towards that is appreciated!


Manifest V3 + multiple feature requests

Soooooo what started as a small change to replace permissions: ["http://*/*", "https://*/*"] quickly devolved into an evening-long refactor and enhancement of the extension. 🙃
QA
If you're interested in testing these changes but don't want to pull and build locally, you can use these builds:

chrome-2.0.0.zip: download and extract, then "Load unpacked" in chrome://extensions/
firefox-2.0.0.zip: download the zip file, then "Load temporary add-on" in about:debugging#/runtime/this-firefox

Manifest V3
With the switch to manifest V3, the spec for Chrome and Firefox splits; those differences need to be taken into consideration when building a manifest.json. Therefore, this PR takes the approach of splitting the manifest into 3 files: manifest.COMMON.json, manifest.FIREFOX.json and manifest.CHROME.json. During the build process, the COMMON file and one the browser files are merged using jq, and the output is saved as manifest.json (which is no longer tracked in git).

background is now split as Chrome now uses a service worker to run the background script
Renamed browser_action to action
Shortcut text removed from default_title
browser_style set to false since now deprecated
browser_specific_settings.id required for Firefox (set to random UUIDv4)
New permission required to support new functionality:

storage: localStorage is deprecated in extension contexts
contextMenus + notifications: quick add link via right-click
scripting + activeTab: get highlighted text for notes

And one more thing: the build script now outputs both packaged and unpackaged builds to /artifacts/chrome and /artifacts/firefox.
Major Changes
Granular permissions
By marking ["http://*/*", "https://*/*"] as optional_host_permissions, we can request only the permission(s) we need to connect to the Linkding instance. When first configuring the extension, you'll be prompted to grant permission to the page that hosts Linkding.

Firefox
Chrome

Dark mode
The extension popup and settings will appear in dark mode, based on the appearance/theme settings of your browser/OS, via the prefers-color-scheme media feature. I may get around to adding a small toggle in the settings to allow for manual control. Closes #41

Context menu entry
Right-clicking on a link will give you the option to quick-add it to your bookmarks, without having to visit the page. A notification is displayed if it's been added, or if it was already bookmarked. Closes #35

Context menu
Notification

Starred pinned badge
On bookmarked pages, a yellow star will appear on the badge (bottom-right on Chrome, top-right on Firefox). The star should appear as soon as a page is bookmarked, or when a new tab is opened. Closes #30

Other smaller changes

The state of the popup changes from "Save" to "Update" on first save
Add a trash button to delete a saved bookmark
Add link to main site (closes #29)
Highlighted text is saved to notes when creating/updating a bookmark (closes #20)
Update all dependencies to their latest versions

Required the removal of the Commonjs plugin in favour of ESM

As always, thoughts, questions and concerns are welcome!
 Thanks so much for this PR, it adds so many feature requests I've been following in one fell swoop :rocket:

@bsecker Glad to hear it! If you're interested in testing the changes but don't want to pull and build locally, let me know and I can upload builds for Chrome and Firefox. Any QA would be appreciated. 🙇🏼

@GAntoine I'll happily test it. A build for Firefox would be awesome!

Awesome, and thank you! Here are the builds and instructions, which I've also added to the body of the PR.

chrome-2.0.0.zip: download and extract, then "Load unpacked" in chrome://extensions/
firefox-2.0.0.zip: download the zip file, then "Load temporary add-on" in about:debugging#/runtime/this-firefox

Hey, thanks for the effort here, but this PR contains way too many changes for me to properly review it. Could you please extract individual features into separate PRs? Likewise for refactorings / manifest + build changes / etc.
Some quick notes:

The Chrome extension already uses Manifest V3, there is a chrome branch with the relevant changes to the manifest that I just rebase for a release. Your solution would be more convenient though, so I'd take a look at a PR for that.
There's probably a lot that I'd change about the dark theme to the point that I should probably do it myself rather than go into an exhausting review loop. Let's exclude this for now.

Initial impressions from 10 mins of playing around:

I get a small blue dot that appears on each page, which disappears after I click the extension.

I don't get any context menu when right clicking to save a bookmark
there also doesn't seem to be any links to the linkding instance (also after saving the link):

If it helps here's a dump of my troubleshooting info

Could you please extract individual features into separate PRs?

Yes absolutely! Since some of the work builds onto itself, I'll start by opening a PR just for the manifest changes that covers permissions.

There's probably a lot that I'd change about the dark theme to the point that I should probably do it myself rather than go into an exhausting review loop

I took a "something-is-better-then-nothing" approach to dark mode that could be done quickly. I'd love to see a more robust dark mode at some point, but until then I'll open a split PR for dark mode.

I get a small blue dot that appears on each page, which disappears after I click the extension.

My understanding is that the dot appears for websites where explicit permission isn't granted in the manifest. I see it as well, and I believe is has to do with the "tabs" permission. If you reload a page where the dot was removed after clicking it, it comes back.

"After installing an extension, if you navigate to a website that requires permissions for the extension to work, the extensions button will display a notification dot. If the extension is already pinned in the toolbar, the notification dot will appear below the extension icon."

https://support.mozilla.org/en-US/kb/extensions-button

I don't get any context menu when right clicking to save a bookmark
there also doesn't seem to be any links to the linkding instance

This makes me think it's interacting with the old linkding extension (I assume) you had installed in Firefox. Could you try installing Firefox Developer Edition and testing it there, as it allows for installing unsigned add-ons (instead of having to load it temporarily)?


Garbage collector min_age should start when SSTs become inactive

The garbage collector's min_age setting for SSTs currently just looks at when an object's last-modify-time is more than N seconds old. I think for WAL SSTs this is fine. But for compacted SSTs, which can be long-lived, this results an in odd behavior:
An SST that's been active (in the manifest) for more than 24h will get GC'd almost as soon as it's removed from the manifest. Consider the case where an SST in an old/big sorted run has been around for a three days. The compactor finally reaches the point where it compacts out that sorted run and removes it from the manifest. The GC's next poll interval (defaults to 5 minutes) will see the SSTs in this sorted run are > 24h old and delete it. This is pretty abrupt, though it shouldn't cause any problems.
The question is: do we want min_age clock to start based on when the SSTs are written or when they become inactive.
If we want the min_age clock to start when SSTs become inactive, we could determine their inactivity based on the manifest history we have. What I'm thinking is the GC could do something like this:

List all manifests (sorted from old to new).
Iterate through each manifest, detecting when L0/compacted SSTs become inactive (when they're no longer referenced).
Set the expiry timestamp for each detected SST based on the first manifest timestamp where they're no longer referenced.
Delete all SSTs with an expiry timestamp > 24h or that are unreferenced by any manifest and have a last modify time > 24h.
Delete all manifests that are not part of any snapshot and reference only SSTs that have all been deleted.

We could theoretically apply similar logic to the wal SSTs starting from last_compacted_wal_id's last modify time rather than each WAL SST's last modify time. If we make the above changes, it's probably worth doing this to make things consistent.
This change makes the manifest deletions a little less predictable since we'd have to keep manifests around until all their referenced SSTs have been deleted. I don't see an issue with that, though. Practically speaking, the manifest and compacted SSTs default to the same GC settings, so all of a manifest's SST should be deleted at about the same time the manifest expires anyway.

for each detected SST

I think there's two ways we can detect SSTs with this approach - by actually listing them all like we do today, or by looking at the SSTs reported in the manifests. The latter is cheaper, but won't include SSTs that were left behind by writers/compactors that never got a chance to write them to a manifest. So, I think we should still be listing all the object store SSTs when looking for candidates to delete.
This came up in a conversation w/ @agavra.

Correct. I didn't mention it explicitly, but you would need to do a full list to complete this part:

or that are unreferenced by any manifest and have a last modify time > 24h.

Documenting one option for supporting this that builds on the proposal in https://github.com/slatedb/slatedb/pull/278. The compactor could leave a checkpoint for the manifest before applying it's updates. The checkpoint could have an expiry time that matches the time that we want to retain SSTs for after they have been removed from the db. The GC will delete this checkpoint once it's expired and then no longer include the referenced manifest in the set of valid manifests that it includes when building up the active SST list. This is basically the same as what's proposed in the original issue description, except it restricts the set of manifest versions to search to those immediately preceding a change to the db that removes SSTs.

Ohhhh, this is a clever approach. It's much simpler to implement! I like it.

+1 on the approach shared above. It requires #319 and #311  to be implemented first.


Typescript Types Support (Ready for Review)

[x] Adding AvionDB specific types (user facing functions)
[x] Adding AvionDB specific types (internal functions)
[x] Adding OrbitDB specific types
[x] Adding IPFS specific types

License: MIT
Signed-off-by: Vaibhav Saini<EMAIL_ADDRESS>

mind sharing how to access the ipfs types in projects that have aviondb installed as a dependency?


Updating OAuth Servers via REST API does not reload OAuth Servers

When oauth_servers is updated via REST API, the changes are saved (if using mongo provider), but are not immediately loaded, requiring a server restart, or API Definition change to trigger configuration reload. It should be immediately reloaded when it is updated.
Reproduction Steps:

Create/Update/Delete an oauth server via REST API

Expected behavior:
Changes are immediately effective.
Observed behavior:
Database is updated, but janus does not reload the updated configuration.
Janus version: 3.8.6

confirm


Write the Docs

[ ] mkdocs setup
[ ] guides similar to cookiecutter-pytest-plugin
[ ] Read the Docs

I do believe documentation seems like a splendid idea! :+1:

@d0ugal I agree. Going to add rtfd soon-ish :grin:


add a test for #8

@clarkevans I'd like to include your case in unit tests if you don't mind.
xref: #8

Thanks!  That looks lovely (very odd case)


not all route can be found

Hi, guys I am using version 1.1 of Nancy self hosting, I have created a some module like this
public UserModule():base("/User")

However some route/module does not appear to be functional( return a 404), so I went to the diagnostics page and found out only some route can be found and access, Nancy seems to ignore the other routes. I have confirmed the issue on both windows and mac(mono)

examples of some modules/route being ignored:

:
I was wondering if there is a workaround for this issue.
Thanks in advance.
Peter

Are they public classes?

@khellang Thank you so much!!!!  You just save the day, It's all working now can't believe I missed that.

It's usually just a simple thing. Been there myself :wink:


Add inline tags to commonly used functions

I've gone through and added inline to some commonly used functions.
If everyone is happy, we could also add these to others.
We should judge what should be inlined. See https://nnethercote.github.io/perf-book/inlining.html
None. The compiler will decide itself if the function should be inlined. This will depend on the optimization level, the size of the function, etc. If you are not using link-time optimization, functions will never be inlined across crates.
#[inline]. This suggests that the function should be inlined, including across crate boundaries.
Amazingly I saw great wins with just this, as I have to call some of these functions a lot when encoding a lot of data.
This is using lto:
before
Operating System: macOS
CPU Information: Apple M1
Number of Available Cores: 8
Available memory: 16 GB
Elixir 1.13.4
Erlang 25.0.2

Benchmarking encode column ...

Name                    ips        average  deviation         median         99th %
encode columns         13.86       72.17 ms    ±21.40%       74.47 ms      114.95 ms

after:
Name                    ips        average  deviation         median         99th %
encode columns         15.12       66.13 ms    ±23.48%       68.36 ms      108.11 ms

@joshuataylor thanks for the PR. Can you share the code for your benchmark?

This looks like a very good idea to me.

Interesting. I would not expect inline to make a real difference. Can we explain why we see a difference in the performance?


Optimize buildVRFMessage in Hare oracle

Motivation
Closes #1569
Changes
warning when beacon gets zero contextually valid blocks
buildVRFMessage is now concurrency-optimized (added lock and cache).
Test Plan
Added UT including a concurrency test.

bors merge

@y0sher waiting for your review

bors merge


adding criteria to push to buffer when there is no wave signal (for large number of OMBs on fast ice to cut Iridium cost).

@jerabaul29
Hi, Jean.
We are not sure whether we implement this or not, but as discussed, we would like to have this as an option. The idea to have a push criteria was suggested by Tsubasa. For now, we simply use Hs and likely a value around 1 cm. The sensor spec noise is about 0.4 cm, we achieved 0.5 cm in NABOS and 0.8 cm in JARE63 (the Antarctica 2022 deployment). So I think 1 cm may be a good start.
I think the added lines of the code is straight forward, but if you can check/have suggestions, please let me know. Thanks so much.
https://github.com/tnn77/OpenMetBuoy-v2021a/blob/85db725043142f9071fa8df4f77205153f3e6669/legacy_firmware/firmware/jare65/wave_analyzer.cpp#L265-L270

@jerabaul29
thanks for taking a look!
the sensors are on the ship already, but I am doing work there this week, so I will test this out and let you know.

I was able to compile and test (on the ship, so it was not practical to get a very small signal, so I tested with 0.05 m, it worked).
when I had it set at 0.008 m, I think ship vibration, even docked produced 0.02 m Hsig).
Anyway, I think we are all good, Thank you so much.


Please Help! Running Console Problem

Hey guys! I seem to be having a problem with Pokefast!
Whenever I try to load the console up, when logging in via Google it brings up the "Connect Device" part of google mail which is expected but the problem is that the console just closes, leaves me no code, just closes.
I have looked through the logs but the logs just cut out the code and just states the starting up process.
I've also had a problem trying to get this to work with my Ptc account. When running the program with Ptc it says that my account isn't verified
"[14:44:31] Exception of type 'PokemonGo.RocketAPI.Exceptions.AccountNotVerifiedException' was thrown. from Pokemon Go Rocket API
[14:44:31] (ERR) Got an exception, trying automatic restart.."
If someone could help It would be greatly appreciated!
Thanks!

Are you using two factor authentication for google? if so you should generate an app password for the bot: https://support.google.com/mail/answer/185833?hl=en

@buettner123 Nope, Im not using a 2step verifcation.

Are you using this account for the first time on a new device with a new IP? If so, then you manually need to log in first using the web interface since google will throw an anti-bot protection when you try to log in from unknown sources. (I know this because I had this problem happening with another non-pokemon bot that I switched to another server).

@mrseeker well the device it needs to connect to is the console, Ive logged into the gmail many of times before, Im not following your train of thought

Problem is that the console just closes as the codes about to pop up

Google has various checks in place, one of them is a check to see if the login is valid. Seeing your error message & the fact that you are trying to log in using a google account threw me off, until I did some quick googling of the error.
Are you trying to log in using the pokemon trainer club or using the google service? The error seems to be related to the fact that you are not authenticated using the pokemon trainer club. Can you check in your configuration setting if this is set correctly:
"auth_service": "google",

My pokemon trainer account is having this problem
[15:41:55] (ERR) InvalidResponseException from HttpClientExtensions.cs
[15:41:55] Exception of type 'PokemonGo.RocketAPI.Exceptions.InvalidResponseException' was thrown. from Pokemon Go Rocket API
[15:41:55] (ERR) Got an exception, trying automatic restart..

Copy-paste from here: http://www.ownedcore.com/forums/pokemon-go/pokemon-go-hacks-cheats/564223-common-errors-solutions-various-bots.html

Go to https://www.pokemon.com/us/pokemon-trainer-club/login, login with your PTC account. The site will ask to send a verification email to your associated address, follow it and verify your account and you're good to go.

Ive already verified it and thats whats annoying me haha!

My google account and ptc account are different accounts, having problems with both accounts

Okay, what do you have in your configuration setting under "auth_service"? Does it say "ptc" or "google", and are you using the right account associated with it?

Yeah when im using google its "Google" and when using ptc its "Ptc", I honeslty dont have a clue what it could be,
My google account closes the console when it brings up the window to enter the code
my ptc has that error above and it does log me in but the error shuts it down sooner or later

when it brings up the window to enter the code

What code are you referring to here?

I am refering to the code to connect devices with google.
When you run the console trying to log in with google a window pops up on google which you have to enter a code to connect device to email.

Are you running the bot or the desktop version of the bot? Since I am running this in a console and I don't get any window pop-ups at all.

Have you tried this? https://support.google.com/accounts/answer/63950?hl=en

@mrseeker I want to log into my google account so we'll focus on that one,
I downloaded the win rar file of pokefast and extracted files to a private file for this bot, I then went into console settings and I have changed the lat&long and put my google details in whilst also changing the auth type to Google. Then I double left click on the console itself and when it is logging in via google lots of writing in white comes up at the bottom and then the console which im running closes bringing up a webpage that from Google asking me to enter the code. I have watched a youtube Video on this and it is the same page.
Mine is the same as what happens in this youtube video
https://www.youtube.com/watch?v=wLN0z3ZHBlw
but my console closes instead of staying open and having a code for me to paste into the webpage. Sorry for the hassle, I appreciate your time in helping me.

You are talking about the "pokefast" .rar file. If this is the case, than this is the wrong place to ask for help because this is an other bot

Installation manual: https://github.com/PokemonGoF/PokemonGo-Bot/wiki/Installation

Yeah Ive been using pokefast sorry, Ive downloaded this other one Pokemon go bot master but this looks confusing! where do I even enter my account details and lat&long

downloaded python

just refer to the installation manual @mrseeker posted before. And take a look at the readme for further information about usage

I keep having import error pgoapi what am I doing wrong

come to slack chat #help and someone can help you
NOT A BUG

[19:54:45] PokeFast V1.0 - Obtained for free at OwnedCore.com
[19:54:46] Make sure Lat & Lng is right. Exit Program if not! Lat: 43,586103 Lng: 1,441305
[19:54:46] Script will continue in 15 seconds!
[19:54:51] Script will continue in 10 seconds!
[19:54:56] Script will continue in 5 seconds!
[19:55:01] Logging in via: Ptc
[19:55:03] Client logged in
[19:55:03] ----------------------------
[19:55:03] PTC Account: jaoudath
[19:55:03] Latitude: 43,586103
[19:55:03] Longitude: 1,441305
[19:55:03] ----------------------------
[19:55:03] Your Account:
[19:55:03] Name: Jaoudath
[19:55:03] Team: Red
[19:55:03] Level: 16 (LvLUp in 8hours // 8585/20000 XP)
[19:55:03] Stardust: 61605
[19:55:03] ----------------------------
[19:55:03] ====== DisplayHighestsCP ======
[19:55:03] InvalidResponseException - Restarting
[19:55:03] PokemonGo.RocketAPI.Exceptions.InvalidResponseException: Une exception de type 'PokemonGo.RocketAPI.Exceptions.InvalidResponseException' a été levée.
à PokemonGo.RocketAPI.Extensions.HttpClientExtensions.d__02.MoveNext() dans C:\Users\Dev\Desktop\Speg\PokemonGo.RocketAPI\Extensions\HttpClientExtensions.cs:ligne 22 --- Fin de la trace de la pile à partir de l'emplacement précédent au niveau duquel l'exception a été levée ---    à System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    à System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    à System.Runtime.CompilerServices.TaskAwaiter1.GetResult()
à PokemonGo.RocketAPI.Client.d__42.MoveNext() dans C:\Users\Dev\Desktop\Speg\PokemonGo.RocketAPI\Client.cs:ligne 383
--- Fin de la trace de la pile à partir de l'emplacement précédent au niveau duquel l'exception a été levée ---
à System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
à System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
à System.Runtime.CompilerServices.TaskAwaiter1.GetResult()    à PokemonGo.RocketAPI.Logic.Inventory.<GetPokemons>d__2.MoveNext() dans C:\Users\Dev\Desktop\Speg\PokemonGo.RocketAPI.Logic\Inventory.cs:ligne 28 --- Fin de la trace de la pile à partir de l'emplacement précédent au niveau duquel l'exception a été levée ---    à System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    à System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    à System.Runtime.CompilerServices.TaskAwaiter1.GetResult()
à PokemonGo.RocketAPI.Logic.Inventory.d__5.MoveNext() dans C:\Users\Dev\Desktop\Speg\PokemonGo.RocketAPI.Logic\Inventory.cs:ligne 52
--- Fin de la trace de la pile à partir de l'emplacement précédent au niveau duquel l'exception a été levée ---
à System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
à System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
à System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
à PokemonGo.RocketAPI.Logic.Logic.d__20.MoveNext() dans C:\Users\Dev\Desktop\Speg\PokemonGo.RocketAPI.Logic\Logic.cs:ligne 470
--- Fin de la trace de la pile à partir de l'emplacement précédent au niveau duquel l'exception a été levée ---
à System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
à System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
à System.Runtime.CompilerServices.TaskAwaiter.GetResult()
à PokemonGo.RocketAPI.Logic.Logic.d__8.MoveNext() dans C:\Users\Dev\Desktop\Speg\PokemonGo.RocketAPI.Logic\Logic.cs:ligne 149
--- Fin de la trace de la pile à partir de l'emplacement précédent au niveau duquel l'exception a été levée ---
à System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
à System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
à System.Runtime.CompilerServices.TaskAwaiter.GetResult()
à PokemonGo.RocketAPI.Logic.Logic.d__7.MoveNext() dans C:\Users\Dev\Desktop\Speg\PokemonGo.RocketAPI.Logic\Logic.cs:ligne 71
[19:55:03] PokeFast V1.0 - Obtained for free at OwnedCore.com
[19:55:04] Make sure Lat & Lng is right. Exit Program if not! Lat: 43,586103 Lng: 1,441305
[19:55:04] Script will continue in 15 seconds!
[19:55:09] Script will continue in 10 seconds!
[19:55:18] Script will continue in 5 seconds!
SA TOURNE EN BOUCLE MEME EN LE RETELECHARGENT ! QUELQU'UN PEU M'AIDER ? Merci

regarde comme si vous êtes interdiction de là serveur


logging sample implementation

Logging Sample Implementation
Literal.ai logging sample implementation is completed.
Still need to complete LangSmith and TruLens.

Seems consensus is to use TruLens for both evaluation and logging, so we can leave this as a PR. Converting to draft.


Missing label of backwards association

In "modifications", I have:
#Missing in RDF vocab definitions
m8g:identifies rdfs:label "identifies"@en .

supplementing the definition from the original vocabulary legal:
<https://data.europa.eu/m8g/identifies> a owl:ObjectProperty;
  rdfs:domain <http://www.w3.org/ns/adms#Identifier>;
  rdfs:isDefinedBy <https://data.europa.eu/m8g>;
  rdfs:range <http://www.w3.org/ns/legal#LegalEntity> .

However, in Dataspecer, I still see the association as "no title":

Maybe this is caused by inability to take parts of the definitions from different files?

Hi, the property IRIs have different schemas - one has http and one https.

Good catch, passed upstream.


Add 'list of all styling parts' link

Fixes #109

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

demo/styling.html, line 30 at r1 (raw file):

URL doesn't seem to work in webcomponents.org demo

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion.

demo/styling.html, line 30 at r1 (raw file):
Previously, samiheikki (Sami Suo-Heikki) wrote…
URL doesn't seem to work in webcomponents.org demo

I don't know how to fix this issue.

Comments from Reviewable


regarding the ValueError: inputs must be a list of at least one Tensor with the same dtype and shape

There is a program that defines the loss function as follows:
reg_loss_col = tf.GraphKeys.REGULARIZATION_LOSSES
weight_loss = tf.add_n(tf.get_collection(reg_loss_col),name='reg_loss')

Running the program raises the following error message

File "/home/ decoder/kitti_multiloss.py", line 86, in loss
name='reg_loss')
File "/devl /tensorflow/tf_0.12/lib/python3.4/site-packages/tensorflow/python/ops/math_ops.py", line 1827, in add_n
raise ValueError("inputs must be a list of at least one Tensor with the "
ValueError: inputs must be a list of at least one Tensor with the same dtype and shape

I am curious how to print out the tensor information of the first parameter tf.get_collection(reg_loss_col) in tf.add_n, so that I can figure out why this cause the error.

This seems to be a program you have written, so this question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!
But the error basically means that the collection with key tf.GraphKeys.REGULARIZATION_LOSSES has no members. If you cannot find the source of this, be sure to post the full source of the program to stackoverflow as it is impossible to help you with some tiny fragment of the program. Thanks!


Idea: be more lenient about line breaks

One of the complaints is that refmt's heuristics doesn't format line breaks too well. First, let's make sure it actually does do line breaks a bit better than today. Second, in the worst case, where line breaking is a bit ambiguous, we can always attach an attribute after e.g a structure called [@@linebreak] or whatever. So
let foo = 1;

let bar = 2;

Parses and attaches [@@linebreak] to let foo, and then the printing is idempotent. But if the user writes
let foo = 1;
let bar = 2;

Then no attribute attached, and the printing still idempotently prints to that.
Again, we should just format line breaks correctly in most cases. But it's good to have a fallback idea, or have both at the same time: prescriptive formatting for e.g. let statements, non-prescriptive for, dunno, class properties or something.
cc @jordwalke @IwanKaramazow @let-def @glennsl

People use whitespace as a tool to structure code. People familiar with Go or Prettier (who isn't these days?), will find the lack of flexibility in whitespace interleaving pretty counter-intuitive.
I did some experiments with the current formatter based on the Location.t of the ast-nodes.
Every node in the parsetree has it's location attached, so why don't we use that as a heuristic for whitespace interleaving? The Location.t contains the line-height, which we can use to compute if we need to attach whitespace. Example:
   line_height_end_structure_one
- line_height_beginning_structure_two
- 1
= whitespace_interleaved_between_structure_one_and_strucure_two

The advantage would be that we don't need to attach new properties and modify the parser.
That is if the location attachment is correct 😅
I think I can make the above work, if this sounds like a good approach.
Not yet 100% sure how this is going to affect comment interleaving.

I don't know the specifics but sounds good. cc @let-def regarding affecting comment

Fixed


Option to change speed

Shouldn't be too hard to implement, see here: https://docs.oracle.com/javase/8/javafx/api/javafx/scene/media/MediaPlayer.html#setRate-double-

bc3f22517808c495ddcfbc62589e974100ed79a4


Feat legend category

更新了分类图例类型定义

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

137 of 155 (88.39%)  changed or added relevant lines in 10 files are covered.
40 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.07%) to 73.813%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ui/legend/continuous.ts
0
1
0.0%

src/ui/legend/category.ts
70
76
92.11%

src/ui/legend/category-item.ts
51
62
82.26%

Files with Coverage Reduction
New Missed Lines
%

src/util/text.ts
3
84.62%

src/ui/legend/base.ts
4
82.69%

src/ui/legend/category-item.ts
4
77.27%

src/ui/legend/category.ts
29
67.23%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
1571

Relevant Lines:
2000

💛  - Coveralls


Make plutarch.cabal cleaner and add tests for GHC 8.10

https://github.com/Plutonomicon/plutarch/issues/145#issuecomment-1022629754

Dup in part: #145

Shouldn't this be closed already?

We can close this when https://github.com/Plutonomicon/plutarch/blob/staging/plutarch.cabal#L153 is removed entirely.

We can close this when https://github.com/Plutonomicon/plutarch/blob/staging/plutarch.cabal#L153 is removed entirely.

Should be done in #302


Update _config.yml

Hi! This looks like custom configurations for your own site. You can update the _config.yml in your own repo to set these fields on your site!


Introduced a new ConvenientLine class to wrap the lines returned from ConvenientHunk.

Also overrode the line content method to provide more useful results.

This looks great! It's a breaking API change but we already have some breaking API changes in for vNext anyways.
As soon as tests pass I'll go ahead and merge unless anybody in @nodegit/owners has objections.

 I don't really know why that one randomly fails. It's like the stash OID gets corrupted somehow.

That might be fixed now after #535 is merged.

Looks like I got that stash test fixed. Going to go ahead and merge this.


vis.table.service.rows_counted vs vis.table.row_count

While fixing https://github.com/CartoDB/cartodb/compare/d7c2f0807266...cb200fd454d7 I found we have two ways of querying the table count:
vis.table.service.rows_counted (PG tables query) and vis.table.row_count (a Sequel count)
Which one is the intended one to use? because at that code vis.table.rows_counted was zero (which is weird, as calling through the service also queries PG classes but returns proper values)

Wow. It's even worse than that: ::Table has a rows_counted that is a sequel count, but exposes a rows_counted public method that is an alias to rows_estimated, which probably explains the behaviour you detected.
There're two complementary behaviours that we want to keep: estimation and actual count. I'll take a look at both and set a proper naming.

oh, if aliases to rows_estimated then it does make sense to return a zero. I didn't digged much, thanks for checking... and yes, an appropiate naming would be better so others (or me again :stuck_out_tongue: ) don't get confused. Thanks!


Handle special characters in keys of the path in change events

Description:
Handle special characters in keys of the path in change events.
We pass the path so the user knows which element of the document has changed.
The special characters . and $ can be included in the key, so they must be processed separately. We can choose one of the methods below:

[ ] Escape special characters. (e.g: '$.o\$j.n\.um')
[ ] Provide path as an array. (e.g: ['$', 'o$bj', 'n.um'])

Why:

If the key contains special characters, the keys cannot be recognized properly.

@hackerwins Hi! I have two questions!

It seems the expected result of '$.o$j.n.um' is '$.o\$j\.n\.um' when we escape special character instead of '$.o\$j.n\.um'. isnt it?
Should we handle the special characters only in Change events?(for example, remote change and local change, not shapshot) 🤔


Incomplete restrictions on resizing large data disks

Resizing a data disk from <= 4095 GiB to > 4095 GiB requires that the data disk be detached from the virtual machine prior to the resize operation.  Attempts to increase the size beyond 4095 GiB while the disk is still attached result in an error:
"Failed to update disk ''. Error: Disks cannot be resized beyond 4095 GiB when attached to a VM. Detach disk  from VM  and try again. If caching is enabled for this disk, it will not be supported for disks greater than 4095 GiB."
This article should be updated to reflect the limitations in resizing beyond 4095 GiB.

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

ID: ccc58c74-7913-f1f6-8681-1823bbd8f050
Version Independent ID: c11bab28-a82a-29a1-84bf-fb3dfb9a5cb2
Content: Expand virtual hard disks attached to a Windows VM in an Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/expand-os-disk.md
Service: virtual-machines
Sub-service: disks
GitHub Login: @kirpasingh
Microsoft Alias: kirpas

@DeclareHigh
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@kirpasingh
Can you please check and add your comments on this doc update request as applicable.

Is it possible to update the docs accordingly? This limitation is quite crucial for VM management.

@VikasPullagura-MSFT, @kirpasingh  - can you please check on this? The documentation is still requiring an update. Thank you!


description of xarray assumes knowledge of pandas

The first sentence a potential new user reads about xarray is

xarray (formerly xray) is an open source project and Python package that aims to bring the labeled data power of pandas to the physical sciences, by providing N-dimensional variants of the core pandas data structures.

Now imagine you had never heard of pandas (like most new Ph.D. students in physical sciences). You would have no idea how useful and powerful xarray was.
I would propose modifying these top-level descriptions to remove the assumption that the user understands pandas. Of course we can still refer to pandas, but a more self-contained description would serve us well.

Agreed!
Here's what @jhamman and I wrote in the abstract for our paper on Xarray:

Xarray is an open source project and Python package that provides a toolkit and data structures for N-dimensional labeled arrays. Our approach combines an application programming interface (API) inspired by pandas with the Common Data Model for self-described scientific data. Key features of the Xarray package include label-based indexing and arithmetic, interoperability with the core scientific Python packages (e.g., pandas, NumPy, Matplotlib), out-of-core computation on datasets that don't fit into memory, a wide range of serialization and input/output (I/O) options, and advanced multi-dimensional data manipulation tools such as group-by and resampling. Xarray, as a data model and analytics toolkit, has been widely adopted in the geoscience community but is also used more broadly for multi-dimensional data analysis in physics, machine learning and finance.

Probably something like that first sentence is a better high level description.

I agree as I was in this situation of jumping straight into xarray (and Python) having never used pandas.  As for other key points that could be emphasised:

, the concept of label-based indexing was new to me and may be something you may want to add more emphasis on in the Page 1 description? (I see it is already nicely explained in the paper in referecend to np.ndarrays.)
the automatic plotting with Matplotlib is super

Closed by #2430 #2657.


Create devcontainer.json

What does this aim to achieve/solve?

I think here are some parts missing, I got a dev container setup running at, https://github.com/K0IN/ollama I can open a pull request if this is wanted.
I use the dev container to debug ollama cgo bindings (c code) on Windows.
It's nice to have a static and functioning environment in development.

Hi @K0IN , thanks for the PR! Local dev files for VSCode are out of scope in terms of files to add to the repo since everyone uses a different development environment. I'll close this for now - sorry about that


Register a processor which has injections

in-n-out version: 0.1.6
Python version: 3.11
Operating System: Ubuntu 22.04

Description
Consider the following code which registers providers and processors:
import in_n_out as ino

COUNTER = 0

class Dummy:
    pass

class Thing:
    def __init__(self):
        self.list = []

def dummy_provider() -> Dummy:
    return Dummy()

@ino.inject
@ino.inject_processors
def thing_provider(dummy: Dummy) -> Thing:
    return Thing()

ino.register_provider(dummy_provider)
ino.register_provider(thing_provider)

def add_item(thing: Thing):
    global COUNTER
    thing.list.append(COUNTER)
    COUNTER += 1

ino.inject_processors(add_item)

ino.register_processor(add_item)
ino.register_processor(add_item)
ino.register_processor(add_item)

@ino.inject
def func(thing: Thing):
    print(thing.list)

func()
# prints [0, 1, 2]

Now if I want to inject a Dummy object in add_item:
@ino.inject
def add_item(dummy: Dummy, thing: Thing):
    global COUNTER
    thing.list.append(COUNTER)
    COUNTER += 1

It doesn't work anymore, as dummy and thing can be injected and the signature doesn't match with a Thing. This works:
def add_item(thing: Thing):
    @ino.inject
    def inner(dummy: Dummy):
        global COUNTER
        thing.list.append(COUNTER)
        COUNTER += 1
    inner()

But I was wondering if there could be a better solution?

thanks @davidbrochart
can you slightly reword or recode that last bit where "it doesn't work anymore", perhaps in the form of an assertion statement?  I think there's something here, but there's a lot going on in this example.  Here's a test I wrote to confirm that processors can indeed themselves have injection requirements, and it currently passes on main ...  but I suspect I've missed the crux of the issue here.
perhaps you could modify this test a bit to make it fail according to your expectations?
def test_processors_with_injections() -> None:
    """Test that we can use processors with injections."""

    class ThingToProcess:
        ...

    class InjectedDep:
        ...

    dep = InjectedDep()
    t2p = ThingToProcess()
    mock = Mock()

    # this function processes a ThingToProcess, but also requires
    # an InjectedDep to be injected before it will work
    @ino.register_processor
    @ino.inject
    def processor_that_has_injections(p0: ThingToProcess, p1: InjectedDep) -> None:
        mock(p0, p1)

    # "inject processors" of this function...
    # meaning: when `return_a_thing ` is called, the t2p return value should be processed
    #  (as a side effect of calling the function)
    @ino.inject_processors
    def return_a_thing() -> ThingToProcess:
        return t2p

    # at the moment, our `processor_that_has_injections` function still doesn't have a
    # p1 InjectedDep, so `return_a_thing`'s return value won't be processed successfully
    with pytest.warns(UserWarning, match="Processor .* failed to process result"):
        return_a_thing()

    # but if we register a something that provides an InjectedDep...
    @ino.register_provider
    def provide_other_thing() -> InjectedDep:
        return dep

    # then the processor should work as expected ...
    return_a_thing()
    mock.assert_called_once_with(t2p, dep)

Sorry, I could probably simplify the example again, but here is a test that fails (the two commented blocks work fine):
import in_n_out as ino

def test_processors_with_injections() -> None:

    class Dummy:
        pass

    class Thing:
        def __init__(self):
            self.list = []

    def dummy_provider() -> Dummy:
        return Dummy()

    @ino.inject
    @ino.inject_processors
    def thing_provider(dummy: Dummy) -> Thing:
        return Thing()

    ino.register_provider(dummy_provider)
    ino.register_provider(thing_provider)

    # def add_item(thing: Thing):
    #     thing.list.append("item")

    @ino.inject
    def add_item(dummy: Dummy, thing: Thing):
        thing.list.append("item")

    # def add_item(thing: Thing):
    #     @ino.inject
    #     def inner(dummy: Dummy):
    #         thing.list.append("item")
    #     inner()

    ino.register_processor(add_item)
    ino.register_processor(add_item)
    ino.register_processor(add_item)

    @ino.inject
    def func(thing: Thing):
        return thing.list

    assert func() == ["item"] * 3

I think it's the normal behavior, because with:
@ino.inject
    def add_item(dummy: Dummy, thing: Thing):
        ...

One cannot say that thing is the thing to process.

ahhh, I see!  sorry, I missed that.  So, to restate the problem slightly, you want to specify that add_item is a processor of the second argument in its signature.  In that case, you can use the type_hint argument for register_processor.  By default, processors  with multiple arguments are assumed to process the first argument in their signature, but you can override that inference with:
type_hint

https://github.com/pyapp-kit/in-n-out/blob/204ed24e8c2010df52a55befffdacdc09308b98a/src/in_n_out/_store.py#L329-L330

OK, looks like I just didn't prepare for processor functions that process anything other than their first argument:
https://github.com/pyapp-kit/in-n-out/blob/204ed24e8c2010df52a55befffdacdc09308b98a/src/in_n_out/_store.py#L568-L572
I think I could fix this by additionally storing the name of the function parameter that the processor processes ('thing' in your case), and then calling the processor with processor(**{param_name: result})

in summary, this boiled down to 2 things:

while processors may override the associated type that they process (using the type_hint argument when registering),  they must process their first argument. A "feature" was implemented in #52, but opted against at this time
there was a latent recursion error when a provider has its output processed.  that is was fixed in #51


Adding a Proxy

It seems that Next.JS doesn't have built-in support for the proxy field like in React. An individual package called http-proxy-middleware has to be installed.

An alternative is to put the backend URL as an environment variable. so request would look like {BACKEND_URL}/api/...}


Microsoft has made substantial improvements to removing knowledge barriers

The documentation and knowledge resources keep getting substantially better.  Thank you

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


Supporting King gets taxed less

As the title suggests, if you decide to support he king you get taxed less but if you overthrow you get taxed more.

Supporters/revolutionaries can get added to the DB in the specific server document. We can use that to work out who's what. Additionally, can assign a Supporter role or a Revolutionary role depending on their stance.

Implementing this at the moment


[BUG] Storage Class not adding labels to volumes created via helm app(s)

Describe the bug
I created a new Storage Class e.g. longhorn db and added "node selector" db on the new storage class.
In my Helm Chart I node use of the "longhorn db" storage class, deleted the old volume and found the new volumes created, but noted that the volume itself had no node tag.
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
To create a volume with the node tag specified in Storage Class, and the node(s) within Longhorn have a matching node tag...
Log or Support bundle
Environment

Longhorn version: 1.2.4
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Rancher Catalog App
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: RKE

Number of management node in the cluster: 3
Number of worker node in the cluster: 3

Node config

OS type and version: Ubuntu 21
CPU per node: 10 variable
Memory per node: 20GB variable
Disk type(e.g. SSD/NVMe): HDD
Network bandwidth between the nodes: 1Gbe

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): KVM
Number of Longhorn volumes in the cluster: 50~

Additional context
Add any other context about the problem here.

Hi @voarsh2,
Could you please provide more detail steps about how did you create the StorageClass and add "node selector" (or manifest?)
Here the document storage-tags may help.


IOError: [Errno 9] Bad file descriptor

Hi Sir,
I am started to use your library to do traffic network.
I have python 2.7 and have installed osmnx with conda. My computer is 64-bit. I use Spyder for analysis instead of jupyter.
The code I am running is a very basic one:
The code can be run without problem on a ipython kernel for the first time, but when I run it for a second time, the following message shows up:
runfile('C:/Users/B***/Desktop/testing.py', wdir='C:/Users/B***/Desktop')
Traceback (most recent call last):

  File "C:\Anaconda_27\lib\site-packages\IPython\core\interactiveshell.py", line 2895, in run_code
    self.showtraceback()

  File "C:\Anaconda_27\lib\site-packages\IPython\core\interactiveshell.py", line 1828, in showtraceback
    self._showtraceback(etype, value, stb)

  File "C:\Anaconda_27\lib\site-packages\ipykernel\zmqshell.py", line 541, in _showtraceback
    sys.stdout.flush()

IOError: [Errno 9] Bad file descriptor

The code is this one:
import osmnx as ox

ox.config(log_file=True, log_console=True, use_cache=True)
# define a bounding box in San Francisco
north, south, east, west = 37.79, 37.78, -122.41, -122.43

# create network from that bounding box
G1 = ox.graph_from_bbox(north, south, east, west, network_type='drive_service')
G1 = ox.project_graph(G1)
fig, ax = ox.plot_graph(G1)

Thank you very much for your kind assistance.
Regards,
yyan162

This doesn't look like an OSMnx issue, but something with your environment. I'd strongly suggest installing OSMnx in a virtual environment (a python 3 one, to be specific) and then running the code again.
conda create --override-channels -c conda-forge -n OSMNX python=3 osmnx


Add JSON Functions to API and JDBC Adapter

Please add support for the new JSON functions to the API and the JDBC adapter.
Example Queries
SELECT '{"a": 1}' IS JSON VALUE WITHOUT UNIQUE KEYS FROM dual; -- returns true
SELECT 'not JSON' IS JSON VALUE WITHOUT UNIQUE KEYS FROM dual; -- returns false

With IS NOT JSON the result of those queries would be negated.
SELECT JSON_VALUE('{"a": 1}', '$.a' RETURNING VARCHAR(100) UTF8 NULL ON EMPTY NULL ON ERROR) FROM dual; -- returns '1'

Capabilities
New capabilities:

FN_PRED_IS_JSON
FN_PRED_IS_NOT_JSON
FN_JSON_VALUE

Predicates
New predicates:
exp1 IS JSON {VALUE | ARRAY | OBJECT | SCALAR} {WITH | WITHOUT} UNIQUE KEYS (requires predicate capability IS_JSON)
{
    "type": "predicate_is_json",
    "expression": {
        ...
    },
    "typeConstraint": "VALUE",
    "keyUniquenessConstraint": "WITHOUT UNIQUE KEYS"
}

The same can be used for predicate_is_not_json (requires predicate capability IS_NOT_JSON).
Notes:

typeConstraint is "VALUE", "ARRAY", "OBJECT", or "SCALAR".
keyUniquenessConstraint is "WITH UNIQUE KEYS" or "WITHOUT UNIQUE KEYS".

Scalar Functions
New special case for scalar functions:
JSON_VALUE(arg1, arg2 RETURNING dataType {ERROR | NULL | DEFAULT exp1} ON EMPTY {ERROR | NULL | DEFAULT exp2} ON ERROR) (requires scalar-function capability JSON_VALUE)
{
    "type": "function_scalar_json_value",
    "name": "JSON_VALUE",
    "arguments":
    [
        {
            ...
        },
        {
            ...
        }
    ],
    "returningDataType": dataType,
    "emptyBehavior":
    {
        "type": "ERROR"
    },
    "errorBehavior":
    {
        "type": "DEFAULT",
        "expression": exp2
    }
}

Notes:

arguments: Contains two entries: The JSON item and the path specification.
emptyBehavior and errorBehavior: type is "ERROR", "NULL", or "DEFAULT". Only for "DEFAULT" the member expression containing the default value exists.

The functions were implemented in
https://github.com/exasol/virtual-schema-common-java/issues/131
Here I'm adding the new API description to the documentation.

Closed with #331


Write a Dart program to swap two numbers without using third variable

Description
Write a Dart program to swap two numbers without using third variable
Input  : 10 20
Output : 20 10

How to contribute

Save the solution in program/swap_two_numbers_without_using_third_variable.dart file
Add swap_two_numbers_without_using_third_variable.dart file in swap-two-numbers-without-using-third-variable folder

!assign

linked to PR #7197


shutdown命令巨慢

Is this a regression?
Yes
Description
shutdown完全关闭需要 20s
Please provide a link to a minimal reproduction of the bug
No response
Screenshots or videos

Please provide the version you discovered this bug in (check about page for version information)
No response
Anything else?
No response

shutdown 需要disable compaction

shutdown 需要disable compaction

好的 我们改下 蟹蟹水哥


feat: emit event to handle chunk load errors

Description
resolves https://github.com/vitejs/vite/issues/12080
This PR emits a vite:preloadError event which can be handled by a framework or user. Primarily this will be useful for saving state and reloading an app in the event of a fresh deployment.
Additional context
If preventDefault() is called on this event, the error will not be thrown. I confess I'm not entirely sure whether this is a good idea, but it provides an escape hatch that may be useful.

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

When is the next team meeting?


Update feather icons to version 4.21.0

I've updated the Feather pack to version 4.21.0

Sure!

Closing this since #926 was merged 🙂


Work with application/xhtml+xml documents

When a document is application/xhtml+xml, tags are case-sensitive, <svg> elements need a namespace in order to be styled, and innerHTML (mostly) doesn't work.  This patch addresses those issues.

No worries, I appreciate the contribution. Sorry it's taking so long to get back to you.
Out of curiosity, which device / browser are you using where autoplay doesn't work? I've tested the demo site with windows, macos, android, and ios, and it works without being muted, because it's being opened in response to a click event.
The other pull request it's breaking the build command so I need to figure out what's happening before merging anything.

Sorry about breaking the build....  I'm not familiar with your build system...  I made the same changes to that file as I made to the final JS to make it work, and then crossed my fingers and hoped for the best.
I know serving as application/xhtml+xml is fairly rare, but if these changes could be incorporated we few who like to do that would be very happy.

I've tried to attach the final JS I've been using, but GitHub won't let me attach a Javascript file here (even after renaming it .txt).  If it would be helpful to have, please let me know and we can figure out a way to send it.

After reading a bit about xhtml I don't think I want to add support to the main script.
It would be fine if the changes didn't add weight, but I don't want to add bytes for everyone to cover a tiny fraction of use cases.
If you feel strongly about it, I encourage you to fork the repo and make bigpicture-xhtml. I'm working on a new lightbox currently and won't be adding new features to this one, so it's essentially fully mature.
Sorry about that.


What is the difference between the two backends ?

The documentation presents two backends:

sqlalchemy
pgstac

I understand they rely on two different libraries. However, why do we need both of them? Why do you maintain 2 different backends?
How to know which backend we should select? Would it be possible to have some documentation to highlight the difference between both? Finally, does it make sense to have both backends up and running in a production environment or should we run only one of them ?
I couldn't find any information on this so if someone could clarify all this, it would be great.
Thanks in advance for your replies.

great question. I'm curious about this as well.

One thing I think I have observed is with Alchemy, since it is an ORM, there are migrations you can run to host the db elsewhere. I have not yet determined how to run the "migrations" to install the database features for pgstac into a seperate hosted db (e.g. functions like get_collection()).

This is a slight variation on the original question, but I wonder if it would make sense to move the backends to separate repos. Given that there are now a few other backends out there (ElasticSearch, MongoDB, etc.) with different repos it seems a bit arbitrary to be packaging both the core/abstract API code and these 2 backend implementations together. It seems like having them in this repo adds complexity to the Makefile, Docker infrastructure, and testing code, and it also adds some development burden because any changes need to be applied to both backends.
Could we instead have a single stac-fastapi-core repo that includes the stac_fastapi.api, stac_fastapi.types, and stac_fastapi.extension packages and move the 2 backends to their own repos?

I think moving the backends into their own repos is a great idea, I see many benefits:

Enforce better separation of concerns between API layer and backend implementations.
Allow us to version everything more independently than we can right now.
Force us to improve testing on the core packages; most of core is tested through a backend implementation instead of directly which is not ideal.
As you mention, core should be easier to maintain if it doesn't include backends.  Changing anything in core right now requires updating the backend implementations as well, and very few people know how both backends work which slows development down.
De-incentivizes forking; I'd much rather have users pip install stac-fastapi + a backend implementation than forking this repo

I'm going to close this as fixed by #555.


make Rav1dDecodeFrameType an enum

Looks good to me! As we go forward and make more of these types into enums we may want to use strum or a similar crate to automatically derive the conversions, but for now the manual impls you wrote are good.

We do already have strum as a dependency btw.


[DE-545] UTF-8 names

It is the responsibility of the server to enable extended names. The driver can send any UTF-8 string along with the request.
Therefore, this PR doesn't really add any new and exciting features. Instead, it adds two tests (one for databases and one for collections), making sure the driver can successfully forward UTF-8 names to the server.
Note that, in order to run the tests, the server has to be started with the --database.extended-names option. Unfortunately, this is only possible starting with 3.11 docker images, and we're currently using 3.10.9 for our GitHub actions. Nevertheless, I have added the parameter as a comment for now. I can confirm the tests have passed locally for versions >= 3.11.0.

Codecov Report

Merging #260 (4045f76) into main (8b09e07) will increase coverage by 0.41%.
The diff coverage is 93.75%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #260      +/-   ##
==========================================
+ Coverage   98.58%   99.00%   +0.41%
==========================================
  Files          26       26
  Lines        3899     3931      +32
==========================================
+ Hits         3844     3892      +48
+ Misses         55       39      -16

Impacted Files
Coverage Δ

arango/collection.py
98.48% <50.00%> (+2.00%)
:arrow_up:

arango/client.py
98.30% <75.00%> (-1.70%)
:arrow_down:

arango/aql.py
95.04% <100.00%> (+0.04%)
:arrow_up:

arango/cursor.py
100.00% <100.00%> (ø)

arango/http.py
100.00% <100.00%> (ø)

please try to create indices on these collections / databases as well.

the test should create indices on database/collections with unicode names.


Logo for Roll Call

@aishwaryachand  this looks nice, please create a separate branch to make the pr not directly in master branch .
@SKAUL05  please help if required


API: choose columns to transfer

Users should be able to select columns that they want to transfer

#82


Migrate docs

What type of PR is this?
(documentation)
What this PR does / Why we need it?
This pull request adapts the documentation from the internal backplane repository for a public audience. It adds the old README and the guide for setting up the bash/zsh PS1.
Which Jira/Github issue(s) does this PR fix?
Resolves # OSD-15318
Special notes for your reviewer
Pre-checks (if applicable)

[ ] Ran unit tests locally
[x] Validated the changes in a cluster
[x] Included documentation changes with PR

/retest

/lgtm
/approve


Added serializetools.

I've added a new package called serializetools. Thanks in advance for the help.

Thanks :)


Improved validation error reporting

Context
Validation error messages (task or work in general) are not really actionable. This PR tries to improve the situation by introducing a new framework for reporting user-facing problems. A problem should tell what happened, where it happened, why it happened, and give possible solutions as well as documentation links.
This introduces the new framework for reporting validation problems, shows one use case. We know need to go through all places and use the new stuff (once this is merged).

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.

mmm, I'm not sure how e0a2265fc123098f32c57abc5e5a7f1967ad4bfc slipped in... this is strange.


Dark mode doesn't allow user to edit or close add config window

When using dark mode, you can't select a new configuration, you can't edit the configuration, and you can't exit this window.

Hi @ninaforsyth - thanks for the report. This looks like a bug with the dark mode settings. We will have a look on patch asap.

Was fixed in version 0.0.2


Ability to diff changes to Angular model before updating Meteor collection

What I consider so cool with angular-meteor is the ability to tie ng-model more or less directly to a collection in the database, but as it currently stands, the entire collection document is swapped with every change. This is suboptimal for performance, and causes big issues when trying to restrict a user's right to update a collection, since I can't restrict the set of fields in the collection that the user can update.
So, the problem was that every update to the model would {$set: {}} the entire client side model on the collection. Here I've added a third argument that can be passed to $meteorCollection, which if set to true, makes the function construct a diff equivalent to what would be passed to Collection.update if we would manually change only the fields that have been updated by the user.
The diff function is fairly simple but still takes in to account nested objects, arrays and performs "actual" equality checking with angular.equals instead of just ===.

@fredrikekelund this is very nice and was our next talk to focus on !
Have tested what will happen if the Collection is ordered in a certain way and then you push an object to the end which is not in the right order?
Also, if you push an object to the array which is out of the cursor bounds? like out of limit or page?

More questions:

If it's working good, why not do it all the time, why do we need the flag?
Have you written the diff code yourself?

Also, have you seen this?
https://github.com/flitbit/diff

Ohhh, look what the Meteor guys sent me:
https://github.com/meteor/meteor/tree/devel/packages/observe-sequence
(look at the tests for usage)
Also Blaze's usage of it:
https://github.com/meteor/meteor/blob/devel/packages/blaze/builtins.js#L101-L170
@netanelgilad look at it

@Urigo nice to hear! Think this change might need some discussion, so here's my attempt at answering your questions.

Have tested what will happen if the Collection is ordered in a certain way and then you push an object to the end which is not in the right order?

This diff function does currently not diff arrays, only objects. If an array doesn't match between the old and new model, it is swapped out entirely in the collection. There is of course room for improvement here, but this change plugged in nicely to the current collection updating that used a {$set: {}} modifier.

Also, if you push an object to the array which is out of the cursor bounds? like out of limit or page?

I think my previous answer answers this question as well, but correct me if I'm wrong.

Would be cool also to add those use cases to the tests actually..

Absolutely, I'll try and add a few tests today!

If it's working good, why not do it all the time, why do we need the flag?

I added the flag since it's such a big change and I wanted to start by making it optional, maybe so we can test it even further and see if improvements can be made, but I agree that this should be the default behavior eventually!

Have you written the diff code yourself?

Yes, it's based of a simple example from StackOverflow but changed to use angular functions rather than underscore, to remove empty objects from result as a result of them being identical between a and b, to be able to deal with arrays and to perform angular.equals equality checking rather than === (to account for identical objects and arrays).
I wanted to start out with a simple diff engine, I saw that there are multiple ones out there, and many construct complex diffs with their own "syntax" in the result. I was after only the simple, actual diff that would allow me to restrict the fields in the collection that the user can update.

@fredrikekelund @netanelgilad look here:
https://github.com/Urigo/angular-meteor/commit/85e72106516ab30b53ab3e389e523efc13eb10bb

Is that meant to diff the collection as a whole? I tested it and it seems to call an update to all documents in the collection, like it did before. But maybe it would work together with my change?

@fredrikekelund are you sure?
It is suppose to call only the right actions to minimongo and then after update the Angular only through the observeChanges in the specific place.
Can you show me how you test it?

@fredrikekelund also, I would love for you to look at the diffArray function, make sure it is doing the diff OK

Looking into it!

updateCursor seems to be aware of the actual changes. But as I understand it from looking at the code, there is even a comment about diffArray not looking at how the documents/objects in the collection change:
// Arrays can contain arbitrary objects. We don't diff the
// objects. Instead we always fire 'changedAt' callback on every
// object. The consumer of `observe-sequence` should deal with
// it appropriately.

I'll merge the code from your branch with mine and see what I come up with

@Urigo I was able to combine your changes with mine and now I'm seeing only the fields I've actually changed showing up in the modifiers sent to the server!

Can you try to pull request our branch?
Just so it will be easier for me to see your changes better.
(Don't worry, if you pull request a branch you still get the credit you clearly deserve).
Also, I think it might be cleaner of you use your object diff inside the diffArray function to fire the cahngedAt event only when needed, that way it will be cleaner in the package code no?
You are amazing.

@Urigo haha nice to hear! :) I'm working on it in the other branch right now, realized I need to test for removed keys as well, I'll make a new pull request as soon as it's ready

Moving this discussion to #165, that's based on the feature/observeAngularChangeLikeBlaze instead of master


Create hardware usage tracking for matching service

What changed?
Add hardware usage tracking for matching hosts

Why?
We need a way to monitor customer utilization on the matching service in order to improve efficiency

How did you test it?
Tested on dev environment

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 0188bbf1-4113-4141-9962-ce8c421e4963

31 of 35 (88.57%)  changed or added relevant lines in 1 file are covered.
20 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.05%) to 57.253%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

service/matching/matchingEngine.go
31
35
88.57%

Files with Coverage Reduction
New Missed Lines
%

common/task/fifoTaskScheduler.go
2
87.63%

common/task/weightedRoundRobinTaskScheduler.go
2
88.6%

service/history/task/transfer_standby_task_executor.go
2
86.93%

service/matching/matcher.go
2
89.57%

service/history/task/task_util.go
4
69.43%

service/history/queue/timer_queue_processor_base.go
8
77.26%

Totals

Change from base Build 0188b6ba-6aaf-4739-8b30-52eaead5800c:
0.05%

Covered Lines:
87082

Relevant Lines:
152099

💛  - Coveralls

The changes broadly look like they work, and I do think we need to do some more of this regarding per-domain/host based limits like this. But I'm a bit confused by the intent of these specifically?
What do we intend to understand by emitting per-domain metrics around matching's CPU and memory? This is a shared resource so it doesn't exactly make sense to me to domain as a dimension? Like the memory might be high for all sorts of reasons, and the fact that you're looking at a domain's view is kind of... not related? Am I missing something?

Closing as part of old PR cleanup effort. Feel free to create a new PR from your fork branch when/if this gets picked up again.


Change Device Property via Azure Function

Hello everybody,
I am trying to get my SigFox location tracking devices (GPS) to Azure IoT Central. I would like to use the Maps functionality of the dashboard to visualize the location of my devices. Therefore I need to change the device property continiously.
I have been trying to follow this instruction:
https://docs.microsoft.com/en-us/azure/iot-central/howto-connect-nodejs#add-a-real-device
I believed the Azure Function is simulating a device and uses the SigFox payload as a input for telemetry, event and/or state. So I tried to expand the engine.js as described above (url), but it lead me there to understand, that the device twin is not callable via http - known issue here:
https://github.com/Azure/azure-iot-sdk-node/issues/351
Is is possible to expand the engine.js so, that I do not need to use the device twin here?
I also tried to use MS Flow but it did not end well either.
Are there any further possibilities? Thank you very much!
Cheers
DD

Please have a look at https://github.com/kartben/iotc-device-bridge/blob/properties-support/IoTCIntegration/lib/engine.js for an example of how to do this. Then just add "properties" in your POST payload in addition to "measurements" and that should work :-)

Note that this is indeed using MQTT instead of HTTP to address the limitation you mentioned in your initial comment, and this hasn't been fully tested, so please use this as a starting point :-)

Microsoft late last night posted an update where they've added an official location telemetry field which offers real time and historical data points, just like the other telemetry fields. The update is detailed here: https://docs.microsoft.com/en-us/azure/iot-central/howto-set-up-template#measurements
I cannot for the life of me get it to work. They don't really provide much detail or documentation beyond Screenshots of the dashboard and a Javascript snippet:
assetloc: {
  lon: floating point number,
  lat: floating point number,
  alt?: floating point number
}

I push data formatted with json from a Particle Dev board and it works fine for all the other fields, but not their new location telemetry. I tried sending it to lat, lon, assetloc. Tried sending a joined set of coordinates to assetloc. Tried nesting the lat and lon fields within the assstloc key in my json and had no luck getting any of it to post.
I think when I tried nesting the data, the bridge wouldn't even connect to Azure properly and failed with error code 400.
Any comments on how push data to the new fields? Is this something the bridge would need to be updated to support?

@kartben Thank you for the hint! Indeed I tried it the way you presented, it is nice, but not working yet correctly due to the device twin properties. It was pain in the *** tbh. I would recommend not to do it like that, since Microsoft seems to implement this feature by themselves as @Danorexic mentions.
So we have been working cutting edge - thats nice! :)
Furthermore I tried that feature, which is not properly working now and it is leading to an error. The Map is not displayed yet. I am really looking forward, since that is going to be fixed soon!

@Danorexic @drunkdev1991 If you want to send location as telemetry, you first need to make sure you've declared a telemetry field in your template of type "location", say you name this field "loc" (it can be anything, really). Your measurement payload should then look something like the following:
{
    "device": {
        "deviceId": "mangoh-bcabe"
    },
    "measurements": {
        "loc": {
            "lat": 12.345,
            "lon": 67.890
        }
    }
}

However, one important caveat is that the "loc" field is now causing the validation of the measurements to fail in the bridge (see: issue #25) so one short term fix suggestion is to simply remove the call to validateMeasurements() in your engine.js file until we deploy a proper fix :)

Confirming that the solution posted by @kartben works and I now have data points on the IoT Central location telemetry map. For anyone new to Azure stuff like me, to update your engine.js file, you go to your Azure portal, Function App, expand down and click on IoTCIntegration. It'll take a moment and initially open index.js. It's easy to miss but on the right hand side of that view, there's an option that says 'View files', click that and you'll be able to navigate to the lib folder and open engine.js.
Commenting out lines 38-40 disabled the check and allowed my coordinates to push to IoT Central:
    //if (!validateMeasurements(measurements)) {     //    throw new StatusError('Invalid format: invalid measurement list.', 400);     //}

My function also provides similar results - it is opening the connection to the device in IoT Central, forwards the message and closes the connection successfully. The payload has same structure as @kartben described.
The data is not forwarded to IoT Central though, and no values are displayed.
Instead I get an unknown error with a error code. I forwarded that to the MS Support.
Additionally the Dashboard of my device template is somehow corrupted, if i want to change or delete existing elements, the tiles are magically moving away below my mouse pointer...
This is obviously not an issue with the function itself, but maybe someone is experiencing the same problems?


TypeError: 'DefaultCfg' object does not support item assignment

Hi @myownskyW7 @V3Det the training on detic open vocabulary detection raise the following error:
TypeError: 'DefaultCfg' object does not support item assignment
Could you pls help

Please install timm==0.5.4 and try again
pip install timm==0.5.4 --upgrade

thanks, it solves the problem


Is there a way to update certificate using only powershell and not use ARM template

Is there an option we can used to use only powershell only and not use ARM template?

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

ID: 30fab368-8a51-14d6-1775-defa5f0ff04f
Version Independent ID: a8fbbe3d-063a-d6c9-b61d-1ebcfcaa1422
Content: Manage certificates in an Azure Service Fabric cluster
Content Source: articles/service-fabric/service-fabric-cluster-security-update-certs-azure.md
Service: service-fabric
GitHub Login: @athinanthny
Microsoft Alias: atsenthi

@tkumark
Thanks for the question! We are investigating and will update you shortly.

@tkumark You can use this cmdlet Add-AzServiceFabricClusterCertificate to add a secondary certificate to the cluster.
Its mentioned in the doc as well.
I hope you are looking for this cmdlet only.  Let me know your thoughts

Is there a command for adding reverse proxy secondary and client certificate?

@tkumark are you asking about the load balancer

Not related the load balancer. In Service fabric Security section you we can see certificates used by Service Fabric. We have Add-AzServiceFabricClusterCertificate for adding Cluster Certificates and Add-AzServiceFabricClientCertificate for adding Client certificates but  I do not see once for Reverse Proxy Certificate

@athinanthny please add your comments

@tkumark I didnt find a way in Azure cli or Azure powershell of sfctl to update the reverse proxy certificate in Azure service fabric cluster.
In the docs ARM template example is given.
@athinanthny Please add your comments

@tkumark
The process to update RP certificate without ARM should be

Add certificate to VMSS https://docs.microsoft.com/en-us/azure/service-fabric/scripts/service-fabric-powershell-add-application-certificate
Update setting in RP to point at new certificate https://docs.microsoft.com/en-us/powershell/module/az.servicefabric/set-azservicefabricsetting?view=azps-3.0.0

there is no powershell cmdlet for this

@tkumark please add your comments

@tkumark
I will close this out for now. If you need additional help please let me know and we can reopen and continue.

@jakaruna-MSFT could you please elaborate on how to use Set-AzServiceFabricSetting for updating Reverse Proxy to point to new certificate. What values would I have to pass for section and parameters?

Section should be "ApplicationGateway/Http" and Parameter should be "ApplicationCertificateValidationPolicy". These settings are detailed here https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-reverseproxy-configure-secure-communication.

@amenarde I think what I am asking for is misunderstood. The link provided above it related to HTTPS forwarding to services. What I am looking for is configuring reverse proxy to accepts HTTPS traffic using powershell. In more specific update the secondary thumbprint using powershell that way I can roll over the certificate when the certificate expires.
http(s): The reverse proxy can be configured to accept HTTP or HTTPS traffic. For HTTPS forwarding, refer to Connect to a secure service with the reverse proxy once you have reverse proxy setup to listen on HTTPS. Ref: https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-reverseproxy

@tkumark let me seek guidance from someone who understands Reverse Proxy better to answer your question. Thanks for waiting.

@amenarde  Thank you it would be of great help

@amenarde sorry to trouble you but were you able to get any information on this?

@tkumark
If you are looking for a powershell command to update reverse proxy certificate, we dont have one.
Plus, even though there are commands to update  if you are updating a client/server certificate, you should try to do it through ARM deployment.


docs(pages): update ack

📝 Changed routes:

/

(view full size)

Commit 5b6acb053a728ddfa188ded5ffaeabc502a4ce54 (https://docs-2fnnovab5-manifold.vercel.app).


Add Color Shifting / 3D Blur Effect

This effect involves a sort of double or triple printing of the ink layers, printed from different channels that are shifted and blurred.
The color shifting / 3D blur effect involves these steps:

split channels from original source (either RGB or CMYK or HSV)
randomly keep color or convert to grey scale or recolorize to a different shade
shift split channels to randomly adjust placement, shifting X/Y positions by random offset and random directions for each channel; most often, channels will go in opposite directions from each other, but might not
most often, reduce intensity of split channels (Poisson-likely to reduce intensity); often adding in a mild blur to one or more split layers
most often, retain the original layer and blend with other split layers; otherwise, print only the split layers

Some additional inspiration might come from tutorials like this one on how to do the similar "glitch effect" in photoshop step-by-step: https://pixelbuddha.net/journal/tutorials/how-to-create-glitch-effect-in-photoshop

@kwcckw Can you demo how we can apply this augmentation to reproduce the image above where the colors are like shadows echoing the text above and below the line in slightly different shades?

@kwcckw Can you demo how we can apply this augmentation to reproduce the image above where the colors are like shadows echoing the text above and below the line in slightly different shades?

The shadow like effect is only applicable for grayscale image only since those displaced color will be in grayscale now and hence creating a shadow like effect. Some grayscale examples:

Thank @kwcckw.  After we finish the other 2 new augmentations (#304 and #303), then we can come back and do the channel splitting so that we can get that effect on black/grey ink layers.


Validation Bulk Fixes

Docs Build status updates of commit 9436a4a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/latest/monitor-query-readme.md
:bulb:Suggestion
View (azure-java-stable)
Details

docs-ref-services/latest/monitor-query-readme.md

Line 2, Column 1: [Suggestion: description-missing - See documentation] Missing required attribute: 'description'.
Line 32, Column 5: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/java/azure/jdk/?view=azure-java-stable' will be broken in isolated environments. Replace with a relative link.
Line 32, Column 5: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: https://docs.microsoft.com/java/azure/jdk/?view=azure-java-stable. Otherwise, remove the view parameter.

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

Docs Build status updates of commit 8ca675c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-services/latest/monitor-query-readme.md
:white_check_mark:Succeeded
View (azure-java-stable)

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

Global effort to fix build validation errors
@srnagar
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you’d like to see. Thank you


Themes not applying at all.

I've installed the extensions but the UI isn't changing at all. I've reloaded and tried multiple times and various accents, but it remains at the default, dark theme or not. I've unloaded all my extensions and tried again with no success. I am running on commit a9fed7c, which is before the 3.23 gradio changes.

I see, so the issue is that there are now two versions of the theme (original and post gradio update). I guess the solution is to have both themes exist and just check for the gradio version? I can work on this soon. Thanks for bringing this to my attention.

The most recent commit should fix the issue for legacy gradio versions, please let me know if it persists. If it doesn't work, then change your theme in the settings, save, then press the reload button.

Most recent commit just fixed the accent color not being applied on legacy gradio versions. I believe this issue is solved so I'll be closing this. Please feel free to reopen this issue, or start a new issue, if there is a problem.