[Bug]: Website - Tutorial blog, missing highlight on code

Which Remix packages are impacted?

[ ] remix (Remix core)
[ ] create-remix
[ ] @remix-run/architect
[ ] @remix-run/cloudflare-workers
[ ] @remix-run/dev
[ ] @remix-run/express
[ ] @remix-run/netlify
[ ] @remix-run/node
[ ] @remix-run/react
[ ] @remix-run/serve
[ ] @remix-run/server-runtime
[ ] @remix-run/vercel

What version of Remix are you using?
Not relevant
What version of Node are you using? Minimum supported version is 14.
Not relevant
Steps to Reproduce
Go here : https://remix.run/docs/en/v1/tutorials/blog
Scroll to the "Action" section
The code block just after "TypeScript is still mad, so let's add some invariants and a new type for the error object to make it happy."
Is missing a green highlight on
import invariant from "tiny-invariant";
Expected Behavior
import invariant from "tiny-invariant";
Should be highlighted green because it was not on the file before.
Actual Behavior
It's not highlighted, like it was on the file before. It's misleading, because the user might think he forgot something before.

Thanks @jakewtaylor, I did not find the way to change the highlights... Next time I'll do it myself


Node Debugger Consistently hangs

Issue Type: Bug
The new debugger in 1.47 needs to be reverted. It consistently hangs on me when executing our tests. I downgraded to 1.46 and it went away. Then VSCode auto updated and the problem is back.
Please turn off this new debugger until you have worked out all the issues. This is very painful.
VS Code version: Code 1.47.2 (17299e413d5590b14ab0340ea477cdd86ff13daf, 2020-07-15T18:18:50.054Z)
OS version: Darwin x64 19.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2200)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 4

Memory (System)
16.00GB (0.26GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

terraform
4op
0.1.10

vscode-eslint
dba
2.1.8

githistory
don
0.6.8

gitlens
eam
10.2.2

prettier-vscode
esb
5.1.3

copy-github-url
mat
0.6.0

vscode-typescript-tslint-plugin
ms-
1.2.3

vim
vsc
1.16.0

You can get the previous node debugger by adding a "debug.javascript.usePreview": false to your settings.

Can you do either of the following?

If it's consistent, are you able to easily create or share a reproduction case?
If not, a trace log using these instructions is also very useful 🙂

If you're able to, add "trace": true to your launch.json and reproduce the issue. The location of the log file on your disk will be written to the Debug Console. Share that with us.
⚠ This log file will not contain source code, but will contain file paths. You can drop it into https://microsoft.github.io/vscode-pwa-analyzer/index.html to see what it contains. If you'd rather not share the log publicly, you can email it to<EMAIL_ADDRESS>
/needsMoreInfo

The new debugger hangs on me as well. Trying to track down the issue it comes to a console.log command that just not returns. Adding this to my launch config "solve" the issue:
"console": "integratedTerminal",

This happens on my server startup on debug mode where the logger is set to "verbose" and there is ton of logging. In order to confirm this is the issue I change the log level to "error" and debugger does not hang.
Hope it helps...

That is issue https://github.com/microsoft/vscode/issues/102254

I didn't think of it previously, but this is most likely a duplicate of https://github.com/microsoft/vscode/issues/102254#issuecomment-658247768


[BEAM-522] Fixes GcsIO.exists() to properly handle files that do not exist

Currently this invocation fails for non existing files instead of returning false.
Updates FileSink.finalize_write() so that we capture and log any transient errors that get thrown at the channel_factory.exists() call.

R: @charlesccychen

Thanks! LGTM.

Thanks Charles.
R: @dhalperi

R: @kennknowles since there seems to be a lot of PR reviews on Dan's plate :)

Thanks Ken. PTAL.

Friendly ping :)

LGTM

Thanks Ken.


Add KafkaJS exporter to list of available exporters

Please add this new exporter to list of available exporters. Please let me know if there is something missing.

Any change to get this PR merged? Again, please let me know if there is anything is missing.

Dear Prometheus team,
do you have any news? I filed a similar PR #2355 which was already merged months ago.


Update catalog2dnx to support SemVer 2.0.0

SemVer 2.0.0 packages will be in the flat container but URLs that contain the version must use the normalized version.

So strip off metadata etc?


stop batch reads after limit is reached

see the TODO by onItem: https://github.com/maxogden/level.js/blob/master/iterator.js#L61

Fixed in #68.


feat(doc técnica): Sumário #43

What issue does this pull request resolve?
#43
What changes did you make?
Mudei o readme da documentação técnica.
Is there anything that requires more attention while reviewing?
Não.

Codecov Report

Merging #44 (64ae94d) into master (cf9b105) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   30.61%   30.61%
=======================================
  Files           4        4
  Lines          49       49
=======================================
  Hits           15       15
  Misses         34       34

Flag
Coverage Δ

integrationtest
30.61% <ø> (ø)

unittests
30.61% <ø> (ø)

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

Continue to review full report at Codecov.

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


SAK-30528 Added some generics to AssignmentSubmission

LGTM

Cheers, Sam.

