chore: consume master of sonar cloud to fix failing PR builds

Issue
No Issue
Description
Consume master of sonar cloud to fix failing PR builds.
This is also in alignment with open-ux-tools https://github.com/SAP/open-ux-tools
Checklist for Pull Requests

[x] Supplied as many details as possible on this change
[x] Included the link to the associated issue in the Issue section above
[x] The code conforms to the general development guidelines.
[x] The code has unit tests where applicable and is easily unit-testable
[x] This branch is appropriately named for the associated issue

Why not use the latest version v2.1.1 rather than master which could break again at anytime?

Why not use the latest version v2.1.1 rather than master which could break again at anytime?

Thanks @donal-tobin-sap, I see we also use master in open-ux-tools and hence decided to use the same here https://github.com/SAP/open-ux-tools/blob/main/.github/workflows/pipeline.yml#L71
If you suggest using v2.1. then I will also do the same change in open-ux-tools

Why not use the latest version v2.1.1 rather than master which could break again at anytime?

Thanks @donal-tobin-sap, I see we also use master in open-ux-tools and hence decided to use the same here https://github.com/SAP/open-ux-tools/blob/main/.github/workflows/pipeline.yml#L71
But I think any version could break anytime, like it has happened now.
If you suggest using v2.1. then I will also do the same change in open-ux-tools

Yes, open-ux-tools I would have expected to use a recent version as well.
The use of a versioned tag is a high likely hood that there will not be a change from build to build.
Using the master branch means that a new merge can break the build at anytime (or worse).
Highest reliance is achieved by using a tag and then letting renovate fill in the actual SHA that officially matches the tag.


Temporary fix for  regex pattern error

Temporary fix for  regex pattern (\W['"]?t['"]?: ?'"['"]) had zero matches

Issue seems to be fixed already -- please re-open if this isn't the case :)


[Task] Change Mode Text Box based on Page Status

Task test
1.) Go to machine edit page
2.) Click add state button
3.) Verify that explanatory text about adding a state appears


[bug] Poll CC Alarm parameter not implemented

Before submitting a bug please read: https://zwave-js.github.io/zwavejs2mqtt/#/troubleshooting/bug_report
Version
Build/Run method

[ ] Docker
[ ] PKG
[x] Manually built (git clone - npm install - npm run build )

zwavejs2mqtt version: 1.0.1.e227f5b
zwavejs version: 6.1.0
Describe the bug
Perhaps not a bug, and more of a "to be implemented", but I have a water leak detector that sends Alarm/Notification CC.  As receiving a one time notification is not always reliable, I would like to poll for its status.
G/W is in Configured Manually/Just Value mode.
Used the GUI to create entry in the G/W table for a Water Leak Detector for parameter Sensor status (113-0-Water Alarm-Sensor status) with the Poll switch set to the "on" position.
I get the following log (note - Node 9 is the water leak detector):
2021-01-30 19:35:18.781 ERROR ZWAVE: Error while polling value 9-113-0-Water Alarm-Sensor status: The pollValue API is not implemented for CC Notification!

Notification CC operates in two modes: "push" and "poll". "push" nodes do reply to polling, so even after this is implemented, it is not guaranteed that it will work for you.


What are the details of using the visual grouding tool?

Hi, I noticed in your README that you would first extract the nouns in the text and then perform visual grounding.
My understanding is to use the nouns in the text as a query and get the corresponding image area by visual grounding.
However, I have some questions about this: (1) if there is more than one noun, is each noun a separate query or all the nouns are one query; (2) in the relational extraction, I found that there are some data with the same text and the same image, only the head entity or the tail entity is different, but the image region is different for these data, according to what you have done in the README, these data have the same text and image, it should be able to get the same image area.
Anyway, I would like to know the exact details of the visual grounding you performed to solve my two doubts above, thanks a lot, your work is great.

Hi,
Thanks for your attention! Your question may be answered as follows: :
(1) If there is more than one noun, each noun will be queried separately.
(2) During the extraction process, we will consider the head entity nouns and tail entity nouns in the text. Specifically, we will utilize these entity nouns to extract image regions in visual grounding tools, resulting in distinct image regions for the data with the same text and the same image but different head entity or tail entity.

Thank you!


stopped working with conda 4.13.0

I just updated conda to 4.13.0 and now conda.el no longer works. It appears to fail because the "conda ..activate" command has been removed.
I tried using the version from the fix/deprecated-conda-activate-command branch. After fixing up my startup PATH variable on macos to get conda--get-installed-version to work, I found that conda--supports-json-activator was still returning nil. I believe the version check has an error:
(defun conda--version>= (v1 v2)
  "Is version vector V1 greater than or equal to V2?"
  (cl-loop for x across v1
           for y across v2
           if (> x y) return nil
           finally return (>= x y)))

That should be "return t" in the second to last line.
After fixing that, it now errors in conda--get-deactivation-parameters because "conda shell.posix+json deactivate %s" does not accept arguments. Removing the %s and the env-dir argument from the format gets that to work.
Finally, the setenv in conda--update-env-from-params fails because the results of json-read-from-string can be symbols or integers, and setenv requires both arguments to be strings. Wrapping the setenv arguments in (format "%s" ...) to coerce them to strings seems to fix that.

Will be closed by #23


undefined symbol: _ZN2at19NonVariableTypeMode10is_enabledEv

If you have torch=1.1.0 and you install torchaudio>v0.2.0, you get
NameError: name 'Optional' is not defined

which means you have to upgrade torch>1.1.0 because Optional was added to jit after 1.1.0.
After you install an upgraded torch, and you finish installation of torchaudio it will not overwrite _torch_sox.cpython-36m-x86_64-linux-gnu.so. Thus it will not do the full compilation with the new torch. The indication of a full compilation is seeing building '_torch_sox' extension or gcc/g++ during python setup.py install/develop.
When you try to import torchaudio, you get
ImportError: ...audio/_torch_sox.cpython-36m-x86_64-linux-gnu.so:
undefined symbol: _ZN2at19NonVariableTypeMode10is_enabledEv

The solution is to run python setup.py clean --all and then re-compiling torchaudio with the new torch. This removes the build/dist which forces a full compilation that overwrites the _torch_sox.cpython-...-gnu.so with the correct file. Someone suggested that this does not always work so it would be nice if someone posts their setup.

I've run into this error myself :)

Another case where you can end up this situation, is if you accidentally have both torch and torch_nightly installed. AFAICT, torch is preferentially picked up.


Remove duplicate class names and spaces on sort

Awesome work on this plugin! The only thing Headwind is doing that's currently missing is also removing duplicate class names and extra spaces. Would be a nice addition.
// before

// after formatting

// expected behavior

 // <-- trim

I did the idea of removing duplicate classes! 👍
However, Prettier should already take care of extra spaces. I just gave this a try:

Hmm, just realizing that while it does it for the class attribute, it doesn't do it for :class:

<header :class="` bg-grey-200 md:flex         hidden        `">

<header :class="` hidden bg-grey-200         md:flex        `">

So, I suspect it probably also won't work with some other variations like className and ngClass (although I haven't tested these).

@reinink I can confirm it doesn't remove extra spaces for className

I did the idea of removing duplicate classes! 👍
will this feature be implemented in next releases?

It doesn't remove the spaces in svelte with class.

:class can include more than just the classes. It can have methods etc, even inside the special kind of ticks. Maybe that's why it does not remove duplicate spaces, because it may have unwanted side effects.

Not only spaces but line breaks also should be removed


Product variant optional attributes

What I'm trying to achieve
Right now, if you want to create new product variant of X product type which have 3 attributes, for example:
color
size
something
you can't leave any variant attribute value empty.
Describe a proposed solution
I think it's kinda useless, we implemented easy fix for that:
AttributeAssignmentMixin
func _validate_input we removed:
        if is_variant:
            return cls._check_input_for_variant(cleaned_input, attribute_qs)
        else:
            return cls._check_input_for_product(cleaned_input, attribute_qs)

and left just
     return cls._check_input_for_product(cleaned_input, attribute_qs)

and it works as charm, but it would be great to remove all is_variant useless logic then.
About frontend: the only change, is to make same thing with variant attributes as with product attributes, so that user can change value to None and then empty list is sent by graphql. (empty list, not None value, that's important tho. But it's just same as with product attribute update)
So basically with removing extra code, and unifying product+variant attributes we can achieve better functionality.
Other solutions I've tried and won't work
Screenshots or mockups
https://i.imgur.com/2Iz759C.png
brak == none, just translation

also - proposed solution above, checks if attribute is required, so it also works with required / non required attributes.

Thanks for the feedback! I'm gathering all issues related to attributes and we'll try to address them when we'll be reiterating on attributes system implementation (which I hope is going to happen before the release of next version).


Use case-insensitive checks for Endpoint name

Closes https://github.com/dotnet/aspire/issues/2521
Microsoft Reviewers: Open in CodeFlow

Should we add tests?

I'd like to see this change centralized in EndpointReference,. Change all of the callers to use EndpointReference and make the fix in there.

I'm taking over this change. Closing for now.


Backends expect string as names in the JSON qobj

