add setFieldError and setError functions to the useField hook

I have a use case where I have to set an error to a field if a specific field his value changes.

Shit, do you have a .prettierrc file @JoviDeCroock  :P ?

Hmm, this feels very imperative since I was actually contemplating on removing setFieldValue all together from useField this in favor of a new hook called useSpy where we can observe a certain value and trigger a callback from it.
Linearly we could do this here I think, how does this not fit in the case of validate?

Codecov Report

Merging #68 into master will decrease coverage by 1.61%.
The diff coverage is 88.37%.

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
- Coverage   98.68%   97.07%   -1.62%
==========================================
  Files          14       14
  Lines         229      239      +10
  Branches       50       51       +1
==========================================
+ Hits          226      232       +6
- Misses          0        4       +4
  Partials        3        3

Impacted Files
Coverage Δ

src/useField.ts
87.5% <80%> (-12.5%)
:arrow_down:

src/Form.tsx
96.1% <90.9%> (-2.47%)
:arrow_down:

Continue to review full report at Codecov.

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

I have the following case. I have an object called 'Dialogstate', which has a name and a very nested config.
So I have a form where the user has to give a name to the Dialogstate, and has to configure the config.
In those configs, there are dropdowns (called nextDialogstate) which can also point to a Dialogstate. But this is a creatable dropdown. So it's not only possible to point to a new dialogstate, but fill in a new name. The back end will create those new dialogstates later.
If I use the code I added in this PR. I can do smth like this:
const [_, {value: currentDialogstateName}] = useField<string>('name')
const [{setError}] = useField<string>('very.very.very.nested.nextDialogstateId')

const onSelectInputChange = ({id, name}: {id?: string, name: string}) => {
        if(!id && name === currentDialogstateName) setError('you can't have the same name as the current diaogstate name)
        //....
}

If I do this with a validate, my code can get nasty because this config can be pretty big

So for your use case useSpy would be perfect. Would you be okay with using useFormConnect for now or? I assume you will need this in 3.2.0 instead of the current target of your PR which is 4.0.0

Indeed, you're right!

If you are okay with using useFormConnect I'll make sure to publish 3.2.0 tonight, if you're not just supply some arguments and I'll consider them when preparing the release.
Big thanks for taking time out of your day to PR already

Any reason why opting for another hook instead of options on the useField hook itself, passing spies? Is it for memoization or?

@Pruxis I think it enforces a paradigm that is very light on bundle size and is semantically a lot easier to read let's compare both examples
MyNameComponent.js

const [_, {value: currentDialogstateName}] = useField<string>('name') // the real field
const [{setError}] = useField<string>('very.very.very.nested.nextDialogstateId') // Irrelevant field

onChange(() => do smth with both the above)

Now with useSpy
MyNameComponent.js

const [_, {value: currentDialogstateName}] = useField<string>('name') // the real field
onChange(() => only changename)

MyNestedField.js
useSpy('name', (newValue, { setError }) => do smth)


Add basic blog functionality

Implements the basic blog functionality with user authentication and CRUD operations for posts and comments.

Models and Migrations: Adds Post, User, and Comment models with corresponding migrations. The Post model includes validations for title and body, and associations to User and Comment. The User model includes validations for email, has_secure_password for authentication, and associations to Post and Comment. The Comment model is associated with both Post and User.
Controllers: Introduces PostsController with CRUD actions, CommentsController with actions to create and destroy comments, UsersController for user registration, and SessionsController for login and logout functionalities.
Views: Adds views for listing, showing, creating, editing, and deleting blog posts; a view for user registration and login; partials for displaying and creating comments.
Routes: Updates config/routes.rb to include resources for posts, comments, users, and sessions, enabling the necessary routes for the blog's functionality.

For more details, open the Copilot Workspace session.

Implement CSRF protection across the application to address Authenticity Token errors.

Global Setting: Enable CSRF protection globally in application_controller.rb by adding protect_from_forgery with: :exception.
Form Helpers: Ensure Rails form helpers are used in users/new.html.erb to automatically include authenticity tokens in forms, enhancing security for user creation.

For more details, open the Copilot Workspace session.

Implements CSRF protection and fixes an authenticity token error in development environment.

Enables CSRF protection globally by adding protect_from_forgery with: :exception in the ApplicationController.
Updates the user sign-up form to use form_with helper, ensuring authenticity tokens are included in forms.
Adjusts default_url_options in the development environment configuration to match the request's base URL, addressing the ActionController::InvalidAuthenticityToken error.

For more details, open the Copilot Workspace session.


Beam Enrich: support the enrichments relying on local files

This is the trickiest enrichment since it relies on local files and so is treated separately.
Beam has a distributed cache similar to hadoop: http://spotify.github.io/scio/examples/DistCacheExample.scala.html

renamed since we now have other enrichments relying on local files, c.f. #3750

We will have a third soon (referer-parser).


Adding my project to the website.

Closes # .

Changes proposed in this pull request:

Hi team,
I tried to run the website locally using hugo. When trying to compile the page using "hugo" as a command, I got the error "render of "page" failed: "/XXX/layouts/page/single.html:53:7". Sounds like there's a specific file missing, but maybe there's also just a different way of rendering the web page.

Looking great! merging...


wayland-pipewire-idle-inhibit: 0.4.5 -> 0.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/wayland-pipewire-idle-inhibit/versions.
meta.description for wayland-pipewire-idle-inhibit is: Suspends automatic idling of Wayland compositors when media is being played through Pipewire.
meta.homepage for wayland-pipewire-idle-inhibit is: https://github.com/rafaelrc7/wayland-pipewire-idle-inhibit/
Updates performed

Rust version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.0 with grep in /nix/store/v14njfi0cpl5j2ljyb44pk3hs0v5ll4w-wayland-pipewire-idle-inhibit-0.5.0
found 0.5.0 in filename of file in /nix/store/v14njfi0cpl5j2ljyb44pk3hs0v5ll4w-wayland-pipewire-idle-inhibit-0.5.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wayland-pipewire-idle-inhibit

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v14njfi0cpl5j2ljyb44pk3hs0v5ll4w-wayland-pipewire-idle-inhibit-0.5.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A wayland-pipewire-idle-inhibit https://github.com/r-ryantm/nixpkgs/archive/9de9c90422910825aa3482e1b7e3d22730e6efd7.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9de9c90422910825aa3482e1b7e3d22730e6efd7#wayland-pipewire-idle-inhibit

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v14njfi0cpl5j2ljyb44pk3hs0v5ll4w-wayland-pipewire-idle-inhibit-0.5.0
ls -la /nix/store/v14njfi0cpl5j2ljyb44pk3hs0v5ll4w-wayland-pipewire-idle-inhibit-0.5.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

wayland-pipewire-idle-inhibit

Maintainer pings
cc @rafameou for testing.

Result of nixpkgs-review pr 298629 run on x86_64-linux 1

1 package built:

wayland-pipewire-idle-inhibit


Event sizes don't reflect "endsAt" property

Hi,  Love the widget! performance is much much much better than full calendar - no jQuery!
Issue is that the end time isn't used to size events in the week view. Would also be nice to have the
day details (click on day) have an agenda view that shows event times overlap, etc..

Sorry I don't quite get what you mean, would you mind explaining a bit more please? Thanks! :smile:

Sure, I was wondering whether it would be possible to render the events on the monthly and weekly calendars as an "agenda" - where they overlap and are ordered top-to-bottom based on their start and end dates. Also, would be nice to use an agenda view in the details section when you expand the day instead of just a list of event details.
Is this possible?

Sorry this is way beyond the scope of this plugin and not something that I would ever have the time to implement (and it's also really difficult to implement well)


Tensorflow

When I try using keras, says that I dont have tensorflow 2.2.0 installed, but i do. Can someone help me? I already uninstalled tensorflow and the same thing happens. Also I reinstalled Anaconda and nothing.
from keras.models import Sequential
Traceback (most recent call last):
File "C:\Users\luiz_\anaconda3\lib\site-packages\keras_init_.py", line 3, in
from tensorflow.keras.layers.experimental.preprocessing import RandomRotation
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow_init_.py", line 41, in
from tensorflow.python.tools import module_util as _module_util
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python_init_.py", line 40, in
from tensorflow.python.eager import context
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python\eager\context.py", line 35, in
from tensorflow.python import pywrap_tfe
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python\pywrap_tfe.py", line 28, in
from tensorflow.python import pywrap_tensorflow
ImportError: cannot import name 'pywrap_tensorflow' from 'tensorflow.python' (C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python_init_.py)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
from keras.models import Sequential
File "C:\Users\luiz_\anaconda3\lib\site-packages\keras_init_.py", line 6, in
'Keras requires TensorFlow 2.2 or higher. '
ImportError: Keras requires TensorFlow 2.2 or higher. Install TensorFlow via pip install tensorflow

Try using Keras from inside TF if you have TF installed.
from tensorflow.keras.models import Sequential

I tried and this happened:
from tensorflow.keras.models import Sequential
Traceback (most recent call last):
File "", line 1, in
from tensorflow.keras.models import Sequential
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow_init_.py", line 41, in
from tensorflow.python.tools import module_util as _module_util
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python_init_.py", line 40, in
from tensorflow.python.eager import context
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python\eager\context.py", line 35, in
from tensorflow.python import pywrap_tfe
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python\pywrap_tfe.py", line 28, in
from tensorflow.python import pywrap_tensorflow
ImportError: cannot import name 'pywrap_tensorflow' from 'tensorflow.python' (C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python_init_.py)

Try one thing, remove all your python installations and CUDA installation and download using Anaconda and you can install TF2 with CUDA bindings using:
conda install -c anaconda tensorflow-gpu

I tried, didnt work...
from tensorflow.keras.models import Sequential
Traceback (most recent call last):
File "", line 1, in
from tensorflow.keras.models import Sequential
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow_init_.py", line 41, in
from tensorflow.python.tools import module_util as _module_util
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python_init_.py", line 40, in
from tensorflow.python.eager import context
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python\eager\context.py", line 35, in
from tensorflow.python import pywrap_tfe
File "C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python\pywrap_tfe.py", line 28, in
from tensorflow.python import pywrap_tensorflow
ImportError: cannot import name 'pywrap_tensorflow' from 'tensorflow.python' (C:\Users\luiz_\AppData\Roaming\Python\Python37\site-packages\tensorflow\python_init_.py)

Can you describe your system details:
System information:

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on a mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Systeminformation:
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Microsoft Windows 10
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): TensorFlow 2.2.0
Python version: 3.7.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.0.130
**GPU model and memory:**GeForce MX130; 8Gb RAM

So I don't think GeForce MX130 has CUDA support, (check here)
PyWrap Error resolved here

