Table are not scrollable

how can i fix the table to make it a scrolllable one because it is being cropped otherwise

Thanks for reporting this issue, I will fix it

Fixed, please update to markdown_viewer: ^0.6.0

Thank you very much! This fixes it!

Don't know if this created the issue but, tables are not 100% width when they don't scroll.
Tables should occupy the whole width area, then scroll. Such as this:

Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column

Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column

Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column

Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column

Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column

Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column

Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column

Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column
Very large column

If this is too hard to implement, maybe there should be an extra setting for us to tell the render if we want or not a scrolling table.

@JCKodel maybe we should have an extra setting to force the tables to occupy the whole width, what do you think?

I think: if you want a table, then build a Table widget =)
Markdown should be simple, and the scroll should be the exception.

The table is a Table widget:
https://github.com/tagnote-app/markdown_viewer/blob/master/lib/src/builders/table_bilder.dart#L104

The table is a Table widget:
https://github.com/tagnote-app/markdown_viewer/blob/master/lib/src/builders/table_bilder.dart#L104

I know =)
What I meant was: if one needs a real Table with scroll and everything, they should go for a Table widget, not trying to make it on markdown =) Because it's so easy to do a Column > Markdown > Table > Markdown.
I'll probably use this widget a lot in some projects I'm working. If this is the case, I'll try to help with PR instead of just issues.


feat(plugins/plugin-kubectl): allow for pinning a Kubernetes Context or Namespace to a tab

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[ ] 🐛 Bug fix
[ ] 💅 Enhancement
[x] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

nice! in the future, we might want to think about the design of the context widgets to better explain that something changed when you switch tabs.
/lgtm


Add Wireless Transfer App.app Latest

Using version :latest Also means sha256 :no_check. See an example. Else, the cask would break with each update.


resource_already_exists_exception

Hi there,
Today I have faced next issue with updated image:

ElasticDump version: latest on 18 June (6.0.2)
Elasticsearch version: 6.4.3
Full command you are having issue with (with inputs and outputs)

docker run --rm --network host registry:5051/taskrabbit/elasticsearch-dump:latest elasticdump --input=http://firsthost:9200/first-index --output=http://secondhost:9200/second-index --type=settings
[14:16:10]	Tue, 18 Jun 2019 11:16:10 GMT | starting dump
[14:16:10]	Tue, 18 Jun 2019 11:16:10 GMT | got 1 objects from source elasticsearch (offset: 0)
[14:16:10]	Tue, 18 Jun 2019 11:16:10 GMT | Total Writes: 0
[14:16:10]	Tue, 18 Jun 2019 11:16:10 GMT | dump ended with error (set phase)  => Error: {"error":{"root_cause":[{"type":"resource_already_exists_exception","reason":"index [first-index/yad_fM7FTGuHg1JS5sfyqQ] already exists","index_uuid":"yad_fM7FTGuHg1JS5sfyqQ","index":"first-index"}],"type":"resource_already_exists_exception","reason":"index [first-index/yad_fM7FTGuHg1JS5sfyqQ] already exists","index_uuid":"yad_fM7FTGuHg1JS5sfyqQ","index":"first-index"},"status":400}
[14:16:11]	Tue, 18 Jun 2019 11:16:10 GMT | Error Emitted => {"error":{"root_cause":[{"type":"resource_already_exists_exception","reason":"index [first-index/yad_fM7FTGuHg1JS5sfyqQ] already exists","index_uuid":"yad_fM7FTGuHg1JS5sfyqQ","index":"first-index"}],"type":"resource_already_exists_exception","reason":"index [first-index/yad_fM7FTGuHg1JS5sfyqQ] already exists","index_uuid":"yad_fM7FTGuHg1JS5sfyqQ","index":"first-index"},"status":400}

ElasticDump 4.3.0 - works.

That's because 4.3.0 would silently fail without returning an error, in 6.0.2 and above we added error log output.

We found out that if we want to dump all 6 types [settings, analyzer, data, mapping, alias, template] with elasticdump command (one by one so total 6 commands) from one ES to another, there will always be resource_already_exists_exception for at least 3 of them; We can run --type settings first but for example, then if you run --type analyzer or mapping we will get such exception

That's because analyzers & settings return the same result. mappings are extracted from settings. All these 3 options must be created at index creation time. It's best to just dump settings or mappings, but never all 3.

Thanks @ferronrsmith ; So to fully migrate one index from one ES to another, dumping settings+data would be enough?

Yes. The other options are just there to provide "different" options for different situations. They aren't meant to be all used at once 😄

Hey @ferronrsmith ,
Sorry for being a noob but i'm running into the same issue when attempting to restore elastic indices and data to our non-prod environment.  We are running both analyzers, mappings, and data.
What combinations of files can/should i use in my scenario?

analyzers = settings
mappings is fine
data is fine.


[BUG] Tracing: support non-string types and fix http.status_code

Library name and version
Azure.Core
Describe the bug
http.status_code has to be int according to hotel and  Azure SDK conventions.
Changing it for DiagnosticSource path would be breaking, but for OTel path, we should be able to change it.
To  do it we'd need to change
https://github.com/Azure/azure-sdk-for-net/blob/f4b6edc0de76a5fda06644c2ac495941d5b070a0/sdk/core/Azure.Core/src/Pipeline/Internal/RequestActivityPolicy.cs#L83
to pass status as is and  support adding (at least) long attributes in DiagnosticScope
https://github.com/Azure/azure-sdk-for-net/blob/f4b6edc0de76a5fda06644c2ac495941d5b070a0/sdk/core/Azure.Core/src/Shared/DiagnosticScope.cs#L88-L95
Activity.AddTag(String, Object) was added in .NET 5 so we need to do our regular reflection hack.
Expected behavior
http.status_code has long or int type
Actual behavior
http.status_code is a string which breaks azure SDK integration with otel-compatible backends
Reproduction Steps

create HttpPipeline, send request to any endpoint
enable Opentelemetry
export traces to AzMon, Jaeger or console

Environment
No response

Fixed in https://github.com/Azure/azure-sdk-for-net/pull/33960


jsx-curly-brace-presence and jsx-no-comment-textnodes conflict with each other

I have both react/jsx-curly-brace-presence and react/jsx-no-comment-textnodes enabled. I want to render the string /* as the child of a component:
const Component2 = () => {
  return /*;
};

This errors for react/jsx-no-comment-textnodes as expected:
/sandbox/src/Component.jsx
  8:16  error  Comments inside children section of tag should be placed inside braces  react/jsx-no-comment-textnodes

Per the docs for react/jsx-no-comment-textnodes, I should be able to use curly braces to fix this:
const Component2 = () => {
  return {'/*'};
};

However, this now errors for react/jsx-curly-brace-presence:
/sandbox/src/Component.jsx
  4:16  error  Curly braces are unnecessary here                 react/jsx-curly-brace-presence

Playing whack-a-mole with linter errors won't get me anywhere here. I could always disable one of the rules for this file, but ideally react/jsx-curly-brace-presence could be smart enough to know that this is something that needs to be in curly braces because of other rules in play and not report this particular code as an error.
Codesandbox (fork and run yarn lint from the terminal): https://codesandbox.io/s/cocky-darwin-1zfs5?file=/src/Component.jsx

Thanks for the repro!
I'm having trouble reproducing this - the test cases pass (the curly-brace-presence warning doesn't appear), both in my local tests and in your codesandbox.
Are you sure you're using the latest version of eslint-plugin-react locally?

This is what I see when I run yarn lint in the sandbox:
sandbox@sse-sandbox-1zfs5:/sandbox$ yarn lint
yarn run v1.22.4
warning package.json: No license field
$ eslint --ext js --ext jsx src
Warning: React version not specified in eslint-plugin-react settings. See https://github.com/yannickcr/eslint-plugin-react#configuration .

/sandbox/src/Component.jsx
  4:16  error  Curly braces are unnecessary here      react/jsx-curly-brace-presence
  8:16  error  Comments inside children section of tag should be placed inside braces  react/jsx-no-comment-textnodes

✖ 2 problems (2 errors, 0 warnings)
  1 error and 0 warnings potentially fixable with the `--fix` option.

Can you show me what output you get for yarn lint? I also realized I wasn't explicit about how the rules were configured in my original post. Here's a verbatim copy of what's in the codesandbox:
{
  "extends": ["eslint:recommended", "plugin:react/recommended"],
  "parserOptions": { "ecmaVersion": 6, "sourceType": "module" },
  "rules": {
    "react/jsx-curly-brace-presence": [
      "error",
      { "props": "never", "children": "never" }
    ],
    "react/jsx-no-comment-textnodes": "error"
  }
}

Is there a way I can just clone the sandbox directly?
The passing tests I landed in bb2b8db use the same eslint config as in the sandbox.

There doesn't seem to be a way to clone a sandbox directly. Here's a new repo that contains the same test case, along with instructions on how to reproduce: https://github.com/nwalters512/eslint-plugin-react-conflicting-rules-repro
Following my own instructions, I see the following output for yarn lint:
yarn run v1.22.4
$ eslint --ext js --ext jsx src
Warning: React version not specified in eslint-plugin-react settings. See https://github.com/yannickcr/eslint-plugin-react#configuration .

/Users/nwalters/git/eslint-plugin-react-conflicting-rules-repro/src/Component.jsx
  4:16  error  Curly braces are unnecessary here                                       react/jsx-curly-brace-presence
  8:16  error  Comments inside children section of tag should be placed inside braces  react/jsx-no-comment-textnodes

✖ 2 problems (2 errors, 0 warnings)
  1 error and 0 warnings potentially fixable with the `--fix` option.

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Thanks, I was able to reproduce and make a fix/test.

Thanks much!


Support using AAD auth for Event Hubs Storage account

Signed-off-by: Bernd Verst<EMAIL_ADDRESS>Description
Ensures the Event Hubs binding can connect to the Storage Account for leaser / checkpointer using AAD. This requires at least Storage Blob Contributor role at the Storage Account level.
We are reusing the blob storage token from AAD credential auth logic that is well-tested and used elsewhere.
Fixes #1566

FYI @mukundansundar

Could you add tests to cover changes?

This PR is actually a bugfix adding missing behavior to the component. Therefore I would appreciate it if you can approve this @daixiang0.
@mukundansundar tested the component locally previously with AAD authentication but did not update the integration test. He will be creating certification tests to make this component stable. At that point we will be adding the E2E tests.
The alternative IMO is to revert the existing PRs - but that is unnecessary.


cli: addition of reindex command

LGTM , besides the cosmetics.

@lnielsen I have amended the commit.


[SILInliner] Handled null access storage base.

During inlining, lexical-ness of arguments is preserved.  In the case of address arguments that come from alloc_stacks, the alloc_stacks are marked lexical.  The base of the storage which is passed to the address is obtained via AccessStorageWithBase.  In some circumstances, a base is not found.  Handled that case by changing a dyn_cast of the base to a dyn_cast_or_null.
rdar://99087653

@swift-ci please smoke test

Please test with following pull request:
https://github.com/apple/swift-source-compat-suite/pull/703
@swift-ci please test source compatibility

@swift-ci please smoke test macOS platform

Project is now a UPASS: https://ci.swift.org/job/swift-PR-source-compat-suite-macos/361/artifact/swift-source-compat-suite/UPASS_Doggie_5.0_BuildSwiftPackage.log


https://go.dev/tour/moretypes/26 error

./prog.go:8:1: missing return

It's an exercise, so you have to fill in the code.


nuclei: 2.8.8 -> 2.8.9

Diff: https://github.com/projectdiscovery/nuclei/compare/v2.8.8...v2.8.9
Changelog: https://github.com/projectdiscovery/nuclei/releases/tag/v2.8.9
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 215893 run on x86_64-linux 1

1 package built:

nuclei


Minify should not error if one file is missing

Minify version: 2.1.3
PHP version: 5.2.9

I really like minify and it is pretty inpressive how many request and KB
you can save with so little extra work. But one thing doesn't work as good
as it could do.

If you use the functionality to combine diffrent files to one JS file by
using the query syntax
http://example.com/min/f=scriptname.js,scriptname2.js and one of the files
is missing on the server, the whole minify process will fail.

The result is that if you have let's say 10 JS files and only one is
missing or misspelled ALL JS files won't be loaded.

It would be nice if you could check for file_exists() to catch that error.
Sure I could do that myself as I already have all JA files to combine in an
array but I think many users might have the same problem and as it was not
so easy for me to find the issue it would be a big improvement to minify.

Another problem I had with grouping files was the
$min_serveOptions['minApp']['maxFiles'] option which I haven't found in
your online examples and as soon as I addded more than 10 files the minify
process failed but without a understanable notice of what has failed. It
might be a good idea to responde a file with a 404 in the header status but
with a debugging message to inform the developer what has failed (but keep
the IE 404 bug in mind: http://www.404-error-page.com/404-error-page-too-
short-problem-microsoft-ie.shtml)

But beside those issues the minify just works perfectly!

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 9 Apr 2010 at 11:11

// As soon as I added more than 10 files the minify
process failed but without a understandable notice of what has failed //

I'm having the same problem. I can't add more than 10 files. It's says
something like
bad request ...

Error message :

HTTP/1.0 400 Bad Request

What is it supposed to mean? Can't we combine more than 10 files?

Regards,
Kathees

Original comment by<EMAIL_ADDRESS>on 22 Apr 2010 at 6:34

Got it guys! In the config.php (Inside your minify folder) you can find
following line

$min_serveOptions['minApp']['maxFiles'] = 10;

This shows maximum number of files. If you want to use more than that all you
have to
do is increase the number.

I'm very new to minify. Just trying use it in our application. So don't know we
will
encounter any performance or whatever issues if we do that. May be someone can
point
out???

Original comment by<EMAIL_ADDRESS>on 22 Apr 2010 at 6:39

@Luc, I should be able to apply the fix to group files as well.

Original comment by<EMAIL_ADDRESS>on 26 Jul 2010 at 1:28

Changed state: Started

2.1.4 beta has this feature.
http://code.google.com/p/minify/downloads/detail?name=minify_2.1.4_beta.zip

Original comment by<EMAIL_ADDRESS>on 8 Jul 2010 at 4:56

Changed state: Fixed
Added labels: Release-2.1.3

I'll remove maxFiles in the next release (not real soon).

For missing files, what I could do is prepend the output with:

/* ATTENTION: Minify could not find /path/to/file1.js */
/* ATTENTION: Minify could not find /path/to/file2.js */

