Bug: TypeScript AST node type PropertySignature as a no-multi-spaces exception does not resolve conflict with key-spacing

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Describe the bug
Given this TS type literal:
type TMyObj = {
    id:     number,
    code?:  string,
};

and the rules:
"@stylistic/no-multi-spaces": ["error"],
"@stylistic/key-spacing": ["error", {
    "singleLine": {
        "beforeColon": false,
        "afterColon":  true,
    },
    "multiLine": {
        "beforeColon": false,
        "afterColon":  true,
    },
    "align": {
        "beforeColon": false,
        "afterColon":  true,
        "on":          "value",
    },
}],

the following is given for id:     number,:

Multiple spaces found before 'number'. eslint@stylistic/no-multi-spaces

Upon resolving that, we get into an inconsitency loop with:

Missing space before value for key 'id'. eslint@stylistic/key-spacing

The two rules are in conflict, the documentation suggests using the exceptions option of no-multi-spaces to resolve such an conflict by ignoring given AST node types.
I checked the node type, it seems to be named PropertySignature: TypeScript AST Viewer
Updating the rule as per documantion to:
"@stylistic/no-multi-spaces": ["error", { exceptions: { "Property": true, "PropertySignature": true, } }],

does not resolve the issue.
I tried other, adjacent node types as well (TypeLiteral, Identifier) just in case, to no avail.
I do have @typescript-eslint/parser configured as the parser.
Does exceptions not allow the use of TypeScript AST nodes?
Repro on stackblitz attached - interestingly, running eslint fix there does nothing even though the issues are reported by it, but manually removing and adding back the spaces does show the inconsistency.
Reproduction
https://stackblitz.com/edit/eslint-stylistic-eslint-stylistic-ueip7q?file=playground%2Feslint.config.js
Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

It seems that using @typescript-eslint/parser as a parser, changes the Node type of some of Nodes, including for interfaces and types.
If you use the AST Explorer and switch the parser to @typescript-eslint/parser and re-inspect the PropertySignature node, it is called TSPropertySignature.
Adding TSPropertySignature in the exceptions key resolved the issue for me.

Great input, thank you, wouldn't have guessed that @typescript-eslint/parser prefixes node types with TS!
Interestingly, swapping PropertySignature to TSPropertySignature did not fix it for me.
Out of curiosity, I added TSTypeAnnotation, and that did "fix" it in the sense that the inconsistency no longer occured, but of course this breaks type annotations everywhere else, e.g. extra spaces in variable declarations are now permitted.
For sanity, I checked with TypeAnnotation, and that does not work as now expected due to @typescript-eslint/parser.
So the TS-prefixed node type is correct, and is picked up, except for TSPropertySignature?
I admit I am rather confused at this point.
Triple-checked everything, can't yet see the reason for this quirk on my end.

Adding TSPropertySignature in the exceptions key resolved the issue for me.

Could you please provide a reproduction of the fix?

It fixed it in terms of type declaration files, but unfortunately as soon as I started going out to other areas of the code base (after posting the original comment) - namely a function with an explicit return type - the spacing started messing up because the @stylistic/key-spacing rule doesn't also have the same level of customisation.
I'm still playing around with eslint configs at the moment to try and get a nice replacement for prettier that allows for custom formatting, which will likely involve some custom rules (I don't have experience with writing eslint rules at the moment), so once I've done that I have a bit more knowledge around the plugins I may try submitting a PR for this.

Hey, I'm having the same issue.
In this case, formatting won't do anything (expected), yet it's complaining.

Now, if I am to miss-align the keys

formatting will change the code to

which isn't correct.
This happens both when too few or too many spaces are present before the colon.
  "@stylistic/no-multi-spaces":
    - warn
    - exceptions:
        Property: true
        TSProperty: true
  "@stylistic/key-spacing":
    - warn
    - align: "colon"

Thanks for the pointers guys, in my case simply adding { TSTypeAnnotation: true } to exceptions fixed all problems for me with this config:
"@stylistic/key-spacing": [
   "error",
   {
      singleLine: {
         beforeColon: false,
         afterColon:  true,
         mode:        "strict",
      },
      multiLine: {
         beforeColon: false,
         afterColon:  true,
         mode:        "strict",
      },
      align: {
         beforeColon: false,
         afterColon:  true,
         mode:        "strict",
         on:          "value",
      },
   },
],
"@stylistic/no-multi-spaces": [
   "error",
   {
      ignoreEOLComments: true,
      exceptions:        {
         Property:           true,
         BinaryExpression:   false,
         VariableDeclarator: true,
         ImportDeclaration:  true,
         TSTypeAnnotation:   true,
      },
      includeTabs: true,
   },
],

Before:

After:

Haven't found anywhere it doesn't work in my case (yet)

Your solution only works when defining a type, not when actually building an object

Pretty sure your case is a different problem

Yup. I've forked the repo and I was actually trying to understand where there would be a loop hole of something.
Turns out, TSProperty: true is actually doing its thing properly when building an object.
In fact, the issue stands with the class decorator: ASTExplorer does not yield anything for it!

Seems this will fix typescript class decorators...
'@stylistic/no-multi-spaces': [
    'error',
    {
        exceptions: {
            Program: true, // !!!!!
        },
    },
],

It does, thanks a lot @LastDragon-ru !

A possible fix here could be something like the ignoredNodes option that the indent rule has.
That could allow key-spacing, no-multi-spaces, and/or type-annotation-spacing rules to specify scenarios that they ignore to avoid conflicts.
Honestly, I don't know why that isn't a more common eslint option anyway. Would definitely help resolve many rule conflicts.


[In Review] Fix for domain command name exercisePart

Checklist

[x] I run yarn run webpack:build:main: the project builds without errors.
[x] I run yarn lint: the project builds without code style warnings.
[ ] I updated the documentation and models.
[x] I tested the changes and all related features on the test server https://artemistest.ase.in.tum.de.
[ ] I added (end-to-end) test cases for the new functionality.

Motivation and Context
In the last PR for integrating the markdown-editor into the programming-exercises I replaced the checkmark in the problem statement with its unicode counterpart (it wasn't stored properly in my database).
However this change doesn't make sense thinking about it, we should directly exchange it with a domainCommand (even though we don't have a button for it yet).
Description

Adjusted the problem statement template with the new domainCommand [exercisePart]
Changed the text parser to use this domain command

One note: The problem statements of programming exercises that were created since the merge of the integration PR for the markdown-editor will not show the exercise part information, as I don't think it is feasible to do an additional check for checkmarks and replace them with the domainCommands.
It should be fine if nobody has created a real problem statement yet.
Steps for Testing

Login to ArTEMiS
Create a new programming exercise
Use the preview of the markdown-editor for the problem-statement: It should show the correct number of exerciseParts

Instead of calling it exercisePart, I would rather call it task.
Could you change it?

@krusche I also prefer task! I just pushed a commit.


minIO na SC

We need to create an instance of Minio on SC and connect it with S3 storage on Pierian. We need this so Pierian can send us data about sequnecing on NextSeq machine.

[ ] Study Minio and run it on Kubernetes
[ ]  Communicate with Michal Růžička about the possibility to connect Minio to Pierian on SC.

[ ] Send an e-mail to trusted.ics.muni.cz about this

[ ]  Communicate with Lukáš Hejtmánke how to technically implement it and set everything to connect with Pierian
[ ]  Communicate with Pierian how to connect with them. (Ask @RadoslavaKacova to initiate a meeting with them)

We need to create an instance of Minio on SC and connect it with S3 storage on Pierian. We need this so Pierian can send us data about sequnecing on NextSeq machine.

[ ] Study Minio and running it on Kubernetes
[ ] Communicate with Michal Růžička about the possibility to connect Minio to Pierian on SC.
[ ] Communicate with Lukáš Hejtmánke how to technically implement it and set everything to connect with Pierian
[ ] Communicate with Pierian how to connect with them. (Ask @RadoslavaKacova to initiate a meeting with them)

I attach presentation created by Pierian prepared for videocall where we disscussed the possibilities of data trannsfer for the first time:
Data egress_MMCI (1).pdf


set-proc-env-on-default after prefix

discord chat

reminder to self - this feature implements the behaviour where calling confighelper.withDefault also sets the process.env variable (prefixed with CONFIG_HELPER_)


Native addons support

Assuming the target machine is the same architecture as the current machine (for this reason the suggestion would be to implement this behind a --binaries flag or similar), here's a suggested approach for remapping and emitting binaries.
The assumption is that the "patterns" for binary emission are generally only of a few types, and that we can build static analysis to detect those patterns and provide an adequate substitution for them through (1) detecting where the binary is based on this static analysis and (2) rewriting the locator code as we do for other fs relocations to point to this new binary location.
Detection Cases (not comprehensive, just what I can find on a quick review):

Statically detect any calls to node-pre-gyp's find method. (literally detecting require('node-pre-gyp').find and all its natural simple variations. Eg like in<EMAIL_ADDRESS>Statically detect any calls to require('bindings')('binding.node') and its associated lookup algorithm.
Others?

Rewriting
The rewriting just full replaces the statically known path. Eg require('node-pre-gyp').find(staticExpression) -> __dirname + '/emitted.node'. This is exactly like the asset relocator already does, but simply extended to these more complex patterns.
There may still be edge cases that are not analyzable, but this partial executor already takes into account all path functions, __dirname and __filename expressions. We can tailor the analysis to the real-world test cases, and likely get quite far here too.

tried ncc on one of our APIs with Newrelic on it - failed due to "node-gyp"
see logs on Node v8.12.0
ʎ  ncc build app.js
ncc: Module directory "/Users/xir/Workspace/epic-nodejs-api/node_modules/@newrelic/native-metrics/lib" attempted to require "npm" but could not be resolved, assuming external.
ncc: Module directory "/Users/xir/Workspace/epic-nodejs-api/node_modules/@newrelic/native-metrics/lib" attempted to require "node-gyp" but could not be resolved, assuming external.
Error: Hash: 763418c0e21c6da9c96a
Version: webpack 4.26.0
Time: 5977ms
Built at: 2018-11-28 16:04:05
 Asset      Size  Chunks  Chunk Names
out.js  6.63 MiB       0  main
Entrypoint main = out.js
  [8] external "path" 42 bytes {0} [built]
 [10] external "fs" 42 bytes {0} [built]
 [25] ./config/config.js 15.6 KiB {0} [built]
 [33] external "os" 42 bytes {0} [built]
 [61] ./utils/logger.js 1.34 KiB {0} [built]
 [96] ./node_modules/express/index.js 224 bytes {0} [built]
 [97] ./node_modules/cors/lib/index.js 6.34 KiB {0} [built]