@ajavadia mentioned that, when non-string are used as identifiers (see #308), backends do not validate the JSON input.
import qiskit
from pprint import pprint
qp = qiskit.QuantumProgram()
qr = qp.create_quantum_register(0, 2)
cr = qp.create_classical_register("", 2)
qc = qp.create_circuit(0, [qr], [cr])
qc.h(qr[0])
qc.measure(qr[0], cr[0])
qp.add_circuit(10, qc)
circuits = [10]
backend = 'local_qiskit_simulator'  # the backend to run on
qobj = qp.compile(circuits, backend=backend, shots=1024)
result = qp.execute(circuits, backend=backend, shots=1024)
pprint(qobj)
pprint(result._result)

results in:
{'status': 'ERROR: Failed to execute qobj (unable to parse qobj, type must be string, but is number)', 'success': False}
This PR converts Quantum and Classical registers names into strings. Tests added.

Thanks! 2 of the tests were failing online, even though local everything was fine. I made those tests be skipped when the cpp simulator is not available.
But this right now only converts qreg and creg names to strings. Still there could be circuit names that are non-strings (in particular, the example you provide in this PR fails!)
I'm now actually thinking maybe the c++ code has to change to accept non-string identifiers. The generated json is valid json (the non string is in values, not keys).

@1ucian0

After talking with @ajavadia, I realize that makes more sense to only allow strings (and None, for auto generate) ids, rolling back #308 and the main point in #290. For me, the convincing reason is that should be possible to build back a QObj object from the JSON representation of a QObj. Supporting non-strings means that the QObj specification should also save the original type in which the name is represented (the name "1" has not enough info to reconstruct the original name). The auto generated name still makes sense. The current state of the PR represents this.

This PR closes #420

While I agree that an idempotent serialization routine would be ideal, I find it hard to stomach that this would dictate requirements for internal names. But I did see some work in removing the requirement of naming registers ourselves, so I think that may be even better. (Why does a user have to come up with a name in the first place, right?)

Thanks, @1ucian0 ! :tada:


Rework topics Notifications

As ...
I want ...
so that ...
Hints / Details
Acceptance Criteria

[ ] In case a notification can not be successfully sent to at least 1 other instance the backend needs to return a http error code to the frontend.
[ ] Frontend displays a loading spinner as long as the notification request takes (no full blocker loading blocker)

Out of Scope

...

https://jira.catena-x.net/browse/TRACEFOSS-3001


Apache Header not found after updating to appcompat v23

When I updated to 'com.android.support:appcompat-v7:23.0.0'
class org.apache.http.Header is not found
the only solution was to change it back to 'com.android.support:appcompat-v7:22.2.1'

Duplicate of #830


relative urls in rss

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have searched for relatived questions
[X] I have read documentations

Describe the issue
There are relative urls in the rss feed, but should be absolut?
https://blog.natenom.com/index.xml
<item>
<title>Falsch gedrehte Bilder in Hugo durch Verwendung von Funktionen zur Bildbearbeitung</title>
<link>/2022/09/falsch-gedrehte-bilder-in-hugo-durch-verwendung-von-funktionen-zur-bildbearbeitung/</link>
<pubDate>Sun, 11 Sep 2022 20:11:13 +0200</pubDate>
<guid>/2022/09/falsch-gedrehte-bilder-in-hugo-durch-verwendung-von-funktionen-zur-bildbearbeitung/</guid>

My website builds on the server with the same hugo extended version as on my local pc.
There the link field looks like this:
<link>//localhost:1313/2022/09/falsch-gedrehte-bilder-in-hugo-durch-verwendung-von-funktionen-zur-bildbearbeitung/</link>

Steps to reproduce
Build.
Expected behavior
Absolute urls.
What operating system(s) are you seeing the problem on?
Linux
What browser(s) are you seeing the problem on?
Chrome, Firefox
What version of theme are you using?
afc9eb4de40683cb22d8d28c2a4edd166356e5de (HEAD -> master, origin/master, origin/HEAD) chore: print the fuse.js options to console
What version of Hugo are you using?
hugo v0.103.0+extended linux/amd64 BuildDate=2022-09-16T20:37:28Z VendorInfo=debian:0.103.0-1
What version of Node.js are you using?
v18.7.0
What version of Git are you using?
git version 2.35.1
Additional context
No response

I guess you haven't specified the baseURL on production.

It was specified, the first line in my config and I didn't change .... oh, I did change it by accident somehow, it was
GbaseURL = "https://blog.natenom.com/"
Thank you :)
I checked that today but missed the G.

Np.


Step 3: convert browserify build to webpack, fixes #976

the vast amount the added lines to this PR are via the yarn.lock file, FYI! this PR is mostly a 120 line webpack config file and some tooling and configuration changes.

[x] browserify min and graphiql bundle replaced with webpack 3, retaining window.GraphiQL interface pattern
[x] windows compatible tooling: replace bash scripts for babel, browserify, uglify, postcss/autoprefixer, etc 'build' turned into build 20s and build-bundle - 40s
[x] use new webpack mincss plugin, and generate a nanocss build at graphiql.min.css too! (note: temporary til styled components)
[x] update yarn dev server to mount webpack dev server with test GraphQL server
[x] fix some issues with cypress + jest battling over the linting of expect() in cypress specs that was failing lint
[x] HUGE reduction in minified build size. I'm not kidding.

(parsed size, from webpack output)