Please always fill in issue template.
Please use ``` before and after code and error blocks to make them readable.
Please post output of pip list.


399 services gateway sidebar

This PR implements sidebar view displaying all services in fabric.
Resolves #399

Added documentation, renamed styled components, cleaned getColor function


Fix overflow in duration

When trying to wait for Duration::MAX, mio will panic with an overflow. This is a work-around for that behavior.

Could you also fix it for Windows:
https://github.com/tokio-rs/mio/blob/7ed74bf478230a0cfa7543901f6be6df8bb3602e/src/sys/windows/iocp.rs#L231
?

Done

@Thomasdezeeuw I've updated the PR to fix some issues encountered by the CI.

Thanks @TethysSvensson


Txt file not changing baud rate for Pixfalcon

Hi all I know this related question have been posted in previous threads but I did not manage to find any solutions for it.
I am trying to change the baud rate for Pixfalcon from 57600 to 921600 since there are some delay when i am trying to send messages to it through multiple nodes at the same time. I tried adding a extra.txt file in the etc folder with various commands in it. (no telem2 in Pixfalcon to use Qgroundcontrol to change SYS COM)
Version 1:
mavlink stop-all
set MAVLINK_F "-r 1200 -d /dev/ttyS0"
mavlink start -r 20000 -m config -b 921600 -d /dev/ttyS1
Version 2:
mavlink stop-all
mavlink start -d /dev/ttyS1 -b 921600 -r 3000
mavlink start -d /dev/ttyACM0
I then changed the baud rate through the browser with the link
http://<IP_ADDRESS>/setparameters?baud=921600.
However doing so gave me the error
[ INFO] [1490094284.176297811]: CON: Got HEARTBEAT, connected. FCU: INVALID
[ WARN] [1490094286.178662038]: VER: broadcast request timeout, retries left 4
[ WARN] [1490094287.180176029]: VER: broadcast request timeout, retries left 3
[ WARN] [1490094288.178617605]: VER: unicast request timeout, retries left 2
[ WARN] [1490094289.177899120]: VER: unicast request timeout, retries left 1
[ WARN] [1490094290.179989010]: VER: unicast request timeout, retries left 0
[ WARN] [1490094291.178454197]: VER: your FCU don't support AUTOPILOT_VERSION, switched to default capabilities
when I tried connecting to Pixfalcon.
I think that the txt file is not executing the commands in it properly, do anyone have any idea on how to solve this problem? Thanks for the help in advance!

It should be on your microsd, under /etc/extras.txt, not "extra.txt".


Pulling objects pushes you back to them

240ping, my character gets pushed backwards towards the item I'm pulling

@metalgearsloth 99% sure this was still joint-less pulling. is this still an issue?

Predicted pulling fixed it which was done like last week.


feat: rename meter-thickness-medium; closes DNA-1224

Description
Renames meter-thickness-medium to meter-thickness large

Related Issue
DNA-1224
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@GarthDB also please fix my commit message typo when you merge!

:tada: This PR is included in version 12.0.0-beta.57 :tada:
The release is available on:

GitHub release
npm package (@beta dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

GitHub release
npm package (@next-major dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 12.0.0 :tada:
The release is available on:

GitHub release
npm package (@next-major dist-tag)

Your semantic-release bot :package::rocket:


Fix syntax for workflow update testing

Ref: https://github.com/mindersec/minder-rules-and-profiles/actions/runs/11562883909
Invalid workflow file: .github/workflows/update.yaml#L7
You have an error in your yaml syntax on line 7

I screwed this up and pushed to the repo, not to my fork.  Closing this in favor of a new PR not on the main repo.


Ruby implementation of generateBitlyUrl

Implement cloud function generateBitlyUrl() with Ruby.
Related issue: https://github.com/appwrite/demos-for-functions/issues/82

@lohanidamodar Please review the code.

@foxy-eyed apart from the requests made by @Meldiron everything looks good. Great job 🚀

@Meldiron @lohanidamodar thanks for review! All done, please check out 🙌

Hi, @Meldiron! Do you have any other suggestions or is it done?)
I fixed all previous and have approve from @lohanidamodar, but now PR is stuck because of your review status.
Could you please check again? Sorry for bothering)

THANK YOU! All changes merged 🥳
Please reach out to eldadfux on our Discord server if you would like to claim your Appwrite swags! As a way of saying thank you, we would also love to invite you to join the Appwrite organization on GitHub. Please share your GitHub username with us on Discord.
You can accept the invite by visiting https://github.com/orgs/appwrite/invitation. By joining our team, you will officially be an Appwrite maintainer on GitHub.
You can change your membership visibility settings, so your new Appwrite team membership badge will show up on your personal GitHub profile.
Please feel free to look for more PRs you might be interested in helping with on our long list of Hacktoberfest friendly issues and help make Appwrite better :)


Index Name- auto populate

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

BCG auto populates if blank.  (MN_BCG).
Need to get bug/fish to do IBI.


pgwire: correctly handle GSSAPI-encrypted connection

Found by @stdrc:
Some users find the error when connecting with psql -h localhost -p 4566 -d dev -U root, if they have configured something like kerberos in the system.
psql: error: connection to server at "localhost" (<IP_ADDRESS>), port 4566 failed: server closed the connection unexpectedly
        This probably means the server terminated abnormally
        before or while processing the request.

The problem appears to be that psql tries to establish a GSSAPI-encrypted connection once it find GSSAPI credentials, according to the default parameter gssencmode=prefer.

https://www.postgresql.org/docs/current/libpq-connect.html#LIBPQ-CONNECT-GSSENCMODE
if there are GSSAPI credentials present (i.e., in a credentials cache), first try a GSSAPI-encrypted connection; if that fails or there are no credentials, try a non-GSSAPI-encrypted connection. This is the default when PostgreSQL has been compiled with GSSAPI support.

Users can use psql -h <IP_ADDRESS> -p 4566 -d "dbname=dev gssencmode=disable" -U root as a workaround. However, I believe we should correctly handle GSSAPI-encrypted connection to avoid this confusion. At least, we should gracefully fallback to a non-GSSAPI-encrypted connection.

This is probably a first step when users try RisingWave. Hope to be fixed asap

Do we really need this? GSS-API is not easy to use.

@TennyZhuang Any updates? Today a user encountered this issue.

I will take a look


record change about advanced auditing feature gate

This record:
https://github.com/kubernetes/kubernetes/pull/51943

/assign radhikapc


nro controller: use predicates for all resources

Our apply logic compares the whole object and also update the whole object
including metadata and status that can be a reason for additional reconciliation
loops. Instead of improving the comparison and updating logic that will make
the code more complex, we can just filter events for objects that did not
have any change under spec or labels.
Signed-off-by: Artyom Lukianov<EMAIL_ADDRESS>

@Tal-or @fromanirh Small improvement.

thanks @cynepco3hahue !

once we have a better test coverage and we are feature complete, I'm ok with ditching/simplifying our in-house logic and replace with controllerutil facilities (e.g. https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/controller/controllerutil#CreateOrUpdate). But that's stuff for the future.


HDFS Create External Data: non-CSV files supported?

@ronychatterjee please assess this from a PM persepective.
All files and folders in HDFS now have the Create External Table from CSV files available. Is this expected? I understand why this might be a decent idea though I think we should handle or early-block some known file types like parquet which are definitively not CSV format.

@corivera this is a good candidate for June.


BCHP Not Working on Legacy

There is an issue with spatial import of CSV that is causing the point coordinates to be transposed so all spatial data fails to import. Worked around it by loading data in DEV instance and uploading using the arches export/import business data function.
Have sent a request to the community to see if someone else has seen this transposition issue. Will address root cause in a separate ticket.
Also fixed issues caused by arches core updates causing map popup functionality to fail.
BCRHP on Legacy now functioning.
FYI @IJGoldie


Add S.reduce function

Those of you that know your Ramda history should appreciate the irony of this one...

Since we define Maybe#reduce (in accordance with the requirements of Foldable), this function should dispatch to a reduce method if present. It'd make sense to update the function's name to match.
We could define Foldable :: TypeClass to constrain the relevant argument.

Dammit, foldl is so much cooler as a name. ;) Not sure what to put for the documentation now that the type takes the more genral type Foldable rather than Array. Any suggestions?

This function seems more at home in the list section than in the function section. Do you agree?
 ;)

Yeah, I wasn't sure where to put it, List seems more appropriate given the standard way the function willl be used.

Let's insert S.reduce immediately before S.unfoldr. These two functions are related, and beginning the list section with the various slicing functions seems appropriate.

:zap: x 2

:zap: x 2

:zap: x 3

:deciduous_tree: Feel free to merge this once the tests pass. :)

:tada:


logged in user can't edit his expense

User is logged in (on mobile) with the email he used and can't see the EDIT button in the expense:

Thank you for taking the time to report an issue 🙏
The easier it is for us to reproduce it, the faster we can solve it.
So please try to be as complete as possible when filing your issue.

URL:
Logged in as: (just provide your username or the url of your profile)
Error message if any:
Expected result:
Browser:
Bonus point if you can add a screenshot :-)
Thank you and have a wonderful day/evening!
(and sorry for inconvenience! We'll do our best to solve this ASAP)

keeping this opened since now this #713

Fixed

(need to go to /expenses - can't edit from homepage, there is not enough room to show the edit form)

@xdamman can we add a link to 'edit expenses' somewhere? how will users know where to go?


Multiple domains

Is it possible to set this up to work with and authenticate with multiple domains?  I am a super admin in two different domains, and would love to be able to use this in both domain systems without having to switch users on my computer.

I have a fairly friendly system on my Mac, where I have three aliases in
.bashrc as follows:
alias gamd1='python ~/sysadmin/googlemail/gamd1/gam.py'
alias gamd2='python ~/sysadmin/googlemail/gamd2/gam.py'
alias gamd3='python ~/sysadmin/googlemail/gamd3/gam.py'
where the folders are each tailored copies of the GAM distribution.
That is to say I expand the source distribution into each of the
tailored folders and in each one I set up the apporpriate .json files
for OAUTH and so on for the relevant domain.
It works well.  One of the really neat things about GAM is that each
'installation' is independent of each other installation.
On 12/06/2015 16:21, Robby Barnes wrote:

Is it possible to set this up to work with and authenticate with
multiple domains? I am a super admin in two different domains, and
would love to be able to use this in both domain systems without
having to switch users on my computer.
—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/69.

Perfect.  I will give something like this a try.  Thank you.
robby barnes |<EMAIL_ADDRESS><EMAIL_ADDRESS>|<PHONE_NUMBER>

On Jun 12, 2015, at 12:33 AM, Andrew Birch<EMAIL_ADDRESS>wrote:
I have a fairly friendly system on my Mac, where I have three aliases in
.bashrc as follows:
alias gamd1='python ~/sysadmin/googlemail/gamd1/gam.py'
alias gamd2='python ~/sysadmin/googlemail/gamd2/gam.py'
alias gamd3='python ~/sysadmin/googlemail/gamd3/gam.py'
where the folders are each tailored copies of the GAM distribution.
That is to say I expand the source distribution into each of the
tailored folders and in each one I set up the apporpriate .json files
for OAUTH and so on for the relevant domain.
It works well. One of the really neat things about GAM is that each
'installation' is independent of each other installation.
On 12/06/2015 16:21, Robby Barnes wrote:

Is it possible to set this up to work with and authenticate with
multiple domains? I am a super admin in two different domains, and
would love to be able to use this in both domain systems without
having to switch users on my computer.
—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/69.

—
Reply to this email directly or view it on GitHub https://github.com/jay0lee/GAM/issues/69#issuecomment-111378594.

You could just use different OAUTH files, whole lot easier than multiple
installs - I manage several Google Apps instance from the single GAM
installation
https://github.com/jay0lee/GAM/wiki/OAuthKeyManagement#selecting-which-oauth-key-file-to-use
Graham Ingleby | Cloud Computing Consultant | 01344 203395 |
<EMAIL_ADDRESS>[image: Google+] http://google.com/+Ancoris
http://www.linkedin.com/company/98835 http://www.ancoris.com
On 12 June 2015 at 07:35, Robby Barnes<EMAIL_ADDRESS>wrote:

Perfect. I will give something like this a try. Thank you.
robby barnes |<EMAIL_ADDRESS><EMAIL_ADDRESS>|
801.419.6332

On Jun 12, 2015, at 12:33 AM, Andrew Birch<EMAIL_ADDRESS>wrote:
I have a fairly friendly system on my Mac, where I have three aliases in
.bashrc as follows:
alias gamd1='python ~/sysadmin/googlemail/gamd1/gam.py'
alias gamd2='python ~/sysadmin/googlemail/gamd2/gam.py'
alias gamd3='python ~/sysadmin/googlemail/gamd3/gam.py'
where the folders are each tailored copies of the GAM distribution.
That is to say I expand the source distribution into each of the
tailored folders and in each one I set up the apporpriate .json files
for OAUTH and so on for the relevant domain.
It works well. One of the really neat things about GAM is that each
'installation' is independent of each other installation.
On 12/06/2015 16:21, Robby Barnes wrote:

Is it possible to set this up to work with and authenticate with
multiple domains? I am a super admin in two different domains, and
would love to be able to use this in both domain systems without
having to switch users on my computer.
—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/69.

—
Reply to this email directly or view it on GitHub <
https://github.com/jay0lee/GAM/issues/69#issuecomment-111378594>.

—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/69#issuecomment-111378907.

--

This message is for the named person's use only.  If you receive this message in error, please delete it and notify the sender.  Ancoris reserves the right to monitor all e-mail communications through its networks. Ancoris Limited, Registered in England Number: 04830784, Registered address: 5a Frascati Way, Maidenhead, Berkshire SL6 4UY. Trading Address: Lily Hill House, Lily Hill Road, Bracknell, Berkshire RG12 2SJ

I create a bash script to do this:
#!/bin/bash
#GAM change Domain Script
echo "Please inform yor domain domain.com"
read filename
if [ ! -f oauth2.txt ]; then
mv oauth2.txt oauth2.txt.old
cp $filename.txt oauth2.txt
else
cp $filename.txt oauth2.txt
python gam.py info domain
fi
Philippe Carvalho
Contato: bakudani<EMAIL_ADDRESS>| *Twitter: *
philippescar http://twitter.com/philippescar
*Linkedin: *http://br.linkedin.com/in/philippescar | *Google+: *http://
http://goog_2091940453google.com/+PhilippeCarvalhoScar
*Blog: *www.casadeinverno.wordpress.com
"Where there is a shell, there is a way"
On Tue, Jun 16, 2015 at 11:34 AM, Graham<EMAIL_ADDRESS>wrote:

You could just use different OAUTH files, whole lot easier than multiple
installs - I manage several Google Apps instance from the single GAM
installation
https://github.com/jay0lee/GAM/wiki/OAuthKeyManagement#selecting-which-oauth-key-file-to-use
Graham Ingleby | Cloud Computing Consultant | 01344 203395 |
<EMAIL_ADDRESS>[image: Google+] http://google.com/+Ancoris
http://www.linkedin.com/company/98835 http://www.ancoris.com
On 12 June 2015 at 07:35, Robby Barnes<EMAIL_ADDRESS>wrote:

Perfect. I will give something like this a try. Thank you.
robby barnes |<EMAIL_ADDRESS><EMAIL_ADDRESS>|
801.419.6332

On Jun 12, 2015, at 12:33 AM, Andrew Birch<EMAIL_ADDRESS>wrote:
I have a fairly friendly system on my Mac, where I have three aliases
in
.bashrc as follows:
alias gamd1='python ~/sysadmin/googlemail/gamd1/gam.py'
alias gamd2='python ~/sysadmin/googlemail/gamd2/gam.py'
alias gamd3='python ~/sysadmin/googlemail/gamd3/gam.py'
where the folders are each tailored copies of the GAM distribution.
That is to say I expand the source distribution into each of the
tailored folders and in each one I set up the apporpriate .json files
for OAUTH and so on for the relevant domain.
It works well. One of the really neat things about GAM is that each
'installation' is independent of each other installation.
On 12/06/2015 16:21, Robby Barnes wrote:

Is it possible to set this up to work with and authenticate with
multiple domains? I am a super admin in two different domains, and
would love to be able to use this in both domain systems without
having to switch users on my computer.
—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/69.

—
Reply to this email directly or view it on GitHub <
https://github.com/jay0lee/GAM/issues/69#issuecomment-111378594>.

—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/69#issuecomment-111378907.

--

This message is for the named person's use only. If you receive this
message in error, please delete it and notify the sender. Ancoris reserves
the right to monitor all e-mail communications through its networks.
Ancoris Limited, Registered in England Number: 04830784, Registered
address: 5a Frascati Way, Maidenhead, Berkshire SL6 4UY. Trading Address:
Lily Hill House, Lily Hill Road, Bracknell, Berkshire RG12 2SJ

—
Reply to this email directly or view it on GitHub
https://github.com/jay0lee/GAM/issues/69#issuecomment-112451409.

Should not be necessary to have multiple installs of GAM. Use the OAUTHFILE environment variable to switch between accounts. Closing this out now.


最新版本报错

Failed to compile with 1 error                                                 下午8:15:12
error  in ./src/styles/element-variables.scss
Syntax Error: Error: Missing binding /Users/jonny/go/src/go-admin-ui/node_modules/node-sass/vendor/darwin-x64-83/binding.node
Node Sass could not find a binding for your current environment: OS X 64-bit with Node.js 14.x
Found bindings for the following environments:

OS X 64-bit with Node.js 13.x

This usually happens because your environment has changed since running npm install.
Run npm rebuild node-sass to download the binding for your current environment.
@ ./src/styles/element-variables.scss 4:14-237 14:3-18:5 15:22-245
@ ./src/main.js
@ multi (webpack)-dev-server/client?http://localhost:9527&sockPath=/sockjs-node (webpack)/hot/dev-server.js ./src/main.js

lijiayi@lijiayideMacBook-Pro go-admin-ui % npm -v
6.14.10
lijiayi@lijiayideMacBook-Pro go-admin-ui % node -v
v14.15.3

npm 安装的包删掉重新安装

npm rebuild node-sass  我执行一下这个就好了~

若不是来翻翻 issues 还真不知道怎么解决，现在 OSX 上 node.js14.x 没有 Run npm rebuild node-sass 这个提示了。
不过，我是用 npm rebuild node-sass 一样解决了这个问题。


Gear colours are reset on resume

When an assembly of gears is made, then the activity quit, and resumed, the colours of the gears have changed.
Save the colors with the journal entry and restore them on resume.
Duplicate from https://github.com/sugarlabs/gears-activity/issues/2

Working on it

Fixed in https://github.com/llaske/sugarizer/pull/340

Closed in https://github.com/llaske/sugarizer/commit/dd3c58d56981e4dfa6d94601b95d7117a3bc9241


[SPARK-20102] Fix nightly snapshot and RC packaging scripts w/ two minor build fixes

What changes were proposed in this pull request?
The master snapshot publisher builds are currently broken due to two minor build issues:

For unknown reasons, the LFTP mkdir -p command began throwing errors when the remote FTP directory already exists. This change of behavior might have been caused by configuration changes in the ASF's SFTP server, but I'm not entirely sure of that. To work around this problem, this patch updates the script to ignore errors from the lftp mkdir -p commands.
The PySpark setup.py file references a non-existent pyspark.ml.stat module, causing Python packaging to fail by complaining about a missing directory. The fix is to simply drop that line from the setup script.

How was this patch tested?
The LFTP fix was tested by manually running the failing commands on AMPLab Jenkins against the ASF SFTP server. The PySpark fix was tested locally.

/cc @marmbrus @holdenk, this patch should fix the build issues blocking Spark 2.1.1 RC1 builds.

Test build #75246 has started for PR 17437 at commit 867ca2c.

Test build #75246 has finished for PR 17437 at commit 867ca2c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75246/
Test PASSed.

I'm going to merge this into master and branch-2.1 then will manually kick off the packaging jobs so that we can get back to publishing nightly package snapshots.


[Deema Diab- MS assignment] Update concepts-migration-planning.md

Fixed table format.
Fixed image syntax to conform with preferred syntax and also added lightboxes and alt text.
Edit non-user-friendly language like "iron out any flaws" and "low hanging fruit". Figurative language also poses a problem for auto-translation.
Edited structure + reorganized paragraphs.
Replaced Next steps with Related content to comply with new Learn content guidelines.

@diab98,
Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request.
Thanks!
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I'm closing this PR because the CLA has not been signed.


Bot will not pause on second server.

I summon the bot like normal in the first server. Leave the first server (bot pauses) and enter the voice channel for a second server. Summon the bot there and then leave. The bot will not pause in the second server.
http://goo.gl/LcISNi
Gif of the action.

The autopause code was never built for multiserver, so it won't work properly with that.  I'll fix that soon and add an option for it.

Good to go. And thank you for your hard work.

This has been fixed (for a while).


SMS Flow for positive tests

[x] None of the other issue templates apply to this issue.
Hi there! I've complained about this on the interwebs, and searched the https://coronalert.be/nl/ for a contact page, but couldn't find one, so as a last resort, I'm bothering you, sorry. Can someone update the flow you need to go through when you're going to get tested?
I think it's explained here: https://devside.atlassian.net/wiki/spaces/CD/pages/11469091/iOS, and I can see in the iOS app that you're taking over the URL, and using javascript to prefill that form. (I would try and make the webpage better instead of making the app do this, but that's another discussion)
But apart from a small note in the release notes of the app, I haven't see any mention of it. If the doctors don't need to get the 17 digit code, this would save quite some time. If people can retroactively link test and app, this would probably help in getting more positive tests into the app.
Twitter:
Complaining about the lack of instructions
Where I dug into the SMS

This is still not rolled out. The apps have been prepared for this, but we need a lot of users to be updated to the latest version or otherwise lots of people could receive an SMS without the app being capable of opening it (since they are still running an old version of Coronalert).
But the basics stay the same. The person registering the test (doctor or other) will need to enter the code generated by your app. The SMS scenario is only supposed to be used for those cases where it didn't happen for some reason. If test requests are received without an app code, those numbers will be notified through SMS to link their app. This is not related to a test result FYI. When the SMS will be sent out the test result is still unknown.

This is still not rolled out. The apps have been prepared for this, but we need a lot of users to be updated to the latest version or otherwise lots of people could receive an SMS without the app being capable of opening it (since they are still running an old version of Coronalert).

They would visit the webform, no? So worst case scenario, that page needs to be update to include that you need to copy over your test code to that form.

The SMS scenario is only supposed to be used for those cases where it didn't happen for some reason.

Since it's copying over a 17-digit number, either over telephone or visually, there are a lot of reasons this fails to happen.
E.g.

"We just don't have the time, to explain it to people and wait for the code, some people can't use their phones"
"I added it but the forms were not supporting it"
"Sometimes it fails, because the submission date is different"

I think, not having to do the code up front could help a lot with app usage, and it seems like the SMS flow would do just that.

This is a discussion you should have with Sciensano / the government I think. It's a bit out of scope (and out of our control) for the iOS Github repository ;)

👍 Agreed! You were just the only contact point I could find :)


Resolver should just take a query, we should internalize anything related to graphiql

Did this in: https://github.com/mwildehahn/hack-graphql/pull/71


Fix vae conversion

Unsure at which moment it broke, but now I can't convert vae(and model as vae it's part) without this fix.
Need further research - maybe it's breaking change in transformers?

Have you tracked down the problem? Is this the correct fix?

Checked - it's internal change from 0.16.1 to 0.17.0 diffusers version:
https://github.com/huggingface/diffusers/blob/v0.16.1/src/diffusers/models/unet_2d_blocks.py#L430C25-L430C25
https://github.com/huggingface/diffusers/blob/v0.17.0/src/diffusers/models/unet_2d_blocks.py#L433
And as we updated diffusers version this fix should be applied.


Add hideOnMouseDown option

This enables hiding the dropdown on mouse down instead of on click (mousedown + mouseup). The use case I am using this great library for prefers this, more responsive, configuration. I'm sure others would benefit from this option as well!

Closing in favour of #50 because this has a bug and the branch name isn't meaningful.


fix(resets.js): use semantic placeholder color token instead of hex

We now have a semantic placeholder text color token, so it might be the time to replace any hex values with respective CSS vars in the resets.js.

We also have a hex value set on a border-color for body selector but I wasn't sure what to replace it with. Any ideas?
*,
::before,
::after {
  box-sizing: border-box; /* 1 */
  border-width: 0; /* 2 */
  border-style: solid; /* 2 */
  border-color: #e5e7eb; /* 2 */
}

Source: https://github.com/warp-ds/css/blob/alpha/resets/resets.js#L13

We also have a hex value set on a border-color for ::before, ::after but I wasn't sure what to replace it with. Any ideas?
*,
::before,
::after {
  box-sizing: border-box; /* 1 */
  border-width: 0; /* 2 */
  border-style: solid; /* 2 */
  border-color: #e5e7eb; /* 2 */
}

Source: https://github.com/warp-ds/css/blob/alpha/resets/resets.js#L13

I guess var(--w-s-color-border) is not the correct color? 🤔 But I would say that it should be semantically correct to use that one. 😄   @adidick ?

I guess var(--w-s-color-border) is not the correct color? 🤔 But I would say that it should be semantically correct to use that one, as that is our default border color. 😄 @adidick ?

Here's a comparison of the --w-s-color-border token value (on the left) with the hex value from resets.js:
Finn's bluegray-300:

Tori's gray-300:


Prompt to install pytest when configured but missing from env

Show an info bar if the project/workspace is configured to use pytest, but pytest is not installed in the current environment.

Implemented: https://github.com/microsoft/PTVS/pull/5456


Barebone HTML roster page in need of styling

@lliu78 the html that needs styling is at master/app/style.html each row of the second table has its own dropdown menu, an example check out lines 110-113, it might be helpful to reference W3School when making the dropdown menus.
There are a lot of rows in the second table, but you don't have to apply styling to all of the rows, first few rows would suffice, or you can delete some of the rows in the second table.

Ignore the part about drop down menus, it will be replaced with buttons instead, the buttons seems to give a better UX than the drop downs.


Update link to etcd faq to etcd 3.5 on setting up prod env page

This PR updates the link to the etcd faq from 3.4 to 3.5 on the Production Environment page
Kubernetes 1.25 uses etcd 3.5.4
From https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.25.md

Etcd: Update to v3.5.4 (https://github.com/kubernetes/kubernetes/pull/110033, @mk46) [SIG API Machinery, Cloud Provider, Cluster Lifecycle and Testing]

/lgtm
Thanks @reylejano

/approve


[Bug]:  cloudflare pages adapter does not support livereload?

Which Remix packages are impacted?

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

What version of Remix are you using?
@remix-run/cloudflare-pages 1.1.1
What version of Node are you using? Minimum supported version is 14.
17.2
Steps to Reproduce
minial project: https://github.com/w2fs/remix-blog-demo
"scripts": {
    "build": "cross-env NODE_ENV=production remix build",
    "build:css": "postcss styles/**/*.css --base styles --dir app/styles",
    "build:css:prod": "npm run build:css -- --env production",
    "dev": "cross-env NODE_ENV=development run-p dev:*",
    "postinstall": "remix setup cloudflare-pages",
    "dev:css": "npm run build:css -- --w",
    "dev:remix": "remix watch",
    "dev:wrangler": "wrangler pages dev ./public --watch ./build ./styles",
    "start": "npm run dev:wrangler"
  }

use  yarn dev
Expected Behavior
when styles or components changed, page should livereload.
Actual Behavior
even fresh manually cannot take effect.
only ⌃ + C and restart works

I am also experiencing this issue

You can temporarily fix this by installing wrangler@pages
https://github.com/cloudflare/wrangler2/pull/71#issuecomment-1002921960

Thanks I'll test this in a few minutes

@lawrencecchen ok I can confirm, that worked

This worked for me also


Any plans for supporting IoT Hub connection strings?

Are you planning to add support for IoT Hub connection strings in the way that is supported in Microsft.ServiceBus.Messaging library?
It's much more convenient to use the IoT Hub connection string directly rather than assembling a Event Hub connection string from IoT Hub's Event Hub-compatible settings.

We are moving ITOHub support to Track 2 SDK therefore closing the feature request here.

Hi @serkantkaraca where did the issue was moved for? I couldn't find it.
Thanks

Azure.Messaging.EventHubs addressed the ask with a sample. Please open new issues if you have any issues or questions regarding the sample. You can find sample here - https://github.com/Azure/azure-sdk-for-net/tree/master/samples/iothub-connect-to-eventhubs


User role doesn't have Graph Write access so can't upload files

Description:
The User role appears to not have sufficient permission to upload files and write to the graph db. The Upload-only role has the GraphDBWrite permission but the User role only has the GraphDBRead permission. This is also contrary to the documentation under User Role Definitions.
Component(s) Affected:

[X] UI
[ ] API
[X] Neo4j
[ ] PostgreSQL
[ ] Data Collector (SharpHound, AzureHound)
[ ] Other (tooling, documentation, etc.)

Steps to Reproduce:

Go to Manual File Ingest as a user with the User role
Click on Upload Files
Add files to upload and select Upload
See error at bottom left: "Permission Denied!"

Expected Behavior:
The User role should be able to successfully upload files manually, similar to the Upload-only role.
Actual Behavior:
The User role is unable to upload files manually.
Screenshots/Code Snippets/Sample Files:
Code: https://github.com/SpecterOps/BloodHound/blob/main/cmd/api/src/auth/role.go
Environment Information:
BloodHound: BloodHound CE latest
Collector: Sharphound 2.0
OS: MacOS Ventura 13.5.1
Docker: 24.0.5
Potential Solution:
For the User role, change the permissions.GraphDBRead to permissions.GraphDBWrite

Hi there - this is by design so that users cannot modify the graph, only Administrators.


I get unkonw 'voice-button' is not a known element while using it with angular

I want to use it with ionic/angular but I get 'voice-button' is not a known element:

If 'voice-button' is an Angular component, then verify that it is part of this module.
If 'voice-button' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

as it says add CUSTOM_ELEMENTS_SCHEMA in your NgModule
follow this link: https://vaadin.com/learn/tutorials/using-web-components-in-angular#:~:text=By default%2C Angular assumes that,use for creating the form.

@Abdulrahman-Khalid could you implement this in Angular. I tried but i get 404 for voicebutton.js and webcomponent-bundle.js. It would be great help in letting me know how could you implement in angular. Thank you.


Add test node package

After #71 is merged, this needs a rebase.

Codecov Report

Merging #72 (4a0d3fa) into master (620b703) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master     #72     +/-   ##
========================================
  Coverage    0.00%   0.00%
========================================
  Files          29       1     -28
  Lines        2059      72   -1987
========================================
+ Misses       2059      72   -1987

Flag
Coverage Δ

unittests
0.00% <ø> (ø)

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

Impacted Files
Coverage Δ

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

... and 20 more

Continue to review full report at Codecov.

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

@destogl ready for rebase, once done, feel free to drop me a note to review this

@bmagyar Before merge please test it with the manual.

@bmagyar just updated... Please one more check.

Codecov Report

Merging #72 (8f1af37) into master (046ce0c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master     #72     +/-   ##
========================================
  Coverage    0.00%   0.00%
========================================
  Files          37       1     -36
  Lines        2664      72   -2592
========================================
+ Misses       2664      72   -2592

Flag
Coverage Δ

unittests
0.00% <ø> (ø)

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

Impacted Files
Coverage Δ

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

...l_demo_hardware/src/rrbot_system_position_only.cpp

... and 28 more

Continue to review full report at Codecov.

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


ja: Make docs/tutorials/services/source-ip.md follow v1.18 of the original text

This is a Feature Request

What would you like to be added
Update docs/tutorials/services/source-ip.md to follow v1.18 of the original (English) text.
Why is this needed
content/ja/docs/tutorials/services/source-ip.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/blob/dev-1.18-ja.2/content/ja/docs/tutorials/services/source-ip.md
Original
https://github.com/kubernetes/website/blob/fb6364d/content/en/docs/tutorials/services/source-ip.md
diff between translated and v1.18
https://gist.github.com/040fe854f6031e225cb849c772777968
Note
Currently, we use dev-1.18-ja.2 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel.

/triage accepted

/assign

@inductor こちらdev-1.18-ja.2では差分が反映されて対処済みでした。問題なければクローズお願いします

/close


sc.pl.embedding with basis "spatial" has side-effects and is missing one row of points

[x] I have checked that this issue has not already been reported.
[ ] I have confirmed this bug exists on the latest version of scanpy.
[x] (optional) I have confirmed this bug exists on the master branch of scanpy.

sc.pl.embedding with basis "spatial" behaves strangely. See code example below.

when "spatial" is an unit array, the resulting plot is missing the top row of points (I think there might be an overflow happening)
when plotting edges and using basis "spatial", adata.obsm["spatial"] is modified after calling sc.pl.embedding`
@giovp