[100] ./node_modules/winston/lib/winston.js 4.12 KiB {0} [built]
[126] ./utils/logglyClient.js 2.08 KiB {0} [built]
[199] external "cluster" 42 bytes {0} [built]
[462] ./app.js 980 bytes {0} [built]
[487] ./node_modules/winston-loggly/lib/winston-loggly.js 7.42 KiB {0} [built]
[580] ./main.js 1.39 KiB {0} [built]
[581] ./node_modules/newrelic/index.js 5.54 KiB {0} [built]
[905] ./node_modules/helmet/index.js 1.17 KiB {0} [built]
    + 1008 hidden modules

node-canvas requires the .node file directly without a helper module, so we should support that use-case as well:
https://github.com/Automattic/node-canvas/blob/490ef47432af83582cf5e491a8ca8797d07c93b0/lib/bindings.js#L3

Could also use / take inspiration from https://www.npmjs.com/package/node-native-loader

Completed in https://github.com/zeit/ncc/pull/93 and https://github.com/zeit/ncc/pull/85. If any new scenarios come up, please post a new issue.


error: took recursive isolate lock; kj::getStackTrace() on storage.put

I get this error:
workerd/jsg/jsg.c++:133: error: took recursive isolate lock; kj::getStackTrace() = 1004af23b 100750017 100772f97 100772f1b 101b9be03 1009e7e0b 10094f88f 10094fab7 101b6e68f 10083f98b 101b6e9b7 100840477 101b6e9b7 100735257 101b70337 101b6e9b7 100735e4b 101b6f663 101b6e68f 100736a7f 101b6e9b7 100736bab 101b6f663 101b6e68f 100736f8f 101b6e9b7 10073822f 101b6f663 101b6e68f

When doing multiple storage.put operations (in a Durable Object alarm - but I assume it happens in other places).
I am running my code with wrangler dev src/index.ts --experimental-local.
Versions:
wrangler 2.17.0
package-lock.json
"node_modules/workerd": {
    "version": "1.20230518.0",
    "resolved": "https://registry.npmjs.org/workerd/-/workerd-1.20230518.0.tgz",
    "integrity": "sha512-VNmK0zoNZXrwEEx77O/oQDVUzzyDjf5kKKK8bty+FmKCd5EQJCpqi8NlRKWLGMyyYrKm86MFz0kAsreTEs7HHA=="
},

The workerd/jsg/jsg.c++:133 file mentions:
 TODO(soon): This shouldn't happen but we know it does in at least one case.
Do you know what I should do in my code to avoid this?

This happens for me on a Remix.run website for Cloudflare Pages, whenever I debug locally and open the website for the first time.

It does seem to work fine, but a full terminal page in red is a bit intimidating 😅

Yeah our logging in workerd is awkward now. In production, we convert these log lines into Sentry errors, and we tend to do a lot of logging that's just meant to be informative for us as the developers of workerd. For someone using workerd in their own infrastructure, the logging is not really appropriate. We need to figure out what to do about this but it's probably a big project...

If it's a warning, should this be KJ_LOG(WARNING, ...) instead of ERROR?

I'm seeing this error locally too. Not sure what it is I am doing to trigger it?

Echoing the above, keep hitting this when using the wrangler pages runner. Its harmless, but it sure makes it tricky to see real errors, especially for newer devs in the team. Its a shame I have to tell people "just ignore the massive wall of red text..."

Note this seems to be the root cause of this pretty common frustration with Wrangler v3: https://github.com/cloudflare/workers-sdk/issues/3631
Wrangler v3 tries to print a stack trace to the console, but doesn't know how. Since that warning appears before this error, it's getting the bulk of the reports (including my original report).
So I think fixing this issue might also close that one.

I’ve submitted cloudflare/workers-sdk#4511 to use the new log suppression feature to also suppress these logs in wrangler, pending a fix for this underlying issue.


Wrap BTC APIs to support wallet/signer interface

Feature request
Currently the BTC APIs are designed to return a PSBT, like the following exmaple:
// Build
const psbt = await sendBtc({
  from: account.address,
  to: {
    address: 'btc_to_address',
    value: 10000,
  },
  source,
});

// Sign
psbt.signAllInputs(account.keyPair);
psbt.finalizeAllInputs();

// Broadcast
const tx = psbt.extractTransaction();
const txid = await service.sendTransaction(tx.toHex());

@duanyytop wish the build/sign/send process can be wrapped/simplified to a single call, like:
const txid = await sendBtc({
  from: signer,
  to: {
    address: 'btc_to_address',
    value: 10000,
  },
  source,
  service,
});

Estimation
In my mind, it would take some steps to support this feature:

Keep the current design of the  BTC APIs, but rename them to be more accurate (sendBtc -> buildSendBtcTx)
Design a Wallet interface that provides publicKey, address and signTransaction to the APIs
Wrap BTC APIs to create new APIs and accept wallet/signer as prop, simplifying things

The first feature was shipped in #77, but the rest of them are not in the plan due to plan changes.


webui: tasking for project requirements, bugs, and feedback

### Bugs (`P0`)
- [ ] "Show capabilities by process" sort by "Namespace" not available
- [ ] For consistency, specify "1" for singular matches e.g. "process match" to "1 process match"

### Requirements (`P1`)
- [ ] https://github.com/mandiant/capa/issues/2192
- [ ] https://github.com/mandiant/capa/issues/2213
- [ ] https://github.com/mandiant/capa/issues/2219
- [ ] https://github.com/mandiant/capa/issues/2217
- [ ] https://github.com/mandiant/capa/issues/2223
- [ ] Make toggled columns persistent (i.e. when a user toggles off a column, it should persist across new uploads/page refreshes)
- [x] Make announcement banner hiding persistent
- [ ] https://github.com/mandiant/capa/issues/2216
- [ ] https://github.com/mandiant/capa/issues/2183

### Nice-to-haves / General feedback (`P2`)
- [ ] https://github.com/mandiant/capa/issues/2215
- [ ] https://github.com/mandiant/capa/issues/2191
- [ ] https://github.com/mandiant/capa/issues/2204
- [ ] https://github.com/mandiant/capa/issues/2218
- [ ] Allow users to hover over a hex feature value to view its decimal represeentation
- [ ] For namespace we could consider adding/using a dropdown with the covered namespaces.
- [ ] Visualize results for an overview at a higher level (see shared document in project drive)

@mike-hunhoff, I fixed the range, optional and not statements. Let me know if you encounter any issues.

@mike-hunhoff mentions:

What are the meanings behind the highlighting used in the "Match Count" and "Namespace" columns? Do we need a key of some sort? This highlighting isn't used in viewing mode "Show capabilities by function"

I initially grouped rule matches which belonged to the same root namespace with a different color to make it easy to distinguish which rules belong to the same category (namespace). Every root namespace is associated with a color.
I have removed this option.

@mike-hunhoff:

"Show capabilities by process" sort by "Namespace" not available

The namespace field is only populated for child nodes (rules) and not for parent nodes (processes).
When sorting is applied, it only affects the top-level nodes (processes in this case).
Since the top-level process nodes don't have a namespace value, sorting this column doesn't change the order of the processes, which is why it appears that nothing happens when you try to sort by namespace.
Please create a seperate issue so I can implement a custom sort if this is desired.


Version numbers

Tesseract published version 4.1.0 just a few days ago.
The version published here are 5.0.0-alpha and on the "older versions" page there's a version 4.1.0 from earlier this year (March). How do those versions relate?
Since the official release 4.1.0 page explicitly links here, it would be nice if that version was actually available.

I added a comment which explains our reasons to the Wiki page. We recommend using 5.0.0-alpha. The older installers from March and May are based on release candidates of 4.1.0, so they don't differ much from the released 4.1.0.

Thanks for the clarifications, but I still find it a bit unfortunate TBH.
Version 5.0.0-alpha might be ok for the standalone tesseract commandline tool, but not so much for the API.
At least the 4.1.0 installers should be called RC on the download page.
Don't get me wrong, I appreciate the effort of providing windows installers. But the link from the release page make the installer somewhat official and not sticking to the official versioning leaves a bit of a bad taste.
The inconsistent versioning IMO makes it uncecessarily hard to integrate the engine into other products.


Feature Request: Side Menu

In order to navigate to specific component quickly like the side menu from http://design-docs.trexcloud.com

I want a side nav component as well, because I want to use it in scaleft. I think Edbury is working on making it part of nim.


Feature request: Make it configurable which traffic is routed via the tunnel device

At the moment all traffic is routed via the tunnel device. This is because on startup 2 routes are set:
<IP_ADDRESS>         <IP_ADDRESS>         <IP_ADDRESS>       U     0      0        0 lispTun0
<IP_ADDRESS>       <IP_ADDRESS>         <IP_ADDRESS>       U     0      0        0 lispTun0

It would be good to have a configuration possibility to route only traffic that has the destination in the lisp network via the lispTun0 device.
At the moment we need to delete the 2 routes and add a new route to send only traffic for the lisp network to the lisp device:
./oor -f /root/oor/oor.conf -D && sleep 5 && route del -net <IP_ADDRESS> gw <IP_ADDRESS> netmask <IP_ADDRESS>; route del -net <IP_ADDRESS> gw <IP_ADDRESS> netmask <IP_ADDRESS>; route add -net <IP_ADDRESS> netmask <IP_ADDRESS> dev lispTun0

This feature has been added with commit 4818f37020ca9a846cd7881c4b1e51fc2cae3a71 .
allowed-dst-eids section added in the configuration file. This feature is only available in linux and openWRT distributions


chore(airtable): upgrade airtable to latest version, v0.7.2

This updates airtable to the latest version, v0.7.2. See the changelog for more info on what changed, but I expect no breaking changes.

Thanks, @EvanHahn :) Haven't heard from @robaxelsen yet (the author of the plugin), but it doesn't look like it will break anything, so I'll merge it.


Add Laravel Livewire detection

Laravel Livewire

A full-stack framework for Laravel that takes the pain out of building dynamic UIs.

https://github.com/livewire/livewire
https://laravel-livewire.com

⭐ 3.7k stars on GitHub
Examples

https://novapackages.com
https://tallstack.dev
Curated list: https://github.com/imliam/awesome-livewire
Used-By GitHub Feature: https://github.com/livewire/livewire/network/dependents?package_id=UGFja2FnZS01OTI1NjcyMDA%3D

Thanks!


[FEATURE] EddieHub Survey ?

Is this a unique feature?

[X] I have checked "open" AND "closed" issues and this is not a duplicate

Is your feature request related to a problem/unavailable functionality? Please describe.
No. It's just a survey when new users come.
Helpful to see where the future of the community wants to go.
Proposed Solution
little option below create your own profile that asks to fill in the little survey.
the survey may ask what do you associate when you visit our website
how do you think our website should look like in a year or so (more bold or more colorful) ?
what features do you look forward to see in our website etc.
just 5-6 questions.
Screenshots
No response
Do you want to work on this issue?
Yes
If "yes" to above, please explain how you would technically implement this
No response

