update fields to retrieve HostID and name

Description
This PR fixes Update fields for Models, Relationships, and components to display host Id and name
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@zakisk

@MUzairS15 ,  I already modified GetModel it's in another PR in Meshery,
here the body response

@MUzairS15 , I already modified GetModel it's in another in this PR #8257, here the body response
Regarding the model, are the available models registered under a host/or registrant?

This is what I was referring to the second screenshot the hostname and id are empty, you need to modify getmodels

@MUzairS15

merge conflict

@MUzairS15, thoughts? i updated the pr

Rady for re-review, @MUzairS15

@leecalcote, sorry forgot to close this one. The changes here have been already merged here #337


Unregistration failed

I try to connect to leshan server in PSK security mode using modified example for mbedclient_linux. During first session everything is fine, but after that unregister failed. In nonsecure mode all work's correctly.
http://pastebin.com/m5RG7TNm

I've enabled dtls logging and got some extended info about error:
dtls_log.txt

ARM Internal Ref: IOTCLT-412

ARM Internal Ref: IOTCLT-417


Parse Error on getAccountHistory

Expected behavior
get 10 account history items for anonsteem user on 2018-10-11
Actual behavior
I get this error
Error: {"name":"RPCError","code":1,"data":{"code":4,"name":"parse_error_exception","message":"Parse Error","stack":[{"context":{"level":"error","file":"string.cpp","line":113,"method":"to_uint64","hostname":"","thread_name":"th_a","timestamp":"2018-02-23T18:17:19"},"format":"Couldn't parse uint64_t","data":{}},{"context":{"level":"warn","file":"string.cpp","line":116,"method":"to_uint64","hostname":"","thread_name":"th_a","timestamp":"2018-02-23T18:17:19"},"format":"","data":{"i":"2017-10-11T00:00:00.000Z"}},{"context":{"level":"warn","file":"variant.cpp","line":405,"method":"as_uint64","hostname":"","thread_name":"th_a","timestamp":"2018-02-23T18:17:19"},"format":"","data":{"*this":"2017-10-11T00:00:00.000Z"}},{"context":{"level":"warn","file":"websocket_api.cpp","line":124,"method":"on_message","hostname":"","thread_name":"th_a","timestamp":"2018-02-23T18:17:19"},"format":"","data":{"call.method":"call","call.params":["database_api","get_account_history",["anonsteem","2017-10-11T00:00:00.000Z",10]]}}]}}

How to reproduce
function accountHistory(user: string, date: string) {
    const then = new Date(date);

    return steem.api.getAccountHistory(user, then, 10, function(err: any, history: any) {
           console.log("History %s", JSON.stringify(history));
           console.log("Error: %s", JSON.stringify(err));
       });
}

accountHistory("anonsteem", "2017-10-11");

Environment information
OS: OS X 10.13.3 (High Sierra)
npm: 5.6.0
node: 9.4.0

Nevermind. I just realized from is not a Date. Glorious.


Use Markdown.js with bootstrap.js

Fixes https://github.com/laravelio/portal/issues/267
Requiring just markdown like so
require('markdown')

Includes a file like so
exports.markdown = require("./markdown");
exports.parse = exports.markdown.toHTML;

Which to the best of my knowledge only exposes require('markdown').markdown hence the need to assign it to the window.
https://github.com/gjrdiesel/portal/blob/834d506705b88add765cef26280a5d86e01ab6a3/resources/assets/js/bootstrap.js#L62

I thought I'd tack one more on here while I was changing the webpack.mix.js file; hope thats ok.

Looks good 👌
Just one question.
Thanks for sending this in!


Infinite Loop

I got this Javascript:
function abc() {
  var s = "Test";
  s = ['<tpl>', s, '</tpl>'].join('');

  var re = /<tpl>.*<\/tpl>/;
  var m, id = 0;

  while(m = s.match(re)){
    s = s.replace(m[0], '{xtpl'+ id + '}');
    ++id;
  }
};

This javascript is reduced to:
function abc(){for(var t,l="Test",p=/<tpl>.*<\/tpl>/,a=0;t=(l=["<tpl>",l,"</tpl>"].join("")).match(p);)l=l.replace(t[0],"{xtpl"+a+"}"),++a}

This results in an endlos loop because variable "l" because it is used in the for condition

Thanks for the report - next time please file them with https://github.com/mishoo/UglifyJS2/issues instead.
Fix is pending at https://github.com/mishoo/UglifyJS2/pull/3139


Wildcard permission not working as expected

 Issue by kbrandwijk
Monday Oct 02, 2017 at 21:08 GMT
Originally opened as https://github.com/graphcool/prisma/issues/722

I have the following permission setup:
permissions:
- operation: "*"
  authenticated: true

I expected this to enable authentication on all permissions, but apparently it doesn't

I tried the exact same configuration, but I thought that it would result in:

All operations are available to authenticated users

But it actually does nothing, it keeps all operations available to everyone.

Is there any news on this ?
It is quite painful without this


OpenSSL does not check the revocation date of the revoked certificate(s)

I created a revoked certificate whose revocation date is later than current time. However, the revoked certificate is still applicable when a certificate chain is verified. OpenSSL 1.1.1d version does not check the revocation date of the revoked certificate(s).
The command I used is:
openssl verify -show_chain -CAfile root.pem -CRLfile test.crl -crl_check_all leaf.pem

The verification returns
C = GB, ST = Berkshire, L = Newbury, O = My Company Ltd
error 23 at 0 depth lookup: certificate revoked
error leaf.pem: verification failed

root.pem:
-----BEGIN CERTIFICATE-----
MIIDNDCCAhygAwIBAgIJAPU0AU3ad04vMA0GCSqGSIb3DQEBCwUAMEcxCzAJBgNV
BAYTAkNOMQswCQYDVQQIDAJTSDENMAsGA1UECgwEU0pUVTENMAsGA1UECwwERERT
VDENMAsGA1UEAwwETkNSTDAeFw0yMDAzMjYwODI3NDlaFw0yMzAxMTQwODI3NDla
MEcxCzAJBgNVBAYTAkNOMQswCQYDVQQIDAJTSDENMAsGA1UECgwEU0pUVTENMAsG
A1UECwwERERTVDENMAsGA1UEAwwETkNSTDCCASIwDQYJKoZIhvcNAQEBBQADggEP
ADCCAQoCggEBAPNLlu+KPCcjj1KiZ1/sUFvFRDt3Z7WZTWjOYeJUFvycHNcYN9cE
laGJ32hfjgaPw9u3cOgs0JJHwIhlQkNhSexUvgGatw336H/3FjPQCJKs48lDJG13
sDF7TK1MvG5wcF1pgRkfvoRSWOyr30aqoeQHaRnqMnT0lWBy26mmV6mgM7LPeNq8
E6jh6aLy7uep3+rO/Ef7LvFi/QWqy+vVVmr5MljXtFyWI+aLs4uFtZZ6rFw5Va4U
Y6OffwchjkVex1eML4D593fobkmkubxZEm2o2Upi/Eiech7CM8HuwgqrAwoIVxi6
FlObraD90sUSUKUwohvl03tkjCTXakXF2TUCAwEAAaMjMCEwEgYDVR0TAQH/BAgw
BgEB/wIBAzALBgNVHQ8EBAMCAeYwDQYJKoZIhvcNAQELBQADggEBACb6hOtUCqD5
sH4VucCO4FYFHM6nfBvB9vx+c2RPC/psam9clOvL5llrUhY070pXbZnd2hwxfnzj
cdr448sVyJkHosukzZj/MyEBV9BERTUMOaY4etQxM2L33uyzn5++/NeRC2Yd53AL
vY/s4znat7txqBK/izvLemLerp1Z5E58VFzLOvYNz+7vEoxMmNaU55TGh88VJIvo
THaZ3LflTc7hv9eUWin0LTV0mg7cvM+/qWrM2N2hyOukztF5gCcMEgoVkpEgUCJP
WsrvOumtDNuXnPr80r4N54n5TaQCTBG22Tj89klc6jUji63+UR9KKACCV44KT8hc
+0ecJPmXqEU=
-----END CERTIFICATE-----

leaf.pem:
-----BEGIN CERTIFICATE-----
MIIDITCCAgmgAwIBAgIBATANBgkqhkiG9w0BAQsFADBHMQswCQYDVQQGEwJDTjEL
MAkGA1UECAwCU0gxDTALBgNVBAoMBFNKVFUxDTALBgNVBAsMBEREU1QxDTALBgNV
BAMMBE5DUkwwHhcNOTYwODAxMDAwMDAwWhcNMjAxMjMxMjM1OTU5WjBMMQswCQYD
VQQGEwJHQjESMBAGA1UECBMJQmVya3NoaXJlMRAwDgYDVQQHEwdOZXdidXJ5MRcw
FQYDVQQKEw5NeSBDb21wYW55IEx0ZDCCASIwDQYJKoZIhvcNAQEBBQADggEPADCC
AQoCggEBAL+N0yePi18I/+MxN/31iBehb2rO5s8MzykUz3aGp3BG/5uEFueqoYZN
CNLA38wIUT/ry8wIw+jlTNj29L7Q9uOX8+10XgF4VTVtN14KT0s7tZ5dLjGRD7ft
fZF03ifbGYp39fW2Wjutjo4Jyop+Bm7g6SrSJB3uaioITpZh8Xf7MHo+kNjJKPsu
ZlVVNQ3T5WQWzoskcpRRIujv7U/NATuRzXODUzqnw+HGavu2qTX3falo5i0dzzrt
9yCtLKqtC+0oX+kZPIi3ib/o20fY3hEXwYstq5sKpvV25xgKTbtwRN1KlMIhfSQN
uFXIg/Rd6rbd9P60zPYxzOTwMsaEysECAwEAAaMTMBEwDwYDVR0TAQH/BAUwAwEB
/zANBgkqhkiG9w0BAQsFAAOCAQEAKhf5CQGxsJCzkFJv26ggzi2HxN/X/eXcwJyy
3gfPP0JZNLzRb6bmracLui58LyCX+0tmY5TA1G3V94Vdu2LIUMRoANwKszTxhW/n
8oNvXDji+E62EsivCtoPgYRAwFE0q4flvcWzDwGlqCfEdaG1uqYGLlLxW8gmHdFs
pKJf4yCzQOn04RmReXOhaAtyUT+xp9AUzawzr2PPGA75x7B07HT4ezLPWy+l1X0o
gMBOWm3AwrwTD8k1B488NiKivCYjBn6UPG0r9/gKxSvdEJEJ6SyM8+Jw+f7lij8i
55LYqy8oyPPknQOAWzB+KZkCbqkcBGJLEPR35agBN/SDSdioXA==
-----END CERTIFICATE-----