Minimal code sample (that we can copy&paste without having any data)
import numpy as np
import anndata as ad
import scanpy as sc
import matplotlib.pyplot as plt
print(sc.__version__)

arr = np.array([[1888,   55],
       [1887,   54],
       [1888,   54],
       [1889,   54],
       [1887,   55],
       [1889,   55],
       [1887,   56],
       [1888,   56],
       [1889,   56]], dtype=np.uint8)
conn = np.array([[0., 1., 1., 1., 1., 1., 1., 1., 1.],
       [1., 0., 1., 0., 1., 0., 0., 0., 0.],
       [1., 1., 0., 1., 1., 1., 0., 0., 0.],
       [1., 0., 1., 0., 0., 1., 0., 0., 0.],
       [1., 1., 1., 0., 0., 0., 1., 1., 0.],
       [1., 0., 1., 1., 0., 0., 0., 1., 1.],
       [1., 0., 0., 0., 1., 0., 0., 1., 0.],
       [1., 0., 0., 0., 1., 1., 1., 0., 1.],
       [1., 0., 0., 0., 0., 1., 0., 1., 0.]])
a = ad.AnnData(arr)
a.uns = {'spatial': {'connectivities_key': 'spatial_connectivities', 'distances_key': 'spatial_distances', 'params': {'n_neighbors': 8, 'coord_type': None, 'radius': 1.5}}}
a.obsp['spatial_connectivities'] = conn
a.obsm['coords'] = arr.copy()
a.obsm['spatial'] = arr.copy()