Thank you for raising.
You mention an EddieHub survey on the website but you have raised this Issue on the BioDrop repo. There is an EddieHub website and also a BioDrop website as these are two different things.
We have not had much success with surveys in the past and also the BioDrop website would not be the place to add this in relation to the EddieHub community.
As to the direction of the EddieHub community, members can raise a GitHub Discussion and also chat in Discord, so I will close this Issue.


image-index: remove platform.features field

While the intended use of the platform.features field was to express
the availability of hardware instructions on CPUs, the practicality of
expressing and implementing such a standard has been called into
question. Much of the functionality is usually hidden deep within kernel
code or is defined by hardware vendors in ways that will make the field
unusable without future additions. Applications that can make use of such
field will likely have other requirements on top of this field that are
specific to a deployment.
In that regard, it is best to remove this field and defer to hardware to
make the decision about what can be run.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>Closes #622

Can we still reserve the field name and mark it MUST NOT BE USED?

Can we still reserve the field name and mark it MUST NOT BE USED?

Yes.

@AkihiroSuda @vbatts Updated to be a reserved property.

L:GTM

LGTM

@opencontainers/image-spec-maintainers PTAL

LGTM

@stevvooe conflicts :(

@philips BAH!!! So close. Wait right there...

@vbatts @philips PTAL

LGTM

@philips review please

LGTM

LGTM.

From: Brandon Philips<EMAIL_ADDRESS>Sent: Friday, May 19, 2017 9:19:06 AM
To: opencontainers/image-spec
Cc: Brendan Burns; Team mention
Subject: Re: [opencontainers/image-spec] image-index: remove platform.features field (#672)
LGTM
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fopencontainers%2Fimage-spec%2Fpull%2F672%23issuecomment-302747439&data=02|01|bburns%40microsoft.com|7a7428aa375b4cc3ebae08d49ed2c81d|72f988bf86f141af91ab2d7cd011db47|1|0|636308075515375941&sdata=88eHsN2ls05fiVj%2BM6LM%2Bm2oVEdOhOKmcxL42T2cMX0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFfDgvOVdbp84TdLS79-DvMg8BRU3Dgxks5r7cD6gaJpZM4NYrLV&data=02|01|bburns%40microsoft.com|7a7428aa375b4cc3ebae08d49ed2c81d|72f988bf86f141af91ab2d7cd011db47|1|0|636308075515375941&sdata=nH%2F6fWuVTABqMc90kVoNFTKTymITMcR7Cf%2BRGFf3p8U%3D&reserved=0.

Thanks, folks!


Doubt regarding apply_similarity_pose function

I have a doubt regarding the function "apply_similarity_pose" in align.py
So given a transform from Camera world to similarity world, this function computes the reverse.
So the existing inverse function is calculated as ,
Rp = R.dot(A.T)
tp = -Rp.dot(b) + s * t

But according to derivation, the inverse function looks like
Rp = (R/s).dot(A.T)
tp = -Rp.dot(b) + t

An example for verification:
Lets assume,

R = A = I(3x3)
b = t = [0, 0, 0]
s = 2

Basically, there is only a scale difference between similarity and camera world. So coordinates in similarity world is obtained by just multiplying 2 with camera coordinates
Now according to the formula in align.py
Rp becomes I(3x3).dot(I(3x3)) = I(3x3)
tp becomes -[0, 0, 0] + s * [0, 0, 0] = [0, 0, 0]
With this Rp, tp the coordinates in similarity world coordinates remains same during Similarity->Camera world conversion and is not scaled down. With the formula that I have posted the similarity world coordinates gets scaled down appropriately.
So Am I missing something in this calculation such that the existing calculation in align.py is correct?. Or is the scale neglected ? since down the pipeline we usually use a normalised unit scale point to do triangulation?

Hi @gouthamvgk ,
@YanNoun can you help resolve with this issue? I just want to make sure that we don't have a bug 😃
Best,
Fabian

Hi @gouthamvgk,
Thanks for the heads-up, this is indeed not very clear and stems out due to the use of the both camera-centric and world-centric coordinate frames for expressing a camera poses, due to many refactors :

We apply  a similarity as   y = s. A. x + b

The camera pose is Rc | tc in camera-centric coordinate frame such as x = [Rc | tc] . X project a world point X to the camera as x

Conversely, we have the world-centric coordinate frame of the camera as : x = Rw (t) . (X - tw) with Rw = Rc (t) and tw = -Rc(t) . tc with (t) denoting the transpose

When applying similarity to the rotation part :
Rwp = A . Rw                                                world-centric coordinate frame
Rcp = (A . Rw) (t) -> Rcp = Rc . A (t)      camera-centric coordinate frame
(see this line https://github.com/mapillary/OpenSfM/blob/7b455d9c424c82d96fc60d57a9c1dbf024aa8b3b/opensfm/align.py#L46)

It gets messy for the translation :
twp = s . A . tw + b                                       world-centric coordinate frame
tcp = -Rcp . (s . A . tw + b)                         camera-centric coordinate frame
tcp = -s . Rcp . A . tw - Rcp . b
But since Rcp . A = Rc :
tcp = -s . Rc . tw - Rcp . b
And since tw = -Rc(t) . tc - > tc = -Rc . tw
tcp = s . tc - Rcp . b
(see this line https://github.com/mapillary/OpenSfM/blob/7b455d9c424c82d96fc60d57a9c1dbf024aa8b3b/opensfm/align.py#L47)

This is utterly complicated and could be written instead as :
R_transformed = A.dot(pose.get_R_cam_to_world())
o_transformed = s. A.dot(pose.get_origin()) + b
pose.set_from_cam_to_world(R_transformed, o_transformed)

Feel free to issue a PR to simplify this mess !
Yann


Rename overloaded to/from_dask_dataframe API

Closes https://github.com/dask/dask-expr/issues/905

@phofl - I've been finding the overloaded/conflicting meanings of to/from_dask_dataframe in dask.array/dask.bag/dask-cudf/etc to be a bit confusing. Hopefully you are still okay with renaming.
You mentioned earlier that we will also need to make changes in dask/dask. Does it make sense to leave the deprecation warning out until we are confident that dask/dask is using the "correct" name?

Yeah leaving the warning out until the next dask/dask release happens would be good

Thanks for the review @phofl!  I addressed your comments.

thx

Can you update the following things in dask/dask:

Add the new method to the api reference
replace usages of to_dask_dataframe/from_dask_dataframe with the new methods

And create a PR here that adds the warning so that we can merge it before the next dask/dask release happens?

Sounds good. Thanks!


Order of primes in factorise

I realize now that this is not actually stated in the documentation, but I always assumed that the primes would be listed in ascending order as they are almost always.  However:
Prelude Math.NumberTheory.Primes.Factorisation> factorise<PHONE_NUMBER>1
[(101701,1),(100801,1)]
it :: [(Integer, Int)]
(0.00 secs, 1,321,416 bytes)

Is that intended?  Running arithmoi 0.8 on Haskell Platform 8.6.3.

Yes, it is intended. Otherwise one cannot factor huge numbers in a lazy fashion, e. g., head . factorise.

If there is an advantage to it, let's keep them returning in non-ascending order.  But perhaps it could be mentioned in the docs that the order of prime factors, while almost always ascending, this is not guaranteed.
I'm a pretty heavy arithmoi user and I didn't know that and hence spent two days tracking down a bug in my code ultimately due to the fact that fst.last.factorise is not a valid way to find an integer's largest prime factor.  Rather, in rare cases one one needs foldl' max 0.map fst.factorise

Sure, it is a good idea to clarify this in docs, thanks for suggestion. Sorry for your troubles.
Small factors below 100000 are found by trial division and thus are always ordered. But when it comes to factorisation over elliptic curves, the order of factors becomes unpredictable. So factors become unsorted only when there are at least two sufficiently big prime factors.


Added option pm2-manage_pm2

Required to manage pm2 with external tools and use only cookbook's LWRP

@luiscvalmeida could you, please, review/merge PR? or someone else?


docs: add installation page to main documentation

Description
resolves #464
Type of change

docs: Documentation only changes

Questions (Optional)

not sure where to put this... first page under "User Guide"?
also not sure what @musslick mean't by the last task -- "We should also consider adding the name of the autora pip/conda package for each tool in the main doc file (index.md) of the respectvie package."

Remarks (Optional)

to me, listing all the optional dependencies seems overkill and likely to get lost in the process of adding new ones going forward. as such, i opted to refer people to the pyproject.toml file for a complete list of options. this is comprehensive, and will necessarily be updated as optional dependencies are added. happy to change and create some sort of table, but aiming for effective viable solution that doesn't add documentation liability in the future...

Hey @benwandrew , regarding the last task:
We should list the name of each optional dependency that can be installed with autora. Ideally in a table, e.g.,
| type | package name | description |  depenency |
-------------------------------------------------
| experimentalist | experimentalist-falsification | samples experimental conditions that are predicted to invalidate a model | optional |

@benwandrew
Regarding the remark: I think we should still list all packages in the doc. Better an old list than no list at all. Pointing the user to the pytoml is cumbersome for them and will scare off unexperienced users.

Now resolved!

ok, all sounds good. and thanks for handling the list!
assume we're happy with placement as first page under User Guide?

ok, all sounds good. and thanks for handling the list!
assume we're happy with placement as first page under User Guide?

Yes, very happy :)

@younesStrittmatter can you approve this when you get a chance? because you requested changes, it is not letting me merge until you approve. or perhaps i'm missing another outstanding requested change?

upon another quick read, wondering why certain samplers are listed as required dependencies? if this needs changing, i might also grab a couple typos in another PR...


Add a command to search for where you can get x drop

Example
+/=getwhere Zenyte Shard
-Zenyte shard
Kill Demonic gorilla 1/x chance
(With or without droprate)
If something gets searched that has more than 10 different monster that drops it
List with 10 monsters per page
-clues
-drop location
-different collecting areas

Until this is added to the bot, there’s a useful alternative you can use right now. If you do +w itemname, you can scroll down to drop sources and get exactly this information with like one extra step.

Until this is added to the bot, there’s a useful alternative you can use right now. If you do +w itemname, you can scroll down to drop sources and get exactly this information with like one extra step.

yes but this is only useful for ingame items.
this command would be amazing in the sense of BSO's custom items & actual OSRS items. (OSB)

I think you should just use the wiki.


Release Google.Cloud.ApigeeRegistry.V1 version 1.0.0-beta07

Changes in this release:
New features

Add IServiceCollection extension methods for client registration where an IServiceProvider is required. (commit 022fab2)

Created release for Google.Cloud.ApigeeRegistry.V1-1.0.0-beta07


[WIP][desktop] Issue 575 detailed torrent transfers

This branch is still work in progress. #575

Ported HexHiveView to Java Swing HexHivePanel.

I'm gonna try to squash this into a single commit and bring it over to my branch

Code moved to origin:advanced-transfers branch
https://github.com/frostwire/frostwire/commit/20271e4c38cbb7493f748984eb647d7b77567938
Please delete this branch from your github and local repos (keep things clean), then do a git fetch origin advanced-transfers so you have the branch with all the code.
Then wait for me to make the next commit demoing how one of the panels will work with the bittorrent engine and BittorrentDownload objects.


Dev hashMap

Codecov Report

Merging #12 (6b20ccd) into master (4f31a71) will increase coverage by 0.03%.
The diff coverage is 98.55%.

@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
+ Coverage   97.53%   97.57%   +0.03%
==========================================
  Files          22       23       +1
  Lines        1258     1320      +62
==========================================
+ Hits         1227     1288      +61
- Misses         23       24       +1
  Partials        8        8

Impacted Files
Coverage Δ

collections/hash_map.go
96.96% <96.96%> (ø)

collections/bucket.go
100.00% <100.00%> (ø)

collections/hash_table.go
97.59% <100.00%> (+0.29%)
:arrow_up:

collections/maps.go
100.00% <100.00%> (ø)

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 4f31a71...6b20ccd. Read the comment docs.


Release v1.1.1

Adds

Adds the text variant in the Button component.

Fixes

Fixes a bug where the defaultValue for a TextInput component was not being passed correctly to the Chakra input element.

Deprecates

Deprecates the link variant in the Button component.

Looks okay but maybe Notification and Header should have their version updated to 1.1.1 as well.

@EdwinGuzman Yes, they should be updated.  Please review one more PR…
https://github.com/NYPL/nypl-design-system/pull/1145


Generate type synonym with all type variables under Maybe

Take for example the type
data Book' c1 c2 c3 =
  Book
    { bookAuthor :: c1
    , bookText :: c2
    , bookBlurb :: c3
    }

type Book = Book' Text Text (Maybe Text)

type BookReadColumns = Book' (Column PGText) (Column PGText) (Column (Nullable PGText))

type BookWriteColumns = Book' (Column PGText) (Column PGText) (Maybe (Column (Nullable PGText)))

type BookNullableColumns = Book' (Column (Nullable PGText)) (Column (Nullable PGText)) (Column (Nullable PGText))

The result of a query returning BookReadColumns is Book, however there is no type synonym for the value of a query returning BookNullableColumns.
It would be great to also have this synonym and function.
type BookMaybe = Book' (Maybe Text) (Maybe Text) (Maybe Text)

sequenceBook :: BookMaybe -> Maybe Book
sequenceBook (Book c1 c2 c3) = Book <$> c1 <*> c2 <*> pure c3

This would certainly be useful when Book is on the right of a left join.
@tomjaguarpaw: Is this a sensible way of dealing with products of nullable columns? As far as I can tell the machinery in product-profunctors isn't quite adequate due to the fact that Nothing is a valid value for bookBlurb and the conversion from BookMaybe shouldn't return Nothing because of that.

On Sun, Nov 13, 2016 at 12:58:30PM -0800, Joe Hermaszewski wrote:

type BookMaybe = Book' (Maybe Text) (Maybe Text) (Maybe Text)

sequenceBook :: BookMaybe -> Maybe Book
sequenceBook (Book c1 c2 c3) = Book <$> c1 <*> c2 <*> pure c3

This would certainly be useful when Book is on the right of a left join.
cc @tomjaguarpaw: Is this a sensible way of dealing with products of nullable columns? As far as I can tell the machinery in product-profunctors isn't quite adequate due to the fact that Nothing is a valid value for bookBlurb and the conversion from BookMaybe shouldn't return Nothing because of that.

Hi Joe, could you explain more about what you mean about products of
nullable columns?  I don't quite understand.

@tomjaguarpaw The type synonym BookNullableColumns in the first post is an example of this.
A use case might be in performing a left join between tables A and B. It seems quite handy to be able to end up with [(ThingA, Maybe ThingB)] after running the query. The query itself could return [(ThingA, ThingBMaybe)] and sequenceThingB seems to fill the gap here.

Have you seen the new FunctionalJoin module?  It's a more sane approach to outer joins.
https://www.stackage.org/haddock/nightly-2016-11-09/opaleye-<IP_ADDRESS>/Opaleye-FunctionalJoin.html

Yeah, I was trying to coerce that into doing what I want earlier today. It's certainly an improvement on leftJoin. Please correct me if I'm wrong, but I don't think it's possible to have this (or something equivalent) columnsResult ~ (columnsL, Maybe columnsR).

It's not possible at all in Opaleye, for good reason.  The constructor of the Maybe must be statically known at Query run time so can't possibly depend on the inputs to the left join.

I thought that might be the case. Looking forward to the monadic interface ;-)

It's not possible for any interface where Query is a functor, otherwises you could map over the constructor of the Maybe and get a query result that depends on a Haskell function.

;-)

;-)