test.crl:
-----BEGIN X509 CRL-----
MIIBtjCBnwIBATANBgkqhkiG9w0BAQ4FADBHMQswCQYDVQQGEwJDTjELMAkGA1UE
CAwCU0gxDTALBgNVBAoMBFNKVFUxDTALBgNVBAsMBEREU1QxDTALBgNVBAMMBE5D
UkwXDTIwMDMwODIwMjYwN1oXDTIxMDIyMTIwMjYwN1owFDASAgEBFw0yMTAyMjEy
MDI2MDdaoA4wDDAKBgNVHRQEAwIBADANBgkqhkiG9w0BAQ4FAAOCAQEAkqQPjXyG
FTyPmHi47KoMaeg0M7wU5KO8SXOHxUEU65+QuXOK5pOSpjAH+JhLk5H19dY+22gW
+Bec3N3PO2Iw7iK58t5jl+UywRl3VNf6g2Pf+7zrZje3XoTHATLp+LHfVkUJLflZ
/FiEtJS6NVsNemrAY+BNCCIocHqBVjMAX1zSmUw9lSUFIH6+9HOtVQ0FlmO5RBA5
6QoCmAO3t/5FZF3LqxGj5z3weBVsU340iebf20remV2pEFwWW9KKDBkx+Rd9njgW
dmrKXY4pLfAUZ+D2eCevHBU7vLSSjmgBOKu+74/l28wGF+7oFtj5txbK4hdc1j11
/gX0RqNhGLarQw==
-----END X509 CRL-----

The date of revocation is not relevant to wether it's been revoked or not.

I created a revoked certificate whose revocation date is later than current time. However, the revoked certificate is still applicable when a certificate chain is verified. OpenSSL 1.1.1d version does not check the revocation date of the revoked certificate(s).

It is not valid for the revocation date to be later than the current time. Here is the text from RFC 5280, Section 5.3.2, with the most relevant portions of it in bold:

The invalidity date is a non-critical CRL entry extension that provides the date on which it is known or suspected that the private key was compromised or that the certificate otherwise became invalid.  This date may be earlier than the revocation date in the CRL entry, which is the date at which the CA processed the revocation.  When a revocation is first posted by a CRL issuer in a CRL, the invalidity date may precede the date of issue of earlier CRLs, but the revocation date SHOULD NOT precede the date of issue of earlier CRLs.  Whenever this information is available, CRL issuers are strongly encouraged to share it with CRL users.

It is not valid for the revocation date to be later than the current time.

I'm still confused here. Should an "invalid" revoked certificate be still used, like a valid one, for revoking certificates? OpenSSL takes it (a revoked certificate with an invalid revocation date) as a normal revoked certificate.

If it's in the CRL, it's revoked. The date in the CRL is just some
meta data that might be useful to known since when something is
wrong with the key.


Add js extrension to imports and named re-exports in es build

Fixes #2587.
Unlike #2588, it doesn't introduce conflicts with pending pull requests (which use no extension in imports and named exports).

@GingerPlusPlus:
Thank you for this and all other recent work.  I've been mostly offline in the last two weeks.  I will try to do more Ramda work this week before I leave for a two-week vacation (sans computer.)
This looks good to me at a quick look, but I will try to look more deeply soon.

@GingerPlusPlus: this has acquired a conflict.  Could you resolve, and I'll merge?
Thanks!

Done.

Thank you very much!  I will be getting to your more complex PRs in a few days.


Add missing PartialBulkMessage to imported exceptions

Currently crashes when a PartialBulkMessage exception is triggered.

@genesluder could this be merged?

@Jacse Done. Sorry for the delay. Thanks @jamesdterry

@genesluder thanks a tonne, and great work with the repo!


version bump 0.8.6

Current coverage is 84.87% (diff: 100%)

Merging #66 into master will not change coverage

@@             master        #66   diff @@
==========================================
  Files            34         34
  Lines          1818       1818
  Methods         641        641
  Messages          0          0
  Branches         49         49
==========================================
  Hits           1543       1543
  Misses          256        256
  Partials         19         19

Powered by Codecov. Last update ce64f54...9355b7b

Let's pod update in the Example project to pick up this new version and include that update in this PR.

@jhersh Done ;)

Please squash and merge, thanks!


Handling transitions

Goal
I'm attempting to implement in-out transitions on a Tooltip component. I'd like to understand if there is some preferred library for implementing such transitions, or if it would be feasible to support this directly in floating-ui.
Sort of a discussion really - I did find https://github.com/floating-ui/floating-ui/discussions/1873, but that is also not solved.
Current state
Example implementation I currently have: https://codesandbox.io/s/beautiful-jennings-zm83g8?file=/src/Tooltip.tsx
It uses the react-transition-state library that provides a hook that handles the animation. To be fair, I'm not too happy with that lib, but I haven't found any better hook-based libraries either. I suppose I could try out react-transition-group as well, though I do not like that component-based approach at all in 2023.
So, the Tooltip component allows both "auto" placement and some specific placement. If a specific placement is set, the positioning and transitions work fine. With the auto placement, however, the first transition is always with placement="bottom", as the auto placement needs time to calculate the final placement. So this takes the transition and actual placement out of sync - the Tooltip is placed on the right, but the transition is for bottom. I assume that flip can also have this issue, whenever it actually needs to flip to some alternative placement.
I do know that there is now the isPositioned boolean returned from the hook that is true only when the final placement is calculated. So I attempted to make the transition state dependent on that: https://codesandbox.io/s/gracious-sound-9c52jf?file=/src/Tooltip.tsx
This version does not have the "in" transition for some reason. It might have something to do with the library I'm using for handling the transition, I don't really know. Either way it's quite cumbersome to mix'n'match these different libraries, it turns into a state variable conditional hell pretty quickly.
Feature request?
I would assume that such transition and animation needs are quite common to be implemented together with different floating elements - at least in my memory this is usually the number 1 feature request from UI designers that I have to implement. So, it would be nice if we could provide some guidance and/or examples how these could be implemented - i.e. which hook/component would be a good match to use together with floating-ui for this.
Perhaps, if there is no good match out there, it could also be a new hook that lives inside floating-ui? As the positioning logic has to be quite tightly tied to the animation and mounting states, this might be a good approach?
const { isMounted, transitionState } = useFloatingTransition({
  open,
  placement,
  isPositioned,
  timeoutMs: 100,
});

framer-motion and react-spring handle this well, but they aren't small so I guess some hooks for handling pure CSS animations would be nice to add.

So to implement placement-aware animations (currently), you can do the following:
https://codesandbox.io/s/reverent-shannon-dp1byq?file=/src/Tooltip.tsx
react-transition-group is used only for the unmounting timeout.

Inside an effect, apply the "out" styles unconditionally;
Then, inside a rAF, apply the transition style and the "in" styles when state.open === true. It immediately cleans up the logic that would start the animation from the false initial placement, so it never runs, only running the logic for the real placement.

isPositioned doesn't appear suitable because it won't wait for the unmounting animation.


feat functionality for deatils form info to firebase

This Pull request covers the following issues job current date check#74, functionality for details form info to firebase#73 and update backend of details form to work with the new design and layout#72
The user can update their personal and professional histories together and see the changes with their information in real-time.
Please review this Pull Request and let me know if you would like to change something.

also are you able to change this to just the name of the job please.

also are you able to change this to just the name of the job please.

Thanks for the detailed review and the requested changes have been made, Cheers!.


fix(curriculum): Use AST-based helpers to check for concatenation or f-strings

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[ ] I have tested these changes either locally on my machine, or GitPod.

Closes #56836

You are going well in using the AST based helpers, but as you are changing a test, please test it on your local machine before marking the PR as ready for review


Jumps to the current chapter when opening chapter list

Originally when opening the chapter list the tableView started at the top. This PR makes the chapter list jump to the current chapter the user is on (based on the currentPageNumber).

@camanjj Thanks, that's a nice to have!


