Fixing issues #13

Summary

This pull request fixes issue #13
Implementation details

Removed "pgrep docker" as when testing in ubuntu and I killed Docker daemon which left child processes hanginhg. When I ran "pgrep docker", it gave me exit code 0 which would result in script going to hung state. Instead modified it to ps -ef |grep docker | grep -v grep

There were cases where Docker daemon was going to hung state due to which docker commands would not yield anything which also caused log collector script to go to hung state.
Added timeout command to mitigate this issue.

The script used to exit abruptly with out making a tar file if Docker daemon or ECS agent wasn't running. Fixing that issue by writing it to a file and allowing the script to tar up rest of the content.

Testing

[Yes] Works properly on Amazon Linux
[Yes] Works properly on RHEL 7
[Yes] Works properly on Debian 8
[Yes] Works properly on Ubuntu 14.04

New tests cover the changes:
Licensing

This contribution is under the terms of the Apache 2.0 License: Yes

@jahkeup I have modified according to your comments. Please take a look.


docs: Update docs to make isvc focal

Motivation
The InferenceService CRD has evolved enough to become the primary interface for interacting with ModelMesh.
The documentation should reflect that.
Modifications
Documentation was adjusted to make the InferenceService CRD focal as opposed to the previous Predictor CRD.
Examples and snippets were added for InferenceServices.
Result
Users will learn and become more familiar with deploying on ModelMesh using the KServe InferenceService.

Thanks, @chinhuang007 for the thorough review!


Automated cherry pick of #2079: fix volcano podgroup update issue
#2130: Refine the integration tests for the immutable PyTorchJob

Cherry pick of #2079 #2130 on v1.8-branch.
#2079: fix volcano podgroup update issue
#2130: Refine the integration tests for the immutable PyTorchJob
I manually squashed commits and modified signs since @ckyuto signed to DCO and CLA with different email addresses.

Commit sha: 18844bd, Author: Weiyu Yen, Committer: Yuki Iwai; Expected "Weiyu Yen<EMAIL_ADDRESS>but got "Weiyu Yen<EMAIL_ADDRESS>
https://github.com/kubeflow/training-operator/pull/2134/checks?check_run_id=25949083128

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 35.414%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controller.v1/common/job.go
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/controller.v1/mpi/mpijob.go
1
91.06%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
4379

Relevant Lines:
12365

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 35.414%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controller.v1/common/job.go
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/controller.v1/mpi/mpijob.go
1
91.06%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
4379

Relevant Lines:
12365

💛  - Coveralls


bertieb: Watch royalurls.com

bertieb requests the watch of the watch_keyword royalurls\.com. See the MS search here and the Stack Exchange search here.

royalurls\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Start new assessment page - minor styling issues