sc.pl.embedding(a, 'coords',  edges=True, neighbors_key = 'spatial')
print('coords', a.obsm['coords'])
sc.pl.embedding(a, 'spatial')
print('spatial no edges', a.obsm['spatial'])
sc.pl.embedding(a, 'spatial',  edges=True, neighbors_key = 'spatial')
print('spatial edges', a.obsm['spatial'])

produces following output:

expected (when plotting "coords")

coords [[96 55]
 [95 54]
 [96 54]
 [97 54]
 [95 55]
 [97 55]
 [95 56]
 [96 56]
 [97 56]]

when plotting "spatial" without edges (obsm is not modified)

spatial no edges [[96 55]
 [95 54]
 [96 54]
 [97 54]
 [95 55]
 [97 55]
 [95 56]
 [96 56]
 [97 56]]

when plotting "spatial" with edges (obsm is modified)

spatial edges [[ 96 255]
 [ 95 254]
 [ 96 254]
 [ 97 254]
 [ 95 255]
 [ 97 255]
 [ 95   0]
 [ 96   0]
 [ 97   0]]

Versions

scanpy==1.6.1.dev70+g7f15d22d anndata==0.7.5 umap==0.4.6 numpy==1.19.4 scipy==1.5.3 pandas==1.1.4 scikit-learn==0.23.2 statsmodels==0.12.1 python-igraph==0.8.3 louvain==0.6.1 leidenalg==0.8.3