[x] - For some reason cypress run is bailing on an obscure error, but jest is passing, and you'll see a working build
[x] - Also to note, netlify for some reason stopped building deploy previews at #999 (netlify build # not GH)
[x] - Lastly, i keep seeing this error from codemirror info addon: ReferenceError: can't access lexical declaration onHover' before initialization`

here is proof that the file utility from the first PR in this series is working:

oof, in order to claim windows interoperability, we still need to add cross-env wherever environment variables are being set!

this is failing because of https://github.com/markdown-it/markdown-it/issues/616, which breaks the webpack build

oh yikes! thanks for reporting this. ill have to see what happened and fix
it immediately
On Mon, Dec 9, 2019 at 1:46 PM Casey Foster<EMAIL_ADDRESS>wrote:

@caseywebdev commented on this pull request.
In packages/codemirror-graphql/.npmignore
https://github.com/graphql/graphiql/pull/1001#discussion_r355618948:

@@ -0,0 +1,4 @@
+babel.config.js
+.gitignore
+resources

Sorry, I got resources and results confused so this isn't the direct
issue. Regardless, there's still an issue with the latest
codemirror-graphql package because the results folder, referenced in the
line above, does not exist.
~ npm install<EMAIL_ADDRESS>ls node_modules/codemirror-graphql/results
ls: node_modules/codemirror-graphql/results: No such file or directory
This broken link is causing my build to fail, so I've resorted back to
<EMAIL_ADDRESS>and manually pinning<EMAIL_ADDRESS>(which
does publish this folder) for now.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/graphql/graphiql/pull/1001?email_source=notifications&email_token=AAKOFF2YWIUCAVC6KMZOBPTQX2G65A5CNFSM4JOS5R2KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCOPD3BQ#discussion_r355618948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKOFF3NESQO2WQCCOGX4ZLQX2G65ANCNFSM4JOS5R2A
.


pull request for e2e test at 01/22/2017 12:26:15

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


CPP: fix static objects initialization for ARC MWDT toolchain

The constructors of static objects are stored in ".ctors" section. In case of MWDT toolchain we have incompatible ".ctors" section format with GNU toolchain. So let's use initialization code provided by MWDT instead of Zephyr one in case of building with MWDT toolchain.
NOTE: As it is done for GNU toolchain, we call constructors of static objects but we don't call destructors for them.

In general this looks like it should work. There are still some "TODO: add mwdt specific ..." in the linker.ld: are these still required, or are these now resolved?
Somewhat related to that, any reason to use #ifdef's instead of linker snippets? It seems the linker snippets are intended for customizations like this.

There are still some "TODO: add mwdt specific ..." in the linker.ld: are these still required, or are these now resolved?

@ruuddw - yep, they are still required - it's related to C++ exception handling - I'll fix that when I'll introduce exception support for Zephyr in MWDT.


Add 'random' and 'hash' pre-defined partitioners for produce APIs

Pull Request Test Coverage Report for Build 916

9 of 17 (52.94%)  changed or added relevant lines in 2 files are covered.
8 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.4%) to 78.096%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/brod.erl
8
12
66.67%

src/brod_utils.erl
1
5
20.0%

Files with Coverage Reduction
New Missed Lines
%

src/brod.erl
2
56.14%

src/brod_group_coordinator.erl
2
87.65%

src/brod_group_subscriber.erl
4
77.21%

Totals

Change from base Build 915:
-0.4%

Covered Lines:
1961

Relevant Lines:
2511

💛  - Coveralls


generate controllers and views

For this milestone, I just created a basic controllers and views.

Hi  @Gardimy  👋 👋,
This PR is completed! There is nothing else to say other than... it's time to merge it 🚀👨🏼🚀
Congratulations! 🎉
STATUS: APPROVED ✔✔

🎯 TOP HIGHLIGHTS 🎯

Cleanly written code ✅
Working as expected ✅


Redshift Materialised views not refreshing through cursor.execute

Describe the bug
I've an Athena table mapped to csv files in S3 that I can successfully upload new files to, and run the Athena repair on:
   wr.s3.to_csv(df,target  ,
                    sep='\t',
                 index=False)
    query_final_state = wr.athena.repair_table(table=tableName, database=database)

I've also got some downstream Redshift Materialised Views that depend on this Athena table (mapped with Spectrum). I can run SQL directly to update them, but trying to run the same statement through aws-sdk-pandas doesn't update the view

def refreshRedshiftMatView(secret, matView):

    print(f"Refreshing Redshift ")
    con = wr.redshift.connect(secret_id=secret)
    try:

        with con.cursor() as cursor:
            cursor.execute(f"REFRESH MATERIALIZED VIEW {matView}")

    except Exception as e:
        print(f"Error refreshing {matView}: {e}")
    finally:
        con.close()
    print(f'Refreshed {matView}')

I've tried editing the credentials in the Secret and the view name, to make sure that the code wasn't just failing silently - it throws an error both times, so I'm confident that it's finding the MatView in question, just not doing anything with it
Appreciate any pointers to how this should be done properly, if I'm missing something
How to Reproduce
Create a Materialised View in Redshift, derived from an Athena/Glue table, and map to Redshift via Spectrum
Update the underlying data
Run the following method to attempt to refresh the view

def refreshRedshiftMatView(secret, matView):

    print(f"Refreshing Redshift ")
    con = wr.redshift.connect(secret_id=secret)
    try:

        with con.cursor() as cursor:
            cursor.execute(f"REFRESH MATERIALIZED VIEW {matView}")

    except Exception as e:
        print(f"Error refreshing {matView}: {e}")
    finally:
        con.close()
    print(f'Refreshed {matView}')

Expected behavior
The matview should be updated, as if the following SQL was run
REFRESH MATERIALIZED VIEW myMatView
Your project
No response
Screenshots
No response
OS
Mac
Python version
3.9
AWS SDK for pandas version
3.2.1
Additional context
Same behaviour locally on a Mac and in a container running in Batch

The wr.redshift.connect API simply returns a redshift-connector  connection. So let me transfer this to the redshift-python-driver team to see if they have more insights.

Hi folks, I maintain redshift-connector. Wondering if you're executing a commit or have autocommit enabled. Could you give that a try and let me know what you see? Thanks! :)

yes, that worked great. I'll close the issue, thanks

Hi - just reopened in order to explain the issue in a little more detail
I've checked the docs, and the to_sql command has the commit_transaction parameter. I used this originally for the REFRESH MATERIALISED VIEW command, but it threw an error as no records were returned
https://aws-sdk-pandas.readthedocs.io/en/3.2.1/stubs/awswrangler.redshift.to_sql.html
I've switched to the cursor side of things, and I understand you're not trying to build an all-purpose data access library, but that's the hole I briefly found myself in ...

In the to_sql API, the commit_transaction is set to True by default, meaning that we will commit the INSERT INTO transaction for you. However any other SQL operation beyond that is the user's responsibility.
You either have to set commit_transaction=False and group together all transactions to commit them at once or leave it True and commit subsequent ones separately.


Possible infinite loop in profile_update hook

Description
When logging in with the auth0 plugin I expect to be able to login, however we've noticed since updating to Auth0 v4, that some users are getting out of memory issues when logging in. It seems that for some users an infinite loops triggers that causes PHP to crash eventually.
Reproduction
I don't know the full details to reproduce yet, however I seem to have fixed it for us by commenting out a bit of code.
It seems that the function wp_auth0_profile_change_email listens to the profile_update hook. Which in turn call the method update_email in the class WP_Auth0_Profile_Change_Email.
This method calls remove_action('profile_update') explicitly to stop infinite loops, however that does not seem to work. I think this might be because there are now 2 functions listening for profile_update, the method in this class listens to it after being called for the first time and the function wp_auth0_profile_change_email I mentioned before.
// Remove this method from profile_update, which is called by wp_update_user, to avoid an infinite loop.
remove_action( 'profile_update', [ $this, __FUNCTION__ ], 100 );

I think this might be a refactoring bug, before this class registered it's own profile_update hook but in v4 that wp_auth0_profile_change_email function was added. The remove_action that should stop infinite loops does not deregister the new wp_auth0_profile_change_email function that calls WP_Auth0_Profile_Change_Email, only it's own function, which I think is wrong.
For now I completely commented out the wp_auth0_profile_change_email function and do an early return in WP_Auth0_Profile_Change_Email::update_email to make sure it doesn't run.
(partial) STACKTRACE
wp-content/plugins/auth0/lib/profile/WP_Auth0_Profile_Change_Email.php:94
wp-content/plugins/auth0/WP_Auth0.php:441
wp-includes/class-wp-hook.php:287
wp-includes/class-wp-hook.php:311
wp-includes/plugin.php:478
wp-includes/user.php:1876
wp-includes/user.php:1995
wp-content/plugins/auth0/lib/profile/WP_Auth0_Profile_Change_Email.php:94
Environment
Wordpress: 5.4.2
Login by Auth0: 4.1.0

This issue is also tracked in a support ticket (SF 00463569). Just FYI.

Hi @joshcanhelp do you think the provided information is sufficient to work on the issue?

@Zae - Apologies for the long delay here. I think you're spot-on with the root cause here and we'll get that fixed and released as soon as possible.


v1alpha2

Per discussions in the working group, this first exemplar CRD will be v1alpha2 because of its distance from the specification as originally conceived.
NOTE: in the future versions will be added, not replaced, in the CRD.  This was a special case where I'd created something called v1alpha1 that was not that and therefore had to be moved, not added to.
[resolves #69]

As a follow-up suggestion, I believe we can also make a release with the version update :) Thoughts @nebhale ?

@Avni-Sharma Once RC2 is officially ratified in the next meeting we'll tag it.


Add Python 3.11 to CI

Separated from the PR bringing 3.11 support to marshalparser because we have to wait till Python 3.11 is in Fedora to test it in the CI.

I see ERROR:  py311: InterpreterNotFound: python3.11 on many architectures :(

You might be able to fix it :wink: https://github.com/fedora-python/fedora-python-tox/pull/53

Oh, I have actually mixed this PR with the other one -- never mind the comment.

All green!


fix: child process proxy

Codecov Report

Merging #98 (b851e42) into master (6eb893a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #98   +/-   ##
=======================================
  Coverage   82.45%   82.46%
=======================================
  Files          34       34
  Lines        2673     2674    +1
  Branches      751      751
=======================================
+ Hits         2204     2205    +1
  Misses        427      427
  Partials       42       42

Impacted Files
Coverage Δ

packages/dev-pack/src/common.ts
90.00% <100.00%> (+0.16%)
: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 6eb893a...b851e42. Read the comment docs.


Update macros.html

Its a Zola theme, I fixed the footer for you to reflect this. (I know you originally ported it from jekyll and probably just missed this.)

I also just added a commit for screenshot.png which is a requirement for this theme to be included here: https://www.getzola.org/themes/

no problem, its a good looking theme!

I updated the pull request to have the new footer text you suggested.

Let me know if you see anything else you want updated? Otherwise I think it is ready for merge.

no problem, its a good looking theme!

yes, very effective theme indeed.
Thank you for your contribution. Very appreciated.


Feat/create model

PR Checklist

[ ] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added or modified a feature, documentation in docs is updated

Description of changes
create registered model from the ui

@creechy I'm not sure why the Scim2User.java formatting change made it into this PR 🤦♀

@creechy I'm not sure why the Scim2User.java formatting change made it into this PR 🤦♀

that looks like previously bad formatted code got auto-formatted


make compile.sh executable

fix up permissions on the script so that users can just run it

Nice!


Eliminate some allocations in client_channel code.

This is the part of #11298 that isn't blocked on combiner improvements.

No significant performance differences

No significant performance differences

This also includes some code cleanups.

I'll review this soon. I'd however like it if #11159 goes in first in order to prevent conflicts in an already very large PR.

No problem -- I'm fine waiting on the other PR.

No significant performance differences

No significant performance differences

No significant performance differences

I had actually already merged in the LB update change, but there was another one that was merged into master right before I pushed the changes to this PR.  I'll update again and see what the tests say.

No significant performance differences

Looks like there's an assertion failure triggering in the Windows Node tests.  I don't understand how it's happening, though, since I can't find a code-path that would cause that.
@murgatroid99, is there a way for me to run those tests such that the output includes the C-core logs?  If so, I can add some debug logging to try to figure out how this is happening.

No significant performance differences

No significant performance differences

No significant performance differences

[microbenchmarks] No significant performance differences

[microbenchmarks] Performance differences noted:
Benchmark                                                     locks_per_iteration
------------------------------------------------------------  ---------------------
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/0    +10%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/1    +6%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/2    +5%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/1/1    +6%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/1/2    +5%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/32k/1  +6%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/4k/1   +6%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/4k/2   +4%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/512/1  +6%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/512/2  +4%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/64/1   +6%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/64/2   +4%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/8/1    +6%
BM_StreamingPingPong<MinTCP, NoOpMutator, NoOpMutator>/8/2    +5%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/0/0       +10%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/0/1       +6%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/0/2       +5%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/1/1       +6%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/1/2       +5%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/32k/1     +6%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/4k/1      +6%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/4k/2      +4%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/512/1     +6%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/512/2     +4%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/64/1      +6%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/64/2      +4%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/8/1       +6%
BM_StreamingPingPong<TCP, NoOpMutator, NoOpMutator>/8/2       +5%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/0        +12%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/1        +12%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/256k     +7%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/32k      +11%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/4k       +11%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/512      +11%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/64       +11%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/0/8        +12%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/1/0        +12%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/1/1        +12%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/256k/0     +7%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/256k/256k  +4%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/32k/0      +9%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/32k/32k    +9%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/4k/0       +11%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/4k/4k      +10%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/512/0      +11%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/512/512    +10%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/64/0       +11%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/64/64      +11%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/8/0        +12%
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/8/8        +11%
BM_UnaryPingPong<MinUDS, NoOpMutator, NoOpMutator>/0/0        +12%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/0           +12%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/1           +12%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/256k        +7%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/32k         +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/4k          +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/512         +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/64          +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/8           +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/1/0           +12%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/1/1           +12%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/256k/0        +7%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/256k/256k     +5%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/32k/0         +9%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/32k/32k       +9%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/4k/0          +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/4k/4k         +10%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/512/0         +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/512/512       +10%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/64/0          +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/64/64         +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/8/0           +11%
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/8/8           +11%
BM_UnaryPingPong<UDS, NoOpMutator, NoOpMutator>/0/0           +12%

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

This PR should be in good shape now.  David, PTAL.

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

Known issues: #11055 #11223


fixed indexing issues - updated apiVersion to 0.0.5

package.json
"dependencies": {
"@graphprotocol/graph-cli": "0.23.2",
"@graphprotocol/graph-ts": "0.23.1"
}

Subgraph is indexing correctly now, thanks for the fix!
For the record the error was :
Subgraph instance failed to run: failed to process trigger: block #20148414 (0x0000…ba7b), transaction 8e1e6dd06a4f2045f9eeed82f0863bca37e9333931dbc72bacde52995a8e3e81: Decoding function outputs for the call Function { name: "stake", inputs: [Param { name: "_amount", kind: Uint(256) }, Param { name: "_recipient", kind: Address }], outputs: [Param { name: "", kind: Bool }], constant: false, state_mutability: NonPayable } failed, raw output: : Invalid name please ensure the contract and method you're calling exist! failed to decode empty bytes. if you're using jsonrpc this is likely due to jsonrpc returning 0x in case contract or method don't exist, code: SubgraphSyncingFailure
The failure happened with failed transaction on Fantom with the hosted-service of The Graph.
Upgrading the subgraph.yml from v 0.0.4 to 0.0.5 has fixed it


Add support for use Trait::method

Rendered
This feature fully supplants https://github.com/rust-lang/rust/pull/73001, allowing you to do things like:
use Default::default;

struct S {
    a: HashMap<i32, i32>,
}

impl S {
    fn new() -> S {
        S {
            a: default()
        }
    }
}

and more.
This is my first RFC, please forgive any missteps I make in the process.
Partially completes #1995.

From The Rust Programming Language section 7.4:

Creating Idiomatic use Paths
Although both Listing 7-11 and 7-13 accomplish the same task, Listing 7-11 is the idiomatic way to bring a function into scope with use. Bringing the function’s parent module into scope with use means we have to specify the parent module when calling the function. Specifying the parent module when calling the function makes it clear that the function isn’t locally defined while still minimizing repetition of the full path.

We have been encouraged for a long time to always import the parent module when calling the function to distinguish locally defined functions and imported functions. However, the syntax sugar suggested in this RFC contradicts to this convention.
I think it is more appropriate to discuss this convention in the RFC to make it more clear how we should use this feature. :)

I'd like to reiterate here what I said on #1995
Many libraries define free functions for constructors to mathy types, like vectors.
Some examples:
https://docs.rs/glam/0.24.2/src/glam/f32/vec2.rs.html#12
https://docs.rs/egui/latest/egui/fn.pos2.html
It would be nice if instead of having to create wrapper functions, the user (or the lib author with pub use) could just do use Vec2::new as vec2;
This is some additional motivation that could go into the motivation section. Default::default is hardly the only use case for this.

That being said, I noticed that this RFC doesn't talk about importing inherent methods, which I think should be addressed.
If this RFC only proposes importing trait methods, but not inherent methods, then the use cases I mentioned above are nullified.
Why should we only support importing trait methods, but not inherent methods?

I discussed that in the future work section. use Type::method is out of scope for this RFC. The difficulty is impl blocks with arbitrary where clauses.

I'm going to go ahead and start the process of seeing if we have consensus on this:
@rfcbot merge
Please do either add support for associated constants inline in the RFC (if it's straightforward to do so) or mention them in future work (if not):
@rfcbot concern associated-constants

Apologies for the delay, wasn't able to work up motivation to work on this again until today. Thanks for the review!

This can be inferred directly from the desugaring described in the RFC. If the trait has generics, they must be inferrable in order for Trait::method() to work, hence the same is true for the imported version. I can spell that out explicitly if you want.

Just wanted to ping this, is there anything I can do to keep this moving?

Nominated for lang discussion.

This is currently implemented on nightly as part of fn_delegation (https://github.com/rust-lang/rust/issues/118212): https://play.rust-lang.org/?version=nightly&mode=debug&edition=2021&gist=fe2a0b28cf36b474400ae3faa10c5408
#![expect(incomplete_features)]
#![feature(fn_delegation)]

reuse Default::default;

fn type_name_of_val<T>(_: &T) -> &'static str {
    std::any::type_name::<T>()
}

fn main() {
    let x: u32 = default();
    let y = default::<u64>();
    let name = type_name_of_val(&default::<()>);
    dbg!(x, y, name);
}

@rfcbot reviewed

This RFC does not propose the ability to import Type::method where method is contained in an impl block. Such a feature would be a natural extension of this work, and would enable numeric features like that discussed in motivation without the need for the num_traits crate. This feature is not proposed in this RFC since initial investigations revealed that it would be difficult to implement in today's rustc.

I would very much like to see this, though I am okay waiting for a future RFC once the implementation challenges are worked out.

If we add a compatibility mechanism to implement a supertrait method when implementing its subtrait, without having to separately implement the supertrait (such that a new supertrait can be extracted from a trait without breaking compatibility), we would also need to lift the limitation on using a supertrait method via a subtrait.

I would also very much like to see this, but yes it belongs in another RFC. While I would prefer to lift the subtrait/supertrait restriction part of this RFC, it sounds annoying to do so it can wait for the next RFC.

The restriction on importing parent trait associated functions is a consequence of this desugaring, see https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=51bef9ba69ce1fc20248e987bf106bd4 for examples of the errors you get when you try to call parent trait associated functions through a child trait. We will likely want better error messages than this if a user tries to import a parent function.

Those errors are bizarre and I agree we should make them better (and as I said, eventually remove them altogether).

@obsgolem Would you add some text elaborating on how trait generics work, so we can get the last concern resolved?

I've updated the links in the RFC and created a tracking issue, so I'm going to merge this. Thanks you @obsgolem!


Update Moshi version

Description
This PR does multiple things:

Update Gradle and tools dependencies so we can use the latest Gradle APIs.
Update Retrofit and Moshi versions.
Fix a bug with a nullable field marked as non-nullable.
Fix a Lint error in AspectRatioImageView.

Tests
Tested in the example demo application and in our Medium app that the changes works as expected.

Let's fork it to move faster but on long term what about opening a pr on the main repo ?

@grosalex There is an 1 year old PR for that with no response from the maintainers: https://github.com/unsplash/unsplash-photopicker-android/pull/16, so the library is no longer maintained.


ninja: error: manifest 'build.ninja' still dirty after 100 tries

I tried to build fuchsia on my macOS 10.15beta, xcode11.0 beta2 following instructions provided  on https://fuchsia.dev/fuchsia-src/getting_started
However, I'm continuously meeting
ninja: error: manifest 'build.ninja' still dirty after 100 tries

How can I resolve this issue?
More log is given below:
(base) kylewong@KyleWongs-MacBook-Pro fuchsia % fx set core.x64 --release

WARNING: Please opt in or out of fx metrics collection.
You will receive this warning until an option is selected.
To check what data we collect, run `fx metrics`
To opt in or out, run `fx metrics <enable|disable>