The styling looks different to the html mock up (http://upcloseandpixelated.com/ODMAT/pages/assessment-overview.html)

[ ] 'Continue assessment' should be 'Start assessment' if the use has not started answering any questions

When the user started using it, then the button should say 'Continue assessment'

[ ] New assessment - it's not obvious that you can change the name. We need to make this more prominent and make it into a button (use the same button style as 'Begin' but in Orange and appear adjacent to the heading "New assessment"

[ ] Background information - again, this is not obvious. Make 'Edit' a button (again in Orange), to appear adjacent to the heading "Background information"

[x] Dimension should be called 'Theme'

[x] Icon (i) does not link to anything. It should link, or show a pop up help text

[ ] Colouring used on tick is too strong/bold, it needs to be fainter as per mock-up

[ ] The status 'Edit' should be orange colour, not grey

[ ] Font for the texts to display the 'Background information' - this should be in the smaller font

[ ] Begin should be 'Start' to be consistent with other phrases we have used so far across the site

Ticked off a couple of these as I've addressed them in other issues

Also, the colouring on the tick is the same green as on the mockup?


Hex numbers are not recognized

Describe the bug

Hex numbers like 0xffffff are not recognized in typescript file, but the same format in strings can be recognized correctly.
The hex number format can be used in some of the WebGL renderer like PIXI.js.
Expected Behavior

Environment

WebStorm 2021.2.1
Build #WS-212.5080.54, built on August 23, 2021
Runtime version: 11.0.11+9-b1504.16 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 1772M
Cores: 12
Non-Bundled Plugins: com.mallowigi.colorHighlighter (8.0)

Indeed, this is because I'm parsing only strings in regular languages, not numerals. Only in Java/Kotlin/Scala...
Too bad I just released a new version now that I've been working on for a while, so this will have to wait

I'd love to see this feature 👍

This doesn't work on C++ in Clion, I'm on version 12.0.0 and Clion 2022.2 on Ubuntu 22.04.

same with version 14.0.0
Doesn't work on C++ in Clion
[x] Detect hex color
checked in settings, but only strings highlighted as in comment above


Comet new version has been released Comet v2024.01.0

Description of feature
A new version of comet has been released Comet v2024.01.0 https://github.com/UWPR/Comet/releases. The new version includes some major changes, including the way to define the precursor tolerances and total number of modifications. We should do the following:

[ ] Generate the corresponding conda package @ypriverol
[ ] Update the OpenMS Adapter @ypriverol @timosachsenberg
[ ] Update quantms @daichengxin @ypriverol

What is the main motivation to make this high-priority?
Quickly skimmed over the changelog and wasn't sure if it is "nice to have" or really needed now.

I will remove the priority. I did see that they fix a couple of bugs with the scores of some low-quality PSMs. But you are probably rigth, is not high priority.


About F-score

Hi, I have two problems about it. Can you help me?
When I input 'python main.py -d citeseer -l 20 ', It says 'UndefinedMetricWarning: F-score is ill-defined and being set to 0.0 in labels with no predicted samples'. How to deal with it?
And the result is ' epoch:12 acc_max: 0.7380 f1_max: 0.6999 '. It is little higher than the result in your paper. I don't know whether it is associated with the warning above.

Hi, Thanks for your attention to our work. In my opinion, this "warning" is different from "error" and it doesn't affect our results. In our experiments, we just ignore this warning and random run AM-GCN five times to get the final results.
If you don't want to see this warning, you can insert the following codes:
import warnings
warnings.filterwarnings("ignore")

Thanks for your reply.


[Docker] Add integration testing for the docker container

This PR adds a cli_test.go file in cmd/tdex which runs a set of cli commands inside tdexd containers
TL;DR

Add a new process in MakeFile: containertest
Add CLI tests in cli_test.go: test for init and unlock the wallet + create, list and open a market

I don't test all the methods because it could be a bit overkill in my opinion: we already test all of the CLI methods in others tests files. Please tell me if it is a good decision.
PS: Note that multiple tdexd containers are resistant to race errors --> could be the solution at the former badger concurrency issue.
@sekulicd @tiero @altafan please review

Guys, sorry for this soo late reply, but don't know why, i'm not able to follow this discussion from my laptop.. always getting a 500 error.
Not ideal to continue this from mobile, but however 😅...
With my latest PR, basically I tried to follow the first 3 points of the list given by dusan, which I essentially agree with.
It would be interesting to know if, using test suits, it would be possible to have isolated envs such that every suit could be ran into a t.Run block.
Running unit tests within t.Run blocks would be nice to speed up their execution. But since some of them, in particular those of the app layer, make use of mocked env vars this is currently not achievable because cause of race condition.

One thing that i want to add to complete point 5 is that every state should be statically generated with a yml (also json file could be an option).
At the moment, there are mocks that are created "dynamically", in the sense they are based on hardcoded data that are used to create the actual mocks. Sometimes this is accomplished by using the same service that should be tested!
Ideally, instead, every unit test should start with a mocked state read from yml, json or whatever, execute a function, check the output and eventually destroy the state.

I agree with @sekulicd
I would suggest to read also this post regarding testing https://threedots.tech/post/microservices-test-architecture and try to re-thing testing architecture from ground up

Closing this in favor of #301


[BUG] input_parameters does not utilize default

The deno-slack versions
"deno-slack-hub/"<EMAIL_ADDRESS>"deno-slack-sdk/"<EMAIL_ADDRESS>"deno-slack-api/"<EMAIL_ADDRESS>
Deno runtime version
deno 1.44.4 (release, x86_64-unknown-linux-gnu)
v8 <IP_ADDRESS>
typescript 5.4.5

OS info
#22~22.04.1-Ubuntu SMP Tue Apr 30 16:08:18 UTC 2024
Describe the bug
The default for parameters in input_parameters is not used
Steps to reproduce
#workflows/example.ts
export const ExampleWorkflow = DefineWorkflow({
  ...
  input_parameters: {
    properties: {
      duration: {
        type: Schema.types.integer,
        default: 30,
      },
    },
  },
});
ExampleWorkflow.addStep(
  Schema.slack.functions.SendMessage,
  {
    channel_id: <Channel>,
    message: `${ExampleWorkflow.inputs.duration}`,
  }
);

Expected result
Outputs message: "30"
Actual result
parameter_validation_failed - Null value for non-nullable parameter `message`

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Yup, seems like a legit problem. I will raise this internally.

Hey @filmaj - any updates on this? I'm also running into this problem.

Hi @ob-uk , FYI I am no longer at Slack, so I cannot help you


Face Verification Flow - Event improvements

Motivation
Right now I am saving the Face Verification events from https://goodid.gooddollar.org/  in a separate Google Analytics.
https://analytics.google.com/analytics/web/#/p415617260/reports/intelligenthome
The goal of this is to put all FV events ( from the wallet, gooddaap, opera and other partners) in the same place.
But, inside this GA I can't say what is the source to analyze the FV of each product, example:

The opera FV converts more than the GoodDaap FV or valora FV?

To achieve this I want to add an extra parameter in the FV events OR push forward the querystring parameter.
Details
From the partners using gooddapp, all of them are using the parameter partner  in the URL:
https://gooddapp.org/?partner=valora
If we can push this parameter forward to https://goodid.gooddollar.org/ example  https://goodid.gooddollar.org/?partner=valora the problem will be addressed.
Other option is add the parameter product to all FV events, who will show the source of that user ( wallet, gooddapp, opera, valora, etc) and I can use to filter.
The first option seems the better.

@johnsmith-gooddollar @sirpy
for specifically partners that come from gooddapp, the best way would to do the paramater passing down as @drptiago suggests right?

@decentralauren @patpedrosa
maybe this should be added to the good-id epic

@L03TJ3 investigate if this is possible now with Reown / updates in new picker modal.


Don't show mnemonics in non-English packs when disabled

Issue Type: Bug
mnemonics are still shown when disabled in non-English packs.
VS Code version: code-oss-dev 1.27.0 (Commit unknown, Date unknown)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.92GB (19.72GB free)

Process Argv
C:\Users\penlv\code\vscode.build\electron\Code - OSS.exe .

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

vscode-pull-request-github
Mic
0.0.13

fixed by https://github.com/Microsoft/vscode/commit/2c9408fecfc0507e78e1ac008e220d433cfef295

To clarify, in language packs like simplified Chinese, mnemonics are shown as a suffix to the Chinese characters in parentheses. e.g.  (F)
The fix is to hide the parentheses and F when mnemonics are disabled. To verify, you can use the old menubar in Windows/Linux as the correct template for what the new menubar should do.


Error in wiki on What's New page

https://github.com/Microsoft/TypeScript/wiki/What's-new-in-TypeScript#computed-properties has the following code example:
function makeNode(name: string, initialNeighbor: Node): Node {
    return {
        name: name,
        neighbors: {
            [initialNeighbor.name] = initialNeighbor
        }
    }
}

On line [initialNeighbor.name] = initialNeighbor it should  be : instead of =

Thanks for the heads up - in the future, if you see a problem with our wiki you can send a pull request yourself to https://github.com/Microsoft/TypeScript-wiki.


screen reader can't read the application

OS:
Windows 10 1709
GPMDP Version:
4.5.
Issue Descriptions:
The free and open source screen reader, NVDA, says, "unknown" whenever I try and run the program. Even with Flash uninstalled, using the HTML 5, it can'r read any part of the program.
Steps to Reproduce:
Download NVDA.
Run NVDA.
Run the program.
NVDA can't see any of the buttons or the controls. I'd be more than happy to do accessibility testing and writing the accessibility section of the wiki when it gets to that point.

Surely this could be an issue with NVDA?


[SPARK-LLAP-157] Use CREATE EXTERNAL TABLE to check permission

What changes were proposed in this pull request?
Cherry-pick from SPARK-LLAP-157 for branch-2.1
How was this patch tested?
Post the result soon.

@dongjoon-hyun @bikassaha  this is the follow up commit for spark-llap-157 for branch-2.2.

Thank you. Can we see the test result on branch-2.2, too?

@dongjoon-hyun  sure, I am testing it now.

Hi, @merlintang .
Could you update the PR description of this PR with test result?
I need that to merge this.

Hi @dongjoon-hyun
I have some issues for testing this, I will update you soon.
thanks

Yep. No problem.

Hi @dongjoon-hyun  my config issue is fixed, and I would post the results here soon.

BTW, the content is the same, but is this patch is not made by cherry-pick because the author is different. :)

