Make Interactive Windown Output Like the Visual Debugger Tab

Will be doable if the output cell shown inside the pop-up tab like the visual debugger tab? Making interactive output inside pop-up tab like the visual debugger tab wouldn't be very difficult as it's been achieved by the visual debugger, is it? Another feature is that when the cursor move outside the cell the pop-up tab will disappear and show another cell's output where the cursor is.
I know vscode python team is currently working on improving the New Notebook, but it won't be compatible with 3rd-party extentions soon. Thus, I think what I suggest above is another better way  to improve the experience of data science coding of VsCode and much more doable than making Notebook compatible with 3rd-party extentions.
Would that be possible?
@rchiodo

I believe we already have this suggestion on our backlog:
https://github.com/microsoft/vscode-python/issues/6368
And yes it is technically feasible, just not sure we'll get to it any time soon. Please upvote the other issue. Thanks.

Reopening. I misunderstood @AeneasZhu's suggestion. This is actually a suggestion to instead of (or in addition to) the interactive window, we could also show outputs in a popup window. So an on hover provider that expands outputs for a cell when hovering over it I believe.

Reopening. I misunderstood @AeneasZhu's suggestion. This is actually a suggestion to instead of (or in addition to) the interactive window, we could also show outputs in a popup window. So an on hover provider that expands outputs for a cell when hovering over it I believe.

Thanks very much. And what about this feature's feasibility? And https://github.com/microsoft/vscode-python/issues/6368 . I don't think it might  be very difficult since it's been achieved in visual debugger, is it?

I don't think this would be impossible as I think the hover provider API allows returning HTML.
https://code.visualstudio.com/api/references/vscode-api#Hover
Not sure what we'd hover over though? The cell header? The last line?

I don't think this would be impossible as I think the hover provider API allows returning HTML.
https://code.visualstudio.com/api/references/vscode-api#Hover
Not sure what we'd hover over though? The cell header? The last line?

 And  be classed as Epic?

Sorry but our todo list is decided based on a number of factors, one of which is how many people are requesting something. A single up vote usually doesn't meet the requirements for being implemented any time soon.

Sorry but our todo list is decided based on a number of factors, one of which is how many people are requesting something. A single up vote usually doesn't meet the requirements for being implemented any time soon.

Is still there any other possibility? I can't call others to upvote this.

You can submit a PR with the change required to make this possible. Or somebody else could as well.

You can submit a PR with the change required to make this possible. Or somebody else could as well.

But how to submit ? Could you submit ?

You follow the guidelines here for contributing:
https://github.com/microsoft/vscode-python/blob/master/CONTRIBUTING.md

Let me also say that if you go down this path, we may not even accept the changes. Especially UI changes. So you might end up with your own private branch as the only source of this change.

You follow the guidelines here for contributing:
https://github.com/microsoft/vscode-python/blob/master/CONTRIBUTING.md

Sorry, it seems too complicated for me to contribute that. Could you submit this suggestion to your team or PM?

Hi @AeneasZhu, we have been mulling over additional changes to the debugging experience for the interactive window and this suggestion is on our radar again. However, I'd like some clarification. In your suggestion, are you specifically talking mainly text based output to appear in the popup window when you hover? In other words, if you hover over a variable that contains a float you see the actual value of that variable. If you hover over a pandas dataframe you see the rows and columns of data in the df?
OR
Are you also wanting to see other visualizations such as plots when you hover over a "figure" variable from plotly or matplotlib?
I believe you're mainly talking about just text based output. Please let us know.

@greazer
Sorry to reply your comment late. Acutually, this suggestion was not about interactive window debugger. What I suggested before is to make the result of cell that have been run shown on a popup like the hydrogen on Atom, since Vim Extension is not available on Notebook so far.


Add call to action to docs site home page

Hi,
The docs site for Orator ORM is, indeed, beautiful.
I'm just thinking it misses a call to action button on the landing page, something like "Get started" or "Read the docs".
Not sure if this is the place to discuss the docs site — I haven't found its repo, but if there's one I'd be happy to transfer the issue over there!

It's a little hidden https://github.com/orator-orm/docs :)


Fix issue with k8s.io/docs/concepts/services-networking/ingress.md

Closes #23709

Preview page: https://deploy-preview-23710--kubernetes-io-master-staging.netlify.app/docs/concepts/services-networking/ingress/#failing-across-availability-zones

/sig network


Xiaomi JTYJ-GD-01LM/BW - extra features (smoke_density, self-test, sensitivity)