Oh, ;-)

Returning to this:

BookMaybe -> Maybe Book

data P a b = P (a -> Maybe b)

is an instance of ProductProfunctor, and then
instance Default P (Maybe a) a where
    def = id

gives you sequenceBook = def.

(which, of course, generalises to all Applicatives)

You may also want
instance Default P (Maybe a) (Maybe a) where
    def = pure

depending on your application.

@tomjaguarpaw I assume you mean
instance Default P (Maybe a) a where
  def = P id

instance Default P (Maybe a) (Maybe a) where
  def = P pure

sequenceJobset :: JobsetMaybe -> Maybe Jobset
sequenceJobset = unP def

I neglected Default when looking for a product-profunctors solution to this. It's certainly nicer than expanding to long applications of <*>.
There's probably still a case to be made for the sequence* functions being around as the type inference around leftJoin and def is (understandably) terrible :)

Also sequence perhaps isn't the best name for this, suggestions welcome.

Indeed, w hat's what I mean.
If you want the function for it's type signature I guess that's fine but you may as well use product-profunctors for the function body.

Closed in

@expipiplus1 Reviewing this now, I feel like this is a useful addition, so would like to merge it. Do you want to change the function body implementation per Toms feedback, perhaps that would further reduce clutter a little bit?
I also intend to (at some point) add another flag to allow enabling/disabling generating the NullableColumn stuff, it's not always relevant so could include this there as well then.
As for naming, since we have BookReadColumns associated with Book perhaps a more natural dual to BookNullableColumns is NullableBook instead of BookMaybe?
Then you could call the function that turns a NullableBook into a Maybe Book something like fromNullableBook, maybeBook or something more "natural" like that?
If you want to make any of these changes, please feel free, if not I will merge anyway and let someone else with a stronger opinion than me change it to what they want :)

@folsen all those suggestions sounds good, I'll see about getting them done today.

Yeah, all the other type synoms start with Book I suppose so it would make sense that that one would as well. I have no strong opinion either way, so do what you feel is best!

I also intend to (at some point) add another flag to allow enabling/disabling generating the NullableColumn stuff

By the way, I'm really hoping to move away from NullableColumn stuff now that FunctionalJoin exists.


[semver:major] cut initial release

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/eld/nix?version=dev:156977ab5873f2bc891028a6affb3cc97458a2d0

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/eld/nix?version=dev:d7228f56fcbb1959b3cf58480cf5aeee020d9b93


Add more Container Platform logs to collect-windows-logs.ps1

Add more containerd logs to collect-windows-logs.ps1 to help troubleshooting for Container Platform team.
We would like the following data to be collected by default with collect-windows-log.ps1 script when ICMs are filed. This will reduced some back and forth with the cx to collect additional data for us to look into:

cd '.\Program Files\containerd'
.\containerd.exe --v
.\crictl.exe pods
.\ctr.exe c ls
.\shimdiag.exe list
for each row got from (5), run the following and share outputs got for each:
   .\shimdiag.exe stacks <id got from each row in (5)>
.\ctr.exe snapshot ls
.\crictl images
9)  get-process containerd-shim-runhcs-v1
get-process CExecSvc
Get-Process vmcompute

@junjiezhang1997 I think that we also need to update https://github.com/Azure/AgentBaker/blob/master/staging/cse/windows/provisioningscripts/loggenerator.ps1.

From the recent ICMs, we noticed that having total commit memory of a container would be really helpful to have as part of the logs too. Can we add something like the following to collect total commit memory consumed by all the process in the container by session id?
get-process | group-object -property si | foreach-object { new-object psobject -property @{ SessionID=$.Name; Memory=($.Group.PagedMemorySize64 | measure-object -sum).Sum } } | format-table

The following information will also be very helpful to have:
kubectl get nodes -o wide
kubectl top nodes
kubectl top pods
kubectl get events -n kube-system

For the first comment, I think we can add it in both collect-windows-logs.ps1 and loggenerator.ps1. My only question is the result of your command only shows memory and session id as follows. Is it enough?

Memory
SessionID

66633728
10

64245760
11

1105395712
0

25440256
2

25530368
3

25284608
4

66252800
5

58503168
7

57171968
6

59957248
9

60559360
8

15704064
1

For the second comment, I'm sorry that we could not implement it since collect-windows-logs.ps1 runs in a windows node and those kubectl commands should be executed at the outside of a node. Of course, we think these commands are really helpful and we add them into our TSG page to help customer supportors or pg engineers collect logs.

For the first comment, I think we can add it in both collect-windows-logs.ps1 and loggenerator.ps1. My only question is the result of your command only shows memory and session id as follows. Is it enough?
Memory	SessionID
66633728	10
64245760	11
1105395712	0
25440256	2
25530368	3
25284608	4
66252800	5
58503168	7
57171968	6
59957248	9
60559360	8
15704064	1
For the second comment, I'm sorry that we could not implement it since collect-windows-logs.ps1 runs in a windows node and those kubectl commands should be executed at the outside of a node. Of course, we think these commands are really helpful and we add them into our TSG page to help customer support or pg engineers collect logs.

For the first comment, I think I wanted to collect the total commit memory of every container running in the windows node. The aggregation is correct, but it would help to associate it with the containers that are running in the system (that is, which container is which session id). We should exec into the container to get information about the session id of a container and co-ordinate the container id with the output got from the command. Not sure if there is a better way to do this on AKS windows nodes? Let me think. @AbelHu thoughts? Do you know alternate ways to get this information?

@kiashok I think that we may not be able to collect such info in loggenerator.ps1. But it may be possible to collect it in collect-windows-logs.ps1 since normally it means that the node is bad when running collect-windows-logs.ps1. Could you help to provide the commands to us so @junjiezhang1997 can help to add it in the script?

@kiashok I think that we may not be able to collect such info in loggenerator.ps1. But it may be possible to collect it in collect-windows-logs.ps1 since normally it means that the node is bad when running collect-windows-logs.ps1. Could you help to provide the commands to us so @junjiezhang1997 can help to add it in the script?

Hi Abel, based on the meeting we had yesterday to retrieve commit memory, i think we can hold off from adding the commit memory usage in this script. Looks like there could be additional work needed to support it

The following information will also be very helpful to have:
kubectl get nodes -o wide kubectl top nodes kubectl top pods kubectl get events -n kube-system