Generating compile_commands took 463ms
Done. Made 28104 targets from 2274 files in 12455ms
(base) kylewong@KyleWongs-MacBook-Pro fuchsia %
(base) kylewong@KyleWongs-MacBook-Pro fuchsia % fx build
WARNING: Please opt in or out of fx metrics collection.
You will receive this warning until an option is selected.
  On Travis OSX CI, Ninja unnecessarily runs CMake right after cmake initial configuration, and runs 100 times.

I have this problem too.  No clue how to fix.  Buggy software.

It may be Mac OS X is buggy :)
But anyhow, this hurts us.  We temporarily switched to Make in CI.

I have the same problem on Centos 7

Encountered same error on Ubuntu 18.04 while building OpenJFX
ninja: error: manifest 'build.ninja' still dirty after 100 tries

One possible reason is your project missing files.
In my case of building chromium, the same problem happened, but running "gclient sync" to get everything updated solved the issue.
Whatever, it will be much better if the error messages of ninja are more informative.

I'm experiencing the same issue

i have the same problem

我也遇到这样的问题，在fedora31下编译initramfs/systemd的时候出现的。I have the same problem while compiling initramfs/systemd in fedora31.有同学解决这个问题吗？麻烦指教，谢谢！Anyone solved this? Please tell me, thank you ：）

i have the same problem

This bit me today on Linux.

The time was two months behind on the build system.  Fixing the system time fixed the build error.  Make sure your systems are synchronized.

Same on Manjaro, ninja version 1.10.2, cannot reproduce. Ninja generated with CMake

Try to sync you system time, then this maybe fixed.

@sutra
thank you works for me!!

I was using CLion on Linux, by removing the cmake-build folders and the .idea and reopening the project I was able to fix this

same problem on windows10

I recently ran into this using WSL and the cross-platform project option in Visual Studio.  Someone mentioned syncing the system time, and when I checked the system time on my Debian VM, it was way out of date, almost as if the system clock had stopped. Once I synced the system time on the VM, this was resolved.  Timestamps are of course not always a perfect proxy for the state of a file, which Ninja is relying on for some validation of file state.

I have the same issue, I have synchronized the system time. VCPKG fails to build.

I have the same problem with my flutter Project

In my case, my computer time was late, I just set it to the right time and tried again. then it worked

Try to sync you system time, then this maybe fixed.

sudo ntpdate pool.ntp.org

Recently encounted this issue on Windows 11 + WSL2 (Ubuntu).
Touched all the files in the source directory. After that it all worked.

I have this problem when building Poppler for installing KDE Plasma desktop on Gentoo. I'm sure that my system time is correct.

 * The complete build log is located at '/var/tmp/portage/dev-util/glslang-1.3.283.0/temp/build.log'.
 * The ebuild environment file is located at '/var/tmp/portage/dev-util/glslang-1.3.283.0/temp/environment'.
 * Working directory: '/var/tmp/portage/dev-util/glslang-1.3.283.0/work/glslang-vulkan-sdk-1.3.283.0_build-.arm64'
 * S: '/var/tmp/portage/dev-util/glslang-1.3.283.0/work/glslang-vulkan-sdk-1.3.283.0'


new URL everytime I update and publish site

steps to reproduce:
-publish site and get one url
-make changes and publish again and get a different url
I expect to get the same url every time I publish a site

posted this issue twice


Add expanded/compact widget toggle to app-header

Screenshots

![screen shot 2016-08-09 at 2 04 43 pm](https://cloud.githubusercontent.com/assets/5818702/17529767/49b2efc0-5e3a-11e6-806f-aa7be3926cbc.png)

Anyway to get rid of the scrolling things in the dropdown?

@timlevett It was taken care of in a merge to frame:
https://github.com/UW-Madison-DoIT/uw-frame/pull/266

Gotcha, then 👍 from me

👍


Fixed typo

Fixed typo, 'using' was written as 'usihg'

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

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

#sign-off

@anandyadavMSFT Please resolve the merge conflict so that the pull request can be merged. Thanks. #hold-off

Fixed in another commit after


cyclonedds: 0.10.4-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository cyclonedds to 0.10.4-1:

upstream repository: https://github.com/eclipse-cyclonedds/cyclonedds.git
release repository: https://github.com/ros2-gbp/cyclonedds-release.git
distro file: humble/distribution.yaml
bloom version: 0.11.2
previous version for package: 0.10.3-1

Holding for Humble sync


Image use in blog post

Hi there
Some unusual question here. I currently write a blogpost about kubernetes operator and would like to include this image  operator-capability-level-transparent-bg.png.
Is it ok if I use this image?
I will publish the article on https://blog.codecentric.de

That should be fine with us.
I'll defer to @dmesser and/or @robszumski on whether that diagram needs any more updates though.

@WtfJoke Feel free to use it, no issues there. Update this issue with a link to the post so we can check it out!

@WtfJoke Forgot to add, can you use a caption that links it back to the original? The capabilities get updated every so often and it would be great for folks to have a way to discover the latest.

I think that would be good to make the reference regards the source as well 👍

Hi everyone, thanks for your fast reply and allow me to use the image. 😃
I will update the link to the post here. It is still in process. Expect nothing fancy, the target audience are beginners.  Post will be in german, maybe I will also publish it in english.
@camilamacedo86 I would have put a link under the image, something like "Source: Operator-SDK Repository" It wasnt my intention to not link to the source.

Hi there,
Wanted to let you know the german blog post just got published:
https://blog.codecentric.de/2019/10/dein-erster-kubernetes-operator/
If you like to have attribution changed, let me know. :)
If I write an english version, will let you guys know here as well

Hi there,
Its me again.  👋
A german tech magazine https://t3n.de/ asked me to write an article about k8 operators.
This time its less about writing an operator by itself and more what is an operator and how they work.  The article is supposed to get printed first and later on published on their website.
I would like to use the image in the article. If its fine for that use too? Just wanted to make sure. If have no problem if thats an issue for you, then I wont use it.
I would give you credits by providing the source as in my blog post in format:
Source:  https://github.com/operator-framework/operator-sdk
Thanks a bunch! 🙏

