Error adding metadata to a dataObject with no preexisting metadata in iRODS

If an object has no existing metadata associated with it then it appears that the object's MetaCollection is nil. If you try to add a new metadata triple then this fails because the MetaCollection is nil and there doesn't appear to be a way to initialize an empty meta collection in order to add new metadata to it.
The error message I'm seeing is:
Problem creating metadata:  2016-07-13 19:46:34.394670434 +0000 UTC: Fatal - iRods Get Meta Failed: 0x7fbfcc09cc70, None
It looks like its coming from MetaCollection.ReadMeta() at 166 (and would also come from 173 for Collections)
https://github.com/jjacquay712/GoRods/blob/master/meta.go#L149
I suspect its being thrown because the code goes all the way down to iRODS to find metadata but then still comes up empty and there isn't anything in place to handle that case and instantiate an appropriate empty MetaCollection that we can then populate going forward.
S.

Once again, thanks for the bug report and the extra effort to trace down the cause. I just committed a temporary fix, but I need to go back and add better error handling  i.e. differentiate between errors and empty meta collections.

Thanks John, I can confirm that I can now create new metadata for objects that had no preexisting metadata associated with them.
S.

Not quite sure why but this is happening again, but only for dataObjects. If I have a line as follows
mc, err := theThing.Meta()
If theThing is a new collection then err == nil and a subsequent theThing.AddMeta() succeeds, if theThing is a brand new dataObject then err != nil and theThing.AddMeta() fails with the following error:
FATAL: 2016-07-19 20:55:30.138266861 +0000 UTC: Fatal - iRods Get Meta Failed: /tempZone/home/alice/NGSData, None, -1
It looks like a new Collection with no existing MetaCollection is getting initialized correctly whereas a similar dataObject is not.

See if this fixes things: https://github.com/jjacquay712/GoRods/commit/309fd42f279f14a32b98d6fca92aa7a4bca41894

That seems to do the trick, thanks!


What would be the best way to inlcude a live wms (image) as my background / banner for my portal homepage?

I tried to include a live wms in a web map and enable automatic updating in the webmap, however, the webmap on the homepage (following directions in the "Tweaking homepage" wiki) wasn't updating. It looks like it was cached in the portal html home page.
 example WMS tile server
http://geoint.nrlssc.navy.mil/nrltileserver/wms?REQUEST=GetMap&VERSION=1.1.1&SERVICE=WMS&layers=BlueMarble24hr&bbox=-180,-90,180,90&format=image/png&width=2000&height=1000

Closing issue because we're retiring this wiki.


Mapbox wants https requests

Issue Type
[X ] Bug
Description and/or steps/code to reproduce the problem
Create an App which points to MapBox as Tilesource.
In src/main/java/org/osmdroid/tileprovider/tilesource/MapBoxTileSource.java the baseurl is pointed to http but MapBox rejects those requests. So it needs to be changed to https.
Version of osmdroid the issue relates to:
6.0.2

@sapia123 I've just checked, and I had the same result for both the current osmdroid MapBox implementation
http://api.tiles.mapbox.com/v4/mapbox.mapbox-streets-v7/1/0/0.png?access_token=my_access_token
and the latest according to the Mapbox doc
https://api.mapbox.com/v4/mapbox.mapbox-streets-v7/1/0/0.png?access_token=my_access_token
By curiosity, on which url did you have problems?

@sapia123 I meant, I had good results for both http and https URLs.
Regardless, I switched the osmdroid code to https for Mapbox in #1177.


mime/multipart: Allow limiting maximum amount of part's header data

This is feature request, to allow user of mime/multipart package's Reader to specify limit of header data loaded into memory when calling NewPart() function.
Reader.NewPart() calls net/textproto package's Reader.ReadMIMEHeader() function which without any limitation loads all header data into memory.
mime/multipart's Reader is also used in net/http package to parse multipart/form-data POST requests, which means that functions like Request.ParseMultipartForm, Request.FormValue, Request.FormFile, Request.MultipartReader are affected by this.
I believe this can be used as remote denial of service attack vector.
Possible solution for this would be limiting HTTP request body size (done by default in some servers like nginx), but this is unsuitable for cases where server needs to accept POST requests with big files (for example, file hosting service).
Similar case like this is net/http package's Server's MaxHeaderBytes field, which limits ammount of header data, but not body.

To illustrate issue, example client code exploiting this weakness: https://www.zerobin.net/?8db1a70f1608fb27#TzogmA1w8sPRNSnByLRcRrwHY0D7urscLFil6AbDdQA=
And test server's code affected by it: https://www.zerobin.net/?6f1057f7dc78ba55#4/v39opPJuwZyqEF8xft8Iqvmkv0QPyhWC189EZy+HE=
This can cause out of memory conditions therefore I recommend turning off swap on machines which run test server and running test server with nice -19.

See also #20169.


Votes can be arbitrarily extended by Manager until they meet quorum

In OptimismGovernorV5.sol, new proposals are created with a snapshot (start time) and deadline (end time):
uint64 snapshot = block.number.toUint64() + votingDelay().toUint64();
uint64 deadline = snapshot + votingPeriod().toUint64();

proposal.voteStart.setDeadline(snapshot);
proposal.voteEnd.setDeadline(deadline);
proposal.votingModule = address(module);

All significant parameters on a proposal are locked once the vote is underway:

a proposal's votingModule is immutable and cannot be changed
updates to quorum are saved historically, so that updates don't change existing proposals
votingPeriod and votingDelay cannot impact timing because calculations are performed up front and saved
all settings and options are immutably set and cannot be changed

The one exception is the proposalDeadline, which can be edited by the Manager with this function:
function setProposalDeadline(uint256 proposalId, uint64 deadline) public onlyManager {
    _proposals[proposalId].voteEnd.setDeadline(deadline);
    emit ProposalDeadlineUpdated(proposalId, deadline);
}

This allows the manager to extend the vote for an arbitrary amount of time by continually pushing back the deadline.
This is especially risky when using the ApprovalVotingModule, because there are no Against votes. If many users are against the proposal, there is no way for them to express their opinion except by not voting. However, extending the vote will inevitably lead to more awareness and a vote that is more likely to pass (either by reaching quorum or by individual options reaching their threshold).
In an extreme case, a Manager could even push back the deadline for a completed vote, reopening it after the fact. In fact, failed votes can be moved from Defeated to Active at any time. This breaks a strong user assumptions, as votes should be considered final once they are completed.
Recommendation
Only allow the deadline to be changed before the vote starts:
function setProposalDeadline(uint256 proposalId, uint64 deadline) public onlyManager {
+   require(block.timestamp < _proposals[proposalId].voteStart.getDeadline());
    _proposals[proposalId].voteEnd.setDeadline(deadline);
    emit ProposalDeadlineUpdated(proposalId, deadline);
}

this mostly depends on optimism intentions and requirements for the function. @kent @yitongzhang @charliecf
We should also consider the currently discussed introduction of againstVotes, which might reduce risk and make it similar to standard proposals.

Currently leaving this unchanged as setProposalDeadline is intended to be unrestricted in this version


Handle 404 through ActionDispatch

Rather than custom logic at the controller level

@mjgiarlo no rush.

This should not be merged until Hyrax 0.1.0 is released.

Needs a rebase/resolution now anyway

Rebased and ready to go.


AFHTTPSessionManager POST request executes successfully but calls failure block

Sometimes a POST request with AFHTTPSessionManager executes the failure block with the following error: The network connection was lost. BUT the server side code executes successfully, sending a 200 OK response. How is that possible and what to do about it? What should happen in general if a network connection is lost while the server executes a routine (successfully)? Could it be a problem with the AFNetworkReachabilityManager? I have it set up like this in the AppDelegate:
  // Monitor network connection
  [[AFNetworkReachabilityManager sharedManager] setReachabilityStatusChangeBlock:^(AFNetworkReachabilityStatus status) {
    switch (status) {
      case AFNetworkReachabilityStatusReachableViaWWAN:
      case AFNetworkReachabilityStatusReachableViaWiFi:
        DDLogInfo(@"AFNetworkReachabilityStatusReachableViaWiFi:");

        // Resume callbacks for remote operations
        [[[TSHTTPSessionManager sharedManager] operationQueue] setSuspended:NO];

        // Notify app
        [[NSNotificationCenter defaultCenter] postNotificationName:TSDidConnectToNetworkNotification
                                                            object:self userInfo:nil];
        break;
      case AFNetworkReachabilityStatusNotReachable:
        DDLogInfo(@"AFNetworkReachabilityStatusNotReachable:");

        // Suspend callbacks for remote operations
        [[[TSHTTPSessionManager sharedManager] operationQueue] setSuspended:YES];

        // Notify app
        [[NSNotificationCenter defaultCenter] postNotificationName:TSDidDisconnectFromNetworkNotification
                                                            object:self userInfo:nil];
        break;
      default:
        break;
    }
  }];

It's possible for a response to time out after the server has received all of the request.
From our contribution guidelines, please take usage questions to Stack Overflow with the "AFNetworking" tag.


application.yml: load server details from environment

Coverage remained the same at 97.552% when pulling d9af67e13b4c0cb94de5d17da00a071de707c2ef on load-server-cred-env into 66d4ad413f55e04534e11420468da8c34964edc6 on develop.


refactor: remove smoke-test-samples

IMO these are pretty obsolete (at least I did not open them for a very long time)

Sure but aren't they used to check backwards compatibility?

Sure but aren't they used to check backwards compatibility?

Probably not, at least that is also covered by the normal samples.
Basically the default samples cover that.


Boxed.cool invalid cert

Getting an HTTPS invalid cert on Boxed.cool

net::ERR_CERT_AUTHORITY_INVALID

Seems to be fixed now


Use HTTPS were available