I updated the PR description a little bit by reusing the real description of the patch.

Yes, since there are conflicts there. Sorry, I would add your name there.
On Tue, Aug 8, 2017 at 2:43 PM, Dongjoon Hyun<EMAIL_ADDRESS>wrote:

I updated the PR description a little bit by reusing the real description
of the patch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hortonworks-spark/spark-llap/pull/162#issuecomment-321090495,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABXY-Qo68Go5j3lTx7o9dbDzZPwJGdS1ks5sWNaFgaJpZM4OwBtn
.

No, it's okay.
What I meant is that is not your id, too :)
You can use your id.

@dongjoon-hyun  the test failures are just for the hostname inconsistency. I have updated it locally. Then, the test passed. please merge it. thanks

+1, LGTM.

It merged into branch-2.2 and master.


How can I get all the highlights from Highlight object ?

Currently I am able to pull the highlights object which is returning.
id=highlight:17848922861142122, latest_reel_media=1614946504, user=Profile(xxx....), title=👼Babies, media_count=90
As of now this is my code:
List<Highlight> highlights= new HighlightsUserTrayRequest(pkId).execute(client).join().getTray();
How can I retrieve all the posts from this highlight object ?

Check the pull request #571 .
Use FeedReelsMediaRequest and FeedReelsMediaResponse for getting each Reels media from the highlight by providing highlight id Ex: highlight:17848922861142122
You can pass multiple highlight ids.