ok, thought a bit and tested it, the problem is indeed a the type np.uint8. If dtype set to anything float then function works as expected.
The reason it fails is because reverting axes returns this, which is clearly not an int:
spatial edges [[1.888e+03 1.000e+00]
 [1.887e+03 2.000e+00]
 [1.888e+03 2.000e+00]
 [1.889e+03 2.000e+00]
 [1.887e+03 1.000e+00]
 [1.889e+03 1.000e+00]
 [1.887e+03 0.000e+00]
 [1.888e+03 0.000e+00]
 [1.889e+03 0.000e+00]]

This is also the only way to revert the axis, since data_points[:,1][::-1 doesn't a different type of transformation (which is not what we want in this case).
What we might want to do is pass a warning/fail if np.uint is passed, something like np.issubdtype(arr.dtype, np.integer)


Add filter when column type is Number

With elements type String smart table setFilter function is working fine, but it's not working with elements of type number.
Let's say I want to list all data with temperature of 30. Example of setFilter function:
this.source.addFilter({ field: 'temperature', search: 30 }, true, false);
I'm getting

No data found

Is there any difference when the number type is used?

yout implements custom search
with onSearch(query: string = "") and  on table component (keydown.enter)="onSearch(search.value)"

 {
            field: "temperature",
            search: query,
            filter: (data: any, query?: string) => {
              return data.toString().includes(query);
}

First of all, thank you @vicenthy.
If understand correctly, I have to modify the temperature field in a way you told me and add the (keydown.enter)="onSearch(search.value)" in HTML inside <ng2-smart-table>.
But where I have to add onSearch(query: string = "") and what I'm writing inside that function?
Regards.

https://akveo.github.io/ng2-smart-table/#/examples/using-filters

I would like to have a filter for each column. Is there an option for a simple convert to String.
I've tried the following but without success:
speed: {
    title: 'Motor Speed',
    valuePrepareFunction: (cell, element) =>
    element.speed.toString(),
},

create a function in your component
onSearch(query: string = '') {

 // this function makes the customized filter

  this.source.setFilter([
    // fields we want to include in the search
    {
      field: 'id',
      search: query
    },
    {
      field: 'name',
      search: query
    },
    {
      field: 'username',
      search: query
    },
    {
      field: 'email',
      search: query
    }
  ], false);
  // second parameter specifying whether to perform 'AND' or 'OR' search
  // (meaning all columns should contain search query or at least one)
  // 'AND' by default, so changing to 'OR' by setting false here
}

on your template

<input #search class="search" type="text" placeholder="Search..." (keydown.enter)="onSearch(search.value)">
    <ng2-smart-table [settings]="settings" [source]="source"></ng2-smart-table>

when Column is a string type ,this works for me.

With elements type String smart table setFilter function is working fine, but it's not working with elements of type number.
Let's say I want to list all data with temperature of 30. Example of setFilter function:
this.source.addFilter({ field: 'temperature', search: 30 }, true, false);
I'm getting

No data found

Is there any difference when the number type is used?

when Column is a string type ,this works for me.


Use Clang's native code coverage workflow

Use Clang's native code coverage compilation, gathering and result reporting if the compiler is Clang, as described in https://clang.llvm.org/docs/SourceBasedCodeCoverage.html.
Also use an option for enabling coverage instead of a build target because they are orthogonal - can have each build target with or without coverage. Mimic -DWERROR.

To test this in a quicker way I temporarily applied this patch to run just a few tests instead of all:
--- i/cmake/script/Coverage.cmake
+++ w/cmake/script/Coverage.cmake
@@ -10,12 +10,15 @@ if(EXTENDED_FUNCTIONAL_TESTS)
 endif()
 if(DEFINED JOBS)
   list(APPEND CMAKE_CTEST_COMMAND -j ${JOBS})
   list(APPEND functional_test_runner -j ${JOBS})
 endif()

+list(APPEND functional_test_runner p2p_ping)
+list(APPEND CMAKE_CTEST_COMMAND --tests-regex addrman_tests)
+
 # Run the tests.

 execute_process(
   COMMAND ${CMAKE_CTEST_COMMAND}
   WORKING_DIRECTORY ${CMAKE_CURRENT_LIST_DIR}
   COMMAND_ERROR_IS_FATAL ANY

General Concept NACK. Anytime we are trying to branch on compiler inside the build system, something is generally going wrong. It's unclear why we need to continue to hard code more cases here, rather than using presets, or, just letting devs set flags as required. I think the only reason the coverage stuff was ported was for parity, but if everyone is using something else, then it'd make more sense to have that (alone) in the build system, if anything, rather than adding more ways to generate coverage.

What is the suggested workflow when using the "Ninja Multi-Config" generator?

It should be the same as the suggested workflow to run the tests when using "Ninja Multi-Config" (without coverage enabled). How do we suggest to the users/devs to run the tests in that case?

Anytime we are trying to branch on compiler inside the build system, something is generally going wrong.

The branching on compiler in CMakeLists.txt can be substituted with try_append_cxx_flags("-fprofile-instr-generate;-fcoverage-mapping") otherwise try_append_cxx_flags("-Og;--coverage") otherwise error: don't know how to enable coverage. Does that sound better? Branching on compiler in cmake/script/Coverage* is already in cmake-stage without this PR, do you consider that "wrong" as well?

It's unclear why we need to continue to hard code more cases here

Because it can be done better. Clang native reports are better.

rather than using presets, or, just letting devs set flags as required.

Are you talking about just the configure step? Presets and/or APPEND_CXX_FLAGS can be used to do the configure step. But the current code (cmake -P build/Coverage.cmake in cmake-staging without this PR) also runs the tests and generates the reports in a gcc-specific way. This PR accommodates the Clang way into that, I am open to suggestions how to do that better.

acb8df9a5a...194758b08e: remove the branching on compiler from CMakeLists.txt
This is incomplete wrt collecting coverage from the fuzz and dealing with ninja multi-config. Will convert it to draft.

Since https://github.com/bitcoin/bitcoin/pull/30454 has been merged, further development in this repository no longer makes sense. Please consider moving this PR to the main repository.

Closing this because, for sure now the point here is not to merge it into hebasto/cmake-staging.


Details needed on licenses

The article says:
"Azure Active Directory Premium licenses for your tenant."
What does this mean? Does this mean your tenant needs a single P1 license? Or a P2 license? Or every user needs a P1 license?

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

ID: cba137cc-67ca-2766-b436-54e3634f0c2f
Version Independent ID: 2f185fba-dcf2-04f5-17a1-b641dc104edd
Content: Azure AD Connect: Group writeback V1
Content Source: articles/active-directory/hybrid/how-to-connect-group-writeback.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@erichenson
Thanks for your feedback! We will investigate and update as appropriate.

Azure AD is always licensed per user. A proper license is required if a user benefits directly or indirectly from any feature covered by that license.
I made a pull request to update the guide to specify that at least a Premium P1 license is required for each user, but I will leave it open for review by @billmath
https://github.com/MicrosoftDocs/azure-docs-pr/pull/192709


auth continually crashes on Windows host

When docker is running in Windows, the auth docker does not run.

What does it do (i.e. what are the logs), and who’s running it in windows?

On Feb 7, 2020, at 5:53 PM, Alex Layton<EMAIL_ADDRESS>wrote:
When docker is running in Windows, the auth docker does not run.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/OADA/oada-srvc-docker/issues/43?email_source=notifications&email_token=AA2KEUF6GGMTFAVUD3IO5WTRBXRAJA5CNFSM4KRWDOUKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IL6DSXA, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2KEUHGS27OXCIFOB2ZRSDRBXRAJANCNFSM4KRWDOUA.

Part of the issue was URIjs changing its name to urijs, which I have pushed a fix for to #33. This created issues because NTFS is not case sensitive.
The remaining issue is that the symlinks are not understood by windows (e.g., it breaks trying to follow the link from domains-enabled/localhost to domains-available/localhost).

I figured out the symlink thing. The tl;dr is you have to enable "Developer Mode" for symlinks to work in Windows for some reason. 🤷♂
So we can call this issue closed when the changes from #33 get merged in.

Fixed by merge of #33


Deferring from the execution of another defer can result in deadlock

version: 0.6.0
rustc: 1.29.1
os: macOS 10.13.6
After modifying some logic in sled to defer transitioning some state, which in turn triggered a nested defer and flush call I introduced a deadlock, which I traced to this recursive call to defer() and then flush().  Removing flush() gets it to work in this case, but I'm pretty sure that this will hit the same deadlock when the local Bag is filled, right?
In this case, I was able to reason that this would always only reach that nested defer from an outer one, so I was able to just remove the nested call.
Maybe it would be good to add some additional documentation to Guard::defer to warn people away from this usage?
To reproduce, run cargo test chunky from the root of the sled repo at commit c74a31f9.

After looking at this more closely, I'm 95% sure this is a deadlock in my own code. Sorry for the noise!


docker-compose recipe: Fresh instance shows blank white page

Full log from docker-compose build && up: http://www.onerussian.com/tmp/docker-compose-dandiarchive-1040994.log
girder is up and accessible (on http://localhost:8091/), but for http://localhost:8092/ browser shows a blank page and console has
DataDetails.vue:72 Uncaught TypeError: Cannot read property 'methods' of undefined
    at Module../node_modules/vuetify-loader/lib/loader.js!./node_modules/cache-loader/dist/cjs.js?!./node_modules/vue-loader/lib/index.js?!./node_modules/@girder/components/src/components/DataDetails.vue?vue&type=script&lang=js& (DataDetails.vue:72)
    at __webpack_require__ (bootstrap:726)
    at fn (bootstrap:100)
    at Module../node_modules/@girder/components/src/components/DataDetails.vue?vue&type=script&lang=js& (DataDetails.vue?5690:1)
    at __webpack_require__ (bootstrap:726)
    at fn (bootstrap:100)
    at<EMAIL_ADDRESS>(DataDetails.vue:1)
    at __webpack_require__ (bootstrap:726)
    at fn (bootstrap:100)
    at Module../node_modules/vuetify-loader/lib/loader.js!./node_modules/cache-loader/dist/cjs.js?!./node_modules/vue-loader/lib/index.js?!./src/views/Home.vue?vue&type=script&lang=js& (DandiMetaViewer.vue:233)
vue.runtime.esm.js:8423 Download the Vue Devtools extension for a better development experience:
https://github.com/vuejs/vue-devtools
vue.runtime.esm.js:8434 You are running Vue in development mode.
Make sure to turn on production mode when deploying for production.
See more tips at https://vuejs.org/guide/deployment.html

Browser: Chromium Version 79.0.3945.130 (Developer Build) built on Debian bullseye/sid, running on Debian bullseye/sid (64-bit)

just an fyi that i cannot replicate this bug on osx catalina with current master.

Tried again after pulling all relevant base images.  Full log: http://www.onerussian.com/tmp/docker-compose-2-1040994.log -- result is exactly the same.  @satra -- what versions of the base images you have?  here is my list including those built ones:
$> docker images | grep -e '^dandi' -e '^node' -e '^girder' -e '^node'
dandi_girder                            latest              44d00353655e        20 minutes ago      1.41GB
dandi_client                            latest              9288faa94496        37 hours ago        143MB
dandi_provision                         latest              04dcbd13b01d        5 days ago          930MB
node                                    latest              7aef30ae6655        6 days ago          943MB
girder/girder                           latest-py3          6c89826e4cf8        6 days ago          1.27GB
node                                    <none>              e0e011be5f0f        2 weeks ago         942MB
girder/girder                           <none>              90b29e2af6ac        3 weeks ago         1.29GB
girder_girder                           latest              429574d14760        5 weeks ago         1.26GB
dandiarchive/pynwb-mpi                  latest              d05cd3020d35        4 months ago        467MB
node                                    8-stretch           52a261fa5e4b        4 months ago        895MB

Found that my repo is "dirty", so I wonder if web/package-lock.json can somehow be implicated

my "dirty" state
(git)lena:~/proj/dandi/dandiarchive[master]
$> git status
On branch master
Your branch is up to date with 'origin/master'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
	venvs/
	web/package-lock.json

nothing added to commit but untracked files present (use "git add" to track)
1 36468.....................................:Thu 12 Mar 2020 12:04:02 PM EDT:.
(git)lena:~/proj/dandi/dandiarchive[master]
$> cat web/package-lock.json
{
  "name": "dandi",
  "version": "0.1.0",
  "lockfileVersion": 1,
  "requires": true,
  "dependencies": {
    "yarn": {
      "version": "1.19.0",
      "resolved": "https://registry.npmjs.org/yarn/-/yarn-1.19.0.tgz",
      "integrity": "sha512-QLiMoj+ZHo2kT17x1t7erO6gzR+9xSa5z7Lmei1qYM1Sj37wiIFv67kQcH+nkshsO0f03ImXClfkEKUgnsMreQ=="
    }
  }
}

will rebuild after removing it

FWIW -- that wasn't it... BTW -- runs fine on another host of mine, so it is something about the state of my laptop which "broke" it (since it was working before), some cache/memory of glorious days somewhere.

FWIW, here is the JS where it is happening:

so it is dateFormatter which is undefined. may be that rings some bell @mgrauer @AlmightyYakob ?

Ha -- comparing the logs (should have just grepped ;)) between good (left) and bad (right) builds saw:

which is in text:
^[[0m WARNING  Compiled with 4 warnings4:42:58 PM

 warning  in ./src/views/Home.vue?vue&type=script&lang=js&

"export 'DataDetails' (imported as 'GirderDataDetails') was not found in '@girder/components/src/components'
^[[91m<s> [webpack.Progress] 100%

^[[0m
 warning  in ./node_modules/@girder/components/src/components/DataDetails.vue?vue&type=script&lang=js&

"export 'dateFormatter' was not found in '../utils/mixins'

 warning  in ./node_modules/@girder/components/src/components/DataDetails.vue?vue&type=script&lang=js&

"export 'dateFormatter' was not found in '../utils/mixins'

 warning  in ./node_modules/@girder/components/src/components/DataDetails.vue?vue&type=script&lang=js&

"export 'usernameFormatter' was not found in '../utils/mixins'

  File           Size                          Gzipped

  dist/app.js    7012.65 KiB                   1003.82 KiB

  Images and other types of assets omitted.

 DONE  Build complete. The dist directory is ready to be deployed.
 INFO  Check out deployment instructions at https://cli.vuejs.org/guide/deployment.html

I think the build should treat such "warnings" as "errors" and fail.  Now to look WTF happened which could have lead to it

I can only suspect vue-cli-service build --mode docker ATM somehow.  Paths listed in the progress log seems to not necessarily be in the same order, and I suspect that in the "bad" run (where it reports built 1308) it just manages to miss some (in the good one it is 1314).

ok, giving up for now -- all what I have tried is provided in https://github.com/dandi/dandiarchive/pull/135 .  Someone who has a clue in those technologies should make build robust, and not fail on random super-fast laptops ;)

FWIW I wasn't able to reproduce this either.

I am ok to be "unique" ;-)  I know that it makes it harder to fix, but if you have ideas on how to e.g. disable parallel builds like I tried to do in #135 (but without notable effect -- may be I did it wrong) -- I could try on my unique speedy laptop ;)

@yarikoptic Are you able to reproduce this if you build from the refactor branch? That has updated a bunch of libraries from the build process.

Fixed

Sorry, missed original request, but the issue persists although differently. Just did a fresh
docker-compose build --pull
docker-compose up

in
(git)lena:~/proj/dandi/dandiarchive[master]docker
$> git describe
fatal: No annotated tags can describe 'd30abc143a776e22de8ff1814c3da4e98b32f918'.
However, there were unannotated tags: try --tags.

and get a blank page with console saying
rest.js:37 Uncaught TypeError: Cannot read property 'endsWith' of undefined
    at eval (rest.js:37)
    at Module../src/rest.js (app.js:1365)
    at __webpack_require__ (app.js:80)
    at eval (AppBar.vue?./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader/lib??vue-loader-options:4)
    at Module../node_modules/cache-loader/dist/cjs.js?!./node_modules/babel-loader/lib/index.js!./node_modules/cache-loader/dist/cjs.js?!./node_modules/vue-loader/lib/index.js?!./src/components/AppBar/AppBar.vue?vue&type=script&lang=js& (app.js:190)
    at __webpack_require__ (app.js:80)
    at eval (VM2345 AppBar.vue:2)
    at Module../src/components/AppBar/AppBar.vue?vue&type=script&lang=js& (app.js:1101)
    at __webpack_require__ (app.js:80)
    at eval (VM2345 AppBar.vue:3)

@mgrauer could someone look into it?  issue persists, might relate to the port number change etc.

I don't think this fits the definition of critical. We are trying to focus on the API unification currently.
@danlamanna looked into it before and couldn't reproduce, @satra also tried to replicate and couldn't.

I guess I have rushed to re-open instead of just filing a new issue.  I would appreciate if docker-compose up + visit frontpage would have been tried again -- original issue from March was likely a different one and this one I believe is appearing ATM for both me and @jwodder .  Meanwhile I have reproduced it again on this master as of  0.1.0-18-g6fcf80c

I removed all docker containers (docker ps -a | f1 | grep -v CON | xargs docker rm)
pruned all volumes (docker volume prune)
cd docker
docker-compose up and waited for provisioning to stop with docker_provision_1 exited with code 0
went to http://localhost:8085/ -- saw blank page.

inspect, console showed that traceback
Uncaught TypeError: Cannot read property 'endsWith' of undefined
    at eval (rest.js:37)
    at Module../src/rest.js (app.js:1365)
    at __webpack_require__ (app.js:80)
    at eval (AppBar.vue?./node_modules/cache-loader/dist/cjs.js??ref--12-0!./node_modules/babel-loader/lib!./node_modules/cache-loader/dist/cjs.js??ref--0-0!./node_modules/vue-loader/lib??vue-loader-options:4)
    at Module../node_modules/cache-loader/dist/cjs.js?!./node_modules/babel-loader/lib/index.js!./node_modules/cache-loader/dist/cjs.js?!./node_modules/vue-loader/lib/index.js?!./src/components/AppBar/AppBar.vue?vue&type=script&lang=js& (app.js:190)
    at __webpack_require__ (app.js:80)
    at eval (VM1161 AppBar.vue:2)
    at Module../src/components/AppBar/AppBar.vue?vue&type=script&lang=js& (app.js:1101)
    at __webpack_require__ (app.js:80)
    at eval (VM1161 AppBar.vue:3)

and looking at that long onelined js smells like `VUE_APP_PUBLISH_API_ROOT` is not setup/passed correctly...

oh, I wonder if that is just due to the manifestation of https://github.com/dandi/dandiarchive/issues/474 and what @jwodder ran into awhile back...?

Closing because the new API provides a Docker compose setup that works for John and Yarik.


Remove the unofficial TypeScript definitions from DefinitelyTyped

Currently there's unofficial, likely unmaintained, and out-of-date TypeScript definitions published via https://github.com/DefinitelyTyped/DefinitelyTyped for the pdfjs-dist library; see https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/pdfjs-dist
Now that we, from version 2.6.347, are shipping official TypeScript definitions through https://github.com/mozilla/pdfjs-dist, it'd be a good idea to submit a PR to DefinitelyTyped to remove its TypeScript definitions; see https://github.com/DefinitelyTyped/DefinitelyTyped#removing-a-package

The official TS definitions bundled in 2.6.347, while much appreciated, only include bindings for the top-level API and therefore lack many of the useful types currently exposed by the DefinitelyTyped definitions, such as:

PDFPageProxy
PDFDocumentProxy
PDFRenderTask
PDFDocumentLoadingTask

It's possible to access these types through the official definitions by using complex sequences of type operations, e.g.:
type PDFDocumentProxyPromise = ReturnType<typeof getDocument>['promise']
type PDFDocumentProxy = PDFDocumentProxyPromise extends Promise<infer U> ? U : never

However, this is inconvenient and also unconventional within the TypeScript ecosystem. Type definitions for libraries usually export typings for important objects/classes (without exporting their values).
Would you consider exposing these types in the official type definitions before removing the typings from DefinitelyTyped?

I would say so since I think they are all part of the public API; see for example https://github.com/mozilla/pdf.js/blob/master/src/display/api.js#L2966-L2967. The TypeScript definitions are generated from our JSDoc comments because PDF.js itself is not a TypeScript project, so any improvements for the typings are much appreciated from the community who have already helped a lot to get the current typings in place.

I would say so since I think they are all part of the public API;

Well, the public API is actually only https://github.com/mozilla/pdf.js/blob/master/src/pdf.js :-)

see for example https://github.com/mozilla/pdf.js/blob/master/src/display/api.js#L2966-L2967

That was exposed there for the unit-tests specifically, as far as I remember.

[...] because PDF.js itself is not a TypeScript project, so any improvements for the typings are much appreciated

I do believe that we said, when landing the TypeScript definitions, that it'll be up to the TypeScript users to help maintain the definitions. (Since as mentioned this is a JavaScript project, and the PDF.js contributors cannot be assumed to know TypeScript.)

Generally speaking: I'm not crazy about the idea of simply exposing those properties in the public API, since it's objects that users should never call/initialize manually.
It would seem preferably, as far as I can tell, if there's a way of exposing those them in the TypeScript definitions without having to extend the official API in the process.

D'oh, I looked at the wrong file indeed. I think the question arose from the observation that the API can return those types, and the TypeScript definitions are used to indicate the type of return value from our API so it's clear how to interact with it. If we were to expose those types in the definitions (using JSDoc comments) that wouldn't change the API itself I think. The objects themselves cannot be initialized since we don't export those, but the TypeScript definitions will then contain the description of those objects returned from the API.
We're indeed open to community improvements in this regard since the community also knows best what would fit their needs and how to test this.

if there's a way of exposing those them in the TypeScript definitions without having to extend the official API in the process

Yes, that should be possible, and it aligns with the TypeScript ecosystem in general. Regardless of whether a library is written in JS or TS, its typings will often expose some internal types in addition to types for the top-level API. This is useful when passing around values that get produced by the library's API. Here's a contrived example; without the PDFDocumentProxy type exported, it would be difficult to type the getNumPagesOfDocument function correctly:
import {getDocument, PDFDocumentProxy} from 'pdfjs-dist'

function getNumPagesOfDocument(doc: PDFDocumentProxy): number {
  return doc.numPages
}

async function getNumPagesOfFile(file: File): number {
  const doc = await getDocument(file).promise
  return getNumPagesOfDocument(doc)
}

Note that getDocument is a value whereas PDFDocumentProxy is a type. TypeScript will not permit the latter to be used as a value. All references to it will be removed when the code is compiled to JS.
Anyway, I'm not sure when I'll find time for this, but, if someone else doesn't beat me to it, I'll look into how the pdfjs type generation currently works and see if there's a simple to way to manually list a few types that should be exported in addition to those of the top-level runtime API. Thanks for being open to the change!

You can import them with the following:
import {PDFDocumentProxy, PDFPageProxy} from 'pdfjs-dist/types/display/api'

We can export the types, not actual classes, at pdf.d.ts with the following JSDoc.
diff --git a/src/pdf.js b/src/pdf.js
index c6a6d1338..4ef9d30b7 100644
--- a/src/pdf.js
+++ b/src/pdf.js
@@ -13,6 +13,11 @@
  * limitations under the License.
  */

+/**
+ * @typedef {import("./display/api").PDFDocumentProxy} PDFDocumentProxy
+ * @typedef {import("./display/api").PDFPageProxy} PDFPageProxy
+ */
+
 import {
   addLinkAttributes,
   getFilenameFromUrl,

Then the generated build/types/pdf.d.ts begins with the following lines:
export type PDFDocumentProxy = import("./display/api.js").PDFDocumentProxy;
export type PDFPageProxy = import("./display/api.js").PDFPageProxy;

To provide some additonal context on the typings discussion, the example at https://github.com/mozilla/pdf.js/blob/master/examples/components/simpleviewer.js, uses the EventBus, PDFFindController, PDFLinkService, PDFViewer - all of which are exported by pdfjs-dist/web/pdf_viewer.js, but do not have typings defined in pdfjs-dist/types/*.

It looks like this was attempted in https://github.com/DefinitelyTyped/DefinitelyTyped/pull/52508, however there were apparently issues with other dependencies :-(
Closing this issue, since I don't think it's necessary for the PDF.js project to keep tracking this (given that it cannot be fixed here).


Refreshing gutter failed!

[I'm supposed to describe steps to replicate my issue here and if I don't do that, I understand that my issue will be probably ignored.]

Brackets 1.4.0-16380 (mac), Git 2.3.2 (Apple Git-55) & Brackets-Git 0.14.32
Refreshing gutter failed!
Error:

Agreeing to the Xcode/iOS license requires admin privileges, please re-run as root via sudo.

call: git status -u --porcelain -- app/app/ux/FieldReorder.js
Error:

Agreeing to the Xcode/iOS license requires admin privileges, please re-run as root via sudo.

    at Object.exports.toError (file:///Users/saenzramiro/Library/Application%20Support/Brackets/extensions/user/zaggino.brackets-git/src/ErrorHandler.js:162:19)
    at NodeConnection.<anonymous> (file:///Users/saenzramiro/Library/Application%20Support/Brackets/extensions/user/zaggino.brackets-git/src/Cli.js:185:44)
    at j (file:///Applications/Brackets.app/Contents/www/thirdparty/thirdparty.min.js:559:26911)
    at Object.k.fireWith (file:///Applications/Brackets.app/Contents/www/thirdparty/thirdparty.min.js:559:27724)
    at NodeConnection._receive (/utils/NodeConnection.js:498:34)

Solution here: https://github.com/zaggino/brackets-git/issues/1191


platformio: 5.0.4 -> 5.1.1

Motivation for this change
Update platformio to its latest release. However, to get this changes working, the following pull requests needed to be merged first:

[ ] #112701, python3Packages.starlette: 0.13.8 -> 0.14.2
[x] #114012, pythonPackages.json-rpc: init at 1.13.0
[ ] #116727, pythonPackages.ajsonrpc: init at 1.1.0

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@ofborg eval

I have updated and rebased this pull request for PlatformIO's latest release 5.1.1. This introduced a new dependency, ajsonrpc, for which I created another PR #116727. However, to make this changes work, the starlette package update needs to be merged first, #112701.
I have successfully tested this package with those two changes applied locally.

After all necessary dependencies were merged, I rebased this PR once again and now it should be ready for a final review.

FWIW I have also tested this locally and it is working as intended.

@SuperSandro2000: Would you please take another look? Thanks!

Result of nixpkgs-review pr 114014 run on x86_64-darwin 1

The PR looks good to me, doesn't seem to be a major change anyway.
I'm going to go ahead and merge this considering everybody is busy with the coming release


problem with console i/o: "NoneType" written to it

Hi,
Trying to load XXDP modules into the emulator to do extra testing of its emulation.
For that I implemented a tape image loader which should be able to read e.g. a .BIC file from the XXDP set.
The loading at itself works fine but as soon as the emulated code wants to write a character to the console, an PDPTraps.AddressError is triggered. I dug a bit into it and klregs from KL11 in kl11.py is called with address 017566 and a value of None, yet the following is logged:
pdp11.DEBUG[004933]: Byte write to 0o17566 value=13
pdp11.DEBUG[004933]: TRAP: AddressError(vector=0o4):

BKTCB0.BIC.zip
tapeloader.py.zip
Stack trace:
* Stack trace, innermost first
*
** byteme at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/mmio.py:215
*** ioaddr: 8054
*** value: None
*** opsize: 2
*** iofunc: <bound method KL11.klregs of <kl11.KL11 object at 0x75ad6c350d90>>
*** self: <mmio.MMIO object at 0x75ad6c346450>

** wordRW at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/mmio.py:280
*** self: <mmio.MMIO object at 0x75ad6c346450>
*** ioaddr: 8054
*** value: None        <=------- here value is suddenly None

** byteme at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/mmio.py:220
*** ioaddr: 8054
*** value: 13
*** opsize: 1
*** iofunc: <bound method KL11.klregs of <kl11.KL11 object at 0x75ad6c350d90>>
*** self: <mmio.MMIO object at 0x75ad6c346450>

** byteRW at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/mmio.py:290
*** self: <mmio.MMIO object at 0x75ad6c346450>
*** ioaddr: 8054
*** value: 13

** byteRW at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/mmu.py:559
*** self: <mmu.MemoryMgmt object at 0x75ad6cf24d50>
*** vaddr: 65398
*** value: 13
*** mode: None
*** space: 1
*** pa: 65398
*** odd: 0

** operandx at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/machine.py:528
*** self: <machine.PDP1170 object at 0x75ad6c5b6010>
*** b6: 55
*** value: 13
*** opsize: 1
*** altmode: None
*** altspace: None
*** rmw: False
*** justEA: False
*** Rn: 7
*** autocrement: 0
*** space: 1
*** extendedb6: None
*** addr: 65398
*** addrmode: 48
*** x: 64786

** op11_movb at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/op4.py:110
*** cpu: <machine.PDP1170 object at 0x75ad6c5b6010>
*** inst: 37943
*** val: 13
*** dst: 55

** run at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/machine.py:585
*** self: <machine.PDP1170 object at 0x75ad6c5b6010>
*** pc: 128
*** breakpoint: None
*** interrupt_mgr: <interrupts.InterruptManager object at 0x75ad6c346410>
*** mmu: <mmu.MemoryMgmt object at 0x75ad6cf24d50>
*** abort_trap: None
*** thisPC: 608
*** inst: 37943

** <module> at /home/folkert/Projects/PDP-11/json/python-pdp11-emulator/./tapeloader.py:63
*** __name__: '__main__'
[...]

In mmio.py line 220 a word is read from the i/o to convert the byte-write into a word-write.
For that to work I think you need to do something like this:
diff --git a/kl11.py b/kl11.py
index b8ad643..9db36fc 100644
--- a/kl11.py
+++ b/kl11.py
@@ -125,7 +125,8 @@ class KL11:
                         self.ub.intmgr.simple_irq(pri=4, vector=0o64)

             # transmit buffer
-            case 6 if value is not None:              # tbuf
+            case 6:
+                if value is not None:              # tbuf
                     # *** WRITING ***
                     value &= 0o177
                     if (value != 0o177):
@@ -133,6 +134,8 @@ class KL11:
                         self.tq.put(s)
                     if self.t_ienable:
                         self.ub.intmgr.simple_irq(pri=4, vector=0o64)
+                else:
+                    value = 0
             case _:
                 raise PDPTraps.AddressError

Ah - can't have a write-only register unless it explicitly implements byte-write (because the automatic byte I/O handling relies on being able to read the register). Doh!
The KL11 manual actually says this register is "load only" ... which is how it is implemented in kl11.py but that's inconsistent with relying on mmio for automatic byte-write conversion as you ran into.
I may think about a more general mod in mmio byteme to allow for write-only registers that still want automatic byte I/O (needs to catch this trap and synthesize the zero - same fix just expressed without making the device write-only registers respond) but this simpler fix to kl11 will be good for now. Until, that is, later we discover a diagnostic that complains about KL11 transmit buf register being readable when it isn't supposed to be :)

Fixed in KL11 as described, at least for now.


test: Test the new e2e tests plus roles mgmt

Thanks for submitting a PR! Please check the boxes below:

[X] I have run pre-commit to check linting
[ ] I have filled in the "Changes" section below?
[ ] I have filled in the "How did you test this code" section below?
[ ] I have used a Conventional Commit title for this Pull Request

Changes
Test the new e2e tests plus real changes of the roles management PR
How did you test this code?
DO NOT MERGE only for testing purposes

Codecov Report
Patch coverage: 67.85% and project coverage change: -0.03% :warning:

Comparison is base (003d782) 95.47% compared to head (a8487b9) 95.44%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2725      +/-   ##
==========================================
- Coverage   95.47%   95.44%   -0.03%
==========================================
  Files         987      988       +1
  Lines       27780    27805      +25
==========================================
+ Hits        26523    26539      +16
- Misses       1257     1266       +9

Files Changed
Coverage Δ

api/e2etests/e2e_seed_data.py
57.14% <57.14%> (ø)

...sts/tests/end_to_end/test_integration_e2e_tests.py
100.00% <100.00%> (ø)

api/e2etests/views.py
100.00% <100.00%> (ø)

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


Remove more styling when  --pretty-print=never is set

I have recently been trying out numbat and how it compares to qalc.
My main use case is quick cli calculations
$ qalc -t '8 GiB to MiB'
8192 MiB
$ qalc -t
> 8 GiB to MiB

  8192 MiB

>

For numbat though I'm not able to get the same level of brevity.
The closest I can get to this with numbat is --pretty-print=never
$ numbat --pretty-print=never -e '8 GiB to MiB'
    = 8192 MiB    [DigitalInformation]

$ numbat -q --pretty-print=never
>>> 8 GiB to MiB

    = 8192 MiB    [DigitalInformation]

>>>

I was wondering if it would make sense to have formatting closer to qalc when --pretty-print=never is set, and if a PR would be welcome for these changes
$ numbat --pretty-print=never -e '8 GiB to MiB'
8192 MiB
$ numbat -q --pretty-print=never
>>> 8 GiB to MiB

  8192 MiB

>>>

Additionally It would be nice get close to qalc -t in use, so maybe -e could default to --pretty-print=never?
Or add an option -t as an alias for --pretty-print=never
$ numbat -e '8 GiB to MiB'
8192 MiB
$ # or
$ numbat -te '8 GiB to MiB'
8192 MiB
$ numbat -qt
>>> 8 GiB to MiB

  8192 MiB

>>>

also, if numbat -e should default to no pretty printing, what about echo 1 + 1 | numbat

Additionally It would be nice get close to qalc -t in use, so maybe -e could default to --pretty-print=never?

This was the default behavior in Insect, and I agree it makes sense not to pretty-print in non-interactive mode by default (even without passing --pretty-print=never). Adding a new command-line flag for this isn't necessary in my opinion.

also, if numbat -e should default to no pretty printing, what about echo 1 + 1 | numbat

Both shouldn't pretty-print IMO.

I was wondering if it would make sense to have formatting closer to qalc when --pretty-print=never is set, and if a PR would be welcome for these changes

Yes :+1:

Additionally It would be nice get close to qalc -t in use, so maybe -e could default to --pretty-print=never?

Yes, good idea

also, if numbat -e should default to no pretty printing, what about echo 1 + 1 | numbat

I also think that the "stdin is not a terminal" case should be handled the same way as numbat -e.


[CVP-3274] Buildah can use registry self-signed certificate (#352)

Signed-off-by: Martin Vala<EMAIL_ADDRESS>

@wcheang @Allda can you have a look and review it? Thanks

Sorry, I had a follow-up question and didn't realize that it never got posted. The change looks good overall, I just didn't want there to be an unused workspace declared unless there is a change coming that will use it.

@wcheang don't merge it yet. this workspace is needed. Let me fix it

Now it is fine

@wcheang don't merge it yet. this workspace is needed. Let me fix it I am getting following error
failed: [localhost] (item=../templates/openshift/pipelines/operator-ci-pipeline.yml) => {"ansible_loop_var": "item", "changed": false, "error": 400, "item": "../templates/openshift/pipelines/operator-ci-pipeline.yml", "msg": "Pipeline operator-ci-pipeline: Failed to apply object: b'{\"kind\":\"Status\",\"apiVersion\":\"v1\",\"metadata\":{},\"status\":\"Failure\",\"message\":\"admission webhook \\\\\"validation.webhook.pipeline.tekton.dev\\\\\" denied the request: validation failed: invalid value: pipeline task \\\\\"build-bundle\\\\\" expects workspace with name \\\\\"registry-cacert\\\\\" but none exists in pipeline spec: spec.tasks[13].workspaces[2]\",\"reason\":\"BadRequest\",\"code\":400}\\n'", "reason": "Bad Request", "status": 400}

That error refers to build-bundle, not generate-index. From this commit, it looks like you had removed the registry-cacert workspace from the ci-pipeline, but it's still needed by the build-bundle task, hence the error.

@wcheang Sorry, my bad. It is fixed now.

@wcheang @Allda can it be merged?


Df 1860 validate forms

Created two new CF jQuery plugins to support validating forms and displaying a message when that validation fails.
Additions

Adds validate library to handle validation logic.
Adds handlebars library for JS templating.
Adds cf_formValidator for validating form inputs for multiple types and returning an object containing the valid and invalid inputs.
Adds cf_notifier plugin for creating a notification using a template and data objects that are customizable.
Adds cf-notifications stylesheet to be ported to CF in the future

Removals

Removes alert.js because not all messages are alerts, the term "notification" covers more of our bases for success and warning messages in the future.
Removes the alert macro so that we can insert messages as needed and remove them when we're done.

Changes

Updated cf-enhancements to include warning states for notifications and form input highlights

Testing

Checkout branch locally
navigate to /blog/ or /newsroom/ and use the subscribe form
navigate to /the-bureau/leadership-calendar/ and use the filtering form at the top (this will be updated, this is just for ease of testing the check groups)

Review

@everyone

Preview

Preview this PR without the whitespace changes
Notes

There's still a lot that could be done, but the plugin is in a pretty solid state. Some expansion will probably occur when adding the subscribe ajax feature.
Before merging down the origin, I wanted to get feedback and make updates first.
Will add tests and update the changelog after merging down origin and ping everyone again.
Using the namespace pattern for jQuery libraries to avoid breaking everything if an unknown method is passed and to ease up on passing data around from function to function.
Odd space between border and outline in Safari is a known bug.

:+1:.  My only suggestion would be precompiling the HandleBars templates but it's outside the scope for this task.

@sebworks How will that affect templates passed in the plugin options? The docs on pre-compilers didn't quite make sense to me.

So we would just call the template directly and pass in the data.
https://www.npmjs.com/package/hbsfy
https://github.com/cfpb/owning-a-home/blob/b891813cd71ee74688f005af2788fdf8425e5b7b/Gruntfile.js#L126

Now it's seriously ready for final sign-off. If you run it locally, be sure to run npm install && bower install

:+1:

:+1: