opt update release verify

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Update see-a-list-of-audio-conferencing-numbers-in-teams.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/7152
Fixed link.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@tonysmit PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@tonysmit

@tonysmit Hope you are well. Wanted to follow up here to see if there is an update? Please let us know if there is anything we can do to help. Thanks.

reviewed + confirmed


Dialog separate session

Responds to this Stack Overflow issue: https://stackoverflow.com/questions/64048543/session-cookie-across-task-pane-and-dialog

Docs Build status updates of commit bda0f92:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/add-ins-with-angular2.md
:white_check_mark:Succeeded
View

docs/develop/auth-with-office-dialog-api.md
:white_check_mark:Succeeded
View

docs/develop/dialog-best-practices.md
:white_check_mark:Succeeded
View

docs/develop/parent-to-dialog.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1d3bb0e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/add-ins-with-angular2.md
:white_check_mark:Succeeded
View

docs/develop/auth-with-office-dialog-api.md
:white_check_mark:Succeeded
View

docs/develop/dialog-best-practices.md
:white_check_mark:Succeeded
View

docs/develop/parent-to-dialog.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Cross Validation

Hi, I would like to clarify how is cross validation performed. With the current settings, it seems that only 1 fold is performed. Is there a need to change the value of ALL_EV in the training shell scripts?

Yes. You train five models with different folds of data.


Feature Request: Separate Repo for Codium (or VS Code) Extension

Hello and thanks for this beautiful repo,
Do you have a separate repo / code base for the IDE extension on your current roadmap? If so, I'd be glad to test it with Ollama and write some comments too.
Best,
Orkut

Sorry for not responding sooner. The VSCode extension is launched and the code is in this repo under mentat-vscode.

Thank you @jakethekoenig :+1:


calloc: Don't zero-out memory allocated with mmap

mmap() already provides zeroed-out memory, so avoid zeroing it out in calloc().

Thanks. In the long term I intend for there to be more logic here, possibly like the existing conditional-fill in musl's old allocator (that avoids touching clean zero pages) and/or possibly using MADV_DONTNEED on largish objects. It's also possible that direct-mmap might in the future get used way below MMAP_THRESHOLD as long as page size is sufficiently small (going all the way up to 128k is mainly because of the existence of archs where pagesize could be up to 64k). But the basic fix you have isn't wrong and should be a good help for now.

Merged


Bug: HTTP_ENGINE custom configuration discarded

It's always overriding the value here https://github.com/dappnode/DAppNodePackage-teku-gnosis/blob/master/beacon-chain/entrypoint.sh#L16

same issue :-1:

We should support custom configuration although it may affect resilience, @dsimog01 @3alpha thoughts?


Fix Jest errors

Dependabot updates for Jest are failing due to a warning. I think it should be quite straight forward to fix.
See:
https://github.com/trimble-oss/modus-web-components/pull/707
https://github.com/trimble-oss/modus-web-components/pull/699

Closing as non-issue. Only Jest 27.x is supported that's why update to Jest 29.x fails with error.


Adding 2 other popular extensions for text files

I'm going to merge this, I'm actually a bit afraid that this list might grow pretty loooong over time. And as in #6 it would make sense to check for the syntax and enable on all text syntax texts. just some random thoughts

thanks for your contribution @harryngh


WIP add XYZ format and QM9 example

No longer requires an explicit check against a known dataset.
Makes RawDataLoader a base class where some methods must be overridden
Adds LSMSDataLoader and XYZDataLoader
Sets up example for QM9 dataset

Still WIP

Draft until we support multiple graph sizes

Closing to be reopened in HydraGNN


[AutoPR signalr/resource-manager] Add signarl/resource-manager/readme.typescript.md

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4097

(message created by the CI based on PR content)


[Next] Entries view cannot be filtered by tags

Create an entry in a collection that uses the tag field
Attempt to filter the entries view by a known value contained in the tag field
No match!

should be fixed now: https://github.com/COCOPi/cockpit/commit/ab44bac6380f4362399e53fb573c3d9ac3a24ed1


Unnecessary panic which causes a double panic.

at pretty_assertions_sorted-1.2.0/src/lib.rs:101. Debug print should not be able to return error. Thanks Finomnis that found the problem.

Should not be able to panic. Returning errors is fine.

Thanks for the bug report! I didn't catch that this case would cause a double-panic, definitely not intended. This is fixed in https://github.com/DarrenTsung/rust-pretty-assertions-sorted/commit/d5892e22263e74fc696013f89aee3ee96a3235b4 and published as 1.2.1.
Re: panic being bad style - Unfortunately the panic in the Debug representation is necessary in order to leverage the pretty_assertions crate as the pretty_assertions::Comparison only has an API for passing in objects that are Debug and not the raw strings.. actually scratch that, maybe passing in the Debug representation rendered as a string would work.
Either way panicking in a test is expected behavior here - the alternative would be to fallback to the unsorted debug representation and I think that should be the user's action instead of the crate automatically doing so (logging something doesn't seem enough because logs are suppressed for passing tests).


Corrupted TIMESTAMPS

Log tracing reveals errors like the following trying to insert invalid Date/Time into the database....
INSERT INTO CVN_CONVERSATIONS (partition_id, conversation_id, creator, tenant_id, title, private, creation_date, modification_date, type) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?); with values: [2,10002,2,1,"some title",false,"2023-09-16T07:34:55.430Z","2023-09-16T07:34:55.430Z",1]
ð  ko: error converting 2023-09-16 07:34:55.430tle to Date/Time

This issues has been fixed with commit 2d4b749acaddc13f3ef01734500e5327e2c3e50a
https://github.com/nuodb/node-nuodb/pull/76/commits/2d4b749acaddc13f3ef01734500e5327e2c3e50a


Sequential event UIDs cause .ics events to be omitted and break format guidelines

Hi,
Thanks for all your work making this system.
Problem
I experienced an issue in Google Calendar where ICS events between 2017 and early 2020 are being omitted. Here's what I see using Google Calendar to view all events:

Replication steps

Visit https://calendar.google.com/calendar/r/settings/addbyurl
Enter https://www.gov.uk/bank-holidays/england-and-wales.ics into the URL field
Submit and observe missing events in the resulting calendar

Alternatively you can view the the problem via the premade example calendar here
Root cause
Surprisingly, when I investigated this the missing events were found correct and present in the latest data here:
https://www.gov.uk/bank-holidays/england-and-wales.ics
For example, 2020 New Year's Day (which is missing from gcal) shows up fine:
BEGIN:VEVENT
DTEND;VALUE=DATE:20200102
DTSTART;VALUE=DATE:20200101
SUMMARY:New Year’s Day
<EMAIL_ADDRESS>SEQUENCE:0
DTSTAMP:20191224T094854Z
END:VEVENT

However it's also notable that the first event shown in Google Calendar is New Year's Day 2012. In the .ics source it's New Year's Day 2015:
BEGIN:VEVENT
DTEND;VALUE=DATE:20150102
DTSTART;VALUE=DATE:20150101
SUMMARY:New Year’s Day
<EMAIL_ADDRESS>SEQUENCE:0
DTSTAMP:20191224T094854Z
END:VEVENT

Given the scenario, the bug I'm experiencing is likely a caching issue external to this service.
However, it's also really likely that the underlying issue is this service's use of sequential UIDs on ics VEVENTS. I notice that 3 years of historical data were recently deleted. Assuming that UIDs are used for cache lookups on events, this would create cache collisions for exactly the missing period described above.
Here's the relevant section of the .ics spec stating that event UIDs should be globally unique. This calendar should definitely follow the format!
Potential solutions
I see several routes to fixing the described issue.
Any of these would work:

Ask the Google Calendar team to manually invalidate their cache for all the GOV.UK bank holiday ics URLs
Revert this commit
Reimplement this function to return globally unique ids as per the .ics standard (any GDS caching layers in front of this service would also need to be invalidated)

1 or 2 would only work if no bank holidays were ever deleted in future.
I'm happy to make a PR for the third implementation route if you need a hand. It seems like the best way forward to me.
Cheers! Happy new year!

Wow, this seems like great work! Thank you all. I'm looking forward to seeing the result go into production


Feedback 22.09.2021

[x] Add default 16 to Participant speed restriction
[x] List item deletion
[x] Import from xlsx (National) - include AD - клуб field
[x] Import from xml (International) Include Athlete club. Also add it in the AthleteView
[x] Horse FeiId and Trainer should not be required
[x] Horse and Athlete changes in ParticipantView are not updated in the parent CompetitionView
[x] HorseListView search not working?
[x] [NOT CONFIRMED] Collapse competition details by default in EnduranceEventView. When clicked - expand that Competition's data

