EV3Dev2 vs PyBricks2: Which is better?

I'll be competing in WRO 2023, and I was planning to use Pybricks2, but one of my teammates suggested to use ev3dev2 instead. We asked our teacher for information about which one is better, which my teacher responded he hadn't any information. For more information about which is better I search on the internet and after some searching, I was not able to find any information. For me and my team this is a very important decision due that the program we choose is going to be use for the next 4 years by our team and then is going to be the one that is going to be teach to the next team.

This is a hard question to answer because what is best for me might not be best for you and your team. What are some important features of the software that would help you do better in the competition?

See also https://github.com/pybricks/pybricks-api/issues/22.

What me and my team mostly need it's the capability for it to use a PID, for the PID to be precise, not having too many variables, being reliable, for it to fast in performing and loading a program, and a small bonus it's for it to be capable of using a neuronal network (this one is not really needed due to my team and I not being sure about using it yet). The PID is the most important one due to it being our main way to adjust correctly and move around the track.

As you'll see in https://github.com/pybricks/pybricks-api/issues/22, Pybricks might be closer to what you need in terms of drive accuracy and configurable PID settings.

Alright thank you for your help and thank you for this information is going to be very useful in deciding which program to use.


ros_control_boilerplate: 0.3.1-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository ros_control_boilerplate to 0.3.1-0:

upstream repository: https://github.com/davetcoleman/ros_control_boilerplate.git
release repository: https://github.com/davetcoleman/ros_control_boilerplate-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.20
previous version for package: 0.3.0-0

ros_control_boilerplate
* API deprecation fix for rosparam_shortcuts
* Switched to better use of rosparam_shortcuts
* Ability to record all controller status data, not just at certain frequency
* Contributors: Dave Coleman

Ready to merge


Remove Microsoft.Extensions.DiagnosticAdapter from the shared framework

Per our last design review of the shared framework, this assembly does not meet the bar for importance or usage. This assembly will continue to be available as a NuGet package for users who want to use this with ASP.NET Core.
cref https://github.com/aspnet/AspNetCore/issues/3755

More IdentityModel cleanup?

Microsoft.Extensions.DiagnosticAdapter was part of the same review that led to #6035, but this is being removed for separate reasons.
But yes, I am incidentally removing some other unused metadata w.r.t. code-signing assemblies. This should have been done in #6035, but we don't have anything that checks for unnecessary codesign config. 🤷♂


Use OXCGRT flag

To detect only national records

We have added the flag into OxCGRT records but still need to filter only national records from OxCGRT. This could probably be best addressed by filtering for only records that do not specify a subnational area.


Realistic expectations for numpy compatibility?

Should we expect all numpy functions to work with fxpmath, or just the common ones?
I'm trying to call numpy.linalg._umath_linalg.eigh_lo and getting the following exception:
  File "{{site-packages}}fxpmath/objects.py", line 1577, in __array_ufunc__
    return self._wrapped_numpy_func(ufunc, *inputs, **kwargs)
  File "{{site-packages}}fxpmath/objects.py", line 1657, in _wrapped_numpy_func
    return self.__array_wrap__(val)
  File "{{site-packages}}fxpmath/objects.py", line 1126, in __array_wrap__
    return self.__class__(out_arr)
  File "{{site-packages}}fxpmath/objects.py", line 226, in __init__
    self._init_size(val, signed, n_word, n_frac, n_int, max_error=self.config.max_error, n_word_max=self.config.n_word_max, raw=raw)
  File "{{site-packages}}fxpmath/objects.py", line 357, in _init_size
    self.set_best_sizes(val, n_word, n_frac, max_error=max_error, n_word_max=n_word_max, raw=raw)
  File "{{site-packages}}fxpmath/objects.py", line 481, in set_best_sizes
    val, _, raw, signed, n_word, n_frac = self._format_inupt_val(val, return_sizes=True, raw=raw)
  File "{{site-packages}}fxpmath/objects.py", line 661, in _format_inupt_val
    val, signed, n_word, n_frac = utils.str2num(val, self.signed, self.n_word, self.n_frac, return_sizes=True)
  File "{{site-packages}}fxpmath/utils.py", line 154, in str2num
    x[idx], _signed, _n_word, _n_frac = str2num(v, signed, n_word, n_frac, base, return_sizes=True)
TypeError: 'tuple' object does not support item assignment

I tried messing with handling for tuples vs. lists a bit but wasn't able to figure much out.
Calling the same code but removing the fixed point conversion before calling eigh_lo works, so it must be something with the fxpmath compatibility.

fxpmath works with numpy through dispatching methods __array_function__ and __array_ufunc__ (more info here) where a Fxp is converted to a ndarray before call the numpy function and create a new Fxp with the returned value. If a numpy function doesn't support this protocol maybe an error could happen.
According to the error you reported, it seems that the problem is with the creation of a Fxp with the returned value. It would be useful to have a piece of code to reproduce the error and perform the debug.

Closing this issue due it's inactive.


Bug: derive_vars_dtm_to_tm - can't convert

What happened?
ad1 <- left_join(pc1, adsl_core, by = c('STUDYID', 'USUBJID')) %>%
+   # Calculate ADTM
+   derive_vars_dtm(
+     new_vars_prefix = "A",
+     dtc = PCDTC,
+     ignore_seconds_flag = TRUE
+     #flag_imputation = FALSE
+   ) %>%
+   derive_vars_dtm_to_dt(vars(ADTM)) %>%
+   derive_vars_dtm_to_tm(vars(ADTM))
Error in `mutate()`:
! Problem while computing `ATM = as_hms(ADTM)`.
Caused by error in `stop_vctrs()`:
! Can't convert <datetime<UTC>> to <time>.
Run `rlang::last_error()` to see where the error occurred.

This may go back to the original UTC issue we had seen previously on different platforms.  We are getting this error in our production environment at GSK.  I confirmed it does not occur in the admiral cloud development environment.
Session Information
> sessionInfo()
R version 4.1.2 (2021-11-01)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Red Hat Enterprise Linux

Matrix products: default
BLAS:   /usr/lib64/libblas.so.3.4.2
LAPACK: /usr/lib64/liblapack.so.3.4.2

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C               LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8    LC_PAPER=en_US.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C             LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

loaded via a namespace (and not attached):
[1] compiler_4.1.2  tools_4.1.2     rstudioapi_0.13

Reproducible Example
No response

We were able to recreate this issue/bug in GSK's R environment using admiral 0.6.3.  However, we also tested admiral 0.7.0 (not yet released) and found that recent updates have resolved this issue/bug.  GSK is going to investigate a little more next week, but it seems like we are in good standing.

GSK internal meeting was cancelled as bug has been resolved


v3:  hide_first_space_indent_level doesn't apply to tabs

I'm not sure if there is a configuration option I'm missing for this hook, but it doesn't seem to want to apply to tabs.  The only hooks.type that seems relevant is WHITESPACE, which I'd expect to work?  But I see the following:

Let me know if you need any more info!  And thank you for maintaining this :)

I made it explicitly not for tabs because conceptually it's a bit weird there, I think.
But I can add a hook for tabs as well.

done, there is now hide_first_tab_indent_level as well.

Amazing, thank you!


The list of available schemas is ordered in a non-friendly way

We need to rewrite:
convert_hed_versions in web_util to order the schemas so that the standard schemas come out first.

Fixed in PR#203


GDNativeLibrary resource support

While 'Switch' shows up as a preset in the 'Export' window, it does not show up as a platform in GDNativeLibrary resource files.  Is there any plan to support this?  How much work would be involved to make it work?

I think I've just realized that I don't think I can make switch dynamic libraries for godot to use anyway.  I have so far been able to make static libraries!  But godot doesn't seem to support using those for GDNativeLibrary.  Sorry for not realizing this before opening the issue!


请问该如何完成code completion功能的配置？

What happened?
When testing the code completion feature, it always shows an error. The screenshot of the error is as follows:
Relevant log output or stack trace
No response
Steps to reproduce

CodeGPT version
2.7.1-241
Operating System
Windows

I am guessing the "ezchat" API is some sort of Proxy for OpenAI's API?
If you are trying to use gpt-3.5-turbo-instruct you should use the /v1/completions, not /v1/chat/completions is compatible with that model for code-completions.
You look at the Azure OpenAI Preset template for the body:

gpt-3.5-turbo-instruct

Thank you, I solved my problem according to your instructions, but what do you mean by Azure OpenAI?

gpt-3.5-turbo-instruct

Thank you, I solved my problem according to your instructions, but what do you mean by Azure OpenAI?

Good to hear that its working for you now.
I just wanted to point out that you can look at the Azure OpenAI Preset template to see how to set the Body for /v1/completions

gpt-3.5-turbo-instruct

Thank you, I solved my problem according to your instructions, but what do you mean by Azure OpenAI?

Good to hear that its working for you now. I just wanted to point out that you can look at the Azure OpenAI Preset template to see how to set the Body for /v1/completions

I see what you mean. Thank you again.


picard: new application icon

Description

Use the current icon for MusicBrainz Picard, which was introduced in 2017 with the 1.4 release.
Motivation and Context

An outdated icon was used.
How Has this Been Tested?