newest human and mice genome annotations

Hi, I have not been able to create a new annotation file in higlass, using the latest human annotation (gencode.v42.primary_assembly.annotation.gtf). The human gene annotation on the higlass server is not the newest. The Refseq file from UCSC, necessary to create the newest annotation in higlass, does not exist (also outdated). We did chat on slack about this issue before and we tried the script you suggested (which uses a GFF3 file to generate: i. Chromosome sizes file  and ii. Bed file - which is used to generate the final gene annotation file for higlass). However, The new scripts do not take into account all the possible annotations and conditions in the GFF file which result in multiple entries per chromosome in the chrom.sizes file as well as for a chromosome labelled as “Unknown”.
The unknown chromosome entries are being added due to the annotations in the GFF file provided by NCBI. If we use these set of files to generate the gene-annotation file, the generation is successful but higlass doesn’t load the file and errors out due to multiple entries per chromosome.
We don't know how to fix this. Do you have suggestions? - or any chance you can add the latest human genome annotations on the server? -
same situation with the mouse genome. mm10 is the latest and not m39. Although, I have not tried yet creating the latter (not sure if the refseq is updated in UCSC).
Thanks!
PS: I lost my access to slack_higlass with my previous email. Any chance I can be added back with<EMAIL_ADDRESS>

The current invite link shouldn't have restrictions, but I sent one to your email anyway.


Feature Request: Support SES metrics

We are using AWS SES (Simple email service) and would like to extra its metrics.
It does not have any additional dimensions as far as I know and does not have tags as well.

Was already merged and released


Validation, Alert message, MinDate, Add footer, Other code improvements

Moved code and functions inside to the $(document).ready(function () { ... });
Validation for destination and dateRange;
Add function to show and hide the Alert message;
MinDate option of DatePicker does not allow selecting past dates;
Add footer;
Other code improvements;

Please ensure that you do not modify the code's functionality before pushing. We need to upgrade the changes already made without changing the code. Please stick to the existing points from the project, don't modify the code without discussing it with the team, and close an issue. This is essential for the development process.


framework: Readded nc.exe as a dependency

Readding nc.exe as a dependency. Also updated readme file with the 3d
party software needed in Tools folder

Change looks ok, but @adriansuhov can you post the screenshot of your Tools folder after a simple test executed in your working branch. Running test from master or other branches may not show nc.exe. Your working branch is the only place shows this binary.


chore(deps): update ci-deps

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
593e4fb -> 0a5cc32

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.10.6 -> v1.10.8

terraform-ibm-modules/common-pipeline-assets
action
minor
v1.15.0 -> v1.17.0

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.10.8
Compare Source
Bug Fixes