Implemented in: https://github.com/HorseSport-achobanov/endurance-judge/pull/16


Add diegolovison as member to kubeflow org

Provide 2-3 links to PRs or other contributions.
Remove outdated versions of Kubernetes: https://github.com/kubeflow/manifests/pull/2640
Add steps to install Kubeflow on kind: https://github.com/kubeflow/manifests/pull/2634
List 2 existing members who are sponsoring your membership.
@hbelmiro @rimolive
Test your PR by running the following:
$ pytest test_org_yaml.py
/home/dlovison/miniconda3/envs/ods-ci/lib/python3.11/site-packages/pytest_logger/plugin.py:104: PytestDeprecationWarning: The hookimpl LoggerPlugin.pytest_runtest_makereport uses old-style configuration options (marks or attributes).
Please use the pytest.hookimpl(hookwrapper=True) decorator instead
 to configure the hooks.
 See https://docs.pytest.org/en/latest/deprecations.html#configuring-hook-specs-impls-using-markers
  @pytest.mark.hookwrapper
========================================================================================================= test session starts ==========================================================================================================
platform linux -- Python 3.11.5, pytest-7.4.2, pluggy-1.3.0
rootdir: /home/dlovison/github/kubeflow/internal-acls/github-orgs
plugins: logger-0.5.1
collected 1 item

test_org_yaml.py .                                                                                [100%]

========================================================================================================== 1 passed in 0.06s ===========================================================================================================

/lgtm
I am sponsoring Diego's membership


Subject Alternative Names(SAN) containing ip address not are considerd and processed

i have a self signed certifiate on my server which has combination of DNS and IP in SAN.
If any certificate has IP address present its not taken into account since its only expecting strings and processing via valueAsStrings(shown below from X509Extension file)
var valueAsStrings: [String] {
var result: [String] = []
for item in block.sub?.last?.sub?.last?.sub ?? [] {
if let name = item.value as? String {
result.append(name)
}
}
return result
}
While for DNS this works but for IP it fails since its added as Data directly in the parse method in ASN1DERDecoder file.
// custom/private tag
let contentData = try loadSubContent(iterator: &iterator)
if let str = String(data: contentData, encoding: .utf8) {
asn1obj.value = str
} else {
asn1obj.value = contentData
}
Another thing i observed was, for few IP's example(127.0.01, <IP_ADDRESS>, etc) which can be successfully encoded using UTF8 are encoded in wrong strings while others(<IP_ADDRESS>, <IP_ADDRESS>) which can't be encoded are added as Data  itself.
@filom Are you planning to include this scenario in any of te future releases?
i know its a rare case but it would be great if you could handle these.

Just add 5fdda99f0ccb4f320142af24b52006c4a6d9f450 to support multiple SAN sub-types included IP-Address.

It's now available on cocoapod version 1.5


PDB variable explorer error?

Description
What steps will reproduce the problem?

Not sure... was in pdb trace, clicked on a dict in variable explorer to see contents and then closed the window.  Then the error came up.

What is the expected output? What do you see instead?
Please provide any additional information below
File "/anaconda/lib/python3.6/site-packages/spyder/widgets/variableexplorer/collectionseditor.py", line 527, in
lambda eid=id(editor): self.editor_accepted(eid))
File "/anaconda/lib/python3.6/site-packages/spyder/widgets/variableexplorer/collectionseditor.py", line 549, in editor_accepted
self.set_value(index, conv_func(value))
File "/anaconda/lib/python3.6/site-packages/spyder/widgets/variableexplorer/collectionseditor.py", line 1370, in set_value
self.set_value_func(name, value)
File "/anaconda/lib/python3.6/site-packages/spyder/widgets/variableexplorer/namespacebrowser.py", line 301, in set_value
value = serialize_object(value)
File "/anaconda/lib/python3.6/site-packages/ipykernel/serialize.py", line 85, in serialize_object
for k in sorted(obj):
TypeError: '<' not supported between instances of 'float' and 'str'
Version and main components

Spyder Version: 3.2.1
Python Version: 3.6.1
Qt Versions: 5.6.2, PyQt5 5.6 on Darwin

Dependencies
pyflakes >=0.6.0 :  1.5.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.1 (OK)
numpy >=1.7      :  1.12.1 (OK)
sphinx >=0.6.6   :  1.5.6 (OK)
rope >=0.9.4     :  0.9.4-1 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
psutil >=0.3     :  5.2.2 (OK)
nbconvert >=4.0  :  5.1.1 (OK)
sympy >=0.7.3    :  1.0 (OK)
cython >=0.21    :  0.25.2 (OK)
qtconsole >=4.2.0:  4.3.0 (OK)
IPython >=4.0    :  5.3.0 (OK)
pylint >=0.25    :  1.6.4 (OK)

Please post a reproducible example so we can test it in our side. Without, it's hard to understand what generated this error.

Closing because of lack of response. Please reopen when you post the code for the dict you were viewing.


Main structure changed

So, like I said. I managed to change the structure of the project. The core can now be placed behind public_html (or whatever you wanna call it)

Hmm, we don't want the default structure to be changed, though. We just want to make it possible to do so...

I understand you. I know how to make it so we don't have to change the main structure at all


Download JSON Schema button in pages doesn't work

The "Download JSON Schema" button in model pages doesn't work.
Example
https://models.accordproject.org/patents/patent.html

This issue is a duplicate of #22 I'm closing it as such.


Add GB HA Dex

Create a Virtual Console/ Hidden Ability option to track Living Dex progress for R/B/Y G/S/C this would be very useful since all Virtual Console pokemon transfer up with their hidden ability

I'd like to see a living-dex option for each generation; some of us still want a gen 3 livingdex with RSE/XD/Colosseum/FRLG included!

@PsyMar see #264 if you want to track that! keeping this ticket for the HA dex request.


update the composite to draft-ietf-lamps-pq-composite-sigs-02

Added changes to composite that implement the changes from draft-ietf-lamps-pq-composite-sigs-02.

MLDSA44-RSA2048-PSS-SHA256 (mldsa44_pss2048) now uses a salt length of 256 instead of 512.
Change references pointing at new version of the draft.

Thanks for the update @feventura . ConceptuallyLGTM (without having checked the Composite spec :-) Please rebase after the Mayo PR has merged to get CI through.


Fix/support more parse dags

Reduce pressure on scheduler by consolidating parse dag tasks
Related PRs
https://github.com/blockchain-etl/ethereum-etl-airflow/pull/466
https://github.com/blockchain-etl/ethereum-etl-airflow/pull/467
https://github.com/blockchain-etl/ethereum-etl-airflow/pull/469

Sorry for the horrible commit history, not sure what's going on there.  Will squash on merge to keep things clean.


Allow setting the Slack channel from ENV

Works on my machine! Code reviews gladly accepted :)

@cdmwebs Awesome, this looks great! I agree on the copy change but other than that :+1:

Done! @jasondew @pusewicz

@cdmwebs Thanks man!

Sure thing! FYI: I didn't update the docs branch.


Add LineHeight property to AutoCompleteBox

Is your feature request related to a problem? Please describe.
I just noticed that the AutoCompleteBox control doesn't have the LineHeight property (inside the text search) like the TextBox, is it possible to add this property? I have a project where I definitely need to include this property for font customization
Describe the solution you'd like
Is it not possible to add the property as in the TextBox control only for the text search font?
Describe alternatives you've considered
No response
Additional context
No response

It should be easy to change with below selector
            <Style Selector="AutoCompleteBox /template/ TextBox">
                <Setter Property="LineHeight" Value="10"></Setter>
            </Style>

I didn't think of that, thank you very much


Transform destination port rules to header rules

Please provide a description of this PR:
Fixes #41204 when Waypoint proxy is deployed.
Tranforming destination port permission rules to header rules that checks the suffix of the authority header's value.

@ymesika: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ymesika: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

integ-operator-controller_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-operator-controller

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

integ-operator-controller_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-operator-controller

integ-helm_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-helm

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

integ-operator-controller_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-operator-controller

integ-helm_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-helm

integ-pilot-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

integ-operator-controller_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-operator-controller

integ-helm_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-helm

integ-pilot-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote

integ-pilot-istiodremote-mc_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote-mc

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

integ-operator-controller_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-operator-controller

integ-helm_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-helm

integ-pilot-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote

integ-pilot-istiodremote-mc_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security-multicluster

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

integ-operator-controller_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-operator-controller

integ-helm_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-helm

integ-pilot-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote

integ-pilot-istiodremote-mc_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security-multicluster

