Wire vector mask

This follows Nicolas's work on https://github.com/iree-org/iree/pull/11796, the work that Matthias did to fix the bufferization and Diego's work to flatten vector.mask.
@dcaballe
The whole thing now fails on some vector.mask issue:
error: 'vector.mask' op expects a maskable operation

Essentially we turn a vector.mask (vector.transfer_read|write) into a vector.mask(vector.load|store) during lower_vectors. I assume we'll want to produce a vector.maskedload|store instead.
High level repro with iree-samples:
benchmark-transform-create -b llvm-cpu <iree-samples>/transform_dialect/benchmark_linalg_reductions.stub.mlir reduction_2d_static f32 121 456

Full repro with iree-opt:
iree-opt '--pass-pipeline=builtin.module(hal.executable(hal.executable.variant(iree-llvmcpu-lower-executable-target)))'  --iree-codegen-llvmgpu-enable-transform-dialect-jit=false --iree-codegen-llvmcpu-use-transform-dialect=full_reproducer.mlir --mlir-disable-threading input.mlir

Repro with mlir-opt (make sure to build with -DMLIR_INCLUDE_TESTS=On to get the interpreter if your building within IREE):
mlir-opt --test-transform-dialect-interpreter      --mlir-disable-threading reduced.mlir -mlir-print-ir-after-failure

input.mlir.txt (Note: there's a syntax difference in the input for mlir-opt vs. upstream mlir-opt. Just uncomment the types on the match operation for upstream.)
full_reproducer.mlir.txt
reduced.mlir.txt

You can use Vector/Transforms/LowerVectorMask.cpp. That will lower a vector.mask { vector.transfer_read } into a vector.transfer_read with a native mask. This is a temporary solution so that we can reuse all the existing lowerings for transfer ops. The idea is to remove the mask from the transfer ops eventually to reduce their complexity.

Thanks @dcaballe, that fix that issue.
Now on to the next one: we fail to legalize some instructions during the lowering to LLVM:
error: failed to legalize operation 'builtin.unrealized_conversion_cast' that was explicitly marked illegal

That indicates a missing conversion pattern. Can you share a bit more context? Perhaps the IR before the error?

Here you go!
crash.mlir.txt
To reproduce:
iree-opt --iree-convert-to-llvm --reconcile-unrealized-casts crash.mlir

At first glance, it looks like we don't convert to llvm the following construct:
  %959 = vector.mask %557 { vector.multi_reduction <add>, %958, %156 [0] : vector<16x8xf32> to vector<8xf32> } : vector<16x8xi1> -> vector<8xf32>

If you need the full pipeline, you can run (or the scripts from iree-samples, first the benchmark-transform-create command listed at the beginning of this PR, then the produced benchmark-transform-run-iree-compile command):
iree-compile input_full,mlir --iree-hal-target-backends=llvm-cpu --iree-flow-enable-aggressive-fusion --iree-codegen-llvmgpu-enable-transform-dialect-jit=false --iree-codegen-llvmcpu-use-transform-dialect=td_input.mlir

[input_full.mlir.txt](https://github.com/ir
td_input.mlir.txt
ee-org/iree/files/10572715/input_full.mlir.txt)

Yeah, reduction support is not committed yet. I'm working on that here: https://github.com/iree-org/iree/pull/12003
I should start submitting patches pretty soon... Once I make sure IREE is stable with all of them.
Perhaps you can start with a code with only loads/stores (and other ops that don't need masking)?

Closing stale PR.


CC01 and patent provision

A team I'm supporting has been considering public domain, specifically CC01 for a project we're open sourcing at CMS.  Based on some of the internal discussion, despite CC01 being used quite a bit at CMS, it may present an issue in regards to not addressing patent provisions. Seems like CC01 could present a legal hazard in relation to patents. Given the US government holds a lot of patents, it would be good to evaluate if we should continue to highlight CC01 within the policy as a option for CMS.
Any thoughts in relation to keeping it as a default or providing additional clarification? @rajesh-nayak-cms

@andres-gov Thank you for your support of Open Source at CMS.
I am working with the Digital Service here at CMS to review our Open Source Policies, and will include this question in our review process. In the meantime, you can join #cms-opensource on slack, happy to discuss there.

@andres-gov I will start with the standard disclaimer of "I am not a lawyer and this does not constitute legal or financial advice."
Based upon analysis of the most recently updated Open Source Policies and best practices from other Federal agencies such as GSA and CISA, CC0 remains the recommended default public domain designation mechanism for Federal Open Source Code.
For additional information about software patent considerations, you can read more in the DOD OSS FAQ by searching for "software patents" in the text.


Update Projects

Automated changes by create-pull-request GitHub action

/hold
Please hold for https://github.com/cncf/tag-contributor-strategy/pull/438 so that we can compare open PRs (this one and a new one that will be created later on)

Closing in favor of https://github.com/cncf/tag-contributor-strategy/pull/441


Consider adding a flag in Horizon to reset captive core DB

It would be useful to have a flag in Horizon that runs new-db on captive core. This is needed in case there is a bug in stellar-core that corrupts the database, so that an operator can easily rebuild the minimal state and join the network. To do this, Horizon would need to run the following core command: stellar-core new-db --minimal-for-in-memory-mode

possibly related to #4454 design aspects for reusing on-disk captive core storage path scenario.

@marta-lokhova @jcx120 , I think the intent requested here for on-disk reuse feature to save time rebuilding has been implemented as of https://github.com/stellar/go/pull/4471, the approach taken was to automate when the captive-core on-disk db can be reused or reset rather than a user set flag, so, I think this issue can be moved to done/closed?

Closing as done


feat: automatically populate 'PROVISIONING_PROFILE_SPECIFIER' if using match in signing configuration

What does this PR do

Automatically populate 'PROVISIONING_PROFILE_SPECIFIER' if using match in signing configuration

Task
resolves #117
Checklist:

[x] I ran make validation locally with success
[x] I have not introduced new bugs
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented on my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new errors

Warnings

:warning:
The library files were changed, but the tests remained unmodified.
Consider updating or adding to the tests to match the library changes.

  Generated by :no_entry_sign: Danger Swift against 9bc377700d31f359c45101037c4d5d88b55c0e98

Warnings

:warning:
The library files were changed, but the tests remained unmodified.
Consider updating or adding to the tests to match the library changes.

  Generated by :no_entry_sign: Danger Swift against 9bc377700d31f359c45101037c4d5d88b55c0e98


Reminders and email notifications

We've added 3 articles for DataHub with screenshots attached.
-Grouping and ordering collection list filters
-Email notifications and reminders
-Add contact form

@IeuanParkerDIT The posts in the design history look amazing. I'll give them a proper scan in the morning but I love seeing new articles being added. The only change I can think of is that would be great is if we put the dates that these features went live as the date of the post, so it doesn't look like we rushed a load of stuff out at once! Could you check that with @uzuncaner and drop them in please. Even if they're approximate. Thanks!

@AndyDaviesDIT posts have been updated with completion date, so the file name and the date inside the post are updated.


Move package org.apache.calcite to org.apache.pinot.calcite

Move the Pinot classes under package org.apache.calcite to org.apache.pinot.calcite to avoid confusion

Codecov Report
Attention: Patch coverage is 0% with 20 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (59551e4) to head (684deed).
Report is 243 commits behind head on master.

Files
Patch %
Lines

...ache/pinot/calcite/sql/fun/PinotOperatorTable.java
0.00%
8 Missing :warning:

...pinot/calcite/rel/hint/PinotHintStrategyTable.java
0.00%
5 Missing :warning:

.../calcite/rel/logical/PinotLogicalSortExchange.java
0.00%
5 Missing :warning:

...inot/calcite/rel/logical/PinotLogicalExchange.java
0.00%
2 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12837       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.76%
=============================================
  Files          2436     2417       -19
  Lines        133233   132307      -926
  Branches      20636    20486      -150
=============================================
- Hits          82274        0    -82274
- Misses        44911   132307    +87396
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
0.00% <0.00%> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <0.00%> (ø)

java-11
?

java-21
0.00% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
0.00% <0.00%> (-61.75%)
:arrow_down:

skip-bytebuffers-true
?

temurin
0.00% <0.00%> (-61.76%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

This change is introducing backward incompatibility because we rely on class path during query plan ser/de, and PinotRelExchangeType is part of the ser/de.
We should fix the ser/de so that it doesn't serialize the class path for both efficiency and forward compatibility


Slug dynamic elements are not created.

When I create a link, only the static elements of "blog /" are applied; [calback] and [field] are not applied. I use wamp "open server", cp ver 0.9.0
PHP Version | 7.1.22
PHP SAPI | apache2handler
System | Windows NT NEKTOBIT 10.0 build 18362 (Windows 10) AMD64
Loaded Extensions | Core, bcmath, calendar, ctype, date, filter, hash, iconv, json, mcrypt, SPL, pcre, readline, Reflection, session, standard, mysqlnd, tokenizer, zip, zlib, libxml, dom, PDO, bz2, SimpleXML, xml, wddx, xmlreader, xmlwriter, apache2handler, mbstring, openssl, curl, exif, fileinfo, ftp, gd, gettext, igbinary, imap, mongodb, mysqli, odbc, Phar, PDFlib, pdo_mysql, pdo_pgsql, pdo_sqlite, redis, soap, sockets, sqlite3, timezonedb, xmlrpc, xsl
Max. execution time | 180 sec.
OPCache enabled | NO
Memory limit | 1536M
Upload file size limit | 100M
Realpath Cache | 2048k / 600 (ttl)

Ok, i think i understand:
"[field:term1]" - working
"[field:term1]/and-some" - working
"[field:term1]-and-some" - not working

Hi @nektobit , sorry for late answer, but can you provide more details on what you want to achieve? If you are using callback as token you need to implement the function you are providing, e.g. slugUniqId
The latest example you provided for field, the implementation only supported tokens between slashes..  happy to handle a PR that enhances that further..


UTF-8 Error

Hi,
The file xavier/src/messages/messages-de.js still gives an UTF-8 error. The comment on line 167 jumped to line 168.
Sander

Is this problem still occurring? Could you perhaps send me a pull request?

Unfortunately I have left the project and I don't have access anymore to
the environment I was working in at the time.
On Wed, Mar 9, 2016 at 12:25 AM, Roland<EMAIL_ADDRESS>wrote:

Is this problem still occurring? Could you perhaps send me a pull request?
—
Reply to this email directly or view it on GitHub
https://github.com/rpbouman/xavier/issues/10#issuecomment-194017137.

Hi Sanderzdb!
hey, that's a pity. Is it possible for you to give the people at your
former project a nudge so they can contact me? I'd be happy to look into it
now that I have a little bit more time.
Best regards,
Roland
On Wed, Mar 9, 2016 at 10:03 AM, sanderzdb<EMAIL_ADDRESS>wrote:

Unfortunately I have left the project and I don't have access anymore to
the environment I was working in at the time.
On Wed, Mar 9, 2016 at 12:25 AM, Roland<EMAIL_ADDRESS>wrote:

Is this problem still occurring? Could you perhaps send me a pull
request?
—
Reply to this email directly or view it on GitHub
https://github.com/rpbouman/xavier/issues/10#issuecomment-194017137.

—
Reply to this email directly or view it on GitHub
https://github.com/rpbouman/xavier/issues/10#issuecomment-194192266.

--
Roland Bouman
blog: http://rpbouman.blogspot.com/
twitter: @rolandbouman
linkedin: http://www.linkedin.com/profile/view?id=5142800&trk=tab_pro
Author of "Pentaho Solutions" (Wiley, ISBN: 978-0-470-48432-6
http://tinyurl.com/lvxa88) and "Pentaho Kettle Solutions" (Wiley, ISBN:
978-0-470-63517-9 http://tinyurl.com/33r7a8m)

I've send you an email on the address of the oil company we used to work at together :)

Ah hey - duurde even voordat dat kwartje gevallen was. Dankje!


MSID: 90400 Version: 1 DOI: 2<IP_ADDRESS>8185

"msas": "Microbiology and Infectious Disease", "Structural Biology and Molecular Biophysics"
"msid": "90400"
"preprintDoi": "10.1101/2<IP_ADDRESS>8185"
"Reviewed Preprint posted": "2023-08-30"
"Sent for peer review": "2023-07-06"
"Posted to bioRxiv": "2023-06-19" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>8185v2/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=90400)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/90400/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>8185)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[x] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/90400/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/90400)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/90400

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/90400

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/90400/v1/90400-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/90400)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