This removes our use of protocol relative URLs (these were generating errors in PhantomJS and would break for anyone else running in a context that uses the "file" protocol).
In addition, this changes the Stamen, MapQuest, OSM, and Bing sources to always use HTTPS.
Because it is no longer used, and to discourage its future use, ol.IS_HTTPS is removed.
Fixes #3574.

Looks good to me. Please merge.


Intentfix

Task/Issue URL: Android: Protect app from crashing due to unsafe input from 3rd parties #953
Tech Design URL: N/A
CC: N/A
Android: Protect app from crashing due to unsafe input from 3rd parties.
Steps to test this PR:

Send invalid intent to app, verify the app does not crash.

Internal references:
Software Engineering Expectations
Technical Design Template

@wseemann are you open to continue working on this PR? Let me know if you are still interested or I can open it for new contributors. Thanks!

@cmonfortep I'm working on the revisions. Sorry for the delay, it's Monday so I was busy catching up on work at my full time job. I will try and get the revisions in within the next day or so.

No problem, take as time as you need here. I will be on the other side for anything you need here 👍

@wseemann Thanks so much for contributing, however, we need to decline this PR (as it is now) since we noticed the code comes from another repository and we only accept unique contributions.

@cmonfortep No problem. I figured an open source project would want to leverage code that already works from another quality open source project rather then reinvent the wheel. After all, that's kind of the point of making things open source.


Update webservice.py

except StopIteration in get_template for newer versions of python

This was fixed in https://github.com/intermine/intermine-ws-python/pull/44 but sadly not released before now, with version 1.12.0. Thanks for creating the PR regardless!


Consider relationship field when creating record

Fix #2121 bug causing the relationship field to be skipped in test at line 53
Also fixes #2141 and possibly also #2144

Hi! Please,  format your code.

#2788

Please check this comment


Refactor influxdb / grafana implementation

Refactor RC to Deployment
Adjust kube.sh script

Signed-off-by: Bryan Stone<EMAIL_ADDRESS>

This change is

Should resolve #1326.  Simplification & redo of PR #1332

@k8s-bot ok to test

LGTM, thanks for the PR. Will merge on green.


python-modules/protobuf: fix darwin build by using $CXX instead of $CC

fixes #26531 (or works around?)
Motivation for this change
This is about issue #26531. The minimal change I could make to get the package to build for me locally. I doubt it's the best way to go about it.
Things done

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

[ ] NixOS
[x] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Ran nix-shell -p nox --run "nox-review wip", which failed to what seem to be unrelated reasons:
Collecting argparse (from worldengine==0.19.0)
  Could not find a version that satisfies the requirement argparse (from worldengine==0.19.0) (from versions: )
No matching distribution found for argparse (from worldengine==0.19.0)
builder for ‘/nix/store/8nfvw7k46wpfq7gkr4q8ng6szwr60a31-python2.7-worldengine-0.19.0.drv’ failed with exit code 1
error: build of ‘/nix/store/8nfvw7k46wpfq7gkr4q8ng6szwr60a31-python2.7-worldengine-0.19.0.drv’ failed
The invocation of "nix-build -A python36Packages.gmusicapi -A python27Packages.python-axolotl -A python27Packages.tensorflowWithoutCuda -A python27Packages.protobuf2_6 -A python27Packages.edward -A python36Packages.protobuf3_0 -A python27Packages.protobuf3_0 -A python36Packages.worldengine -A python27Packages.protobuf3_2 -A python27Packages.gmusicapi -A electrum-ltc -A python27Packages.protobuf -A python36Packages.python-axolotl -A python27Packages.tensorflow -A python27Packages.tflearn -A python27Packages.gplaycli -A python36Packages.protobuf3_2 -A python27Packages.worldengine -A python27Packages.ddar /s/nixpkgs" failed

Curious travis failures: linux travis-nox-review-pr.sh nox pr fails because output exceeds 4MB, but seems fine otherwise. osx travis-nox-review-pr.sh nox pr fails because of a segmentation fault in python2.7-protobuf-3.0.2 unit test testMergeFrom (google.protobuf.internal.message_test.Proto3Test)

Closing in favour of #26709.


Allow default Tailwind's darkMode : media strategy

Describe what feature you'd like. Pseudo-code, mockups, or screenshots of similar solutions are encouraged!
I would not like to have a LightSwitch, I would rather have the UI follow the desired system option only, as tthe default Tailwind "media" dark mode strategy.
I tried setting darkMode: media under tailwind.config.cjs but got compilation issues. Am I missing something here?
What type of pull request would this be?
Enhancement
Any links to similar examples or other references we should review?
No response

There shouldn't be any part of Skeleton that would cause compilation issues for you. I'd recommend you verify all your settings in your config are set properly. Perhaps share your config here along with the error your received?
I haven't personally used the media option but in theory it should work. If not it seems like something we could support. There just hasn't been a lot of interest until now. I've marked this as a feature request to review though.

@endigo9740 you are absolutely right, it was something on my side (a node_modules cleanup did the trick).
Sorry for bothering you guys with that, great work by the way!

Glad you got it sorted! Let us know if if we can help with anything else!


Clean up and fix some netcode

Fixes some areas where SendExtraAI or ReceiveExtraAI were reading/writing data wrong
Cleans up a lot of the SendExtraAI or ReceiveExtraAI calls and removes redundant overloads and base calls

LGTM, merged!


feat: updated packages

Resolved the error:
38076: bleach <=3.1.1 resolved (3.1.1 installed)!

Does this close also #63? @CorsoGabriele

I think so


Incorrect advice regarding DNS caching

When a backend pool's server is configured with a Fully Qualified Domain Name (FQDN), Application Gateway performs a DNS lookup to get the domain name's IP address(es). The IP value is stored in your application gateway's cache to enable it to reach the targets faster when serving incoming requests.
The Application Gateway retains this cached information for the period equivalent to that DNS record's TTL (time to live) and performs a fresh DNS lookup once the TTL expires.

This information is incorrect. I have AppGw in a vnet with a linked private DNS zone. I updated the DNS records to point to a newer instance of APIM. Hours later the AppGw was still routing to the original APIM instance, despite DNS records having a 3600 TTL.
Only after I made a slight edit to the Backend pool was the DNS change recognised by AppGw and traffic forwarding update.
[Enter feedback here]

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

ID: 0373061e-623f-f6ca-7d42-e32ea0299c66
Version Independent ID: 20930963-7740-a05e-8743-2210404675da
Content: How an application gateway works
Content Source: articles/application-gateway/how-application-gateway-works.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

@adamgattsd
Thanks for your feedback! We will investigate and update as appropriate.

@adamgattsd
I'm sorry to hear that you experienced an issue with the DNS caching behavior of Application Gateway. While Application Gateway does cache DNS lookup results, it should refresh the cache once the TTL expires and perform a fresh DNS lookup. However, there may be situations where the cache is not refreshed as expected.
In your case, it seems that making a slight edit to the Backend pool forced Application Gateway to recognize the DNS change and update the traffic forwarding. This behavior is not expected, and I apologize for any inconvenience it may have caused.
If you encounter similar issues in the future, you can try restarting the Application Gateway to force it to refresh the DNS cache. You can do this with Azure CLI:
az network application-gateway stop --resource-group myRG --name myAppGw
az network application-gateway start --resource-group myRG --name myAppGw

I hope this helps! Let me know if you have any other questions.

@adamgattsd
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation


Add .js suffix to all local imports

Seems that the lack thereof breaks Next.js application build, and possibly some other builds as well.
As a distraction, Next.js development server doesn't have problems with these imports.
Anyway, I'd say it doesn't hurt to go with the most compatible option.