.NET Core Compatible Build

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
Referencing ReactiveUI (v8 alpha) in .NET Core crashes in PlatformRegistrations since it is pulling in the .NET Standard build that throws the exception.
What is the expected behavior?
ReactiveUI references common services, or registers nothing at all without throwing an exception when referencing it in .NET Core.
What is the motivation / use case for changing the behavior?
No .NET Core apps or UI frameworks that run on .NET Core can use ReactiveUI (such as Avalonia).
Which versions of ReactiveUI, and which platform / OS are affected by this issue? Did this work in previous versions of ReativeUI? Please also test with the latest stable and snapshot (http://docs.reactiveui.net/en/contributing/snapshot/index.html) versions.
All versions have this bug in their PCL versions, but it is only applicable to v8 since v8 is the only one that is referencable by a .NET Core app.
Other information (e.g. stacktraces, related issues, suggestions how to fix)
Either make the .NET Standard build not throw or add a .NET Core build that doesn't throw and keep the .NET Standard build throwing.

Let's keep the platform throwing exception. It puts people on the happy path and reduces the amount of frivolous GitHub issues.
:+1: to adding netcore build target. We have people who run ReactiveUI serverside and I've been meaning to do this. I welcome a PR - if you need help let me know.

I'm trying to use the 8.0.0-alpha0055 in a aspnet core app. When I do, I get this exception "You are referencing the Portable version of ReactiveUI in an App. Reference the platform-specific version.".
Is this linked to the issue above and the referenced one?

@pellea can you please try with .8.0.0-alpha0061 from NuGet as .NET Core support was only merged recently. Open a GitHub issue if you run into problems.

@ghuntley  It's working. Thanks.

<3


add top level readme

ticket

test is failing bc of error in testanchor, I pinged anchor-eng team about


Copy Paste selection of files

Using the plugin is great to ensure manageable environment variables between several configurations.
In intellij we can generate on the fly some test configurations but some have to use environment variables.
It will be useful if we can copy/paste selection of env files.
Here is an example of copy paste tools in the environment variables configuration window

Is it possible to add them on this section ?

I definitely can imagine that as something doable. I'd welcome a contribution like that!
May be a while until I will have free time to look into implementing it on my own.


Feat/gateway v2 sushiswap connector

Before submitting this PR, please make sure:

[x] Your code builds clean without any errors or warnings
[x] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Tests performed by the developer:
Tips for QA testing:

@Faouzijedidi1 Could you please resolve branch conflicts?

PR update:

Tested on Kovan testnet / Papertrade: ok
Tested on Mainnet: PENDING
Tested on source: In progress
Tested on docker: Not started


v0.1.16 - KeyError: 'promotion_tag'

Hi.
Just got this issue when running 0.1.16
Job "tangata.<locals>.run_first_load (trigger: date[2021-06-27 21:29:07 AEST], next run at: 2021-06-27 21:29:07 AEST)" raised an exception
Traceback (most recent call last):
  File "/home/xxx/envs/dbt19/lib/python3.8/site-packages/apscheduler/executors/base.py", line 125, in run_job
    retval = job.func(*job.args, **job.kwargs)
  File "/home/xxx/envs/dbt19/lib/python3.8/site-packages/tangata/tangata.py", line 108, in run_first_load
    tangata_api.reload_dbt(sendToast)
  File "/home/xxx/envs/dbt19/lib/python3.8/site-packages/tangata/tangata_api.py", line 446, in reload_dbt
    refreshMetadata(sendToast)
  File "/home/xxx/envs/dbt19/lib/python3.8/site-packages/tangata/tangata_api.py", line 66, in refreshMetadata
    assemblingFullCatalog = tangata_catalog_compile.compileCatalogNodes()
  File "/home/xxx/envs/dbt19/lib/python3.8/site-packages/tangata/tangata_catalog_compile.py", line 89, in compileCatalogNodes
    tempCatalogNodes[key] = populateFullCatalogNode(catalog['nodes'][key], "node", catalog, manifest)
  File "/home/xxx/envs/dbt19/lib/python3.8/site-packages/tangata/tangata_catalog_compile.py", line 31, in populateFullCatalogNode
    if len(tangataConfig["promotion_tag"]) > 0 and tangataConfig["promotion_tag"] in manifestNode['tags']:
KeyError: 'promotion_tag'

Fixed in 0.1.18 - new config values were causing issues if they don't exist on version upgrade


routing instructions broken

Distance values are broken in the routing instructions. At the moment, wrong values (all well below 1km) are returned for long distances. Maybe the edge distances up to the next instruction aren't accumulated, and only the distance of the current edge is returned?

Thanks, will look into that tomorrow.

This happens only for the rather old build on the current demo machine. It does not occur with the latest source code.


replacing use of deprecated ruamel.yaml.safe_load

With the latest release of ruamel.yaml version 0.18.2, now safe_load is removed. In this PR updating a remaining usage of it.

How about just using ruamel.yaml >=0.18.0?

Hello, any update on this?


Is there any pre-trained models on COCO dataset?

If not, any guideline to train the model from scratch?
Thanks.

see
https://github.com/Alibaba-MIIL/ASL

Thanks for your help. The ASL repo does contain the URL to the COCO model. Unfortunately, when I click them, it says that "You have no right to access this object because of bucket acl". I think the model is not public available.


MAPREDUCE-7400. New MapReduce example - Sentiment Analysis

Description of PR
I am looking to add a new map reduce example, i.e, sentiment analysis. Sentiment analysis map reduce job helps in determining the sentiment score for a user. It takes each tweet made by an user and assigns a sentiment score for that tweet/sentence for a particular user and then aggregates the sentiment scores for all tweets made by all users.
This example takes the twitter dataset which contains users and the tweets made by users and gives the output as <username, sentiment score>. For each user, the sentiment score is calculated for all the tweets made by that particular user.
How was this patch tested?
This patch was tested using the twitter dataset on a single node Hadoop cluster in pseudo-distributed mode.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  2s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
38m 19s

trunk passed

+1 :green_heart:
compile
0m 39s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
0m 36s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
0m 44s

trunk passed

+1 :green_heart:
mvnsite
0m 42s

trunk passed

+1 :green_heart:
javadoc
0m 53s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
0m 44s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
1m 11s

trunk passed

+1 :green_heart:
shadedclient
20m 18s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 26s

the patch passed

+1 :green_heart:
compile
0m 28s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
0m 28s

the patch passed

+1 :green_heart:
compile
0m 24s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
0m 24s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 22s
/results-checkstyle-hadoop-mapreduce-project_hadoop-mapreduce-examples.txt
hadoop-mapreduce-project/hadoop-mapreduce-examples: The patch generated 181 new + 15 unchanged - 0 fixed = 196 total (was 15)

+1 :green_heart:
mvnsite
0m 30s

the patch passed

+1 :green_heart:
javadoc
0m 25s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
0m 24s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

-1 :x:
spotbugs
0m 56s
/new-spotbugs-hadoop-mapreduce-project_hadoop-mapreduce-examples.html
hadoop-mapreduce-project/hadoop-mapreduce-examples generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)

+1 :green_heart:
shadedclient
19m 59s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
0m 57s
/patch-unit-hadoop-mapreduce-project_hadoop-mapreduce-examples.txt
hadoop-mapreduce-examples in the patch passed.

-1 :x:
asflicense
0m 43s
/results-asflicense.txt
The patch generated 1 ASF License warnings.

92m 19s

Reason
Tests

SpotBugs
module:hadoop-mapreduce-project/hadoop-mapreduce-examples

Found reliance on default encoding in org.apache.hadoop.examples.SentimentAnalysis$SentimentMapper.getList(Mapper$Context):in org.apache.hadoop.examples.SentimentAnalysis$SentimentMapper.getList(Mapper$Context): new java.io.InputStreamReader(InputStream)  At SentimentAnalysis.java:[line 84]

org.apache.hadoop.examples.SentimentAnalysis$SentimentMapper.getList(Mapper$Context) invokes System.exit(...), which shuts down the entire virtual machine  At SentimentAnalysis.java:down the entire virtual machine  At SentimentAnalysis.java:[line 92]

Failed junit tests
hadoop.examples.TestSentiment

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4808/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4808

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint spotbugs checkstyle

uname
Linux f43454b9fe70 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / cf1463cb72b7e3e10c97a0235fcfa30cfcc26463

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4808/1/testReport/

Max. process+thread count
727 (vs. ulimit of 5500)

modules
C: hadoop-mapreduce-project/hadoop-mapreduce-examples U: hadoop-mapreduce-project/hadoop-mapreduce-examples

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4808/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Update index.html

Add or modify some css changes to make more attractive
Fixes : #1122
Thank you for the chance
Please review my pull request and merge it and label me level2
@Niketkumardheeryan

Please review my pull request

Please review my pull request

Added html only??

Change css properties in html file
On Tue, 29 Oct 2024, 12:30 pm Niket kumar Dheeryan, <
@.***> wrote:

Please review my pull request
Added html only??
—
Reply to this email directly, view it on GitHub
https://github.com/Niketkumardheeryan/ML-CaPsule/pull/1154#issuecomment-2443361495,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A7SRZ6DEYYILAA4DSYR7ORLZ54W75AVCNFSM6AAAAABQTNO5COVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINBTGM3DCNBZGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


python3Packages.scrapy-fake-useragent: fix license

change meta.license to licenses.mit
Motivation for this change
Upstream changed to MIT with alecxe/scrapy-fake-useragent@0ea7614 (so with Version 1.4.0, it seems), but looks like it got overlooked in b5d6f771c2636933757d85aeddc4992dc977bdbe, which entered master as part of PR #95609.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

[x] meta.description is capitalized
[x] meta.description doesn't start with the package name
[x] meta.description doesn't have a period at the end
[x] meta.license is set and fits the upstream license
[ ] meta.maintainers is set

Result of nixpkgs-review pr 101278 1


performance regression in 0.10.2

Environment
0.10.2
Python
Environment:
Cloudflare R2

Bug
sorry the bug report is not very helpful, but I notice when I updated to the latest 10.2, the performance has degraded considerably, I am using Google Cloud functions to write to Cloudflare R2, I had to go back to 10.1

same with writing to Azure, using Fabric notebook, the resource usage just increased significantly ?

Do you happen to have an example of some write code you can share? There are a few reasons I can imagine this changing.
What would also be helpful would be the pyarrow version being used (and/or pandas version)

here is the code, I am not sure how to get the pandas version, I just run this code in google cloud functions
https://github.com/djouallah/aemo_tracker/blob/main/writedelta.py

@djouallah can you check against v0.13 if you still see performance issues?


Cknieling/enum fixes

check if the enum was initialized before redrawing it

Found two additional bugs in the enums, that should now be fixed with the above changes


Adds Custom Field Blog

Closes #133
Closes #23
This blog focuses only on custom inline field components. Thinking of making a follow up post where we create a Field Plugin! In the process of making a little demo repo to link to for folks, and also making some additional media.

Reminder: Add consumer metadata

Reminder: need to tweak the sidebar image..its so big

It is unclear to me where I am supposed to put the RangeInput function.

Yeah that's fair, the answer really depends on the situation you're trying to use it with. I'll add a note to try and make that more clear.


Create PodSecurityPolicy and RBAC when needed.

Fixes #272

Thanks for your PR @czunker.
Can you explain a bit more on the benefit, please? I've never used PSP in production yet (as its new with 1.19) so I'm interested in some insights.
Apart from that: the PR would need a version bump to 9.5.0, please.
Thanks!

Hi @SantoDE ,
I've bumped the chart version.
I don't know when PSPs where introduced, but the docs range back to at least 1.15:
https://kubernetes.io/docs/concepts/policy/pod-security-policy/
Pod Security Policies allow to limit the possibilities of PODs, e.g.:

which volume types to mount
whether a container might run privileged or not
whether a container might run as root or not
whether a container might access host ports or not
...

As the policies are controlled by an admission controller, the caveat is, PODs will not start, when the Admission Controller is enabled and no matching PSP is found.
PSPs in general enhance security and when integrated into helm charts, free users from having to write their own.


Functionality request: column header sortable boolean

In order to disable sorting on that column, so column header can be clicked and event generated

v1.14.1 allows custom templates for headers along with ability to add your own events.
closing.


Testing in Jest

Hello. I am using your react-slick library. Thank you very much.
There is a question: there is an afterChange event. How can it be tested in Jest?
I tried to trigger an event via fireEvent, but since afterChange is a field in a variable (
const settingsReserveGoodsSlider = {
    afterChange: (current) => {
      setPrevProduct (current - 1);
    }}

)
, then the event is not called in it.
https://codesandbox.io/s/upbeat-bose-pe1ow?file=/src/App.js

You can check this
https://github.com/akiran/react-slick/blob/master/__tests__/afterChange.test.js


Caliper on a specific variable

Hi,
Thank you for the package.
I saw in the help file that it is possible to exactly match on some variables and to put a caliper on the distance measure, but is there a way to put a caliper on a specific variable? For example, for each treated, I want to keep only the controls that have an age difference of 5 years maximum.
Best regards,
Thomas

Hi, if you only have one such variable, then you can do this using metric(euclidean). Here is an example using nearest-neighbor matching:
webuse cattaneo2, clear
kmatch md mbsmoke fage, ematch(prenatal1 mmarried fbaby) att ///
    metric(euclidean) caliper(3) ///
    nn(5) idgenerate