We have version 1 loaded instead of version 2.


How to choose a specific set of blocks for the exported schematic?

Hello!
First time trying to use your tool, i imported a couple of .obj but cant see any option to specify what blocks should be picked by the converter when doing the process of converting to schematic.
I see a block pallete option in assign blocks, but it doesn't have options like use only wool, stained clay or concrete "palletes".
I'm doing something wrong of unfortunately this isn't a feature supported by the tool?

@vico93 It sounds like you want to create your own block palette. If you follow the steps here, then next time you open the program there will be a new block palette to select from. Alternatively, you can go into the block palettes folder (which is found in resources/app/res/palettes, copy-paste one of the existing palettes, open it and type in the block names that you want. Make sure to rename this file to something new and it must be named something like my-new-palette.

Closing the issue for now, feel free to reopen it if you're having issues or need help.

@LucasDower just one question: does ObjToSchematic need to be rebuilt to read the pallete? Or the release can load it as well?

@vico93 You don't need to rebuild it but you need to restart the program for it to show up.

Well, i went ahead and created a coloured-blocks2.palette containing only terracotta and concrete blocks:
{
    "blocks": [
        "black_concrete",
        "black_terracotta",
        "blue_concrete",
        "blue_terracotta",
        "brown_concrete",
        "brown_terracotta",
        "cyan_concrete",
        "cyan_terracotta",
        "gray_concrete",
        "gray_terracotta",
        "green_concrete",
        "green_terracotta",
        "light_blue_concrete",
        "light_blue_terracotta",
        "light_gray_concrete",
        "light_gray_terracotta",
        "lime_concrete",
        "lime_terracotta",
        "magenta_concrete",
        "magenta_terracotta",
        "orange_concrete",
        "orange_terracotta",
        "pink_concrete",
        "pink_terracotta",
        "purple_concrete",
        "purple_terracotta",
        "red_concrete",
        "red_terracotta",
        "white_concrete",
        "white_terracotta",
        "yellow_concrete",
        "yellow_terracotta"
    ]
}

But when i try to select this palette on assign blocks i get this error:


update obcs-configuration summary

What changes does this PR introduce?
missing feature
What is the current behaviour?
some parameters are not in the obcs configuration summary
What is the new behaviour
print most (if not all) R/I/L parameters of OBCS_PARAMS.h
Does this PR introduce a breaking change?
Other information:
Suggested addition to tag-index

pkg/obcs: update configuration summary

OK, I am not sure why I didn't see the code in obcs_check.F nor the associated output. I'll try to merge my redundant contribution and we'll see how much remains.

@mjlosch Thanks. Going to merge this PR very soon.


[Feature Request] Favorite/Default Tag Items

I've been playing with Yttr recently, and it adds tags for things like iron, redstone, and gold. Whenever I used them in a recipe tree, it does not resolve by default.
I would want to pick vanilla iron, redstone, and gold to resolve those tags, and regular vanilla glass for the glass tag for example.
At the moment, I can select tag resolutions within one recipe tree, but they're not savable as a default across multiple trees.

Added in 1.0.0


Symfony 5.2.7

Coverage remained the same at 96.273% when pulling d204995ccccfaaccab9bea242ca654aafb21eec9 on symfony-5.2.7 into aed2cb84661110a86d5e14586b67de43f43b93c4 on master.


Restrict non-staff user's access to stock records of linked partners only

Currently a non-staff user has access to all the stock records, if there is at least one stock record from their linked partner for the product. This is wrong in a market place scenario. Access should be restricted to the stock records of the linked partner only. With this fix, non-staff user can create/delete/update stock records for their linked partners only.

Good catch. We should probably at a test case for this in the future.


 I tried with all the tapfire settings and nothing worked. It's not about my config, I've tried legit and HvH config and no result. And yes, I made sure im not double tapping. Sometimes it works fine, most of the time doesn't. It looses ammo when shooting, but the sound is like a pistol and shoots like a pistol. I've tried with fake lag and with no fake lag, and again, no result. And I've also tried going into the test place where nobody is here so I can get a fps boost, and guess what, no results.

do u have antibackstab/aa on?

Doesn't interfere with firing speed as far as I can tell, but I understand if ur annoyed by the audio constantly stopping.


Make SimpleCharacter more flexible

What Happened
SimpleCharacter is inflexible because it imposes a collision shape. Even if no texture is provided, it fallbacks to a 100x100 px square. Also the texture can't be resized from the canvas, but that can be workarounded by adding a Sprite2D as child. So let's do the same with the collision node: don't impose one.
Still maintaining the simplicity of passing a texture image and automatically gain a collision box matching the texture size.
How to Reproduce
Explained above.
Block Coding Plugin Version
main
Godot Engine Version
v4.3
Operating System
Endless OS
Logs or command-line output
No response
Anything else?
No response

Fixed by #216


ROX-13165: Removing security overview section and redesigning network security section

Description
Making modifications in the DeploymentDetails component based on the updated mocks shown here: https://docs.google.com/presentation/d/1DdKZ0PoLk6yDyV9vFM4Q-HpB0N_lXwJ53G-CYAc56Sc/edit#slide=id.g1669bc43c7d_13_51
Checklist

[x] Investigated and inspected CI test results
[ ] ~Unit test and regression tests added~
[ ] ~Evaluated and added CHANGELOG entry if required~
[ ] ~Determined and documented upgrade steps~
[ ] ~ Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs) ~

Screenshots

/retest


Fix sequence rendering

Thanks for this patch. I did not explicitly test this, but reviewed the code changes, and they look good to me.


Feat: Expand auto height implementation to handle auto height use cases.

Description
Expand auto layout implementation to handle auto height use cases.
Use cases handled in this PR:

Change canvas and container-like widget height on adding / removing widgets.
Container height update on content change of individual props, e.g. text, checkbox groups.
Tabs widget use cases - change height on tab change, shouldShowProps update.
Correct modal widget height.
List widget updates - disable auto height, enable manual resizing of item container.
Fix resize loop.

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Manual
Jest
Cypress

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] 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
[ ] PR is being merged under a feature flag

/build-deploy-preview env=release

/build-deploy-preview env=release

/build-deploy-preview env=release

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/ok-to-test

/build-deploy-preview env=release

/build-deploy-review env=release

/build-deploy-preview env=release

/ok-to-test

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/ok-to-test

/build-deploy-preview env=release

/ok-to-test

/ok-to-test


[DOCFIX] Update the chinese document of Configuring-Alluxio-with-OSS.md

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16091/
Test PASSed.


how to pass json data to pivotUI

I converted the object returned from the api into Json by JSON.stringify() and passed the result to pivotUI but got error. I too followed the Input Formats doc and my json data in the same format but still got error. Pls suggest where i went wrong

The output of JSON.stringify is a string, which is not the correct input format: you need to pass in the actual object or array. Basically just skip the stringify :)

Yes, i figured it out later, thanks for your response sir