Here's how the failure looks like (I masked paths manually):
info  - Collecting page data .Creating cached promise
Creating cached promise
info  - Collecting page data ..Error [ERR_MODULE_NOT_FOUND]: Cannot find module 'PATH_MASKED/node_modules/workerboxjs/lib/encodeArg' imported from PATH_MASKED/node_modules/workerboxjs/lib/scopeToString.js
    at new NodeError (node:internal/errors:393:5)
    at finalizeResolution (node:internal/modules/esm/resolve:323:11)
    at moduleResolve (node:internal/modules/esm/resolve:916:10)
    at defaultResolve (node:internal/modules/esm/resolve:1124:11)
    at nextResolve (node:internal/modules/esm/loader:163:28)
    at ESMLoader.resolve (node:internal/modules/esm/loader:841:30)
    at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:76:40)
    at link (node:internal/modules/esm/module_job:75:36) {
  code: 'ERR_MODULE_NOT_FOUND'

I might also further suggest to use ESLint and Prettier for linting and consistent formatting.
There's a plugin that can be configured to require the .js extension in imports for ESLint: https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/extensions.md

Ahh nice one. Thanks. Yeah I use semistandard for linting my projects, but it's not customisable (which is why I like it).
Normally it's fine because NodeJS would fail if you don't have the .js extension, but since the tests are actually bundled and run in the browser, it doesn't catch this.
I think ESLint might be a good option in this case.


Osp gingo suite

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


Scrolling up not being detected

I have some divs with a class of .slide. Each one has a data-hue attribute. I am trying to grab the data-hue attribute when it gets to the top of the window and add it as a body class. I mostly have it working when scrolling down on the page. However, it does not work scrolling up the page.
Here is my code:
` $('.slide').each(function() {
var $element = $(this);
  $element.waypoint(function (direction) {
    var hue = $element.data('hue');
    console.log('Scroll direction: ' + direction);

    if (direction === 'down') {
      $("body").removeClass().addClass(hue);
    this.destroy();
    } else {
      $("body").removeClass().addClass(hue);
    this.destroy();
    }
  },{
    offset: "50px"
  });
});`

And a link to my test site.
http://kingswaychurch.fishhookcms.com/
I am not sure what I am doing wrong...
Any insights?
Thanks!
Shawn

Nevermind. I was using code for an older version of Waypoints.


ci: Benchmark for specific Prisma versions

Run benchmarks for specific versions instead of current checkout.
Benchmark: https://github.com/prisma/prisma/blob/master/src/packages/client/src/tests/benchmarks/huge-schema/huge-schema.bench.ts
Action: https://github.com/rhysd/github-action-benchmark
Temporary results location: https://prisma.github.io/prisma/janpio/pr/ + https://github.com/prisma/prisma/tree/gh-pages/janpio/pr

Closing as old, we can reopen if needed


Bring tag behavior in line with builtin messages

This fixes #21 -- adds a level_tag property to the message model, and adds a custom message manager with a method that automatically adds any applicable level_tag to any user-defined tags. Tag behavior now matches builtin messages. I didn't add any new tests for this functionality; I just changed the existing tag-related assertions to match the new expected behavior. I'm happy to add new tests if you think it's necessary.

I'm closing this PR so that I can submit a new one with cleaner commit history and no merge conflicts.

Hi @nthall !
Thank you. I'm just going to work on the project again during this weekend so I can merge everything, taking a look at your PR.
Thanks for your code!


Memory leak when required repeatedly

(This has been my first effort at debugging a memory leak, so if you have any advice on how I can do this better or be more helpful, please let me know!)
I think I've run into a memory leak in this module. Here's the short backstory: I work on a project, stylelint, that has a lot of tests. We end up running >20K assertions across hundreds of test files. We've been experimenting with the test runner Jest, but it always crashes on Node 4. I logged heap statistics and with each test file saw the heap growing, with inadequate cleanup, until the crash. Through some trial-and-error experiments, I think I tracked this behavior down to a dependency, cosmiconfig, that uses this module. When I change cosmiconfig's usage of graceful-fs to Node's fs, stylelint's memory problem goes away: the garbage collection adequately reduces memory usage when the heap has grown too much.
I'm not sure what the next steps would be for diagnosing this, but I can try to help if anybody has any suggestions.

This is still a problem

Yeah! This is still a problem.
@isaacs  can you please respond to people?

Still a problem for us as well (using Jest and fs-extra, which depends on node-graceful-fs)

great work! waiting for it to be released and included in Jest, etc.

It has been released, and is within semver range in jest. So you should be able to upgrade on your side (although a PR to explicitly bump itnin Jest is welcome)

Unfortunately this does not fix the issues. Jest tests, although using graceful.js in version 4.1.15, quit working with memory heap errors. The RAM consumption increases with every new test. Anything one can do to drop graceful.js on PC entirely?


Reduce default session timeout

This was previously increased to cater for forms that have large text fields and may take some time for the user to input the required information.
There are not many forms with this requirement so we can set the session timeout individually for those particular forms without having to do it globally.
https://trello.com/c/QoSSTIXg/746-default-session-timeout

Can you test the timeout warning with this timeout?


cloud I use it with docker-compose?

I want build CiCD with my phone
so,udocker cloud't use docker-compose.
when we will get udocker-compose ?

not in the plans


Handle API Key and API Secret properly

Right now when a project is set-up using the Twilio CLI we are actually setting the API Key and API Secret that the Twilio CLI is using as ACCOUNT_SID and AUTH_TOKEN in the .env file. That works for deployment and interacting with the Twilio Serverless API because it doesn't require the knowledge of the Account SID.
However, it causes two problems.

If you use context.getTwilioClient() to get a Twilio client this is going to fail for some APIs that require the knowledge of the Account SID (SMS, Calls etc.)
We cannot verify locally the Twilio Signature for protected webhooks because we need the auth token for that which we don't have in that situation.

Why do we use the Auth Token in the first place?
In the legacy Functions environment we have ACCOUNT_SID and AUTH_TOKEN available on the context object. This enabled the use of context.getTwilioClient() but people could also access these variables directly. Therefore we are setting them upon project creation in your .env file.
By default these are also read for other interactions with the Serverless API for consistency. For deploy and other commands we introduced two flags --auth-token and --account-sid to override the ones that are read from the .env file.
When the Serverless Plugin was built, this introduced another complexity. The logic that is being used right now is the following:

When serverless:init is called we'll pass username and password of the Twilio Client in the CLI to create-twilio-function. This will write them as ACCOUNT_SID and AUTH_TOKEN into .env.
When you run serverless:start both ACCOUNT_SID and AUTH_TOKEN are being read and made available on context argument inside Function executions
For deployment and other API interactions we follow the following logic:
i) if a specific project has been passed with -p or --project we will use that username and password to interact with the API.
ii) if the user passed in --account-sid and --auth-token we'll use those credentials
iii) if the user as ACCOUNT_SID and AUTH_TOKEN in the .env file we'll use those
iv) otherwise we'll fallback to the default credentials of the Twilio CLI that are under username and password.

Right now my train of thought here is:

Rename --account-sid and --auth-token for API-related commands to --username and --password for consistency and deprecate the other ones
Still keep ACCOUNT_SID and AUTH_TOKEN and set them to username and password essentially. Meaning these could be API Key and API Secret
Detect if the pair is actually API Key and API Secret and display a warning to the user for the start command that client.getTwilioClient() and protected routes might not operate as expected and that if they want to make this more realistic, they have to change to Account SID and Auth Token.

Alternatively we can fix client.getTwilioClient() by detecting in create-twilio-function already that we are setting an API Secret not an Auth Token and instead set ACCOUNT_SID to the actual account SID and introduce TWILIO_API_KEY and TWILIO_API_SECRET instead and use that in creating the getTwilioClient() locally but then we will have the issue that context.AUTH_TOKEN will be available in the deployment but not locally. And since that's a sensitive piece of information, I wouldn't want to surprise the user that it exists.
Would love to have some input from you folks on how to solve this problem.
@philnash @dprothero

I prefer if we encourage teas to use API Key and Secret. That's a better/safer way to be using credentials anyway since you can rotate them easily, and if you leak an API Key, you can just revoke it.

I understand that we want to encourage use of API Key and Secret. But for Functions we also need:

Account SID for when we use context.getTwilioClient and need an account sid in the path
Auth Token so that we can validate incoming webhooks in a protected endpoint (which should be every webhook)

I'd be happy to add an optional prompt to ask for an API Key and Secret when initialising a project with create-twilio-function, but the most important things I feel that needs is the Account SID and Auth Token
Will new Functions continue to have ACCOUNT_SID and AUTH_TOKEN? And will there be any other blessed environment variables? Should there be some sort of consistency across the CLI, these projects and the Functions deploy environment.

Like @philnash said, some APIs require an Account SID in the API path. The Twilio CLI has that aspect so we could bridge that functionality.
However, the problem would remain that ACCOUNT_SID and AUTH_TOKEN will be available in Twilio Functions during runtime if credentials are enabled (which if they use this tool to deploy, they are automatically atm).
I think the fact that we wouldn't be able to verify the signature during local development would be a minor issue as this would be something that we could send a warning message about. Something like:

WARNING: You are currently using API Keys & Secret. While this is the preferred way, it currently does not allow you to verify webhook requests. Therefore local Functions won't be verifying the request signature. Learn more at: xxxx

This has been solved for now by using ACCOUNT_SID and AUTH_TOKEN as fields for either API Keys/Secret or Account SID / Auth Token. If in local development someone uses context.getTwilioClient() with an API Key/Secret pair, it will display an error as introduced in #45.


Run directly if no line number specified

I try to use m with spring and it takes extra 3-4 seconds compare if I run tests directly
If no line number specified require file directly and it will work fater

If no arguments are passed, m shells out to rake test, no parsing at all is done. The only price you pay is of loading m into memory.
What exact commands are you comparing?

Original:
require 'm'
M.run(ARGV)

Faster:
Dir.glob('./test/**/*_spec.rb').each do |file|
    require file
end

Right, but how are you running this? Are you using rake on both situations?

I try to use spring-commands-m, generate binstub and then try to change it little.
Currently it looks like this:
#!/usr/bin/env ruby
begin
  load File.expand_path('../spring', __FILE__)
rescue LoadError => e
  raise unless e.message.include?('spring')
end

require_relative '../config/boot'

#require 'm'
#load Gem.bin_path('m', 'm')

Dir.glob('./test/**/*_spec.rb').each do |file|
  if ARGV[0]
    require file if file.include?(ARGV[0])
  else
    require file
  end
end

It's easier to type m filename then take test FILE=filename

The reason I'm asking is that running rake is not super fast. I'll take a look at this and see if there's way we can make this faster.

Cannot reproduce this, closing the issue. If it is still a problem, please open again.


Fix an error for InternalAffairs/LocationLineEqualityComparison when inside block

This PR if fix an error for InternalAffairs/LocationLineEqualityComparison when inside block.
Code to reproduce:
nodes.select do |node|
  node.first_line == nodes.first.first_line
end