integ-telemetry-mc_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-mc

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

integ-operator-controller_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-operator-controller

integ-helm_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-helm

integ-pilot-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote

integ-pilot-istiodremote-mc_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security-multicluster

integ-telemetry-mc_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-mc

integ-security-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security-istiodremote

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

@ymesika: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests-arm64

unit-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test unit-tests

integ-basic-arm64_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-basic-arm64

release-test_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test release-test

analyze-tests_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test analyze-tests

integ-ds_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-ds

integ-telemetry-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-istiodremote

integ-security_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security

integ-operator-controller_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-operator-controller

integ-helm_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-helm

integ-pilot-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote

integ-pilot-istiodremote-mc_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-istiodremote-mc

integ-security-multicluster_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security-multicluster

integ-telemetry-mc_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-telemetry-mc

integ-security-istiodremote_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-security-istiodremote

integ-pilot-multicluster_istio_exp
11d08550be900f4396b56307bfe989349ab19c1f
link
true
/test integ-pilot-multicluster

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

Still relevant

Thanks John!  I was thinking the same too - opened https://github.com/istio/istio/issues/42208 to track it


Fix llama tokenization issue by using tokenizer initialized with add_bos_token=True

Description
This PR ensures that the tokenizer used by the model is initialized with add_bos_token=True.
Previously, the tokenizer that  does not prepend bos was used as default. When we wanted to prepend a bos token to the input, we manually added it to the input, e.g., tokenizer.bos_token + input. However, this resulted in a bug for LLaMA tokenizers because they behave differently when the bos token is manually prepended and when it is automatically prepended by the tokenizer (https://github.com/neelnanda-io/TransformerLens/issues/373#issuecomment-1701017935).
It is necessary to ensure that the tokenizer is initialized with from_pretrained(add_bos_token=True) because add_bos_token cannot be dynamically controlled after initialization (https://github.com/huggingface/transformers/issues/25886).
Because some tokenizers doesn't automatically prepend bos token even when they are initialized with add_bos_token=True, cfg.tokenizer_prepends_bos is introduced to check if the tokenizer does prepend bos token or not when initialized with add_bos_token=True.
The test for tokenizers are in test_only_tokenizer.py. (I also manually tested llama tokenizer using the same test.)
Fixes https://github.com/neelnanda-io/TransformerLens/issues/373
Type of change

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

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have not rewritten tests relating to key interfaces which would affect backward compatibility

Thanks! Sorry for the delayed merge.


Multi-site / Multi-platform

Hi I have a .net web app which I'm publishing from VS2017 to my EC2 instance running IIS.
I recently added a .Net Core Web App which I'm also publ;ishing to the same instance.
When I publish the original app, the new core app gets deleted.
When I publish the core app, both sites remain stable.
I'm not sure how to change my original app publishing process to protect the core app.
Please help / advise if possible.

ps the .net core app is in VS2019

The Toolkit does not support publishing more than one application to the same destination.

Well that's not strictly true, you can publish multiple apps using manifest.


Improving documentation on profile templates.

Improving documentation on profile templates where also known as custom entitlements.
Checklist

[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Struggled for over a month trying to add custom entitlements to a fastlane ci setup, only to find out that it was already supported just under a different name.
Resolves #11049
Description
Updated docs to mention 'custom entitlements' so that the template_name key is searchable using 'custom entitlements'.

ping @lukewakeford

Hey! Sorry for the delay, yes I think it should be somewhere in the text of the Match page.
Where does the list of examples get used?

I think here: https://docs.fastlane.tools/actions/match/#4-examples

Ah cool, yeah I'll take a look at this.

I don't think it needs to be in the examples as it's not a very common use case, I have added it to body of that page tho, in the fastlane section underneath Multiple Targets.


Create categories

The ability to group items under dynamically named categories is powerful.
The idea is that the user can add/remove/modify categories, and then associate items with that
category. That way when they add an item to their list it will get auto-categorized.
Ideally this has some pre-populated categories.

A category object has been created, as well as a category manager object (factory).


Add ConsumerGroup to API docs

@wddimple6 any chance we can these docs updated? I assume you're the new maintainer of this project. Is there anything I can do to help?

@twm can you confirm if this was done and ready to be merged?


Extend GCM with an ability to split partially-dead nodes

The code is based on the Benedikt Meurer's idea first implemented in V8. See: https://codereview.chromium.org/899433005

This increases the speed of minilua compiled with IR almost twice. (0.091 sec instead of 0.180)
clang -Wno-everything -O -Xclang -disable-llvm-passes -w -c -emit-llvm -o tmp.bc dynasm/minilua.c
opt tmp.bc --passes='function(mem2reg)' -S -o minilua.ll
ir --llvm-asm minilua.ll --run ./bench/mandelbrot.lua

hi @cliffclick
I'm very interested in getting your opinion about this GCM extension (and may be some related ideas).
I'm not the author of the algorithm, and it was never described in papers.

Sounds reasonable.  I wrestled with the notion, but had too many other fish to fry at the time.  Seemed low payoff, but more or less free.  Also, the X86 machine ops would clone "simple" ops for free into the larger X86 addressing mode, so again some simple/cheap cases got handled already.  Never did the general case.


Add autocorrect for RSpec/ExpectActual

Adds an autocorrect to RSpec/ExpectActual, swapping the arguments to expect and eq when possible.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the changelog if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

Do you need a hand with this @dduugg ? Please let me know if you get stuck with something.

Hi @pirj I'll close this until I have more bandwidth to cover the various examples. It seemed much simpler when I was able to autocorrect the cop's spec and my code with ease 😸


[#100410408] terraform jobs: run make prep

After merging https://github.com/alphagov/tsuru-ansible/pull/166 , ETCD_CLUSTER_ID files are prepared by make prep command. Add this to the terraform job, so that it runs correctly.

Sorry!


Move notifications to console panel.

Overview
This moves all console and logging into an easy to access console panel for users.
This will be done when the following are finished

[x] Move all logging into a single class to wrap both log operations and printing to console
[x] Implement different error levels
[ ] Add a level filter

Example

Going to pass on the log level filter for now. I think it might be more useful if I start adding debugging logs to the output but atm I haven't


Saving your code closes editor panes

Describe the Issue
I didn't test this too thoroughly, but when writing code for one of the savable projects, I saved my code and the CSS pane closed.
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/build-a-survey-form-project/build-a-survey-form
Steps to Reproduce

Go to https://www.freecodecamp.org/learn/2022/responsive-web-design/build-a-survey-form-project/build-a-survey-form
Open both the HTML and CSS editor panes
Write some code in one of them
Click the "save your code" button or press ctrl+s
The CSS pane closes

Expected behavior
The panes should stay how I had them.
Screenshots
No response
System

Device: [e.g. iPhone 6, Laptop]
OS: [e.g. iOS 14, Windows 10, Ubuntu 20.04]
Browser: [e.g. Chrome, Safari]
Version: [e.g. 22]

Additional context
Possibly somewhat related: https://github.com/freeCodeCamp/freeCodeCamp/issues/55619

Along with not keeping the styles.css tab open it is also not adding the styles to the page when the css is written in the styles.css

Can I work on this isuue?

@moT01 You have to be signed in as well. At least, it didn't happen when signed out and saving to local storage.

@sanjeevmurmu  It is applying the styles just fine for me. Did you remember to link to the CSS file?
@vinayak9303 We typically do not assign issues. Instead, we accept the first pull request that comprehensively solves the issue.
Issues labeled with help wanted or first timers only are open for contributions.
Please make sure you read our guidelines for contributing. We prioritize contributors following the instructions in our guide. Join us in our chat room or the forum if you need help contributing - our community will be happy to assist you.

I totally understand the issue you are facing. I would be really delightful to write some code and solve the problem...assign this issue to me please.

@sanjeevmurmu It is applying the styles just fine for me. Did you remember to link to the CSS file?

Yes my bad i forgot to link it as i thought it will already be linked to it

I am statring new here I cloned and extracted repo to my local machine and I made changes in the code. I am not getting how to run this locally (hoe to run this project locally ) need guide or steps can any one help me?

There's instructions linked above: contribute.freecodecamp.org/how-to-setup-freecodecamp-locally/ - is there anything specific you are stuck on @dhanushkn97 - I recommend just running the code following the instruction before making changes to make sure it's all working.


Perform a package query returns internal server error

https://api.npms.io/v2/package/react
{
code: "INTERNAL",
message: "Internal server error"
}

Is the project still being maintained?

Sorry, it's up again. Doing some migrations, upgrading to a better machine.


Limit sdk based on api config

Project organization

Dependant on https://github.com/liteflow-labs/libraries/pull/64
Dependant on https://github.com/liteflow-labs/backend/pull/2239

Description
Update hooks to raise an error if the API does not has the feature enabled rather than calling the API and failing anyway
How to test
Run the API without the plugins you want to test and test that the hooks fail correctly
Checklist

[x] Update related changelogs
[x] Update docs if necessary

Good idea 👍


Use EntityHashMap for EntityMapper

Objective

There is a specialized hasher for entities: EntityHashMap
[EntityMapper] currently uses a normal HashMap<Entity, Entity>
Fixes #10391

Solution

Replace the normal HashMap with the more performant EntityHashMap

Questions

This does change public API. Should a system be implemented to help migrate code?

Perhaps an impl From<HashMap<K, V, S>> for EntityHashMap<K, V>

I updated to docs for each function that I changed, but I may have missed something

Changelog
TODO

This section is optional. If this was a trivial fix, or has no externally-visible impact, you can delete this section.

What changed as a result of this PR?
If applicable, organize changes under "Added", "Changed", or "Fixed" sub-headings
Stick to one or two sentences. If more detail is needed for a particular change, consider adding it to the "Solution" section

If you can't summarize the work, your change may be unreasonably large / unrelated. Consider splitting your PR to make it easier to review and merge!

Migration Guide
TODO

This section is optional. If there are no breaking changes, you can delete this section.

If this PR is a breaking change (relative to the last release of Bevy), describe how a user might need to migrate their code to support these changes
Simply adding new functionality is not a breaking change.
Fixing behavior that was definitely a bug, rather than a questionable design choice is not a breaking change.

This is my first time contributing in a while, and I'm not familiar with the usage of EntityMapper. I changed the type definition and fixed all errors, but there may have been things I've missed. Please keep an eye out for me!

I checked out the CI error, but I'm unsure why it's happening. Cargo.lock has async_lock 2.8.0 and 3.0.0 both. The blocking crate calls get_or_init_blocking here. blocking requires async_lock 3.0.0. Maybe it's trying to pull in the 2.8.0 version?

Failure was #10412: not your fault!

Failure was #10412: not your fault!

Glad to hear it wasn't something I did! Could you please re-run the CI?

Nice! LGTM. Need to fix the PR description before merging though.

I updated the description. Please tell me if the migration guide or changelog is not adequate. :)

I don't think the CI will run unless I make another commit. Could someone with write privileges please manually re-run it?

Re-running :)

Darn, the CI is still failing. It appears that larger changes are going to need to occur in async_executor before things can get running again. @ Bluefinger suggested removing all references to LocalExecutor when compiling for WASM and making them use wasm-bindgen-futures instead, but that is a larger change that is definitely out of scope for this PR.
https://github.com/smol-rs/async-executor/issues/72#issuecomment-1795019874
https://github.com/smol-rs/async-executor/pull/73#issuecomment-1795799398
https://github.com/smol-rs/async-executor/pull/73#issuecomment-1795996660


[backport] ROCm: Fix bugs and test suites to make ROCm/HIP happy - Part 2

Backport of #3835

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 13625ed7085c9e9dbcda961e3adc9dc738c46b9b, target branch v8) succeeded!

Thanks!


degit --help not working. Help file is missing.

What is the issue?
The degit --help command is not working cause the help file is not being distributed on the release.
Error: ENOENT: no such file or directory, open '**/node_modules/degit/dist/help.md'

What do you expect to happen?
degit --help should show the help information for degit.

I have submitted PR #338 that addresses this problem.
Note aside, there is some CI problem regarding a syntax error coming from eslint that is stopping every PR to merge well.
Someone might want to check for that. This is an awesome utility from R.Harris and will be a pity if it ends in the shadow.

Closing this due to the lack of activity.


Missing init for slackOAuthToken in SlackReporter.ts

Missing:
this.slackOAuthToken = slackReporterConfig.slackOAuthToken;
This cause the error:
Neither slackOAuthToken nor process.env.SLACK_BOT_USER_OAUTH_TOKEN were found

@hananmalka can you let me know if you it works when using an environment variable SLACK_BOT_USER_OAUTH_TOKEN as opposed to setting a key in the config?

@ryanrosello-og
When using the env variable SLACK_BOT_USER_OAUTH_TOKEN it does works but I prefer to set it in the config file

attempt to fix here https://github.com/ryanrosello-og/playwright-slack-report/pull/28
@hananmalka try v1.1.1

@ryanrosello-og tried v1.1.1 - Still getting the error.
The change you made doesn't reflect onSlackReporter.js file under dist/src

@ryanrosello-og Any update with this one?

@hananmalka try v1.1.2
Seems something got messed up publishing the npm package.

@ryanrosello-og
Thanks. It works now with v 1.1.2


Fixed typo

Fixed typo in REST Hooks section

Hi @robjmills   👋
Thanks for contributing to webhooks.fyi.
We're launching our site on Product Hunt tomorrow and I'd love to include our contributors as makers (to spread the love and street creds).
Please sign up to producthunt.com and send me your profile url here if you would like to be included and I'll take care of the rest.
ty,
Fred

Hey. This is my product hunt profile @.*** @.***>
Cheers!

On 31 Oct 2022, at 23:36, Frederico Hakamine @.***> wrote:

Hi @robjmills https://github.com/robjmills 👋
Thanks for contributing to webhooks.fyi.
We're launching our site on Product Hunt tomorrow https://www.producthunt.com/posts/webhooks-fyi and I'd love to include our contributors as makers (to spread the love and street creds).
Please sign up to producthunt.com https://producthunt.com/ and send me your profile url here if you would like to be included and I'll take care of the rest.
ty,
Fred
—
Reply to this email directly, view it on GitHub https://github.com/ngrok/webhooks.fyi/pull/72#issuecomment-1297816292, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA25K3WNQR77U2DGMXV72TWGBJWPANCNFSM6AAAAAAQRB7XAY.
You are receiving this because you were mentioned.


Improvement suggestions

From today's call:

Rename with to as in import and export statements. #16
Accept both identifiers as well as strings in the as clause of those statements. #28
Alter the behavior of passing an item by identifier for new expressions:

The current behavior is to try to infer the instantiation argument name for any kind of item, passing that item as a single argument.
Instead, for instances, "splat" the instance such that any exports from the instance
are automatically provided to matching arguments that aren't otherwise explicitly specified; at least one match must occur otherwise it is an error.
For non-instance items, continue to infer the argument name as a single argument.
Scalar instance items can continue to be passed with explicit names, e.g. foo: instance or "foo": instance.

Similar to the "splatting" behavior of instances as arguments for the new expression, there should be a syntax for easily "splatting" an instance's exports as the exports of the composition itself.
Some suggestions:

export i.*; where i is an instance (as would be prohibited in this form).
export i; where i is an instance (to export a scalar instance, users would need the as, e.g. export i as "i";)
Borrowing from JavaScript's spread syntax: export ...i;

As I'm now working on the "splat" (a term borrowed from Ruby's * operator) of instance exports for new expressions and export statements, and given the future potential for having wave record literals being "splatted" into transform expressions using a similar syntax, I propose we use a syntax similar to JavaScript "spread" syntax:
// Assumption: `i` names an instance
new a:b {
   // Passes `i` with inferred argument name
   i,
   // Splats/spreads the exports of `i` across any unspecified and unsatisfied arguments names
   ...i,
   // Passes `i` with explicit name `a`.
   a: i,
   // Invalid: `...` is unexpected token in this position
   b: ...i,
   // Satisfy any unsatisfied arguments with implicit imports
   ...
}