deps: update module github.com/gruntwork-io/terratest to v0.43.6 (#&#8203https://github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper/issues/576es/576)) (93c8433)

v1.10.7
Compare Source
Bug Fixes

deps: update gomod (#&#8203https://github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper/issues/572es/572)) (93b5090)

terraform-ibm-modules/common-pipeline-assets (terraform-ibm-modules/common-pipeline-assets)
v1.17.0
Compare Source
Features

CRA will now use SCC instances api instead of deprecated account api (#&#8203https://github.com/terraform-ibm-modules/common-pipeline-assets/issues/500es/500)) (4044cc1)
default CRA profile is now FSCloud 1.3.0 for v2 pipeline (#&#8203https://github.com/terraform-ibm-modules/common-pipeline-assets/issues/500es/500)) (4044cc1)

v1.16.1
Compare Source
Bug Fixes

fix bug in CRA config logic (#&#8203https://github.com/terraform-ibm-modules/common-pipeline-assets/issues/498es/498)) (49cd1df)

v1.16.0
Compare Source
Features

update default CRA SCC profile to FSCloud 1.3.0 (v1 pipeline only) (#&#8203https://github.com/terraform-ibm-modules/common-pipeline-assets/issues/494es/494)) (588ea85)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 2.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Code block direction in RTL interface

Describe the Bug
When we create a code block in a RTL page and modify its direction to LTR, it looks flawless in the editing mode. But when we save the document, it goes RTL and is not properly readable.
Steps to Reproduce

Go to an RTL interface.
Create a code block in a page.
In edit mode, make it left to right.
Save the page.
See? It is right to left.

Expected Behaviour
The code is expected to be rendered left to right regardless of the interface language.
Screenshots or Additional Context

Browser Details
Google Chrome Version 104.0.5112.102 (Official Build) (64-bit) On Windows 10
Exact BookStack Version
BookStack v23.12

Thanks for reporting @okaeiz, I've assigned this to be fixed in the next feature release.
If a temporary workaround was desired, you can add this below code to the "Custom HTML Head Content" customization setting to force all code blocks to ltr:
<style>
  .cm-editor, .page-content pre {direction: ltr;}
</style>

This has now been addressed within commit 999d41a7f5455f4c7f9f175b5718d82ee529df2b.
This will be part of the next feature release.


Meteor 1.4 not working

I'm trying to use
https://atmospherejs.com/zurb/motion-ui
by running
 meteor add zurb:motion-ui
and adding the sass file:
@import '{zurb:motion-ui}/src/motion-ui';
But the basic orbit example works with motion turned off, once I enable it, just freezes after click on the 2nd slide.
But if I just include manually the motion-ui.min.css it works.

Same problem! Same thing happen also with package imported from npm.

You need to include Sass mixins:
@include motion-ui-transitions;
@include motion-ui-animations;

I updated the Meteor installation instructions

See https://github.com/zurb/motion-ui/pull/120


Screenshots

Hello, please include screenshots of your interface and hopefully some of the keyboards you support.
I think you are targeting this software:

Example keyboards:

Here is the first-party application from a big seller on AliExpress. I got write access, when we are ready I can link to this repo from that folder.
https://drive.google.com/drive/folders/1ELKgQ2BUPMGbhAVZO2o962LVMWzFgmp3

Yes, The App on the screenshot is the one that I reverse engineered. I Can add a screenshot of it of course, but I can't include screenshots that I support, because I only have one of these and I'm not the official developer. I just reverse engineered the stuff and ported it to a more modern .NET version. However I'm open to accept pull requests for this repository.


Why does qr(a, lwork=0) not fail?

In [1]: import numpy as np

In [2]: from scipy import linalg

In [3]: a = np.random.randn(5,4)

In [4]: linalg.qr(a)
Out[4]:
(array([[-0.27823641,  0.07326068, -0.42930639, -0.18236735,  0.83645415],
        [-0.71836156, -0.47889962,  0.22947092,  0.4490064 ,  0.01865926],
        [-0.27878177, -0.39251849, -0.03310475, -0.8370379 , -0.25784023],
        [-0.52567893,  0.77308739,  0.29000444, -0.15934457, -0.12846899],
        [ 0.22909289, -0.11641799,  0.82331077, -0.19776819,  0.46584377]]),
 array([[-1.40356203,  1.54993858, -0.36631044, -0.44073894],
        [ 0.        , -2.40858304,  1.16474483, -0.25626037],
        [ 0.        ,  0.        , -0.98774949, -1.32820173],
        [ 0.        , -0.        , -0.        , -2.16491456],
        [-0.        ,  0.        , -0.        ,  0.        ]]))

In [5]: linalg.qr(a, lwork=0)
Out[5]:
(array([[-0.27823641,  0.07326068, -0.42930639, -0.18236735,  0.83645415],
        [-0.71836156, -0.47889962,  0.22947092,  0.4490064 ,  0.01865926],
        [-0.27878177, -0.39251849, -0.03310475, -0.8370379 , -0.25784023],
        [-0.52567893,  0.77308739,  0.29000444, -0.15934457, -0.12846899],
        [ 0.22909289, -0.11641799,  0.82331077, -0.19776819,  0.46584377]]),
 array([[-1.40356203,  1.54993858, -0.36631044, -0.44073894],
        [ 0.        , -2.40858304,  1.16474483, -0.25626037],
        [ 0.        ,  0.        , -0.98774949, -1.32820173],
        [ 0.        , -0.        , -0.        , -2.16491456],
        [-0.        ,  0.        , -0.        ,  0.        ]]))

It seems lwork is getting lost somewhere, even if it seems of somewhat dubious value.  I think the problem is in decomp_qr.py::safecall  since rq doesn't seem to have this problem and doesn't use safecall.  AFAICT there are no tests for lwork on either function.

fixed in gh-4344


Items Needed To Be Added To APP Store

Could we add the following items to the app STORE:
Under ACTION SHOOTING: Steel Rental $10.00
Under SHOOTING SUPPLIES: Safety Glasses $2.00
Under TRAP: Shot Shells $9.00

you should be able to do this as an Admin now.   "Admin Functions" -> "Store Items" -> "Create New"!  Let me know if you have any issues.


Public Comment: Offstreet Parking Theme

Ensure that the offstreet feature layer is displaying correctly for all parking themes.  Make sure that when user selects a different theme layer, the offstreet parking data updates accordingly.

The off street parking theme does not show at all for a few locations when selecting any occupancy based themes.  Please verify that there is actually no data for some of the Study Areas.  For example the following study areas don't show facility data:
Berkeley Elmwood
Berkeley Telegraph
Berkeley Downtown
Downtown Mountain View
etc.

Berkeley Elmwood
Berkeley Telegraph
Berkeley Downtown
These areas do not actually have facility occupancy data
Downtown Mountain View
This area has only some facilities with occupancy shown (less than mxd shows)
I am trying to figure out why


Add workflow to deploy to AKS

Add workflow to deploy to AKS

Automated Deployment Warning
Ensure your manifest at ./kubernetes/deployment.yaml, ./kubernetes/ingress.yaml, and ./kubernetes/service.yaml uses the image name "registry1730320582381.azurecr.io/image-workflow-1730320443784:latest" for the application you wish to deploy.
For example, the image line in your deployment would look something like
image: registry1730320582381.azurecr.io/registry1730320582381.azurecr.io/image-workflow-1730320443784:latest


V3 unclear how  is supposed to be used

Documentation Issue
I was trying to get the sample multi-tenant-single-domain working, there a custom TenantField is used. With version beta75 or 76 (I think) the usage of this field has changed and now you need to provide a field property to <RelationshipField>.
I tried to update this according to the original attributes
export const TenantFieldComponent = () => {
  const { user } = useAuth<User>()
  const { path, readOnly } = useFieldProps()

  if (user) {
    if ((user.tenants && user.tenants.length > 1) || user?.roles?.includes('super-admin')) {
      return (
        <RelationshipField
          field={{ relationTo: ['tenants'], label: 'Tenant', name: path, required: true }}
        />
      )
    }
  }
  return null
}

but in that case, when posting the Page document, instead of the current tenant id, it posts the following in the REST call to the server
_payload: {"slug":"t6","title":"test1","tenant":{"relationTo":"tenants","value":1},"updatedAt":"2024-08-19T06:04:39.040Z","createdAt":"2024-08-19T05:57:03.283Z"}

so value property of this field is not properly set.
Additional Details
I checked the whole source repo and currently no <RelatinpshipField> is being used with the correct syntax so its hard to figure this out on my own.

This also is shown in the call to http://localhost:3000/api/form-state where I get a similar validation error
{...
"tenant": {
    "errorPaths": [
        "tenant"
    ],
    "initialValue": {
        "relationTo": "tenants",
        "value": 2
    },
    "passesCondition": true,
    "valid": false,
    "value": {
        "relationTo": "tenants",
        "value": 2
    },
    "errorMessage": "This relationship field has the following invalid relationships: [object Object] 0"
}

@rburgst I can work on updating that example, likely today. But for your case, the field is not polymorphic so it would look like this:
<RelationshipField
  field={{
    type: "relationship",
    _path: path,
    label: "Tenant",
    name: path,
    relationTo: "tenants",
    required: true,
  }}
  readOnly={readOnly}
/>

Key difference from yours to mine, is the relationTo property.

thanks a lot for the rapid response. This does indeed work fine now. I will keep this open until the docs have been updated (or the sample app).

@JarrodMFlesch in case you plan on improving the multi tenant sample for 3.0 you might also consider this https://github.com/payloadcms/payload/pull/7692/files

@rburgst updated that example to use import paths!


Cloudfront deploy

Update aws.sh script to enable cloudfront distribution.
Now if you run ./etc/testing/deploy/aws.sh --create --use-cloudfront
it'll create a pachyderm cluster with:

an s3 bucket w the correct bucket policy so that CF can access the bucket
a cloudfront deployment
with the correct caching rules so that it does NOT cache error responses (which pachd needs to see)
pachd configured to use cloudfront

This PR does not:

change cloudfront to be on by default
update cloudfront to lock down access

Tested the fruit stand example ... seemed to work fine for the filter step ... the sum step reported an error ... but seems like its related to the transform ... seems like a bug w fruit stand?
  "transform": {
    "cmd": [ "sh" ],
    "stdin": [
        "for fruit in apple orange banana; do",
        "   [[ -e /pfs/filter/$fruit ]] || continue",
        "   cat /pfs/filter/$fruit | awk '{s+=$1} END {print s}' >/pfs/out/$fruit",
        "done"
    ]
  },

the error is:
17-06-04[18:16:11]:pachyderm:0$pc get-logs --job b9bd2040-bdda-40bc-9297-82ace8372775
sh: 2: [[: not found

sh: 2: [[: not found

sh: 2: [[: not found

sh: 2: [[: not found

sh: 2: [[: not found

sh: 2: [[: not found

sh: 2: [[: not found

sh: 2: [[: not found

sh: 2: [[: not found

Oh and I should mention that I tested ./etc/testing/deploy/aws.sh with and without the --use-cloudfront flag and it worked fine.
Once cloudfront becomes the default (in the next PR) this deployment code path will be hooked into the nightly test suite.

Re: the fruit stand, that's a bug (probably my fault). [[ is a bash-only enhancement of [ which I'm too in the habit of using.

The checks actually succeeded. I'm guessing this is a relic of the GH outage this afternoon.
Merging now.


Revert to previous txt file deletion behavior

Prior to the filtering system rewrite, if a user entered a message that was too long, the bot would delete the message and respond with this message in the channel where the deleted message was posted.

The current behavior is that the bot will send a direct message to the user.
Anecdotally, the current behavior seems to be less effective. I frequently see people try to post the same message multiple times and complain that the bot keeps deleting their message, without the user seeing or acting upon the direct message from the bot.
We should revert to the previous behavior of sending a message in the channel where the long message (or txt file) was posted.

is this being worked on, or can I work on it?

Doesn't look like anyone has assigned to this issue yet, I will assign you now!

It looks like the current filtering system doesn't have a setting to send embeds in the original channel, so that would have to be added

assuming we also want this for .py files

Yeah some setting(s) needs to be added here.
I'm wondering what the correct approach here is: We could have a setting for whether to DM the message or send it in the public channel (in which case it might make sense to change the names of dm_content and dm_embed since it wouldn't necessarily be a DM anymore), or we could add a setting that allows sending the message publicly as a fallback if sending the DM fails (usually because they have DMs off).

People don't necessarily check their DMs in the middle of using a help channel, so in this case we probably want to put an embed in the channel even if they have DMs open.

Yeah that makes sense


iPhone support

Please does this tool work with iPhone (windows + iPhone to be specific). If yes could you please list the processes involved with how to use it on one

No, it's for Android.

Okay, thanks @rom1v


tsc with "vitest" version 2 requires "@types/node" to run without errors

Describe the bug
tsc with "vitest" version 2 requires "@types/node" to run without errors, but that's not the case with version 1 of vitest.
Reproduction
https://github.com/hamirmahal/vitest-debug/actions/runs/9915513422/job/27396470395#logs
System Info
System:
    OS: Linux 6.5 Ubuntu 22.04.4 LTS 22.04.4 LTS (Jammy Jellyfish)
    CPU: (8) x64 Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz
    Memory: 22.94 GB / 31.03 GB
    Container: Yes
    Shell: 5.1.16 - /usr/bin/bash
  Binaries:
    Node: 20.9.0 - ~/.nvm/versions/node/v20.9.0/bin/node
    Yarn: 1.22.22 - ~/.nvm/versions/node/v20.9.0/bin/yarn
    npm: 10.1.0 - ~/.nvm/versions/node/v20.9.0/bin/npm
    pnpm: 9.4.0 - ~/.nvm/versions/node/v20.9.0/bin/pnpm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 126.0.6478.126
    Chromium: 126.0.6478.126

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

You can see that tsc --noEmit fails after updating from "vitest": "^1.6.0" to "vitest": "^2.0.2", whereas it was passing before.


Fix a typo in setting the max brake

From self._max_steer to self._max_brake

Description

Fixes #
Where has this been tested?

Platform(s): ...
Python version(s): ...
Unreal Engine version(s): ...

Possible Drawbacks

This change is

Sorry for the delay, I really thought I answered this. This was definitely an error, thanks for spotting that :slightly_smiling_face:


Spiral Region sent when lawnmower was displayed

Not sure how to replicate, look into GPS logs and video logs to confirm/debug.
Also not sure how this would happen since the sent waypoints are taken from a list generated by the PolyArea which is the list that is also used for displaying the region.

Unable to reproduce this, might have been a misunderstanding by the user?


Update multiple-nics.md

Hi,
I would like to have the page extended with an additional description what is required to in the case that a VM gets created with more than one NIC. The problematic with the incorrect routing setup seems to be quit common and therefore customers are going to open SRs about this issue. I therefore think it is the best to extend our docu with additional information how to avoid a misbehaviour

@malachma,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@malachma - Please update your GitHub profile to include your Microsoft email address (go to https://repos.opensource.microsoft.com/ and join the Microsoft organization). Then, close and reopen this pull request. This will enable the automation to verify your account and should remove the CLA requirement.  Thanks

Joined the Microsoft org

@malachma,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@malachma : Thanks for your contribution to the Azure documentation! The author, @iainfoulds, has been notified to review your proposed change.

@malachma : Thanks for your contribution to the Azure documentation! The author, @iainfoulds, has been notified to review your proposed change.

Those changes look good now, thanks! #sign-off


Question - How do you rename/move an object in a Storage Bucket?

What the efficient way to do a rename or move?  Storage shows update(), delete()... but I didn't find what I was looking for.

There's no single move operation in the Storage API. For .NET, as per https://github.com/GoogleCloudPlatform/google-cloud-dotnet/issues/398, we decided not to try to fake one in the client library - there's too much that would be app-specific in terms of how to handle failures.

@danoscarmike why is this in the First 4 (GA) project? We don't have it marked Blocking GA.

I am going to resolve this for now because we don't plan to support it in the library as a feature.


update Magento <IP_ADDRESS>

Magento <IP_ADDRESS> included the follow paths:
dev/
app/bootstrap.php
lib/Net

Ignoring dev like this would not allow you to add your own tests, which would be a good practice.

Superseded by #1746


fix avoid calculating sqrt of negative in agg function CORR

Proposed changes
Issue Number: close #xxx
before:
mysql [sqlfunctest]>SELECT CORR(data, 89.999999) FROM DOUBLEDATA_NOT_EMPTY_NULLABLE;
+---------------------------------------+
| corr(data, cast(89.999999 as DOUBLE)) |
+---------------------------------------+
|                                  NULL |
+---------------------------------------+
1 row in set (0.53 sec)

--- actually here's a silent nan by float exception. now sure in all platform.
after:
mysql [sqlfunctest]>SELECT CORR(data, 89.999999) FROM DOUBLEDATA_NOT_EMPTY_NULLABLE;
+---------------------------------------+
| corr(data, cast(89.999999 as DOUBLE)) |
+---------------------------------------+
|                                     0 |
+---------------------------------------+
1 row in set (0.13 sec)

--- is stable

run buildall

run buildall

run buildall

run buildall


[SPARK-46653][SQL] Code-gen for full outer sort merge join output row by row

What changes were proposed in this pull request?
Be consistent with closing code-gen, update code-gen for full outer sort merge join output row by row.
For example:
val a = Seq((1, 2), (2, 3)).toDF("a", "b")
val b = Seq((2, 5), (3, 4)).toDF("a", "c")
a.join(b, Seq("a"), "fullouter")

before this pr, generated code: https://gist.github.com/zml1206/aff18fc313a7164d6f65096a97d233eb
after: https://gist.github.com/zml1206/a27350b8849951e6efac0fb6088e527f
Why are the changes needed?
Avoid oom. When code-gen for full outer sort merge join is enbaled  and the parent of SortMergeJoin cannot codegen, full outer sort merge join needs to append the output of the same key to BufferedRowIterator.currentRows which type is LinkedList. If there are a large number of duplicate keys, it is likely to cause executor oom.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing UT and local test.
val df1 = spark.range(10000).map(_ => ("testkey", "testvalue1")).toDF("key", "value")
val df2 = spark.range(10000).map(_ => ("testkey", "testvalue2")).toDF("key", "value")
df1.join(df2, Seq("key"), "fullouter").show()

Local mode and driver memory 1G.
Before this pr will oom.
java.lang.OutOfMemoryError: Java heap space
	at java.base/java.util.LinkedList.linkLast(LinkedList.java:146)
	at java.base/java.util.LinkedList.add(LinkedList.java:342)
	at org.apache.spark.sql.execution.BufferedRowIterator.append(BufferedRowIterator.java:73)
	at org.apache.spark.sql.catalyst.expressions.GeneratedClass$GeneratedIteratorForCodegenStage5.smj_consumeFullOuterJoinRow_0$(Unknown Source)
	at org.apache.spark.sql.catalyst.expressions.GeneratedClass$GeneratedIteratorForCodegenStage5.processNext(Unknown Source)
	at org.apache.spark.sql.execution.BufferedRowIterator.hasNext(BufferedRowIterator.java:43)
	at org.apache.spark.sql.execution.WholeStageCodegenEvaluatorFactory$WholeStageCodegenPartitionEvaluator$$anon$1.hasNext(WholeStageCodegenEvaluatorFactory.scala:43)

After this pr is ok.
Was this patch authored or co-authored using generative AI tooling?
No.

@cloud-fan Can you help take a look if you have time? Thanks.

cc @cloud-fan @wankunde @ulysses-you do you have any thought about this? Thanks.

I'm sorry where is smj_leftIndex_0 reset to 0 ?
https://gist.github.com/zml1206/a27350b8849951e6efac0fb6088e527f#file-full_outer_sort_merge_join_codegen_after-L280-L305

I'm sorry where is smj_leftIndex_0 reset to 0 ?
https://gist.github.com/zml1206/a27350b8849951e6efac0fb6088e527f#file-full_outer_sort_merge_join_codegen_after-L280-L305

https://gist.github.com/zml1206/a27350b8849951e6efac0fb6088e527f#file-full_outer_sort_merge_join_codegen_after-L126

Whenever processNext is called, the buffer will be consumed first, and then findNextJoinRows will be called to reset the index and buffer and match the new row written into the buffer. @wankunde


Docker error messages

Two cases for docker error handled with appropriate messages

Docker not installed.
docker.sock permission denied.

This will be a bit more complicated than this. The main things we want to detect for the user are --

If Docker is not installed at all. Unfortunately, we won't be able to detect this by attempting to run the container because most Docker operations now use the Docker API directly via dockerode. So, we need to check for the client binary with something like this line to know if Docker is installed.
If Docker is installed but not running. Docker needs to be running and available for the page to work.

I think you should do these checks in a useInterval that starts up when Validator page is loaded, and is cancelled when Docker is detected to be running (thus, exposing the real page). If Docker is not installed or detected, we'd show only a prompt which warns the user either "Docker is not installed, please install Docker", or "Docker is installed but not running, please start Docker".
Curious to know @SvenDowideit's thoughts on the matter
Does that all make sense?

@nathanleclaire I guess by docker not running you mean to check if the docker deamon is up or not. right?

Correct! The daemon should be up and ready to receive a request on the Docker socket.

going to close this one in favor of https://github.com/workbenchapp/solana-workbench/pull/252

replaced by #252