gen dmax = -1
foreach id of var _ID_* {
    qui replace dmax = max(dmax, abs(fage-fage[`id'])) if mbsmoke==1 & `id'<.
}
qui replace dmax = . if dmax==-1
tab dmax

       dmax |      Freq.     Percent        Cum.
------------+-----------------------------------
          0 |        772       90.82       90.82
          1 |         67        7.88       98.71
          2 |         11        1.29      100.00
------------+-----------------------------------
      Total |        850      100.00

metric(euclidean) uses the identity matrix as scaling metric (i.e. do not change the scaling); the "distance" on which the matching is based will then just be the age difference in the original scaling of the variable; caliper(3) then means that the maximum allowed age distance is 3 years. In the example I use idgenerate() to store the observation numbers of the  matched controls, which I then use to compute the maximum age difference between treated and matched controls. We see that in 772 cases there is a max age difference of 0, in 67 cases a max dif of 1, in 11 cases a max dif of 2. That there are no cases with a max dif of 3 seems to be precision issue; according to the help file, caliper() should allow max differences of 3, but it seems that these comparisons have been excluded nonetheless. You can prevent this by setting the caliper to a slightly larger value.
webuse cattaneo2, clear
kmatch md mbsmoke fage, ematch(prenatal1 mmarried fbaby) att ///
    metric(euclidean) caliper(3.0001) ///
    nn(5) idgenerate
gen dmax = -1
foreach id of var _ID_* {
    qui replace dmax = max(dmax, abs(fage-fage[`id'])) if mbsmoke==1 & `id'<.
}
qui replace dmax = . if dmax==-1
tab dmax

       dmax |      Freq.     Percent        Cum.
------------+-----------------------------------
          0 |        772       90.61       90.61
          1 |         67        7.86       98.47
          2 |         11        1.29       99.77
          3 |          2        0.23      100.00
------------+-----------------------------------
      Total |        852      100.00

However, kmatch does not support the specification of a maximum distance for each variable separately. Furthermore, as soon as you use multiple variables, the distance to which the caliper is applied is a composite distance across all variables.

Hi,
Thank you very much for your detailed reply. That's very useful!
Based on your last paragraph, I suppose though that it is not applicable to kernel matching? (since caliper cannot apply to multiple variables, and caliper is a synonym for bwidth in the package)
Ideally, I would have liked to compare the results between nearest neighbor matching and kernel matching, both using a caliper on age.
Anyway, thanks a lot!
Thomas


Delete application from aws

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm i -g @aws-amplify/cli
If applicable, what version of Node.js are you using?
v16.13.2
Amplify CLI Version
v7.6.25
What operating system are you using?
Mac V11.6.5 Big sur
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
i have delete all AWS S3 compartiments on the console AWS, i have delete amplify of my project with amplify cli.
Amplify Categories
storage, hosting, Not applicable
Amplify Commands
publish, pull
Describe the bug
I want to delete the application from amplify, but an error appears :

Invoking remove all backends execution failed

I went back to the S3 compartment, and created an empty compartment with the same name as when I created the project. I made a copy of the amplify folder before deleting amplify from the project.
after that i went back to the VScode project. and i reinstalled aws cli, and i tried to pull the application registered on aws. i choose -aws profile -> default key but i have this error:
Error downloading #current-cloud-backend.zip from deployment bucket: amplify-travelwebsite-dev-104849-deployment, the error is: The specified key does not exist.
it's impossible. I try to init amplify, same another error
Initializing your environment: devCould not initialize 'dev': The specified key does not exist.
i need help please ,I don't know what to do anymore
Expected behavior
I hope to be able to pull the project so that I can properly remove the amplify application. i can't currently delete a project
Reproduction steps
amplify pull --appId d2uuzr317tcvrq --envName dev
amplify init
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

Additional information
No response

Hey @damienfmx74 :wave: thanks for raising this! To clarify, were all s3 resources deleted via the console and we're wanting to use amplify delete to remove other resources (such as auth, functions) associated with the project?

hi ,
i haven't s3 ressources, all are delete. and i can't delete application because of this I think.

yes, I just made this request "aws amplify delete-app --app-id (id)", it works perfectly thanks


Issue on download method

I end up with a corrupted image when looping through a node's attachments and downloading the attachment.
I am probably making some silly mistake but I saw no other information for this method other than it downloads the image.
//const path = ...
//const node = ...
//const fs = ...
node.listAttachments().each(function () {
           const attachment = this;
           if (attachment.getId() === 'default') {
                attachment.download(function (binaryData) {
                      fs.writeFile(path, binaryData, 'binary', function (err) {
                           if(err){
                              // do something else
                           }
                     });
                     if (!binaryData) {
                        // do something else
                     }
              })
           }
})

I didn't see anything wrong with your code at first glance. Please send this to<EMAIL_ADDRESS>so that it can go through the right channels.

This was fixed in a newer commit.


Save tx at transaction submit eventHandler for nightfall-client

What does this implement/fix? Explain your changes.
Changes in this PR includes

implement transactionSubmittedEventHandler for nightfall-client
save transaction in transactionSubmittedEventHandler if any of commitments and nullifiers are store in db for nightfall-client.
update blockProposedEventHandler of nightfall-client to delete duplicate transactions and their commitments from db, if the transaction not in L2 i.e deleted commitments are not nullified on chain yet.

Does this close any currently open issues?
This PR fixes #1335
What commands can I run to test the change?
Any other comments?
temporary workflow and branch created https://github.com/EYBlockchain/nightfall_3/actions/runs/4076625738/jobs/7026024663 to test the logic - where user created two transfers transaction using same set of commitments while proposer setMakeBlock was off,  Note: second transfer with higher fee.
then proposer starts making block pick second tx and make a successful block.
At nightfall-client side all transaction belongs to user using nightfall-client(logic explained in PR description point 2) are saved.
and at  blockProposedEventHandler second tx update and first tx deleted with change commitment first transfer tx created.

Please add a test for your client synchronisation code (like the optimist resync test)

added


Available language colors

Get available colors for new languages
Hello, is there a more efficient solution for getting available language colors than just manually crawling the colors tool by @lildude ?
And, how will the management go over, when there are more languages than color codes? Will languages have same colors?
Thanks and best regards
Timo Sarkar

There is no longer a restriction in colour choice, so pick whatever colour is best suited, though we recommend sticking to official branding colours to reduce unnecessary churn or unhappy users. Yes, languages will share colours but it's rare for two languages used together to have the same colour.

As @lildude pointed out:

There is no longer a restriction in colour choice,

This restriction was abolished in #4978, so the set of "available" colours is logically equivalent to the universal set.
Closing this as there isn't really anything to do here.


CC51566: updated title and changed one link #7

Hello, @barbkess,
This proposed file change comes from this Community Contribution: https://github.com/aspnet/Docs.it-it/pull/7/files.
Please, help to check this proposed file change into the article and help to merge if you agree with fix. If not, please, let me know either if you would like me to fix it in another way within this PR, if you prefer to fix it in another PR or if I should close this PR as by-design. In case of using another PR, please, let me know of your PR number, so we can confirm and close this PR.
Many thanks in advance.
When creating a new PR, please do the following and delete this template text:

Reference the issue number if there is one:
Fixes #Issue_Number

The "Fixes #nnn" syntax in the PR description causes
GitHub to automatically close the issue when this PR is merged.

Thank you for the PR; however, we prefer the xref style of linking. It has some advantages over the other style.


SocketException: Write failed while doing pub get

I've been following instructions from README
and on pub get got:
Loading sass transformers...
in ShutdownIsolate: Unhandled exception:
SocketException: Write failed, port = 0
#0      _rootHandleUncaughtError. (dart:async/zone.dart:895)
#1      _microtaskLoop (dart:async/schedule_microtask.dart:41)
#2      _startMicrotaskLoop (dart:async/schedule_microtask.dart:50)
#3      _Timer._runTimers (dart:isolate-patch/timer_impl.dart:394)
#4      _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:414)
#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)
^C
tomek@tora:~/projects/clay_tablet_connector_ui$ dart --version
Dart VM version: 1.14.1 (Wed Feb  3 20:13:16 2016) on "linux_x64"
my pubspec:
name: my_app
version: 0.0.1
environment:
sdk: '>=1.13.0 <2.0.0'
dependencies:
angular2: 2.0.0-beta.3
browser: ^0.10.0
bootstrap_sass: any
sass: any
transformers:

sass
angular2:
platform_directives:

'package:angular2/common.dart#COMMON_DIRECTIVES'
platform_pipes:
'package:angular2/common.dart#COMMON_PIPES'
entry_points: web/main.dart

Doesn't seem like a bootstrap_sass issue but sass itself - I think it can be closed

it's just sass command not installed on my box lame ;)  Worth metioning in README maybe

Ok, I'll add it to Readme file


Added Sendable to public structs and enums

I was in the process of prepping my app for having 0 warnings from SwiftSetting.enableExperimentalFeature("StrictConcurrency"), and some Sendable issues surfaced from AcmeSwift.
Adding Sendable to most public enums and structs made the warnings go away, without introducing new ones.
I could not add Sendable to AccountCredentials because it references Crypto.P256.Signing.PrivateKey which is not Sendable yet.
I also tried bumping the swift-tools-version to 5.10 and applying StrictConcurrency to this project, and no warnings were raised. I did not add this to the PR, because I am not sure when you want to bump the required version of Swift.

I could not add Sendable to AccountCredentials because it references Crypto.P256.Signing.PrivateKey which is not Sendable yet.

We might want to either prefix the import with @preconcurrency, or tag AccountCredentials  as @unchecked Sendable, since PrivateKey is a non-mutable struct and should be safe to use in this scenario.

Is this something you want me to do in this PR, or would it make more sense to take these changes as-is and revisit the leftovers later?

I'm happy to take these changes as-is, just need to take a few moments to test and validate

Is this something you want me to do in this PR, or would it make more sense to take these changes as-is and revisit the leftovers later?

Thanks for your PR.
You are very welcome to "revisit the leftovers" any time!


Change default address type and add config to setup

We should change the default address type to bech32 because we will need to have this anyway to integrate coinjoin.
This would affect:

watches, and watchbyxpub
get new address from spender
pycoin derive utilities

I think it should be possible to change this config in the cyphernode configs
priority: high

All the derivations in Cyphernode already use the provided xpub to derive the good address type:
xpub/tpub = 1abc/mabc (P2PKH)
ypub/upub = 3abc/2abc (P2WPKH in P2SH)
zpub/vpub = bc1/tb1 (P2WPKH)
Ref.: https://github.com/satoshilabs/slips/blob/master/slip-0132.md#registered-hd-version-bytes

It is now possible to supply addresstype as an arg to getnewaddress like this:
https://<IP_ADDRESS>:2009/getnewaddress/bech32 to get a bech32 address.
Currently in the dev github branch.

<EMAIL_ADDRESS>

Released in v0.2.1.


add ONNX test suite support in migraphx

the onnx project contains a unit test suite that has good coverage of the scenarios we need to test as an inference engine. This task is to integrate the test suite to migraphx to we have a better test for mgiraphx

We need to inherit the class onnx.backend.test.BackendTest in ONNX, then following the same way in the onnxruntime unit test onnx_backend_test_series.py to do that.

PR merged.


Update about Schedule re-evaluation for deployment

I asked this in Feb.2018 from Djam what is recommendation of Software deployment Schedule re-evaluation for deployments, he said "I think this setting is much more invasive to the local client, than it is to the network or sitewerver.   I do not like that global eval is 7 days.  That is far far too slow!   You could always set it faster, and carefully monitor the situation, and if it doesn’t work slow it back down.".
I suggest our SCCM people change this schedule, but this document has to change first.
Summarize the change in the pull request title
Describe your change, specifically why you think it's needed.
Fixes #Issue_Number (if necessary)

@sandytsang : Thanks for your contribution! The author, @aczechowski, has been notified to review your proposed change.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions

#sign-off
Once merged, this should be live with the next daily publishing process. Right now that should be tomorrow (Thursday, 7 February) around 11:00 Pacific time. No further action needed from you.


startup.sh: mv: cannot move '...' to 'api-server/public/assets': Directory not empty

mv: cannot move '/app/frontend/dist/assets' to '/app/api-server/public/assets': Directory not empty

Possible solution in
https://unix.stackexchange.com/questions/127712/merging-folders-with-mv


Fix for BlameDataCollectorShouldOutputDumpFile

Fix flakiness of BlameDataCollectorShouldOutputDumpFile acceptance test

This is masking the behavior. Abandoning this for now, let me know if you disagree.


Function to Convert Address to String, and Vice-Versa (debug::print can't read ascii)

So, as far as I've been able to do, it's currently impossible to convert an address-type to a string, and a string to an address-type.
If you use sui::bcs::to_bytes(address) what you get is a vector of u8s, which debug::print interprets as a pair of hexadecimals, i.e. 253u8 = fd, 154u8 = 9a, etc.
If you use std::ascii::string it will accept an array of u8s, however it expects all u8s to be < 128, because ascii-characters are supposed to be 7 bits, so it will throw an error every time it sees a u8 >= 128.
What I did was create a function that splits each byte into an upper and lower half, and then converts those into their corresponding ascii-encoding; this turns a length-20 bcs-encoded address into a length-40 ascii. Unfortunately debug::print will print off an ascii-string of bytes incorrectly; for example 102u8 in ascii is supposed to be printed as = f, however debug::print interprets 102u8 as the hexadecimal pair = 66.
You can find the ascii and extended ascii charts here:
https://www.rapidtables.com/code/text/ascii-table.html
The problem is that std::ascii::String is really just a generic wrapper of bytes (vector), without its own print function, and the other modules have their own non-ascii interpretation of how to print those bytes.

https://github.com/MystenLabs/sui/pull/6714 is working toward addressing this, and we'll also fix the debug printing of ASCII strings on the Move side (thanks for filing that issue over there).
In the meantime, I think you should be able to unblock yourself by creating a UTF8-encoded std::string::String from your ASCII-encoded bytes. That should work because UTF8 is a superset of ASCII and debug::print knows how to print std::string::String correctly.

#6714 is working toward addressing this, and we'll also fix the debug printing of ASCII strings on the Move side (thanks for filing that issue over there).
In the meantime, I think you should be able to unblock yourself by creating a UTF8-encoded std::string::String from your ASCII-encoded bytes. That should work because UTF8 is a superset of ASCII and debug::print knows how to print std::string::String correctly.

Thanks! I've switched to using utf8 for anything rather than ascii. Unless you can think of a reason I should prefer ascii?


match() - Conditions

So while looking through the code in commands.py I noticed we are using almost the exact same commands to check the conditions in match(). It starts with msg.startswith("!cmd ") and then sometimes have additional conditions like "The second word has to be an emote." / "The line can only contain 3 words.", etc. Some of these condition checks can get a little messy and we are basicly repeating code with different values.
My idea is to use a standardized 'condition - line' method. First we use a function that parses the written line and extracts all the information available (number of words, text for each word, is the word a number/emote?, etc). Then we define specific condition for each class/function ("First word has to be '!', second word has to be an emote, etc.). And finaly have a function that returns wether the line matches the condition for the class or not.

Write functions that can give you information about a written line.

How many words are in the line? -> n
Information about each word:

Is the first word a command? (!<cmd>)
Is the second word an emote/emoji, NOT an emote, a number, etc.
Length of a word
Total length of the line

Maybe structure like:
line = {
"n": "n-words",
"length": "length-line",
"words": [["text": text1, "type": type1, "length": length1], ["text": text2, …], …]
}
types could be:
0: default (no emote, no number, etc.)
1: a command (startswith '!' or is in commandlist)
2: a number
3: a username (user should be in the channel)
4: basic twitch emote
5: basic bttv emote
6: bttv emoji
7: channel bttv emote
8: special emote the user has access to (subscriber-emotes, turbo, etc.)
ETC.

2.) Then we can define condition for the command. E.g.:
(Should probably have a better structure)
condition = [
[0, {"text": "!call"}],	->	"First word is '!call'."
[1, {"type": ["emote", "emoji"]}, <- Should be able to handle multiple conditions
[3, {"length": "< 10"},	->	"Third word is shorter than 10 letters."
[n, "< 10"]
]
3.) Then in match() we can go like:
if check_match(line, cond):
return True
Instead of making the check for each condition in the match function. It would make the transition from the wording "The second word needs to be a number." into actual conditions easier/more standardized.

My brief thoughts:
The current command structure is originated from SimpleTwitchBot.
match() is called from bot.py in process_command, the line match = cmd.match(self, user, msg)
I think we can categories commands into 3 types, briefly:

commands in simplereply list (Note: I rarely see this used by mods in Zeta's channel ...)

something slightly complicated and requires special logic that we need to write (Oralpleasure for example, not really difficult logic but we need to remember the state in monkalot's memory)

quite complicated one, like Question command (what is 2Head + 2Head in msg) and Spam command

If I am not mistaken, you want to rework on the match() of those in category 2.
My concern is that category 3 wouldn't fit for your proposed change.
So maybe we can have a ...
QuiteSimpleCommand class which is inherited from Command class, then its match() will call something like common_check(). Then it will do the default? type of checking we want, while leaving those complicated category 3 commands intact.


bug fix: tcp/udp port may conflict

fixes: #11658
Hope to fix #11658, @Stono Would you test this?

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hzxuzhonghu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [hzxuzhonghu]

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

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

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
5867f40b4b836233eaadc47641ca2d8fe53461bd
link
/test istio-integ-k8s-tests

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 kubernetes/test-infra repository. I understand the commands that are listed here.

@hzxuzhonghu: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
5867f40b4b836233eaadc47641ca2d8fe53461bd
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
5867f40b4b836233eaadc47641ca2d8fe53461bd
link
/test istio-pilot-multicluster-e2e

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 kubernetes/test-infra repository. I understand the commands that are listed here.

Just tested, this doesn't seem to fix it.
Test config:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: fake-dns
spec:
  selector:
    matchLabels:
      app: fake-dns
  template:
    metadata:
      labels:
        app: fake-dns
    spec:
      containers:
      - name: fake-dns
        image: maniankara/tcp-echo
        resources:
          limits:
            memory: "128Mi"
            cpu: "500m"
        ports:
        - containerPort: 8445

---
apiVersion: v1
kind: Service
metadata:
  name: fake-dns
spec:
  selector:
    app: fake-dns
  ports:
  - port: 9053
    targetPort: 8445
    name: dns
    protocol: UDP
  - port: 9053
    targetPort: 8445
    name: tcp-dns
    protocol: TCP
---
apiVersion: v1
kind: Service
metadata:
  name: alt-fake-dns
spec:
  selector:
    app: fake-dns
  ports:
  - port: 8053
    targetPort: 8445
    name: tcp-dns
    protocol: TCP
  - port: 8053
    targetPort: 8445
    name: dns
    protocol: UDP

ik pc endpoints | grep 53 should show 8053 and 9053 - it only shows 8053, when TCP port comes first

close this in favor of #12766


fix private module registry sources

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
fix #302
Generalizes PR #276

@nimrodkor @tsmithv11 @rotemavni Workflow approval and review is very much appreciated


Longest Common Subsequence in Julia | GSSoC'21 | DP

Have you read the Contributing Guidelines on Pull Requests?
YES
Description
Longest Common Subsequence in Julia
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #4715

Please make an issue here and raise a PR, for an article as part of our Docs repository: https://github.com/TesseractCoding/NeoAlgo-Docs
It would be much appreciated. Thanks!


Linkable belongsToMany

Fix #178

Hey @mrleblanc101! Thanks a ton for your PR-s! I merged all of them and released them in version 4.2.0. Good luck!


POST application/json

how to send application/json ?
my Request Body:
{     "cmd": "request.post",     "url": "https://jsonplaceholder.typicode.com/posts",     "userAgent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/83.0.4103.0 Safari/537.36",     "postData": {         "title": "foo",         "body": "bar",         "userId": 1     },     "headers": {         "content-type": "application/json"     } }
my Response:
{     "status": "error",     "message": "Unable to process browser request",     "startTimestamp":<PHONE_NUMBER>053,     "endTimestamp":<PHONE_NUMBER>678,     "version": "2.1.1" }
CURL:
curl --location --request POST 'http://localhost:8191/v1' \ --header 'Content-Type: application/json' \ --data-raw '{     "cmd": "request.post",     "url": "https://jsonplaceholder.typicode.com/posts",     "userAgent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/83.0.4103.0 Safari/537.36",     "postData": {         "title": "foo",         "body": "bar",         "userId": 1     },     "headers": {         "content-type": "application/json"     } }'
ERROR in Console
ERROR REQ-15 TimeoutError: Navigation timeout of 30000 ms exceeded

same

Address: http://localhost:8191/v1
Response code: 500 (InternalServerError)
Received headers:
Content-Type: application/json
Date: Sun, 30 Apr 2023 17:50:05 GMT
Connection: keep-alive
Keep-Alive: timeout=5
Transfer-Encoding: chunked
Received cookies:
Response Source:
{"status":"error","message":"Unable to process browser request","startTimestamp":1682877004677,"endTimestamp":1682877005340,"version":"2.1.1"}
BUMP


TEmma BuildHits not setting variable properly

Describe the bug
This is for version <IP_ADDRESS>
During the last beam time I attempted to draw GetPosition() (from TEmmaHit), but everything just came out as zeros. GetPosition() is calculated from a bunch of variables (GetLeft(), GetRight(), GetTop(), GetBottom()) which are set in the BuildHits function. Drawing any of these though returns 0. I put some cout statements and in the BuildHits part of the code and when I unpack the code I can see them being set as the correct values, but when I attempt to draw directly from a tree they are being set as 0.
To Reproduce
Unpack a run and try to draw GetLeft() from the TEmma hit, the calibration file is attached
CalibrationFile.cal.txt
Expected behavior
GetLeft(), GetRight(), GetTop(), GetBottom() are set as a value. In version <IP_ADDRESS> used during the last beam time everything works as it should.
Additional context
I have not modified the Emma class since this version so I don't think the problem is with TEmma

So this used to work with v<IP_ADDRESS> and is not working anymore? This might be related to the deep copy pull request. I noticed that that also introduced some issues for DESCANT (which is why we've been using an older version to sort the data during the experiments with DESCANT).

I think so, but I never looked at EMMA data with v<IP_ADDRESS> so I can't be sure. I think you mentioned the problem we were having with TIGRESS addback in v<IP_ADDRESS> was the deep copy request and I know that was working in v<IP_ADDRESS>, so I suspect EMMA was probably working as well.

I tried re-creating this issue with a small test program, but had no success. I will keep looking into this to see if I can find where and why the variables aren't written properly.

I just tested again a small script I wrote to check the streamers:
void streamer(int n)
{
	TFile output("streamer.root","recreate");

	TTree tree("tree","tree");

	TEmma* em = new TEmma;
	TEmmaHit* emHit = new TEmmaHit;
	em->AddHit(emHit);

	tree.Branch("TEmma", &em);
	tree.Branch("TEmmaHit", &emHit);

	for(int i = 0; i < n; ++i) {
		emHit->SetLeft(i+1);
		for(int j = 0; j < i; ++j) {
			TEmmaHit* tmp = new TEmmaHit;
			tmp->SetLeft(emHit->GetLeft());
			tmp->SetRight(emHit->GetLeft()*2);
			tmp->SetTop(j);
			em->AddHit(tmp);
		}
		tree.Fill();
	}

	tree.Write();
	output.Close();
}

Reading the second entry of the generated root file:
GRSI [4] tree->GetEntry(2); em->Print(); std::cout<<std::endl; emh->Print()
virtual void TEmmaHit::Streamer(TBuffer&)
virtual void TEmmaHit::Streamer(TBuffer&)
virtual void TEmmaHit::Streamer(TBuffer&)
virtual void TEmmaHit::Streamer(TBuffer&)
virtual void TEmmaHit::Streamer(TBuffer&)
TEmma: 4 fHits
fFilter = 0
fTdcNumber = 0
fLeft = 3
fRight = 0
fTop = 0
fBottom = 0
fAnodeTrigger = 0
fFail = 0
fICSum = 0
fFilter = 0
fTdcNumber = 0
fLeft = 2
fRight = 4
fTop = 0
fBottom = 0
fAnodeTrigger = 0
fFail = 0
fICSum = 0
fFilter = 0
fTdcNumber = 0
fLeft = 3
fRight = 6
fTop = 0
fBottom = 0
fAnodeTrigger = 0
fFail = 0
fICSum = 0
fFilter = 0
fTdcNumber = 0
fLeft = 3
fRight = 6
fTop = 1
fBottom = 0
fAnodeTrigger = 0
fFail = 0
fICSum = 0

fFilter = 0
fTdcNumber = 0
fLeft = 3
fRight = 0
fTop = 0
fBottom = 0
fAnodeTrigger = 0
fFail = 0
fICSum = 0

Which is just as expected. The first hit for TEmma is the one created outside the loop with only the left set to a value, followed by three more entries (one from the second pass of the outer loop, and two from the third pass), which all have left, right, and top set.
So it seems that writing and reading TEmma and hits works in principle, just not the way we are doing it when sorting data?
This would indicate that it's not an issue of the streamers but rather the way we try and write the data. I thought that last time I had looked into this everything looked okay, up to the moment when we write the data to file, the issue occurs just when we read it back from file.
I might try and hard-code writing a separate branch for TEmma just to test this (similar to the way it's written in the test-script above).

I re-sorted run 52038_000 with the newest version, and it seems to work. The attached plot shows the result of these commands:
TEmma* em = nullptr; AnalysisTree->SetBranchAddress("TEmma",&em);
TH2F* h1 = new TH2F("h1","h1",1000,0,5000,1000,0,5000);
for(int i = 0; i < AnalysisTree->GetEntries(); ++i) { AnalysisTree->GetEntry(i); for(int j = 0; j < em->GetMultiplicity(); ++j) { h1->Fill(em->GetEmmaHit(j)->GetLeft(),em->GetEmmaHit(j)->GetRight()); } } ;
h1->Draw("colz");

I don't know if this is exactly what we expect, but it isn't zero anymore. So I think pull request #1182 solved this.
I don't know if you have any time to test this @sgillespie90, but I will leave this issue open for a little bit longer in  case there are still problems with this.


Adding styles

Initial PR to add in some basic styling, branched off of https://github.com/mccorkle/SEWebFE/pull/6

Can you whack those .sass-cache// binary files or are they needed in PR?

Going ahead and merging, but let me know if you need more explanations / for me to update something.

Thanks Erin -- you covered all my concerns I rose here.  Looking forward to the next PR :-)


Insert project description into metadata (Chrome)

followup of #18

Functionality
Our Extension produces noice while you are normally surfing the web. In the Background many Websites will be opened. These websites are random but connected to a specified stereotype. (There are some defined keywords, the Extensions searchs for. Additional keywords will be found automatically) Do you want to be seen as a Business man? Or a cool Skateboarder?
Key Features:

Browse the web while your user profile is obscured by fake activity!
Random traffic is generated to obscure your real user profile!
You pick what your fake profile should look like!
Illegal traffic is prevented by using up-to-date blacklists!
If you have no data flat: define the limit of traffic generated!

@CaroBa - in welchem commit is das bei Chrome dabei gekommen?

Link: Website
https://mystique-extension.jimdo.com/


Access & secret specifications

Hello,
I have some issue to implement this plugin. I would like to know some specifications about this following command:
velero install --provider tardigrade \
    --plugins storjlabs/velero-plugin:latest \
    --bucket $BUCKET \
    --backup-location-config accessGrant=$ACCESS \
    --no-secret

May I ask you to confirm:

The variable $ACCESS refers to the serialized access from "uplink share sj:/// ". Which can also be remplace by one access key generated from WEB UI.
--no-secret refers to the bucket passphrase, so we need to use --secret-file with the bucket passphrase stored in file or it refers to a k8s secret ?

Thank you

Hi @Guigouu

The variable $ACCESS refers to the serialized access from "uplink share sj:/// ". Which can also be remplace by one access key generated from WEB UI.

Yes, that's correct, it's the serialized access grant that you can get from uplink or in the Web UI and whatever else that you constraint with uplink share ....

--no-secret refers to the bucket passphrase, so we need to use --secret-file with the bucket passphrase stored in file or it refers to a k8s secret ?

No, an access grant is self-contained, there is no concept of a secret as other providers like AWS S3, hence, the --no-secret flag tells Velero that this plugin doesn't require one.
Whenever you could verify my answers and if they resolve your doubts, please close the ticket, thank you.

Thank you for the clarification, I currently have default permission issues with API key. I am trying to figuring out how to revoke default root access with this thread https://forum.storj.io/t/how-do-you-revoke-an-access-grant-using-uplink-cli-with-examples/16320.


Files generated by snapshots are pruned away

In Nanoc 4.1.0rc2, a compilation rule with a #snapshot call along with a routing rule matching that snapshot, will cause the item rep to be written, but pruned afterwards.

Fixed in #779.


Toa points tracker

Tracks your ToA points based on Jagex's recent blog post found here: https://secure.runescape.com/m=news/tombs-of-amascut-drop-mechanics--osmumtens-fang?oldschool=1
Future updates will include pet unique chance and MVP points.

Adam asked me to take a look at this before it gets merged, and I have a couple comments:

This is also missing puzzle points, which were not really specified in the blog post but we have a decent idea of how they work
MVP points are going to be very hard to estimate because if you want to do it right, you'll need to account for everyone else's room points
Calculating based on hitsplats is going to be somewhat inaccurate since multiple people can get points for overkill damage (basically, damage you get XP for, but it doesn't splat because the NPC has already hit 0). If possible, I would suggest reworking to base it on XP drops rather than hitsplats
There are a few monsters that give no points that weren't given in the blogs – a prominent one is the Ba-Ba boulders.
Het's Seal (weakened) appears to have a space in it ingame, but not in your code
The blog wasn't clear on this, but the 5000 points you get at the beginning are subtracted off at the end – they're only there to make early deaths more painful. I think I would suggest displaying the overlay as totalPoints - 5000, which means the display would start at 0. IMO that's better than the alternative of just overestimating the final points by 5000 (or subtracting off 5000 when they enter the reward room)

Het's Seal (weakened) appears to have a space in it ingame, but not in your code

Thanks for the follow up, currently i'm reworking it to use IDS instead of NPC names to solve some of the above issues.

This is also missing puzzle points, which were not really specified in the blog post but we have a decent idea of how they work

Is there information for this I can find somewhere?

There are a few monsters that give no points that weren't given in the blogs – a prominent one is the Ba-Ba boulders.

Is there a way you've been able to see this? I've only been going off of what was posted in the blog

The blog wasn't clear on this, but the 5000 points you get at the beginning are subtracted off at the end – they're only there to make early deaths more painful. I think I would suggest displaying the overlay as totalPoints - 5000, which means the display would start at 0. IMO that's better than the alternative of just overestimating the final points by 5000 (or subtracting off 5000 when they enter the reward room)

Same thing as above, where can I see that happening?

Calculating based on hitsplats is going to be somewhat inaccurate since multiple people can get points for overkill damage (basically, damage you get XP for, but it doesn't splat because the NPC has already hit 0). If possible, I would suggest reworking to base it on XP drops rather than hitsplats

Noted, going to change this over once I'm happy with everything else as a few extra points being calc'd is fine with the lack of MVP.

A discussion we had on discord yesterday is about Kephri, how does it calc shield damage, are you able to farm it by letting it heal ? or does it have a total cap ? does it count the newest or oldest damage if it is capped.

Is there information for this I can find somewhere?

You can see the specifics by looking at how the wiki calculator was made: https://oldschool.runescape.wiki/w/Module:Tombs_of_Amascut_loot
For example, looking at line 108, it seems like it's 100 Points per Scabaras path puzzle completed.

Is there information for this I can find somewhere?

Since the wiki calculator was ready right as jagex released the official blog, it's safe to assume they got the info to make the calculator from another source than that blog. It seems like the blog misses some of that info.

A discussion we had on discord yesterday is about Kephri, how does it calc shield damage, are you able to farm it by letting it heal ? or does it have a total cap ? does it count the newest or oldest damage if it is capped.

Yes, you can farm it by letting it heal. Not worth doing though, looking at a loot/hr perspective.

You can see the specifics by looking at how the wiki calculator was made: https://oldschool.runescape.wiki/w/Module:Tombs_of_Amascut_loot
For example, looking at line 108, it seems like it's 100 Points per Scabaras path puzzle completed.

Thanks! Added a few updates already, when I get some time today I'll go through and add changes based on this.

Took out the free 5k points
Added an MVP assumption config option while MVP is being worked on.
Fixed Wardens points going crazy
Added Palm watering points.
Next:
Add monkey room vent/pillar/dd points
Add scarab puzzle points
Add Better MVP tracking
Add better warden p2 tracking

Added points for missing puzzle rooms, currently assumes 300 (3puzzles x 100) + 450 (75 x 6traps).  So overall this should be extremely accurate for solos, and decently accurate for teams. Few more changes and improvements could be made, but i'm happy with how this is now

@TylerBesecker Are you interested in adding a points tracker to the existing Tombs of Amascut plugin to keep it all in one place? I'm sure users would appreciate having it all-in-one as well.
It would also be easier for us to review rather than a whole new plugin.

@LlemonDuck Not at the moment, personally I like the way CoX plugins are where each functionality is a different plugin. Obviously this is a personal thing, but I find I ignore the config options of large plugins where there's just to much going on.

At some point this good integrate with the party plugin to help calculate mvp damage and communicate points among team members

At some point this could integrate with the party plugin to help calculate mvp damage and communicate points among team members

That's the plan! At least that's the only reasonable way I can think of calculating team MVP accurately

Changed my github name (and the plugin properties to match) which added the non-author plugin change label. So it can be removed if needed.

You should update the plugin manifest file to match as well

You should update the plugin manifest file to match as well

Not sure how I missed that, thanks!

Hey, when could we get this merged please?

I only have two points to revisit:

VarbitChanged is a very hot event, it can be called many many times per tick, and you're doing a lot in this event without actually seeming to care that a varbit has changed. At the very least it would be preferred that you use onGameTick here https://github.com/AbusiveTuna/ToA_Points/blob/b5d41944b1f54b82b03b21f2401350b325f292dc/src/main/java/com/tuna/toa/ToAPointsPlugin.java#L122
While you're changing things, you should also consider using a matching config group name like toapointsoverlay or just toapoints https://github.com/AbusiveTuna/ToA_Points/blob/b5d41944b1f54b82b03b21f2401350b325f292dc/src/main/java/com/tuna/toa/ToAPointsConfig.java#L7

Updated varbitchanged to ongametick, changed config group name to match project


charmrun under causalft should respect ++local

Original issue: https://charm.cs.illinois.edu/redmine/issues/543

When running a message logging (and possibly just online checkpoint/restart) FT net build of Charm++, charmrun will try to launch replacement processes to take the place of failures. In testing environments, it's convenient to run with ++local so that there's no need to SSH to remote nodes. When running the causalfttest targets, I found that charmrun tries to use ssh to launch the replacement, in spite of ++local.
Obviously, fixing this is not really helpful to production-like uses of the online FT builds, since losing a single process on a node is not really the kind of failure we expect. The testing use case is valuable, though.

Original date: 2015-10-26 21:24:01

Might involve some refactoring of charmrun. At the point where we restart a node, a lot of the stuff used in local startup has already been lost. Charmrun could probably use a bit of refactoring as it is anyways.

Original date: 2018-04-29 23:02:02

causalft and mlogft are not production features and have been broken since at least 64-bit IDs were merged for 6.8.0


WIP: Support for AFNetworking 3.0

Fixes #47
There are still a few unresolved issues with the test suite, mostly related to existing JSON recordings no longer being used. Unfortunately, they seem rather non-trivial to fix and the base for the recordings don't necessarily exist anymore (sometimes because the spaces were removed at some point or because they are sync tests which need multiple steps). The best course of action is likely to recreate the affected tests from scratch, ideally with some kind of script to recreate them in the future.
Note: this branch will only work after running make pod because the Pods submodule is only updated for the master branch.

Deleting branch as work is being continued from AFNetworking-3 branch.


Inconsistent behaviour when used on child record

Considering two models Parent and Child where Child has a uniqueness constraint using rescue_from_duplicate. When we instantiate Parent with a Child (not saved yet) that is not unique, there are two possible outcomes:
Scenario 1
The uniqueness validation fails because the other identical record is already visible in the db to the current process. Parent doesn't get saved and #save returns false.
Scenario 2
The uniqueness validation passes but saving fails through a race condition and rescue_from_duplicate rescues ActiveRecord::RecordNotUnique. Parent#save still succeeds and returns true but the Child is not saved. For the caller of Parent#save it looks like both were saved correctly.
I think the right solution is to rollback the Parent transaction (including any other related models) but I have no idea if that's possible.
@gmalette @byroot

cc @tskorupa

@casperisfine

I think this is solved by https://github.com/rails/rails/pull/32796. I'll close this issue, and if you can repro, can you provide an example please?


[SPARK-39938][PYTHON][PS] Accept all inputs of prefix/suffix which implement str in add_predix/add_suffix

We need to follow the pandas behavior of prefix/suffix parameter validation in add_prefix/add_suffix.
Now, we force to validate it as a String type. But pandas looks all values which can be formated as String(implement str func). So it's different here.
What changes were proposed in this pull request?
We support all kind inputs which can be formated as string.
Why are the changes needed?
As pandas behavior is different with PySpark when we input other types into add_prefix/add_suffix funcs.
PySpark
>>> from pyspark import pandas as ps
>>> df = ps.DataFrame({'A': [1, 2, 3, 4], 'B': [3, 4, 5, 6]}, columns=['A', 'B'])
>>> df.add_suffix(666)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/spark/spark/python/pyspark/pandas/frame.py", line 9060, in add_suffix
    assert isinstance(suffix, str)
AssertionError
>>> df.add_suffix(True)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/spark/spark/python/pyspark/pandas/frame.py", line 9060, in add_suffix
    assert isinstance(suffix, str)
AssertionError

Pandas: 1.3.X/1.4.X
>>> pdf.add_suffix(0.1)
   A0.1  B0.1
0     1     3
1     2     4
2     3     5
3     4     6
>>> pdf.add_suffix(True)
   ATrue  BTrue
0      1      3
1      2      4
2      3      5
3      4      6

Does this PR introduce any user-facing change?
No
How was this patch tested?
Input any can be stringable input into add_prefix/add_suffix funcs.

cc @xinrong-meng @itholic @zhengruifeng FYI

Can one of the admins verify this patch?


Task-04: Styling

Acceptance Criteria for 1

[x]  Mentor is added to pr
[x]  AC copied to pr description
[x]  Link to deployed version is added to pr
[ ]  Added layout of Search Bar
[x]  Rewritten all styles with Module Styles
[x]  Implemented dynamic theme change logic

Acceptance Criteria for 1*

[ ]  Implemented logic for fetching user's theme.
[ ]  Saved theme to the Local storage.
[ ] Note: if we have saved theme in Local storage, we use it first after reloading of the page

Dima, you've made a great job, and it will be good to fix this layout issue.

Your colors are a bit different. It isn't a big problem, it's up to you


Modal title padding named wrong

Version: v4.0.0-alpha.5
In source variables we can find $modal-title-padding. As the name indicates it should be applied to .modal-title, but in fact it's applied to: .modal-header.
Either change variable name to $modal-header-padding, or apply this value to .modal-title
Source:
_variables.scss on line 690
$modal-title-padding:         15px !default;

_modal.scss on line 77
.modal-header {
  padding: $modal-title-padding;
  border-bottom: $modal-header-border-width solid $modal-header-border-color;
  @include clearfix;
}

#21118 was merged, so this is fixed.


flutterw is deactivated after running test

Is there an existing issue for this?

[X] I have searched the existing issues.

Version
0.5.2
Description
After running dart test in flutterw, the global activated flutterw is deactivated.
Steps to reproduce

Install flutterw by dart pub global activate flutterw
git clone this repo
cd path/to/flutterw/packages/flutterw
dart test

Expected behavior
No change for global activated flutterw
Screenshots
No response
Additional context and comments
No response

As #12 changes, this is fix.


OPC UA

Please use the form below, leaving the prefilled data to help us. Thank you.
Page link: OPC UA
Document link: opc-ua.md
My Issue/Suggestion
Hi, the following link is broken:
.....
"You can check out a sample module, which gives an example of how the connector can be used."
.....

@ifo234 Thank you for contacting us. Sorry that the link didn't work for you. I just checked the link myself and it works. Is it still broken on your side?


Update the project for Xcode 9.3 and Swift 4.1

Hi guys, below are the list of changes in the project:
• Update the .travis.yml to run on the Xcode9.3beta version.
• Replace the deprecated flatMap operator in favor of compactMap.
• Fix a lint warning with nesting type.
• Remove unnecessary implementation of the InstanceWrapper  by the ImplicitlyUnwrappedOptional.
• Fix an issue causing the compiler was not inferring the types for the + operator correctly declaring the types explicitly for the closure.
• Disable the line_length rule in the ContainerSpec.Arguments file to avoid the warnings.
Regarding the + operator problem with the Swift compiler I solve it in one of the several ways, we can solve it (explicitly declaring the types of the arguments in the closure, as the most of the times in type inference problems of the compiler). The joined function can be another solution as pointed by Rob Napier here.

Thanks for the PR! 👍
In order to properly run the tests on Linux, we need to update .swift-version file.
One issue with this PR is, that it removes InstanceWrapper functionality for ImplicitlyUnwrappedOptionals - which we ideally want to keep - if there is no way to declare conformance to InstanceWrapper, we might need to find other ways to make that work..

@jakubvano You're welcome !!! Regarding the SPM failing test, I was able to enable flatMap for swift version <4.1. The job that is failing and I don't know exactly why? because TravisCI is not giving too much information about it is the JOB="XCODE" DEST="OS=8.4,name=iPhone 5" SCHEME="Swinject-iOS" SDK="iphonesimulator" ACTION="test". If you could take a look it could be great!


Viewbox being removed

I was using the 1.6.0 and I've updated to the 2.1.1 to use the svgAttributes. Now all of the svg view boxes are being removed.
My webpack config:
            loader: require.resolve('@svgr/webpack'),
            options: {
              replaceAttrValues: {
                '#000': 'currentColor',
                '#000000;': 'currentColor',
              },
              svgAttributes: {'aria-hidden': 'true'},
            },
          },

@lucasjs this is now delegated to SVGO, as mentioned by @sonaye you have an example in #142.

Thanks guys!


Syntax highlighting for punctuations, null and undefined.

There is currently no syntax high-lightning for the following punctuations:
!=
==
!==
===
!
?
:

and more... But the above are some of those I expect to have syntax high-lightning.
Also null and undefined are missing syntax high-lightning:
null
undefined

I made it high priority to double check. But after checking there is some syntax highlighting for these compare ts and js:

Marking this as a duplicate of https://github.com/TypeStrong/atom-typescript/issues/71 which is tracking advanced highlighting desires :rose:


python version and tag

Use pyproject.toml to define the Python version.
Fix final image tag

/retest


Fix issue #5748: Rename "Ran a Jupyter Command" to "Ran a Python Command" in UI

This pull request fixes #5748.
The issue has been successfully resolved. The AI agent made the following changes:

Located the text in the JupytrerCellInput component within the frontend code
Changed the display text from "EXECUTE" to "Ran a Python Command" to better reflect that it's running generic Python commands rather than Jupyter-specific operations
Verified the change through frontend tests, specifically the Jupyter component test which passed successfully

The change directly addresses the original issue by removing the strong Jupyter/data science association and making it clearer that the functionality is for general Python command execution. While there were some Python formatting issues in the pre-commit checks, these were unrelated to the UI change and don't affect the functionality of this fix.
This PR can be reviewed with confidence as it:

Makes a targeted change to improve UI clarity
Passes relevant frontend tests
Maintains existing functionality while improving user understanding
Addresses the core concern about Jupyter/data science associations

Automatic fix generated by OpenHands 🙌

OpenHands started fixing the pr! You can monitor the progress here.

Overview:
✓ All issues resolved successfully. The requested text changes from "Jupyter Command" to "Python Command" were implemented in both locations:

Running message updated
Completion message updated

No remaining issues to address.


Use streams instead of direct video urls

Over the last couple of months, the addon has become unusable for me for everything other than the live stream - video starts for maybe 5-10 seconds and then stops.
By using the same strategy used by the new player on ruv.is and fetching an m3u8 catalog, the startup slows down a bit but watching the video works again.

I also did some linting here and there... Hope that's ok!

Fantastic! I'll do a little testing and then ask Kodi to update the plugin 👍

Thanks!
Really nice response time :)


Update zh-cn.ts

Hi, I've completed and made some small improvements to the Simplified Chinese translation. Please merge this. Thanks for your work!

Thank you 🙏🏼


Add .env.template file

Description
Added the .env.template file
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?
Added the file as well as the links
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Thanks @muditmahajan21!

Thanks @muditmahajan21!

Cool! happy to contribute :))

Feel free to contribute more if you need to for HF or just want to make the bot even better! Im happy to merge and grow Clarence!

Feel free to contribute more if you need to for HF or just want to make the bot even better! Im happy to merge and grow Clarence!

I can add more similar commands (random facts from APIs or something like this), if you would like? Or if you have anything specific feature you would like?

I can add more similar commands (random facts from APIs or something like this), if you would like? Or if you have anything specific feature you would like?

Any new commands that you want to create, go ahead whether theyre cool API ones or just ones for a general discord server, or keep an eye out on the issues, as thats where ill put any new commands pd features!

I can add more similar commands (random facts from APIs or something like this), if you would like? Or if you have anything specific feature you would like?

Any new commands that you want to create, go ahead whether theyre cool API ones or just ones for a general discord server, or keep an eye out on the issues, as thats where ill put any new commands pd features!

Okay, I will search and add a couple of commands then. :)


Provide a way to retain tags such as  when serializing

Attempting to serialize components which have rainbow or gradient tags results in the tag getting converted to a lot of individual color tags (see below). It would be useful to keep the rainbow tags when e.g. the user wants to convert from non-strict to strict format.
Original text:
<rainbow>test</rainbow>
Expected output once deserialized and serialized back:
<rainbow>test</rainbow>
Actual output:
<#f3801f>t</#f3801f><#4bff2c>e</#4bff2c><#0c80e0>s</#0c80e0><#b401d3>t</#b401d3>

See #842


Qwen2.5-14B-Instruct eval shape wrong.

my command is
CUDA_VISIBLE_DEVICES=2,3 torchrun --nproc-per-node=2 --master_port 8866 --no-python lm_eval --model hf --model_args pretrained=Qwen/Qwen2.5-14B-Instruct, --tasks hellaswag  --batch_size auto:4
but i miss the wrong:
ValueError: Trying to set a tensor of shape torch.Size([152064, 5120]) in "weight" (which has shape torch.Size([151646, 5120])), this look incorrect.
additionally, it will be right if i replace the model with Qwen2.5-3B-Instruct.

Stupid cry. i changed my config.json in Qwen2.5-14B-Instruct, when i rewrite the vocab_size, things be ok.


added telemetry opt-in banner

Description of the change
Added the telemetry opt-in banner to the following files: README.md and charts/backstage/Chart.yaml
Existing or Associated Issue(s)
Closes https://issues.redhat.com/browse/RHIDP-1906
Additional Information

Checklist

[ ] Chart version bumped in Chart.yaml according to semver.
[ ] Variables are documented in the values.yaml and added to the README.md. The pre-commit utility can be used to generate the necessary content. Use pre-commit run -a to apply changes.
[ ] JSON Schema template updated and re-generated the raw schema via pre-commit hook.
[ ] List tests pass for Chart using the Chart Testing tool and the ct lint command.

/hold
As commented out in https://github.com/janus-idp/operator/pull/372#issuecomment-2141642047, https://access.redhat.com/documentation/en-us/red_hat_developer_hub/1.2/html-single/administration_guide_for_red_hat_developer_hub/index#assembly-rhdh-telemetry_admin-rhdh currently returns a 404.
Should we wait until confirmation that the URL is correct?

As commented out in janus-idp/operator#372 (comment), access.redhat.com/documentation/en-us/red_hat_developer_hub/1.2/html-single/administration_guide_for_red_hat_developer_hub/index#assembly-rhdh-telemetry_admin-rhdh currently returns a 404. Before merging this PR, should we wait until confirmation that the URL is correct?

Got confirmation from @nickboldt that the URL appears to correct.
/hold cancel


Added option for StructuredMeshViews.

This adds the option to convert StructuredMeshViews.

This seems redundant now. I appears that we can already convert StructuresMeshViews to vtk.


Event time two hours off cest

Event times are two hours wrong compared to CEST time of events. I suspect this is caused by storing and displaying times as they are in UTC. We should probably set the timezone of the date object before pushing them into the event template.

The bug stemmed from the docker container running with a wrong timezone, not the Paraply2 application itself. Closing.


Updated to cloudevents/sdk-go v2.0.0-preview4

Signed-off-by: Francesco Guardiani<EMAIL_ADDRESS>Release Note

Bumped cloudevents/sdk-go to v2.0.0-preview4

/lgtm
/approve


AWS sso:ListPermissionSets crashes on child accounts with AccessDeniedException

Description:

What issue is being seen?  Describe what should be happening instead of the bug, for example: Cartography should not crash, the expected value isn't returned, the data schema is wrong, etc.

The cartography sync crashes with an AccessDeniedException when an account that does not have identitycenter configured attempts to run sso:ListPermissionSets. This happens even when the role running cartography is explicitly allowed to listpermissionsets.
To Reproduce:

Steps to reproduce the behavior.  Provide all data and inputs required to reproduce the issue.

Configure an AWS organization with more than 1 child account.
Set up identitycenter on the main aws account.
Run the cartography sync.

Expected behavior: the cartography sync will show that the identitycenter is attached to the main account and correctly list permissionsets on that main account. When it syncs the child accounts, it should gracefully continue when it encounters the access denied exception.
Actual behavior: the cartography sync crashes. It seems like a weird quirk of AWS to throw an accessdeniedexception here when the identitycenter doesn't exist on this child account.
Logs:

If applicable, copy and paste your console log with the failing stack trace.

botocore.errorfactory.AccessDeniedException: An error occurred (AccessDeniedException) when calling the ListPermissionSets operation: User: arn:aws:sts::ACCOUNT1:assumed-role/cartography-read-only/botocore-session-1734851166 is not authorized to perform: sso:ListPermissionSets
ERROR:cartography.intel.aws:AWS sync failed for accounts ['ACCOUNT2', 'ACCOUNT1']
ERROR:cartography.sync:Unhandled exception during sync stage 'aws'
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/cartography/sync.py", line 113, in run
    stage_func(neo4j_session, config)
  File "/usr/local/lib/python3.10/site-packages/cartography/util.py", line 198, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/cartography/intel/aws/__init__.py", line 298, in start_aws_ingestion
    sync_successful = _sync_multiple_accounts(
  File "/usr/local/lib/python3.10/site-packages/cartography/intel/aws/__init__.py", line 196, in _sync_multiple_accounts
    raise Exception('\n'.join(exception_tracebacks))
Exception: 2024-12-22 07:06:06.043570 - Exception for account ID: ACCOUNT2
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/cartography/intel/aws/__init__.py", line 169, in _sync_multiple_accounts
    _sync_one_account(
  File "/usr/local/lib/python3.10/site-packages/cartography/intel/aws/__init__.py", line 64, in _sync_one_account
    RESOURCE_FUNCTIONS[func_name](**sync_args)
  File "/usr/local/lib/python3.10/site-packages/cartography/util.py", line 198, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/cartography/intel/aws/identitycenter.py", line 273, in sync_identity_center_instances
    permission_sets = get_permission_sets(boto3_session, instance_arn, region)
  File "/usr/local/lib/python3.10/site-packages/cartography/util.py", line 198, in timed
    return method(*args, **kwargs)
  File "/usr/local/lib/python3.10/site-packages/cartography/intel/aws/identitycenter.py", line 67, in get_permission_sets
    for page in paginator.paginate(InstanceArn=instance_arn):
  File "/usr/local/lib/python3.10/site-packages/botocore/paginate.py", line 269, in __iter__
    response = self._make_request(current_kwargs)
  File "/usr/local/lib/python3.10/site-packages/botocore/paginate.py", line 357, in _make_request
    return self._method(**current_kwargs)
  File "/usr/local/lib/python3.10/site-packages/botocore/client.py", line 569, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/usr/local/lib/python3.10/site-packages/botocore/client.py", line 1023, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.errorfactory.AccessDeniedException: An error occurred (AccessDeniedException) when calling the ListPermissionSets operation: User: arn:aws:sts::ACCOUNT2:assumed-role/cartography-read-only/botocore-session-1734850578 is not authorized to perform: sso:ListPermissionSets

Screenshots:

If applicable, add screenshots to help explain your problem.

Please complete the following information::

Cartography release version or commit hash [e.g. 0.12.0 or 95e8e11913e2a44a4d4682506d8364a638ceac69]
0.96.2

cc: @SecPrez - i think we were talking about this briefly a couple weeks ago