How to define accesslog as json format?

I use istioctl --set values.global.proxy.accessLogFile="/dev/stdout" --set values.global.proxy.accessLogEncoding="JSON"
to customize accesslog format as json.
2020-03-24T03:35:14.957143Z	info	accesslog.instance.istio-system	{"apiClaims": "", "apiKey": "", "clientTraceId": "", "connection_security_policy": "none", "destinationApp": "miop-ui", "destinationIp": "<IP_ADDRESS>", "destinationName": "miop-ui-7f49c98cf6-wck7h", "destinationNamespace": "miop-dev", "destinationOwner": "kubernetes://apis/apps/v1/namespaces/miop-dev/deployments/miop-ui", "destinationPrincipal": "", "destinationServiceHost": "miop-ui.miop-dev.svc.cluster.local", "destinationWorkload": "miop-ui", "grpcMessage": "", "grpcStatus": "", "httpAuthority": "www.xxx.xxx.pub", "latency": "471.475μs", "method": "GET", "permissiveResponseCode": "none", "permissiveResponsePolicyID": "none", "protocol": "http", "receivedBytes": 1190, "referer": "http://www.xxx.xxx.pub/static/umi.6cb9a301.css", "reporter": "destination", "requestId": "6736f886-5864-979d-a684-67e745ce94c7", "requestSize": 0, "requestedServerName": "", "responseCode": 304, "responseFlags": "-", "responseSize": 0, "responseTimestamp": "2020-03-24T03:35:14.957530Z", "sentBytes": 152, "sourceApp": "istio-ingressgateway", "sourceIp": "<IP_ADDRESS>", "sourceName": "istio-ingressgateway-b988595-zlc6r", "sourceNamespace": "istio-system", "sourceOwner": "kubernetes://apis/apps/v1/namespaces/istio-system/deployments/istio-ingressgateway", "sourcePrincipal": "", "sourceWorkload": "istio-ingressgateway", "url": "/static/static/background.7ff88c18.jpg", "userAgent": "Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.132 Safari/537.36", "xForwardedFor": "<IP_ADDRESS>"}
There is some text not json fromat at the beginning when I collect telemetry by filebeat, how to fix it？

what text is not json?

@howardjohn
I mean "2020-03-24T03:35:14.957143Z info accesslog.instance.istio-system"  this part of texts at the beginning.

are those mixer logs?

are those mixer logs?

Yes

Could you use proxy based stdout log instead? The option you enabled will enable istio-proxy access log. Mixer is going to be deprecated in the near future. So better avoid setting up anything new for it.

But from istio-proxy log message, it is not json format
[2020-04-10T16:52:49.798Z] "GET /rest/unauth/devops/healthCheck HTTP/1.1" 200 - "-" "-" 0 117 29 28 "-" "kube-probe/1.15" "9f428fae-24d6-94a7-9fa4-71bb89afa611" "<IP_ADDRESS>:8080" "<IP_ADDRESS>:8080" inbound|80|http-sys-web|sys-web.miop-dev.svc.cluster.local - <IP_ADDRESS>:8080 <IP_ADDRESS>:52612 - default

Probably because of this: https://github.com/istio/istio/issues/19763 Could you try a build >= 1.4.4?

@bianpengyuan
It seams that's the reason of version, I use istio-1.4.2
Could you please give me a field name list of current text format of  istio-proxy log message? I will splite it the other tools.
[2020-04-10T16:52:49.798Z] "GET /rest/unauth/devops/healthCheck HTTP/1.1" 200 - "-" "-" 0 117 29 28 "-" "kube-probe/1.15" "9f428fae-24d6-94a7-9fa4-71bb89afa611" "<IP_ADDRESS>:8080" "<IP_ADDRESS>:8080" inbound|80|http-sys-web|sys-web.miop-dev.svc.cluster.local - <IP_ADDRESS>:8080 <IP_ADDRESS>:52612 - default

Default Envoy access log format is used if you don't configure anything: https://www.envoyproxy.io/docs/envoy/latest/configuration/observability/access_log#default-format-string

Default Envoy access log format is used if you don't configure anything: https://www.envoyproxy.io/docs/envoy/latest/configuration/observability/access_log#default-format-string

Thanks. I have another question.
I deploy istio-ingressgateway service as NodePort type, but get x_forwarded_for all values is "<IP_ADDRESS>", and can not get real IP of clients.  How to fix it?

You will need to set your ingressgateway use Local as externalTrafficPolicy: --set components.ingressGateways.k8s.service.externalTrafficPolicy=Local. Take a look at https://kubernetes.io/docs/tutorials/services/source-ip/ for implication.

But I get error:
Error: failed to apply manifests: failed to generate tree from the set overlay, error: bad path=value: components.ingressGateways.k8s.service.externalTrafficPolicy=Local

@bianpengyuan
I set  Local as externalTrafficPolicy and NodePort, but only on node that istio-gateway assigned nodeport is exposed, so I need to deploy  istio-gateway as DaemonSet to make all nodes exposed nodeport.
any advice? thanks.


include operator pending mode in pythom motion mappings