Log:
An error occurred while InternalAffairs/LocationLineEqualityComparison cop was inspecting /ydah/rubocop-rspec/a.rb:2:2.
undefined method `method?' for s(:lvar, :node):RuboCop::AST::Node
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/internal_affairs/location_line_equality_comparison.rb:54:in `extract_receiver'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/internal_affairs/location_line_equality_comparison.rb:41:in `on_send'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:107:in `public_send'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:107:in `block (2 levels) in trigger_responding_cops'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:171:in `with_cop_error_handling'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:106:in `block in trigger_responding_cops'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:105:in `each'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:105:in `trigger_responding_cops'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:69:in `on_send'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-ast-1.29.0/lib/rubocop/ast/traversal.rb:158:in `on_block'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:71:in `on_block'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-ast-1.29.0/lib/rubocop/ast/traversal.rb:20:in `walk'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/commissioner.rb:87:in `investigate'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/team.rb:156:in `investigate_partial'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cop/team.rb:98:in `investigate'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:345:in `block in inspect_file'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:344:in `each'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:344:in `flat_map'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:344:in `inspect_file'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:287:in `block in do_inspection_loop'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:321:in `block in iterate_until_no_changes'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:314:in `loop'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:314:in `iterate_until_no_changes'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:283:in `do_inspection_loop'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:164:in `block in file_offenses'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:189:in `file_offense_cache'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:163:in `file_offenses'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:154:in `process_file'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:135:in `block in each_inspected_file'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:134:in `each'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:134:in `reduce'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:134:in `each_inspected_file'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:120:in `inspect_files'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/runner.rb:73:in `run'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli/command/execute_runner.rb:26:in `block in execute_runner'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli/command/execute_runner.rb:52:in `with_redirect'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli/command/execute_runner.rb:25:in `execute_runner'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli/command/execute_runner.rb:17:in `run'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli/command.rb:11:in `run'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli/environment.rb:18:in `run'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli.rb:118:in `run_command'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli.rb:125:in `execute_runners'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli.rb:51:in `block in run'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli.rb:77:in `profile_if_needed'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/lib/rubocop/cli.rb:43:in `run'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/exe/rubocop:19:in `block in <top (required)>'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/benchmark.rb:311:in `realtime'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/rubocop-1.54.0/exe/rubocop:19:in `<top (required)>'
/ydah/.rbenv/versions/3.2.2/bin/rubocop:25:in `load'
/ydah/.rbenv/versions/3.2.2/bin/rubocop:25:in `<top (required)>'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/cli/exec.rb:58:in `load'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/cli/exec.rb:58:in `kernel_load'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/cli/exec.rb:23:in `run'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/cli.rb:492:in `exec'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/vendor/thor/lib/thor/invocation.rb:127:in `invoke_command'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/vendor/thor/lib/thor.rb:392:in `dispatch'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/cli.rb:34:in `dispatch'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/vendor/thor/lib/thor/base.rb:485:in `start'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/cli.rb:28:in `start'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/bundler-2.4.10/libexec/bundle:45:in `block in <top (required)>'
/ydah/.rbenv/versions/3.2.2/lib/ruby/3.2.0/bundler/friendly_errors.rb:117:in `with_friendly_errors'
/ydah/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/bundler-2.4.10/libexec/bundle:33:in `<top (required)>'
/ydah/.rbenv/versions/3.2.2/bin/bundle:25:in `load'
/ydah/.rbenv/versions/3.2.2/bin/bundle:25:in `<main>'

RuboCop version:
rubocop -V
1.54.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.29.0, running on ruby 3.2.2) [x86_64-darwin21]
  - rubocop-performance 1.18.0
  - rubocop-rake 0.6.0
  - rubocop-rspec 2.22.0

Replace this text with a summary of the changes in your PR.
The more detailed you are, the better.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[-] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


numba.core import issues

Reporting a bug

[x ] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[ ] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

@clsmith12 thank you for submitting this. In order to debug this further, please supplement this issue with additional information:

What did you do?
What did you expect?
What happened insteadd?

Hi, thought I had written more in my original but it doesn't seem to be there...
downloaded numba zip from github (due to permissions issues with a company device I am not able to use pip install)
The numba version is 0.45.1 (and I cannot get the latest 0.52?) according to numba.version from this install. (Note, the only way I can install is using the zip file from github! )
I can successfully import numba. but I cannot import numba.core, even when making sure the folder is in sys.path.
I can, however, successfuly "import core" instead (and similar applies to the other sub modules). So by changing all numba.core to core this works. However, this means any thing I am running that uses numba.core needs changing (and same for the other sub modules)

@clsmith12 thank you for updating this. First of all, this is a very, very unusual situation you are in and probably very few people (if any?) in the Numba community have encountered this kind of situation before. I am afraid I am going to need some additional details in order to continue debugging this:

What operating system are you on?
What compilers are available to you on this system?
What link did you use to download the zip the file?
What is the name of the zip file that you obtained?
How did you obtain/install Python in the first place?
What version of Python is it?
How did you install Numba into the Python installation from the zip file? What commands did you use? How did you compile it?
How are you setting up sys.path?
What commands do you use to launch the Python interpreter?
What error messages do you see, if any?

It's OK if you don't have all the answers here, but please try to answer as best you can so we can get to the next step of debugging. Listing exact commands, their output and any error messages verbatim is likely to help. Thanks!

Are you trying to run numba from the directory in which is was downloaded? That might cause this issue. Another potential problem is that 0.45.1 didn't have numba.core, the first version of Numba that had numba.core is 0.49 which was following a large refactor when Python 2.7 support was removed. As for downloads of source, you should be able to get a zip of the latest release from here: https://github.com/numba/numba/releases

Hi, Yes this could be the issue! I'll try next time I can to see if moving
it solves it.
Where should I "extract" the zip file to to make it work best? I have
usually just installed and extracted in my working folder for that project
(as it is quite rare I need to download anything separate and this is the
first time this sort of problem occurred)
I thought that was where I got the install from, but I'll double check and
redownload when I can
Thanks for all your help so far :)
On Mon, Dec 14, 2020 at 9:54 AM stuartarchibald<EMAIL_ADDRESS>wrote:

Are you trying to run numba from the directory in which is was downloaded?
That might cause this issue. Another potential problem is that 0.45.1
didn't have numba.core, the first version of Numba that had numba.core is
0.49 which was following a large refactor when Python 2.7 support was
removed. As for downloads of source, you should be able to get a zip of the
latest release from here: https://github.com/numba/numba/releases
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/numba/numba/issues/6569#issuecomment-744322902, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHXD5GHK3CY7VPJQL5VQRL3SUXOFPANCNFSM4UWMNXBA
.

You can extract Numba anywhere you like, it then needs building, there's some instructions here. Am going to close this issue as resolved, if you encounter issues please do post to the Numba discourse forum https://numba.discourse.group or ask questions on IRC: https://gitter.im/numba/numba. Thanks!


chore(main): release 1.9.0

:robot: I have created a release beep boop
1.9.0 (2024-06-02)
Features