Hi @junjiezhang1997 do you know if these commands were added to AKS TSGs like you suggested in this comment - https://github.com/Azure/AgentBaker/issues/2733#issuecomment-1443329794? I am working on few ICMs where having this information would have really helped. They are currently missing from the logs shared by CSS.

One last commands that I think will be useful is -
.\hcsdiag.exe

Oh, these commands are already added into TSG pages. But I think we and CSS miss them recently. I will clarify these commands with my colleagues and CSS. Thanks for reminding.


fix range B5 -> C5 to match example code

change the sample description on line 21 to match the address range shown in the code sample on line 26

Docs Build status updates of commit 3b3b22f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-add-ins-ranges.md
:white_check_mark:Succeeded

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

@chadmando thanks for fixing this typo -- we appreciate the contribution!


Stop toString from descending into arbitrary objects

The function toString has code to convert Elm-records into a string representation.
This code missed a checked whether the JS object is really an Elm-record.
Problem: If toString is called on an recursively linked JS object (not an Elm-record, e.g. from a native library function), this will lead into an infinite recursion.
The check used to be in the code up to commit 08bf8095e24feefc1b13acd7e428081e6d5c20c9. I don't know if Evan removed it intentionally for some reason.

Yes, it was removed because records no longer have a _ field that distinguishes them.
Can you give concrete examples of values that mess this up?

Ah, didn't know about the dropped _ field.
My example is a test app for elmfire. The native code returns a reference to internal data of the Firebase library. This data happens to have a cyclic structure.
I now have a workaround, where I drop this cyclic Json data from the record before printing.
Does this mean, that cyclic structures are disallowed as a Json.Encode.Value? This would be kind of a burden for native wrappers around JS libraries. But it would be a valid argument to not have to do cycle checks in functions like toString (and probably in comparing and equality checking too).


Misleading Wiki page about self-hosting

On step 13 of this page: https://github.com/NuGet/NuGetGallery/wiki/Hosting-the-NuGet-Gallery-Locally-in-IIS, it says to update dbo.GlobalSettings to match your desired configuration.  With this migration,<PHONE_NUMBER>18537_MovingGallerySettingsToConfiguration, it looks like columns from that table were dropped to prefer a web.config based configuration.
Should this be updated to say something like this?

Update the appropriate Web.Config appSettings to match your desired SMTP/Email and Branding settings

I spent a few minutes tripping over this to find that the GlobalSettings table is no longer intended for email configuration.

Yes, those instructions are very out of date. Apologies for that. The up-to-date instructions are located here.

I have added a notice to the wiki page directing to the instructions in our README.md. I don't think we can entirely remove the outdated instructions because they concern hosting the gallery on IIS, which is not covered by the README.md instructions. This issue can be used for tracking of the eventual modernization of the instructions for IIS.

We have additional users asking for instructions: https://github.com/NuGet/NuGetGallery/issues/5621

Removed the misleading instructions.


Bug 2082441: Add docs for tagging requirements in UPI clusters

Add additional documentation regarding AWS resource tagging requirements in UPI clusters.
Signed-off-by: thejasn<EMAIL_ADDRESS>

/lable qe-approved-label

/label qe-approved


blog post #5

1,000+ Open Source Projects rely on dotenvx
1,000+ open source projects now rely on dotenvx. 🎉

It's a far cry from dotenv's 18.7 Million, but it is growing fast - faster than dotenv initially grew. - Mot

It's truly a pleasure to see the community adopt it. Here's some notable projects making use of it.
Facebook's Rapid

https://dotenvx.com/blog/2024/09/12/from-zero-to-1000.html


How can I update a variety of installations in an enterprise without accidentally installing additional options?

Hello,
We have a number of developers in our environment with a wide variety of Visual Studio install options working in an offline environment. Due to the lack of WSUS capability for VS2019, it has become very difficult to update Visual Studio in our environment. For example, it seems necessary to create a layout or a command structure for every single devloper layout to prevent installing additional or missing features when updating these installations. If I create a generic layout with all options, and then try to run it against the various system, it will invariably install the entirety of options to the workstations rather than simply updating the currently installed options. It would be ideal to use a generic layout but have the ability to easily update installations without modifying the installed options. A parameter such as -updateonly which instructs the layout to only update existing options would be useful. It would honestly be much better if there was an easier way to update visual studio offline. Currently, I have to go machine by machine to run the layout. This is very slow and tedious.

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

ID: 228076d1-94bf-c502-c018-08f7c0faf377
Version Independent ID: c64a5cd5-6236-4192-2bfc-923aa20c94da
Content: Create an offline installation - Visual Studio
Content Source: docs/install/create-an-offline-installation-of-visual-studio.md
Product: visual-studio-windows
Technology: vs-installation
GitHub Login: @jillre
Microsoft Alias: jillfra

@ducttapecoder-vt, it looks like you’re having an issue with installation. If your Visual Studio installation fails, see Troubleshooting Visual Studio installation and upgrade issues for step-by-step guidance.
We also offer a Live Chat (English only) support option for installation-related issues.
Here are a few more support options:

Report product issues to us via the Report a Problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Suggest a feature, track product issues, and find answers in the Visual Studio Developer Community.
Use your GitHub account to talk to us and other Visual Studio developers in the Visual Studio conversation in the Gitter community.

And you may find answers here:

https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.

Hi @ducttapecoder-vt and @Anton-V-K, I believe you are looking for information on how to update a network installation of Visual Studio. See here for how to deploy an update to client machines.


Create a deployment page for Windows

We should have https://flutter.dev/docs/deployment/windows, as we do for all the other platforms (including macOS and Linux); see https://github.com/flutter/website/pull/6436#pullrequestreview-795449494 for some background.
It should ideally:

Move content that's currently on flutter.dev/desktop
Expand to cover changing other things like the version that are in the .rc file.

/cc @domesticmouse

This is done.


Validate the input to Optimizer::Run

Run the validator in the optimization fuzzers.

The optimizers assumes that the input to the optimizer is valid.  Since
the fuzzers do not check that the input is valid before passing the
spir-v to the optimizer, we are getting a few errors.

The solution is to run the validator in the optimizer to validate the
input.

For the legalization passes, we need to add an extra option to the
validator to accept certain types of variable pointers, even if the
capability is not given.  At the same time, we changed the option
"--legalize-hlsl" to relax the validator in the same way instead of
turning it off.

Follow up issue #1810 was opened to improve the consistency of the message consumers.

I think I have most of the changes needed to run the validator in the optimizer.  All of the tests pass.  The only remaining issue is the output of the messages from the validator.  The PR in this state will change the output, and I don't think it is as understandable.

The current version of this PR will not change the format of the output for validator errors, but will change the information in the messages from replace invalid opcode.
I think this is fine because those messages rely on OpLine debug information to be meaningful anyway, and in general the line information is not present.  Issue #1810 was opened to follow up with this and make messages more consistent.


Adjust bottom panel's minimal height for the expand icon

Prevents the elements inside the bottom panel going up and down when toggled.

I'm not sure to understand what issues this commit aims to fix.
Trying on the current master branch (ec1e8843), I don't see things going up and down when toggled. Also if this fix is needed, it might be better to fetch the icon's size directly instead of hardcoding 24 (the size would differ depending on the HiDPI scaling).

@akien-mga Here:

Also I'm trying to get the icon's size via gui_base->get_icon("ExpandBottomDock", "EditorIcons")->get_height() but it doesn't seem to be working.

That's interesting, I can't reproduce the jump of the "Output" label seen in your gif when expanding the dock. For me it stays at the same position (the upper boarder does "jump" as there's a bit more padding with the TextEdit above, but that's ok).

Anyway, if it fixes it for you, it doesn't hurt :)


Hanging tests

Hi, just a heads up in case you don't receive any mails from our Trac installation. The original bug report states:
See http://lists.nongnu.org/archive/html/chicken-users/2016-07/msg00040.html for more context.
Basically, async-io tests hang sometimes. To reproduce the issue, just run the tests repeatedly until they hang.
It was making salmonella hang on the linux machines. I installed a hack to kill the hanging test process after some time. Although the hack is in place and working, salmonella runs take longer when the test process hangs, as the hack polls to detect hanging processes.

I just added some debugging output to reader-read!, showing the number of chars it read and the chars.  A successful run looks like this:
read 2 --> he
read 3 --> llo
read 1 --> h
read 4 --> ello
read 4 --> (+ 1
read 3 -->  1)

A hanging run looks like this:
read 2 --> he
read 4 --> lloh
read 4 --> ello

I'm not 100% sure, but I think the problem lies in the reader test's sep-proc.
(define (sep-hello str)
  (if (equal? str "hello")
    (values str "")
    (values "" str)))

In the hanging test, it reads twice and sees "helloh", which means the separator will never return the token. I think something like this should do the trick:
(define (sep-hello str)
  (if (and (>= (string-length str) 5) (substring=? str "hello"))
      (values (substring str 0 5) (substring str 5))
      (values "" str)))


Clangd doesn't work correctly when using the windows path separator ()

compile_commands.json

[
    {
        "directory": "G:\\VsCode\\STL",
        "command": "g++ -IG:\\VsCode\\STL\\include -o G:\\VsCode\\STL\\main.cpp -c G:\\VsCode\\STL\\main.cpp",
        "file": "G:\\VsCode\\STL\\main.cpp"
    }
]

log

I[11:37:12.422] Updating file g:\VsCode\STL\main.cpp with command
[G:\VsCode\STL]
g++ --driver-mode=g++ -IG:VsCodeSTLinclude -o G:VsCodeSTLmain.cpp -c G:VsCodeSTLmain.cpp -fsyntax-only -resource-dir=D:\lib\clang\9.0.0\

This problem caused by Triple.getEnvironment will return EnvironmentType::GNU when compiling clangd with MinGW-W64.
I think we shhould always use TokenizeWindowsCommandLine on Windows.
https://reviews.llvm.org/D70164?id=229011

I'm not familiar with Phabricator linked above. @lh123 has the fix been merged to a release? I'm on clangd 12.0.0 and still seeing the error Found unexpected ':' while scanning a plain scalar.
Is there a workaround for clangd 11 or 12?

20 minutes later I realize that the error was due to having invalid YAML in the .clangd file. Surrounding the string that includes a colon in quotes resolves the issue. The .clangd file should look like this:
CompileFlags:
    Add: [
        -DSOME_SYMBOL=symbol_definition,
        "-IC:\\path\\to\\compiler\\include"
    ]

Sorry for the spam!


Make module.json compatible with Foundry V10

A quick test shows that the module still works in Foundry V10, but it is reporting compatibility warnings.
The changed module.json contains the new syntax required by Foundry V10 to not show these warnings.

Thanks for doing this! I'll merge it and update the artifacts the release point to.


Support async

The handler interface does not support async implementations. Either add a new async method implementation or replace existing one to return a Task.
Add async wherever else it makes sense as well.

