Clean up navigation logic (2/x).

This is the second PR in the series that began with #4428.
It converts some more components to Hooks-based components, to prepare for future work, and it also makes some preparatory changes to ZulipStatusBar, which I'd like to focus on in the next PR in this series. (ZulipStatusBar, I've found, isn't the best place to put logic that handles the safe-area view at the top; that PR will relocate that logic.)
I found that I could reasonably include a fix for #4267 in this PR, to help it land sooner. The status bar still appears in the lightbox after this PR, despite passing hidden to the lightbox's ZulipStatusBar. I'll get to the bottom of that, later, but it's orthogonal to what's really needed to fix #4267:

having the lightbox header clear the unsafe area at the top (which contains the status bar, if present) when it slides down
having the footer clear the unsafe area at the bottom, when it slides up
padding the contents of the header and footer horizontally, so they don't appear in the horizontal unsafe areas while the device is in landscape mode.

(I neglected 2 and 3 in my #4268; so, I'm closing that.)
After those fixes, the lightbox looks like this on a newer iPhone (showing the image in this message). These screenshots show the state when the header and footer are visible; tapping the screen makes them slide cleanly off the screen. I've also tested on the office Android device, and on my own, older, iPhone; all works as expected.

Thanks!
I think you mean #4267 as the issue this will fix.
I'll look at the code shortly, but first a couple of thoughts on those helpful screenshots:

It looks like this causes the header and footer to get separated from the top and bottom of the screen. That seems like an odd effect. Better would be for the background of the header and footer to extend all the way to the top and bottom, respectively.

This is what Apple's Photos app does, for example. They have a nav bar at top and a tab bar at bottom, and they extend all the way up and down (but keep their actual controls within the safe area.)
Actually, more than that: when the header is in view, we should be showing the actual status bar. (With a background coordinated with our header: see https://developer.apple.com/design/human-interface-guidelines/ios/bars/status-bars/. In the Photos app, the nav bar / header is visually continuous with the status bar.) We should hide the status bar just when we're hiding the header and footer, and showing purely the image.

This part is maybe a separate issue from #4267 but I think is also part of #3066: it looks like we're trying to show the image, the one the user is trying to look at, all the way out to the edges of the full screen, and as a result trying to show it in places like the sensor notch where nothing can show. Instead, we should be scaling the image so it fits in the area you can see.

... Well, I wrote that and then I went and looked at Apple's Photos app, viewing some screenshots (which conveniently, like I guess the image in your example message, have exactly the screen's size as their natural size) -- and they show them across the full screen, just like we do.
Ah, but if I rotate the device into landscape mode, then the screenshot gets scaled down to fit -- it gets "pillarboxed". We should do the same, I think. It looks like we're instead doing some unhelpful cropping.

Thanks!
I think you mean #4267 as the issue this will fix.
I'll look at the code shortly, but first a couple of thoughts on those helpful screenshots:

It looks like this causes the header and footer to get separated from the top and bottom of the screen. That seems like an odd effect. Better would be for the background of the header and footer to extend all the way to the top and bottom, respectively.

This is what Apple's Photos app does, for example. They have a nav bar at top and a tab bar at bottom, and they extend all the way up and down (but keep their actual controls within the safe area.)
Actually, more than that: when the header is in view, we should be showing the actual status bar. (With a background coordinated with our header: see https://developer.apple.com/design/human-interface-guidelines/ios/bars/status-bars/. In the Photos app, the nav bar / header is visually continuous with the status bar.) We should hide the status bar just when we're hiding the header and footer, and showing purely the image.

This part is maybe a separate issue from #4267 but I think is also part of #3066: it looks like we're trying to show the image, the one the user is trying to look at, all the way out to the edges of the full screen, and as a result trying to show it in places like the sensor notch where nothing can show. Instead, we should be scaling the image so it fits in the area you can see.

... Well, I wrote that and then I went and looked at Apple's Photos app, viewing some screenshots (which conveniently, like I guess the image in your example message, have exactly the screen's size as their natural size) -- and they show them across the full screen, just like we do.
Ah, but if I rotate the device into landscape mode, then the screenshot gets scaled down to fit -- it gets "pillarboxed". We should do the same, I think. It looks like we're instead doing some unhelpful cropping.

I think I'll tackle #4267 in a separate PR after this one—I've just pushed a revision that excludes that part. 🙂

I think I'll tackle #4267 in a separate PR after this one—I've just pushed a revision that excludes that part. 🙂

Looks good, thanks! Merged.
I edited the PR description first to replace "what's really needed to fix #4267" with "what's really needed for fixing #4267", so it wouldn't accidentally cause that issue to be closed :slightly_smiling_face:

Looks good, thanks! Merged.
I edited the PR description first to replace "what's really needed to fix #4267" with "what's really needed for fixing #4267", so it wouldn't accidentally cause that issue to be closed :slightly_smiling_face:

@chrisbobbe Can I help you with the cleanup and in conversion of component to hooks based components?
I can start with subscriptions and user status screens.

@chrisbobbe Can I help you with the cleanup and in conversion of component to hooks based components?
I can start with subscriptions and user status screens.

Sure, @karanb1, thanks! Several of our components don't have any lifecycle methods beyond render, so they're low-hanging fruit for Hooks conversions. For one that's even a little bit complicated, it would be helpful to handle it in multiple commits, as you can see I've done in this PR with Lightbox. I'd stay away from the really complicated components like ComposeBox, for now; there's a good chance that some subtlety of how things work isn't clearly documented, which makes it unfortunately easy to introduce bugs in a refactor like this. If you'd like to follow or join the discussion about converting to Hooks, it's here. 🙂

Sure, @karanb1, thanks! Several of our components don't have any lifecycle methods beyond render, so they're low-hanging fruit for Hooks conversions. For one that's even a little bit complicated, it would be helpful to handle it in multiple commits, as you can see I've done in this PR with Lightbox. I'd stay away from the really complicated components like ComposeBox, for now; there's a good chance that some subtlety of how things work isn't clearly documented, which makes it unfortunately easy to introduce bugs in a refactor like this. If you'd like to follow or join the discussion about converting to Hooks, it's here. 🙂

(Just split off some work on the lightbox in #4442.)

(Just split off some work on the lightbox in #4442.)


chore(monitoring): remove config

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

v1.3.10
v1.3.10
GitHub release

Your semantic-release bot :package::rocket:


jsoncpp conflicts？

Hi, I use this repo as a start project,
when I wrote in .cpp
#include <json/json.h>

I got
fatal error: json/json.h: No such file or directory

if I install jsoncpp via vcpkg, and add
target_link_libraries(Draw PRIVATE
    jsoncpp_object
    jsoncpp_static
    Open3D::Open3D
    )

then I got many multiple definition link errors like:
multiple definition of `Json::StreamWriterBuilder::validate(Json::Value*) const'; /home/hong/devs/vcpkg/installed/x64-linux/debug/lib/objects-Debug/jsoncpp_object/json_writer.cpp.o:/home/hong/devs/vcpkg/buildtrees/jsoncpp/src/3918c327b1-034a82149a.clean/src/lib_json/json_writer.cpp:1204: first defined here
collect2: error: ld returned 1 exit status

I looked Open3D code, seems jsoncpp lib is not installed as 3rdparty lib when make install?
any suggestions or  work around?
Thanks

with -DGLIBCXX_USE_CXX11_ABI=ON, problem solved
Thanks


Support interactive netlist plotting for Component.plot_netlist

Is your feature request related to a problem? Please describe.
Leverage the code in https://github.com/gdsfactory/gplugins/pull/199 for being able to interactively look at a netlist from components.

close by https://github.com/gdsfactory/gplugins/pull/199
thank you Niko!


Ignore symfony dymp buttons in preview

Adds a fix to ignore symfony dump buttons so you can dd a preview and expand items.

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


empty application.yml in main resources

There is an empty application.yml in the main resources (which means it'll end up in the jar and  corresponds to the default location Spring Boot uses to load config)

Thanks for the hint, removed the unnecessary YAML file.


Check if shims is already in path before adding it

I did a new pull request because I did a bad rebase

We had this feature but I had to revert (e2173df4aa91c8d365ca1596fb857fcac9fdd787) because it bit too many people who had duplicated PATH configuration due to starting terminal multiplexer such as tmux. These users needed the shims path to get prepended to the PATH again with high precedence, but because of non-duplication logic, the shims path would stay in PATH with low precedence while other system paths were repeated on top.
So, for the sake of simplicity and not being overly clever in code, we're not gonna do this.
BTW you don't have to reopen a new PR when you mess one up. You can force-push to the same branch to refresh your PR after a bad rebase.

@mislav Makes sense. Thanks.
For now I'm just doing which rbenv > /dev/null && [[ ":$PATH:" != *":$(rbenv root)/shims:"* ]] && eval "$(rbenv init -)"
Thanks

@bradynpoulsen In which case do you get duplicate paths? Also tmux?

I've never really used tmux before, but just to figure it out I adjusted it to
if which rbenv > /dev/null; then
    [[ "$PATH:" != "$(rbenv root)/shims:"* ]] && eval "$(rbenv init -)"
    export PATH="${PATH//:$(rbenv root)\/shims:/}"
fi

I use a pathAdd <newPath> function to conditionally add paths to the PATH. I've adjusted it to do the following
function pathAdd() {
    if [[ -n "$1" ]]; then
        [[ ":$PATH:" == *":/sbin:"*":$1"* ]] && PATH=`echo $PATH | sed -e "s/:${1//\//\\\/}//"`
        if [[ -d "$1" ]] && [[ ":$PATH:" != *":$1:"* ]]; then
            PATH="$1:$PATH"
        fi
    fi
}


Request: auto-complete for lints

When adding attributes to adjust lint levels, such as warn(missing_docs), allow(non_snake_case), and so on, there is no auto-complete support.
Since lints are pulled from a fairly fixed list, this is hopefully simple enough to provide suggestions for.

Where exactly is no autocomplete support? Typing inside a #[allow($0)] ($0 being the cursor) should give you completions.

Ah, then perhaps it's missing only for inner attributes.

Ah no, the problem here is that the parentheses is unclosed, #![allow($0) works but #![allow($0 does not. Will have to investigate if we can fix this as effectively the entire file turns into the attribute's input.


Setting method doesn't work correctly

Hi,
I have simple page with default PIGNOSE Calendar. I added checkbox:
<input type="checkbox" name="datepicker_range">
And onChange event:
$('input[name=datepicker_range]').change(function () {
   var range_checked = this.checked;
   $('.calendar').pignoseCalendar('setting', {
       multiple: range_checked
   });
});

Unfortunately this doesn't work, the calendar has single-selected date all time. I replaced 'setting' with 'configure' but the effect is the same. Is it bug?

Hey I have the same problem, has anyone ever found a solution? In the documentation the method is written as "settings" instead of "setting" but settings is not valid and setting (as well as configure) is not working.
Can anyone help?


⚠ Samsung has degraded performance

In ee36fa5, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2122 ms

Resolved: Samsung performance has improved in d9cb7c8.


Incorrect argument label in call (have 'withAttributes:', expected 'attributes:')

Im getting this error on PageIndicator.swift. Any tips on how to solve this?

you kin fix it like that:
self.frame.size = maximumString.size(attributes: [NSFontAttributeName: font])


chore: Test branch for contributors stuff

Signed-off-by: Marcos Iglesias<EMAIL_ADDRESS>
Summary of Changes
PR to add contributors

Documentation

CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely, including a title prefix.
[ ] PR includes a summary of changes.
[ ] My commits follow the guidelines from "How to write a good git commit message"

@all-contributors please add @ttannis for bug, code, content, data, doc, example, ideas, infra, maintenance, mentoring, platform, plugin, projectManagement, question, review, security, tool, test, tutorial

@all-contributors please add @ttannis for bug, code, content, data, doc, example, ideas, infra, maintenance, mentoring, platform, plugin, projectManagement, question, review, security, tool, test, tutorial


rename columns to match NEAR

This PR renames columns in transactions and events_actions to use txn_hash and block_height, matching current NEAR nomenclature.
Both of these tables would need to be rebuilt, not simply refreshed, due to the column change.
Testing:
root@0d1d2327f398:/near# dbt run -m transactions
Running with dbt=0.21.1
Found 5 models, 41 tests, 0 snapshots, 0 analyses, 175 macros, 0 operations, 0 seed files, 2 sources, 0 exposures

19:33:39 | Concurrency: 4 threads (target='dev')
19:33:39 |
19:33:39 | 1 of 1 START incremental model DEV.transactions...................... [RUN]
19:38:26 | 1 of 1 OK created incremental model DEV.transactions................. [SUCCESS 1 in 287.32s]
19:38:26 |
19:38:26 | Finished running 1 incremental model in 292.80s.

Completed successfully

Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

root@0d1d2327f398:/near# dbt run -m actions_events
Running with dbt=0.21.1
Found 5 models, 41 tests, 0 snapshots, 0 analyses, 175 macros, 0 operations, 0 seed files, 2 sources, 0 exposures

21:31:22 | Concurrency: 4 threads (target='dev')
21:31:22 |
21:31:22 | 1 of 1 START incremental model DEV.actions_events.................... [RUN]
21:32:40 | 1 of 1 OK created incremental model DEV.actions_events............... [SUCCESS 1 in 78.05s]
21:32:40 |
21:32:40 | Finished running 1 incremental model in 84.01s.

Completed successfully

Done. PASS=1 WARN=0 ERROR=0 SKIP=0 TOTAL=1

Missed blocks! I'll update. And yep, I totally forgot about the documentation files. Will fix those, too.

oh just realised and your action_events table too while i was looking at it :p

@forgxyz actually don't worry about the renaming for transactions, i've addressed that in this PR with other changes, so don't wanna have a clash. https://github.com/MetricsDAO/near_dbt/pull/26
Would be great if you can update the one for blocks too

Ok, so I will update for blocks and will ignore transactions as that is addressed in another PR.

transactions.sql reverted to original commit from main.


Added "Collège des Mûriers"

@Pixfri First stage of verification passed, official domain is clg-lesmuriers.sarthe.e-lyco.fr as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://clg-lesmuriers.sarthe.e-lyco.fr/page/6/

@Pixfri Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course. Thank you.

@Pixfri You might miss my previous comment. Please react.

@Pixfri Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course. Thank you.

Hello, my school's website is indeed this one. I'm sorry I couldn't answer as soon as possible because I was not at home.

@Pixfri Thank you for the clarification. Could you please not only confirm that this website belongs to your school, but also the fact that the school offers an IT-related long-term (>1 year) course. If the school does not offer such a course, we for sure will not be able to add the school's domain to this repository.

@Pixfri I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Bump PHP to 8.3

The configuration does not supported PHP8.2
https://github.com/elgentos/magento2-store-switcher/blob/main/src/Model/Configuration.php#L21
https://3v4l.org/kdHDk

Thanks!


chore: promote test1 to version 0.0.3

chore: promote test1 to version 0.0.3
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign q1323829945
You can assign the PR to them by writing /assign @q1323829945 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@q1323829945: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
d44a798946a9203507255debecffd22b8fe80eb0
link
/test verify

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


Update docker image to use the SCDF-based app launcher

TBD: we might revert the docker image name to use the old module launcher name, until the module launcher is removed.

This is not applicable anymore.


Performance Test Script missing from repo: examples/provisioner/general-purpose-azure-smallnodes.yaml

Version

Karpenter Version: v0.0.0

Kubernetes Version: v1.0.0
Expected Behavior
FIle: examples/provisioner/general-purpose-azure-smallnodes.yaml is missing
Actual Behavior
make az-perftest100
hack/azure/perftest.sh 100

'[' -z 100 ']'
replicas=100
FMT=+%Y-%m-%dT%H-%M-%SZ
++ date +%Y-%m-%dT%H-%M-%SZ
START=2023-11-09T00-18-16Z
++ date --iso-8601=seconds
STARTKUBECTL=2023-11-09T00:18:16+00:00
mkdir -p logs
exec
exec
++ tee -i logs/az-perftest-2023-11-09T00-18-16Z-100.log
logk=logs/az-perftest-2023-11-09T00-18-16Z-100-karpenter.log
kubectl apply -f examples/provisioner/general-purpose-azure-smallnodes.yaml
error: the path "examples/provisioner/general-purpose-azure-smallnodes.yaml" does not exist
make: *** [Makefile-az.mk:212: az-perftest100] Error 1

Steps to Reproduce the Problem
make az-perftest100
Resource Specs and Logs
N/A
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Thanks for sharing, there is a couple of references to our provisioners that did not get cleaned up. Also in Makefile.az for gpu provisioning, the readme, and a couple of other places.
Also note those perf scripts dont run in macos, and have room for improvement. Let me clean those up!


Can't call setState (or forceUpdate) on an unmounted component.

Error:
Can't call setState (or forceUpdate) on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method.
Description:
Pressing on a "Place" item in the listview from react-native-google-places-autocomplete triggers the error. I've stepped through it in Chrome's devtools and found the error occurs right after TouchableHighlight from _renderRow (please see screenshot below).

Code:
  <GooglePlacesAutocomplete
    autoFocus
    debounce={200}
    enablePoweredByContainer={false}
    fetchDetails={false}
    GooglePlacesSearchQuery={{
      rankby: 'distance',
      types: 'establishment'
    }}
    listViewDisplayed={destinationInput}
    minLength={2}
    nearByPlacesAPI="GooglePlacesSearch"
    onPress={data => setDestination(data)}
    placeholder="Where to?"
    placeholderTextColor="black"
    ref={setRef}
    returnKeyType="search"
    styles={searchDestStyle}
    textInputProps={{
      onBlur: blurDest,
      onFocus: focusDest
    }}
    query={{
      key: config.apiKey,
      language: 'en',
      types: ['establishment', 'geocode'],
      location: '37.786279,-122.406456',
      radius: '15000',
      components: 'country:us'
    }}
  />

destinationInput is a Boolean from redux state.
setDestination, blurDest, focusDest are a redux action creators.
setDestination sends the location data to redux state, which react-native-maps-directions uses to send direction coordinates to react-native-maps. Then a react-native modal is triggered.
Relevant dependencies:
    "expo": "^29.0.0",
    "react": "16.4.2",
    "react-native": "https://github.com/expo/react-native/archive/sdk-29.0.0.tar.gz",
    "react-native-google-places-autocomplete": "^1.3.9"

Same here! But I just get a warning.

set debounce to 0 if it is fine with your usecase.

I have the same problem and setting debounce to 0 didn't help. The warning is still there half of the times.

Do you know if this is still an issue?

This sounds like it might be caused by redux updating the props when the component is unmounted, hence the error.  See this article for more.

@alexpopa-work  @frankfaustino  did you find any fix for this?


Use V1 API for actions.scale_deployment

Hey all,
This PR fixes the actions.scale_deployment action by bringing it inline with the rest of the class and using the AppsV1 API.
I've also included it in __all__, now that it seems to work properly (for me at least!).
Thanks so much for the software!

Apologies - a rogue extra file had slipped into this PR - removed now!

Great, thanks!


Account information not displaying

Navigate to Sign Up
Activate Sign Up button
System displays Accounts page
Press the browser back button
Press Sign Up button again

Account details do not display. Page does not seem to recognize log in attempt - Link your GitHub account button is enabled (normally disabled post sign up / log in)

duplicate of #32


rewrite: FetchContent for abseil-cpp

This now works for me. I also tested with -DLP_STL_UNORDERED=ON.
[LangBench] mkdir build && cd build
[LangBench/build] cmake .. -DLP_ABSL_FLAT=ON -DLP_REGEX_STD=ON
[LangBench/build] make

Thanks!


Prepare secureCodeBox CRDs for v2 Stable Release

The CRD's of the secureCodeBox Operator are currently use the experimental.securecodebox.io as group name.
For the stable v2 release this should be changed to just securecodebox.io.

Closed via #130


Image text align correction

The float:  used for image alignment, doesn't match the output from the editor.
This changes the image so it is wrapped in a div, with the text-align property.
I beleive this this fixes #28, https://github.com/jpuri/react-draft-wysiwyg/issues/838 , https://github.com/jpuri/react-draft-wysiwyg/issues/723

Is there any reason why this commit hasn't been merged yet?


Few things to understand

hi yhenon
Thanks for providing in the implementation. I have below things to understand from you...

in Losses .py file
why are we subtracting from width & height anchor[:,0] & anchor [:,1] respectively i think we initi anchors  0,1 columns to zero
anchor = anchors[0, :, :]
 anchor_widths  = anchor[:, 2] - anchor[:, 0]
 anchor_heights = anchor[:, 3] - anchor[:, 1]

what does anchor ctr_X denoes ,why we use 0.5 here
anchor_ctr_x   = anchor[:, 0] + 0.5 * anchor_widths
anchor_ctr_y   = anchor[:, 1] + 0.5 * anchor_heights

what is the purpose of shifted anchors in model.py.
shifted_anchors = shift(image_shapes[idx], self.strides[idx], anchors)
all_anchors     = np.append(all_anchors, shifted_anchors, axis=0)

In general  does x and y cordinates  in   bbox  vector  [pc,x,y,h,w...] deonotes top left corner cordinates ?

In general does x and y cordinates in bbox vector [pc,x,y,h,w...] deonotes top left corner cordinates ?

This is just convention. When x/y are top denote top left corner, it's easy to find the extent of the bounding box using height and width.

what does anchor ctr_X denoes ,why we use 0.5 here

It denotes center_x (ctr_x). If you take the x-coordinate of the top left corner and add half the width (then do the same for y), you get the center of the anchor box.

why are we subtracting from width & height anchor[:,0] & anchor [:,1] respectively i think we initi anchors 0,1 columns to zero

I haven't actually checked, but I'm fairly sure anchor[:, 2] is x2, anchor[:, 0] is x1. So x2-x1 is the width of the anchor box.

@jaideep11061982 please close this issue if @MikkelAntonsen answer clears your doubts.


Train error 'KeyError: 'MASTER is not in the models registry''

work_dir = 'weights'
gpu_ids = range(0, 1)
Traceback (most recent call last):
File "tools/train.py", line 221, in
main()
File "tools/train.py", line 185, in main
test_cfg=cfg.get('test_cfg'))
File "/home/thorpham/Documents/challenge/mmocr/mmocr/models/builder.py", line 140, in build_detector
cfg, default_args=dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "/home/thorpham/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/mmcv/utils/registry.py", line 212, in build
return self.build_func(*args, **kwargs, registry=self)
File "/home/thorpham/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/mmcv/cnn/builder.py", line 27, in build_model_from_cfg
return build_from_cfg(cfg, registry, default_args)
File "/home/thorpham/anaconda3/envs/open-mmlab/lib/python3.7/site-packages/mmcv/utils/registry.py", line 45, in build_from_cfg
f'{obj_type} is not in the {registry.name} registry')
KeyError: 'MASTER is not in the models registry'
Can you help me fix error ? Thank you

@JiaquanYe do you modify code  in mmdetection-2.11.0 . Which code you modify .I use mmocr-0.4 so it must  mmdetection 2.14.0

@JiaquanYe do you modify code in mmdetection-2.11.0 . Which code you modify .I use mmocr-0.4 so it must mmdet-2.18.1

Yes, I have modify the mmocr and mmdetection source code. You should install mmocr and mmdetection-2.11.0 in this repo.
The installation details can be found in README.


Az CLI shell

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
Unknown
Errors:
az: '&&[smd]' is not in the 'az' command group. See 'az --help'. If the command is from an extension, please make sure the corresponding extension is installed. To learn more about extensions, please visit https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview

'NoneType' object has no attribute 'in_tree'
Traceback (most recent call last):
pip-install-363drsgw\knack\knack\cli.py, ln 215, in invoke
cli\core\commands\__init__.py, ln 654, in execute
cli\core\commands\__init__.py, ln 718, in _run_jobs_serially
cli\core\commands\__init__.py, ln 711, in _run_job
Temp\pip-install-363drsgw\six\six.py, ln 703, in reraise
...
interactive\azext_interactive\azclishell\app.py, ln 600, in handle_scoping_input
    if self.completer.command_tree.in_tree(tree_path):
AttributeError: 'NoneType' object has no attribute 'in_tree'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az &&[smd]

Expected Behavior
Environment Summary
Windows-10-10.0.17134-SP0
Python 3.6.6
Installer: MSI

azure-cli 2.8.0 *

Extensions:
deploy-to-azure 0.2.0
dev-spaces 1.0.5
interactive 0.4.4

Additional Context

Shell is account language so graph if version lastest someone process and extension

Sorry I don't understand your statement:

Shell is account language so graph if version lastest someone process and extension

Could you let us know

What you are trying to do?
How you are doing that, with what commands?

Thanks for the information.

firsts not single to download async

firsts not single to download async

Sorry I can't understand your comment. Could you provide some more details?

Resource to get (|pragna) alone is status function or project asset [have to remote; draft]

The author seems to be a bot and also created https://github.com/git-lfs/git-lfs/issues/4212 before. Closing.


[turborepo] turbo prune fails in docker environment with turbo > 1.5.5

What version of Turborepo are you using?
1.5.6-1.8.3
What package manager are you using / does the bug impact?
Yarn v1
What operating system are you using?
Linux
Describe the Bug
turbo prune (and sometimes yarn global add) fails when turbo version at line RUN yarn global add<EMAIL_ADDRESS>pnpm is set to<EMAIL_ADDRESS>or greater. Copies of errors are below the dockerfile.
# syntax=docker/dockerfile:1

FROM node:18-alpine3.16 AS builder
# Set working directory
WORKDIR /app

RUN yarn global add<EMAIL_ADDRESS>pnpm

COPY --link . .

RUN turbo prune --scope=api --docker

# Add lockfile and package.json's of isolated subworkspace
FROM node:18-alpine3.16 AS installer

RUN apk update && apk add --update --no-cache curl libc6-compat openrc openssl1.1-compat-dev && \
    rm -rf /var/cache/apk/*

RUN curl -fsSL https://tailscale.com/install.sh | sh

RUN yarn global add pnpm

WORKDIR /app

# Install the dependencies
COPY --link --from=builder /app/out/json/ .
COPY --link --from=builder /app/out/pnpm-lock.yaml ./pnpm-lock.yaml
RUN --mount=type=cache,target=/root/.local/share/pnpm/store/v3 \
    pnpm install

# Build the project
COPY --link --from=builder /app/out/full/ .
COPY --link turbo.json turbo.json

RUN pnpm turbo devserver.build --filter=api...

 Os { code: 2, kind: NotFound, message: "No such file or directory" }', crates/turborepo/src/main.rs:50:10
#17 0.418 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
------
executor failed running [/bin/sh -c turbo prune --scope=api --docker]: exit code: 101

<EMAIL_ADDRESS> => CACHED [builder 2/5] WORKDIR /app                                                                                0.0s
 => CACHED [builder 3/5] RUN yarn global add<EMAIL_ADDRESS>pnpm                                                                                0.0s
 => CACHED [builder 4/5] COPY --link . .                                                                                0.0s
 => ERROR [builder 5/5] RUN turbo prune --scope=api --docker                                                                                0.4s
------
 > [builder 5/5] RUN turbo prune --scope=api --docker:
#17 0.419 thread 'main' panicked at 'Failed to execute turbo.: Os { code: 2, kind: NotFound, message: "No such file or directory" }', crates/turborepo/src/main.rs:50:10
#17 0.419 note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
------
executor failed running [/bin/sh -c turbo prune --scope=api --docker]: exit code: 101

Expected Behavior
I expect that the command runs and sends nothing to stdout
To Reproduce
File structure:
root

apps

api

Dockerfile

web
...

packages

...

**set the dockerfile's turbo version to turbo >= 1.5.6, <= 1.8.3
docker build . -t api -f ./apps/api/Dockerfile

Unfortunately the repo is private and I cannot share it.
Reproduction Repo
No response

I tested install again in a non-Docker setup, but I can say pretty confidently that the install works:
~/repos/triage$ yarn global add turbo
yarn global v1.22.19
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 🔨  Building fresh packages...
success Installed<EMAIL_ADDRESS>with binaries:
      - turbo
✨  Done in 2.40s.

You're using yarn to install, and pnpm to run. You should unify that.
1.7+ error is https://github.com/vercel/turbo/blob/ce52ed4508140f63ad192e66ab8be262ab7adaf1/crates/turborepo/src/main.rs#L51

Looks like the asset that is supposed to be present (go-turbo) isn't, which is I assume that it is a docker layer cache.
I don't know how that might have been triggered, but if you reset your build's Docker cache it may probably start working. Alternatively, if you can create a reproduction you can share, I'll be happy to investigate further!
For additional help through this you can visit our Discord! Somebody there may have encountered this configuration before.

Unfortunately I can't install turbo and pnpm with pnpm on this docker base image. I'll look for one with pnpm installed.

corepack enable
corepack prepare pnpm@7 --activate


Add CBlockVersion from Namecoin

The CBlockVersion type used in Namecoin to manage block version and AuxPoW flags independently significantly reduces risks, and is also expected by their unit tests. This should be merged in once Bitcoin Core 0.10 patches are in place.

In Dogecoin Core 1.10 now.


fix(telegram): simply return text when received at, fix koishijs/koishi#907

Codecov Report

Merging #62 (1bfbb07) into master (6963972) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #62   +/-   ##
=======================================
  Coverage   82.71%   82.71%
=======================================
  Files           1        1
  Lines         405      405
  Branches       83       83
=======================================
  Hits          335      335
  Misses         70       70

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Change from just "Loading..." to an actual spinner

The time delay in getting data back from our custom GitHub contributor API is such that it needs a visual notification to the user.
There is a npm library React Spinners that offers a choice of spinners.
We could use their Pacman spinner with a cute saying as is seen in the below gif.
I respectfully request that I be assigned to this issue.

Sounds good to me! Assigned. :D

10/10 on the Pac-man spinner and sayings

I have 2 observations:

The contributors data doesn't have large changes. (Yes, commit numbers will change and, from time to time, a new person with a small number of commits will appear close to the bottom of the list.)
The list on the website could be slightly out-of-sync with the current data without the world crashing come crashing down.

This suggests that we should take the load of making the API call off of the Gatsby page. Instead, put the effort of calling the API somewhere else and store the resulting data in a JSON file.
The Gatsby page could then do a fetch from that JSON file and its response time to the user will be much faster.
I have successfully written a GitHub action in my fork that schedules a call to the API and saves the results into a file called contributors.json in the root of the repo.
Then I modified the fetch inside the Gatsby page to load that file.
It is indeed much faster.
By compromising on the frequency of the scheduled GitHub action, it will also have the benefit of limiting calls to your Azure account.

Based on this, we could do something similar to what we have currently for Doxygen, where we can:

Make a new branch
Update the contributors.json file
Auto merge the change based on a rule we set within the repo

Considering it'd probably be as simple as using a cURL request to grab the most up to date json, this wouldn't be terribly hard to set up.
@RubyNova any issues with this approach? The only ones I see are maintenance (which is minimal) and its not "real time" (but we don't have PRs pumping into our repos every hour so I think it's a non-issue)

I agree with using curl. My GH Action doesn't create a new branch, but its logic is similar to what @capnkenny is suggesting.
# This is a basic workflow to help you get started with Actions

name: Fetch Contributors

# Controls when the workflow will run
on:
  # Triggers the workflow on push or pull request events but only for the main branch
  push:
    branches: [ main ]

  schedule:
    - cron: "05 3 * * *"

  # Allows you to run this workflow manually from the Actions tab
  workflow_dispatch:

# A workflow run is made up of one or more jobs that can run sequentially or in parallel
jobs:
  # This workflow contains a single job called "fetch"
  fetch:
    # The type of runner that the job will run on
    runs-on: ubuntu-latest

    # Steps represent a sequence of tasks that will be executed as part of the job
    steps:
      # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
      - uses: actions/checkout@v3

      # Runs a single command using the runners shell
      - name: Run curl
        run: curl https://novelrt-functions.azurewebsites.net/api/novelrtcontributors -o contributors.json

      - name: Set email
        run: git config --global user.email<EMAIL_ADDRESS>
      - name: Set username
        run:  git config --global user.name "GitHub Action"

      - name: stage changed files
        run: git add .
      - name: commit changed files
        run: git commit -m "Auto updating contributors.json"
      - name: fetch from main
        run: git fetch origin main
      - name: push code to main
        run: git push origin HEAD:main

@capnkenny what does this mean for the existing azure function?

@RubyNova We would still use it, it would just be used by the action and only hit so often instead of on every page load.
Not only helps with site performance, but reduces the chance someone spams the function and drives your bill up :^)

I see.
In that case, I think we should just go ahead and resolve https://github.com/novelrt/NovelRT-Azure-Functions/issues/3 by committing to .NET 6.
The concerns outlined in the ticket are probably no longer even relevant if we do this.

I'm not entirely buying that switching the function to .NET would alleviate the concern here. Not without at least seeing a comparison with what we have now

Why wouldn't it?
the function could be written in ALGOL for all the GHA could care for, no?

Why wouldn't it?
the function could be written in ALGOL for all the GHA could care for, no?

Yes, GHA would not care. But are you agreeing to switch to the GHA update? Or are you saying to not resolve this and just update the function?
I may be misunderstanding but it sounds like we're conflating two issues that are somewhat independent of each other.

They are not independent of each other.
Currently the Azure function takes x amount of time to return the list. This is mostly due to node.js + contacting GH API, probably. To reiterate my concerns of the other ticket:

I've been told that node.js supposedly has 0 startup time but I don't entirely buy into that. It's Node. Nothing about it is fast.
.NET has a startup time, but I know that its startup times have been reduced as of .NET 6.
The majority of the maintainer team doesn't really write JS and this is a back end thing reaching into our org's statistics; this is the most important thing for me here.

If we do this switch to the JSON file with a GHA that just, say, runs nightly or something, then .NET and node startup times are no longer even relevant and we get the benefit of having the Azure function codebase for better accessibility for the maintainer teams. My main line of thinking here is that since the guy who volunteered to help us maintain the website above and beyond being a regular contributor is no longer around, it would basically be all on you to maintain the function as it stands in the current codebase.
That's not fair to you and we can't expect any contributor to just stand up and take that heightened responsibility right now.
Therefore, for the sake of everyone involved, I think the azure function should be rewritten to .NET 6. I'm the one currently handling building and deploying of it anyway. I'm like a lemon when it comes to Node. The GHA doesn't care what it's written in, but other humans somewhat do, given the current situation.

If you are looking at making a couple GH API calls once daily, you will not come close to hitting GH's anonymous rate limitations. That means it could be done without any need for GH secrets. We could write a script that the scheduled GHA runs to make the JSON file.

Then there's a misunderstanding on my part, apologies.
How often were you thinking this JSON file would get updated?

For example: Here's the endpoint for anonymously getting the contributors for the one repo.
https://api.github.com/repos/novelrt/NovelRT/contributors

As Kenny is saying, maybe we do once per day. Even once every 6 hours or once per hour should not be rate limiting.

We have a list of repos we want to include and loop through them creating a single JSON file. Then the Gatsby page reads and parses the file.


[resolvers: bundle] Cache resolved bundle to not fetch it all the time

Feature request
Provide an option, on the bundle resolver, to be able to cache/store the resolved bundle in order to not have to fetch it each time.

If the bundle is identified by the digest, we can cache it directly.
If the bundle is identified by a Tag, we could query the registry if the digest we stored for this bundle is still the one targeted by the tag — if not, we re-fetch the bundle, if yes, we just use the cache.
This could be optional.
We would need to make the "storage" extensible (aka in emptydir, in PVC, in CSI, in s3, …)

Use case

Reduce the number of "pull" from registries. For a PipelineRun with 10 tasks, all using the same bundle, I think we pull the bundle 10 times.
Reduce the possible impact of service outage / flakyness. If a registry is having an outage or is flaky, as long as the bundle is already in the cache, Pipelines and Tasks wouldn't be affected.

cc @abayer

Slightly related to https://github.com/tektoncd/pipeline/issues/6385 as well 👼🏼

https://github.com/tektoncd/pipeline/issues/6386

/lifecycle frozen
/area roadmap


Wrong event var in Modal

https://github.com/twbs/bootstrap/blob/master/dist/js/bootstrap.js#L1053
Var  event is not defined, document !== event.target replace to document !== e.target

Duplicate of #18908. Please search more in the future. Thanks.


Release Microsoft.Azure.Management.ResourceManager

Can you please release Microsoft.Azure.Management.ResourceManager 3.15.1-preview

This release was completed. @ArthurMa1978 as FYI.


Albums in the Collection should show the Artist rather than AlbumArtist if AlbumArtist isn't set, but the Artist field is the same

As you can see, the Artist is common throughout the group, but the AlbumArtist is not set, so the attribute does not show next to the Album name in the collection view when collapsed.
So, the front end sends
{"action":"FETCH","data":{"path":["Root"]}}

The server responds with
...
            {
               "Key" : "9c509",
               "AlbumArtist" : "Brookes Brothers",
               "Name" : "Carry Me On (feat. Chrom3)"
            },
            {
               "Name" : "City Of Gold",
               "Key" : "2098b"
            },
            {
               "AlbumArtist" : "Logistics",
               "Key" : "5ea2a",
               "Name" : "Crash Bang Wallop!"
            },
...

And the tag for 'Artist' is consistent across the entire album, confirmed with picard.

Fixed in 06685fcc


clip_gradients no longer exists?

clip_gradients - still in the defaults for NAFModel, but not used anywhere.  i recall this being here in the past and it is a common practice - not sure when it was lost.

tau - this is in a few configs and docstrings, but isn't actually used

We were doing loss clipping not gradient clipping in the q models, so this has been renamed to clip_loss. Gradient clipping might be implemented separately at some point.
fixed tau -> update_target_weight renaming in https://github.com/reinforceio/tensorforce/commit/4884c47e6ff2dac622351bc564782752b31f037e


Copy files over SSH: Cannot parse privateKey

Question, Bug, or Feature?
Type: Bug
Enter Task Name: Copy files over SSH
Environment

Server - Azure Pipelines
Agent - Hosted

Issue Description
Copy files over SSH does not work with a private key. I am able to connect with username and passwort only but not with a private key. In the error log is visible that "Cannot parse privateKey: Unsupported key format", however I tried multiple versions like:
ssh-keygen -t rsa -b 2048
ssh-keygen -t rsa -m PEM -b 2048
ssh-keygen -t rsa -m PEM

I used powershell version 5.1.18362.145 for the key generation.
I also stumbled over Issue 8818 which is already closed, however the same keygen options do not work for me.
My service connection looks like the following

My pipline like:

Error logs
[2019-09-26T17:24:57.5708149Z ==============================================================================
2019-09-26T17:24:57.5708283Z Task         : Copy files over SSH
2019-09-26T17:24:57.5708379Z Description  : Copy files or build artifacts to a remote machine over SSH
2019-09-26T17:24:57.5708485Z Version      : 0.151.1
2019-09-26T17:24:57.5708570Z Author       : Microsoft Corporation
2019-09-26T17:24:57.5708684Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/deploy/copy-files-over-ssh
2019-09-26T17:24:57.5708804Z ==============================================================================
2019-09-26T17:24:58.1932243Z ##[debug]agent.workFolder=d:\a
2019-09-26T17:24:58.1938497Z ##[debug]loading inputs and endpoints
2019-09-26T17:24:58.1943022Z ##[debug]loading ENDPOINT_AUTH_93ebd23f-ee98-41a1-8ff1-6361292f2e0d
2019-09-26T17:24:58.1945702Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_93ebd23f-ee98-41a1-8ff1-6361292f2e0d_PASSWORD
2019-09-26T17:24:58.1948155Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_93ebd23f-ee98-41a1-8ff1-6361292f2e0d_USERNAME
2019-09-26T17:24:58.1948949Z ##[debug]loading ENDPOINT_AUTH_PARAMETER_SYSTEMVSSCONNECTION_ACCESSTOKEN
2019-09-26T17:24:58.1949268Z ##[debug]loading ENDPOINT_AUTH_SCHEME_93ebd23f-ee98-41a1-8ff1-6361292f2e0d
2019-09-26T17:24:58.1949554Z ##[debug]loading ENDPOINT_AUTH_SCHEME_SYSTEMVSSCONNECTION
2019-09-26T17:24:58.1949821Z ##[debug]loading ENDPOINT_AUTH_SYSTEMVSSCONNECTION
2019-09-26T17:24:58.1950086Z ##[debug]loading INPUT_CLEANTARGETFOLDER
2019-09-26T17:24:58.1950347Z ##[debug]loading INPUT_CONTENTS
2019-09-26T17:24:58.1950694Z ##[debug]loading INPUT_FAILONEMPTYSOURCE
2019-09-26T17:24:58.1951023Z ##[debug]loading INPUT_FLATTENFOLDERS
2019-09-26T17:24:58.1951287Z ##[debug]loading INPUT_OVERWRITE
2019-09-26T17:24:58.1951549Z ##[debug]loading INPUT_SOURCEFOLDER
2019-09-26T17:24:58.1951803Z ##[debug]loading INPUT_SSHENDPOINT
2019-09-26T17:24:58.1952078Z ##[debug]loading INPUT_TARGETFOLDER
2019-09-26T17:24:58.1952336Z ##[debug]loading SECRET_SYSTEM_ACCESSTOKEN
2019-09-26T17:24:58.1952590Z ##[debug]loaded 16
2019-09-26T17:24:58.1952858Z ##[debug]check path : d:\a_tasks\CopyFilesOverSSH_67cec91b-0351-4c2f-8465-d74b3d2a2d96\0.151.1\task.json
2019-09-26T17:24:58.1953150Z ##[debug]set resource file to: d:\a_tasks\CopyFilesOverSSH_67cec91b-0351-4c2f-8465-d74b3d2a2d96\0.151.1\task.json
2019-09-26T17:24:58.1953426Z ##[debug]system.culture=en-US
2019-09-26T17:24:58.1953689Z ##[debug]sshEndpoint=93ebd23f-ee98-41a1-8ff1-6361292f2e0d
2019-09-26T17:24:58.1954066Z ##[debug]93ebd23f-ee98-41a1-8ff1-6361292f2e0d auth param username = ***
2019-09-26T17:24:58.1954687Z ##[debug]93ebd23f-ee98-41a1-8ff1-6361292f2e0d auth param password = ***
2019-09-26T17:24:58.1954886Z ##[debug]93ebd23f-ee98-41a1-8ff1-6361292f2e0d data host = my-server.com
2019-09-26T17:24:58.1955061Z ##[debug]93ebd23f-ee98-41a1-8ff1-6361292f2e0d data port = 22
2019-09-26T17:24:58.1955410Z ##[debug]Using private key for ssh connection.
2019-09-26T17:24:58.1955673Z ##[debug]contents=**
2019-09-26T17:24:58.1956288Z ##[debug]sourceFolder=d:\a\1\s\scripts\doc
2019-09-26T17:24:58.1956713Z ##[debug]check path : d:\a\1\s\scripts\doc
2019-09-26T17:24:58.1956988Z ##[debug]targetFolder=/home/test
2019-09-26T17:24:58.1957249Z ##[debug]cleanTargetFolder=true
2019-09-26T17:24:58.1957507Z ##[debug]overwrite=true
2019-09-26T17:24:58.1957758Z ##[debug]failOnEmptySource=true
2019-09-26T17:24:58.1958055Z ##[debug]flattenFolders=false
2019-09-26T17:24:58.1958285Z Setting up SSH service connection to remote host my-server.com.
2019-09-26T17:24:58.1958555Z ##[debug]task result: Failed
2019-09-26T17:24:58.2015161Z ##[error]Error: Failed to connect to remote machine. Verify the SSH service connection details. Error: Cannot parse privateKey: Unsupported key format.
2019-09-26T17:24:58.2025471Z ##[debug]Processed: ##vso[task.issue type=error;]Error: Failed to connect to remote machine. Verify the SSH service connection details. Error: Cannot parse privateKey: Unsupported key format.
2019-09-26T17:24:58.2042179Z ##[debug]Processed: ##vso[task.complete result=Failed;]Error: Failed to connect to remote machine. Verify the SSH service connection details. Error: Cannot parse privateKey: Unsupported key format.
2019-09-26T17:24:58.2042504Z ##[debug]Closing the client connection
2019-09-26T17:24:58.2071986Z ##[section]Finishing: CopyFilesOverSSH]

I have similar issue with the task SSH command. need to log with the privatekey option . hope this issue will be resolved soon

I have the same issue. Key was working on self hosted TFS server, on cloud Azure DevOps server its not working with "Error: Cannot parse privateKey: Unsupported key format". Installed locally Azure Devops Server on my machine to test it locally and its working. The only difference i've found so far is that on local Azure Devops server version of extension visible in the logs is 0.142.2 on cloud version of Azure Devops it's 0.151.1. On older TFS server  it's 0.121.0. Keys generated on powershell in windows and bash on centos. Ale with -m PEM option. Always the same error.

I finally found a workaround:
The preview feature "service connection" has to be disabled. Without the preview feature enabled there is a button "Upload SSH private key file...". The same key files as before works perfect.

I finally found a workaround:
The preview feature "service connection" has to be disabled. Without the preview feature enabled there is a button "Upload SSH private key file...". The same key files as before works perfect.

Could you share the screenshot where did you disabled 'preview feature "service connection"'?

Official there is this manual:
https://docs.microsoft.com/en-us/azure/devops/project/navigation/preview-features?view=azure-devops
For me it looks a bit different:
Click right top on your name then press the "..." behind azure devops profile and then select preview features.

@epsmae Thx for the tip. I can confirm that disabling The preview feature "service connection" fixed the issue for me.

@epsmae Thanks for the tip! Disabling the preview feature Service connection fixed the issue for me as well.

@epsmae Thanks you.

@epsmae Thanks a lot for your answer.
Disabling the preview feature Service connection fixed the issue for me as well.

Jeeez thanks @epsmae, had the same issue and after testing different types of keys this is what worked for me in the end! 👍

I am having this issue, and still cannot use a 4096 byte key

It would be really nice if the new connection experience (and old upload) did some validation. It's non-obvious that we have to include PEM.
Anyway, I got 4096 keys to work, using:
ssh-keygen -t rsa -m PEM -b 4096

U smell

From: Arno<EMAIL_ADDRESS>Sent: Thursday, November 28, 2019 3:34 AM
To: microsoft/azure-pipelines-tasks<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: Re: [microsoft/azure-pipelines-tasks] Copy files over SSH: Cannot parse privateKey (#11439)
@epsmaehttps://github.com/epsmae Thanks for the tip! Disabling the preview feature Service connection fixed the issue for me as well.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/azure-pipelines-tasks/issues/11439?email_source=notifications&email_token=ALUUZLYXUAJFUNOYZBK2OZDQV6NEXA5CNFSM4I25C37KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFMFVPA#issuecomment-559438524, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALUUZLY4RHPTMIP2TIITMP3QV6NEXANCNFSM4I25C37A.

Could someone from the release side take a look at this, looks like a service connections issue


Unable to launch the Angular 8 project After AOT Build

Hello all,
i have an issue While launching the product from Azure . Here is the issue
I got an exception like " unexpected end of input " in console (see attached pic ) .

I have cleared the Browser ( I used Chrome and IE) then reload the product then i got anther issue " Uncaught SyntaxError: missing ) after argument list "

I have cleared and reload the project many times  it finally launched after 20 mins .  Can you please help this , i have already lost few customers because of this issue .
Here is my Work Environment

Angular 8
Typescript 3.4.5
Visual Studio 2019
.Net Core 2.0
and AOT build Query is :    node --max_old_space_size=8192 node_modules/@angular/cli/bin/ng build --prod

Thanks
Praisy Athoti.

Hi, can you update to latest version of the CLI and see if the problem persists lease submit a reproduction.
A good way to make a minimal repro is to create a new app via ng new repro-app and add the minimum possible code to show the problem. Then you can push this repository to github and link it here.

Hello  alan-agius4,
Thank you for your response, In my project we are using Telerik Kendoui Controllers and this project size 350 MB.  My self i have tested with few files that time it launching correctly , if i use all my apps then it give that error .
Is there a way to build AOT without using " --max_old_space_size=8192 " .
Thanks
Victor Athoti

max_old_space_size is needed is your project is large. This error occurs when the memory allocated for the execution build is less than the required memory when run the build.

The problem reported though is a runtime error which is not related to the max_old_space_size flag. Can you setup a minimal repro please to replicate the runtime error?
Thanks.

Hello alan-agius4,
I will create  a  Repro-app  and will provide the github link .
Thanks

This looks like a network problem. Your server might not be serving the full .js files, which would give those errors.

Hello filipesilva ,
Thank you for your reply , May be it is network issue , but  we have Good bandwidth like 25 Mega Bytes speed and other sites loading very quickly.

Hello alan-agius4,
I have create a Repro-app with specific files , but this is loading ( did not get any issue ). i am trying to add more files one by one , when ever I get exception then I will upload to GitHub and will provide the link.
Thanks


Default Dispatch method implementation

We could allow for a default dispatch method implementation if the user supplies the type to the dispatch method like so:

export class IncAction { constructor( public amount: number ) {} }

export class MyComponent {
  @Dispatch(IncAction)
  add: (amount: number) => IncAction; // note that this defines the type and not a value

  @Dispatch(IncAction)
  increaseBy(amount: number) { } // note that this defines a method with an undefined return

}

@arturovt @splincode What do you think?

the best option for OOP

export class MyComponent {

 public ngOnInit() {
   this.increaseBy(2); // -> this.store.dispatch(new IncAction(2));
   this.onMultiAction('en'); // this.store.dispatch(new SetLanguage('en'), new Navigate('/home'));
 }

  @Dispatch()
  private increaseBy(amount: number) {
    return new IncAction(amount);
  }

  @Dispatch()
  private onMultiAction(lang: string) {
    return [new SetLanguage(lang), new Navigate('/home')];
  }

}

Also
@State({ })
class MyState {

  @ActionDispatcher()
  public static increaseBy(ctx, action) {
   ctx.setState(action.payload);
  }

}

export class MyComponent {

  @Dispatch(MyState.increaseBy) // @Emitter(MyState.increaseBy) add;
  private add: (amount: number) => MyState.increaseBy; // -> add.emit(amount)

 public ngOnInit() {
   this.increaseBy(2);
 }

}

i like multi action feature, i think we can think of its integration


Running multi-pages pipelines for SharePoint SPFx

Hi, i would like some help and input on the CI/CD in DevOps using the multi-stages YAML specify in this site > https://learn.microsoft.com/en-us/sharepoint/dev/spfx/toolchain/implement-ci-cd-with-azure-pipelines
I'm able to get this working if my SFPx web part is in it's own DevOps repos. But when i have a repos with multiple folders (web parts), it doesn't able to find the gulp.js or package-locked.json file.
(My repos have two webparts and each has it's own folder and i created 3 CI/CD YAML file in a template folder)

I got this error referencing the gulp. Do you know fi this YAML is working with folder or has to be on top level?

Thanks.

Hey @SeeBeeDev, thanks for reporting this. As this is more related to SPFx development, I'll move this issue to the SP Dev Docs repo where more SPFx related questions can be found. 😄

I've created a new issue over there as I couldn't move this one.


[5.10] Cherry-pick Fix divergent overwrite behavior of FileManager.copyItem(at:to:) on Linux and Windows

This is a cherry-pick of #4821 to 5.10.
On Apple platforms, the FileManager APIs copyItem(atPath:toPath:) and copyItem(at:to:) refuse to overwrite the destination file (or link, or directory) if it already exists; this behavior is even explicitly documented. And indeed, any such attempt results in a fileWriteFileExists error.
However, on Linux and Windows, the destination is silently overwritten. This PR changes that behavior to match what's documented. On POSIX platforms, this is accomplished by adding the O_EXCL flag to the relevant open(2) call, which results in an EEXIST errno if the path already exists. For Windows, the bFailIfExists parameter of CopyFileW() is changed to true.
Fixes #3368

@swift-ci please test

@swift-ci please test Windows platform

@swift-ci please test Windows platform

@swift-ci please test Windows platform


zo_snapshot triggering UITableView reload

I'm trying to use ZOZolaZoomTransition with a tableView, both in simulator and device running iOS 10 when popping a view controller "zo_snapshot" triggers a UITableView reload which causes the tableView to scroll back to top (in my case), this is not the behaviour with the collectionView in the demo app. Is there any way to avoid this ?

Thank you.

Weird. But regardless, a UITableView reload shouldn't change the content offset of your UITableView. Are you sure you're not doing anything that would trigger that? I'm using this library all over the place in Objective-C and and Swift and haven't seen that behavior. Is this reproducable on iOS10 only?

Ah my bad, it was my code that causes the reload, I'm closing the issue, Thank you !


[Backend] PATCH/PUT API for updating the isApproved field of Question & Answers stored in Db

Deliverables-Deliverables-

[ ] Create an API for updating the isApproved field of questions data [the api will resider under here]
[ ] Create an API for updating the isApproved field of answers data [The api will reside under here]

I would like to work on this issue

please go ahead @prafullapal
Happy Learning  :)

Any updates? @prafullapal

Can i work on this ?

Can i work on this ?

Sure, please go ahead

@Kajol-Kumari, i have made the PR for this issue, can you please review that ?


The use of the 'describe' verb is not valid.

There is no validation of verb names in Kubernetes so this won't cause
an error but it also won't do anything. The 'get' verb is enough for
getting the jobs and their status.
It will generate an error if the account creating the role has been
restricted in what it can create.
Description of the change
Removes invalid verb from kubernetes Role.
Benefits
This prevents an error when installing the helm chart with an account that is only allowed to 'get' Jobs.
Drawbacks
None known.
Additional information
Allowing the verb 'describe' has no use because it's not a valid verb anyway.

Looks good to me - could you update the Chart version in Chart.yaml?
@APIM-KiranVaddadi @ksaladi


⚠ Bofrost.de has degraded performance

In f391369, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 921 ms

Resolved: Bofrost.de performance has improved in 2c28884 after 7 minutes.


problem

I recently had MikTeX reinstalled since I was switched to Windows10, now some of my LaTeX files that previously worked stopped working, I get an error message
"Package pdftex.def Error: File `but.pdf' not found"
I tried to google this error, could not find it. None of the files that I designed start with but. Please help

Note this is a bug report tracker not a help forum (tex.stackexchange.com would be more suitable) but that means you have \includegraphics{but.pdf} but that file can not be found.

I understand that, the problem is that I am using slide files that follow a pattern including the statement  \paneloverlay{but.pdf} This file is supposed to be provided by the package graphics-def which is part of MikTeX.
In the past, this all worked, but now when I reinstalled MikTeX and WinEdt, for some reason, the system cannot find its own file but.pdf
I can probably find it and add it again, but the file should be already there, this part of the MikTeX installation.
To me, if something worked OK and stopped working in a new version without any warning, it is a bug.
Here is a standard beginning that I copies for all previous files that worked OK:
\documentclass[a4paper]{article}
\usepackage[latin1]{inputenc}
\usepackage{xspace,colortbl}
\usepackage[screen,panelright,code,sectionbreak,paneltoc,chocolate]{pdfscreen}
\usepackage{pause}
\begin{screen}
\margins{.35in}{.45in}{.35in}{.35in}
\screensize{5.25in}{7in}
\paneloverlay{but.pdf}
\def\pfill{\vskip6pt}
\definecolor{panelbackground}{rgb}{0.5,0.5,0.9}
\backgroundcolor{white}
\paneloverlayempty
\end{screen}
\pagedissolve{Wipe /D 0 /Dm /V /M /O}
\setlength{\parindent}{0cm}
\begin{document}
?

From: David Carlisle<EMAIL_ADDRESS>Sent: Tuesday, May 21, 2019 1:53 AM
To: latex3/graphics-def
Cc: Kreinovich, Vladik; Author
Subject: Re: [latex3/graphics-def] problem (#21)
Note this is a bug report tracker not a help forum (tex.stackexchange.com would be more suitable) but that means you have \includegraphics{but.pdf} but that file can not be found.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/latex3/graphics-def/issues/21?email_source=notifications&email_token=AMEBHI33KLWVPBWHDP3YFXLPWOTAHA5CNFSM4HOH3RHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV3CGJQ#issuecomment-494281510, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMEBHI2VG347LWNESPHKEBTPWOTAHANCNFSM4HOH3RHA.

FWIW, I find the following in TeXLive 2018.  Haven't checked TeXLive 2019.
/usr/local/texlive/2018/texmf-dist/tex/latex/pdfscreen/but.pdf

To me, if something worked OK and stopped working in a new version without any warning, it is a bug.

true, but then you need to report the problem to maintainer of the software. TeX/LaTeX is a huge universe and the LaTeX Project only maintains the core LaTeX format and and core packages but not the distributions, the engines, the editors the contributed packages etc. Granted it is not always easy to understand whom to address, but we aren't the right people as we can neither fix pdfscreen or change miktex etc.
By the way, on the Mac here with TL 2019 your sample find bug.pdf, so perhaps it is an issue with MikTeX, which we can't help with , sorry.
This is why we on the bugtracker for latex2e ask to always produce a running example  that shows the problem and that starts with
\RequirePackage{latexbug}

on the first line.
This would then have clearly indicated that your issue needs to be handled elsewhere.
Obviously for this sub-repository this guidence hasn't been set up (which should happen)

but.pdf is in my miktex and is found without problems. If you still have problems you should ask with a complete example e.g. a tex.stackexchange or in the miktex issue tracker.


[5.8] PHP 7.4 support for ServiceProvider

Resolve issue https://github.com/laravel/lumen-framework/issues/1008 for lumen running PHP7.4

Officially, Laravel 5.8 is not accepting bug fixes, but I'd advocate for this to be allowed through, to help people upgrade to PHP 7.4. Lesser changes have been merged into 5.7 and 5.8 before, after they were officially not accepting bug fixes.


[kitchen] fixing broken pipeline

What does this PR do?
Pins bundler everywhere necessary to ensure we don't bring in the new bundler 2.0.0 that requires an updated ruby.
Motivation
Broken kitchen tests.
Additional Notes
We should maybe look into updating ruby in our builders, I don't really see any strong reason not to do so.

Integrations failure unrelated - going to use my superpowers  💪 to merge this to unblock the kitchen tests.


fix syntax error for example

Python does not use 'new classname' to create an instance of an object.

Docs Build status updates of commit 4854077:
:clock10: Full build: the average full build time is 139 min(s) 39 sec(s), this is based on the last 332 build(s) for this repository.

Docs Build status updates of commit 4854077:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/storage-queue-readme.md
:warning:Warning
View (azure-python)
Details

docs-ref-services/storage-queue-readme.md

Line 57: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/azure/storage/common/storage-auth?toc=/azure/storage/queues/toc.json' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

👍 This is automatically synced from the azure-sdk-for-python repository. I am accepting this contribution, but it looks like the readme content has been updated to fix the issue you're resolving here. Next time we release storage-queue it should be updated to reflect it.


feat(route-rules): add support for both the path and params

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Wouldn’t it be beneficial to support both the path and query parameters in proxy and redirection route rules?
I may not fully understand the entire scope, so I’d greatly appreciate any clarification if I’m mistaken.
However, in my opinion, supporting both would be highly advantageous. Currently, I need to create a patch for each project to implement this functionality.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Can you please open an issue + reproduction of the unsupported case first? 🙏🏼


Bound refresh max rows to this.rows not this.lines

this.row is the size of rows in the viewport, this.lines is the buffer. It's
only possible to refresh 0 to this.rows - 1.
Fixes #86

This prevents an exception when calling refresh incorrectly.

Thanks a lot for this!

@Tyriar I think that there might be a chance that this PR made the focus/blur events go a little bit crazy. Could you please try setting a listener to each event and compare the event emission rate before and after the attached commit?


generate spritesheet per pixelRatio

Because of not resizing image based on provided pixelRatio, client is required to load oversize image which is not necessary.
I am willing to send a pull request to cover this for only 'compositor: gm'.
@selaux, please let me know, if this is something you would be willing to accept?

Hey. A bit late, but can you check wether the PR #33 accomplishes this?


Add files via upload

JetBrains Products for Learning - add NTSU to domain, thx!

@Khalil-NTST First stage of verification passed, official domain is ntsu.edu.tw as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Khalil-NTST Please provide us with an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course. Thank you.

@Khalil-NTST You might miss my previous comment. Please react.

@Khalil-NTST I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


[QUESTION] Job application in Laravel-excel chain function

Versions:
PHP version: 7.3
Laravel version: 6+
Package version: 3.1
Example:
Business Layer:
$excel_path = "product.xlsx";
$productImport = new productImport();
($productImport)->import(storage_path($excel_path), null, Excel::XLSX)
->allOnQueue('imports-excel')->chain([new productImportJob()]);

Import Class:
class productImport implements ToModel, WithBatchInserts,WithChunkReading,ShouldQueue
{
    use Importable;
    use Queueable, SerializesModels;

    /**
     * @param array $row
     *
     * @return productExcel | null
     */
    public function model(array $row)
    {
        return new productExcel([
            "StockNumber" => $row[0],
            "ProductName" => $row[1]
        ]);
    }

    public function batchSize(): int
    {
        return 1000;
    }

    public function chunkSize(): int
    {
        return 1000;
    }
}

Model:
class productExcel extends Model
{
    protected $fillable = [
        "StockNumber",
        "ProductName"];
}

Question: How do I throw job lines in the chain function?
This is the error I get: PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'databasename.product_excels' doesn't exist in.

Like the exception says, you have a database table that doesn't exist.
Closing due to age of ticket. If you need more help to understand why the table doesn't exist, stackoverflow or laracasts might be good fits.


fix:  the static resource loading exception encountered

When using a global proxy, the page cannot be accessed normally:

Changes:

remove unavailable CDN domains.
use the original domain name: apisix.apache.org

Since the CDN vendor has fixed this issue, this PR is closed.


fix(tsi): compact inactive log files on startup

If there are multiple log files, inactive ones would not be compacted
and some memory space would be occupied.
This may happen when option max-index-log-file-size is adjusted, or the
program crashes during a tsi compaction.

@timhallinflux @dgnorton  can you get this PR processed?

I've opened a more complete overhaul of this code at https://github.com/influxdata/influxdb/pull/21943/files


Add Dark Mode Toggle

Add a button to toggle between light and dark mode on the website, updating the CSS accordingly.

I'd like to work under this issue, thanks.

Thank you for your interest in contributing to this issue! You're welcome to work on it. Please feel free to fork the repository and create a pull request once you've implemented the dark mode toggle.
Looking forward to your contribution! 😊


OAuth integration and code refactor

Refactor the existing code in shiny-server-develop to integrate changes from shiny-server-oauth (without losing git commit history).
The refactor strategies are described in issue #145.
To test the code, I would recommend spinning up an instance of the Data Curator App on the Shiny Server and also installing the schematic backend via pip install schematicpy and also following the updated documentation here.

Added an additional commit to modify the default path to the config.yml file (root of the Data Curator App installation) which will be used by schematic.

Added commit to remove unnecessary call to getModelManifest() which will save some time when metadata_model.py is sourced in server.R.

@rrchai @milen-sage: I've tested this code, and I believe it's ready to be merged. You can take a look at the deployed app, HTAN-oauth which has the code from this branch.

Great! @sujaypatil96 I'll test the app on the HTAN oauth link above. I'll also send the link to one of our collaborators to see if oauth works as expected for users on Synapse teams other than DCC (it should; but it's always good to check).
@rrchai if could you test installing and setting up the refactored app following the updated instructions. @sujaypatil96 I think you intended to link the updated docs (see 'here' in your original comment?

Thanks @milen-sage. That's great, the more people there are testing it, the better.
Also, I was thinking I'd save the docs update for a separate PR. I think this PR has grown big enough and want to avoid further cluttering it with more commits. Do you think that's a good idea? I can update the docs here if that works better too. In the meantime, I can help @rrchai with the setup and everything as needed.
I'll followup with a documentation update PR on this branch.

Yes separate PR for the docs would be great!

I've added a couple of commits to address what we discussed above @rrchai. Let me know if this is enough.

Merged this PR since all conversations were marked as "resolved". We can revert if something else comes up.


keyboard doesn't work

this is the keyboard example:
11-11 19:57:36.985 18850 18850 W InputMethodManager: Ignoring showSoftInput() as view=android.widget.EditText{20a16a4 VFED..CL. ......ID 0,0-1080,119 aid=1073741842} is not served.

using "imm.showSoftInput(editText, InputMethodManager.SHOW_IMPLICIT)" instead of "imm.toggleSoftInput(InputMethodManager.SHOW_FORCED, 0)" at line VKeyboardActivity:113 works somehow.

Thanks for the bug report. It used to work so maybe something has changed somewhere in Android-land 🤔

no problem. also there's this: error: unknown type 'os.NativeActivity' in android.c.v when building it as a desktop app. should i create a separate issue?

I temporarily fixed it by making desktop and android clients as separate modules to avoid importing jni while using desktop client

no problem. also there's this: error: unknown type 'os.NativeActivity' in android.c.v when building it as a desktop app. should i create a separate issue?

Thanks. Yes, that would be a big help

Right it turns out it has been deprecated/removed from API level 33:

using "imm.showSoftInput(editText, InputMethodManager.SHOW_IMPLICIT)" instead of "imm.toggleSoftInput(InputMethodManager.SHOW_FORCED, 0)" at line VKeyboardActivity:113 works somehow.

Thanks for hinting this. It should work on master branch now 👍🏻


Disable leader election

Describe the feature request
I am working on running k8s on edge device. One of the goal is to reduce writes to etcd as much as I can. In return, this will also reduce disk write. I noticed that the following PUT are going into etcd.
(Captured over 8 minutes)
73 /registry/configmaps/app-ingress/istio-namespace-controller-election
73 /registry/configmaps/app-ingress/istio-leader
73 /registry/configmaps/app-ingress/istio-gateway-status-leader
73 /registry/configmaps/app-ingress/istio-gateway-deployment-leader

I would like to disable those leader election leases writes in istio. Currently, it seems like those parameter are hard coded.
https://github.com/istio/istio/blob/53eef240ecb2c81eca120db55ba14dbe14718b75/pilot/pkg/leaderelection/leaderelection.go#L207
I would like to add a feature flag to disable the leader election.
Describe alternatives you've considered
Instead of disable the leader election. We can also make the leader-election parameters tunable. However, this seems more work than just to disable leader election for my use-case
Affected product area (please put an X in all that apply)
[ ] Docs
[ ] Installation
[ ] Networking
[x] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[x] Multi Control Plane
Additional context

Do you want to disable leader election or customize interval?
If it is disable leader election, there can be a race when multiple istiod running

I want to disable the leader election.
If it is disable leader election, there can be a race when multiple istiod running
does istiod need to be run on all nodes? And what's the race condition?
kube-scheduler has this feature.
--leader-elect     Default: true
--
  | Start a leader election client and gain leadership before executing the main loop. Enable this when running replicated components for high availability.

https://kubernetes.io/docs/reference/command-line-tools-reference/kube-scheduler

IIRC, kubescheduler has this option to allow disable leader election when one instance run

IIRC, kubescheduler has this option to allow disable leader election when one instance run
if I have a multi-node setup, does istiod need to run on all nodes? I know I can still run one stance of kubescheduler in mulit-node setup if i don't care about HA.

Istiod does not need to run on all nodes.  I think we can provide such option.

@hzxuzhonghu sounds good. I can put a PR for this since this should be a quite simple change.

Awesome, thank you

@hzxuzhonghu I am digging into the code. is this the place that i need to disable the leader-election? https://github.com/istio/istio/blob/master/pilot/pkg/serviceregistry/kube/controller/multicluster.go#L259

Also in pilot/pkg/bootstrap/configcontroller.go


RESTMapper to resolve unqualified resource names (e.g. ingress) to fully qualified ones

Is your feature request related to a problem? Please describe.
With addition of  --resources=ingress to --args in syncer.yaml leads to workloadcluster deployment not being True and syncer logs report errors as below
E0513 15:47:25.098968       1 syncer.go:127] Failed to retrieve GVRs from kcp: The following resource types were requested to be synced, but were not found in the KCP logical cluster: [ingress]
Describe the solution you'd like
Would like RESTMapper to resolve unqualified resource names to qualified ones .
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi, I'd like to try working on this issue. Is this still relevant?

Hi @mra-ruiz I think so. Although at this week's community call, we discussed #2718, which is semi-related. You might want to look at them together. And/or possibly chat with @davidfestal and @jmprusi. Thanks!

In fact I tend to think that this issue is somewhat obsolete. #2718 is the right one indeed.
Now the workload sync CLI command produces some YAML that includes RBAC rules for additional resources to be applied in the physical cluster. And these RBAC rules should use the full GR. So it seems that the place where we should use the RestMapper would be inside the workload sync command, in order to grab the complete GroupResource to add in the RBAC rules of the generated YAML.
However since the workload sync command will not use the discovery of the target physical cluster to feed the RestMapper, I wonder if this all really makes sense.
Should we simply require the whole resource.group data in the whole flow, and thus remove the use of the RestMapper in the APIImporter ?
cc @ncdc

@davidfestal I'm not fully up to speed on the full details - what is the exact issue?

If you pass route in the --resources arg of the workload sync command, the routes.route.openshift.io will effectively be imported by the Syncer APIImporter, because this component uses a RestMapper, and will have mapped the route partial singular name to the complete routes.route.openshift.io name.
However the Syncer will not be able to sync, since the Syncer service account will not have the right permissions, on the downstream cluster, for routes.route.openshift.io.
The reason for this is that the worklod sync command will create RBAC (in the generated yaml) with the short name route and not with the full GR-based name.
Which is why you'll end up with a SyncerAuthorized condition to false on the SyncTarget.

Thanks. It sounds like a possible solution is to have the workload sync command use a RESTMapper against kcp. (I think it's ok that it's kcp and not the workload cluster ... right?)

Well, how is it going to work for routes for example, or any other resources that, at the time we run the workload sync command, isn't known on the KCP side ?
It's precisely the goal of the --resources argument to allow importing additional (initially unknown to KCP) resources from the physical cluster to the KCP workspace.

Ah, I was thinking they'd already be in kcp 😄. So, short-term, it's probably easiest to require fully qualified resource+group.

/transfer-issue contrib-tmc


Improves the R_omega function in jax_util

The JIT compiler needs to unroll the entire for loop in order to compile it. By replacing the standard for loop with jax.lax.fori_loop, the compiler no longer needs to unroll the entire loop for compilation, saving a lot of time.

Thanks a lot @ahuang314 !

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

7 of 8 (87.5%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 23.607%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

herculens/Util/jax_util.py
7
8
87.5%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
1521

Relevant Lines:
6443

💛  - Coveralls


StorageCircuit - Error when key doesn't exist

While generating storage proof for a non-existent account at a particular block height or on a non-existent/null storage slot on an account, the verification fails:
End:     Create proof ..............................................................69.599s
thread '<unnamed>' panicked at 'assertion failed: {\n
    let mut transcript_read =\n
        PoseidonTranscript::<NativeLoader,\n
                &[u8]>::from_spec(&proof[..], POSEIDON_SPEC.clone());\n
    VerificationStrategy::<_,\n
            V>::finalize(verify_proof::<_, V, _, _,\n
                    _>(params.verifier_params(), pk.get_vk(),\n
                AccumulatorStrategy::new(params.verifier_params()),\n
                &[instances.as_slice()], &mut transcript_read).unwrap())\n
}', /Users/sohamzemse/Workspace/cloned-repos/axiom-snark-verifier/snark-verifier-sdk/src/halo2.rs:125:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Expected behavior: it should still generate proof for the non-existent slots since they would have an implied value of 0.

fixed in https://github.com/axiom-crypto/axiom-eth/pull/24


Ladder bot support for running games

Disruptor bot wasn't really supposed to be part of this PR...


Question: what if one is running a different antivirus?

Hello. I'd like to use the program to clear out any Defender-cruft. However, I read that the script restarts Defender - and I use a second-party anti-virus and have Defender disabled (or as disabled as it can be without extreme measures).
I could be brave and just try the script. Yet, even if so doing would cause no problems on my system, you might want to put something in the README about this situation.
Thanks for your time.

I would expect that if a different AV is installed, the script will not cause Defender to start. Note that the stop/start is done by     Set-MpPreference -DisableRealtimeMonitoring 1/0 as opposed to starting an exe. When I get a chance, I'll test it to be sure.

Thanks. I had not actually looked at the script. Just now I did. I know little of Windows's guts so the script did not mean that much to me, but I decided to run it anyway. I did so via a Powershell prompt. The script completed, without giving a visible error, and my third-party antivirus remains functional.


Multilingual Support

Just leaving this here because I can envision someone bringing this up eventually... and because it hasn't quite been discussed between all of us.
A super-comprehensive app would support multiple languages (as Baka-Tsuki supports multiple languages).
However I vote no on this.
Unless someone else volunteers to do everything related to multilingual support (models, translations, everything)... because it won't be me. >_____< This app is complex enough. I feel like we'd be decreasing the likelihood this will finish by adding this requirement.
Finishing something that's good in English is all I'm really hoping for right now. ;-;

Maybe implement localization with this, and following the model suggested here should allow different languages without any extra work.
Maybe have tags for Volumes that show it's status (wip, completed, unedited), and the language that it's in.

Localization usually addresses static content in an app (like the sidebar, navbar translations, etc).
We run into a more complicated modeling issue because we'll need to provide translations for series descriptions, volume titles... pretty much every field that an end-user can provide.
If we'd like to keep different translations of the same series related (link related series between languages the way it's currently done in Baka-Tsuki), a possible model to use is:

Series (not-language specific)
Series (language-specific): 1-to-many relationship to Series

However, even if we choose to model this way, we need to think about UX. In the case where any translator wishes to add a new series, they somehow must go through a select-box of Series (not-language-specific) to ensure that they add it to the right place if it already exists in a different language (do we expect them to be able to read the other language?).
It's a complicated user experience. Could probably bypass this end-user UX complexity if we delegated the responsibility of figuring out whether a translation project exists in another language to mods.
Another model approach is to simply put a field on series. However, with this approach, it wouldn't be possible to link associated series (of different languages) together... at least, the normal way. Is it possible to do a self-to-self(many) relationship? lol We'd be modeling a network.

I like the last approach. A record for a language is good since the series pages will be managed independently. Instead of more weird relationships, Series could just have an identifier field that's constant across different languages.

Agreed. Though actually, having an identifier field that's constant across different languages is actually the same as the first approach (it's the same as having a foreign key one-to-many relationship to a table that has series (not-language-specific)).
The important difference though is that if you make this field optional, users/translators aren't forced to go through complicated UX when creating a new series, and somebody else can add it later. That solves the UX problem.
I guess you've changed my opinion about multilingual support. I'd be willing to do it. However, you have to ask at what is the priority of this requirement over everything else; and we have figure out what is a good time to do it.

As for UI localisation; this'll be done through the settings that have yet to be added. Should be a simple implementation


TensorFlow: Unsupported attribute value type

When trying to load a .pb I am getting the following error:

I know that this model opens properly with release v.5.6.9 so maybe you can re-create that release? That would be great

@goelbenj can you share a model file to reproduce the issue?

The change in behavior come from this commit (message get changed latter)
Where unknow case are no more ignored, but generate errors.
Can this be reverted?

@goelbenj what is the root cause for the invalid attribute? Please share a repro file and how it was created.

I am also seeing the same issue. I have attached a sample graph (I had to zip it since GitHub does not support the pb format).
graph.pb.zip


JavaScript SDK does not work: The ScriptProcessorNode is deprecated. Use AudioWorkletNode instead.

The JS SDK does not work as the ScriptProcessorNode is deprecated on the [https://aka.ms/csspeech/jsbrowserpackageraw](https://aka.ms/csspeech/jsbrowserpackageraw) when it calls return context.createScriptProcessor(bufferSize, 1, 1);

@YounKimAtAri What code are you using to produce this issue? Could you describe "does not work" in more detail? Are you not receiving results when called JS SDK APIs?

@glharper
I found out why it happened. There is a still JS error message, but not it's working.
Thank you.


XIVY-13167 use dev mode flag

@ivy-lli  & @ivy-edp  FYI you must set -Ddev.mode=true in your eclipse debug configurations to make the lsps work.

I normally test against my dev designer, but good to know 👍


Using Settlement Date for Superficial losses

Looks like you use the daily currency exchange rate. So I'm guessing you're entering the transaction date instead of settlement date however you're supposed to use the settlement date instead of the transaction date for calculating superficial losses. I think this can change the rate if you buy a share on Friday. Then settlement is usually after two business days so Tuesday so you effectively moved your transaction ahead can get even more conflicts if public holiday.
Not sure the best way to fix this. Maybe add 2 days + holidays when calculating superficial losses?
https://www.adjustedcostbase.ca/blog/what-is-the-superficial-loss-rule/
https://www.adjustedcostbase.ca/blog/understanding-trade-dates-and-settlement-dates/
https://help.wealthsimple.com/hc/en-ca/articles/360058451913-US-and-Canadian-exchange-holidays

You are right in that the exchange rate should be applied on the settlement date instead of the transaction date. But why not just enter the settlement date instead of the transaction date in the CSV input file for each row, and then it will properly get the exchange rate for the settlement day?

No that is not what I'm saying. The exchange rate is supposed be from the transaction date:
https://www.adjustedcostbase.ca/blog/calculating-adjusted-cost-base-with-foreign-currency-transactions/

So what I was saying is superficial losses are supposed to use the settlement date and currency exchange rates should use the transaction date. Thus, you need both dates,

Ah I see what you mean now. We're going off his opinion for this, and as he says the CRA doesn't have a firm stance on this. I've read some comments from this site from others who suggested that it's fine to just use the settlement date for both exchange rate and superficial loss purposes. The author's own tool, if you pay for premium, only asks for a single date to be inputted and then automatically finds the exchange rate for the given day. I'm not a tax expert so I'm not sure what the correct approach is here, but in my own case I use the settlement day for everything. For USD <--> CAD transactions, it really shouldn't be a big deal as the currency fluctuates very marginally on a day-to-day basis.
I think if we want to do this as he suggests, I would need to expand the CSV file to take in an optional transaction date and required settlement date. Then we could do exchange rate lookups on the transaction date, and do superficial loss calculations using the settlement dates, if both dates are provided, or otherwise do both operations on just the settlement date if that one date is only provided. This allows the tool to support both approaches.
I don't like the idea of trying to calculate what the settlement date is. It's not consistent, and trying to account for holidays across the world is not a trivial endeavor. I'm also not sure if brokerages have standardized using T+2 for the settlement date. Certainly before 2017, this was not the case. I think it's simpler if the user provides that information without having to do guesswork.

Yea not sure about T+2 settlement but from the ones I looked at they said T+2 (ex. QuestTrade and Wealthsimple and Robinhood). But anyway I agree would be good to have the user provide both if possible.
Ah I didn't know the tool does it automatically for premium. I only looked at normal where you had to enter in the exchange rate you wanted to use. Anyway I think what you said about exchange rate not differing too much makes sense especially since apparently another valid solution is to calculate the exchange is to use the avg for year if I recall correctly.


OpenSSF Best Practices Badge for devfile/devfile-web

Which area/kind this issue is related to?

/area registry-viewer
Issue Description

This task aims to complete the OpenSSF Best Practices Badge as part of a greater effort to improve our adherence to best practices across all repositories. In order to complete this task the badge should be completed to 100%. You can find the badge on the OpenSSF Best Practices site and it clearly outlines the necessary tasks that must be completed.
At the bottom of the link there should be an edit button available to anyone who has commit permissions to the repository.
Acceptance Criteria

[x] Badge completed to 100%

Badge entry here: https://www.bestpractices.dev/en/projects/8255


Missing method setConditonClasses in \yii\db\QueryBuilder

What steps will reproduce the problem?
Try to configure query builder as mentioned here with custom codition
What is the expected result?
To don't throw Unknown Property – yii\base\UnknownPropertyException
Setting unknown property: yii\db\mssql\QueryBuilder::conditionClasses
What do you get instead?
throw Unknown Property – yii\base\UnknownPropertyException
Setting unknown property: yii\db\mssql\QueryBuilder::conditionClasses
Additional info
The method setConditonClasses in \yii\db\QueryBuilder is missing.
The property $conditionClasses is protected but it could be public, because the init() method in QueryBuilder overrides the initial value (see line 125: $this->conditionClasses = array_merge($this->defaultConditionClasses(), $this->conditionClasses);)

Q
A

Yii version
2.0.14

PHP version
7.0

Operating system
*

Thank you for the report


Adding Vizlab content.css

Note: requires vizlab #349. I should have requested that you increment the patch version number in vizlab before I merged that PR, but I didn't. I'll put up another PR in vizlab with a tiny fix plus that increment, and then let's add that new requirement to your viz.yaml here. Gimme a few minutes...

oh, fudge. sorry, @mwernimont , i just introduced merge conflicts by merging @lindsaycarr 's PR. i guess the resolution had to get dumped on one of you...let me & lindsay know if you have any troubles

OK, just merged https://github.com/USGS-VIZLAB/vizlab/pull/351, which declares vizlab version 0.3.2.
Now we just need to edit the required-packages in viz.yaml so that everybody else gets the message. I think this should work:
  required-packages:
    vizlab:
      repo: github
      version: 0.3.2
      name: USGS-VIZLAB/vizlab
      ref: 6bcdf08319d00f73b0b54ecfb73d1de00837a2de


chore: const values supported in validate

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Post installation page for apps

A lot of apps have steps that need to be taken after installation. For example, the stale plugin requires a .config/stale.yml, and the DCO plugin optionally suggests required statuses.
It might be nice if all of the listed apps had a post-install page (e.g. https://probot.github.io/apps/stale/install) with info on what to do after installing the app.

Let me try and implement this.


500 [VITE_ERROR] /pages/[...slug].vue?macro=true

The build works fine, when  I go to http://localhost get this:
[vite-node] [plugin:vite:vue] [VITE_ERROR] /pages/[...slug].vue?macro=true

<template>
    <main class="max-w-none mx-auto sm:w-3/5 prose prose-reader-light prose-reader-base prose-reader-compact px-4 pt-10 pb-20 sm:px-6">
        <ContentDoc />
    </main>
</template>

<script setup lang="ts">
definePageMeta({
  middleware: ["refresh"],
});
</script>

at /pages/[...slug].vue?macro=true

Hi @franksheppard Does this also happen for you outside of Docker (running frontend at http://localhost:3000 direct)? And did you update Vite at all? I'm not having success duplicating this.

Should be fixed in latest release.


Blank Option Page

Options/Configuration page is displaying blank.
In recent update version(2.0.55) for some users google payment API is returning error and because of that options page is showing blank. We regret for inconvenience caused.
We have figured out issue and provided fix for same in version 2.0.56


[feature request] Update control to provide quick toggle for the popular chart types

Description
Spinning off from #6235.
@mistercrunch :

"Side mission would be for the control to offer a quick-toggle for the popular chart types that would not require opening the modal.
Here's a horrible sketch of what I'm thinking, where the last button ... opens the modal :"

It would bring some common interactions to the forefront and add a nice dash of color to the explore view.

@elibrumbaugh ^