🐛 Fixed trapped focus in menu stories

resolves #1924

Is there now way to better fix interaction with Menu and keypress Enter? Maybe we need to step back and have a look at what we are trying to achieve.
These "fixes" feels like we are adding lots more unnecessary complexion for our users (in keypress on anchor button).
* Do we need a global keypress for `Enter`?

* Can this be done in `MenuList` or `MenuItem` instead?

As a user I would expect the component to handle focus on enter internally, but you mentioned last week there was some reason why we need to handle onkeydown outside of the component (I forgot what that reason was though 😅 )

Is there now way to better fix interaction with Menu and keypress Enter? Maybe we need to step back and have a look at what we are trying to achieve.
These "fixes" feels like we are adding lots more unnecessary complexion for our users (in keypress on anchor button).
* Do we need a global keypress for `Enter`?

* Can this be done in `MenuList` or `MenuItem` instead?

As a user I would expect the component to handle focus on enter internally, but you mentioned last week there was some reason why we need to handle onkeydown outside of the component (I forgot what that reason was though 😅 )

There was something that didn't work with attaching handlers on passed dom elements (anhchorEl). Maybe we should try to again to handle this internally in the Menu as the current approach feels cumbersome for our users. Especially since we've seen other component libraries do this internally aswell...

Made a new issue about moving keydown handling of anchor into the component #1936


Change default combine key

I'm recommending a small suggestion to change the default combine key to a tilde " ` " key.  With multiple monitors it's easy to press your combine hotkey on another window and have it go crazy.
There might be issues with laptops or other smaller keyboards where the tilde is missing or requires a function key.
Also an option to pause hotkey detection without closing the app might also work well.

Yeah, I really don't like the '9' key, myself. I have no objection to changing it to a back-quote, though I'm not sure how easy it is to access on international keyboards.

A back-tick in the Italian keyboard is Alt-Gr + ì, not something easy to access.
Also, you cannot trigger the combine when you have a modifier key pressed, to let people type Shift + 9 without problems.
There is an option to change the key exactly for this reason, I see no point in changing the default value.

Well, closing, as there is nothing to do.
If @jforr happens to have stronger arguments for a default key change, he/she can reopen this.


Fix exports on 5to6 to be able to update jscodeshift

What does this PR do?
I couldn't update jscodeshift to its latest version due to an issue on ast-types, and since 5to6-codemod hasn't been updated in 3 years, I made a patch using patch-package.
How should it be tested manually?
npm run prepublishOnly
# or
yarn prepublishOnly

# and...

npm test
# or
yarn test

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
310

Relevant Lines:
310

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
310

Relevant Lines:
310

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
310

Relevant Lines:
310

💛  - Coveralls


How to fetch selected?

Hi!
How can I fetch the selected @users in a array for example?

@ichord , how can we get the selected items in case of an TextArea control?

@raghu1286 It can't, for it don't know when the selected item was removed.


Fix var interpolation

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.78%. Comparing base (7f29a12) to head (d44a8b1).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #199   +/-   ##
=======================================
  Coverage   61.78%   61.78%
=======================================
  Files           2        2
  Lines         785      785
=======================================
  Hits          485      485
  Misses        249      249
  Partials       51       51

Flag
Coverage Δ

unit
61.78% <ø> (ø)

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

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