Screenshot (if appropriate, usually isn't needed):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[ ] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[ ] The changes only affect a single package (not including meta package).

Original Location

Original Repository
Open Issues Add the different issues underneath, and tick those that are fixed in this PR

[ ] Issue 1 link
[ ] Issue 2 Link

[ ] Include the link to the opened PR that removes the package from the original location
[ ] The migration guidelines have been followed

:white_check_mark: Package verification completed without issues. PR is now pending human review

:white_check_mark: Package verification completed without issues. PR is now pending human review

@phw thank you for your contribution.
The new icon will be part of the next version of the picard package.


try to provide bcc and headers in CI for unit test on linux

try to provide bcc and headers in CI for unit test on linux

@marceloamaral , here you are.


Development

#Deployment
User Story
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
Acceptance Criteria
GIVEN a Prework Study Guide website,
WHEN I visit the website using the URL,
THEN I can access my website from any browser

use api and fetch


Make possible set headers for request

My ExchangeRateService need auth (basic) and i want set headers for request.
It possible when add headers parameter for request method or set protected visibility for properties httpClient and requestFactory. Or both

And i want send Accept: application/json header in request

Closing in favor of https://github.com/florianv/exchanger/pull/9


General Issue: Dependencies conflict with tidyverse recomendations

Background Information
Currently dependencies on external functions are handled via importing the functions into the namespace in the admiral-package.R script.
Hadley's book on packages recommends not to do this, and instead to write package::function(. Some example reasons why :: is preferred are;

It makes it clear which functions are from this package, and which are external. This makes it easier to onboard to the codebase, debug the codebase, and generally understand what a function is doing
There is no risk of namespace conflicts
You don't end up with mismatches as the package evolves and dependencies are dropped, as dependencies are called where they are used, rather than in a separate file

He does though note 3 exceptions where it can be useful to import,

you use the function a lot (and it's clear it comes from somewhere else - e.g. for admiral mutate or left_join will obviously be dplyr to devs and users).
it's a loop, as it's slower - but only noticeably if running millions of times
it's an operator, e.g. %--% from lubridate

As a side note, it's more redundant rather than a bug - but there is least one case where we accidentally have done both. e.g. in this line of code the user wrote stringr::str_match despite the fact it's imported into the namespace
ref: https://r-pkgs.org/dependencies-in-practice.html#in-code-below-r
Definition of Done
Consider a switch to following Hadley's best practices and move to ::

ugh... i really dislike this part of R with :: every where in your code - so distracting.
I guess if Hadley is saying this is best practice we have to obey... just aesthetically it is so wacky to look at a function with stringr:: and dplyr:: peppered everywhere making it hard to look at and understand
@pharmaverse/admiral what do you think?

We discussed this as a team already and I would be reluctant to change anything at this point.

@bms63 @thomas-neitmann can you raise at the team call tomorrow please and if team decision is to stick with current approach then the rationale should be explained in our programming strategy. I remember @sgorm123 raising this same concern previously but I don't see it documented anywhere why we chose not to follow this.

I guess if going against normal trends it would be good to document. Hadley's textbook is the defacto standard now for good in r packages. We also use both currently in admiral, it's just sometimes the :: isn't doing anything due to the import.

In the link above :: is used even though it doesn't do anything due to the import from.
there is a purrr::pmap, but that actually is doing something as bunch of purrr functions are imported, but not pmap.
derive_summary_records also uses dplyr::

At the more general level Hadley's point was that it decreases confusion, as you are being that being explicit about internal vs external which makes the code easier to follow.   @rossfarrugia pointed out  (when we discussed this change for admiralroche) though admiral has very specific naming, so you run into less issues around reading the code and not knowing what functions are part of the package vs imported than a normal R package.
With admiralroche we switched to :: where it is a Suggests, as you shouldn't import from at the package namespace level if it's not a Depends.

This is seperate to admiral, as it doesn't have to follow the majority - but I was kinda interested now to know what is the most common, once you remove the tidyverse. This is a bit hard though, as many of the famous names in R are either from pre-tidyverse (e.g. survival and data.table), which is an era where people hated dependencies 😆 - or they are posit employees or tidyverse contributors.
But it looks like Jeeron Ooms (Ropensci) uses :: based on magick, but Will Landau does it the admiral way based on drake. And then Karl Broman seems to do a mixture, but mostly aligns with admiral and in his package guide mentions both ways and then says he uses import.

Placed onto tomorrow's agenda - I think @zdz2101 did this one

@bms63 i can't make the team call so sharing my 2 cents here... happy to go with the majority and i don't have major concerns not following the tidyverse approach here, but if we stick with current rule then I'd ask:

lets make it consistent (e.g. the examples of :: that James identified above and let's check any others in the codebase)
when we add rationale to the programming strategy include why we don't see this as a major risk of namespace conflicts (i.e. admirals strict function naming conventions)

Hi all,
The decision is to keep our admiral-package.R convention.
We will update our code base to not use :: #1842.
We will update our programming documentation to explain our reasoning and best practice - there might be a slight exception to the rule from time to time.
https://github.com/pharmaverse/admiraldev/issues/253
@pharmaverse/admiral any other thoughts?


5.4.0 WIP Integration Branch

(Currently) A PR to track the changes of 5.4.0-wip against master.

Will re-open as 5.5.0 WIP branch...


documentation bug in kivy.core.text

The first example starts with "from kivy.core.label import Label as CoreLabel".  Should be "from kivy.core.text…"

Thanks for catching that!


fix: .snyk & package.json to reduce vulnerabilities

The following vulnerabilities are fixed with an upgrade:

https://snyk.io/vuln/npm:hoek:20180212
https://snyk.io/vuln/npm:npm:20180222

The following vulnerabilities are fixed with a Snyk patch:

https://snyk.io/vuln/npm:hoek:20180212

Snek is pretty neat 💯

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Not sure why its saying I haven't signed the CLA. Seems to be a bug there, as it redirects quickly.

Because the author has to sign the pull request, not the committer. :/


More validation in functions that need it

Retiring this. When needed, we can make more specific issues.


[Bug]:  system notifications

Describe the bug
not sure if this is a bug or feature request of both.  I set reminders to use system notifications.  I am on Win11.  The notification is in the notification panel, but  it says the app name is electron.app.Obsidian.  And in settings/system/notifications in the list of apps that did recent notifications it isn't there.  This is needed to be able to customize notifications, as I usually have notifications with sound off but want to  make this an exception.
B
Expected Behavior
The app name would be obisdian in the notifications sidebar and the Obsidian app would appear the the settings list of recent notifications.
Steps to reproduce
set a remidner in win11 and set to use systems notifications.
Operating system
Windows

You may have to make the request to Obsidian itself.
As far as I read the Electron's doc, there is no such settings.


Move Drupal-specific configuration files

With an in-place deployment for local environments, all Drupal-specific configuration files can be moved from ´drupal/conf´ to their final location, leaving only WudnderTools-specific code in that folder.

There is only settings.local and vagrant.settings left. Need to resolve #115 to make this fully working solution.

I'll close this as only the settings issue remains and it has it's own separate issue.


Welcome page has 3 broken links

"Introduction to the class","Current Computing Technology","Computer System Evaluation" links lead to GitHub 404 pages from https://jlpteaching.github.io/comparch/

Fixed! That was more painful than it should have been.


Fix code coverage

Coverage remained the same at 0.0% when pulling 9660683dd81f432fe5d098728a2b43c72ad1a71a on coverage into 5108aba126abcadccaa6d1528212944e3cbf7b49 on master.


VS code is extremely slow

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Performance Issue
Whenever i open a file to code it's stucks on hanging position in the window even after uninstalling unused extensions.
VS Code version: Code 1.52.1 (ea3859d4ba2f3e577a159bc91e3074c5d85c0523, 2020-12-16T16:34:46.910Z)
OS version: Windows_NT x64 10.0.19041

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-4005U CPU @ 1.70GHz (4 x 1696)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
3.91GB (0.53GB free)

Process Argv
D:\Programmings\web dev apna college --crash-reporter-id 7760dc67-9273-4582-8a80-fb7664e33091

Screen Reader
no

VM
67%

Process Info
CPU %	Mem MB	   PID	Process
    1	    85	  3372	code main
    0	    96	  1800	   shared-process
    0	   167	  2704	   window (Welcome - web dev apna college - Visual Studio Code)
    0	   119	    32	     extensionHost
    0	    23	  3332	       c:\Users\harsh\.vscode\extensions\ms-vscode.cpptools-1.1.3\bin\cpptools.exe
    0	    10	  5516	         console-window-host (Windows internal process)
    0	    11	 10964	         "c:\Users\harsh\.vscode\extensions\ms-vscode.cpptools-1.1.3/bin/cpptools-srv.exe" 3332 0
    0	    64	  3628	       "C:\Users\harsh\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\harsh\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\node\jsonServerMain" --node-ipc --clientProcessId=32
    0	    88	 10468	       "C:\Users\harsh\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\harsh\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\html-language-features\server\dist\node\htmlServerMain" --node-ipc --clientProcessId=32
    0	    14	 10160	     watcherService
    0	    11	   696	       console-window-host (Windows internal process)
    0	    66	 11664	     searchService
    1	    78	  4076	   window (Issue Reporter)
    0	    22	  4952	   crashpad-handler
    0	    36	  8512	   utility
    2	   137	 10736	   gpu-process

Workspace Info
|  Window (Welcome - web dev apna college - Visual Studio Code)
|    Folder (web dev apna college): 5 files
|      File types: html(3) c(2)
|      Conf files:;

Extensions (12)

Extension
Author (truncated)
Version

code-gnu-global
aus
0.2.2

dart-code
Dar
3.17.1

competitive-programming-helper
Div
5.0.1

code-runner
for
0.11.1

python
ms-
2020.12.424452561

jupyter
ms-
2020.12.414227025

remote-wsl
ms-
0.52.0

cpptools
ms-
1.1.3

debugger-for-chrome
msj
4.12.11

autodocstring
njp
0.5.4

vscode-readme-pattern
tho
1.3.0

vim
vsc
1.18.5

A/B Experiments
vsliv368:30146709
vsreu685:30147344
openlogontheside:30221877
python763:30178808
python383:30185418
pythonvspyt617cf:30237717
vspor879:30202332
vspor708:30202333
vspor363:30204092
python504:30227505
vswsl492cf:30211402
wsl2promptcf:30224613
unusedpromptcf:30224611
folderexplorer:30224614
openfilemenu:30224647
pythonvsded773:30236629
pythonvspyt600:30239448

/needsMoreInfo
Is it reproducible with all extensions disabled?
You can try this with F1 and >Developer: Reload Window With Extensions Disabled
And what kind of file you are trying to open? HTML, C, or something else?

Does it mean that VS Code will open file quickly with all extensions disabled but slow if live server extension is enabled?

Yes. This is the only issue
On Tue 29 Dec, 2020, 2:55 AM Andrii Dieiev<EMAIL_ADDRESS>wrote:

It's working with all disabled extensions.
Does it mean that VS Code will open file quickly with all extensions
disabled but slow if live server extension is enabled?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/113317#issuecomment-751869172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANACTH35HIY5FR5P5L5IYU3SXDZVVANCNFSM4VGGYJUA
.

/causedByExtension
If it's not reproduced with all extensions disabled - please report this issue in extension repo.
Extension bisect could help to find one causing problem if you are not sure which one causes it.
If it's Live server in your case - they already have couple of issues related to performance so new info should be helpful in those issues.


Difference between speakerNet and speakerNet of NIVIDIA?

What is the difference between your model and speakerNet proposed in this paper SPEAKERNET: 1D DEPTH-WISE SEPARABLE CONVOLUTIONAL NETWORK FOR TEXT-INDEPENDENT SPEAKER RECOGNITION AND VERIFICATION
Both you use the same name?

What is the difference between your model and speakerNet proposed in this paper SPEAKERNET: 1D DEPTH-WISE SEPARABLE CONVOLUTIONAL NETWORK FOR TEXT-INDEPENDENT SPEAKER RECOGNITION AND VERIFICATION
Both you use the same name?

I think they just simply name the model class as 'SpeakerNet'.

The two models are different.


REQUEST: turn off tiflash-repo's automatic inviter review mechanism

The tiflash-repo always invites inappropriate people as reviewers. We hope to disable this mechanism so that the PR owner can invite people to review on their own.

fixed in #886


camo: Replace with go-camo implementation.

The upstream of the camo repository[1] has been unmaintained for
several years, and is now archived by the owner.  Additionally, it has
a number of limitations:

It is installed as a sysinit service, which does not run under
Docker
It does not prevent access to internal IPs, like <IP_ADDRESS>
It does not respect standard HTTP_proxy environment variables,
making it unable to use Smokescreen to prevent the prior flaw
It occasionally just crashes, and thus must have a cron job to
restart it.

Swap camo out for the drop-in replacement go-camo[2], which has the
same external API, requiring not changes to Django code, but is more
maintained.  Additionally, it resolves all of the above complaints.
go-camo is not configured to use Smokescreen as a proxy, because its
own private-IP filtering prevents using a proxy which lies within that
IP space.  It is also unclear if the addition of Smokescreen would
provide any additional protection over the existing IP address
restrictions in go-camo.
Fixes #18351 by moving to supervisor.
Fixes zulip/docker-zulip#298 also by moving to supervisor.
[1] https://github.com/atmos/camo
[2] https://github.com/cactus/go-camo
Testing plan: Deployed in a test production instance.

This should ideally use the zulip::external_dep package from #20286, so I'm going to mark this as a draft until that lands and I can rebase this to use it.

Yeah, sounds good.

The commit message paragraph about headers has a typo "co-camo".

Fixed the typo in the commit message and rebased.

lgtm, feel free to merge once CI passes.  (Do we also want to backport this to 4.x?)

Yup, I think it's worth backporting, and I previously tagged it as such. :)


Certain javadocs no longer resolvable via static.javadoc.io

Consider this:
https://javadoc.io/doc/jakarta.persistence/jakarta.persistence-api/2.2.2
When hit with a browser, you get javadocs in HTML format.  Fine.
Normally one may also hit this:
https://static.javadoc.io/jakarta.persistence./jakarta.persistence-api/2.2.2
…but for this project (and many others with what looks like JDK 9+ javadocs) this doesn't work.
Contrast that with this:
https://javadoc.io/doc/org.eclipse.microprofile.config/microprofile-config-api/1.3
…and:
https://static.javadoc.io/org.eclipse.microprofile.config/microprofile-config-api/1.3
…which works fine.

Another arbitrary example:
https://static.javadoc.io/org.jboss.weld.environment/weld-environment-common/3.1.1.Final gives a 403.

with static domain you have to refer to the page directly: https://static.javadoc.io/org.jboss.weld.environment/weld-environment-common/3.1.1.Final/index.html
closing now. Feel free to reopen if you have more issues

OK, I will reopen this—well, no, I'll file a new bug.
You can obviously make your tool do whatever you want it to; I have no control over that.  I hope that I can eventually convince you that javadoc and the maven-javadoc-plugin all need the following resources to NOT give any kind of error (and I personally don't care whether it's static.javadoc.io or not):

https://static.javadoc.io/some.groupId/some-artifact-id/some-version
https://static.javadoc.io/some.groupId/some-artifact-id/some-version/
https://static.javadoc.io/some.groupId/some-artifact-id/some-version/package-list
https://static.javadoc.io/some.groupId/some-artifact-id/some-version/package-list/ # must refer to the same resource as the line above does
https://static.javadoc.io/some.groupId/some-artifact-id/some-version/element-list
https://static.javadoc.io/some.groupId/some-artifact-id/some-version/element-list/ # must refer to the same resource as the line above does

I said "any kind of error" but I should say that in the case of element-list vs. package-list: element-list is what JDK 9+'s javadoc looks for first, and if it can't find it (404, which your tool does not return) then it looks for package-list instead.
There will always be a package-list resource in any javadocs generated by the javadoc tool.  It must also be able to be addressed via package-list/.


command flag should support human readable integer

Hi @overvenus
I find that command doesn't support human readable integer, like:
tikv-server --capacity=10gb

I think we should support this in command flag too.

+1


Fixes register and login cycle

Fixed the login/register cycle

@tronicdata Please look over this when you have the time.


O sorensen issue 9

Created new course based on Issue #9 created by Brian Buffet

Closing this one and creating one single pull request for the last 3 courses


New release?

Hey @stricaud, the last tag/release is from 2016, any plans to create a new release anytime?

You are right, there has been enough improvement to have a new release. Thanks for  your comment! I will see what needs to be done right now to make it happen.


Fix padding fields default value being ignored when using Msb

This PR fixes #56 which describes using a padding field with Msb, its default value won't get set. This change will ensure that the default value for padding fields will be set when using Msb order.
This PR works by passing the order enum to the Members struct then for padding fields without a setter, it checks the order then calculates the right shift offset.

First of all, good catch. Thanks for finding this.
And I have to admit that this macro code is a bit ugly. So I completely understand that fixing something here might be a bit difficult. Still, your changes are a bit more involved than I expected. I'll give you a few comments on them.

Ok now everything looks good, thanks a lot :D


Contributor Docs / Contributor Tags

We should probably add some contributor docs in the top level of the crate -- we can probably copy Rust analyzer's structure https://github.com/rust-analyzer/rust-analyzer/tree/master/docs but just do docs.
User docs are kind of hard for us to do since imgui's visual, and the C++ demo is sufficient.
@thomcc what are your thoughts about tagging "beginner friendly" issues too? This repo is both very large, but actually quite easy to contribute to, especially on the actual API. It'd be nice to expand who's contributing beyond those who had to.
We can have this be the end dest for https://github.com/imgui-rs/imgui-rs/issues/435

We now have a docs folder with development docs, and #664 may move towards a user-facing documentation
I do like the beginner-friendly issue idea - made a good-first-issue label for this


Feature request: support for SAP HANA

Overview of the request

I'd like to enhance JHipster to support SAP HANA as a database type when creating an application.

Motivation for or Use Case

Recently I've been adding support for SAP HANA to several open source projects like Hibernate, Spring, Liquibase, and others, and I'd like JHipster to support SAP HANA as a database as well.

Browsers and Operating System

n/a

Related issues

n/a

Suggest a Fix

If you accept the feature request, I can contribute the support for SAP HANA.

@breglerj this is not the correct project for the request, anyway, I don't think we have the bandwidth to have this in the core project, I suggest you explore adding it as a module or blueprint and if it gets popular we can consider for the core. https://www.jhipster.tech/modules/marketplace/#/list


83 context provider

closes #83

Ups, this is failing on Mac.... Need to investigate

That was a flaky test - will create seperate issue for that and ignore in this issue


revert troublesome line

What does this PR do?
Does not default to CPU provider when multiple providers are available.
Fixes # (issue)
https://github.com/huggingface/optimum/issues/274
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Please check my response in

#274

You can easily move your model around with the .to() method.
Regarding your question about TensorRT. We haven't sketched out a plan on how to fully integrate it yet. Maybe we are not going to use onnxruntime for it and go with TensorRT directly. But when we add support for it we'll make sure the model can be correctly loaded.

Please check my response in
* [Running into Issues Optimizing CausalLM Model for GPU #274](https://github.com/huggingface/optimum/issues/274)

You can easily move your model around with the .to() method.
Regarding your question about TensorRT. We haven't sketched out a plan on how to fully integrate it yet. Maybe we are not going to use onnxruntime for it and go with TensorRT directly. But when we add support for it we'll make sure the model can be correctly loaded.

Responded in the issue. FYI it seems like if the TensorRT execution provider is available and you select it then it sort of just works. I have been able to get some really good performance


#2259 Fix potential thread safe problem in InterceptorScope

Current coverage is 39.84% (diff: 78.26%)

Merging #2260 into master will increase coverage by 0.05%

@@             master      #2260   diff @@
==========================================
  Files          1611       1613     +2
  Lines         55846      55863    +17
  Methods       10972      10976     +4
  Messages          0          0
  Branches       7709       7713     +4
==========================================
+ Hits          22220      22258    +38
+ Misses        33626      33605    -21
  Partials          0          0

Powered by Codecov. Last update 6465a0f...fc62055


Bump Dependencies #3122609613

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/3122609613

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add a version command

There is no way to know which version you are running...

When you use antibody version it show master and I think that it should show a number in "x.y.z" format

@ozkxr It should be doing that... will check that out in #57


A Class Name Consisting of More Than One Word on the Joint.js Diagram Causes an Infinite Generation of Errors

When the user edits the class name on a Joint.js diagram or in the text editor (when the Joint.js diagram type is chosen) with more than 1 word, the event triggers an infinite generation of error messages in the text editor.  Changing the diagram type or selecting a different example doesn't stop the error messages and it seems like refreshing the page is the only option the user is left with.
Consider the picture below for reference:

Proposed Change
The program should probably react in the same manner as the Editable Class diagram, where a single message appears on top of the diagram panel to notify the user that their edit causes an error. Refer to the screenshot below:

Closing because I don't want to do further work on joint.js


Day 31

Good morning! Here's your coding interview problem for today.
This problem was asked by Google.
The edit distance between two strings refers to the minimum number of character insertions, deletions, and substitutions required to change one string to the other. For example, the edit distance between “kitten” and “sitting” is three: substitute the “k” for “s”, substitute the “e” for “i”, and append a “g”.
Given two strings, compute the edit distance between them.

Going to implement a basic algorithm:
https://en.wikipedia.org/wiki/Wagner-Fischer_algorithm#Calculating_distance


docs: add documentation using jsdoc

closes #39
Changes Proposed

add config file for jsdoc
create documentation using jsdoc

@Dun-sin requested few minor changes. Please review. Other than that, it looks good.

@Dun-sin your repository seems to be deleted. Wondering if it was intentional? Is there anything I can help with in order to take this one ahead and merge?

Yep it was intentional, totally forgot about this😅
I really don't know what's with the Github security errors tho


Fix/visa adapter

Hi,
Here is the fix of the visa adapter so that write_termination is now a safe keyword (#170) .
Looking at the adapter, could we also add an argument to configure the backend?
something like:
self.manager = visa.ResourceManager(backend)
Right now my way to change the backend is to set the environment variable
os.environ['PYVISA_LIBRARY'] = '@py

@thosou, thanks for the PR. It looks like the term_chars field was dropped in PyVISA 1.6. I think a backend argument is a good idea -- feel free to add to this PR since it is within the scope.

I added the selection of the backend with the visa_library parameter. This is the one that is passed to the ressource manager in pyvisa. The default value is empty string.

@thosou, nice work! Thanks for the fixes.


Moving protocol constants into a consistent enum

various interfaces exists in CAS that do nothing other than to host protocol specific constants. It is suggested that these be merged into relevant ENUM types so we can get consistent terminology across protocols (i.e. Protocols.CAS.getServiceParameter() and Protocols.SAML.getServiceParameter(). Would allow us to create more shared components where you just passed in the protocol enum and it could extract everything it needed.

Not applicable anymore.


MSEARCH-109 Implement search by items/holdings electronic access

Following stories are similar and done in single PR:

MSEARCH-109 Implement search by items electronic access
MSEARCH-110 Implement search by holdings electronic access

Wondering if there is a way to reduce/simplify the repeats in src/main/resources/model/instance.json .

It definitely possible to implement, however for this story we don't consider refactoring of metadata loading, but I've created a story to reduce duplication for resources:
https://issues.folio.org/browse/MSEARCH-122


Make rocketmq client's stats of RT make sense, ISSUE #2088

What is the purpose of the change
fix issue #2088, to make the log output of RT stat makes sense.
Brief changelog
Change the log output format for RT stat
Verifying this change
Start consumer and consume message. After a few minutes, check the log output, the stat of RT should be like
[CONSUME_RT] [topic_xxxx@xxxx-consumer] Stats In One Minute, TIMES: 256807  AVGRT: 812.68

Coverage decreased (-0.04%) to 51.161% when pulling 6b3f339c23a57e8a2da87c6f788e3abf4659f520 on Jaskey:dev/rtstat-optimize into 2fa51fc1bd3be4d31156f40854eaddcf2b790a6f on apache:develop.

I would approve this PR, but I don't have write access to this repo, so probably on Monday we'll get it merged. Sorry for any inconvenience.

@imaffe thank you .


no response after clicking

点击"之前"按钮后没反应，不显示之前的日志
logseq版本：0.10.8
MAC客户端

Click on another row.
The line is from Logseq app and should have been hidden by this plugin as it is unnecessary.
TODO:

[ ] Remove the confusing button

After watching the demo GIF in the readme, I found that there is no line like "(---display before today)" for me, just Today journal and "Earlier"。 Sometimes, "Earlier" also disappears when I scroll up and down the Today journal.

It could be a bug, or it could be by design.　There are Journals and journal single pages. Neither is displayed on a journal single page.

Thx. Is there any native method to control the number of daily pages shown in the Journal view (namely, before "Earlier")?
I remember there could be, but failed to find it now.

There is no way to customize Journals to open multiple journal. To achieve this, it needs to be implemented as an option in the Logseq core, not as a plugin.
I think such options have the potential to make Logseq a breakthrough, but the developer team doesn't seem to be too keen on adding more small options.

For example, it should be possible to add an option to this plugin to show 3 days (starting from today) instead of just today, but without any control over those dates. In this case, it would have to be implemented in Logseq core rather than in a plugin, and the developer team doesn't seem to be too fond of small options.

same thing in
logseq 0.10.9 (83)
plugin version: 1.2.0

debug
CORS problem I guess
	Uncaught (in promise) DOMException: Blocked a frame with origin "lsp://logseq.io" from accessing a cross-origin frame.
      at zr (lsp://logseq.io/logseq-plugin-single-journal/dist/assets/index-aFl0cje4.js:12:85269)
      at nt.<anonymous> (lsp://logseq.io/logseq-plugin-single-journal/dist/assets/index-aFl0cje4.js:12:84998)
      at O.emit (lsp://logseq.io/logseq-plugin-single-journal/dist/assets/index-aFl0cje4.js:3:1788)
      at #lspmsg# (lsp://logseq.io/logseq-plugin-single-journal/dist/assets/index-aFl0cje4.js:12:12929)
      at lsp://logseq.io/logseq-plugin-single-journal/dist/assets/index-aFl0cje4.js:12:7777
  zr @ index-aFl0cje4.js:12
  (anonymous) @ index-aFl0cje4.js:12
  O.emit @ index-aFl0cje4.js:3
  #lspmsg# @ index-aFl0cje4.js:12
  (anonymous) @ index-aFl0cje4.js:12

It was the right chase!


Add support for shift+enter to insert line breaks on mac

Chrome and windows does this by default but mac/safari does ctrl+enter.
This update allows people in safari to also use shift+enter so that the behaviour is consistent.

I really wanted to write tests for this and I spent some time installing the dependcies to get selenium working but I got stuck with firefox booting and immediately crashing whenever I ran npm test ...any thoughts?

I was looking into this because of this new support ticket: https://app.bananastand.org/organizations/52/teams/1/tasks/5677662
That change to wysihtml5 does not seem to work properly for me in Chrome - often the cursor jumps down without changing the text, and if the cursor is sitting at the end of a bolded string it doesn't work at all (just acts as if enter was hit).


Confusion over Dataset Processing

Excuse me, I would like to ask you a question. I am very interested in your paper and want to do an experiment. But I don't quite understand the data processing part. Which directory should ACE2005 dataset be placed in? And what format is it needed to handle? Thank you very much for your answering.

Hi, we did not provide any pre-processing script to prepare the ACE 2005 dataset, but we outlined the pre-processing steps in the paper. You can find the format of the preprocessed data in #2.


Fetch PR info from GH before trying to open it

This allows users to open PR's outside of av and then let av know about
the state of that PR. This is mostly useful in scenarios where users open
the first (root) PR using the GH UI, then realize that they want to stack
their work on top if it. Before this commit, that wasn't possible (without
closing the original PR and then re-creating it with av pr create).
When we implement av stack submit, this should also happen automatically
(for every PR in the stack), but that's a bridge we haven't quite reached.

/aviator stack merge


MAISTRA-730 Remove istio-galley-mesh cluster role and binding

Don't merge into rc1. While it doesn't look like Galley actually ever reads nodes, we need more testing to see if that's really the case.


How to Import Existing Membership Data into IdentityServer3.AspNetIdentity Tables

I have a script that moves membership table users to identity tables. The part that imports user's password into the AspNetUsers.PasswordHash column is as follows:
INSERT INTO AspNetUsers
(
...
[PasswordHash],
...
SELECT
...
(aspnet_Membership.Password+'|'+CAST(aspnet_Membership.PasswordFormat as varchar)+'|'+aspnet_Membership.PasswordSalt),
...
To give you an example for one user I did:
insert into [IdentityServer3.AspNetIdentity].[dbo].[AspNetUsers]
(Id, PasswordHash, SecurityStamp, UserName, EmailConfirmed, PhoneNumberConfirmed, TwoFactorEnabled, LockoutEnabled, AccessFailedCount) values
('824B3B74-909A-4CD5-A7B3-C1243142C398',
'JKPDfgg/63IBjIf0EC4Pm5WRe90=' + '|' + CAST(1 as varchar) + '|'+ 'eembO/PkE9C4gXBqSX20Yg==' ,
NEWID(),
'oo7',
0, 0, 0, 0, 0);
When I use a regular MVC project that uses local accounts security I'm able to authenticate users from the Identity tables that have the imported data. But when I do the same in the AspNetIdentity sample that's part of the IdentityServer3 samples it doesn't work and throws an exception. I looked at the log file and it says:
...
System.FormatException: The input is not a valid Base-64 string as it contains a non-base 64 character, more than two padding characters, or an illegal character among the padding characters.
at System.Convert.FromBase64_ComputeResultLength(Char* inputPtr, Int32 inputLength)
at System.Convert.FromBase64CharPtr(Char* inputPtr, Int32 inputLength)
at System.Convert.FromBase64String(String s)
at Microsoft.AspNet.Identity.Crypto.VerifyHashedPassword(String hashedPassword, String password)
...
Does anybody know what I need to do with existing membership passwords so they can be used in the AspNetIdentity sample?

Correction for the tsql script in the previous post:
insert into [IdentityServer3.AspNetIdentity].[dbo].[AspNetUsers]
'(
Id, PasswordHash, SecurityStamp, UserName, EmailConfirmed, PhoneNumberConfirmed, TwoFactorEnabled, LockoutEnabled, AccessFailedCount) values
('824B3B74-909A-4CD5-A7B3-C1243142C398',
'JKPDfgg/63IBjIf0EC4Pm5WRe90=' + '|' + CAST(1 as varchar) + '|'+ 'eembO/PkE9C4gXBqSX20Yg==' ,
NEWID(),
'oo7',
0, 0, 0, 0, 0);

I realized that some custom code has to be added to the Identity system to verify imported passwords from membership differently.


Metadata lost when copying cross account

Hello, thanks for the simple utility which makes setting up aws lambda s3 copying very easy!
I was trying it out for cross account copying and noticed that the destination bucket is getting copies without the Content-Type set. It seems .. MetadataDirective: 'COPY' doesn't work in this case, have you tried copying across accounts or maybe you did this in a different way?

@gpip when copying S3 objects between S3 accounts, the object metadata will be lost if you "push" the file. Instead, you need to "pull" the object.
There are 2 ways to copy an object between AWS accounts:

"Push" the object to the target account's bucket.

Account B has the target bucket. You apply a bucket policy allowing Account A to write to the bucket.

In the Lambda function, you're executing as Account A, and you perform the copy.

"Pull" the object from the source account's bucket.

Account A has the source bucket. You apply a bucket policy allowing Account B to read from the bucket.

In the Lambda function, you're executing as Account B, and you perform the copy. This means you need to use credentials in your Lambda function aside from the role the lambda function is executing as.

In the "Pull" case, the metadata should be copied correctly. I've dealt with AWS support on this issue and they say that you need to "pull" the file rather than "push" it.  I received no indication from them if this will ever change.

@mwhouser awesome! Indeed, I was using the push model. Do you remember how you configured the bucket on Account A for this? I have a policy in place for allowing a specific user to read the bucket on Account A from the Account B, but I probably need a different policy (or a different setup) in order to display that bucket when creating a S3 trigger for the lambda func.

I've configured the bucket on Account A to allow s3:Get* and s3:ListBucket for the lambda role in Account B. I also configure an event notification on the bucket on Account A to send it to lambda function, and this part is working. The lambda function is running on Account B and receiving the events, it's role is configured to allow s3 operations but for some reason every execution fail due to access denied.
I skipped the UI to configure the lambda function and instead used the aws cli for lambda add-permission --action "lambda:InvokeFunction" --principal s3.amazonaws.com --source-arn ... --source-account ... and this part seems to be working, as it's triggering the lambda func on Account B based on events happening on the bucket on Account A.

Ah, the bucket was configured to allow s3:Get* on bucketname/*, I missed that for just bucketname so get-bucket-tagging could run. Awesome, it's working now @mwhouser


Navigation drawer drop arrow and text alignment a bit off

Bug Report
In the navigation-drawer, the vertical alignment of the name or email and the drop arrow are a bit off. Also, the cursor is a pointer on the arrow, but not the text even though the action triggers for both.
Other information
I'm guessing we can just change https://github.com/Teradata/covalent/blob/develop/src/platform/core/layout/navigation-drawer/navigation-drawer.component.html#L13 to add class cursor-pointer and attribute layout-align="start center".
So from

to

(or move the click event down to the arrow button for the pointer issue)

latest looks ok


Bug: Javascript error

Chat2DB Version
3.0.9
Describe the bug
windows, when I installed, the error happened:
Uncaught Exception:
Error: Command failed: 9windir96 System32 REGexe QUERY
HKEY LOCAL MACHINEYSOFTWARE Microsoft Cryptography /v MachineGuid
at checkExecSyncError (node:child process:865:11)
at execSync (node:child process:936:15)
at node:electron/js2c/asar bundle:2.12758
at u
(F: chat2dbresources applnode modules (node-machine-id dist indexjs:1:7825)
at new Analytics4 (F: chat2db resources app src main ga4,js:7:49)
at registerAnalytics (F: chat2dblresources applsrc(main analysis,js:7:21)
at App.sanonymouss (F: chat2db resources app src main indexjs:56:3)
at Appemit (noderevents:513:28)

请提供下系统信息


Implement Drop for Device

Device context field (ID3D11DeviceContext) have to call .Release() in Drop impl.

bors r+

bors retry


Add Recorded Future Identity Solution.

Change(s):

Add Recorded Future Identity Solution, consist of 5 playbooks:

"RecordedFutureIdentity-search-workforce-user" playbook - to search new exposures for Workforce users
"RecordedFutureIdentity-search-external-user" playbook - to search new exposures for External users
"RecordedFutureIdentity-add-AAD-security-group-user" playbook - to add risky user to Active Directory Security Group for users at risk
"RecordedFutureIdentity-confirm-AAD-risky-user" playbook - to confirm to Active Directory Identity Protection that user is compromised
"RecordedFutureIdentity-lookup-and-save-user" playbook - to lookup additional information on a compromised user and save results to Log Analytics

Reason for Change(s):

This Solution shows use of new Recorded Future Identity API via Recorded Future Identity Connector.

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

@anki-narravula Can you please approve playbooks.

Hi @recordedfuture-dev
Can you please incorporate the changes requested by @anki-narravula
Thanks

We now have a mainTemplate.json that validates by Test-AzTemplate. But we run in to a issue when testing using Custom deployment.
Deployment template validation failed: 'The resource 'Microsoft.Resources/templateSpecs/OskarFreeRF-pl-b4mxdohwsqsq4' at line '82' and column '9' is defined multiple times in a template.
Do you have any advise?

@v-mchatla Please look into this.

@NikTripathi We now have a Working solution


args.half_or_full_lora can't set to full

Hello, I just change the args.half_or_full_lora  from half_skip_attn to full and run train.py. Then a dtype issue occurs. Do you know how to fix this? Thanks!

try to disable xformer attention


C: Enhance Face hover effect and change it's page title

I would like to pick up this issue under GSSOC'22. Midway through it. Kindly assign it to me.

I would like to work on this issue

We assign on FCFS basis, @777ahesh you can checkout other issues

@aachal28 do fast

Dont close this issue my project is ready ill upload tommorrow

i would like to work on tjis

@aachal28 what's the update?


Use a more descriptive subtitle in the README

:+1:

:-1: from me. I like the sarcastic nature of the current README subtitle and think it's important to not take this library too seriously. The subtitle for the repo is more descriptive, and is what shows up in search. The rest of the README should be able to describe what this lib is.

The rest of the README should be able to describe what this lib is.

Yup, I agree. Maybe this is really #26 all over again. I think examples would help soften the blow of the current subtitle. I'll close.


.cody/ignore

Introduced in https://github.com/sourcegraph/cody/pull/1382
Currently behind the cody.internal.unstable configuration.
Once we have gathered extensive internal feedback, we should move it to experimental stage and enable it by default for external input.

[x] Work in a single workspace
[x] Work in non-indexed workspace
[x] Work in multi-root workspace
[x] Work with nested repositories
[x] Work with Chat
[x] Work with Commands
[x] Work with Edits
[x] Work with Autocompletes
[x] Work on Mac
[ ] Work on Windows

To be discussed
Do we need it to work with the following?

[ ] Work with local embeddings during indexing stage https://github.com/sourcegraph/bfg-private/issues/104
[ ] Work with remote embeddings during indexing stage
[ ] Work with symf during indexing stage

Follow up

UX / UI

how to let user know that their files were ignored (if needed)

[ ] in chat
[ ] in commands
[ ] in autocompletes

how can users find out about this feature

[ ] Doc update

### Tasks
- [ ] https://github.com/sourcegraph/bfg-private/issues/104

Question for you @DanTup 🙂
Can the windows test we just merged help us with confirming/checking if the .cody/ignore file is working on Windows?

If there are e2e tests, it should (the other tests were already running on Windows).
I can manually test things on Windows too if you want some extra validation (let met know if there are specifics things you want tested).

We do have a context test here that validates ignores that works correctly on Windows:
https://github.com/sourcegraph/cody/blob/4fb31b80f5a1d6fcaa661d22d9911f2133428cef/vscode/src/editor/utils/editor-context.test.ts#L64
I don't know if it covers all the places where files could be returned though. It's checking that getFileContextFiles() is filtering out the ignored files returned in vscode.workspace.findFile but it might not be verifying if those ignored files show up from other context sources.

@DanTup thanks for the info. I'll create an issue to add an e2e test for Cody/ignore.
Will also create a separate issue for verifying if the context returned from different context sources are also filtered correctly.
I was thinking since all the context will be turned into a context message, we can add do that during that process like we did for the interaction class. Do you think that'd work?

I was thinking since all the context will be turned into a context message, we can add do that during that process like we did for the interaction class. Do you think that'd work?

Sorry, missed this. I'm not sure I understand the question though - isn't the interaction class already filtering the context messages? I'm not very familiar with how the other context sources (like embeddings etc.) work, but I guess we should ensure that if any of those sources accidentally return ignored files, they get filtered out (not just going to the LLM, but in the UI so it doesn't look like they would be sent).

nm - I saw https://github.com/sourcegraph/cody/issues/2911 / https://github.com/sourcegraph/cody/pull/2913 :)

nm - I saw https://github.com/sourcegraph/cody/issues/2911 / https://github.com/sourcegraph/cody/pull/2913 :)

Haha yea! Just added you as a reviewer! Broke the e2e tests while I was trying to fix an issue on windows, so will need to fix that on Monday lol

@kalanchan are we good to close this out since all the initial items have been completed?
I understand there are more works that the product team is planning on, and we can create a new issue for to track those when they are ready. Does this sound good to you?

@abeatrix yeah it seems like embeddings is the last thing to work on from this tracking issue? if so then its fine to close this as we wait for the next PRD to be approved and ready to go


Mac OS X

Hi,
Any plan to support Mac OS?
Thanks

Good question.
I wrote this FAQ Can Javet Support Mac? when I started this project. So far, I have no plan on purchasing a decent Mac device in the near future.
I wonder if someone else could work on that, or anyone could donate one to me.

@caoccao Is it possible for you to use some sort of VM for this?

@caoccao Is it possible for you to use some sort of VM for this?

I seriously doubt there might be slight difference. Some Intel NUC boxes supports MacOS and I have a pretty decent one. However, that only resolves x86-64. I haven't heard any news on VM for arm64.
Also, I think 2 Mac Mini boxes (x86-64 + arm64) might be quite ideal in terms of the ROI. I believe if the demand was so high, some organizations would donate me these assets. At least, I don't have plan to purchase them in few years (mainly for financial reason).

Oh, I see. Well you've already done a lot of great work in supporting Windows and Linux, and before I was going to use GraalJS, but seeing how Javet uses V8 directly, I plan to use it instead. Your work is very promising, thank you for working on Javet like this!

Oh, I see. Well you've already done a lot of great work in supporting Windows and Linux, and before I was going to use GraalJS, but seeing how Javet uses V8 directly, I plan to use it instead. Your work is very promising, thank you for working on Javet like this!

Good choice, and thank you.
From architectural perspective, GraalJS might impress people in PoC, but keep surprising the development team in production environment, simply because evil hides in details. There could be thousands of tiny inconsistencies between GraalJS and V8. These annoying things would drain the development resource and cause serious outages from time to time. Going with the authentic V8 is the right decision to make, I think.

Updated: I think a virtual macos environment (VPS) may also work.

Hi Folks,
My MacBook mid-2012 was revived with Mac OS Catalina which is the lowest version supported by latest V8. I managed to have built snapshot version (x86_64) only for testing purpose. However, this won't last long because next time V8 abandons Catalina, I won't be able to create any new builds unless someone takes care of the Mac OS build or I get enough donation for a new device.
You are welcome taking a try and I look forward to your feedback. Please visit the dev branch for detail.

The x86_64 version is published.

The arm64 version is supported.


NPE in #onMessage when retryTemplate!=null and recoveryCallback==null

From the code in RetryTemplate it is valid to have no recoveryCallback.

@urs-ikentoo-com Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@urs-ikentoo-com Thank you for signing the Contributor License Agreement!

Agreed.

Merged as https://github.com/spring-projects/spring-integration-kafka/commit/6b9d8e22bc6ca562f6aafa04972873e6deab0747.
Back-ported to 3.0.x as https://github.com/spring-projects/spring-integration-kafka/commit/7f135baa95016c0413fefc0a027fd27ee90e28f8 after fixing conflicts.
Cherry-picked into master as https://github.com/spring-projects/spring-integration-kafka/commit/7f135baa95016c0413fefc0a027fd27ee90e28f8 with some conflict resolutions, too.
@urs-ikentoo-com ,
thank you very much for your patience and effort.
Looking forward for more contributions!


jetson-utils target missing

Hi,
After building jetsen-inference (as described here) I try to depend on it from a custom package.
However the find_package cmake call cannot fully find it and throws the following warning:
CMake Warning at custom_package/CMakeLists.txt:25 (find_package):
  Found package configuration file:

    /usr/local/share/jetson-inference/cmake/jetson-inferenceConfig.cmake

  but it set jetson-inference_FOUND to FALSE so package "jetson-inference" is
  considered to be NOT FOUND.  Reason given by package:

  The following imported targets are referenced, but are missing:
  jetson-utils

There seems to be an installation error which prevents jetson-utils to be found.
Has anyone experienced anything similar before? Any help would be appreciated.
System info:
Jetson tx2
Ubuntu 16.04
ros kinetic

Did you try find_package(jetson-utils) first?
There is an example CMakeLists in the 'Creating your own Image Recognition Program (C++)' step.

From: Paul Verhoeckx<EMAIL_ADDRESS>Sent: Friday, September 27, 2019 7:42:44 AM
To: dusty-nv/jetson-inference<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [dusty-nv/jetson-inference] jetson-utils target missing (#417)
Hi,
After building jetsen-inference (as described herehttps://github.com/dusty-nv/jetson-inference/blob/master/docs/building-repo.md) I try to depend on it from a custom package.
However the find_package cmake call cannot fully find it and throws the following warning:
CMake Warning at custom_package/CMakeLists.txt:25 (find_package):
Found package configuration file:
/usr/local/share/jetson-inference/cmake/jetson-inferenceConfig.cmake

but it set jetson-inference_FOUND to FALSE so package "jetson-inference" is
considered to be NOT FOUND.  Reason given by package:
The following imported targets are referenced, but are missing:
jetson-utils
There seems to be an installation error which prevents jetson-utils to be found.
Has anyone experienced anything similar before? Any help would be appreciated.
System info:
Jetson tx2
Ubuntu 16.04
ros kinetic
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/dusty-nv/jetson-inference/issues/417?email_source=notifications&email_token=ADVEGK5EXYASDLICFICV2N3QLXWTJA5CNFSM4I3FLDD2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HOEGPBA, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADVEGK6NNQF5M2LTVYRBVG3QLXWTJANCNFSM4I3FLDDQ.

This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.

That fixed it, thanks!


Automate nginx config

The arm templates in this repo automatically install the vms using nginx as a dns forwarder, but as written they require the user to manually configure the nginx dns forward after deployment.
I've modified the scripts to automate this and eliminate the need for manual config.
Please see PR #2

It has been committed. Thanks a lot for your contribution.


fix typo

Coverage remained the same at 98.904% when pulling 00dab39b630d8dabcb83950cb44afab78f60eeaa on Balatzar:patch-1 into 47ce4ab8c6aa64c229d1d340efd1f3cc3e2750bb on coryhouse:master.

Thanks!


Add Getting Started section to README

README Update
Please add a getting started section to the README so folks Know how to start with this projecst

Added Getting Started section in README file


Update dependencies

This will be required in order to be able to support latest features of LSP (like labelDetails)
It's not ready to get merged because there is a bug in ZLS
Removing this field fixes the issue: https://github.com/zigtools/zls/blob/fc5b1c64579e0ec33ea374051bcfef59ebfc0c5c/src/types.zig#L245
Something about ranges is wrong, i tried to investigate today, but i couldn't find the cause, so i just commented that line.. so far it seems to be ok
I'll try to spend more time on it tomorrow, since i need that to get sorted out to be able to send a PR to add labelDetails support

LGTM, thank you so much! Yeah, ranges are always a pain point, best of luck figuring them out. <3


[release-4.8] Bug 2032321: Avoid hardcoding the JAR names that contain the JMSAppender class

/bugzilla refresh

/label backport-risk-assessed

/label lgtm

Manually adding the requisite labels since the images CI check passed. For more context see https://coreos.slack.com/archives/CJARLA942/p1639507843420000.


Instant tooltips

Moved from the v2022.3 release criteria:

On anything with title text, show our own popup with the text right away, rather than make people wait for the browser.

GitHub has done this forever, for example. It’s a big big usability win.

Focusing on a title-text element with keyboard navigation should ideally also pop the tip.
(Nice to have:) Today there are elements where if their text content overflows, we truncate the text with an ellipsis; we also add title text. The title text is added even if the text content does not actually end up overflowing: the title text is added if the text content might overflow. However, ideally we show an instant tooltip only if the text content actually overflows.

Tested wit Success!

Tested with Success!

I've filed an issue with design review feedback at #753. We didn't end up making those changes for .2, but we're planning to do so for .3.


Retire old site and delete related resource

As a result of this release, we should retire original website, and delete Azure SQL server.
Switch all bits to new created droplet.

all things are moved to new droplet.


Winter update; Audio asset list & review list

For the winter update, heres the audio to update, revisit, proof up;
===================================================
==================================== vanilla speed dial =
review sounds =

drugs
music (integrate adesi music mixer?)

add new sounds =

stinger game start (8 seconds long)
stinger game over (8 seconds long)
stinger domination (dynamic sound, intensity rises with kill count)
stinger revenge

voiceline sounds = (TBD)

character voiceline [game start] "lets do this"
character voiceline [got a kill] "Heh" "MMmmm"
character voiceline [dead] "Doh!" "Ow!"
character voiceline [domination] "I'm amazing!"
character voiceline [revenge kill] "Back at ya!"
character voiceline [winner MVP] "I'm the best!"

soundscapes assets (indefinite rn)
zombie music assets (indefinite rn)

==== Decide implementation for zombie music, heres rough concept layout:

pregame-no-action = play soundscape (includes spooky sounds, distant zombies, ambient musical elements)
wave-begins = music begins (rhythmic, dramatic, possibly interactive & random elements?)
wave-over = music stinger and fadeout (uses beat sync transition w/ adesi music system)

music sidenote: interactive music elements require specific code design for that composition, this would complicate the 'randomized-playlist-of-all-composers' approach for music we typically do at Eagle One. Possibly we randomize the music of all Eagle One composers on a "per-wave-basis". Everything here can be accomplished with just a bit of effort. TBD.

Love that you put stingers for game start and over on there, I only recently thought about that haha
We could also use some generic damage/impact sounds, that's part of the fx I wanted to improve on for classic speed-dial
You're also an MVP for putting so much consideration into the implementation for more complex music features. The s&box sound API is extremely lacking at the moment, coming up with proper use cases for features to be added would be an excellent use case for this


Create user email subscription history

https://app.clubhouse.io/enmarche/story/3660/stocker-l-historique-des-choix-de-notification-des-utilisateurs

@illusionOfParadise Please don't squash your commits on big PR like this so that we don't need to review everything every time.

I am taking over this PR while @illusionOfParadise is out.

@illusionOfParadise @ottaviano @Remg @cacahouwete Ready for review


SOS signal format

From Wikipedia:

[...] to indicate that the Morse code equivalents for the individual letters of "SOS" are transmitted as an unbroken sequence of three dots / three dashes / three dots, with no spaces between the letters.

The SOS signal shouldn't be as it is currently:
... [space] --- [space] ... [space] ... [space] --- [space] ...
but instead
...---...[space]...---...[space]...---...[space]

Hmm, I did not realize that SOS was essentially treated as its own letter in Morse Code - always thought it was three letters. Thank you for pointing this out, I'll fix this for the next release.


calculate_remove_liquidity

Add a function to the Rust SDK that calculates the amount of proceeds and withdrawal_shares received for removing liquidity. Traders and UIs could use this to preview their returns without hitting the chain.

This is related to #862. Both of these will require _distributeExcessIdle to be ported over.

Several PRs in flight for this:
https://github.com/delvtech/hyperdrive-rs/pull/17
https://github.com/delvtech/hyperdrive-rs/pull/11
One landed:
https://github.com/delvtech/hyperdrive-rs/pull/7
Will need to wait for updateLiquidity update due to k-inflation changes, then I'll put up a PR for calculate_distribute_excess_idle_share_proceeds and calculate_distribue_excess_idle.  The functions are written already, but need to add testing and wait for these PRs to land.


Update censys to use new dataset

Closing as duplicate of https://github.com/cisagov/crossfeed/issues/1857


Replace Translate component with native Google Translate UI

Overview
Recent changes to the app broke the previous implementation of Google
Translate using the Translate.jsx component. Further investigation
revealed that the way the Translate component had been set up was not
really in harmony with how Google Translate is supposed to be set up --
which increased its surface area for bugs and made it incompatible with
existing documentation like
https://www.w3schools.com/howto/howto_google_translate.asp
To remedy this, we drop the Translate component and use the standard
method for setting up Google Translate: giving Google Translate a div on
which to create its own UI that's managed outside of React. This
circumvents having to manually set the hash, call window.reload, and
sync the cookies. It also puts the Google Translate usage back into
harmony with the documentation that is available online.
Note that it turns out that Google Translate for websites seems to be
deprecated, so although it is currently working here we should probably
be prepared for it to break at any point in the future.
Connects #281
Demo

Notes
Just to reiterate: it appears that Google translate for Websites is not accepting new signups. The landing page here https://translate.google.com/intl/en/about/website/ suggests that it will continue to work for websites that have already signed up, but it also suggests directing users to make use of browsers with built-in translation extensions.

One additional thing: as depicted in the demo gif above, there's an interval between when the app starts and when translation is available -- and in this moment, the navbar does not have a "Select Language" or selected language tab. This is a consequence of waiting for the function call to return, and although it's visually different than what was there before I believe this is the correct way for it to behave.
Testing

serve this branch then use the translation element and verify that the bugs described in #281 no longer occur

What do you think about incorporating these changes?

I wasn't really able to find any documentation about terms of service for using Google Translate, and I assume it's mostly been removed since the service stopped accepting new signups. However, I was wondering whether there's some terms of service thing that required displaying the icon too?
Not having documentation around means we'd probably just be speculating one way or another. If you think it's ok to go without the icon, I'm happy to add the patch above. Alternatively, we could keep
b.childNodes[0].innerHTML = 'Language';

from the patch but drop
+.goog-te-gadget-icon {
+    display: none;
+}

... which would just shorten the string back to "Language" but keep the icon.
Happy to do whatever we deem best, though! As mentioned above, it seems like the translation thing is no longer supported to it's hard to guess accurately about what its terms were.

I wasn't really able to find any documentation about terms of service for using Google Translate

I would like to hide the icon since there is no indication that we can't do so.

I would like to hide the icon since there is no indication that we can't do so.

:+1:

Sounds good. This was accomplished in
09ba486

Hm, that timeout doesn't seem to be universally working. Going to see if there's a onload handler.

I force pushed
c9a4764 which adjusts the style on load by using a MutationObserver: https://developer.mozilla.org/en-US/docs/Web/API/MutationObserver that gets removed once it has been executed.
I also adjusted how the code manipulated the navbar nodes: using .innerHTML on the "Select Language" node broke a feature whereby the language was propagated into the Navbar, so I kept "Select Language" around but added some logic for removing the dropdown arrow icon.

If we just want to have it always say "Language" then I can revert back to using = innerHTML

I added 83ac59c  which is an unrelated commit that removes the LandingAlert component which we decided to drop in #325

c6f9d2f replaces the for loop which removed some elements from the tab with a while loop. This was done because the for loop would run past the end of the childNodes list since removing an element from it would change its size.

Great! Thanks for your help with this. I'm going to squash then merge once the tests pass.


Add Set, Set.Int, Set.String, Set.Dict docs

Resolves #20

this is fantastic, thank you very much @baransu 👏👏👏

Great work @baransu 🚀


Update Score Calculation

Edit: References #369
It will be necessary to change the high score calculation based on game difficulty and gamemode (survival mode will have a different score factor)
Update the way high scores are calculated to account for game difficulty. Harder difficulty will grant a higher score modifier while lower difficulty will do the inverse.

Spent about 20 minutes on this task figuring out how to modify the score and playtesting a little bit to make sure my changes worked and are correctly affecting the score.


Role authorization scoping for 2.0

Describe Problem
There is no current authorization scoping in 2.0
This means that anyone can run any command
Suggest Changes
Add different scopes with different levels of privledges
Provide Examples

This would be an advanced issues scoping. As command permissions cannot be revoked, the minimum role that can use this command is required.
The config for role permissions would look like
command_permissions:
    <command name>:  "<role name>"

example:
command_permissions:
    whereis:  "everyone"
    ping: "pong"

In this example, everyone can use whereis but only people with the role pong can use the ping command.
This would be done for every command

This is taken directly from Isaacs branch
global_features:
  - pong

guilds:
  "your_guild_id_here":
    name: "your amazing server"
    features:
      - purge
    role_perms:
      "Admin":
        - purge

so this works in 2 parts. Global features and guild features.
Global features are enabled on all servers for everyone.
Guild features are enabled on the specific server for the specific role.
under the guilds section, you have features and role_perms
features is what features are enabled.
role_perms are what features can be used at that role or above.
for example, if I have 2 roles called admin and guest. admin is above guest. We want to have a purge command that only admin can use and a map command that everyone can use no matter the role.
global_features:
  - map

guilds:
  "123455":
    name: "example server"
    features:
      - purge
    role_perms:
      "admin":
        - purge

since admin is above guest and admin has purge both enabled and perm set, it will work for only admin.
lets say we have a security issue in purge, we would want to disable it without removing the role perm config. That is easy, you would just do
global_features:
  - map

guilds:
  "123455":
    name: "example server"
    features: []
    role_perms:
      "admin":
        - purge

this isnt needed anymore. discord integrations can do it now. permissions and scoping will done manually by server owners.


fix implied_bounds_entailment lint

Hey :wave: this crate unintentionally relied on a soundness bug in the Rust compiler, see https://github.com/rust-lang/rust/issues/105572 for more info. This lint will be a hard error in a future release and is fixed by this PR.
The issue was that inside of deref, we assume that Self::Target has to life to 'static even though that's never proven by the caller of deref because deref returns &'static Self::Target. Users of Deref for Static<T> always only used the signature of the trait method, so the stronger bound on the impl did not impact them.

Thank you for fixing this here, and throughout the ecosystem!


Adding the IDisposable interface to ECDsaSecurityKey, RsaSecurityKey and X509SecurityKey

Fixes #1434.
One of the points brought up in the issue mentioned above is that we should avoid caching and retrieving signature providers that have been disposed of. I've made a note to handle this in the caching PR (https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/1575).

we decided not to do this.


experience cloud search

[ ] shortcuts must look in Network  object which contain list of Experience Cloud site.
[ ] evaluate other missing object from SF list : https://developer.salesforce.com/docs/atlas.en-us.object_reference.meta/object_reference/sforce_api_objects_list.htm

not sure if really usefull anyway, we can look to object to check which one are usefull...


Default iteration details

Added more details to better outline what Default iteration is expected to affect -- namely Backlog creation of work items.

@KathrynEE could you review this?
@KimPlausible In the future, you can use the vsts-docs-pr repo (the private repo) rather than the vsts-docs repo (the public repo) for changes to articles. You should have permissions to vsts-docs-pr.

@KimPlausible - the new repo to use is azure-devops-docs-pr.
#sign-off


Date Types

Thinking about dates & timestamps…
Currently your types are restricted to various known permutations of strings.
Maybe you can add some things like:
            date_in_past
            date_in_future
            date (default, goes either direction, unbounded)
            date_in_past_N
            date_in_future_N
            date_N

The “N” would be the number of YEARS for the bounding.  Perhaps offer just a couple out of the box.
For “time” data type, I would not offer a control option.  Time is just a simple random 24-hour time.
For a timestamp, I would offer only permutations that control the date range.
By doing the past, future, and either direction, you don’t have to read multiple parms.

This was released in 0.000011.


fix(PrettyPrinter/Delaborator): unify default behaviour of pp.proof with false

Previously the behaviour of set_option pp.proofs false was different from not setting the option at all, this seems surprising and possibly unintended to me.
This can be seen in the following example:
def List.nthLe (l : List α) (n) (h : n < l.length) : α := sorry

set_option pp.proofs false -- comment and uncomment this line

example : List.nthLe [1, 2] 1 (by simp) = 2 := by
  -- cursor here, and click on (_ : 1 < List.length [1, 2]) to display the widget
  -- with this change commenting and uncommenting line 3 does nothing
  -- without change the widget display changes from
  -- @of_eq_true (1 < List.length [1, 2]) (eq_true_of_decide (Eq.refl true)) : 1 < List.length [1, 2]
  -- to @of_eq_true (1 < List.length [1, 2]) (_ : (1 < List.length [1, 2]) = True) : 1 < List.length [1, 2]
  sorry

It is intentional.  This is the same as the Lean 3 behavior (minus the type of the proposition, I think we only showed the underscore).  By default, proofs are hidden unless you are printing a proof.
Note that there's also pp.proofs.withType to customize this even further.


Added primitive piecelists

These seem to be a net slowdown? But I'm reasonably confident I can get it to be an overall speedup.
If @japanoise will let me, anyway.

Codecov Report

Merging #40 into master will increase coverage by 0.89%.
The diff coverage is 95.83%.

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   89.41%   90.31%   +0.89%
==========================================
  Files           3        3
  Lines         633      671      +38
==========================================
+ Hits          566      606      +40
  Misses         60       60
+ Partials        7        5       -2

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 108fe3b...0fc690c. Read the comment docs.

We even get a positive codecov score for the diff.

lovely. Good job.


Feature to clip the data point along with radius.

Describe the bug
The Scatter chart in the image below does not clip the bubble.It considers only the data points. The ScatterChartData has a attribute called clipData , which checks if the data point is within the chart and then decides to clip it or not.
However, it doesn't consider the radius of the bubble and hence, even if the data point is inside, the bubble might be so big that it causes it to go outside the axis, which is not a good UI/UX.
To Reproduce
It is fairly easy to reproduce. Take any example of scatter chart and increase the radius until you see it cross the axis.
Screenshots

Possible Solution
So, currently we have clipped the bubble (see the topmost bubble), and it doesn't cross the chart, but it does cross the axis. So, what I feel would be a better idea is to crop clip the bubble inside the chart. So, we get something like this

Here, the bubble doesn't cross the axis.
Versions

Flutter version: 2.10.0
fl_chart version: 0.41.0

It has been fixed in 0.45.0 with your help :))
Thank you!


Is maintenance like optimize, vaccuum and rewrite API in scope

Sorry for the incomplete issue, I pressed enter and it submitted the incomplete issue.
I would like to know whether table maintenance commands as described in https://stackoverflow.com/a/77763196 is in scope.
Thanks.

Sorry for the incomplete issue, I pressed enter and it submitted the incomplete issue.
I would like to know whether table maintenance commands as described in https://stackoverflow.com/a/77763196 is in scope.
Thanks.

Hi, @ibotty Thanks for your interest. The goal of this library  is to have same complete features as java/python API, so it will includes transaction API for these  features.
But the optimization commands rely on a distributed query engine, and the  roadmap is to integrate this library with rust based engines, such as datafusion, polar-rs.


Right alignment broken

In updating the tests for #152, I noticed an anomaly in the output tests/alignment.sil. The test is passing but I don't think it should be. There is some sort of issue with the last line of the ragged right paragraph. For some reason the last line overflows into the margin that it should be starting from.

There is a bug somewhere in the line ratio code; you can see it in the Conan Doyle test, as there are spaces at the end of lines which are meant to be justified. I think it may have come in after the changes to discretionaries.


[EDA] System Tools: Refresh All Administrative Account Names

Critical Changes
Changes
Issues Closed
New Metadata
Unpackaged Metadata
Deleted Metadata
Testing Notes

QA Passes - Merging


mouse

Microsoft PowerToys version
V0.71.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Mouse Utilities
Steps to reproduce
The mouse lookup function remains effective even when the game mode is enabled.
✔ Expected Behavior
The mouse lookup function does not take effect when the game mode is enabled.
❌ Actual Behavior
No response
Other Software
No response

What games did you use?
/needinfo

LOL，and turn on the game mode.
发自我的iPhone
------------------ Original ------------------
From: Aaron Junker @.>
Date: Thu,Aug 10,2023 5:26 PM
To: microsoft/PowerToys @.>
Cc: LY199610 @.>, Author @.>
Subject: Re: [microsoft/PowerToys] mouse (Issue #27926)
What games did you use?
/needinfo
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

LoL == league of legends


C++ Maximum Width of the Binary tree - medium Level

🚀 Feature
(A clear and concise description of what the feature is.)
Have you read the Contributing Guidelines on Pull Requests?
Yes I have read all the contributing guidelines on Pull requests
Motivation
It is a simple yet medium-level problem of the Binary tree that asks you to find the Maximum width of a binary tree given.
Pitch
this problem surely will help many to understand how level-order traversal works and the implementation of a binary tree

@HarshCasper @ankitaggarwal23 I am assigning this issue to work under GSSOC'21
Please arrange to add the suitable labels to the issue

/assign

Problem already present #3656


Using FileProviders without views compilation?

I have multi-modular asp net core application which is working perfectly in core 2.2. Unfortunately core 3.0 brings some braking changes and we have problem now.
We use PhysicalFileProvider for loading views from referenced mvc projects (where each is separated module). We do not compile views.
Thanks to that:

in Debug mode I can modify views without recompilation of any project
in Release mode we can copy views from referenced projects to proper structured folders in the main application and publish it

After upgrading to core 3.0 this solution doesn't work.

First of all - issue #7647
I found that I can disable compilation setting RazorCompileOnBuild to false but that causes my ViewLocationExpander (we use Feature Folder structure) to stop working
Setting MvcRazorRuntimeCompilationOptions have no effect without calling AddMvcRazorRuntimeCompilation. Again: We do not compile views

services.Configure<MvcRazorRuntimeCompilationOptions>(opts =>
    opts.FileProviders.Add( new myFileProvider() )
);

We want to use PhysicalFileProvider to indicate where the views are located but without compilation.
Is there any possibility to get it working again like in core 2.2?

I would like to add my weight to this issue.
I was previously using (and still do in the .net core 2.2 version of my project) the following approach:
services.Configure<RazorViewEngineOptions>(options =>
{
     InstallerProviderItem.RazorViewEngineFileProviders.ForEach(x => options.FileProviders.Add(x));
});

Here the RazorViewEngineFileProviders contains a DatabaseFileProvider and an EmbeddedFileProvider for each 'plugin' assembly, along with IChangeToken similar to this article and all works fine. Views that are edited in the database are refreshed after being edited on next page load.
Using this approach:
services.Configure<MvcRazorRuntimeCompilationOptions>(options =>
{
     InstallerProviderItem.RazorViewEngineFileProviders.ForEach(x => options.FileProviders.Add(x));
});

It does not work the same. The documentation states that code has been 'moved', but it also does not function as previously. Also, the DatabaseChangeToken no longer works and any views edited in the database no longer refresh unless the app is restarted.

@dario-l, using RazorViewEngineOptions.FileProviders was a feature of runtime compilation. Configuring it effectively meant you were configuring the behavior of compiling views at runtime. To get the effective behavior in 3.0, you will need to call AddMvcRazorRuntimeCompilation.
I've added a sample that shows using runtime compilation in dev time using IFileProviders while using compiled views on publish: https://github.com/aspnet/samples/blob/master/samples/aspnetcore/mvc/runtimecompilation/. Here's the relevant bits to set up runtime compilation: https://github.com/aspnet/samples/blob/master/samples/aspnetcore/mvc/runtimecompilation/MyApp/Startup.cs#L32-L41 in Debug \ Development.

@pranavkm So, I can not use FileProviders now without any kind of views compilation (like in core 2.2)?

OK, I found it.
Adding <MvcRazorCompileOnBuild>false</MvcRazorCompileOnBuild> to the MyApp.csproj causes that compilation is not executed but view is refreshed.
I hope that behavior will be not removed (silently) in the future releases. :)


SilkTouchProtection is globally disabling all xp and money gain irregardless of restrictedBlocks.yml.

Compiled source as of https://github.com/Zrips/Jobs/commit/3f71080b028ac26e85fab586f37b319961e08ee3
Paper dev 1104 (Spigot 1.11.2)
SilkTouch exploit protection is globally disabling all experience and money gain irregardless of what is set in the restrictedBlocks.yml. Shouldn't it only disable gains for items in the restrictedBlocks list and even then, only disable the money payout, not both experience and money per the config note?
    # Enable silk touch protection.
    # With this enabled players wont get paid for breaked blocks from restrictedblocks list with silk touch tool.
    SilkTouchProtection: true

Player joined digger class.
Player tried to dig up sand with a silk touch enchanted diamond shovel.
Player did not receive any money or experience for digging up sand.
Sand is not in my restrictedblocks list.

restrictedBlocks.yml
http://paste.ubuntu.com/24663408/
Digger class.
http://paste.ubuntu.com/24663412/

Will include only blocks from restrictedBlocks list with next build


Add Gravitino into hadoop-testing

project: https://github.com/datastrato/gravitino

We use supervisord to manage the services, it requires the sub-process to run in frontend while Gravitino does not support that, opened https://github.com/datastrato/gravitino/pull/3749 to implement such a feature.


As a user, I want the vars & props for the Button component documented on the site

Scenario: Vars Table

[ ] Create a new folder in /docs-src/src/pages/components called InputGroupTables
~- [ ] Create a new file in that folder called InputGroupVars.js~
~- [ ] Table details:~
~- [ ] Table Headers: Variable Name, Default Value, Description~
~- [ ] On small screens, there should be a Table.HeadSmall that says InputGroup Vars~
~ - [ ] The Table Data should include [Column Header] before the table data so that the headers shows on small screens when the table stacks.~
~- [ ] Make sure to check the settings file for any vars that were not listed at the top of the components stylesheet.~
~- [ ] Import the InputGroupVars file into the main example page: /docs-src/src/pages/components/InputGroup.js~
~- [ ] After the ExampleSection include
Variables:
~
~- [ ] Then call in the table: <InputGroupVars />~

Scenario: Props Table

[ ] Create a new file in the InputGroupTables folder called InputGroupProps.js
[ ] Table details:

[ ] Table Headers: Name, Type, Default, Description
[ ] On small screens, there should be a Table.HeadSmall that says InputGroup Props
[ ] The Table Data should include [Column Header] before the table data so that the header shows on small screens when the table stacks. Example:

    Name: shrink

[ ] Include all possible props (also check the components file to make sure you didnt miss any props that were not listed on the site.)

[ ] Import the InputGroupProps file into the main example page: /docs-src/src/pages/components/InputGroup.js
[ ] After the Variables section include
Properties:

[ ] Then call in the table: <InputGroupProps />

I've pushed up what I have for this ticket to my branch 269-inputgroup  -tables.
Props Table:
Ready for someone to take over:

[ ] Need to double check that I have included all the possible props.
[ ]  Need to double check that the type column has the correct types for each prop.
[ ] Check that descriptions are accurate + finish filling in the ones that are incomplete


[Viewer] Fix #144 empty viewer output for Discrete Rates

I'm missing some controls that could be there:

ability to color the lines
ability to control the width of the lines
ability to filter lines by their attributes (posterior prob or bayes factor)


index.md: corrected title & localeTitle's format

Included the "" so that the format for title and localeTitle would be correct and more easily readable.

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

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

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for opening this pull request.
Book recommendations are beyond the scope of the freeCodeCamp Guide, so we've decided to go ahead and remove these book recommendation articles.
I encourage you to review this book in the ratings category of the freeCodeCamp forum. Your recommendation will reach way more people there.
Thanks again recommending this book, and for contributing to the freeCodeCamp Guide.


Consolidate checks for (uninstantiated) templates to use shared helper function in compiler

This confuses us a lot and the logic should be in one place. See also: https://github.com/microsoft/cadl/pull/600#discussion_r892505440

estimate: 3


Add more tests for Date Matchers

The code for Date Matchers (package io.kotlintest.matchers.date, file matchers.kt) has a lot of extension  functions to match Dates (LocalDateTime, LocalDate, ...).
There is one test class, called DateMatchersTest. However, some methods are not tested in this class. Some examples:

fun LocalDateTime.shouldHaveSameYearAs(date: LocalDateTime);
fun ZonedDateTime.shouldBeAfter(date: ZonedDateTime)

and some others.

The code is very trivial (only delegate to another function), but unit testes are needed nonetheless.
What does this solve?
It doesn't solve anything now, but for future refactors of these matchers, there should be a test scenario for each of them.

Currently working on this.

It will be good to cover this.

Merged, resolving this issue.