[Feature]: Fix Warning in bundle: some deprecated and minkubeversion missing

Summary 💡
When execute make bundle , some deprecated and csv.Spec.minKubeVersion missing is shown. We should fix it.
Detail message:
❯ make bundle
test -s /Users/kevin/workspace/develop/golang/src/github.com/zncdata-labs/listener-operator/bin/controller-gen && /Users/kevin/workspace/develop/golang/src/github.com/zncdata-labs/listener-operator/bin/controller-gen --version | grep -q v0.15.0 || \
        GOBIN=/Users/kevin/workspace/develop/golang/src/github.com/zncdata-labs/listener-operator/bin go install<EMAIL_ADDRESS>/Users/kevin/workspace/develop/golang/src/github.com/zncdata-labs/listener-operator/bin/controller-gen rbac:roleName=manager-role crd webhook paths="./..." output:crd:artifacts:config=config/crd/bases
/opt/homebrew/bin/operator-sdk generate kustomize manifests -q
cd config/manager && /Users/kevin/workspace/develop/golang/src/github.com/zncdata-labs/listener-operator/bin/kustomize edit set image controller=quay.io/zncdatadev/listener-operator:v0.0.1
/Users/kevin/workspace/develop/golang/src/github.com/zncdata-labs/listener-operator/bin/kustomize build config/manifests | /opt/homebrew/bin/operator-sdk generate bundle -q --overwrite --version 0.0.1 --channels="stable" --default-channel="stable"
# Warning: 'patchesStrategicMerge' is deprecated. Please use 'patches' instead. Run 'kustomize edit fix' to update your Kustomization automatically.
# Warning: 'patchesJson6902' is deprecated. Please use 'patches' instead. Run 'kustomize edit fix' to update your Kustomization automatically.
WARN[0000] ClusterServiceVersion validation: [CSVFileNotValid] (listener-operator.v0.0.1) csv.Spec.minKubeVersion is not informed. It is recommended you provide this information. Otherwise, it would mean that your operator project can be distributed and installed in any cluster version available, which is not necessarily the case for all projects.
INFO[0000] Creating bundle.Dockerfile
INFO[0000] Creating bundle/metadata/annotations.yaml
INFO[0000] Bundle metadata generated successfully
/opt/homebrew/bin/operator-sdk bundle validate ./bundle
WARN[0000] Warning: Value : (listener-operator.v0.0.1) csv.Spec.minKubeVersion is not informed. It is recommended you provide this information. Otherwise, it would mean that your operator project can be distributed and installed in any cluster version available, which is not necessarily the case for all projects.
INFO[0000] All validation tests have completed successfully

Examples 🌈
No response
Motivation 🔦
No response

#104 for fixing minkubeversion missing


Header gets lost when printing to pdf

Just was about to add this to your issue tracker. Great to see you already beat me to the punch!


Player moved too fast! issues on setting location and bizzare coordinate reported

Folks aren't sure if this is an implimentation or API issue:
Setting a player location sometimes causes the server to block it as a moved-too-quickly event, which is unavoidable - we're moving them instantaneously across the map :)
But in addition, the logs have mysterious output when it cancels.  The coordinates randomly get negated, even the Y coordinate in back to back trials:
BooMod moved too quickly! -605.0,-20.09800000190735,170.0 (605.0, 20.09800000190735, 170.0)
[16:50:40] [Server thread/WARN]: BooMod moved too quickly! -43.0,13.0,-259.0 (43.0, 13.0, 259.0)
[16:50:46] [Server thread/WARN]: BooMod moved too quickly! 43.0,-13.0,259.0 (43.0, 13.0, 259.0)
[16:51:43] [Server thread/WARN]: BooMod moved too quickly! 86.5,71.0,-194.5 (86.5, 71.0, 194.5)
Forge Mod Loader version <IP_ADDRESS>7.

This is an implementation issue. Currently, there's no method in the api to disable the check.

would the odd coordinate sign-swapping in the output be a totally different issue then within the moved-too-fast handling code?  Its not like the system is always negating the z in reporting the output, for example.

Would it be worth adding a method to teleport players as opposed to setting their locations?

I consider setLocation the same as teleport.
So i don't see a reason to add another method for that.

I've found this too, I'm in the process of writing a command system plugin which includes home, spawn etc an on random occasions this does occur. Given the usage of spawn command on so many servers I would view this is a high priority bug in the implementation

@bloodmc
You know 100x more about this than me due to your time with Cauldron. Could you rectify this in SpongeCommon?

Sure, I'll take a look tomorrow.

@theboomer Please provide me with a copy of a test world and exact code you are using to trigger the issue. I have a fix but need to confirm it works before pushing. Thanks!

Its been n number of builds since the tests were done, and I dont know if I have a working state plugin for what did that, but im sure i can try to put something together to test, and will figure out a way to make the files available.  If not in 30min, then will have to wait til tomorrow, but will-do

I managed to recover the functional test part of the code out of comments but while trying to recompile to test it, its not cooperating.  My CommandExecutor implementing classes /command builder things dont seem to be happy as they are.    I haven't written a command handler code in a long time, so in case something strange has changed that I am unaware of in recent builds...  I'm currently using the 1446 forge and 1446-dev482 sponge versions on my test server.
I'm getting Caused by: java.lang.NoSuchMethodError: org.spongepowered.api.util.command.spec.CommandSpec$Builder.setDescription(Lorg/spongepowered/api/text/Text;)Lorg/spongepowered/api/util/command/spec/CommandSpec$Builder;
Yet there are no errors apparent in Eclipse, the error suggests that the setDescription() part of the commandspec builder is not in the current code, yet, its still in the intelletext if I create a new command, and if i comment it out, then the error is on the next command-spec property..  I haven't tried any coding in several weeks, and have switched forge versions a few times recently, so i dont know to what extent this
I will keep pushing to try to get this to compile, I have even written a clean new plugin just copy/pasting the module code over to it, with same results...

https://forums.spongepowered.org/t/mystery-eclipse-issue-with-magical-nonimporting-imports/7989  Is what is holding me back from retesting this and giving you anything.  I dont know if its something weird in a configuration, or if my Eclipse has decided to rise up against me, but I'm stuck in a rut unable to compile a working plugin and until I can resolve it, your request will be stuck.