+1 just stumbled into a situation, where I needed to call async/await code from the HandleMessage callback. It turns out that just creating a Task via calling an async method seems to work in a console app, however I’m not at all sure that this is the correct sync/async transition. Does the code need to wait for completion? What about exceptions? SynchronisationContext? Does it drain the queue and re-queue all messages in the ThreadPool queue? Does it deadlock itself?
Yes, I guess there is need for either a comprehensive example or even better an async compatible message handler.


[BUG] ReactiveCommand's IsExecuting not set to False after command throwing exception

Describe the bug
If a command throws an exception the IsExecuting observable is not set to False
Steps To Reproduce
This is a test to reproduce the issue. Thanks to @anaisbetts and @glennawatson for the constant support on Slack.
        [Fact]
        public void Test1()
        {
                    var subj = new Subject<Unit>();
                    var isExecuting = false;
                    Exception fail = null;
                    var fixture = ReactiveCommand.CreateFromTask(async _ => {
                    await subj.Take(1);
                    throw new Exception("die");
                    });
                    fixture.IsExecuting.Subscribe(x => isExecuting = x);

                    Assert.False(isExecuting);
                    Assert.Null(fail);

                    fixture.Execute().Subscribe();
                    Assert.True(isExecuting);
                    Assert.Null(fail);

                    subj.OnNext(Unit.Default);
                    Assert.False(isExecuting);  //here it fails
                    Assert.NotNull(fail);
        }

Expected behavior
IsExecuting should be set to False after exception
Environment
Blazor Server on .NET 5

OS:       Windows 10

Thanks for the bug report. One of us should be able to look at it.


Adjust Counter Seed

Is there a way to reset or change the value of the seed of a counter after the first time it's been set?
I've tried to change the value in code, but it seems it's still incrementing based off the first value I ever set.
Example:
patch: $[counter('myProj.patch', 450)]
results in, 450, 451, 452, ...
Then after adjusting the value to:
patch: $[counter('myProj.patch', 500)]
should result in 500, 501, 502, ...
but instead it's 452, 453, 454, ...

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

ID: dd7e0bd3-1f7d-d7b6-cc72-5ef63c31b46a
Version Independent ID: dae87abd-b73d-9120-bcdb-6097d4b40f2a
Content: Pipeline variables - Azure Pipelines & TFS
Content Source: docs/pipelines/process/variables.md
Product: devops
GitHub Login: @andyjlewis
Microsoft Alias: alewis

Any update on this?

You can use the reset counter and reseed counter APIs

@mosswald to comment on this suggestion...

@DominicCabral Changing the value of the counter seed does not reset the current value to that seed. I suggest opening a Developer Community thread on the subject, as that is one way we prioritize future development.
@tspascoal Unfortunately, those endpoints are incorrect. They belonged to an experimental implementation of the counters feature and accidentally got added to the documentation. The endpoints themselves don't exist, and the docs are slated to be removed.


NullPointerException

Hi Lorensius, Thanks for your great work.
I'm still using your July-16 commit and one of my users encountered the following exception. Has this been fixed in the latest release? Thanks!
java.lang.NullPointerException
at android.widget.PopupWindow$PopupViewContainer.dispatchKeyEvent(PopupWindow.java:1340)
at android.view.ViewRoot.deliverKeyEventToViewHierarchy(ViewRoot.java:2471)
at android.view.ViewRoot.deliverKeyEvent(ViewRoot.java:2431)
at android.view.ViewRoot.handleMessage(ViewRoot.java:1741)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:123)
at android.app.ActivityThread.main(ActivityThread.java:4627)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:521)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:878)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:636)
at dalvik.system.NativeStart.main(Native Method)

Sorry. I don't. This was reported by an anonymous user and I couldn't reproduce it on my handsets. Searching online I found a few people encountered similar issues. At least one of them looks like also related to QuickAction.
http://stackoverflow.com/questions/6896356/android-error-nullpointerexception-view-dispatch-key-event
http://stackoverflow.com/questions/6328050/how-can-i-track-down-the-origin-of-this-exception
http://stackoverflow.com/questions/7768728/popupwindow-crash-on-dispatch-event


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_305mmh156q/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_305mmh156q/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_305mmh156q/clone/recipe/meta.yaml:36: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_305mmh156q/clone/recipe/meta.yaml:36: missing_description: The recipe is missing a description
ERROR: /tmp/abs_305mmh156q/clone/recipe/meta.yaml:36: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_305mmh156q/clone/recipe/meta.yaml:36: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found


fix: avoid react native require cycle warning

Description
Fixing the "Require cycle" warning encountered when running @legendapp/state in a React Native environment.
The warning is:
[javascript] Require cycle<EMAIL_ADDRESS>-><EMAIL_ADDRESS>
Changes Made

replace some import statement by relative path

Testing

I tested the 0.69.0 version of react native and it works without warnings

Fixes #59

I have implemented a temporary solution to resolve the Require cycle warning by updating the import statements. However, since my knowledge of rollup is limited, I believe there might be a more optimal solution using plugins. I am open to suggestions and would appreciate any guidance or alternative approaches to effectively address this issue.

These are good fixes, thank you!


Product option URLs are not respected when loading PDP

When changing an option selected, e.g. from "Small" to "Large" the url state is updated.
If you attempt to load the page from that url, the default option is automatically selected.

Closing b/c duplicate of this ticket https://github.com/reactioncommerce/reaction-next-starterkit/issues/408


 When i run it i get an error that says:
[18:52:11 INF] UnturnedMapMergeTool v<IP_ADDRESS> has been loaded!
[18:52:11 INF] Start combining 2 maps
[18:52:11 ERR]
System.NullReferenceException: Object reference not set to an instance of an object.
   at UnturnedMapMergeTool.DataMergeTools.ObjectsDataMergeTool.ReadData(CopyMap copyMap)
   at Program.Start(Config config)
   at Program.Main(String[] args)
[18:52:11 INF] Press any key to exit the program..

Im guessing that i need to give the program an argument that says where the maps are located but i can't figure out how to do that.
Looking forward to your reply.

I ran the program again and it game me some config files. Im just wondering what the ShiftX and ShiftY options are

nvm i figured it out. great tool btw

i still need help with the ShiftX and ShiftY options


Add a Json Patch Action Kamelet