In detail, missing files would cause 1 Minify_Source object to be the first
source.
It would always have the same ID (to prevent poisoning the cache with lots of
bad
URLs) and have the message about as its content.

@bernhardkau: I wouldn't put too much fail in Groups being tremendously faster.
PHP
certainly does less work (and w/ cleaner URLs), but I've never ABed the
performance
difference.

Original comment by<EMAIL_ADDRESS>on 22 Apr 2010 at 5:50

Changed title: Minify should not error if one file is missing
Changed state: Accepted

Hi all,

Thanks for the beta fix - it is a much better approach just reporting the
missing files.  In my case for instance, when I upload a quick fix to our site
during the upload process somebody may be making the request for the file at
the same time, and it's better to see a slightly broken page rather than a
completely useless page.

More importantly, I noticed that this fix only applies to individual files - it
still failes completely if a group contains missing files?

in my groupsConfig.php:

 array( 'js_puntersparadise' => array('//js/jquery-1.3.2.min.js','//js/missingfile.js'));

400 Bad Request
Please see http://code.google.com/p/minify/wiki/Debugging.

Please help!

Original comment by<EMAIL_ADDRESS>on 26 Jul 2010 at 12:07

I already mentioned the maxFiles variable in my post. But I don't see any need
of
such a function. It only leads to a failure if you doesn't know about it. And
the
error file doesn't contain any useful information about the error, so you
cannot
find out what was going wrong.

The bigger issue is the one I mentioned first. If a file is missing you want to
group, the whole group will fail which will in the worst case results in zero
JS
and zero CSS on your page. That really is a big issue for me because it is very
likely, that a file might be missing or missspelled or renamed.

@katheeskumar: We could improve the load speed and overall performance a lot
using
the group funtionality. And as Google takes the speed of a website in
consideration
when calculating the ranking, you should really give it a try. It is very easy
to
integrate (if use r.g. use the simple query style links) and it saves a lot of
traffic and HTTP requests.

Original comment by<EMAIL_ADDRESS>on 22 Apr 2010 at 10:23

The controller is
http://code.google.com/p/minify/source/browse/branches/2.1.4/min/lib/Minify/Cont
roller/MinApp.php
Line 139 has the check in the f= option. I need similar code at line 72.

Original comment by<EMAIL_ADDRESS>on 26 Jul 2010 at 2:09

In git master c8e903e

Original comment by<EMAIL_ADDRESS>on 4 Sep 2011 at 1:11

Changed state: Fixed

Thank-you!  It worked for me.  I'm pretty sure i've done a really dodgie hack
because I have no idea what I'm doing, but it works now.  Attached is the new
file.

Original comment by<EMAIL_ADDRESS>on 26 Jul 2010 at 2:36
Attachments:

MinApp.php

Thank-you!  I've been going through the code but can't seem to find out how to
fix it.  Line 83 of Groups.php seems to be where it was already applied, but
for some reason it doesn't work?

Original comment by<EMAIL_ADDRESS>on 26 Jul 2010 at 2:01


README changes

Here is a R notebook R/README.Rmd that is rendered with rmarkdown into README.md.
This is the first batch ie: how atomic kdb objects are sent to R.
It shows problems already (which almost makes this file a unit test)

What are the problems that you mention? Could you fill an issue for them(or fix :)) ?
You also mentioned adding parameter with host/port to rkdb.test() function. Would you like to add it in this change?

Hello, I can fill an issue yes, I think minute second and a few other types
should be converted to string or ITime maybe but then you'll need to import
data.table which might not be a bit overkill.
About rkdb.test I did not do anything as I think it should be removed, it
does not strike me as very usefull...
Can you tell me how to send R objects to kdb now that it is coded?
I'll add docs for this too
On Mon, 26 Jun 2017 at 22:41, Sergey Vidyuk<EMAIL_ADDRESS>wrote:

What are the problems that you mention? Could you fill an issue for
them(or fix :)) ?
You also mentioned adding parameter with host/port to rkdb.test()
function. Would you like to add it in this change?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/KxSystems/rkdb/pull/12#issuecomment-311190284, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADcfOfISTlT1cK-Q_eAeN__E9BGVRuQWks5sICV8gaJpZM4OF2yG
.

ok. agree on rkdb.test - it is intended as a quick sanity check that library works without running all the tests.
examples of sending R objects to kdb are
print(execute(h, "raze", list(c(1,2,3), c(4,5,6))))
execute(h, "+", 2, 5)
execute(h,'{`tmp set x}',data.frame(a=c(1,2,3),b=c("a","b","b")))

Yes. It takes too much space and not useful for normal user. Table from K
to R is useful, but maybe put it in a package doc, not a readme
On Tue, 27 Jun 2017, 7:41 am statquant<EMAIL_ADDRESS>wrote:

@statquant commented on this pull request.
In R/README.Rmd
https://github.com/KxSystems/rkdb/pull/12#discussion_r124188883:

+```

+Assigning a variable in q workspace:
+{r test 2, echo=TRUE} +execute(h, 'x:1+1') #assign x hopefully to 2 +execute(h, 'x') # get back the value +
+
+# Checks
+## Atoms from kdb to R
+
+As per Q for mortals kdb uses the following basic types:
+
+```{r atoms 0, echo=FALSE, warning=FALSE, message=FALSE}
+# install data.table
+if(! 'data.table' %in% rownames(installed.packages())) install.packages('data.table')
Not sure I agree on this, I like the table, I can add another R column for
K<->R and remove the "null" column. Are you strong on removing the table ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/KxSystems/rkdb/pull/12#discussion_r124188883, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAA3Bg2K3MVxfslmAH0ZXp8yxGoo6GWmks5sIKQlgaJpZM4OF2yG
.

Ok it appears that most R type are wrongly converted to  kdb...
I think for instance that a vector of length 1 should be converted to an atom an not a list in kdb...
I have added some more 'checks' but it now become a very long list and might not be very nice to read for the user.
Shall we use those as tests then and only show a few q -> R / R -> q examples ?

There is an issue #11 for names vectors/lists.
data.frame can have complex rownames. maybe we can drop it for trivial case 1..n(same as i)
not sure about atoms... how do you create list of length one if you convert that to atom on K side.

@statquant regarding checks - as i mentioned before instead of readme could you please put this tests into tests/testthat/test-basic.R, so I can integrate them into testing of the package?
Would be nice if readme contains couple of useful examples and leave simple cases for tests.
Example with data.frame,async execution,etc

About data.frame: this just does not make sense to keep the rownames as q tables does have this feature, from R -> kdb it will never be used, else one can add another column holding the rownames.
I think
||r type||kdb type||
|unnamed list|list|
|named list|dictionary|
|vector length 1|atom|
|vector length >1|list|

could you raise an issue for unnamed list,rownames and atom(separate)?

Here are some tests.
Most of the kdb -> R are successful.
Most of the R -> kdb are failing.
Before filling issues for each of them I think we should clarify what kdb <-> types mappings are.
I have added a kdb -> R map in the md of the README
I'll update the README for kdb -> R soon