Resolved the mystery issue (answer was D'oh) for part of it, and wrong api dependancy version, but now I need to rewrite the scheduler part to compile..

I said screw-the-scheduler and went straight to moving the player to coordinates, which is what I was using before implimenting a delay that didn't help anyways...
I have not been able to get the same moved-too-quickly errors at any time from over 100 tp jumps at different times and frequency of jumping with sponge 1446-dev500  and forge 1446
I am including all the module code that should have been the code used to get the spotty motion results, map coordinates are hardcoded to move the player within the same world.
The map I've uploaded to here:
http://www.ethicraft.net/world.zip
I wish I could generate the failure results still, I'm sure its possible by matching the right versions of api and dependancies, which i have no idea how to do at this time.


Minor spelling correction

Learn Build status updates of commit 46f6a66:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

powerbi-docs/developer/embedded/embed-azure-analysis-services.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks for pointing this out, @AKomonen!
The fix will be live soon.


High cpu load when letsencrypt is set to ondemand

traefik consumes > 4.0 load when ondemand is set to true on RPI

@aacebedo could you be more specific and give more details?

I am running traefik on a rpi with 5 docker containers.
Each of them are have one specific subdomain.
I ran traefik with docker watch flag to true and letsencrypt ondemand mode to true.
When running top, traefik seems to consume 400% cpu.
Requests are not served as they reach the http connection timeout.

Traefik is also running in a container

Does High load occur only during first request (OnDemand) or all the time?

All time.
I can investigate more tonight playing with the ondemand flag if it can help you.

@aacebedo any news on this ?

I close it, the high cpu demand is caused by ssh key generation when starting traefik.
sorry for trouble


[5.1] Cleanup route function

This was triggered by https://github.com/laravel/lumen-framework/issues/119.

Can you explain the problem and what this fixes.

It doesn't fix anything code wise. It just makes it way clearer and provides readable phpdoc.

I am seeing that a code fix is necessary. Lumen returns an array as the route and Laravel ruturns an Object (Illuminate\Routing\Route).
In Lumen - \Laravel\Lumen\Application line 643  - setRouteResolver returns $this->currentRoute
    protected function registerRequestBindings()
    {
        $this->singleton('Illuminate\Http\Request', function () {
            return Request::capture()->setUserResolver(function () {
                return $this->make('auth')->user();
            })->setRouteResolver(function () {
                return $this->currentRoute;
            });
        });
    }

$this->currentRoute returns an array and is documented as such in that same class (Laravel\Lumen\Applications) line 115
    /**
     * The current route being dispatched.
     *
     * @var array
     */
    protected $currentRoute;

So in Illuminate\Http\Request the function route() returns the array in Lumen. If you pass a parameter on route() then it is trying to access the parameter function on an array which can't be done.
So something like this would work
    /**
     * Get the route handling the request.
     *
     * @param null|string $param
     * @return mixed
     */
    public function route($param = null)
    {
        // is either an array (Lumen) or an instance of \Illuminate\Routing\Route (Laravel)
        $route = call_user_func($this->getRouteResolver());

        if (is_null($route) || is_null($param)) {
            return $route;
        } else if(is_array($route)) {
            return $route[2][$param];
        } else {
            return $route->parameter($param);
        }
    }

Now from Illuminate\Http\Request is can call route($param) in both laravel and lumen and get my route parameter back as a string.  Hope that makes sense.
I can submit a pull request if needed.

So we basically need to add
        } elseif (is_array($route)) {
            return $route[2][$param];

Laravel and Lumen use different routing engines. Nothing was broken in
Laravel so I'm tempted to revert al this
On Friday, June 26, 2015, Graham Campbell<EMAIL_ADDRESS>wrote:

So we basically need to add
    } elseif (is_array($route)) {
        return $route[2][$param];

—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/9417#issuecomment-115741603.

Yeah adding the elseif would be all that needs to be added.

WHAT ARE WE FIXING?!
On Friday, June 26, 2015, Jason Yaede<EMAIL_ADDRESS>wrote:

Yeah adding the elseif would be all that needs to be added.
—
Reply to this email directly or view it on GitHub
https://github.com/laravel/framework/pull/9417#issuecomment-115742273.

Laravel so I'm tempted to revert al this

There's nothing to revert yet. No commit has been pushed to laravel to fix any lumen specific issue.

The only commit there was a fix for readability and to remove recursion.

Yeah just discussing a possible solution to two different routers sharing one the same request class.  The only functionality that breaks in lumen is in the route() function when a parameter is passed because it wants the laravel router when a param is passed not the lumen. Not a urgent fix or anything. Just something I found.

:-1:

This commit was NOT meant fix that lumen issue.

Roughly 2 years ago I submitted a PR to add something like that routing ability to Laravel and it was rejected for the reasons listed above. I then subclassed the affected Laravel routing components for Lumen, added the functionality there, and submitted a PR to Lumen to fix the routing issue. That PR was subsequently rejected because it was too big of a change to put in mid-release, or some such. My requests for assistance/guidance as to how to get a "correct" PR in to fix the issue were met with silence.
So, when it really comes down to it, I get the impression that it isn't a high priority to get feature-complete routing functionality in Lumen. Even after 2 years since the original bugs and PRs were filed.
The Lumen issue is a blocker for people's projects. I follow the bug and I get notified every few months when someone makes another comment on it. My project was young enough, and the Lumen bug was a critical blocker, that I ended up moving my Lumen project to another framework rather than run a private fork of Lumen that fixed the actual issue.


Build/push arm64 image and multiarch manifest

Update docker-otelcol make target for ARCH var and docker buildx
Update Dockerfile to only download/extract smart agent bundle for amd64
Update circleci to build/run images for both archs
Build/tag/push quay.io/signalfx/splunk-otel-collector:<version>-[amd64|arm64] images from gitlab
Create/tag/push quay.io/signalfx/splunk-otel-collector:<version> manifest with both archs from gitlab

What the implications of manifests being experimental and of these changes when using an older docker client and trying to run quay.io/signalfx/splunk-otel-collector:latest?

What the implications of manifests being experimental and of these changes when using an older docker client and trying to run quay.io/signalfx/splunk-otel-collector:latest?

According to https://docs.docker.com/registry/spec/manifest-v2-2/#backward-compatibility:

When pulling images, clients indicate support for this new version of the manifest format by sending the application/vnd.docker.distribution.manifest.v2+json and application/vnd.docker.distribution.manifest.list.v2+json media types in an Accept header when making a request to the manifests endpoint. Updated clients should check the Content-Type header to see whether the manifest returned from the endpoint is in the old format, or is an image manifest or manifest list in the new format.
If the manifest being requested uses the new format, and the appropriate media type is not present in an Accept header, the registry will assume that the client cannot handle the manifest as-is, and rewrite it on the fly into the old format. If the object that would otherwise be returned is a manifest list, the registry will look up the appropriate manifest for the amd64 platform and linux OS, rewrite that manifest into the old format if necessary, and return the result to the client. If no suitable manifest is found in the manifest list, the registry will return a 404 error.

Also, since our current images use the v2 manifest schema (same as the multiarch manifest), it should not be an issue.

A test multiarch manifest was pushed to quay.io/signalfx/splunk-otel-collector-dev:bc4b79dd130cbd179a21b61f728cb3c20d3f5a9a (based on the v0.37.1 release):
$ docker manifest inspect quay.io/signalfx/splunk-otel-collector-dev:bc4b79dd130cbd179a21b61f728cb3c20d3f5a9a
{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
   "manifests": [
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 3034,
         "digest": "sha256:e1e9a78fe54b9c953671332d2567f754667ea74829a06f9296e401bf7b296466",
         "platform": {
            "architecture": "amd64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
         "size": 3028,
         "digest": "sha256:ffa8af1cc4219ba6b140c579c4f54059a2a80a0beac1843230a88d6b9bbfd346",
         "platform": {
            "architecture": "arm64",
            "os": "linux"
         }
      }
   ]
}


Add some motivation to the CI job on MacOS

It currently fails with this message:
Error: Refusing to uninstall /usr/local/Cellar/ruby/3.3.5
because it is required by asciidoctor, which is currently installed.
You can override this and force removal with:
  brew uninstall --ignore-dependencies ruby

This should fix the failing CI of #2224 and #2225


rustfmt stops working if the comment just after the => of a "match" contains "=>"

Hello,
I've noticed that rustfmt stops working if the comment just after the => of a "match" contains "=>".
info :
rustfmt 1.7.0-stable (82e1608 2023-12-21)
rustc 1.75.0 (82e1608df 2023-12-21)
here's an example code :
fn main() {
    let a = 50u32;
    match a {
        10 =>
        // comment => 10
                {
            println!("A")
        }
        20 => {
                            println!("B")
        }
        _ => {
            println!("C")
        }
    }
}

the problem is in the comment :
// comment => 10
the comment contains the symbol "=>", if you try to use rustfmt it won't work.
but if you delete the "=>" from the comment, or simply add a space like this "= >", rustmt works again.
you can test the bug here :  ( click on the top-right "Tools -> Rustfmt" button to start formatting, but it won't work as long as the comment contains "=>" )
Playground
Thank you for the wonderful work that you do :+1:

@rxdiscovery thanks for the report!
Using rustfmt 1.7.0-nightly (85e21fab 2023-12-28) I can confirm that this first (minimized) example reformats
fn main() {
    match a {
        _ =>
        // any comment here will prevent formatting
                {
            println!("A")
        }
    }
}

while this second example does not.
fn main() {
    match a {
        _ =>
        // comment with =>
                {
            println!("A")
        }
    }
}


Make the README file appear on the PyPI project page.

H/t @ColCarroll, inspired by https://github.com/ColCarroll/callisto/blob/master/setup.py

Coverage remained the same at 95.257% when pulling 4cb7581942e903bfb421682f2bbf261cb4e5fdb1 on add_long_description into 427481bc81fcd29144843c8814dd8c4006df6f50 on master.

Needs a new release before it shows up, right?

Right, it won't be visible on pypi until the next release.
On Wed, Sep 19, 2018, 11:17 AM Colin<EMAIL_ADDRESS>wrote:

Needs a new release before it shows up, right?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/kensho-technologies/graphql-compiler/pull/133#issuecomment-422843422,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACPWSiq2ifmO2YC6KRXQUObqSUcYdnt7ks5ucmAdgaJpZM4WuxX0
.


Class not found on OS X El Capitan

Whenever I want to use jpype.JClass() to import a class, I constantly get the following message:
jpype._jexception.RuntimeExceptionPyRaisable: java.lang.RuntimeException: Class TestApi not found

I'm using Java 1.6 and Python 2.7. Anyone has the same problem on a Mac?

double check your classpath argument you've passed to the jvm. This has most likely nothing to do with the underlying operating system (osx, etc.). And please avoid double posting (copy paste issues) the same thing twice in the future, you are wasting other people time by doing so...
To clarify: JClass and JPackage are very related to each other. Perhaps you should read some literature.


fix(dashboard): better dashboard hints

Prioritize +doom-dashboard-mode-map normal mode bindings on the dashboard, and add a
small note about it in the documentation.

Abort mission for now. This fails if the map has no normal mode binding

Got a working version, that is slightly dirtier

Might be a bit unrelated, but it would be good to fix docstring in  https://github.com/hlissner/doom-emacs/blob/42e5763782fdc1aabb9f2624d468248d6978abe2/modules/ui/doom-dashboard/config.el#L85
since it seems like :key STRING is no longer used, and will throw an error if you try to define a PLIST with :key element in it, eg.:
("Find File Menu"
     :icon (all-the-icons-octicon "search" :face 'doom-dashboard-menu-title)
     :key "f"
     :action find-file)

will give you error:
Error (+doom-dashboard-functions): Error running hook "doom-dashboard-widget-shortmenu" because: (error Keyword argument :key not one of (:icon :action :when :face))

Unfortunately, a little more finesse was needed for this, as precedence (with where-is-internal) is a real pain to get right, whilest avoiding calling where-is-internal more than we have to. I done this in b5093a5.
I've also restored :key, so it can now be used to statically define a key shortcut string.
Still wondering if rebinding keys needs documenting at all. Doing so in a hook isn't necessary (and should never be necessary except in extreme cases, *cough*eshell). Anyhow, I'll put it in the rewrite-docs branch if/when I decide to.
Thanks for bringing it to my attention!

Thanks a lot for working on the issue, I will change my config accordingly.
Have a nice day


Switch from secrets to rails credentials

UPDATE:  The code has been updated to use fallbacks of default values in tests so CI and users don't need to actually specify Rails.application.credentials unless they are recording new cassettes.
TODO: Verify we can rerecord the cassettes using this PR.
Here's what you have to do:

To run the tests, do nothing, we have defaults specified to match the cassettes.

To record updated cassettes:

Apply the commit here to switch to credentials
Edit rails credentials in the rails application (manageiq) in the test environment:
IMPORTANT: cd to manageiq and not the rails engine directory.  It looks for the encryption key and encrypted
credentials file relative to the rails application, not the engine.

EDITOR=vi be rails credentials:edit --environment test
Specify the real values in your editor(change the values below):
autosde:
  appliance_host: autosde-appliance-host
  site_manager_user: autosde
  site_manager_password: change_me

Save this.  It should generate the following files in the rails app:

config/credentials/test.key (if not previously created)
an encrypted credentials file in config/credentials/test.yml.enc

Both files should be .gitignored.
You can now run record updated cassettes.

@jrafanie yeah we need some way to have default credentials for vcrs but be able to override that in order to re-record the VCRs using real credentials.

@jrafanie yeah we need some way to have default credentials for vcrs but be able to override that in order to re-record the VCRs using real credentials.

Default credentials feels like something that's separated from rails credentials.  You could theoretically run EDITOR=vi be rails credentials:edit --environment test to set defaults like I did above, then ship the config/credentials/test.key and config/credentials/test.yml.enc where the .key is like our v2_key and the .yml.enc is the encrypted yaml of the credentials for use in CI or similar.
There are problems with this though.  You wouldn't be able to run the tests at all without first following those steps.  I wonder if it makes more sense changing usage of Rails.application.secrets or Rails.application.credential to have a default value if nil in the code/test that uses it so you only need it when re-recording VCRs.
In other words:
Rails.application.credentials.dig(:autosde, :appliance_host) || "autosde-appliance-host"

In other words:
Rails.application.credentials.dig(:autosde, :appliance_host) || "autosde-appliance-host"

I'll just do it as additional commits... it's easier to talk about code once it's in front of you

@agrare @nasark No rush but if one of you is able to rerecord cassettes, it would be great if you could verify the process I listed above where you run rails credentials:edit in the main app repo to set the correct actual credentials works when you try to update the cassettes.

Ok, removing WIP.  I still need to record updated cassettes using this procedure but I think all of the concerns raised here have been addressed.

@jrafanie and just to make sure I understand how the new credentials system works would we have to use rails credentials:edit --environment test here?  Can that be per-plugin or is it for the whole app?  We should probably include something in the plugin/providers developers guide on how to re-record vcrs

@jrafanie and just to make sure I understand how the new credentials system works would we have to use rails credentials:edit --environment test here? Can that be per-plugin or is it for the whole app? We should probably include something in the plugin/providers developers guide on how to re-record vcrs

rails credentials are for the rails application so I guess you'd have some .gitignore(d) credentials for each plugin.  I need to add the .gitignore for the main app.  I'll do that next.

We also need to update the plugin generator.
For some reason I'm not a fan of the defaults in the code as opposed to the YAML file, but in the end it doesn't make much difference, and I won't let that hold up a merge. In the end as long as @agrare is happy then I'm happy 😅

We also need to update the plugin generator.

Yes, it's on the list.  I wanted to get a concrete example done and then it's on to all the other places we use secrets.  I'm going to try to find an environment I can try to rerecord cassettes at some point.

For some reason I'm not a fan of the defaults in the code as opposed to the YAML file, but in the end it doesn't make much difference, and I won't let that hold up a merge. In the end as long as @agrare is happy then I'm happy 😅

I don't like it either.  One alternative is requiring CI/devs create their credentials for use with the test but that seems cumbersome.

However I would like to have the "real" secrets gitignored ahead of time in core.

However I would like to have the "real" secrets gitignored ahead of time in core.

On the way...

@agrare @Fryguy see: https://github.com/ManageIQ/manageiq/pull/23263

Leaving as wip until I can run through recording cassettes along with the updated guide: https://github.com/ManageIQ/guides/pull/553


Bundle the boost git repo instead of using the .tar.gz archiv

Motivation / Current Behaviour
We currently bundle only a minimal extract of boost library as an archive that contains only the libraries of boost that we use. E.g. in PR #929 we need additional functionality from boost. The resulting bundle of boost is too big to include it in the repository.
Enhancement description
The preferred solution is to bundle the git repo of boost instead of using the .tar.gz archive. A disadvantage is that the build step of the project takes a while because boost is downloaded.
Additional context
No response
Checklist

[X] Attached labels, especially loc:: or model:: labels.
[X] Linked to project

I already tried this. As far as i know, there is no elegant way to do this.
Do you have an idea in mind?
There currently is minimal CMake support for boost for this but to use it i think it is not ready yet. Last time i looked into it was 1/2 year ago.

Yes, I have a few ideas and some things have already worked out. I've also talked to @dabele. Maybe I'm missing something that's causing the whole thing to fail. I'll try a few things. Will keep you updated.


Removed duplicate dataframe / transpose

Didn't think there should be two in a row .. ? Just poking around

Whoops!


Add CXR Foundation and Derm Foundation model libraries (Google Health AI)

For two Google Health AI models, a primary usage pattern (with supporting documentation and notebooks) is downloading only the precomputed .npz embeddings. As such, I'm submitting for two new model libraries which are essentially an extension of the TF Keras model library, but they cleanly handle the non-saved_model.pb usage pattern in the countDownload override. This PR was designed to conform with https://huggingface.co/docs/hub/en/models-download-stats and https://huggingface.co/docs/hub/en/models-adding-libraries#register-your-library.
Public models https://huggingface.co/google/cxr-foundation and https://huggingface.co/google/derm-foundation will be switched over once this PR is merged.

Correct - I think we should be all set. Thanks @Vaibhavs10

Merging since the CI errors look unrelated.


498 mimic camchem substep convergence failure integration acceptance

Closes #498
From my understanding, if you fail to converge too many times in a row and try to cut the time step too many times in camchem, you forgo the current integration, but accept what you have done up to the last successful integration and continue on.
I don't actually know how to test this

Codecov Report
Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 93.20%. Comparing base (85232c5) to head (9519972).

Files
Patch %
Lines

include/micm/solver/backward_euler.inl
66.66%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #582      +/-   ##
==========================================
+ Coverage   92.89%   93.20%   +0.30%
==========================================
  Files          49       49
  Lines        3477     3473       -4
==========================================
+ Hits         3230     3237       +7
+ Misses        247      236      -11

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


Always On feature needs detailed explanation

Here is a good detailed answer on what it does:
https://stackoverflow.com/questions/26578934/azure-website-always-on
An important detail is that you can't control the endpoint, it is always application root. So if your app does not have a root route, you are going to see a lot of 404s in your logs.

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

ID: 1e25899c-1ed8-7cfc-2c37-73bb010a87f0
Version Independent ID: 8b366c42-43f8-d5eb-b619-b356e75a9916
Content: Configure apps in the portal - Azure App Service
Content Source: articles/app-service/configure-common.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@poctob Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks for your input! I have submitted a PR to fix the issue, the change is merged internally and will go-live in the public docs shortly.


[WIP - DO NOT MERGE][Scheduler][Stability] Enable forkserver as a default forking mode

fixes issues with threads locking - switched to forkserver, refactored code to be picklable
fixes issues with CUDA - #620

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I know that this is WIP but I'd like to provide the example to help us locate potential issues. I tried the following code with the changes in this PR. But it reports an error:
import numpy as np
import autogluon as ag
import autogluon.core
import torch as th
import torch.nn as nn

class Net(nn.Module):
    def __init__(self, in_units, num_hidden):
        super().__init__()
        self.linear1 = nn.Linear(in_units, num_hidden)
        self.out_layer = nn.Linear(num_hidden, 1)

    def forward(self, x):
        return self.out_layer(nn.ReLU()(self.linear1(x)))

def train_nn(args, reporter):
    num_hidden = args['num_hidden']
    lr = args['lr']
    batch_size, in_units = 32, 8
    net = Net(in_units=in_units, num_hidden=num_hidden)
    net.cuda()
    net.train()
    optimizer = th.optim.Adam(net.parameters(), lr=lr, amsgrad=True)
    for i in range(10):
        optimizer.zero_grad()
        random_data_in = th.normal(0, 1, (batch_size, in_units)).cuda()
        random_data_out = 5 * random_data_in ** 2 + 1
        pred_out = net(random_data_in)
        loss = ((random_data_out - pred_out)**2).mean()
        loss.backward()
        with th.no_grad():
            print(a_gpu)
            print(loss.cpu().detach().numpy())
            reporter(mean_loss=loss.cpu().detach().numpy())
        optimizer.step()

def run_tuning_jobs(fn, search_space):
    args_decorator = ag.core.args(**search_space)
    scheduler = ag.core.scheduler.FIFOScheduler(args_decorator(train_nn),
                                       resource={'num_cpus': 4, 'num_gpus': 1},
                                       num_trials=20,
                                       reward_attr='loss',
                                       time_attr='iteration')
    scheduler.run()
    scheduler.join_jobs()
    return scheduler

search_space = {
    'num_hidden': ag.core.space.Int(16, 32),
    'lr': ag.core.space.Real(1e-3, 1e-2)
}

a_gpu = th.ones((10,)).cuda()

scheduler = run_tuning_jobs(train_nn, search_space)

Error message:
Traceback (most recent call last):
  File "/home/ubuntu/autogluon/core/src/autogluon/core/utils/custom_process.py", line 16, in run
    mp.Process.run(self)
  File "/usr/lib/python3.6/multiprocessing/process.py", line 93, in run
    self._target(*self._args, **self._kwargs)
  File "/home/ubuntu/autogluon/core/src/autogluon/core/scheduler/scheduler.py", line 147, in _worker
    fn = dill.loads(pickled_fn)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/dill/_dill.py", line 283, in loads
    return load(file, ignore, **kwds)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/dill/_dill.py", line 278, in load
    return Unpickler(file, ignore=ignore, **kwds).load()
  File "/home/ubuntu/.local/lib/python3.6/site-packages/dill/_dill.py", line 481, in load
    obj = StockUnpickler.load(self)
  File "/home/ubuntu/.local/lib/python3.6/site-packages/dill/_dill.py", line 471, in find_class
    return StockUnpickler.find_class(self, module, name)
AttributeError: Can't get attribute 'Net' on <module '__main__' (built-in)>

Could you pass the branch to Thibaut, so he can run the examples that failed for him?

Closing this version - moved to #788


Bump version to "v13.2.0" with changelog.

NOTE: This is a non-standard release since it is a minor. This PR includes manual edits to latest-versions.ts and peer deps on the framework to drop prerelease tags and bump the versions to 13.2.0 where appropriate.

Actually looks like FW did just release, so we should be good to go if CI passes.


Stack trace length on Android limited to 10 lines

Problem description
Any result of new Error().stack is limited to 10 lines.
Expected behavior
Print entire stack trace (within reason).
Environment
Any Android version of Tabris.js.
Code snippet

function a() { console.log(new Error().stack); }
function b() { a(); }
function c() { b(); }
function d() { c(); }
function e() { d(); }
function f() { e(); }
function g() { f(); }
function h() { g(); }
function i() { h(); }
function j() { i(); }
function k() { j(); }
function l() { k(); }
function m() { l(); }

m();

Result:
Error
    at a (./dist/imageview.js:16:28)
    at b (./dist/imageview.js:17:16)
    at c (./dist/imageview.js:18:16)
    at d (./dist/imageview.js:19:16)
    at e (./dist/imageview.js:20:16)
    at f (./dist/imageview.js:21:16)
    at g (./dist/imageview.js:22:16)
    at h (./dist/imageview.js:23:16)
    at i (./dist/imageview.js:24:16)
    at j (./dist/imageview.js:25:16)

Related: https://stackoverflow.com/questions/7697038/more-than-10-lines-in-a-node-js-stack-error


Announcement on Julia Discourse

Hello,
This package looks very exciting - have you considered making an announcement on the Julia Discourse to increase visibility?

Hey @dillondaudert, thank you so much! I didn't think about it before, but I agree that I should post something on the discourse. I wonder if I should wait for the JOSS reviews to finish before posting something.

Myself, I'd make an announcement as soon as I felt comfortable with the implementation and functionality, and wanted people to use the package. I wouldn't worry about waiting for the publication.

That makes perfect sense. Thanks for the advice. I'll prepare the announcement!


Chromedriver Default Port?

What:
When I run the program it halts to tell me Chromedriver needs to run on port 4444 specifically.
Waiting...
Downloading brightwheel...
thread 'main' panicked at 'chromedriver must be running on port 4444', /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/apk-downloader-0.2.1/src/main.rs:175:19
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

"Fix": By default, chromedriver ran on port 9515 for me. I am not sure if that is due to a previous configuration for me, or that is the true default. I ran chromedriver --port=4444 and that allowed the program to run normally.

Oh! I also used APK Pure obviously to run into this.

What:
When I run the program it halts to tell me Chromedriver needs to run on port 4444 specifically.
Waiting...
Downloading brightwheel...
thread 'main' panicked at 'chromedriver must be running on port 4444', /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/apk-downloader-0.2.1/src/main.rs:175:19
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

"Fix": By default, chromedriver ran on port 9515 for me. I am not sure if that is due to a previous configuration for me, or that is the true default. I ran chromedriver --port=4444 and that allowed the program to run normally.

Yes, probably a good idea to make this the default port.  Fixed in cd188d89ba40af646bc20696598f8943f4ba76e2


Search option

Add search option to select node like my 1 lava server on garman and other on us after i connect german lava it give worng results
i want search it on us node so i request add node name option on search function so i can get what i want result

This will available on v4


runtime warning in bittrex API call

I am getting a warning when getting deposit address balance.
[2018-07-02 10:54:27.320] [pool-6-thread-2] WARN si.mazi.rescu.RestMethodMetadata@create:111 [] [] - GET request declared as consuming method body as application/json. While body is allowed, it should be ignored by the server. Is this intended? Method: public abstract org.knowm.xchange.bittrex.dto.account.BittrexDepositAddressResponse org.knowm.xchange.bittrex.BittrexAuthenticated.getdepositaddress(java.lang.String,si.mazi.rescu.ParamsDigest,si.mazi.rescu.SynchronizedValueFactory,java.lang.String) throws java.io.IOException
BittrexAuthenticated should not contained @Consumes annotation

I am receiving this same warning.

My issue was related to bittrex v1 api. You should migrate to v3 api (xchange-bittrexV3).
According to API v1:
Warning: The v1.1 API will reach its planned end of life and no longer be supported as of 9/30/2020.
I will close this issue as no longer relevant.


Not possible to have more than 64 submit_candidacy extrinsics per block

any more will trigger "invalid candidate slot" error in runtime
Output from a test script (including substrate logs interleaved):
2019-06-21 14:27:20 Starting consensus session on top of parent 0x1b2e112ff2bb4977b13e4eee24c4f13ac81083128e8c5cb53490b09b8f1fd508
2019-06-21 14:27:20 Prepared block for proposing at 8 [hash: 0xdde85dab8fc077ef128fc1443ab103d833161c68326b70a05f7ed71fcd70bafc; parent_hash: 0x1b2e…d508; extrinsics: [0x0973…181e, 0xcb35…226d]]
2019-06-21 14:27:20 Pre-sealed block for proposal at 8. Hash now 0x28932adf2ff509ed960d1e455193845886d22dc7ae4e99cae3ca4181c8d7a0bb, previously 0xdde85dab8fc077ef128fc1443ab103d833161c68326b70a05f7ed71fcd70bafc.
2019-06-21 14:27:20 Imported #8 (0x2893…a0bb)
Sending transactions for: 'council::submit_candidacy' ...
******* Candidate count = 0, Counter = 0, Best block = 8
******* Candidate count = 0, Counter = 1, Best block = 8
******* Candidate count = 0, Counter = 2, Best block = 8
******* Candidate count = 0, Counter = 3, Best block = 8
******* Candidate count = 0, Counter = 4, Best block = 8
******* Candidate count = 0, Counter = 5, Best block = 8
******* Candidate count = 0, Counter = 6, Best block = 8
******* Candidate count = 0, Counter = 7, Best block = 8
******* Candidate count = 0, Counter = 8, Best block = 8
******* Candidate count = 0, Counter = 9, Best block = 8
******* Candidate count = 0, Counter = 10, Best block = 8
******* Candidate count = 0, Counter = 11, Best block = 8
******* Candidate count = 0, Counter = 12, Best block = 8
******* Candidate count = 0, Counter = 13, Best block = 8
******* Candidate count = 0, Counter = 14, Best block = 8
******* Candidate count = 0, Counter = 15, Best block = 8
******* Candidate count = 0, Counter = 16, Best block = 8
******* Candidate count = 0, Counter = 17, Best block = 8
******* Candidate count = 0, Counter = 18, Best block = 8
******* Candidate count = 0, Counter = 19, Best block = 8
******* Candidate count = 0, Counter = 20, Best block = 8
******* Candidate count = 0, Counter = 21, Best block = 8
******* Candidate count = 0, Counter = 22, Best block = 8
******* Candidate count = 0, Counter = 23, Best block = 8
******* Candidate count = 0, Counter = 24, Best block = 8
******* Candidate count = 0, Counter = 25, Best block = 8
******* Candidate count = 0, Counter = 26, Best block = 8
******* Candidate count = 0, Counter = 27, Best block = 8
******* Candidate count = 0, Counter = 28, Best block = 8
******* Candidate count = 0, Counter = 29, Best block = 8
******* Candidate count = 0, Counter = 30, Best block = 8
******* Candidate count = 0, Counter = 31, Best block = 8
******* Candidate count = 0, Counter = 32, Best block = 8
******* Candidate count = 0, Counter = 33, Best block = 8
******* Candidate count = 0, Counter = 34, Best block = 8
******* Candidate count = 0, Counter = 35, Best block = 8
******* Candidate count = 0, Counter = 36, Best block = 8
******* Candidate count = 0, Counter = 37, Best block = 8
******* Candidate count = 0, Counter = 38, Best block = 8
******* Candidate count = 0, Counter = 39, Best block = 8
******* Candidate count = 0, Counter = 40, Best block = 8
******* Candidate count = 0, Counter = 41, Best block = 8
******* Candidate count = 0, Counter = 42, Best block = 8
******* Candidate count = 0, Counter = 43, Best block = 8
******* Candidate count = 0, Counter = 44, Best block = 8
******* Candidate count = 0, Counter = 45, Best block = 8
******* Candidate count = 0, Counter = 46, Best block = 8
******* Candidate count = 0, Counter = 47, Best block = 8
******* Candidate count = 0, Counter = 48, Best block = 8
******* Candidate count = 0, Counter = 49, Best block = 8
Sent 50 transactions
******* Candidate count = 0, Counter = 50, Best block = 8
******* Candidate count = 0, Counter = 51, Best block = 8
******* Candidate count = 0, Counter = 52, Best block = 8
******* Candidate count = 0, Counter = 53, Best block = 8
******* Candidate count = 0, Counter = 54, Best block = 8
******* Candidate count = 0, Counter = 55, Best block = 8
******* Candidate count = 0, Counter = 56, Best block = 8
******* Candidate count = 0, Counter = 57, Best block = 8
******* Candidate count = 0, Counter = 58, Best block = 8
******* Candidate count = 0, Counter = 59, Best block = 8
******* Candidate count = 0, Counter = 60, Best block = 8
******* Candidate count = 0, Counter = 61, Best block = 8
******* Candidate count = 0, Counter = 62, Best block = 8
******* Candidate count = 0, Counter = 63, Best block = 8
******* Candidate count = 0, Counter = 64, Best block = 8
******* Candidate count = 0, Counter = 65, Best block = 8
******* Candidate count = 0, Counter = 66, Best block = 8
******* Candidate count = 0, Counter = 67, Best block = 8
******* Candidate count = 0, Counter = 68, Best block = 8
******* Candidate count = 0, Counter = 69, Best block = 8
******* Candidate count = 0, Counter = 70, Best block = 8
******* Candidate count = 0, Counter = 71, Best block = 8
******* Candidate count = 0, Counter = 72, Best block = 8
******* Candidate count = 0, Counter = 73, Best block = 8
2019-06-21 14:27:22 Idle (0 peers), best: #8 (0x2893…a0bb), finalized #8 (0x2893…a0bb), ⬇ 1.0kiB/s ⬆ 1.0kiB/s
2019-06-21 14:27:24 Starting consensus session on top of parent 0x28932adf2ff509ed960d1e455193845886d22dc7ae4e99cae3ca4181c8d7a0bb
2019-06-21 14:27:24 Prepared block for proposing at 9 [hash: 0xa68a7a123a7a270b25306aab3c2905eec5dd93f5f336bb23f5e2371367a6a4c4; parent_hash: 0x2893…a0bb; extrinsics: [0x7afd…8a65, 0xfbbe…273f, 0x2b84…aae6, 0x8ea5…529e, 0x298e…f83a, 0x1f14…eae8, 0xfc55…2c10, 0x7bee…e739, 0x81ec…3c9a, 0x03cc…a6ea, 0x7f07…81db, 0xc20d…a51e, 0x2c28…4f60, 0x671d…b1ac, 0xcb1b…fa45, 0xd7c3…9d0f, 0xd8db…9b26, 0x5f1c…1c0c, 0xbcca…da4f, 0xcbad…d4a8, 0x1ab6…2af9, 0x58f8…5d1e, 0xc82f…13d2, 0x742f…b92b, 0xa0df…15b2, 0x6d2c…f13a, 0x738d…7732, 0xe9c4…6bdb, 0xf62f…d90f, 0x611e…b195, 0xd3dd…c87b, 0xbc63…34b2, 0x2ddc…c937, 0xfd4b…1451, 0x8d28…4629, 0x6699…4e10, 0x8c6c…473a, 0xc7cf…b11d, 0xff64…4dfa, 0x2f82…4952, 0x6acb…e7c2, 0x4611…738d, 0x675b…5f04, 0xed25…b28e, 0xee42…227e, 0x92f4…6b74, 0xc1be…2621, 0x323f…0e8b, 0xa56c…f09a, 0x0fd0…2ba1, 0xb403…f9fa, 0x4d27…ff35, 0x5511…9439, 0xf0c3…6a02, 0xd2ad…86e6, 0xc808…add7, 0x048e…b963, 0x5244…19fb, 0xb14b…a5d2, 0x6f61…3397, 0xa083…463a, 0xd3a0…3266, 0x187d…1428, 0xa04d…cabc, 0x7590…ec4b, 0xff7f…8877, 0xaec4…6102, 0xe4b9…1eeb, 0x90d2…d6d1, 0xfe2a…7bc0, 0xc09f…a4bb, 0x125c…56fb, 0x2f5b…17cd, 0xf892…4194, 0xfe4d…da63, 0xe54b…11bc]]
2019-06-21 14:27:24 Pre-sealed block for proposal at 9. Hash now 0x67b495cce1770a544050f4f667b192cd8398a39d11e96776905aa23a58584a3f, previously 0xa68a7a123a7a270b25306aab3c2905eec5dd93f5f336bb23f5e2371367a6a4c4.
Runtime: invalid candidate slot
Runtime: invalid candidate slot
Runtime: invalid candidate slot
Runtime: invalid candidate slot
Runtime: invalid candidate slot
Runtime: invalid candidate slot
Runtime: invalid candidate slot
Runtime: invalid candidate slot
Runtime: invalid candidate slot
Runtime: invalid candidate slot
2019-06-21 14:27:24 Imported #9 (0x67b4…4a3f)
******* Candidate count = 64, Counter = 74, Best block = 9

from the looks of it, and what we discussed on a call, I assume you are sending fn submit_candidacy(origin, #[compact] slot: u32) with different accountIds with sequentially increasing slots, and the issue arises once we proceed to the next block.
I can't see why this (going to the next block) can cause an issue as long as you are still in the voting period. Can you maybe:

Print clearly Self::candidate_count(), Self::candidates().len() and your input slot per transaction
More importantly: print this on_oninitialize and on_finalize of the module?

cc @gavofyork

from the looks of it, and what we discussed on a call, I assume you are sending fn submit_candidacy(origin, #[compact] slot: u32) with different accountIds with sequentially increasing slots,

Correct.

and the issue arises once we proceed to the next block.

More accurate to say the issue becomes visible.

I can't see why this (going to the next block) can cause an issue as long as you are still in the voting period.

Going to the next block does not cause the problem, it's not going to the next block, ie. too many calls to this fn in a single block.
I have printed the above info (obtained from RPC) and there is no divergence in the values returned from  candidate_count() and candidates().len() - both remain static until next block is finalized.

(Self-assigning but if anyone wants to look into it sooner, go ahead)

I just tried this in a runtime test. Whatever is going wrong is not in the runtime logic.
@mattrutherford would be good if you report this again in your new set of benchmarks. Note that this is now called election module.

It's possible that the problem got fixed as part of recent changes, although I'm sure it's something that happens in the runtime as all TX from pool are received by runtime and processed with either correct result or error. I'm re-working the benchmarking setup at the moment but will try and update here once that's ready, if that's ok?

It's possible that the problem got fixed as part of recent changes, although I'm sure it's something that happens in the runtime as all TX from pool are received by runtime and processed with either correct result or error. I'm re-working the benchmarking setup at the moment but will try and update here once that's ready, if that's ok?

Yeah, no rush in it imo as it is unconfirmed.


Getitem not always work with Local storage

I want to store some data in local storage with javascript. But there is some problem - because getitem method always return 0.
Here is sample code:

import random

def rand_str():
    s = [chr(random.randint(ord('a'), ord('z'))) for _ in range(30)]
    return ''.join(s)

import streamlit as st
from streamlit_javascript import st_javascript

st.subheader("Javascript API call")

b1 = st.button('read')
if b1:
    # j_code = "console.log(localStorage.getItem('local_sandbox'));"
    j_code = "localStorage.getItem('local_sandbox');"
    v = st_javascript(j_code)
    st.session_state['text'] = f"Read from storage: {v}"

b2 = st.button('write')
if b2:
    j_code = f"""localStorage.setItem('local_sandbox', '{rand_str()}');"""
    v = st_javascript(j_code)
    st.session_state['text'] = f"Write to storage: {v}"

st.json(st.session_state)
print(f"{st.session_state.get('text')}")

Here is stdout: (1 press write, 2 press read, 1 write again, 2 read again)
Write to storage: 0
Read from storage: 0
Read from storage: 0
Write to storage: 0
Read from storage: 0
Read from storage: 0

in browser i can see that variable created - but getitem always return 0
if use string with console.log( - then in browser log shows correct values from storage
What is the reason of such behaviour?

I am getting the same problem around here in my project. Do you have any updates on how to solve it?


ignore signal for linux

prevents crashes when a lot of requests

hi @Hanul
what do you think about change it to the following code:
#if os(Linux)
let s = send(self.socketFileDescriptor, pointer.baseAddress + sent, Int(data.count - sent), MSG_NOSIGNAL)
#else
let s = write(self.socketFileDescriptor, pointer.baseAddress + sent, Int(data.count - sent))
#endif
best
dk

omg! that's cool.


Support wildcards for -Product

Fixes issue #14

@heaths,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:x: Build vssetup.powershell 1.0.49+g44b68429a7 failed (commit https://github.com/Microsoft/vssetup.powershell/commit/44b68429a7 by @heaths)

:white_check_mark: Build vssetup.powershell 1.0.50+gb2087d24fc completed (commit https://github.com/Microsoft/vssetup.powershell/commit/b2087d24fc by @heaths)

:white_check_mark: Build vssetup.powershell 1.0.51+g2d62993bf6 completed (commit https://github.com/Microsoft/vssetup.powershell/commit/2d62993bf6 by @heaths)


Add information about newly added options inside the XAML designer op…

…tions pane
Recently XAML designer team added new options for setting the default behavior to view XAML document. This PR is adding more information about those options for the customer.

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


Not working on iPhone devices

This package is not working on iPhone devices, ics file can be downloaded but not inserted into iPhone's calendar. Not working on all iOS devices.

I am facing the same issue, Has this been resolved?

I think it has not been solved. @ManishaBhandari-cyber

@nwcell

Fixed with this PR k6bOjo0SKVnGPE5j


Documentation questions/errors?

In README.md

Carthage users can point to this repository and use whichever generated framework they'd like, Moya, RxMoya, or ReactiveMoya. The full Moya framework is bundled in each of those frameworks; importing more than one framework in a single file will result in ambiguous lookups at compile time.

This appears to be true for basic usage. However, I get undefined symbol errors for Response if I don't also include Moya with RxMoya. I run into this whenever I want to create custom Rx operators for Observable<Response>. I have not run into any ambiguous lookups yet.

In docs/RxSwift.md

In the error cases, the error's domain is MoyaErrorDomain. The code is one of MoyaErrorCode's rawValues, where appropriate. Wherever possible, underlying errors are provided and the original response data is included in the NSError's userInfo dictionary using the "data" key.

The domain of the NSError that I see is Moya.Error. Also, it doesn't appear that the userInfo dictionary contains any items, and I can't see where in the code this would be the case. Maybe there is some magic happening under the hood when bridging Error to NSError?
In any case, when I need to process the response in error cases, I don't bridge to NSError and instead:
if
   let error = error as? Moya.Error,
   let data = error.response?.data
{
   // Do something with the data
}

On the error: am I right you are expecting the error's response data to be in the NSError's userInfo? I think that would make sense to do.

Well, at least my reading of the docs lead me to believe that I could extract the original response data by casting the Moya.Error to an NSError and then retrieving its userInfo["data"] value. That didn't work for me. Retrieving the response is straightforward the way I mentioned above, it's just different than what the docs say.

@scottrhoyt thank you for checking out the README. The issue you ran into with RxMoya not having Response defined stems from changes made in https://github.com/Moya/Moya/pull/563.
Previously both RxMoya and ReactiveMoya were fat (they also contained all of the Moya source code), but this change made them slim (they dont contain Moya, so are dependent on Moya being imported as well).
Not updating the README was an oversight, it should have been updated in tandem with #563. We should create a pull request to update that documentation.


Make encoders more reusable

When reusing and customizing encoders I'm limited though non exported functions or struct attributes. Example:

To overwrite command in encoders.X264ImageEncoder we must be able to set cmd in the struct X264ImageEncoder which is not allowed because it's not exported. See below's example:

import (
    // ...
    "github.com/amitbet/vnc2video/encoders"
)

// ...

type Encoder struct {
	encoders.X264ImageEncoder
	quality       int
	present       string
	vcodec        string
	fileExt       string
}

func (enc *Encoder) Init(videoFileName string) {
        // works fine, as Framerate is exported
	if (enc.Framerate <= 0) {
		enc.Framerate = 12
	}

	cmd := exec.Command(
		enc.FFMpegBinPath,
		"-f", "image2pipe",
		"-vcodec", "ppm",
		"-r", strconv.Itoa(enc.Framerate),
		"-an", // no audio
		"-y",
		"-i", "-",
		"-vcodec", "libx264",
		"-preset", enc.present,
		"-crf", strconv.Itoa(enc.quality),
		videoFileName,
	)

	// XXX: This fails as input is not exported. Rename to Input
	enc.input, _ = cmd.StdinPipe()
	// XXX: This fails as cmd is not exported. Rename to Command
	enc.cmd = cmd
}

If I implement a whole new encoder with my own Encode function, I cannot reuse the encodePPM function in encoders package, as it's also not exported.

Do you accept PRs on this?

Sure, you can send me some PR to expose what you need,
What are you working on?

I want to embed encoders.X264ImageEncoder in a custom encoder which adds additional stuff to the video as sound or text overlay. But since cmd and input are private, i'm not able to for example change the exec.Cmd for encoding.
So most simple way for now would be to export cmd as Cmd and input as Input.

@amitbet why did you close this?


Base implementation of Markdown lexical analyzer

Markdown lexical analyzer takes in a markdown file and creates a stream of tokens. This is still a working branch, but wanted to move some work into dev, then into main. Also included is an initial implementation of a github actions file to allow for running go unit tests upon each commit, push, and PR. This will improve code quality and display that functionality is passing tests.
This branch will still be worked on as the converters from markdown tokens to HTML and Gemtext files needs to be implemented
In order to run tests, pull down branch and navigate to the ianus folder. Then run
go test -v ./...
@kraileth looking for your input before merging into dev. If you approve, I will go ahead and merge it into main without approval.

Ran the tests on FreeBSD 13.0 and they all passed. Only thing that I noticed was that you accidentally committed and pushed the compiled binary. You may definitely want to remove that before merging (and probably add ianus/ianus to .gitignore?).
Couldn't figure out how to comment specifically on a binary file, so I'm including this here.

Good catch, I'll remove it tonight. Can't believe that escaped me, I'm usually pretty good at filling out gitignore or just paying attention to the status of files in magit. Probably escaped me since I have all the source in an ianus folder. Will get it removed tonight and let you review again for approval.

Ran the tests on FreeBSD 13.0 and they all passed. Only thing that I noticed was that you accidentally committed and pushed the compiled binary. You may definitely want to remove that before merging (and probably add ianus/ianus to .gitignore?).
Couldn't figure out how to comment specifically on a binary file, so I'm including this here.

Change requested has occured. Deleted the binary file.


Issue 1684: Add default impl for checkRepositoryAccess

Pull Request Description
Things made:

new endpoint /{storageId}/{repositoryId} instead of greet in MavenArtifactController
default implementation of endpoint in BaseArtifactController

This pull request closes #1684
Other PR's which can be closes after this PR: #1805, #1770
Acceptance Test

[x] Building the code with mvn clean install -Dintegration.tests still works.
[x] Running mvn spring-boot:run in the strongbox-web-core still starts up the application correctly.
[x] Building the code and running the strongbox-distribution from a zip or tar.gz still works.
[x] The tests in the strongbox-web-integration-tests still run properly.

Questions

Does this pull request break backward compatibility?

[ ] Yes
[x] No

Does this pull request require other pull requests to be merged first?

[ ] Yes, please see #...
[x] No

Does this require an update of the documentation?

[ ] Yes, please see strongbox/strongbox-docs#{PR_NUMBER}
[x] No

Thanks for your contribution, @mslowiak ! :)


init by position without meshgrid

be able to initialize large ImageData set without meshgrid --> vectorize or for loops with parallelization or numba.

this is shown in example02, added in #9


Correct default download URL's

As per this issue:
https://github.com/cake-build/website/issues/275
Extension is currently using:
// The Cake bootstrapper URIs.
  "cake.bootstrappers": {
    "powershell": "http://cakebuild.net/download/bootstrapper/powershell",
    "bash": "http://cakebuild.net/download/bootstrapper/bash"
  },

So they should be changed to:
http://cakebuild.net/download/bootstrapper/packages
http://cakebuild.net/download/bootstrapper/osx
http://cakebuild.net/download/bootstrapper/windows

Since this was handled by the redirects that @devlead has put in place, I don't think that there is anything to do here.


simulator: Only print sim state on state change

Stops the spam in the terminal

I could also do that in setters in SimulatorState if that would be preferred.

can we just print the status on any keystroke so we don't have to add a bunch of conditions for things that might change later?

can we just print the status on any keystroke so we don't have to add a bunch of conditions for things that might change later?

Not really as I would prefer the bridge to inform when the status changes.
I updated the PR and now I'm using setters in SimulatorState. Now if somebody changes the logic, they won't need to update the logging code.


CHANGELOG: prepare for 0.5.1 release

I think it would be a good idea to do a release! I've added the latest commits to the CHANGELOG.
If there are more things we should merge then we can do another release shortly after this one. We need to start getting in the habit though.

I don’t think so, I think we should get in the habit of releasing more
frequently so no reason we can’t do another one in a few weeks!
On Sun, Feb 17, 2019 at 05:38 Jordan Krage<EMAIL_ADDRESS>wrote:

@jmank88 approved this pull request.
LGTM though I haven't been watching closely enough to notice if anything
isn't mentioned.
Any reason to wait for go 1.12 release? Guessing no, since it could be
weeks, and this version already increments a version. (Did we officially
drop support for 1.9 when adding 1.11? Maybe mention that.)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/dep/pull/2108#pullrequestreview-204561689, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAOSI-NXDFae9qu_k16YcNNNzQ1jGqmoks5vOVs0gaJpZM4a_UdB
.

--
Kevin Burke
phone:<PHONE_NUMBER> | kev.inburke.com

Hi,
Is this expected to happen in the immediate future? I'm trying to port a different project that relies on dep to AArch64 and I wonder whether it's worth building dep from sources on the fly or waiting for this release.

This is blocked on #2114


When the Image Slider move to it's last index the slider does not go back to it's 1st index.

Slider Stop when it go to it's last index

Same issue on android

Fixed it by removing props "loop" or "loopBothSides", left only "autoPlayWithInterval"


Mirror ReadyCondition with the actual current condition

This patch covers two things:

Mirror() relies on sorting conditions in each condition group by their LastTransitionTime, so it swaps the RestoreLastTransitionTimes() call with the Mirror() call in the defer function

It setup ReadyCondition as Unknown instead of False, so we can avoid it to be the only one mirrored in the
top-level status: this allows the other conditions to compete for the top-level mirroring

/retest
rabbitmq failure

/test glance-operator-build-deploy-kuttl

/test glance-operator-build-deploy-kuttl


Docker

Early branch to build swan docker image using gfortran, mpi and netcdf, and push the image to docker registry using github actions.

There are conflicts there @rafa-guedes - I might leave them to you...?


Use versioned Thaliak GraphQL endpoint

2023 is approaching fast - this ensures your API usage will continue to work after the unversioned endpoint is deprecated! 😄

Thanks for the PR and reminder @avafloww! 😃
Will definitely roll this out later this evening.

This has been rolled out, you should not see any hits from us to the un-versioned endpoint anymore 🙏🏼


Add support for a=framerate

https://tools.ietf.org/html/rfc4566#page-30

LGTM

Yup. Thanks!

landed in<EMAIL_ADDRESS>


Add delay between exit of window manager and termination of Xvnc

kwin_x11's termination is not waited by startkde. Xvnc is closed
by sesman after sesman detects exit of startkde.
kwin_x11 runs indefinitely on getTimestamp due to the display is
closed. Thus kwin_x11 cannot process exit event from event loop.
Therefore, kwin_x11 will run a loop and consume a whole core of
CPU.
The fix adds a g_sleep(1000) to allow kwin_x11 to exit gracefully.
Fix #1712 kwin_x11 runs indefinitely because Xvnc exits too early

I'm still not sure why this is required to Xvnc only.

This is my post on kde-devel. In the post, I gave three solutions:

KDE: use a preemptive method to handle exit condition; or find a way to delay ending X server
Qt: provide a non-blocking getTimestamp() (may lead to many changes)
Xrdp-sesman: delay ending Xvnc; or use systemd slice to find active processes

In short, I am still working on patches for KDE now in my spare time.
The reason I create pull-request for Xrdp first is:

This is the minimum change that makes this problem gone.
I did not read about that startx will terminate all processes, which are created by it. So I guess maybe some other WM may have this issue some how. This is a precaution.
Xrdp is closer to downstream distros. Users can have fixes soon. Distros are slow on updating Qt and KDE.

Thank you!


Fields cloned using altInput: true lose custom styling

We're using a third party CMS which adds style attributes directly to fields. If we use altInput: true the cloned fields don't inherit the custom styles.
Repro Link
https://jsfiddle.net/zbrpvfn7/
Your Environment

flatpickr version used: 4.6.3
Browser name and version: All (tested in Safari, Firefox and Chromium)
OS and version: Multiple (tested on macOS Catalina 10.15.3, iPadOS 13.3.1 and iOS 13.3.1)

Related?
https://github.com/flatpickr/flatpickr/issues/359

Hi team, I'm also experiencing this - the cloned field seems to have some Bootstrap classes on it too, which isn't useful to me as I'm not using Bootstrap. I'm happy to make a PR if someone could point me in the right direction.


Read user's bio?

Is there a way to extract a user's bio text and write it into a file?

No way?

I can do it.

I added this feature, , it's about adding:
try:     bio = container.find_element_by_class_name('_tb97a')\                         .find_element_by_tag_name('span').text                         except:     print ("\nBio is empty")     bio = ""
into the get_user_info method.
I will wait for Tim to approve it.

@timgrossmann can be closed


Code block running markdown rules inside it

The following markdown code:
{ path: '**',   redirectTo: '', pathMatch: 'full' }

Gets rendered like this:

This is clearly because ** is evaluated as Markdown syntax even though it's inside a code block.

@PedroEsperanca I should work fine. I tried your code on online demo. It's good.
https://dimpu.github.io/angular2-markdown/


Added instructions to start port forwarding before trying to access Web UIs

Added instructions to start port forwarding before trying to access Web UIs, after installation via Minikube.
Also created short code to share commands with relevant section in "Accessing UIs" page.
(addresses issue #656)

This change is

Thanks!
/assign
/ok-to-test
/lgtm
/approve


[FLINK-3634] [docs] Fix documentation for DataSetUtils.zipWithUniqueId()

The example results now reflect recent improvements to the zipWith functions while contrasting the dense and sparse indexing.

Changes look good.
Shall we add that zipWithIndex requires two passes and is hence not pipelined and that zipWithUniqueId should be preferred if possible?

The documentation for zipWithIndex now references requiring two passes and notes zipWithUniqueId as an alternative.

Thanks for the update @greghogan. +1 to merge.


Updated for HAPP insulin treatments parsing

It may also make sense to look for !(current.timestamp) && current.insulin since I believe HAPP may give you the ability to edit the "Entered by" string (need to confirm that though).

Merging this for now.  Should probably revisit a possibly more general solution at some point.


operator [N] [CI] oracle-database-operator (1.1.0)

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[ ] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description about the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@psilberk you are submitting two bundles with the same name and version in two different directories: please remove the v1.1.0 directory. Also, I think the name field in the metadata section of the csv should be oracle-database-operator.v1.1.0 instead of oracle-database-operator.1.1.0.

Hi @mporrato, thanks for the help, my first approach was to use v1.1.0 but then I had an error that suggested that the field should be an int and not a string (then moving from v1.1.0 to 1.1.0). Now it the latest PR request I executed that change and still got the errors above. The v1.1.0 directory should have been removed, do you still see it?

Hi @mporrato I tested this approach. I still fail in one check and it's not clear what the error is by looking at the logs.
Thanks for you help in advance.

Hi @mporrato I tested this approach. I still fail in one check and it's not clear what the error is by looking at the logs.
Thanks for you help in advance.
@haripate @tomasbakk if you could also check this and let me know what the error seems to be I'll appreciate.
Thanks

@mporrato @haripate @tomasbakk any hints on how can I move forward? Should I just drop the contribution?

Probably the last attempt before dropping the contribution: @mporrato @haripate @tomasbakk any update or hints? Thanks

@psilberk I have very low understanding how operators work internally. I guess you are able to deploy the operator in your local/testing k8s cluster? Is the deployment stable? The kiwi test is failing to find the running pod, so it might be already shut down for some reason and the test does not find it (it just calls oc get pods ... internally) - or there might be some identifier mismatch.


Fix compile warnings.

Issue:

GCC compiler warns about "return statement prevents copy elision" like below.
Since this is a common header file many warnings like one below will be fired.

/home/cbalint/work/TVM/tvm.marlann/include/tvm/ir/attrs.h:694:29:   required from ‘void tvm::AttrsNode<DerivedType>::InitByPackedArgs(const tvm::runtime::TVMArgs&, bool) [with DerivedType = tvm::relay::Conv2DAttrs]’
/home/cbalint/work/TVM/tvm.marlann/include/tvm/ir/attrs.h:658:8:   required from here
/home/cbalint/work/TVM/tvm.marlann/include/tvm/ir/attrs.h:478:25: warning: moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
/home/cbalint/work/TVM/tvm.marlann/include/tvm/ir/attrs.h:478:25: note: remove ‘std::move’ call

Details:

GCC version used:

$ gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/libexec/gcc/x86_64-redhat-linux/10/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none
OFFLOAD_TARGET_DEFAULT=1
Target: x86_64-redhat-linux
Configured with: ../configure --enable-bootstrap --enable-languages=c,c++,fortran,objc,obj-c++,ada,go,d,lto --prefix=/usr --mandir=/usr/share/man --infodir=/usr/share/info --with-bugurl=http://bugzilla.redhat.com/bugzilla --enable-shared --enable-threads=posix --enable-checking=release --enable-multilib --with-system-zlib --enable-__cxa_atexit --disable-libunwind-exceptions --enable-gnu-unique-object --enable-linker-build-id --with-gcc-major-version-only --with-linker-hash-style=gnu --enable-plugin --enable-initfini-array --with-isl --enable-offload-targets=nvptx-none --without-cuda-driver --enable-gnu-indirect-function --enable-cet --with-tune=generic --with-arch_32=i686 --build=x86_64-redhat-linux
Thread model: posix
Supported LTO compression algorithms: zlib zstd
gcc version 10.2.1 20200723 (Red Hat 10.2.1-1) (GCC)

Origins:
Mentioned line was introduced here by @hanzz2007 on PR #6128 .
@hanzz2007 , @tqchen please help with the review.
Thank You !

warning: moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
/home/cbalint/work/TVM/tvm.marlann/include/tvm/ir/attrs.h:478:25: note: remove ‘std::move’ call

https://stackoverflow.com/questions/19267408/why-does-stdmove-prevent-rvo
I think the right solution is to just remove the std::move.

warning: moving a local object in a return statement prevents copy elision [-Wpessimizing-move]
/home/cbalint/work/TVM/tvm.marlann/include/tvm/ir/attrs.h:478:25: note: remove ‘std::move’ call

https://stackoverflow.com/questions/19267408/why-does-stdmove-prevent-rvo
I think the right solution is to just remove the std::move.

remove std::move may help in common modern compilers,  but the standard said this kind of optimization is allowed, not must, i am afraid it is not a good way to rely on certain implementations not the standard

When certain criteria are met, an implementation is allowed to omit the copy/move construction of a class object, even if the constructor selected for the copy/move operation and/or the destructor for the object have side effects. In such cases, the implementation treats the source and target of the omitted copy/move operation as simply two different ways of referring to the same object, and the destruction of that object occurs at the later of the times when the two objects would have been destroyed without the optimization. This elision of copy/move operations, called copy elision, is permitted in the following circumstances (which may be combined to eliminate multiple copies):
in a return statement in a function with a class return type, when the expression is the name of a non-volatile automatic object (other than a function or catch-clause parameter) with the same cv-unqualified type as the function return type, the copy/move operation can be omitted by constructing the automatic object directly into the function’s return value
[...]
when a temporary class object that has not been bound to a reference (12.2) would be copied/moved to a class object with the same cv-unqualified type, the copy/move operation can be omitted by constructing the temporary object directly into the target of the omitted copy/move
[...]

i suppose there are two ways to solve:

add some pragma  to inhibit this warning
neither rely on move nor rvo,  just refactor the AttrInitEntry , add ref counted internal state to delay destructor, as codes showing bellow
https://gist.github.com/hanzz2007/803b28a7915c31686cffce0ac6450b73
https://gist.github.com/hanzz2007/e42b14b51c8c9c765cca107867e64d49

i suppose there are two ways to solve:

add some pragma  to inhibit this warning
neither rely on move nor rvo,  just refactor the AttrInitEntry , add ref counted internal state to delay destructor, as codes showing bellow
https://gist.github.com/hanzz2007/803b28a7915c31686cffce0ac6450b73
https://gist.github.com/hanzz2007/e42b14b51c8c9c765cca107867e64d49

@hanzz2007 , @MarisaKirisame

Updateded PR, added a GCC specific single-line effect pragma for now.
I myself like 2. solution (cean & readable & trackable) but @hanzz2007 please make it in a separate PR.

Thank You !

Any advice?  @tqchen

let us go with the warning disable for now. Given that the move can be more efficient(as efficient as RVO) and we do not want to introduce additional complexity here

Thanks @cbalint13 @MarisaKirisame @hanzz2007


Lockpick use range short

Previously, I could use lockpicks across any distance with my hotkey. Now i practically have to stand next to the chest to open it - which kinda defeats the purpose of the hotkey imho. Is it bugged? I have no idea why this was killed.

It’s been alleged for a while that some types of direct-packet sends (dialogs, interacts, etc) which circumvent client UI mechanics are flagging accounts for suspicious behavior. In response to that, Jon and friends have been slowly migrating toward more UI interaction.
This was one of the features impacted by the latest builds. You can read up on it more in TB discord as it’s been discussed many times over the last few years.

Ok that seems fair. However, I would suggest to build this as the default with an option to revert back to the old settings. This serves as a secure basis that some people can change at their own risk.

This won't be coming back to toolbox


3D Movie Maker

What's the Product's Name?
3D Movie Maker
Describe the product in a single sentence.
Children program to make films by animating 3D characters.
When was the product launched? Provide a specific date, if possible.
Around 1995
When was the product discontinued? Provide a specific date, if possible.
It was a single release in 1995 and a few themed editions. None of editions was ever continued.
What type of product was it? App, Service, or Hardware?
Windows 95 application.
https://en.wikipedia.org/wiki/3D_Movie_Maker

Thanks for the suggestion!
Added here https://github.com/fabianoriccardi/killed-by-microsoft/commit/d9b5026966d46d73541c1146f54eb339817ed774

They just resurrected it! https://www.windowscentral.com/nerd-sniping-convinces-microsoft-open-source-3d-movie-maker-1995
https://github.com/microsoft/Microsoft-3D-Movie-Maker

Incredible 😂 however I would leave it as dead, the repository is published as "archived" and unsupported.

That's right! But maybe - in future - you could add some indication of projects that were released as open-source etc. Maybe they will do it again with other ones?

Maybe they will do it again with other ones?

Probably yes, I think that WPF and Winform are examples of this: they were released as open-source to avoid the complete and fast discontinuation.

But maybe - in future - you could add some indication of projects that were released as open-source etc

Nice idea, I will consider it!


Enable AWS SQS Long Polling.

It seems that the attribute ReceiveMessageWaitTimeSeconds of SQS queue is not configurable with Django-Q on sqs.create_queue to enable long polling. Without that, It would produce a huge amount of requests with empty receive that increases cost spent on AWS SQS.
Is there any better way to automatically configure this attribute than manually editing the existing queue on AWS console?

This issue just bit us, when we got charged on SQS for a queue that was sitting empty the whole month. The way we fixed it was to create a custom broker class that subclassed the Sqs broker class and adding a WaitTimeSeconds parameter to the dequeue method. Here's the code for the custom broker class we're using:
from django_q.brokers.aws_sqs import Sqs as DQ_Sqs

class Sqs(Sqs):
    @staticmethod
    def get_connection(list_key=Conf.PREFIX):
        return Session(region_name=Conf.SQS['aws_region'])

    def dequeue(self):
        """Custom dequeue that stops the broker from generating potentially millions of SQS requests if the queue
        is sitting empty. SQS charges all requests, even the ones that return empty responses, and this can lead
        to us being charged even when we don't do anything."""
        # sqs supports max 10 messages in bulk
        if Conf.BULK > 10:
            Conf.BULK = 10
        wait_timeout = Conf.SQS.get("wait_timeout", 0)
        tasks = self.queue.receive_messages(MaxNumberOfMessages=Conf.BULK, VisibilityTimeout=Conf.RETRY,
                                            WaitTimeSeconds=wait_timeout)
        if tasks:
            return [(t.receipt_handle, t.body) for t in tasks]

We continue to use the sqs key in the settings.Q_CLUSTER dict to configure this class, as it's easier than having to rewrite all the different configuration options.
Hope this helps anyone who comes looking for this issue.

FYI: https://github.com/Koed00/django-q/pull/506. Django-q now takes receive_message_wait_time_seconds as an argument.


[BUG] Triggering the 1750 Hz tone by pressing PTT+Fn2 also triggers scanning

Describe the bug
Some repeaters need to be "opened" by sending a 1750 Hz tone. To do this you press PTT and the Fn2 key afterwards.
The bug is that the unit is starting a frequency or channel scan, which is something the users normaly not wants to do in this scenario.
To Reproduce
Steps to reproduce the behavior:

Turn the device on
Tune to a frequency or a memory channel which allows TX
Press PTT and Fn2 in order to send a 1750 Hz tone
A scan is started which is not expected

Expected behavior
As a 1750 Hz tone is used to open the repeater, it is expected that the device stays on the currect memory channel or frequency.
Pictures/video
A video can be provided if necessary.
Details:

Firmware version: 20.1
Bandwidth: any
Power: any
RxAGC: any
Gain settings (LNAs/LNA/PGA): any
other: -

Additional context
There is a info that this bug has not existed in v19.1. If necessary, we can test which version introduced the bug.
Also it seems not every device is affected. But at least 2 known users are affected.

I have to add, that when sending DTMF tones while pressing PTT also triggers the standard function of the key, when it's being pressed a little longer. For example Pressing the number 6 for half a second (while pressing PTT) also switches the output power.
These function shouldn't be executed while pressing PTT.
Thanks guys!

https://github.com/kamilsss655/uv-k5-firmware-custom/assets/4473704/a4e9bbac-54a3-40a0-a3bb-f08f81320a12
Here's the video which show's the issue in VFO mode. The same happens also in memory mode.

More info on the version where this issue has been introduced:
With v19.10 this bug did not exist. It seems to be introduced with v20

since v20 i have a similr problem on a k6 and R5, while pressing PTT and another key, like A, B , C, D, the action of that key will happen, instead of just sending the tone.
 With 19.10 everything was fine...


Recipes and mass balancing

The furnace is 2800 and blast_furnace is 3000, therefor I believe it's appropriate for the engine to be 3200.
The anchor is iron block + 1000.

Please speak your minds about this. Even if it's not a change suggestion, please discuss any balancing concerns and ideas.

I think these masses are fine. There is a point to be made about these common parts being too heavy to operate a ship, but I can't imagine that it's that big of a problem.

I think these masses are fine. There is a point to be made about these common parts being too heavy to operate a ship, but I can't imagine that it's that big of a problem.

Made an airship using 4 balloons holding up the anchor. If anything it's still too easy. I think the balloons should get nerf'd.

Then I see no problem with this. Eureka is meant to be very easy to use and operate, and it is very configurable for the servers that do not want that.

Yes, that is why I have not changed that. but I belive a slight change might make it better. 👍

idk about the lead in the recipe, getting leads can be difficult. or you have some luck and meet a wandering trader and you can make 2 leads or you need to get slime wich sometimes is very hard.

The balloon amounts seem to me a bit extreme. While i understand that 4 strings shoudln't give allot of balloons strings are more expensive then wool in the game. that recipe seems useless based on the cost and return. For the wool imo its also extreme, i imaigne a casual player with a sheep farm of 8 big in a nice fence. This would mean that each shearing would only produce 16 balloons, this sounds too harsh for me. I agree with that the leather should have more balloon, but 14 balloons is odd. Also it would be cool to have leather make brown balloons.

The balloon amounts seem to me a bit extreme. While i understand that 4 strings shoudln't give allot of balloons strings are more expensive then wool in the game. that recipe seems useless based on the cost and return. For the wool imo its also extreme, i imaigne a casual player with a sheep farm of 8 big in a nice fence. This would mean that each shearing would only produce 16 balloons, this sounds too harsh for me. I agree with that the leather should have more balloon, but 14 balloons is odd. Also it would be cool to have leather make brown balloons.

I have been playing around in a survival world and found the numbers work well, the string recipe is not about the rarity of string but an option if you don't have the resources to craft it any other way. In my survival work, I was out of all others, but I needed some balloons for a quick nether transportation, the string I had on hand was above enough.
You can build a ship with just one balloon. I imagine that if one wants to have an oversized balloon or giant ship, one would build a farm to gather those resources. Strings are very common to find and are not an optimal resource to craft balloons.
Leather gives 14 because phantom membrane gives 16. the phantom membrane is way harder to farm or find.

idk about the lead in the recipe, getting leads can be difficult. or you have some luck and meet a wandering trader and you can make 2 leads or you need to get slime which sometimes is very hard.

The anchor is not necessary to have on your ships. Therefore it does not matter if it's a little harder to get your hands on one. slime can be commonly found in dungeon chests or can be hunted in slime chunks or swamps. Additanly, the traveling merchant is not only very common but also drops two leads when killed.
looking forward to hearing back.

I suppose the string recipe is fair. Also you are right about the anchor, I suppose its not a must thus indeed not too harsh to make it need something more rare.
I get that you would expand the farm based on the need of balloons but I still feel like 4 to 8 is a not enough.
But maybe this is just me. Il do a poll in vs2 to choose that.

Ok so the poll results are:
Wool 4
Membrane 32
Leather 4 (we need to make it start with brown color but also a way to wash? the balloon back)
Paper 2
String none (in the poll i said "the results can change the recipes not will we can keep it if we want so)

Ok so the poll results are: Wool 4 Membrane 32 Leather 4 (we need to make it start with brown color but also a way to wash? the balloon back) Paper 2 String none (in the poll I said "the results can change the recipes not will we can keep it if we want so)

The brown Leather craft would be restrictive without a way to wash/recolor the ballons.
Regarding the string recipe, you can craft string into wool.

Ok time to merge, the leather, and washing can be done in a seperate PR.


api shows wrong email address

Gitea version (or commit ref): 1.5.1
Git version:
Operating system: linux
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
The API shows my email address as<EMAIL_ADDRESS>but I changed it in my profile. Is this an error?
[
    {
        "id": 37,
        "url": "https://code.ryanburnette.com/api/v1/repos/ryanburnette/decently-smart-target/issues/1",
        "number": 1,
        "user": {
            "id": 1,
            "login": "ryanburnette",
            "full_name": "Ryan Burnette",
            "email"<EMAIL_ADDRESS>            "avatar_url": "https://secure.gravatar.com/avatar/80ca3efd5f4440f4f7103e0a3ad932bb?d=identicon",
            "language": "en-US",
            "username": "ryanburnette"
        },
        "title": "Test",
        "body": "This is a test issue.",
        "labels": [],
        "milestone": null,
        "assignee": null,
        "assignees": null,
        "state": "closed",
        "comments": 0,
        "created_at": "2018-09-09T15:55:11Z",
        "updated_at": "2018-09-09T15:55:47Z",
        "closed_at": null,
        "due_date": null,
        "pull_request": null
    }
]

Screenshots

I couldn’t replicate this though... Are there more steps to take than just updating the email address?

@adelowo  You got the correct email address in the API response?

Yes, I did...  I got the correct email in the "issue created" webhook

Hmm. Maybe try changing your email then see if you get the old one.
On Mon, Sep 24, 2018 at 17:33 Lanre Adelowo<EMAIL_ADDRESS>wrote:

Yes, I did... I got the correct email in the "issue created" webhook
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/go-gitea/gitea/issues/4899#issuecomment-424133293,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACJfOZvbKUEjmxx8y2zr1kx24E79kwYiks5ueU-7gaJpZM4WgVtq
.

@adelowo My apologies for the suggestion. This is an issue on my instance... so some set of circumstances lead to it.


Apply systray patch

Hello,
I have been using this build of dwm for a long time but I really miss having a systray in my statusbar.
We could use the https://dwm.suckless.org/patches/systray/ patch.
Or is there another way to get the systray inside the statusbar ?
How would you do need ? Just using the patch is breaking some other patch already included.

Nevermind people already proposed it before, going to apply it on a new fork for myself.


Crawling Website does not use extra headers specified in configuration

Problem Statement
In my project, we are building a Monitoring workflow to generate lighthouse reports on the website to check performance to keep track each release.
The Website in this case, uses cloudflare bot fight which has a specific user agent in the allow-list for our synthetic tools. For this reason, I am configuring unlighthouse to use that user agent to make the requests to the website.
Expected Behaviour
Upon setting the configuration lighthouseOptions.extraHeaders['User-Agent'], requests to sitemap.xml should use this User Agent so cloudflare doesn't block my agent.
Current Behaviour
Unlighthouse is always sending the 'Unlighthouse' user agent to the axios requests. This is caused by the line below:
https://github.com/harlan-zw/unlighthouse/blob/8a79f62c6bd2161f0df8ff9e3ab486a632ebcdd4/packages/core/src/util.ts#L139
When spreading the configuration, the latest item takes precedence when object attributes conflict. So in this code, my custom header for User Agent will be ignored, making sitemap.xml requests fail with 403.

I cannot add Unlighthouse as an allowed user agent in cloudflare, as It poses a risk.

Hi, please configure using the extraHeaders config, not modifying the lighthouseOptions directly.
https://unlighthouse.dev/guide/guides/authentication#custom-headers-authentication

I have set the configurations as you mentioned (I am omitting sensitive information)
My configuration file looks like this:
`const config = {
site: 'xxxx.xx',
ci: {
budget: {
performance: 0.9,
accessibility: 0.9,
'best-practices': 0.9,
seo: 0.9
}
},
scanner: {
exclude: ['/.?pdf', './amp', 'en-', '.?mp4'],
samples: 1,
sitemap: true,
robotsTxt: false,
throttle: false
},
extraHeaders: {
'User-Agent': 'XXXX'
},
debug: true
}
export default config`
My github action (Im running it in a github worflow) is still failing with the message:
[warn] [Unlighthouse] Request to site https://bujairi.sa`/ threw an unhandled exception. Please check the URL is valid and not blocking crawlers. Request failed with status code 403`
I double checked and the user agent white listed in cloudflare is correct from the configuration. It's just not receving this user agent string.

New information.
This doesnt happen when I run locally. only in github ci environment.

Hey, I think I caught the bug, can you try out v0.13.1?


feat: statically sized Vector type

This PR adds a static vector type to batteries akin to Mathlib.Data.Vector with one difference: It is based on Array instead of List. The PR attempts to make the API of Batteries.Data.Vector isomorphic to Mathlib.Data.Vector to the maximum possible extent.
For discussion see this Zulip thread

Changing all operations to array operations is my next goal. This causes quite a bit of proof breakage that will have to be fixed. It doesn't make sense to turn operations like head and cons into array operations I suppose?

It doesn't make sense to turn operations like head and cons into array operations I suppose?

head certainly! Not sure about cons. Maybe it shouldn't be there? Or at least with a warning.

(also I should mention that I'm just offering unsolicited opinions here, the maintainers have authority here)

I have started porting the large Array API under Init.Data.Array.Basic. This is a very large file, so I expect this to take some time.

I would like to merge this PR in order to avoid putting too many things. Subsequent PRs will:

Add a separate file for monadic defs and their dependents.
Port Array lemmas.

awaiting-review

A lot of the changes around newlines or whitespaces seem like they could be automatically solved by a tool like rustfmt

Added five N suffix functions to  Batteries.Data.Array.Basic

Squashed everything into one commit.

Took a suggestion by @dupuisf conveyed by @fgdorais that #800 need not depend on this PR. Hence I closed that PR  and made a simpler replacement #803

I went ahead and golfed the size proofs since that was easier than individual suggestions. You can revert any changes you are unhappy with.

I went ahead and golfed the size proofs since that was easier than individual suggestions. You can revert any changes you are unhappy with.

Another question : I wrote some of these proofs with rw or simp only, which now use simp proofs. I am not sure what the .. does and whether it makes sense to have proofs based on simp.

The .. fill in the arguments automatically. It's okay to have these proofs use simp since they are very quick.

Could you explicitly indicate in the docstrings the cost of each operation? See List.map for what I mean.

We maintained those complexity annotations where the corresponding Array docstring stated the complexity. For the rest, at some point, we just agreed that predicting whether an operation is an in-place edit is not trivial.


Modify clickable area function to accept two values - TEAM-1484

Pull Request
ClickUp link: https://app.clickup.com/t/80p6f3
Acceptance criteria

[ ] Passes all regression tests
[ ] Is cross-browser compatible
[ ] Is cross-device compatible
[ ] Fullfils General Acceptance Criteria (see Wiki)
[ ] Fullfils Specific Acceptance Criteria (see ClickUp card)
[ ] Adheres to code standards
[ ] All code reviews have been addressed
[ ] No linting errors

QA Wiki: Generic Criteria

Task linked: TEAM-1484 Modify clickable area mixin to accept two values for padding


Passing cov to extend_STR_reads

The usage and example given for extend_STR_reads in the README does not include the cov parameter unlike the actual usage. If you do run the program and exclude cov, it does run, provided there are not other options so the example given with the test dataset does actually work. This is because it just checks for the number of arguments [extend_STR_reads.c:1016 (argc < 4)].

Thanks for catching that. Changed it to check for (argc < 7).


OpenSSL 1.0.2o Compilation warning/error rsa_oaep.c: error: 'dblen' may be used uninitialized in this function

Compiler: gcc 4.8.5
With some of the recent changes added to rsa_oaep.c adding OPENSSL_cleanse() functions, it looks like a compiler warning is being generated. When used with -Werror this causes a compilation failure.

We don't support -Werror on the 1.0.2 releases.  Yes there are some code-cleanup things in that release.  I am going to close this.

-Werror works however on the 1.0.2 release before this though? It also continues to work if this one issue is fixed...

Thank you very much.

Linked PR was merged so closing.


Variable Biomes along Y axis resulting in reduced Biome information stored on disk

Starting with 19w36a Minecraft supports variable Biome along the Y Axis.
Information on disk is stored in 1024 integers instead of 256 integers like before. This is less information along X and Z axis. Now one element represents a 4x4x4 part of the Chunk. Real Biome data is interpolated inside the Minecraft Client during run-time and no longer present in the stored data on disk.
To be able to use Biome data per Block (as possible with older versions) this algorithm has to be extracted from the Minecraft source code. Also neighboring Chunks would be needed, as interpolation depends on Biome data in adjacent Chunks. This is not possible with the current Minutor architecture.
-> we will support only 4x4x4 parts as Biome

Stored data on disk:

Real Biome data used in Minecraft Server and Client:

Interesting exploration, thanks. I ran into the same thing in Mineways, and am not bothering being too clever with this stuff for now. I have to admit I don't see an obvious algorithm as to how they decide what biome to use for each individual pixel - maybe there's some noise function layered on? Ugh.

I assume that the stored data is enough to let the normal Chunk generation algorithm know how to recalculate the finer resolution. So, just repeat some part of the typical Chunk generation.
Main problem for Minutor is, that it seems to be dependent on neighboring Chunks. And we are not able to look into them during Biome loading.


Fraudulent Website Detection

Detect fraudulent websites using the Malicious and Benign Websites dataset by analyzing URL characteristics (length, structure), server attributes (location, reputation), and content features (malicious scripts, deceptive information), leveraging machine learning for effective classification.

Iam a gssoc contributor 2024, i have already worked on the issue.
@sanjay-kv please assign me this issue under gssoc level3
thank you

How come even before assigning you did PR for this?
This is not the right approach.


feat(eks): support IAM Role for Service Account (IRSA)

Using IAM User requires rotating Access / Secret key. This PR supports "IRSA" for using dynamic credentials based on IAM Role.

Hi @teevans will this ever be merged? I'm not allowed to use hardcoded keys in my env ;/

Hey @pragmaticivan - I'll try to get someone to review it this week!

Closing as stale, @carnei-ro if you're interested in this let me know and we can engage in next steps to get this validated.