Display the users global rating and games played above the list of open lobbies

Many lobbies require you to have a specific rating or game count to join. To speed up the process of identifying which lobbies you can join, i'd suggest to write the users global rating and games played above the list of open lobbies.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Or colour the games green/red (maybe other colours ) to provide visual support ....

If there is no way to enter that game then it's useless to the user. It shouldn't pollute his  viewing space. Coloring does seem like a good idea but gray is more of a indicator of uselessness/ that something is disabled. But what about the users that don't see colors?

For now, nobody is prevented from joining a game even if his rating doesn't match. Once the server supports min/max ratings, we can add additional options. But right now we don't even know what the requirements are, because all we can do is parse the game title - not very reliable.

Do will still want this? From the recent discussions about rating (and trying not to put that much emphasis on it), I'd say showing it in the profile information (2 more clicks) should be enough.

no i think we dont want it


Why is the INT8 range [-127,127] instead of [-128, 127]

As in this line of plugin code:
https://github.com/NVIDIA/TensorRT-LLM/blob/main/tensorrt_llm/quantization/functional.py#L63

layer.get_input(0).set_dynamic_range(-127, 127)

and this line of the cuda kernel code: https://github.com/NVIDIA/TensorRT-LLM/blob/main/cpp/tensorrt_llm/kernels/layernormKernels.cu#L175

const float dynamic_per_token_scale = 127.f / abs_max_f;

Shouldn't [-128, 127] make the model more accurate?
To utilize the range [-128,127], one could do scale = 255 / (abs_max_f * 2) instead.

Yes, you are right that the range of [-128, 127] has one more fixed point than [-127, 127] and has better accuracy theoretically. However I think the improvement is minor cus 1. We already have 255 fixed points. 2. This only affects when negative value range is larger than positive.

Thanks for the reply @Tracin. Is there any downside to changing it to [-128, 127]? I am thinking of implementing this locally.

I am not sure about the downside of this modification, have you verified the improvement of this changing? Like any comparison using Pytorch or something.

When simulating it using Tensorflow, I observed a 1% change in evaluation.

That is interesting, what is the model and benchmark dataset?


[FEATURE]Integrating Conversation and Retrieval QA Chains in Flowise Chatbots

I am working on a Flowise chatbot and have created two chain flows. The first is a conversation chain that begins when the user starts a conversation, collecting their email and name. The second is a Conversation Retrieval QA chain that assists with questions related to the website or documents. My challenge is connecting these two nodes and ensuring they function correctly, so that the user's questions are routed to the Conversation Retrieval QA chain.

this can now be achieved using the new seq agent approach


[PLAT-5669] Move Carthage to BK, ensuring it uses Xcode 11

Goal
Moves away from Travis by moving remaining tests to buildkite, ensuring they'll work correctly by forcing the usage of a compatible version of Xcode.
Targeting the barebones-pipeline branch as a base, as both will potentially require some updating of github status settings before merging.

Moves Carthage static build to buildkite
Uses DEVELOPER_DIR environment variable to enforce Xcode 11 usage
Removes Travis CI file

This I like. :-)
Nick's suggestion above seems reasonable - presumably it just a matter of jumping on the box(es) quickly and renaming the folder?  Anyway, if he's happy then I'm happy.

This I like. :-)
Nick's suggestion above seems reasonable - presumably it just a matter of jumping on the box(es) quickly and renaming the folder?  Anyway, if he's happy then I'm happy.

Do we know that changing the directory name won't break other things relying on the Xcode tools?

Do we know that changing the directory name won't break other things relying on the Xcode tools?

Do we know that changing the directory name won't break other things relying on the Xcode tools?

We don't have any hard-coded paths to Xcode so as long as xcode-select -p returns a valid path nothing should break.

Do we know that changing the directory name won't break other things relying on the Xcode tools?

We don't have any hard-coded paths to Xcode so as long as xcode-select -p returns a valid path nothing should break.


Passport js + Fitbit + Webpack : Cannot read property 'prototype' of undefined

I am trying to authenticate Fitbit using Passport strategy for Fitbit OAuth 2.0.
My application is React + Express 4 web application and doing this authorization in react component's constructor.
connect.js:33 Uncaught TypeError: Cannot read property 'prototype' of undefined
See details on Stackoverflow