// Export `i` with inferred export name
export i;

// Export `i` with explicit export name
export i as a;

// Export all of the exports of `i`
export ...i;

// Invalid: `as` clause cannot be used with `...`
export ...i as a;

I like the inverse form for exports, so we can go with that for now unless there are complains.


How to share my location in the background?

If I want to update my location in the background every 15 minutes for an hour, how would I do so with this library? Would I have my own NSTimer to end it and set the Position.sharedPosition.timeFilter to 15?
EDIT:
How do I update location in the background? The documentation is a bit unclear on how to go about this process. Does oneshot imply only one attempt to update location? What method do I use to update location every time the user moves 40 meters?
EDIT 2:
If I set a distance filter and a time filter, is the time filter to let the singleton know when it expires and the distance filter know when to update the user location?

Hi @jayisidoro,

timeFilter is not fully integrated yet, so you have two options

Set up your own timer and call the one shot method every 15 minutes manually
Call startUpdating to receive continuous background location updates and set adjustLocationUseWhenBackgrounded and adjustLocationUseFromBatteryLevel to minimize battery usage in the background

Re: Edit 1

Background updates: make sure you've called requestAlwaysAuthorization and have set an NSLocationAlwaysUsageDescription to be able to get the user's location in the background. Then, call startUpdating to receive background location updates.
The performOneShotLocationUpdateWithDesiredAccuracy method can be used to quickly get the user's current location with a desired accuracy (in meters).