Works really well on camel-jbang
A new lock file was created, delete the file to stop running:
/home/oscerd/workspace/apache-camel/camel/dsl/camel-jbang/camel-jbang-main/dist/./.run9527842704156958828.camel.lock
Starting CamelJBang
2021-12-21 13:14:44.821  INFO 19033 --- [           main] o.a.c.m.BaseMainSupport                  : Auto-configuration summary
2021-12-21 13:14:44.826  INFO 19033 --- [           main] o.a.c.m.BaseMainSupport                  :     camel.main.name=CamelJBang
2021-12-21 13:14:44.826  INFO 19033 --- [           main] o.a.c.m.BaseMainSupport                  :     camel.main.shutdownTimeout=5
2021-12-21 13:14:44.826  INFO 19033 --- [           main] o.a.c.m.BaseMainSupport                  :     camel.main.routesReloadEnabled=false
2021-12-21 13:14:44.826  INFO 19033 --- [           main] o.a.c.m.BaseMainSupport                  :     camel.main.routesIncludePattern=file:pippo.yaml
2021-12-21 13:14:44.826  INFO 19033 --- [           main] o.a.c.m.BaseMainSupport                  :     camel.component.kamelet.location=file:///home/oscerd/workspace/apache-camel/camel-kamelets/kamelets/,classpath:/kamelets,github:apache:camel-kamelets/kamelets
2021-12-21 13:14:53.918  INFO 19033 --- [           main] o.a.c.m.DownloaderHelper                 : Downloaded dependency: org.apache.camel:camel-json-patch:3.14.0 took: 8s933ms
2021-12-21 13:14:54.072  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           : Routes startup (total:3 started:3)
2021-12-21 13:14:54.072  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           :     Started route1 (kamelet://timer-source)
2021-12-21 13:14:54.072  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           :     Started timer-source-1 (timer://tick)
2021-12-21 13:14:54.072  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           :     Started json-patch-action-2 (kamelet://source)
2021-12-21 13:14:54.073  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           : Apache Camel 3.14.0 (CamelJBang) started in 9s216ms (build:41ms init:9s51ms start:124ms)
2021-12-21 13:14:55.091  INFO 19033 --- [ - timer://tick] route1                                   : { "id": "9", "title": [ "Doc 1", "Doc 2", "Doc 3" ], "genre": "Sci-fi" }
2021-12-21 13:14:55.188  INFO 19033 --- [ - timer://tick] route1                                   : {"id":"9","title":["Doc 1","Doc 2","Doc 3"],"category":"Sci-fi"}
2021-12-21 13:15:05.072  INFO 19033 --- [ - timer://tick] route1                                   : { "id": "9", "title": [ "Doc 1", "Doc 2", "Doc 3" ], "genre": "Sci-fi" }
2021-12-21 13:15:05.076  INFO 19033 --- [ - timer://tick] route1                                   : {"id":"9","title":["Doc 1","Doc 2","Doc 3"],"category":"Sci-fi"}
^C2021-12-21 13:15:14.230  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           : Apache Camel 3.14.0 (CamelJBang) shutting down (timeout:5s)
2021-12-21 13:15:14.238  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           : Routes stopped (total:3 stopped:3)
2021-12-21 13:15:14.238  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           :     Stopped json-patch-action-2 (kamelet://source)
2021-12-21 13:15:14.238  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           :     Stopped timer-source-1 (timer://tick)
2021-12-21 13:15:14.238  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           :     Stopped route1 (kamelet://timer-source)
2021-12-21 13:15:14.240  INFO 19033 --- [           main] o.a.c.i.e.AbstractCamelContext           : Apache Camel 3.14.0 (CamelJBang) shutdown in 10ms (uptime:20s292ms)


⚠ ClamAV has degraded performance

In f129879, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 211 ms

Resolved: ClamAV performance has improved in 80fc179.


chore: update git links for npm to display correct urls

When trying to search for the original code with this version, it's redirecting to the fork source instead of this fork. That's a bit confusing.

@Akryum ?


Improve search for finding worker without block

What changes are proposed in this pull request?
Resolve #16405
It replaces the list with set for finding worker without block function which is being called in for loop.
Why are the changes needed?
It improves the search time from O(N) to O(1).
Does this PR introduce any user facing changes?
No

@jja725 Could you please help in merging this PR as we dont have access. Thank you!

Thank you @jja725 and @maobaolong for reviewing the PR.
@jja725 It seems I don't have permission to merge this PR. Can you please help me to merge it. Thank you again.

alluxio-bot, merge this please


called Option::unwrap() on a None value

Hi Prisma Team! My Prisma Client just crashed. Perhaps related to #10586?
This is coming up repeatedly and really making it difficult to work with prisma...
This is the report:
Versions

Name
Version

Node
v18.15.0

OS
darwin-arm64

Prisma Client
4.16.2

Query Engine
4bc8b6e1b66cb932731fb1bdbbc550d1e010de81

Database
sqlserver

Logs
prisma:tryLoadEnv Environment variables loaded from .env
prisma:client checkPlatformCaching:postinstall false
prisma:client checkPlatformCaching:ciName
prisma:tryLoadEnv Environment variables loaded from .env
prisma:client dirname node_modules/.prisma/client
prisma:client relativePath ../../../prisma
prisma:client cwd prisma
prisma:client protocol graphql
prisma:client clientVersion 4.16.2
prisma:client clientEngineType library
prisma:client:libraryEngine internalSetup
prisma:client:engines:resolveEnginePath enginePath node_modules/.prisma/client/libquery_engine-darwin-arm64.dylib.node
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:libraryEngine sending request, this.libraryStarted: true
prisma:client:request_handler {"name":"PrismaClientRustPanicError","clientVersion":"4.16.2"}
prisma:client:libraryEngine sending request, this.libraryStarted: true
pri

Client Snippet
client.getData
    .query()
    .then((data) => {
      // ... code to process data
      return data;
    });

Schema
model actions {
  request_id       Int
  request          all_request? @relation("ActionFirstRequest", fields: [request_id], references: [request_id], onDelete: NoAction, onUpdate: NoAction)

  // ... more fields

  @@index([request_id])
}

view all_request {
  request_id                          Int       @unique
// ... more fields
}

Prisma Engine Query
getData: authProcedure.query(async () => {
    return prisma.actions.findMany({
      select: {
        request: { select: { request_id: true } },
      },
    });
  })

@jlkravitz thanks for the report, could you please provide the missing information so we could reproduce the issue (the query that is failing and the schema)?

@jlkravitz thanks for the report, could you please provide the missing information so we could reproduce the issue (the query that is failing and the schema)?

@aqrln Just added a minimal example based on my code that produces the error.

@jlkravitz That makes a lot of sense. Optimally Prisma would of course not crash here, but return some useful error message to the user what went wrong - according to your description it queried for 1 entry (as indicated by the relation and @unique) but got back multiple ones. Seems we currently just do not have any code to handle that case.
But I have trouble to reproduce the situation you are describing. When I try to manually create a schema similar to the one above, 2 tables with a relation to a non-unique field, with this SQL:
-- CreateTable
CREATE TABLE [dbo].[actions] (
    [id] INT NOT NULL IDENTITY(1,1),
    [request_id] INT NOT NULL,
    CONSTRAINT [actions_pkey] PRIMARY KEY CLUSTERED ([id])
);

-- CreateTable
CREATE TABLE [dbo].[request2] (
    [id] INT NOT NULL IDENTITY(1,1),
    [request_id] INT NOT NULL,
    CONSTRAINT [request2_pkey] PRIMARY KEY CLUSTERED ([id]),
    -- CONSTRAINT [request2_request_id_key] UNIQUE NONCLUSTERED ([request_id])
);

-- AddForeignKey
ALTER TABLE [dbo].[actions] ADD CONSTRAINT [actions_request_id_fkey] FOREIGN KEY ([request_id]) REFERENCES [dbo].[request2]([request_id]) ON DELETE NO ACTION ON UPDATE CASCADE;

(Note the commented out UNIQUE constraint)
I get an error message from SQL server on creation:
There are no primary or candidate keys in the referenced table 'dbo.request2' that match the referencing column list in the foreign key 'actions_request_id_fkey'.

Seems SQL Server does not allow you to create a schema with a foreign key (to represent the relation) that points to a non unique column?
Is this maybe where the views come in?
Or does your table not actually have a foreign key, but you only added that to the Prisma schema?

Does that mean the @relation was also not really backed by a foreign key, but that it only existed in the Prisma schema so that the API is generated to use it like one? Then we could indeed end up in a situation that could work like yours.

Yes- isn't that by definition the case since this is a view? AFAIK, I
couldn't set up a foreign key onto a view, right?
Sent from iPhone
On Fri, May 24, 2024 at 5:04 PM Jan Piotrowski @.***>
wrote:

Does that mean the @relation was also not really backed by a foreign key,
but that it only existed in the Prisma schema so that the API is generated
to use it like one? Then we could indeed end up in a situation that could
work like yours.
—
Reply to this email directly, view it on GitHub
https://github.com/prisma/prisma/issues/21219#issuecomment-2130354099,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWWZK6D7VP6LN6Q5V4UMQLZD6TOHAVCNFSM6AAAAAA5GKBKVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZQGM2TIMBZHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi @laplab Makes complete sense. I will emphasize, though, that the lack of specificity in the error message made this an extremely difficult thing to debug. Even if it had said that "there is a mismatch between the database and prisma schemas," that may have saved several hours. I have encountered this same error message in different circumstances, too. It's quite confusing when a low-level error (in Rust) appears on what is supposed to be a high-level tool (in Typescript).
More than anything, I guess this is my pitch for fixing the error message, even if the use case is (understandably) not supported. If you'd like me to include a ticket or link this one to an existing one, let me know. I'm happy to help, but am not a regular contributor to this project so am not familiar with its processes.


enum Pays for PaysFee

small follow up for https://github.com/paritytech/substrate/pull/5686

enum Pays to be more clear with payment.
expose maximum block weight and length as const so that UI can show them.

Is it actually more clear though? I don't yet see the advantage, unless we expect there to be another option besides yes and no :)

honestly I can't think of a third option now. I was recommended to do this by @bkchr and to me it just made sense as an assurance and being more explicit.

Is it actually more clear though? I don't yet see the advantage, unless we expect there to be another option besides yes and no :)

The point is, if you read the code you will directly understand what this parameter means, without opening the docs and find the place where this is documented.

FYI, this is a backwards incompatible change in the way this is encoded in SCALE for non-Rust clients. Where previously true encoded as 1 and false encoded as 0, the enum discriminant values default to integer values in order, which makes Yes encode as 0 and No encode as 1.

@maciejhirsz Thanks for picking this up. Yeap, the API decoding broken with this, so will have to do an update.

@maciejhirsz @jacogr does it make sense to just flip the order of the enum here?
@kianenigma

This is already deployed on Kusama, so no.

@maciejhirsz @jacogr does it make sense to just flip the order of the enum here?
@kianenigma

OMG, I debug this for a whole afternoon.


Implement basic readline keybindings.

This lets us press CTRL-W to cut a word and CTRL-U to cut to the start of the line in input mode just like default readline bindings (emacs).
None of the other bindings are feasible at the moment because we are not tracking cursor position. Unfortunately the handling for CTRL-W will have to be changed once we track cursor position, but the logic should be the same as the one implemented here so any changes should be trivial.

Definitely going to add more keybinds like these in the future. Maybe specifically emacs/vim mode?


Data visualization and representation

i would like to work on this project. we can use python libraries to visualize the data which would help in clear understanding of the data and will help u to detect pattern, correlation and dependencies.

Please elaborate on the problem statement, i.e., the data you wish to visualize.

I agree to follow this project's Code of Conduct
I'm a GSSOC'24 contributor
I want to work on this issue.

I'm a GSSOC'24 contributor.
Please assign this issue to me.


Draw(w,h) was not producing a bitmap of the required size

I thought that the objective for this was to produce a bitmap of the desired size containing the image at 1:1 scaling but found that this was not the case.
I created DrawAtSize accepting width and height and using them in the Bitmap constructor and invoked DrawAtStdSize which did as Draw did with a ScaleTransform(1, 1);

I had used an svg produced by System Architect which had this in the svg element
width="100%" height="100%" viewBox="75 33 1042 897" preserveAspectRatio="xMinYMin meet"
In comparison, an svg from Visio created by saving an individual symbol to svg worked with the original Draw(w,h) function whereas the function I created does not scale the image to the new extents.
The visio svg contained this in the svg element
width="0.720833in" height="1.32153in"
viewBox="0 0 51.9 95.15" xml:space="preserve"

I had the same issue drawing the bitmap in PictureBox. I ended up using this:
        svgDoc.Width = New SvgUnit(SvgUnitType.Pixel, pbSVG.Width)
        svgDoc.Height = New SvgUnit(SvgUnitType.Pixel, pbSVG.Height)
        pbSVG.Image = svgDoc.Draw()

This worked for my purpose.

I solved it in the manner I described, the code is attached
code excerpt.txt
Sample data attached. The test.svg is what is being rendered, the pic2.png is what Draw(270,232) gives me and pic1.png is what I obtained with the alternative approach. The shaded background covers the area the bitmap was supposed to cover, and the grid-background is a measurement indicator.
dmCapability.svg is an svg which did not give this problem.
files.zip

Ok, thanks for the example - it may help someone else...


Schema for ce_registry does not exist

Received the following error:
{"errors":["Schema for ce_registry does not exist"]}
I have attached a document with the error, json payload and envelope.
I am using:
"envelope_community":"ce_registry","
the exact same community is used with an organization publish but I don't get an error.
Uploading Schema for ce_registry does not exist.docx…

Hi,
I changed these fields.
The update is running on the CI and in a couple minutes will be deployed on the server.

@mparsons1953
The issues seems to have changed.
The first one regarding the duns, and ipedsID fields is fixed.
About the new, regarding the error with "ce_registry does not exists",  can you provide the resource?  (the link on the issue description is wrong)
I tested with the one you've given before (on the previous error), and seems to work fine, but that was an Organization.  I guess you are receiving the error for a credential sub-type right?

Sorry you were too quick for me.
I had started with the ipedsID as an issue. I then saw the issue was related to the schema expecting a string, and we were publishing an array.
For clarity, I will create a new issue, so this one can be closed.


Add/Show components in Device Types

As ...
Austin - Network Automation Engineer
I want ...
to be able to add components correctly in the Device Family and see them
So that ...
just creating a new device derived from the Device Family i can have all the related interfaces, console ports, etc like a blank sheet.
I know this is done when...

I can create, update and delete components within the Device Types
I can create a new Device derived from the Device Types with all the interfaces done.

Optional - Feature groups this request pertains to.

[X] Automation
[ ] Circuits
[X] DCIM
[X] IPAM
[ ] Misc (including Data Sources)
[X] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[X] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
No response
External Dependencies
No response

Hi, thanks for the report. I'm a bit confused as it seems like you're using "Device Family" and "Device Type" terms interchangeably but they're two different data models with different feature sets. It's already possible to add component templates to a Device Type - are you asking for the ability to add components to a Device Family and have those somehow "bubble down" to its child Device Types, or is there something I'm misunderstanding?


More reasonable handling of request messages after test has completed

Currently, after a test has completed, but before the next one starts, the test harness provides a response with responseCode = 103 to all request messages.
For some CBSD this may not lead to a good state at the end of a test case. So, it may require a state machine reset or hard reset to move to the next case.   In some cases, allowing a CBSD to RLQ and/or DRG may provide for faster test execution.
I propose that we hook up the Engine.py generate_canned_response() to the FlaskServer for the case that a request messages arrive after a test has completed, but before the next one is run.  This already exists since it was needed to keep CBSD1 happy in the case that CBSD1 completes all steps in a testcase, but CBSD2 has not yet completed (for DP + 2 CBSD test cases).  It provides the following response messages:
registrationRequest:  responseCode = 200 (does not allow new registrations)
spectrumInquiryRequest:  responseCode = 0 (shows 3550 - 3555 MHz available)
grantRequest:  responseCode = 400 (does not allow new grants)
heartbeatRequest: responseCode = 0 (allows a heartbeating CBSD to continue)
relinquishmentRequest:  responseCode = 0 (allows CBSD to relinquish a grant)
deregistrationRequest:  responseCode = 0 (allows CBSD to deregister)
This has the advantage that, after a test has completed, the CBSD could RLQ and DRG, to prepare for the next test case, rather than requiring (possibly) a hard reset.  If a vendor prefers instead to perform a state machine reset on their device between test cases, then this should not impact them.

If a test case ends with CBSD grant suspended, then the above would allow it to move back to AUTHORIZED state when the test ends, potentially failing the test since the CBSD could start transmitting again  after the last step in the test has completed.
Therefore, need to modify the logic above as follows:
heartbeatRequest:
if in 'AUTHORIZED' state, then responseCode = 0
if in 'GRANTED' state, then responseCode = 501 (i.e. keep in suspended state)
That way, if the test ends with grant suspended, CBSD behaving properly should not start transmitting again after the test has completed.

I made the changes as indicated above, to latest working branch.
I included two flags to file cbrsPython/model/flaskServer.py to enable/disable functionality.
As shown below, I have defaulted nice msg handling to ON.  If this has any unforeseen problems, you can go back to the old msg handling (responseCode=103 for all messages) by setting it to 'False'.
Also as shown below, I added a flag to allow printing these "in between test" requests/responses to stdout (i.e. using print statements, not logging to file).  This is default OFF, but available for development if you want to see the messages happening after a test has completed, but before the next test starts (since these are not logged to the per-test log files).
NICE_MSG_HANDLING_BETWEEN_TESTS: Turns on/off nice msg handling in time
between test cases.
True:  (DEFAULT) - enables nice msg handling
False: reverts to old msg handling (resp with responseCode=103 to all
req msg)
PRINT_MSG_BETWEEN_TESTS:  For use in debugging, if problems occur
between test cases:
False = no printing of msg received in idle time between test cases
(DEFAULT)
True = prints req/resp msg to std out if request msg is recevied during
idle time between two test cases


Generated code with the new https://github.com/protocolbuffers/protobuf-go depends on the old packages

When I generate code using the new protoc-gen-go I get the following generated code. It seems strange to depend on the old "github.com/golang/protobuf/proto" just for the "ProtoPackageIsVersion4" constant:

// Code generated by protoc-gen-go. DO NOT EDIT.
// versions:
// 	protoc-gen-go v1.25.0
// 	protoc        v3.12.3
// source: proto/zipkin_proto3/zipkin.proto

import (
	proto "github.com/golang/protobuf/proto"
)

// This is a compile-time assertion that a sufficiently up-to-date version
// of the legacy proto package is being used.
const _ = proto.ProtoPackageIsVersion4

Am I missing something?

Duplicate of #1077


Old Opencore Version (No Support for MacOS Monterey)

Hi AniKulkarn, The Opencore version of this EFI folder is 0.6.8. Apple has announced MacOS Monterey which has support starting from Opencore 0.7.0. Please Update the OC folder for better performance and Better Support for Monterey.

Yes. I definitely will update the OC version and also update the macOS version once I observe the stability of other Hackintoshes running Monterey.

hello, I have created an EFI folder for E14 Gen1 with all working (Wifi,Bluetooth) and OpenCore 0.7.3 and updated all kexts and With Macbook Pro 16,3 SMBIOS as its very similar - Link for Download is https://drive.google.com/file/d/1hXNDxx2JUZxWP3nrwGtIKcV5lXkRusNu/view?usp=sharing

Hello, I Have created an EFI folder for E14 gen 1 with all kexts updated and with opencore 0.7.3 with everything except Boot Chime working. Link is - https://1drv.ms/u/s!Aq8vmrQDnxgKeHaOHg21AklbrJw?e=GISIth

Thanks @avneesh-go I'll try to update this repository soon!

And is HDMI working too? 😃😃😃

Hello, can you share your EFI<EMAIL_ADDRESS>thank you

您好，我已经为 E14 gen 1 创建了一个 EFI 文件夹，其中所有 kexts 都已更新，并且 opencore 0.7.3 中除了 Boot Chime 之外的所有内容都可以正常工作。链接是 - https://1drv.ms/u/s!Aq8vmrQDnxgKeHaOHg21AklbrJw?e=GISIth

Hello, can you share your EFI<EMAIL_ADDRESS>thank you


(portis) integrate portis as popup

Is your feature request related to a problem? Please describe.
To be able to send a card, web2 users need to get a web3 wallet and ether. (@Lililashka edit if you want!)
Describe the solution @Lililashka would like
Integrate portis into the sender flow, as the primary choice for any user. Both on web2 desktop and web2 mobile.
Describe alternatives you've considered
Don't integrate portis as primary choice, but as one of a few choices. Explain to the user precisely what the tradeoffs are.

Related to 'custom guides / warnings for specific users' https://github.com/RadiCards/radi.cards/issues/59
and https://www.figma.com/file/fYUJNZKCwLxGFIfC7VNa3GW1/CryptoNewYear?node-id=300%3A25

@Lililashka please propose your version as screenshot or figma. Then lets get to a nice version.

now more speficic in https://github.com/RadiCards/radi.cards/issues/73


ReentrantLock is buggy

The following compiles correctly so it is not a TS+ bug but a ReentrantLock one:
https://github.com/Effect-TS/core/blob/main/packages/core/_test/concurrent/ReentrantLock.test.ts#L110

Actually it looks like the concurrent data types are not yet properly maintained in ZIO and have a degree of duplication, I'll kill it in favour of direct usage of TReentrantLock from STM


DRAFT: BED-4435

Description
Implemented validation checks to prevent creation of asset groups with empty names/tags or duplicate tags.
Motivation and Context
Previously, asset groups could be created with tags that duplicate existing tags or with empty names/tags. This could potentially cause widespread bugs in analysis and display given our extensive reliance on string matching for these tags.
How Has This Been Tested?

Updated existing unit test function TestResources_CreateAssetGroup
Added new test cases for:

Empty asset group name
Empty asset group tag
Whitespace in asset group tag

Manually tested with API Explorer UI

Screenshots (if appropriate):
Types of changes

[ ] Chore (a change that does not modify the application functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] Documentation updates are needed, and have been made accordingly.
[x] I have added and/or updated tests to cover my changes.
[x] All new and existing tests passed.
[x] My changes include a database migration.

recheck

I have read the CLA Document and I hereby sign the CLA

recheck

Per https://github.com/SpecterOps/BloodHound/pull/665#discussion_r1658152670, this PR is marked as blocked as we work on a solution to safely migrate database to v5.12.0 while preserving the integrity of the existing database.


Update homework_1.md

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)  I added a logic model for the farmers market data (vendor to vendor_booth_assignment)
What did you learn from the changes you have made?  It is a one to many relationship
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of? No other approaches
Were there any challenges? If so, what issue(s) did you face? How did you overcome it? yes - not easy to build image in Lucid.app
How were these changes tested?  viewed image in a separate browser
A reference to a related issue in your repository (if applicable) NA
Checklist

[X] I can confirm that my changes are working as intended

NA


What is best place to use GraphQL with clean architechture?

There is trends for GraphQL as  a API For Mobile apps and GraphQL is also adopted in most big company to solve some coomon issue with Rest API.
What is best place to use GraphQL with clean architechture?
With or without CQRS patterns?

I think I would need to experiment with this to have a good answer. If anyone has a fork or suggestion or wants to make a PR that would be helpful as well.

Closing due to inactivity. I don't have direct experience with adding GraphQL using this approach that I would share here yet.

Personally, I think you would definitely want to do CQRS if you’re using GraphQL, because it’s all about ultra-flexible querying of the underlying data. Trying to provide that flexibility with domain-centric repositories just wouldn’t work: it’d be really inefficient, lead to a lot of convoluted code where repositories are filled with tonnes of read methods and optional parameters, and your domain model would probably suffer as you’d be concerned about mapping the data from an object model to DTOs.


Update cri-tools to include the supplemental groups test.

https://github.com/kubernetes-incubator/cri-tools/pull/127
I've tried locally:
Running Suite: E2ECRI Suite
===========================
Random Seed:<PHONE_NUMBER> - Will randomize all specs
Will run 1 of 37 specs

SSSSSSSSSSSS
------------------------------
[k8s.io] Security Context runtime should support container with security context
  runtime should support SupplementalGroups [security context]
  /home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/security_context.go:268
[BeforeEach] [k8s.io] Security Context
  /home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/framework/framework.go:50
[BeforeEach] [k8s.io] Security Context
  /home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/security_context.go:49
[It] runtime should support SupplementalGroups [security context]
  /home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/security_context.go:268
STEP: create pod
STEP: create container for security context SupplementalGroups
STEP: Get image status for image: busybox:1.26
STEP: Pull image : busybox:1.26
STEP: Create container.
Aug 30 04:56:06.127: INFO: Created container "689432342bb788bd04448ef2cf4a85726864c40bae4f939c684fd6fde6cac88c"

STEP: start container
STEP: Start container for containerID: 689432342bb788bd04448ef2cf4a85726864c40bae4f939c684fd6fde6cac88c
Aug 30 04:56:06.204: INFO: Started container "689432342bb788bd04448ef2cf4a85726864c40bae4f939c684fd6fde6cac88c"

STEP: Get container status for containerID: 689432342bb788bd04448ef2cf4a85726864c40bae4f939c684fd6fde6cac88c
STEP: verify SupplementalGroups for container
STEP: execSync for containerID: 689432342bb788bd04448ef2cf4a85726864c40bae4f939c684fd6fde6cac88c
Aug 30 04:56:06.241: INFO: Execsync succeed
[AfterEach] runtime should support container with security context
  /home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/validate/security_context.go:235
STEP: stop PodSandbox
STEP: delete PodSandbox
[AfterEach] [k8s.io] Security Context
  /home/lantaol/workspace/src/github.com/kubernetes-incubator/cri-tools/pkg/framework/framework.go:51
•SSSSSSSSSSSSSSSSSSSSSSSS
Ran 1 of 37 Specs in 3.366 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 36 Skipped PASS

Ginkgo ran 1 suite in 3.434110904s
Test Suite Passed

Close. Fixed in https://github.com/kubernetes-incubator/cri-containerd/pull/189.


fix(parser)::Fixed errors with nested table_reference brackets and ci…

Task Description
close #98
Solution Description

make set_operation_stmt to list
add LPAREN RPAREN around relations

Passed Regressions

Unittest

LGTM