Question: How do you use url_for to get a route specified in your yaml file?

Question is in the subject.
I've tried using operationId to no avail. Given the following snippet, how would I use url_for to fetch a URL?
---
swagger: "2.0"
info:
  description: Example for Issue Reporting
  version: "1.0.0"
  title: Example API
basePath: /v1.0
paths:
  /hello/{name}:
    get:
      summary: "Says Hello, $name"
      operationId: controllers.example.say_hello_name

Looking at the code, it seems that the Api is created as a Blueprint using basePath as the blueprint name. I've tried using the <blueprint name>.<route name> convention and it didn't work.
Examples tried:
v1_0.controllers.example.say_hello_name
v1_0.controllers_example_say_hello_name
The error I get back is:
werkzeug.routing.BuildError: Could not build url for endpoint 'v1_0.controllers_example_say_hello_name' with values ['name']. Did you mean '/v1_0.controllers_example_say_hello_name' instead?

It seems to work with the leading /, but why does it add a slash to the start overriding the normal way url_for works with Blueprints? http://flask.pocoo.org/docs/0.12/blueprints/#building-urls

@hello-josh I am also hitting the same issue. Did you find any solutions so far?

@hello-josh you're missing the initial slash, this will work: url_for("/v1_0.controllers_example_say_hello_name").

@hello-josh you're missing the initial slash, this will work: url_for("/v1_0.controllers_example_say_hello_name").
To inspect urls, just print app.url_map variable ;)

I don't have this code around any longer to verify, but I believe as stated that when I used a leading slash, that the URL was returned with two leading slashes. //v1...


Proccessors cyclic import

File-processors or any other sub package are not allowed to import procesdors, so we cannt use init ( processors.Register(p))
But maybe try breaking out Register to an own package?
ProcessorManager etc

ProcessManager didnt help seeing as our sub packages needs to have a func that return processor.Processor.
Dereferencing processor also dsnt work.

Next test should be Adding a
Func new() interface{} to Processor interface
Then in processormanager run
original, ok := entity.(Processor)
if ok {
println(original.b())
}
To cast into Processor

processmanager now accepts a function that is
new() interface,
Its not part of the proccessor interface to not force allocating a new proccesor .
Instead its up to each processor to come along with a
New() interface function that can be used wihout first creating a processor


fix: make python3 dep just a default

When trying to build a package that had python3 = nixpkgs.python311, I got an error similar to:
       Definition values:
       - In `/nix/store/...-source/nix/modules/dream2nix/default': <derivation python3-3.11.4>
       - In `/nix/store/...-source/modules/dream2nix/core/paths': <derivation python3-3.10.12>
       Use `lib.mkForce value` or `lib.mkDefault value` to change the priority on any of these definitions.»; }

This seems like a sane usage, so I'm expressing here that what dream2nix adds is just a default.

Oh, it seems I got confused and should have added python as the dependency, not python3: https://github.com/nix-community/dream2nix/blob/77eeb06927e94bc196444fcee2d08a20514443a3/modules/dream2nix/pip/default.nix#L9
Maybe this could be closed then.

Hm, maybe we should refactor the whole thing to use python3, not python what do you think?

I think it's better to use python, not python3, because:

You may want to build a python2 package.
Some day, you may want to build a python4 package.

So, the fact you use python for the pip backend is better IMHO.
The fact you're using python3 for path module also makes sense, because it's the supported version: https://github.com/nix-community/dream2nix/blob/5a729cb75ae914b2e97114e1c0e9280953b7b488/modules/dream2nix/core/paths/find-root.py#L1
So, IMHO the better fix would be to package find-root.py separately, so you add that as a dependency, and not its python interpreter directly. Then we only have one python variable around, and it's the one for building python packages.

@mergify rebase

I can't rebase your PR's. Mind giving me permissions?

Sorry, I can't. Github doesn't allow that when the fork belongs to a team, like this one. Let me rebase it.

I think this is fixed?
On current main branch, python3 gets a mkOverride-priority of 1003, which is even a bit higher than mkDefault - so can be easily overridden with a simple setting (with or without mkDefault/mkForce)


Fix harvester cloud credential migration when using non-existent token

Issue:  https://github.com/rancher/rancher/issues/47583

Problem

When attempt to migrate existing harvester credentials to add the rancher.io/expiration-timestamp annotation, if the cloud credential secret's kubeconfig contains a token which no longer exists, a NotFound error will be returned when attempting to get the v3.Token object.
Solution

Ignore not found errors during migrations (note: the error is still valid during a norman Update, since we don't want users to create invalid credentials, but we have no control over whether the token was already removed during migrations).
Testing

Tested Manually & with unit tests
Engineering Testing
Manual Testing

Updated existing unit tests to cover this case
QA Testing Considerations

N/A
Regressions Considerations

N/A
Existing / newly added automated tests that provide evidence there are no regressions:

Updated tests in store_test.go

Are there any side effects for the Harvester cloud credential that was skipped during migration due to its use of a non-existent token? One clear side effect is that the UI will not be able to easily display the expiration time according to rancher/dashboard#11332

The alternative is Rancher crashloops, and I don't think we settled on a sentinel time to indicate it already expired.

Are there any side effects for the Harvester cloud credential that was skipped during migration due to its use of a non-existent token? One clear side effect is that the UI will not be able to easily display the expiration time according to rancher/dashboard#11332

The alternative is Rancher crashloops, and I don't think we settled on a sentinel time to indicate it already expired.

Crashloop is too bad, and we need to avoid 😆

Merging as this is a release blocker for 2.9.3


Add editor option for automatically closing the output when stopping the game (2.1)

(back-ported from 1bd1af776c111f1efdd9a3b0259d8f916c052ef2)

I think it would be the best if that could be semi-automatic. If user had turn it on in editor then it should not hide automatically. If user hid it in editor then it should hide when game is stopped.

Well, this is just a backport of what's already merged for 3.0. So it's
done as long as it works.
Nonetheless, your suggestion can be studied as a separate feature
request. But I'm not sure I get what you're telling.
By "turn on" do you mean having the output pane visible or having the
setting enabled?
El 21/03/2017 a las 18:08, kubecz3k escribió:

I think it would be the best if that could be semi-automatic. If user
had turn it on in editor then it should not hide automatically. If
user hid it in editor then it should hide when game is stopped.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/8099#issuecomment-288148883,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALQCtrFlhq4kYGtoXTjTseJap4T1Xggqks5roAQcgaJpZM4Mjr1N.

I meant having the panel visible, I think that way we could maybe even get rid of additional setting since behavior would be based on what user was doing recently with this panel. Was not aware of the fact it's a backport from 3.0, just wanted to share what seemed the best from my perspective. Thanks!

That may be interesting. Please open a feature request issue so others can share their impressions.
So we can consider this one complete.


[Feature Request] Allow to rename an entry

Some applications have strange names like "Start.exe".
Please let me rename them to give them a better name (like "PortableApps").

In <IP_ADDRESS> you allow parameters, now renaming would be really cool.
I have 2 entries with the explorer icon and the "Explorer" title, would like to rename them to "Windows Updates" and "Windows Store".

In <IP_ADDRESS> you allow parameters, now renaming would be really cool.
I have 2 entries with the explorer icon and the "Explorer" title, would like to rename them to "Windows Updates" and "Windows Store".

In <IP_ADDRESS> you allow parameters, now renaming would be really cool.
I have 2 entries with the explorer icon and the "Explorer" title, would like to rename them to "Windows Updates" and "Windows Store".

In <IP_ADDRESS> you allow parameters, now renaming would be really cool.
I have 2 entries with the explorer icon and the "Explorer" title, would like to rename them to "Windows Updates" and "Windows Store".

@OLLI-S this has actually been added too in v<IP_ADDRESS>! But i forgot to add it in the changelog.
To change a name you simply click on it and you should be able to edit it. Let me know if it doesnt work as intended

@tjackenpacken Maybe you add a small "Edit" icon (pen icon) behind the name to indicate this feature?
Can you please re-open this issue?

@tjackenpacken Maybe you add a small "Edit" icon (pen icon) behind the name to indicate this feature?
Can you please re-open this issue?

@tjackenpacken Maybe you add a small "Edit" icon (pen icon) behind the name to indicate this feature?
Can you please re-open this issue?

I'll see what i can do!

@tjackenpacken Maybe you add a small "Edit" icon (pen icon) behind the name to indicate this feature?
Can you please re-open this issue?

I'll see what i can do!

Thank you!

Thank you!


auth.resetPassword throws error when providing options

When trying to provide an options object to auth.resetPassword like so:
auth.resetPassword({
  email<EMAIL_ADDRESS>
  option: { redirectTo: 'http://localhost:3000/en-ca/change-password' }
 })

I get a 400 error: Error validating request body. "options" is not allowed.
changePassword accepts params of type ResetPasswordParams
export declare type ResetPasswordParams = {
    email: string;
    options?: {
        redirectTo?: string;
    };
};

Looking at endpoint user/password/reset
router.post(
  '/user/password/reset',
  createValidator().body(userPasswordResetSchema),
  aw(userPasswordResetHandler)
);

userPasswordResetHandler accepts following body
type BodyType = {
  email: string;
  options?: {
    redirectTo?: string;
  };
};

And we can see in the code this: const { email, options } = req.body;
However, it seems that userPasswordResetSchema only accepts the email
export const userPasswordResetSchema = Joi.object({
  email: emailRule.required(),
});

I've created this PR to fix the issue: https://github.com/nhost/hasura-auth/pull/51

@alveshelio thank you! We will review the PR.

Thanks for posting because I just ran into this as well.

Thanks for your contribution, the PR has been merged


Add longer activity periods

Gitea version (or commit ref): 1.9.1

It would be great to have an activity overview for longer periods (yearly, all time) to get a feel for a projects scope over its full life time.
Was the maximum value of 1 month a performance consideration?

I would like an all time activity period please, I am not bothered about the performance of it, I also do not mind if it has to be enabled in a setting somewhere.
Thanks :)

This is already fixed by #8829. It added yearly period. All-time period would be performance overkill