@WtfJoke No issues! We'd love to see the article when its published :)


kotlin: add library request interfaces

https://github.com/lyft/envoy-mobile/issues/119
Adding a primitive Request and RequestBuilder with basic features:

URL
method
headers
trailers
retry policy

Tests are basic setting a builder and ensuring the result is expected. Additionally, the request build transform creates an equivalent object when built back.
Signed-off-by: Alan Chiu<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description: kotlin: add library request interfaces
Risk Level: low
Testing: unit
Docs Changes: n/a
Release Notes: n/a
[Optional Fixes #Issue]
[Optional Deprecated:]

thanks for the review @rebello95 @goaway


🔥 Loading .dylib has failed, This is likely because Swift code being injected refers to a function with a default argument. Rebuilding and re-running your project can resolve this.

Excuse me, can I only re-run if I encounter this problem?

🔥 Loading .dylib ...
🔥 Loading .dylib has failed, This is likely because Swift code being injected refers to a function with a default argument. Rebuilding and re-running your project can resolve this.
🔥 *** dlopen() error: dlopen(/Users/tate/Library/Developer/CoreSimulator/Devices/6BB1496B-CE17-4A53-A2E9-7A16CE440C6D/data/Containers/Data/Application/18B0CA15-7294-48C2-91E0-9A69C91FED5E/tmp/eval101.dylib, 2): Symbol not found: $s4Time10UNoticeBarC4show8duration9completedySd_ySbcSgtFfA0
Referenced from: /Users/tate/Library/Developer/CoreSimulator/Devices/6BB1496B-CE17-4A53-A2E9-7A16CE440C6D/data/Containers/Data/Application/18B0CA15-7294-48C2-91E0-9A69C91FED5E/tmp/eval101.dylib
Expected in: flat namespace
in /Users/tate/Library/Developer/CoreSimulator/Devices/6BB1496B-CE17-4A53-A2E9-7A16CE440C6D/data/Containers/Data/Application/18B0CA15-7294-48C2-91E0-9A69C91FED5E/tmp/eval101.dylib ***

Hi, these symbols should be defined but only after the second build after a clean build. Did it work after your rebuilt?

Hello,thank you for your response， restarting has no effect. The main reason is that commenting the following lines of code does not work. I don't really understand what is wrong with the code here?
//        containerView.addSubview(startBtn)
//        startBtn.snp.makeConstraints { (make) in
//            make.centerX.equalToSuperview()
//            make.bottom.equalTo(-28)
//            make.height.equalTo(50)
//            make.width.equalToSuperview().multipliedBy(0.75)
//        }
Reported error：
🔥 Loading .dylib ...
🔥 Loading .dylib has failed, This is likely because Swift code being injected refers to a function with a default argument. Rebuilding and re-running your project can resolve this.
🔥 *** dlopen() error: dlopen(/Users/tate/Library/Developer/CoreSimulator/Devices/D69A82CD-CD24-4923-8CE2-CB48EC989754/data/Containers/Data/Application/FAB2FE49-E7EA-4B96-928E-C7B7DFD6661A/tmp/eval101.dylib, 2): Symbol not found: $s4Time10UNoticeBarC4show8duration9completedySd_ySbcSgtFfA0
Referenced from: /Users/tate/Library/Developer/CoreSimulator/Devices/D69A82CD-CD24-4923-8CE2-CB48EC989754/data/Containers/Data/Application/FAB2FE49-E7EA-4B96-928E-C7B7DFD6661A/tmp/eval101.dylib
Expected in: flat namespace
in /Users/tate/Library/Developer/CoreSimulator/Devices/D69A82CD-CD24-4923-8CE2-CB48EC989754/data/Containers/Data/Application/FAB2FE49-E7EA-4B96-928E-C7B7DFD6661A/tmp/eval101.dylib ***

I just found out that an error was reported when running on a real machine. The error log:
dyld: Library not loaded: @rpath/SwiftTrace.framework/SwiftTrace
Referenced from: /private/var/containers/Bundle/Application/C7F794AA-6352-4931-92CA-B48D2C4E1CF8/Time.app/Frameworks/HotReloading.framework/HotReloading
Reason: image not found
dyld: launch, loading dependent libraries
DYLD_LIBRARY_PATH=/usr/lib/system/introspection
DYLD_INSERT_LIBRARIES=/Developer/usr/lib/libBacktraceRecording.dylib:/Developer/usr/lib/libMainThreadChecker.dylib:/Developer/Library/PrivateFrameworks/DTDDISupport.framework/libViewDebuggerSupport.dylib
Message from debugger: Terminated due to signal 6

Hi, sorry this is holding you up. The problem you reported is strange one as the symbol should have a "_" at the end if it is for a default argument.
$ swift demangle
$s4Time10UNoticeBarC4show8duration9completedySd_ySbcSgtFfA0_
default argument 1 of Time.UNoticeBar.show(duration: Swift.Double, completed: (Swift.Bool) -> ()?) -> ()

I have to believe what you pasted is correct so I've made a small change to the HotReloading package to  "unhide" symbols like the one you have seen available on branch "default-argument". Can  you give it a try please?

On the second problem about SwiftTrace, how do you mean "Real Machine"? Do you mean an actual phone? I've never tried running HotReloading on a real device but I don't think it will work.

Hi, sorry this is holding you up. The problem you reported is strange one as the symbol should have a "_" at the end if it is for a default argument.
$ swift demangle
$s4Time10UNoticeBarC4show8duration9completedySd_ySbcSgtFfA0_
default argument 1 of Time.UNoticeBar.show(duration: Swift.Double, completed: (Swift.Bool) -> ()?) -> ()

I have to believe what you pasted is correct so I've made a small change to the HotReloading package to "unhide" symbols like the one you have seen available on branch "default-argument". Can you give it a try please?

Hello, I updated the code of 1.4.3, but still encountered a new problem, modified the previous code, and then saved and reported a new error as shown in the figure:

Not sure what's happing there, it shouldn't be breaking on throw which is because of the undefined symbol. Did you try re-running or  the "default-argument" branch?

Hi~ o(￣▽￣)ブ，I probably know why. As long as the Swift code file I wrote refers to the function with default parameters, there will be no hot update. I can compile the code by commenting it, but this will reduce the efficiency. I hope you can support this. To fix the problem, many of my functions have default parameter values, thank you!
public init(title: String? = nil,                 image: UIImage? = nil,                 textColor: UIColor = UIColor.white,                 backgroundColor: UIColor = UIColor.orange,                 barStyle: UNoticeBarStyle = .onNavigationBar,                 animationType: UNoticeBarAnimationType = .top) {         self.title = title         self.image = image         self.textColor = textColor         self.backgroundColor = backgroundColor         self.barStyle = barStyle         self.animationType = animationType     }

If the code you're injecting refers to a function that has default arguments this is where you get the problem. One solution is to prove the adult argument in this code but you shouldn't be being this as the "Unhide" code has a work-around which is included in HotReloading.

Sorry, does the question of'Unhide' mean this? I have already set it up, or am I doing something wrong?

You're not doing anything wrong. That's the setup for HotReloading which includes my internal "Unhide" function so injecting default arguments should work (after the second build).

Thank you！

You're not doing anything wrong. That's the setup for HotReloading which includes my internal "Unhide" function so injecting default arguments should work (after the second build).

I meet the same problem after I build more than one time.
🔥 Loading .dylib ...
🔥 Loading .dylib has failed, This is likely because Swift code being injected refers to a function with a default argument. Rebuilding and re-running your project can resolve this.
🔥 ⚠ dlopen() error: dlopen(/Users/xxx/Library/Developer/CoreSimulator/Devices/ACA0C019-A7F1-42DF-B31D-A4AFD1EE31F3/data/Containers/Data/Application/FFE45CEA-828B-48BB-9A74-9B37093BB8BA/tmp/eval101.dylib, 2): Symbol not found: _$s8ByteView6RouterC6sharedACvau
Referenced from: /Users/xxx/Library/Developer/CoreSimulator/Devices/ACA0C019-A7F1-42DF-B31D-A4AFD1EE31F3/data/Containers/Data/Application/FFE45CEA-828B-48BB-9A74-9B37093BB8BA/tmp/eval101.dylib
Expected in: flat namespace
in /Users/xxx/Library/Developer/CoreSimulator/Devices/ACA0C019-A7F1-42DF-B31D-A4AFD1EE31F3/data/Containers/Data/Application/FFE45CEA-828B-48BB-9A74-9B37093BB8BA/tmp/eval101.dylib
How to fix it? It's encountered a new problem when I remove default argument func. The console log as follow:
🔥 Compiling /Users/xxx/Workspace/xxxr/ToolBarPhoneViewController+Test.swift
🔥 Loading .dylib ...
🔥 Loaded .dylib - Ignore any duplicate class warning ^

@johnno1962 plz.

Not a problem I've seen before. What version of Xcode is this?
Johns-Mac-mini HotReloading % swift demangle
_$s8ByteView6RouterC6sharedACvau
ByteView.Router.shared.unsafeMutableAddressor : ByteView.Router

Can you type the following command on your executable:
nm /<path_to_executable> | grep '$s8ByteView6RouterC6sharedACvau'
and tell me what it says?

@johnno1962
Thanks for reply.
My Xcode is Version 12.3 (12C33).
The grep of '$s8ByteView6RouterC6sharedACvau' as follow:
0000000105eb8ec0 t _$s8ByteView6RouterC6sharedACvau

I've never encountered this before but have put in what may be a fix. Can you update HotReloading but from the branch "unhiding-eval" in the Swift Packages inspector.

@johnno1962
I use the branch "unhiding-eval", but it has a compile error as follow:

Sorry I'll delete that branch. You can use the 1.6.2 tagged release now.

@johnno1962
I try the version of 1.6.2, but encountered the same problem as follow:
🔥 Loading .dylib has failed, This is likely because Swift code being injected refers to a function with a default argument. Rebuilding and re-running your project (without a build clean) can resolve this.
🔥 ⚠ dlopen() error: dlopen(/Users/xxx/Library/Developer/CoreSimulator/Devices/ACA0C019-A7F1-42DF-B31D-A4AFD1EE31F3/data/Containers/Data/Application/E4A3C57B-499A-4E61-8CAC-335A7BD43DAF/tmp/eval101.dylib, 2): Symbol not found: _$s8ByteView6RouterC6sharedACvau
Referenced from: /Users/xxx/Library/Developer/CoreSimulator/Devices/ACA0C019-A7F1-42DF-B31D-A4AFD1EE31F3/data/Containers/Data/Application/E4A3C57B-499A-4E61-8CAC-335A7BD43DAF/tmp/eval101.dylib
Expected in: flat namespace
in /Users/xxx/Library/Developer/CoreSimulator/Devices/ACA0C019-A7F1-42DF-B31D-A4AFD1EE31F3/data/Containers/Data/Application/E4A3C57B-499A-4E61-8CAC-335A7BD43DAF/tmp/eval101.dylib

Did you rebuild and re-run (without doing a build clean) and try again? This is necessary to have the patched object files make it into the app.

@johnno1962
It works! 🎇 But use "@objc func injected()" have an error, It's an infinite cycle in "func sweep(_ value: Any)", and use notification is working.
The error as follow:
🔥 Loaded .dylib - Ignore any duplicate class warning ^
🔥 ⚠ Adding or removing methods on Swift classes is not supported. Your application will likely crash. ⚠
🔥 Injected class 'xxx.ToolBarPhoneViewController'
🔥 As class ToolBarPhoneViewController has an @objc injected() method, HotReloading will perform a "sweep" of live instances to determine which objects to message. If this fails, subscribe to the notification "INJECTION_BUNDLE_NOTIFICATION" instead.

Great! Strange about the cycle through You can set environment variable "DEBUG_SWEEP" in your scheme's launch settings to track down what is happening. I've never had an issue of an infinite loop.


Change approach to compute the stress

Previously shear stress was computed in the cell centers and interpolated onto the cell vertices. The new kernel computes shear stress directly on the vertices; this also avoids and extra MPI comm step.

[x] 2D stress kernel
[ ] 3D stress kernel
[ ] Clean up

CC @ChristianSchuler @aelligp


feat: class/singleton properties

Add an easy way to create properties on a class, probably similar to GMod's AccessorFunc method.

Example usage:
ClassName:Property("Name", "boolean", true)

Args:

The name of the property. This is used for the method names of getters/setters (Get{name}/Set{name}) and the name of the property variable (_{name}).
The type of the property, should respect use of ClassLoader types (see PR #7).
An optional default value, should be set using the newly created setter.


Optimization and code documentation

Codecov Report

Merging #241 (71db366) into main (da729c5) will increase coverage by 0.99%.
The diff coverage is 99.38%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
+ Coverage   82.39%   83.38%   +0.99%
==========================================
  Files          69       69
  Lines        6851     7007     +156
==========================================
+ Hits         5645     5843     +198
+ Misses       1206     1164      -42

Impacted Files
Coverage Δ

pallets/asset_management/src/lib.rs
77.94% <ø> (ø)

pallets/asset_management/src/mock.rs
95.83% <ø> (ø)

pallets/payment/src/mock.rs
100.00% <ø> (+45.83%)
:arrow_up:

pallets/tenancy/src/lib.rs
80.00% <ø> (ø)

pallets/tenancy/src/mock.rs
95.23% <ø> (ø)

runtime/src/lib.rs
10.28% <ø> (ø)

pallets/asset_management/src/functions.rs
91.35% <88.88%> (+0.16%)
:arrow_up:

pallets/payment/src/tests.rs
99.52% <100.00%> (+0.15%)
:arrow_up:

pallets/payment/src/lib.rs
77.16% <0.00%> (+1.57%)
:arrow_up:

pallets/payment/src/weights.rs
11.11% <0.00%> (+11.11%)
:arrow_up:

... and 2 more

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


Using Net.CredentialCache.DefaultCredentials with SetProxy()

In order to set proxy settings what we usually use is something similar to the following -
var wp = System.Net.WebRequest.DefaultWebProxy; wp.Credentials = System.Net.CredentialCache.DefaultCredentials;
Is there anyway to use this or similar with SpeechConfig.SetProxy method?
Currently receiving the following error -
CANCELED: ErrorDetails=Connection failed (no connection to the remote host). Internal error: 1. Error details: 11001. Please check network connection, firewall setting, and the region name used to create speech factory. SessionId: f59b665c6e724ca386fc2e9c3e870c45
This is received when using the SetProxy method as so -
_config.SetProxy("http://<IP_ADDRESS>", 8080);
Could I use the SetProxy() overload method with default credentials?
Any help much appreciated!

Hi, you can use SetProxy also with proxyUserName and proxyPassword parameters - see the API reference (C#):
https://docs.microsoft.com/en-us/dotnet/api/microsoft.cognitiveservices.speech.speechconfig.setproxy?view=azure-dotnet
Also please note that proxyHostName should not have a protocol prefix (http://).
Unfortunately there is currently no support for using a DefaultCredentials object (System.Net.ICredentials) as a parameter directly.


Question: Is it possible to determine what ACTIONS are defined for a particular state?

Bug or feature request?
Question
Description:
Hi David, I have enjoyed your youtube videos, and have started investigating xstate for use in my project.  It was a breeze to implement the traffic light example, and step through the states with a button. Thank you!  Hat Tip for the Ryan Florence and the State Machine reference 👍
For my use case, I have a question.
Is it currently possible to determine what ACTIONS are defined for a particular state?
i.e. on the review state, I can determine the actions SUBMIT and REJECT are available?
I can then show these as actions on my form.
{
  key: "employment",
  initial: "draft",
  states: {
    draft: {
      on: {
        SUBMIT: "review"
      }
    },
    review: {
      on: {
        SUBMIT: "manager",
        REJECT: "draft"
      }
    },
    manager: {
      on: {
        APPROVE: "complete",
        REJECT: "review"
      }
    }
  }
}

You can get all events for a machine via:
myMachine.events; // ['SUBMIT', 'REJECT', ...]

You can get then for a specific state node via the same:
myMachine.states.draft.events; // ['SUBMIT']

You can also see if a state node (or machine) handles a certain event:
myMachine.handles('SUBMIT'); // true

Brilliant! Thanks so much.
Is it worth me adding this to the doco, or have my noob eyes not read it correctly?

Is it worth me adding this to the doco

It is!

Trying to do the same - need to get a list of the transitions that can occur on a node. These are known as EVENTS here, yes?
state.events seems to always just returns an empty array...

@davidraine David K's guidance above worked a treat for me.  I created a simple app that cycles through the various states, showing buttons of the EVENTS available at a given state.  Perhaps it will assist with your question.  https://github.com/seankellyaus/xstate-implementation

Thank you - that is really helpful.

@davidraine (et. al.) The future API for this in v4 will be:
const nextState = someMachine.transition(...);

// get next possible events for the given state
nextState.nextEvents;
// => ['FOO_EVENT', 'BAR_EVENT', 'BAZ_EVENT']

The .nextEvents property holds all the possible next events for the given state value; that is, all events that can be executed from the state nodes that represent the state value in the given state, excluding forbidden events. So if you have an event that should be "forbidden":
const myMachine = Machine({
  // ...
  states: {
    foo: {
      on: { FORBIDDEN_EVENT: undefined }
    }
  }
});

Then FORBIDDEN_EVENT will not be returned in .nextEvents when you're in the foo state.
See 5240c3e for more implementation details.

Hi,
state.nextEvents returns available transitions for a given state, but its seems to not run the guards.
Is there a better way than this ?
machine.initialState.nextEvents.filter(next => machine.transition(machine.initialState.value, next).changed);

state.nextEvents returns available transitions for a given state, but its seems to not run the guards.

Guards are dependent on context and event data, so they will not run on plain events - this is by design.
You can do this:
currentState.nextEvents.filter(nextEvent => {
  return machine.transition(currentState, { type: nextEvent, /* ... */ }).changed;
});

Thanks, that matchs the exemple i've given.
My second second exemple returns also an resolved event where changed is not triggered.
This might be helpful for a more "workflow" approach.
Why not push them into the documentation if it's valid?

In the future we might improve SCXML compatibility and nextEvents API might get cimberse with that as in SCXML transitions might work on event type prefixes so its impossible to give a user true possible „nextEvents” as that set wont be fully known by XState


Problematic issue with generation of non-shared temporary files

Looking at P.getTempFilename() in CGATPipelines/Pipeline/File.py consider:

A user runs a pipeline on head_node
A "to_cluster=yes" task does "tmp_file = P.getTempFilename()"
P.getTempFilename() will safely write a temporary file in PARAMS["tmpdir"] on the head_node! and return the name to the python variable.
The job goes to the cluster
The user will inadvertently write an unsafe (i.e. name collision possible) temporary file in PARAMS["tmpdir"] on a compute_node according the python variable set on the head_node.

There are then 2 temporary files which both need to be cleaned up - likely partly because of this there are e.g. quite a lot of files in /scratch on cgat150:

stephens@cgat150:/ifs/home/stephens/
$ ls /scratch/ | wc -l
179850`

More serious is if the files are cleaned up only on the head node - they will then remain on the compute node scratch: this will happen unless users explicitly rm the tmpfile in the job statement...
It seems clear that a python script executing on a head node cannot give a safe temporary filename for local scratch space on a compute node.
Furthermore, the arbitrary requirement to open a file of the same name on the head node prevents specification of the node scratch location in an environment variable by the queue manager (e.g. in a job prolog script).
To non-destructively fix I suggest:
(1) A new a PARAM["local_tmpdir"] variable that cluster jobs requiring a temporary filename can use to safely get a temporary file on the node, e.g. using the bash utility i.e. "mktemp" in the job statement i.e.

statement='''tmpfile=`mktemp -p %(local_tmpdir)s`;
cat my_data > $tmpfile;
rm $tmpfile'''

(2) Moving forward use of PARAMS["tmpdir"] is depreciated and P.getTempFile() and P.getTempFilename() only use PARAMS["shared_tmpdir"] if a folder is not specified.
Note that this will also allow for specification of the compute node scratch space location via a shell environment variable so that queue managers can deploy and clean up scratch resources (i.e. PARAMS["local_tmpdir"] can then be set to contain the name of a shell environment variable.)
Can I please implement these changes?

'local_tmpdir' added to CGATPipelines/Pipeline/Parameters.py.
See:
https://github.com/CGATOxford/CGATPipelines/commit/49dfd998e32728c1aa5c805e1d77e001158694b1
Note that PARAMS["local_tmpdir"] is meant to be directly passed to the job script and used as follows:

mvar=`mktemp -p %(local_tmpdir)s`

Please do not reference in python code or code portability will be broken!
I suggest depreciating use of PARAMS["tmpdir"] and re-writing existing code as above.

Hi @snsansom , I am not sure why local_tmpdir is required? This is exactly what tmpdir is!
Yes, the temporary file gets created on the head-node, but this means that within a CGAT environment, temporary files are unique across the cluster, which is an even stricter requirement
than being unique on a node. The ctmp prefix ensures that there is no conflict with other tools creating a temporary file.
Yes, clean-up could be improved, but I think the changes are unnecessary.

The difference is that "local_tmpdir" is not meant to be evaluated on the head node.
In our setup, a SLURM prolog sets a per-job scratch dir on the execution node in an an environment variable. It is necessary therefore to pass this straight through to the job statement in the way described above.
If I used "tmpdir" for this purpose i.e. PARAMS["tmpdir"]=$SCRATCH_DIR all existing references to P.getTempFile() will fail.
Also:

Temporary files are not necessarily unique across the cluster by your design - the head node ultimately has no way of knowing what has happened on a node.

The current system lends itself to abuse - i.e. users pass a P.getTempFilename() var to a task and then do an os.unlink after P.run(): the tempfile is not deleted from the execution host.

Thanks. Providing a job-specific tmp dir is a good solution and I see the need to feed the environment variable to job scripts.
I am worried that we now have two ways to use temp files. Does the following not work out-of
the box?
    statement = '''
    my_temp_file=`mktemp -p %(tmpdir)s`;
    # do stuff with $my_temp_file;
    rm $my_temp_file;
    '''
    P.run()

In the current solution, temporary names are all handled by the head-node, hence I would think that they are unique on each node as well? I don't see the way there would be a conflict unless intentionally creating one?
Indeed, there is no principled solution for clearing up the temp files and people need to use the correct pattern in order to not have debris lying around.

Thanks.
Your statement is correct for new pipelines, but all existing references to P.getTempFile() (and there are a lot in CGATPipelines!) would trip over when passed e.g. "$SCRATCH_DIR" on the head node.
Currently if a tmpfile is cleaned up the head node but not on the execution node (I am sure this happens frequently) you are open to name space collision.
Here our solution for cleaning up node scratch is an accompanying SLURM epilog that does "rm -rf $SCRATCH_DIR". It seems to work nicely :).

I see, I am getting there. Ok, let us do both.

Currently if a tmpfile is cleaned up the head node but not on the execution node (I am sure this happens frequently) are you not open to name space collision?

If two pipelines are running concurrently, they will coordinate tempfiles via the head node and there should be no conflict. If one pipeline has finished, but failed to clean up, and another job by chance gets the same temporary filename, then the existing tmpfile will simply be overwritten, but nobody will be missing its contents.
I fully admit that this is not the best solution and providing per-job scratch is much cleaner.

For reference this is the nub of the issue:
>>> tempfile.NamedTemporaryFile(dir="$SCRATCH_DIR") Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/gfs/apps/apps/python-2.7.11/lib/python2.7/tempfile.py", line 475, in NamedTemporaryFile     (fd, name) = _mkstemp_inner(dir, prefix, suffix, flags)   File "/gfs/apps/apps/python-2.7.11/lib/python2.7/tempfile.py", line 244, in _mkstemp_inner     fd = _os.open(file, flags, 0600) OSError: [Errno 2] No such file or directory: '$SCRATCH_DIR/tmp1c6SCo'

Cool thanks for taking the time to go through it. If CGAT moves to SLURM I'll be happy to share our setup.


Best practies for github actions: permissions and dependabot

Related: #221
Description of changes:
Explicitly set permissions required inside github actions
For dependabot what you'd prefer for schedule interval?
And for the preference you expressed:

raise PRs unless there is a semver-breaking version bump

I think the closest behavior can be ignoring semver patch release for all actions,
instead minor, major usually can contain some breaking changes. Is it ok?
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I think we can lean on "increase-if-necessary" for Dependabot? (see here)

Yeah, you're right. Let's stick to your suggestion then.

To benefit from dependabot update checks the last thing that remain to do is to explicitly pin the complete tag version for all github actions used (to avoid rolling tags such as checkout@v3) even better the full commit hash.
Fortunately for this error prone task we can use https://github.com/step-security/secure-repo, I've already preview changes for full length commit sha pinning: https://github.com/step-security-bot/cargo-chef/commit/760092bca981d81aa9d3cb35a7678b17b4c3a55b
If you are ok with this approach, I can let the bot open a PR, and then incorporate its changes here

I prefer to keep a rolling tag, which is why I want to skip patch updates (and perhaps minor, depending on the frequency).
Pinning would just lead to unnecessary PR churn on this repository.


fix(minimal-slp-wallet): Updating to v3.7.0

:tada: This PR is included in version 2.19.0 :tada:
Your semantic-release bot :package::rocket:


luminance-hdr: use Qt5's mkDerivation

Motivation for this change
Fix luminance-hdr as per https://github.com/NixOS/nixpkgs/issues/65399
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Cosmetic: maybe put the mkDerivation at the beginning of the argument list?

Done.

backported to:

19.09 f6c1d3b113ce9e7c908991df2e8eed02f03df1bc
20.03 3add50e56db4a618c19889652e92ef79b3e631bb


Cancel partial posted purchase invoice updates received quantities

When I cancel a partial posted purchase invoice, in purchase order the Qty to Receive and Qty Received are updated also. It should update only field Qty to Invoice and Qty. Invoiced

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

ID: 83c8f216-2ecd-ae0e-a754-05c14bebe5bf
Version Independent ID: 65149fe7-b68f-68a7-d37e-ac88469a289a
Content: Amend or Cancel Unpaid Purchase Invoices - Business Central
Content Source: business-central/purchasing-how-correct-cancel-unpaid-purchase-invoices.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: sgroespe

Hello,
Thank you for your feedback. Since your question is not about the Help, we suggest that you submit a feature request here: https://aka.ms/businesscentralfeedback


Notebook language icons missing for .NET

Testing #114902
Unsure if this something the extensions have to set, but python has their language icon for code cells:

And .NET has the default:

I will take a look but it's like that no icon is defined that therefore the default icon is used. Outline simply uses the theme service for this

I will take a look but it's like that no icon is defined that therefore the default icon is used. Outline simply uses the theme service for this

Adding @aeschli
The same issue can be observed for with GH issue notebooks, which also have a special icons. What's special with notebooks is that they don't have file names from which to derive an icon but the use the language name, e.g getIconClassesForModeId, and I am unsure if that has a bug or if themes must do something here

Adding @aeschli
The same issue can be observed for with GH issue notebooks, which also have a special icons. What's special with notebooks is that they don't have file names from which to derive an icon but the use the language name, e.g getIconClassesForModeId, and I am unsure if that has a bug or if themes must do something here

@misolori What exact theme are you using? Some themes don't define icons for language ids, but just for file extensions.

@misolori What exact theme are you using? Some themes don't define icons for language ids, but just for file extensions.

@aeschli I'm using default (seti)

@aeschli I'm using default (seti)

Can you inspect what the class names for label node are?

Can you inspect what the class names for label node are?

Can you inspect what the class names for label node are?

@aeschli you can check that yourself when using GH issue notebooks

Can you inspect what the class names for label node are?

@aeschli you can check that yourself when using GH issue notebooks

We don't have a language called github-issues.. But Seti has an icon icon for the file extension github-issues.
This can be fixed by adding a language github-issues, and regenerate the seti-theme.
However, having a language github-issues probably doesn't make much sense except for the icon case.
Therefore I'll add that language just in the seti font build script where I have a list of "nonBuiltIn" languages.
Fixed as as part of cba8232d778fe751d69edab7536bf89b66906caa

Still need to check what's the problem with .net notebook.

However, having a language github-issues probably doesn't make much sense except for the icon case.

@aeschli that language exists but it is contributed by an extensions: https://github.com/microsoft/vscode-github-issue-notebooks/blob/27bf6ea885bdb983567013fa4d6489995c014c09/package.json#L41

The C# interactive notebook uses dotnet-interactive.csharp as language, which of course is not known to the seti-theme.
Couldn't/shouldn't they use csharp as language for the cell?

The issue is in the C# extension, closing the issue

To verify, open the outline for a Github notebook as described here and verify that you see the Github Icon


Using uploads_dir Placeholder in CDN Custom File list Breaks Page

CDN Generic Mirror doesn't work with "{uploads_dir}/*" in Custom file list as discussed in #296.
The problem is that the {uploads_dir} placeholder when converted is becoming a full url (e.g. http://mydomain.com/wp-contents/uploads/* ) -- unlike {wp_content_dir} and {plugins_dir} which are filesystem paths not urls -- and so, upon closer inspection it seems full urls in the Custom Files list such as http://foo.bar/* are mishandled.
This fix tries to remedy that (and the {uploads_dir} issue) by handling full urls more correctly so that when the CDN host name is used in the page it can properly replace any given domain (if given in the custom files list).

I think {uploads_dir} is meant to be an URL path, the path part of the URL.
https://github.com/szepeviktor/w3-total-cache-fixed/blob/v0.9.5.x/Cdn_Util.php#L214-L220
Originally comes from wp_upload_dir() in function wp_upload_dir() in Util_Environment.php
A custom (relocated) wp-content dir or uploads dir may cause problems.
I think its value should be the path relative from the document root:
$upload_dir = str_replace( Util_Environment::document_root(), '', $upload_dir['basedir'] );

@amiga-500 Your could debug it here
https://github.com/szepeviktor/w3-total-cache-fixed/blob/v0.9.5.x/Cdn_Util.php#L221
error_log( 'U=' . $upload_dir );

Yea i was thinking the same about {uploads_dir} and one can easily make it relative by modifying https://github.com/szepeviktor/w3-total-cache-fixed/blob/v0.9.5.x/Cdn_Util.php#L233 (this is the specific spot that CDN custom file list uses).  It just needs to be changed to:
$uploads_uri = Util_Environment::url_to_uri($upload_dir['baseurl']);

I tested it out earlier today and it worked fine -- it solved the {uploads_dir}/* problem.  That's when i discovered that files listed in the Custom Files box can potentially have full urls, not just paths.  So this PR takes care of full urls given by users correctly now.
I will add that line code above to Cdn_Util.php so {uploads_dir} is now a relative path.  Earlier I  came across a WP article showing how one can modify the upload dir via wp-config.php but after checking different articles just now it seems although the upload dir can be modified it is always relative to the domain (unless one uses a plugin which then can use other domains).
It's interesting that if i were to use the line you indicated:
$upload_dir = str_replace( Util_Environment::document_root(), '', $upload_dir['basedir'] );

instead of the one i was planning:
$uploads_uri = Util_Environment::url_to_uri($upload_dir['baseurl']);

then if the $upload_dir host happens to be different from the document root host then the upload_dir's host will remain in $upload_dir, whereas my line would strip the hostname at all times, leaving the path only.  So, then maybe your line would work better as a fail safe if somehow the user does mess around with wp to modify the domain of the upload_dir.
I will add the line:
$upload_dir = str_replace( Util_Environment::document_root(), '', $upload_dir['basedir'] );

to https://github.com/szepeviktor/w3-total-cache-fixed/blob/v0.9.5.x/Cdn_Util.php#L233

Please consider opening a separate PR for this So this PR takes care of full urls given by users correctly now.

It's interesting that if i were to use the line you indicated:

There is a confusion here.
I've examined and linked the previous function, replace_folder_placeholders(), not replace_folder_placeholders_uri()

My debug output shows that {uploads_dir}/* get converted to full URL not a path.
M=favicon.ico
M=/static/gallery/*
M=/static/uploads/avatars/*
M=/static/plugins/wordpress-seo/css/xml-sitemap.xsl
M=/static/plugins/wp-minify/min*
M=/static/plugins/*.js
M=/static/plugins/*.css
M=/static/plugins/*.gif
M=/static/plugins/*.jpg
M=/static/plugins/*.png
M=http://short.keszul.tk/static/uploads/*

Investigating...

There is a confusion here.
I've examined and linked the previous function, replace_folder_placeholders(), not replace_folder_placeholders_uri()

Yes i was aware of that.  I was doing some tests to see which of the lines would be more efficient to use.  Both end up with the same result with neither being better than the other.   So i will use my line of: $uploads_uri = Util_Environment::url_to_uri($upload_dir['baseurl']); in the next PR.

My debug output shows that {uploads_dir}/* get converted to full URL not a path.

Precisely!  If i dont change anything then this new PR will handle it fine.  But i can adjust this to become a relative path so at least it is more inline with the other placeholders -- they are just paths.

After adding $uploads_uri = Util_Environment::url_to_uri( $uploads_uri ); } into the isset() branch:
M=favicon.ico
M=/static/gallery/*
M=/static/uploads/avatars/*
M=/static/plugins/wordpress-seo/css/xml-sitemap.xsl
M=/static/plugins/wp-minify/min*
M=/static/plugins/*.js
M=/static/plugins/*.css
M=/static/plugins/*.gif
M=/static/plugins/*.jpg
M=/static/plugins/*.png
M=/static/uploads/*

As this one line solves #296 I think this PR could be closed.

As this one line solves #296 I think this PR could be closed.

Actually that is what i was thinking earlier today but remember that because people can still add their own full urls into the custom files list box then without this PR the problem a site can still break.  The {uploads_dir} placeholder was only showing us what happens when a full url is used.  Because we fixed the {uploads_dir} placeholder by making it a relative path it doesn't solve the other issue when users use full urls in the box.
So i think the PR might actually still be needed.

OK!
Testing....

Please consider calling a - perhaps new - utility function here
https://github.com/szepeviktor/w3-total-cache-fixed/blob/v0.9.5.x/Cdn_Plugin.php#L1010

The current output of a full URL is
http\\://short\\.keszul\\.tk/customdir/[^\\?\\*:\\|'"<>]*\\.jpg

Why is it incorrect? I think we should find the real cause.

Why is it incorrect? I think we should find the real cause.

I can explain what is happening....Your output of:
http\\://short\\.keszul\\.tk/customdir/[^\\?\\*:\\|'"<>]*\\.jpg

Is correct.  The problem is caused by this: https://github.com/szepeviktor/w3-total-cache-fixed/blob/v0.9.5.x/Cdn_Plugin.php#L1025
This line is what handles full urls.  The problem is the regex is not properly formed.  I will explain in more detail shortly.

I think I got it.
This input field expects URL paths only.
Entering something others causes undefined behavior.

The L1025 has the "?".  In its present form that "?" is what is causing all the havoc.  When its removed the full url will now be inserted.  But there is a problem....

The problem though with that is the regex does not separate the scheme, domain, and path into separate groups.  That's important. Without that then the proper replacing with the CDN host cant happen correctly.  It replaces the matches with just the cdn host name (you lose the path).

So that regex needs to be rewritten to be more aware to handle full urls.
And that is what this PR does.
Notice that i rewrote it so it could handle the

I think we should sanitize full URL just after if is_url()
How to strip off everything except URL path? parse_url()?

Is it allowed to enter foreign URL-s?

Yup it allows entering foreign urls.
Remember, this PR already fixes the full URL problem.  So we dont need to strip a full url down to its path.

Why is it incorrect? I think we should find the real cause.

The answer is no as there is an input field File types to import: to "Automatically import files hosted with 3rd parties".

handles remote (foreign) urls.

I think we need a project-level decision here. This is not my task.

The answer is no as there is an input field File types to import: to "Automatically import files hosted with 3rd parties".

Ahhhh thats interesting.  I didnt see that.  Seeing it now.

The line if ( Util_Environment::is_url( $mask ) ) suggests that Full URL-s are expected.
Could we settle on simple stripping off everything except the URL path?
The next official release will take care of this.
Good night!

Ok then...this becomes a lot simpler...great!
So  I will:

I will deal with the {uploads_dir} placeholder thing we discussed (add the line of code)
sanitize any user full URL to strip out and leave just its path to be processed.

And thanks for your assist @szepeviktor!  Have a good night!


Updating element classname removes aos-animation class

This is:

Bug

Expected Behavior

Element should not fade out / aos-animate class should not be removed
Actual Behavior

aos-animate class is removed even though elements are in viewport
Steps to Reproduce the Problem

1.I'm using react, when I update my class with a prop, the aos-animate class is removed and therefore the elements disappear even though they are in the view port

same issue here

same issue here

What I did to solve it for now was add an outer div with no styles which I apply the animation on. Something like

Had the same issue in Nuxt.js as well. When adding data-aos on a parent element which content can change (even just a checkbox), makes the entire element disappear as it removes the AOS classes.
I solved it by surrounding that element with <client-only> tags like:
<client-only>
    <div data-aos="fade">
    ....
    </div>
</client-only>

It's not an ideal solution for a SSR app, but it works.


chore(main): bump version to 2.2.4

:robot: I have created a release beep boop
2.2.4 (2023-06-11)
Continuous Integration

fix update swagger workflow error (3ce98bc)
optimize Dockerfile (ba7a5c9)

Chores

added types for code styling tools config file (1b304fc)
deps: upgrade dependency packages (5257d4f)
optimize linter config (94a2e3c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kenso312/nestjs-v9-webpack-boilerplate/releases/tag/v2.2.4 :sunflower:


Add support for Composite Tuples

This fixes https://github.com/hodur-org/hodur-datomic-schema/issues/5

This does not always work, as the schema produced by hodur-datomic-schema returns the attributes in alphabetical order, but this means that for a composite tuple not all the attributes that comprise the composite might exist yet, causing :db.error/invalid-tuple-attrs error when trying to apply the schema


Multilingual Bot Translator for Spanish, German, Portuguese to english and vice versa

📝 Description
Developed a multilingual translation AI system that accurately translates text from Spanish, French, German, and Portuguese into English and vice versa.

Dear @Adrija-G , thank you very much for your contribution. We found that your AI Agent may not be behaving as expected. You can try changing the content in the prompt into commands for GPT (instead of introducing functions to users)

After optimization

For your reference:
Act as a multilingual translation AI system capable of accurately translating text from Spanish, French, German, and Portuguese into English and vice versa with a focus on fluency, contextual understanding, and natural language processing.
Start the conversation with {Hello, please send me your text in Spanish, French, German, or Portuguese}


Feature request : Need an api to return StructType when xml-string is passed

Struggling to find a direct api which gives StructType when xml-string is passed.
There is api present but that takes SparkSession which returns Dataset and by using we need to do .schema(), Currently we are doing like below.
new XmlReader().withRowTag(rowTag).xmlRdd(spark_session, rdd));

You could use schema_of_xml - https://github.com/databricks/spark-xml/blob/master/src/main/scala/com/databricks/spark/xml/package.scala#L124-L126

But the method 'schema_of_xml' takes 'Dataset[String]' as an argument.

You can do schema_of_xml(Seq("xml-string").toDS)

which scala version of Seq has toDS method?
https://www.scala-lang.org/api/current/scala/collection/Seq.html
I don't find it in the link above.

Spark's implicit has it. You might have to do import spark.implicits._.

Dear @HyukjinKwon, Thank you very much, I'm able to solve, I did a small reverse engineering and found what is happening under the hood.

Dataset ds = session.createDataset(Arrays.asList(xml_string), Encoders.STRING());
StructType xmlStructType = InferSchema.infer(ds.rdd(), new XmlOptions());

Oh you're using Java API. Sure that should be the right way.


Revert "xrootd: provide fuse support (xrootdfs) for Darwin and fix the tests"

Reverts NixOS/nixpkgs#206969. See https://github.com/NixOS/nixpkgs/pull/206969#issuecomment-1415202826

Just found that the passthru.tests.test-xrdcp got dropped during the revert and redo.
I'll redo that, since it ensures the functionality of this library and the xrdcp executable, and can be run on all supported platforms. With this test, we could just merge the automatic update from r-ryantm once all the tests are passed.


#144155241

…update or delete document/folder[144155241]

Changes Unknown when pulling 4f7309e4a944a89ab427fd4314d1a6e29d046d93 on feature/144155241/add-user-dashboard into ** on development**.


chore(package): update lint-staged to version 10.0.7

Closes #46

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Disable CSS of Sublime Text 3 get an error

After installed CSS3, when I Disable CSS of Sublime Text 3, which popuped an error but only once. I can't show it again. So strange?

Unfortunately, every time when I will install it, it said something wrong in CSS3...

I can't be sure without more information, but I suspect the problem is that you had a file open with the syntax set to CSS when CSS was disabled.

If you have CSS3 installed, try cmd+shift+p (mac/linux) / ctrl+shift+p (windows) and Set Syntax: CSS3 to switch to the new syntax highlighter.
If that doesn't fix the problem, could you please screenshot the error message and post it here?