Ability to configure deployment environments (including their secrets/variables) (#20) (5499a84)
Ability to configure repo Collaborators (teams + users) (#18) (19b7dbc)
Ability to configure repo Variables (#19) (29ea9a8)
Adjust repo for self-management (#16) (61853a2)

Bug Fixes

Auto Docs action-docs-action workflow (#21) (156e9e7)

Documentation

Update Readme (3b8ee6e)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/actuarysailor/gha-repo-manager/releases/tag/v1.9.0 :sunflower:


Minimal code completion support

@devoncarew PTAL
This is the first very minimal cut of code completion integration.

lgtm! You may want to add a TODO to the DartCompleter, that it currently requests on each keystroke, rather than on each user completion request.

Thanks @devoncarew, wrt the comment I think you had already written one a couple of lines above :)


Feature request: Add feature to auto hide and rename nested messages

Hello,
We're using nested messages for debug information and facing two issues:

Nested Messages Auto-Expand: We need an option to set these messages to be collapsed by default.
Customizable Text for Nested Messages: Instead of the default "Took {nr} steps" text, we want to customize this text.

Proposed Enhancements:
Visibility Setting: Allow setting nested messages to be either expanded or collapsed by default.
Custom Marker Text: Enable custom text for the markers in nested messages.

Nested messages should not auto expand unless the default_expand_message parameter is set to true in your config.
For the default text customisation, it might come in the future but is not prioritised atm.

Yes, seems that we missed that config. Thanks for the help


Custom  Annotations

Hi,
is it possible to extend the template of the secret to add additional custom annotations?

Hey there,
i dont think i support custom annotations yet, just labels. But i'll look into it. maybe this can be a quick win

reasonable quick win. annotations are now merged with the default once, so you can define what you need @chrthal

Thank you!
For upgrading the api version - registry credentials must be recreated. Schema merging does not seem to work.
After recreation the custom annotations are working perfectly!


encode method generate diff vector output

So, encode method in tool.py may generate diff results after each run. add model.eval() may solve the problem. also adding a vector saving and reloading function may by helpful

Hi,
In my case the embedding is the same every time. Do you mind providing the environment you used for us to reproduce the error?


Update installation insctructions

Installation instructions needs an update to work with my app using Symfony 2.8.8.
The file M6WebDaemonBundle.php required by my AppKernel is appears only since V 1.2.0.
Installation (that works for me)
Via composer :
"require": {
    "m6web/daemon-bundle":"1.4.*"
}

then enable the bundle in your kernel:
<?php

$bundles = [
    new M6Web\Bundle\DaemonBundle\M6WebDaemonBundle()
];

I think, this will be fixed with #21 :)

Sure, I hadn't seen this PR sorry.


{Cognitive Services} Fix #22545: az cognitiveservices account identity remove: Return LROPoller

Related command
az cognitiveservices account identity remove
Description
client.begin_update returns a LROPoller instance. During instance initialization, a thread named LROPoller(xx) is created. If the command's return type is poller, azure core will call LROPoller.result() to get the final result. Thus the LROPoller(xx) thread is joined and terminated.
Without return, azure core will not wait the thread and this thread may be still active when call tearDown().
Issue: #22545
Testing Guide
Create a cognitive services, then
az cognitiveservices account identity remove -g <rg> -n <service-name>
You should see the running progress bar and the final json output.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Cognitive Services

The reason for the CI failure is because identity_remove doesn't return the poller:
https://github.com/Azure/azure-cli/blob/e1a6afe9b69d4541863fb08a194fe70d95457880/src/azure-cli/azure/cli/command_modules/cognitiveservices/custom.py#L265-L271
cognitiveservices account identity remove's PATCH request was recorded: https://github.com/Azure/azure-cli/blob/e1a6afe9b69d4541863fb08a194fe70d95457880/src/azure-cli/azure/cli/command_modules/cognitiveservices/tests/latest/recordings/test_cognitiveservices_identity.yaml#L603-L658
It's polling result 671a5b1e-ef0a-4be7-a205-025e2aa9a650's GET request was also recorded
https://github.com/Azure/azure-cli/blob/e1a6afe9b69d4541863fb08a194fe70d95457880/src/azure-cli/azure/cli/command_modules/cognitiveservices/tests/latest/recordings/test_cognitiveservices_identity.yaml#L709-L756
However, the polling request's order is wrong, as it appears after cognitiveservices account identity show's request:
https://github.com/Azure/azure-cli/blob/e1a6afe9b69d4541863fb08a194fe70d95457880/src/azure-cli/azure/cli/command_modules/cognitiveservices/tests/latest/recordings/test_cognitiveservices_identity.yaml#L659-L708
The actual order in the YAML is:

cognitiveservices account identity remove - PATCH
cognitiveservices account identity show - GET
cognitiveservices account identity remove - polling GET

Instead, the correct order should be

cognitiveservices account identity remove - PATCH
cognitiveservices account identity remove - polling GET
cognitiveservices account identity show - GET

VCRPY is merciful enough to allow that. Still, this is a dangerous usage since VCRPY has no concurrency support, meaning a corrupted YAML recording file can be generated in any live run randomly.


Add class to elm debugger

This PR adds the class __elm__internal__debugger__ to the widget that lives to the bottom-right of the screen and is used to open the debugger.
This is useful for possible overrides that users may want to do (move the debugger to the right, lift if up a little, etc).
I'm open to discussion, but this would be very useful for us, since we've a toast notification system that doesn't work well in development since the huge z-index of the said widget doesn't allow for any exact override. A class would be a huge helper, and hopefully has a minimal impact

I wonder where you would document this, so that people know this CSS class is available for modifying the debugger?

Would it be better to have an id instead of a class? Only going to be one debugger.


Running only Solid-OIDC provider without storage (with existing WebIDs only!)

It's a very early draft, this draft PR serves as an invitation to offer feedback and possibly collaborate.

I improvised with the way I'm overwriting the HTML templates. The way I do it doesn't seem to fully work, for example, I can't use variables like isBlankForm  or prefilled so I just removed them for now not to get errors. I will probably also need to replace some inputs with hidden inputs if my overwrite doesn't allow selecting any other option.

I'm not sure if putting everything needed for this in recipes is the solution. For example, for the config part of only having OIDC, which comes down to disabling access to the LDP part, it might be more interesting to just have another import option for the config/http/handler import in the CSS repository itself..
The main goal of this repository was also to showcase how the configurations could be combined, so-called recipes, so adding custom HTML templates here might be too much.

I can't use variables like isBlankForm  or prefilled

Having a look at the code, I think prefilled might have been removed from the code at some point but never updated in the templates as the fact that it disappeared didn't actually impact the result.

This PR is mostly learning by doing for me, plus I saw on the chat that someone else also needs something similar so doing it in the open might be a good idea.
Would you be able to start a branch in the CSS repo with an initial example of another import option you suggested?
I  haven't used EJS
When it comes to templates, do you think it would be possible that based on configuration, only some parts of the specific page would be rendered? I pretty much only deleted from them any functionality which I don't want to enable, doing it via config would be much cleaner.
The current state of modified /idp/register/

I still would like to remove the radio button and the checkbox, only leaving input for existing WebID and the account credentials.

Would you be able to start a branch in the CSS repo with an initial example of another import option you suggested?

I can have a look at it later, but it would be another variant of this import: https://github.com/CommunitySolidServer/CommunitySolidServer/blob/main/config/http/handler/default.json,
but then with the necessary lines removed.

When it comes to templates, do you think it would be possible that based on configuration, only some parts of the specific page would be rendered?

The only way to do this is to pass extra parameters to the templates, so these would have to be included in the configuration as well. This is done here for example: https://github.com/CommunitySolidServer/CommunitySolidServer/blob/fbe60371b9643961200c6b7aff736c6269a8331f/config/identity/registration/enabled.json#L44
But this can get ugly quite fast if you are dropping template-specific variables all over the place. The one linked above is probably going to disappear in the future specifically for that reason. It would probably be better to have specific templates for different configurations.

I agree, also templates using those variables would end up with a lot of conditionals, especially showing radio buttons or not could get pretty convoluted.

It would probably be better to have specific templates for different configurations.

I guess all of those templates could still use a common set of reusable partials/components. This way each template would only include what I need to show.

I'm not sure if I'm specifying which template should be used in the correct way. I ended up copying into my config the whole content of config/identity/registration/enabled.json just to set "HtmlViewHandler:_templates_key" to a different value. TBH I hope there is an easier way to accomplish it.

TBH I hope there is an easier way to accomplish it.

You can use Components.js overrides. See https://componentsjs.readthedocs.io/en/latest/configuration/configurations/overrides/ and/or https://github.com/CommunitySolidServer/tutorials/blob/main/custom-configurations.md#overriding-preset-values

"@graph": [
    {
      "@type": "Override",
      "overrideInstance": { "@id": "urn:solid-server:auth:password:HtmlViewHandler" },
      "overrideParameters": {
        "templates": [
          {
            "HtmlViewHandler:_templates_key": "./templates/register.html.ejs",
            "HtmlViewHandler:_templates_value": { "@id": "urn:solid-server:auth:password:RegistrationRoute" }
          }
        ]
      }
    }
  ]

results in error
Error while parsing file "/home/elf-pavlik/code/solid/CSS-Recipes/solo-oidc/config-solo-oidc.json": Invalid predicate IRI: templates

When I look at Component.js test for Override it uses prefixed predicates, which works fine.
Context generated from CSS uses Scoped Context
"HtmlViewHandler": {
        "@id": "css:dist/identity/interaction/HtmlViewHandler.jsonld#HtmlViewHandler",
        "@prefix": true,
        "@context": {
          "templateEngine": {
            "@id": "css:dist/identity/interaction/HtmlViewHandler.jsonld#HtmlViewHandler_templateEngine"
          },
          "index": {
            "@id": "css:dist/identity/interaction/HtmlViewHandler.jsonld#HtmlViewHandler_index"
          },
          "templates": {
            "@id": "css:dist/identity/interaction/HtmlViewHandler.jsonld#HtmlViewHandler_templates",
            "@container": "@list"
          },
          "gine": {
            "@id": "css:dist/identity/interaction/HtmlViewHandler.jsonld#HtmlViewHandler_templateEngine"
          }
        }
      },

I've added some config options in https://github.com/CommunitySolidServer/CommunitySolidServer/pull/1739 which should cover relevant cases.


Canary Wallet Flow

Feature description: We're implementing this feature to alert the user of any unauthorized use of one or more of their keys.
Flow 1: Setup
Within the Inheritance Planning Section:
Home Screen > Inheritance Planning > Key Security > Canary Wallets  - the following screen is to be implemented:

When a user clicks on Canary Wallets, the following screen will be shown:

The user has to be taken to the Manage Keys section from the CTA in the screen above.
In the Manage Keys section:
Manage Keys > Within settings of each key (Or Recovery Key> Settings), there will now be the option to view and fund a Canary wallet as shown in the image below:

On clicking Canary Wallet the user will be shown the following single-key wallet that they can fund to set up the Canary:

Note: Canary wallets are single-sig wallets for the key using standard single sig path and purpose.
End of Flow 1.
Flow 2: Alert and responses
If and when a key is used, the UAI will let the user know when the user logs into the app next. This will take precedence over all other UAI messages - Cc: @pratyakxx
( UAI Message Content:
Heading: Canary Key Alert
Sub-heading: Your [Key Name] may have been compromised.
CTA: Primary: Manage; Secondary: SKIP)
User clicks on "Manage" in the UAI: They will be taken directly into the key's details> Settings section with the relevant message model "If you have not made this transaction and think that the key is compromised, then replace (change key) from the Key Settings in all the associated vaults" @ASN-BitHyve
End of Flow 2.
Learn More text for Key Details section:
Health Check: Follow the steps to check if your signer is properly accessible.
Change Signer: Signer/key lost or compromised? Swap this key out and create a new key.
Settings: Access advanced feature from here.
Primary CTA: Need help? Secondary CTA: Back

Test Cases:
Flow 1: Setup
Test Case 1: Navigation to Canary Wallet Setup
Objective: Verify the navigation path Home Screen > Inheritance Planning > Key Security > Canary Wallets leads to the correct screen.
Steps:
Navigate to the Home Screen.
Select Inheritance Planning.
Go to Key Security.
Click on Canary Wallets.
Expected Result: The screen for Canary Wallet setup is displayed as per the designs.
Test Case 2: Manage Keys Access from Canary Wallets
Objective: Verify that clicking the CTA on the Canary Wallets screen navigates to the Manage Keys section.
Steps:
From the Canary Wallets setup screen, click on the CTA.
Expected Result: The Manage Keys section is displayed.
Test Case 3: Canary Wallet Option in Key Settings
Objective: Verify that the option to set up a Canary wallet is available within the settings of each key.
Steps:
Navigate to Manage Keys.
Enter the settings for a key.
Look for the Canary Wallet setup option.
Expected Result: The option to setup a Canary wallet is available for each key, including the Recovery Key.
Test Case 4: Setup Process for Canary Wallet
Objective: Ensure the user can fund a single-key wallet to set up the Canary.
Steps:
From the key settings, select the option to set up a Canary wallet.
Fund the single-key wallet as instructed.
Expected Result: The Canary wallet is successfully funded and set up.
Flow 2: Alert and Responses
Test Case 5: UAI Message Precedence and Content
Objective: Verify that the UAI message for a compromised key takes precedence over other messages and contains the correct content.
Steps:
Compromise a key.
Log into the app.
Expected Result: The UAI message regarding the compromised key is the first thing shown, with content as specified.
Test Case 6: Replace Journey
Objective: Test the journey when the user clicks "Replace" from the UAI.
Steps:
Click "Replace" on the UAI message.
Follow the prompts to change the signer.
Expected Result: The user is taken directly to the compromised key's Settings, the Change Signer button has a red dot, and upon completion, a toast message with specific instructions is shown.
Test Case 7: Ignoring Replace and Navigating Manually
Objective: Verify the user experience when not immediately clicking "Replace" and navigating manually instead.
Steps:
Ignore or close the UAI message without choosing "Replace".
Navigate to Manage Keys and then to the compromised key.
Expected Result: The Manage Keys and the compromised key should have a red dot, directing the user to change the signer.
Test Case 8: Skipping the Replace Option
Objective: Verify the flow and developer decisions needed when the user clicks "Skip" instead of "Replace".
Steps:
Click "Skip" on the UAI message.
Note the behavior and any recurring prompts.
Expected Result: Developer decisions on message frequency and red dot indicators need to be validated against implementation.
General Test Cases:
Functionality: Each button, link, and CTA works as expected without crashes or bugs.
Security: Ensure that the feature does not expose any additional security vulnerabilities, especially in the handling and storage of keys.
Usability: Confirm that the feature is intuitive and easy to use, with clear instructions and feedback.
Performance: The feature should not significantly impact the app's performance, especially in terms of loading times and responsiveness.
Compatibility: Ensure the feature works across all supported devices and OS versions.
Error Handling: Verify that any potential errors are handled gracefully, with informative messages for the user.
Each test case should be performed with a variety of keys (e.g., newly created, existing, recovery keys) and under different network conditions to ensure robustness and reliability.

Tasks:
@antuz123 to review flow and give the 2 decisions.
@summikhan11 to share the screens (final link)
@pratyakxx to review this issue, breakdown backend tasks, and assign feature to devs
@cakesoft-swati to review the test cases and test!

Three parts:

Inheritance part and redirection @Raheel1258
Canary Wallet @PraneethGunas
UAI @pratyakxx

Moving this to sprint 31, As UAI will be done in sprint 31.
For other enhancements is already a ticket

Task picked in sprint 31

[ ] https://github.com/bithyve/bitcoin-keeper/issues/4336

[ ] https://github.com/bithyve/bitcoin-keeper/issues/4517

Completed


[Bug] Minecraft 1.21.1 (with Fabric mod loader) crashes when trying to launch the game

Link to the Pastebin containing the crash log: https://pastebin.com/WwGAZCCV
Description
This bug occurred when I was launching Minecraft 1.21.1 (with the Fabric mod loader).

Minecraft Version: 1.21.1
ModLoader: Fabric
MCPitanLib Version: 2.5.0

Steps to Reproduce

Install the following mods:

Advanced XRay (Fabric) (Version: 21.0.0)
Architectury (Version: 13.0.6)
CC: Tweaked (Version: 1.113.1)
Fabric API (Version: 0.105.0+1.21.1)
Item Alchemy (Version: 1.0.5)
Litematica (Version: 0.19.50)
MaLiLib (Version: 0.21.0)
MCPitanLib (Version: 2.5.0-1.21-fabric)
Xaero's Minimap (Version: 24.4.0)

Try launching Minecraft.

Expected Behavior
I expected Minecraft to not crash when trying to launch it.

2.5.0 works on SP but prevents my server from starting up.
2.4.9 is fine on all systems

After updating this mod (MCPitanLib) from 2.5.0 to 2.5.1, this bug no longer occurs.

For Fabric 1.20.1 it is not yet visible in the Modrinth app. I'd like to test it.

And 2.5.1 is visible and it works on the server. Bug fix confirmed.
Thank you!


Jekyll Front Matter skip

Description
I am using VS Code which uses beautify for formatting document. I'am doing project with Jekyll. I think you know is is default static page generator for Github Pages.
Every document that I want to be processed by Jekyll have to include as they call it Front Matter.
---
title: This is my document
layout: default
tags: tag1, tag2
category: news
---

This is YML. It tells Jekyll meta data about this file.
If beautification is applied this turns into single line
--- title: This is my document layout: default tags: tag1, tag2 category: news ---

Which is not recognized as Front Matter by Jekyll.
So I have to turn beautification off in VS Code completely for every HTML document.
Is it possible to detect Front Matter and not to format it? Looks like there is not parameter to exclude it from formatting because it is not in HTML tag.
Before
The code looked like this before beautification:
---
layout: default
---

{{ page.title | escape }}

        {{ content }}

Expected
The code should have looked like this after beautification:
---
layout: default
---

{{ page.title | escape }}

        {{ content }}

Actual Output
The  code actually looked like this after beautification:
--- layout: default ---

{{ page.title | escape }}

        {{ content }}

Steps to Reproduce

Install VC Code
Insert code snippet
Click Command+P and type format. Select format code.

Environment
OS: macOS
Visual Studio Code
Settings
{
    "preserve_newlines": true,
    "max_preserve_newlines": 1
}

See #510.


Running tango_ros on Lenovo Phab2

Hello,
Is the app available on Google Play (version 1.0-5) tested with the Lenovo Phab2 phone? I can run the app and see the published topics in my Linux machine, but I cannot see any published data using rostopic echo command.
Thank You,
Ahmad

Hello @aseyfi , were you able to see the published data? Did you encounter some problem?

Hi @jubeira, yes, I was able to see the published data running the app on Lenovo Phab2. Somehow the topics in app's menu were getting turned off.
The other issue that I am experiencing is that when I look at the video from fisheye camera in RVIZ, sometimes I see that video freezes for a few seconds. Is it the app's issue or you think there is something wrong with my network?
Thank You,
Ahmad

We haven't seen that yet on our side, except when wifi is really bad. How did you setup your networking, could you retry with USB tethering?

@smits , Thank you for the suggestion. I used USB tethering and now I don't see the issue any more.
Thank You,

@smits , with tethering freezing the frames are not happening, but I am getting low frame rates. Would you please let me know what is the best rate that I can expect from different topics, specifically, color camera, fisheye and depth?
Thank You,
Ahmad

Which version are you using? What framerates are we talking about?
Op do 2 mrt. 2017 23:15 schreef Ahmad<EMAIL_ADDRESS>
@smits https://github.com/smits , with tethering freezing the frames
are not happening, but I am getting low frame rates. Would you please let
me know what is the best rate that I can expect from different topics,
specifically, color camera, fisheye and depth?
Thank You,
Ahmad
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Intermodalics/tango_ros/issues/143#issuecomment-283799942,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAG0m7jtlZpGdonY0BFUXy5wtGuke_9qks5rhz-RgaJpZM4L2lPH
.

--
Ruben Smits, Roboticist - Founder
+32 479 511 786
Intermodalics - Gaston Geenslaan 9, 3001 Heverlee - BELGIUM
www.intermodalics.eu

@smits , I am compiling the app based on tag v1.0-13 and run it on Lenovo Phab2 and my question is about the following topics, assuming that I subscribe for all three at the same time:
/tango/camera/color_1/image_rect/compressed
/tango/camera/fisheye_1/image_rect/compressed
/tango/point_cloud
Thank You,
Ahmad

Here are the rates we observed for the following topics:
/tango/camera/color_1/image_rect/compressed ~3 Hz
/tango/camera/fisheye_1/image_rect/compressed ~5 Hz
/tango/point_cloud ~5 Hz
For better rates, I would recommend using the non rectified images if possible :
/tango/camera/color_1/image_raw/compressed ~8 Hz
/tango/camera/fisheye_1/image_raw/compressed ~30 Hz

I'm very curious if subscribed from another node within the same device if the rates are better.
If that is true then we would know that the bottleneck is in the network but also makes it more interesting to add more logic on the device side.

Thank you @PerrineAguiar .@adamantivm, when it comes to publishing compressed color images, this line is the bottleneck and its execution takes in average about 110ms on Lenovo Phab2. I have done some experiments to solve these problem by running multiple threads to compress and publish color images here and assigning task to each thread here. By running 4 extra threads I was able to publish compressed color images @ ~15Hz. The problem is that since tag v1.0-13, CameraPublisher was used for transferring images and it makes it difficult to implement a similar multithreading scheme.
Also I have opened this issue for running android native tracing. If we can get it to work, then it would be easier to analyze the code.


Question: Replication per Table pgsql2pgsql

I don't understand Chinese, so I am not sure if this has been asked before. Is the pgsql2pgsql support specific table replication, instead of entire database?
Thanks

sorry for the late reply
currently, this tool only supports copy of whole database.


CreateFirstAdmin reset parameter

The reset parameter in the createFirstAdmin call (Signup.vue component) should not be placed in the body attribute of the request input, but rather at the root of the request input object.

Duplicate: #225
(and already fix in 2.x branch)


use ingame ids

In order to avoid confusion, I would recommend using in-game product IDs everywhere.
logistic-chest-active-provider -> active-provider-chest (and all other chests)
small-lamp -> lamp
stone-wall -> wall
solar-panel-equipment -> portable-solar-panel
fusion-reactor-equipment -> portable-fusion-reactor (and all other equipment)
category (name of recipe group) intermediate-products -> intermediate

Agreed. I started on this in the Rationals branch I think, but this should be finished out. I originally got the ids I use from the Factorio wiki, I'm not sure why they are different there.


PassCore with AD LDS

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Update:
I changed some things in my configuration. Now i get the message.
Failed to update password: The supplied context type does not match the server contacted. The server type is ApplicationDirectory.

Can you elaborate on the requirement?


PlanningCalendarRow cannot be given id

OpenUI5 version: ALL
Browser/version (+device/version): ALL
Any other tested browsers/devices(OK/FAIL):
URL (minimal example if possible):
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:
1.Add "id" property to a PlanningCalendarRow control
What is the expected result? Planning Calendar row is rendered with the given ID
What happens instead? Error loading the app
PlanningCalendarExample.zip
Any other information? (attach screenshot if possible)
This seems odd. Is that by design or is that a bug?
If this is by design, why?

The problem is not that you add an ID to the PlanningCalendarRow, but that you try to bind it.
Data binding is supported only for properties and aggregations. "id" is neither of the two, it's a special setting (search for "specialSettings") and therefore cannot be bound.

Just in case my reply didn't provide enough "why":
It's a fundamental requirement of UI5 that the "id" of a ManagedObject is know and given in the constructor and not modified afterwards. Data binding, on the other side, most of the time is asynchronous and therefore only applied later (and values might change).
As those two aspects don't fit together, we introduced the category of "special settings" for controls (or more general, for ManagedObjects) to better describe such special cases.


Shark errors out v673- failed to legalize operation

Verson 673
I get this error. :0: error: failed to legalize operation 'gpu.subgroup_mma_store_matrix' that was explicitly marked illegal

@dtroupe2 We are aware of the issue. We'll release 675 which has the fix later today.

Any news ?
Running on an 6800XT (RDNA2) I'm still having this issue on 667 (iree 367)


enhancement: one commandline, map integer to string (e.g., -cs n)

I love the commandline string at the bottom of the GUI. However, many of the options are locked to using numbers. Can those show their mapped strings?
for example:
craterstats -ep 1
would be good to show
craterstats -ep moon
or
craterstats -cs 5
would be good to show
craterstats -cs neukumivanov
This might be easy or could be very tricky. So, consider this an optional enhancement.

Possibly easy enhancement. Is there a list of all the abbreviations for the chronology systems? I was unsure on how to write the different abbreviations for the different systems like how to differentiate Moon, Neukum (1983) and Moon, Neukem et. al. (2001)

The chronology system names are taken from functions.txt which are read into the dict f in cli.main (sorry for the 1-letter name! I use them for brief scope variables)
The function decode_abbreviation() in cli.py decodes any abbreviation to an index. You could use this to expand back to the full name.

I was able to get abbreviations for all functions except Mars, Ivanov (2001) and Mercury, Le Feuvre and Wieczorek 2011 porous. Would either of you happen to know abbreviations for these two?

This is a problem with my decode_abbreviation function. "Mars, Ivanov (2001)" could be considered an abbreviation of "Mars, Neukum-Ivanov (2001)", as "Le Feuvre and Wieczorek 2011 porous" can be of "Le Feuvre and Wieczorek 2011 non-porous". I'll have to think of a way to deal with that. The intention was that you should be able to write anything unambiguous, e.g. NI01 or Moon83, and still get what you want. I didn't consider this case.
You should still find them all in the dict f in cli.main.
I should add that I also very much like the reconstruction of the command line below. Especially if it will work both ways (i.e. you can also enter a command line to change the UI and plot). An advantage of this would be that the UI is more robust against future additions of functionality... you will be able to do most of the work in the UI and add an extra option in the command line if needed.

I fixed this to resolve ambiguous names in favour of the shortest. Thus MarsIvan01 will give "Mars, Ivanov (2001)", and you need to include a differing part to get the superset name, e.g. MarsNeukIvan01 or FW11np for "Le Feuvre and Wieczorek 2011 non-porous"

The command line string is now showing abbreviations rather than index numbers. I think the pip module might need to be updated for these abbreviations to work, I'm not 100% sure though.


DynamoDB reco tool not excluding Infrequent Access usage for calculations

DynamoDB reco tool not excluding Infrequent Access usage for calculations
DynamoDB reserved capacity recommendations tool should be excluding the usage of Standard Infrequent Access tables, as these are not included in the reservations scope.
As confirmed by AWS team, this case is not considered and was suggested to raise the issue.
Version used: 1.0.0

Can you provide some sanitized billing data with Standard-IA storage in it so we can see what you see?

In addition, could you ask the AWS representative you mentioned to send details of this issue to<EMAIL_ADDRESS>This email address is only open to Amazon senders. Or, send them a link to this issue and have them reach out to me.

@switch180 Adding some sanitized CUR data with both Standard (eu-west-1) and Standard-IA (eu-central-1) data on both Read & Write consumption.
sanitized_cur_data.csv

@gcalv0  @arjanschaaf would either of you be able to checkout the code in https://github.com/awslabs/amazon-dynamodb-tools/pull/10 and verify that the patch excludes your S-IA units? The new regex passes the unit and integration tests, which are strong evidence that the code now filters out S-IA usage in its calculations.

Fixed and released. See #reco-v1.1.0


show content titles on hover of weekly cards

PR Fixes:

1 Now we are able to see content title on hover of weekly course cards
2 Some type fixes

Resolves #[1312]
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

https://github.com/user-attachments/assets/34a0d777-5c71-43d6-882b-67a7dab1d51f

Amazing

Thank you so much sir!!!


10xEngineers Repository Credentials

Cloud-V requires owner credentials for triggering the jobs when a pull requested is created. As 10xEngineers owner credentials are not added in Cloud-V, so the PR events cannot be configured as trigger for builds.

Known issue. Temporarily use non org repo.

Closing, going internal.


Range Error with this file

Hello Angus, I've obtain a Range Error (enabling Range Check Error in compiler) with this file in previous version (committed on 18/08/2024).

@carloBarazzetta  I would imagine that Angus would at least like the call stack at the point where the error occurs, so he has something to work from. Ditto for you AV problem.
If you install and enable madExcept then that will provide a bug report with all the details.

Thank you very much!
Please @AngusJohnson make a new "release" when you consider a version "stable", so I can adopt the new official release for my components.


[11.x] Upgrade WildFly Core to 3.0.10.Final

We have no 11.x PR jobs yet, so custom ones are at:
https://ci.wildfly.org/viewLog.html?buildId=79652&tab=buildResultsDiv&buildTypeId=WF_MasterIgnoreLinux
https://ci.wildfly.org/viewType.html?buildTypeId=WF_MasterIgnoreWindows&branch_WF=%2B|%2Fupcore3010%3B%2B|%2Fbstansberry%3B-|%2Fmaster-ignore
https://ci.wildfly.org/viewLog.html?buildId=79656&tab=buildResultsDiv&buildTypeId=WF_MasterElytron


Incorrect defaults shown for kubelet feature-gates reference

This is a...

[ ] Feature Request
[ X] Bug Report

Problem:
The feature-gates section in the kubelet reference is not consistent with the correct defaults.
I'm comparing https://kubernetes.io/docs/reference/command-line-tools-reference/kubelet/ with
https://github.com/kubernetes/kubernetes/blob/release-1.13/pkg/features/kube_features.go
Specifically, multiple areas of the documentation state that PodPriority is enabled by default in 1.11. However, the kubelet reference shows the default as ALPHA - default=false. This is confusing. I do not believe it is the case the apiserver and kubelet have different defaults, rather the documentation is not up to date.
Proposed Solution:
I'm happy to submit a PR for this issue, but I'd just like to make sure this issue just lies with documentation. I'd like to update the documentation for 1.13 with the 1.13 release branch for kubernetes/kubernetes, but I can start with master.
Page to Update:
https://kubernetes.io/docs/reference/command-line-tools-reference/kubelet/

cc @bsalamat

kubelet --help gives the expected defaults, so this issue is with the generated doc.

Heh, not an issue but an open PR because other folks have tried to address the issue also. See https://github.com/kubernetes/kubernetes/pull/66034. I made sure the bot wouldn't close it, or at least not for another while anyway.
@ojmhetar if you've got ideas for how to address beyond what folks have already attempted, sig-docs would most definitely appreciate it. And feel free to ping me on Slack (where I'm jrondeau) if I can help in any way.


Support realtime resource metrics without prometheus

What would you like to be added:
Support realtime CPU/RAM metrics without prometheus needing to be installed, by querying the Kubernetes metrics APIs for nodes/pods where kube-metrics-server is installed
Why is this needed:
Our clusters do not run prometheus, yet this information is still available in an alternative format.
Environment you are Lens application on:

Kubernetes distribution: Custom, running on AWS EC2
Desktop OS: macOS 10.15

Long time ago (when Lens was still closed source and was running in-cluster) we did pull all the metrics data from metrics-server. Reason why we switched to prometheus was time series data (metrics-server only stores the last value). Do you have already have an idea how Lens could use metrics-server for time series? Local cache?

There are some pages where only the last value is used (getLastMetricValues), e.g. I was thinking it'd be useful to show the last value of CPU/memory usage when listing pods/nodes, or as a fallback if timeseries data from prometheus isn't available.
I also started experimenting on a branch with running Prometheus locally, but that has a problem - that that will require a tunnel to access the metrics endpoint of every pod/node in the cluster, which doesn't actually help our internal use case where engineers only have access to the API and not to run the port-forward command.

Would love to see this implemented. Time series metrics are nice, and I understand the necessity of something like Prometheus for that, however for the occasions when you just want to see local CPU, RAM, and disk usage in real time, the k8s metrics APIs seems like they would be perfect for that. As @adammw points out, when looking at a list of pods, nodes, etc., you only care about the instantaneous metrics values. This is one of the main reasons I keep going back to k9s.

I have to agree we use some other TSDB for our metrics but it would be a great way yo have metric-server included as an options for current info

I also want to chime in here that the lack of this integration makes it hard to use lens in the same way I use k9s.  Getting a list of pods with current CPU/Mem which is updated every so often is useful when checking system health.  The time series part could be interesting, but shouldn't be a requirement, as not all clusters have prometheus.  For instance, by default a GKE cluster has a metrics-server but not prometheus.

I also want to chime in here that the lack of this integration makes it hard to use lens in the same way I use k9s.  Getting a list of pods with current CPU/Mem which is updated every so often is useful when checking system health.  The time series part could be interesting, but shouldn't be a requirement, as not all clusters have prometheus.  For instance, by default a GKE cluster has a metrics-server but not prometheus.

In our use case, we have a victoria metrics cluster for multiple k8s clusters. So we don't want to install Prometheus to each cluster

Completely agree.  We don't install the entire prometheus stack in all of our clusters.  Many development clusters are very lightweight with only the bare essentials.  Supporting metrics-server there would be awesome.

Same thing here - I don't have prometheus installed and didn't intend to, because Kubernetes Metrics Server works just fine with k9s. So I'm using both, k9s and Lens depending on situation.

We have now migrated to datadog instead of prometheus. So getting the historical values of cpu and memory would be had there, but for instantaneous usage metrics i end up having to falling back to k9s.

I use k3s , do not want to add Prometheus, just want support real time Metrics Server

This is still relevant, please it would be a big enhancement to be able to see metrics from metric-server in real-time despite of not having time series data.

Any chance we can make metrics pluggable?  For instance, if there were a way to set a metrics plugin, then as a datadog shop, I could put in an API key and pull time series data from there.  The same is probably true of other third party monitoring solutions.

it would be great if Lens could get those metrics from Datadog

subscribing here for updates
do not want to have extra prometheus in my test cluster but want to see real time metrics for nodes in lens

Metrics-server support has been added to Lens Desktop.

Metrics-server support has been added to Lens Desktop.

where?


Custom sending domain

See issue #54
This allows setting the domain option to auto so the Deliverer will automatically set the domain based on the from address.
Any feedback is welcome.

Nice PR. Bumped into the exact same issue. However, our setup uses 3 different mailgun accounts, with 3 different API Keys. I guess that can be also solved, but the current PR doesn't take this into account. The domain is dynamically set, but there is only 1 API key which is fetched from the config.
Is this project still active btw, will the PR get merged at some point?


Remove duplicated recipe maintainers key

I'm surprised this wasn't caught by the CF linter. 🤷

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@xhochy, could you merge this? I sadly can't because I'm still lacking the maintainer permissions, which I hope to gain by merging this.

@xhochy, thanks! Could you please also merge?


[JENKINS-69569] Changes the build status of previous build which gets cancelled

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[x] Link to relevant issues in GitHub or Jira
[x] Link to relevant pull requests, esp. upstream and downstream changes
[x] Ensure you have provided tests - that demonstrates feature works or fixes the issue

See discussion in Jira.


line fix

please fix the following
with open('data/cf-subnet.txt', 'wb') as fd:
to get real path

will be included in next release, unless if you push a pull request with this change then I can merge? Thanks for the help!


Create F2_23.c

The application you will be writing for is intended to run on a small, inexpensive embedded device that does not have a floating point unit. This means that the processor can only do integer arithmetic. However, the device does have a display and needs to display the result of floating point arithmetic to the user.  Because this is a small inexpensive processor there is no support for strings or any mathematical library functions. You must write all the code yourselves performing all the floating point math with integers only and no strings! Character arrays, or C strings, can't tell you their size but they do end with a '\0' character.
The requirements for the code are to write one of the functions presented below. All the functions return a boolean value indicating the success or failure of the operation. Improper data is the most likely cause for a function to fail. You may add your own additional helper functions (I had at least a half dozen) but you must include them for the review. Do not change the interface to these functions.
These two functions take as parameters two sets of mantissa and characteristic and a char array to hold the result of the arithmetic operations. The result of the add or subtract should be converted into char's and placed in the result array. The array must end with a '\0'. The 'len' parameter tells how many characters can be placed on the result array. For these functions to return true you must at a minimum store the characteristic of the result. If the result is a non-integer place as many of the digits as will fit in the result after a decimal point.
bool add(int c1, int n1, int d1, int c2, int n2, int d2, char result[], int len);
bool subtract(int c1, int n1, int d1, int c2, int n2, int d2, char result[], int len);
...
...
char answer[100];
int c1, n1, d1;
int c2, n2, d2;

c1 = 1;
n1 = 1;
d1 = 2;

c2 = 2;
n2 = 2;
d2 = 3;

//if the C string could hold at least the characteristic if(add(c1, n1, d1, c2, n2, d2, answer, 100))
{
    //display string with answer 4.166666666...
}
else
{
    //display error message
}
'''

I am working on this

The logical structure of the code is clear and relatively easy to follow with respect to the flow of operations.
Functions follow provided interfaces to ensure they return the appropriate boolean values.
The error messages should be clear so that the users will be able to understand what the problem is we need to make sure it includes all the errors possible and it doesn't miss any error.
Complex parts can be modularized into smaller functions that are reusable, hence improving readability and making the code easier to test and update.
More comments are needed to explain what each function is trying to achieve and how that fits into the overall architecture of the program; this would prove helpful for a future developer that may work on this project.


Added identity collections during build to avoid instance loops

Addresses #66 and #76 by storing and checking an object identifier collection during serialization.  If an object is not found in the corresponding collection then it is either written to or read from the Parcel.  If it is found in the corresponding collection, we avoid writing it to the Parcel (we write the identifier instead) and read it from the collection instead.
This handles the following cases:
1: A loop in the instances:
A a = new A();
B b = new B();
a.b = b;
b.a = a;

See Test: https://github.com/johncarl81/parceler/blob/graph_map/examples/test/src/test/java/org/parceler/CircularReferenceTest.java#L138
2: Objects referenced in multiple spots:
Five five = new Five();
Six six = new Six();
five.a = six;
five.b = six;
five.c = six;

See Test: https://github.com/johncarl81/parceler/blob/graph_map/examples/test/src/test/java/org/parceler/CircularReferenceTest.java#L194
Worth noting, this does not handle cases of loops through @ParcelConstructor annotated constructors or @ParcelFactory annotated methods and an exception is thrown if this case is detected:
1: @ParcelConstructor:
Eight eight = new Eight();
eight.name = "8";
Seven seven = new Seven(eight, "7"); // @ParcelConstructor
eight.seven = seven;

Parcels.unwrap(ParcelsTestUtil.wrap(seven)); // Throws exception

Eight unwrappedEight = Parcels.unwrap(ParcelsTestUtil.wrap(eight)); // Successful.

See Test: https://github.com/johncarl81/parceler/blob/graph_map/examples/test/src/test/java/org/parceler/CircularReferenceTest.java#L256
2: @ParcelFactory:
Ten ten = new Ten();
ten.name = "10";
Nine nine = Nine.build(ten, "9"); // @ParcelFactory
ten.nine = nine;

Parcels.unwrap(ParcelsTestUtil.wrap(nine)); // Throws exception

Ten unwrappedTen = Parcels.unwrap(ParcelsTestUtil.wrap(ten)); // Successful

See Test: https://github.com/johncarl81/parceler/blob/graph_map/examples/test/src/test/java/org/parceler/CircularReferenceTest.java#L326

I think the field setting is missing from the "Objects referenced in multiple spots" example, and also the ordered bullets are all 1s.

Thanks @WonderCsabo, updated.


Response payload size issue

Response payload size is 24813879, and has exceeded the limit of 16777216. Please consider querying less data at a time and make paginated call if needed...."
How to fix it?

Hi @atulgcp ,
I would love to understand a little more details.

Where do you get this message?
How many Subscriptions do you query with the workbook?

Hi,
I received this message when go to list out orphaned subnetworks.
roughly 300+ subscriptions.

Does this issue occur only with orphaned subnets, or does it affect other resource types as well? And did you still encounter this error when selecting a single subscription?


Some ideas about extending the GP modular

I will do most of the following things in the order.

Sparse inference (batch update)
Stochastic inference (online/streaming update)
GPU related stuff
deep GP
Inference for structure GP
EP inference
Inference for multi-class classification

Working in progress
(ref https://github.com/shogun-toolbox/shogun/issues/2779)
Sparse inference (batch update)

sparse KL inference for regression (done)
sparse KL inference for classification (this summer)
sparse KL inference for dimension reduction (this summer)

Stochastic inference (online/streaming update)

stochastic KL inference for regression (this summer)
stochastic KL inference for classification (this summer)

To do (next year)
GPU related stuff
GPU speed up for some inference methods (need some help from @lambday )
Deep GP and beyond (deep probabilistic graphical models)

extending GPLVM (one hidden layer) to deep GP

Inference for structure GP
implementing inference methods base on Andrew' work (http://www.cs.cmu.edu/~andrewgw/)
*

in future (once I get time)
EP inference for t distribution

Robust/parallel EP inference for t distribution
EP inference for sparse GP

Inference for multi-class classification

@karlnapf
take a look at this

I would like to add some non-algorithm related points.

Benchmarks against other toolboxes. We want Shogun to be among the fastet ones, while having the most complete set of algorithms. This is really important
Notebook for all methods, with clear examples that maybe reproduce experiments from the relevant papers. Users get confused if we just offer 20 different inference methods. Need some decision tree what to use.
GPU and linalg stuff should be done on-the-fly. Chances are that most of the code is not touched again in quite a while, so I would rather go a bit slower but make full use of the linalg framework.

@karlnapf
I agree with you. Yes, we do need some user guide.
One question about benchmarks
We will compare the GP modular with three GP tools (GPML, GPy and GPStuff).
I want to users can reproduce benchmark results, I may need some blog or framework to share benchmark Matlab codes for GPML and GPStuff.

User guide:
I suggest a tutorial style notebook, like the one you have, but with way less math and more hands on.
Benchmarks:
@lambday did some benchmarking, and I think we could use his ideas to do this in a principled way. Thoughts @lambday  ?

@karlnapf
@lambday
For GPU speedup, the first thing I need is the Cholesky decomposition.

@Ialong also needs this for some work he is doing
@yorkerlin you could also try to add it yourself.  Or @lambday can do it, he is fast in this stuff :)


Edit the README.md file

Issue Title
Edit the README.md file
Description

The README.md file requires some editing. The main heading and description.
The JS notes are currently in alphabetical order. They should instead be in order of difficulty (basics to advanced).
This is my first time contributing to open source, hence why I found an easy issue. 😅

Screenshots

Tagging Reviewers
Requesting review from maintainers who can provide guidance and/or permission: @allanoguis

go ahead @khd33j

don't forget to leave a star and if you have projects I can work on please join our discord
https://discord.gg/3Y4HsvxX