Re: Edit 2

The distanceFilter property specifies the minimum distance (meters) that your device must move in order for Position to generate an update event, similar to how CLLocationManager's distanceFilter works. Read more here.
In your case, to update location every time the user moves 40 meters, set distanceFilter = 40 and as long as you've called startUpdating, all observers will receive location updates every time your device moves >40 meters from the previously delivered location.

Hope this helps.


[Bug] SQL dialect does not work

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Code
monaco.editor.create(document.getElementById('container'), {
	value: "",
	language: 'sql'
});

Reproduction Steps
Simply type select * from f in the playground editor.
Actual (Problematic) Behavior
The editor suggests "from" when you type the "f", which doesn't make sense. "Select" and "from" are not autosuggested.
Expected Behavior
"Select" and "from" should be suggested, and table names (if provided) should be suggested after "from".
Additional Context
No response

The editor suggests "from" when you type the "f", which doesn't make sense.

You can disable it via wordBasedSuggestions: false

"Select" and "from" should be suggested, and table names (if provided) should be suggested after "from".

This can be implemented using a CompletionItemProvider, it's nothing Monaco has built in
https://microsoft.github.io/monaco-editor/playground.html#extending-language-services-completion-provider-example
https://microsoft.github.io/monaco-editor/api/interfaces/monaco.languages.CompletionItemProvider.html

That makes sense, but shouldn't it work given that the SQL option is available?

This language option is for syntax highlighting https://microsoft.github.io/monaco-editor/monarch.html
Autocomplete is an entirely separate thing. You can't expect Monaco to ship with autocomplete support for every language and every SQL flavor.
From the website https://microsoft.github.io/monaco-editor/

That makes sense, I guess why is there any autocomplete whatsoever? Wouldn't it be better if it didn't suggest anything?

That makes sense, I guess why is there any autocomplete whatsoever? Wouldn't it be better if it didn't suggest anything?

Depends who you ask. I imagine there are users that like word-based suggestions which is why it is enabled by default.

This is out of scope.


Update data displayed in clients grid

The clients grid should display:
Name, Organization, Email, Status

I didn't realize you had this assigned, I kind of need to do it in order for the modal for client creation to make sense. Do you mind if I just take it over and push it with my other PR?


[6.0] Limit PHPUnit to version 8.0 and above.

The changes has been done on develop branch for laravel/laravel. It is logical that we does this as well on laravel/framework.

Then we need to increment php: ^7.1.3 version. Because PHPUnit 8 requires php: ^7.2.

Heya thanks. We made the change on the skeleton repo to encourage people to make use of the 8.0 release. But we'd like to keep using the ^7.5 constraint on the framework one for now to still test that PHPUnit version for the "lowest" build. PHPUnit 7 is still actively supported until february next year. We'll probably drop support for it in Laravel 7.

@driesvints I don't see the value of testing against lower than 7.5 going to 5.9 or 6.0. It just means that whatever we doing with Illuminate\Foundation\Testing should be working with phpunit/phpunit version 8.0.0 and above.
Based on travis-ci setup the core is now tested only against 7.5.0 and latest 8.x version.

@crynobone right. But that's what I meant. We're still supporting PHPUnit ^7.5. Or did you mean something else? Seems like there's some confusion here?

@crynobone I'll re-open this to see what Taylor says.

Or did you mean something else? Seems like there's some confusion here?

We support "phpunit/phpunit": "^7.5|^8.0" in Laravel 5.8 because it still needed to be able to tests against installation with PHP 7.1, and that the reason why we made laravel/laravel 5.8 to use "phpunit/phpunit": "^7.5".
For 6.0, we already bumping the minimum PHP version to 7.2 which make it possible to completely stop supporting 7.5. The laravel/laravel app skeleton has been updated to 8.0 for quite sometime https://github.com/laravel/laravel/commit/0582a20adddc0e6bd16ca05eeae93e6412924ad6#diff-b5d0ee8c97c7abd7e3fa29b9a27d1780
Technically developer should be able to use phpunit/phpunit=^7.5 when upgrading to 6.0, but when we say we still supporting 7.5 that would mean if any bug related phpunit usage we also need to use code that are supported by both 7.5 and 8.0, if we upgrade the version now that would mean we just need to fix based on 8.0 and above.

@crynobone PHPUnit 7.5 supports 7.2. The version constraint you removed here was in require-dev, not require. It was only meant to use PHPUnit 7 to run the laravel framework test suite when applying "lowest". Because you removed it here the framework won't be tested with PHPUnit 7 anymore. At no time a PHPUnit is enforced in any of the require lists on any of the component's or main composer.json file. PHPUnit is still actively supported until February 2020 and dropping support for it now is a little too early imo. But since Taylor merged it I guess that's fine by me as well.

Just fyi: because this was removed, the test suite didn't caught a breaking change with https://github.com/laravel/framework/pull/30989 and PHPUnit 7 support is now broken for 6.x. I suggest we re-add this so the test suite actually runs PHPUnit 7 on the lowest matrix.

laravel/laravel v6.0.0 comes with phpunit 8.0 as minimum. https://packagist.org/packages/laravel/laravel#v6.0.0
laravel/framework v6.0.0 comes with phpunit 8.3 as minimum. https://packagist.org/packages/laravel/framework#v6.0.0
Is there a reason why we need to readd PHPUnit 7 support?

@crynobone Laravel 6 definitely kept on supporting PHPUnit 7. Like I said before: the versions you're linking to are "dev" requirements for the test suite and not "required" versions. We usually drop support for PHPUnit versions in February releases when PHPUnit has dropped support themselves.

Also see https://github.com/laravel/framework/issues/31099

Then this shouldn't have happened.
https://github.com/laravel/laravel/commit/0582a20adddc0e6bd16ca05eeae93e6412924ad6#diff-b5d0ee8c97c7abd7e3fa29b9a27d1780

@crynobone yes it does. It's only on the Laravel skeleton to encourage people to start using PHPUnit 8 when they start with a fresh Laravel install.


Alternative solution for correct rib/spar intersection computation

The code made the wrong assumption that the spar segment between two spar positionings is a straight line in eta/xsi space. The fix changes the computation to be done in 3D space, where the assumption of the straight line holds (according to the way the spar geometry is generated)

I don't like the assumption, that the spars have only z-orientation. If this is not true, the whole code fails.
I want a solution that is as generic as possible.

You are right, even though the spar geometry code can't generate spars with an up vector different to the z-axis right now, this can change in the future. All our testcases seem to work so we'll take your solution.


And do not forget to publish latest darwin image.

The android line is actually disable as we do not support android build from an Arm64 yet. We use Makefile to upload the android image at the moment.


NPC system - fix bugs

Fix some bugs in the NPC system.

The NPC object runs away when the main character gets closer to it.
I think it is a side effect of the move_and_slide() function in the NPC scripts.
But after commenting on this function, the main character cannot conflict with the NPC objects anymore.

reworked NPCs entirely.


Uncaught SyntaxError: Unexpected Identifier in 15.0.92

One of our users seemingly randomly gets the following error while logged into a realm on our self-hosted Zulip server:

Aforementioned user is using v.15.0.92 of the Zulip mobile app on a Galaxy S7 running Android 8.0- any ideas what might be causing this?

Thanks for the detailed report!
This looks like a case of #2505. The issue was fixed in 787c4a53b (from #2854) a couple of weeks ago in master; the fix is in version 16.2.96, which went to beta this week and will go out to everyone next week if all is well.
(Meanwhile, if you're interested in giving early feedback on beta versions, we'd be glad to have you join!)


[Remote State] Create a common interface for reader/writer in PersistedClusterStateService and RemoteClusterStateService

Is your feature request related to a problem? Please describe.
PersistedClusterStateService contains method to read and write cluster state to Lucene. RemoteClusterStateService contains method to read and write cluster state to remote blob store. We should refactor these to have a common read/write interface so that it will be easier to switch from local store to remote store in the future.