feat: Added whenIdle

What?
whenIdle is a function that takes a callback which is executed either:
when window.requestIdleCallback runs
when the 500ms timeout is reached
or, if requestIdleCallback is not supported, after 300ms
Why?
This offers people a utility for deferring code execution.
This function is proposed in this PR for React Islands but it is also something that could be useful in the work to defer downloading YouTube javascript.

Coverage decreased (-0.4%) to 99.283% when pulling 90c81b3a3a11494e68d246bc558909ef993a77eb on oliver/when-idle into d494fe6e5b673ef6156aeef95efebe5d92ed5a34 on main.

what's the benefit of this over using requestIdleCallback natively?

what's the benefit of this over using requestIdleCallback natively? the timeout?

It's mostly the api. I like how I can write code like
			switch (when) {
				case 'idle': {
					whenIdle(() => {
						doHydration(name, props, marker);
					});
					break;
				}
				case 'visible': {
					whenVisible(marker, () => {
						doHydration(name, props, marker);
					});
					break;
				}
				case 'immediate':
				default: {
					doHydration(name, props, marker);
				}
			}

Bit it also removes the need for people to have to know that this is an available option. By having a suite of these sorts of defer when functions we give people the tools to construct good deferred experiences without needing to do the research.

I think with requestIdleCallback, the timeout is optional but it's also best practice to provide one. So this abstraction removes the need for developers to know the best practice.

without needing to do the research