Same error here, but with Vue.js - Express, passport-jwt strategy.
Uncaught TypeError: Cannot read property 'prototype' of undefined at eval (eval at exports.__monkeypatchNode , bug on 'http.IncomingMessage.prototype.login ='
exports.__monkeypatchNode = function() {
  var http = require('http');
  var IncomingMessageExt = require('../http/request');

  http.IncomingMessage.prototype.login =
  http.IncomingMessage.prototype.logIn = IncomingMessageExt.logIn;
  http.IncomingMessage.prototype.logout =
  http.IncomingMessage.prototype.logOut = IncomingMessageExt.logOut;
  http.IncomingMessage.prototype.isAuthenticated = IncomingMessageExt.isAuthenticated;
  http.IncomingMessage.prototype.isUnauthenticated = IncomingMessageExt.isUnauthenticated;
};


Update DVMP printer columns

Description
Check each of the following when appropriate to help reviewers verify work is complete.
Modifying an existing version

[ ] I modified operator permissions in the OLM CSV and operator.yml
[ ] I modified the operator deployment in the OLM CSV and operator.yml
[ ] I modified operand permissions in the OLM CSV and ansible role
[x] I modified CRDS in the OLM CSV and ansible role

Adding a new release version

[ ] I created a new z release directory in deploy/olm-catalog/crane-operator
[ ] I updated channels in the crane-operator.package.yaml
[ ] I created a new release directory in deploy/non-olm
[ ] I created or updated the major.minor link in deploy/non-olm
[ ] I updated the spec.skips parameter in the CSV

Related controller PR https://github.com/konveyor/mig-controller/pull/1101


Configure admin authentication to point to ROC Active Directory

Ensure that ROC admins can be authenticated against AD

so spent a while trying to set up this for staging, looks like our application asks for something call 'allatclaims' which doesn't seem to exist on the resource in staging. not super clear what that allatclaims is maybe @bion has some context. i added an api scope with 'allatclaims' and updated the manifest but couldn't get it to work by EOD. Also FWIW i think a lot of this stuff has to be done in the portal as i don't see anything in the docs https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/active_directory_domain_service that reference active_directory outside of these too maybe @shubha-rajan can sanity check me though!

you can see the error on the staging instance if you log in as an admin with your google account :
Message: AADSTS650053: The application 'sgtest-full-mammal' asked for scope 'allatclaims' that doesn't exist on the resource 'xxxxxx'. Contact the app vendor.

for staging: https://github.com/seattle-uat/civiform/pull/1857 there is a readme here that should explain how roc will do this


Prevent reduce receiving an empty list

Coverage increased (+0.04%) to 88.413% when pulling 3a047c99ba3f1a40f30b14f5036f394a4fecf68c on SmileyJames:master into 62bedeb5df13e01506a883a5f04f543a05da315e on inonit:master.

Coverage increased (+0.04%) to 88.413% when pulling 63c8fec2e245eebe3e81f9c88b3625f424e786a1 on SmileyJames:master into 62bedeb5df13e01506a883a5f04f543a05da315e on inonit:master.


"Already exists" error message. Save some files of the archive on filesystem but not in DB

On some occasions when I try to add a new archive I receive an "Already exists" message and I can't see the new archive on the page.
Although, I see that some files of the archive it's really saved successfully.
More info
Already exist message error

archivebox add command output
[+] [2023-03-18 20:21:46] Adding 1 links to index (crawl depth=0)...\n    > Saved verbatim input to sources/1679170906-import.txt\n    > Parsed 1 URLs from input (Generic TXT)\n    > Found 0 new URLs not already in index\n\n[*] [2023-03-18 20:21:46] Writing 0 links to main index...\n\r    √ ./index.sqlite3\n

archivebox server

It's a known bug, the error message itself may also be the a bug. If you manage to debug it please share your findings.

Sorry for the delay. I was trying to replicate the error and that was I found:
Problem
The problem comes from the archivebox output because in both cases (Bookmark app and Terminal) the command archive the website, but in the Elixir app, the output given is randomly wrong.

Elixir executes the command archivebox add

  def archivebox(url) do
    command = "cd #{directory()}; archivebox add #{url}"

    System.cmd("sh", ["-c", command], cd: directory()) <--------
  end

The command is executed, the website is archived (in fact, System.cmd() takes a long time to download the files), but the output received in Elixir indicates that the file already exists (this output shouldn't allow archive the website, therefore, the problem is that archivebox is giving a wrong output)

[+] [2023-03-18 21:09:04] Adding 1 links to index (crawl depth=0)...
    > Saved verbatim input to sources/1679173744-import.txt
    > Parsed 1 URLs from input (Generic TXT)
    > Found 0 new URLs not already in index <------ This output is wrong.

[*] [2023-03-18 21:09:04] Writing 0 links to main index...
    √ ./index.sqlite3

The Elixir app expects this output, but how it is not given from archivebox, launch the "Already exists" message:

    > Saved verbatim input to sources/1679258473-import.txt
    > Parsed 1 URLs from input (Generic TXT)
    > Found 1 new URLs not already in index     <-----------  Expected output behaviour

[*] [2023-03-19 20:41:13] Writing 1 links to main index...
    √ ./index.sqlite3

[▶] [2023-03-19 20:41:13] Starting archiving of 1 snapshots in index...

[+] [2023-03-19 20:41:14] "bitcoinnews.com/women-bitcoin-adoption"
    https://bitcoinnews.com/women-bitcoin-adoption/
    > ./archive/1679258473.967474              <---------------- Bookmark needs this line to get the identifier
    .....

Posible workaround
I try to change the function System.cmd to System.shell :or.cmd, but I the result was the same....
Finally, I found the IS_TTY flag (interactive session) in the archivebox configuration.
Disabling the isatty option, ArchiveBox assumes that its output is being redirected to a file or another process and adjusts its output behavior accordingly.
This works for me at the moment (I couldn't replicate the wrong behavior until now).
I'm going to create a pull request if you are okey.

That's really great, thank you so much for debugging that, your PR is very welcome.


PUPIL-751 feat(options): ensure tier link link to correct tier

Description
Music year: 1978

List of changes

Issue(s)
Fixes #
How to test

Go to https://deploy-preview-2458--oak-web-application.netlify.thenational.academy/pupils/programmes/combined-science-secondary-year-10/options-l
Click on tier links
You should direct to correct tier page

Screenshots
How it used to look (delete if n/a):
{screenshots}
How it should now look:
{screenshots}
Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

:tada: This PR is included in version 1.448.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add nucleus operator API

/assign pmorie
/assign deads2k

looking pretty close to me.

I have followups I want quickly, but I'm willing to merge this to unblock progress.
/lgtm
/hold
holding for pmorie to have a say.

/hold cancel

/approve

/lgtm


Socket error in logs

2023-08-05 17:21:43 (node:1) Warning: An error event has already been emitted on the socket. Please use the destroy method on the socket while handling a 'clientError' event.

No idea what this is, don't care.


Selection of folder when bookmarking reverts immediately

This is a Bug Report
Description
When using the Blue Star icon in the address bar to bookmark a new site, if I change the folder location it immediately reverts to the last selection. Doesn't matter what I select, it always reverts. However if I use the "Choose ..."option, then I can select a different folder and it will stay selected (and remembered until next time). Note that I do have "Last folder used" set in the config. This used to work as I have used this extension in Firefox for a long time now.
Steps to reproduce the problem

Click Blue Star icon in address bar to bookmark
Change the Folder by selecting any other location shown

Expected behavior
I should be able to select a different location.
Specifications

Version of the add-on: 2.12.1
Firefox edition: Firefox (regular edition)
Firefox version: 73.0.1
Firefox language: English
Platform/OS: Windows
Platform/OS version: Windows 10 1909

Thank you for reporting the issue David! 🙂 I was able to reproduce it and will be looking into a fix as soon as possible.


DOCS: README.md - ap-southeast-3 mentioned twice

Towards the top of the readme I can see the following sentence:

The AWS Cloudformation template provided in this sample does not work in ap-southeast-3, af-south-1, ap-northeast-3, ap-southeast-3, me-south-1 and us-gov-east-1 AWS regions due to service limitations.

ap-southeast-3 is mentioned twice. is one of these mentions maybe supposed to be ap-southeast-1 or ap-southeast-2?

Yes, it is meant for ap-southeast-1


Bump crypto version

Describe the bug
I don't think that tusd uses crypto/ssh (and is therefore not really subject to this CVE) however from a static analysis standpoint it'd be nice if we could pull in a version of this library that isn't flagged as vulnerable.
I'm not super familiar with golang, so not sure how hard this is to do, as this is a dependency of a dependency.
To Reproduce
See https://security.snyk.io/vuln/SNYK-GOLANG-GOLANGORGXCRYPTOSSH-551923

To be honest, I am not an expert with Go dependencies. I tried to follow the steps in https://medium.com/@majdasab/how-to-upgrade-an-indirect-dependency-in-golang-988751f92a6e to require a version of crypto that fixes this issue, but the Go tool chain always overwrites my changes. So maybe we already use a new enough version. But I personally do not know how to tackle this.

@Acconut thank you so much for looking into this. I've also chatted with golang folks at my company it and it sounds like this is not really possible (and is a widespread issue in the go ecosystem). I'm going to close this for now. Thanks again for looking into it ❤


Update deployment to fly.io guide

https://docs.platformatic.dev/docs/guides/deployment/deploy-to-fly-io-with-sqlite still reference platformatic.db.json and was written in the pre-runtime era.

Will be delivered next week, added as a priority high task.

Created a PR for this.


Upgrade Spark 3.0.2 to 3.0.3

1. Describe the feature
2. Motivation
3. Describe the solution
4. Additional context

I would like to do it.

@zhaomin1423 thanks for the help


Same log level in different channels

Laravel Version: 5.7.27
PHP Version: 7.2.14
Database Driver & Version:

Description:
There are two channels with same driver=monolog and handler (for example StreamHandler), but different levels (debug, error). After configuration handlers for this channels will be have same log levl debug
Steps To Reproduce:
config/logging.php
'channels' => [
        'stack' => [
            'driver' => 'stack',
            'channels' => ['stderr', 'stdout'],
        ],
        'stderr' => [
            'driver' => 'monolog',
            'level' => 'error',
            'handler' => StreamHandler::class,
            'formatter' => env('LOG_STDERR_FORMATTER'),
            'with' => [
                'stream' => 'php://stderr',
            ],
        ],

        'stdout' => [
            'driver' => 'monolog',
            'level' => 'debug',
            'handler' => StreamHandler::class,
            'formatter' => env('LOG_STDERR_FORMATTER'),
            'with' => [
                'stream' => 'php://stdout',
            ],
        ],
    ],

Create console command with content in function handler Log::debug('test', ['context' => 'test_content']); There are two messages with same log level in output. Link on code in framework

Issue can be closed, bug is fixed in 5.8 at least.

Thanks


Handle Amazon Product API errors

I'm breaking this out of #56 since it's really a different issue.
The Issue
Sometimes, Amazon queries return errors. The current behavior is that the page stays blank. We should have a graceful way of handling this that gives some clue about the problem.
Here are the errors I've encountered so far (to give a sense of how they look):

 "Error"=>{"Code"=>"SignatureDoesNotMatch", "Message"=>"The request signature we calculated does not match the signature you provided. Check your AWS Secret Access Key and signing method. Consult the service documentation for details."}

 "Error"=>{"Code"=>"RequestThrottled", "Message"=>"AWS Access Key ID: <redacted>. You are submitting requests too quickly. Please retry your requests at a slower rate."}

 "Error"=>{"Code"=>"AWS.InvalidParameterValue", "Message"=>"{:ASIN=>\"B00TFT77ZS\"} is not a valid value for ItemId. Please change this value and retry your request."}}

And here's an example error response for the ItemLookup endpoint: gist
Reproducing the Issue
The easiest error to reproduce: search for an item with a ' character (ex. k'nex) (until #56 is solved).
Task
We should handle Amazon error responses in a generic way. Maybe re-render the search page with an flash alert? Display a different error page?
We should also make sure any sensitive data (ex. keys) are redacted from the displayed messages.
Do any of these errors need special handling? I assume the Amazon API docs list all possible errors.

@leesharma I would like to take a shot at this issue. Do you have any extra details on the issue before I get started?

@hmasila Not at the moment! This will probably require playing around with the response classes (lib/amazon_product_api/*_response.rb) to hold/handle errors.

@leesharma I think I would like to try to solve this, but I wasn't sure if someone was already working on it. Is it ok if I pick this issue up?

@leesharma Do you still need help resolving this issue?  If so, I'd like to help!


Add tests for rate-control/compositeRate.js

This PR aims to create unit-tests for compositeRate.js class within our caliper-core package.
However, the current tests are failing due to following reasons.
Within our packages/caliper-core/lib/worker/rate-control/compositeRate.js, we find the following line :
https://github.com/hyperledger/caliper/blob/21a98f496c850840c211a670c32fcfa9240612bb/packages/caliper-core/lib/worker/rate-control/compositeRate.js#L76
However, I tried logging out this.options and was returned with an empty Object as output.
console.log('Options: ', this.options);
let weights = this.options.weights;
let rateControllers = this.options.rateControllers;

I was returned with following logs :
Options:  {}
2024.05.12-00:43:27.856 error [caliper] [composite-rate-controller]     Weight and controller definitions must be arrays.
      2) should throw error when weights and rateControllers lengths are not the same
(Similar errors for other tests too)

Due to this our initial check at line 86, that is :
https://github.com/hyperledger/caliper/blob/21a98f496c850840c211a670c32fcfa9240612bb/packages/caliper-core/lib/worker/rate-control/compositeRate.js#L79
Always throws an error. However, if we replace the line 76 above with
let weights = this.testMessage.content.weights;

Then, our tests pass.
What would be the most optimum fix for such cases?
Checklist

[x]  A link to the issue/user story that the pull request relates to
[ ]  How to recreate the problem without the fix
[ ]  Design of the fix
[ ]  How to prove that the fix works
[ ]  Automated tests that prove the fix keeps on working
[ ]  Documentation - any JSDoc, website, or Stackoverflow answers?

Issue/User story

Steps to Reproduce

Existing issues

[ ] Stack Overflow issues
[ ] GitHub Issues
[ ] Discord history

This issue is a small part of #1557 which aims to increase overall code-coverage for the caliper-core package.

Considering above changes have been applied for proper data-handling within _prepareControllers, we get a new error causing recursive functional callings as follow :
 at CompositeRateController._prepareControllers (lib/worker/rate-control/compositeRate.js:173:37)
      at new CompositeRateController (lib/worker/rate-control/compositeRate.js:69:14)
      at createRateController (lib/worker/rate-control/compositeRate.js:275:12)
      at new RateControl (lib/worker/rate-control/rateControl.js:59:27)
      at CompositeRateController._prepareControllers (lib/worker/rate-control/compositeRate.js:173:37)
      at new CompositeRateController (lib/worker/rate-control/compositeRate.js:69:14)
      at createRateController (lib/worker/rate-control/compositeRate.js:275:12)
      at Context.<anonymous> (test/worker/rate-control/compositeRate.js:49:35)
      at process.processImmediate (node:internal/timers:478:21)

@davidkel can you help me with this in any way possible?

@Sweetdevil144 Unfortunately your change is not correct. The reason you are getting an error and options is an empty object is because that's what you have set it to in your test when you created a mock message.
            rateControl: {
                type: 'composite-rate',
                opts: {},

@Sweetdevil144 Unfortunately your change is not correct. The reason you are getting an error and options is an empty object is because that's what you have set it to in your test when you created a mock message.
            rateControl: {
                type: 'composite-rate',
                opts: {},

Thanks. The test cases work fine now. However, I found a bug within our packages/caliper-core/lib/worker/rate-control/compositeRate.js. Can you confirm whether I'm correct or now?
I notices that we had been extracting Weights and rateControllers via following configurations :
        let weights = this.options.weights;
        let rateControllers = this.options.rateControllers;

Whereas, it should have been as below :
        let weights = this.testMessage.content.weights;
        let rateControllers = this.testMessage.content.rateControllers;

Can you confirm whether I'm correct or not @davidkel ?

@Sweetdevil144 Sorry, no your change is not correct. Not sure how you have come to this conclusion but if you look in the documentation https://hyperledger.github.io/caliper/v0.6.0/rate-controllers/#composite-rate you will see the format of the rate controller spec. weights and rateControllers are properties of opts and opts is extracted into this.options in the constructor see https://github.com/hyperledger/caliper/blob/56ecafa073ddb751375f3357200f84b4264ee13e/packages/caliper-core/lib/worker/rate-control/rateInterface.js#L33

@davidkel fixed. Can you review it?


Command Migration: ('HSET', 'HGET', 'HSETNX', 'HDEL')

This issue tracks the migration of the mentioned commands - ('HSET', 'HGET', 'HSETNX', 'HDEL') to make them compatible across the three protocols supported by the Dice database: RESP, HTTP, and WebSocket.
DiceDB now supports more than one protocols (Resp/http/websocket). We don't want eval operation of each command to be strictly bound with any of the protocols. Currently eval function return statements of each command is bounded to the RESP protocol.
The migration is required because:

The current implementation is specific to the RESP protocol
Evaluation functions of each command must be refactored to ensure protocol-agnostic behavior.
Evaluation functions should return raw responses instead of protocol specific responses.

The goal is to make the command logic protocol-independent, allowing all three protocols to call the same core functionality seamlessly.
Requirements

Refactor the evaluation function of [Command Name] to be generic.
Ensure the evaluation function does not include protocol-specific logic.

Migration Steps

Analyze Current Implementation

Review the current codebase to understand how the command logic is implemented.

Refactor Return Logic

Create a new function with the function definition as evalXXX(args []string, store *dstore.Store) *EvalResponse under the file /internal/eval/store_eval.go
Analyse return statements of the eval and modify them to send raw types without encoding
Use errors in the return statements from /errors/migrated_errors.go file
Use perdefined responses from /internal/clientio/resp.go file as RespType variables

Command/Worker specific Changes

Make IsMigrated flag to true in the commands information under /internal/eval/commands.go file.
Use the newly written eval function against the NewEval parameter in the same command structure.
Delete the old eval function from the /internal/eval/eval.go file.
Add the migrated command to the /internal/worker/CommandsMeta map and make it's type as SingleShard

Update Unit Tests

Refactor existing unit tests to accommodate the new implementation.
Add new unit tests if necessary to cover all possible cases.

Integration Tests

Run all integration tests to ensure successful migration.
Ensure that each protocol (RESP, HTTP, WebSocket) works correctly after migration.

Checklist

[ ] Migrated the evalXXX function with the latest definition
[ ] Update or add unit tests for the new implementation.
[ ] All unit tests pass successfully.
[ ] Ensure all integration tests pass successfully.

Additional Notes

Describe any edge cases that need to be handled in the generic function.
Mention any protocol-specific optimizations required in wrappers.

If there are any questions or concerns about this migration, please mention them here.
Related Issues/PRs

Sample implementation for the Get, Set, GetSet and SetEx can be found in this Pull Request.

@AshwinKul28, can I pick this up?

Hey @anchalsinghrajput thanks! Go for it. 🚀

HI @anchalsinghrajput , I hope you are doing well. Do you happen to have any updates on this? If you've any doubts please let us know on the discord.

Hi @AshwinKul28 @vpsinghg, I am new to go, i am still working on it, will give the update once i complete it.

Hi @AshwinKul28 , can you please reassign this issue to this Git profile.

Done @anchalsingh25

Hello @anchalsingh25,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.

Hey @anchalsingh25 hope you are doing well. I saw on the discord that you had lost your changes, by any chance were you able to recover them?

Hey @anchalsingh25 Sorry to bother you again, hope you are doing well, but we need to chase the timeline for the production readiness, hence for now I'm opening up this issue for others to pick it up, you can pickup any other issue once you are available and back. Thanks a lot and apologies.

Hi @AshwinKul28 , I will be picking this up on priority. This also unblocks me to work on hashmap with expiry.

@lucifercr07 I can work on this

Lets close this issue together @bhima2001 .

Hi @AshwinKul28 @apoorvyadav1111,
I was able to recover my changes. Can you please review the PR?


GET/POST image/images inconsistency

I'm trying to wrap my head around the images API and got confused by the naming. Is it images (plural) or image (singular) in the two cases below:
	case strings.HasPrefix(path, "/ob/images"):
		i.GETImage(w, r)

and
	case strings.HasPrefix(path, "/ob/images"):
		i.POSTImage(w, r)

@JustinDrake this seems more suited for a slack question rather than an issue. If it is an issue then it would be nice for it to be structured as an actionable ask.

This is an issue. Notice the inconsistency "/ob/images" versus the internal API GETImage. The other endpoints don't have this issue.

this was fixed


PR for OML Fundamentals Workshop. WID: 7401

Hi,
PR request for Oracle Machine Learning Fundamentals on Autonomous Database (WMS ID 7401). Contains new content on OML4R (lab 1) and text policy content in AutoML (lab 4)
Thanks
Moitreyee
Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Hi,
Under oml folder, I work on 3 different workshops - oml-fun-on-adb, oml, and oml4py-adb.

I wanted to do a PR only for the workshop oml-fun-on-adb (WMS: 7401). And for that, I clicked on oml-fun-on-adb in my fork, and clicked Pull Request. See attached screenshot. Let me know what I have missed here.  Thanks


Proposal bugs and issues

A few bugs and issues I've found reading the proposal:

Typo: captureValue vs. capturedValue
Typo: indicies
Why bother calling mapfn with indices -1 and -1 for missing matches? It seems that just returning undefined would be a bit more reasonable.
The mapfn logic doesn't work in full unicode mode because it conflates Input offsets with S offsets. They're not the same because one is counting code units and the other is counting code points. This is important but quite tricky to get right.

Also, the OffsetMapper approach seems a bit too confusing. When in mapped mode, either RegExpBuiltinExec should always return ranges or never. Having it sometimes call the user's mapfn and sometimes return ranges (via OffsetMapper) confuses where exactly the user's mapfn is called. It should always be at the same level of abstraction.

@ljharb yeah I just noticed that it is only used to advance lastIndex.

@waldemarhorwat I think these items should be addressed as of ffc9657de3b10bf2e86991e16ee94668e214e292.

you can't even omit missing matches from the output.

Can you clarify? re.exec(input, (start, end) => start === -1 ? undefined : [start, end]) would create an array with undefined for the value of any failed match. Are you stating that you'd rather we not create an entry in the array (sparse)?
Since the results from mapfn are user defined, it would be perfectly acceptable to have a callback like this:
// give me `true` or `false` based on whether the capture group was a match:
re.exec(input, (start, end) => start === -1);

// create an object representation for the match:
re.exec(input, (start, end) => {
  if (start === -1) {
    return { start, end, matched: false, value: "" };
  }
  else {
    return { start, end, matched: true, value: input.slice(start, end) };
  }
});

Just returning undefined does not provide that flexibility, and also doesn't align with existing behavior (where a non-captured match is an empty string, rather than undefined).
The following two expressions should be virtually identical:
re.exec(input);
re.exec(input, (start, end) => input.slice(start, end));

If we were to just return undefined, then they would definitely not be identical.

-1 seems like a hacky value.

I wouldn't categorize -1 as a hacky value, considering that is a well understood value to mean no offset was found in regards to String.prototype.indexOf, Array.prototype.indexOf, Array.prototype.findIndex, etc. Using a consistent integer value can also help to avoid polymorphism in v8, where you call a function once using numbers, but another time using undefined. In practice I would say that it's better to avoid polymorphism of this nature where possible in native APIs to avoid poor performance.

The following two expressions should be virtually identical:
re.exec(input);
re.exec(input, (start, end) => input.slice(start, end));

I agree. That's what I want!

If we were to just return undefined, then they would definitely not be identical.

That's incorrect. If we don't return undefined, then they would definitely not be identical.
re.exec(input) returns undefined for missed capture group matches, while re.exec(input, (start, end) => input.slice(start, end)) replaces them with empty strings, indistinguishable from successful matches that happen to match empty strings.


Question about fintune LayoutModel

[Question] about: "Fintune layoutml with other language"
Thank for great repo.
I want to fintune layoutml with vi-layoutxlm-base-uncased architecture portugal lang.
I am very thankfull if you can give step by step to fintune or give the documents fintune.
One more times, many thanks.

@phamkhactu Hi,
What is the downstream task?

@phamkhactu Hi,
What is the downstream task?

hi @linjieccc, I want to fintune SER task. After some research, I found out that: if I want to fintune ser, I need to fintune LayoutXLMModel, and if want to fintune  LayoutXLMModel I must fintue LayoutXLMTokenizer. So I my opinion, I need fintune LayoutXLMTokenizer before and after that LayoutXLMModel. I am sorry if I wrong.
I very happy if you give me guide to solve it or help me can combine LayoutXLMTokenizer and LayoutXLMModel
Thank you.


fix: sub-command region conflict with options_define region

$ rm -rf /root/.tccli/default.configure
$ tccli --version
3.0.924.1
$ tccli region  DescribeRegions  --Product cvm
usage: tccli [options] <command> <subcommand> [<subcommand> ...] [parameters]
To tccli help text, you can run:

  tccli help
  tccli configure help
  tccli service[cvm] help
  tccli service[cvm] action[RunInstances] help

config file:/root/.tccli/default.configure error, string indices must be integers

$ cat /root/.tccli/log/tccli.log
... ...
[2023-08-01 11:06:36,780] [4874] [tccli.main] [INFO] tccli region DescribeRegions --Product cvm
[2023-08-01 11:06:36,893] [4874] [tccli.main] [ERROR] config file:/root/.tccli/default.configure error, string indices must be integers
Traceback (most recent call last):
  File "/root/venv/lib/python3.6/site-packages/tccli/services/region/region_client.py", line 265, in parse_global_arg
    version = conf["region"][OptionsDefine.Version]
TypeError: string indices must be integers

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/root/venv/lib/python3.6/site-packages/tccli/main.py", line 41, in main
    CLICommand()()
  File "/root/venv/lib/python3.6/site-packages/tccli/command.py", line 49, in __call__
    return command_map[parsed_args.command](remaining, parsed_args)
  File "/root/venv/lib/python3.6/site-packages/tccli/command.py", line 189, in __call__
    return command_map[parsed_args.operation](remaining, parsed_globals)
  File "/root/venv/lib/python3.6/site-packages/tccli/command.py", line 286, in __call__
    return self._action_caller(action_parameters, vars(parsed_globals))
  File "/root/venv/lib/python3.6/site-packages/tccli/services/region/region_client.py", line 73, in doDescribeRegions
    g_param = parse_global_arg(parsed_globals)
  File "/root/venv/lib/python3.6/site-packages/tccli/services/region/region_client.py", line 271, in parse_global_arg
    raise ConfigurationError("config file:%s error, %s" % (conf_path, str(err)))
tccli.exceptions.ConfigurationError: config file:/root/.tccli/default.configure error, string indices must be integers

$ rm -rf /root/.tccli/default.configure
$ tccli --version
3.0.924.1
$ grep "region" /root/.tccli/default.configure
  "region": "ap-guangzhou",

git bisect show https://github.com/TencentCloud/tencentcloud-cli/commit/b7a2dee32c972e4be984cbd4c09bab75ddf07d06 (tccli>=3.0.687.1) that introduced a bug. After this PR, it will keep backwards compatibility for user existed *.configure and then fix it silently
$ rm -rf /root/.tccli/default.configure
$ tccli --version
3.0.924.1
$ tccli region  DescribeRegions  --Product cvm
usage: tccli [options] <command> <subcommand> [<subcommand> ...] [parameters]
To tccli help text, you can run:

  tccli help
  tccli configure help
  tccli service[cvm] help
  tccli service[cvm] action[RunInstances] help

config file:/root/.tccli/default.configure error, string indices must be integers
$ grep "region" /root/.tccli/default.configure
  "region": "ap-guangzhou",

$ python -m pip uninstall -y tccli && python -m pip install /tmp/dev/tccli-3.0.924.1-py3-none-any.whl
$ tccli region  DescribeRegions  --Product cvm
{
    "TotalCount": 18,
    "RegionSet": [
... ...

$ grep "region" /root/.tccli/default.configure
  "region": "ap-guangzhou",
  "regionms": {
    "endpoint": "region.tencentcloudapi.com",

Thank you for reporting this bug, we will review it as soon as possible

After some discussions and code review, we think it might be better to indent the system configuration to prevent future protential conflict, so this pr will be closed, fixed by https://github.com/TencentCloud/tencentcloud-cli/pull/70, we've added you to the co-author list.
Thank you very much!


Option to hide user/schema when show allí tables in a single list is checked

Hi,
I would like to have the option to customize the navigator view when you use the option "show all tables in a single list". In my case i don't care in what schema/user is the table, i only want to be able to list the tables from there alphabetically. As it is right now It first order the list by schema/user name and then by table name so i don't really get the global table name order. Would be nice if table names where shown just as the names of the tables and not with "database"+"user/schema"+.+"table name"

Hello @kot7k
What database do you use?
Also, I'm not sure that this is a good option. Because then you will see duplicates from different schemas/catalogs with equal names in the navigator tree.

Additionally, you can open the "Tables" folder and use ordering there:

It's informix. I see what you say, it's an option, but would be better It they appear on navigator already ordered to be able to write en sql editor while you watch. About duplicates, that's right, It could happen, but exceptionally at least in my case. Then i would have to care about the schema by deactivating this option i'm suggesting you to implement. I used another DB manager before and has that option. I love dbeaver and would like to have that option also . I see i only have the option to close this with a comment... I Hope you could reconsider this

Reopen

Ok, thanks for the idea.


Fix ARM Neon compilation and MulAdd implementation

This fixes an incorrect implementation for FMA on NEON.
I also had issues compiling the library due to the the enable_if constraint on NEON f32 register constructor. Looking at the other implementations, they don't have the enable_if, but I wasn't entirely sure what it's purpose was.
Here's a before an after for comparison:

Thanks, did you run the tests on MacOS ARM? I don't have a Mac device to test it myself

Yes, I ran FastNoise2 (NewFastSIMD branch) on a MacBook with the M1 CPU.

The enable_if on the mask register is to avoid ambiguous implicit conversions. The AVX512 m32 also uses it, without it there are compiler errors. What error are you getting with the enable_if?


Update issue 984 testimages due to a bug in greyscale conversion

Related to PR #1026

Thanks! Will merge now so you can continue

thank you!


Why is |x|/|x-1/x| so slow compared to |x/(x-1/x)|?

Describe the bug

When using the graphing mode, |x|/|x-1/x| is noticeably slower (sub 1FPS on my machine) than |x/(x-1/x)|.
Steps To Reproduce

Go to graphing mode
input |x|/|x-1/x| into function 1
input |x/(x-1/x)| into function 2
compare performance.

Expected behavior

Since the two are largely equivalent one would expect them to preform similarly. Logically it would only be adding one extra operation to each calculation, which I wouldn't expect to cripple performance like it does.
Device and Application Information

OS Build:
Major  Minor  Build  Revision
10     0      22543  0
Architecture: X64
Application Version: 11.2112.3.0
Region: en-US
Dev Version Installed: False

Requested Assignment

I'm just reporting this problem.  I don't want to fix it.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

I found that the two expressions given actually represent the same function. In algebra, we can observe that:
The first function: |x|/|x - 1/x| = |x^2|/|x^2 - 1|
The second function: |(x/(x - 1))/x| = |x|/|x - 1| = |x^2|/|x^2 - 1|
This means that their plots and properties should be the same. Now, let's focus on why there is a difference in plotting speed for these two equivalent expressions in the Windows 10 built-in calculator.
The reason could be related to algorithm optimization and numerical computation. During the plotting process, the calculator calculates the function value for each x-value within the defined interval and generates the plot based on these points. Different expressions might cause the calculator to calculate the numerator and denominator separately, or apply some other optimization strategies, such as simplifying the expression. These calculation strategies might affect the drawing speed.
For example, in the first expression, the calculator might need to calculate |x| first, then calculate |x - 1/x|, and finally divide them. In the case of the second expression, the calculator might directly compute the simplified form (|x^2|/|x^2 - 1|). This may result in different computation processes and different numbers of calculations, thereby affecting the plotting speed.
In summary, this speed difference might be due to the calculator's plotting algorithm handling the two expressions differently. Although the two functions are actually equivalent, they may be subject to different evaluation strategies when plotted by the calculator.


CCUBE-1297

Changes

Created display provider
Created display hook functions
Created display provider, hook, state, action types
Created display state
Created toast component.
Amending the the toast type
Amending the state.toast variable to be an array,
Amended the hook and provider handling of the showing and dismissing of the toast.

Additional information

You may refer to this CCUBE-1297

can close this PR as I reviewed all the changes in CCUBE-1298


Solution-Code-for-finding-longest-consecutive-sequence-in-an-array

Hi @saurabh47 @maheshmnj ,
Thanks for accepting my previous contribution, looking forward to make more in future. Kindly please merge this pull request as well.


Local testing assigning 'model' variable doesn't work

while testing locally I cannot get away with something like
	var mdlJson = File.ReadAllText("../../../../test/tower-developer.json");
	var mdl = Model.FromJson(mdlJson);
	var mdl = model;

this fails to write any elements to the output model.json or the model.gltf
if I build my elements in a separate variable, for instance 'mdl' and then add them to the otherwise untouched 'model' which comes into the Execute function arguments, then it works.
	var spaces = mdl.ElementsOfType<Space>();
	model.AddElements(spaces);

can't figure this one out.  not urgent

I believe this has been resolved with subsequent releases of Elements.


Segmentation fault (core dumped)

What happened?
Not necessarily a bug - I'm testing Frankenphp for the first time on my Yii2 project.
Running frankenphp php-server -a -v in my web directory. It works for seconds, it handles a few requests and then randomly crashes with no meaningful message, just Segmentation fault (core dumped).
Am I missing something?
Build Type
Standalone binary
Worker Mode
No
Operating System
GNU/Linux
CPU Architecture
x86_64
PHP configuration
default php8.3 php.ini

Relevant log output
No response

Trying a the worker mode using `frankenphp php-server -a -v --worker /var/www/api/web/index.php
Results with the following errors. Knowing that I have no function called apache_request_headers() in the entire code.
2024/11/01 22:40:43.256 ERROR   unexpected termination, restarting      {"worker": "/var/www/api/web/index.php", "exit_status": 1} 2024/11/01 22:40:43.256 DEBUG   apache_request_headers() called in non-HTTP context     {"worker": "/var/www/api/web/index.php"} 2024/11/01 22:40:43.254 DEBUG   apache_request_headers() called in non-HTTP context     {"worker": "/var/www/api/web/index.php"} 2024/11/01 22:40:43.257 ERROR   unexpected termination, restarting      {"worker": "/var/www/api/web/index.php", "exit_status": 1} 2024/11/01 22:40:43.255 ERROR   unexpected termination, restarting      {"worker": "/var/www/api/web/index.php", "exit_status": 1} 2024/11/01 22:40:43.257 DEBUG   starting        {"worker": "/var/www/api/web/index.php", "num": 40} 2024/11/01 22:40:43.255 DEBUG   apache_request_headers() called in non-HTTP context     {"worker": "/var/www/api/web/index.php"} 2024/11/01 22:40:43.253 ERROR   unexpected termination, restarting      {"worker": "/var/www/api/web/index.php", "exit_status": 1}

It's probably one of your library that calls apache2_resuests_headers().
For the crash, please check that your or code or the libraries you use aren't using an extension known to crash (the usual suspect is OpenSSL): https://frankenphp.dev/docs/known-issues/#buggy-php-extensions
If you're using OpenSSl, please switch to the Docker image we provide or compile FrankenPHP locally. This will fix the issue.
If it's another thing, could you follow these steps to get a stack trace please? https://frankenphp.dev/docs/contributing/#debugging-segmentation-faults-with-static-builds


Writing anndata to h5ad requires transformation_matrix's column to be of str type

Description
Hi there,
I'm using squidpy to analyse the public human breast cancer dataset provided by Vizgen. The data is available here (https://vizgen.com/data-release-program/) under the "MERSCOPE FFPE Human Immuno-oncology" heading.
After loading the data up into anndata, I was not able to write the object out as a h5ad file using the write function.
It is because the column heading of the transformation_matrix in the uns slot was of type int64. Converting the column type to str seem to fix the problem.
...
Minimal reproducible example

# Download the data from Vizgen website first.

from pathlib import Path
import scanpy as sc
import squidpy as sq

vizgen_dir = Path().resolve() / "HumanBreastCancerPatient1"

adata_bk = sq.read.vizgen(
    path=vizgen_dir,
    counts_file="cell_by_gene.csv",
    meta_file="cell_metadata.csv",
    transformation_file="micron_to_mosaic_pixel_transform.csv",
)

adata_bk.write("test.h5ad")

Traceback

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:246, in report_write_key_on_error.<locals>.func_wrapper(*args, **kwargs)
    245 try:
--> 246     return func(*args, **kwargs)
    247 except Exception as e:

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:311, in Writer.write_elem(self, store, k, elem, dataset_kwargs, modifiers)
    310 else:
--> 311     return write_func(store, k, elem, dataset_kwargs=dataset_kwargs)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:52, in write_spec.<locals>.decorator.<locals>.wrapper(g, k, *args, **kwargs)
     50 @wraps(func)
     51 def wrapper(g, k, *args, **kwargs):
---> 52     result = func(g, k, *args, **kwargs)
     53     g[k].attrs.setdefault("encoding-type", spec.encoding_type)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/methods.py:562, in write_dataframe(f, key, df, _writer, dataset_kwargs)
    561 group = f.create_group(key)
--> 562 col_names = [check_key(c) for c in df.columns]
    563 group.attrs["column-order"] = col_names

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/methods.py:562, in <listcomp>(.0)
    561 group = f.create_group(key)
--> 562 col_names = [check_key(c) for c in df.columns]
    563 group.attrs["column-order"] = col_names

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:113, in check_key(key)
    108 # TODO: Should I try to decode bytes? It's what h5py would do,
    109 # but it will be read out as a str.
    110 # elif issubclass(typ, bytes):
    111 # return key
    112 else:
--> 113     raise TypeError(f"{key} of type {typ} is an invalid key. Should be str.")

TypeError: 0 of type <class 'int'> is an invalid key. Should be str.

The above exception was the direct cause of the following exception:

TypeError                                 Traceback (most recent call last)
Cell In[33], line 1
----> 1 adata_bk.write("test.h5ad")

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_core/anndata.py:1951, in AnnData.write_h5ad(self, filename, compression, compression_opts, as_dense)
   1948 if filename is None:
   1949     filename = self.filename
-> 1951 _write_h5ad(
   1952     Path(filename),
   1953     self,
   1954     compression=compression,
   1955     compression_opts=compression_opts,
   1956     as_dense=as_dense,
   1957 )
   1959 if self.isbacked:
   1960     self.file.filename = filename

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/h5ad.py:99, in write_h5ad(filepath, adata, as_dense, dataset_kwargs, **kwargs)
     97 write_elem(f, "varp", dict(adata.varp), dataset_kwargs=dataset_kwargs)
     98 write_elem(f, "layers", dict(adata.layers), dataset_kwargs=dataset_kwargs)
---> 99 write_elem(f, "uns", dict(adata.uns), dataset_kwargs=dataset_kwargs)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:353, in write_elem(store, k, elem, dataset_kwargs)
    329 def write_elem(
    330     store: GroupStorageType,
    331     k: str,
   (...)
    334     dataset_kwargs: Mapping = MappingProxyType({}),
    335 ) -> None:
    336     """
    337     Write an element to a storage group using anndata encoding.
    338
   (...)
    351         E.g. for zarr this would be `chunks`, `compressor`.
    352     """
--> 353     Writer(_REGISTRY).write_elem(store, k, elem, dataset_kwargs=dataset_kwargs)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:248, in report_write_key_on_error.<locals>.func_wrapper(*args, **kwargs)
    246     return func(*args, **kwargs)
    247 except Exception as e:
--> 248     re_raise_error(e, elem, key)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:246, in report_write_key_on_error.<locals>.func_wrapper(*args, **kwargs)
    244         break
    245 try:
--> 246     return func(*args, **kwargs)
    247 except Exception as e:
    248     re_raise_error(e, elem, key)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:311, in Writer.write_elem(self, store, k, elem, dataset_kwargs, modifiers)
    302     return self.callback(
    303         write_func,
    304         store,
   (...)
    308         iospec=self.registry.get_spec(elem),
    309     )
    310 else:
--> 311     return write_func(store, k, elem, dataset_kwargs=dataset_kwargs)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:52, in write_spec.<locals>.decorator.<locals>.wrapper(g, k, *args, **kwargs)
     50 @wraps(func)
     51 def wrapper(g, k, *args, **kwargs):
---> 52     result = func(g, k, *args, **kwargs)
     53     g[k].attrs.setdefault("encoding-type", spec.encoding_type)
     54     g[k].attrs.setdefault("encoding-version", spec.encoding_version)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/methods.py:281, in write_mapping(f, k, v, _writer, dataset_kwargs)
    279 g = f.create_group(k)
    280 for sub_k, sub_v in v.items():
--> 281     _writer.write_elem(g, sub_k, sub_v, dataset_kwargs=dataset_kwargs)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:248, in report_write_key_on_error.<locals>.func_wrapper(*args, **kwargs)
    246     return func(*args, **kwargs)
    247 except Exception as e:
--> 248     re_raise_error(e, elem, key)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:246, in report_write_key_on_error.<locals>.func_wrapper(*args, **kwargs)
    244         break
    245 try:
--> 246     return func(*args, **kwargs)
    247 except Exception as e:
    248     re_raise_error(e, elem, key)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:311, in Writer.write_elem(self, store, k, elem, dataset_kwargs, modifiers)
    302     return self.callback(
    303         write_func,
    304         store,
   (...)
    308         iospec=self.registry.get_spec(elem),
    309     )
    310 else:
--> 311     return write_func(store, k, elem, dataset_kwargs=dataset_kwargs)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:52, in write_spec.<locals>.decorator.<locals>.wrapper(g, k, *args, **kwargs)
     50 @wraps(func)
     51 def wrapper(g, k, *args, **kwargs):
---> 52     result = func(g, k, *args, **kwargs)
     53     g[k].attrs.setdefault("encoding-type", spec.encoding_type)
     54     g[k].attrs.setdefault("encoding-version", spec.encoding_version)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/methods.py:281, in write_mapping(f, k, v, _writer, dataset_kwargs)
    279 g = f.create_group(k)
    280 for sub_k, sub_v in v.items():
--> 281     _writer.write_elem(g, sub_k, sub_v, dataset_kwargs=dataset_kwargs)

    [... skipping similar frames: report_write_key_on_error.<locals>.func_wrapper at line 248 (2 times), report_write_key_on_error.<locals>.func_wrapper at line 246 (1 times), write_spec.<locals>.decorator.<locals>.wrapper at line 52 (1 times), Writer.write_elem at line 311 (1 times), write_mapping at line 281 (1 times)]

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:246, in report_write_key_on_error.<locals>.func_wrapper(*args, **kwargs)
    244         break
    245 try:
--> 246     return func(*args, **kwargs)
    247 except Exception as e:
    248     re_raise_error(e, elem, key)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:311, in Writer.write_elem(self, store, k, elem, dataset_kwargs, modifiers)
    302     return self.callback(
    303         write_func,
    304         store,
   (...)
    308         iospec=self.registry.get_spec(elem),
    309     )
    310 else:
--> 311     return write_func(store, k, elem, dataset_kwargs=dataset_kwargs)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/registry.py:52, in write_spec.<locals>.decorator.<locals>.wrapper(g, k, *args, **kwargs)
     50 @wraps(func)
     51 def wrapper(g, k, *args, **kwargs):
---> 52     result = func(g, k, *args, **kwargs)
     53     g[k].attrs.setdefault("encoding-type", spec.encoding_type)
     54     g[k].attrs.setdefault("encoding-version", spec.encoding_version)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/specs/methods.py:281, in write_mapping(f, k, v, _writer, dataset_kwargs)
    279 g = f.create_group(k)
    280 for sub_k, sub_v in v.items():
--> 281     _writer.write_elem(g, sub_k, sub_v, dataset_kwargs=dataset_kwargs)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:248, in report_write_key_on_error.<locals>.func_wrapper(*args, **kwargs)
    246     return func(*args, **kwargs)
    247 except Exception as e:
--> 248     re_raise_error(e, elem, key)

File /stornext/data/mambaforge/envs/spatial/lib/python3.11/site-packages/anndata/_io/utils.py:229, in report_write_key_on_error.<locals>.re_raise_error(e, elem, key)
    227 else:
    228     parent = _get_parent(elem)
--> 229     raise type(e)(
    230         f"{e}\n\n"
    231         f"Above error raised while writing key {key!r} of {type(elem)} "
    232         f"to {parent}"
    233     ) from e

TypeError: 0 of type <class 'int'> is an invalid key. Should be str.

Above error raised while writing key 'transformation_matrix' of <class 'h5py._hl.group.Group'> to /

Version

1.3.1.dev4+gecee104

Made a pull request @giovp: https://github.com/scverse/squidpy/pull/732

@giovp when i was trying to run the unit tests on my mac using tox -e py310-macos, I ran into a couple of issues, one of which I managed to solve:

When installing the dependencies, installing pyQT will error out at the step "Preparing metadata..". I then manually installed pyQT using pip, which also resulted in the same error. Apparently this is because pyQT5 is stuck asking for a license. Fixed this by re-running pip using pip install pyqt5 --config-settings --confirm-license= --verbose.
Managed to run most of the unit tests apart from the ones that require datasets to be downloaded from figshare, such as from this url: https://ndownloader.figshare.com/files/28241139. I got a urllib.error.HTTPError: HTTP Error 403: Forbidden error. When I visited the url from my browser, I also get the same 403 error.

Problem 2 seems to be specific on my end as the ci tests ran for the pull request did not indicate any problem.
Happy to create another issue and pull request to fix them if required.

thank you for the PR, just merged!

Is it maybe due to the mac bein an M1 or M2? In that case, a solution would be to install pyqt via conda
I just tried and I was able to download it, I think this is usual HTTP request being rejected sometime, nothing that can be done.

thanks again, will close this for now, feel free to reopen otherwise!


Icon is abnormal in windows explorer.

Prerequisites
Version: 0.14.0
Install: by npm
Operating system: Windows 10
Bits: 64 bits
Description
When see large icon in windows explorer, some of icon is abnormal. like alert-triangle.

But it works in Firefox, edge, chrome, gimp, inkscape.
I check detail in feather and lucide for alert-triangle:
lucide : ... <path d="M10.29 3.86L1.82 18a2 2 0 001.71 3h16.94a2 2 0 001.71-3L13.71 3.86a2 2 0 00-3.42 0z" /> ...
feather: ... <path d="M10.29 3.86L1.82 18a2 2 0 0 0 1.71 3h16.94a2 2 0 0 0 1.71-3L13.71 3.86a2 2 0 0 0-3.42 0z"> ...

yeah just switch to linux

Hmm interesting, I think this has something to do with SVGO optimisations, I think windows doesn't support the short syntax. Do you use additional plugins in windows explorer to display svgs or is it natively supported?

I use default config in windows. but set Open with for Microsoft Edge

Doesn't it use IE to render those though?

It does not matter what it opens with

I think we can say this is not our fault, and a windows bug - this doesn't happen anywhere else.


Add pre/postInstall hooks to npm install handling

[x] New feature (non-breaking change which adds functionality)

Proposed changes
Introduces a set of new API hooks that can be used to add code before and after npm install is run. This applies to both installing new nodes and Function node external modules.
There are four new hooks:

preInstall
postInstall
preUninstall
postUninstall

The *Install hooks are passed the following event object:
{
    "module": "<npm module name>",
    "version": "<version to be installed>",
    "url": "<optional url to install from>",
    "dir": "<directory to run the install in>",
    "isExisting": "<boolean> this is a module we already know about",
    "isUpgrade": "<boolean> this is an upgrade rather than new install"
}

The *Uninstall hooks are passed the following event object:
{
    "module": "<npm module name>",
    "dir": "<directory to run the uninstall in>",
}```

### Example usage

RED.hooks.add("postInstall",  (event, done) => {
// An npm install of event.module has just completed in the directory event.dir.
// Optionally do some extra work before allowing the install to complete
done();
});

### Handling errors

 - If the `*Install` hooks throw an error or return a rejected promise, the install will be cleanly failed.
 - If the `npm install` itself throws an error (such as 404), the `postInstall` hook will *not* be triggered
 - If the `preUninstall` hook throws an error, the uninstall will be cleanly failed.
 - If the `postUninstall` hook throws an error, it will be logged, but the uninstall will complete cleanly as we cannot easily rollback an `npm remove` after it has completed.

## Checklist
<!-- Put an `x` in the boxes that apply -->

- [x] I have run `grunt` to verify the unit tests pass
- [x] I have added suitable unit tests to cover the new/changed functionality

@tfmf @kazuhitoyokoi this feature will allow custom code to be added in the module install code path - such as running electron-builder.
Let me know what you think.

Coverage increased (+0.07%) to 67.45% when pulling d2432716ea5054c83188342f5c336a50f5f8e9b7 on npm-install-hooks into c021b4c3682f9c9b717b5318e93de4b18b73d84e on dev.

@knolleary I'm going to try this out and come back to you, but at glance it seems that this will allow to remove the hardcode workaround I did.

@knolleary Thank you for the mention. I tried this code. To replace the npm command with the npm module which was defined in the package.json as a dependency, I thought that it will be useful to support return null in preInstall as same as other hooks. In terms of electron builder, we need to define when the Node-RED nodes need to rebuild. In my understanding, Node-RED nodes need to rebuild when the electron builder executes the first time if the .node-red directory exists.

@knolleary Thank you for accepting my idea. I will check it.

@knolleary I tried this out and worked like a charm, here's my simple example to run electron-rebuild after the installation:
RED.hooks.add("postInstall",  (event, done) => {
    console.log(event);
    child_process.exec("npm run rebuild " +  event.module, {cwd: event.dir},  (err, stdout, stderr) => {
      // the *entire* stdout and stderr (buffered)
      console.log(`stdout: ${stdout}`);
      console.log(`stderr: ${stderr}`);
    });

   done();
});

A couple of things that do not allow me to use this as it is:

since the electron-rebuild is set as devDependencies it gets removed when installing any node, for my example I had to move it to dependencies I have a workaround in the installer.js where I remove the '--production'
It would be great to see the output in the event log, I also accomplished this by adding the electron-rebuild command after the success of the node installation in the installer.js

@tfmf looks good.
Being able to modify the npm args was something I'd considered, but wasn't sure if it was really necessary (for the first version at least). But as you've identified a specific need for it, we can add it in.
I've just pushed an update that adds an 'args' property to the event object:
{
    "module": "<npm module name>",
    "version": "<version to be installed>",
    "url": "<optional url to install from>",
    "dir": "<directory to run the install in>",
    "isExisting": "<boolean> this is a module we already know about",
    "isUpgrade": "<boolean> this is an upgrade rather than new install",
    "args": [ "an array", "of the args" ,  "we will pass to npm"]
}

The preInstall trigger can modify that object in place - filtering out the unrequired args, or adding other ones.
When we run the npm install command we will then do: npm install <insert the args from event.args> module@version - so the args array does not contain install or the name of the module to be installed.
On the event log side - you should be able to use the @node-red/utils.exec module as that does the event logging - it's just a question of how you get access to it.


Updated ios x86 simulator build to match target version of real devices.

On ios the x86 build uses an old target that does not support c++11, I have updated the x86 to match the one used on real device

Why did you close it? It's not needed?


Exception is thrown when receiving error different than 401 for the first request

When the first request get error code other than 401, an exception is thrown and the error handler doesn't get called.

fixed with PR #18

merged.


Fix #1977, Functional requirements grammar cleanup

Describe the contribution

Fix #1977

Grammar cleanup only, no actual change to meaning.
Testing performed
None, doc change only
Expected behavior changes
None
System(s) tested on
N/A
Additional context
None
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

https://github.com/nasa/cFS/wiki/Community-CCB-Agenda:-September-29-2021 APPROVED


Added Berry Lexer

Added berry lexer, example file, and tested against lexer with no errors. Regenerated _mapping.py to include berry lexer.

Thanks for the feedback and suggestions. Hopefully I have done the requested changes correctly. The HTML and test output looks good.

Thanks for the updates; there are still a few unresolved threads and one new (easy) one.

And two more:

Don't commit the macOS system file .DS_Store (I'll prepare a patch to make it ignored by Git),
Add Berry to doc/languages.rst.

Added Berry to doc/languages.rst. Sorry I don't know how to delete the .DS_Store file I accidentally pushed.

Right. @Beormund, do you have the time to address that minor issue before we merge this?

Right. @Beormund, do you have the time to address that minor issue before we merge this?

Done!

Then thanks for the patience!


Any plans to address deprecation warnings?

There's a list of deprecation warnings installing Strider. A sample as follows:
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>This project is unmaintained
npm WARN deprecated<EMAIL_ADDRESS>Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools.
npm WARN deprecated<EMAIL_ADDRESS>'native-or-bluebird' is deprecated. Please use 'any-promise' instead.

Will the code author accept submissions to fix them?

Would totally accept submissions. Although minimatch will be hard to address since it's in almost every dependency.

Closing for #1080, also v2.1.0 addressed a few.


[PWX-33177] Skip pool expansions when app has only FBDA volumes.

Since FBDA volumes do not get created on PX pool, px pool expansion in case of FBDA volumes is not required.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional)
Closes #
Special notes for your reviewer:
Successful job run: https://jenkins.pwx.dev.purestorage.com/job/Users/job/Vinayak/job/FBDA-MultiTenancy/29/console

Thanks @vprabhakar-px

Merging the code as per PTX-23855.


Can't use NPM task to install globally on Linux agent

Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Npm
Environment

Server - Azure Pipelines.
Agent - Hosted .

Issue Description
When installing NPM packages globally, the sudo command is required on Linux agents (e.g. 'ububtu-18.04'), other wise the installation fails with the following error -
2020-02-21T12:01:54.0304530Z [command]/usr/local/bin/npm install -g newman
2020-02-21T12:01:58.8292544Z npm WARN checkPermissions Missing write access to /usr/local/lib/node_modules
2020-02-21T12:01:58.8294427Z npm ERR! code EACCES
2020-02-21T12:01:58.8296234Z npm ERR! syscall access
2020-02-21T12:01:58.8297634Z npm ERR! path /usr/local/lib/node_modules
2020-02-21T12:01:58.8299374Z npm ERR! errno -13
2020-02-21T12:01:58.8301029Z npm ERR! Error: EACCES: permission denied, access '/usr/local/lib/node_modules'
2020-02-21T12:01:58.8302716Z npm ERR!  [Error: EACCES: permission denied, access '/usr/local/lib/node_modules'] {
2020-02-21T12:01:58.8304549Z npm ERR!   stack: "Error: EACCES: permission denied, access '/usr/local/lib/node_modules'",
2020-02-21T12:01:58.8306755Z npm ERR!   errno: -13,
2020-02-21T12:01:58.8308493Z npm ERR!   code: 'EACCES',
2020-02-21T12:01:58.8310151Z npm ERR!   syscall: 'access',
2020-02-21T12:01:58.8311672Z npm ERR!   path: '/usr/local/lib/node_modules'
2020-02-21T12:01:58.8312862Z npm ERR! }
2020-02-21T12:01:58.8314121Z npm ERR!
2020-02-21T12:01:58.8315352Z npm ERR! The operation was rejected by your operating system.
2020-02-21T12:01:58.8316661Z npm ERR! It is likely you do not have the permissions to access this file as the current user
2020-02-21T12:01:58.8318103Z npm ERR!
2020-02-21T12:01:58.8319554Z npm ERR! If you believe this might be a permissions issue, please double-check the
2020-02-21T12:01:58.8321302Z npm ERR! permissions of the file and its containing directories, or try running
2020-02-21T12:01:58.8322698Z npm ERR! the command again as root/Administrator.

However, when using the NPM task, there is no option to use the sudo command, even when selecting custom.
There appears to be no mention of this in the documentation.
Task logs
tasklog_7.zip

Guess MS don't care about this, so closing.

This issue hit me again yesterday, so reopening. Hopefully it'll get picked up...

I'm also having this issue! Any advice, or can this be picked up?

I got this issue since yesterday. It was building normally two days ago

Hey folks, sorry for the delay here. We have a few suggestions:

Add the new npm authenticate task that we recommend using moving forward
Run a script to 'npm install newman'
Run a script to "sudo npm install -g newman" [may need an OS constraint on the task]

Let us know how it goes, I'll close this for now but please re-open if you're still having problems.

@herenhuang sudo npm solved my issue


docs: fix broken link

Closes https://github.com/t3-oss/create-t3-app/issues/1346
✅ Checklist

[ ] I have followed every step in the contributing guide (updated 2022-10-06).
[ ] The PR title follows the convention we established conventional-commit
[ ] I performed a functional test on my final commit

Changelog
[Short description of what has changed]

Screenshots
[Screenshots]
💯

Duplicate of #1347 or isn't it?

yea lets merge the other one


"Error Loading Clusters" repeatedly appears when no cluster ever created in user account

A new user who has used Terra for just 1-2 days during Hackathon saw an Error Loading Clusters pop-up each time he refreshed. He had never started a cluster and was a brand new Terra user. The message cannot be permanently disabled either. The issue persisted even after a refresh and a sign-in/out.
Why is he getting it and can he be able to permanently dismiss it?

ZD #1673

I think this is an issue for the Leo team, we're just passing along an error that the user is getting. Despite the fact that this user hasn't created a cluster, we poll to check if the user has any clusters. And if Leo is down, then they'll get this error. I'm not sure why this user kept getting that error though.

@katevoss I can pass it along to the Leo team! Thanks.

@schaluva if this was indeed an issue for the Leo team, I'll close this one.

@katevoss Yes, I passed the ticket and details to Leo and Adrian said he would have someone on his team look at it! Thanks


Immolate Support Gem is being ignored

Hello,
the damage from immolate is not being added when using this gem.
Currently using this gem in an Oro's Sacrifice Flickerstrike setup.
kind regards

You need to enable the "Is the enemy Burning?" option in the Configuration tab. You may not see the option unless you have Immolate currently linked to your main skill.


[0.5-nexus] Disable Fields Minus in grammar to avoid data correctness issue

Description
The Fields Minus Command fields - <field list> has been supported from https://github.com/opensearch-project/opensearch-spark/pull/698 which is required Spark 3.5.0+
But its grammar presented at the beginning of this repository created, which the - in fields - <field list> is ignored by default. So in branch 0.5-nexus whose spark version is 3.3.2 will cause data correctness issue (the command results equal to command fields + <field list>).
Issues Resolved
Resolves https://github.com/opensearch-project/opensearch-spark/issues/731
Check List

[ ] Updated documentation (ppl-spark-integration/README.md)
[x] Implemented unit tests
[x] Implemented tests for combination with other commands
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

2024-10-03T03:38:34.6161986Z [info] *** 23 TESTS FAILED ***
2024-10-03T03:38:34.6170835Z [error] Failed tests:
2024-10-03T03:38:34.6172098Z [error] 	org.opensearch.flint.spark.FlintSparkMaterializedViewITSuite
2024-10-03T03:38:34.6174059Z [error] 	org.opensearch.flint.core.scheduler.OpenSearchAsyncQuerySchedulerITSuite
2024-10-03T03:38:34.6203919Z [error] 	org.opensearch.flint.spark.FlintSparkCoveringIndexITSuite
2024-10-03T03:38:34.6205644Z [error] 	org.opensearch.flint.spark.FlintSparkSkippingIndexITSuite
2024-10-03T03:38:35.4267885Z [error] (integtest / IntegrationTest / test) sbt.TestsFailedException: Tests unsuccessful
2024-10-03T03:38:35.4304050Z [error] Total time: 3041 s (50:41), completed Oct 3, 2024, 3:38:35

All above IntegTest failures are not related to this PR.
@ykmr1224 @noCharger @dai-chen @YANG-DB Please help to review and merge.
@ykmr1224, make sure above failures are taken care before 0.5-nexus releasing, they have been introduced for a while.

@dai-chen can u plz review the failed tests ?
thanks

Failed tests are caused by external scheduler cases


Seg Fault when returning Data type?

This could easilly be my fault, I have what appears to be a trivial test
example code here https://github.com/polygonhell/HintTest
On my Mac this segfaults when I cabal run (or just execute) it, but completes if I run main from inside the cabal repl.
On my Windows box it segfaults in both cases.
Since I could get basic types like Int to work correctly including functions on basic types, I assume the issue is the shared SomeType data definition.
I couldn't find an example that demonstrated the use of more complex types.

Confirming on linux/amd64:
Running HintTest...
["TestHint","SomeType"]
[Fun "test"]
[Data "SomeType" ["Ploop","Floop"]]
got something
zsh: segmentation fault (core dumped)  cabal run

To be more precise:
 $ ghc --version
The Glorious Glasgow Haskell Compilation System, version 7.10.3

I'm not familiar with debugging segfaults in Haskell, so any help is welcome.

I ran the code in LLDB, and this is all I could get out of it, unfortunately I'm not familiar enough with the guts of GHC for it to provide any real insight.
It dies here, with a bad memory reference
HintTests1bh_info:
0x1000037b0 <+0>:  leaq   -0x10(%rbp), %rax
0x1000037b4 <+4>:  cmpq   %r15, %rax
0x1000037b7 <+7>:  jb     0x10000385e               ; c1cM_info + 46
0x1000037bd <+13>: leaq   0x34(%rip), %rax          ; c1cG_info
0x1000037c4 <+20>: movq   %rax, -0x10(%rbp)
0x1000037c8 <+24>: movq   0x10(%rbx), %rax
0x1000037cc <+28>: movq   0x18(%rbx), %rbx
0x1000037d0 <+32>: movq   %rax, -0x8(%rbp)
0x1000037d4 <+36>: addq   $-0x10, %rbp
0x1000037d8 <+40>: testb  $0x7, %bl
0x1000037db <+43>: jne    0x1000037f8               ; c1cG_info
->  0x1000037e1 <+49>: jmpq   *(%rbx)
0x1000037e3 <+51>: nopl   (%rax,%rax)
0x1000037e8 <+56>: addl   %eax, (%rax)
0x1000037ea <+58>: addb   %al, (%rax)
0x1000037ec <+60>: addb   %al, (%rax)
0x1000037ee <+62>: addb   %al, (%rax)
0x1000037f0 <+64>: andb   %al, (%rax)
0x1000037f2 <+66>: addb   %al, (%rax)
0x1000037f4 <+68>: addb   %al, (%rax)
0x1000037f6 <+70>: addb   %al, (%rax)`
The registers at this point are
       rax = 0x000000010349f4b1  ghczmprim_GHCziTypes_ZMZN_closure + 1        rbx = 0x0000010555e02800        rcx = 0x0000000000000001        rdx = 0x0000000105501700        rdi = 0x0000000106982010        rsi = 0x000000010555c380        rbp = 0x00000001055fcf70        rsp = 0x00007fff5fbfb7e8         r8 = 0x0000000000000006         r9 = 0x000000010555c678        r10 = 0x000000010555c368        r11 = 0x0000000106968700        r12 = 0x000000010555c6a8        r13 = 0x00000001034a0e18  HintTestMainCapability + 24
r14 = 0x0000000000000006
r15 = 0x00000001055f50c0
rip = 0x00000001000037e1  HintTests1bh_info + 49     rflags = 0x0000000000010246         cs = 0x000000000000002b         fs = 0x0000000000000000         gs = 0x0000000000000000
It's dereferencing rbx @ 0x0000010555e02800 which is memory not mapped to the application.
this is called from the following callstack
ghczmprim_GHCziTypes_ZMZN_closure stg_upd_frame_info_dsp
All in all not a very useful stack trace.
My best guess is it's trying to dereference a thunk, and pulling out the wrong address to jump to. Could it be something like differing compiler options for the application and the Hint compiled code resulting in a different header for the closure or something?

Actually looking at it further, while 0x0000010555e02800 is not mapped, it looks like 0x000000010555e028, so possibly some sort of misaligned write?

This could easilly be my fault, I have what appears to be a trivial test example code here

Your code is not as trivial as it first appears: you have a dependency loop. If I was reimplementing your example code, I would have divided it into three layers:

package some-type, exposing SomeType
file plugin/TestHint.hs, depending on some-type
package HintTest, depending on some-type and hint at compile-time and on plugin/TestHint.hs at runtime

Note how each layer only depends on layers which come before it. By comparison, here are the logical dependencies of your code:

file plugin/TestHint.hs, depending on hint-test's SomeType
package HintTest, exposing SomeType, depending on hint at compile-time and on plugin/TestHint.hs at runtime

Notice the loop. That's not what causing the problem though, because the way you've implemented it doesn't actually involve a loop. Instead, you have have references to two types which happen to have the same name but are actually different entities. Here's what I think is happening:

you compile src/SomeType.hs and src/Main.hs into an executable named HintTest. Within this codebase, there is a type called "SomeType", which I'll call "SomeType1". Since this is a program and not a library, nobody else can refer to SomeType1 and the compiler is allowed to use any binary representation it wishes.
at runtime, you interpret files plugin/TestHint.hs and src/SomeType.hs. It just so happens that src/SomeType.hs contains the same code as the src/SomeType.hs which was used to compile the HintTest executable, but in Haskell types use nominal equality, not structural equality, so this coincidence is irrelevant. This is a new type which I'll call "SomeType2".
the line of code interpret "test 5" (as :: SomeType) becomes the runtime expression "test 5 :: SomeType". Note that since the type annotation (as :: SomeType) is part of the code of the HintTest executable, this first annotation says that the expression "test 5" is expected to have type SomeType1. By contrast, since the expression "test 5 :: SomeType" is evaluated at runtime, this second annotation says that the expression "test 5" has type SomeType2.
hint uses ghc to interpret "test 5 :: SomeType", and since ghc did not report any type error, hint assumes that "test 5" has the expected type, and so it unsafeCoerces the result to SomeType1. This is the bug.
Later on, the HintTest executable attempt to print a variable of type SomeType1, but the bits to which this variable points to actually has the shape of a SomeType2. Since one type is compiled and the other is interpreted, the bit patterns are very different, and so a segfault ensues.

When you run HintTest in interpreted mode, the incorrect coercion still occurs, but since SomeType1 and SomeType2 are both interpreted (and are both defined by the same code), the bit patterns are the same and so no segfault occurs.

Yes thanks that makes sense.
I was assuming that the SomeType being added to the context would be the one from the host application rather than another copy.
I assume then that there is no way to expose functionality from the host application to the compilation context without factoring the shared functionality out as a separate package?

Closing this as it's my bug, not Hints

I assume then that there is no way to expose functionality from the host application to the compilation context without factoring the shared functionality out as a separate package?

I was curious about this too, so I tried to implement the circular dependency I described earlier, and it worked fine! I sent you a pull request (which you already merged, you're fast!) to demonstrate how I did it.
If the functionality of the host application which you want to expose doesn't require new types, you can also expose it to the program executed by hint simply by passing that functionality as an argument. For example, suppose the host application implements square :: Int -> Int but does not expose it as a library. The plugin can still use square by expecting it as an argument:
test :: (Int -> Int) -> Int -> SomeType
test square x = Ploop (square x)

And then the host can pass its square implementation by asking hint to return a function and then passing arguments to that function:
interpret "test" infer <*> pure square <*> pure (5 :: Int)

Yes, thanks for the example, it was only fast because I happened to look just after you checked in.
It looks like the "magic" is exposing the the module and then not exposing it's source path to the Hint context.
I did get it working by factoring out the shared code, it was straight forwards once I realized that ghc couldn't see the modules installed to a sandbox unless it was explicitly added to the GHC_PACKAGE_PATH. unsafeRunInterpreterWithArgs is a much better way that I was using as well
cabal exec cabal run
was pretty ugly.
Cabal just isn't the best build system for working with dependent libraries, so I prefer this approach.
My original example actually returned a function, I only changed it to return the Type to to minimize complexity, my actual use case is more of the form
fn :: StateObject -> Args -> (ResultType, StateObject)
Which should be simple enough.


Get Started button updates

Changed the links on the get started button and try-it nav link
Deleted try-it.md and try-it.html files

/lgtm

Still trying to rebase

Reopening this


Add DOI lookup to frontend

Add DOI lookup input field to frontend.
Add logic for fetching the DOI data.
Refactor.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (58487ad) 100.00% compared to head (94eede6) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #46   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          106       106
  Branches         7         7
=========================================
  Hits           106       106

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


go.mod: Update to latest k8s v0.28.3

This should sync with the k8s version used by OpenShift 4.15

/hold

I've force-pushed to the PR branch, in order to update openshift in addition to k8s, and to address the inet.af issue differently.

/unhold

/test integration-crc


outdated links

Original bug filed by jldarrow at b/239093750
Link to CreateBlob and UpdateBlob are out of date (may be other locations in client still too) - https://github.com/googleapis/java-storage/blob/main/google-cloud-storage/src/main/java/com/google/cloud/storage/package-info.java#L22

We just got another internal bug filed for this, please take another look whenever you can!


New version: Microsoft.AzureDataStudio version 1.48.1

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Aware of the other PR - it will be closed as it was generated by someone outside of the release team.

/azp run

/azp run

@erpett are you going to be submitting the latest version?


oc deploy jobs should wait for deployment to finish

we should have the oc deploy commands wait for the deployment to finish and then have them report the status, into the same jenkins job that they were called from. Typically apps should go live/fail rapidly, so it wont add a lot more time to the overall job run

is this something you might be able to look at @kadel ?

At the moment, if the build succeeds, we assume that the code meets acceptance criteria for itself, and that the containers were build and pushed into the relevant place. after which point we simply run an "oc deploy --latest" and hope for the best; ref: https://github.com/almighty/almighty-jobs/blob/master/devtools-ci-index.yaml#L96

If I remember correctly I've already tried to solve this once, and I'm afraid that currently it i not that easy to wait for deploy to finish, but there might by some workaround for this. I will investigate.

in current alpha Origin release  is --follow option for deploy this prints deployment logs and waits for it to finish.
--follow=false: Follow the logs of a deployment

But there is no way to do this in current stable release v1.2.1.

:sparkles:  It looks I found how to do that without --follow
this could work:
oc deploy core --latest
oc logs -f dc/core # this displays logs and blocks until finished

do you want to propose a PR for this ? https://github.com/almighty/almighty-jobs/blob/master/devtools-ci-index.yaml#L97 is the area you'd need to edit up.

related issue in Origin: https://github.com/openshift/origin/issues/10864

related issue in Origin: openshift/origin#10864

This issue is currently assigned to an openshift dev. @kadel is going to talk with tomas nozicka tomorrow and then possibly mfojtik to see what the status of this is.

implementing https://github.com/openshift/origin/issues/10864 is planned for OpenShift release

implementing openshift/origin#10864 is planned for OpenShift release

which release?

Origin 1.4.  More specifically oc rollout status should get  to 1.4 (see https://github.com/openshift/origin/pull/9874#issuecomment-244774448) and that should solve this problem also.

So this is blocked on the openshift issue, but we believe we have a workaround for now in https://github.com/almighty/almighty-jobs/pull/20. Is this PR satisfactory?

upstream pull request has been merged: https://github.com/openshift/origin/pull/9874


Component get repeatedly rendered on button click

I'm trying to adapt the load more example and got it all set up and working, but there is a litte part of if I can't reproduce the  result of the sprig cookbok. https://putyourlightson.com/sprig/cookbook?page=3#load-more
{% set categ = categ ?? null %}
{% set limit = limit ?? 1 %}

{% set categories = craft.entries({
    section: 'category',
}).all() %}

{% if categ %}
    {% set selectedCateg = categories|filter(v=> v.id == categ) %}
{% endif %}

{% set entriesQuery = craft.entries({
    section: 'contentStructure'
}|merge(selectedCateg is defined ? {relatedTo: selectedCateg} : {})
).limit(limit)
%}

{% set entries = entriesQuery.all() %}
{% set count = entriesQuery.count() %}

    {% for category in categories %}
        <button sprig
                s-val:categ="{{ categ == category.id ? '' : category.id }}"
                s-val:limit="{{ 1 }}"
                id="category-{{ category.id }}"
                class="{{ categ == category.id ? 'load-more--selected' : '' }}"
        >{{ category.title }}</button>
    {% endfor %}

        {% for entry in entries %}

{{ entry.title }}

        {% endfor %}

    {% if entriesQuery.count() > entries|length %}
        {# Increments `offset` by the value of `limit` and swaps itself out on click #}
        <button sprig
                id="test"
                s-val:limit="{{ limit + 1 }}"
                s-target="this"
                s-swap="outerHTML"

                >
            Load another
        </button>
    {% endif %}

    {{ hiddenInput('limit', limit) }}
    {{ hiddenInput('categ', categ) }}

When using this code, the whole component get repeated when clicking the "Load another" button. But if I remove the s-target="this" and the s-swap="outerHTML", everything works fine. From the documentation and sprig cookbook I see that this should swap the button itself, not the entire component. What am I missing here?
Note: Great plugin by the way, working with it is fun!

I forgot to add the screenshot, here we can see that the whole component get's rendered multiple times.

It looks like you’re missing an offset, see https://putyourlightson.com/sprig/cookbook#load-more

I'm not sure to understand. I intentionally did not use the offset. As I understand it, the whole template will be rerendered, so this is not a problem. Also, if I use the sprig.isInclude, the whole part with the categories just dissapears.

Please use the code from the recipe I linked, to see how it works. If you are still stuck then I can look at your code again.


Added type indicators to NbtDouble and NbtFloat

Type safety is important for Minecraft to recognize float and double arrays properly.

Minecraft logs an error because it doesn't know that the numbers in this list are supposed to be floats.
java.util.concurrent.CompletionException: java.lang.IllegalArgumentException: Whilst parsing command on line 1: Can't insert TAG_Double into list of TAG_Int at position 246: ...cale:[1,1,<--[HERE]

Doesn't this make toPrettyString add two d or f? And why are you only adding the d to the integer case of NbtDouble?

Looks good! Thanks again!


Why i can save bitlocker key only in usb?

I cleared the TPM and still the problem remains what to do? is there a way to fix without using the BIOS?

Saving to USB works.
Looks like your key doesn't had a drive letter / don't get recognized at all. Check that first

@yonathanluiz . Bitlocker key can be saved in Microsoft account when device is connected to the internet  by enabling BitLocker.
It gives options to take a printout, save key to usb device  and  to Microsoft account

The problem is when I run the option to turn the bitlocker on it automatically tries to generate the key via usb without letting me choose how I want to generate it, and there is not even the option to go back as you can see in the print.

@yonathanluiz . It will gives  the options afterwards , because i enabled once  , saved key in usb device , taken printout and key is also saved in microsoft account .
which version of Windows 10, you using, i am using 1909 64bit 18363.815
https://support.microsoft.com/en-us/help/4026181/windows-10-find-my-bitlocker-recovery-key

@yonathanluiz . When i was tested it does not ask Pendrive to connect while unlocking BitLocker from booting, instead, i typed key every time, the computer starts.
I don't know the way to remove the dependency

@yonathanluiz - May we know which Microsoft document link you used in reference to this issue? Or are you having some technical issues which requires technical support?
If you have some issues with a particular doc, we would really appreciate it if you are able to help by providing more details as to what should be edited and/or included. Thank you!

@yonathanluiz - We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.  If you're experiencing a technical issue, please contact support and open a service ticket for Windows 10 products  so this can get resolved ASAP.
If there is a specific area of the docs that we can improve on, please feel free to open a new issue by clicking on the Feedback section on the specific Microsoft doc which is found at the bottom of the page.


merge package template from arlac77/npm-package-template

Updated standard to latest version

Coverage remained the same at 76.744% when pulling 9f9d8a62ebba46f5a89b3537b50a8bdad004d9d5 on template-sync-2 into f943ab378a2bae8301294f20fe751f2dd1ff4814 on master.

Codecov Report

Merging #84 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   76.74%   76.74%
=======================================
  Files          13       13
  Lines          86       86
=======================================
  Hits           66       66
  Misses         20       20

Continue to review full report at Codecov.

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


URL Parameter Encoding

Using marketing URL's, any parameters that are URL encoded get replaced by a "/". Any insight on maintaining url encoding when branch converts this into NSDictionary?
Thanks!

I have a temporary fix in place for our implementation needs using regex; however I feel there may be a more robust way to handle it; if not already one provided by your SDK.
Please & Thanks!

@oursevendev could you provide an example? That will help us isolate where the issue is occurring and determine the best fix.

@derrickstaten sure, here's a more detailed explanation:
https://bnc.lt/a/key_live_{our_account_livekey}?feature=marketing&channel=email&agentKey=9941356&el=~~{urlEndodedToken}~~
"el=" (encoded login is the parameter that we have generated by our system and is url encoded): {
"$one_time_use" = 0;
"+click_timestamp" =<PHONE_NUMBER>;
"+clicked_branch_link" = 1;
"+is_first_session" = 0;
"+match_guaranteed" = 1;
agentKey = 9941356;
el = {url encoded pieces get converted to a /, space etc. Any way to maintain the urlEncoded value? };
"~channel" = email;
"~feature" = marketing;
"~id" =<PHONE_NUMBER>62655642;
}
*The problem is that once it's on the branch side in iOS, as an NSDictionary; url encoded characters are removed and/or replaced!
I have a work around in place, that is working for our current release of a product; however this would be a good to have fix to the branch SDK.
Cheers 🍻!

@oursevendev thanks for letting us know. Can you send us a concrete string where this is happening -- the value that you are putting in the marketing link, what value you are getting, and what you expect to get passed in.
Also, would be great if you could share with us your workaround.

@oursevendev we can't selectively decode some values. We will always decode any query parameters. You are always welcome to re-encode on the client.

I am running into an issue with this. For me, we are passing an email address as a parameter on a branch link. It's not very common, but when there's an email with a '+' character in it (i.e. test+email@gmail.com), then by the time Branch passes the parameters to my app it has become test<EMAIL_ADDRESS>I have also tried url encoding the email first (test%2Bemail%40gmail.com), but I got the same result. Any suggestions on how to address this?

Have you tried double URL encoding it?
That is, URL encode it, URL encode that, and then send it.
When you get the value on the iOS side you will have to URL decode the value.

@cmmouritsen  See above ^^.

Thanks for the response. Something else may be going on. I keep getting the same params sent to my app (the first params that I used with the url) no matter what params I send. I've even tried another device and I still get the params from the first link that I clicked. It doesn't seem like that is how it is supposed to work. In any case, I'm no longer sure my second statement above is true about it not working when I encode the parameter. I realize this is a different issue, but any help would be appreciated.

@cmmouritsen Can you create a new issue and more fully describe what you're trying to do and what your problem is?


Add Dependabot

Because I'm terrible at remembering to check for updates.

Done in PR #50. Closing.


added cargo space command easter egg

I appreciate this as an Easter egg. However, cargo built-in subcommands have precedence over 3rd party subcommands and aliases. This may cause some issues to others having those in their workflows.

Previous effort 😆
https://github.com/rust-lang/cargo/pull/8618

Closing this for now, as I don't think the implementation can be landed without affecting others.