The normal mode mappings worked fine with motion pending (omap).
This just required changing nnoremap -> noremap in the ft file.
This seemed not to break things, and now I can delete functions with d[[ etc

Am 2016-09-01 07:27, schrieb OrenLeaffer:

The normal mode mappings worked fine with motion pending (omap).
This just required changing nnoremap -> noremap in the ft file.

This should be first discussed with the maintainer of the file (see top
of the file)
Best,
christian

This would also enable the mappings in Visual and Select mode, which most likely doesn't work.

@brammool - the design was to enable the mappings in select and visual mode (as well as command pending and normal) with noremap and then immediately override them for select and visual with xnoremap (which was already there). That seemed cleaner than repeating the same mappings a third time by having nnoremap, onoremap, and xnoremap.
The slightly terser approach seemed better than the more repetitive one, but it probably deserves a comment, explaining what's going on.

@brammool - thank you for the feedback - @sullyj3 explained to me that select mode should be excluded and I've since updated my patch and worked with him on getting it right. Thanks!

I think this has been fixed since then. Closing.


Bug in the Aberdeenshire scraper

Describe the bug
The manual tags in the webscraper are not getting mapped correctly.
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/OpenDataScotland/the_od_bods/blob/main/data/scraped-results/aberdeenshire.csv
Click on the file
Scroll down to 'Committee areas boundaries within Aberdeenshire' and you will see that manual tag is given as 'transport'  instead of 'council and government'
See error

Expected behavior
The manual tag mapping needs to be done correctly
Screenshots
None
Hardware and software used

Device: laptop
OS: windows
Python version - 3.9

Additional context
None

I just had a look at the mentioned csv file. There are only three categories present: education, burial grounds, and transport, with transport being the most occurring category. These categories are also the first three handled in aberdeenshire_council_scraper.py, with transport being the third. So I would guess, the error is in the elif for transport.

hmm, wonder why we're even processing categories at the scraper level. Should be at merge_data.py level.

This point is very valid @KarenJewell :D For me it was just: code was there, it was broken, so I fixed it. And it was a welcome excuse not to focus on my studies :D
I'll check, whether the key words are already implemented in merge_data.py and then update the PR.


Change obs error for GNSSRO bending angle and update build

Description
This PR adds changes for assimilating GNSSRO bending angle (ROPP1D and MetOffice operators) with the NRL observation error. Previous error parameters using the Desroziers et al 2005 method are commented out for reference. Here, the built code is also updated to account for recent changes in mpas-jedi to pass geometric height as geopotential height for bending angle forward simulations.
In addition, the ObsSpaceV2-to-V3.yaml file and scenario 3denvar_O30kmIE60km_WarmStart.yaml are updated.
Tests completed
Tier 1:

[x] 3dvar_OIE120km_WarmStart
[x] 3denvar_OIE120km_IAU_WarmStart
[x] 3dvar_OIE120km_ColdStart
[x] 3dvar_O30kmIE60km_ColdStart
[x] 3denvar_O30kmIE60km_WarmStart
[x] eda_OIE120km_WarmStart
[x] getkf_OIE120km_WarmStart
[x] ForecastFromGFSAnalysesMPT

Tier 2:

[x] 3denvar_O30kmIE60km_WarmStart (under scenario folder)

Tested three scenarios. All passed: scenarios/3dvar_OIE120km_WarmStart.yaml scenarios/3denvar_OIE120km_WarmStart_VarBC.yaml scenarios/3dhybrid_O30kmIE60km_SpecifiedEnsemble_VarBC.yaml Thank you so much @ibanos90 !

Thanks so much for testing @junmeiban!


Add callbacks

https://github.com/user-attachments/assets/e8ce9c4b-097b-4fbc-ac78-92df456bab57

Please also update the docs


Please create a new github tag; 1.0.3 is not on bower

1.0.3 fixed the two way binding issue, but it's not on bower.  Can someone create a new tag for it?  Thank you.

please use tag 0.0.3


symboliclinker.rb: update URLs

It seems like the homepage and download URL is no longer available.

Thank you for the contribution. It needed some fixes, so they were made in 012d03a2e0008b8f49cd7d2dc3bbbd8fcac3fea6. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


GitHub actions

Moves to GitHub actions CI.

This is ready! I will merge once #35 is in 🎉. Closes #32


Add support for new players flag in the players overview table

Add the new players tag for the player overview table:

I also fixed an issue where the player overview tables weren't using the shared win percentage helper. So I updated that :)


match making

as a solo player, it isn't easy to find friends and play.

[ ] on making a lobby choose whether is it public or private
[ ] create lobby action
[ ] on play if there is a public lobby join, else make a lobby
[ ] On lobby creation choose the limitation of users

You are correct, and I will implement the 'joining public lobby' feature once we have a minimum of 500 users. It is not beneficial with a small user base because finding active lobbies would be so hard.

Hi @geeksesi.
I new release you are able to create public lobbies.
Later on I'll work on private lobbies and user capacity.


Ability to draw sprites with a different origin

I needed a way to draw a sprite with a different origin

Don't merge yet, i think i can make this a bit cleaner :p


TaskCanceledException: A task was canceled. [HTTP 200 returned while handling GET Request]

Raygun report https://app.raygun.com/crashreporting/258s6a2/errors/219862335002 has more details about the error

Related to #788?


DNN-7169 Created 2 more helpful error messages when trying to reset the password

More accurate reporting of failures when password reset doesn't work

After fixing typo, looks good to me :+1:

I've removed the milestone as assigning PR's to milestones is the PM's decision (i.e. @jbrinkman )

OK, I thought it was related to risk.

it often is i.e. a large PR that touches on elements that run on each request (such as CDF) will be assigned to a "large" release such as 8.0.0 to ensure that we have a longer test cycle, or a change in a popular API (such as tabcontroller) may not be taken too close to codefreeze, but we also use it to organise what goes in each release i.e. Joe assigns items to a release, and Charles and I process them. In this case 7.4.2 is effectively codefrozen so when Joe reviews this I would expect it to be assigned to 8.0.0, unless he deems it low-risk (and important enough to come after the codefreeze) - we expect we may take an occasional additional PR but that's up to PM to decide


REMOVE THE CONTENT, JUST KEEP THE COMMENTS

https://github.com/UoA-eResearch/ResBaz24Python/blob/8a50006e601dd9d716b446a7607674329628b5fc/ResBaz_Intro_to_Python.ipynb#L1

accidental duplicate


fix: implement icon in backend and show icon in table if available

Hey, I just made a Pull Request!
Fixes https://github.com/backstage/backstage/issues/26255

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Nice, LGTM!


NameError("name 'ET' is not defined")

[x] This is an actually a bug report.
[x] I am not getting good LLM Results
[x] I have tried asking for help in the community on discord or discussions and have not received a response.
[x] I have tried searching the documentation and have not found an answer.

What Model are you using?

[ ] gpt-3.5-turbo
[ ] gpt-4-turbo
[ ] gpt-4
[x] Other (please specify). Claude Haiku

Describe the bug
When running this code I am getting the error: NameError("name 'ET' is not defined")
Traceback (most recent call last):
  File "/Users/akarshghale/Desktop/Projects/scripts/gpt_functions/instruct_openai.py", line 21, in <module>
    user_response = anthropic_client(
  File "/Users/akarshghale/.pyenv/versions/3.10.11/lib/python3.10/site-packages/instructor/patch.py", line 123, in new_create_sync
    response_model, new_kwargs = handle_response_model(
  File "/Users/akarshghale/.pyenv/versions/3.10.11/lib/python3.10/site-packages/instructor/process_response.py", line 285, in handle_response_model
    tool_descriptions = response_model.anthropic_schema
  File "/Users/akarshghale/.pyenv/versions/3.10.11/lib/python3.10/site-packages/instructor/function_calls.py", line 64, in anthropic_schema
    from instructor.anthropic_utils import json_to_xml, extract_xml, xml_to_model
  File "/Users/akarshghale/.pyenv/versions/3.10.11/lib/python3.10/site-packages/instructor/anthropic_utils.py", line 43, in <module>
    root: ET.Element, model_dict: Dict[str, Any], references: Dict[str, Any]
NameError: name 'ET' is not defined. Did you mean: 'T'?

Code from the lib site:
https://python.useinstructor.com/blog/2024/03/20/announcing-anthropic-support/

To Reproduce
Running the code shared above seems to give an error only when running with Anthropic but works fine with other model like OpenAI or Together.
Expected behavior
It should return a response from Anthropic API

Claude structured responses are XML, not JSON. Don't ask in the prompt for a JSON output. You can still use pydantic response_model classes for the expected output, instructor will take care of that XML conversion for you.
On 28 Mar 2024, at 20:10, Akarsh Ghale @.***> wrote:

This is an actually a bug report.
I am not getting good LLM Results
I have tried asking for help in the community on discord or discussions and have not received a response.
I have tried searching the documentation and have not found an answer.

What Model are you using?

gpt-3.5-turbo
gpt-4-turbo
gpt-4
[x ] Other (please specify). Claude Haiku

Describe the bug
When running this code I am getting the error: NameError("name 'ET' is not defined")
summary_response: SummaryGenerator = anthropic_client( #  summary_response: SummaryGenerator = openai_client.chat.completions.create(
model=MODEL_NAME,
max_tokens=1024,
response_model=SummaryGenerator,
messages=[
{"role": "system", "content": "You're a helpful assistant. Your task is to summarize in the json response model given"},
{"role": "user", "content": topic_assignment_msg.format(customer_responses_list=json.dumps(context_list), query_input=query)}
],
)
To Reproduce
Running the code shared above seems to give an error only when running with Anthropic but works fine with other model like OpenAI or Together.
Expected behavior
It should return a response from Anthropic API
—
Reply to this email directly, view it on GitHubhttps://github.com/jxnl/instructor/issues/538, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABAGMTKGD2K6VLLCV24IA3Y2R2LFAVCNFSM6AAAAABFNNGCCOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYTIMBTHE3TMMA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Claude structured responses are XML, not JSON. Don't ask in the prompt for a JSON output. You can still use pydantic response_model classes for the expected output, instructor will take care of that XML conversion for you. On 28 Mar 2024, at 20:10, Akarsh Ghale @.> wrote:  * This is an actually a bug report. * I am not getting good LLM Results * I have tried asking for help in the community on discord or discussions and have not received a response. * I have tried searching the documentation and have not found an answer. What Model are you using? * gpt-3.5-turbo * gpt-4-turbo * gpt-4 * [x ] Other (please specify). Claude Haiku Describe the bug When running this code I am getting the error: NameError("name 'ET' is not defined") summary_response: SummaryGenerator = anthropic_client( # summary_response: SummaryGenerator = openai_client.chat.completions.create( model=MODEL_NAME, max_tokens=1024, response_model=SummaryGenerator, messages=[ {"role": "system", "content": "You're a helpful assistant. Your task is to summarize in the json response model given"}, {"role": "user", "content": topic_assignment_msg.format(customer_responses_list=json.dumps(context_list), query_input=query)} ], ) To Reproduce Running the code shared above seems to give an error only when running with Anthropic but works fine with other model like OpenAI or Together. Expected behavior It should return a response from Anthropic API — Reply to this email directly, view it on GitHub<#538>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABAGMTKGD2K6VLLCV24IA3Y2R2LFAVCNFSM6AAAAABFNNGCCOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYTIMBTHE3TMMA. You are receiving this because you are subscribed to this thread.Message ID: @.>

Thanks for the reply, I have even run this code with the sample code you have provided in the lib site I mentioned. So the issue is with the lib itself.

pip install xmltodict anthropic

pip install xmltodict anthropic

Thanks this fixed the issue!


ip camera stream doesn't work

Hi,
Thanks a lot for your software!
I successfully installed ffmpeg and homebridge, but I can't get the streaming in HomeKit working.
I checked if ffmpeg was installed by executing ffmpeg.
The snapshot works great. Here's my config:
{
"bridge": {
"name": "Homebridge Synology",
"username": "AC:22:3D:E3:CE:31",
"port": 51826,
"pin": "031-45-154"
},
"description": "This is an example configuration file. You can use this as a template for creating your own configuration file containing devices you actually own.",

"accessories": [
],

"platforms": [{

"platform": "Camera-IP",
"cameras": [
{
"name": "Camera",
"videoConfig": {
"source": "-re -i http://<IP_ADDRESS>:8081",
"stillImageSource": "-i http://<IP_ADDRESS>/picture/1/current/",
"maxStreams": 2,
"maxWidth": 1280,
"maxHeight": 720,
"maxFPS": 25
}
}
]
}
]
}
In my log I can see:
-re -i http://<IP_ADDRESS>:8081 -threads 0 -vcodec libx264 -an -pix_fmt yuv420p -r 30 -f rawvideo -tune zerolatency -vf scale=1280:720 -b:v 299k -bufsize 299k -payload_type 99 -ssrc 1 -f rtp -srtp_out_suite AES_CM_128_HMAC_SHA1_80 -srtp_out_params 4pXKX4bLVc0+SbPl8bUTDv8JW6+mqyEMZwJI4z/c  srtp://<IP_ADDRESS>:58441?rtcpport=58569&localrtcpport=58569&pkt_size=1378
Any ideas?

Sorry, got no idea.

Got exactly the same problem.
See: http://github.com/KhaosT/homebridge-camera-ffmpeg/issues/120

Closing this issue. Seem to be invalid settings.


can't get work with rn 0.31.0

Hello, thanks for your awesome plugin but i can not get work with rn 0.31.0, this is my package json
"dependencies": {
    "moment": "^2.13.0",
    "react": "15.2.1",
    "react-native": "0.31.0",
    "react-native-maps": "^0.7.1",
    "react-native-scrollable-tab-view": "^0.5.4",
    "react-native-share": "^1.0.16",
    "react-native-swiper": "^1.4.11",
    "react-native-vector-icons": "^2.0.3",
    "react-redux": "^4.4.5",
    "redux": "^3.5.2",
    "redux-thunk": "^2.1.0"
  }

logs
iMac-de-Dante:MiDiocesis dantecervantes$ npm install react-native-audio-streaming --save
npm WARN peerDependencies The peer dependency<EMAIL_ADDRESS>included from react-native-audio-streaming will no
npm WARN peerDependencies longer be automatically installed to fulfill the peerDependency
npm WARN peerDependencies in npm 3+. Your application will need to depend on it explicitly.
npm ERR! Darwin 16.0.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "react-native-audio-streaming" "--save"
npm ERR! node v4.4.4
npm ERR! npm  v2.15.1
npm ERR! code EPEERINVALID

npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/dantecervantes/Documents/react-native/MiDiocesis/npm-debug.log

Thanks!

+1
Having the same problem with React Native 0.33.0

Same problem with RN 0.35.0

This should be fixed with latest release (2.2.0), see https://github.com/tlenclos/react-native-audio-streaming/commit/79ef1dbe1031a198c183a480b2441c853da4ac14.


Skip mean over empty axis

Avoids changing the datatype if the data does not have the requested
axis.

[x] Closes #4885
[x] Tests added
[ ] Passes pre-commit run --all-files
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

I have now changed so that several wrapped functions preserve the data.
It is more generic, and hopefully still readable. The flag might also be called identity_0d because the function is the identity function for 0d data, while invariant_0d means that any 0d data is invariant under this function.

Excellent! Thanks @dschwoerer .
Any thoughts from others before we merge, particularly on the identity_0d approach?

Merging — as ever, any additional feedback we can address in another PR.
Thank you very much @dschwoerer , we're happy to have you as a contributor!


u128 comparison fails with opt-level 1

This little program is fine with opt 0, but fails with opt 1.
fn main() {
    const X: u128 = 0x8000_0000_0000_0000;
    assert_eq!((|| X)(), X);
}

$ rustc +nightly foo.rs -Zcodegen-backend=cranelift -Copt-level=1 && ./foo
thread 'main' panicked at foo.rs:3:5:
assertion `left == right` failed
  left:<PHONE_NUMBER>854775808
 right:<PHONE_NUMBER>854775808
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Note that this constant could fit in u64, but it looks like it's sign-extending that %r8 MSB into %rcx:
    845c:       e8 86 00 00 00          call   84e7 <_ZN3foo4main28_$u7b$$u7b$closure$u7d$$u7d$17h8cd7c7807c97b1efE>
    8461:       48 8d 34 24             lea    (%rsp),%rsi
    8465:       48 89 06                mov    %rax,(%rsi)
    8468:       48 89 56 08             mov    %rdx,0x8(%rsi)
    846c:       48 8d 34 24             lea    (%rsp),%rsi
    8470:       48 8b 06                mov    (%rsi),%rax
    8473:       48 8b 56 08             mov    0x8(%rsi),%rdx
    8477:       49 b8 00 00 00 00 00    movabs $0x8000000000000000,%r8
    847e:       00 00 80
    8481:       4c 89 c1                mov    %r8,%rcx
    8484:       48 c1 f9 3f             sar    $0x3f,%rcx
    8488:       4c 39 c0                cmp    %r8,%rax
    848b:       41 0f 94 c0             sete   %r8b
    848f:       48 39 ca                cmp    %rcx,%rdx
    8492:       41 0f 94 c1             sete   %r9b
    8496:       4d 21 c8                and    %r9,%r8
    8499:       49 f7 c0 01 00 00 00    test   $0x1,%r8
    84a0:       0f 84 09 00 00 00       je     84af <_ZN3foo4main17hb0af6480194903a8E+0x62>
    84a6:       48 83 c4 50             add    $0x50,%rsp
    84aa:       48 89 ec                mov    %rbp,%rsp
    84ad:       5d                      pop    %rbp
    84ae:       c3                      ret

The closure properly zeros the top half instead:
00000000000084e7 <_ZN3foo4main28_$u7b$$u7b$closure$u7d$$u7d$17h8cd7c7807c97b1efE>:
    84e7:       55                      push   %rbp
    84e8:       48 89 e5                mov    %rsp,%rbp
    84eb:       48 b8 00 00 00 00 00    movabs $0x8000000000000000,%rax
    84f2:       00 00 80
    84f5:       48 31 d2                xor    %rdx,%rdx
    84f8:       48 89 ec                mov    %rbp,%rsp
    84fb:       5d                      pop    %rbp
    84fc:       c3                      ret

$ rustc +nightly -Vv
rustc 1.79.0-nightly (f9b161492 2024-04-19)
binary: rustc
commit-hash: f9b16149208c8a8a349c32813312716f6603eb6f
commit-date: 2024-04-19
host: x86_64-unknown-linux-gnu
release: 1.79.0-nightly
LLVM version: 18.1.4

-Zmir-opt-level=2 without -Copt-level=1 reproduces this too. Looks like
v3 = icmp_imm eq v2, 0x8000_0000_0000_0000

gets legalized to
v12 = iconst.i64 0x8000_0000_0000_0000
v13 = sextend.i128 v12  ; v12 = 0x8000_0000_0000_0000
v3 = icmp eq v2, v13

which is to be expected as immediates in clif ir are signed. The problem is that cranelift_frontend::switch::icmp_imm_u128 uses icmp_imm here despite the value to check against not fitting in an i64.

https://github.com/bytecodealliance/wasmtime/pull/8422 fixes this bug.

Fixed with the latest Cranelift release.


Updated m_suppress_notifications flag

The m_suppress_notifications should be 4096, but it was 512.

thanks I already fixed this when I saw your message, I appreciate the pr!


Update vpn-gateway-about-vpn-devices.md

Hi @stegag, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Stefano Gagliardi). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

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

#sign-off


Hi, im trying to hide title

Can i  hide title?

Please upgrade
npm install<EMAIL_ADDRESS>--save


Pipeline stages using 'withProject' require a openshift.withCluster() {} closure

Addressed by adding openshift.withCluster() above each invocation of openshift.withProject()


Jira ticket hyperlinking doesn't work when the ticket number is inside of parens

Hyperlinking works correctly when a whitespace character is at the beginning of the Jira ticket, but does not correctly hyperlink when the Jira ticket is surrounded by parentheses:
e.g.
ABC-12345 works, but (ABC-12345) does not.

hyperlinks from tickets inside parens are linking to /browse/(ABC-12345


Allow fetching more rows in table explorer

Currently, the table explorer just displays the first 1000 rows. Having a button to fetch more rows would be useful.

The lazy-loading that the CSV file reader/explorer uses is amazing. It can handle files with a million rows, and yet feels like it's got the whole thing available (at least, on my laptop with a fast SSD).


Load-test production api-backend with the created locustio code

Here the repo and the code for load-testing https://github.com/opentargets/genetics-loadtests

For a load of 200 users

1 api machine
1 ch backend


Ubuntu installation and multiple improvements

Enables choosing version of strongswan by building from source rather than installing with apt  (Debian only)
Tweaks sysctl to disable RPF and redirects; also enables forwarding
Adds updown script for tunnel establishment (disabled by default, but may be used according to README)
Enables separate logfile instead of using syslog
Disables adding strongswan_conns parameters if vars are undefined
strongswan_config_setup is a dict now - charondebug parameter needs to be enclosed in quotes unlike others in this section

Every change is documented in README and defaults/main.yml with usage examples.

Hey @torian can it get reviewed please :)


Extract api_error_message from HTTP 4XX responses

Currently, AssemblylineClient extracts the api_error_message field from the JSON response for 5XX responses. It does not extract the error message for 4XX responses. It seems like the client should be extracting messages for 4XX responses to make debugging easier.
I have seen at least one example where a 400 BAD REQUEST response body had a message that would have been useful:
{
  "api_error_message":"You cannot start a scan with higher classification then you're allowed to see",
  "api_response":{},
  "api_server_version":"4.0.0",
  "api_status_code":400
}

The error generated by the client for this response was:
WebClientResponseException$BadRequest: 400 Bad Request from POST https://malware.cyber.gc.ca/api/v4/submit/

Instead, it would have been nice to get:
WebClientResponseException$BadRequest: 400 Bad Request : You cannot start a scan with higher classification then you're allowed to see from POST https://malware.cyber.gc.ca/api/v4/submit/

(This is the same formatting used for 5XX errors.)

Extracting errors from both 4XX and 5XX seems to be safe (I was worried that modifying the 4XX errors would cause issues somewhere else). The only minor hiccup is that the test cases for logging in don't include a bodies for their mock 401 UNAUTHORIZED responses, which means that attempting to read the message throws an error. That part is easily fixable, but there is one test case that seems tricky: testIsSubmissionCompleteUnauthorizedTwice.
My understanding is that the test is trying simulate a case where a login fails (non-existent user, wrong API key, etc). From what I understand, the sequence of events in the client would be approximately:

Initial request fails because session token is not included in request.
Before retying, attempt to log in.
Retry initial request.

Given that sequence of events, I would expect the mock responses to be:

401 (Session Rejected)
401 (non-existent user, invalid API key, etc)
401 (Session Rejected)

However, ordering the responses as described above does not work and the "correct" ordering of responses is:

401
200 (Successful login)
401

Even though I can see how the current mocks produce the expected result, I don't understand what situation they are simulating.

This test was written to test that the Retry.max(1) worked and would stop after that. I can see that the test itself does not make much sense though as you are correct and the situation would never rise. The test can be removed I believe.
As well, I agree it makes sense to add the 400 error messages.

I'll change the test (or replace it) so that it verifies that a failed login attempt via the retry mechanism propagates a useful message back to the user (e.g. Invalid API key).


mapsApiKey for Geo Charts

Is there a way to pass a mapsApiKey to the Chart component?  I'm having trouble loading my GeoChart because it seems to be requiring an API key.
https://developers.google.com/chart/interactive/docs/basic_load_libs#load-settings
Thanks!

Hey @jrlegrand,
You should be able to pass the key using the mapsApiKey prop.
Cheers !


Use margin instead of padding to offset "On this page"

This makes the border extend much less further, making the aside look
much cleaner.

main

This PR

This PR + #578 + #579


COMMONSRDF-11 make simple subclassable

Fixes COMMONSRDF-11 by making all simple classes (except Types) non-final and have public constructors.
A new SubClassRDFTermFactoryTest verifies this by constructing each of them as an empty, anonymous subclass. (although it does not try to override any of their methods).

What is the rationale for any other implementation needing to extend this implementation and include different semantics to those used here, and implied by the various interactions between package-private classes? If any of those assumptions are changed by the extended implementation then they would think the bug is here and not in their implementation.
It was quite intentional not to publicise any of the implementation classes or constructors for this module, and to make the implementations final so that the only entry point is the SimpleRDFTermFactory that has its public API, except for its constructor, set by the commons-rdf-api module, and all other visible objects and methods from that point on have their contracts also set by commons-rdf-api.
I also do not want to be supporting the simple module in terms of binary compatibility, as that will influence the versioning on the API module unless we split them up, which may be more confusing than it is valuable. Ie, if we make a change here that is binary incompatible, but the API doesn't have any binary incompatible changes, the API may get a major version bump when that isn't necessary.
Our core focus is on maintaining the stability and applicability of the core API, so we want to ensure that its versioning and binary compatibility doesn't change without a clear reason once version 1.0 is released. The simple module here is only being implemented here as a test harness, not as a basis for any other implementations, or else it will fall foul of our general principle not to be focused on any particular implementations.

I thought it might be more in line with the general Apache Commons style of exposing straight forward classes "as is", rather than being overly protective. But not having to keep binary compatibility here could be a big win strength release-wise - which I know Commons is quite touchy on. (See https://commons.apache.org/releases/versioning.html ).
I don't see the simple implementation as just a test harness, it could also be used by code that do not themselves want to bind to a particular 'framework' implementation, but need to be able to return a couple of Triples for instance.
JSONLD-Java's core module is a fine example of that, e.g. in RDFDataset which could be significantly simplified with Simple (even if we don't add "Dataset" in Commons RDF yet)

I have always seen simple to be solely a test harness to verify the contractual effects and our compliance tests. The main reasoning other than the stable versioning issue, is that simple is a well encapsulated implementation with its own internal assumptions that are be much simpler to implement if it is never extended, and we can guarantee that objects are immutable.
In addition, I would much prefer a defensive programming approach where the internal implementation details are not exposed if the user has no way of modifying them without violating other internal assumptions. Ie, this pull request as it is currently still has private final fields that cannot be overridden or replaced and any overriding would need extra fields with extra memory usage to match, as in most cases there is either no default constructor, or the default constructor sets immutable fields itself. The fact that the fields are private final means that third parties would need to extend the implementation to change the behaviour for all parts by adding their own fields, which defeats the purpose of an extensible framework.
If there is benefit to providing extensibility (which isn't clear so far) then the versioning would need to be changed to split up the versions between the api and the simple module. We want excessively stable versioning (corresponding to excessively stable binary compatibility) on the commons-rdf-api module, but commons-rdf-simple would not have (or need) excessively stable versioning if it is intended to be extended that way, and its major version could be incremented regularly as its internal implementation advances independent of the base contract. As it is currently, the binary compatibility is completely encapsulated via the package-private classes and constructors so that the binary compatibility concerns of simple are effectively the same as the api module (except for the SimpleRDFTermFactory constructor that is an unavoidable extension). Opening that up extends the binary compatibility to the particular methods we have used to implement the API in simple, which extends our scope massively (comparative to the scope without it).
In addition, we will need to think further about the way BlankNodeImpl(Object, String) needs to function if we can't use "instanceof BlankNodeImpl" to scope the mapping, and I can't see how any other method would be viable.
Basically, there are massive effects to exposing the simple implementation classes and very little tangible benefit so far except for a general guideline that could be interpreted to agree with a single entry point into the simple module as a well-encapsulated state machine.

As a reference/testing implementation "simple" has one role; as an extensible framework, it would have a different role.  I think we need to cover the first and treat the second as additional if it does not interfere with the reference/testing role.

OK, I will close this unmerged, as I agree to not go for COMMONSRDF-11 and keep simple focused as a black box non-extensible implementation which we use primarily to exercise the API and tests - I don't want to muddle versioning and binary compatibility now.
It is still very much usable for dummy and testing use, just not extensible - in which case you probably should implement the whole class yourself.
I suggest we leave SimpleRDFTermFactory non-final (as it is now), so the only remaining extensibility option is to partially use the Simple classes in a derived factory - which given the non-complexity of the SimpleRDFTermFactory is straight forward (it is just calling constructors) and does not add any binary compatibility issues.


COPY from s3 can only process the first 1000 documents in a bucket

Only the first 1000 documents are processed from the s3 bucket. The MapD Copy command returns successfully and with 0 rejections. However this is very misleading because potentially 1000s of documents may have been ignored.
I would guess this is caused by not checking that more results are available using the GetIsTruncated flag on the ListObjectsRequest result.

The ListObjectsResult object has two values that will help you: GetIsTruncated and GetNextMarker. GetIsTruncated tells you if you've reached the end of the result set. If you haven't reached the end, you fetch the next set of results by setting the marker on the request (ListObjectsRequest::SetMarker) to be the value of ListObjectsResult::GetNextMarker from the result of the previous request.

@andrewrimmer Just wanted to follow-up with you on this, this bug should be resolved in our 4.1.1 release which will come out in the next several days. I will leave this issue open until it's released, then close it

Released:
https://www.mapd.com/docs/latest/7_0_release.html#4-1-1-released-september-4-2018


The splitbar position defaults to middle on page load

I was using Jquery's UI Layout plugin initially and recently started using angular and so the ui layout directive . In Jquery's UI Layout the resizer (the splitbar) once repositioned does not change on page reload . Unlike this in ui-layout directive we move the splitbar according to our convenience refresh the page and then splitbar will come back to its initial position .

+1

+1 Or maybe would be more powerful to put in the api a callback on the end of the resize event with some useful information as arguments.

@mohsen1 mentions that it may be a good idea to use localStorage for this purpose.


Back/forward navigation buttons

I've found myself reaching for the back/forward style buttons that Apple seems to be littering around macOS these days. It might be nice to add a back stack.

If we can get to a point where the whole app is drive by searches, the back forward list should be really easy to implement (infrastructurally at least).

Won't fix. It's not at all clear to me that this adds significant value, and it only seems to be supported within a very few macOS apps.


Bolt is using non-puppet ruby by default on Windows

Describe the Bug
When running a simple bolt manifest on Windows through WinRM looks like it's failing before the manifest even starting. It's trying to use our system ruby C:/software/ruby24/ by default, even though Puppet Agent comes with an on-board Ruby. And indeed, I removed system Ruby from PATH and Bolt fallbacks to agent's ruby automatically. I can't leave the system without system ruby in PATH, is there a way to strictly define the version Bolt should be using?
Starting: install puppet and gather facts on win11
[win11] out: 5
[win11] out: C:\Users\builduser\AppData\Local\Temp\jeixtomp.su0
[win11] out: Version parameter not defined and agent detected. Nothing to do.
[win11] out: 5
[win11] out: C:\Users\builduser\AppData\Local\Temp\hkuemzwt.vge
[win11] err: ruby.exe : C:/software/ruby24/lib/ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in `require': cannot load such file -- puppet (LoadError)
[win11] err:     + CategoryInfo          : NotSpecified: (C:/software/ruby24...pet (LoadError):String) [], RemoteException
[win11] err:     + FullyQualifiedErrorId : NativeCommandError
[win11] err:    from C:/software/ruby24/lib/ruby/2.4.0/rubygems/core_ext/kernel_require.rb:55:in `require' from C:/Users/builduser/AppData/Local/Temp/hkuemzwt.vge/custom_facts.rb:5:in `<main>'
Finished: install puppet and gather facts with 1 failure in 22.88 sec
Starting: apply catalog on
Finished: apply catalog with 0 failures in 0.0 sec
Failed on 1 target: win11

Expected Behavior
Bolt should use Puppet Agent's onboard ruby by default.
Steps to Reproduce
Steps to reproduce the behavior:

Install ruby
Make sure it's in the system PATH

Environment

Version 3.21.0
Platform Windows 11

I wasn't installing Bolt on Windows. I assume Bolt operates through Puppet Agent and there is none it tried to install one. So I thought Bolt could be used as a starting point to bootstrap clean Windows machines from scratch, where Bolt will setup everything it needs, including Chocolatey.
And no, I have not used apply_prep. Though I do not have access to that environment anymore so I won't be able to test this exact issue anymore.
Though, let me know if I am wrong in assuming the above.


Build output is not available to guests

Currently guests do not have access to build output as co-debugging initiates only after build and launch has competed.  Build output should be shared once a build begins even if the actual debugging session does not kick off until some time later.

@Chuxel I certainly think this would be helpful to get the build output on a guest view. Ran into an issue building earlier this week that we though was because of LS (turned out to be completely unrelated). At the time we were in the same room so I was able to go over to his workstation and look at the build output and was able to track down the issue. A lot of the value we're seeing in LS is quick collaboration to debug/troubleshoot issues. Working with C# there's not too many pain points with just getting something to build, but for other languages I could certainly see getting the build output being helpful for just getting a project to build in the first place

@RyanWilsonBIBend That's great feedback! We definitely view build output as a core piece of context that needs to be shared, for the exact reason you mentioned. While we certainly don't expect Live Share to completely replace existing forms of collaboration (e.g. walking over to someone's desk), we certainly want it to be able to accomodate the most common developer scenarios, and identifying/diagnosing build errors is something we've heard pretty frequently thus far.

This has been resolved in the latest VS Live Share update 🙌 Let us know if you run into any issues or questions with the experience, and thanks again for the feedback!


Transition from web.config to appsettings

Incremental Configuration Migration
A lot of code uses System.Configuration.ConfigurationManager.[AppSettings|ConnectionStrings] and migrating away from that may be difficult. This is somewhat similar to HttpContext.Current as it is a static property that had always been available, and now the option to replace it requires rearchitecting things to use IOptions<> or some other pattern to propagate configuration.
As applications started to move to the cloud, ConfigurationManager didn't work as well as it expected all values to come from local sources. With .NET 4.7.1, .NET Framework added configuration buidlers that provide a way to populate the values of ConfigurationManager from different sources. This infrastructure is not available in the .NET Core implementation of System.Configuration.ConfigurationManager.
Potential directions
There are a number of potential directions we could go:

Bring over configuration builders to .NET Core. This is potentially a lot of code that would mimic what IConfigurationSource does, but in a different way.
Provide analyzers/codefixers to identify and refactor code using ConfigurationManager. This is difficult as the static AppSettings/ConnectionStrings doesn't have an analog and would probably require a sizeable refactoring to support.
Enable IConfiguration and ConfigurationManager to stay in sync of what values they are exposing.

(3) appears to be the simplest option and will be the basis for this proposal.
Design Goals
This proposal is built on the following principles to enable an incremental configuration migration:

Values must be able to be accessed via ConfigurationManager.[AppSettings|ConnectionStrings] so legacy code continues to function
Values must be accessible via IConfiguration to facilitate migration away from ConfigurationManager
Values must be able to be updated externally to the app (i.e. environment variables, AppService, etc)

The following are commonly discussed, but violate at least one of these principles:

Recommending replacing web.config with appsettings.json: This can solve (2) and (3), but fails to satisfy (1)
Support web.config being loaded by ConfigurationManager: By default, this file is not loaded, and must be renamed to [main-assembly].dll.config to be loaded. This can solve (1), but does not satisfy (2) or (3)
Support web.config as an IConfiguration source: This achieves (2) and (3), but fails to satisfy (1)
Continue using ConfigurationManager where you are, but update incrementally to IConfiguration: This fails all 3 principles

Proposal
A migration path can be created by combining the recommendations into a single approach:

We add support for web.config/app.config as a source for IConfiguration
We add support to update ConfigurationManager entries with values from IConfiguration (and potentially support reloading)

By supporting these two scenarios, we can enable users to use their current set up, but also have a pathway to migrate to IConfiguration as the data in both of the configuration storage systems will be the same.
Example
An example of how this could work is available here. The usage in a core app would be simple:
var builder = WebApplication.CreateBuilder(args);

// Register `web.config` for `IConfiguration`
builder.Configuration.AddWebConfig();

// Enable binding `IConfiguration` to `ConfigurationManager` (with potential for reloads
builder.Services.AddConfigurationManager();

Challenges
There are a few challenges with this:

System.Configuration.ConfigurationManager.ConnectionStrings is readonly. The POC works around this by some reflection trickery, but we would want to handle this better
If you link a file (i.e. web.config) from a separate project into the core project, it will work fine on publish. However, on F5 debug it will not find the web.config file because the content root is the project directory when debugging and not the published output.

Original Posting
When migrating to core, configuration is drastically different. In framework, people would often use web.config as the source of configuration, but in core, the source of configuration is composed from many sources. Out of the box, core will read from environment variables, appsettings.json, and command line. Additional sources are available from NuGet as well.
During a migration, with a framework and a core app running, this may cause a system to have to duplicate the configuration values and can create keeping them in sync. There are a few options we should explore to ensure this pathway is seemless:

Explore adding web.config as a potential source for configuration values
Point to using config builds in ASP.NET framework to move the source of the config to some other file

Some APIs that may be related here:

HttpContext.GetSection
HttpContext.GetGlobalResourceObject
HttpContext.GetLocalResourceObject

While thinking about this some more, I realized an additional challenge.  Users will most likely be using System.Configuration.ConfigurationManager.AppSettings to grab the values throughout the code. This makes anything to use Microsoft.Extensions.Configuration more work in the migration process and potentially difficult to update all the code.
In order to help, we may want to initially ensure that the values from web.config show up in AppSettings - this moves the migration for configuration after they have successfully moved to core. I have a prototype of this in https://github.com/dotnet/systemweb-adapters/tree/tasou/web-config that adds some msbuild logic:

When adding Microsoft.AspNetCore.SystemWebAdapters.CoreServices, some msbuild targets will be injected into their project
These tasks search for any web.config entries
An XSL transformation is performed on the supplied web.config that will keep just the AppSettings/ConnectionStrings
The final web.config is placed next to the output with the name of [ProjectName].dll.config so that the configuration manager will pick it up.

Some open questions:

ConfigurationManager only appears to read the settings automatically if the name is [ProjectName].[exe|dll|.config. If that's the case, we may need to merge with any existing config
We'd probably want to help people move off of it - maybe adding an analyzer that flags usage of ConfigruationManager.[AppSettings|ConnectionStrings] and pushes people to the optinos pattern

We talked about this one today to see if it should be considered. It is unclear if a) this problem is common enough were we want to provide a solution, and b) if this is the right approach to addressing this as opposed to having aspnetcore be able to understand web.config. @twsouthwick will follow up on this.


Heading Attributes

Heading Attributes
User Story
AS A marketing agency
I WANT a codebase that follows accessibility standards
SO THAT our own site is optimized for search engines
Acceptance Criteria
GIVEN a webpage meets accessibility standards
WHEN I view the heading attributes
THEN they fall in sequential order

changed footer to h4 to ensure sequential order in HTML; moved content CSS above benefit CSS to ensure sequential order in CSS

reopened to problem-solve how to create proper headings that meet accessibility standards but are not visible on the page/

reorganized HTML attributes to be sequential and work with screen reader without changing the visible website


Cache raster redirects longer

Haven't seen error reports on these, so let's cache these a bit longer to reduce our server load.

Warnings

:warning:
This PR modified the server but none of its tests. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS against a1e0db24c17aad0c779a8f42a631b2c8deeb0a7c


bump microsoft/hcsshim to 0.8.7

Update vendored dependency hcsshim to v0.8.7
It has been a long time since the last version update, some bugs were fixed in the middle, and a new version was finally released recently
https://github.com/microsoft/hcsshim/releases/tag/v0.8.7

https://github.com/kubernetes/kubernetes/pull/86958

LGTM. Not sure what the DCO issue is since this is vendor only.

DCO is still required on vendor commits

@dmcgowan - LOL. Its the signature from this commit. :). I thought it was something wrong with vendored code

@wawa0210 - Use git commit -s

@wawa0210 - Use git commit -s

Done.

Your files commited into vendor/ don't seem to match what a run of vndr with that hash is expecting; CI output says these files don't match:
These files were modified:
 D vendor/github.com/Microsoft/hcsshim/Protobuild.toml
 D vendor/github.com/Microsoft/hcsshim/appveyor.yml
 D vendor/github.com/Microsoft/hcsshim/functional_tests.ps1
 D vendor/github.com/Microsoft/hcsshim/go.sum
 M vendor/github.com/Microsoft/hcsshim/hcn/hcn.go
 M vendor/github.com/Microsoft/hcsshim/hcn/hcnglobals.go
 M vendor/github.com/Microsoft/hcsshim/hcn/hcnloadbalancer.go
 M vendor/github.com/Microsoft/hcsshim/hcn/hcnsupport.go
 M vendor/github.com/Microsoft/hcsshim/internal/hcs/process.go
 M vendor/github.com/Microsoft/hcsshim/internal/hcs/system.go
 M vendor/github.com/Microsoft/hcsshim/internal/vmcompute/vmcompute.go
 M vendor/github.com/Microsoft/hcsshim/osversion/windowsbuilds.go

Can you re-run vndr and see if you get changes?

Please sign (git commit -a -s --amend)

Done

vndr

I am very sorry, I am not very familiar with vndr. It seems that after implementing vndr, a lot of redundant packages have been added.

Your files commited into vendor/ don't seem to match what a run of vndr with that hash is expecting; CI output says these files don't match:
These files were modified:
 D vendor/github.com/Microsoft/hcsshim/Protobuild.toml
 D vendor/github.com/Microsoft/hcsshim/appveyor.yml
 D vendor/github.com/Microsoft/hcsshim/functional_tests.ps1
 D vendor/github.com/Microsoft/hcsshim/go.sum
 M vendor/github.com/Microsoft/hcsshim/hcn/hcn.go
 M vendor/github.com/Microsoft/hcsshim/hcn/hcnglobals.go
 M vendor/github.com/Microsoft/hcsshim/hcn/hcnloadbalancer.go
 M vendor/github.com/Microsoft/hcsshim/hcn/hcnsupport.go
 M vendor/github.com/Microsoft/hcsshim/internal/hcs/process.go
 M vendor/github.com/Microsoft/hcsshim/internal/hcs/system.go
 M vendor/github.com/Microsoft/hcsshim/internal/vmcompute/vmcompute.go
 M vendor/github.com/Microsoft/hcsshim/osversion/windowsbuilds.go

Can you re-run vndr and see if you get changes?
Edit: You might also be running an old version of vndr--try to update with go get -u github.com/LK4D4/vndr

I am very sorry, I am not very familiar with vndr. It seems that after implementing vndr, a lot of redundant packages have been added.
Can you give me some pointers

Looks like it worked fine for me: https://github.com/containerd/containerd/compare/master...estesp:update-hcsshim-vendor?expand=1
I can open a PR, but if you are wanting to make a contribution I can wait for you to attempt the same steps:
# go to your containerd clone and make sure you are up to date with master

$ cd ~/go/src/github.com/containerd/containerd
$ go get -u github.com/LK4D4/vndr
$ vi vendor.conf

# update the one line change for hcsshim to the new hash

$ vndr
$ git add vendor*
$ git commit
$ git push ...

This worked perfectly for me and generated the update to 10 files in vendor/, which should be what you get, and the 11th file in the commit is the modified vendor.conf.

Thank you very much. I used to operate on centos and it was very weird. Later, I changed a windows computer and everything worked fine.

@estesp  The previous vendor-related errors have been fixed, and the new ci error is reported, I am a bit confused

Codecov Report

Merging #3946 into master will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3946      +/-   ##
==========================================
+ Coverage   42.41%   42.63%   +0.21%
==========================================
  Files         130      130
  Lines       14738    14759      +21
==========================================
+ Hits         6251     6292      +41
+ Misses       7567     7548      -19
+ Partials      920      919       -1

Flag
Coverage Δ

#linux
46% <ø> (+0.21%)
:arrow_up:

#windows
38.22% <ø> (+0.28%)
:arrow_up:

Impacted Files
Coverage Δ

snapshots/btrfs/btrfs.go
57.39% <0%> (-0.9%)
:arrow_down:

metadata/snapshot.go
47.57% <0%> (-0.15%)
:arrow_down:

platforms/platforms.go
80.21% <0%> (+0.87%)
:arrow_up:

filters/parser.go
81.65% <0%> (+1.28%)
:arrow_up:

filters/scanner.go
82.19% <0%> (+13.83%)
:arrow_up:

Continue to review full report at Codecov.

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


CatClearLag deleting celestial crystals from Astral Sorcery

CatClearLag deletes celestial crystals sitting in starlight with /re -h. They are whitelisted, but they are still cleared. The crystal does change nbt, but the difference is maybe that the item is sitting in a liquid.

Update

I think i have the latest version...


Feature Request: 3-speed Fan Control IR device as single Home app fan device

Problem
Switchbot IR fan device with 3 speed buttons and 1 Off button shows up in Home app as On/Off fan without any possibility to switch different speeds.
Solution
Fan device with speed control in Home app which turns on Speed 1 switchbot IR control button when for example speed set in Home app between 1-30%, turns on Speed 2 button when set 30-60% etc...
Alternatives
Alternatively I have created 3 fan devices in switchbot app which each turns On different speed and then there are 3 single On/Off fans in Home app each for turning different fan speed. Works but one single fan device would be better.
Additional context
Using switchbot plugin version 3.4.0 because of issue #1021 - so if it is already implemented, please let me know. When I check the box Enable speed control nothing happens, device in Home app is still On/Off fan.

Closing all issues now that v4.0.0 is out. If you are still having an issue after trying v4.0.0, please open a new issue. Keep in mind v4.0.0 has some breaking changes with the config. You will need to re set your config up, however there are some nice added bonuses since you can set settings at a deviceType level or individual level.


chore: upgrade vite version

#1489

The test is failing. I guess the override is still needed to make sure a single version of Vite installed.

#1505


Wikipedia parity

I was browsing the wikipedia list of status codes and I noticed some discrepancies between python-status and this page: https://en.wikipedia.org/wiki/List_of_HTTP_status_codes
This pull request should bring python-status into parity with the wikipedia list of HTTP status codes as of 2016-02-26

Hey, thanks for the PR! Can you squash all your commits into one and update the PR?

Whoops, looks like force pushing closes PRs

Problem solved. PR reopened.

Thank you!
:beer:


Added X-ETOS-ID header to all external providers

Current dependencies on/for this PR:

main

PR #54

PR #55

PR #56

PR #57   (this PR)

PR #58

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: ⏳ Queued to merge
This pull request is currently queued to be merged as part of a stack.
This comment was auto-generated by Graphite.

Job Reference: BlPbstb4LcZJpY2xQRx9


Set 1st email/phone no. to be primary /app/contact

Conditions:
Only if users didn't check 'Is Primary' in Email ID table. and
Only if users didn't check 'Is Primary Phone' and 'Is Primary Mobiles' in contact number table.

Function works correctly✅ but has 'contact not found' warning. Please investigate and fix this issue.

I found 2 new issue related to this function.

Issue 1

Contact below is contact that I left all primary checkbox as a blank. Then, I deselected one of it.
and wait for a moment (or sometime refresh the page). Checkbox that I've deselected was selected again.

Issue 2
2. Contact didn't display email/phone number in preview even it's primary.

How to reproduce an issue.

Go to customer
Add contact by using this button

Add detail and don't check primary option.

Contact is created without primary mark.

Go into that contact again and you will see that primary option is just checked.

Come back to linked customer and you will see the problem.

Fixed!! Now user can deselect the option to make primary.

Issue 1 is fixed.✅ Issue 2 is still persisting.❌

Issue no.2 is still persisting.

Expectation⤵
If meet the condition, phone no. will be both 'Primary Phone' and 'Primary Mobile' like the image below.

What's happening now.
'No Primary Mobile'

The issue above caused this issue. No Primary Mobile, then no phone mobile no. display here.

@Muzmal This is persisting. Can you check from this video?

[ ] 'No Primary Mobile' issue
[ ] Customer Primary Contact was selected but no email id and phone no. show up.

https://www.loom.com/share/d30a04b312be4d7283de145b44f83d2d?sid=3ab93ba0-fac0-47b4-b6f2-0b870bae881c

Fixed. you can check

Primary phone option is working fine now.✅
Only Customer Primary Contact preview is pending.

Need solution from Umer.
So, will continue this after Muzammal and Umer don't have any urgent task left.

I tested with three cases: Phone number + Email, Email only, and Phone only.
The issue persists only with the Phone-only case. Please check the detail below.
-> if this issue took >1.5hr, please contact me for the next step.
Phone no. + Email✅

Email only✅

Phone only❌

Issue above is fixed.✅ @Muzmal
But I just found issue when 'Saving' the document and 1 question.

In this video, first, 'Did we have any script that have added to hide Enabled label??'

Please focus on 'Not Saved' label. After I clicked SAVE, document was saved for a sec and instantly changed to Not Save. So I need to click SAVE button again to resolve this issue.
https://www.loom.com/share/1efef102be29475cbfaa712ef6589fbd?sid=6cb28c3e-4ca4-48f9-a51c-603c94c524bb

This is since Jan 8, please focus or takes contact with senior and close it within next few days @Muzmal


[libconfig] Cleanup, fix libconfig++ cmake export

Fixes https://github.com/microsoft/vcpkg/issues/30906.
Minimal patch here.  Upstream variant: https://github.com/hyperrealm/libconfig/pull/234

CC @HunterZ.

Usage test passed with x64-windows triplet.


sf apex run test : --output-dir parameter ignored

Summary
When I run the sf apex run test command, the --output-dir parameter is ignored.
Steps To Reproduce
Run sf apex run test command with --output-dir parameter.
Expected result
File in the output directory specified with the test run ID.
Actual result
Nothing
System Information
Bash
$ sf version --verbose --json
{
"cliVersion"<EMAIL_ADDRESS>"architecture": "win32-x64",
"nodeVersion": "node-v18.15.0",
"osVersion": "Windows_NT 10.0.19044",
"shell": "C:\Program Files\Git\usr\bin\bash.exe",
"rootPath": "C:\Users\A0468184\AppData\Local\sf\client\2.0.2-3bd4327",
"pluginVersions": [
"@oclif/plugin-autocomplete 2.3.1 (core)",
"@oclif/plugin-commands 2.2.17 (core)",
"@oclif/plugin-help 5.2.11 (core)",
"@oclif/plugin-not-found 2.3.26 (core)",
"@oclif/plugin-plugins 3.1.6 (core)",
"@oclif/plugin-search 0.0.18 (core)",
"@oclif/plugin-update 3.1.22 (core)",
"@oclif/plugin-version 1.3.6 (core)",
"@oclif/plugin-warn-if-update-available 2.0.41 (core)",
"@oclif/plugin-which 2.2.22 (core)",
"@salesforce/cli 2.0.2 (core)",
"apex 2.3.4 (core)",
"auth 2.8.4 (core)",
"data 2.4.5 (core)",
"deploy-retrieve 1.13.5 (core)",
"info 2.6.25 (core)",
"limits 2.3.23 (core)",
"login 1.2.15 (core)",
"org 2.9.19 (core)",
"schema 2.3.17 (core)",
"settings 1.4.17 (core)",
"sobject 0.1.32 (core)",
"source 2.10.22 (core)",
"telemetry 2.2.1 (core)",
"templates 55.5.2 (core)",
"trust 2.4.27 (core)",
"user 2.3.20 (core)"
]
}
Additional information

That starts an asynchronous test run, so the command finishes without any results, so there's nothing to put in your dir.
the output is trying to tell you what to do:
sf apex run test -o UAT -l RunLocalTests --output-dir testResults --synchronous would get results, which would then be savd
(--wait with some large enough number also works)

Sorry but this is not correct.
That should create a testrunID.txt file with the ID of the test run. Then we can use this ID to build the test report.
It was the case with sfdx. I have a CI/CD job that fail because of this file no more created.
How can I schedule my CI/CD tests jobs with this behavior?

Oh, wow.  I never knew it did that (that's awful!).  This would also be the same in sfdx since it's the same plugin underneath.
If you're trying to script with the CLI, use --json output.  If you need a file written (ex: for artifacts for later user) then pipe the --json output to a file.
Json output is considered sacred because of these kind of use cases and there's tests to make sure we don't break the format.
apexTestRepro ../../plugin-apex/bin/dev apex:run:test --class-names Test --json > out.json
➜  apexTestRepro cat out.json
{
  "status": 0,
  "result": {
    "testRunId": "7078I00001uqDzk"
  },
  "warnings": []
}

We *could* try to restore the old behavior, but I'd rather

yep, it's still asynchronous (unless you tell it to wait).
I think I can restore the old file behavior.
use jq for parsing json.  You could even > the output from it to your file while you wait for the fix.
$ sf apex run test --class-names Test --json | jq -r '.result.testRunId'
7078I00001uqH3h


Lab 4 knitting to HTML

Do we have to knit this lab to a HTML document?  The instructions say to simply run the app to see if it functions.  I made a second markdown document with the YAML header output switched to HTML, which works, but the graph's appearance isn't the same. I also had to use {r echo = FALSE} to make the code disappear so you could see the selection box over the graph, otherwise the code took up too much space requiring the user to scroll down to see their input.

Andrew,
You just need to submit the lab as a .RMD file. I will be running each file to evaluate them when grading.


fix: prevent currentSlide from going out of bounds of totalSlides

What:
Prevent currentSlide from going out of bounds of totalSlides.
Why:
It's possible for a component to dynamically change the number of slides. If the number of slides is reduced such that currentSlide is past totalSlides, it results in a blank view.
How:
On CarouselProviders update, currentSlide is being constrained to totalSlides - 1 (i.e. the last slide).
Checklist:

~[ ] Documentation added/updated~ N/A, bugfix
~[ ] Typescript definitions updated~ N/A, no type changes
[x] Tests added and passing
[x] Ready to be merged

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


not seeing error when I have a fake url

Testing #10911

The error should only show up when you hit enter.


Select messages in chat and email it

Add option to select individual or all messages and email them to yourself of any email. This would be helpful in situations where you discussed something important enough that leaving it in the chat log might now be the best option.
Another good option would be to just select some of the messages and then hit copy to the clipboard from the menu. This would avoid having to copy individual messages several times and it would make easier to keep notes/logs with less clutter.

Other ideas:

Copy to a task or wiki page
Forward to another XMPP contact or to a room

Different but related task: Each message should be bookmarkable
Ex.: https://irc.tiki.org/irclogger_log/tikiwiki?date=2019-01-22,Tue&sel=92#l88

Copy to a task or wiki page

See https://github.com/igniterealtime/Pade/issues/133

Each message should be bookmarkable

See https://github.com/conversejs/converse.js/issues/1497

Add option to select individual or all messages and email them to yourself of any email.

There are many ways to do this, but the path of least resistance for me is to use a modal window to set the start and end time stamps and then make a REST call to the Openfire Chat API plugin to extract the conversation, create a PDF file and use the mailto: URI to post it by email

It is cool to create a pdf as well, but I don’t know if I would want a pdf all the time. Mostly I would like to have it as searchable and editable text.
As you suggested it seems that it would create a block with start and end time stamps. So if I want to be picky and not save all other messages in that timeframe I would have to send multiple emails., and in some large rooms that might be chats that overlap, I think selecting individual messages and email them as text would be better.
Would it use the server as the mail provider or would the user have to use some other server?

I think selecting individual messages and email them as text would be better.

It would be better, but it will be challenging to do the the UI for cherry picking messages. Personally, I would prefer to read all messages within a time-frame for context when viewing much later in the future.

Mostly I would like to have it as searchable and editable text.

Best tool at the moment would be standard browser search (CTRL F), select text, copy (CTRL C) and paste (CTRL V)

Shouldnt these features be available in converse rather than pade?

Shouldnt these features be available in converse rather than pade?

Probably, but I suspect that the real feature you need for taking notes and creating tasks during groupchat is existing issue  https://github.com/igniterealtime/Pade/issues/119

I consider this issue to be same as #195


[tokenizers] load truncation and padding settings from options, updat…

[tokenizers] load truncation and padding settings from options, update apis
Description

loading truncation/padding settings from options when creating new tokenizer instance
Updated APIs for setting truncation/padding post tokenizer creation. These apis are more in line with the interface exposed by Transformers/Fast Tokenizers
Truncation and Padding behavior must be set together (can't set one without the other). This is in line with HuggingFace interfaces, and it ensures that the user cannot make a mistake by setting max_length to different values for truncation and padding

Codecov Report

Merging #1926 (4e755dd) into master (bb5073f) will decrease coverage by 2.32%.
The diff coverage is 67.01%.

@@             Coverage Diff              @@
##             master    #1926      +/-   ##
============================================
- Coverage     72.08%   69.75%   -2.33%
- Complexity     5126     5687     +561
============================================
  Files           473      543      +70
  Lines         21970    25119    +3149
  Branches       2351     2715     +364
============================================
+ Hits          15838    17523    +1685
- Misses         4925     6266    +1341
- Partials       1207     1330     +123

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

.../cv/translator/StyleTransferTranslatorFactory.java
40.00% <ø> (ø)

... and 443 more

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


fix expired certificates

generating the test TLS certifactes now just in time (JIT),
using code based on the standard pkg/crypto/tls/generate_cert.go code
fixes #10

Thanks!


Clean up operator aliases

Some of the aliases don’t conform to what the equivalent is in the XPath spec.

+ should be addition
| should be union

Also removes the & alias for consistencies sake.
Also adds unary ! as an alias for inverse

@twalpole I'm really unsure about this change, it does make sense, but it's the uncomfortable kind of breaking change which silently changes behaviour, so I'm not too keen on that.
What do you think?

I have the same concerns as you about this being a completely silent change that will just lead to different but valid XPath expressions.   I lean towards 👎  on this.   What do you think it being a configurable selection?  Either way I'm going to update Capybara to call the union method rather than using the relevant alias.

Okay, I'm closing this PR. The resulting breakage is not worth it.


.SetSound is deprecated

.SetSound (RingtoneManager.GetDefaultUri(RingtoneType.Ringtone))
is deprecated. Why update with a deprecated line, because this has been recently updated.

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

ID: 78e442e3-b387-f538-5430-e4538cf60cb9
Version Independent ID: 44dc930e-09e2-56e8-e693-18a69af0a747
Content: Local Notifications - Xamarin
Content Source: docs/android/app-fundamentals/notifications/local-notifications.md
Product: xamarin
GitHub Login: @mgmclemore
Microsoft Alias: mamcle

unless it's a Visual Studio Xamarin bug, because it underlines it green as deprecated.

You're correct .SetSound was deprecated in API 26. Ideally, apps should be using NotificationChannel.SetSound instead.
Thanks for reporting this, we'll add it to our list of things to fix up.


Add an OpenGL enum values in spv_target_env

There's now an extension to GL to allow use of SPIR-V for shader programs:
https://www.opengl.org/registry/specs/ARB/gl_spirv.txt
We probably need to add entries to spv_target_env for any GL-based environments that will use SPIR-V.  The gl_spirv extension provides validation rules.
The extension describes validation rules in seciton "A.spv.2" and allows for variations for OpenGL 4.0, 4.1, 4.2, 4.3, 4.5 due to progression of functionality in those specs.

Fixed with c296701964


Docs-new: missing version number in new docs website.

Currently old docs website docs.kanister.io has a version number visible, but new docs docs-new.kanister.io doesn't.
It looks like the old website was putting the version in dynamically in https://github.com/kanisterio/kanister/blob/master/docs/conf.py#L66
The new website should specify version it was build for, but doesn't have to use the same mechanism, it could be just a constant as long as it is updated by ./build/bump_version.sh.

@anishbista60 we already addressed it in https://github.com/kanisterio/kanister/pull/3096


Evaluate alternatives to RapidNJ

[x] IQ-TREE - @ConnorChato encountered issues to access its NJ functionality via undocumented decentTree binary
[x] FastME 2.0 - potentially faster

Need a test case to confirm reproducibility between ML, RapidNJ and others
See #158

Ran some initial speed tests overnight on samples of GISAID data. All other things being equal, fastME has better time complexity for smaller trees. I currently can't remember if the pipeline is ever going to build 2000 tip trees but that seems to be the changeover point

Visuals comparison on a 100 Tip sample...
RapidNJ (Left) FastME (Right)

Running treedist()
     symmetric.difference   branch.score.difference           path.difference quadratic.path.difference
<PHONE_NUMBER>000              0.0000644981<PHONE_NUMBER>001              0.0002468756

Looks like you have to do some re-rooting and branch rotation to reconcile the two trees, but on a glance I think they are comparable.

Right - Here's how they both look when Midpoint-Rooted

Figured out how to run Decent Tree - It does not perform better than RapidNJ (The BIONJ-R and NJ-R algorithms both preformed similairly)

Here are the Cophylo plots for
rapidNJ (left) vs. fastME (right)

rapidNJ (left) vs. decentTree (right)

Thanks for this analysis @ConnorChato - we'll stay with RapidNJ in light of these results.

Thanks for this analysis @ConnorChato - we'll stay with RapidNJ in light of these results.


[5.0] SILOptimizer: avoid accessing the class properties if not necessary in redundant load elimination

The fix here is to just swap two bail-out conditions.
It's a NFC regarding the performed optimization, but it avoids importing class properties, which is not needed.
This is a speculative fix for rdar://problem/45806457

@swift-ci test

Build failed
Swift Test Linux Platform
Git Sha - 4634880e767051b361963d69940af77e9ab6f8a3

Build failed
Swift Test OS X Platform
Git Sha - 4634880e767051b361963d69940af77e9ab6f8a3

@swift-ci smoke test macos

@swift-ci test macos


move cluster name to the begining of gcp infra and workload machinesets

Description
https://github.com/cloud-bulldozer/scale-ci-deploy/issues/169
https://github.com/cloud-bulldozer/scale-ci-deploy/issues/170
Fixes
From To Hung in QE team:

We do see this problem (firewall not cleaned up) when a machine is created using yaml especially if the machine does not have a name that matches the cluster infra id.
Most, if not all, of our automation do not create machine using a yaml.

So If we include cluster infra id to the machine name, both the firewall and storage issues could be fixed.
https://github.com/cloud-bulldozer/scale-ci-deploy/pull/171 did a change to include cluster-name in name of gcp infra and workload machinesets, but the cluster-name is in the middle.
After reading https://coreos.slack.com/archives/C68TNFWA2/p1631654865169700?thread_ts=1631629766.139500&cid=C68TNFWA2, I checked the code, destroyer checks clustername as prefix https://github.com/openshift/installer/blob/422e17192462f2b5d5572d8600096949d3c14634/pkg/destroy/gcp/gcp.go#L220, so begin with clustername is necessary, that means we need to change name: infra-{{cluster_name.stdout}}-a to something like name: {{cluster_name.stdout}}-infra-a. @mohit-sheth Please help to review and let me know if you think there is any side effect of moving 'infra' and 'workload' from beginning to the middle.

@kedark3 FYI

Can one of the admins verify this patch?


Viper configuration support for model data

Is your feature request related to a problem? Please describe.
There will be a lot of shared metadata between models where a configuration file could provide enhanced automation with the context that is missing.
Describe the solution you'd like

Given a lula config file exists
When any action is taken that produces OSCAL artifacts
Then Viper could provide an interface for exposes data to be included

Additional context
Potential to explore other automation or preferences for how Lula is configured or executed.

Closing this issue as duplicative and more specific than areas of discovery we want to perform more design analysis around.


Rollout #121

Rollout #121

canary - Pending
uswest - Pending (run after canary)
useast - Pending (run after canary)


use xml open capability from pmp instead of xcdat

@acordonez I found this has no impact, but would you mind double checking with mean climate demo in your env?

@acordonez thanks for checking!


It's annoying to have to sign in frequently

Right now, when you close the browser, you get signed out. This is annoying. We should persist cookies and not clear them, so that users ~never have to log back in.

https://github.com/shaldengeki/fitbit-challenges/pull/254 might solve this; let's see.

I think this is fixed! 🎉


making changes to enable splunk as log destination

adding operator API spec for splunk storage
making changes to log collector controller
making changes to fluentd to render splunk env vars
adding test case to render splunk, updating others

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

 All committers have signed the CLA.


Syncthing relies on insecure version of go

Describe the bug
Syncthing refuses to install from the unstable channel
To Reproduce
Steps to reproduce the behavior:

nix-shell -p syncthing

error: Package ‘go-1.14.15’ in /nix/store/pspr3icn3frvzv0nnqikcsd0qw22vpk9-nixos-20.09.3223.38eaa62f283/nixos/pkgs/development/compilers/go/1.14.nix:247 is marked as insecure, refusing to evaluate.
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.97, NixOS, 20.09.3160.5c53c720ff6 (Nightingale)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.10`
 - channels(root): `"nixos-20.09.3223.38eaa62f283, nixos-unstable-21.03pre249162.1dc37370c48"`
 - channels(alex): `"unstable-21.05pre271444.9816b99e71c"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@peterhoeg @andrew-d

Hmm, I'm getting the same issue on release actually:
error.txt
System info:
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.60, NixOS, 20.09.3157.95ce0f52ec1 (Nightingale)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.10`
 - channels(root): `"nixos-20.09.3223.38eaa62f283"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`
 - ```

There was a [recent merge](https://github.com/NixOS/nixpkgs/pull/113809) which I hoped it'd fix the 20.09 at least, apparently not.

This is because the NixOS 20.09 channel hasn't advanced to that version yet. Presumably it is waiting for some rebuilds to finish and be published to the binary cache.
You can see where the channel is at on the nixos-20.09 branch and the changes yet to hit the channel using the GitHub compare feature.
I briefly checked on Hydra and the synctest builds are green again for release-20.09, so syncthing should be working again when the channel advances.
All in all this backport caused unfortunate evaluation errors which could have been prevented for little gain as there were no actual known Vulnerabilities for Go 1.14 (afaik at least), so there was no immediate necessety to get it of people's machines.

IMHO it was correct to update the go version as soon as it got unsupported. I would assume that security tests are not done against EOL versions as much as against supported versions.

IMHO it was correct to update the go version as soon as it got unsupported. I would assume that security tests are not done against EOL versions as much as against supported versions.

Fair enough, but it should not be done if it breaks evaluation of packages on NixOS stable. The correct way to implement that change would've been to patch/update packages that depend on Go 1.14 first and then remove / mark as insecure Go 1.14, so there is no intermittent breakage for stable users.

I see this as a larger failure in the maintenance of the Go package itself.
We know that only the two latest Go releases are supported and we roughly know the release schedule of Go so I think this could have been entirely prevented by having a process at NixOS release branch-off.

The channel has since advanced, so this issue can be closed.


fix: update import and Sass var reference #75

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #75
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
This commit removes an unnecessary import of design tokens into the local scope. It also updates a Sass variable reference to use interpolation to process the Sass variable into a CSS value.
On branch dsande/sassError
Changes to be committed:
modified:   demo/api.min.js
modified:   demo/index.min.js
modified:   src/style.scss
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team
Summary by Sourcery
Fix import and Sass variable reference by removing an unnecessary import and updating a Sass variable to use interpolation for CSS value processing.
Bug Fixes:

Remove unnecessary import of Sass custom properties to prevent redundant code inclusion.

Enhancements:

Update Sass variable reference to use interpolation for processing into a CSS value.

Output CSS is validated
[auro-icon] {
  --ds-auro-icon-size: var(--ds-size-300, 1.5rem);
  margin-right: var(--ds-size-100, 0.5rem);
}

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Just food for thought.

I do find as great as this library is, its missing some general features which I think are quite important.
The first (unless Im missing something) is the fact that you dont show the actual version detected.
Its nice to know its IE whatever but I actually need to know what version that IE is trying to be.
Currently Im having to use two libraries. I could do some Regex on parts of yours but its seems like something it should have.
This is an example as to what (Im guessing) most are using in some form. But notice in my case the two libraries.
SomePage.htm

<title>Browser</title>

<meta name="viewport" content="width=device-width, initial-scale=1.0, shrink-to-fit=no">
<meta name="robots" content="noindex,nofollow">

Hi @WHBitbucket!
This isn't really intended as a sniffing or detection lib to toggle features.
It's more for reporting purposes.

Its nice to know its IE whatever but I actually need to know what version that IE is trying to be.

platform.version does report the version of the browser, so it would be great if you could clarify what this means.
As mentioned though, for feature detection, you should probably use another library such as Modernizr as Platform.js is more for providing detailed information on the platform.

It quite a while ago I wrote this but from memory the main "issue" was platform.version does correctly tell you the actual browser. It doesn't inform you if the browser is pretending to be a different version. I think in this use case (from memory) it was because IE 10 on an intranet pretends its IE 8. platform.version reports IE 10 which is correct. It would be nice to maybe have an additional platform.pretending.to.be.version or suchlike.


dockerfiles/pixiecore/Dockerfile: glide -> dep

fix #78
I don't know if there's an alternative to glide nv but I saw some project doing it like this (run dep ensure and then go test $(glide nv))

Any intent to merge this?


docs: Complete ReadMe \ Installation steps for Google Cloud Build Plugin

Signed-off-by: PurseChicken
Hey, I just made a Pull Request!
This PR fixes #19514
I modified the existing readme.md file to include all the information required to successfully install and use the Google Cloud Build plugin.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

The pipeline error may be fixed by running prettier:
https://github.com/backstage/backstage/actions/runs/6793075030/job/18467374984?pr=21069

Thanks for this :tada:


x11forwarding not woking

VSCode Version:1.24.1
OS Version:centos7

I use vscode via x11forwarding. I'm sure that my environment is ok, because I test by xterm. Vscode can't lanuch, the error:
..... [main 11:14:25 AM] Resolving machine identifier... [main 11:14:25 AM] Resolved machine identifier: ef82848e38b0b2a5622203fa032a1531839642715badf2c32f47de2aa103f2e3 [main 11:14:25 AM] update#setState idle [26794:0704/111425.448216:ERROR:browser_main_parts.cc(122)] X IO error received (X server probably went away)

I am closing this as we do not officially support x11forwarding


Events locations update

REMOVE from site :
Launchpad Colo Week v13
July 15 - 22, 2023
Paris, France
Launchpad Colo Week v14
August 19 - 26, 2023
Location TBD
Launchpad Colo Week v11
May 6 - 13, 2023
Austin, TX

UPDATE Entry
eventName: "" #launchpad
eventDescription:
#web3 #filecoin #IPFS #libp2p
eventTopic:

"<Topic_1>"
"<Topic_2>"
"<Topic_3>"
"<Topic_4>"

eventType: "Conference" or "Virtual" or "Social" #Type of the event: Please choose one among the below options or just leave it blank
location: "Paris, France"
venueName:
venueAddress:
venueMapsLink:
website: "<your_website_url>" #make sure to have all the relevant information: dates, venue, program, ticketing (if any), etc. or just leave it blank
startDate: "07/16/2023"
endDate: "07/22/2023"
tag: "PLN Event"
#Preferred means of contacts -<EMAIL_ADDRESS>preferredContacts:
#Event host names and their respective logos(preferred size is 48px width, 48px height)-
eventHosts:
Comments:
Update to current Launchpad Colo Week v12 dates and location, please do not add another entry.

Please take proper pull from the main branch (as it shows 60 files changed) and commit only those files that you require changes/updates.

Merged the above mentioned changes using the PR - https://github.com/memser-spaceport/pln-events/pull/335
@pandeyanujk @brookeoneill


Support for Cloud Native Buildpacks as a workload.

Summary
Cloud Native Buildpacks are one of the standard ways to deploy applications to Platforms as a Service. They're in use by PaaS such as Heroku and Cloud Foundry. Buildpacks are enjoyed by developers because of their ease of use (almost 0 manual configuration).
Adding support for Cloud Native Buildpacks would complement the other workloads that DevTron can deploy.
Motivation
Buildpacks fulfill the case of simply being to write code without configuration of the underlying infrastructure, especially helpful for twelve-factor apps. Users looking for simplicity may choose to deploy a Buildpack using DevTron similar to other platforms that provide AppOps such as Heroku or certain Tanzu features.
Proposal
Create a workflow to deploy apps using Buildpacks. Add relevant templates, CI/CD pipelines.

Support was added in https://github.com/devtron-labs/devtron/pull/2562


Issue #372: Add automatic module name to core

Libraries without a set automatic module name causes build warnings for
all clients who build with Java >= 9.
Also bump version of the Exec library, to use a version which has the
automatic module name set.
The following pull request for Exec adds an automatic module name:
https://github.com/vorburger/ch.vorburger.exec/pull/40

This will fail to build unless a new https://github.com/vorburger/ch.vorburger.exec v3.1.2 is released.. I'll see if I can get to that some time.

Hey @vorburger , I don't want to bug you about this, but do you think we could have some progress on this? It's been almost 4 years since Java 9 was released.
Running tests on the module path really requires stable module names. Using something like requires exec is highly unstable and will break as soon as another jar called "exec" is introduced to the module path.

See #334.


[FEATURE REQUEST] Improve Elasticsearch cluster status checking during an upgrade

Is your feature request related to a problem? Please describe.
Currently, during an upgrade we checking if cluster status is yellow or green.
Describe the solution you'd like
During an upgrade procedure, verify if we have single instance of Elasticsearch or instance>1 and check cluster state in proper way.
Proper status:
One node = yellow
More than one node = green
Additional context
Useful calls:
GET _cat/nodes
GET _cat/allocation
GET _cluster/health
GET _cat/shards

Things to improve as well:

tests
documentation


Tutorial for Quantum Autoencoder to compress quantum states

Summary
This is a tutorial which goes through the theory of a Quantum Autoencoder, followed by several examples of compressing different quantum states.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: murphytarra:x: Tara MurphyTara Murphy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 87.616%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
3028

Relevant Lines:
3456

💛  - Coveralls

Thanks for the comments. There are a few updates, see below what has been changed.

While QAE is the natural abbreviation for Quantum Auto Encoder - it is the abbreviation that is generally used for Quantum Amplitude Estimation. Better change it to QAuEn or something along those lines.

Now it is just plain "Quantum Autoencoder" to avoid any confusions.

There is a minor typo in the classical auto encoder section which has a less nodes.

Fixed.

Please try to avoid the term "ancilla". This term is connected to some negative history. The preferred terminology is "auxiliary qubit".

Good point, replaced with "auxiliary".

The goal of the algorithm should be explained in detail in the introduction.

Added some additional explanations in the introduction.

Why do you use a Swap test instead of a simple overlap? The increase in the number of qubits could be problematic for near-term applications.

The tutorial does not pretend to be the only right solution. It is merely implements what is stated in the referenced paper. Other approaches may require additional research and trials.

I suggest renaming Sec. 7. to A Quantum Auto Encoder for Digit Compression

Renamed.

What I would also like to see is a discussion on applications of quantum auto encoders. This can be very beneficial for the user's understanding.

Added a few applications.

@woodsp-ibm I have updated the tutorial to the new QNNs. I'm happy with the content, Christa had a look as well. If you could take a look it would be nice, but no pressure now. I guess, if I mark it as "stable backport" it will be backported and published on the website, right?

It needs to go over to stable to get published yes. I think the docs need to be triggered @manoelmarques
I am curious on this below though in regards of what warning is being ignored here. Whatever it is I guess its of no concern, but then if its raising some warning - or maybe this is from something its using - can it not be sorted?
# we suppress a warning that comes from RawFeatureMap
warnings.filterwarnings("ignore")

qnn = SamplerQNN(
    circuit=qc,
    input_params=fm.parameters,
    weight_params=ae.parameters,
    interpret=identity_interpret,
    output_shape=2,
)

At the start of section 5 it has some imports which still include Aer - I do not think thats needed any longer right (I cannot comment in the file at all since github says its too large to show the diff)
import json
import time
import warnings

import matplotlib.pyplot as plt
import numpy as np
from IPython.display import clear_output
from qiskit import ClassicalRegister, QuantumRegister
from qiskit import QuantumCircuit, Aer <<<-------------

At the end of the notebook, in the applications of section 8.
This is commonly used to describe electron - electron interactions in molecules.
I had to re-read that as I first tried to read it as a dash - not a hyphen.
Think it should more be (no spaces around the - so words (electron) are joined i.e. use it as a hyphen.
This is commonly used to describe electron-electron interactions in molecules.

I am curious on this below though in regards of what warning is being ignored here. Whatever it is I guess its of no concern, but then if its raising some warning - or maybe this is from something its using - can it not be sorted?
# we suppress a warning that comes from RawFeatureMap
warnings.filterwarnings("ignore")

qnn = SamplerQNN(
    circuit=qc,
    input_params=fm.parameters,
    weight_params=ae.parameters,
    interpret=identity_interpret,
    output_shape=2,
)

Not required anymore, good point. This was need  to suppress warnings from CircuitQNN with RawFeatureVector when the former tries to evaluate a gradient circuit. Removed now.

At the start of section 5 it has some imports which still include Aer - I do not think thats needed any longer right (I cannot comment in the file at all since github says its too large to show the diff)

Aer is removed now.

At the end of the notebook, in the applications of section 8.

Updated - and the numbered list, should look good now (locally it is good).


Auto create compilation album when the following conditions are met:

A song is parsed and its 'album' tag already exists in the DB
The 'artist' tag does not match the artist found from the matching album in DB
The song is in the same folder as the previous song with the matching 'album'.

Thanks, but being in the same folder doesn't guarantee anything – I used to group all of my songs under one flat folder, for instance.