This is not a fix. To reiterate @hamarb123:
Can we let users (i.e. admins of small personal instances) decide what a performance overkill is? i.e. make 1 month the default max, but give the option to specify higher time ranges.
(Also: To make this view scalable, we could compute & store its state on each push, instead of an expensive query. Though I'm not sure about the complexity-usefulness tradeoff here.)


build fails (due to missing resources?)

Trying to build v0.2.2 fails at :core:verifyReleaseResources:
Execution failed for task ':core:verifyReleaseResources'.
> A failure occurred while executing com.android.build.gradle.tasks.VerifyLibraryResourcesTask$Action
   > Android resource linking failed
     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:25: AAPT: error: attribute layout_constraintTop_toTopOf (aka ly.com.tahaben.core:layout_constraintTop_toTopOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:25: AAPT: error: attribute layout_constraintStart_toStartOf (aka ly.com.tahaben.core:layout_constraintStart_toStartOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:34: AAPT: error: attribute layout_constraintTop_toBottomOf (aka ly.com.tahaben.core:layout_constraintTop_toBottomOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:51: AAPT: error: attribute layout_constraintStart_toStartOf (aka ly.com.tahaben.core:layout_constraintStart_toStartOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:51: AAPT: error: attribute layout_constraintTop_toBottomOf (aka ly.com.tahaben.core:layout_constraintTop_toBottomOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:61: AAPT: error: attribute layout_constraintTop_toTopOf (aka ly.com.tahaben.core:layout_constraintTop_toTopOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:61: AAPT: error: attribute layout_constraintStart_toEndOf (aka ly.com.tahaben.core:layout_constraintStart_toEndOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:76: AAPT: error: attribute layout_constraintStart_toStartOf (aka ly.com.tahaben.core:layout_constraintStart_toStartOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:76: AAPT: error: attribute layout_constraintTop_toBottomOf (aka ly.com.tahaben.core:layout_constraintTop_toBottomOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:86: AAPT: error: attribute layout_constraintTop_toTopOf (aka ly.com.tahaben.core:layout_constraintTop_toTopOf) not found.

     ERROR: /build/repo/core/build/intermediates/merged_res/release/mergeReleaseResources/layout/usage_widget_preview.xml:86: AAPT: error: attribute layout_constraintStart_toEndOf (aka

Could it be something was not checked in to this repo?

@tahaak67 could you please take a look? ^^ (guess you've missed the notification for this)

I'll take a look later tonight

Ok so i just looked through the files and everything seems to be there.
i also cloned the project again and tried running it and building a signed apk and everything was successful, from what branch are you tring to build? if its the main branch can you please pull and try again?

from what branch are you tring to build?

From the commit the tag points to: 876c925dc0ec76d7073d5ca5d1bd8b9f7c1c49e6

if its the main branch can you please pull and try again?

I can of course. Though it would be strange if that suddenly would work out. If the resources were missing at that commit, how should they suddenly show up? But then, strange things happen (non-deterministic builds), so let's see… Nope, as expected, error remains.

ok so i took a deeper look and it seems the file usage_widget_preview.xml has some constriant layout attributes (i don't use constraint layout) so i removed thoes attributes and also fixed some missing dependencies in the end2end test, the changes are at 1f5f1cf please pull from main and try again

Thanks, now it builds! As the background of my question are Reproducible Builds (see Reproducible Builds, special client support and more at IzzyOnDroid), could you provide me an APK you've built and signed from the same commit to compare against – so we can see if we can establish RB for your next release, or what might need adjustment? Thanks in advance!

app-release.zip
here you go, i had to put it in a zip file because github doesn't allow .apk here

i had to put it in a zip file because github doesn't allow .apk here

It's MS, so it just goes by file extensions. Simply rename it to *.zip and suddenly it's fine :man_shrugging: But never mind, works this way as well.
But it's not RB.  That smells a bit like zlib-ng was used instead of zlib, but that cannot be as then all other files would be affected as well (tested: confirmed, it's not that). Still, I cannot even get the compression level of the file via zipfile. Not sure what happened to that font.
The PNG on the other side also differs when uncompressed. Could it be you use PNGCrunch or something like that? Those tools produce non-deterministic results. If so, for this I would have a hint. But if we cannot resolve the TTF file, that won't help much…
Do you have any ideas there? Maybe let's compare build environments: I've built the APK on Debian bookworm using JDK-17 and gradle at the command line. What did you use?

It's MS, so it just goes by file extensions. Simply rename it to *.zip and suddenly it's fine 🤷♂ But never mind, works this way as well.

aha, thanks for the info.

Do you have any ideas there? Maybe let's compare build environments:

I'll let you know if i have any ideas just kind of in a rush at the moment 😅
I built it with Android Studio 2024.2.1 Patch 3 using the "Generate signed app bundle / APK" tool and i have jbr-17.0.9 (jetbrains runtime) set as the JDK.

Generate signed app bundle / APK

Ouch. Not sure, but if that creates the APK from the bundle (which I'm afraid it does), that could be the culprit: those APKs are not RB. Is there an option to (first clean and then) only create the APK?

jbr-17.0.9 (jetbrains runtime)

that could also be an issue. if possible, can you select OpenJDK 17 instead?  That TTF still confuses me, being identical when uncompressed but differs in compression by 4 bytes. And I cannot determine its compression level even in your APK:
$ list-compression-levels *upstream.apk res/Ad.ttf
Traceback (most recent call last):
  File "/mnt/data/src/git/forks/obfusk/reproducible-apk-tools/list-compresslevel.py", line 144, in <module>
    list_compresslevel(args.apk, *args.patterns, levels=clevels)
  File "/mnt/data/src/git/forks/obfusk/reproducible-apk-tools/list-compresslevel.py", line 52, in list_compresslevel
    raise Error(f"Unable to determine compresslevel for {info.filename!r}")
__main__.Error: Unable to determine compresslevel for 'res/Ad.ttf'

$ list-compression-levels *unsigned.apk res/Ad.ttf
filename='res/Ad.ttf' compresslevel=9

("upstream" is yours, "unsigned" the one my builder produced). Guess if we cannot solve that one, trying to fix the PNG is moot (for RB at least, as we cannot reach RB if that remains different). I have no idea how to control the compression level with gradle (except that I know you can use aaptoptions to specify noCompress for file extensions you name) – so I'm rather out of ideas there. What OS are you building on? And would using e.g. Github workflows be an option for you? Of course we can gladly offer you using our "RB tooling": rbtlog can also be used to "just build, don't compare". It supports Podman and Docker containers for building (using "ephemeral" containers ensures you always have a clean build environment).

jbr-17.0.9 (jetbrains runtime)

that could also be an issue. if possible, can you select OpenJDK 17 instead? I know the option exists, but it could be you'd first need to download & install it (which AFAIK is possible from within Studio).

We just had the very same issues with another app, also using Jetbrains. Have you switched to OpenJDK meanwhile?


Clarified PIN config

Trying to fix:

drduh/YubiKey-Guide/issues/248

This is an attempt to unblock @andrewpmartinez's work on drduh/YubiKey-Guide/pull/249, and to add more details.

I'm fine. I've tried to get my PR merged twice I believe. Both times other work was done before someone who was capable of merging hid the merge button.


Update Meteor integration

With this update of the Meteor integration I've tried to integrate it more with the current environment of the project.
After a one-time setup, publishing a new release to Atmosphere will be as easy as pushing a new tag to the Github repository.
What you need to do is this.

Go to Autopublish
Login with github
Navigate to the list of your repositories
Activate autopublish for Vide
Add the user meteorpublish as a Meteor package maintainer with this command:

meteor admin maintainers vodkabears:vide --add meteorpublish

Done.
Before pushing a new release make sure to update the version number in the file package.js as you do in bower.json and package.json alike.
I've integrated Meteor testing to your Grunt workflow so Meteor testing will be run when running grunt test. spacejam is used for running Meteor tests.
I'll try and answer any questions you have as best I can.

:+1:

Thanks.

No problem. It's nice to have Meteor support which is maintainable in the original library instead of having a wrapper package. :smile:


Added query param text & made MLT/Faceting mixins more usable

Added these two features

URL param can be defined in the view itself. We need to define attribute facet_query_params_text on intended view to  replace selected_facets query text in urls.

Made MoreLikeThisMixin and FacetMixin more usable by removing dependency of viewset. MoreLikeThisBaseMixin and FacetBaseMixin can be used in regular views.

Coverage increased (+0.2%) to 88.54% when pulling 43ab12cc4e6fb8ba446f6c3d6037ad14f1b94303 on geekashu:master into 62bedeb5df13e01506a883a5f04f543a05da315e on inonit:master.

Great, I'll have a look at it tomorrow!
Thanks

Update..  I just realized that point 2 was unnecessary. Removing it... Thanks

Coverage increased (+0.06%) to 88.431% when pulling 9cf3ee7126d049d31947fff1cdd0aa7906642d52 on geekashu:master into 62bedeb5df13e01506a883a5f04f543a05da315e on inonit:master.

Coverage increased (+0.06%) to 88.431% when pulling 9cf3ee7126d049d31947fff1cdd0aa7906642d52 on geekashu:master into 62bedeb5df13e01506a883a5f04f543a05da315e on inonit:master.

Coverage increased (+0.06%) to 88.431% when pulling 9cf3ee7126d049d31947fff1cdd0aa7906642d52 on geekashu:master into 62bedeb5df13e01506a883a5f04f543a05da315e on inonit:master.

Coverage increased (+0.06%) to 88.431% when pulling 9cf3ee7126d049d31947fff1cdd0aa7906642d52 on geekashu:master into 62bedeb5df13e01506a883a5f04f543a05da315e on inonit:master.

Merged!


ML.NET Anomaly Detection Task output should include PredictedLabel

The Anomaly detection inputs and outputs section should have PredictedLabel in addition to Score.  Note: There is an issue with the current release of ML.NET where PredictedLabel always evaluates to true.
Additionally, the description for Score reads:

The non-negative, unbounded score that was calculated by the anomaly detection model.

This does not indicate how the value returned by the model for Score should be interpreted.  Should higher scores be interpreted as anomalies, or scores closer to zero?

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

ID: bd15175c-5b7d-4310-ecf5-32ffc7bc588b
Version Independent ID: 38adc676-934e-0ff9-c868-527f075c96df
Content: Machine learning tasks - ML.NET
Content Source: docs/machine-learning/resources/tasks.md
Product: dotnet-ml
GitHub Login: @natke
Microsoft Alias: johalex

Confirmed here, Predictions that are anomalies will have higher scores than those that aren't anomalies.

What it is essentially computing is 1 minus the ratio of the length of the projected vector and the input vector. We expect the higher the scores the higher the chance of an anomaly because in case of no anomaly the data point should be able to be projected without changing the length much.

Confirmed here, Predictions that are anomalies will have higher scores than those that aren't anomalies.

What it is essentially computing is 1 minus the ratio of the length of the projected vector and the input vector. We expect the higher the scores the higher the chance of an anomaly because in case of no anomaly the data point should be able to be projected without changing the length much.

Confirmed here, Predictions that are anomalies will have higher scores than those that aren't anomalies.

What it is essentially computing is 1 minus the ratio of the length of the projected vector and the input vector. We expect the higher the scores the higher the chance of an anomaly because in case of no anomaly the data point should be able to be projected without changing the length much.

Confirmed here, Predictions that are anomalies will have higher scores than those that aren't anomalies.

What it is essentially computing is 1 minus the ratio of the length of the projected vector and the input vector. We expect the higher the scores the higher the chance of an anomaly because in case of no anomaly the data point should be able to be projected without changing the length much.

Thanks @colbylwilliams. We will add this as part of the updates for  #13880

AB#1596627


Feat/use environment variables with default values

What kind of change does this PR introduce?
feature
What is the current behavior?
Server binds to hard coded address <IP_ADDRESS>:5000
Closes #217
What is the new behavior?
Uses the variables HOST and PORT for the address to bind to when provided. If not set, defaults to <IP_ADDRESS>:5000
See below, logs from the modified storage-api on fly.io

@sduduzog looks great!
There are 2 lint issues would you be able to run prettier on these files pls, will merge right after

Actually no worries, I'll fix the lints
Thanks a lot for the contribution


security-setup: on OSX bcrypt install fails without xcodebuild

Issue:  `sudo pip install -r requirements' fails with the following error if xcodebuild not installed.
Collecting pycparser (from cffi>=0.8->bcrypt==1.1.1->-r requirements.txt (line 6))
  Downloading pycparser-2.10.tar.gz (206kB)
    100% |################################| 208kB 1.8MB/s
Installing collected packages: pycparser, cffi, bcrypt
  Running setup.py install for pycparser
    error: could not create '/Library/Python/2.7/site-packages/pycparser': Permission denied
    Complete output from command /usr/bin/python -c "import setuptools, tokenize;__file__='/private/var/folders/fr/dt03ln8j66nblz0z3f0yf6540000gn/T/pip-build-qlcsoi/pycparser/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /var/folders/fr/dt03ln8j66nblz0z3f0yf6540000gn/T/pip-lZryvJ-record/install-record.txt --single-version-externally-managed --compile:
    running install

    running build

    running build_py

    creating build

    creating build/lib

    creating build/lib/pycparser

    copying pycparser/__init__.py -> build/lib/pycparser

    copying pycparser/_ast_gen.py -> build/lib/pycparser

    copying pycparser/_build_tables.py -> build/lib/pycparser

    copying pycparser/ast_transforms.py -> build/lib/pycparser

    copying pycparser/c_ast.py -> build/lib/pycparser

    copying pycparser/c_generator.py -> build/lib/pycparser

    copying pycparser/c_lexer.py -> build/lib/pycparser

    copying pycparser/c_parser.py -> build/lib/pycparser

    copying pycparser/lextab.py -> build/lib/pycparser

    copying pycparser/plyparser.py -> build/lib/pycparser

    copying pycparser/yacctab.py -> build/lib/pycparser

    creating build/lib/pycparser/ply

    copying pycparser/ply/__init__.py -> build/lib/pycparser/ply

    copying pycparser/ply/cpp.py -> build/lib/pycparser/ply

    copying pycparser/ply/ctokens.py -> build/lib/pycparser/ply

    copying pycparser/ply/lex.py -> build/lib/pycparser/ply

    copying pycparser/ply/yacc.py -> build/lib/pycparser/ply

    copying pycparser/_c_ast.cfg -> build/lib/pycparser

    running install_lib

    creating /Library/Python/2.7/site-packages/pycparser

    error: could not create '/Library/Python/2.7/site-packages/pycparser': Permission denied

    ----------------------------------------
    Command "/usr/bin/python -c "import setuptools, tokenize;__file__='/private/var/folders/fr/dt03ln8j66nblz0z3f0yf6540000gn/T/pip-build-qlcsoi/pycparser/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /var/folders/fr/dt03ln8j66nblz0z3f0yf6540000gn/T/pip-lZryvJ-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /private/var/folders/fr/dt03ln8j66nblz0z3f0yf6540000gn/T/pip-build-qlcsoi/pycparser

Workaround: sudo xcode-select --install
@stevendborrelli I'm concerned about adding complexity for new users that are trying to get started.  I almost wonder if we should revert back to installing bcrypt on the hosts ... thoughts?

I'm thinking about it.
Passlib has a native bcrypt implementation, but on my mac it takes about 52 seconds to run:
>>> t= timeit.Timer('bcrypt.encrypt("password")', "from passlib.hash import bcrypt")
>>> t.timeit(number=1)
51.58764600753784

Decent performance is going to require native code, which makes the install more complicated.
I think we need to revert the changes and run it on a host again.

it's pretty standard to have to set up xcode on a Mac to install pretty much anything. IMO It would be OK to just add a warning in the relevant places.

52 seconds is fine -- the xcodeinstall thing took nearly 5 minutes to download and install. Maybe just add a message saying "encrypting ... this will take about 1 minute ...". Security / encryption related tasks are perceived by many users to take a long time away. :+1:

It'd probably be possible to make a small wrapper function over it and then catch an ImportError if the native version isn't available.
What I meant by saying that xcode is a common task is that I would imagine that a lot of our users would have it installed already. If they've installed anything from Homebrew, it's the first step.

I reverted the change until we get a solution.
I think always using the pure python method is too slow. I like the fact that the script sets everything up quickly. The script also may be encrypting more than one password in the future.
The import/ImportError pattern is something we can do. Basically try to import bcrypt, and if it fails, put out a warning that native is being used and takes much longer to complete, with a solution of pip install bcrypt.
In order to use the pure python version the env. variable PASSLIB_BUILTIN_BCRYPT=enabled has to be set.

I like this idea.

@keithchambers looking at your original error again, that's a permission thing. I'm curious how installing xcode was the workaround there?

I posted the wrong error.  The error was something like this and I found the solution here.

@stevendborrelli do you want to move this out to a later release?

Moving to 0.3 for now.

We are still encrypting the password on the nodes themselves. Are we interested in trying @BrianHicks's suggestion of a try/catch import and doing something like this again? I'd be happy to set that up.

First-time setup can now be performed in Docker or Vagrant. Closing, please open another ticket if it's still an issue.


New Python Projects using Cookiecutter

May want to consider "archetype" functionality existing in mvn which allows a given archetype to auto generate a basic temple projects of a specific archetype
I think a good way of doing so is possibly using something like:

https://github.com/cookiecutter
https://github.com/cookiecutter/cookiecutter

From https://github.com/cookiecutter/cookiecutter

Documentation: https://cookiecutter.readthedocs.io/
GitHub: https://github.com/cookiecutter/cookiecutter
PyPI: https://pypi.org/project/cookiecutter/

May also think about http://springerle.github.io/py-generic-project/
For this use case it will involve:

Select File...New Project...Cookiecutter
In wizard, populate a list of available cookie cutter template types
Execute the necessary commands to cookie cutter to generate the project specific files.
Any post processing of the project for use in Python4NB project contexts

Possible examples of templates found on github by searching by topic https://github.com/topics/cookiecutter-template

https://github.com/TezRomacH/python-package-template


feat(action): bump default release and binary locations to v0.5.1

Hi, this change is mainly necessary, because the binary names changed in newer releases :)

thanks, but this repo will be archived soon
please use: https://github.com/puerco/release-actions/blob/main/setup-bom/action.yml


extend features to list of features for metrics with multiple types

This PR enables having multiple Features listed in a Metric for metrics that can have different kinds of inputs. On the first element or batch which feature is used is inferred from the data. It assumes that fields of each feature are the same but their values can differ (i.e. they all have the same column names).
This is related to #10 and #15.

Thanks @lhoestq! I actually added tests and forgot to commit them 😂


ngx-translate is not always working with ng-bootstrap accordion

Description
When using ngx-translate with accordion content inside accordion is not always translated when changing language.
Minimal Reproduction

Just open the stackblitz example. Change default language and expand accordion label and placeholders won't be translate. Actually sometimes they are translated sometimes not.. Just play with dropdown change and expanded/collapsed state of accordiion.
https://stackblitz.com/edit/angular-6pupmw
Your Environment

Angular version: 9.1.3
Formly version: 5.6.0

Additional context
Add any other context about the problem here.

@aitboudad  I see you pushed fix for this, thanks! I have one more question. I want to translate validation messages instead of hard code strings. Currently I have separate file validation.messages.ts where i add my custom messages and just register them in validationMessages array. How can i translate content in those messages? Can I reuse translate.extension.ts file somehow or? Stackblitz is updated:
https://stackblitz.com/edit/angular-6pupmw

This issue has been fixed and released as part of v5.6.1 release.
There is one limitation to the tab label where it should be translated inside AccordionFieldComponent type: https://stackblitz.com/edit/angular-6pupmw

I want to translate validation messages instead of hard code strings.

check:

https://github.com/ngx-formly/ngx-formly/issues/2140#issuecomment-598253799
https://github.com/ngx-formly/ngx-formly/issues/2140#issuecomment-598268027

@aitboudad  I succeed with validationLoader. But one more question, sorry. I am using dynamic module where i create custom types, validation messages.. etc.. and then just import that module. But what i need is to 'combine' en.json files from dynamic module and from module for example when i am using this dynamic formly module. I have translate loader in dynamic module:
export function HttpLoaderFactory(httpClient: HttpClient) {     return new TranslateHttpLoader(httpClient, './assets/i18n/', '.json'); }
and en.json file in assets/i18n in this module.
But also in another module  i have en.json file in assets/i18n and with current code only from second module en.json file is loaded.

@aitboudad  I updated to 5.6.1 but it seems like it is not fixed... Again if i change language from dropdown, expand accordion translation is again not updated with chosen language. But now strange things happen also.. like for example  if i collapse accordion translation is changed back to previous chosen dropdown language...

I updated to 5.6.1 but it seems like it is not fixed..

Your stackblitz is not updated, just make sure to take account of the limitation mentioned in the comment above.

Also one more thing about validation messages. I succeed to implement with validation loader. But if message is shown and i change language message is not translated with newly chosen language.

Could you please include the code in your stackblitz in order to fix both issues at once.

@aitboudad Can you please check the stackblitz now:
https://stackblitz.com/edit/angular-6pupmw
Issue 1: Just focus test input and leave it empty. This field is required message will be shown. Change language to 'fr'. Message is not translate to 'fr' it is still in 'en'.
Issue 2: I added | translate to ngb-panel tab itself. But there are still issues. Do this: expand accordion and change language to 'fr'. Everything is translated fine (sometimes not...). Now collapse accordion and change language to 'en'. Main accordion label is not translate it is still in 'fr'.
Let me know if you need some more clarification. Thanks!!!

Issue 1: I'll try to find a solution when time allows :pray:!
Issue 2: for the Main accordion, the translation should be done in the component template instead of extension:
skip translate for the accordion label and use translate pipe:
<ngb-panel [title]="tab.templateOptions.title|translate">

Thanks for speedy response @aitboudad !!! Looking forward for issue 1 solution 👍
Btw does this mean that for custom components that are more complex like accordion or maybe tabs we should use translate pipe instead of translate extension? Thanks!

Btw does this mean that for custom components that are more complex like accordion or maybe tabs we should use translate pipe instead of translate extension?

When the accordion is collapsed, the fields inside accordion is destroyed which means unsubscription from the translate stream, so the solution is either to use  translate pipe for the accordion tab or rely on the parent field to get the label of each tab instead something like:
{
  type: 'ac-accordion-field',
  templateOptions: {
  	tabs: ['HOME.accordionParticipant']
  },
  fieldGroupClassName: 'row',
  fieldGroup: [...]
}

@aitboudad  got you. But for second approach i will need to add some condition in translate.extension.ts to check for this scenario... cause currently only  label and placeholders which are in templateOptions are translate. I think pipe translate will do for now :)

it'll be part of v5.7 release, I'll let you know once done (most probably will be released on Monday).

@Bukic v5.7 is released which resolve the validation message issue, I've added an example in https://formly.dev/examples/advanced/i18n-alternative:
The ValidationLoader is not needed anymore and can be removed.
export function registerTranslateExtension(translate: TranslateService) {
  return {
+   validationMessages: [
+     {
+       name: 'required',
+       message() {
+         return translate.stream('FORM.VALIDATION.REQUIRED');
+       },
+     },
+   ],
    extensions: [{
      name: 'translate',
      extension: new TranslateExtension(translate)
    }],
  };
}

Hey @aitboudad updated to 5.7.0 it seems to be working now!! Thanks! :)
One question. As you mention ValidationLoader can be removed now. I still want to keep my messages in separate validation.messages.ts file. How can i call them in validationMessages extension? For example:
validation.messages.ts file
export function requiredValidationMessage(translate) {     return translate.stream('validations.required'); }
form.extension.ts file
export function registerFormExtension(translate: TranslateService) {     return {         extensions: [             {                 name: 'translate',                 extension: new TranslateExtension(translate),             },         ],         validationMessages: [            // this works             {                 name: 'required',                 message():  return translate.stream('validations.required');             },            // this does not work             {                 name: 'required',                 message:  requiredValidationMessage             },         ],     }; }

export function requiredValidationMessage(translate) {
-    return translate.stream('validations.required');
+    return () => translate.stream('validations.required');
}

        validationMessages: [
            {
                name: 'required',
-                message:  requiredValidationMessage
+                message:  requiredValidationMessage(translate)
            },
        ],

Yeah i asked  the question in wrong way...  This will work and
return (err, field) => translate.stream('validations.required');

still returns me err and field itself so all good 👍


Filter search can be very slow

In big jobs with a large number of objects the search works slowly. It is related with the need to interpolate objects for each frame. Probably there is faster approach.

Suppose this issue has been resolved in new UI


fix bug with rmi multiple tag

fixes #11414 once and for all
this code desperately needs to be refactored, but for now here is the fix and a new test, I will refactor after ;)

LGTM if janky is good.

@jimmyxian can you please make sure swarm follows the same behaviour ? Thank you :smile:

LGTM

:heart:  thanks Jess!

@vieux Sure, I will check it  :)

@vieux I have checked it. It's OK. swarm can follow this.

cherrypicked


Add authorization

Blog app: add authorization rules:

In this project, you will add authorization to your app using CanCanCan.
The following were implemented in this PR as the project requirements using pair programming.

Installed CanCanCan as a gem .
Added a role column to the users table
A user can delete a post if it is theirs or if they have an admin role (column role has value "admin"). Used CanCanCan for this authorization.

Implemented the post deleting functionality. Add the "Delete" button to the view and make sure that only authorized users can see it.
A user can delete a comment if it is theirs or if they have an admin role (column role has value "admin"). Used CanCanCan for this authorization.
Implemented the comment deleting functionality. Add the "Delete" button to the view and make sure that only authorized users can see it.

[x] Required: Using the gem cancancan as mentioned above, this file should be protected against unauthorized posts. Kindly consult the cancancan documentation.
[x] The above requirement requested by the previous reviewer has been implemented.

@brytebee Thank you for your constructive feedback

@sja-thedude Thanks for your approval.


StringRenderer gets no update

Hi Andrew,
thanks for doing this, it's a nice start for me doing stuff in Unity and VR!
I think I found a small bug. I'm using Unity 2020.3.9f1 with XR Interaction Toolit 1.0.0.-pre.4.
The StringRenderer get no update for the middle-transform. I think something like
private void UpdateColor(Vector3 pullPosition, float pullAmount)
{
    **middle.position = pullPosition;**
    // Using the gradient, show pull value via the string color
    Color color = pullColor.Evaluate(pullAmount);
    lineRenderer.material.color = color;
}

has to be added. Then the string moves, but the arrow does not go off. It just sticks after releasing the trigger.
Regards

I have the very same issue on Unity 2020.3.26f1 with XR Interaction Toolkit 2.0.0.


Black formatting

WIP. Fixes #15
Added black formatting, pre-commit & black config file and updated travis file.
I would need some information about the coding format for this project before I apply the formatting via black.

I am going to add support for auto-formatting with black and isort based on this once I've figured out how likely my VPS is going to handle installing Python 3.6 (black requirement) as I just noticed it still runs on 3.5.


node.js version example CLI

remove last empty lines of node.js/index.js plz

done

remove inquirer to minify the example

Test by @nella17, thanks for that


Test by Sam Dulam

Before you start please confirm the following.

[X] Yes, I've searched similar issues on GitHub.
[X] Yes, I've checked whether this issue is covered in the Portainer documentation or knowledge base.

Problem Description
Test
Expected Behavior
Test
Actual Behavior
Test
Steps to Reproduce
1,2 3
Portainer logs or screenshots
Test
Portainer version
2.22.0
Portainer Edition
Business Edition (BE/EE) with 5NF / 3NF license
Platform and Version
Docker Swarm 24.0.3
OS and Architecture
Ubuntu 22.04, AMD64
Browser
No response
What command did you use to deploy Portainer?
No response
Additional Information
No response

Test


Importer[period] should be a number and not a string

Current status

From the README.md file the period can be written as 60s to express "60" seconds.

http POST localhost:8080/api/v1/importer/redhat-csaf csaf[period]=60s

The list of importers at http://localhost:8080/api/v1/importer can return period=1m

[
    {
        "name": "redhat-csaf",
        "configuration": {
            "csaf": {
                "disabled": false,
                "period": "1m",
                "source": "https://redhat.com/.well-known/csaf/provider-metadata.json",
                "v3Signatures": false
            }
        },
        "state": "waiting",
        "lastChange": "2024-04-16T06:32:13.941898Z",
        "lastRun": "2024-04-16T06:32:11.428871Z",
        "lastError": "Visitor error: Invalid signature: Invalid key: \"Subkey of 77E79ABE93673533ED09EBE2DCE3823597F5EAC4 not bound: No binding signature at time 2024-04-16T04:20:22Z\""
    }
]

Problem
There is no unique way of expressing period.
period should be a number (seconds, milliseconds or whatever else) as an Input and Output. The Clients (e.g. UI) and Backend should then translate the number to the format of their preference.

Things like duration should always have a unit. There are so many cases where a missing unit leads to unexpected results.

I agree, duration should always have a measurement unit

I was suggesting to have one and only one unit which would be for instance milliseconds (a number and not a string).
If having multiple measurement units is better, then can we document which are the units supported?

Right however. that doesn't carry that information in the value: timeout: 1000 … what unit is it? timeout: 1000s is pretty clear.
Bugs this his have happened so many times. Specifically stating the unit solves this.

No problem. Can we document which are the supported suffixes to refer to measure units?

    nsec, ns -- nanoseconds
    usec, us -- microseconds
    msec, ms -- milliseconds
    seconds, second, sec, s
    minutes, minute, min, m
    hours, hour, hr, h
    days, day, d
    weeks, week, w
    months, month, M -- defined as 30.44 days
    years, year, y -- defined as 365.25 days

I'm going to close this PR.

On input data: the REST apis should accept any of the formats mentioned above.
On output data: I expect the server to be predictable and consistent when using formats. I expect the output to use any of the units s,m,h,d,w,M,y. So if period="50 seconds" then the server should always return 50s and never 50sec nor 50seconds.


Add CentOS 6 support

This commit adds a control recipe for CentOS 6 audits based of off v1.1.0 - 03-02-2015 from CIS. Additionally, the .kitchen.yml file has been updated for platform specific testing.
Specific chages include:

Add recipes/centos6-110.rb
Update .kitchen.yml to support platform specific testing, which excludes other platforms- Update the README to show support for CentOS 6

Super :+1:

:+1:

:+1:

https://github.com/chef-cookbooks/audit-cis/commit/4667640002c5fafef306237d0ea1541d945be251


core: duplicate symbols from libevent

Similar to https://github.com/lyft/envoy-mobile/issues/617, we're seeing duplicate symbols in the final Envoy.framework/Envoy binary coming from libevent.

Proposed upstream PR to fix: https://github.com/envoyproxy/envoy/pull/10002
Started Slack discussion: https://envoyproxy.slack.com/archives/CNAK09BSB/p1581403500006600


Add export, import settings

It would be great if we could export all settings, shortcuts, snippets, extensions in one step and import them in another machine.

fyi @bpasero @joaomoreno

Can this be made way that it actually is integrated with git?
What VsCode requires to integration may be user to set "git repository for user settings", and whenever user wants they can push changes to github and they can be pulled (syncronized in other locations).
I can currently do this manually because VSCode relies on json files (which is good thing), however some tooling may make this kind of syncronizing killer feature.
Why i propose this is truth that in environments where export/import is made by hand (create file -> import file), that is often actually syncronized hard way (via email for example). Its true that that file can be placed under version control (where it should be), however then cycle is following:

Export file to version control location and overwrite existing.
Add, Commit, Push etc.
Pull etc.
Import.
Question is, should this really be that hard?

Git integration makes syncronizing environments between team easy if required (however because workspace settings exist in vscode this may be rare case). If you want to use common settings just use master, if you want to modify it based on master and use it as base, just fork or branch it and merge following changes whenever you want and configure VSCode to use your branch instead.

Comment so I'm participating.

This would be very useful

How can I export all settings to another machine now?

Duplicate https://github.com/Microsoft/vscode/issues/2743

Reopened this to treat this as a separate settings sync request

@sandy081 but we already have a way of doing this now, why add another way that no one has asked for yet and even fewer people would use now?

@Tyriar Some users asked for it in #2743. I recommended to open new issue instead of polluting the original one. - https://github.com/microsoft/vscode/issues/93572 (Since I did not know this issue already exists).
Now that I have seen this request, I reopened this. We shall not mark this as dup of #2743 as they are different requests.


script-setup: Types need to be imported separately with import type ....

Version
3.0.5
Reproduction link
https://github.com/JSerFeng/bug/blob/master/src/App.vue
Steps to reproduce

What is expected?
nothing happens
What is actually happening?
Uncaught SyntaxError: The requested module '/node_modules/.vite/vue.js?v=6cc5afeb' does not provide an export named 'Ref'

It's a Bug in the compiler that doesn't differentiate between type exports and runtime exports.
As a workaround, you can use import type { Ref } from 'vue' to import types explicitly as types.

Is it related to vite? https://github.com/vitejs/vite/issues/1661

Don't think so, the issue talks about runtime exports (unref, openBlock).
There were also comments about the problem with Type exports in the RFC comments that found the same solution/workaround:
https://github.com/vuejs/rfcs/pull/227#issuecomment-766427503

Oh, the code it generates looks like this:
import {Ref} from 'vue'

export default {
  expose: [],
  setup(__props) {
    return { Ref }
  }
}

But I can't find any way other than list all types through enumeration, or we can consider imports beginning with capital letters as types, but exclude such things as Teleport/ErrorCodes.

Yeah, don't think that would be a reliable heuristic. I don't think the compiler can reliably determine wether or not an import is a Type or a runtime object, so in theory we would have to clean up the imports with Ts somehow before generating the code.
No idea if something like this is even feasable, not really familiar enough with compiler details.
Should we consider this an enhancement rather than a bug as import type works for now, and the RFC is still in flux?

Should we consider this an enhancement rather than a bug as import type works for now, and the RFC is still in flux?

Agree

see

https://github.com/vitejs/vite/issues/1266
https://github.com/vitejs/vite/issues/731
https://github.com/evanw/esbuild/issues/341#issuecomment-678661170
https://github.com/evanw/esbuild/issues/314#issuecomment-668401819

This workaround breaks eslint-prettier, so weird.

It's a Bug in the compiler that doesn't differentiate between type exports and runtime exports.
As a workaround, you can use import type { Ref } from 'vue' to import types explicitly as types.

how exaxctly?

@LinusBorg
The problem is I didn't install prettier as dependency.
(I thought that eslint-plugin-prettier has the dependency)
Everything is okay right now, sorry for that.

For Types the import type solution worked for me. But I cant get interfaces working. I always get the warning, that the interface is not exported.
"export 'Consultation' was not found in '@/libs/shared/interfaces/consultation'

I think compiler has no a perfect way to solve this problem, but IDE already report error for this use case, so maybe we don't need to do anything in compiler side.

This impacts types, which you can work-around by using import type; and const enums, which you can't work around in a practical way (both import and import type create errors, in different places).
One solution to this problem could be to check what is actually used by the template and not export identifiers that it doesn't use.

Not sure if that would fully work with HMR?
It's more work as it would require analyzing the globals of every expression in template, but at least it's a local solution to the problem (i.e. no need to do cross-files type analysis to figure out what is a type).
It only partially solves the const enum problem: now you can import them and use them in code, but you still can't use them in template.

I believe none of this is an issue when building for production and the template becomes a lambda inside the setup code, which simply has local scope automatically captured.
That's actually great and the best solution, the only issue is that the other way is used in dev to enable HMR / exposing state to dev tools. I wonder if we could figure out a way to do those things with the lambda design instead... (I don't have an idea just yet 🤔)

@yyx990803 it's fantastic you could fix that!
On top of what was already said above, it really messed up with VS Code auto-complete.
When you try to auto-import a type, VS Code always adds it to an existing plain import and you must go back and manually split things in two imports.


Fix security_enable_chrony: no not work issue

Crony package is installed no matter security_enable_chrony is yes or
no, which will remove ntp package automatically.

Thank you for contributing to openstack/openstack-ansible-security!
openstack/openstack-ansible-security uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Enable master branch protection

Would prefer to wait to merge rather than accidentally break CI. What do you think?
Experience from $other projects suggests I'm too impulsive to manage a reliably working master without assistance :)

A prerequisite for this is probably to set up working CI and to actually setup some tests of the server.
I'm not sure how to properly do feature/integration tests of the server but a simple CI that checks everything compiles and runs the unit tests is doable.
Perhaps we should migrate this repo to airmash-refugees and setup the infrastructure there?

I'm ambivalent about putting your nice clean roomesque effort in the same GitHub org as the copyright frontend. Really doubt anything bad will happen, but one DMCA might be enough to make your repo unavailable forever.
On the other hand, having a single GitHub org for everything really helps show the project is not dead, and give some sense of identity that others might want to get involved with.
Your call :) Quite happy either way. (Also quite happy if the name of the org/avatar/etc are bikeshedded). The -eu fork that's already in there could also be renamed and just remain a perpetual mirror of your real repo, where I could work from a branch. It might afford some protection in the case of a DMCA request

I think having a single github org for everything outweighs the potential downsides. In addition, we can keep my copy as an up-to-date mirror which should hopefully protect against DMCA requests.
Aside: IANAL and I don't know much about the DMCA laws but aren't DMCA requests restricted to the repo that would actually be infringing?

I've enabled branch protection for the master branch. From now on everything needs to be approved before it can be merged into master.


comments are good

@ari23 \o/

Also @zeeshanlakhani I just tested with ping4d and it works :) I enabled EchoReply and EchoResponse for ipv4 (which are currently commented out). Just wanted to let you know that in case you want to enable them in code.

Also @zeeshanlakhani I just tested with ping4d and it works :) I enabled EchoReply and EchoResponse for ipv4 (which are currently commented out). Just wanted to let you know that in case you want to enable them in code.

@ari23 commented out? If any code were, it wouldn't compile? Maybe point me to where? If so, I'll def fix that.

These guys https://github.com/williamofockham/nb2/blob/master/core/src/packets/icmp/v4/mod.rs#L278

@ari23 derp. good catch.

(also https://github.com/williamofockham/nb2/blob/master/core/src/packets/icmp/v4/mod.rs#L258) I am sure you would have caught it, but just in case ;)

thanks @ari23


Question about training custom dataset

I want to train the model to track one object.
My dataset has 2 part. 1.image folder :images. 2.train.json, which can be correctly uesd in CenterNet.
python main.py tracking --exp_id coco_tracking --dataset custom --custom_dataset_ann_path ../data/Cap2/train.json --custom_dataset_img_path ../data/Cap2/ --input_h 960 --input_w 1280 --num_classes 1 --pre_hm --ltrb_amodal --same_aug --hm_disturb 0.05 --lost_disturb 0.4 --fp_disturb 0.1 --gpus 0
1.KeyError: Caught KeyError in DataLoader worker process 0.
2./cls_id = int(self.cat_ids[ann['category_id']]) KeyError: 0
I set the num_workers in opt.py to 0. But still meet question 1.
And I have no idea how
Thank you for your reply : D

I find that if I input one image, it works. However, if input a image folder, it shows this problem.

I find that if the number of images in the folder is less than 34, the training part works. However, if the number of images in the folder is large, it still has the problem

I find that when I modify this value, and the number of training images it smaller than default value, it works.
self.parser.add_argument('--batch_size', type=int, default=32,
help='batch size')

thanks for your solution! when you train with your own dataset, are there any modify in the code?


The aws-auth submodule for the aws/eks module no longer validates

In the terraform-aws-modules/eks/aws module version 19.17.2 this worked...
module "eks" {
  manage_aws_auth_configmap = true
  aws_auth_roles = [
    {
      rolearn  = data.aws_iam_role.karpenter_instance.arn
      username = "system:node:{{EC2PrivateDNSName}}"
      groups   = ["system:bootstrappers", "system:nodes"]
    },
  ]
  aws_auth_users = var.eks_additional_users
}

But with version 20.14.0 I get...
 Error: Unsupported argument
│
│   on eks-cluster.tf line 90, in module "eks":
│   90:   manage_aws_auth_configmap = true
│
│ An argument named "manage_aws_auth_configmap" is not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on eks-cluster.tf line 91, in module "eks":
│   91:   aws_auth_roles = [
│
│ An argument named "aws_auth_roles" is not expected here.
╵
╷
│ Error: Unsupported argument
│
│   on eks-cluster.tf line 98, in module "eks":
│   98:   aws_auth_users = var.eks_additional_users
│
│ An argument named "aws_auth_users" is not expected here.

But checking the docs for aws-auth, the example seems to show that this should still work.

that is not true - its no longer in the root module, you need to define a new sub-module definition to use that

First, thank you for the response. The docs should probably be updated... And not to be too dense, but by submodule definition, I should insert something like...
module "eks" {
  aws_auth = {
    manage_aws_auth_configmap = true
    aws_auth_roles = [
      {
        rolearn  = data.aws_iam_role.karpenter_instance.arn
        username = "system:node:{{EC2PrivateDNSName}}"
        groups   = ["system:bootstrappers", "system:nodes"]
      },
    ]
    aws_auth_users = var.eks_additional_users
  }
}

...because I make my way through these things by documentation as opposed to actually knowing what I'm doing.

I figured out the module declaration. Thnks.

@zenbones what's the proper module declaration? I'm experiencing the same issue.

Whether something is true or not it is complex topic, but I can say that I found the same issue. I understand that this is a change in the contract of the module, and as such it should be treated gently.
For other humans arriving here, taking the original example that opened the issue:
module "eks" {
  manage_aws_auth_configmap = true
  aws_auth_roles = [
    {
      rolearn  = data.aws_iam_role.karpenter_instance.arn
      username = "system:node:{{EC2PrivateDNSName}}"
      groups   = ["system:bootstrappers", "system:nodes"]
    },
  ]
  aws_auth_users = var.eks_additional_users
}

We need to move all auth related things to a new module definition, like
module "eks" {
   source = "terraform-aws-modules/eks/aws"
   cluster_name = "...."
   # All the rest of cluster configuration that was not moved
}

module "aws_auth" {
  source = "terraform-aws-modules/eks/aws//modules/aws-auth"
  manage_aws_auth_configmap = true
  aws_auth_roles = [
    {
      rolearn  = data.aws_iam_role.karpenter_instance.arn
      username = "system:node:{{EC2PrivateDNSName}}"
      groups   = ["system:bootstrappers", "system:nodes"]
    },
  ]
  aws_auth_users = var.eks_additional_users
}

But if this problem is coming from an upgrade, you might find the following links helpful to find out what else is failing and for a little bit of context:

https://github.com/terraform-aws-modules/terraform-aws-eks/blob/master/docs/UPGRADE-20.0.md
https://github.com/clowdhaus/terraform-aws-eks-migrate-v19-to-v20
https://github.com/aws-ia/terraform-aws-eks-blueprints-addons/issues/389
https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/a66db17ae26f0683f61314e8142922e6f3c4f124/patterns/karpenter/main.tf#L160


QUESTION: How to represent a Component (dependency) with multiple download formats?

It's commonplace in the Python ecosystem for a given package version to be available in both whl and tar.gz formats (and possibly others.
When we define a Component in a CycloneDX schema, this Component can include a list of hashes that provide an integrity to the definition. This is complicated in the Python eco-system as we do not know? which format/download has been used.
For example, in a Pipfile.lock for toml==0.10.2 you will see two hashes which correlate to both the whl and tar.gz formats:
"hashes": [
    "sha256:806143ae5bfb6a3c6e736a764057db0e6a0e05e338b5630894a5f779cabb4f9b",
    "sha256:b3bda1d108d5dd99f4a20d24d9c348e91c4db7ab1b749200bded2f839ccbe68f"
],

Including both hashes may be the correct answer, but I'm seeking clarification from the @CycloneDX/core-team prior to implementation.

Include the hash for the component that represents reality.
If there are multiple distributions of a component, then I'd recommend using the distribution external reference type. Refer to https://cyclonedx.org/use-cases/#external-references. Each distribution can have its own set of hashes.


Fix character class range matching

Previously we performed a lexicographic comparison with the bounds of a character class range. However this produced surprising results, and our implementation didn't properly handle case sensitivity.
Update the logic to instead only allow single scalar NFC bounds. The input is then converted to NFC in grapheme semantic mode, and checked against the range. In scalar semantic mode, the input scalar is checked on its own. Additionally, fix the case sensitivity handling such that we check both the lowercase and uppercase version of the input against the range.
Resolves #401
Resolves #395
rdar://96898279

@swift-ci please test

Hrm, I thought the macOS CI would be new enough by now

@swift-ci please test

So it turns out the macOS CI does have a new enough toolchain, it's just that when testing through swift test with a development toolchain, the OS stdlib is used. This differs from when testing within Xcode where it seems dyld is told to use the toolchain stdlib instead. Seems like it would be nice to figure out a way to get the macOS CI using the toolchain stdlib, but for now let's guard certain tests against older stdlibs.

@swift-ci please test

Splitting off the _CharacterClassModel work into https://github.com/apple/swift-experimental-string-processing/pull/578

We talked about this, and we want to do the following errors. Workaround for 1-2  is to use a scalar escape, which is clearer and more explicit anyways and doesn't have the bug potential (specially since copy-past might normalize to NFC on them, etc)

Parse-time: error for any non-NFC literal content (using old stdlib SPI)
Parse-time: error for any literal multi-scalar custom character class range bound
Run-time compilation: error for any (even escaped) multi-scalar custom character class range bound

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test


rcon command in auto mode not displaying correctly

Running >r to display the current server, will never show what server when running in auto mode.

Fixed issue, will be in latest commit.


Docs fixes for action invocations

More accurate description of blocking calls and the situation when application error is returned upon invoking an action.
Description

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[x] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[x] I updated the documentation where necessary.

Hey @rabbah @markusthoemmes , what do you think now? I've fixed the description a little bit in separate commits.

Thanks for improving the docs - this is important and appreciated.

hey @rabbah, wdyt now?

LGTM now - I’ll leave it to Markus to clarify the init vs run timeout.

@rabbah Clarified in the other PR, I don't see a reason not to make them both behave the same. One could argue that an action timeout is a form of an uncaught exception (as the action is aware of its own deadline). Making them both non-application errors makes sense to me.

@rabbah @markusthoemmes alright, thanks. So let's wait with this PR until the other one is completely sorted out. Then I can make a little change to this one if necessary.

hey, I've updated the description of errors in the event of timeouts, after https://github.com/apache/incubator-openwhisk/pull/3941 was integrated.


Clean up legacy global variables in Zulip javascript

The Zulip web codebase has an ongoing (and mostly complete) migration in process to eliminate global variables and replace them with calls into well-defined modules.
You can see our current list of globals in the sections of check-all.js that aren't lists of modules:
    // index.html
    + ' page_params'

    // common.js
    + ' status_classes password_quality'

    // setup.js
    + ' csrf_token'

    // zulip.js
    + ' all_msg_list home_msg_list narrowed_msg_list current_msg_list'
    + ' keep_pointer_in_view unread_messages_read_in_narrow'
    + ' respond_to_message recenter_view last_viewport_movement_direction'
    + ' scroll_to_selected get_private_message_recipient'
    + ' viewport process_loaded_for_unread'
    + ' have_scrolled_away_from_top'
    + ' maybe_scroll_to_selected recenter_pointer_on_display suppress_scroll_pointer_update'
    + ' fast_forward_pointer recent_subjects unread_subjects'
    + ' furthest_read server_furthest_read'

To clean one of these up, we basically just need to move the relevant functions/variables into either an existing module or a new one.  A good place to start for anyone interested would probably be the globals with unread in their name (except process_loaded_for_unread which is badly named) which should be moved into unread.js or moving the pointer-related code into a new pointer.js module; I'm happy to post more thoughts/details on how to do this if there's interest.
If you work on this, please do it with commits that are easy to read/review, e.g. moving a single function per commit, and please check carefully (e.g. using grep) that you've eliminate all references to the old identifier.  It's very common for bugs to be introduced in this sort of refactoring.

@timabbott Is it fine to move all_msg_list home_msg_list narrowed_msg_list current_msg_list to message_store.js ?

Hmm.  I think given _msg_list occurs 242 times in the frontend code, switching to super long names like message_store.home_msg_list would be pretty sad.  Thinking about this more, I think these probably belong in message_list.js, since that defines the MessageList type anyway.
If we do that, then we can use more abbreviated names, e.g. message_list.home, which is similar enough in length to the existing home_msg_list to not substantially impact the line-wrapping situation.
So I guess my proposal would be to move these to message_list.js.  Does that seem reasonable to you @hackerkid ?
If you work on this moving those, please do it as one commit per variable, change nothing other than moving the variable in each commit; use a tool (sed or your editor's find-replace tool) to do the replacements so there's no risk of typos, and use grep to confirm you got them all.  Since they are used everywhere, and frontend test coverage is incomplete, it's important to do this sort of refactoring very carefully.

@timabbott Sounds good. I will move them to message_list . Should I send unique PR for each commit?

I think 1 PR with 5 commits is probably best here.

@timabbott Sorry for the delay. Since the message_list.js is already exporting MessageList how do you want to handle the exporting of variables?

Hmm.  I think we don't use that export:
tabbott@zaset:~/zulip$ git grep message_list static/
static/js/compose_fade.js:exports.update_message_list = function () {
static/js/narrow.js:        compose_fade.update_message_list();
static/js/narrow.js:    compose_fade.update_message_list();
static/js/reload.js:function clear_message_list(msg_list) {
static/js/reload.js:        clear_message_list(all_msg_list);
static/js/reload.js:        clear_message_list(home_msg_list);
static/js/reload.js:        clear_message_list(narrowed_msg_list);
static/js/stream_list.js:    var recipients_dom = templates.render('sidebar_private_message_list',
static/styles/zulip.css:.message_list {
static/styles/zulip.css: .message_list .recipient_row {
static/styles/zulip.css:.message_list .stream_label {

So there's no real issue here.  I think it'd be good to make module.exports in message_list.js work how it does in the other files.
So maybe as a preliminary change, we should make message_list.js work like the other modules where we start by defining exports and adding elements to it, and access message_list.MessageList (rather than having MessageList be a global).  Seems like an overall valuable refactoring that wouldn't be a ton of work (and probably we should do the same thing to message_list_view.js).

@hackerkid it looks like we're almost done with cleaning these up -- let me know know if you need any help with this!

@hackerkid when you get a chance, can you clean up these ones?
    + ' home_msg_list current_msg_list'
    + ' respond_to_message'

Once those are resolved, we'll be very close to being able to eliminate zulip.js and its legacy globals.
(as a sidenote, the pointer-related code in main should be moved to static/js/pointer.js).

@timabbott Yeah. Sure. I had some trouble moving home_msg_list and current_msg_list last time. Will give it another try. :+1:

@timabbott To which module should I move respond_to_message

compose.js seems natural.

OK zulip.js now just has home_msg_list and current_msg_list.  We've made their friends narrowed_msg_list and all_msg_list globals, but I'm not sure we actually want to de-globalize those variables; if there's something that should stay as globals, it's them.  I opened https://github.com/zulip/zulip/issues/2231 for making a decision on that question; but given that leaving them as globals might be the right decision for the project, I think at this point it makes sense to close this issue as no longer having any value beyond #2231.


Fix unknown encoding

We were parsing some rtf files from users that were causing the "windows-10000" encoding to be set. This is not a valid Ruby encoding so the parsing was failing. With the code updates invalid encodings will be ignored.

Awesome, thank you.

Thank you for the super fast approval & merge :-)


add student mail domain for Guangzhou University

homepage: https://www.gzhu.edu.cn/
university name: Guangzhou University
address: 230 Wai Huan Xi Road, Guangzhou Higher Education Mega Center, Guangzhou 510006 P.R.China
The domain e.gzhu.edu.cn is provided as students email domain by Guangzhou University and the previous mail domain gzhu.edu.cn is available for teachers.
mail portal url: http://mail.e.gzhu.edu.cn

@LemonPrefect I am sorry, the domain e.gzhu.edu.cn is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

@LemonPrefect I am sorry, the domain e.gzhu.edu.cn is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


KOGITO-1899: BDD tests: Test travel-agency scenario in native mode

JIRA Ticket: https://issues.redhat.com/browse/KOGITO-1899
Description: Configure Travel Agency tests with native enabled
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[x] You've tested the new feature/bug fix in an actual OpenShift cluster

I've run the tests in quarkus native with the workaround and they worked fine.


Smooth scrolling!

I made scrolling perfect on my iPhone 5s without removing auto-layout.
I commented on every commit.
Basically the changes in order of their importance:

textView.scrollEnabled=NO causes scrolling problems for some reason, but it can be safely disabled because the layout takes care of that
NSDateFormatter optimized by having separate Date & Time formatters and not changing the style of a single formatter all the time.
make sure we don’t cause unnecessary work with setText=nil upon cell reuse
remove forced rasterization to unburden the CPU
move default dataDetector setting to textView.awakeFromNib instead of cellForRowAtIndexPath so library users can have better control

The biggest bottleneck left seems to be dataDetectors as they heavily block the main thread.
When testing this PR I advise you to disable them to see the smoothness achieved by these commits (without being affected by dataDetectors).
Maybe later I’ll come up with something to make that part faster as well.

Combined with TTTAttributedLabel (which has way faster rendering and does data detection on a background thread) I managed to eliminate ALL jankiness. Seriouosly, it flies.
I think it's a good call to switch to that for 8.0

Just opened a PR for TTTAttributedLabel that will make it fly even more with data detectors.
Just look at these number UITextView (1.662s) vs TTT (0.084s).

thanks @galambalazs!
wow - UITextView :cry:

Good news:
I managed to implement a version of TTTLabel which allows the native data detector actions to appear.
(Currently you'd have to implement them yourself, plus localize etc...)
Jesse: there are two quick demo projects and two videos in the smooth scrolling issue thread.
Please do take a look at the performance characteristics show in those.
i'm now 100% sure TTT is the way to go.
I'll probably break this PR up to separate small ones and 1-2 may be merged so 7.x branch gets somewhat faster. But 8.0 is TTT all the way.

Thank you @galambalazs
It works like a charm.

I've taken some of these quick fixes and tried to apply to my project. I would really like to see this go somewhere....

Still waiting for @jessesquires to decide what's the plan for 8.0
The biggest change would come from switching to TTTAttributedLabel
In that case, only the date formatter is relevant from here.
Maybe I'll add a separate PR for that's easy to merge.

Also you had a SPViewController  that was one error, couldn't get it to work in my project either.

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


Dead links in documentation

Awesome work on this project! Just a note: Navigating to the README, there's three "Getting started" links which are all dead.

It seems there's also other dead links. E.g. on the Introduction page the "callbacks" link points to https://fluxml.ai/FluxTraining.jl/dev/docs/callbacks/reference.md.html, which is dead.


Weird routes

https://routeplanner.bike.brussels/#13.72/50.8649/4.3152?sb=true&o=4.3220772,50.8698411&d=4.3106358,50.8581213&p=profile1

This is a problem with us completely excluding primary roads:
https://routeplanner.bike.brussels/#16.88/50.86180/4.31651?sb=true&o=4.3168491,50.8626573&d=4.3150098,50.8615379&p=profile1

In this case this road looks reasonable:
https://www.mapillary.com/app/?pKey=obClReV5QumT2gmApfe9jA&focus=photo&lat=50.86330801&lng=4.31749205&z=17

This is probably a duplicate of #32

Fixed by the primary roads allowed update


Fix Lookup component scrollbar height after open and closing menu

Note: this is not reproducible in Wikit, please reproduce in special-new-lexeme. Instructions in the phabricator ticket.
Bug: T325822

I reproduced this error in storybook on master (with wrapping the added results in a window.setTimeout()) and confirmed that it works as expected with this change.

Super! It didn't occur to me before to test that way ,thanks for the tip.


updates to article

@danielsollondon : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@RicksterCDN - can you approve these please


[Live Trains] Whitlocks End via Whitlocks End

When using Live Trains at Birmingham Moor Street, services to Whitlocks End are announced and displayed as Whitlocks End via Whitlocks End.

This is a limitation of the Darwin data feed caused by WMT inputting garbage.
The announcement system can only be as good as what the operators provide, unfortunately.


build not successful failed to create bus connection

Hello,
trying to install and I'm getting the following error:
timezone from host: America/Los_Angeles
Failed to create bus connection: No such file or directory
error while setting timezone
build not successful
stopping & removing build container ...
fmsinstall
fmsinstall

Hello,
is it possible the error message says it can't create the d-bus connection?
Which OS are you using?

Thank you, MacOs 10.15.7

On Dec 20, 2021, at 12:55 AM, S @.***> wrote:

Hello,
is it possible the error message says it can't create the d-bus connection?
Which OS are you using?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Alright, that rules out problems based on Big Sur and up. And the FMS version? Most recent on Ubuntu?
It looks like timedatectl can not be executed correctly inside the container. For that, elevated permissions and the /run directory are necessary, which should be provided by the install script that starts the container. Are there any other errors prior to this one by any chance?

Okay, i get the same error. Something has changed with Ubuntu I suspect. I'm gonna look into it.

Thank you.  It was working fine and the recent Docker update broke it.
MacOs: 10.15.7
Docker: Version 4.3.1 engine 20.10.11
Tried to reinstall with no luck and tried the following FMS:
filemaker-server_<IP_ADDRESS>_amd64.deb
filemaker-server_<IP_ADDRESS>_amd64.deb
filemaker_server-19.2.1-23.x86_64.rpm
Here is more info on the error:
Current default time zone: 'Etc/UTC'
Local time is now:      Mon Dec 20 01:47:54 UTC 2021.
Universal Time is now:  Mon Dec 20 01:47:54 UTC 2021.
Run 'dpkg-reconfigure tzdata' if you wish to change it.
Failed to create bus connection: No such file or directory
error while setting timezone
build not successful
stopping & removing build container ...
fmsinstall
fmsinstall
-Alex
From: S @.>
Date: Monday, December 20, 2021 at 1:45 AM
To: fmgarage/ft-fms-docker @.>
Cc: Alejandro Yescas @.>, Author @.>
Subject: Re: [fmgarage/ft-fms-docker] build not successful failed to create bus connection (Issue #12)
Okay, i get the same error. Something has changed with Ubuntu I suspect. I'm gonna look into it.
—
Reply to this email directly, view it on GitHubhttps://github.com/fmgarage/ft-fms-docker/issues/12#issuecomment-997761873, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACK2GMJKXJJ5WUNZUI2IV6TUR33NFANCNFSM5KMS2DSQ.
You are receiving this because you authored the thread.Message ID: @.***>

It seems to me that running a Docker container with a systemd environment inside is not possible any more, at least in the way we did that. I suspect one the recent versions of Docker (Desktop). The root problem seems to be that the cgroup volume that is mounted into the container and that is needed by systemd does not work the way it did anymore. Systemd can't be started and so all services started by systemd itself. I'm not sure at this point if this can be fixed.
See also:
https://github.com/systemd/systemd/issues/19245
https://serverfault.com/questions/1053187/systemd-fails-to-run-in-a-docker-container-when-using-cgroupv2-cgroupns-priva/1054414#1054414
https://github.com/moby/moby/issues/42275

I tested with Windows WSL and Linux (Debian 10) and these still work. I guess because they can provide /sys/fs/cgroup from the host and are still on cgroup v1.

closed with https://github.com/fmgarage/ft-fms-docker/commit/bfec8bb04ea28121c1006b07569043ae18ab3b76


Enable Bitcoin Cash Payments (WIP)

:construction_worker: Purpose
Enables BCH as a supported crypto payment method. This will also lay 75% of the foundation needed to accept BTC as a supported crypto payment method, so it is essentially killing 2 birds with one stone.
:rocket: Changes

Enable coinmarketcap price lookups of BCH
Remove function validateBlockchain that was only being used in a test within the v2 api.

:warning: Breaking Changes
None

In order to successfully handle payments, each payment creation will need to use a unique address. This will involve leveraging bchwallet to create handle management of a bchwallet.

Strange, the test build I did for RC1 didnt seem to pickup the API calls for BCH

Current method of generating an address for payment can be better optimized. See this issue for more information


assume -c -ar does not work with --sso option

To reproduce:
run
 assume --sso --sso-start-url https://{DIRECTORY}.awsapps.com/start --sso-region ap-southeast-2 --account-id {ACCOUNT} --role-name GrantedAdministratorAccess

you will receive an output similar to the below
❯ assume -c -ar
you tried to use the -active-role flag but GrantedAdministratorAccess does not match any profiles in your AWS config or credentials files

the activeRole flag is currently meant to assume based on the AWS_PROFILE variable.
Do you want this ticket to add granted_ exports for the sso params so that when running --ar it detects that an sso profile was used, and assumes it?
@chrnorm

@JoshuaWilkes hmmm this might be problematic due to #212, where we need to remove the AWS_PROFILE export from --sso as it breaks the AWS CLI. My desired effect is that after running assume --sso --sso-start-url ... (having pasted this snippet from Granted Approvals) I can easily pop open a console to the role which I've assumed.
I realised that running
assume --sso --sso-start-url ... -c

actually works but this is a bit annoying as I need to open up my web browser again and copy the snippet a second time.
To clarify - by adding granted_ exports, do you mean adding an env var like GRANTED_AWS_PROFILE so that -c -ar will work?

We could export all the sso params
GRANTED_SSO=true
GRANTED_SSO_START_URL=...
GRANTED_SSO_ROLE_NAME=...

These are exported when the sso command is used and unset when a normal profile is used.
Then if you use --sso ... you can always put --ar in front of subsequent commands to launch using the sso profile form earlier.
This will work by checking for these env vars, and if they exist, using them in the same way that the AWS_PROFILE is used

That sounds like a good approach!


Hiding categories from inspector via EditorInspectorPlugin

Godot version:

OS/device including version:
Mac OSX Mojave
Issue description:
Even though it is possible to hide properties from the inspector (even builtin ones), using the "EditorInspectorPlugin"
extends EditorInspectorPlugin

func can_handle(object):
	if object is Attribute:
		return true;

func parse_property(object, type, path, hint, hint_text, usage):
	match path:
		'resource_local_to_scene':
			return true;
		'resource_path':
			return true;
		'resource_name':
			return true;
		'script':
			return true;

the resource category still gets drawn to the inspector is there a way to intersect the inspector draw and avoid drawing the category?! the parse_property function has a bool return that allows that but the category_parse doesn't!! Shouldn't it have that functionality?!
I was trying to remove the "Resource" category from resources while using them in arrays to avoid clutter!

This is still open to date. I'm fighting to get this properly with a category I define in custom nodes in my plugin.
Is there another solution that I'm not seeing to hide groups and categories from EditorInspectorPlugin - or another Editor class?

Categories were not designed to be conditionally hidden like that. Semantically, categories are designed to represent the class hierarchy, so hiding them doesn't make sense. That said, both categories and groups should get hidden if all of their properties are hidden, since they have no content to display.

@YuriSizov the problem to me is that programmatically hiding the property with _parse_property() doesn't hide the category, so I end up with an empty category. I took that's a desired behavior for granted, but from what you say, the right path may be fixing this so that if a category has no visible property, it's also hidden.

@stickgrinder Hiding properties with _parse_property() is not an expected use case for this method. Properties are normally hidden by the class' script itself, using _validate_property() and _get_property_list(). Just removing or hiding the control from within _parse_property() will not lead to the category or the group being hidden as well, we don't handle that.

You can now hide properties using _validate_property(), which should be working as expected.


Simplify SEBlock by broadcast of binary op

instead of explicit broadcast_to. The main motivation of this
change is to simplify the exported ONNX, but this would also
improve performance.

pfnCI, test this please

Rebased to pull #901
pfnCI, test this please


cmd/vet: false positives for FP assembly

What version of Go are you using (go version)?

$ go version
go version go1.13.3 linux/amd64

Does this issue reproduce with the latest release?
Yes.

What did you expect to see?
No complaints.
What did you see instead?
# gonum.org/v1/gonum/internal/asm/c64
c64/axpyinc_amd64.s:53:1: [amd64] AxpyInc: invalid MOVSD of alpha+0(FP); complex64 is 8-byte value containing alpha_real+0(FP) and alpha_imag+4(FP)
c64/axpyincto_amd64.s:57:1: [amd64] AxpyIncTo: invalid MOVSD of alpha+40(FP); complex64 is 8-byte value containing alpha_real+40(FP) and alpha_imag+44(FP)
c64/axpyunitary_amd64.s:48:1: [amd64] AxpyUnitary: invalid MOVSD of alpha+0(FP); complex64 is 8-byte value containing alpha_real+0(FP) and alpha_imag+4(FP)
c64/axpyunitaryto_amd64.s:49:1: [amd64] AxpyUnitaryTo: invalid MOVSD of alpha+24(FP); complex64 is 8-byte value containing alpha_real+24(FP) and alpha_imag+28(FP)
c64/dotcinc_amd64.s:159:1: [amd64] DotcInc: invalid MOVSD of sum+88(FP); complex64 is 8-byte value containing sum_real+88(FP) and sum_imag+92(FP)
c64/dotcunitary_amd64.s:207:1: [amd64] DotcUnitary: invalid MOVSD of sum+48(FP); complex64 is 8-byte value containing sum_real+48(FP) and sum_imag+52(FP)
c64/dotuinc_amd64.s:147:1: [amd64] DotuInc: invalid MOVSD of sum+88(FP); complex64 is 8-byte value containing sum_real+88(FP) and sum_imag+92(FP)
c64/dotuunitary_amd64.s:196:1: [amd64] DotuUnitary: invalid MOVSD of sum+48(FP); complex64 is 8-byte value containing sum_real+48(FP) and sum_imag+52(FP)
# gonum.org/v1/gonum/internal/asm/f64
f64/sum_amd64.s:99:1: [amd64] Sum: unknown variable sum; offset 24 is ret+24(FP)
f64/sum_amd64.s:100:1: [amd64] Sum: RET without writing to 8-byte ret+24(FP)

See gonum/gonum#79.

Thanks, @randall77. I had thought I had tried that change in Sum. Clearly not.


[ZEPPELIN-665] travis CI change to not rerun interpreter tests

What is this PR for?
While investigating CI failure with PR #706, I noticed that we were running all tests in each of 6 jobs in the test matrix.
It seems we have accidentally switched back to running all tests even with older versions of Spark. (from PR #626
Filtering out interpreter tests for these runs help make test jobs faster (~30%) and also reduce the chance that an interpreter test might intermittently fail, thus hopefully improve CI reliability.
Also changing Spark 1.2.1 to 1.2.2 (last 1.2.x release)
What type of PR is it?
Improvement
Todos

[x] - Travis CI yml changes

Is there a relevant Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-665
How should this be tested?
Run Travis CI
Screenshots (if appropriate)
N/A
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@Leemoonsoo Please review

Thanks for the improvement.
LGTM

merging...


Action does not trigger on my repo

https://github.com/kgashok/cyberdojo-exercises/blob/master/.github/workflows/jsformat.yml
I have no idea why it doesn't do a PR with the changes?
Am I missing something?

I have no idea why it doesn't do a PR with the changes?

Because it doesn't. I may implement this feature, but for now, it does only checking.

Do you know of any other action that does this already?

[this one] commits directly to your repo.

[this one] commits directly to your repo.
Link is missing. Thanks,

updated

Thanks much, @KSXGitHub !


added carbon footprint for Dell products

Description
source: https://www.dell.com/learn/us/en/uscorp1/corp-comm/environment_carbon_footprint_products

added carbon footprint for XPS, Precision, & Latitude laptops
added carbon footprint for the newest OptiPlex All-in-One, OptiPlex Towers, Precision desktops (bottom of the list in Desktops tab)
added hidden attribute to all electronics

put hidden: false for all existing Apple products
put hidden: true for older Dell products

Related Issues
Addresses #199

https://github.com/tmrowco/tmrowapp/issues/1331 needs to be fixed before


openai cli --stream error

Describe the bug

When adding the --stream parameter to the OpenAI command, an error occurred.
To Reproduce
When adding the --stream parameter to the OpenAI command, an error occurred.
> openai api chat_completions.create -m gpt-3.5-turbo -g user "Hello" --stream
Traceback (most recent call last):
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\Scripts\openai.exe\__main__.py", line 7, in <module>
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\site-packages\openai\_openai_scripts.py", line 70, in main
    args.func(args)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\site-packages\openai\cli.py", line 144, in create
    sys.stdout.write(c["message"]["content"])
KeyError: 'message'

Code snippets
No response
OS
Windows
Python version
Python v3.10.6
Library version
openai v0.27.4

It looks like you get this error because OpenAI-Python library does not support streaming. You can use OpenAI-Python package to handle this issue.   https://pypi.org/project/openai/
I would like to know if this works. Have a nice day!

The completions request for the "ada" with stream=True works correctly.

The ChatCompletions request with "gpt-4" with stream=True returns the response iterator correctly.

One of the confusing issue is that completions requires a message=[string] input, while a ChatCompletions requires a message=list[dict] input in the format of {"role":"user|system|assistant", "content":"[user message]"}
It can also be confusing that depending on a stream output or a bulk result the type of the result is different.

The problem with the CLI that it expects a ["message"]["content"] in the response from the returned function value after making the call:

Here is a sample response if you receive a single response:

This is fixed in the v1 SDK!
$ openai api chat.completions.create -g user 'say hi!' -m 'gpt-4' --stream


Allow for favorite creators (artists)

Need more robust local storage to allow for favorites

Added localforage and working with fav artists and recordings


IP address of smartthings changed...

My home network changed, and along with it my home network subnet.
So when I am running the bridge it appears to try to connect to my old IP address instead of the new one.
How do I fix this?
For reference, I've been running the bridge for well over a year on my raspberry pi (not docker). I poked around through the node_module folder for the globally installed package and didn't find my old IP address anywhere. Not clear where that gets established...

OOps, firgured it out, I had to update the address of the mqtt device in the app. All set!


How to upload files like pdf, word and excel

Is it possible to upload other type of files.?

You can upload any type of files in the same way.


Changing the target version of an FTC does not result in sync controller restart

It is possible to upgrade the target version of an FederatedTypeConfig (FTC), but the FTC controller does not currently respond to the change:
# Initial enable
kubefed2 enable mytypes.example.com --version=v1

# Sync controller targeting v1beta1 endpoints is launched

# Subsequent update
kubefed2 enable mytypes.example.com --version=v2

# Sync controller continues to target v1beta1 endpoints until controller manager restart

The FTC controller should detect when the target version of an FTC has changed and restart its sync controller.

/kind bug

/assign


Please add a LICENSE file

All published code should have a LICENSE file.
See https://help.github.com/en/articles/adding-a-license-to-a-repository

Hi @mithro
Thank you for sharing this. It will be addressed before the official release in July 2019.

Thanks again @mithro for pointing out to this. License is added.


希望Upload组件 fileList 能增加自定义操作区的插槽

[ ] I'm sure this does not appear in the issue list of the repository

Basic Info
What are the similar cases of this feature
无
What problem does this feature solve?
可以解决自定义下载按钮功能

同类问题 close: #1727