There are more features for the smoke sensor available and not visible with this plugin:
- smoke_density
The result received is:
{"battery":100,"voltage":3115,"smoke_density":2,"linkquality":94,"smoke":false}
This is a value
- self-test
A self-test can be triggered with publishing zigbee2mqtt/[devicename]/set {"selftest": ""}
This gives a beep in the smoke sensor
- sensitivity
Same as self-test can be triggered with publishing zigbee2mqtt/[devicename]/set {"sensitivity": "value'}
where "value" is low, medium or high
extra info: https://www.zigbee2mqtt.io/devices/JTYJ-GD-01LM_BW.html
If you need more info, please ask.
Thanks.

Could you please update plugin and check

I checked, it works.


HIVE-25920: Bump Xerce2 to 2.12.2.

What changes were proposed in this pull request?

Bump Xerce2 to 2.12.2
Why are the changes needed?

Xerce 2.12.1 has high-level vulnerabilities.

https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2022-23437

Does this PR introduce any user-facing change?

How was this patch tested?

Merged, Thanx @tasanuma for the contribution

Thanks for your review, @ayushtkn!


feature: adds edit btn to image widget

This pull request fixes the following issues:

adds edit button to filer image widget

@lory87 please also provide a screenshot, eventually big and when its small (narrow)

@FinalAngel screenshots are added

@lory87 the pencil icon is not really vertically centered

@FinalAngel


"excludes" setting is ingored

In SublimeLinter.sublime-settings I have:
"PythonLinter": {
     "excludes": ["test_*.py"],
    ...
}

but it still produces warnings in files that match.
Not seeing any error messages in console.

Hi! The settings structure is different. Under the key "linters" comes another dict where you config individual linters. Here the name of the linter is just "pylint". Always the name you also see in the statusbar. We have some links about the settings in the Readme you might want to follow.

Closing as hopefully resolved.


Issue 1075: Improve performance of Emitter.analyzeScalar

I realize this is a PR into a mirror, this seemed like the easiest way to share my proposed fix for https://bitbucket.org/snakeyaml/snakeyaml/issues/1075/heavy-allocation-in-emitteranalyzescalar

@carterkozak please run the tests (respect all the chars, not only the first 2)

@carterkozak for your information, bitbucket also has PR:
https://bitbucket.org/snakeyaml/snakeyaml/pull-requests/


GUI click event

Hi, Ricardo.
Great work for cardboard in iOS.
I've got one question about GUI in unity sdk. I run the unity sample in your package, but can't see pointer image and also can't click on button.
Does your sdk for unity support click event of gui button?
Regards. Johanne.

I didn't and don't have much time to look into this. Did you make any progress with this?


Introduce exclusion patterns for files that shouldn't be sent over the network

Cody should exclude .env files and other files that may contain secrets from being sent over the network.
One potential solution here is to exclude files that aren't part of the Git repository.
Customer requests:

https://sourcegraph.slack.com/archives/C04MSD3DP5L/p1694539385641899
https://sourcegraph.slack.com/archives/C04MSD3DP5L/p1694591949182719?thread_ts=1694539385.641899&cid=C04MSD3DP5L

More requests: https://github.com/sourcegraph/cody/discussions/14

10/18 RFC create, pending for review - https://docs.google.com/document/d/15XJnQ-qa-Y3SM2eltuKfuESmgYYfx7JhEdi2447YF_0/edit?usp=sharing

@DanTup @abeatrix what's the latest status on this? Still to be done?

I think Beatrix was/is working on supporting multi-folder workspaces but I don't know if it's complete yet. The current PR is at https://github.com/sourcegraph/cody/pull/1382 but looks like it has a bunch of merge conflicts now that will need resolving.

The work was done in the linked PR, including working with milti root repo and nest repo, and was waiting for Cody strat team to confirm but then became stale because it was implemented before simple chat was introduced and doesn't work with bfg yet. We will have to pick it back up after GA as we discussed in one of our product meetings @kalanchan


Support for multiple themes

Is it possible to use multiple themes with the same app ? I want some piece of code using Coy and others using the Dark theme. Is there someway to do this ?

Hi! Prism does not currently provide a way to do this out of the box, though you could probably change the selectors used in those themes to fit your needs.
If it were to be a feature, how would you see it? How could we allow both a global default theme and block-specific ones?

You'd have to scope the CSS or something to the context you want to highlight it in; I can't imagine how you'd support this by default, if you even wanted to.

We could try to rely on selectors specificity, but this would require all themes to properly reset any property the default theme would have set...

Hi, do you mean to say that I change the code and pre selectors in the css and js files of one of the themes to some other selectors and use the new selectors to style an element ?

You shouldn't have to change anything in the JS.
You just need to add a class name on every code blocks, for example:

and change the dark theme like this, for example:
code.dark-block[class*="language-"],
pre.dark-block[class*="language-"] {

(just make sure you change all the required selectors in the theme)
And you can do the same with the Coy theme.

Thanks @Golmote that works. Please close this issue.

I've managed to implement this by

depending on some new Prism themes in package.json
  "dependencies": {
    "prism-themes": "^1.9.0",
    "prismjs-darcula-theme": "^1.0.3",
  },

Activating the Docusaurus Sass plugin

add "docusaurus-plugin-sass": "^0.2.2", in package.json,
in docusaurus.config.js activate the sass plugin: plugins: ['docusaurus-plugin-sass'],)

in docusaurus.config.js, define a new global.scss client module
const config = {
  clientModules: [
    require.resolve('./src/modules/global.scss'),
  ]
}

Then in ./src/modules/global.scss I @import the two Prism styles inside a css rule, which means the rule in each imported style will have an additional qualifier.
.language-typescript {
  @import "prism-themes/themes/prism-vsc-dark-plus";
}

.language-kotlin {
  @import "prismjs-darcula-theme/darcula";
}

Because the Kotlin and TypeScript styles have different fonts/colours/sizes that look weird aside each other, I !important override the incongruent properties
pre[class*="language-"], code[class*="language-"] {
  font-family: Source Code Pro, PT Mono, Consolas, Monaco, Andale Mono, Ubuntu Mono, monospace !important;
  font-size: 1em !important;
  line-height: 1.3 !important;
  background: #1e1e1e !important;
}

Now Kotlin and TypeScript have different colours, which makes the difference more apparent.


Add discrimnator to received events to get auto-mapping

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

@lmazuel is this PR still required?

Hi, @lmazuel. The PR has be closed for a long time and it's related branch still exist. If no further used for over 14 days, I will delete this related branch. Please tell me if you still need this branch.

Hi, @lmazuel. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.


Consolidate Holdings Records

Currently we are generating a new FOLIO Holdings record for each row in the TSV file. We want instead to have only one holdings record per Symphony LIBRARY and HOMELOCATION combination but have each row still generate a separate FOLIO Item record.

One holdings record PER INSTANCE RECORD per Symphony LIBRARY and HOMELOCATION.
One item record per csv row, linked to the correct holdings record.

From FOLIO Slack https://folio-project.slack.com/archives/C01HRQKSLL8/p1645714333005169,

Breaking change:
When the tools create holdingsrecords from Items, we define conditions of similarity between items that groups them into > one holdingsrecord. Previously, you defined this using the following cryptic statement:
"holdingsMergeCriteria": "clb",
But from now on, using the develop branch, you can specify any holdingsrecord property name (on the first level) to group > by:
"holdingsMergeCriteria": ["instanceId", "permanentLocationId", "callNumber"],
Properties are available here: https://github.com/folio-org/mod-inventory-storage/blob/master/ramls/holdingsrecord.json

With this change, we should be able to define the instanceId to merge holdings into a single record.

In the current main branch, the default Holdings fields instanceId, permanentLocationId, and callNumber are used to merge holdings as a default. Confirmed with local testing so closing this issue as these are the fields we want to de-dup as well.


Better error messages

Finally getting around to adding custom error messages for the errors that aren't clear. I'm opening this PR before I've added all the custom errors I'd like to add, so please suggest some!
I went to go find the issue(s) that this would close but they've existed for so long that our friend stale[bot] has already closed em.

[ ] Add tests

It'd be great to show a friendlier error for this as well:

@bkeepers, the remaining failing tests made me realize that trying to get the app's details via the API isn't reliable, because if a common error is to forget the PEM file and you need the PEM file to get those details for the error message than the error reporting falls apart.
I removed that bit then added some tests - if you'd like to give it one last review that'd be good (same question to @probot/maintainers)

I just tried this locally, and you still get an error on startup:
$ PRIVATE_KEY=nope npm start
…
02:58:00.994Z  INFO probot: Listening on http://localhost:3000
02:58:00.997Z ERROR probot:
  Promise {
    <rejected> Error: error:0906D06C:PEM routines:PEM_read_bio:no start line
      at Sign.sign (crypto.js:331:26)
      at Object.sign (/Users/bkeepers/projects/probot/probot/node_modules/jwa/index.js:55:45)
      at Object.jwsSign [as sign] (/Users/bkeepers/projects/probot/probot/node_modules/jws/lib/sign-stream.js:23:24)
      at Object.module.exports [as sign] (/Users/bkeepers/projects/probot/probot/node_modules/jsonwebtoken/sign.js:186:16)
      at Robot.app (/Users/bkeepers/projects/probot/probot/lib/github-app.js:12:16)
      at Robot.auth (/Users/bkeepers/projects/probot/probot/lib/robot.js:157:61)
      at getInstallations (/Users/bkeepers/projects/probot/probot/lib/plugins/stats.js:32:32)
      at refresh (/Users/bkeepers/projects/probot/probot/lib/plugins/stats.js:25:33)
      at module.exports (/Users/bkeepers/projects/probot/probot/lib/plugins/stats.js:12:24)
      at load (/Users/bkeepers/projects/probot/probot/lib/index.js:75:5)
      at apps.concat.forEach.app (/Users/bkeepers/projects/probot/probot/lib/index.js:86:45)

I think the try/catch needs to be around the JWT loading in lib/github-app.js or lib/robot.js

Another error to show a friendly message for: No X-Hub-Signature found on request. I think this means that there isn't a webhook secret set at all.

Found the culprit for your comment; there was another unhandledRejection handler that I've gone ahead and used. Now this works as expected:

I'd also like to handler this error:
{"message":"A JSON web token could not be decoded","documentation_url":"https://developer.github.com/v3"}

Which I think means that your PEM key is wrong. @bkeepers can you confirm that?

A JSON web token could not be decoded...Which I think means that your PEM key is wrong

I just ran into this yesterday. In my case that's definitely what the error was. I had regenerated the key to use for one app, but forgot to update it in the other. Not sure how difficult this would be, but would it be useful to show the fingerprint of the private-key that's being used in the error?

would it be useful to show the fingerprint of the private-key that's being used in the error?

Would definitely be useful - where can we get it from?

Based on this SO response:

The fingerprint is the MD5 of the Base64-encoded public key.

I think you'd do this, which I did inside probot run by getting program.privateKey, but that should be also be available as this.app.cert, I think?
const crypto = require('crypto')
const fingerprint = crypto.createHash('md5').update(program.privateKey).digest('hex')

I don't have that feature flag on, so you'd want to test that against what you're seeing in the private key screen.

@tcbyrd I tried out your suggestion above, and the output is most definitely not the fingerprint shown in the GitHub UI. Upon chatting with our friend @tarebyte, it seems that that fingerprint is created using the private-key's public key, which we don't have access to right now. So, I've gone ahead and grouped this error with the other "incorrect private key" error message.

This PR is good for a review minus one thing - I wasn't able to add a test for that last error message. It appears that on startup, the app is making a request (via octokit) and returning that error, but I can't find where that request is being made. I can run probot.start() in a test, but then the server is hard to close and I'd like to avoid that (plus it'd make for a slow test). Any ideas?

@JasonEtco I'm pretty sure the call is made from github.authenticate: https://github.com/probot/probot/blob/b0f69c307f14944370b95b85f083beb32bd3f33b/lib/robot.js#L146-L157
That takes this.app(), which has the certificate in it. Can we mock that call with a good and bad certificate?


Adding support for firmware loading

mapping of the host firmware directory into the load/unload pod only
in case firmware for the Module is defined
during Loading operation, worker pod is copying all firmware files and
directories from image onto the mapped host directory
during Unloading operation, worker pod is deleting all firmware
files, but not directories ( since those can be shared with other
modules' firmware files
unit tests

Codecov Report
Attention: 44 lines in your changes are missing coverage. Please review.

Comparison is base (20fe966) 79.27% compared to head (455cc54) 78.73%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #596      +/-   ##
==========================================
- Coverage   79.27%   78.73%   -0.55%
==========================================
  Files          49       49
  Lines        4849     4937      +88
==========================================
+ Hits         3844     3887      +43
- Misses        829      866      +37
- Partials      176      184       +8

Files
Coverage Δ

cmd/worker/funcs_kmod.go
83.33% <100.00%> (+2.08%)
:arrow_up:

cmd/worker/main.go
0.00% <0.00%> (ø)

internal/controllers/nmc_reconciler.go
77.05% <70.58%> (-0.40%)
:arrow_down:

internal/worker/worker.go
79.81% <46.66%> (-12.78%)
:arrow_down:

... and 1 file with indirect coverage changes

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

/lgtm


Open ID Connect (OIDC) for GHEC Audit Log Streaming to AWS S3 - [GA]

Summary
Today, GitHub’s audit log streaming feature requires storage of cloud secrets in GitHub when configuring your stream. Going forward, the audit log feature will support OpenID Connect (OIDC) for streaming partners. OIDC allows for the use of short-lived tokens that are automatically rotated for each configuration.
Intended Outcome

With the new OpenID Connect (OIDC) support, you can stream to one of our five streaming partners
OpenID token exchange eliminates the need for storing any long-lived cloud secrets in GitHub
Enterprise owners can use the security mechanisms of their cloud provider to ensure minimal access to cloud resources

How will it work?
OIDC will establish an identity layer between GitHub and AWS for the purposes of authenticating GitHub to stream audit log events to a specified AWS S3 bucket. Enterprise owners will establish trust with the GitHub audit log application and assign audit log a role with write permissions to the S3 bucket. When streaming GitHub events via audit log streaming, GitHub will authenticate the cloud role and the Github audit log identity using short lived tokens.

🚢 This has shipped: https://github.blog/changelog/2022-10-04-stream-audit-log-to-aws-s3-with-openid-connect-oidc/


Support std::unique_ptr

It's not possible to use std::unique_ptr as arguments. It works as returned value only.
Here is an example unique_ptr.zip
The problem is that even if we write the appropriate MappedType, the generated code that call the method with a std::unique_ptr argument should std::move this one
extern "C" {static PyObject *meth_Test_setObject(PyObject *, PyObject *);}
static PyObject *meth_Test_setObject(PyObject *sipSelf, PyObject *sipArgs)
{
...
    {
        std::unique_ptr<::AnObject>* a0;
        ::Test *sipCpp;

        if (sipParseArgs(&sipParseErr, sipArgs, "BJ3", &sipSelf, sipType_Test, &sipCpp, sipType_std_unique_ptr_0100AnObject, &a0, SIP_NULLPTR))
        {
          // sipCpp->setObject(*a0); // generated code, doesn't work, we need to move
	  sipCpp->setObject(std::move(*a0));

            Py_INCREF(Py_None);
            return Py_None;
        }
    }
...
}

Could it be supported by SIP ?

Ideally the conversion from python -> C++ would also check that the object IS owned by python first, and if not then raise some kind of Python exception (RuntimeError?) stating that it's not possible to call the method with an object which isn't owned by python.
(Otherwise we'll eventually get a double free crash somewhere)

Adding a /Movable/ (or similar - /NonCopyable/?) mapped type annotation shouldn't be too much of a problem.
I don't see a problem with ownership so long as /Transfer/ is used properly. It may be possible that mapped type conversion code doesn't currently have enough context to handle transfers of ownership properly in this case.
Can I assume that the common use case is going to be when unique_ptrs are going to be passed and returned by value?


Compose 1.13 docs

Proposed changes
Commits that need to be merged to master for 1.13 docs release
@shin- @londoncalling PTAL. You said there were supposed to be 6 commits. Can you find the missing one and link me to it on one of the branches?
Unreleased project version (optional)
Compose 1.13
Related issues (optional)
Reopens #3184

@fermayo can you please make this auto-tagging stop?

The 5 commits here were merged into vnext-compose via #3091. @londoncalling if you did any other commits, they are not here yet.

LGTM, merging. If there is any missing  content we can cherry-pick it in later.


Design Visual Indicator (and Text?) for Indicating Sender's Processing Mode

Acceptance Criteria

What does it look like?
Where do we want to go with this?
What are we calling this?
Is the data being sent right now to a PHD?

A couple examples to try out, saw these while clicking around this week:
In between gov banner and nav

Under nav


support for rust out of the box?

You probably dont want this to get bloated but with two more plugins, you can get a pretty excellent rust experience out of the box.
Inlay hints are pretty important for using rust IMO since it hugely cuts down on the amount of times you have to compile.
If no, then that is fine - I can fork it.
...
  use('simrat39/inlay-hints.nvim')
  use 'simrat39/rust-tools.nvim'

...

require("inlay-hints").setup()
local rt = require("rust-tools")

rt.setup({
  server = {
    on_attach = function(_, bufnr)
      -- Hover actions
      vim.keymap.set("n", "<C-space>", rt.hover_actions.hover_actions, { buffer = bufnr })
      -- Code action groups
      vim.keymap.set("n", "<Leader>a", rt.code_action_group.code_action_group, { buffer = bufnr })
    end,
  },
})
local ih = require("inlay-hints")

require("rust-tools").setup({
  tools = {
    on_initialized = function()
      ih.set_all()
    end,
    inlay_hints = {
      auto = false,
    },
  },
  server = {
    on_attach = function(c, b)
      ih.on_attach(c, b)
    end,
  },
})
-- The line beneath this is called `modeline`. See `:help modeline`
-- vim: ts=2 sts=2 sw=2 et

went through previous issues, I will close


Use exec.CommandContext for timeout control

For https://github.com/go-gitea/gitea/issues/1453 and probably https://github.com/go-gitea/gitea/issues/1398.

LGTM

looks much cleaner, good

LGTM

@typeless  Looks great, but maybe you can give some tests on another PR?

@lunny I'll thinking about it.


can we pass bwa index pointer directly?

I see that the index is passed to the JNI laver via the BWAIndex object. Really though I think only the pointer is actually used. Can it be passed directly rather then passing a Java object that needs to be unwrapped in the C layer? I mean changing this API
	private native AlnRgn[] align(BwaIndex bwaIndex,byte bases[])  throws IOException;
	private native String[] align2(BwaIndex bwaIndex,final ShortRead ks1[],final ShortRead ks2[])  throws IOException;

to
	private native AlnRgn[] align(long bwaIndexPtr,byte bases[])  throws IOException;
	private native String[] align2(long bwaIndexPtr,final ShortRead ks1[],final ShortRead ks2[])  throws IOException;

@SHuang-Broad wdyt?

If you look at this core-file, it seems that though in Java it is a long, when it is actually used in the "core" function align2:
/*
 * Class:     com_github_lindenb_jbwa_jni_BwaMem
 * Method:    align2
 * Signature: ([Lcom/github/lindenb/jbwa/jni/ShortRead;[Lcom/github/lindenb/jbwa/jni/ShortRead;)I
 */
JNIEXPORT jobjectArray JNICALL QUALIFIEDMETHOD(BwaMem_align2)(
	JNIEnv *env,
	 jobject self,
	 jobject bwaIndex,
	 jobjectArray ks1,
	 jobjectArray ks2
	 )
  	{
  	bseq1_t *seqs=NULL;
  	int i;
  	int numPairs=0;
  	jclass shortReadClass;
  	jmethodID shortReadGetName;
  	jmethodID shortReadBases;
  	jmethodID shortReadQualities;
  	jobjectArray samRetArray=NULL;
  	mem_opt_t* opt= _getBwaMem(env,self);
	bwaidx_t* idx= _getBwaIndex(env,bwaIndex);

It actually calls the C-style template for not just getting reference (see lines 255 and 285), the scheme may not be that simple?
static retType _get##method(JNIEnv * env, jobject self)\
	{\
	jlong ptr;jfieldID field;\
	jclass c= (*env)->GetObjectClass(env,(self));\
	VERIFY_NOT_NULL(c);\
	field = (*env)->GetFieldID(env,c, ref, "J");\
	VERIFY_NOT_NULL(field);\
	ptr= (*env)->GetLongField(env,self,field);\
	return (retType)ptr;\
	}\

There could exist a better way of not using this C-style template (I would guess that because mem_opt_t is simple but contains loads of parameters, doing this template-style makes sense), and not exposing an BwaIndex object, but rather handles index loading inside the binding would be better?
Idea:
In align2, simply check if a "global" bwaidx_t * is not 0, instead of calling _getBwaIndex() (which is the call to the template of converting the long representation to the true C ptr representation).

@akiezun i you could pass the pointer/long directly, but IMHO you would lose the semantics of the method  (my IDE can see  that it needs BwaIndex and not any kind of long value).
The C template are a generic (and paranoid) way to safely set/get the internal C pointer stored in the java objects as a long.
@SHuang-Broad @akiezun as I said before in another thread, jbwa is a playground/proof-of-concept for calling bwa from java. I havent' used it since I've published on my blog. I you want to make some major changes feel free to submit a PR or I can even add you as collaborators for this project to speed-up things.

@SHuang-Broad can you try both and report performance? That's all I care about here.

@akiezun Will do.
@lindenb I'll let you know as well, and may submit PR's along the way.


Change app icon

Task/Issue URL: https://app.asana.com/0/72649045549333/1161651092205713
Description:
The iOS team saw a positive response from users after adding the feature that enabled users to change the app icon. We think that the same outcome can be achieved on the Android side.
Disclaimer:
It's known that this feature won't work on all devices. Depending on the launcher and OEM, it's possible that after selecting an icon it closes the app. If it doesn't, be aware that the icon change may take a few seconds before it's applied.
Steps to test this PR:

Open app
Open Settings and Change App Icon
Select a new icon
Navigate to Apps Menu
Check new icon has been applied

Screenshots:

Settings screen
Change Icon screen

The amount of launcher icon assets we have now is huge. This might be the only way to do it, but thought I'd ask the question anyway to question assumptions: do we still need to provide all these PNG or could any of them be replaced with XML instead?

Unfortunately not @CDRussell. Adaptive icons are only available API level 26 and above. If and when we have minSDK 26 we can get rid of those pngs.

@CDRussell I've fixed the issue with the flag after app close. The tests were missing the flag appUsedSinceLastClear so they were green without having to be. I've added a few extra tests to make sure we are on the right side of things


Load config and secret with an immutable .env

Hello,
As far as I understand OpenObserve config, secrets included, are loaded through environment variables or a .env file.  In general both methods are considered secure but there's a debate about using environment variables in Kubernetes. The best security practice seems to mount an ephemeral volume with the secrets (in our case a .env with both secrets and config). This allows fine tuned permissions on the file and configuration immutability using immutable ephemeral volumes.
Would you be interested in a pull request implementing this?
For Kubernetes, see, for instance:
https://blog.nillsf.com/index.php/2020/02/24/dont-use-environment-variables-in-kubernetes-to-consume-secrets/
https://medium.com/israeli-tech-radar/favor-config-files-over-env-vars-d9189d53c4b8
About immutability:
https://www.linkedin.com/pulse/deep-dive-kubernetes-immutable-secrets-configmaps-enhancing-i8iic
This proposal is implemented but the doc don't seems to be up to date: https://github.com/kubernetes/enhancements/blob/master/keps/sig-storage/1412-immutable-secrets-and-configmaps/README.md
Thank you very much.

Thanks @julienkosinski . This is a good concept but I think this is taking things a bit too far.


Doesn't work when inside of a Bootstrap modal

When inside of a Bootstrap modal, the slider doesn't work. See https://jsfiddle.net/ovw8hzLw/.

This is a known anomaly.
There are solutions in the document.
https://github.com/NightCatSama/vue-slider-component#exceptions
This is one of the ways you can refer to https://jsfiddle.net/ovw8hzLw/5/.

I see. Thanks.

It doesn't work for me. Using Bootstrap 4.1 - tried each method from js fiddle.

It doesn't work for me. Using Bootstrap 4.1 - tried each method from js fiddle.
Range is enabled, but value not set default v-model

Can someone explain why this is not working?


Improve output of system-information

Feature-request
Description
The current output of the system-information coming from Azuki is only a plain printed json. This has to be packed in a better graphical structure.

moved to https://github.com/kitsudaiki/Hanami-AI/issues/55


Error running on Mac OS 13.1 _tkinter.TclError: unknown option "-relief"

Not sure if it's supposed to work here (Mac OS 13.1), so I tried it to see -- got some errors that I fixed by installing more packages.
Then got this error that I couldn't figure out with some searches:
python3 picklescan_gui.py
Mac OS Version is 13.1 and patch enabled so applying the patch
Applyting Mac OS 12.3+ Alpha Channel fix.  Your default Alpha Channel is now 0.99
Traceback (most recent call last):
File "/Users/robertw/Stable-Diffusion-Pickle-Scanner-GUI/picklescan_gui.py", line 409, in
main()
File "/Users/robertw/Stable-Diffusion-Pickle-Scanner-GUI/picklescan_gui.py", line 222, in main
widget.Widget.config(relief='flat')
File<EMAIL_ADDRESS>line 1675, in configure
return self._configure('configure', cnf, kw)
File<EMAIL_ADDRESS>line 1665, in _configure
self.tk.call(_flatten((self._w, cmd)) + self._options(cnf))
_tkinter.TclError: unknown option "-relief"

No its made for mac because i cant test on mac, but maybe it will work.
Try to comment out the widget in line 222:
 "input_files_directory_widget"

if it wont work try to comment out :
in lines 227-228
 for widget in widgets:
    widget.Widget.config(relief='flat')

Commenting out the two lines in the second suggestion work though the line numbers given are 221-222 for me and not 227-228.
I got it to scan my files as shown here:
https://prnt.sc/etPTPBzb8Tsl

Great, soon ill update fix,
If you can share what needed to get to to work on mac it will be great.
Thank you.

Here are instructions you can give people though they may not be perfect I am looking through command line logins of what I did and had to make a few attempts for some of them.
Download the files from here using:
git clone https://github.com/diStyApps/Stable-Diffusion-Pickle-Scanner-GUI
Change to the directory downloaded to run: python3 picklescan_gui.py
You might get errors related to certain files.
If related to PySimpleGUI try: python3 -m pip install PySimpleGUI
If related to tk/tkinter try: python3 -m pip install tkinter
If you get an error that says "_tkinter.TclError: unknown option "-relief" "
then comment out in the file  picklescan_gui.py   by putting in # before the lines (currently lines 221-222 or closeby):
 for widget in widgets:
    widget.Widget.config(relief='flat')

After doing each fix run this again and see what errors remain: python3 picklescan_gui.py
You may need some other files/packages not mentioned here that you might be able to find with a google search on the messages.

Thank you for the instructions.
It looks like the problem may be related to installing packages.
One potential solution is to try installing miniconda, which is useful for managing different python environments.
There is a version for Mac available at the following link: https://docs.conda.io/en/latest/miniconda.html.
This may help to resolve the issue with packages.
And then clone the repository again and try the following steps:
Type the following command:
   cd into the Stable-Diffusion-Pickle-Scanner-GUI folder

Type the following command:
   conda env create -f conda.yaml

These steps should help to resolve any issues with the packages.
Let me know if you have any further questions or need additional assistance.
ill update installation guide and if it working for mac users.
Thank you.

I can confirm the code update works now so I don't have to edit the files myself to comment out those lines. I don't know about the package issue though since I had already fixed all that manually. Probably best to see if anyone else who tries it on a Mac has an issue because it's fully working for me.

Great thank you.


Alw outerlands full backend

Full backend implementation for Outerlands fundamental index performance calculation.
Includes queries for:

Calculating monthly asset weights.
Calculating performance of index based on daily changes in component asset prices.

Job runs cleanly:

Data looks good:

Also noting I added layerzero and axelar to the 'bridge' category/list to account for the changes in this PR


Issue #81 Updated Dockerfile

Updated winglian/axolotl to OpenAccess-AI-Collective/axolotl in Dockerfile

Unfortunately, I don't think there was a move for the image to a new org yet for Dockerhub. I'll close this temporarily for now.


Usbong iOS default tree ends after choosing a class (e.g. Warrior, Rogue, Mage)

Salut! I've finally tested the Usbong app (iOS) on an actual device, and I noticed that the default tree ends after choosing a class (e.g. Warrior, Rogue, Mage). The default tree that I use for the Usbong app (Android) includes other questions, such as "What is your name?", "What is 1+1?". Could it be that the default tree that is used in Usbong app (iOS) is not the updated version? Merci beaucoup!
p.s. I like how the font of the text scales and becomes larger on an ipad.

Yes, this app hasn't been updated in a while so it's still using the old tree.

D'accord! If I want to update the .utree via Xcode, is it a simple as downloading Usbong-iOS, replacing the .utree file, and exporting the output to iTunes? Or would I still need to configure certain things apart from the ones mentioned in the LIKHA-Guide? Merci beaucoup!

As with all apps, you would also have to update the version and build number. The build number is usually incremented every time a build is uploaded to iTunes Connect. This is because if a build with a version and build number already exists in iTunes Connect, a build with the same version and build number can't be uploaded again. Xcode will show this error before uploading.
Also, you would have to make sure you are using the correct provisioning profile and code signing identities.

On Oct 19 2016, at 04:32, Masarap Mabuhay<EMAIL_ADDRESS>wrote:
D'accord! If I want to update the .utree via Xcode, is it a simple as downloading Usbong-iOS, replacing the .utree file, and exporting the output to iTunes? Or would I still need to configure certain things apart from the ones mentioned in the LIKHA-Guide? Merci beaucoup!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/usbong/Usbong-iOS/issues/1#issuecomment-254786946, or mute the thread https://github.com/notifications/unsubscribe-auth/ALtXUjDxMu_q3duu3VYmAuvp0GqKm_SSks5q1f_CgaJpZM4KYluW.

Je comprends. I'll upload my first iOS app using the LIKHA-Guide first, and then see if I run into issues when modifying the .utree of Usbong-iOS then. Merci beaucoup!


Update use-windows-event-forwarding-to-assist-in-intrusion-detection.…

issue #1071
Removed reference to the supported OS, as it makes no sense (WEF could be used for any current Window OS and Windwos Server OS)

@tedhardyMSFT - can you please assist to check, approve and merge the content for this PR based on issue #1071?
Thank you.

Hello @Justinha ,
As far as I understand, this feature works not only for Windows 10 and Windows Server 2016, but also for all OS that supports WinRM, that's why I think that "Applies to" shouldn't be used here.
What do you think?
Thank you

If it does ally to the other versions of Windows then they should also be listed in the applies to section instead of removing the applies to section. This would create a positive affirmation of its relevance.

I concur, much better to confirm in which versions of Windows it actually works, than not saying anything about which versions are relevant.

Dear @egallagh , @illfated ,
Don't you think that "Aplies to" should be for the features that works for some OS, but doesn't work for others?
As this feature works for ALL OS (at least since Windows Vista and Windows Server 2008), should we specify all OS there? What do you think?
Thank you

Well, you sort of have got a point there. Speaking for myself, I would (more often than not) expect from a Microsoft page to state in which OS versions the documented program or software package is supported (or is expected to work). If the versions are not listed, I would expect that the documented software would be available in all versions of Windows (or the software environment in question).
I could very well be mistaken, so I refer to the Microsoft Docs team members for verification and proper feedback.

Dear @Justinha , added Windows Server, as you recommended, could you please have a look?
Thank you

@MaratMussabekov Thanks for the addition!


Support deployment to Kubernetes

Objective
Highlight the integration between OpenTelemetry and the greater cloud-native ecosystem by supporting Kubernetes.
The following are a non-exhaustive list of requirements for this --

Minikube/local k8s as the minimum requirement for installation
Self-deployment of services via ArgoCD
OpenTelemetry deployment via OpenTelemetry Operator
Kubernetes API Server Tracing enabled
Kubernetes Metrics scraped by Collector
Kubernetes resource detectors on Collector (and associated telemetry enhancement)

Done


Question: development.config.php vs development.local.php.dist

Currently to enable development and Whoops, the skeleton installer suggest using two files:

./config/development.config.php.dist -> ./config/development.config.php
./config/autoload/development.local.php.dist -> ./config/autoload/development.local.php.dist

Is there any reason to not use only one ?
Could we merge both in ./config/development.config.php.dist and remove the other ? For example:
use Zend\ConfigAggregator\ConfigAggregator;
use Zend\Expressive\Container;
use Zend\Expressive\Middleware\ErrorResponseGenerator;

return [
    'debug' => true,
    ConfigAggregator::ENABLE_CACHE => false,
    'dependencies' => [
        'invokables' => [
        ],
        'factories'  => [
            ErrorResponseGenerator::class       => Container\WhoopsErrorResponseGeneratorFactory::class,
            'Zend\Expressive\Whoops'            => Container\WhoopsFactory::class,
            'Zend\Expressive\WhoopsPageHandler' => Container\WhoopsPageHandlerFactory::class,
        ],
    ],
    'whoops' => [
        'json_exceptions' => [
            'display'    => true,
            'show_trace' => true,
            'ajax_only'  => true,
        ],
    ],
];

@belgattitude

Is there any reason to not use only one ?

You will find the explanation in ./config/development.config.php.dist:
/**
 * For use with the zf-development-mode tool.
 *
 * Provide your own development-mode settings by editing the file
 * `config/autoload/development.local.php.dist`.
 *
 * Because this file is aggregated last, it simply ensures:

https://github.com/zendframework/zend-expressive-skeleton/blob/master/config/development.config.php.dist#L6

Thanks @froschdesign
Make sense, I'll close it then.


Is this still maintained? The documentation is outdated.

Issue Type (delete the irrelevant ones)

[x] Question

Specs
What version are you using?
v2.19.1
Question
As the subject. I am finding a lot of documentation inconsistencies. One example is the server side rendering section: https://xaksis.github.io/vue-good-table/guide/advanced/remote-workflow.html
The handler methods are not inline with the main Configuration options section (in particular sorting).
Also the Table Options columns section, lists a column type of 'filterable' which doesn't appear to work when specified. Instead Column Filter Options specifies an object which is set on a column.
Overall it's costing a lot of time to figure out precisely what is valid information and what is not.

It's still maintained, documentation is hard, please create PR's when you find inconsistencies or things that are out of date,

Totally agree with @TheJaredWilcurt. @skillshrine vue-good-table is complex and has quite a lot of customization options. Apart from doing our best to maintain it on our own, we depend on folks such as yourself who are benefiting from this library, to help us identify and resolve inconsistencies. Creating PRs is a great way to contribute and help the community.
closing.


Deleting a branch crashes gogs

Hi,
I just played around with gogs. It's already looking great, but I run into a few issues.
I can crash the server by simply trying to delete a branch.
Steps to reproduce (I run gogs in a VM with a custom domain behind an nginx proxy):
$ git clone https://gogs.dev/jer/repo.git
$ cd repo
$ echo "gogs rocks" > README.md ; git add README.md; git commit -m init
$ git push origin master
...
$ git push origin :master
error: RPC failed; result=22, HTTP code = 502
fatal: The remote end hung up unexpectedly
fatal: The remote end hung up unexpectedly
Everything up-to-date

Console output of gogs:
[martini] Started POST /jer/test-repo.git/git-receive-pack
2014/05/02 12:46:34 [FATAL][github.com/gogits/gogs/models] update.go:39: runUpdate GetCommit of newCommitId: Object not found<PHONE_NUMBER>000000000000000000000000000000

this should be handled more gracefully. GitHub for instance rejects deletions if it is the default branch. Non-default branches are handled as expected there.

In case this ever becomes relevant again, this is what drone does to set up the workspace for testing:
+ git init
+ git remote add origin https://github.com/mkaito/dotfiles
+ git fetch --no-tags origin +refs/heads/master:
+ git reset --hard -q 10b978000779084454c93be75d629f6dc14a6930
+ git submodule update --init --recursive


Allow "choose on map" for routing start/end location

It would be nice to have the option of clicking/tapping a spot on the map to enter that as the start (or end) point of the route.
Just wanted to put it out there, because, as other tickets and HN comments have mentioned, the address autocomplete isn't super sophisticated yet, so adding this functionality would make a it just a bit easier to use the routing capabilities in the meantime.
For example, my home address doesn't appear in the autocomplete at all, so I typed in a nearby restaurant which did pop up. With that location set as the destination, having the option to click near my house would be super convenient. For a variety of reasons, I find that I use that functionality on Google Maps quite a lot, actually.
Thanks for this ambitious project! I'm looking forward to following it :)

This is definitely something I want to support! Currently you can drop a pin on desktop only by long-pressing the map while nothing is selected, but as soon as you're in the directions screen you can't drop pins anymore, so it's currently only useful for selecting a destination. It's also not discoverable enough.

Ok, I see the long-press now, I like the use of that event a lot. That same interaction could auto fill the "From" box.

That's what I'm thinking. In the directions screen if exactly one of the boxes is not filled, the long-press should drop a pin and fill it.
Do you think that will be discoverable enough?

That's what I'm thinking. In the directions screen if exactly one of the boxes is not filled, the long-press should drop a pin and fill it.

And if neither are filled, I think it should fill the destination, on any screen.

Do you think that will be discoverable enough?

Why not a simple tap/click instead of a long one? This way should be way more discoverable.


Refactor so the bot can work on multiple servers

Refactor so the bot can work on multiple servers

Done in (a637f5c2cddc26f4c3da305969a81f9e019a03cc).


Fixes #3

Changed NullUnityNativeToastsAdapter to NullUnityNativeSharingAdapter

Thanks for spotting this issue
This PR will fix the error, it doesn't solve the issue
The class name in (NullUnityNativeToastsAdapter.cs)[https://github.com/NicholasSheehan/Unity-Native-Toasts/blob/master/Adapters/NullUnityNativeToastsAdapter.cs] was not renamed while refactoring, the class name needs to be changed to NullUnityNativeToastsAdapter from NullUnityNativeSharingAdapter
I'll have a look at this when I wake up to make sure nothing else has been missed

Been fixed, will be pushed in the v1.0.3 update


Add some more JS keywords

Looks good to me, but could you please fix the merge conflicts?

@LeaVerou, done.


[New] Edit features with feature linked annotation

Description
This PR implements Edit features with feature-linked annotation in Edit and Manage Data category.
URL to README: README
Linked Issue(s)

swift/issues/5377

How To Test

Tap on a point to edit it's address and tap again to move the point.
Tap on a straight polyline and tap again to move it's nearest vertex.
See the annotations update accordingly.

Screenshots

Thank you!


Option to use service account token in mgt api

we should also add correspondent changes in the readme file in the architecture section and installation guide.


Error on iOS 'Attempting to change access mechanism for an unconfigurable property'

Hi,
I got the following exception in a React Native app (iOS) when I added p-memoize.
2018-11-12 11:33:00.365 [fatal][tid:com.facebook.react.ExceptionsManagerQueue] Unhandled JS Exception: Attempting to change access mechanism for an unconfigurable property.

The props that are problematic are arguments, caller, length and name.
I'm using it like this:
import pMemoize from 'p-memoize';

const memoized = pMemoize(async (path, version) => {
 // [...]
});

Not yet sure about the best way to fix this for the React Native env.

This is a Node.js module. I have no intention to add extra code to support IE or React Native, sorry.

The error on IE11 should now be fixed with the new release.
The one on React native might be fixed when using the ignoreNonConfigurable option.


Want to be able to make pre-release copr builds a bit more automatically

Description
We want to make pre-release copr builds. Specifically,  we would like a way to influence packit's automatic rewriting of the spec file so that it updated the Version field to a pre-release value and left the Release field alone. In this way, rpmdev-vercmp would sort the two versions so that the pre-release version preceded the final release version.
Something like:
update_release: false
update_version: true

is what I'm thinking.
Benefit
I think other users would use this if it were available.
Importance
This problem is not blocking us, since we can override the  action for fixng up the specfile, or, in fact do any number of things to get around the fact that this feature is not currently available.
Workaround

[X] There is an existing workaround that can be used until this feature is implemented.

Participation

[ ] I am willing to submit a pull request for this issue. (Packit team is happy to help!)

hi! have you already tried using update_release configuration option? And could you provide please an example of the version Packit provides and the one you would like to have with this?

Hi!
We've been having successes, so we have examples. Take a look at https://copr.fedorainfracloud.org/coprs/packit/stratis-storage-stratis-cli-1022/builds/ .
The top entry has the version 3.6.0~202309080248gitc07c796, note the ~, not -, and that version sorts before 3.6.0 which we are planning to release in the future. It has release D.
The bottom one was produced by a more standard packit configuration and has version 3.6.0 and release  77.20230905212147420030.pr1022.90.g2aa00fe. It sorts after 3.6.0.
We want our numbers to sort before, because we have not released 3.6.0 yet.
This is interesting for people who want to make pre-releases, but the source is already set at the new version.

We obviously have a workaround, and it's fine. So, I think you shouldn't do it for me, this would be more of an ease-of-use thing for adopters.

@mulkieran thanks for the explanation!
Keeping this in the backlog for now, if more people would be interested in this.


Refactor tidy up main blueprint

What is the context of this PR?
Tidying up blueprint

* Removing unnecessary `/` route handling
* Throwing exception for multiple surveys
* Renaming session

How to review
Test should pass and agree fits best practices

It was intended that it was a multi purpose message page, however this has made it into just the 2 surveys open message, I am not sure we want to do that.


Use libc for all ffi declarations

I checked the history of the libc repository, and all omissions or
mistakes these declarations were working around have been fixed as of
libc 0.2.33.

Great work. Thanks! Surely, this was the original intention, but I haven't find time to do it yet.


Make miter limit and arc tolerance configurable for polypath offsetting

Supersedes #29758 (makes API not consistent due to binding limit without including core/method_bind_ext.gen.inc which would result in engine binary size bloat, AFAIK).
Closes #29886.
The first commit breaks compat by combining PolyJoinType and PolyEndType into a single PolyConnection enum. The second commit exposes both miter and arc parameters which was impossible to do due to binding limit of 5 parameters. I can squash these together if the changes are acceptable.
Example
# Produces a thin corridor with round corner passages and dead-ends.
var polyline = [Vector2(0, 0), Vector2(100, 0), Vector2(100, 100), Vector2(0, 100)]
var polygons = Geometry.offset_polyline_2d(polyline, 20.0, Geometry.JOIN_ROUND | Geometry.END_BUTT)

var polygons = Geometry.offset_polyline_2d(polyline, 20.0, Geometry.JOIN_MITER | Geometry.END_ROUND)

Test project
geometry-clipper-test.zip

Closing this for now due to the same reasons as in https://github.com/godotengine/godot/pull/35929#issuecomment-634566288.


[CI][Python] CI failures on Python builds due to pytest_cython

Describe the bug, including details regarding any error messages, version, and platform.
We are seeing sporadic CI failures on Python builds due to pytest_cython:
  File "/opt/conda/envs/arrow/lib/python3.8/site-packages/pytest_cython/plugin.py", line 77, in <module>
    def _without_suffixes(path: str | pathlib.Path) -> pathlib.Path:
TypeError: unsupported operand type(s) for |: 'type' and 'type'

https://github.com/apache/arrow/actions/runs/8526575827/job/23356153535?pr=40949#step:6:5476
Upstream issue has been merged https://github.com/lgpage/pytest-cython/issues/67 but not yet released so we should add an upper pin for pytest-cython to have green CI.
Component(s)
Continuous Integration

Issue resolved by pull request 40975
https://github.com/apache/arrow/pull/40975


The equivalent of "rpm -ql" in powershell

I met a problem when I worked on outlook.
I want to remove all files or settings that associated with a user in outlook. or just uninstall office 2016 ,then install it again.
But I am afraid after I uninstall office 2016 , it is still have some user setting remain in user profile.
If I need to remove office 2016 completely  ,then I need to know all the path to those office installation folders and files and registry key.
In linux , we can use rpm -ql packagename (let's say software package name in windows). to find all files and folders for a specific package.
[root@www ~]# rpm -ql openssh-server
/etc/pam.d/sshd
/etc/ssh/sshd_config
/etc/sysconfig/sshd
/usr/lib/systemd/system/sshd-keygen.service
/usr/lib/systemd/system/sshd.service
/usr/lib/systemd/system/sshd.socket
<EMAIL_ADDRESS>/usr/lib64/fipscheck/sshd.hmac
/usr/libexec/openssh/sftp-server
/usr/sbin/sshd
/usr/sbin/sshd-keygen
/usr/share/man/man5/moduli.5.gz
/usr/share/man/man5/sshd_config.5.gz
/usr/share/man/man8/sftp-server.8.gz
/usr/share/man/man8/sshd.8.gz
/var/empty/sshd

How about windows , will it possible to this kind of cmdlet ?
Waiting for good news.

If Office is installed from msi - the Microsoft Window trusted installer subsystem is tracking all files.
If Office is installed by ClickToRun from Cloud - I don't know how the ClickToRun works, it seems it is not public and not documented.
In any case the question is not related to PowerShell.


fix sample list subscription err reference

Purpose

As seen on the documentation below, the catch condition is defining the variable error while the function to stringify is referencing err:
https://docs.microsoft.com/en-gb/azure/developer/javascript/core/nodejs-sdk-azure-authenticate?tabs=azure-sdk-for-javascript#3-list-azure-subscriptions-with-service-principal

Does this introduce a breaking change?
[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?
[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code and run as stated by the documentation, an error should no longer appear:
Previously:

ReferenceError: err is not defined
    at listSubscriptions (/Users/azure/dev/main.js:58:34)
    at Object.<anonymous> (/Users/azure/dev/main.js:62:1)
    at Module._compile (node:internal/modules/cjs/loader:1101:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:81:12)
    at node:internal/main/run_main_module:17:47

What to Check
Verify that the following are valid

Verify that the documentation also reflects these changes.

Any chance that reviewer can get it merge?


return_sequences=False

Hi,
I'm trying to use your NTM implementation to do sequence classification.  Footnote 1 of your PDF says that return_sequences=False is broken.  But is this still true?  By setting that parameter, I am getting tensors of the right shape, i.e. [batch_size, output_size] with no time dimension.
Performance is not great yet, but I want to make sure that's due to finnicky hyper-parameters and not something still being broken with return_sequences.
Thanks!

Hello!
The PDF refers to the branch https://github.com/flomlo/ntm_keras/tree/SemDL, which is frozen and nowadays several dozens commits behind. The whole PDF was for an assignemt for some seminar in my university and will no longer be updated (I should probably document that somewhere).
As for the return_sequences=False-problem: Yes, I think I fixed that at some point. I can remember faintly being annoyed by it myself some time ago as I wanted to use it myself for sequences.
With the bug still in place the code crashed turing compile time with some unambiguous errors, so if it doesnt, I fixed it.
Performance can be quite a hassle indeed. If you ever use it successfully on some non-trivial problem please tell me about it :)
Best,
oml


Implementation seem to be not quite efficient

After reading the paper, I believe the KAN model should have $O(NL(G + k))$ parameters, which is only slightly larger than an MLP by a factor of (G + k) and should still be manageable in terms of size (in certain specific settings).
However, when I attempted to use model = KAN(width=[8,16,32,16,8,1], grid=15, k=3, seed=1, device=device) with an input batch size of 90, my 24GB VRAM GPU ran out of memory unexpectedly. I mean, yes, the KAN model requires more parameters than just (G + k), but with the above settings, it shouldn't exceed 24GB of VRAM, right? I suspect this might be due to the implementation, but the code is too complex to analyze in a short period of time. Has anyone here found anything related to the implementation causing such high VRAM usage? I would appreciate it if anyone could share their insights.

If you use code similar to this to count the number of parameters you use, it may be a misunderstanding. (Total Trainable Params: 30985)
from kan import *
from prettytable import PrettyTable

device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
print(device)

model = KAN(width=[8,16,32,16,8,1], grid=15, k=3, seed=1, device=device)

def count_parameters(model):
    table = PrettyTable(["Modules", "Parameters"])
    total_params = 0
    for name, parameter in model.named_parameters():
        if not parameter.requires_grad:
            continue
        params = parameter.numel()
        table.add_row([name, params])
        total_params += params
    print(table)
    print(f"Total Trainable Params: {total_params}")
    return total_params
count_parameters(model)

In your case, G + k = 15 + 3 = 18. Suppose you work on MNIST for 90 batch size, and the input data for each batch will be (90, 28, 28) or (90, 784). In the first layer, you will take 890784*18 = 10160640 ~ 10M parameters.


Allow copy functions to follow symlinks

Some package managers (pnpm) use symlinks to be faster and more space conscious. This adds the dereference: true option to all the copySyncs we have.
References
Node's FS documentation on cpSync.
fs-extra's docs

@lariciamota on both places you mentioned the contents are being copied from the .faststore dir (tmpDir). It has no symlinks because it is where we copy everything to.


Warn users to not try to run Spyder from source inside a directory with non-ascii characters

From ccordoba12 on 2011-11-13T10:48:47Z
This is currently failing in Windows and Linux with an ugly traceback. I noticed it while trying to fix issue #812 .
The warn has to go in bootstrap.py and shouldn't be so hard to code. bootstrap has to detect the directory from which is run, then try to decode it in ascii. If it fails, it must give a loud warning to the user, informing he/she what's the problem.
Anatoly, could you take care of this one?
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=834

From<EMAIL_ADDRESS>on 2011-11-13T12:58:02Z
The same problem should appear when Spyder is installed into directory with non-ascii character. I believe all localized Windows versions fall under this category. Maybe it is possible to fix the issue instead?

From pierre.raybaut on 2011-11-14T04:51:54Z
@-techtonik: even on localized versions of Windows, the default installation path will not contain non-ASCII characters. Only the HOME directory -where the settings are stored- will eventually do. So the only situations I can think of will be:

Running Spyder from source within a directory which path contains non-ASCII characters
Installing Spyder using 'python setup.py install' with an installation prefix containing non-ASCII characters
But case 2. won't happen very often...

From ccordoba12 on 2011-11-13T07:54:32Z
Blocking: 812

From<EMAIL_ADDRESS>on 2011-11-14T09:24:04Z
I think it is better to try to trigger specific exception we're getting when running Spyder before it occurs. On my system it is:
{{{
...
File "/home/user09/p/ыва/spyderlib/spyderlib/config.py", line 388, in get_image_path
full_path = osp.join(img_path, name)
File "/usr/lib/python2.7/posixpath.py", line 71, in join
path += '/' + b
UnicodeDecodeError: 'ascii' codec can't decode byte 0xd1 in position 16: ordinal not in range(128)
}}}
But I am able to do this from the same dir:
{{{
import os
os.path.join(os.getcwd(), 'stuff')
}}}

From ccordoba12 on 2011-11-14T04:53:22Z
Anatoly, thing is python packages can't be installed into directories with non-ascii characters. This will be fixed in 3.3 or was in 3.2, I don't remember. So we are safe just adding a warning to bootstrap.

From ccordoba12 on 2011-11-14T13:49:52Z
That's the first error of possibly many. Every time we try to join a file-system path with a unicode string, we'll get the same kind of error.
I don't think it's worth the effort because of what Pierre and I told you. Besides, if a user is on Linux or Mac, the solution is as easy as to move Spyder source to an ascii dir (home dirs are not allowed to have non-ascii characters in Mac and in Linux that's rare). If he/she is on Windows things are more complicated, but Windows users are not usually the ones living on the bleeding edge and wanting to try something from source.
It's your call though.

From<EMAIL_ADDRESS>on 2011-11-15T07:41:14Z
This issue was closed by revision 8b9331d89bf9 .
Status: Fixed

From<EMAIL_ADDRESS>on 2011-11-15T07:55:07Z
This issue was updated by revision b4d45ded7cc4 .
follow-up to the previous commit - it is not the path Spyder is executed
from, but the path Spyder source in located, that should be non-ASCII free

From<EMAIL_ADDRESS>on 2011-11-15T07:56:25Z
I see that Mercurial handles this correctly. Let's hope Spyder will be able to handle non-ASCII paths too eventually.


Image brightness

Image contrasts should be adjusted for the algorithms to work correctly at different daylight levels.

Solved!


NodeJS Distroless Docker image does not exist

Is this a BUG REPORT or FEATURE REQUEST?:
BUG REPORT
What happened:
The kubeless-v1.0.0 YAML files define the nodejs_distroless docker image as
henrike42/kubeless/runtimes/nodejs/distroless:0.0.2

This image does not exist. The image exists at henrike42/kubeless.runtimes.nodejs.distroless:0.0.2
https://hub.docker.com/r/henrike42/kubeless.runtimes.nodejs.distroless/tags/
When docker attempts to pull this image the following error is returned:
$ docker pull henrike42/kubeless/runtimes/nodejs/distroless:0.0.2
Error response from daemon: pull access denied for henrike42/kubeless/runtimes/nodejs/distroless, repository does not exist or may require 'docker login'

What you expected to happen:
The correct image should be defined in the YAML files. Docker should be able to pull the image if this runtime is requested to be executed.
How to reproduce it (as minimally and precisely as possible):
docker pull henrike42/kubeless/runtimes/nodejs/distroless:0.0.2

Anything else we need to know?:
Nope
Environment:

Kubernetes version (use kubectl version): N/A
Kubeless version (use kubeless version): N/A
Cloud provider or physical cluster: N/A

Hi @TR-jboykin,
You are right, sorry for the inconveniences. That image is now hosted as kubeless/nodejs-distroless@sha256:1fa0469c5520f4e08d89b1fafd2cacf03f098b96ea04997fa52bb9ef2a180fb3. You can modify the Kubeless configuration to use that image instead of the current one.
I will do a release next week with the update of the different runtimes.

I have released kubeless 1.0.1 that should fix this issue. Let us know if you find a different issue!


Fix Jit Pack classpath and publish on maven central

Error Log Reference
Seems to be related to an old Java Class Path which will not compile with most recent ones.

[x] Fix JitPack build.
[x] Add Maven Central.

Error caused by:
[ERROR] Failed to execute goal com.spotify.fmt:fmt-maven-plugin:2.21.1:format (default) on project at4j:
Execution default of goal com.spotify.fmt:fmt-maven-plugin:2.21.1:format failed:
Unable to load the mojo 'format' in the plugin 'com.spotify.fmt:fmt-maven-plugin:2.21.1' due to an API incompatibility:
org.codehaus.plexus.component.repository.exception.ComponentLookupException: com/spotify/fmt/FMT has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 54.0

Both fixed!


Accept rtsp client without interleaved parameter

Fix #1510

Your changes are incorrect. The client can come for UDP transport.

The existing code already expects TCP-only transport, so there is no regression with this PR?

But it return 461 Unsupported transport error, so smart client can change UDP to TCP request. Many clients support this out of box.

Got the point. Should be fixed now.

It's wrong again. Some clients don't send unicast word. I think only first prefix should be enough.

Fixed.
By the way, there was a typo between tr and transport variables, fixed!
I'll build & deploy on my side.


Fix build

Somehow the build was broken. I just put a parenthesis back and it worked.
Can someone check it just to verify?

Thank you!   I did not see this before I landed a similar patch, but thank you!


[silx view] Fixed support of NXData image with 0-length axis

This PR updates the calibration to raise exceptions for empty arrays and it takes it into account for NXData views in silx view.
It also moves the signal connections before updating the plot so they are registered even if an exception occurs later, just in case.
closes #3766

CI is broken due to issues with cython3.0.0b1. PR #3770 should fix it.

Tests added and rebased to include PR #3770.


[shape_poly] Add support for jnp.cum{sum,prod,max,min} with shape polymorphism

Unfortunately, on CPU and GPU where we use associative scan, we cannot support shape polymorphism with native lowering, when the axis over which we reduce is non-constant size. Improve the error in that case.

@rxsang @maxwillzq PTAL


Add support to ignore "Schema circular referecnce changed"

Is your feature request related to a problem? Please describe.

Describe the solution you'd like
Add a cli option to ignore "Schema circular referecnce changed" differences

Hi @JSSAggie,
In order to address your request, I need some more info.
First, could you please explain why you want to ignore this change, it is erroneous?
Could you also please provide a pair of specs and the command-line to replicate this behavior?
Thanks,
Reuven


Add bootsrapper for mainnet by Qubelabs

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

Project coverage is 45.42%. Comparing base (2469e7a) to head (bdc41e1).
Report is 194 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3760      +/-   ##
==========================================
+ Coverage   44.83%   45.42%   +0.58%
==========================================
  Files         265      281      +16
  Lines       14620    16059    +1439
==========================================
+ Hits         6555     7294     +739
- Misses       7313     7916     +603
- Partials      752      849      +97

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


fix: show sign if bucket is empty

Delete code instead of commenting - bit hard to tell what exactly changed here.  Also I'm pretty sure we did key count < 1 on purpose because it === 1 when a subfolder is empty

Yeah, if there exists a subfolder then the bucket is not empty right? Previous logic returns isEmpty to true when key count is 1.


Alexa won't respond

It won't respond, i think is because of the mic I'm using (it's the ps3 camera), hope someone can help me. Thanks!
pi@raspberrypi:~ $ sudo systemctl status AlexaPi.service
● AlexaPi.service - Alexa client for all your devices
Loaded: loaded (/usr/lib/systemd/system/AlexaPi.service; enabled)
Active: failed (Result: exit-code) since Sun 2017-05-21 18:50:26 UTC; 23s ago
Docs: https://github.com/alexa-pi/AlexaPi/wiki
Process: 1382 ExecStart=/usr/bin/python /opt/AlexaPi/src/main.py --daemon (code=exited, status=1/FAILURE)
Main PID: 1382 (code=exited, status=1/FAILURE)
May 21 18:50:26 raspberrypi python[1382]: surround50:CARD=CameraB409241,DEV=0
May 21 18:50:26 raspberrypi python[1382]: surround51:CARD=CameraB409241,DEV=0
May 21 18:50:26 raspberrypi python[1382]: surround71:CARD=CameraB409241,DEV=0
May 21 18:50:26 raspberrypi python[1382]: iec958:CARD=CameraB409241,DEV=0
May 21 18:50:26 raspberrypi python[1382]: dmix:CARD=CameraB409241,DEV=0
May 21 18:50:26 raspberrypi python[1382]: dsnoop:CARD=CameraB409241,DEV=0
May 21 18:50:26 raspberrypi python[1382]: hw:CARD=CameraB409241,DEV=0
May 21 18:50:26 raspberrypi python[1382]: plughw:CARD=CameraB409241,DEV=0
May 21 18:50:26 raspberrypi systemd[1]: AlexaPi.service: main process exited, code=exited, stat...LURE
May 21 18:50:26 raspberrypi systemd[1]: Unit AlexaPi.service entered failed state.
Hint: Some lines were ellipsized, use -l to show in full.

Hello,
You have to find your device's PCM with:
arecord -L
You must find theses lines:
plughw:CARD=[YOUR CAMERA NAME],DEV=0
    Webcam [YOUR CAMERA NAME], USB Audio (or something like that)
    Hardware device with all software conversions

Edit the AlexaPi config file:
sudo nano /etc/opt/AlexaPi/config.yaml
and change this line:
  input_device: "default"
with:
  input_device: "plughw:CARD=[YOUR CAMERA NAME],DEV=0"
And finally reboot your PI or restart AlexaPi daemon
I don't know why theses steps are not on the Installation wiki page...
Emerick

I have PS3 eye too and initially had the same problem. It turned out that the record was bad, it was just random noise. I fixed by modify one line in src/alexapi/capture.py.
			# now do VAD
			while (force_record and force_record[0]()) \
					or (do_VAD and (thresholdSilenceMet is False) and ((time.time() - start) < self.MAX_RECORDING_LENGTH)):
				length, data = inp.read()
				if length > 0:  # changed line here, ignore overrun frame
					yield data

Explanation: usually one period in a record is overrun, which returns negative length and one-byte buffer. Original code writes that one byte to audio stream and destroy all the subsequent good frames. The solution is to skip that bad period. Luckily it happens only once in a long series, so the overall audio isn't significantly affected.

@JesusGarc See the full log & docs: https://github.com/alexa-pi/AlexaPi/wiki
You need to set your input device name properly.

@JesusGarc have you solved your issue?

Closing due to inactivity. Feel free to reopen.


Windows client auth

The Windows client will need to handle user authentication to the portal within a secure webview and then save the resulting token to a persistent, secure location for establishing the control plane websocket.
One solution could be to use UWP's WebAuthenticationBroker
The URL to use for authentication should be AUTH_BASE_URL/ACCOUNT_ID?client_platform=windows where AUTH_BASE_URL and ACCOUNT_ID are from settings.
After authentication is successful, the portal will redirect the browser to AUTH_BASE_URL/handle_client_auth_callback?token=<TOKEN> which the client will need to intercept in order to store the token and connect the WebSocket.
Note that the token returned from the portal is an opaque token. The client should not try to introspect it to determine lifetime or other parameters. Token validity is determined by connlib -- if the Session connect call succeeds, the token is valid. Otherwise, the token is invalid and should be discarded, moving the app to the signed out state.
If a token expires while the client is connected, the portal will disconnect the websocket, causing connlib to call onDisconnect, at which point the client should discard the token and move to the signed out state.

I'm not sure what my account ID is, so I'll ask about that some time.

We use firezone for internal testing

That's the URL, right? Looking at Session::connect, I'm not sure where the get the device ID and token. When I open that URL and put in my firezone email, I don't get any email.

@ReactorScram Sign in with Google workspace (not magic link)
The token is returned from the portal in a final redirect to AUTH_BASE_URL/handle_client_auth_callback?token=<token>

See this PR for inspiration for retrieving the SMbios:
https://github.com/firezone/firezone/pull/2477/files#diff-9d5ec593addfe28407db74bd8b8b4b9a918258ddf4da6e92dff0ba67dcfee6c8L76

I'm seeing people having trouble with the UWP API, saying maybe it doesn't work on desktop, or it needs the app to be UWP.

https://github.com/microsoft/WindowsAppSDK/issues/441
https://github.com/microsoft/WindowsAppSDK/issues/398

Some people mentioned spinning up a temporary localhost web server and making the OAuth flow redirect through that, so the web browser can get the end URI and token back to the app. But I'm not sure if this is compatible with Firezone's particular OAuth setup. I'd have to somehow tell the server "Don't tell me the URI, redirect the user to my localhost callback URI"
And it would show up in the default browser, it sounds like phones and Apple do some system-controlled webview for security.
If that could work, there is a crate for it - https://docs.rs/yup-oauth2/8.3.0/yup_oauth2/#installed-flow-usage

We discussed on the call today that app-links / deep-links / custom uri scheme protocol / whatever it's called might be usable instead of a local web server.
The auth endpoint doesn't allow me to request a custom redirect / callback URI, so I couldn't use an ephemeral port, and picking a designated port is a little unappealing. So I'll try app links first. I think the only catch is that I need to register the exe with Windows, but it doesn't look complicated.

App links work in that commit. firezone:// is registered whenever the app runs, and we could have the installer register it if we need.
So the local web server is not needed. Unfortunately there isn't a secure webview yet, it just opens in the default browser, on my system it's Firefox.

I think both problems would be more bullet-proof if the client could attach an in-memory single-use public key to these requests, then the server would encrypt them so only the client could read them. But that is a gigantic change.

It's not too bad actually from the portal's perspective. This might be the winner here. We're discussing how this'd work in the other clients. In the meantime, maybe just keep the custom URI scheme you have here for now while we wait on the portal changes?

By the way, is there a good place to document things like "We considered URI schemes and rejected it for this reason"? Like a decision journal? I'll just stick it in code comments for now.

Good question, we started a MADR document here but it didn't seem to stick. Welcome to add more records there if useful.

Sure. I was already looking into the local webserver so I'll just wrap that up and leave it un-called in the code somewhere.

Awesome.
Just a note on retrieval via SMbios -- it looks like that may not be as reliable as we thought after chatting with @conectado (and introduces another dependency).
What if we just generate a random UUID on first launch and persist it with the store plugin? Key value is how we store settings in the other clients (i.e. no need for SQL)
Then we could even use the stronghold plugin for storing the token.

@jamilbk Is that SHA256 hash intended for any obscurity or security or just to normalize the length?
I've heard that SHA256 is vulnerable to length extension attacks, so personally I've switched to other hashes - https://en.wikipedia.org/wiki/SHA-2#Comparison_of_SHA_functions

@ReactorScram Great question -- the sha256 is for normalization and obscurity. We don't want to store a raw device ID in case the client sends something private for example.
Noted on the length extension attack, but for our purposes the security of SHA256 is fine since we don't use it for authentication or authorization, only client metadata.

Closing, we didn't get the secure webview and we're waiting for the new auth flow to be complete but in general the Windows client does auth with the portal.


Return empty string to render instead of null or undefined when using block loader

Prevents null from showing when the most popular block is not returned in the case like on content.nrwa.org

 Changing our ad manager - new code for content portal


Thai rendering broken

Open http://www.cl.cam.ac.uk/~mgk25/ucs/examples/UTF-8-demo.txt
and look at lines 123-130, compare with nano or vi.
Version: 1.3.4-dev.38
Commit hash: e99fd13
Compiled on October 08, 2017

Ubuntu 16.04.3
Sakura, MATE Terminal, ROXTerm -- doesn't matter here.

Go is supposed to be UTF-8 ready, right? Why would this bug be the case??

cat UTF-8-demo.txt:

Display in micro:

I'm using MacOS, so it looks like it's not platform specific.

Micro is an excellent editor, but this is a blocking issue for adoption. There are many terminal based editors out there that do this well. This is a main area for general improvement. It is probably not a huge thing, just some oversight in the handling of utf-8 that doesn't get caught with other types of fonts.

All the other sections of  http://www.cl.cam.ac.uk/~mgk25/ucs/examples/UTF-8-demo.txt seem to be handled well, there is just a bug in the Thai rendering. Would be a great jump in usability and being bug-free when fixed!

With the new support for unicode combining characters this text now displays correctly.

Improvement! Tried this with the latest Nightly build:  Version: 2.0.4-dev.19 Commit hash: d0f7ecf Compiled on May 16, 2020
When you move with the cursor over lines 123-130 you see that the rightmost paragraph is not left-alinging like it should, and the characters shift while the cursor passes the line.


Exercise information on Tutor Exercise Dashboard is not displayed correctly

Describe the bug
When assessing submissions as an instructor, the number of "submissions assessed by you" on the tutor course dashboard is not displayed correctly. It always shows 0, even after submissions have been assessed. This happens with accounts that only have the instructor group, but it does not happen with accounts that have the instructor group as well as the tutor group.
To Reproduce
Prerequisites:

You need one Instructor account
You need a submission for an exercise that you can assess

Steps:

Go to the Tutor Course Dashboard
Locate your exercise and open the tutor exercise dashboard
Assess your prepared submission
Go back to the tutor exercise dashboard
"Submissions assessed by you" is still at 0

Expected behavior
The correct amount of assessed submissions must be displayed next to "Submissions assessed by you" in the tutor exercise dashboard.
Screenshots

Additional context
This issue is related to this PR and this comment

This view does not exist anymore, therefore this issue can be closed.

outdated


Update concepts-business-continuity.md

Changing hyperlink for flexible server https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/postgresql/flexible-server/concepts-high-availability.md

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

Looks good; verified the links.


Is this better than video retalking?

Is this better than video retalking? I'm using video retalking here but the output lacks quality.
Another question, do you have the project on Google collab?

Try it and tell me what you think.
Try it easily at:
https://replicate.com/xiankgx/lipsick

@xiankgx  looks like the replicate has not input fields.


Divide foxai explainer into deterministic and non-deterministic

Some explainers, like CV_INPUT_X_GRADIENT_EXPLAINER are deterministic, and some like CV_DEEPLIFT_EXPLAINER, CV_GRADIENT_SHAP_EXPLAINER are not deterministic. It would be nice to divide explainers into two subsets in the readme file. Some projects require to have deterministic results and it would be nice to have a look-up table.
Probably all of those methods are deterministic if the same seed is chosen, but I'm not sure if there is such an option right now and anyway it would be still good to note it down somewhere.

Some of them might behave non-deterministic based on provided arguments, e.g. Integrated Gradients passing every time a new random baseline matrix, but you probably wanted to classify the algorithm itself.


Adaptive Card in Microsoft teams getting trimmed from b

Screenshots

Version
Bot framework : v3
Adaptive Cards : 1.0
Describe the bug
We have a customer who needs the user profile cards to be displayed like the attached. At the bottom of the card should be two icons :

Chat - Should take the user to the teams chat of the user being reported in the user profile card
Email - Should open default email client with user being reported in the user profile card in To.

Now when I send this card, the two icons have been rendered inside a Column Set which in-turn is placed inside a container. The card when received in web-chat, works fine, but in Teams, the icons are trimmed as shown in the screenshot. I have to scroll up and down a bit, or minimize and restore teams window to get the cards to render in full (without trimming the icons)
To Reproduce
Steps to reproduce the behavior:

Create a adaptive card
Create three containers for body
Add something to the first two containers
Add a column set to the third container and add three columns
Add Images to the first two columns and leave the third empty
Send the card to teams client

Expected behavior
I would expect the card to be shown fully without trimming the bottom.
Additional Information
I think, because the icons are images fetched from a URL, it takes time for the teams client to load the images, hence the card gets trimmed. This should not happen however, if the images are load async, the card should auto-resize and the user should be able to see the full card with icons. like the following

Thanks,
Yogesh

Incorrect Forum. Closing the issue.


feat: use release namespace throughout chart

This PR bundles several housekeeping tasks for the chart quickstart.
The most impactful change is 256e1a4b99b8730b3b57870373dea175dc3168f5, which swaps a hard-coded ochami namespace for the release namespace.
Most resources were already using namespace: {{ .Release.Namespace }}, and several resources had dependencies on the resources previously always in ochami. I don't think we'd want a static namespace for anything unless we had resources we wanted to share between releases.

Other changes are creating a directory for scripts, fixing a broken dependency (dunno what happened, but it looks like the upstream maintainer no longer publishes an official image), and ignoring downloaded chart artifacts.

Where have you tested these changes?  Since the rest of us don't generally use GKE, we can't readily verify that they work.  Can you add testing instructions and outcomes?

I don't have access to any GKE instances either :D KinD provides a local miniature cluster suitable for most development/test work.
helm-test.txt shows the current image failures and misplaced configmaps, followed by a cluster rebuild and install with the PR branch.
We should be able to add CI to the repo that checks, if nothing else, a basic confirmation that the chart installs and has resources in a happy state, but the current need for out of band Postgres management makes that a bit difficult.
FWIW I cherrypicked these commits out of https://github.com/rainest/ochami-deployment-recipes/tree/rainest/oneshot, which can manage Postgres automatically, but significantly changes how we'd manage Postgres, so I was deferring that part til I could discuss it with Brian.


[WIP]cloudlfare adapter wip

Description
Work in progress for cloudflare adapter.

Just want to say, really keen for this. I've just discovered remix and this repository. All my stuff is completely built on Cloudflare pages, can't wait to use these devtools :)

@AdiRishi i am so happy to hear that! Well maybe it isn't clear but you CAN use the CLIENT part fwith cloudflare just fine, it's the server part you can't. I am working on an adapter that will bring as much functionality as possible to the server side part.

Hey @AlemTuzlak! Thanks for the awesome work here. Would this PR help to allow Hydrogen to use the dev tools?
Right now, I seem to be getting a lot of errors trying to run this with the custom server directions, but also possible that I'm not setting it up correctly.
Hydrogen uses Mini-Oxygen for dev environment which is base off of cloudflare, I think.
Here are some of the errors, I'm getting trying to put dev tools in Hydrodens' server.ts file:

Errors
 [ERROR] Could not resolve "http"

    ../node_modules/ws/lib/websocket.js:7:21:
      7 │ const http = require('http');
        ╵                      ~~~~~~

  The package "http" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

✘ [ERROR] Could not resolve "net"

    ../node_modules/ws/lib/websocket.js:8:20:
      8 │ const net = require('net');
        ╵                     ~~~~~

  The package "net" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

✘ [ERROR] Could not resolve "tls"

    ../node_modules/ws/lib/websocket.js:9:20:
      9 │ const tls = require('tls');
        ╵                     ~~~~~

  The package "tls" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

✘ [ERROR] Could not resolve "crypto"

    ../node_modules/ws/lib/websocket.js:10:44:
      10 │ const { randomBytes, createHash } = require('crypto');
         ╵                                             ~~~~~~~~

  The package "crypto" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

✘ [ERROR] Could not resolve "stream"

    ../node_modules/ws/lib/websocket.js:11:37:
      11 │ const { Duplex, Readable } = require('stream');
         │                                      ~~~~~~~~
         ╵                                      "./stream"

  The package "stream" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

✘ [ERROR] Could not resolve "url"

    ../node_modules/ws/lib/websocket.js:12:24:
      12 │ const { URL } = require('url');
         ╵                         ~~~~~

  The package "url" wasn't found on the file system but is built into node. Are you trying to bundle for node? You can use "platform: 'node'" to do that, which will remove this error.

But Hydrogen dev server requires the serverPlatform to be set to  'neutral'.

@DanPete hey Dan! Thank you for reaching out, I have actually run a lot of testing here with cloudflare and running them in a wrangler env with node but there is a lot at play here that stops this from happening. I believe once the remix team releases cloudflare support with Vite the dev tools will automatically work there without any issues. I don't know if that would help with Hydrogen support?
I have put this on hold to see what happens when Vite comes into the picture and they deprecate the old compiler but would love to hear if that would still cause issues for hydrogen, although it's important to note that the client side can run anywhere

@AlemTuzlak Thanks for the hint about client side being able to run anywhere! That's exactly what I was doing wrong. 🤦
I was thinking I had to follow the Custom Server Setup to get it working properly.
But once I re-read the README, I got it working correctly with Hydrogen! 🙂

@DanPete that is awesome to hear! no worries, I plan on making a video and a docs site pretty soon to avoid these issues, the github readme is not really the best place for all of this info

Remix Dev tools work with cloudflare on Vite, closing this branch!


useHoverDirty warning should be in effect, not hook body

What is the current behavior?
useHoverDirty always console.warns on initial render(s) when refs have yet to be assigned. this is pretty loud, especially given that this is expected behavior for any component using useHoverDirty.
the correct place to include this warning would be in the effect itself, when ref is actually expected to be defined.
https://github.com/streamich/react-use/blob/master/src/useHoverDirty.ts#L6

yes, pretty annoying warning that could be removed entirely


Use modern META filename

The META.info is a legacy, pre-Christmas name. While it's still currently supported, META6.json is the new name. And since a lot of people simply copy some module's  structure, the old name still proliferates, so we're trying to get rid of it for good  by sending PRs to any modules that use the old name, to switch to the modern name.

Thanks for this btw, although this module is mostly a relic now and it might be worth removing from the.. 6pan.


[Felix] Why each model's train loss will rise  in the last epoch?

When I use felix model training Discofuse dataset, the model's train loss will rise in the last epoch no matter which size of epoch I chosen.  My training parameters : num_training_examples = 450,000 ,num_eval_examples = 20,000, batch_size = 32, max_seq_length=128, lr = 0.00003

Okay that's good ej483$83×¢=¢emd

Hi @BlackBearBiscuit  I've never noticed anything like this before. Could you maybe post your logs/tensorboard? Also you say you set the num of examples to 450,000, how many examples does the dataset actually contain(are you using all of DF)?

Dkz48-*


Allow unstable dependencies in Kotlin DSL buildscripts

Fix for https://github.com/gradle/gradle/issues/25460

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge


Swagger Doc - Ignore Controller

Is your feature request related to a problem? Please describe.
I have a controller that I would like to hide from the documentation. I have been unable to find a solution in the documentation. It would also be useful to hide individual routes as well.
Describe the solution you'd like
Controller - Add a comment at the top of the file to ignore.
Route - // @Tags Hide
Describe alternatives you've considered
Only way it appears I can hide it is to remove my existing controller.

I suggest you put  your individual routes away from your searching dir.

@grtn316
I have the same requirement #661

Let me guess: Security by obscurity !
Creating hidden routes doesn't make the software more secure. On the other hand, you shouldn't add comments on the controller you don't want to make visible.

Let me guess: Security by obscurity !
Creating hidden routes doesn't make the software more secure. On the other hand, you shouldn't add comments on the controller you don't want to make visible.

Actually, I'd just like APIs only used by the system to be aware of them by hiding them in my public swagger docs. Even if they are "secured", I don't desire bots "testing" my secure endpoints they found in my public swagger docs. It takes resources from my server and is unnecessary when hiding them in the documentation will quickly cut down a lot of that traffic.
I should not have to build another app to host system only APIs when I can hide them with decorations in other programing languages.


Fix for 1291

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.
Closes #1291.

Sure, I will write the unit tests, make the appropriate changes you mentioned, and will let you know @FyzHsn

Hi @FyzHsn , Above mentioned changes were made and wrote the unit test. can you review it.

Hi @FyzHsn , Above mentioned changes were made and wrote the unit test. can you review it.

Hi @RaviPsiog, the diff coverage is still at 83% and needs to be 95% before this PR can be merged. You just need to cover line 330 in electron.py in your tests. This PR should be good to be merged as soon as the diff coverage meets the minimum requirement

Hi @FyzHsn , The unit test that I wrote is supposed to cover line 330.  Then I saw that the line is not covered as we are using the "is" operator, which checks the reference. If i am not wrong, checking if they are identical is enough in this case. And hence i changed it to == operator. Now, The coverage should meet the requirements and also it's working as it's supposed.


Trap Ctrl C

Trap Ctrl C to skip current step

Great idea and how about a pause menu, so i can switch vpn ip's then either resume current scan or skip to the next scan.

It's a good option too, but also remember that when you cancel a scan and rerun the same target, the tool skip the steps already performed ;)

not working


Wrong URL for release notes in Help menu

API Services Portal Issue
User Story
As a developer,
I want to read the latest release notes
so that I can understand how a release will impact me.
Test Case
ENV

[ ] DEV
[x] TEST
[x] PROD

TESTCASE

Browse to api.gov.bc.ca
Open the 'Help' dropdown menu in the header nav
Click 'Release notes'

EXPECTED

Browser should navigate to https://bcgov.github.io/aps-infra-platform/releases/2023-mar-1.2.11/

ACTUAL

Browser navigates to https://bcgov.github.io/aps-infra-platform/releases/2022-may/

Since we probably don't want to keep updating the link in the Portal, perhaps we just have it got here:
https://bcgov.github.io/aps-infra-platform/releases/
@elisafw-ux

Yes please!

@Jonesy A miscellaneous UI fix when you have a minute :)


[BUG] Refactor and merge details and config tabs for the host

For the host, currently there are two tabs - refactor and explore if we can club them together in a single tab.

Fixed