chore(map-dependencies): Update map dependencies (#269-update-mapbox-gl)

Udpated the following dependencies:

react-map-gl@latest
deck.gl@latest
@types/react-map-gl@latest
@danmarshall/deckgl-typings@latest

For now, one visible change is the difference between yellow tones while very zoomed out.

Before
After

I feel like this is related to the yellow border that surrounds each dot on the map.
I will investigate.

It does indeed fix the problem of dots not being displayed on apple silicon M1 MacBooks! 🥳
FYI: @Esshahn

I see the trees but not the city map

I see the trees but not the city map

This is normal this is just a preview thing. You won't have this problem when it's merged.


Fix growing tf buffer

Decreases the time for writing .ply for me from > 24h to < 20min for a 8 GB bag.

I am lacking context, but that seems to make the tf_buffer entirely useless. Or am I missing something?

Yeah, nevermind I guess.
After looking into it a bit more, still not sure why this actually works at all for me...
Nevertheless there seems to be a problem with the tf_buffer growing too large.
Code wise it should adjust it's buffer automatically when a tf is looked up.
This seems to not happen for me for some reason..

Fixed by #879


ZFS fails to build with latest Kernel on NixOS unstable

Describe the bug
I am running NixOS unstable on an encrypted ZFS root, upgrading the system fails with the error message that the builder for the latest zfs-kernel package failed.
Currently I am running linux-5.5.13 with zfs-kernel-0.8.3-5.5.13, the upgrade pulls in linux-5.6.2 with zfs-kernel-0.8.3-5.6.2.
Expected behavior
I would have expected an successful system upgrade.
Additional context
I am sorry if I am overseeing something here, I am quite new to NixOS and I haven't seen this issue open somewhere else. If I forgot some information please tell me then I will update this.
Error Message
This is the error message I am seeing:
In file included from /build/source/include/spl/sys/condvar.h:33,
                 from /build/source/include/sys/zfs_context.h:38,
                 from /build/source/include/sys/lua/lua.h:13,
                 from /build/source/module/lua/lctype.h:11,
                 from /build/source/module/lua/lctype.c:11:
/build/source/include/spl/sys/time.h:88:15: error: unknown type name 'time_t'
   88 | static inline time_t
      |               ^~~~~~
/build/source/include/spl/sys/time.h: In function 'gethrtime':
/build/source/include/spl/sys/time.h:108:18: error: storage size of 'ts' isn't known
  108 |  struct timespec ts;
      |                  ^~
/build/source/include/spl/sys/time.h:109:2: error: implicit declaration of function 'getrawmonotonic' [-Werror=implicit-function-declaration]
  109 |  getrawmonotonic(&ts);
      |  ^~~~~~~~~~~~~~~
/build/source/include/spl/sys/time.h:108:18: warning: unused variable 'ts' [-Wunused-variable]
  108 |  struct timespec ts;
      |                  ^~
/build/source/include/spl/sys/time.h:109:2: error: implicit declaration of function 'getrawmonotonic' [-Werror=implicit-function-declaration]
  109 |  getrawmonotonic(&ts);
      |  ^~~~~~~~~~~~~~~
/build/source/include/spl/sys/time.h:108:18: warning: unused variable 'ts' [-Wunused-variable]
  108 |  struct timespec ts;
      |                  ^~
cc1: some warnings being treated as errors
make[6]: *** [/nix/store/6mhxg6wwpi7rzx7f3qdiq3gn18a55yxa-linux-5.6.2-dev/lib/modules/5.6.2/source/scripts/Makefile.build:268: /build/source/module/lua/ldebug.o] Error 1
cc1: some warnings being treated as errors
cc1: some warnings being treated as errors
make[6]: *** [/nix/store/6mhxg6wwpi7rzx7f3qdiq3gn18a55yxa-linux-5.6.2-dev/lib/modules/5.6.2/source/scripts/Makefile.build:268: /build/source/module/lua/lcorolib.o] Error 1
make[6]: *** [/nix/store/6mhxg6wwpi7rzx7f3qdiq3gn18a55yxa-linux-5.6.2-dev/lib/modules/5.6.2/source/scripts/Makefile.build:268: /build/source/module/lua/lctype.o] Error 1
In file included from /build/source/include/spl/sys/condvar.h:33,
                 from /build/source/include/sys/zfs_context.h:38,
                 from /build/source/include/sys/lua/lua.h:13,
                 from /build/source/module/lua/ldo.c:12:
/build/source/include/spl/sys/time.h:88:15: error: unknown type name 'time_t'
   88 | static inline time_t
      |               ^~~~~~
/build/source/include/spl/sys/time.h: In function 'gethrtime':
/build/source/include/spl/sys/time.h:108:18: error: storage size of 'ts' isn't known
  108 |  struct timespec ts;
      |                  ^~
/build/source/include/spl/sys/time.h:109:2: error: implicit declaration of function 'getrawmonotonic' [-Werror=implicit-function-declaration]
  109 |  getrawmonotonic(&ts);
      |  ^~~~~~~~~~~~~~~
/build/source/include/spl/sys/time.h:108:18: warning: unused variable 'ts' [-Wunused-variable]
  108 |  struct timespec ts;
      |                  ^~
cc1: some warnings being treated as errors
make[6]: *** [/nix/store/6mhxg6wwpi7rzx7f3qdiq3gn18a55yxa-linux-5.6.2-dev/lib/modules/5.6.2/source/scripts/Makefile.build:268: /build/source/module/lua/ldo.o] Error 1
make[5]: *** [/nix/store/6mhxg6wwpi7rzx7f3qdiq3gn18a55yxa-linux-5.6.2-dev/lib/modules/5.6.2/source/scripts/Makefile.build:505: /build/source/module/lua] Error 2
make[4]: *** [/nix/store/6mhxg6wwpi7rzx7f3qdiq3gn18a55yxa-linux-5.6.2-dev/lib/modules/5.6.2/source/Makefile:1683: /build/source/module] Error 2
make[3]: *** [/nix/store/6mhxg6wwpi7rzx7f3qdiq3gn18a55yxa-linux-5.6.2-dev/lib/modules/5.6.2/source/Makefile:180: sub-make] Error 2
make[3]: Leaving directory '/nix/store/6mhxg6wwpi7rzx7f3qdiq3gn18a55yxa-linux-5.6.2-dev/lib/modules/5.6.2/build'
make[2]: *** [Makefile:30: modules] Error 2
make[2]: Leaving directory '/build/source/module'
make[1]: *** [Makefile:839: all-recursive] Error 1
make[1]: Leaving directory '/build/source'
make: *** [Makefile:708: all] Error 2
builder for '/nix/store/v93dbkws4x9b952x9qv6n23dc2pcc436-zfs-kernel-0.8.3-5.6.2.drv' failed with exit code 2
cannot build derivation '/nix/store/r749p1qabkk5bdkkis0s12nxdhp2ag59-kernel-modules.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/wgx3n9506ncf2wm7kspqy0iv71iagk0s-linux-5.6.2-modules.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/a32np42pgid39n8r9mqck2sy38fkd83p-nixos-system-Desktop-20.09pre220217.39247f8d04c.drv': 1 dependencies couldn't be built
error: build of '/nix/store/a32np42pgid39n8r9mqck2sy38fkd83p-nixos-system-Desktop-20.09pre220217.39247f8d04c.drv' failed

Metadata
❯ nix-shell -p nix-info --run "nix-info -m"
these paths will be fetched (0.05 MiB download, 0.28 MiB unpacked):
  /nix/store/qr6dpv0wl37p00rdj5pijn1mkz0wfkwd-bash-interactive-4.4-p23-dev
copying path '/nix/store/qr6dpv0wl37p00rdj5pijn1mkz0wfkwd-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.5.13, NixOS, 20.09pre218613.ae6bdcc5358 (Nightingale)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.3`
 - channels(root): `"nixos-20.09pre220217.39247f8d04c"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

cf. #84595

Ah thanks, haven't seen that one. I will re-test as soon as the channel got updated.

Should be fixed now.


Any plans to update this for use with the latest XCode & Swift?

Looks to me from the commit messages, that this was done

Updated project to compile with Xcode 9.2

Oops, clearly not - i get a ton of compile errors

I created a fork and fixed it for Xcode 10 and Swift 4.2.
https://github.com/innoreq/SwiftyWalkthrough


Strict Mode Email Validation blocks new TLD Addresses

The new TLD style emails<EMAIL_ADDRESS>for example) is not considered valid by the email_validator when strict_mode is true.
What's the best move to allow emails formatted like this in Clearance?

Are you sure? I looked quickly at the code for EmailValidator, which the validation uses, and enabling strict mode seems to have no impact on the regex used for the domain.
EmailValidator.regexp(strict_mode: true) =~<EMAIL_ADDRESS>#=> 0

I also just updated clearance's user_spec.rb locally to add:
  it { is_expected.to allow_value("hugh@sanctuary.computer").for(:email) }

This passes. What made you suspect it was something to do with strict mode or even clearance?

Oh!  It may be a collision with a different lib in that case...  I'll keep looking!  Thanks @derekprior !


Centralized config

I wish to store all my openmmlab configs in one place, lets say ${config_project}.
However, when I ran mim mmdet train ${config_project}/my_config.py ..., it always try to find this path under ${mmdet}/.mim or ${mmdet}, resulting in a FileNotFound error. Do I have to copy-paste all configs to my mmdet installation? Or there are other ways to do this?

Sorry, it seems to have been a wrong base config path...


[WB-9831] Rename wandb local to wandb server

Fixes WB-9831
Description
Renames wandb local to wandb server

@jsbroks I just made this handle all the cases, let me know how much you hate it ;)

I would argue we should not have wandb server as an alias for wandb server start. My reasoning: It makes the tool harder to use correctly (since its hard to discover the subcommands), other tools follow a help menu convention for sub commands, and its easier to add this feature later then to remove it.
E.g Here is googles CLI

My argument is that a user should never need to learn or know about any sub commands.  They really only need to be able to start a server.  What are the critical sub commands that we want the user to be able to discover by typing wandb server?

@jsbroks I implemented your suggestions and added a wandb server stop to provide some justification for making a new group of commands anyway.  Let's get this into the next release.  CC @sephmard


feat: add support for poll_answer

[x] All tests pass  (except game-related tests, which I managed to skip)
[x] I have run npm run doc

Description

Add support for poll_answer.
Currently, this library doesn't support poll_answer in getUpdates. Actually, this is easy to implement with several lines of code.
References
https://core.telegram.org/bots/api#update (the last line of the table)
response format: (some data wiped)
{
  "update_id": 12888888,
  "poll_answer": {
    "poll_id": "0000000000000000000",
    "user": {
      "id": 123456789,
      "is_bot": false,
      "first_name": "123456789",
      "last_name": "123456789",
      "username": "123456789",
      "language_code": "en"
    },
    "option_ids": [ 0 ]
  }
}

LGTM

Thanks for the PR, your commit was cherry-picked in #796.


bow character require short aim shot for proccing headshot

need to find out the frames for this

Have frames here. Just need to figure out syntax for it
https://docs.google.com/spreadsheets/d/1e8to2PGZgpykGsi2G-mkAWiY2OSLUBgUkWfM5N4B6fk/edit#gid=0

Got permission from BowTae#0141 to use this

updated sheet:
https://docs.google.com/spreadsheets/d/187T-SngEZUUordjY_K_tF_DdvHjQju9CoBJdp2eJOis/edit?usp=sharing

stuck in limbo because of syntax problems (#772)

done in #1980


Get a failed jobs notification periodically

This is to fetch job failures via jenkins api and post a message on the unresolved art threads
thread in team slack channel.
The motivation is to get a simple very visible place for release artists and team to notice the
overall health of our jenkins jobs in slack.
Test notification: https://redhat-internal.slack.com/archives/C03U0TPBQMR/p1717512811407959?thread_ts=1717512811.239669&cid=C03U0TPBQMR
Failed aos-cd-jobs in last 3 hours:
{
  "build/build-sync": 6,
  "build/ocp4": 1,
  "build/ocp4_scan": 1,
  "build/olm_bundle": 1,
  "build/scan-for-kernel-bugs": 1
}

/approve


Revert "Temporary change for nutanix image-builder test"

Reverts aws/eks-anywhere-build-tooling#3216

/lgtm
/approve


Fix/test stop grad beh values

What?

Stop gradient on beh values.

Why?

To improve ppo implementation.

How?

Extra

Closing as new results are inconclusive as to whether this has an impact on performance.


[help want] change to custom pt model in android demo app, it's failed.

Hi I get a pt from my custom model's ckpt. so I change pyvideo pt model to my pt model in android demo app.
the app building is success, but the inference is failed.
console log is here,
RuntimeError: input image (T: 4 H: 5 W: 5) smaller than kernel size (kT: 8 kH: 7 kW: 7)
when I get pt model, I don't use build_model.py. my build_model code is this.
import torch
from pytorchvideo.accelerator.deployment.mobile_cpu.utils.model_conversion import (
convert_to_deployable_form,
)
from pytorchvideo.models.accelerator.mobile_cpu.efficient_x3d import EfficientX3d
from torch.hub import load_state_dict_from_url
from torch.utils.mobile_optimizer import (
optimize_for_mobile,
)
checkpoint_callback = ModelCheckpoint(save_top_k=-1,
monitor='val_loss',dirpath='~/pytorchvideo/weights/',filename='sample-{epoch:02d}-{val_loss:.2f}')
trainer = pytorch_lightning.Trainer.from_argparse_args(args)
trainer = Trainer(callbacks=[checkpoint_callback])
classification_module = VideoClassificationLightningModule(args)
data_module = KineticsDataModule(args)
trainer.fit(classification_module, data_module)
checkpoint_callback.best_model_path

input_blob_size = (4, 3, 8, 256 , 256) #(batch, channels, sampling time, cropped image's height, cropped image's Width)
input_tensor = torch.randn(input_blob_size)
trace_model = torch.jit.trace(classification_module, input_tensor, strict=False)
optimized_traced__model = optimize_for_mobile(trace_model)
optimized_traced__model.save("./video_custom_classification.pt")

when I saving pt from ckpt, I didn't get error msg(RuntimeError: input image (T: 4 H: 5 W: 5) smaller than kernel size (kT: 8 kH: 7 kW: 7)). so I think all right.
where is wrong?

In the demo app, the input is (1, 3, 4, 160, 160), which is different from your model training/saving. You may need to change the input blob size in demo app for your model to work, and I believe you can find more details/ask question there

@liyilui hi, I knowthatso I change a relative part of code (1,3,4,160,160) to (4,3,8,256,256).
But the app is crushed with that error msg. Is it still a problem of my pt model? And I don't confirmed that my build_model.py is right evenif they dont use ckptfile. They are Just callback in code.

According to the error message, I don’t think the change of input blob size is successful (as it is showing T=4 not 8). I would suggest ask in demo app github repo for best way to change or validate changing input blob size.

Closing the issue due to inactivity. Please feel free to re-open the issue if the problem is not resolved.


Use C# wrapper objects for PowerShell cmdlet output

Related to: #2796
Currently, the PowerShell module outputs a COM object which, based on feedback from the community, is not easily usable and also causes issues with certain behaviors such as getting properties and sorting by specific fields (id, version, etc.). This change fixes those issues by using a better formatted C# wrapper object around the data from the COM object and outputs that to PowerShell.
Tests:
I added basic Pester tests but did not include them in the pipeline as the module looks for the wingetdev package to be installed on the build machine (specifically executing WindowsPackageManagerServer.exe) before creating any COM instances. I will work on that in a separate PR but I have included the basic tests here which can be run locally while having the localhost webserver running. Since we already have basic E2E test coverage for the module, it is not absolutely necessary right now.
Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run


how to use the results.tsv

Thanks for your impressive work. I run the command of prediction and got the 'results.tsv'.
This file contian 3 colunms, the last col should be the distance between ligand and protein embedding.
lig-5092	Protein1	0.0
C43734921	Protein1	0.0
C51120531	Protein1	0.0
C63349888	Protein1	0.004318551
C50397193	Protein1	0.013059744
What is the threshold of this distance? e.g.  a value less than that is considered a positive sample, and vice versa.

ConPLex scores are generally well-calibrated, such that the values between 0-1 roughly correlate to model confidence/likelihood of an interaction. In the default case, where the model is in classification mode and the latent distance is the cosine similarity, you can think of values closer to one as more likely to interact. I'd recommend against a strict cutoff, and say that ConPLex is best used for large-scale screens and relative probability/ranking interactions.


python3Packages.pytradfri: 8.0.0 -> 8.0.1

Motivation for this change
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

3 packages built:

home-assistant-component-tests.tradfri
python310Packages.pytradfri
python39Packages.pytradfri

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

3 packages built:

home-assistant-component-tests.tradfri
python310Packages.pytradfri
python39Packages.pytradfri

@ofborg eval


Add best practice for designing lean operator

Description of the change:
Add documentation for best practices for designing lean operator.
#5382
Motivation for the change:
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

true true.

@rocrisp you will need a signature to pass CI. more info
Content is great thank you!
/lgtm

I signed the commit and force pushed. Hope I didn't break anything.

@rocrisp it looks like the DCO check is still failing. Looking at the DCO check steps you should be able to do:

git rebase HEAD~2 --signoff
git push --force-with-lease origin <branch-name>

I hope this helps!

@everettraven  Worked this time. Thank you.


Update documentation for 1.0.

Basic overhaul to show usages of static Sentry API.
TODO: Drop breadcrumb/context mentions from logging integrations if we don't figure out a solid solution.

TODO: Drop breadcrumb/context mentions from logging integrations if we don't figure out a solid solution in #390


Delete all documents from the collection.

I'm trying to figure out, how to delete all elements from the collection or drop the collection? (Laravel 4.2)
Could you help me with this.
Thank you!

I found the answer Stackoverflow


incorrect link on Deploy CockroachDB on Microsoft Azure (Insecure) page

The Secure button on the new Deploy CockroachDB on Microsoft Azure (Insecure) page points to the secure GCE instruction page instead of the secure Azure instruction page.

Thanks for catching that!


DateTime#to_time and Time#to_time

Part of #473
Specs for Features #12271 and #12189.
DateTime#to_time creates a Time object with the same time at the same timezone as the DateTime.  Time#to_time just returns itself.

Ah, my misunderstanding. I’ll wrap the time zone test in a version guard and keep the time object test outside the guard.

On Oct 11, 2017, at 8:08 AM, Benoit Daloze<EMAIL_ADDRESS>wrote:
@eregon commented on this pull request.
In library/datetime/to_time_spec.rb:

@@ -1,6 +1,25 @@
require File.expand_path('../../../spec_helper', FILE)
require 'date'

-describe "DateTime#to_time" do

it "needs to be reviewed for spec completeness"
+ruby_version_is "2.4" do

describe "DateTime#to_time" do
it "yields a new Time object" do

 DateTime.now.to_time.should be_kind_of(Time)

end
Yes, that's why I would only move this example out of the 2.4 block.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[FEATURE] Update to 1.18

Describe the solution you'd like
As the title says, begin work on updating to 1.18.
This will be the next thing that's worked on.

Apparently serialization and client syncing has changed quite a bit for block entities according to this issue:
https://bit.ly/3d1GLSy
There's a few things I'll need to do:

Remove anything related to BlockEntityClientSerializable.class.
Create a .sync helper method on GraveBlockEntity.
Implement client syncing using the new methods found on BlockEntity.class (toUpdatePacket and toInitialChunkDataNbt.
Revise writeNbt and readNbt a little.

According to the issue, I can leverage BlockEntityUpdateS2CPacket.create alongside toUpdatePacket to easily generate an update packet for the client.

Commit e123d49 resolves this.

Ported from 1.17.1 to 1.18 (Caves & Cliffs: Part II)
In the process I updated the mod to use Java 17 & the most recent fabric loader (1.12.8).
Make sure you update as well! :)

Closing.


Library Does not work with User Account Credentials

Thanks for stopping by to let us know something could be better!
PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.

Is this a client library issue or a product issue?
No, this is specific to generating authenticated tokens with user account credentials.

Did someone already solve this?

Search the issues already opened: https://github.com/googleapis/google-auth-library-nodejs/issues
Search the issues on our "catch-all" repository: https://github.com/googleapis/google-cloud-node
Search or ask on StackOverflow (engineers monitor these tags): http://stackoverflow.com/questions/tagged/google-cloud-platform+node.js
Another person seems to be experiencing a very similar issue, https://stackoverflow.com/questions/61754530/use-user-account-credential-for-reaching-private-cloud-run-cloud-functions

Do you have a support contract?
Please create an issue in the support console to ensure a timely response.

If the support paths suggested above still do not result in a resolution, please provide the following details.
Environment details

OS: macOS Big Sur 11.1
Node.js version: v8.15.1
npm version: 6.4.1
google-auth-library version: 6.0.6

Steps to reproduce

$ gcloud auth login --project
$ gcloud auth application-default login
Attempt to execute getIdTokenClient()

Example Code:
import { GoogleAuth } from "google-auth-library";
const auth = new GoogleAuth();
const aClient = await auth.getIdTokenClient(<iapClientAudience>);

This results in an error:
(node:25566) UnhandledPromiseRejectionWarning: Error: Cannot fetch ID token in this environment, use GCE or set the GOOGLE_APPLICATION_CREDENTIALS environment variable to a service account credentials JSON file.
Setting the GOOGLE_APPLICATION_CREDENTIALS to point to the generated user account credentials does not resolve this.
Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

Hi @jbgrunewald, I think what's going on is that you're authenticating with two different accounts between the first and second step, and they are probably trying to authenticate different accounts from different environments. Read more about the commands here and here. Try just using the second command before running the script, and see if that works for you.

Hi @jbgrunewald, I think what's going on is that you're authenticating with two different accounts between the first and second step, and they are probably trying to authenticate different accounts from different environments. Read more about the commands here and here. Try just using the second command before running the script, and see if that works for you.

@sofisl I agree that I was making an error by authenticating twice with two different accounts. I tried each individually and still received the same error.
I also used the --update-adc  with the option to explicitly overwrite the ADC when running the auth command.
gcloud auth login --update-adc

Fundamentally, this error doesn't tell me there's a permissions issue. This is telling me that it can't find the format it's expecting for the credentials (i.e. service account credentials). Without looking at the code, I'm only guessing, but it appears that it doesn't try to use the Application Default Credentials at all, or the format is not acceptable for some reason.

@sofisl I agree that I was making an error by authenticating twice with two different accounts. I tried each individually and still received the same error.
I also used the --update-adc  with the option to explicitly overwrite the ADC when running the auth command.
gcloud auth login --update-adc

Fundamentally, this error doesn't tell me there's a permissions issue. This is telling me that it can't find the format it's expecting for the credentials (i.e. service account credentials). Without looking at the code, I'm only guessing, but it appears that it doesn't try to use the Application Default Credentials at all, or the format is not acceptable for some reason.

@jbgrunewald the reason we throw this error is that as of a year ago, ID tokens cannot be fetched using gcloud credentials.
They CAN be fetched using client credentials, but the use-case there is very fringe: you would be asking an end-user to grant you permission to get ID tokens on their behalf.

@jbgrunewald the reason we throw this error is that as of a year ago, ID tokens cannot be fetched using gcloud credentials.
They CAN be fetched using client credentials, but the use-case there is very fringe: you would be asking an end-user to grant you permission to get ID tokens on their behalf.

Yeah that’s what I’ve gathered. My take away so far is that we are using IAP for service to service communication sometimes, but that’s not really it’s strong suite.
James Grunewald

On Jan 21, 2021, at 2:07 PM, Brent Shaffer<EMAIL_ADDRESS>wrote:

@jbgrunewald the reason we throw this error is that as of a year ago, ID tokens cannot be fetched using gcloud credentials.
They CAN be fetched using client credentials, but the use-case there is very fringe: you would be asking an end-user to grant you permission to get ID tokens on their behalf.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yeah that’s what I’ve gathered. My take away so far is that we are using IAP for service to service communication sometimes, but that’s not really it’s strong suite.
James Grunewald

On Jan 21, 2021, at 2:07 PM, Brent Shaffer<EMAIL_ADDRESS>wrote:

@jbgrunewald the reason we throw this error is that as of a year ago, ID tokens cannot be fetched using gcloud credentials.
They CAN be fetched using client credentials, but the use-case there is very fringe: you would be asking an end-user to grant you permission to get ID tokens on their behalf.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

In addition, other language libraries (such as java) make this possible without requiring service accounts.
const client = await auth.getIdTokenClient(https://cloud-run-url.app);
const token = await client.idTokenProvider.fetchIdToken(https://cloud-run-url.app);

The above lines are what we use to fetch an OIDC to call a gRPC cloud run service. The above only works when the ADC is for a service account, but ideally it would work for an ADC for the user account (as it does for java / go).

It shouldn't be this complicated to just execute a function securely!!!

The following worked on our end - tested and validated the token with Axios too.
const {GoogleAuth} = require('google-auth-library');
const targetAudience = 'https://my-cloud-run.nn.a.run.app/
const auth = new GoogleAuth();
const client = await auth.getIdTokenClient(targetAudience)
const accessToken = await client.idTokenProvider.fetchIdToken(targetAudience);

@ckrebeca how did you end up authenticating once you had the idToken (accessToken in your case)? I am adding it to a header like so
const response = await fetch(url.toString(), {
  method: "POST",
  headers: {
     Accept: "application/json",
     Authorization: `Bearer ${idToken}`
   },
   body: formData
})
      ```
But I continue to get a 403. It was working when i used the standard `idTokenClient.request()` method but this was giving issues when I used FormData

From the docs:

You can use service account impersonation to set up a local Application Default Credentials (ADC) file. Client libraries that support impersonation can use those credentials automatically.

I confirmed that this works on google-auth-library v8.9.0 or higher. But earlier versions don't seem to support impersonation; they throw this error:
The incoming JSON object does not contain a client_email field


Question: Controlled swiper

Controlling the index of Swiper component
Can the Swiper component be controlled externally, so that an 'active id' could be set?
This is so an external navigation could be used, that could allow the swiper to 'jump' to a specific index.
I would of expected something like this maybe...

const [activeId, setActiveId] = useState(0)

<Swiper
  activeId={activeId}
  onActiveIndexChange={(slider) => setActiveId(slider.activeIndex)}
...
>

Nevermind, answered my own question.
For anyone else interested
function App() {
  // store swiper instance
  const [swiper, setSwiper] = useState(undefined);

  return (
    <>
      {/* Controls outside of Swiper that use swiper instance */}
      <button onClick={() => swiper.slideTo(0)}>Go to 1st slide</button>
      <button onClick={() => swiper.slideTo(1)}>Go to 2nd slide</button>
      {/* Swiper component */}
      <Swiper onSwiper={setSwiper} slidesPerView={1}>
        <SwiperSlide>first slide</SwiperSlide>
        <SwiperSlide>2nd slide</SwiperSlide>
      </Swiper>
    </>
  );
}

Excellent thanks

this

Nevermind, answered my own question.
For anyone else interested, here's how to do it.
function App() {
  // store swiper instance
  const [swiper, setSwiper] = useState(undefined);

  return (
    <>
      {/* Controls outside of Swiper that use swiper instance */}
      <button onClick={() => swiper.slideTo(0)}>Go to 1st slide</button>
      <button onClick={() => swiper.slideTo(1)}>Go to 2nd slide</button>
      {/* Swiper component */}
      <Swiper onSwiper={setSwiper} slidesPerView={1}>
        <SwiperSlide>first slide</SwiperSlide>
        <SwiperSlide>2nd slide</SwiperSlide>
      </Swiper>
    </>
  );
}

how do i get the most recent active/real index


Stateless function components cannot have refs.

Hi,
I have a chatbox with custom-scrollbars, when i change the channel from A to B there is no problem, but when i change back to channel A, i got this error.
Uncaught Invariant Violation: Stateless function components cannot have refs.
any ideas?

You're obviously using Stateless function components anywhere in your code. However, this shouldn't affect the Scrollbars component directly. Are you sure this is a problem with Scrollbars? Could you provide some code?

Yep, i use stateless function components but not anywhere related to scrollbars when i remove scrollbar component and try the same thing i don't  get any errors.
import React, { Component, PropTypes } from 'react';
import fireproof from '../utils/fireproof';

import MessageBox from './MessageBox';
import Message from './Message';
import Card from 'material-ui/lib/card/card';
import CardTitle from 'material-ui/lib/card/card-title';
import List from 'material-ui/lib/lists/list';
import CircularProgress from 'material-ui/lib/circular-progress';
import { Scrollbars } from 'react-custom-scrollbars';

class MessageList extends Component {

  componentWillReceiveProps(nextProps) {
    if (this.props.channelKey) {
      fireproof.child('channelMessages').child(this.props.channelKey)
      .off('value', ::this.onSnapshot);
    }
    if (nextProps.channelKey && nextProps.isLoading) {
      fireproof.child('channelMessages').child(nextProps.channelKey)
      .limitToLast(50).on('value', ::this.onSnapshot);
    }
  }
  componentDidUpdate() {
    this.scrollBottom();
  }
  componentWillUnmount() {
    if (this.props.channelKey) {
      fireproof.child('channelMessages').child(this.props.channelKey)
      .off('value', ::this.onSnapshot);
    }
  }

  onSnapshot(snapshot) {
    this.props.onMessagesReceived(snapshot.val());
  }
  scrollBottom() {
    this.refs.chat.scrollToBottom();
  }
  render() {
    const messageKeys = Object.keys(this.props.messages);
    let children = null;
    let title = null;

    if (this.props.channelKey && this.props.isLoading) {
      children = (
        <CircularProgress mode="indeterminate" style={{
          paddingTop: 20,
          paddingBottom: 20,
          margin: '0 auto',
          display: 'block',
          width: '60px'
        }} />
      );
    } else if (messageKeys.length) {
      const messages = this.props.messages;
      children = messageKeys.map(key => {
        const message = messages[key];
        return (
          <Message {...message} key={key} />
        );
      });
    } else {
      title = (
        <CardTitle subtitle="No messages." />
      );
    }
    return (
      <Card style={{
        flexGrow: 1,
        marginLeft: 30,
        maxWidth: 600
      }}>
        {title}
        <List>
          <Scrollbars ref="chat" style={{ width: 600, height: 420 }}>
            {children}
          </Scrollbars>
        </List>
        <MessageBox user={this.props.user} channelKey={this.props.channelKey} />
      </Card>
    );
  }
}
MessageList.propTypes = {
  user: PropTypes.object,
  isLoading: PropTypes.bool.isRequired,
  messages: PropTypes.object.isRequired,
  channelKey: PropTypes.string,
  onMessagesReceived: PropTypes.func.isRequired
};
export default MessageList;
</code>

Hey i am so sorry. I checked again  with using div instead of scrollbar and got the same error so it is from my codes.

@tarkanlar Did you find what was causing this error ? I have the same error and also in places where I use react-custom-scrollbars (although I don't know if it is related).

Hey @HenriJ  I found and fixed but really don't remember. But to see if it is because of scrollbars are not you can try the code by trying the same thing with divs. And if you still getting the error means the scrollbars aren't the reason of the bug.
```js

            {children}
          ```


Polygon parsing can follow non-contiguous corners

The corner characters "." (period) and "'" (apostrophe) specify rounded corners. The wall following routine will blindly follow any corner to another corner, not taking into consideration standard ASCII diagramming conventions. For example, period is used to specify a top corner; apostrophe is used to specify a bottom corner. When moving vertically, the parsing code should not try to traverse from one period to another (or similarly, from one apostrophe to another).
This should fix the logo's rendering bug, and be more consistent with what people will find in most ASCII diagrams. This is kind of a big deal since the intention is to not support slashes as rounded corners (in lieu of supporting them as diagonals).

Bitbucket: https://bitbucket.org/dhobsd/asciitosvg/issue/5
Originally reported by: Devon O'Dell
Originally created at: 2012-03-13T22:40:23.569

Fixed in [[https://bitbucket.org/dhobsd/asciitosvg/changeset/d3d67e8d71fd|d3d67e8d71fd]].

Original comment by: Devon O'Dell

Issue #2 was marked as a duplicate of this issue.

Original comment by: Devon O'Dell


Use ngx_small_light v0.7.3

WHY
ngx_small_light v0.7.3, which includes the fix of Animation GIF (https://github.com/cubicdaiya/ngx_small_light/pull/43) is released.
WHAT
Use ngx_small_light v0.7.3

🎉 ㊗ 🎉


Enable chacha and poly1305 plugins for openssl

The current openssl package does not support chacha and poly1305 plugins. We need these plugins for the Unity Cloud product as this is required for VPP support in our product.

The ChaCha and Poly1305 algorithms are not on our list of approved cyphers.  Will not fix.


Colour output on different terminals

Weird one here... I'm slightly abusing Owl to render an interface for a MUD that is displayed over a LiveView, and I'm hitting a funny issue when deploying.
The ansi colour codes work fine on dev, when I'm usind a LiveScreen and updating a block, I observe its io_request message and the colour codes are present.
But when I do this in prod (Debian / Fly.io), the same messages contain no colour codes. I'm observing the raw data in logs:
dev
ello world

prod
Hello world

(No colour codes)
Any pointers for what might be happening? I've been trying to debug this morning and my best guess is that Owl is doing some kind of terminal support check and disabling colours...

Owl doesn't disable colours, Elixir does this by itself. Owl uses IO.ANSI.format/2:
https://github.com/fuelen/owl/blob/5b05678c094ea84fa5aacd7e66e4a85cde5e9a8a/lib/owl/data.ex#L266
which uses IO.ANSI.enabled?/0 by default. Looks like IO.ANSI.enabled/0 returns false in prod.
You can enable it manually https://github.com/fuelen/owl/blob/5b05678c094ea84fa5aacd7e66e4a85cde5e9a8a/test/test_helper.exs#L1
but this keep in mind that this change is global

I think we can add an optional second parameter to Owl.Data.to_ansidata/1 so it behaves similarly to IO.ANSI.format/2 and you'd be able to force ANSI formatting even though it is disabled globally. Will it solve your issue?

Thanks for the pointer, that's exactly the cause :)
I have no problem enabling ansi colours for this app, so that works for now. Happy to close this issue if you are.


Scrollbar in NestedScrollView is not visible

Expected behavior: When using the following code example, the scrollbar should be visible.
Actual behavior: The scrollbar is actually not visible.

Minimal code example
void main() {
  runApp(MyApp());
}

class MyApp extends HookWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    final scrollCtrl = useScrollController();

    return MaterialApp(
      scrollBehavior:
          ScrollConfiguration.of(context).copyWith(scrollbars: false),
      home: Material(
        child: Scrollbar(
          controller: scrollCtrl,
          child: NestedScrollView(
            controller: scrollCtrl,
            headerSliverBuilder: (context, innerBoxIsScrolled) => [],
            body: CustomScrollView(
              slivers: [
                SliverList(
                  delegate: SliverChildBuilderDelegate(
                    (_, __) => ListTile(
                      title: Text('Shiat'),
                    ),
                  ),
                )
              ],
            ),
          ),
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel beta, 2.6.0-5.2.pre, on macOS 11.6 20G165 darwin-x64, locale en-GB)
    • Flutter version 2.6.0-5.2.pre at /Users/kristian/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 400608f101 (3 weeks ago), 2021-09-15 15:50:26 -0700
    • Engine revision 1d521d89d8
    • Dart version 2.15.0 (build 2.15.0-82.2.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/kristian/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.61.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.71

• No issues found!

Hey @KristianBalaj, you should use the ScrollBehavior to build the scrollbar in this case instead adding one manually to the CustomScrollView. You are right, assigning a ScrollController to the inner scroll view of the NestedScrollView breaks the nesting functionality.
The automatically applied scrollbars on desktop are done in the same way via the ScrollBehavior, we actually have a guide available here that has a couple of different examples of how you can use this. You should be able to have the scrollbar work as you like by following that guide.


Add google.api.core.path_template

Towards #3842

@lukesneeringer @dhermes this gave me more trouble than I anticipated, but the tests give me confidence that this works as expected.

Still simpler than a lex-yacc implementation. ;-)
Sanity check: Correct?

Much simpler
On Mon, Aug 21, 2017, 7:48 PM Luke Sneeringer<EMAIL_ADDRESS>wrote:

Still simpler than a lex-yacc implementation. ;-)
Sanity check: Correct?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/pull/3851#issuecomment-323904362,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc6HWLCb78HkRVkBspwrTMa-n4IIbks5sakF9gaJpZM4O-Cw2
.


Implement Rust traits for Screeps types

Things like Index, IndexMut, etc. should be implemented for better integration with Rust code

In general, agreed!
Were there any specific Screeps types you were thinking Index/IndexMut would be appropriate for?

I was specifically thinking the dict types (creeps, spawns, memory) should have Index/IndexMut, and probably also be iterable?

Alright- that makes sense.
The current design of the Game.* dicts could be redone so they support implementing traits. Right now they're modules, but we could make them structs with no fields. It could then implement IntoIterator, which would work well, but I'm not so sure about Index.
The main problem I see with implementing Index for them is that Index::index needs to return some sort of reference borrowed from the structure. We don't currently store Creep / Structure / etc. instances in screeps-game-api, and we would need to in order to implement Index. I'm not sure about how to solve this unless we just went ahead and allowed fetching all values into a HashMap<String, Creep> which could then be indexed.

For memory, the main question I have is what type the index should result in. I went with methods for each specific type in order to have checked/typed access.
Do you think introducing a MemoryValue enum with different possible types would be good, or would Index return stdweb::Value?
Even here though, there's still the problem with references. Rust's Index trait seems designed for data structures which really hold their data, and don't fetch it from an external place.


moveo_moveit/ArmJointState.h: No such file or directory

Basically for last month or so i try to work out this problem, I've tried everything that was written about this problem. I tried reinstalling the library many times. I even reinstalled my virtual machine few times.  I have the exact same problem that other folks do but i can't repair it, we are trying for weeks with my friend and it seems like it just doesn't work. Pls give us some advice, we are very eager to complete this robot and we just don't have a clue what to do anymore.
We use Ubuntu 16.04.07 and ros kinetic. We have everything working but rosserial library. It's the one thing that doesn't work.
Thank you in advice.

I have the same problem and I don't know how to fix it?

Try to do what it says in the Pull Request https://github.com/jesseweisberg/moveo_ros/pull/26 and after run CMake.
If the compile is ok then do what it says for the library.

For me following approach resolved the issue (using ROS kinetic and Ubuntu 20.04.2) but it should  also work with ROS kinetic:
1. In the package.xml of moveo_moveit you need to add:
1.1 <build_depend>message_generation</build_depend>
1.2. <exec_depend>message_runtime</exec_depend>
**1.3.**change <run_depend> to <exec_depend>  for all tags (don't forget to change   </run_depend> to  </exec_depend> as well ;) )
**1.4.**change <package>to <package format="2">
In the CMakeList.txt of moveo_moveit you need to change:
2.1. add message_generation to the find_package function():
find_package(catkin REQUIRED
  COMPONENTS
    roscpp
    rospy
    std_msgs
    moveit_core
    moveit_ros_planning
    moveit_ros_planning_interface
    pluginlib
    geometric_shapes
    moveit_visual_tools
    message_generation
)

2.2 add message_runtime to the catkin_package() function:
catkin_package(
  CATKIN_DEPENDS
    moveit_core
    moveit_ros_planning_interface
    interactive_markers
    message_runtime
  DEPENDS
    EIGEN3
)

Following tutorial helped me to resolve the issue: http://wiki.ros.org/ROS/Tutorials/CreatingMsgAndSrv#Creating_a_msg
Unfortunately at the moment i have issues to attache the files (package.xml and CMakeList.txt). I will try to add them later.


Messaging Node in Workflow missing

I have installed and enabled the Messaging module and Email Messaging module.  I have also set the email settings and testing them with a Contact Us form.
I am now trying to create a workflow that messages me when a comment is created, but there is no "Messaging" action node available in the Workflow toolbox.
What am I missing?

Please ask such questions on the forum, or on Orchard Pros. This is the bug database for the documentation.
You need to enable "email workflow activities". Then'you'll have a "send email" activity.


Incorrect layout when using both left & right margin?

https://github.com/delight-dev/Delight/blob/698e660073ff492d0091571fc9a311f62e37a73c/UnityProject/Delight/Assets/Delight/Content/Views/UI/UIView.cs#L408
Should RectTransform.offsetMax actually read like this instead?
            RectTransform.offsetMax = new Vector2(                 offsetMaxX + margin.Left.Pixels - margin.Right.Pixels + offset.Left.Pixels - offset.Right.Pixels + offsetFromParent.Left.Pixels - offsetFromParent.Right.Pixels,                 offsetMaxY + margin.Bottom.Pixels - margin.Top.Pixels - offset.Top.Pixels + offset.Bottom.Pixels - offsetFromParent.Top.Pixels + offsetFromParent.Bottom.Pixels);
As in, adding margin.Left.Pixels and margin.Bottom.Pixels, otherwise setting a left margin on a rect with a fixed size within a group shrinks it instead of moving to the right.
Example:
  <Group Id="MainMenu" Alignment="Left" Orientation="Horizontal" Width="300" Height="100%">     <Region Id="Meme" BackgroundColor="Red" Width="10" Height="100%" Alignment="Left" Margin="105,0,0,0"/>   </Group>

Hmm it breaks the case where Width=100%. For that to work, I'm guessing the percentage should be applied after the margin is computed.

Which ... I guess you can't do because UpdateRectTransform doesn't have the size of the parent as an argument


Change "file" filetype and size

I am trying to put this in my form:
   file: {
      type: String,
      optional: true,
      autoform: {
         type: 'file'

      }
}

But people can upload whatever they want which is not good. Is there a way to restrict what people can upload (For instant only pdf and png) and maybe the size (5mb) as well? Couldn't find anything

Please use ostrio:meteor-files wit ostrio:autoform-files to achieve this goal


parse(location.search) always start with '?'

xxx.com?id=1
queryString.parse(location.search)
"query-string": "^6.9.0"
now return: {?id: 1};
expect：{id: 1};

The same


Added support of empty string field_name.

Description:
This PR worked on issue #140.
The binary writer doesn't include the case that field_name is an empty string.

TEST:
Test code:
from amazon.ion.simpleion import dumps

obj = {'': ''}
print(dumps(obj, binary=True))

Output:
b'\xe0\x01\x00\xea\xe7\x81\x83\xde\x83\x87\xb1\x80\xde\x82\x8a\x80'

which is: $ion    ion_symbol_table::{  symbols:[""]  }     {"" :  ""}

Unit test:

Two questions:

There are test files in ion-tests that exercise empty field names. Why were they not failing? Are they on the skip list?
Were our pypy Travis builds failing before previously?

We had specific tests that cover empty filed name. The reason causing this issue is because of the difference ion_type(or if they are ion nature).

When we are running the unit test: We load unicode (from vector files) to a IonPyDict object which is an ion nature object. Then when we dump the object, we go through this branch to create an ionEvent by to_event method that will convert field name to a symbol token.

When we dump a python object {'': ''} directly: Since it's not an ion nature, it will go through this branch calling IonEvent() constructor to generate an ionEvent. IonEvent supports both unicode or symbol token as filed name so "" will be treated as unicode first and later on we will call the derive_field_name method here to convert the filed name from unicode to symbol token.

The issue happenes when we are going to call the derive_field_name method here. we don't include the case that field name can be an empty string ""  which throws an error later.

I'm not very sure about the pypy builds failing, I noticed that some of our previous PRs faced the same issue.

Added a new unit test covering this case. Before this PR, the new test fails due to the empty string field issue, and passed after this PR change.
before:
now:

Sure, I'll see if this commit fails again to ensure it's not a random issue.


Move test mock injection out of public interfaces

Summary: Removes method from public interface that allows for mock injection. New method in implementation header to achieve this.
Type: Cleanup
Test Plan: Unittests.
Jira: NO-JIRA

https://jenkins.code.rdkcentral.com/job/github-triggered/44430/ started

https://jenkins.code.rdkcentral.com/job/build-component/83088/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/10097/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/44432/ started

https://jenkins.code.rdkcentral.com/job/build-component/83090/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/15420/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/10098/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/15421/ started.

build-rpi-hybrid-wpe-dunfell Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/fda7eba02ff65d98b31464c99ac69f30

build-rpi-mc-wpe-dunfell Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/b05c44d6b37391bf7ddc344608f95de0

build-rpi-refapp2 Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/132ea21090409350144f258055d74f12

build-rpi4-mc Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/1c45d09f5a7ac73363a495be71565797

build-rpi4-hybrid Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/bce65cccbfd37810993e5b4bd2e8f3a3

https://jenkins.code.rdkcentral.com/job/github-triggered/44434/ started

https://jenkins.code.rdkcentral.com/job/build-component/83093/ started.

https://jenkins.code.rdkcentral.com/job/build-component/83096/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/10099/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/15424/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/10102/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/15427/ started.

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/32028/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/32034/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/32041/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/32042/


fix workspace document searching

Fix bug where any embedding searches would always return nothing.

@shatfield4 merging this but lets double check we dont do this elsewhere. This is a holdover from the SQLite use days pre-prisma and pre-postgres. I know ive written that line more than once haha


Replace outdated use of c_types.h with stdint

Replace outdated use of c_types.h with stdint
uint32 -> uint32_t

Thanks Rudi! I've added this PR and #5 into our internal change queue.
For future reference, if you make changes on the same branch then you can add commits to an existing PR. You don't need to create a new PR each time (but github will do this if you create a new branch each time.)

ah - thanks - have not seen "Add more commits by pushing to the" - ok now i know.
you guys are hard working team! where we can - we support you :+1:


Ambiguous interpretation of 'free'

The access attributefree has ambiguous behavior

"" --> blocked
false --> open to anonymous users
true --> open to anonymous users
False  --> open to anonymous users

NOT TESTED:

'FALSE'
'True'
'TRUE"

Note: this also impact the entitlement service of videodock

@marcovandeveen we can accept true, True and other case variation as TruE. Other values will be considered to be falsy. However there may also be a problem with requiresSubscription param which we also use. Not only false value is accepted. In case we set it to False or FaLSE, the item would be considered to be free.
It seems that we need a general approach for these cases. I suggest:

We accept false / true string values and their upper / lower case variations.
Empty string value is a falsy one.
Boolean values (not sure if it may happen) have their original values.

@dbudzins fyi

I would also accept 0 and 1 as strings to be false and true respectively.  Actually, we only need to talk about what is false and then everything else we'll assume to be true.  So false is:

blank value / empty string
any casing of the word false
0

@dbudzins then for free param we can set any other value which would be true. I don't know whether it is what we want.

Oh I see... some properties may be better defaulted to false if not explicitly true... that's fine.  So we do:

requiresSubscription - true unless:

blank value / empty string
any casing of the word false
0
any casing of the word No

free - false unless:

any casing of the word true
1
any casing of the word Yes

Works?

@dbudzins makes sense. I also wonder why we decided to use requiresSubscription param as an indication that the item is free (so basically item is not free by default, and it could be more logical to call it isFree or useNoSubscription). For me it seems to be a bit confusing as we also support free which requires the opposite value for the item to be free. Maybe we can just use free for both cases? In this case we won't have to integrate different logic for two params (default true for requiresSubscription and default false for free). @marcovandeveen maybe you have better understanding of the possible reason.

The reason is that Applicaster uses attribute 'free'.
Consistency between applicaster-webapp is more important than having a good pure model at this moment. Asking Applicaster to change is too much of burdon now.  Especially as we need to productize our access model in future iterations further. Also, the attribute 'productIds' is not a clean solution.

@marcovandeveen I am not against using free param :) I suggest to use it for both us (and get rid of requiresSubscription ) and Applicaster

I suggest to deprecate requiresSubscription but don't remove it.
Fanschoice, TBN and others are using it. Also our documentation still states you need to use 'requiresSubscription'. New docs are in progress


Does AWS API-Gateway support the swagger documentation server?

I would like to have a documentation and mock-up server of my api, like the swagger document service provides for a server (in contract to serverless) setup environment, please click this http://petstore.swagger.io for what I meant here.
My customer(s) can use the above link to experience my API in an intuitive way. Does AWS API-Gateway provide such functionality?

I know APIGW gives you client SDKs, but I am not sure if it supports generating Swagger UI page. You can do it yourself using https://swagger.io/swagger-ui/ from your Swagger template

are you able to do this? even i am looking for the same
me to


TypeScript support

Hi,
Impressive work ! Love your dependency-free single-file approach.
I'm planning on using it on a TypeScript project so I made a typed schema.ts version with:

added some type conversion (in validDate/validTime) to prevent implicit conversion
fixed some typo (foraign=>foreign, vocal=>vocab)
added two arguments (fetch/URL) to the new Schema() constructor to be platform-agnostic and easyly mockable

If you are interested, I could send a PR along with a new compile step in gh-action to continue publishing the JS version
If you are not interested I will keep it in a fork :)

Hi, yes i am interested!

I've never mixed JS/TS on the same TS project so I'm not sure. I'll try and get back to you.
You can add QuickJS (and nodeJS?) to the list of supported engine if you are okay with passing the 2 additional argument to the Shemas() prototype ;)

I just tested and --allowJs seems to do the job (but got no typehint on the error() return values)
I'll close this as typescript is not a goal and keeping JS won't block usage.
I'll send a PR with the changes I did to make it work on every platform like node/quickjs (which require the move some global object into the class to forward them the platform fetch/URL)


Incorrect mapping of sub-classes when nested as member of another sub-class

Note: This issue was posted in springdoc-openapi first. It was closed, since the issue seams to be located in swagger-core.
Describe the bug

Inherited sub-classes are not mapped correctly if they are nested in a sub-class that by itself is derived from a super class.
I'd expect the mapping to be onOf() which is true for the nested class itself, but not for the member.
See the example to make things clear:

To Reproduce
Steps to reproduce the behavior:

Spring Boot starter parent, version 3.0.2
Spring Doc modules starter-webmvc-ui and starter-common in version 2.0.2.

Here's an example controller:
@RestController
@RequestMapping(path = "/v1/test", produces = MediaType.APPLICATION_JSON_VALUE)
public class TestOpenApiError {

    @GetMapping("/a")
    public A testGetA() {
        return null;
    }

    @GetMapping("/b")
    public B testGetB() {
        return null;
    }

    @GetMapping("/c")
    public C testGetC() {
        return null;
    }

    @Schema(
            description = "Some out class",
            subTypes = { A1.class, A2.class, },
            discriminatorProperty = "@type",
            discriminatorMapping = {
                    @DiscriminatorMapping(schema = A1.class, value = "A1"),
                    @DiscriminatorMapping(schema = A2.class, value = "A2"),
            }
    )
    public class A {}
    public class A1 extends A {}
    public class A2 extends A {
        public List<? extends B> listOfBs;
    }

    @Schema(
            description = "Some nested class",
            subTypes = { B1.class, B2.class, },
            discriminatorProperty = "@type",
            discriminatorMapping = {
                    @DiscriminatorMapping(schema = B1.class, value = "B1"),
                    @DiscriminatorMapping(schema = B2.class, value = "B2"),
            }
    )
    public class B {}
    public class B1 extends B {}
    public class B2 extends B {}

    public class C {
        public List<? extends B> listOfBs;
    }
}

I have three classes A, B, and C. A1 and A2 are derived from A. B1 and B2  are derived from B.
B as list is member of A2 (nested in a class that is derived, too) and B as list is member of C (not nested in an inheritance)
This is the generated outcome fragment for this as yml
B:
      type: object
      description: Some nested class
      discriminator:
        propertyName: '@type'
        mapping:
          B1: '#/components/schemas/B1'
          B2: '#/components/schemas/B2'
    B1:
      type: object
      allOf:
        - $ref: '#/components/schemas/B'
    B2:
      type: object
      allOf:
        - $ref: '#/components/schemas/B'
    C:
      type: object
      properties:
        listOfBs:
          type: array
          items:
            oneOf:
              - $ref: '#/components/schemas/B'
              - $ref: '#/components/schemas/B1'
              - $ref: '#/components/schemas/B2'
    A:
      type: object
      description: Some out class
      discriminator:
        propertyName: '@type'
        mapping:
          A1: '#/components/schemas/A1'
          A2: '#/components/schemas/A2'
    A1:
      type: object
      allOf:
        - $ref: '#/components/schemas/A'
    A2:
      type: object
      allOf:
        - $ref: '#/components/schemas/A'
        - type: object
          properties:
            listOfBs:
              type: array
              items:
                $ref: '#/components/schemas/B'

When B is member of C the generation is as expected. Is maps to oneOf. As expected:
    C:
      type: object
      properties:
        listOfBs:
          type: array
          items:
            oneOf:
              - $ref: '#/components/schemas/B'
              - $ref: '#/components/schemas/B1'
              - $ref: '#/components/schemas/B2'

When B is member of A2 it just refers to an array of type B. No oneOf mapping:
    A2:
      type: object
      allOf:
        - $ref: '#/components/schemas/A'
        - type: object
          properties:
            listOfBs:
              type: array
              items:
                $ref: '#/components/schemas/B'

Expected behavior
I'd expect A2 to me mapped like this:
    A2:
      type: object
      allOf:
        - $ref: '#/components/schemas/A'
        - type: object
          properties:
            listOfBs:
              type: array
              items:
                oneOf:
                  - $ref: '#/components/schemas/B'
                  - $ref: '#/components/schemas/B1'
                  - $ref: '#/components/schemas/B2'

Maybe I'm doing something wrong. Maybe somebody can confirm, that this is a real issue.
Thx for any help.

I came across this issue searching for something similar. In my case B is abstract and I wanted to show oneOf B1 and B2.
I was able to show the correct options this by adding this @ArraySchema
public class A2 extends A {
    @ArraySchema(schema = @Schema(implementation = Objects.class, oneOf = {B1.class, B2.class}))
    public List<? extends B> listOfBs;
  }

I came across this issue searching for something similar. In my case B is abstract and I wanted to show oneOf B1 and B2. I was able to show the correct options this by adding this @ArraySchema
public class A2 extends A {
    @ArraySchema(schema = @Schema(implementation = Objects.class, oneOf = {B1.class, B2.class}))
    public List<? extends B> listOfBs;
  }

That's the best solution, but you have to remember, that fields from parent abstract class won't be included in docs.


Update handling of binary port errors

This PR adds:

missing variants for TryFrom<u16> for ErrorCode
test to ensure that all variants are covered in the future (uses strum as dev-dependency)

bors try

bors r+

bors r+

bors r+

bors r+


"No mixin named mdc-states" compile error

I keep on getting an install error on Windows - first with fsevents (which happens since fsevents doesn't work on Windows and I think is optional for vue-cli), but after that, with mdc-states.
Each time, I get a compile error:
"error in ./src/theme.scss
Module build failed:
Undefined
^
No mixin named mdc-states"
Following that, there's a backtrace.  Just trying to get through that particular rabbit hole, I went through and deleted the first couple locations it mentions to see if I could get the package to compile, but so far, no luck.

I fixed it by manually go into package.json and update material-components-web to the latest verison

thanks - that worked! Changing the line in package.json to
...
"dependencies": {
"material-components-web": "^0.29.0",
...
did the trick.

Was it warning you about peer dependencies when you installed npm dependencies?

@thotze
I guess I can close this one ?
plz re-open if you still have issues


Feature Request : Copy text from current view (llil, mlil and disasm view)

I'd like to see a feature where you can select multilines using Ctrl + Cursor and then copy the lines out into the clipboard :)
Example : In Disassembler view, select multiple lines or press Ctrl-A to select all lines in the current block and then press Ctrl-C to copy the text into the clipboard (for graphical view, llil and mlil pseudo-code)
Currently, this is only possible using python console, which is a bit inconvient if I need to copy out stuff fast.
Example code for llil for current block :
for bb in current_llil.basic_blocks:
  for instruction in bb:
    print instruction

Duplicate of these: #91 #373


Fix usage typo in info.go

Fix usage information (drone info) typo;
should information about the current user should read show information about the current user

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Improve Sourcery-ai and Coderabbitai configuration

Enhance AI assistants to improve the sequence on tests and reviews:

To use first sourcery-ai before coderabbitai does a revision (if applicable)
To avoid execution at the same time: this is for the fix of multiple comments of the same issue and/or unnecessary bug reports.

Documentation:

Soucery-ai config: https://docs.sourcery.ai/Coding-Assistant/Reference/Configuration/sourcery-yaml/
Coderabbitai config: https://docs.coderabbit.ai/configure-coderabbit/

Coderabbitai configuration can be done at their app dashboard.
@coderabbitai I need to create a path instructions for you to follow up on next PRs revisions. Provide an initial instruction that follows code standards that we have been following for all paths.
@sourcery-ai plan this update by first giving an initial prompt for all paths.

@coderabbitai amazing! Yes, I have a set of rules to follow code standards. We made it for full-stack development with javascript/typescript but I summarized it for you. Please refactor the previous given path instructions to add the new points to consider while reviewing:
Masterbots Code Standards
JavaScript/TypeScript:

Use semantic, idiomatic, functional, and declarative code
Avoid default exports, use named exports
Boolean variables: prefix with auxiliary verbs (is, has, does, should)
Use regular function declarations for components
Type everything, avoid any
Let TypeScript infer types when possible
Use interfaces for objects/classes, types for unions/complex types

Components:

Follow single responsibility principle
Break into small, focused components with minimal props
Use composition over inheritance
Declare components with function keyword
Order: component declaration → styled components → TypeScript types

File Structure:

Lowercase with dashes for directories/names (e.g., auth-wizard)
Extensions: .config.ts, .test.ts, .context.tsx, .type.ts, .hook.ts
Group similar components in feature directories
Keep folder structure flat where possible

NextJS 14/15:

Use Server Components by default.
Add use client only when needed for client-side features.
Implement nested layouts via App Router.
Use streaming and suspense for large data sets.
Follow metadata file conventions for SEO.
Fetch data on server when possible.

API Development:

Optimize GraphQL/SQL queries to minimize data transfer.
Implement proper error handling and logging.
Use generated GraphQL/Supabase/DrizzleORM client for type-safe queries.
Follow Hasura/Supabase/DrizzleORM best practices for database operations.

Styling:

Use TailwindCSS utility-first approach
Apply shadcn/ui components for consistent UI.
Use CVA for component variants.
Avoid arbitrary values in Tailwind classes.

Testing & Quality:

Write unit tests for critical functionality. Always suggest the unit test code. Ask to commit.
Use ESLint and Prettier for code formatting.
Keep dependencies updated.
Follow TypeScript strict mode guidelines.


Flaky test: TestHedgedResponseWins

I've packaged this library for Debian, as a dependency of other packages, and have noticed that there is a test that seems to be flaky across multiple builds/autopkgtests for the latest release (0.8.0):
=== RUN   TestHedgedResponseWins
    hedged_test.go:363: Unnexpected actualRoundTrips: 4
--- FAIL: TestHedgedResponseWins (0.60s)

(See https://buildd.debian.org/status/fetch.php?pkg=golang-github-cristalhq-hedgedhttp&arch=all&ver=0.8.0-1&stamp=1692386052&raw=0, https://ci.debian.net/data/autopkgtest/testing/ppc64el/g/golang-github-cristalhq-hedgedhttp/36961327/log.gz, https://ci.debian.net/data/autopkgtest/testing/i386/g/golang-github-cristalhq-hedgedhttp/36953254/log.gz)

Oh, interesting! First sorry for the failed build. Second: I saw this failure once and was sure was this a dream or not. I will take a closer look today-tomorrow.
Thanks for the report!

Fix is available in https://github.com/cristalhq/hedgedhttp/releases/tag/v0.8.1

Awesome, thanks for the quick fix!


add BCC v0.9.0

Thanks for your submission. I've landed #11 which addresses adding bcc 0.9.0 support to this library. Closing this out.


demo.py

i train it with PVAnet,What should i change in demo.py to use the output/voc_2007_trainval/ckpt files? @CharlesShang

i got the problem that is W tensorflow/core/framework/op_kernel.cc:1192] Invalid argument: Assign requires shapes of both tensors to match. lhs shape= [25088,4096] rhs shape= [13824,4096]

i really want your help!thanks @CharlesShang


Layer stack

I don't see the need for Layers in the Engine. this may be used for Editor but i really can't imagine how we would use this inside the engine. Maybe you could explain it to me, please?

Actually, you've got a point there. I will move it to Editor

After a bit of thought it might be used like this:
LayerStack

World (3D Scene)
UI (Health bar, text, etc)
Debug (If we wanted this. Like in unreal, a small in-game console that shows message)
Editor (ImGui)


Searching packages aways fails

Getting really wound up by the latest update.
Appears it just HANGS are various points like when trying to search for a package. Install, Select a option
You have no idea if its accepted your response or just frozen.
Its taken over 30 attempts to load the packages to install 1 package.... Really.
Thinking of going back to PHPEd after this, seriously messed up my day!!
This may be a irrelivent Issue to track, But i wanted to provide my feedback to why i am now leaving Atom.
Its not ideal for someone trying to do alot of work under tight deadlines.

I take that back, it never even installed the package after all that!!! Geezzz

Works just fine for me, Where does it get stuck for you?

@dave7802 atom.io is under heavy load right now (since the 1.0 announcement :stuck_out_tongue_closed_eyes:)  - a fix is in the works though!

Loading packages,
Searching packages for "XXXXXX" Always returns.
Searching packages failed: atom.io is temporarily unavailable, please try again later.
Thanks for the response,

Hi, i know its 2017...just started learning to program and i liked the way atom's UI looks and the support of many features... but i have been experiencing difficulty from day 1 installing THEMES and PLUGINS... displaying errors: Fetching local packages failed.Hide output…spawn C:\WINDOWS\system32\cmd.exe;C:\Program Files (x86)\Java\jdk1.7.0_07\bin;C:\Users<username>\AppData\Local\atom\app-1.18.0\resources\app\apm\bin ENOENT
UPDATE:

THEME:

PACKAGES:

i still face this issue
how did you guys close this issue?


import ncon_solver at contractors level

Currently it's not imported in the __init__.py

Hey,
can I take up this issue?

Of course, PRs are always welcome!

I want to work on this. How to start can you guide. I am new to contribution.
Thank you .

What is contractors level ?

I think this is fixed? I see ncon_solver in the contractors init.py

yes it is! thanks!


Y24-411 - As a long read user (Michelle) I would like to be able to create multiple libraries at once (24) to save time and reduce errors

User story
As a long read user (Michelle) I would like to be able to create multiple libraries at once (24) to save time and reduce errors
Who are the primary contacts for this story
Steve / Michelle
Who is the nominated tester for UAT
Michelle / Long Read
Acceptance criteria
To be considered successful the solution must allow:

[ ] The libraries endpoint should be able to receive a request to create multiple libraries OR a new endpoint for creating multiple libraries
[ ] each library should be validated as a standard library
[ ] the libraries should be returned with the newly created tubes so the barcodes can be printed

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

References
This story has a non-blocking relationship with:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Do we want a BatchedLibrary object, similar to the reception model, that gets created on successful library batch creation that tracks what libraries were created in that event. (not sure if its useful in this case)


Prevent Unmounting?

Hey There.
Is there any easy way to prevent views from unmounting?
I've setup my app to support androids multi window feature which allows running more than 1 app on the screen at a time in separate windows.
But there is a bit of an issue if you resize the app.
The current view gets unmounted then mounted again which results in the views state being reset.
What i'd like to do if possible is prevent that.
As a workaround currently i just feed the views state back into itself.
Any ideas?

Pretty much just decided to rig up a HOC to deal with it.
Seems to do the job.


az ad sp reset-credentials --name parameter not resolving as expected

I tried to use the az ad sp reset-credentials --name command to reset the credentials on an existing service principal.
I first ran az ad sp list to find the service principal details that I was after, it returned the following for the service principal I was interested in:
    "appId": "appid-redacted",
    "displayName": "AzureContainerService",
    "objectId": "objectid-redacted",
    "objectType": "ServicePrincipal",
    "servicePrincipalNames": [
      "appid-redacted"
    ]
  }

I then ran az ad sp reset-credentials --name "appid-redacted" because that was the only value shown in the collection of servicePrincipalNames. This resulted in the following error: can't find an application matching 'appid-redacted'
I also tried running the command again using the value AzureContainerService resulting in the same error.
Note I have replaced the actual guids with appid-redacted.

I have the same issue. Looks like the code tries to look up by identifierUris. Why not just filter by appId like in the rest of the az ad sp commands? ... /applications?api-version=1.6&$filter=appId+eq+'d1d54911-161d-4296-8ffe-b6912fee5a10' seems to work fine

i should fix it soon along with #1596. The background is the reset-credentials pairs with create-for-rbac has a assumption that the name is supposed to be app-id-uri, but this concept is not well accepted, so time to adjust it.


Addition of data header for cases per million population

Is your feature request related to a problem? Please describe.
As we all know India is second-largest populous country in the world. According to worldometers.info, we are 1,380,004,385 as of April, 2020. It would be better to show "total, active and death" cases at per million population base. I believe this will give our users a broad and better perspective on COVID 19 spread in India.
Describe the solution you'd like
In header before data table, a new information should be added with totalCases/1M population, activeCases/1M population, deathCases/1M population and totalCases-India/totalCases-World.
Describe alternatives you've considered
NA
Additional context
NA

I think any statistics in India is skewed by the large population. I feel a better judge of the testing numbers would be to compare it with the number of cases detected. If more cases are detected, that means those cases have contacts and those contacts need to be tested aggressively.
https://github.com/covid19india/covid19india-react/issues/945 talks about this.
And definitely, top header is not the place for this. This could be a graph in deep-dive.

I saw issue #945 and I think that's also a good idea. Please don't be confused that my idea is also the same. I will try to give a clear view of my idea:
The idea is to show the users how good/bad we are tackling corona virus spread in India. I talked about the ratios (no stats or graph needed), like totalCases-India/totalCases-World (this will give the spread ratio in India over world), totalCases/1M (as of now: 8453/1000000 = "0.008453", you see how small this is). The smaller the ratio would be, the  more optimistic we would be, or if it increases we could be more cautious about the spread. We can also show totalCases/totalPopulation instead of per million population, if you want. Adding these ratios in header with other counts will inform our users about the progress (and position in the world) of our battle against corona virus. If we can show the ratios' skewed graph in Deep Dive then it would be better, but adding the ratios in Home header will fulfil the idea for now.

Closing in favor of #1496


[vtk] FindPEGTL fix

Fixes #35223

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

@microsoft-github-policy-service agree

Please adjust the pegtl.patch already there instead of adding another one

Please adjust the pegtl.patch already there instead of adding another one

Should be fixed now.

@martinfalk seems like this PR is ready to merge, hopefully it can go in soon as there is another large PR waiting on this one:  https://github.com/microsoft/vcpkg/pull/37119  🤞

Thanks for the fix and sorry I read it backwards before

No worries, and happy to contribute :)


Max Bid Fixes

Related to #1631.

Looks like there was a mismatch after https://github.com/artsy/emission/pull/175 was merged, the Podfile was still pointing to an old version of React that had been updated in Emission. I bundle exec pod install'd, committed + pushed changes to eigen-artefacts ( https://github.com/artsy/eigen-artefacts/commit/09019c40ea3a7a22421903b564a1b684801b1c0d ) and updated the Podfile/.lock and submodule commit. Fingers crossed.

        2 Warnings

  </th>
 </tr>

:warning:
Needs testing on a Phone if change is non-trivial

:warning:
Detected some Swift building time outliers

Time | Class | Function |
| --- | ----- | ----- |260.5ms | AuctionViewController.swift | @objc dynamic func showRefineTappedAnimated(animated: Bool)
197.4ms | SwiftyJSON.swift | get {}
173.9ms | WebSocket.swift | private func processRawMessage(buffer: UnsafePointer, bufferLen: Int)
77.7ms | Observable.swift | private final func nextTokenHash() -> Int
74.0ms | LiveAuctionsAdminViewController.swift | @objc override func viewDidLoad()
71.6ms | AuctionLotMetadataStackScrollView.swift | required init(viewModel: LiveAuctionLotViewModelType, sideMargin: String)
71.3ms | UIViewController+BlurredStatusView.swift | func ar_presentBlurredOverlayWithTitle(title: String, subtitle: String, buttonState: BlurredStatusOverlayViewCloseButtonState = default)
69.3ms | UIViewController+BlurredStatusView.swift | (closure)
61.4ms | RefinementOptionsViewController+Private.swift | func stackView() -> ORStackView

  Generated by :no_entry_sign: danger


Exception on nodejs 11.9+ EVP_CIPHER_asn1_to_param:cipher parameter error evp_lib.c

I am uncertain if this is compatible with nodejs 11.9 and up currently.
I have posted an error with them first. I am uncertain what is the problem.
https://github.com/nodejs/node/issues/27213

I have just checked 11.14 is also not working as of 2019-04-17

nodejs 12 also not working.

Still errors in 12.10


Replicate _users database to table couchdb_medic_users

#137
Change to<EMAIL_ADDRESS>is https://github.com/medic/couch2pg/pull/45

@njogz You might think you've seen this exact PR because of #136. Truly very similar!
I appreciate your eyes here again.

Are there any materialized views that you think are useful to have across all deployments? For example, we create multiple views and materialized views during initial setup based on which file is being run here. It will be nice to have those views/materialized views created and executed as a part of c2pg rather than requiring us to recreate same objects later.

Thanks for the feedback @yrimal. I personally find the _user data to be quite simple. The table is usually small (<12000 users max for CHT instances). We might consider a reusable view, but I'm not sure it adds that much value. Would you use it and prefer to have it?


refactor: update icons

Codecov Report

Merging #846 (13ee64a) into release/1.0.0 (fb65ac5) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           release/1.0.0     #846   +/-   ##
==============================================
  Coverage          57.14%   57.14%
==============================================
  Files                  1        1
  Lines                 70       70
  Branches              15       15
==============================================
  Hits                  40       40
  Misses                30       30

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 fb65ac5...13ee64a. Read the comment docs.


nit: update sample heading in Sample05_UploadDownloadImage.md

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


Typing in the query bar loses/removes every other keystroke after adding $filter query

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
Add ?filter=
Type more
Get frustrated
https://clipchamp.com/watch/CJ9UsEGtNLJ
Expected behavior
Users should love typing in a filter.
Screenshots
Video: https://clipchamp.com/watch/CJ9UsEGtNLJ
Desktop (please complete the following information):

OS: Windows 11
Browser Edge
Version 110.0.1587.63 (Official build) (64-bit)

Hey @gavinbarron. Apologies for this experience. I have created a PR to fix it.


Release 0.16.5

Checklist

[x] Ensure zstandard is unconstrained
[x] Include rdkit !=2024.03.6 constraint
[x] Move back to main label
[x] Used a personal fork of the feedstock to propose changes
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


RSO (Statistical Rejection Sampling Improves Preference Optimization)

Implementation of Statistical Rejection Sampling Improves Preference Optimization
Respond to #816.

To do:

[X] Add a function to make generation from the SFT model
[X] Add a function to score the generated sentences using the reward model
[X] Example script on how to perform statistical rejection sampling
[X] Implement the two ranking methods introduced in the paper: ie tournament ranking and first-round ranking

To be discussed:

[ ] I have added a  generate  and  score  function to  trainer.utils as those two functions will also be useful for raft and rest implementations (#704). It would also be possible to add those functions to a utils.py script inside the examples folder. What do you think?
[ ] Same for the conduct_rejection_sampling function.
[ ] RSO nearly left unchanged the loss function of DPO and mainly introduces a method to generate preference pairs from the target optimal policy. Thus, the training script is the same as dpo.py. I don't think adding another similar script is useful. What's your opinion?
[ ] Not sure of the location to put those RSO example scripts.

cc @kashif @younesbelkada @lvwerra @philschmid

thanks @gaetanlop I added the `"hinge" loss function which is what the RSO authors also use right?
I am also refactoring the DPO code a bit to remove the reference model log probs calculation to be done externally to the trainers (as a dataset mapper)
I had some questions about the seq-2-seq  models for these methods (asked in the seq-2-seq merged PR)

Hello @kashif, thanks for looking at this. It looks like the RSO authors experimented both with sigmoid (used in dpo) and hinge (used in SliC) loss function. Can you send the link where you posted the questions for the seq2seq models please? I do not find it

@gaetanlop see here: https://github.com/huggingface/trl/pull/586/files#diff-f2d3c6530916998b7d251573afa193b5f0dfeca5b1a96147fe368c967ea8e079R380

Stay calm stale-bot :D

Looks cool!
Are there any recent updates?


The new version release is missing files and cannot be run directly.

Environment

Operating System: Darwin
Node Version:     v20.17.0
Nuxt Version:     3.13.2
CLI Version:      3.13.2
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, imports, devServer, colorMode, extends, modules, css, dayjs, elementPlus, runtimeConfig, compatibilityDate, hooks, components
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

                                                                                1:55:17 AM

👉 Report an issue: https://github.com/nuxt/nuxt/issues/new?template=bug-report.yml
👉 Suggest an improvement: https://github.com/nuxt/nuxt/discussions/new
👉 Read documentation: https://nuxt.com
Reproduction
I need to manually copy the vue and @ vue dependency packages to. output in order to run properly.
[nuxt] [request error] [unhandled] [500] Cannot find module '/Users/xuzhibin/Desktop/Project/ADLWebNuxt3/.output/server/node_modules/vue/server-renderer/index.mjs' imported from /Users/xuzhibin/Desktop/Project/ADLWebNuxt3/.output/server/chunks/routes/renderer.mjs
  at finalizeResolution (node:internal/modules/esm/resolve:265:11)
  at moduleResolve (node:internal/modules/esm/resolve:933:10)
  at defaultResolve (node:internal/modules/esm/resolve:1169:11)
  at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:540:12)
  at ModuleLoader.resolve (node:internal/modules/esm/loader:509:25)
  at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:239:38)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:96:40)
  at link (node:internal/modules/esm/module_job:95:36)
[nuxt] [request error] [unhandled] [500] Cannot find module '/Users/xuzhibin/Desktop/Project/ADLWebNuxt3/.output/server/node_modules/vue/server-renderer/index.mjs' imported from /Users/xuzhibin/Desktop/Project/ADLWebNuxt3/.output/server/chunks/routes/renderer.mjs
  at finalizeResolution (node:internal/modules/esm/resolve:265:11)
  at moduleResolve (node:internal/modules/esm/resolve:933:10)
  at defaultResolve (node:internal/modules/esm/resolve:1169:11)
  at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:540:12)
  at ModuleLoader.resolve (node:internal/modules/esm/loader:509:25)
  at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:239:38)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:96:40)
  at link (node:internal/modules/esm/module_job:95:36)
^C%                                                                                (base) xuzhibin@xuzhibindeMacBook-Pro ADLWebNuxt3 % npx nuxi info
Working directory: /Users/xuzhibin/Desktop/Project/ADLWebNuxt3                                             1:55:17 AM
Nuxt project info: (copied to clipboard)                                                                   1:55:17 AM

------------------------------
- Operating System: Darwin
- Node Version:     v20.17.0
- Nuxt Version:     3.13.2
- CLI Version:      3.13.2
- Nitro Version:    2.9.7
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      devtools, imports, devServer, colorMode, extends, modules, css, dayjs, elementPlus, runtimeConfig, compatibilityDate, hooks, components
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

Describe the bug
I need to manually copy the vue and @ vue dependency packages to. output in order to run properly.

[nuxt] [request error] [unhandled] [500] Cannot find module '/Users/xuzhibin/Desktop/Project/ADLWebNuxt3/.output/server/node_modules/vue/server-renderer/index.mjs' imported from /Users/xuzhibin/Desktop/Project/ADLWebNuxt3/.output/server/chunks/routes/renderer.mjs
  at finalizeResolution (node:internal/modules/esm/resolve:265:11)
  at moduleResolve (node:internal/modules/esm/resolve:933:10)
  at defaultResolve (node:internal/modules/esm/resolve:1169:11)
  at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:540:12)
  at ModuleLoader.resolve (node:internal/modules/esm/loader:509:25)
  at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:239:38)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:96:40)
  at link (node:internal/modules/esm/module_job:95:36)
[nuxt] [request error] [unhandled] [500] Cannot find module '/Users/xuzhibin/Desktop/Project/ADLWebNuxt3/.output/server/node_modules/vue/server-renderer/index.mjs' imported from /Users/xuzhibin/Desktop/Project/ADLWebNuxt3/.output/server/chunks/routes/renderer.mjs
  at finalizeResolution (node:internal/modules/esm/resolve:265:11)
  at moduleResolve (node:internal/modules/esm/resolve:933:10)
  at defaultResolve (node:internal/modules/esm/resolve:1169:11)
  at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:540:12)
  at ModuleLoader.resolve (node:internal/modules/esm/loader:509:25)
  at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:239:38)
  at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:96:40)
  at link (node:internal/modules/esm/module_job:95:36)
^C%                                                                                (base) xuzhibin@xuzhibindeMacBook-Pro ADLWebNuxt3 % npx nuxi info
Working directory: /Users/xuzhibin/Desktop/Project/ADLWebNuxt3                                             1:55:17 AM
Nuxt project info: (copied to clipboard)                                                                   1:55:17 AM

------------------------------
- Operating System: Darwin
- Node Version:     v20.17.0
- Nuxt Version:     3.13.2
- CLI Version:      3.13.2
- Nitro Version:    2.9.7
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      devtools, imports, devServer, colorMode, extends, modules, css, dayjs, elementPlus, runtimeConfig, compatibilityDate, hooks, components
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -
------------------------------

Additional context
No response
Logs

Is this happening in an existing project? Could you reproduce it in a new one?

Is this happening in an existing project? Could you reproduce it in a new one?

The project is quite complex, there was no problem before. I am now building it on Windows 10 and Mac, and it works the same when running it locally. However, there is a lack of @vue node_module and vue node_module

I had the same issue with<EMAIL_ADDRESS>and<EMAIL_ADDRESS>installed.
Manually upgrading vue (pnpm upgrade vue) to the latest version (3.5.13) fixed the issue for me.

Seems like this was a vue or lockfile issue ? I'm going to close this. Thank you !