Java 7 introduced the diamond operator (https://docs.oracle.com/javase/7/docs/technotes/guides/language/type-inference-generic-instance-creation.html), for stuff like that that isn't a bugfix I think it's good to use things that make the code simpler  as I don't expect this to get backported.


radish decomposed aggregates

fixes #478
Making Radish support decomposed groupbys was relatively simple. It involved

implementing GrappaShuffle (the code for which is inside of the implementations of Grappa global hash tables).
pulling DecomposeGroupBy rule out of myrialang to rules.py, where it can be used by any distributed algebra
implementing GrappaPartitionGroupBy as a simple override of some code templates from GrappaGroupBy

(apologies for throwing with into bmyerz/tpc-h-fixes...that branch needs to be merged finally)

result of the optimization on TPC-H


Fix Antrea IPsec e2e tests

The tests were passing even when IPsec support was broken and the
antrea-ipsec container was crashing on start. See #1043.
We update the tests to run additional checks:

make sure the containers do not crash after updating the YAML (with
IPsec -> without IPsec, and vice versa), by monitoring container
status for 20 seconds
make sure a Security Association is created correctly between Nodes
when checking inter-Node Pod connectivity

An alternative solution for the first item could have been to use
minReadySeconds for the Antrea Agent DaemonSet, but I didn't want to
modify the YAML manifest.
Fixes #1043

Codecov Report

Merging #1215 into master will decrease coverage by 8.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1215      +/-   ##
==========================================
- Coverage   56.19%   47.33%   -8.86%
==========================================
  Files         105       70      -35
  Lines       11550     6158    -5392
==========================================
- Hits         6490     2915    -3575
+ Misses       4491     2968    -1523
+ Partials      569      275     -294

Flag
Coverage Δ

#integration-tests
47.33% <ø> (ø)

#unit-tests
?

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


mypy: Improve typing of request.pyi (REQ).

This is a minimal version of #7261, involving expanding request.pyi to cover some of the perceived advantages while retaining a separate stubs file.
Other files have been updated to pass mypy where necessary, though there is one outstanding case which I've not yet been able to resolve.
Converters and defaults must now now match the specified type in the function signatures, which wasn't previously checked by mypy, in addition to the typing of the other REQ parameters.
Should the types be removed from request.py?

Requesting review from maybe @gnprice?
I think I got a general LGTM from @timabbott and @showell (in the integration meeting on Friday), but I would appreciate help with the remaining error.
Removing the type annotation for report_send_times complains about an absence of annotation, but continues to complain with the same problem:
zerver/views/report.py:36: error: Argument 1 to "REQ" has incompatible type "Callable[[str], int]"; expected "Callable[[str], str]"
I'm guessing this is the time parameter, whose converter function is Callable[[str], int], but I'm unsure where the expected version comes from (particularly with no annotation). Any ideas?

Just a minor heads-up that I managed to tie the REQ parameters every so slightly more, and added some explanatory text at the top of request.pyi. Hopefully it's accurate and reasonable to add :)

For the zerver/views/report.py thing, the issue is actaully the received and displayed parameters; that union isn't working correctly.
I worked around that by making the types a bit more consistent, and merged this as 42f5eea61ff0079fef1b2744cddda9a173596535.  This is great, thanks @neiljp!
I'm not sure about removing annotations in request.py -- I'd want to ask someone like @gnprice who knows mypy better (he's on vacation in India this week though).

Certainly for now I'm rethinking removing the annotations, though keeping them in two places feels rather questionable.
I thought about the report.py change to just being an int, but erred against this - glad to see it solved the problem 👍


empty folders created

https://i.imgur.com/GGrZJ5v.png
The course link is https://www.udemy.com/trig-by-krista-king
My commands are simply the -u username -p password courseurl
before maybe 1 week or so it worked well not sure what suddenly happened, it created empty folder for each section in course
My OS is windows 7 professional and i am using latest python 3.6.5

@Redlava60 please read this : Issue reporting guideline. anyways you should clone the recent repository already added a fix for this


Bump golang.org/x/net

What this PR does / why we need it:
Bump golang.org/x/net

Is this resolving a CVE?


Fixing merge conflicts

Thanks @BanisharifM -- please either create a new fork or update your fork before commiting your midterm blog entry. Then create a new pull request. If done correctly, your pull request will only contain changes related  to your midterm blog post.


How do I remove dropdown from top level folder

How do I remove the dropdown menu here from Reference. It shouldn't be a collapsable menu.
My folder structure is /pages/help (renamed to Reference)/all other categories
I tried to also hide help instead but I can only rename it.

https://github.com/shuding/nextra/issues/2319#issuecomment-1734547874

@dimaMachina is there also a way so it doesnt highlight on hover?


fix: fix compatibility with latest WSL update

The latest WSL update makes /dev/shm a mount point, not a symlink, as it should be.
Unfortunately, this does not work well with us trying to remove it. Oops.

A quick "my system is broken pls help" guide:

Check out this PR
nix build .#staticShim
Copy result/bin/nixos-systemd-shim somewhere on the Windows file system
From a Windows shell, run wsl --system --user=root
mount -o remount,rw /mnt/wslg/distro
rm /mnt/wslg/distro/sbin/init
cp /mnt/.../nixos-systemd-shim /mnt/wslg/distro/sbin/init
Exit WSL and run wsl --shutdown
Restart NixOS on WSL
Rebuild your config with changes from this PR - otherwise the activation script will replace the shim with the old broken one.

Code looks good so far. Will this work on the older WSL version as well?

It should.


{Extension} Fix _run_pip

Description

Resolve #13706
_run_pip is also used by the azure-devops extension, make it backward-compatible.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

add to S170


Drop HDRSCAN and use cc -M -MF instead

The header-scanning system that the original Jam used, and which Ham partially (but not fully) implements to determine header dependencies, is quite outdated and often is one of the slowest steps for Jam to use. These days, most modern build systems let the C/C++ compiler report what headers a file used on last compilation directly, using the -M -MF flags to emit such lists. Somehow the general rules file and Ham should be modified to do this, too, to avoid rescanning headers manually (which often gets such dependencies incorrect leading to improper partial rebuilds.)

Thanks for this great tip! I'd like to adopt some kind of adoption mechanism for this, both for unexpected uses of HDRSCAN and for more exotic preprocessors that might not implement those flags (as that's a GNU flag as far as I can tell). I'll probably trigger this on the presence of a new variable, but I'll give it some thought. Thanks!

MSVC has an equivalent flag (but a different output format.) Basically all compilers support this, ninja requires it I think. (For that matter you might also look at just generating Ninja files to run actual builds with and let Ninja take care of this part too; it's much faster overall for rebuilds anyway.)

MSVC has an equivalent flag (but a different output format.) Basically all compilers support this, ninja requires it I think. (For that matter you might also look at just generating Ninja files to run actual builds with and let Ninja take care of this part too; it's much faster overall for rebuilds anyway.)

Huh, Ninja looks really interesting. It's already available on Haiku, so I'll definitely consider it for any future reworks. Using ninja looks like it would simplify a lot of the make phase and likely increase performance.

I've written Ninja generators from scratch before so feel free to shoot questions my way about it.

Err, how is this completed?

Err, how is this completed?

I had been planning since the beginning of the project to alter the architecture, but especially after your note about Ninja now seemed like the best time, as just last week all the Jam evaluation logic was completed, and it was just fixing bugs in the execution phase.
After talking with @stippi last week and running through a proof-of-concept/motivation, we decided to move to a new architecture so that Ham could have a better value proposition than "less buggy Jam". Namely, all the evaluation logic (what was worked on previously) will be moved over, but the custom parser, test suite, and make phase will be replaced. The motivation for everything, tools that will be used , and features that will be added are discussed in ADR #8-13 (located in docs/development/decisions on the v2 branch).
I'm just beginning to transition the project issues/docs, and so this issue was closed because part of the rework is using Ninja directly, so this change is obsolete. I'd love to talk with you more since you have experience creating Ninja build systems (that will be the last part of the refactor), so you can shoot me an email if you have time.

Personally I prefer using either GitHub or IRC/Matrix rather than email directly. Drop me a line whenever.


Add option to avoid _glfwPlatformPollEvents in glfwInit

It looks like the call to _glfwPlatformPollEvents was added with the helper window in commit 4cd493dd9a461e9c06f9cae1ac27d015d38a71f7.
I would like an option to avoid this call (or if it's not necessary remove it without an option). In my specific scenario I already have a GUI toolkit that has it's own Windows message loop. I'm simply using glfw for offscreen OpenGL for video post processing. The post processing effect is an optional piece of our software which is why I don't want to just call glfwInit on startup which would eliminate the problem.
The problem with _glfwPlatformPollEvents being called in glfwInit is that we may end up calling this while we have multiple video streams being received and displayed. While all of this is going on if there is heavy enough load then the Windows message queue may never be emptied and therefor glfwInit will never actually return until all of the messages are processed. I'd really rather there not be a message loop inside glfwInit, but if there must be it would be nice if it exited after it got the particular message it's looking for. In my case I don't see it ever doing anything in windowProc other than going to DefWindowProcW so I don't even think the poll is necessary at all, but I could be wrong.

This was fixed with 7420814fe20346d8e8b074ede9d77bc5546bb5fe, but I forgot to tag it.

This still isn't fixed.
_glfw.win32.helperWindowHandle will be NULL until createHelperWindow() returns.
So its passing a NULL handle to PeekMessageW and still processing foreign windows messages.

@nidefawl You are absolutely correct.

@nidefawl This has (hopefully) been fixed with 5292914ce19ed2dc6c6abfd591c4df18820a466a.


Poor performance and warning for GradientTape.jacobian with experimental_use_pfor=False

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Scientific Linux 7
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): conda
TensorFlow version (use command below): 2
Python version: 3.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
Extremely slow gradient calculation.
Also the following warning:
WARNING:tensorflow:From /home/grad3/cmich/miniconda3/envs/3.7/lib/python3.7/site-packages/tensorflow_core/python/ops/array_grad.py:502: _EagerTensorBase.cpu (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.identity instead.

Describe the expected behavior
There most be a much faster way of getting these gradients. After all the TensorFlow optimization must use these and those evaluations do not take so long.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
import time
import tensorflow as tf
import numpy as np

class Model(tf.keras.Model):
    def __init__(self,):
        super(Model, self).__init__()
        self.layers_list = [tf.keras.layers.Dense(2)] + \
            [tf.keras.layers.Dense(1000)] + \
            [tf.keras.layers.Dense(500)]

    def call(self, x):
        for layer in self.layers_list:
            x = layer(x)
        return x

model = Model()
model.build((1,2))
weights = model.trainable_variables

input = np.zeros([1,2])

with tf.GradientTape(persistent=True) as tape:
    ts = time.time()
    output = model(input)
    print(f'Forward took {time.time()-ts:.2f}s')

ts = time.time()
gradients = tape.jacobian(output, weights, experimental_use_pfor=False)
print(f'Backward took {time.time()-ts:.2f}s')

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
Output:
Forward took 0.01s
WARNING:tensorflow:From /home/grad3/cmich/miniconda3/envs/3.7/lib/python3.7/site-packages/tensorflow_core/python/ops/array_grad.py:502: _EagerTensorBase.cpu (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.identity instead.
Backward took 5.75s

The gradient calculation takes several orders of magnitude more time (for my real case the respective times are about 0.03s and 18s which is too much).
Is this the best way of doing this or this a faster way? Also should I care about the warning?

It looks like you are computing jacobian on an output of size 500. Without pfor, it would translate to 500 evaluations of gradient, so a slowdown of 500x is as expected. It looks like pfor vectorization provides a 5x speedup on top of an iterative strategy which also sounds in line with expectation. So I'd say that this is working as intended.
As an aside, If you plan on using a batch size larger than 1, I'd recommend batch_jacobian instead of jacobian.

If this is how slow it is supposed to be than I guess there's nothing to do. Unfortunately, I cannot use batch normalization for my case. I was just surprised it was so slow when the optimization/training in tensorflow is not this slow and it requires gradient calculation. But from what I've been reading it seems tensorflow never calculates the full jacobian when doing back-propagation/gradient descent.

@agarwal-ashish Can you PTAL? Thanks!

@cmichelenstrofer can you clarify the requirement ? Typically networks have a scalar loss at the top which makes the jacobian computation cheap. For non-scalar output, batch jacobian would be faster if you can use it (batch entries are independently transformed). If you compute full jacobian with non-scalar outputs, then yes, it will be exepected to be slow.

@agarwal-ashish The requirement is for non-scalar output and no batches. Very high-dimensional input and output spaces. And I need the Jacobian of all outputs w.r.t. all inputs (no batch Jacobian). I was under the impression that the full Jacobian needed to be computed as an intermediate step even when using a scalar output (a scalar output is after all one extra step in the forward model). This seems to not be the case and you use a numerical trick to avoid computing the large intermediate Jacobian (is this correct?). So counter-intuitively adding an extra step to your forward model makes the back-propagation cheaper. If this is indeed the case, it sounds like there is no way around the jacobian being so expensive.

Gradient functions are implemented as vector-jacobian products, and they often use tricks for more efficient computation which avoids creating jacobian matrices.
GradientTape.Jacobian still builds upon those tricks, but effectively it is now running many of the vector-jacobian computations, which it vectorizes for 5-10x speedups.


Fixing the null check for agreements

This PR fixes #14109

I was doing a null check on the container that was always created. Instead, a null check on the agreement object was needed here.

I was doing a null check on the container that was always created. Instead, a null check on the agreement object was needed here.

Pull Request Test Coverage Report for Build 521639964

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

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/resource-deployment/src/ui/toolsAndEulaSettingsPage.ts
0
1
0.0%

Totals

Change from base Build 519749688:
0.0%

Covered Lines:
24620

Relevant Lines:
52086

💛  - Coveralls

Pull Request Test Coverage Report for Build 521639964

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

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/resource-deployment/src/ui/toolsAndEulaSettingsPage.ts
0
1
0.0%

Totals

Change from base Build 519749688:
0.0%

Covered Lines:
24620

Relevant Lines:
52086

💛  - Coveralls


Remove debug prints preventing debug to load on 5.10

Some of the debug prints are not accepted by the 5.10 verifier, commenting them out so we can load the debug BPF programs if needed on older kernels:
Relates to: https://github.com/grafana/beyla/issues/1055

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.14%. Comparing base (ada2a72) to head (f9247b4).

:exclamation:  There is a different number of reports uploaded between BASE (ada2a72) and HEAD (f9247b4). Click for more details.
HEAD has 1 upload less than BASE

Flag
BASE (ada2a72)
HEAD (f9247b4)

k8s-integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1063       +/-   ##
===========================================
- Coverage   79.10%   68.14%   -10.96%
===========================================
  Files         139      139
  Lines       11297    11297
===========================================
- Hits         8936     7698     -1238
- Misses       1820     3087     +1267
+ Partials      541      512       -29

Flag
Coverage Δ

k8s-integration-test
?

oats-test
37.02% <ø> (ø)

unittests
52.22% <ø> (ø)

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

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


Clicking the duplicate button does nothing

In addition, the "edit ballot" feature doesn't work, either. No changes are ever saved.

This has been fixed. Thanks!


QnA Maker scoreThreshold validation does not allow 1

Versions
botbuilder-ai - v4.8.0
nodejs - v13.6.0
macOS Mojave - 10.14.6
Describe the bug
When calling QnAMaker.getAnswers() with { "scoreThreshold": 1 } the following validation error is thrown:
TypeError: \"1\" is an invalid scoreThreshold. QnAMakerOptions.scoreThreshold must have a value between 0 and 1."

The API docs allow for a score of 1 to be passed through to force exact matching but the validation in the SDK prevents this.
To Reproduce
Steps to reproduce the behavior:

Call QnAMaker.getAnswers() with { "scoreThreshold": 1 }

Expected behavior
A scoreThreshold of 1 should be valid.
Additional context
The validation code can be found here:
https://github.com/microsoft/botbuilder-js/blob/master/libraries/botbuilder-ai/src/qnamaker-utils/generateAnswerUtils.ts#L157
[bug]

Thank you for reporting @matthew-clark02 . Made a PR to fix the issue.


Readme misleading for android user manual link import com.zoontek.rnpermissions.RNPermissionsPackage

Hi Everyone,
I couldn't tell for sure were to add this issue under, but the Readme for manual linking is kinda misleading
as it is no longer
import com.zoontek.rnpermissions.RNPermissionsPackage;
but now
import com.reactnativecommunity.rnpermissions.RNPermissionsPackage;
I am not sure who this might help, but to who ever this is relevant to, I wish you best of luck and good luck

Also, I will love to add if you're facing the RNPermission null thing on Android, while working on a windows OS or any other OS it may pop up, I think it is simply best you manual link

@zico4reel Are you using v3 (current version, current README)? Did you clean your Gradle cache?
Because the namespace is correct: https://github.com/zoontek/react-native-permissions/blob/3.0.1/android/src/main/java/com/zoontek/rnpermissions/RNPermissionsPackage.java#L1

Hi, my apologies, I just checked, it is version 2.2.2, as it was installed as a dependency package.

@zico4reel Check the documentation for your version here: https://github.com/zoontek/react-native-permissions/tree/2.2.2


mouse scroll stops when cursor in polars output

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
using mouse wheel to scroll down. the scroll stops when the cursor goes to polars output.
VS Code Version
1.72.2
Jupyter Extension Version
2022.9.1202862440
Jupyter logs
(NA)

Coding Language and Runtime Version
No response
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
No response

What OS are you using?
Do you have a sample code cell that would produce the output that causes this? the only example on that github readme just displays a dataframe, does that cause the scrolling issue?

I'm using Windows.
Here is what I find:
There is a style that makes line-height to be 95% (see https://github.com/pola-rs/polars/pull/3691), and it seems vscode cannot handle that.
I created a sample notebook to help debug, and feel free to ask if it's not reproducible, thanks!
test.zip

thanks, I can repro with that notebook, although not very consistently


⚠ GitLab Server has degraded performance

In 02366e0, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11577 ms

Resolved: GitLab Server performance has improved in 51a1112.


CI: build versioned stable docs

tested here https://github.com/gtk-rs/gtk4-rs/runs/2600462595?check_suite_focus=true

cc @GuillaumeGomez another one here :)

Thanks!


Add toolbar to dashboard page

Description
Add toolbar to dashboard page

a before / after screenshot of the Dashboard Page would be nice.


feat: adding pre-commit linting

Looks good! :) Will also need to update pyproject.toml and add pre-commit as a dev dependency
I also think we should update template_content/README.md.jinja to include instructions on how to enable pre-commit (pre-commit install)

@aorumbayev

so the idea is that we rely on poetry to run pre-commit? What is the main advantage for not using the self-isolated pre-commit hooks that don't require reliance on existing repository?

The idea here is to leverage the local install and make sure the version stays in sync, this is what we do in algokit-cli and it has worked well so far.
Not all hooks would need to run poetry, these are system hooks so they can use whatever command.
In addition, this setup actually makes extra sense for a monorepo like the full-stack template, since there is no way with pre-commit to run a hook in a subdirectory, you would need to use system and cd into the say the backend subdirectory to run the commands in that location.


Workspaces - background color is green

Can we change the green background color?

I thought it would be this code
module=left,X,0xff8000ff,0xffffffff,1,workspaces
When I remove the line - I no longer see the workspaces
but if I change the colors nothing changes?
Can we change colors for active and non active workspaces?
+
Is it possible to click on the workspaces and then change to the workspace?

Is it possible to click on the workspaces and then change to the workspace?

No

OMG - sorry for that
Looked over it completely
and how can we change the font color of the active workspace?

done

I am probably too fast

based on last commit 371

you prolly arent on 371 my best guess

I have rebuilt the package and now it is alright.
Closing this topic


1.12.2 - Can't place blocks after installing your mod

Hey,
when using the mod on 1.12.2 i got the Problem that placing blocks is no longer possible, as they are erased out of inventory and not placed in the world, even though the placing sound is played. I confirmed deleting your mod stops this and playing with your mod alone has this effect while playing without any mods doesn't have it.

Hmm okay. No resting for 1.12.2 has happened yet. However, that is serious side effect so Ill look into it.

Well, as most mods just work on 1.12.2 i just expected yours to do so, too.
Then i had this big problem and needed to test every Single mod.
Am 26.09.2017 00:33 schrieb "Clayborn"<EMAIL_ADDRESS>
Hmm okay. No resting for 1.12.2 has happened yet. However, that is serious
side effect so Ill look into it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/orbwoi/UniversalRemote/issues/11#issuecomment-332033018,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYtVcFhi3nrX7MmGE09Q7RJYuR42JXuUks5smCofgaJpZM4PiFQK
.

I have it working in 1.12.2 - mojang changed some stuff with no functional change. Code cleanup? But fixing it broke 1.12.1 and 1.12.0. I am going to try to find another way to fix it that works with all 1.12 versions.

And on this topic (sorta), there are two other bugs likely caused by this: Forcing survival mode no matter what was selected on the world create screen, and a forge state engine corruption error.
I ended up on this mod by doing a binary search over my modlist.

1.12.2 is NOT supported right now.

I see. It's just unusal for mods to have such big Problems between Version, you might consider adding a warning to your curseforge page, if not already done. To be honest i just switched out my forge when the new Version came out and everything worked - despite this mod.

It wasnt supposed to be unfixed this long. Real life day job murdered me. Im trying to make improvements so patches are less likely to break things. Just realize this mod literally rewrites parts of forge and minecraft to make remote gui even possible.

My issue that I just tracked to this mod is simply that when I interact with a bed, I sleep but don't animate.  And when I go to the nether or back I get desync from server for blocks.
Very strange, but 1.12.2 does definitely seem to touch something that this mod depends on.

Finally found some time. Fixed in 1.12-0.9.5! :)


CIV-10782 Bulk print hearing

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-10782
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ x ] No

Dev Blocked


Add an error string when user is missing a name, prevents infinite spinner

Background
Fixes the blocker faced in OpenEdx LTI Integration, there the user does not have a name and the experience shows an un-ending Loading spinner on the screen.
@Lee - Would you please help with the appropriate string here? The case is that we would like the user to have a name to be shown on the screen. Without that, I've added an error instead for now.

Updating to provide error message if the user has no name
this.errorContent = { errorMsg: 'Please add a name for the user.', icon: 'BlockContact' };


[main] Upgrade to latest dependencies

release -psschwei
/cc knative-sandbox/client-writers
/assign knative-sandbox/client-writers
Produced by: knative-sandbox/knobots/actions/update-deps

/test integration-tests

Invalid. Fixed with https://github.com/knative/test-infra/pull/3696


Query string all

Description & motivation
Initially, I wanted to add an *all* flag to the remove query string feature.
When writing the tests, I discovered that the remove query string only removed question and ampersand query strings but not hash strings which may not technically be query strings but are sometimes used as such.
In fixing the regular expression, I ended up simplifying it and added tests to ensure that the new regex works as expected.
We may want to add the following to the release notes when pushing this live.

The query_parameter_exclusions variable now removes fragments, hash # parameters, from page_location and page_referrer. In most cases this will be an improvement in functionality. However, if you have query parameter keys using question mark, ?, or ampersand, &, that you want to remove and fragments that you don't want to remove, then you will need to override the remove_query_parameters macro.

Checklist

[y ] I have verified that these changes work locally
[y ] I have updated the README.md (if applicable)
[y ] I have added tests & descriptions to my models (and macros if applicable)
[y ] I have run dbt test and python -m pytest . to validate existing tests

Looks great. thanks for adding the unit test. I need to figure out how to get the dbt unit tests to run on PR updates, but I can do that separately. The unit tests passed.


Log home many files have been modified by usemin

I made a incremental deployment. But everty .html file will be transmit, even they acctually haven't been modified.
So I done this, the .html file haven't been changed will not be transmit.

@snasc can you add a test ?

ping @snasc
Your PR is only about adding some log, - the number of files modified and not the list of file modified -, and (still) missing a test.
I don't see the interest of adding only a log output, but I might be wrong, but unless someone vote for this PR, I will close it.


feat: --send-all option

fix formatting and typos (and refactored text in docs/testnet11.md) 667ffb304539bdc5bfc47123d62b6f782f8b5a11

added --send-all option to send command, allowing users to send their entire mature balance minus an optional priority fee
retains feature parity between Go and Rust implementations (the Go wallet has --send-all) 07fd91b527643e39dd5c4c6971f9fe6fac0eb51e

@KaffinPX this being --send-all is probably to match what it used to be in the kaspad go wallet: https://github.com/kaspanet/kaspad/blob/6085d1fc84d71f74a571fad2b7e5b38413a3db5a/cmd/kaspawallet/config.go#L67

@KaffinPX this being --send-all is probably to match what it used to be in the kaspad go wallet: https://github.com/kaspanet/kaspad/blob/6085d1fc84d71f74a571fad2b7e5b38413a3db5a/cmd/kaspawallet/config.go#L67

Yes exactly! It was a missing feature in the rust wallet so I submitted this PR to maintain feature parity.


Adding ARIA Hidden True to Font Awesome Icons

"Modern versions of assistive technology, like screen readers, will read CSS generated content (how Font Awesome icons are rendered), as well as specific Unicode characters. When reading our default markup for rendering icons, assistive technology may have the following problems."

The assistive technology may not find any content to read out to a user
The assistive technology may read the unicode equivalent, which could not match up to what the icon means in context, or worse is just plain confusing
Many dyslexic people find it helpful to swap out a website’s typeface for something like OpenDyslexic. But icon fonts get replaced as well, which makes for a frustratingly broken experience.

http://fontawesome.io/accessibility/
As a first step to providing better accessibility, we can "hide" the font icons from the accessibility tree.
aria-hidden="true"

Here is the output from bgrep : https://gist.github.com/cam156/200e9c0869ed1c4246f272ac50e3e357

So only update the views in ScholarSphere.  Ignore the ones in the other gems for now.

Need to replace <i> with <span> and add `aria-hidden="true" to both glyphicons and fa icons.


guarantees that win_size won't be nil (#405)

in some cases win_size is nil, which makes the round thrown an error
fixes #405

Thanks for the PR!


PLZ-743 Automatically refresh auth token

With this change we now store a refresh token along with our access token. When
the access token is soon to expire, we will fetch a new one by exchanging our
refresh token. This uses the api.plz.review/auth/github/device/refresh endpoint
added in plz.review/review/4818.
plz-review-url: https://plz.review/review/4821

2 new comment(s) @ https://www.plz.review/review/4821


NuGet package for Microsoft.ServiceFabric.Services.Remoting version 3.3.624 is no longer available.

Any official answer as to why the NuGet package for Microsoft.ServiceFabric.Services.Remoting version 3.3.624 is no longer available? 3.3.622 is now the latest.

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

ID: 8902374b-b631-9d50-16ef-1899ea2dd29d
Version Independent ID: 55351e69-aa2a-901c-a563-106b62e3d6ac
Content: Service remoting by using C# in Service Fabric
Content Source: articles/service-fabric/service-fabric-reliable-services-communication-remoting.md
Service: service-fabric
GitHub Login: @vturecek
Microsoft Alias: vturecek

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

@DivideBySara looks like it was removed by the owner...

@vturecek @aljo-microsoft @dkkapur @masnider any idea why/ the details on this?

According to the release notes, the Runtime release are version x.x.624 and x.x.622, Seems that the .624 does not need new packages, the release details mentions only bugs fixes on windows, the new SDK version release is 3.3.622.9590.
Probably they released the .624 by mistake, as there is no difference between both and it would not be compatible with older version (.623).

It seems you removed a working version and now the nuget version is 3.3.638 for Microsoft.ServiceFabric.Actors, but I don't see a download for that version for the runtime.   What are the correct versions and where should I get them from?

Thanks for the patience while I looked into this.
A new version of nuget package has been published https://www.nuget.org/packages/Microsoft.ServiceFabric.Services/3.3.638
Issue explanation https://github.com/Microsoft/service-fabric-services-and-actors-dotnet/issues/166

Also, for any other issues related to nuget packages you can open them here:
https://github.com/Microsoft/service-fabric-services-and-actors-dotnet
That way the engineering team sees them directly and can look into it right away.


Considered a virus by windows defender [BUG]

Not sure if this is the forum for it since it's not really a 'bug'.
Anyhow, windows defender detected and deleted the x64 msi installer(QR-Code-Reader-1.1.2-x64.msi) as soon as I downloaded it.
These are the details provided by windows defender.

Unfortunately, there is nothing I can do about Windows Defender except send a false positive report to the Microsoft Security Intellegence Team.
However, Virustotal is not complaining. https://www.virustotal.com/gui/file/e8e2d43277b675a534bf9e920b1daf4e67b691ef05391a87bb687f3071111ea0?nocache=1
Try switching windows Defender off temporarily. Sorry for the inconvenience.

@ottozumkeller I can confirm I am having the same issue, when trying to update to 1.1.2 through WinGet.
Hopefully MS resolves the issue sometime soon.
It worked fine when I initially installed 1.1.0 (though I'm not sure I used WinGet back then. Doubt it matters.).

While using the x86 build is a fine workaround, it doesn't really solve the issue, and running non-64-bit isn't ideal.
I know the Minecraft Forge team has had similar issues with their installer in the past, and reporting the false positive to MS eventually does result in them removing it.

While using the x86 build is a fine workaround, it doesn't really solve the issue, and running non-64-bit isn't ideal. I know the Minecraft Forge team has had similar issues with their installer in the past, and reporting the false positive to MS eventually does result in them removing it.

Then why doesn't the developer report to microsoft for false positive ?

I've actually already done that several times. Unfortunately, false positives are a known problem with many open source projects. Unfortunately there is nothing I can do except wait until or if MS adjusts the detection algorithms.

running non-64-bit isn't ideal.

Why it isn't ideal ? I want to know about it. Please tell me

Realistically, it probably won't make any real difference for this particular app, but other applications will suffer performance hits from doing that.
And 32 bit programs are limited to using (if I recall correctly) a maximum of 4GB RAM.

connect.exe is considered malicious by TrendMicro as well:
Infected file: connect.exe File path: C:\Program Files (x86)\QR-Code Reader
TROJ_GEN.R002C0XCR24


Webhooks are not fire the last attempt from ATTEMPT_SCHEDULE

The problem
ATTEMPT_SCHEDULE list has attempts with time delays it should be fired:
https://github.com/bullet-train-co/bullet_train-outgoing_webhooks/blob/ba6fc92326e9e36ac6eb481baac416881ddc8941/bullet_train-outgoing_webhooks/app/models/concerns/webhooks/outgoing/delivery_support.rb#L12-L18
The last attempt is not actually taking and not sent.
Possible solution
Change still_attempting? logic and set "less or equal" instead of just "less" in the condition: attempt_count <= max_attempts
The original method:
https://github.com/bullet-train-co/bullet_train-outgoing_webhooks/blob/ba6fc92326e9e36ac6eb481baac416881ddc8941/bullet_train-outgoing_webhooks/app/models/concerns/webhooks/outgoing/delivery_support.rb#L50-L53

Moved to core repo https://github.com/bullet-train-co/bullet_train-core/issues/921


OCPBUGS-36014: CVE-2024-6104: bump github.com/hashicorp/go-retryablehttp to v0.7.7

cc @openshift/storage

/retest

/retest

Closing - not shipped in 4.16 and newer.


Trying a new framework approach with nested with

I tried to do such and such
@mahdirajabi96 please review the enter/exit syntax
@kimkolody How does this look?

@kimkolody did you have the chance to review?


test(api): navigation menu permissions

:books: Summary

:link: Links / References

#409

:construction_worker: Tasks

[ ] :firecracker: Contains breaking-change Any Breaking change(s)?
Breaking Change must also be notated in the commit that introduces it and in Conventional Commit Format.

[ ] :notebook: Release notes updated

[x] :blue_book: Documentation written
All features to be documented within the correct section(s). Administration, Development and/or User

[ ] :checkered_flag: Milestone assigned

[x] :gear: :test_tube: Functional Test(s) Written

[ ] :test_tube: Unit Test(s) Written
ensure test coverage delta is not less than zero

[ ] :page_facing_up: Roadmap updated

Python Scan Results (requirements.txt)
Scan Date: 2024-12-09T13:57:24.040954344Z File: requirements.txt
High and Critical Vulnerabilities
Vulnerability ID	Package	Version	Severity	Description
CVE-2024-53908	Django	5.1.2	HIGH	django: Potential SQL injection in HasKey(lhs, rhs) on Oracle

fixed in c03b7e7d49a866a313d5586ad72a252d63d1df17


JavaModule's run command arguments should be a Task

Currently, we can't define a def a: Task[String] = ??? and use that to call the run command from the JavaModule. See the minimal repo:
import mill._
import mill.scalalib._

object proj extends JavaModule {

/*

/private/tmp/def/build.sc:10: Target#apply() call cannot use `value args` defined within the T{...} block
    run(args)()
        ^
/private/tmp/def/build.sc:8: `T.sources` definitions must have 0 parameter lists
  def aCommand() = T.command {
      ^
Compilation Failed
*/
  def myArgs: T[String] = T { "arg" }

  def aCommand() = T.command {
    val args = myArgs()
    run(args)()
  }
}

As per a conversation on gitter with @lefou , a straightforward fix for that is to turn def run(args: String*): Command[Unit] into def run(args: Task[Seq[String]]): Command[Unit]

See also https://github.com/com-lihaoyi/mill/discussions/1857

I tried a prototype in https://github.com/com-lihaoyi/mill/pull/2452. Thinking about this, it appears there are a few options here:

def run(args: Task[String]*): That should work already I think, but it limits the number way arguments can be passed to run since the number of arguments is fixed up front

def run(args: Task[String*]) is not allowed by the Scala type system

def run(args: Task[mainargs.Leftover[String]]) would require some patches to the mainargs library, because currently we hard-code support for mainargs.Leftover[T]. We'd need to un-hard-code it somehow to make the Task[Leftover[String]] param signature work, and then maybe provide an implicit constructor so people can pass in run(Seq("foo")) rather than run(mainargs.Leftover(Seq("foo"))) at the callsite

Leave def run(args: String*) and just make it forward to an anonymous task def run(args: Task[Seq[String]]) = T.task{...}. This is probably the easiest thing to do to solve the immediate issue without major surgery, but doesn't generalize to other T.commands unless we repeat the boilerplate of T.command/T.task pairing for each one

It feels like either (3) or (4) are the best options here, depending on how much we want to generalize this v.s. how difficult it is to make the upstream mainargs changes necessary to support (3)

Beside what this means for the codebase, I think what we want is a args: Task[Something]. For different reasons.

Something can be a Seq[T] to support variable count
Something can be a case class or the like, which is easier to evolve in a binary compatible way

The JavaModule.run is a good example for a Seq-like parameter.
If we look at a more complex use case like JavaModule.ivyDepsTree, we see currently the following signature:
def ivyDepsTree(args: IvyDepsTreeArgs = IvyDepsTreeArgs()): Command[Unit]

with IvyDepsTreeArgs, which is annotated with various mainargs annotation to be parseable.

https://github.com/com-lihaoyi/mainargs/pull/62 has the mainargs necessary to make (3) work. It would be a binary compatibility breakage for Mainargs, but IMO it's probably worth doing sooner or later, as apart from supporting this feature it would also clean up a lot of cruft within the MainArgs codebase


Sandpack doesn't handle web components out of the box

Bug report
Packages affected

[x] sandpack-client
[ ] sandpack-react

Description of the problem
Creating a sandpack client with a template that uses web components works fine.
When running the client.update() method, we get the following issue:
Failed to execute 'define' on 'CustomElementRegistry': the name "new-el" has already been used with this registry

What steps can we take to reproduce the problem?
I've created a minimal repro case here: https://github.com/andrico1234/sandpack-wc-repro
The instructions on how to run it are in the README.
The expected behaviour would be to either:

support web components out of the box
provide a documented way for how to avoid the issue

Thanks all!
Your Environment

Software
Name/Version

Sandpack-client version
2.6.9

Browser
Chrome

Operating System
MacOS

Update: I managed to get it working. This was due to two issues:

the setup object for loadSandpackClient should have had template: node set
I did not pass a startRoute in the client options. this prevented the site from loading. setting it to "/index.html" solved the problem

I'm not sure if this requires some documentation update, or whether to automatically infer the startRoute if an index.html file is present in the files object. Happy to open a PR, based on the team's thoughts.


Wrapping completions and chat/completions endpoint

What does this PR do?
This PR implements a comparability endpoint, by wrapping generate and generate_stream in server.rs. It does so by minimally modify existing files, packing the required changes into a mod completion.rs making it as modular as possible.
I guess there is a couple of applications who want to post exactly the following request, which is hard-coded in several ways in some python applications (e.g. Langchain, AutoGPT, stuff that is already legacy).
For background: Also a lot of issues were created, up until recently, e.g. https://github.com/huggingface/text-generation-inference/issues/497 https://github.com/huggingface/text-generation-inference/issues/735
Note: I recently (last week) learn RUST, and ist quite a leap for a Python dev - any constructive feedback is welcome.
Post:/completions
{
  "max_tokens": 20,
  "messages": [
    {
      "content": "What is the capital of Bavaria?",
      "role": "assistant"
    }
  ]
}

And get an answer in this format:
{"id":"chatcmpl-12345","object":"chat.completion","created":1690902873,"model":"gpt","choices":[{"index":0,"finish_reason":"stop","message":{"role":"assistant","content":"The capital of Bavaria is Munich (München in German)."}}],"usage":{"completion_tokens":13,"prompt_tokens":16,"total_tokens":29}}

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR.

Current state.

[ ] chat/completion:

[x] non-stream working
- [ x] compliant
- [  ] tested
[ ] stream working
- [ ] tested

[ ] completion:

[x] non-stream working
- [ x] compliant
- [  ] tested
[ ] stream working
- [  ] tested

Chat/completion stream

[ ] model name correct
[ ] fixed id and timestamp -> both are not non-fixed and depend on the streaming time.
[ ] special event: start with deltaMessage {"role":"assistant","content":" ""}
[ ] special event: Done Token
Stream as expected for protocol.

data: {"id":"chatcmpl-7ikx6GJPtSN7lcmdRHgj22hSGPCze","object":"chat.completion.chunk","created":1690901272,"model":"gpt","choices":[{"index":0,"finish_reason":null,"delta":{"role":"assistant"}}],"usage":null}

data: {"id":"chatcmpl-7ikx6GJPtSN7lcmdRHgj22hSGPCze","object":"chat.completion.chunk","created":1690901272,"model":"gpt","choices":[{"index":0,"finish_reason":null,"delta":{"content":")."}}],"usage":null}

data: {"id":"chatcmpl-7ikx6GJPtSN7lcmdRHgj22hSGPCze","object":"chat.completion.chunk","created":1690901272,"model":"gpt","choices":[{"index":0,"finish_reason":"stop","delta":{}}],"usage":null}

data: [DONE]

Current state:
data:{"id":"id-1690901237","object":"text_completion","created":1690901237,"model":"TGImodel","choices":[{"delta":{"role":"assistant","content":" smile"},"finish_reason":null,"index":0}]}

data:{"id":"id-1690901237","object":"text_completion","created":1690901237,"model":"TGImodel","choices":[{"delta":{"role":"assistant","content":"."},"finish_reason":null,"index":0}]}

data:{"id":"id-1690901237","object":"text_completion","created":1690901237,"model":"TGImodel","choices":[{"delta":{"role":"assistant","content":" \""},"finish_reason":"length","index":0}]}

I think the chat API is critical to avoid prompt injection. I just want to understand this PR better on how to let the server add special token while API callers cannot.

Changes are addressed now, stream and non stream are both working.

Could you run rustfmt?

Also I have concern about the complexity of _generate_stream_typed. Basically this PR is trying to implement 3 different endpoints in a single _generate_stream_typed function, making it complicated. What is worse is that if in the future OpenAI adds more features to their streaming API and we want to also adopt the same features, we will have to add more code into _generate_stream_typed.
I think a more sustainable way is to implement different streaming protocols in different functions, and reuse fine-grained code pieces as self-contained functions, instead of trying to reuse the whole function.

I would argue, that the internal protobuf protocol is more likley to change, that the openai's stable external streaming api. It is therefore important to wrap the internal generate_stream as much as possible, enable code-reuse, and do the conversion at the latest point in time.
It is not possible to wrap the fn generate_stream function as a whole.

It is not possible to wrap the fn generate_stream function as a whole.

Sorry I didn't propose to wrap generate_stream. Instead, I proposed the reverse approach: don't reuse generate_stream as a whole.
My suggestion was to reuse fine-grained code pieces as self-contained functions if any.

I will merge this PR if we could avoid patching to original generate_stream.
To be honest, the consideration here is that Preemo (or other contributors) are expected to change generate_stream in the future and we don't want to add complexity to generate_stream and the risk to break the patch function.
Duplication is relatively tolerable. Even if we want to deduplicate similar code, I would extract common code in a utility module, instead of patching existing implementation.

Sorry for messing up history.
Commits are not squashed, changes are addressed.
Docker image is at:
docker pull docker.io/michaelf34/tgi:0.9.4-completion_endpoint

Hello, is OpenAI API Wrapper now available? Can you give me an example? Thank you.

I can merge it any way, but I cannot test this PR in our internal CI because our internal implementation is already partially refactored for modularization as we promised in README.
Once the internal version is ready to publish to this open-source repository, I expect that changes like this PR will need to migrate to new architecture.


txn: Move tag and cmd_metric to each command

Signed-off-by: longfangsong<EMAIL_ADDRESS>
What problem does this PR solve?
Minor refactor in src/txn/commands.rs
What is changed and how it works?
What's Changed:
Move tag and cmd_metric to each command and abstract them with a CommandExt trait.
Check List
Tests

Unit test

@nrc PTAL

/merge


Fix conversation scheduling

Currently, we are not sending the conversation_key to the task for bulk_send and dialogues.

Ready for review.

:+1: when Travis is happy.

Test failures look unrelated. Looking into why it's failing on travis.

:+1:
I'll open a new issue for the Moto failures.


Airflow HTTPD Proxy development and configuration

Dependency: CS setup HTTPD proxy service on each target venue
Must proxy both the Airflow UI and the Airflow API (which are at the same base URL).
It will require consulting the Airflow documentation since Airflow assumes all URLs are served by a root URL,.
Instructions/documentation from CS:

https://app.gitbook.com/o/xZRqGQeQXJ0RP4VMj7Lq/s/UMIRhLdbRQTvMWop8Il9/developer-docs/common-services/docs/developers-guide/httpd-server-deployment
https://github.com/unity-sds/unity-proxy

To integrate with the API Gateway, we must create a new resource in the venue api gateway:

The venue API Gateway should have the naming convention of unity-{project}-{venue}-rest-api-gateway.
a resource at the root of the API gateway must be created titled ogc (discussion, should this be "processing" ?)
after that resource is created, we want to create another , proxy resources as a child of the ogc/path. It should essentially be ogc/{proxy+} .
The proxy resource should point to the ogc load balancer (preferably over HTTPS), and should include the proxy+ content in the request to the OGC ALB:

Had to poke at it a little bit, as Airflow unfortunately decided to use absolute pathing on its Location headers.
<Location "/sps/">
    ProxyPassReverse "/"
</Location>
<LocationMatch "^/sps/(.*)$">
    ProxyPassMatch "http://k8s-airflow-airflowi-8261b72237-194646091.us-west-2.elb.amazonaws.com:5000/$1"
    ProxyPreserveHost On
    FallbackResource /management/index.html
    AddOutputFilterByType INFLATE;SUBSTITUTE;DEFLATE text/html
    Substitute "s|\"/(?!sps/)([^\"]+)|\"/sps/$1|q"
</LocationMatch>

The Substitute call, above seems to work fine so far (no issues visible in the UI), but it'll only match a URL in quotes, which in my opinion could miss a edge case or two, but I haven't stumbled on one.

In this case, what is the URL that is proxied to http://k8s-airflow-airflowi-8261b72237-194646091.us-west-2.elb.amazonaws.com:5000/$1?

with the above configuration, the /sps/ path on the unity-proxy endpoint (currently, the unity-venue-dev "main" instance http://unity-dev-httpd-alb-285256043.us-west-2.elb.amazonaws.com:8080/sps/)
But that /sps/ path isn't set in stone, just picked during testing.

Note: the following links don't seem to be working:
o Clicking on the Airflow logo on the upper left
o Submitting a job from the UI (the. job gets submitted, but there is a problem displaying the confirmation page)

Hmmm. That first link is a charming link to /
<a class="navbar-brand" rel="home" href="/" data-original-title="" title="">

I might be able to fiddle the regex to catch that.
The second issue is an odd redirect to /home, which seems to double dip in picking up /sps prefixes (becomes /sps/sps/home). I'll take a look at that.

Should be fixed by #170


fix switching of entrypoint

Codecov Report

Merging #564 (4911f91) into main (efc7f0b) will increase coverage by 0.41%.
The diff coverage is 100.00%.

:exclamation: Current head 4911f91 differs from pull request most recent head 6602ce6. Consider uploading reports for the commit 6602ce6 to get more accurate results

@@            Coverage Diff             @@
##             main     #564      +/-   ##
==========================================
+ Coverage   81.64%   82.06%   +0.41%
==========================================
  Files          31       31
  Lines         752      747       -5
  Branches      138      134       -4
==========================================
- Hits          614      613       -1
+ Partials      138      134       -4

Impacted Files
Coverage Δ

platform/src/kernel/component/Scoped.jsx
72.00% <ø> (+3.42%)
:arrow_up:

platform/src/component/Module.jsx
77.27% <100.00%> (+1.08%)
:arrow_up:

platform/src/kernel/reducer/shared.js
82.69% <100.00%> (+0.33%)
:arrow_up:

platform/src/kernel/registry/loader.js
79.54% <100.00%> (+0.72%)
:arrow_up:

Continue to review full report at Codecov.

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


LTSB not supported list should grow

The LTSB not supported list should ideally grow, noting that no cloud services are supported. Would be nice to continue to add things like CMG, Desktop Analytics, Azure AD authentication, Windows Store for Business, etc. to clarify that the list of unsupported features will continue to increase.

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

ID: 8b4e6c18-babd-5009-0af4-786fa91e3d5a
Version Independent ID: b42f0c87-a277-185c-1b61-ec20f1367a40
Content: Introduction to the Long-Term Servicing Branch - Configuration Manager
Content Source: sccm/core/understand/introduction-to-the-ltsb.md
Product: configuration-manager
Technology: configmgr-other
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @MSFTJerryAb, good feedback 😄

I revised this article, it will publish live on Monday around 11am Pacific


As a Ext Operator, I want to have a Submission ID for my application, so that I have a reference until RFP ID is assigned

Description:
Capturing the work needed to be done for having a 'Submission ID' in place for the CIF External App:
"The verdict of the RFP ID was to have a temporary 'Submission ID' created when proponent creates an application. The CIF team can then assign an RFP when they review it manually, and the new ID will reflect in the system then. Some outstanding question I have are 1) how should the 'Submission ID' be displayed to the Internal team? and 2) What should the format of the temporary 'Submission ID' look like?"
Acceptance Criteria:
Given
When
Then
Development Checklist:

[ ] Checklist item
[ ] Checklist item
[ ] Checklist item
[ ] Meets the DOD

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

Flagging this for the business area - the team needs to reach out to find out an appropriate format for a submission ID


chore(package): update @kadira/storybook to version 2.17.0

https://greenkeeper.io/

This pull request is now managed with CodeReviewHub. Refresh.


Issue specifying a Reader

I'm following the examples of reading a document into the tool shown here: http://chemdataextractor.org/docs/reading
I am using version 1.2.2 installed using the conda install on linux and working with Python 3.5, I'm testing from a Jupyter Notebook.
If I use the first example on that page to read a locally stored html file, then I successfully read in the HTML and can query the data stored in the doc variable.
If I then try and explicitly call the correct reader RscHtmlReader() I get an error stating that the name is not defined, and the same if I try to specify the AcsHtmlReader().
`NameError                                 Traceback (most recent call last)
 in ()
1 f = open(//Downloads//chemdataextractor-journal-articles-source//evaluation journal articles//articles//rsc.nj.c5nj01594d.html', 'rb')
----> 2 doc = Document.from_file(f, readers=[AcsHtmlReader()])
NameError: name 'AcsHtmlReader' is not defined
`
I was also able to use the command line interface to generate an output json file, I'm not sure if these are generated by successfully calling the correct reader or if the application is just falling back to to the generic html reader.
An ideas?

Sorry, I left out the necessary imports from the documentation. Before you use the readers, you will need:
from chemdataextractor.reader import AcsHtmlReader, RscHtmlReader

Ok I tried that and it worked. Thanks. I did try using
from chemdataextractor import *
but hadn't thought to go for a more specific import.


Update node operator guide to reference simple-celo-node

The node operator docs have become much simpler, because the simple-celo-node
project is now encapsulating a lot of that complexity.

Great to see it's so much shorter now!
Questions:

is EigenDA fully covered? It's not mentioned any more, nor in https://github.com/celo-org/simple-celo-node
what about historical state & execution? the previous setup required running a proxy to an L1, what's the new approach?

Things I'd add:

"node operators must upgrade their existing L1 nodes to the latest release" is too vague -> point to a specific one for Alfajores and Baklava

"node operators must upgrade their existing L1 nodes to the latest release" is too vague -> point to a specific one for Alfajores and Baklava

"node operators must upgrade their existing L1 nodes to the latest release" is too vague -> point to a specific one for Alfajores and Baklava

@lvpeschke We don't actually have those releases yet. So I'd suggest merging this and later we can make a release of celo-blockchain and then update the docs.

Ack. But then I'd change the wording to make it clear that specific releases are needed and that the info is coming (like the mainnet migration block) instead of referring to the latest one.


./insmod.sh fails

Dear,
We have several GPU nodes (Skylake processors with 4x P100 cards per each node), and I would like to test if the RDMA is available on these nodes or not.
When I try to build the gdrcopy, I get the following error message:
mknod: ‘/dev/gdrdrv’: Operation not permitted
Here is the specification of the host:
$> uname -a Linux r23g34 3.10.0-693.21.1.el7.x86_64 #1 SMP Wed Mar 7 19:03:37 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
In fact, there is not such a file at /dev/gdrdrv on our current system. Do you have an idea what is wrong here?
Thanks
Ehsan

Ehsan, insmod.sh requires that the user issuing the command have sudo privileges.

I definitely have a root permission. Let me copy-paste what I get when running "make":
gdrcopy$> sudo ./build.sh &> log.txt
And the tail of the log.txt reads:
make PREFIX=/vsc-hard-mounts/leuven-data/sys/x0090231/easybuild/work/gdrcopy/install CUDA=/apps/leuven/skylake/2018a/software/CUDA/9.1.85 all install echo "GDRAPI_ARCH=X86" GDRAPI_ARCH=X86 cc -O2 -fPIC -I /apps/leuven/skylake/2018a/software/CUDA/9.1.85/include -I gdrdrv/ -I /apps/leuven/skylake/2018a/software/CUDA/9.1.85/include -D GDRAPI_ARCH=X86  -c -o gdrapi.o gdrapi.c cc -O2 -fPIC -I /apps/leuven/skylake/2018a/software/CUDA/9.1.85/include -I gdrdrv/ -I /apps/leuven/skylake/2018a/software/CUDA/9.1.85/include -D GDRAPI_ARCH=X86  -c -mavx -o memcpy_avx.o memcpy_avx.c cc -O2 -fPIC -I /apps/leuven/skylake/2018a/software/CUDA/9.1.85/include -I gdrdrv/ -I /apps/leuven/skylake/2018a/software/CUDA/9.1.85/include -D GDRAPI_ARCH=X86  -c -msse -o memcpy_sse.o memcpy_sse.c cc -O2 -fPIC -I /apps/leuven/skylake/2018a/software/CUDA/9.1.85/include -I gdrdrv/ -I /apps/leuven/skylake/2018a/software/CUDA/9.1.85/include -D GDRAPI_ARCH=X86  -c -msse4.1 -o memcpy_sse41.o memcpy_sse41.c cc -shared -Wl,-soname,libgdrapi.so.1 -o libgdrapi.so.1.2 gdrapi.o memcpy_avx.o memcpy_sse.o memcpy_sse41.o ldconfig -n /vsc-hard-mounts/leuven-data/sys/x0090231/easybuild/work/gdrcopy/gdrcopy ln -sf libgdrapi.so.1.2 libgdrapi.so.1 ln -sf libgdrapi.so.1 libgdrapi.so cd gdrdrv; \ make find: ‘/usr/src/nvidia-*’: No such file or directory dirname: missing operand Try 'dirname --help' for more information. make[1]: Entering directory /vsc-hard-mounts/leuven-data/sys/x0090231/easybuild/work/gdrcopy/gdrcopy/gdrdrv'
Picking NVIDIA driver sources from NVIDIA_SRC_DIR=NVIDIA_DRIVER_MISSING. If that does not meet your expectation, you might have a stale driver still around and that might cause problems.
make[2]: Entering directory /usr/src/kernels/3.10.0-693.17.1.el7.x86_64' find: ‘/usr/src/nvidia-*’: No such file or directory dirname: missing operand Try 'dirname --help' for more information.   CC [M]  /vsc-hard-mounts/leuven-data/sys/x0090231/easybuild/work/gdrcopy/gdrcopy/gdrdrv/nv-p2p-dummy.o /vsc-hard-mounts/leuven-data/sys/x0090231/easybuild/work/gdrcopy/gdrcopy/gdrdrv/nv-p2p-dummy.c:48:20: fatal error: nv-p2p.h: No such file or directory  #include "nv-p2p.h"                     ^ compilation terminated. make[3]: *** [/vsc-hard-mounts/leuven-data/sys/x0090231/easybuild/work/gdrcopy/gdrcopy/gdrdrv/nv-p2p-dummy.o] Error 1 make[2]: *** [_module_/vsc-hard-mounts/leuven-data/sys/x0090231/easybuild/work/gdrcopy/gdrcopy/gdrdrv] Error 2 make[2]: Leaving directory /usr/src/kernels/3.10.0-693.17.1.el7.x86_64'
make[1]: *** [module] Error 2
make[1]: Leaving directory /vsc-hard-mounts/leuven-data/sys/x0090231/easybuild/work/gdrcopy/gdrcopy/gdrdrv' make: *** [driver] Error 2
I am building against CUDA/9.1.85.

I made some progress with the previous errors, and now, I get a new error:
insmod: ERROR: could not insert module gdrdrv/gdrdrv.ko: Unknown symbol in module

Hard to tell.
Are you building and installing on the same machine?
There should be a detailed error in the kernel log. You could use 'dmesg' to dump that log and copy the relevant lines here.

Alright ... I'm coming back to this ticket, because I need gdrcopy for a CUDA-aware OpenMPI. I am attaching the redirected stderr/stdout from building gdrcopy in here, together with the very simple build script I am using.
In brief, I have two complains now, one about NVIDIA_SRC_DIR, and the other about CONFIG_RETPOLINE during the "make" step. In fact, I am not sure how to set these, so that they propagate properly to the make.
Furthermore, I need to know what is expected to be inside NVIDIA_SRC_DIR?
What do you see on your platform?
gdrcopy.zip

I would like to attract your attention to this ticket. In fact, my installation of CUDA-aware MPI is pending on compiling gdrcopy. Could you please take a look at my error logs, and also the questions I raised above?
Thanks a lot.
E.

Ehsan,
thank you for trying gdrcopy.
The excerpt from your build log, copied below, is clear enough:

NVIDIA_SRC_DIR is auto set based on your local install dir of the GPU driver
CONFIG_RETPOLINE is apparently not supported by your host compiler. I am not an expert, but I don't believe you are supposed to tweak the compiler command line for a kernel module. Either your Linux kernel automatically detects and enables retpoline or not.

make[1]: Entering directory `/easybuild/work/gdrcopy/gdrcopy/gdrdrv'
Picking NVIDIA driver sources from NVIDIA_SRC_DIR=/usr/src/nvidia-418.40.04/nvidia. If that does not meet your expectation, you might have a stale driver still around and that might cause problems.
make[2]: Entering directory `/usr/src/kernels/3.10.0-957.10.1.el7.x86_64'
arch/x86/Makefile:166: *** CONFIG_RETPOLINE=y, but not supported by the compiler. Compiler update recommended..  Stop.
make[2]: Leaving directory `/usr/src/kernels/3.10.0-957.10.1.el7.x86_64'
make[1]: *** [module] Error 2
make[1]: Leaving directory `/easybuild/work/gdrcopy/gdrcopy/gdrdrv'
make: *** [driver] Error 2

Thanks Davide for your message; it brought some activity back to this ticket.
My problem is, whether or not I set the two env vars NVIDIA_SRC_DIR and/or CONFIG_RETPOLINE, my build always crashes at the same location, and throws the same error message. That made me wonder I am not doing it right.
Do you have any idea why my build crashes? And how to resolve this?

That kernel module build error is discussed on the net, e.g. on RH/CentOS forums/bugzilla.
For example see https://bugzilla.redhat.com/show_bug.cgi?id=1566297#c12

Thanis Davide for the hint.  However, I purge the GCC module, and stick to the system gcc and it builds flawlessly.
I still cannot comprehend why gdrcopy builds with an older GCC rather than a newer one!

It looks like retpoline support is in gcc 7.3 or 8.x.
Most probably RH backported retpoline support onto gcc 4.8.5.
closing as this is a local customer server issue

dear ,  how dou you fix the problem "insmod: ERROR: could not insert module gdrdrv/gdrdrv.ko: Unknown symbol in module" ?  i

Hi @zhuanwancaishi ,
There are multiple possibilities:

Was nvidia driver (nvidia.ko) loaded before you tried insmod.sh?
When you compiled gdrdrv, there should be a message printed out. Did it pick the correct nvidia driver and the linux kernel version you are running?


TS implementation suggestion for #23

This one could do #23 in TS part. I really need some approve that I got things correctly before moving on to Circom part.
Also pay attention to the note in the initial commit for this issue, pls. I suspect that Rust implementation actually should panic when SEC1 decoding gives number greq then the field order.
So the inconsistency might be much deeper.

wrapping around field instead of curve order
difference in decoding of the hash into an element

[ ] think about breaking the interface with switching to bigint: remove excessive conversions, closer to the Rust implementation which initiated this issue, interface isn't stable anyway, but I'd double check this switch doesn't introduce potential exploitation

It's entirely possible this is a great bug you've found, I hadn't considered that. Is the bug in all of the implementations or just the rust one?

 wraps correctly though on c reception, didn't check
I'm still in the beginning of my ZK journey, so I'd appreciate an explanation of what c is. (Here, a link, in an email, in discussions, ...)
X)
If it is a hash semantically, than maybe its type should be changed in the crates.

Right. C is described in both the plume blog post and the slides, both of which are linked from the repo readme! Blog post will be easiest to understand.

Closing this in favor of solution from the other end (taking Rust implementations to be consistent with TS).


Increase NBGL serialization buffer size to 200 bytes

Description
The goal is to be able to serialize bigger NBGL objects for Stax/Nano Testing
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[*] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Tests
[ ] Documentation
[ ] Other (for changes that might not fit in any category)

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (22d2f0c) 59.47% compared to head (4a3b148) 59.47%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #463   +/-   ##
=======================================
  Coverage   59.47%   59.47%
=======================================
  Files          12       12
  Lines        1683     1683
=======================================
  Hits         1001     1001
  Misses        682      682

Flag
Coverage Δ

unittests
59.47% <ø> (ø)

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

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


[ENHANCEMENT] Use less blurry font rendering method like other windows apps

Please confirm these before moving forward

[X] I have searched for my feature proposal and have not found a work-in-progress/duplicate/resolved/discarded issue.
[X] This improvement refers to an existing feature. If you want to suggest a new feature, please use this template
[X] This improvement is not a bug. If you want to report a bug, please use this template

Describe the improvement
On a fractionally-scaled monitor text in UniGetUI looks blurrier vs. text in other apps
Top: Wordpad
Bottom: UniGetUI 3.1.0-alpha0

When you zoom a lot you can see that Wordpad has some "yellowish" color around, I think it's due to a specific type of antialiasing. Can UniGetUI add an option to use the same type?
Thanks!
Describe how this improvement could help users.
This would improve text readability

WingetUI uses WinUI3 rendering, powered by the Windows Conposition Engine. I cannot change how the text is rendered

This seems to be the core framework issue indeed https://github.com/microsoft/microsoft-ui-xaml/issues/768, and given the team's attitude that's it's better to wait a couple of more decades until the hardware fleet catches up, it's unlikely to ever get fixed

Subpixel-Rendering is an intended feature on all common operating systems. It depends on the person and on the monitor which rendering is prefered, I recommend reading https://de.wikipedia.org/wiki/Subpixel-Rendering
@eugenesvk
To disable the subpixel-redering on Windows I recommend: (increases the preformance)

disabling font smoothing / anti-aliasing in Windows 10
disable ClearType


Browsing S7 OPCUA Server, Outputs not returned (FunctionalGroupType?)

Hi Nauful,
we are trying to browse Nodes in a S7 OPCUA Server. We are browsing with this command:
client.Browse(new BrowseDescription[]
{
new BrowseDescription(
topnodeid,
BrowseDirection.Forward,
NodeId.Zero,
true, 0xFFFFFFFFu, BrowseResultMask.All)
}, 10000, out browseResults);
In OPCWatch we are seeing several Outputs (Q_Test....) but the browse result by the code above just gives us 2 results, the icon and a Node FunctionalGroupType of NodeClass ObjectType. Do you know how to browse the Outputs?
Thanks a lot
Thomas

Hello Thomas,
Could you share a Wireshark capture of browsing without encryption? I would guess it has something to do with references but I need to see to make sure.
Thanks,
Nauful

Hi Nauful,
here is the capture:
opcuacapture.zip

Can you capture the browse operation in the other client which shows the correct list of children?
Thanks,
Nauful

It should be somewhere in here:
opcuacapture2.zip

Looks like Browse returns a continuation point for you to call BrowseNext:
client.Browse(new BrowseDescription[]
{
		new BrowseDescription(
			new NodeId(2, 0),
			BrowseDirection.Both,
			NodeId.Zero,
			true, 0xFFFFFFFFu, BrowseResultMask.All)
}, 1000, out browseResults);

while (browseResults[0].ContinuationPoint != null)
{
	client.BrowseNext(new[] { browseResults[0].ContinuationPoint }, false, out browseResults);
}

At every step, browseResults will be recreated as a new array.

Hi Nauful,
thanks a lot for your great support again. This resolved our issue. Something I did not knew.
Best regards
Thomas

Great! It's uncommon but if results overflow Browse, servers support continuation points to continue a browse with BrowseNext.
Thanks,
Nauful


test-branch/execute-job/quoted-authorization/1345c22e-1a6f-41bf-8211-734363d0421c

PR for testing 2d0ca54256bb60d6ff6712f08a00fbc4777a5875

/canonical/self-hosted-runners/run-workflows 2d0ca54256bb60d6ff6712f08a00fbc4777a5875


Add Tables type

Hey.
I have another small feature that I need before I can move from schemats to knex-types.
Its a type called Tables connecting table name with its type.
PS Should I create Issue first? Or is PR OK when I have code ready? 🙇

Hey @d7ark, the PR looks good! Did you want to add anything else to it before merging? BTW, no need to create an "issue" for it.

Hey. Thanks! I think this one is ready.

When will this be released? I happen to need to use this feature😂

+1. Any plans to release it? =)

@imjamesku @swayf I think it was released already. Check out v0.3.2.

@d7ark @imjamesku @swayf, oh, it seems like I forgot to publish it. Check out v0.4.0.


Update speech usage document for streaming.

Thanks to @FCtj for finding this.

Thank you!

LGTM


Offline Access clientViaUserConsent()?

How do we grant offline access as defined in the official documentation via clientViaUserConsent()? I am unable to find the parameter.
clientViaUserConsent(_credentialsID, _scopes, prompt)
I am looking to have the client authorize my project to access their calendar on the backend and refresh their token. The current issue is on the backend, I am getting an "unauthorized_client" error when calling the refresh.

hey, did you find a solution yet? i'm also facing the same issue.

I have found a solution. You should get one time consent using clientViaUserConsent(_credentialsID, _scopes, prompt) function, then save some properties of your AuthClient object to your database then use it when refreshing authentication using refreshCredentials().
Something like this:
AccessCredentials _accessCredentials = await refreshCredentials( _clientID, newCredentials , _newClient);
Documents mention
If you use a Google API Client Library, the [client object](https://developers.google.com/identity/protocols/oauth2/web-server#creatingclient) refreshes the access token as needed as long as you configure that object for offline access.
The flutter package has AutoRefresh class for this purpose. Client refreshes itself basically.

@lowkeyboard Your solution didn't work for me as error below.
  'package:googleapis_auth/src/auth_http_utils.dart': Failed assertion: line 98 pos 16: 'credentials.refreshToken != null': is not true.
  dart:core                                               _AssertionError._throwNew
  package:googleapis_auth/src/auth_http_utils.dart 98:16  new AutoRefreshingClient
  package:googleapis_auth/auth_io.dart 144:10             clientViaUserConsent

So, I created a new class that extends AuthorizationCodeGrantServerFlow. By doing this, It allows us to override method authenticationUri() to return a URI by passing offline parameter to the method createAuthenticationUri() as below. It's a workaround for sure, Because we have to import two files under src/ folder  (which means private implementation) to reuse codes behind the package. Hope this issue will be resolved soon.
https://github.com/google/googleapis.dart/blob/9e090203b1bb56e13c3b2392848fc5ed8ddbe7b4/googleapis_auth/lib/src/oauth2_flows/auth_code.dart#L19-L27.

@Gasol I'm glad you were able to find a workaround, happy coding! :)

@Gasol Thanks for the info!! Made my life so much easier! = )  To pay it forward, here is my hacked code of your advice. (Note, I was using clientViaUserConsent, so created a Hack version of that)
// Attempt at implementing @Gasol's solution at:
//  https://github.com/google/googleapis.dart/issues/378
import 'package:googleapis_auth/src/oauth2_flows/authorization_code_grant_server_flow.dart';
import 'package:googleapis_auth/src/oauth2_flows/auth_code.dart';
import 'package:googleapis_auth/src/auth_http_utils.dart';
import 'package:http/http.dart';

// Copied and pasted from auth_io.dart. 2 lines changed below.
Future<AutoRefreshingAuthClient> clientViaUserConsentHack(
  ClientId clientId,
  List<String> scopes,
  PromptUserForConsent userPrompt, {
  Client? baseClient,
  String? hostedDomain,
  bool offline = false, // First line changed.
}) async {
  var closeUnderlyingClient = false;
  if (baseClient == null) {
    baseClient = Client();
    closeUnderlyingClient = true;
  }

  // Second line changed
  final flow = AuthorizationCodeGrantServerFlowHack(
    clientId,
    scopes,
    baseClient,
    userPrompt,
    hostedDomain: hostedDomain,
    offline: offline,
  );

  AccessCredentials credentials;

  try {
    credentials = await flow.run();
  } catch (e) {
    if (closeUnderlyingClient) {
      baseClient.close();
    }
    rethrow;
  }
  return AutoRefreshingClient(
    baseClient,
    clientId,
    credentials,
    closeUnderlyingClient: closeUnderlyingClient,
  );
}

// Overrides required to get offline = True into the system. 3 line changes from base version
class AuthorizationCodeGrantServerFlowHack extends AuthorizationCodeGrantServerFlow {
  final bool offline; // First line

  AuthorizationCodeGrantServerFlowHack(
    super.clientId,
    super.scopes,
    super.client,
    super.userPrompt, {
    super.hostedDomain,
    this.offline = false, // Second line
  });

  @override
  Uri authenticationUri(
    String redirectUri, {
    String? state,
    required String codeVerifier,
  }) =>
      createAuthenticationUri(
        redirectUri: redirectUri,
        clientId: clientId.identifier,
        scopes: scopes,
        codeVerifier: codeVerifier,
        hostedDomain: hostedDomain,
        state: state,
        offline: offline, // Third line
      );
}


generics: introduce one-line Immediate If (IIf) assignment helper

for things like:
bits := core.IIf(rootCA, 3072, 2048)

Forgot the README

v0.7.1 tagged @karasz


[Service Bus] README Updates

Summary
The focus of these changes is to resolve some broken links and remove references to .NET Core 2.1.
References and Related

Service Bus Readme issues (#31492)

API change check
API changes are not detected in this pull request.


[release/1.7] vendor: google.golang.org/grpc v1.58.3

Upgrade gRPC in anticipation of upgrading OTel, which raises the lower limit.
As gRPC upgrades have been known to cause issues, this is being done pre-emptively. Additionally, it ensures we avoid v1.58.2, which has a known CVE, but would otherwise be chosen by MVS from go mod.
See #9256.

Looks like this needs a rebase; also probably should have a counterpart in main (separate from the OTEL PR?) and backported.

All right, pending real CI jobs succeeding, this should be all set 🤞🏻
See #9297 for the equivalent PR to main.

#9297 is merged 🎉 so hopefully this is all set too

/ok-to-test


'No OpKernel was registered to support Op 'RoiPoolGrad' with these attrs' in Training

The environment on my computer is Tensorflow 1.0.1 + python 2.7 + cuda 8.0.
I can run the  demo.py successfully follow the guidance.
But when I go to the training part, and run:
> python ./faster_rcnn/train_net.py --gpu 0 --weights ./data/pretrain_model/VGG_imagenet.npy --imdb voc_2007_trainval --iters 70000 --cfg  ./experiments/cfgs/faster_rcnn_end2end.yml --network VGGnet_train --set EXP_DIR exp_dir

There is an error says:
  tensorflow.python.framework.errors_impl.InvalidArgumentError: No OpKernel was registered to support Op 'RoiPoolGrad' with these attrs.  Registered devices: [CPU], Registered kernels:
  device='GPU'; T in [DT_FLOAT]

It maybe that there is something wrong on my setting, but I just follow the guidance, and try to replicate the result on my computer till now.
I'm totally not sure what the problem is.  Is there anyone who has some ideas? Thank you!
Following is the detailed error message.

/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/ops/gradients_impl.py:93: UserWarning: Converting sparse IndexedSlices to a dense Tensor of unknown shape. This may consume a large amount of memory.
"Converting sparse IndexedSlices to a dense Tensor of unknown shape. "
Traceback (most recent call last):
File "./faster_rcnn/train_net.py", line 108, in
restore=bool(int(args.restore)))
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 400, in train_net
sw.train_model(sess, max_iters, restore=restore)
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 148, in train_model
sess.run(tf.global_variables_initializer())
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 778, in run
run_metadata_ptr)
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 982, in _run
feed_dict_string, options, run_metadata)
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1032, in _do_run
target_list, options, run_metadata)
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1052, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: No OpKernel was registered to support Op 'RoiPoolGrad' with these attrs.  Registered devices: [CPU], Registered kernels:
device='GPU'; T in [DT_FLOAT]
[[Node: gradients/pool_5_grad/RoiPoolGrad = RoiPoolGrad[T=DT_FLOAT, pooled_height=7, pooled_width=7, spatial_scale=0.0625](conv5_3/Relu, roi-data/rois, pool_5:1, gradients/fc6/transpose_grad/transpose)]]
Caused by op u'gradients/pool_5_grad/RoiPoolGrad', defined at:
File "./faster_rcnn/train_net.py", line 108, in
restore=bool(int(args.restore)))
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 400, in train_net
sw.train_model(sess, max_iters, restore=restore)
File "./faster_rcnn/../lib/fast_rcnn/train.py", line 142, in train_model
grads, norm = tf.clip_by_global_norm(tf.gradients(loss, tvars), 10.0)
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/ops/gradients_impl.py", line 560, in gradients
grad_scope, op, func_call, lambda: grad_fn(op, *out_grads))
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/ops/gradients_impl.py", line 368, in _MaybeCompile
return grad_fn()  # Exit early
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/ops/gradients_impl.py", line 560, in
grad_scope, op, func_call, lambda: grad_fn(op, *out_grads))
File "./faster_rcnn/../lib/roi_pooling_layer/roi_pooling_op_grad.py", line 23, in _roi_pool_grad
data_grad = roi_pooling_op.roi_pool_grad(data, rois, argmax, grad, pooled_height, pooled_width, spatial_scale)
File "", line 74, in roi_pool_grad
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 768, in apply_op
op_def=op_def)
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2336, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1228, in init
self._traceback = _extract_stack()
...which was originally created as op u'pool_5', defined at:
File "./faster_rcnn/train_net.py", line 100, in
network = get_network(args.network_name)
File "./faster_rcnn/../lib/networks/factory.py", line 29, in get_network
return VGGnet_train()
File "./faster_rcnn/../lib/networks/VGGnet_train.py", line 17, in init
self.setup()
File "./faster_rcnn/../lib/networks/VGGnet_train.py", line 84, in setup
.roi_pool(7, 7, 1.0/16, name='pool_5')
File "./faster_rcnn/../lib/networks/network.py", line 36, in layer_decorated
layer_output = op(self, layer_input, *args, **kwargs)
File "./faster_rcnn/../lib/networks/network.py", line 235, in roi_pool
name=name)[0]
File "", line 45, in roi_pool
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 768, in apply_op
op_def=op_def)
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2336, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/yuanjial/tensorflow/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1228, in init
self._traceback = _extract_stack()
InvalidArgumentError (see above for traceback): No OpKernel was registered to support Op 'RoiPoolGrad' with these attrs.  Registered devices: [CPU], Registered kernels:
device='GPU'; T in [DT_FLOAT]
[[Node: gradients/pool_5_grad/RoiPoolGrad = RoiPoolGrad[T=DT_FLOAT, pooled_height=7, pooled_width=7, spatial_scale=0.0625](conv5_3/Relu, roi-data/rois, pool_5:1, gradients/fc6/transpose_grad/transpose)]]

Hello, I have encountered the same problem, did you have solved this problem?

My problem is that I installed tensorflow, not tensorflow-gpu. Once I changed it, the whole project could work expectedly.