I was discussing something along these lines with @owaiskazi19 a couple of days ago.
Specifically, we were talking about storing config in indexes (like many of our plugins do) versus storing config in cluster state (as search + ingest pipelines do).
In my opinion, we could do better than treating this a cluster state problem and instead think of it as a "config store" problem, with a config store interface. Cluster state is one such config store (managed by cluster managers and replicated to all nodes). An index (either managed by the data nodes or in a remote store) could be another config store. Those would be two implementations that would make the most sense to ship with the OpenSearch min distribution. (At that point, plugins that move their config to the new interface could offer users the choice of storing config in an index or cluster state.)
Other config store implementations could be provided via plugins: Apache Cassandra, Zookeeper, DynamoDB, files on an NFS-mount, etc. All could provide trade-offs in terms of update propagation latency, setup / maintenance cost, scalability, etc.


Support .env config

It's a very common practice to use a .env file to load server configuration instead of the config.json file, that being said, this issue suggests to add support to loading config from .env file.
an example .env file would be as follows
[DB_CLIENTS]
RDBMS=postgresql
HOST=<IP_ADDRESS>
PORT=5432
DBNAME= postgres
USER=postgres
PASSWD=""
IS_FAST=true
NUMBER_OF_CONNECTIONS=1
TIMEOUT=-1.0

[REDIS_CLIENTS]
HOST=<IP_ADDRESS>
PORT=6379
PASSWD=""
DB=0
IS_FAST=true
NUMBER_OF_CONNECTIONS=1
TIMEOUT=-1.0

[LISTENERS]
ADDRESS=<IP_ADDRESS>
PORT=80
HTTPS=false

[LISTENERS]
ADDRESS=<IP_ADDRESS>
PORT=443
HTTPS=true
CERT=""
KEY=""
USE_OLD_TLS=false

[APP]
NUMBER_OF_THREADS=0
ENABLE_SESSION=true
SESSION_TIMEOUT=0
DOCUMENT_ROOT="./"
USE_IMPLICIT_PAGE=false
UPLOAD_PATH="uploads"
MAX_CONNECTIONS=100000
MAX_CONNECTIONS_PER_IP=0
LOAD_DYNAMIC_VIEWS=false
ENABLE_UNICODE_ESCAPING_IN_JSON=true
RUN_AS_DAEMON=false
HANDLE_SIG_TERM=true
RELAUNCH_ON_ERROR=true
USE_SENDFILE=true
USE_GZIP=true
USE_BROTLI=false
STATIC_FILES_CACHE_TIME=0
IDLE_CONNECTION_TIMEOUT=30
SERVER_HEADER_FIELD=drogon/version
ENABLE_SERVER_HEADER=true
ENABLE_DATE_HEADER=true
KEEPALIVE_REQUESTS=0
PIPELINING_REQUESTS=0
GZIP_STATIC=true
BR_STATIC=true
CLIENT_MAX_BODY_SIZE=1M
CLIENT_MAX_MEMORY_BODY_SIZE=64K
CLIENT_MAX_WEBSOCKET_MESSAGE_SIZE=128K
REUSE_PORT=true

[LOG]
LOG_PATH=""
LOGFILE_BASE_NAME=drogon
LOG_SIZE_LIMIT=10000000 # 10MB
LOG_LEVEL=DEBUG

this's the equivalent of the config.json file created with new dragon project.

Thanks for the feature request.
Could you please clarify one question: Do you suggest that Drogon should adopt the INI file format in addition to JSON, or do you suggest that Drogon should support all configuration parameters via environment variables, and the .env file would be a means to an end for feeding the framework with variables?


scripts: gen_app_partition: Fix broken typo'd sys.exit()

os.exit() doesn't exist.
Also use the nifty sys.exit(msg) feature, which prints msg to stderr
and exits with status 1.
https://github.com/zephyrproject-rtos/ci-tools/pull/37 (just to link)

Codecov Report

Merging #14764 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14764   +/-   ##
=======================================
  Coverage   52.51%   52.51%
=======================================
  Files         309      309
  Lines       45048    45048
  Branches    10419    10419
=======================================
  Hits        23657    23657
  Misses      16583    16583
  Partials     4808     4808

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 8e85d7d...2cd769f. Read the comment docs.


Moon Card Error

Mod version: Current 1.20-1.5.7
Describe the issue: The moon card times out too quickly to maintain its effect, creating a distracting flicker.
How to Reproduce:

Go to game
Equip Moon card
See error

Additional info:
Add any other info about the problem here.

Discovered the issue. If you have underwater night vision as an effect from another item active, it interferes with the Moon effect.

what effect is underwater night vision?

Enigmatic Legacy provides an item that gives vision underwater but not above water.


 causes an import error?

Was trying to make an svg sprite work, and encountered some behaviour I can't explain. Not sure if it's a bug or I'm missing something, if it's the latter I apologise in advance, but:
This works
<svg fill="none" viewBox="0 0 399 398">
  <symbol id="circle"></symbol>
    <circle cx="50" cy="50" r="50" fill="white" />
</svg>

This doesn't
<svg fill="none" viewBox="0 0 399 398">
  <symbol id="circle">
    <circle cx="50" cy="50" r="50" fill="white" />
  </symbol>
</svg>

And throws the following error
code: 'BABEL_PARSE_ERROR',
    reasonCode: 'UnexpectedToken',
    loc: Position { line: 1, column: 70, index: 70 },
    pos: 70,
    syntaxPlugin: undefined,
    plugin: 'solid-svg',
    id: '{...}/socialSprite.svg?component-solid',
    pluginCode: 'export default (props = {}) => <svg fill="none" viewBox="0 0 399 398"/{...props}>',
    frame: '1  |  export default (props = {}) => <svg fill="none" viewBox="0 0 399 398"/{...props}>\n' +
      '   |                                                                        ^'
  },
  statusCode: 500,
  fatal: false,
  unhandled: true,
  statusMessage: undefined,
  data: undefined
}

Am I missing something?
Is this expected? If so, is there a way to make this work?
Here's a link to stackblitz relevant file is random.svg.
Thanks!

Hi @vallafederico!
Thank you for the report and the demo.
This also works:
<svg fill="none" viewBox="0 0 399 398">
  <symbol id="circle">
    <circle cx="50" cy="50" r="50" fill="white" />
  </symbol>
   <use href="#circle" x="100" y="100" />
</svg>

It seems to be an optimization by SVGO. I tried disabling all SVGO optimizations, and it also worked. You can configure SVGO to disable the rule that is affecting this.

disabling svgo removeUselessDefs fix the problem. You can configure svgo in its own file, or as solidSvg params:
solidSvg({
      svgo: {
        enabled: false,
        svgoConfig: {
          plugins: [
            {
              name: 'preset-default',
              params: {
                overrides: {
                  removeUselessDefs: false,
                },
              },
            },
          ],
        },
      },
    }),

Thanks for your patience, this makes a lot of sense now.
And thanks for the amazing work on this!


Dndungu/unexport find handler creator

Current coverage is 100% (diff: 100%)

Merging #10 into master will not change coverage

@@           master   #10   diff @@
===================================
  Files           3     3
  Lines         165   165
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
  Hits          165   165
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 2fc0aa2...e2cfe1a


Return pystan

This moves away from the original stanmagic model towards a model that simply returns a pystan model.

Apologies, I was meant to merge this into my repo!


Output syncable API in status

From a customer feedback: https://github.com/aws-amplify/amplify-js/issues/4535#issuecomment-563363748
If a customer uses amplify add/update api it's hard to know if their endpoint is setup as syncable without looking in the metadata. We should show this with a mechanism such as amplify status as a property of the API.

Closing as we show it now in amplify update api.


MolsonCoors - The message "Milwaukee is not a city in Romania"  is displayed on scrapers.peviitor.ro

ENVIROMENT: production
URL: https;//scrapers.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open the URL in the browser
Run scraper for the " MolsonCoors" company
Chek the location for the jobs:  "Team Lead - Procure-to-Pay (Accounts Payable)"  and "Solution Designer"

EXPECTED RESULTS:
The location for the jobs, on the web's company, must be a city in Romania
ACTUAL RESULTS:

The message "Milwaukee is not a city in Romania" is displayed on scrapers.peviitor.ro for the 2 jobs:  "Team Lead - Procure-to-Pay (Accounts Payable)"  and "Solution Designer"

On scrapers.peviitor.ro the jobs: "Senior Analyst - Group Consolidation" has only Bucuresti city as the location.
On the website company's this position has also following location: Bucuresti, Brasov, Iasi, Timis

Fixed!

Retesting:

Run scraper.

EXPECTED RESULTS:

On the website's company the location for this job "Senior Analyst - Group Consolidation" is Timis too.

ACTUAL RESULTS:

On scrapers.peviitor.ro the job "Senior Analyst - Group Consolidation" has only 3 location: Bucuresti, Brasov si Iasi.

if you consider that bug has not been solved, please change the label from "Fixed" to "Bug", otherwise i consider it fixed


Count Primes

If an integer n is given , My code will return the number of prime numbers that are strictly less than n.
Please Assign me this  issue. Thank you

@Kumar-laxmi  sir please assign me this issue , I really want to make a start in open source, kindly assign me this issue. I can solve it using C, C++,  JAVA

Would like to contribute in this issue. Please assign me this issue

@Kumar-laxmi I would like to contribute to this issue. Please assign me this issue.

@Kumar-laxmi can you assign me this under SSOC'23
Method 1: A simple solution is to traverse all the numbers, check if current number is prime. If yes, increment the count. Finally, return the count.
Method 2: An efficient solution is to use Sieve of Eratosthenes algorithm to find all primes up to the given limit.
Time Complexity : O(n * log(log(n)))
Space Complexity : O(n)


Iface only tests

@richbrowne
What issues does this address?
Fixes #554
What's this change do?
This change add the "ifaceonly" tests to the nightly regression (NOTE They error out at this time).

Ifaceonly turns out to be a mislabel on my part.


Fix undefined property 'remoteAddress'

Sometimes Error: TypeError: Cannot read property 'remoteAddress' of undefined occurs, try to fix it.

@igrigorik I'm using node v0.12.7, and I will receive this error almost every day.
Can you help me find the reason and fix it ?
TypeError: Cannot read property 'remoteAddress' of undefined
    at handleRequest (/root/.nvm/versions/v0.12.2/lib/node_modules/spdyproxy/lib/server.js:136:81)
    at emit (events.js:118:17)
    at TLSSocket.socketOnData (_http_server.js:368:14)
    at TLSSocket.emit (events.js:107:17)
    at readableAddChunk (_stream_readable.js:163:16)
    at TLSSocket.Readable.push (_stream_readable.js:126:10)
    at TCP.onread (net.js:538:20)

@igrigorik got the same error on node v4.2.4

@xuemengfei I suggest you use spdylay/nghttpx + squid instead of this deprecated project, this one does send neither NPN nor ALPN.


Add CI integration to run tests and deploy site to SSL enabled server

To be done with Travis, docker and (potentially) docker-hub

@DinisCruz Could you please give some insight what you plan here?


Fetch EventRequestPaused fail to intercept and edit POST data

What versions are you running?

$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.5.4-0.20200714133707-62bd38cdf4f0
$ google-chrome --version (I'm using '/Applications/Google Chrome.app')
Version 87.0.4280.67 (Official Build) (x86_64)
$ go version
go version go1.14.5 darwin/amd64

What did you do? Include clear steps.
I want to Intercept request and change the POST data, means I want to send arbitrary POST request, I succeed in past, but now it fails in latest chrome version
Here some snippet code
switch ev := ev.(type) {
		case *fetch.EventRequestPaused:
			go func() {
				c := chromedp.FromContext(cctx)
				cctx := cdp.WithExecutor(cctx, c.Target)
                                newreq := fetch.ContinueRequest(ev.RequestID)
				newreq.URL = "http://my-vps:4321/fse/ooo.php"
				newreq.Method = "POST"
				newreq.PostData = "aaaaa=1234"
                                newreq.Do(cdp.WithExecutor(cctx, c.Target))
                        }()
}

On my vps i listen on port 4321, I can see the method change to POST, but the post data is empty
Here the dump of newreq using spew
(*fetch.ContinueRequestParams)(0xc00038cf00)({
 RequestID: (fetch.RequestID) (len=20) interception-job-1.0,
 URL: (string) (len=34) "http://my-vps:4321/fse/ooo.php",
 Method: (string) (len=4) "POST",
 PostData: (string) (len=10) "aaaaa=1234",
 Headers: ([]*fetch.HeaderEntry) <nil>
})

Here the request that i got on my vps
POST /fse/ooo.php HTTP/1.1
Host: my-vps:4321
Connection: keep-alive
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_2) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/87.0.4280.67 Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9
Accept-Encoding: gzip, deflate
Accept-Language: en-US

What did you expect to see?
The POST Data "aaaaa=1234"
What did you see instead?
Empty Post Data
Pls note that it was succeed in past (around 2 months ago), then chrome update something and it fails.

The PostData should be encoded as a base64 string. And you should set the content-type header to help the server interpret the posted data (but the content-length header is not required because Chrome will set it for you).

Thank @ZekeLu , it works!


Release v0.6

Okay, so...