[#noissue] Upgrade dependencies

Codecov Report

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

@@            Coverage Diff             @@
##           master    #6924      +/-   ##
==========================================
+ Coverage   39.26%   39.27%   +0.01%
==========================================
  Files        3221     3221
  Lines       95272    95272
  Branches    11885    11885
==========================================
+ Hits        37409    37419      +10
+ Misses      54874    54866       -8
+ Partials     2989     2987       -2

Impacted Files
Coverage Δ

.../pinpoint/rpc/client/PinpointClientHandshaker.java
65.00% <0.00%> (-1.43%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
85.36% <0.00%> (+2.43%)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.98% <0.00%> (+2.81%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
66.66% <0.00%> (+11.11%)
:arrow_up:

Continue to review full report at Codecov.

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


Error when using make on Windows

Followed the instructions on the Getting Started page on wren.io. I got the following error:
mingw32-make[1]: Entering directory 'C:/wren'
process_begin: CreateProcess(NULL, printf "%10s %-30s %s\n" run util/build_libuv.py, ...) failed.
make (e=2): The system cannot find the file specified.
mingw32-make[1]: *** [util/wren.mk:241: build/libuv.a] Error 2
mingw32-make[1]: Leaving directory 'C:/wren'
mingw32-make: *** [Makefile:16: release] Error 2```

Do you have a Python interpreter installed? It is needed to build on of Wren's dependencies.

There are various issues in the steps on Windows. I'm working on fixing those with the 0.3.0 stuff, so for now it's better to hold off briefly on trying to fix master.

On windows 32/64, there were some issues need to be corrected , such as file  path separator, included header files ...  @underscorediscovery

yes I stated that right before :) They'll be addressed.

There are various issues in the steps on Windows.

Fixed in 0.3.0, use the visual studio project for a seamless experience.
http://wren.io/getting-started.html


Automate HIL deployment

One of the project for cloud computing class is Automated Secure Cloud Deployment.
For that, all the components of secure cloud need to be deployed automatically. For BMI, there's already an ansible script to do it.
One of the students from the class will be working on automating HIL deployment. I was thinking of doing it in Ansible. Opening up this issue if someone has any feedback on what route to take.
cc// @okrieg @zenhack @mosayyebzadeh

Can I loop in https://github.com/CCI-MOC/hil/issues/45?

@xuhang57 Good one. We need to know all related issues to be taken care of.

An ansible role sounds like a good idea.

It's an open issue, anybody can take this.


Update codebase for SS4

Use namespace ?. (TBC) - either CWP\AgencyExtensions or SilverStripe\CWPAgencyExtensions cc @tractorcow
Sub-tasks can be added separately or as items in a checklist on this issue.

cwp is still the vendor in composer, so I would use this with your namespace also.


Support for letter-spacing

Thank you for creating dropflow! It looks really cool. 🙂
I was wondering if you are also planning to add letter-spacing at some point? It’s the one essential typographic feature I’m currently missing.

Thank you! I'd love to add letter-spacing support. I think it's simple: walk the glyph array and increase advances at the end of each unicode grapheme:

This property specifies additional spacing (commonly called tracking) between adjacent typographic character units. Letter-spacing is applied after bidi reordering and is in addition to kerning and word-spacing. [CSS-WRITING-MODES-4] [CSS-FONTS-3] Depending on the justification rules in effect, user agents may further increase or decrease the space between typographic character units in order to justify text.
For the purpose of letter-spacing, each consecutive run of atomic inlines (such as images and inline blocks) is treated as a single typographic character unit.
Letter-spacing must not be applied at the beginning of a line. Whether letter-spacing is applied at the end of a line is undefined in this level.

— CSS Text §7.2

I did more reading and thinking on this. It's harder than I thought because the spacing has to be applied after bidi reordering. But letter-spacing also needs to be followed when measuring text for fit. If bidi text splits an inline, the common ancestor's letter-spacing is used. It sounds like I'm supposed to do a simple test of "will this split an inline", rather than trying to calculate the "hypothetical" bidi order.
Currently, none of the major browsers do this correctly: test.
LineWidthTracker, ShapedItem.measure, and createLineboxes all need to be updated to have letter spacing state. Those use logical order, but the total number of spaces will be the same as the after-bidi letter spacing. Also ShapedShim will need to gain an advance property since inline-blocks and <img> in the future also get spaces after them through letter-spacing.


Adding KafkaWriter

KafkaWriter writes into more than one Kafka Topic. It needs the following settings:

metadata.broker.list
zk.connect
serializer.class
topics

For example:
"metadata.broker.list" : "<IP_ADDRESS>:9092",
"zk.connect" : "<IP_ADDRESS>:2181",
"serializer.class" : "kafka.serializer.StringEncoder",
"topics" : "topic1,topic2"

[ ]   File: jmxtrans-output/jmxtrans-output-core/pom.xml:L53-64
 The goal of the jmxtrans-output-core module is to have only core output writers, with minimal dependencies. Could you move the output writer to a new module, probably named jmxtrans-output-kafka ?
[ ]   File: jmxtrans-output/jmxtrans-output-core/src/main/java/com/googlecode/jmxtrans/model/output/KafkaWriter.java:L1-134
 It seems that all those getters / setters are exposing internal state. You could probably remove them.
[ ]   File: jmxtrans-output/jmxtrans-output-core/src/main/java/com/googlecode/jmxtrans/model/output/KafkaWriter.java:L1-134
 Complexity of this method is quite high. It seems we can at least extract a method writeSingleValue().
[ ]   File: jmxtrans-output/jmxtrans-output-core/src/main/java/com/googlecode/jmxtrans/model/output/KafkaWriter.java:L1-134
 This isTest condition is dubious. If you want a way to unit test this class, you should provide a way to inject the producer and mock it.

This pull request is now managed with CodeReviewHub. Refresh.

Looks good ! A few minor corrections would be nice. I can do those corrections if you do not have the time / energy / ... let me know ...

Yes, I think you have a better idea about the project structure , so it might be far quicker for you to edit these things as compared to me. But the writer works and writes to Kafka as needed. I tested it thoroughly. Thanks for the quick review and hopefully I contribute more in the future on this project. :+1:  :)

I did some rework and created a new PR (#305). Your review of what I did would be welcomed !

Closed as this is replaced by #305


Make CurrencyAmount comparisons throw when currencies are different

Currently
CurrencyAmount.fromRawAmount(DAI, 1).equalTo(CurrencyAmount.fromRawAmount(USDC, 1))
returns true
Made these comparisons throw if currency is different

@willpote re-opening

Fixed in https://github.com/Uniswap/sdk-core/pull/14/commits/279c52b6c9406d74c45d1c417f951e686594bf90


WIP: fix optional keyword typo

Fix #13

i think this doc is autogenerated so  i will look on the other repo also

yop thx @fabiand  i think is a good entry-point to learn the openapi-spec. I don't think i would need help for this small task, is just about start reading :radio: :bookmark: :sun_with_face:

So many nice emojis!
:tada:
Good luck then :+1: :)

ciao @fabiand, one question since i where around on this issue and i investigate a little.
I discovered the open-api and found out you use this tool for generating the doc. ( never heard about before :smile: )
https://github.com/kubevirt/kubevirt/tree/bf3b2e02837ece11d7ad2666d67033649456f2e7/hack/gen-swagger-doc
I will try this out, but i think we might add in this repo README.md about how to generate the doc, once i will have time to experiment about it ( adding in my todo list)
From curiosity pov, why was choosed the swagger.java client and not something not in golang?
https://github.com/go-swagger/go-swagger, like this one. I am pretty unexpert atm so i just ask blindly :dog:

Yo!
java is only needed to generate the swagger documentaion (not the openapi spec), and maybe there is just no golang tool available generatig swagger docs.

@MalloZup Hi, it would be great if you added readme into this directory !!
Regarding generation itself, we have a target in Makefile for that apidocs, there you can follow what are steps to generate it.

@MalloZup the background is that our golang swagger-generator does not recognize +optional as something special. We may be able to switch to https://github.com/kubernetes/code-generator/tree/master/cmd/openapi-gen. If not, we would have to teach https://github.com/rmohr/go-swagger-utils to interpret +optional.

Ok, so we can switch. The steps on how to do this are outlined in https://github.com/kubevirt/kubevirt/issues/1227


Fix: Bump ktor version to avoid "no implementation found" error

After the initial clone and build of the sample app I stumbled upon the "no implementation found" described in this issue.
Then I updated ktor to 2.3.2 as suggested in the issue and the app stated to compile and run on both platforms, so it fixed the issue.
Interesting observation was that after being compiled with ktor 2.3.2 once, the app also stated to compile with 2.3.1, which initially caused the issue.

Thank you, I'll certainly take a look. I didn't experience this issue on 2.3.1, but an update is always good!


Suggestion: Use PHP builtin methods for IP lookups

Instead of using an external API, there's a builtin method to do DNS lookups instead:
https://www.php.net/manual/de/function.dns-get-record.php

This was returning very minimal data from my testing:
Array ( [0] => Array ( [host] => removedhost.com [class] => IN [ttl] => 3788 [type] => HINFO [cpu] => RFC8482 [os] => ) )

This was returning very minimal data from my testing:
Array ( [0] => Array ( [host] => removedhost.com [class] => IN [ttl] => 3788 [type] => HINFO [cpu] => RFC8482 [os] => ) )

Interesting... did you really use removedhost.com for testing. That one has no A/AAAA records.
Could you try the following snippet?

<?php
$host = "mirror.netcologne.de";

$result = dns_get_record($host, DNS_A);
print_r($result);

$result = dns_get_record($host, DNS_AAAA);
print_r($result);
?>

Interesting... did you really use removedhost.com for testing. That one has no A/AAAA records.
Could you try the following snippet?

<?php
$host = "mirror.netcologne.de";

$result = dns_get_record($host, DNS_A);
print_r($result);

$result = dns_get_record($host, DNS_AAAA);
print_r($result);
?>

Ok, no I removed it for privacy. I see now that you have to define what you want and not let it run the default type. I will look at it or feel free to do a commit :)

Ok, no I removed it for privacy. I see now that you have to define what you want and not let it run the default type. I will look at it or feel free to do a commit :)

@cp6 I have applied a fix in a89f624b082a81a7dc4d8deef7c33964a849b40c
needs to be tested, available in my pull request

@cp6 I have applied a fix in a89f624b082a81a7dc4d8deef7c33964a849b40c
needs to be tested, available in my pull request


Research Uniswap V4

Research Uniswap V4 and the functionality of creating a liquidity pool for each new round will likely require some time and practical testing. Here's how you can approach this task:

Study Uniswap V4: The first step is to study Uniswap V4 and its documentation. A fundamental understanding of how Uniswap V4 works and how to create liquidity pools is necessary for further exploration.

Create a Liquidity Pool: Create your own test project to explore creating a liquidity pool using Uniswap V4. Try to create a pool, add assets, and configure pool parameters.

Practical Testing: Launch several test rounds, each of which creates a new liquidity pool. Evaluate how long it takes to create a pool, how easy it is to configure parameters, and how the functionality expands for each new round.

Distributing Winnings: Develop a mechanism for distributing winnings to each winner. Keep in mind that Uniswap V4 may require you to develop your own smart contract to manage winnings.

Performance Evaluation: Assess the performance of the system, especially as the number of rounds and pool creations increases. Check how convenient it is to manage all this functionality.

Documentation: It's important to thoroughly document all the steps, results, and issues encountered during your research.

Conclusions and Recommendations: Based on your research, draw conclusions and provide recommendations for using Uniswap V4 to create liquidity pools for each new round.

I wanted to share some links I've found and think may be useful:
Introduction:

Alice Henshaw & Sara Reynolds - Introducing Uniswap v4
Uniswap 🛠 Introduction to Uniswap V4

V4 is not deployed yet, but there is a testnet called Conduit Testnet where we can deploy Uniswap v4 and deploy hooks
v4-core and tutorials on Uniswap V4 Documentation are made using Foundry (maybe we should consider switching from Hardhat to Foundry)
If we're gonna keep using Hardhat, we should set it up for EIP 1153 opcodes
I've watched and coded along this lesson, although I don't think I've grasped it all the math yet:

Uniswap v4: Create your own on-chain limit orders hook for Uniswap using Foundry and Solidity
^ text version on LearnWeb3
^ repo I've done following this lesson and updating it to use the latest v4-core

Here are some links I'm thinking of exploring next:

univ4-hooks-portfolio (Compilation of V4 hooks)
awesome-uniswap-hooks (Compilation of V4 hooks)
Uniswap v4 hook for rebalancing a pool
v4-template (Template for writing V4 Hooks)
Dev resources (^ these repos and more)
How to integrate Uniswap 4 and create custom hooks (Solidity Developer . com)
Uniswap V4 Deep Dive with Head of Protocols Sara Reynolds (1:16 hs video)
Uniswap v4 - threat modeling for secure integration

Thank you, we have received answers to many questions and I have checked various things based on them.
Is it possible to add liquidity to the pool for the totalAmount in the closeRound function?


Exercise topics

Add topics for the exercises. These are my initial "estimations" based on how I solved the exercises, but I think it should be an iterative process starting there.

This is conflicting, would you mind rebasing onto the most recent master?

I rebased. Let me know if anything else is needed. Thanks.


warning! unknown event group with latest sdk

Not big issue, but annoying.

And what is it exactly? What number does it have?

[sciter] warning! unknown event group 0020
[sciter] warning! unknown event group 0020

BTW, could ypu help answer https://sciter.com/forums/topic/menu-not-be-hidden-when-open-dialog-on-linux/#post-75593?

Huh. How is 020 unknown if it's a size event 🤔

Confirmed that I'm seeing the same thing.

It as indeed annoying. Thanks for the catch!

It as indeed annoying. Thanks for the catch!


Example docs are wrong/old?

@iainnash just discovered some issues in the example docs we provide on the readme for the zdk for getting set up.
this is what we have:
const zdk = new ZDK({
endpoint: API_ENDPOINT,
networks: [
{
chain: Chain.Mainnet,
network: Network.Ethereum,
},
],
apiKey: API_KEY // optional!
);
this is what it should be:
const zdk = new ZDK({
endpoint: API_ENDPOINT,
networks: [
{
chain: ZDKChain.Mainnet,
network: ZDKNetwork.Ethereum
}
]
})
notice that the current version is missing the "ZDK" tag on ZDKChain and ZDKNetwork. Wiithout those (what is currently shown on the readme) you are unable to identify the correct params

just realized the formatting looked completely wrong on that comment ^ so adding screenshots here for further clarification
current:

what it should be:

Fixed in #81


Fix migration bugs

Fixes two bugs with the migration: multiple dialogs opening and
sometimes setting the runtime image property of the nodes to the
display name instead of the value.
The first bug was partial fixed in #2045 but the code setting the
flag to true was lost during revision.
The second bug described in #2208 was due to a mistake I made in #1860
by using  the transient copy of the pipeline used by the GUI instead of
the persisted content when running the migration.
This caused a race condition when the API call to get the runtime
images returned and triggered the changeHandler to update to the
display names before the user clicked the migrate button in the dialog.
Fixes #2208
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Can we make sure we have a more descriptive pr/commit description before merging.


Can't Login Owncloud

TypeError: Cannot read property 'data' of undefined
at owncloud.js:8
at owncloud.js:1
at owncloud.js:1
at MutationObserver.a (owncloud.js:8

mind sharing the code you are using? THX

I'll reopen this issue once there is feedback


Rename the link to blocksite.net to walrus.site in flatland move contract

Was the contract redeployed? Should this issue be closed?

no, not yet.

Fixed after the testnet update


setLiveWallpaper doesn't work on xiaomi 10 miui 14

Describe the bug
Everything works fine when I set up static images(jpg/png) on MIUI14，but when i use liveWallpaper ,both the lock screen and home screen changed back to the default wallpaper, and show toast successfully.
This may be a problem with MIUI itself, as I am still testing on Samsung and Huawei phones, and they are normal.
I want to know if there is a way to catch this exception and handle it.
sorry,my english not good,thank you watch this.
code

Screenshots

Flutter:

version 3.7.11

Dart:

version 2.19.6

Additional context
phone: xiaomi 10
android version Android 13
MIUI 14.

I have fix this issue and created a pull request modify your pubspec.yaml
async_wallpaper:
  git:
    url: https://github.com/codenameakshay/async_wallpaper.git
    ref: 10ae084c26d644a8c653aa0e63265d41a5603c8c


Reuqest for simple

This is not actually issue i just got stacked about implementing validation in multi step form. is there any good example of how to implement this so that next previews is enabled based on validation and inputs get appropriate validation visual? i tried to wrap everything in  tah but angular validation does not work in this case.

I found good example of this in samples page. closing this issue.


Improve Arabic messages

Just simple and minor orthography fixes in Arabic messages and adding missing markup and punctuations that exists in English messages but not in Arabic ones.
Also adds Arabic long vowels to vowels list to make character's mouth move in Arabic.
While reviewing please note that as Arabic is written from right to left the space before comma is on its right not left so I've removed the one on its right (before) not left that might be confusing to the reviewer but not someone who uses the script everyday.

Also is it okay to add another language here?

Thank you for the contribution! Much appreciated.
Regarding adding another language, we'd prefer not to add more, as this demo aims to show the support for one latin script, one CJK language, and one right-to-left language.

GitHub seems to be having some issues at the moment. I can´t merge the PR or push changes right now. I'll try again later.

Oh, I almost have forgot this, thanks for considering this 😊

Regarding adding another language, we'd prefer not to add more

That's totally understandable.

GitHub seems to be having some issues at the moment. I can´t merge the PR or push changes right now. I'll try again later.

I can rebase if you think is needed. Thanks 😊


[Suggestion] Blast proof redstone torch

Add a blast proof redstone torch which is built with obsidian.

If this ever happens, it might be a while from now, because I've no clue how to make a torch that behaves like a redstone torch - I need to read over how the burn-out logic works.


Brazilian Portuguese translation

Hi, I saw that is missing a Brazilian Portuguese translation. Could I try creating it and making a pull request?

Ok, thank you!


No schema validation tests for update service instance and create binding

There are tests checking service instance create schema validation, but not for update service instance or create binding. Can someone look at adding these?
The existing schema tests should also be optional, as schemas are not a required part of the OSBAPI specification.

I'm interested in this project, so I can take a look if nobody else wants : )

Already solved by #63


Fix coveralls reporting

Resolves #88
We should wait until my fork CI finishes to check, but since it worked for janus_core, hopefully this should work as is.

The build for this PR seems to match my local results now:
---------- coverage: platform linux, python 3.11.8-final-0 -----------
Name                          Stmts   Miss  Cover
-------------------------------------------------
aiida_mlip/__init__.py            1      0   100%
aiida_mlip/calculations.py       27      0   100%
aiida_mlip/cli.py                31      2    94%
aiida_mlip/data/__init__.py      22      0   100%
aiida_mlip/data/model.py         59      4    93%
aiida_mlip/helpers.py            34     25    26%
aiida_mlip/parsers.py            23      3    87%
-------------------------------------------------
TOTAL                           197     34    83%


Merge pull request #1 from rhelmot/main

Merge with Updates

You should generally avoid merging stuff from my repository - I like to use the squash or rebase merge options, which will make it problematic to merge. You should do your work on a branch other than main, and then after I merge your changes you pull the main branch and create a new branch based on the new main in order to start your next round of changes.


The table column name "Temp storage (SSD) GiB" is incorrect for the D series

The table column name "Temp storage (SSD) GiB" is incorrect for the D series.
It should refer to regular HDD. That is why the size is bigger than the DS series.

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

ID: b681fb36-d142-204e-85a7-0ab5f86edc75
Version Independent ID: f71468a6-899e-5c3d-044f-606fca3d4c88
Content: Dv2 and DSv2-series - Azure Virtual Machines
Content Source: articles/virtual-machines/dv2-dsv2-series.md
Service: virtual-machines
Sub-service: vm-sizes-general
GitHub Login: @joelpelley
Microsoft Alias: jushiman

Are you suggesting using HDD instead of SSD in braces in the column name? Can you please clarify on the requested change in the doc. Thanks.

When on the original hardware, the D-series was on HDDs.  That hardware has been retired, but we maintain the disk volume as it was then, now on SSDs.

Thanks @joelpelley for the clarification.
@pavelrozenberg
Hope that helps. We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.

Thank you all for the clarification :)


[Suggestion] GTEU Support

As the title says :D

I'm not sure what you mean by GTEU support. Could you elaborate on what you are hoping will be supported?


render coastline with indication of water side/land side

To help users drawing coastline to set the correct direction (which I've been told is still important).
Maybe a water gradient or pattern?

Closing as #1475 has a more complete list.


fix: fix unnecessary recomputations due to macros

This computes a macro's fragment kind eagerly (when the calling file is still available in parsed form) and stores it in the MacroCallLoc. This means that during expansion we no longer have to reparse the file containing the macro call, avoiding the unnecessary salsa dependencies (https://github.com/rust-analyzer/rust-analyzer/pull/8746#issuecomment-834776349).
Marking as draft until I manage to find a test for this problem, since for some reason typing_inside_a_function_should_not_invalidate_expansions does not catch this (which might indicate that I misunderstand the problem).
I've manually confirmed that this fixes the issue described in https://github.com/rust-analyzer/rust-analyzer/pull/8746#issuecomment-834776349:
    7ms - parse_query @ FileId(179)
   12ms - SourceBinder::to_module_def
       12ms - crate_def_map:wait
            5ms - item_tree_query (1 calls)
            7ms - ???

Hmm, why does that stop us from recomputing the ItemTrees? Their input still changed after all

Ok, I think I figured that out. We re-run parse_macro_expansion query, and this query returns a green node of the expansion. The green node is the same, so item tree is not recaltucated.
Why is it recalculated for completion though? Because parse_macro_expansion is an LRU query! We seem to have more that 128 trees, and recalculate all of them, so LRU gets busted.

I guess  we can change the test to look at both item trees and macro-expansion queries.

Yup, upping this const: https://github.com/rust-analyzer/rust-analyzer/blob/b43921cdddb4e703787ab87b0ea3f8eaff94f6c9/crates/base_db/src/lib.rs#L92 removes recomputations on master.

Yup, that seems to catch it, nice!

bors r+


Fallback to theme icon when no custom icon is set in autocompletion

Partially fix- #95789

Thanks!

Cherry-picked for 4.3.1.


How about accounts?

Maby we should start thinking about creating views and models for users accounts?

Yes, I think we should. I was thinking of doing it when I do the log in
with Twitter function
On Friday, October 26, 2018, HallBregg<EMAIL_ADDRESS>wrote:

Maby we should start thinking about creating views and models for users
accounts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ykdojo/editdojo/issues/19, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABukw6Xne51rgyezhSoHrfdWqzeprgqAks5uo2uKgaJpZM4X85Lz
.

Maby I could do some mock-up of it?

Sure! That’ll be great!
On Saturday, October 27, 2018, HallBregg<EMAIL_ADDRESS>wrote:

Maby I could do some mock-up of it?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ykdojo/editdojo/issues/19#issuecomment-433604919, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABukw4PP36Wt5N6QnqT_j_QBjlkKThkwks5upCUZgaJpZM4X85Lz
.

@HallBregg What do you use to make mock-ups please?

@HallBregg I was just asking about the software you're going to use. Actually, I know about Balsamiq ; it's a commercial software. But I am not sure if it exists an equivalent or other softwares which are used.

Yeah, I understood. Maby I should use word "mock up" . All I meant was to make fully working login, and registration (accounts) views and models in Django. I am more backend developer. Dont know much about style and how should it look visually.

@HallBregg Oh I see, thanks for your reply.

@HallBregg
http://pencil.evolus.vn
Pencil is built for the purpose of providing a free and open-source GUI prototyping tool that people can easily install and use to create mockups in popular desktop platforms.

@HallBregg thanks so much. I'm wondering if it exists an online similar tool (open-source free will be great !)

Should there be a username or is email find?
There are pros and cons of Usernames so it is important to think about.
If you decide not to use Username, you would have to use Cumstom User model for Django users.

Okay I'll think about this when I work on #11.
Closing this issue for now.


[Macros] Remove plugin related flags from toolchain Info.plist

-external-plugin-path flags pointing XcodeDefault.xctoolchain were needed to use macro plugins in Apple SDK. But since Apple macro plugins are not in .xctoolchain anymore, these flags are useless.
rdar://125498074

@swift-ci Please smoke test

@swift-ci Please Build Toolchain macOS Platform


style url-generator

Follow up: Styling for the KLIPS URL-Generator.

Very nice @sdobbert !

Will be closed in favor of #228


Add note about source link

@weshaggard is it expected that check-enforcer won't work for such files?

Correct if there is no pipeline triggered then check-enforcer will not pass. In such cases that are expected you just have to override as you did.


Achieve 50 TPS or 300 txs per a block in test harness

Currently, test harness reports up to 10 TPS which in most cases means  around 56 transactions accepted within 5-6 block time.
Obviously, the main limitation comes from two constants we've got:

// MaxTxSetSize defines the maximum amount of transactions.
// It is TBD along with block size and processing.MaxFrameSize.
const MaxTxSetSize = 150000

// MaxFrameSize is set at 250000 bytes.
MaxFrameSize = uint64(250000)

Highest result achieved so far.
	 msg="Measured TPS 11.20 (56/5)" height=14
	 msg="Measured TPS 9.33 (56/6)" height=15
	 msg="Measured TPS 9.33 (56/6)" height=15
	 msg="Measured TPS 9.33 (56/6)" height=16
	 msg="Measured TPS 11.20 (56/5)" height=17
	 msg="Measured TPS 11.20 (56/5)" height=17
	 msg="Measured TPS 9.33 (56/6)" height=18
	 msg="Measured TPS 9.33 (56/6)" height=19
	 msg="Measured TPS 9.33 (56/6)" height=19
	 msg="Measured TPS 9.33 (56/6)" height=19
	 msg="Measured TPS 9.33 (56/6)" height=19


Add RUSH E 2 by sheet music boss

Song Name
RUSH E 2
Artist Name
sheet music boss
Source
Youtube
Youtube Link
FuMWmnhjCEQ
Direct File Link
No response
Song ID
845288
Start Offset [ms]
10
Extra
Created from Auto Nong.
Created for level: 64026102
Downloaded: No

accept
сб, 9 нояб. 2024 г., 08:52 auto-nong[bot] @.***>:

Thanks for contributing!
Someone will soon review your request.
Here is the JSON result for the reviewer:
{
"name": "RUSH E 2",
"artist": "sheet music boss",
"source": "youtube",
"startOffset": 10,
"yt-id": "FuMWmnhjCEQ",
"songs": [
845288
]
}
To accept the request, a reviewer needs to write "accept".
—
Reply to this email directly, view it on GitHub
https://github.com/FlafyDev/auto-nong-indexes/issues/6151#issuecomment-2465981942,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BMBAZ3NYI53GSFXJ4EAF6YLZ7VTF3AVCNFSM6AAAAABROTFBM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRVHE4DCOJUGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Detect default dtype on array-likes

This is a fix for issue #8461 where da.full refused Dask arrays for fill_value as the dtype would be detected as object.

[X] Closes #8461
[X] Tests added / passed
[X] Passes pre-commit run --all-files

I was a bit confused that pytest was raising errors for FutureWarnings related to np.ndim and np.dtype, not sure whether it was due to versions in my dask-env environment or something I need to implement differently in this PR.

Can one of the admins verify this patch?

Thank you @aeisenbarth for submitting this PR and for adding in the test! I think @ian-r-rose can review this?

ok too test

Those are telling you that dask will trigger immedate computation on your fill_value. @scharlottej13 mentioned that in her response on the original issue. To get past these warnings in the tests you can explicitly ignore them in the test.

We've seen a number of people accidentally hitting this, and the error message isn't particularly helpful. I wonder if we can come up with a better message.

We've seen a number of people accidentally hitting this, and the error message isn't particularly helpful. I wonder if we can come up with a better message.

Yes! We can come up with a better warning for sure! We can also implement more methods on dask array.

@aeisenbarth I think my suggestion was missing an import that you need to add to test_wrap.py from dask.array.utils import assert_eq

Thanks for working on this @aeisenbarth! I think this will be ready to go once you:

merge in main
undo your changes in setup.cfg
change assert len(record) == 2 to assert len(record) == 1 since #8502 is merged.

Let me know if any of that is unclear or if you would prefer for me to take over and push to your branch.

Thanks for noticing my local setup.cfg change which I hadn't intended to include in the commit.
Now I have fixed setup.cfg, changed the assertion and merged aeisenbarth:da-full-dtype-detection into aeisenbarth:main. Is this the normal procedure? Do I need to edit the pull request to aeisenbarth:main?

Sorry I meant merge dask:main into this branch.

Now it's upstream main is merged into this branch.

Now it's upstream main is merged into this branch.

@jsignell This looks close to ready, shall we resolve the remaining things and merge?

@jsignell is this good to merge?

Yes thanks for the ping @scharlottej13! This looks great @aeisenbarth thanks for sticking with it :)

rerun tests

I noticed that the GPU tests hadn't run, but once those pass I'll merge this in!


Publish the gem

Currently it's only available through github, bundle it an upload to rubygems. Then update README.

I think it would be useful to have a TODO list for this. Here are the things I can think of:

[ ] Add code of conduct
[ ] Add sponsorship section
[ ] Add our standardrb style guide
[ ] Apply our code style guide (e.g. I've seen some usage of single and double quotes in some files)
[ ] Improve documentation to talk about the usual usage with Rubygems.org

Anything else @bronzdoc @rdormer?

another question for the @fastruby/black-bunny-brigade , should we modify the author too? add some reference to ombulabs/fastruby there too?

[ ] Add licence
[ ] Contributing Guide

I wonder if we should use the<EMAIL_ADDRESS>as the author or maybe<EMAIL_ADDRESS>that way the gem is not tied to any of use but the company

I wonder if we should use the<EMAIL_ADDRESS>as the author or maybe<EMAIL_ADDRESS>that way the gem is not tied to any of us but the company

Not sure about this. I added everyone who pushed a commit in the project.
I have pushed version 1.0.0 to Rubygems.org and I've created issues for the remaining items in the TODO list:

Add badges to the README: https://github.com/fastruby/dotenv_validator/issues/17
Add code of conduct: https://github.com/fastruby/dotenv_validator/issues/16
Add sponsorship section: https://github.com/fastruby/dotenv_validator/pull/19
Add our standardrb style guide: https://github.com/fastruby/dotenv_validator/pull/18
Apply our code style guide: https://github.com/fastruby/dotenv_validator/pull/18
Improve documentation to talk about the usual usage with Rubygems.org: https://github.com/fastruby/dotenv_validator/issues/20

Closing since 1.0.0 was released


Added irth (myself) to the contributors

What:
I added myself to the contributors table :)

Why:
Because I contributed some code.

How:
Using algorithms and coding.

Using algorithms and coding.

😂

I'm really sorry about all the trouble you had! I should probably try and figure out what's wrong with that script!


[Bug] 100% CPU loop when zoomed far in

When map is zoomed beyond the provider's zoom ability (?), SimpleInvalidationHandler continuously sends invalidate() leading to 100% CPU usage.
I could repro this in the sample app; start it in Profiler, take any map and continue zooming on some meaningfull location, say New York downtown, Wall Street. After a while you will hit zoom level with constant 100% CPU in Profiler (where you see Trinity Church full screen)

When I debug my app, I see SimpleInvalidationHandler
	public void handleMessage(final Message msg) {
		switch (msg.what) {
		case MapTileProviderBase.MAPTILE_SUCCESS_ID:
			if (mView!=null)
				mView.invalidate();
			break;
		}
	}

flooding the invalidations. Here's the log:
https://gist.github.com/pasniak/10e5863ded98a21cf132c3e21c36a9d8
You'd say: do not zoom so far... yet I want to zoom further to show overlayed images, which are at smaller scale than the available tiles...

Just found an elegant solution.
I may PR tonight.

Just PR'ed #1107
@pasniak Could you test your issue with this fix?

I meant 100% core - this can drain the phone overnight if left zoomed in... :-) Let me try tonight, or, if not, tomorrow.

Tested. Works great (in emu and my !  I upgraded from my frozen master as of Apr 2018 and all seems in order.

great, i'll merge and hopefully cut a new release tonight

https://github.com/osmdroid/osmdroid/issues/131 related

@pasniak I have the impression that the recent code merges fix at the same time this issue, #131 and #989. Ready to close them all?


Automatically inject preload scripts

With this PR, we automatically inject the correct preload scripts via the Electron session.setPreloads() API.
This has a number of advantages:

Lots of code removed and simplified
Users no longer have to supply appName in any process
Users no longer have to consider preload scripts

No more bundling issues with incorrect target for preload (#351)

Renderer code no longer needs to reference Electron

No need to add electron to externals when bundling (#355)

No longer need to 'ping' the main process to check the SDK has been enabled

Disadvantages

Requires a major version bump as it contains breaking changes
The session.setPreloads() API was added in Electron v2 so that would become our minimum supported version (previously v1.8)

Electron v1.8.8 reached EOL 3 years ago

preload scripts have to be available on disk which presents some issues if the main process is bundled. Currently preload scripts get transpiled to variables in code and are written to userData directory at runtime
If a user is already using the session.setPreloads() API, they will likely overwrite our preload scripts. I've added a link to the docs where we can help users diagnose and fix the issues they may encounter.

IMO no tool or framework should make architectural decisions like this for apps by default, this kind of thing (if it had happened without us noticing) could cause untold security issues for downstream apps that the Sentry SDK might not consider but we definitely do.

I posted this PR to the Electron Discord security channel specifically because I doubted it was a sensible thing to do!

You can solve this using additionalArguments in newly created webContents instead

appName is only required in the renderer with Electron pre v9 because it's required when calling crashReporter.start().
I didn't know about additionalArguments and it looks like it's been there for years.
The problem with additionalArguments is that this SDK does not get involved in creating BrowserWindows, and there's currently no way to intercept their creation (https://github.com/electron/electron/issues/13846).
It's probably simpler to just require passing appName in the renderer if you're using older versions of Electron 🤷♂.

Can you provide more context on why this ping is needed? I might have alternatives to remove it

I think it is there mainly to improve developer experience. If the renderer cannot get an IPC response from the main process it logs a console.warn reminding devs that you also need to init the SDK in the main process if you want anything to work.

Closed in favour of #361


Using set_error_handler() in StreamHandler

Currently StreamHandler has:
set_error_handler(array($this, 'customErrorHandler'));

and then:
private function customErrorHandler($code, $msg)
{
    $this->errorMessage = preg_replace('{^(fopen|mkdir)\(.*?\): }', '', $msg);
}

Setting an error handler to a private method won't work, as the handler gets called from outside the object scope and the callback method needs to be public. Furthermore, when monolog is used to handle php errors,  set_error_handler() gets called from within the error handler callback itself, which can lead to pretty unexpected results.
I have spent several hours trying to figure out why not all the messages are being logged using monolog, and it turned out that one of the error handlers was being popped from the stack when StreamHandler is used.
Commenting out set_error_handler() and restore_error_handler() within write() method fixed the problem.
It seems that the problem only appears when there is more than one error handler stacked.

As per https://3v4l.org/D729B - this seems to all work as expected, and does not drop error handlers. If you can't provide more details I am afraid we can't do much here.


Update to Minecraft 1.17

Also add translations and bump version to 0.1.2.

Welp, it's not perfect.
Will add some comments to the changes soon.

Forgot to mention: This is built for 1.17 but I tested it on 1.17.1, so it should work on both versions.

very nice PR. thanks!
tbh i'd prefer if it would not do 10x in one PR tho. like why did you have to include translations in here? don't get me wrong it's nice, but it's messed up in german so it's holding back the whole PR.
Please either remove the german translation or resize the box.

why did you include translations in here? (...) it's holding back the whole PR

Because I wanted to leave that to you and the MOD_ID thing was also an open question.
I removed the translations again (but left the TranslatableText) so it's like before and I also think there's no real benefit of changing MOD_ID, so I left it as-is too.

The rendering of the sun & moon actually is broken. It should not have a white circle around.
In this branch

before

Getting the fix into LibGui might take a while so I added a temporary workaround for now.

thanks again!


feat(navigation): remove a11y warning

containers with click listeners should be interactive. This PR ensures
that only clickable elements receive click listeners.

closes #14


SSH gateway support

Xenon 1.0 supported SSH gateways (see #148). This is useful if you need to access resources via an
intermediate server (also called a "jump server", "door node", etc).
In 2.0 we do not support this feature yet.
In 1.0 properties where used to enable gateways. For example by setting:
 "xenon.adaptors.ssh.gateway" = "ssh://example.com"

The SSH connection would be routed via an intermediate SSH link to "example.com"
I feel the properties approach is a bit of a hack and it would be nicer to explicitly state this in the location.
For example:
"somemachine.org/workdir via:example.com"

One issue here is that the user name and authentication may differ for both machines.Since we can only supply a single Credential, we need to include this information for the intermediate machine in the location as well:
"somemachine.org/workdir<EMAIL_ADDRESS>
we can even include a password if necessary:
"somemachine.org/workdir<EMAIL_ADDRESS>
What is a bit weird now is that we allow the credentials to be added for the gateway, but not the target machine. We could also allow this for the target machine, giving us:
<EMAIL_ADDRESS><EMAIL_ADDRESS>
or even:
<EMAIL_ADDRESS><EMAIL_ADDRESS>
Any opinions ?

After some discussion we decided that the simplest approach would probably be the cleanest option:
 "somemachine.org/workdir via:example.com"

Where Xenon would do the following:

a connection is set up to example.com using  DefaultCredential()

if this fails, a connection is set up to example.com using the Credential specified by the user

if this fails, we cannot create the connection

through example.com we setup a tunnel to somemachine.org
we connect to this tunnel and authenticate using the  Credential specified by the user

This implies one of the following:

the user has to setup the local .ssh/config such that example.com can be reached. The correct username and certificate are specified in this config.
the user uses an SSH agent to handle the authentication.
the user has the same username and password/certificate on both machines

I'll see if this is actually implementable using the SSHD library.

After some more discussion we concluded that the handling of the credentials (as described above) is too vague. It is better to explicitly provide a Credential for every step.
This does require an implementation of Credential that can store multiple Credential which we don't have at the moment.
The result would then be:
 "somemachine.org/workdir via:example.com"

and a call to create which gets a ArrayCredential (or whatever it's called) that contains one Credential for somemachine.org and one for example.com.


feat(app): gather all protocol and calibration data

fix #8097

Overview

This PR includes all of the selectors needed to gather data on required labware from a json protocol and check that against the labware that is currently on the robot.
Changelog

Create selector to synthesize needed pipette and tiprack info from protocol
Create selector to check if necessary labware is on the robot
Create selector to synthesize matching labware and calibration info
Create selector to determine whether robot calibration step is complete
Display this data in plain text in the robot calibration step in the app
TODO: Fix TS errors and add testing

Review requests

Am I missing any possible reasons that the calibration step is not complete? Is this all of the data we need to consider to display all scenarios?
Risk assessment

Mid - behind feature flag

Also, I see you moved RobotCalibrationStep into RunSetupCard. When I did labware setup I kept LabwareSetup as it's own directory as a sibling to RunSetupCard, and I see @jerader did the same for ModuleSetup.
It doesn't really matter to me which structure we choose, but we should be consistent in where we place each of these steps (since they're siblings), and be consistent in the naming (postfix with Step or not).

@shlokamin I don't have a strong opinion on how these should be organized. I'll drop the postfix Step since I'm last to merge. For file organization I was going off of @b-cooper's suggestion in this comment, which I do think makes sense since all three of these are technically children of RunSetupCard, but since that's more of a structural component without a lot of logic nI don't mind if the children are outside of the folder. https://github.com/Opentrons/opentrons/pull/8182#discussion_r679999105


Handle standard changes to origin getter

Ref: https://github.com/nodejs/node/issues/48157

Note that I recommend at least a minor version bump... if not a major version bump... because you are changing how things are supposed to work (admittedly in a small way).


Consider adding additional unit tests from peripheral Schnorr library

We have a further library of tests for secp256k1 Schnorr signatures that you may wish to port and use to test your code.
https://github.com/decred/dcrd/tree/master/dcrec/secp256k1/schnorr
The library could use more testing and feedback as well.

we're planning on adding schorr at some point, but I don't think it's very near on our priorities list.
Bit confusing to see a file called 'ecdsa.go` in a schnorr library.

ah forgot which repo I'm in. Glad to see other people are finding it

We used "ECDSA" in each of the libraries to refer simply to elliptical curve digital signature algorithms, but I can see how that could be confusing. The cleanup of these Schnorr libraries in Decred is an eventual goal. There is too an Ed25519 library that you might find useful for tests and multisignature.
https://github.com/decred/dcrd/tree/master/dcrec/edwards


Why does SleepPhase need ImageDecoder as input?

One of the key inputs to SleepPhase is the ImageDecoder, but every reference to it in sleep.py and predict.py is either an assertion or used to get a setting (e.g. "n_tiles_per_image"). This is because "ImageDecoder" actually has impact through the "SimulatedDataset" during training.
To avoid duplication and possibly conflicting state by having two "ImageDecoder" objects, I propose eliminating the one that is given as input to SleepPhase.

Needed for plotting while training


sdm --shrink elicits a kernel error on Bullseye if IMG on NFS share

Title says it all. See https://forums.raspberrypi.com/viewtopic.php?t=325703 for details.
It doesn't appear to cause data corruption, but it is disconcerting to see in the system log!

Still an issue in Bookworm, but since it doesn't seem to be affecting anything, and nobody else is identifying it as a problem, closing this issue.


what is the wrong thing i did here?

Challenge Use an ID Attribute to Style an Element has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.103 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form action="/submit-cat-photo" id="cat-photo-form">
  <input type="radio" name="indoor-outdoor" checked> Indoor
  <input type="radio" name="indoor-outdoor"> Outdoor
  <input type="checkbox" name="personality" checked> Loving
  <input type="checkbox" name="personality"> Lazy
  <input type="checkbox" name="personality"> Energetic
  <input type="text" placeholder="cat photo URL" required>
  <button type="submit">Submit</button>
</form>

You forgot to change #cat-photo-element {   background-color: green; } to #cat-photo-form {   background-color: green; }

@tareqnofal Github If you are stuck on a challenge, please use the Help button. This will take you to a dedicated chatroom for coding help. Or you can use google (or a search engine of your choice) to find a solution.
GitHub issues are for problems relating to the FreeCodeCamp website itself.
Many Thanks

thanks i will do that

its not working, tried all the option

the same problem with me and i have the same code

Hi, @Reicelo, the form id = "cat-photo-form" not the same with "cat-photo-element". please check your code.


No need to wait for HTML here

Module scripts defer by default!
https://github.com/follywolly/web-app-from-scratch-18-19/blob/960a98a709d761ef74e11f55198b69b69f2db8b2/src/js/app.js#L26

Done!


update crypto (2.4)

Signed-off-by: Wei Liu<EMAIL_ADDRESS>

refer to https://github.com/stolostron/backlog/issues/18840

/lgtm


[Request] .env.* file icons

Hi guys, how about standard icons for env?
I found out that you parse only *.env but how about .env, i.e. in react apps I have next list of env files:
.env.development
.env.production
.env.test
.env.development.local
Could you add it pls?

duplicate of https://github.com/vscode-icons/vscode-icons/issues/1563


ExistValidator for relations

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
yes

Fixed issues
#8092

this PR leads to a Single Responsibility violation. IMO, this should be implemented with a RelationExistsValidator. Its already getting confusing when which Validation logic will apply.

@samdark , @SilverFire do you agree it should be a new validator?

@dynasource I do not agree, as far as I see targetRelation is just another way of setting targetClass and targetAttribute so a new class would duplicate a lot of the functionality.

This requires #13618 to work properly:
$model->relation_id = 1;
$model->validate();
$model->relation_id = 2;
$model->validate(); // validation uses old relation for $model->relation_id = 1

@cebe

I do not agree, as far as I see targetRelation is just another way of setting targetClass and targetAttribute so a new class would duplicate a lot of the functionality.

Can't this new class just extend the ExistValidator and configure it correctly?

@SamMousa that is a good idea! :)

how about something like
class ExistRelationValidator extends ExistValidator
{
     public $relationName;

     public function validateAttribute($model, $attribute)
     {
           $relationQuery = $model->getRelation($this->relationName);
           // exception will be thrown if relation is invalid.
           $this->targetClass = $relationalQuery->modelClass;
           $this->targetAttribute = $relationalQuery->link;
           return parent::validateAttribute($model, $attribute);
     }
}

Maybe add some php doc to $targetClass and $targetAttribute properties that they will be overwritten by the relation defined in $relationName.
This would also solve the issue presented by @rob006

I'll make a PR, have something simpler in mind.

@yiisoft/reviewers need reviews.

I still think that #13618 should be merged first to avoid weird bugs with cached relations on validation.

#13618 is now merged. @SamMousa do you remember what was that simpler variant you were writing about?

Yes, the idea was to use the relation definition itself as parameter.
[['author_id'], RelationExistValidator::class, 'relation'=> $this->getAuthor()]

And then:
public function init() {
    parent::init();
    $this->targetClass = $this->query->modelClass;
// Add some checks here for exception cases where there are multiple link thingies.
    $this->targetAttribute = key($this->query->link)
    // Add any other query filters to
    $this->filter = $this->query->where;
}

This is by heart untested and not 100% sure if this was the exact original idea.

What's the benefit compared to using relation name?

Type checking / IDE auto complete

Well, the con I see is that Query is being built at configuration time instead of validation time...

That could affect index type of actions significantly...

That's a fair point; valuable thing is the configuration of the validator so using name is fine as well.
Note that instantiation in php is fairly cheap though, and validators themselves already are lazily loaded.

Right but for a 100 ARs per page 100 extra inits for validators may be significant.

But you don't initialize them unless you validate them?

Merged. Thank you!

We do. A call to relation inits the relation class.

I noticed that it checks for the relation existence with
if (empty($model->{$this->targetRelation})) { .
It will fetch the relation's row(s) in memory?  If so, it wouldn't better to check with a ->exists() call?

https://github.com/yiisoft/yii2/pull/15647


NUM_STYLES already defined

NUM_STYLES is included in the variables at https://codelabs.developers.google.com/codelabs/tensorflow-style-transfer-android/index.html?index=..%2F..%2Findex#4 (Section Add the inference code) that wea re supposed to add to the Activity.
But it is already defined in StylizeActivity at https://github.com/googlecodelabs/tensorflow-style-transfer-android/blob/codelab-finish/android/src/org/tensorflow/demo/StylizeActivity.java#L74
It seems to have several uses already, so rather than doing a Pull Request and removing it from the code, it would be best to drop it from the CodeLab text

Great point, thanks for the submission. I've added some comments to the codelab, does that look OK to you? Codelab step 5
(closing, will reopen if there's further suggestions)

Perfect!


Gantt Charts

Hi,
Is it possible to add a Gantt chart (http://en.wikipedia.org/wiki/Gantt_chart)-type functionality to the Kanboard dashboard, or perhaps to one of the project or analytics pages?  It'd be incredibly helpful :)
Thanks,
Algy

Gantt charts the a project using JQuery: http://taitems.github.io/jQuery.Gantt/

Duplicate of #832

Gantt chart is available in the dev version, see https://github.com/fguillot/kanboard/blob/master/docs/gantt-chart.markdown


: update selenium standalone to version 6.5.0

This update is needed to resolve chrome to open in maximize mode
while runnig in headless mode.
This commit is dependent on below commit
https://github.com/vvo/selenium-standalone/pull/285
Once above commit gets merged selenium standalone would be updated
to have version 6.5.0 and will point to use chrome driver 2.30.

These failed errors will be resolved once below PR gets merged
https://github.com/vvo/selenium-standalone/pull/285/files

Dependent PR has already been merged so @christian-bromann  pls merge it

@christian-bromann  @vvo  can you please merge it ?

Published to NPM as v0.0.9. Thanks!


Fixing PMD Issues

Contains
Brief description of what the PR does like "Fixes #12345"
How to test
Brief description of how to test / confirm this PR before merging
Outstanding before merging
If anything. You can use neat checkboxes! Feel free to delete if not needed

[ ] Need to consider use case x
[ ] Still have to adjust the wiki doc
[ ] Will need translation work

Can one of the admins please verify this patch?

Hey there,
Thanks for taking the time to do these changes and make a PR. Unfortunately due to the size of this PR and the lack of documentation of these changes in your commit message I'm going to close it. In the future it would be beneficial to aim for smaller Pull Requests and ensuring your commit message is clear and precise as to exactly what your changes will be doing in detail.
Also the issue of file conflicts would need to be addressed. Feel free to make a new Pull Request with the above feedback taken into consideration.
-engiValk

To add onto the above, the reason why there are so many files is because you have included thing that should not have been committed. For instance, the IntelliJ project files.
Another reason is that there are a number of changes that are not to do with the logging changes you've made. This is likely from autoformatting files with some seeing turned on.
It seem like you didn't set up the game by following our developer guide. Try setting it up with that and then we can try and review the PR properly for you.


Refreshing doesn't retain state

Describe the bug
I'm not sure if this is a bug or an intended behavior, but it is a deal breaker for me if it's intended.
Say I edit some files. Then I refresh the page. On local VSCode, all my files I edited are still there & ready to be staged. But on Stackblitz, using Codeflow, when I refresh, all my changes are lost.
Is this expected behavior?
Link to the blitz that caused the error
Private repo, sorry
Steps to reproduce

Make some changes in a codeflow repo, and save them
Don't commit anything
Refresh

Expected behavior
All file changes should still be there.
Parity with Local

[ ] I have run the project in my local machine and I could not reproduce the issue.

I couldn't reproduce on VSCode...I can do this locally with stackblitz too if you think this is a bug.
Screenshots
No response
Platform

OS: macOS
Browser: Chrome

Additional context
No response

Hi @nandorojo! Thank you for raising this.
For now, the changes you introduce don't persist between sessions so you need to remember to commit your work often or before closing the browser.
This is, however, one of the features that we are prioritizing and that's on our product roadmap so stay tuned for the feature announcement 💙


new enpoint for SmesherService.SmeshingStatus

adding new endpoint for getting info about requested epochs
original issue https://github.com/spacemeshos/go-spacemesh/issues/3281
example response
type SmeshingStatusResponse struct {
  status: 'not_eligible',
  epoch: uint64;
  estimatedTime: number;
  reason: string;
  processedBytes: string // like "Processed 176 GB of 256 GB"
  processedPercentage: string // like "86% completed"
}

is there an open pr? i don't quite understand some fields.
what is status: not eligible?  you are always eligible to smesh

is there an open pr? i don't quite understand some fields.
what is status: not eligible? you are always eligible to smesh

related pr
https://github.com/spacemeshos/go-spacemesh/pull/3314
not eligible is related to specific epoch X. if you not build file and not submit proff before epoch start - you not eligible build block and get reward for started epoch X


Remove BackUp CR

Describe the bug
Backup CR needs to be removed as the namespace for backup will be placed in a different namespace
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Actual behavior
A clear and concise description of what actually happens.

@AsfaMumtaz why do you want to put backup in different namespace?


Is it possible to use the namespace optimizer on its own with standard .net fx xml attributes

Hi,
I have types with standard xml attributes defining the namespaces and attributes and would like to get them outputted in the result document. I can't find a way to force your library to use those attributes. Is that possible somehow?
Thanks

Hi @alexf2017 thank you for writing in and for your question.  If the document is being read into memory to instantiate a CLR object graph, when it is written out as a resulting XML document all attributes will be preserved accordingly w/ ExtendedXmlSerializer's own devices.  I am not sure if there is a way to preserve 1:1 of the input document's namespaces.
Note that ExtendedXmlSerializer's primary use case is the above.  That is, it is primarily designed to create and persist .NET object graphs.  Anything else is "maybe" supported, but there are limitations:
https://github.com/ExtendedXmlSerializer/home#known-issues
https://github.com/ExtendedXmlSerializer/home/wiki/FAQs#systemxmlserializer-vs-extendedxmlserializer
Please do give those a read and if you still feel this is something that can be solved with ExtendedXmlSerializer I can try to help further assist you.  I may need more code/reference to do so, however. :)

Going through older questions and it appears this one has been answered.  Please do let me know if you have any further comments/questions and I will further assist you. 👍


npm install -g doesn't work as expected

I'm trying to install all my dependencies in my package.json globally, but when I do, npm doesn't seem to resolve them globally.
My package.json looks like this https://gist.github.com/1272550#file_package.json and when I initially run npm ls, I get this https://gist.github.com/1272550#file_npm+ls .  When I run npm install -g, I get this https://gist.github.com/1272550#file_npm+install+_g (occluding the bcrypt build logic), however npm ls still shows all the same unmet dependencies as before.
I don't get it -- did the install work correctly or not?

Installing stuff globally doesn't work that way. Global install is for command line programs only, not for your programs dependencies. It's not only something you shouldn't do, it's something that just doesn't work. @isaacs, why is it even possible to do npm install -g?

@thejh yeah that should probably be changed to either install the currently referenced directory, or throw an error.
Also is there any way to call a specific app on the command line without making it global?
like npm run ender or some other equivalent?

@limeblack That would still need an installation of "ender" that's in some way global, right? Also, I don't see what benefit this could have.

@thjh Yeah you would have to install the node_module ender prior to running the command.
The benefit is to prevent global pollution and avoid conflicting names.  For example there is no way to call the npm module touch because it would override the unix touch program.  Considering so many bash scripts depend on touch, I would definitely not want to override it :D but I guess you could

@limeblack something like this?
#!/bin/bash
node_modules/$1/bin/$1 <some construct for passing on arguments>

@thejh Yeah that is the basic concept

Alright, I guess I assumed npm was similar to Ruby's gems and/or Bundler, allowing there to be some "system"-wide libraries and some project-specific libraries.  Thanks for clearing this up...

@nanodeath
Yes, it does allow there to be some system-wide libraries, and some project-specific libraries, but not in a way similar to ruby's gem/bundler approach.  Please read npm faq and npm help folders.
There is never a need to install your module's dependencies globally.  You may do this, but you never need to.  To execute bins, just add scripts to the package.json scripts hash.  npm will put ./node_modules/.bin as the first entry in the PATH environment when those scripts are run, so you'll always get your dependencies' bins before anything else.
For example:
{ "scripts": { "test": "vows test/*.js", "preinstall": "cake build" },
  "devDependencies": { "vows": "1.2.3"}
  "dependencies": { "coffee-script": "1.2.4" } }

That would run cake build at install time, using the "cake" executable provided by coffee-script 1.2.4, and vows test/*.js when you run npm test.
(or you could install them globally, and do npm link coffee-script and npm link vows to pull them in locally, it's up to you.)


Node label change in Jenkinsfile

Change node label linux_test to mesh-test

@kimlep01 Are you aware of these label changes?


Base the redis queue on Bull or Bee-Queue instead?

https://github.com/OptimalBits/bull
https://github.com/bee-queue/bee-queue

Chose Bee-Queue for now.


Support SSO for portal login

Please up-vote if you want this feature. Do not submit a new feature request.

This is a duplicate of #7, potentially one should be closed.

This is a duplicate of #7, potentially one should be closed.


Loading flash at address 0x0ABCD000 causes example to hang

Applying this patch causes the example to hang:
diff --git a/kernel.c b/kernel.c
index be748fd..07da912 100644
--- a/kernel.c
+++ b/kernel.c
@@ -18,7 +18,7 @@ int start() {
     // (0 - 0x2000) are reserved by spike and the third page contains the
     // host-target interface symbols defined in the kernel (0x2000, 0x2008).
     uint32_t sram_addr = 0x12340000;
-    uint32_t flash_addr = 0x81234000;
+    uint32_t flash_addr = 0x0ABCD000;

     const void* elf_data = cm; // App to load.

It may just be that this causes two memory regions to overlap but by my calculations there is enough space.

The flash address must have the MSBit set (i.e. (flash_addr & SENTINEL) != 0). I'll make that clearer in the documentation and code comments, and add an assertion. It's properly documented in tock.c, but it's not obvious if you don't look there or aren't familiar with the Tock loader. Thanks!


Configuring the Parts Unlimited team project - Which Organization do I select? No Permissions

Hi,
I am trying to go through the Parts Unlimited Demo but every time I try to make a template I get a permissions error. Which organization do I need to select?

Hey @TedB-MSFT.
You need to select an organization where you have permission to create a new project.

I have the same issue and do not have access to any organization.  How do I get access to one of the organizations?

All around organizations: https://docs.microsoft.com/en-us/azure/devops/organizations/accounts/organization-management?view=azure-devops
Hope this helps.


Swift Compiler Error: Editor place holder in source file

I'm facing a Swift Compiler Error in my app, I am using xcode 11.4. My App used to build fine before now I'm completely lost, I don't know what exactly happened. I've been looking for a fix for the last 3 days with no success, even tried to run it under previous xcode versions.
Any help will be much appreciated.
I get the following compile error:

Hi @Samoun15
Please provide your flutter doctor -v
and flutter run --verbose?
Also, to better address the issue, would be helpful
if you could post a minimal code sample to reproduce the problem
Please don't post screenshots of text where is not necessary, post the text instead.
Screenshots are not searchable and it's not possible to copy-past text out of it.
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so it's always better to open new issues and reference related issues.


The items parameter is no longer supported

The line parameter has been changed to the lineItems in the redirectToCheckout.
We need to rewrite it to the lineItems.
https://stripe.com/docs/payments/checkout/migration#client-products

Cool! Thanks for the heads up @ryoju-ohata I will keep this issue open. Are you planning to do a PR? Just let me know.

Thanks @jofftiquez
I'm not familiar with this code, so I'm not planning to do a PR.

I understand. Thanks. Will add this to my to-do list.


请问下载模型一直报签名错误是什么情况？

modelscope.hub.errors.FileIntegrityError: File /home/apple/.cache/modelscope/hub/._____temp/qwen/Qwen2-VL-7B-Instruct-GPTQ-Int4/model-00002-of-00002.safetensors integrity check failed, expected sha256 signature is e41d9af09037c50f37b14d9f20b04f984ebde3259545641aa8206b8c21361b20, actual is ec708eee6fed25eb9924486d990f9807d132d0c12bb240bb032042b898ee1199, the download may be incomplete, please try again.
请问如何解决呢？

modelscope.hub.errors.FileIntegrityError: File /home/apple/.cache/modelscope/hub/._____temp/qwen/Qwen2-VL-7B-Instruct-GPTQ-Int4/model-00002-of-00002.safetensors integrity check failed, expected sha256 signature is e41d9af09037c50f37b14d9f20b04f984ebde3259545641aa8206b8c21361b20, actual is ec708eee6fed25eb9924486d990f9807d132d0c12bb240bb032042b898ee1199, the download may be incomplete, please try again.
请问如何解决呢？

please refer to discussion on modelscope github: https://github.com/modelscope/modelscope/issues/968