I know that you mean "without needing to know about the timeout" in this case. But the advantage of requiring devs to do the research (i.e. by not providing this API) is that they would have to look at MDN and other sources of information that discuss when you should be using requestIdleCallback (i.e. for low priority tasks that may never actually get run, and you're okay with that fact). By providing this API, the burden would be upon us to reiterate that advice and ensure we keep it up to date.

But the advantage of requiring devs to do the research (i.e. by not providing this API) is that they would have to look at MDN and other sources of information that discuss when you should be using requestIdleCallback

On the other hand, the disadvantage to not providing the api is devs might not know it's an option. I didn't know about it until I saw it in the Astro api. There's also the risk that by not providing it in the api you send the message that it's been excluded, or isn't approved.
Searching around. I see we have previously abstracted this function once before and used in inline once before. Arguably, they were the both the right choice in their own contexts. I'm not sure if either of them give or take weight from this PR 🤷
I do agree that it's good to encourage devs to do their own research but in this case if the only information is about when to employ this method I think that could be achieved better in the context of a suite of options where you're explaining which one to choose. The assumption is the developer has already had the thought they want to defer execution, the decision is more about how.
Perhaps this all makes more sense if we share the same vision. In my head, I'm imagining a situation where we provide a suite of functions or patterns to defer execution of code. Currently, we're proposing doing this using Islands but maybe there are other places where we can implement the same patterns but perhaps they've not been employed because it's never been considered?
Or, and this is possible, maybe I'm guessing future use cases that don't exist based on assumptions that are false and we don't need this abstraction here? I'm perfectly happy with this outcome!

Just wanted to chime in with Polyfill.io’s source. It’s chunky.

In my head, I'm imagining a situation where we provide a suite of functions or patterns to defer execution of code.

I must admit this is super compelling 😄 I like the vision of the suite of functions you're imagining and I believe people would enjoy thinking in terms of visible | idle | whenever. I wonder if there's a way to organise this suite of functions to ensure people understand the available options... Would deferUntilIdle and deferUntilVisible send a stronger signal of relatedness?

Would deferUntilIdle and deferUntilVisible send a stronger signal of relatedness?

I like having the word 'defer' in there. Maybe it helps to list all the potential deferations? The things I can imagine we might use to defer stuff are:
if idle
if visible
if media (some media query is true)
if mouse over (the mouseover event fires)
if mouse near (not sure how we achieve this but it's maybe nice to have for smaller components)
So, with your pattern, we could have
deferUntilIdle
deferUntilVisible
deferUntilMedia
deferUntilMouseOver
deferUntilMouseNear
with the 'when' construct it would be
whenIdle
whenVisible
whenMedia
whenMouseOver
whenMouseNear
I think both work. With defer it's a little more opinionated about the motivations for using the function but when is a bit more concise.

Thanks for laying all those examples out. I don't have strong opinions, but looking at the examples, when feels more like natural language, whereas deferUntil is a bit clunky (you need to think for a second) and noisy. Im happy to stick with when, and happy with this whole pattern 😊 Thanks @oliverlloyd!

Do we want to resurrect this one or is everyone happy is I close this for now? Perhaps there's no use cases beyond DCR for this function, which is fine.


adding children to more than one parent

Hi guys, I am using angular gantt for my master thesis and I am very pleased with it so far. Thanks for the great work, by the way. But now I came across an issue which I can't fix that easily: I have modified the data object in such a way, that the first entry of the tree is always a person in charge of some tasks and when I open the tree I can see all the tasks from this person. But now it could happen, that more persons in charge have the same task. So is it possible to add one task to more than one person in charge? Or generally speaking is it possible to add the same children to more than one parent? Any help will be appreciated. Cheers!

It's not possible, you have to duplicate tasks in your model.

Yes, I pressumed so... but thank you anyway!


Not all type issues show up in integrated problem window

Hi, we are using the mypy type checker plugin as the old Python plugin no longer works with newer versions of VSCode. We're noticing that quite a few problems reported by mypy are not being captured in the integrated problems window. When I check the mypy Output window, there are a lot of issues reported, also when running via the CLI. These do not show up in the problem output.
Trivial typing issues such as a: int = 'a string' get parsed correctly and show up in the problems window. We are using mypy version 1.7.1.
I'd expect all errors to be reported in the problems window.

@bverhoeve Check the logs and see what is being reported. Output > Mypy Type Checker is where you can see it. If you see it is using dmypy then it might be this bug in dmypy (https://github.com/python/mypy/issues/15677) in that case try using mypy-type-checker.preferDaemon to false.
If something is reported by mypy in the logs and is not picked up by the Problems view let us know.

I've fixed the issues in the meantime, I'll see if I can reproduce the problem. The output is from a closed source codebase though so I'll have to see what I can share.

Try using mypy instead of dmypy , in case that is what is causing the problem. All I need to see is the pattern of the error reported, you can replace the message part with any text. What I am looking for is something like <file path>:#:#:#:#: <message>.

This was the type error that was not shown in problems:
<path_to_file>.py:80: error: Invalid type: try using Literal[True] instead?  [valid-type]

@bverhoeve Please try the pre-release version of the extension. This is actually a bug in mypy (https://github.com/python/mypy/issues/15463), but we have also added a fallback for these cases. The issue here is mypy does not repro a column number for the error.
Related https://github.com/microsoft/vscode-mypy/issues/35
Related https://github.com/microsoft/vscode-mypy/issues/156

Okay, good to know! Thanks for investigating 🙏

@karthiknadig out of curiosity I also tried this with dmypy and then it indeed does not work. Are there plans to support this or should we use the pre-release version? We have a rather large codebase making mypy quite slow.

@bverhoeve This repo is for the VS Code extension, all we do in this extension is wrap the mypy and dmypy tools. The bugs you see with dmypy are also something that should be addressed in the mypy repo. dmypy has known issues with caching that can sometimes cause it to skip showing issues.
All we do in this extension is run mypy/dmypy on your behalf. The fix we did for issue with missing column number in the extension is only for the part that parses the output from mypy/dmypy. We will publish stable extension after some weeks of testing pre-release.

Can you share some of the output? I want to make sure that this is getting parsed correctly.

Hi @karthiknadig, this is some sample output:
2024-03-05 17:06:27.042 [info] [Trace - 17:06:27] Received notification 'window/logMessage'.
2024-03-05 17:06:27.042 [info] file:///path-to-file.py :
/path-to-file.py:66:1:66:22: error: Need type annotation for "VARIABLE" (hint: "VARIABLE: Optional[<type>] = ...")  [var-annotated]
/path-to-file.py:87:28:87:118: error: Incompatible types in assignment (expression has type "Union[Any, None, tuple[Optional[Any], list[XMLSchemaValidationError]]]", variable has type "tuple[Any, ...]")  [assignment]
/path-to-file.py:176:45:176:55: error: Argument 1 to "etree_tostring" has incompatible type "Union[Any, tuple[Any, list[XMLSchemaValidationError]]]"; expected "ElementProtocol"  [arg-type]
/path-to-file.py:177:16:177:30: error: Item "bytes" of "Union[str, bytes]" has no attribute "encode"  [union-attr]
/path-to-file.py:74:46:74:94: error: Argument 1 to "XMLSchema10" has incompatible type "list[str]"; expected "Union[Union[str, bytes, Path, IO[str], IO[bytes], XMLResource, Element, ElementTree], list[Union[str, bytes, Path, IO[str], IO[bytes], XMLResource, Element, ElementTree]]]"  [arg-type]
/path-to-file.py:74:46:74:94: note: "List" is invariant -- see https://mypy.readthedocs.io/en/stable/common_issues.html#variance
/path-to-file.py:74:46:74:94: note: Consider using "Sequence" instead, which is covariant
/path-to-file.py:77:33:77:43: error: Argument 1 to "loads" has incompatible type "Union[XMLSchemaValidationError, str, None]"; expected "Union[str, bytes, bytearray]"  [arg-type]
/path-to-file.py:276:16:276:86: error: Item "bytes" of "Union[str, bytes]" has no attribute "encode"  [union-attr]
/path-to-file.py:276:41:276:51: error: Argument 1 to "etree_tostring" has incompatible type "Union[Any, tuple[Any, list[XMLSchemaValidationError]]]"; expected "ElementProtocol"  [arg-type]

@karthiknadig were you able to check the output?

@bverhoeve These are all parsed by the pre-release version of mypy extension. Was there one you could not see in the problems window.

I'll check with the pre-release :)

There are more outputs that cannot be parsed...
2024-04-23 19:29:11.114 [info] file:///path-to-file.py:
/path-to-file.py:176:47:176:80: error: Cannot determine type of "XXX"  [has-type]
/path-to-file.py:143:47:143:80: error: Cannot determine type of "XXX"  [has-type]


add logging to fit and score

What
Sets up a way to log params during fitting and metric during scoring

nice! definitely makes sense. I think if we wanna follow the logger approach I laid out on the base branch we'll wanna iterate over step_names, _ = zip(*self.steps) and pull the specific logger for each one (or just the default for now)
feel free to merge this into the rubicon-pipeline branch


HeroBanner: Smoother Animations, CSS Adjustments - Behind main branch

Description
Linked to Issue: N/A
On high resolution screens, the cloud animations in the HeroBanner are a bit overly jittery. This is fixed by engaging GPU acceleration (by setting a translate3d value) and by setting the will-change property on our moving scenes. Animations are now set to change the transform property using translate3d rather than left.
This PR also adjusts the outline on the animated logo in the HeroBanner so that it is better vertically centered.
Changes

Smooth out the animation in pages/components/HeroBanner/SceneMaroon/styles.cloudscapebackground.module.scss
Smooth out the animation in pages/components/HeroBanner/SceneOrange/styles.cloudscapeforeground.module.scss
Tweak the vertical centering of the hex backdrop in pages/components/HeroBanner/styles.module.scss

Steps to QA

Pull down and switch to this branch
Run this branch locally and verify on a high resolution monitor (> 1922px) in browser that the animations in the Hero Banner on / are smoother than on the main branch

Commit is behind main branch - closing


[ci] speed up conda setup for some jobs

In #5648 , I saw that conda create-ing an environment in one shot was significantly faster in some situations than conda  create-ing an environment and then updating it with conda install or conda update (https://github.com/microsoft/LightGBM/issues/5647#issuecomment-1373148054).
That PR was for Windows CI jobs.
This PR proposes similar changes to some of the macOS and Linux jobs.

moves lint, if-else, and swig job code up earlier in .ci/test.sh

swig job doesn't need a conda environment, so time in that job spent waiting for conda create is wasted time
lint and if-else have very different conda-installed dependencies than the other jobs, so they can be totally self-contained and just conda create their own environment with everything they need

benefits of this change
I observed the following timing differences in these jobs on this PR, compared to master:

job
latest master
this PR

(GHA) macOS if-else
06m33s
07m04s

(GHA) check-docs
04m08s
03m30s

(Azure) Linux inference
04m52s
04m36s

(Azure) Linux swig
04m55s
04m02s

(Azure) Linux_latest inference
04m07s
03m56s

(Azure) macOS swig
07m42s
05m05s

this PR:

macOS GitHub Actions: https://github.com/microsoft/LightGBM/actions/runs/3870849816
static_analysis GitHub Actions: https://github.com/microsoft/LightGBM/actions/runs/3870849804
Azure: https://dev.azure.com/lightgbm-ci/lightgbm-ci/_build/results?buildId=14120&view=results

latest master build:

macOS GitHub Actions: https://github.com/microsoft/LightGBM/actions/runs/3849602797
static_analysis GitHub Actions: https://github.com/microsoft/LightGBM/actions/runs/3849602798
Azure: https://dev.azure.com/lightgbm-ci/lightgbm-ci/_build/results?buildId=14109&view=results

Notes for Reviewers
Given all the things that can change from run-to-run, it's hard to trust small differences in the timings based on 1 or even a few of runs. But I'm still pretty confident that these changes make these CI jobs at least a small amount faster, without reducing their stability.
If this PR is accepted, I'll propose another one after that consolidates these into a single conda create for the remaining tasks:
https://github.com/microsoft/LightGBM/blob/61e464bc02bd325672ce71daabfa57d569cf02bc/.ci/test.sh#L37
https://github.com/microsoft/LightGBM/blob/61e464bc02bd325672ce71daabfa57d569cf02bc/.ci/test.sh#L126-L140

@jmoralez could you help with a review on this? I think it should be quick and non-controversial.

thanks @jmoralez !


Enable contributors developing on Apple Silicon (M1 - Arm64)

Description
The Emissary-Ingress build system supports and tests 'linux/amd64' containers which presents a problem when developing locally on newer M1 Macs which have Apple Silicon (arm64). To enable developers to build locally on Apple Silicon we have introduce a new environment variable CONTAINER_BUILD_ARCH which defaults to linux/amd64 so that running make images will produce the correct artifacts.
Important - This does NOT introduce a multi-architecture image but rather to decrease the friction for contributors on Apple Silicon. The multi-architecture support is a separate stream which some of the discussion can be found in #4184.
Related Issues
N/A
Testing

Manually verified that make images is able to properly build the containers
Manually verified that make deploy can push containers out to development K8s cluster.
Verified that Emissary-Ingress ran successfully within cluster after forcing linux/amd64 architecture
Tested that Make variable CONTAINER_BUILD_ARCH defaults linux/amd64 but can be overridden by setting the Environment Variable with the same name

Checklist

[ ] I made sure to update CHANGELOG.md.

I haven't because this doesn't really apply to a specific release...however, it might be good to cherry pick these changes to any release branch that is still supported?

[x] This is unlikely to impact how Ambassador performs at scale.

[x] My change is adequately tested.

It might be good if someone without an M1 Mac or another developer on the M1 Mac tries it out as well but so far there doesn't appear to be any issues with using this approach in the short-term.

[] I updated DEVELOPING.md with any any special dev tricks I had to use to work on this code efficiently.

@kflynn @LukeShu

Are you cool with the variable name I introduced and location or is their a more preferred location?
Should I update the /docs/releaseNotes.yml since this is more of a contributor facing PR?
Do we cherry-pick this kind of stuff into release branches?

@LanceEa Normally before the next release we will create the release branch off of master so landing that change here is fine. We normally wouldn't cherry-pick it into release branches such as 2.1 that have already been released since the state of those branches should be everything that was in that release.

@LanceEa Looks good from here! One request: please drop a note into DEVELOPING.md about this? but yeah, I'm good with the content. (I would've been OK with BUILD_ARCH for the name as well, but I'm good with CONTAINER_BUILD_ARCH.)

I originally had BUILD_ARCH so since you will accept that I changed it to keep it smaller 😄 . I have added a few changes to DEVELOPING.md so let me know if that works for you.


Let DescriptorSource include source code info in multiple files

Currently, DescriptorSource only contains the source code info in a single file, but linter may want to check descriptors across files, e.g., a service may include the request and response message in a different file.
We may let DescriptorSource use {filename, path} as the key to lookup the source code info (currently, we only use path as the key).

#109


getTestCaseCustomFieldExecutionValue not working

Hi,
I have seen already issues regarding getTestCaseCustomFieldExecutionValue here and here. First one seems still to be opened, the second one closed. We are using testLink 1.9.13 and still the same error is present:
Caused by: org.apache.xmlrpc.client.XmlRpcClientException: Failed to parse server's response: Expected methodResponse element, got pre
    at org.apache.xmlrpc.client.XmlRpcStreamTransport.readResponse(XmlRpcStreamTransport.java:188)
    at org.apache.xmlrpc.client.XmlRpcStreamTransport.sendRequest(XmlRpcStreamTransport.java:156)
    at org.apache.xmlrpc.client.XmlRpcHttpTransport.sendRequest(XmlRpcHttpTransport.java:143)
    at org.apache.xmlrpc.client.XmlRpcSunHttpTransport.sendRequest(XmlRpcSunHttpTransport.java:69)
    at org.apache.xmlrpc.client.XmlRpcClientWorker.execute(XmlRpcClientWorker.java:56)
    at org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:167)
    at org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:158)
    at org.apache.xmlrpc.client.XmlRpcClient.execute(XmlRpcClient.java:147)
    at br.eti.kinoshita.testlinkjavaapi.BaseService.executeXmlRpcCall(BaseService.java:90)
    at br.eti.kinoshita.testlinkjavaapi.TestCaseService.getTestCaseCustomFieldExecutionValue(TestCaseService.java:689)
    ... 30 more
Caused by: org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 6; Expected methodResponse element, got pre
    at org.apache.xmlrpc.parser.XmlRpcResponseParser.startElement(XmlRpcResponseParser.java:101)
    at org.apache.xerces.parsers.AbstractSAXParser.startElement(Unknown Source)
    at org.apache.xerces.impl.XMLNSDocumentScannerImpl.scanStartElement(Unknown Source)
    at org.apache.xerces.impl.XMLNSDocumentScannerImpl$NSContentDispatcher.scanRootElementHook(Unknown Source)
    at org.apache.xerces.impl.XMLDocumentFragmentScannerImpl$FragmentContentDispatcher.dispatch(Unknown Source)
    at org.apache.xerces.impl.XMLDocumentFragmentScannerImpl.scanDocument(Unknown Source)
    at org.apache.xerces.parsers.XML11Configuration.parse(Unknown Source)
    at org.apache.xerces.parsers.XML11Configuration.parse(Unknown Source)
    at org.apache.xerces.parsers.XMLParser.parse(Unknown Source)
    at org.apache.xerces.parsers.AbstractSAXParser.parse(Unknown Source)
    at org.apache.xerces.jaxp.SAXParserImpl$JAXPSAXParser.parse(Unknown Source)
    at org.apache.xmlrpc.client.XmlRpcStreamTransport.readResponse(XmlRpcStreamTransport.java:186)
    ... 39 more

Is there any update on this? Can maybe someone point me to this method exposed on TestLink interface itself? I would try to write it on my own then and see if it works

Caused by: org.apache.xmlrpc.client.XmlRpcClientException: Failed to parse server's response: Expected methodResponse element, got pre

If I remember well, I would see similar message when my end point was not returning the XML-RPC response.
If the problem persists, feel free to re-open. Try to include as much information as possible to reproduce the issue, such as TestLink version, API version, code snippet if possible, etc.
Thanks


Youtube link on website

We need to add a sub-menu link to our YouTube page, probably under news and events
https://www.youtube.com/channel/UC7HcWhSetq6nTlpMXPHKz_A/featured

The link is already in the icon at the top of the page using the Youtube icon: https://www.youtube.com/channel/UC7HcWhSetq6nTlpMXPHKz_A?disable_polymer=true

Closing for now as we have a Youtube link in icon form in nav


[niconico] fix a small mistake of thumbnail extraction

In order to be accepted and merged into youtube-dl, each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Thumbnails are still not downloading, but it's my fault. I realized try_get() only accept a list of lambda, so using dict_get() now. This time I have tested my PR to make sure things work.

🤦 I shouldve caught it...


print / no_print in profiles.yml has no effect in dbt v1.7

I tried out specifying no_print: within profiles.yml, and it didn't have any effect.
So maybe it was fully deprecated sometime between dbt v1.5 and v1.7 but we didn't yet update the docs?
Originally posted by @dbeatty10 in https://github.com/dbt-labs/docs.getdbt.com/pull/4740#discussion_r1454071332
To do
Try out the following within each of v1.5, v1.6, and v1.7. Figure out if it was fully deprecated or not and update the docs accordingly.

Create this simple model file:

models/my_model.sql
{{ print("") }}
{{ print("*** Parsing, compiling, or running: " ~ this) }}
{{ print("") }}

select 1 as id

Add the following config to profiles.yml:

config:
  print: False

See if it shows up in the logs or not:

dbt parse

Hey @dbeatty10, I came across a possibly related Slack convo. It maybe needs to be wrapped in a macro until it's fixed in Core?

Hey @dbeatty10, I came across a possibly related Slack convo. It maybe needs to be wrapped in a macro until it's fixed in Core?

Oooh, good call about that Slack convo 🧠
Digging into that convo deeper, it seems it is mainly related to other issues like these:

https://github.com/dbt-labs/dbt-core/issues/8756
https://github.com/dbt-labs/dbt-core/issues/9174

Looking at this more closely, I think this might be a bug in dbt-core, so I'm going to transfer this issue to that repo.


Preprocessor directives are falsely reported as E001 Syntax error

On Windows 10, with fortitude 0.6.1, when using preprocessor directive (#define) before the module, fortitude reports a synthax error
#define set_error(id, msg) \
 if (present(ierr)) ierr = id; \
if (present(errmsg)) errmsg = msg;
module test
...

This is unfortunately coming from the tree-sitter-fortran grammar, currently it can't handle multiline preprocessor #defines like this, there's a clash with the line continuations.
I'd like to improve this aspect, but the preprocessor complicates the grammar a lot unfortunately!


fix flaky integration tests for security analytics

Signed-off-by: Subhobrata Dey<EMAIL_ADDRESS>Description
fix flaky integration tests for security analytics
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[X] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

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

Codecov Report

Merging #241 (44d6ad4) into main (2fcbdba) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #241      +/-   ##
============================================
+ Coverage     38.89%   38.93%   +0.04%
- Complexity      888      891       +3
============================================
  Files           176      176
  Lines          6747     6747
  Branches        823      823
============================================
+ Hits           2624     2627       +3
+ Misses         3880     3878       -2
+ Partials        243      242       -1

Impacted Files
Coverage Δ

...g/opensearch/securityanalytics/model/Detector.java
69.89% <0.00%> (+1.03%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Great find! Thanks for fixing this.


Update to GitOps Operator GA

Update to GitOps Operator GA release and associated dependencies and references

@sabre1041 Is it more involved than the subscription update to openshift-gitops in PR 65?

@mbach04 yes there are multiple changes that are required (mainly related to GA):

GitOps operator uses a different namespace
GitOps operator is named differently
Process to enable OpenShift Authentication is different

Additional steps are needed to align with different namespace and SA that runs the application controller

I see. I think 1 and 2 are minor, I could PR a fix for those if you'd like? Part 3 though would likely take me a fairly long time to investigate.

Resolved via #68


Corruption on launch

If you see this on startup remove the file c:\sys\zxdb.cfg and relaunch.
It's an issue with the config file which will be resolved on the next release.

Backend was down for a short time. This is now resolved. Thanks


chore(): export PluginBase type from @schedule-x/calendar package

I fell into a rabbit hole, join me for a story!
In my project i'm using Deno with Fresh framework. @schedule-x works just fine, but when i run deno check (similar to typechecking with tsc --noEmit), i'm getting an error:
error: Module not found "https://esm.sh/v135/<EMAIL_ADDRESS>
i'm using esm.sh but it's irrelevant here.
The module is indeed missing! At the time of writing this, this esm.sh link shows Module not found.
Why tho?
package.json says that "types": "dist/types/index.d.ts, so i'm looking at that file:
export { ScheduleXCalendar } from './schedule-x-calendar.tsx';
export { useCalendarApp } from './use-calendar-app.tsx';

huh, the .tsx extension is a bit strange within a .d.ts file. Let's
see how the file is created.
Me, the consumer of this package, i get the file from a released @schedule-x/preact tarball. The tarball is built with a github action, which eventually runs npm run build which is vite build && npm run build:dts
ok, what da npm run build:dts do? it runs node scripts/build-declarations.mjs, which is:
https://github.com/schedule-x/preact/blob/main/scripts/build-declarations.mjs
bit weird, we're running node to run tsc and some shell and content.replace. And it's all in try/catch but within catch it just logs some output, but doesn't actually fail. Strange, but i don't know the history and i don't care. What i care is if it works or not.
so i check recent github action runs and here i see an error:
https://github.com/schedule-x/preact/actions/runs/12253666418/job/34182889290#step:5:25
since the script failed on running shell, the output is a buffer. Ok i
clone and setup the repo locally to see wtf. I get the same error, but
now i can .toString() it and i see:
src/use-calendar-app.tsx:4:17 - error TS4058: Return type of exported function has or is using name 'PluginBase' from external module "/schedule-x/preact/node_modules/@schedule-x/calendar/dist/core" but cannot be named.

4 export function useCalendarApp(config: CalendarConfig) {
                  ~~~~~~~~~~~~~~

Found 1 error in src/use-calendar-app.tsx:4

ugh, this is annoying error, i never know what this means. gotta go deeper. I google around and find that
this error often happens due to a missing export.
so i go to node_modules and just manually export the missing type. And voila it works!
awesome, let's make a PR! You're reading it! :grin:
To be honest i'm not 100% sure if this is a proper fix, but it seems
harmless, we're only adding extra export of a type (non-runtime code).

 Thanks for writing it with such suspense, was a good read 😄
I'll try to remember to make a patch version with this in the morning!

and about the weird script in preact:
The way I remember it, the reason for it's existence is exactly what you already highlighted. The d.ts file is pointing to some .tsx files, while in reality it should be .ts


Added JS SDK implementation.

@jmcudd I did not see this pop up, but I already had a working js / ts sdk on the old repo, just never had time to port it to this one. The code is in #402 if you want to take a look at it.
I think I'll take the doc comments from your code if you dont mind, I forgot to add some to mine. I also went with separate implementations of the sdk for different runtimes, since deno, bun and node dont do everything the same way, but I'll have to actually test that, maybe they all use the same Request / Response type.

I'm having some trouble getting the consts template to render for some reason.

Also I think I need to add a build pipeline to publish the npm module.

I'm having some trouble getting the consts template to render for some reason.

I you want to keep building a parallel sdk to mine I recommend looking at the ts const template in my branch. That one renders correctly.

I think we need to coalesce on one.  @Superpat's version is definitely farther ahead atm.


Add arm64 workflow

This commit adds arm64 to the build-matrix for PRs and checks if the PR is labeled with arm64 before running.
ARM64 tests are run on self-hosted runners while x86_64 tests are still run on github macos-latest runners.
Fixes #1754

Codecov Report
Patch and project coverage have no change.

Comparison is base (a45bd22) 76.57% compared to head (ad63102) 76.57%.

:exclamation: Current head ad63102 differs from pull request most recent head 80fcae7. Consider uploading reports for the commit 80fcae7 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1779   +/-   ##
=======================================
  Coverage   76.57%   76.57%
=======================================
  Files          59       59
  Lines        5584     5584
=======================================
  Hits         4276     4276
  Misses       1012     1012
  Partials      296      296

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


how to check best itag

I know that the download function, if no itag is passed, downloads the best available. But sometimes the best available is a webm, not mp4. How can I check what its the best available mp4 itag?

edit: I want to download itag 22, if its not available 18. I tried with property exists, no success. tried with array_key_exists, also no luck. could anyone help me? thank you very much! really appreciate.


How do permissions on deferred transactions work?

I'm playing around with deferred transactions and encountered an oddity.
Say, we have a contract deployed on contract, and an account account. contract has<EMAIL_ADDRESS>as part of its active permission, account does not.
The EOS Portal developer documentation states:

Deferred transactions carry the authority of the contract that sends them.

In my deferred action handler I require_auth(account) and it passes. Shouldn't it fail as the transaction has authorization<EMAIL_ADDRESS>and thus fail as account@active does not contain<EMAIL_ADDRESS>If I add a deferred transaction to transfer eosio.token it however does not correctly pass the authorization check.
Exception Details: 3090003 unsatisfied_authorization: Provided keys, permissions, and delays do not satisfy declared authorizations
transaction declares authority '{"actor":"account","permission":"active"}', but does not have signatures for it under a provided delay of 3000 ms, provided permissions [{"actor":"contract","permission":"eosio.code"}], and provided keys []
    {"auth":{"actor":"account","permission":"active"},"provided_delay":3000,"provided_permissions":[{"actor":"contract","permission":"eosio.code"}],"provided_keys":[],"delay_max_limit_ms":3888000000}

Why does one require_auth check pass, but the other doesn't? Here's the code I deploy on contract
#include <string>
#include <eosiolib/eosio.hpp>
#include <eosiolib/transaction.hpp>
#include <eosiolib/currency.hpp>

using namespace eosio;
using namespace std;

class deferred_example : public eosio::contract
{
public:
    deferred_example(account_name self) : contract(self)
    {
    }

    /// @abi action
    void deferred(account_name from, const string &message)
    {
        // why does this pass? shouldn't the permission be<EMAIL_ADDRESS>        // and thus fail as from@active does not contain<EMAIL_ADDRESS>        require_auth(from);

        print("Printing deferred ", eosio::name{from}, " : ",  message);
    }

    /// @abi action
    void send(account_name from, const string &message)
    {
        require_auth(from);

        eosio::transaction t{};

        // sending this passes the require_auth(from) check in _self::deferred
        t.actions.emplace_back(
            permission_level(from, N(active)),
            _self,
            N(deferred),
            std::make_tuple(from, message));

        // sending this fails the require_auth(from) check in eosio.token::transfer
        // because from@active does not contain<EMAIL_ADDRESS>to
        // t.actions.emplace_back(permission_level(from, N(active)),
            // N(eosio.token),
            // N(transfer),
            // currency::transfer{
                // .from = from,
                // .to = _self,
                // .quantity = asset(10000, S(4, EOS)),
                // .memo = "transfer EOS"});

        t.delay_sec = 3;
        // use now() as sender id for ease of use
        t.send(now(), from);

        print("Deferred transaction scheduled");
    }
};

EOSIO_ABI(deferred_example, (send)(deferred))

In order to focus our efforts on issues that are currently creating difficulty for the community we are closing tickets that were created prior to the EOSIO 2.0 release.  If you believe this issue is still relevant please feel free to reopen it or create a new one.  Thank you for your continued support of EOSIO!


[WIP] RF: Use upstream TShift improvements

First commit: Verify that nipy/nipype#2608 does not break existing usage
Upcoming commit: Use TShift.inputs.slice_timing instead of helper Function interface

I don't think we actually use slice timing in any of our tests, so we can't go by Circle on this...

Status update: nipy/nipype#2608 will be in 1.1.1, so I'll update this and we can merge after 1.1.1.
I tested on ds000030/sub-10159. Output files of the TShift step differed by timestamp.


HeapDump.exe /ForceGC fails to complete

HeapDump.exe fails to complete. This is all I get in the output:

[EXECUTING: PerfView ForceGC WinFormsApp1]
Starting Heap dump on Process 22808 running architecture Amd64.
Exec: "C:\Users\XXX\AppData\Roaming\PerfView\VER.2023-06-<IP_ADDRESS>.657\Amd64\HeapDump.exe" /ForceGC 22808
0.1s: Enabling JScript Heap Provider

The process is a new .NET 7 WinForms app from the template. I have tried to force GC via the UI, via PerfView.exe and directly via HeapDump.exe and nothing works. It was previously working for other apps, but not I can't get it to run for anything. I've rebooted my PC, but that didn't help.
OS: Windows 10.0.19045 Build 19045

It's hard to say what's happened here if there isn't any information in the log.  It looks like you're trying to trigger a GC.  I have confirmed on my local machine that forcing a GC and generating a gcdump works for the Winforms template.
How long does it take to fail?  I'm wondering if it hangs, and if so, can you grab a process dump of HeapDump.exe?

It never fails, it just hangs. I've attached the dump.
HeapDump.zip

Thanks @MisinformedDNA.  What I actually meant was a process dump of HeapDump.exe when it's hung.  If you have windbg installed, you can attach to the process and then run .dump /ma C:\path\to\dump.dmp.  Then share the resulting dmp file.


Corrects the group management samples for C#

The samples for the group management were not working, the string userId could not be bound correctly.
By passing a ClaimsPrincipal we can correctly get the claims the user has used call the method, and therefore we can get its UserId.
Fixes #34409

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

Thanks for your contribution, @Gimly!
#sign-off


exec: new package

This change is

Also, there are a few testing errors related to some changes in the disasm package. WIll push a fix soon.

CI tests with Go 1.7 fail:
wagon_test.go:20: error running go vet:
		exec/internal/bits/bits.go:11: ^uint(0) might be too small for shift of 32
		exit status 1

However, this doesn't seem to be the case with 1.8.

yeah... I was actually trying to devise some workaround for this...

please cherry-pick 876f81a. it should work.

that's a lot of files to review in one sitting...
could you first send a PR with only exec/internal/compile. it seems that package has manageable dependencies.

sure.

thanks. BTW: traditionally, test data directories are called testdata, not test_data.
(this has a nice little optimization in the go toolchain: go get ./... won't inspect the testdata directories nor its children)


ImportError from distribute_setup.py, line 143

When trying to "pip3.6 install --user jsonpipe" I get this error:
%  pip3.6 install --user jsonpipe
Collecting jsonpipe
  Cache entry deserialization failed, entry ignored
  Cache entry deserialization failed, entry ignored
  Downloading https://files.pythonhosted.org/packages/47/e6/c635a9ce1769559167bb04e1106c4b22223084005d440472e5ead1d6b346/jsonpipe-0.0.8.tar.gz
    Complete output from command python setup.py egg_info:
    Downloading http://pypi.python.org/packages/source/d/distribute/distribute-0.6.14.tar.gz
    Traceback (most recent call last):
      File "/home/me/tmp/pip-build-sfedaf0j/jsonpipe/distribute_setup.py", line 143, in use_setuptools
        raise ImportError
    ImportError

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/home/me/tmp/pip-build-sfedaf0j/jsonpipe/setup.py", line 5, in <module>
        use_setuptools()
      File "/home/me/tmp/pip-build-sfedaf0j/jsonpipe/distribute_setup.py", line 145, in use_setuptools
        return _do_download(version, download_base, to_dir, download_delay)
      File "/home/me/tmp/pip-build-sfedaf0j/jsonpipe/distribute_setup.py", line 124, in _do_download
        to_dir, download_delay)
      File "/home/me/tmp/pip-build-sfedaf0j/jsonpipe/distribute_setup.py", line 193, in download_setuptools
        src = urlopen(url)
      File "/usr/lib64/python3.6/urllib/request.py", line 223, in urlopen
        return opener.open(url, data, timeout)
      File "/usr/lib64/python3.6/urllib/request.py", line 532, in open
        response = meth(req, response)
      File "/usr/lib64/python3.6/urllib/request.py", line 642, in http_response
        'http', request, response, code, msg, hdrs)
      File "/usr/lib64/python3.6/urllib/request.py", line 570, in error
        return self._call_chain(*args)
      File "/usr/lib64/python3.6/urllib/request.py", line 504, in _call_chain
        result = func(*args)
      File "/usr/lib64/python3.6/urllib/request.py", line 650, in http_error_default
        raise HTTPError(req.full_url, code, msg, hdrs, fp)
    urllib.error.HTTPError: HTTP Error 403: SSL is required

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /home/cthulhu/tmp/pip-build-sfedaf0j/jsonpipe/

Never mind, this will just open a new can of worms because distribute is deprecated, etc...

Never mind, this will just open a new can of worms because distribute is deprecated, etc...


How to put the title of the legend on top, whereas the values should be distributed horizontally ?

This question was initially asked on SO
Question
I am trying to put the title of the legend on top, whereas the values are distributed horizontally but I cannot.
library(tidyverse)

df1 <- data.frame(
  sex = factor(c("Female","Female","Male","Male")),
  time = factor(c("Lunch","Dinner","Lunch","Dinner"), levels=c("Lunch","Dinner")),
  total_bill = c(13.53, 16.81, 16.24, 17.42))

ggplot(data=df1,
       aes(x=time, y=total_bill, group=sex, shape=sex, colour=sex)) +
  geom_line() +
  geom_point() +
  theme_bw() +
  theme(
    legend.direction = "horizontal",
  ) +
  scale_color_manual(values=c("#0000CC", "#CC0000"),
                     name = 'Gender')

Solution
df1 %>%
  ggplot(aes(time, total_bill, group = sex, shape = sex, colour = sex)) +
  geom_line() +
  geom_point() +
  theme_bw() +
  scale_color_manual(values=c("#0000CC", "#CC0000"),
                     name = "Gender")

See the plot


custom_dataset docs: Workflow for bundler -> colmap for RealityCapture, Metashape etc

Is your feature request related to a problem? Please describe.
For testing Gaussian-Splatting, I wanted to create a workflow to ingest data from RealityCapture. The bundler standard, although dating from 2007, is one of the more common formats for storing camera registration.
I used naver/kapture toolset for bundler -> kapture -> colmap conversion. This workflow can be useful to others and could be added in the custom_dataset doc - can do a PR if anyone think it's worth it!
Describe the solution you'd like
The complete workflow is descibed in this thread, and relies on this kapture_export_colmap.py script
:: Export from RC an imagelist.lst + bundle.out
:: Convert RC imagelist.lst to imagelist-local.lst

:: Install Kapture
pip install kapture

:: Bundler -> Kapture
py C:\Python310\Scripts\kapture_import_bundler.py -v debug -i dataset-bundler\bundle.out
  -l dataset-bundler\imagelist-local.lst -im dataset-bundler\images
  --image_transfer link_absolute -o dataset-kapture --add-reconstruction

# Kapture crop remove black strips + radial->pinhole conversion using kapture-cropper.py
# https://gist.github.com/jo-chemla/258e6e40d3d6c2220b29518ff3c17c40
py kapture-cropper.py -v info -i dataset-kapture\ --border_px 10

:: Kapture -> Colmap
py C:\Python310\Scripts\kapture_export_colmap.py -v debug -f -i dataset-kapture
  -db dataset-colmap\colmap.db --reconstruction dataset-colmap\reconstruction-txt

:: Optional Colmap txt to bin conversion
mkdir dataset-colmap\sparse\0
COLMAP.bat model_converter --input_path dataset-colmap\reconstruction-txt
  --output_path dataset-colmap\sparse\0 --output_type BIN

:: Convert cameras from RADIAL to PINHOLE NOT NEEDED ANYMORE
:: xcopy dataset-bundler\images dataset-colmap\images /i
:: curl https://raw.githubusercontent.com/graphdeco-inria/gaussian-splatting/main/convert.py -o gaussian-splatting-pinhole-cameras-convert.py
:: py gaussian-splatting-pinhole-cameras-convert.py -s dataset-colmap --colmap_executable ..\COLMAP-3.8-windows-cuda
:: [--resize] #If not resizing, ImageMagick is not needed

Describe alternatives you've considered
A nerf-studio workflow already exists for ingesting RealityCapture or Metashape data, but this one, based on bundler data, is more generic and could be applied to any software exporting Bundler.

@jo-chemla  hello! thanks for your fantastic awesome job!
I run your codes successfully for running gaussian splatting from inria code(not nerf studio).
But when I run convert.py, I got same errors repeatedly like
'
Processed file [1/1047]
Name:            00001.jpg
ERROR: Single camera specified, but images have different dimensions.
Processed file [2/1047]
Name:            00000.jpg
Dimensions:      3743 x 2160
Camera:          #1 - OPENCV
Focal Length:    4491.60px
Features:        9594
Processed file [3/1047]
Name:            00002.jpg
ERROR: Single camera specified, but images have different dimensions.
Processed file [4/1047]
Name:            00003.jpg
ERROR: Single camera specified, but images have different dimensions.
Processed file [5/1047]
Name:            00004.jpg
ERROR: Single camera specified, but images have different dimensions.
Processed file [6/1047]
Name:            00005.jpg
ERROR: Single camera specified, but images have different dimensions.
Processed file [7/1047]
Name:            00006.jpg
ERROR: Single camera specified, but images have different dimensions.
Processed file [8/1047]
Name:            00007.jpg
ERROR: Single camera specified, but images have different dimensions.
Processed file [9/1047]
Name:            00008.jpg
ERROR: Single camera specified, but images have different dimensions.
...
'
in bundle.out file, there is 1047 cameras and yes I have 1047 undistorted images(so all images have different size) exported together with the bundle.out.
why it shows "Single" camera?? do you have any idea or guess?
Please help me!
Thank you!

even after reconstruction step and export bundle out again, it shows "0" number of point like
"# Bundle file v0.3
1047 0
....
"
but there are 3 milion lines for points. can you give me some hint? what is correct pipeline or option settings for exporting bundle out? I saw your export setting in "https://forums.unrealengine.com/t/support-colmap-export/1277698/5" but it shows only undistored image exporting options and it's not related to point option.

I have the same problem!

Hey, for those who might still have struggle with this: I solved the problem by keeping the exporting images resolution "fixed" during the bundler export (though now I have other errors)

Hi all,
Sorry I did not receive the notifications on the thread. Indeed the workflow has some hiccups at the moment, like processing the reconstruction in RC is preferred to have a point count different than zero. A recent RC release also fixed the black borders errors, so a portion of this workflow can now be avoided. Please don't hesitate to report back if you have more precise errors!

Hi
Thanks for the scripts! How do we use the generated colmap.db file for, say, Gaussian Splatting?

Hi @nerfstudio-project team. Interested in adding the workflow described in the original thread to the custom_dataset doc?
Now that parsers are implemented for metashape, realitycapture etc it might seem less useful, but allowing import for any bundle.out alignment might still be useful to the doc!
Should I make a simple PR copy-pasting my solution?

Closing this since RealityCapture now supports, thanks to RC 1.5 release, colmap export
If anyone still want to use this workflow based on kapture for bundler to colmap, don't hesitate!


Mutation Conventions errors missing "data" field response results in 500

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
It seems that when using MutationConventions any GraphQL error results in a 500 response instead of 200 because of missing "data" field in response.
This causes issues in Gateway and clients trying to unnecessary retry.
Full repro at https://github.com/sergiu-enache/HC-Mutation-Conventions-Issue
Steps to reproduce
services
      .AddGraphQLServer()
      .AddQueryType<Query>()
      .AddMutationType<Mutation>()
      .AddAuthorization()
      .AddType(new TimeSpanType(TimeSpanFormat.DotNet))
      .AddMutationConventions(applyToAllMutations: false);

public class Mutation
{
    [UseMutationConvention]
    public Person? AddPerson(string name, TimeSpan time) => new Person(name);

    public Person? EditPerson(string name, TimeSpan time) => new Person(name);

    [Authorize]
    [UseMutationConvention]
    public Person? AddPersonAuthorized(string name, TimeSpan time) => new Person(name);

    [Authorize]
    public Person? EditPersonAuthorized(string name, TimeSpan time) => new Person(name);
}

Open BCP
Run mutation that uses MutationConvention

mutation addPerson {
  addPerson (input: {
    name: "test"
    time: "24:59"
  })
  {
    person {
      name
    }
  }
}

which results in 500 because "data" result is missing
{
  "errors": [
    {
      "message": "TimeSpan cannot parse the given literal of type `StringValueNode`.",
      "path": [
        "input",
        "time"
      ],
      "extensions": {
        "field": "AddPersonInput.time",
        "fieldType": "TimeSpan"
      }
    }
  ]
}

Run mutation that does not use MutationConvention

mutation editPerson {
  editPerson (
    name: "test",
    time: "24:59"
  )
  {
    name
  }
}

which results in 200 because of "data" result
{
  "errors": [
    {
      "message": "TimeSpan cannot parse the given literal of type `StringValueNode`.",
      "locations": [
        {
          "line": 16,
          "column": 5
        }
      ],
      "path": [
        "time"
      ],
      "extensions": {
        "field": "Mutation.editPerson(time)",
        "fieldType": "TimeSpan",
        "responseName": "editPerson"
      }
    }
  ],
  "data": {
    "editPerson": null
  }
}

Same thing with [Authorized] mutations with/without Mutation Conventions

Relevant log output
No response
Additional Context?
No response
Product
Hot Chocolate
Version
12.12.1

This is correct. Watch this video which explains GraphQL non-null propagation:
https://www.youtube.com/watch?v=Zx0nvTUfjn4


Add ony_smart_detections param

Added a param to only upload smart Detections of Unifi G4 models.
I've 15 G4 cameras and personally only care about Smart Detections, so I made this PR.

Hi, thanks for the pull request. I like the concept and agree it should be a setting.
I think it would be great if instead of a boolean just for smart dectections it was instead a list of what types to backup. So motion person and vehicle as options, defaulting to all of them.

Great idea. Didnt think of it.
I can try to adjust the PR, but i'm kind of rusty in python, so I need some time :)

More than happy to review code and help where needed! I didn't want to just come along and do it for you incase you wanted to give it a go.

I have made some improvements. What do you think?

Thanks for your help with this feature! It will be in the next release


Create the FastEddy-2.0 release

Describe the Task
Create the FastEddy-2.0 release.

Update docs/conf.py
Update the Release Notes
Update the Downloads page
Add two new publications

Time Estimate
1 hour
Relevant Deadlines
Should be created on Friday 8/9/24
New Parameters
None
Technical References
Adding the following:
Application of the NCAR FastEddy® Microscale Model to a Lake Breeze Front
https://www.mdpi.com/2073-4433/15/7/809
Implementation and Validation of a Generalized Actuator Disk Parameterization for Wind Turbine Simulations Within the FastEddy Model
http://doi.org/10.1002/we.2941
Define the Metadata
Assignee

[x] Assign this Task to the appropriate person

Labels

[x] Select pillar
[x] Select priority
[x] Select requestor(s)

Closed with PR #49


Create hello.cairo

Motivation and Resolution
...
RPC version (if applicable)
...
Usage related changes

Change 1.
...

Development related changes

Change 1.
...

Checklist:

[ ] Performed a self-review of the code
[ ] Rebased to the last commit of the target branch (or merged it into my branch)
[ ] Linked the issues which this PR resolves
[ ] Documented the changes in code (API docs will be generated automatically)
[ ] Updated the tests
[ ] All tests are passing

@Jensnicolaigustavsen please stop spamming, we will block you :)


ServiceClient.Clone() uses more connections than expected

Hi, a potential issue with how Clone() works in 1.1.16
Were using some old .Net Framework code that have been upgraded to .Net 6 with as few changes as possible, due to how parallel code works in .Net Framework all our calls to the serviceclient are wraped in code like
Using Clone = _staticServiceClient.Clone();
Clone.Execute(Message);
When running the above on Consumption based Azure Functions the function host very quickly reaches 600 active connections (upper limit) and throws the following error. An exception was thrown by the invocation.
Exception: System.AggregateException: One or more errors occurred. (An attempt was made to access a socket in a way forbidden by its access permissions. (xxx.crm4.dynamics.com:443))
This error occurs no matter if the "useUniqueInstance" flag is set to true or false.
As far as we can tell it dosen't share the HTTP connection but open up its own connection (it retrys the call inside the HTTP client and eventually gets a connection to call dataverse).
As the new recommended approach is to not use clones in .Net 6 we have changed the code and it works as intended without clones (sharing the HTTP Connectionpool).
As were moving all our code away from .Net Framework we have not tested to see if the same issue is also present when running on .Net Framework.

@FarmerenHE ,
.net core ( specifically .net 6 ) introduces some pretty drastic changes in how multithreading and networking work under the covers.  95% of these are very good, long needed, things that improve performance and such. 5% of them cause some challenges, but that is the cost of moving the development frameworks and setting aside older tech that has been significantly superseded by newer concepts.
I would strongly encourage you to move to using the Async methods as quickly as possible.
That said, the behavior you're seeing with .clone is a byproduct of changes made to how garbage collection works in .net core+.   We have seen this a few where our internal teams updated to .net core.  When you create a clone, as you have seen, a totally new connection is created using a local httpclient. It is very important that you explicitly call dispose on the cloned connection when your done with it, then null the pointer to make sure .net core will dispose it and allow the port to release.   .net framework was much more aggressive about cleaning up released objects that had not been explicitly disposed.
In the Dataverse ServcieClient from 1.1.10 + UseUniqueInstance is unwired and does not perform a function.  All of the connection caching code that was a part of that system has been removed or deactivated.

We found the issue as we changed to doing Async, should I take it as we should avoid the non async method entirely? Because then I'm going to mark them as obsolete in our own extension on the IOrgservice to nudge everyone to make sure they use the async version.
Are any of the 4 recommendations in https://learn.microsoft.com/en-us/power-apps/developer/data-platform/send-parallel-requests?tabs=sdk#optimize-your-connection stil relevant? Or can those be ignored as legacy .Net Framework as well?

@FarmerenHE in the article you reference, you will notice there is .net 6+ section and a .net framework section and they are different.  This is, as I said above, a byproduct of the significant changes made in .net 6+
The recommendations are correct and tested for each.
Insofar as guiding using Async in your external clients directly, Yes, that is what we are encouraging strongly :)
thanks.


Remove loggingexporter from trace gateway config

Description
The trace gateway comes with a loggingexporter configured for every pipeline. That is causing a log entry at least every 10s. The data provided is not needed as for operations it is available via metrics as well. Furthermore, it will increase the amount of logs transfered and stored when collecting system logs. Furthermore, the loggingexporter got deprecated and replaced with a new exporter called "debugexporter" which also indicates that it should not be used for regular operations.
Goal: remove the feature of having a loggingexporter configured per pipeline.

Criterias

No otel-collector instance is logging with the loggingexporter anymore
The otel-collector image bundles the new debugexporter so that it can easily be activated on demand

Reasons

Attachments

Implemented as described, removed usage of loggingexporter on the resources deployed by the manager.
Switched mock backends to debugexporter


[Frontend] Error on changes to global vars used in kernels.

Consider the following code.
GLOBAL_VAR = 1

@triton.jit
def kernel():
  a = GLOBAL_VAR

kernel[(1,)]()

GLOBAL_VAR=2
kernel[(1,)]()

Before this patch, the code above would run without error, but the second time
it ran, a would get the old value of GLOBAL_VAR, i.e. 1.  This is because
the value of GLOBAL_VAR "baked in" to the kernel that Triton compiles.
Now this instead raises an error on the second invocation.

cc @IzzyPutterman


Poor performance on custom data comparing to yolov5

I have a set of brain MRI images with tumor bounding box annotated. The image is grayscale, 8bit, 1024 x 1024. Previously, I trained a yolov5 (yolov5m) tumor detector. In default settings (no change from git clone), after 100 epochs I got<EMAIL_ADDRESS>= 0.7 on validation data.
Using yolox, after 100 epochs training, I got<EMAIL_ADDRESS>= 0.3 on the same validation data. I wonder if I missed any important setting? My yolo_m.py reads below
import os

from yolox.exp import Exp as MyExp

class Exp(MyExp):
    def __init__(self):
        super(Exp, self).__init__()
        self.depth = 0.67
        self.width = 0.75
        self.exp_name = os.path.split(os.path.realpath(__file__))[1].split(".")[0]

        # Define yourself dataset path
        self.data_dir = "datasets/brainMRI"
        self.train_ann = "train.json"
        self.val_ann = "val.json"

        self.num_classes = 1

        self.input_size = (1024, 1024)

        self.max_epoch = 100
        self.data_num_workers = 6
        self.eval_interval = 2

my training command is
python tools/train.py --exp_file exps/example/custom/yolox_m.py --devices 1 --batch-size 4 --fp16 --occupy --ckpt model/yolox_m.pth

mark

@kail85 You may also need to add the following setting:
self.random_size = (32, 32)
self.test_size = (1024, 1024)

The first line means the training scales of your input size ranges from 1024 to 1024. You can modify the value 32 based on your own need. For example: (28, 36).
The second line means using (1024, 1024) for testing.
Let us know if you make any progress.

According to #578, this issue has been fixed.


KAFKA-9991: Fix flaky unit tests

The latest commit #8254 on this test deleted all topics after each test, but the topic was actually shared among tests before. And after that we are relying on the less-reliable auto-topic generation to get the topic which makes the test flaky.
I'm now using different topics for different tests, also setting the app.id for tests differently.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mjsax @ableegoldman


feat(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1278

Questions ?
Please refer to the Backport tool documentation


Try to upgrade Vue.js to v3

This would be future proof, and the earlier we try, the easier it should be

Done (initially in vue3 branch, then merged)


if ~/.npmrc has prefix=~/.npm-global the path solved incorrectly

How to reproduce
Adding the following to ~/.npmrc
prefix=~/.npm-global

global-dirs won't parse the ~ as home directory but as a literal string ~, it should parse it as os.homedir().

Note to self: We can use https://github.com/sindresorhus/untildify for this.

Note to self: We can use https://github.com/sindresorhus/untildify for this.


Upgrade issue from 1.0.3 to 1.0.5: agent node NODE_PRIVATE_KEY is missing Error.

Describe the bug
I upgraded my bootstrap from 1.0.3 to 1.0.5 and after that my agent container won't run anymore...
Agent container says "Error: Configuration could not be loaded! NODE_PRIVATE_KEY is missing." (Please see attached log.)
Should I change something in config file for supernode for 1.0.5 upgrade? Thanks
To Reproduce
My upgrade commands were below:
symbol-bootstrap stop
sudo npm install -g npm
sudo npm install -g symbol-bootstrap@latest
symbol-bootstrap config -p mainnet -a dual -c my-preset.yml --upgrade
symbol-bootstrap run -d

My config file (my-preset.yml) didn't changed. (I masked address and private keys to paste here)
beneficiaryAddress: XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
transactionSelectionStrategy: maximize-fee
nodes:
        - friendlyName: Nakamoto Satoshi Node
          rewardProgram: 'supernode'
          mainPrivateKey: XXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
          nemesisSignerPrivateKey: XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
          host: l.symbol-nember.tokyo
          agentUrl: https://l.symbol-nember.tokyo
          restGatewayUrl: http://l.symbol-nember.tokyo
          voting: true
          maxUnlockedAccounts: 50
          minFeeMultiplier: 100

Expected behavior
node-agent container runs successfully.
Logs
The result of docker logs node-agent is this:
Error: Configuration could not be loaded! NODE_PRIVATE_KEY is missing. Are you providing the right .env file, custom --config file or ENV variables?
    at Function.load (/snapshot/symbol-node-rewards/packages/agent/build/Config.js)
    at Object.<anonymous> (/snapshot/symbol-node-rewards/packages/agent/build/app.js)
    at Module._compile (pkg/prelude/bootstrap.js:1320:22)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1156:10)
    at Module.load (internal/modules/cjs/loader.js:984:32)
    at Function.Module._load (internal/modules/cjs/loader.js:877:14)
    at Function.Module.runMain (pkg/prelude/bootstrap.js:1375:12)
    at internal/main/run_main_module.js:17:47
Loading custom config file 'agent.properties'...
pkg/prelude/bootstrap.js:1320
      return wrapper.apply(this.exports, args);

Desktop (please complete the following information):

OS: Ubuntu 20.04 LTS
Tool Version: 1.0.5
Docker Version: Docker version 20.10.5, build 55c4c88
Node Version: node v10.19.0, npm 7.11.2

Additional context
Add any other context about the problem here.

try:
symbol-bootstrap config -p mainnet -a dual -c my-preset.yml --upgrade
symbol-bootstrap compose -upgrade
symbol-bootstrap run -d

Or better:
symbol-bootstrap start -p mainnet -a dual -c my-preset.yml --upgrade

Aha by
symbol-bootstrap start -p mainnet -a dual -c my-preset.yml --upgrade -d

, new node-agent container image was pulled and now all work very well. Thanks a lot!!!


properly explode newlines from streamContext headers

In order to receive an array of headers which are then later used within the package, it is required to explode default EOL merged headers into separate array elements. Otherwise successive calls using the merged headers will fail

They fail because the headers still contain leading/trailing whitespace. I fixed this in #29.

I'm closing this in favor of #29


How to find the define of x and delta ?

Hi, Mr.Eric,
I find the code :
      param_idx = lambda n, p: self.shape_dict[self.get_shape(p)].index(n) if self.config.mend.shared else None  # noqa: E731             transformed_factors = {                 n: self.mend[str(tuple(self.get_shape(p)))](p.__x__, p.__delta__, param_idx(n, p))                 for n, p in _inner_params(self.model.named_parameters(), self.config.model.inner_params)             }
I want to know where can I find the define of x and delta in code  ?
Is it in torch or python?
Thank you!

Check out hooks.py to see where these are defined, using PyTorch hooks in the forward/backward pass. Feel free to re-open if this doesn't clear things up (and sorry for the delay in responding!).


Update CakePHP to 3.6.11

Pulls in the latest CakePHP 3.6.x and updates other dependencies to match.

/assign hhorak


Send system message only once in infer_type_names

Infer Type Names sends the system message everytime with user message. Fix it such that the system message is only sent once for all the types who's name needs to be inferred, and the user message is sent one time for each type whose name needs to be inferred.

/bounty 25$

/attempt #2665

/attempt #2665

/attempt #2665


Added header section and created hw03_carnesa.py.

Leave the Pull Request in Github. Your code is empty though (it's the stubs file I gave you with none of your own code). Usually a Pull Request is issued after coding is complete, to signal to the repo owner (me) that it is ready for review.
 Your PR gets updated automatically with new changes. Message me again when these fixes are complete.

Leave the Pull Request in Github. Your code is empty though (it's the stubs file I gave you with none of your own code). Usually a Pull Request is issued after coding is complete, to signal to the repo owner (me) that it is ready for review.
 Your PR gets updated automatically with new changes. Message me again when these fixes are complete.

Leave the Pull Request in Github. Your code is empty though (it's the stubs file I gave you with none of your own code). Usually a Pull Request is issued after coding is complete, to signal to the repo owner (me) that it is ready for review.
 Your PR gets updated automatically with new changes. Message me again when these fixes are complete.

Apologies for taking so long. I am ready for review.

@sheggen Apologies for writing another comment. I just wanted to make sure that the other comment had been sent properly after I had accidentally closed the conversation.

No need to apologize, PRs are a conversation back-and-forth!


Include MEGAHIT assembly

The dataset should also be assembled with MEGAHIT (appeared in 2015 in Bioinformatics) for comparison. Ray Meta is 3 years old.

Related to #14

I just started the MEGAHIT assembly to compare. Won't switch to another assembly, though, as groups are working on the Ray one (which is more than decent already!). Still interested in the results...

Assembly
Total bases
No. Contigs
N50
"NG50_Ray"
"NG50_Mega"
Largest Contig

Ray_k31
228,382,457
54,489
9,796
9,796
3,400
333,979

MEGAHIT
319,169,696
113,513
3,566
5,587
3,566
197,663

"NG50" is a pseudo-normalized N50 value: I took the total assembly size of Ray_k31 or Megahit as the genome length. This way, we can compare the contiguity between both assemblies.
So in conclusion, judging by this crude measure we assemble more of the metagenome with Metahit, but the "upper part" of the assembly is more fragmented. Will run a gene prediction next.

Interesting. How many reads map back to these assemblies?

On 22 May 2015, at 23:21, Andreas Bremges<EMAIL_ADDRESS>wrote:
Assembly	Total bases	No. Contigs	N50	"NG50_Ray"	"NG50_Mega"	Largest Contig
Ray_k31	228,382,457	54,489	9,796	9,796	3,400	333,979
MEGAHIT	319,169,696	113,513	3,566	5,587	3,566	197,663
"NG50" is a pseudo-normalized N50 value: I took the total assembly size of Ray_k31 or Megahit as the genome length. This way, we can compare the contiguity between both assemblies.
So in conclusion, judging by this crude measure we assemble more of the metagenome with Metahit, but the "upper part" of the assembly is more fragmented. Will run a gene prediction next.
—
Reply to this email directly or view it on GitHub https://github.com/metagenomics/2015-biogas-cebitec/issues/31#issuecomment-104774512.

For the Ray_k31 assembly, see #13
I did not map the reads in the new assemblies, yet.

Mapping rates are in the 8x range for the MEGAHIT assembly. It might be better at reconstructing low-abundance players, and thus does not include that many more reads (but it's only a first and uneducated guess). Alternative explanations are a higher duplication rate, but I haven't looked into it.
As there are people working on our Ray_k31 assembly already, and we have no extensive experience with MEGAHIT's assembly quality (but know Ray quite well and in general are very happy with its results), I will argue like this in our response letter - closing this.


Fix normalize for empty mark tags

Review: @kesla @iefserge
Type: Patch
Empty mark tags (https://github.com/micnews/html-to-article-json/pull/42) was messing up the normalize step, this fixes that. And also makes sure we keep all mark tags.

LGTM! 👊 👍 🚂🚋🚋🚋


Fix AggregatedMetrics not being posted to Mackerel

Closes #33
Aggregated metrics are now posted to Mackerel when post_aggregated_metrics: true.
The message for posting service metrics is chs.ServiceMetrics, so I used that, and since chs.AggregatedMetrics is no longer used, I deleted it.

@cohalz Thank you! I unintentionally broke posting aggregated metrics at v0.7.0.


Provide name and icon for the electron app

There is no real electron app. This are spawns of webtorrent-hybrid which is just used for the torrent delivering the tiles content and has no interface.

https://github.com/webtorrent/webtorrent-hybrid

It creates a hidden Electron process (which is based on Chromium, so WebRTC support is great!) and communicates with that process to enable WebRTC in Node.js. This adds a lot of overhead, so we are looking forward to using a pure JavaScript implementation, like perhaps Node-RTCPeerConnection when it's ready.

OK, cool, thanks.


Fixes incorrect focused element when creating new file/folder and renaming in file explorer

This PR fixes #87044
The tree could rerender an element and not fire the onFocusChanged event making this.compressedNavigationController invalid.

I have verified this fixes the issue.
Also this is a good change, since we keep less state in the explorerView.
Thanks a lot for this great pr!


feat(cli): add collapsible animations by default

Adds the same collapsible animations shown in the docs: #185

Thanks for the PR @raxelo !
Can we also update the docs (installation section) for collapsible similar to accordion?

Updated the instructions to include the animation step 😄
Thanks for the library 💚


[New Feature]: ADDING ABOUT SECTION

🎮 feature Request
As for now About Us Section is missing in the wesbsite. I would like to work on this issue. Please assign me this issue under GSSoC'24.
Point down the features
Having About section is the most important part for a website. It helps user to know about the website.
I will make an Impressive about us section which will help users to know about your website.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@Dishika18 Could you please explain more what you want to do, if possible you can also add screenshot.

Can you please assign this issue to me under Gsoc

@ChromeGaming assign this to me