[x] Write a full draft a v0.6 announcement devlog post
[x] Record a video (don't forget to check the volume level)
[x] Create English subtitles for the video
[x] Create Russian subtitles for the video
[x] Prepare a short summary of the devlog post
[x] Update screenshots and GIFs in the README
[x] Make the final v0.6 commit:

[x] Change Cargo.toml's version from v0.6.0-snapshot to v0.6.0
[x] Update the roadmap in the readme

[x] Commit a tag and make a github release
[x] Upload release binaries to itch.io and github's release

[x] linux
[x] win64
[x] win32 (!)
[x] macos

[x] Update itch.io's web version
[x] Publish the video to youtube (with subtitles)
[x] Replace mockup links in the devlog post with real ones
[x] Publish the devlog post
[ ] News time!:

[x] patreon
[x] /r/rust
[x] /r/rust_gamedev
[x] /r/devblogs
[x] twitter
[x] URLO
[x] facebook
[x] itch.io
[x] tigsource
[x] indiedb
[x] vkontakte
[x] gamedev.ru

[x] Remove the weekly post/video experiment where possible (urlo, itch.io, tigsource, patreon, gamedev.ru?)
[x] Make the first v0.7.0-snapshot commit

WIP: https://github.com/ozkriff/ozkriff.github.io-src/blob/41a049d9b/content/2019-08-05--devlog-zemeroth-v0-6/index.md

Done!


v0.6.7

#771  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alanvardy and the rest of your teammates on  Graphite


Alias connection

This pull request adds an optional alias parameter on the create and receive connection invitation endpoints.
It also adds the ability to query for connections by alias.

Codecov Report

Merging #149 into master will decrease coverage by 0.02%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   73.48%   73.45%   -0.03%
==========================================
  Files         188      188
  Lines        8224     8230       +6
==========================================
+ Hits         6043     6045       +2
- Misses       2181     2185       +4


Connect Azure Logs Monitor to Power BI

User Story

As a stakeholder at the CDC, I want to view ReportStream product metrics on the CDC's PowerBI instance, authenticating with my CDC credentials and not requiring any special permissions.
Description/Use Case

The following document outlines how to connect azure monitor logs to Power BI:

https://learn.microsoft.com/en-us/azure/azure-monitor/logs/log-powerbi#export-analytics-queries
The CDC Power BI instance to connect to: https://app.powerbigov.us/

We should create queries based off the work done in #12723 and graph something interesting in PowerBI to prove out the connection and have something to demo.
Risks/Impacts/Considerations

Dev Notes

Acceptance Criteria

[ ] Power BI is able to read ReportStream's events and graph interesting data from it

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @jack-h-wang @jalbinson @JessicaWNava @JFU-NAVA-PBC @jimmyfagan @mkalish @thetaurean

Please add your planning poker estimate with Zenhub @brick-green


Simplify result on polygons depends on the cyclic order of the vertices

Expected behavior and actual behavior.
I noticed the results of the simplify() method on polygons depends on the cyclic order of the vertices.
Take a polygon, shift the order of the vertices in the initialization, it is the same polygon (poly1 and poly2 in the example below). But the application of the Ramer–Douglas–Peucker algorithm through  the simplify() method on these two identical polygons will lead to simplified polygons with different number of vertices.
Note: I know this problem is more related to Geos than Shapely, but the registration on their Trac requires a registration on Osgeo which looks like a long process :(
Steps to reproduce the problem.
from shapely.geometry import Polygon

#In poly2, the vertices 0, 1 and 2 are perfectly aligned
# along the  line -0.35001187*x + -0.0756343*y + 0.64449771
poly2 = Polygon([[1.341343356332423, 2.313918776834345], [1.344881773910638, 2.297544087047788], [1.416996781165046, 1.963818389997046], [0.06662927567958832, 0.002057790756225586]])

# For poly1, we shifted the vertex 0 to put it at the end
poly1 = Polygon([[1.344881773910638, 2.297544087047788], [1.416996781165046, 1.963818389997046], [0.06662927567958832, 0.002057790756225586], [1.341343356332423, 2.313918776834345]])

# Our polygons are identical
poly1.difference(poly2).area
>> 0.0

# As expected, we get a simplified polygon with 3 vertices
# the first one is repeated in a line ring, so 3+1=4
poly2_simplified = poly2.simplify(0.001, False)
print len(poly2_simplified.exterior.coords)
>> 4

# Here get the original polygon, no simplification occurred while some points were perfectly aligned
poly1_simplified = poly1.simplify(0.001, False)
print len(poly1_simplified.exterior.coords)
>> 5

Operating system
Mac OS X 10.11.6
Shapely version and provenance
1.5.17 installed from PyPI using pip

I appreciate you bringing this issue up @vins31. I did not know that this was the case. I'm not going to change the GEOS algorithm, but I will see that Shapely's docs are updated.

Documentation updated by 197b052.


fix: lint_version compatibility with bash

Blocks #1499
Overview
Bash doesn't support nested parameter expansion [1] [2] [3], which currently breaks #1499. This PR breaks them into multiline.
This error was originally raised in v0.4 branch so targeting this PR to v0.4. It'll be merged into master as part of #1499.
Changes
Updated lint_version circleci job to break down the version step by step.
Testing
CircleCI tagged builds should pass.

This only affects our CI so no version release needed.


[CP] DevTools hotfix 2.28.2

Issue Link
https://github.com/flutter/devtools/pull/6644
Commit Hash
8db42fc4a0b7a4ca180ba73fc387b81b0927e8a0 (from flutter/devtools)
Target
beta
PR Link
https://dart-review.googlesource.com/c/sdk/+/334580
Changelog Description
I am not sure how to push this DevTools cherry-pick through, so I'll be very descriptive here and hopefully can get some guidance from the Flutter / Dart SDK rollers.
There was an issue in DevTools that was fixed by https://github.com/flutter/devtools/pull/6644. DevTools extensions were broken for test debugging targets because we were only looking for lib/main.dart as an app's entry point. This PR modifies the RegExp search to support other targets within a project.
The current DevTools version set to go into the November Dart / Flutter stable release is 2.28.1. I created a branch on the flutter/devtools repo 2.28.2. This branch is created from 2.28.1 and cherry picks the fix from https://github.com/flutter/devtools/pull/6644 as well as bumps the version to 2.28.2 (see diff).
Impacted Users
DevTools extension authors
Impact Description
DevTools extensions are not supported for debugging targets that are not lib/main.dart. For example, a package that is for debugging tests might want to provide an extension to give their users tooling. The DevTools extension author who submitted this fix will not be able to publish their extension unless this cherry-pick goes through. This will delay them shipping their extension until the next flutter stable.
Since this upcoming Flutter stable release will be the first that includes DevTools extension support altogether, it would also be ideal to not release with this bug that limits the type of debugging tools DevTools extensions authors can provide.
Workaround
There is no workaround. This DevTools extension author cannot expect their users to work from Flutter master in order to get access to the extension.
Risk
low
Test Coverage
yes
Validation Steps
No response

I am not sure how to proceed with this cherry-pick. The Dart SDK CL here (https://dart-review.googlesource.com/c/sdk/+/334580) updates the "devtools_rev" hash in the SDK DEPS file to 8db42fc4a0b7a4ca180ba73fc387b81b0927e8a0, which is the tip of flutter/devtools @ branch 2.28.2. However, this hash does not correspond to a devtools build in CIPD because the DevTools LUCI builder only uploads DevTools builds to CIPD for commits to flutter/devtools @ master. @sortie
A lot has changed with the DevTools release process since we last had to do a cherry pick, so I will document the DevTools cherry-pick process for DevTools with the recommended steps once we outline them here. Thanks.

Cherry pick CL is submitted (https://dart-review.googlesource.com/c/sdk/+/334940). Next step is for @itsjustkevin to re-build the stable branch.

This was released in the 3.16.0 stable.


Change "id" to "uuid"

The master branch of sciunit now defines a read-only attribute id for all classes. We therefore need to use something else for identifying models and tests, I am proposing uuid.
All sciunit.Model classes registered with the Validation service will need to be updated, unfortunately.

Sample use of from_existing() (for future reference - could add to docs):
from hbp_validation_framework import ModelCatalog, TestLibrary
a = ModelCatalog(username="shailesh")
b = TestLibrary.from_existing(a)


A bug was detected when using the prefix method without group method

Laravel Version
10.32.1
PHP Version
8.2.12
Database Driver & Version
No response
Description
When using the prefix method on routes that are not groups, it incorrectly generates the destination URL.
It should be like this:
POST api/orders/some ..... orders.index › OrderController@index
POST api/webhook/some ............. webhook › WebhookController

But this option is displayed instead:
POST api/orders/some ..... orders.index › OrderController@index
POST webhook/api/some ............. webhook › WebhookController

Steps To Reproduce

Create a controllers:php artisan make:controller OrderController
php artisan make:controller WebhookController -i

Create routes:use App\Http\Controllers\OrderController;
use App\Http\Controllers\WebhookController;

app('router')
    ->name('orders.')
    ->prefix('orders')
    ->controller(OrderController::class)
    ->group(static function () {
        app('router')->post('some', 'index')->name('index');
    });

app('router')
    ->name('webhook')
    ->prefix('webhook')
    ->post('some', WebhookController::class);

Execute the console command:php artisan route:list

https://github.com/laravel/framework/assets/10347617/c9ad88ab-653a-4f32-94e3-b4ea6a0402da

Repository: https://github.com/andrey-helldar/laravel-issue-49039
Commit: https://github.com/andrey-helldar/laravel-issue-49039/commit/3f6a81c4063129cf324ae4338e0a03558ca61e49
Result:
Helldar@HellPC MINGW64 /d/domains/bug-report (main)
$ php artisan route:list

  POST       api/orders/some .................................................... orders.index › OrderController@index
  POST       webhook/api/some ............................................................ webhook › WebhookController

Hello, I have tested with the examples provider, and I have the same result as @andrey-helldar

I've also noticed that it doesn't matter what route and controller names are used. The only important thing is to call the prefix method without using a group:
// incorrect
app('router')->prefix('some')->get('', SomeController::class);

// correct
app('router')->prefix('some')->group(
    fn () => app('router')->get('', SomeController::class);
);

I would feel the current behavior is expected, since prefix is more useful for grouped routes. Feel free to submit PR if you want to improve the behavior


Suggest using a pull request checklist template

Include a list of common things to check before completing the pull request to prevent forgetting about essential stuff.

Could it be added as a PR template? https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/about-issue-and-pull-request-templates

Yeah, I considered that and experimented a bit with it. However, I found out that the commit description is completely overriden by the template which is a bit unfortunate. Filed this in the following discussion:

https://github.com/orgs/community/discussions/43392

As for now, it seems, there is no elegant solution to include both. So we have to either copy-paste the checklist or the commit description. Not completely sure which of the options is better. Let's gather some feedback before choosing.

Vote here: I'd rather copy-paste the commit description.

Ok, so the voting is perfectly balanced, I'll do the decision then: Let's start with including the checklist on the Contribute page, we can add the template later if needed/desired. Still missing at least one review to merge it...


ERROR  Cannot read properties of undefined (reading 'options')

I'm suddenly running into an issue with this plugin
# This file contains the result of Yarn building a package (my-app-8b06de@workspace:apps/my-app)
# Script name: postinstall

Nuxi 3.0.0

 ERROR  Cannot read properties of undefined (reading 'options')

  at nuxtModule (/home/user/workspace/my-project/node_modules/@nuxtjs/eslint-module/dist/module.js:25:13)
  at installModule (/home/user/workspace/my-project/node_modules/@nuxt/kit/dist/index.mjs:416:9)
  at async initNuxt (/home/user/workspace/my-project/node_modules/nuxt/dist/index.mjs:1825:7)
  at async loadNuxt (/home/user/workspace/my-project/node_modules/nuxt/dist/index.mjs:1857:5)
  at async loadNuxt (/home/user/workspace/my-project/node_modules/@nuxt/kit/dist/index.mjs:493:19)
  at async Object.invoke (/home/user/workspace/my-project/node_modules/nuxi/dist/chunks/prepare.mjs:30:18)
  at async _main (/home/user/workspace/my-project/node_modules/nuxi/dist/cli.mjs:50:20)

The same problem (from RC14 btw),

I'm having the same issue with the stable Nuxt 3.0.0, any fixes?


Only customize resources during export if there are any export plugins in use

Fixes #72263
This bug was introduced during PR #70377 when changing the approach to use an export plugin: this tiny bit of code should have been reverted during the change, but I missed it. Sorry, Everyone!

Thanks!