lxml.etree.ParserError: Document is empty

Traceback (most recent call last):
File "/home/kali/.local/lib/python3.10/site-packages/pyquery/pyquery.py", line 57, in fromstring
result = getattr(etree, meth)(context)
File "src/lxml/etree.pyx", line 3252, in lxml.etree.fromstring
File "src/lxml/parser.pxi", line 1913, in lxml.etree._parseMemoryDocument
File "src/lxml/parser.pxi", line 1800, in lxml.etree._parseDoc
File "src/lxml/parser.pxi", line 1141, in lxml.etree._BaseParser._parseDoc
File "src/lxml/parser.pxi", line 615, in lxml.etree._ParserContext._handleParseResultDoc
File "src/lxml/parser.pxi", line 725, in lxml.etree._handleParseResult
File "src/lxml/parser.pxi", line 654, in lxml.etree._raiseParseError
File "", line 1
lxml.etree.XMLSyntaxError: Document is empty, line 1, column 1
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/kali/Desktop/WebScraperAsync/main.py", line 94, in
res = asyncio.run(main())
File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete
return future.result()
File "/home/kali/Desktop/WebScraperAsync/main.py", line 69, in main
return await asyncio.gather(*file_url)
File "/home/kali/Desktop/WebScraperAsync/main.py", line 52, in work
if len(session.html.xpath("//h1[@class='text-center']")) > 0 and len(
File "/home/kali/.local/lib/python3.10/site-packages/requests_html.py", line 654, in html
self._html = HTML(session=self.session, url=self.url, html=self.content, default_encoding=self.encoding)
File "/home/kali/.local/lib/python3.10/site-packages/requests_html.py", line 421, in init
element=PyQuery(html)('html') or PyQuery(f'{html}')('html'),
File "/home/kali/.local/lib/python3.10/site-packages/pyquery/pyquery.py", line 217, in init
elements = fromstring(context, self.parser)
File "/home/kali/.local/lib/python3.10/site-packages/pyquery/pyquery.py", line 61, in fromstring
result = getattr(lxml.html, meth)(context)
File "/usr/lib/python3/dist-packages/lxml/html/init.py", line 875, in fromstring
doc = document_fromstring(html, parser=parser, base_url=base_url, **kw)
File "/usr/lib/python3/dist-packages/lxml/html/init.py", line 763, in document_fromstring
raise etree.ParserError(
lxml.etree.ParserError: Document is empty

changed the encoding to 'iso-8859-1' when reading and writing the files, which will probably fix the error.


ci: pre release

https://github.com/stackblitz-labs/pkg.pr.new
Thanks @Aslemammad for this cool feature

Thank you so much for using pkg.pr.new! we really appreciate that ❤


Use existing fetch if XMLHttpRequest not available

This might be more of a conversation than a pull request, but I want to attempt to address an issue that's been noticed in a dependency, cross-fetch, that this might help solve.
As it stands, XMLHttpRequest doesn't exist in some runtimes, such as Deno or Cloudflare Workers. Tweaking this library might have a positive cascade of solving some issues, such as these:
https://github.com/supabase/supabase-js/issues/154
https://github.com/lquixada/cross-fetch/issues/78
https://github.com/supabase/supabase-js/issues/161

@jcs224, we already have similar code in-place which only adds the polyfills to the global object if the native fetch does not exist:
https://github.com/github/fetch/blob/master/fetch.js#L600-L605
This package wasn't written with those runtimes (deno, cloudflare workers) in mind, is there a way to run our test suite in those runtimes?

@JakeChampion I changed the title of the PR to more accurately reflect the actual change I'm proposing.


Fix raw ubuntu builds - Set directory path properly

Description of changes:

Set the remote repo directory path properly
Do not clean up repo when running builds on CI

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm


[FEATURE]: allow INSERT in CTEs (WITH clauses)

Describe what you want
Drizzle ORM supports SELECT queries in CTEs (WITH clauses). From the docs:
const sq = db.$with('sq').as(db.select().from(users).where(eq(users.id, 42)));
const result = await db.with(sq).select().from(sq);

Currently, Drizzle does not support INSERT queries in CTEs. Example of such a query:
with inserted as (
  insert into orders (user_id, customer_id, order_date, shipping_product, payment_method)
    values (1, 1, now(), 'default', 'cash')
    returning *
)
insert into order_positions (position, order_id, product_id, quantity, price)
  values
    (1, (select id from inserted), 1, 1, 99.90),
    (2, (select id from inserted), 2, 1, 49.90),
    (3, (select id from inserted), 3, 4, 149.90)
  returning *
;

As you can see, this would be very useful for nested inserts in a single query and should be supported by Drizzle to be a feature complete SQL query builder.

@AndriiSherman should this also include DELETE as well?
WITH deleted AS (DELETE FROM table WHERE condition IS TRUE RETURNING *) SELECT count(*) FROM deleted;

IMO it should include INSERT, SELECT, UPDATE and DELETE as Drizzle aims to be full featured SQL-in-TS

This needs to be added, as it is something that appears in complicated apps! And both MySQL and Postgres support such CTE queries.

+1
In my case, I need to use it with UPDATE. All operations are important for this feature.
Maybe there needs to be a note on the documentation that it only supports SELECT for now.

Please add this!

Is this why I get the error qb.getSelectedFields is not a function (pg-core/db.js:88:44) with the following code?
const newTimezoneId = db
	.$with("new_timezone_id")
	.as(
		db
			.insert(timezones)
			.values({ name: "UTC" })
			.onConflictDoNothing()
			.returning({ id: timezones.id }),
	);

This would be a game changer.

Has anyone come up with a workaround?

I also need this.

@michrome Yes.

I wanted a find_or_create, so was trying to insert then return the value if it already existed. In my app specifically, I know the record will exist most of the time so just broke it into two queries:
async function getTimezoneId(timezoneName) {
	const existingTimezone = await db
		.select({ id: timezones.id })
		.from(timezones)
		.where(eq(timezones.name, timezoneName))
		.limit(1);
	if (existingTimezone.length !== 1) {
		return await createTimezone(timezoneName); // executes a new INSERT
	}
	return existingTimezone[0].id;
}

Not really a workaround but suitable (better?) ifor my use case 😀

Have you come up with a workaround?

I wanted a find_or_create, so was trying to insert then return the value if it already existed. In my app specifically, I know the record will exist most of the time so I broke it into two queries as below. Not a workaround but suitable (better?) for my use case 😀
I agree with comments above though: Drizzle should add support for INSERT with CTEs as there are many valid use cases for it.
async function getTimezoneId(timezoneName) {
	const existingTimezone = await db
		.select({ id: timezones.id })
		.from(timezones)
		.where(eq(timezones.name, timezoneName))
		.limit(1);
	if (existingTimezone.length !== 1) {
		return await createTimezone(timezoneName); // executes an INSERT and returns the new ID
	}
	return existingTimezone[0].id;
}

Bummer, I was hoping to avoid multiple queries. My company will have to stick with Kysely for data-intensive applications.

Great feature! It can improve multiple inserts to many tables

really need this in my app, is there a workaround available even if not pretty and/or not fully typesafe that keeps the whole thing in one DB query? i was trying to figure out if i could use toSQL() to build the insert and drop it into a CTE with the sql template helper but wasn't able to get it working with parameters.

I want to add that a nested insert would be useful here.  Basically a with clause, but more user friendly.
await db.insert({
  authors: {
    name: 'Jane Austen',
    books: {
      create: [
        { title: 'Pride and Prejudice' },
        { title: 'Sense and Sensibility' },
      ],
    },
  },
});

J

I want to add that a nested insert would be useful here. Basically a with clause, but more user friendly.
await db.insert({
  authors: {
    name: 'Jane Austen',
    books: {
      create: [
        { title: 'Pride and Prejudice' },
        { title: 'Sense and Sensibility' },
      ],
    },
  },
});

J

it more about db.query.create

I want to add that a nested insert would be useful here. Basically a with clause, but more user friendly.
await db.insert({
  authors: {
    name: 'Jane Austen',
    books: {
      create: [
        { title: 'Pride and Prejudice' },
        { title: 'Sense and Sensibility' },
      ],
    },
  },
});

J

it more about db.query.create

What if we want to insert the author but update the book?  We need a way to differentiate the nested table action.
Note: Going to use insert instead of create for consistency from SQL CRUD (insert, select, update, delete). You could also get into connect or disconnect, but that is a different feature.
await db.insert({
  authors: {
    name: 'Jane Austen',
    books: {
      insert: [
        { title: 'Pride and Prejudice' },
        { title: 'Sense and Sensibility' },
      ],
    },
  },
});

For update, we would need a key, so:
await db.insert({
  authors: {
    name: 'Jane Austen',
    books: {
      update: [
        { id: '0x123', title: 'Pride and Prejudice' },
        { id: '0x234', title: 'Sense and Sensibility' }
      ],
    },
  },
});

J

I want to add that a nested insert would be useful here. Basically a with clause, but more user friendly.
await db.insert({
  authors: {
    name: 'Jane Austen',
    books: {
      create: [
        { title: 'Pride and Prejudice' },
        { title: 'Sense and Sensibility' },
      ],
    },
  },
});

J

it more about db.query.create

What if we want to insert the author but update the book? We need a way to differentiate the nested table action.
Note: Going to use insert instead of create for consistency from SQL CRUD (insert, select, update, delete). You could also get into connect or disconnect, but that is a different feature.
await db.insert({
  authors: {
    name: 'Jane Austen',
    books: {
      insert: [
        { title: 'Pride and Prejudice' },
        { title: 'Sense and Sensibility' },
      ],
    },
  },
});

For update, we would need a key, so:
await db.insert({
  authors: {
    name: 'Jane Austen',
    books: {
      update: [
        { id: '0x123', title: 'Pride and Prejudice' },
        { id: '0x234', title: 'Sense and Sensibility' }
      ],
    },
  },
});

J

i talk that it more about relation API
this differ from SQL writing - dont like it in query builder
but proposal is good i need it


Support for filter extension

Need to add support for registering custom filters. A user can implement custom filters like Between, NotBetween, And, Or, Xor etc.

Duplicate of issue #7


Ensure Dart Sass file:// sources use relative paths

In preparation for v4.6.0 release we noticed Dart Sass writes absolute file paths to source map sources:

https://github.com/sass/dart-sass/pull/1021

You can see we last released source maps (via Node Sass) with relative paths:
dist/govuk-frontend-4.5.0.min.css.map
dist/govuk-frontend-4.5.0.min.js.map
But for Dart Sass builds an absolute path to govuk-frontend will be included:
Node Sass source maps
{
  "sources": [
    "components/accordion/_index.scss"
    "components/back-link/_index.scss",
    "components/breadcrumbs/_index.scss"
  ]
}

Dart Sass source maps
{
  "sources": [
    "file:///Users/username/path/to/govuk-frontend/src/govuk/components/accordion/_index.scss"
    "file:///Users/username/path/to/govuk-frontend/src/govuk/components/back-link/_index.scss",
    "file:///Users/username/path/to/govuk-frontend/src/govuk/components/breadcrumbs/_index.scss"
  ]
}

This PR ensures we keep relative paths

The GOV.UK Design System website also uses Dart Sass but isn't built locally:


Build report error fixes

Docs Build status updates of commit 1c3e004:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 1c3e004:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch lindalu-MSFT-patch-1 has been deleted or has been force pushed: fatal: Couldn't find remote ref refs/heads/lindalu-MSFT-patch-1

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


Option to cancel editing a saved draft

Proposed solution

I a saved draft is opened from Choose from drafts, it opens a form with pre-filled values.

However if a user change his/her mind and doesn't want to continue with this draft and have a new empty form, there should be a way for the user to escape/cancel from the editing want to cancel editing the current draft.
Also when a user submit a saved draft, the values of submitted form still exists, there should be a way to clear the submitted form and have a new empty form.

DoD (Definition of Done)

User is able to cancel editing the current saved/submitted draft and have a new empty form.
Testing

Integration test, maybe also e2e test, peer to peer review

we can have a sticky bar that contains form controls such as:

clear form
new form
save draft
submit

we don't need the hide button.


fix: 修改容器品类

Owner

[x] Nickname(Tuya): 萧瑟
[x] Intra-group review（组内审阅人员）：萧瑟

Description
fix ipc container
Package
Please select the relevant package.

[ ] tuya-panel-animation-sdk
[ ] tuya-panel-api
[ ] tuya-panel-electrician-sdk
[x] tuya-panel-ipc-sdk
[ ] tuya-panel-lamp-sdk
[ ] tuya-panel-lock-sdk
[ ] tuya-panel-robot-sdk
[ ] tuya-panel-outdoor-sdk

Type of change
Please select the relevant option.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Self CheckList
Please check all options.

[x] Intra-group review
[x] ESLint lint
[x] Example Run
[x] Jest Run
[x] No Grammer Error
[x] No Spell Error
[x] No Merge Conflict

@XiaooSe jest 失败了


fix: fix the issue with applying text size change to a specific container

This PR should fix the issue with the withPreview example that the text size change doesn't apply to the preview container.
One further issue is running node resolveIncludes.js rebuilds individual enactor sass into css, but doesn't rebuild the "Enactor.css" which is a collection of all individual enactor css files. Since "Enactor.css" is the css loaded by the withPreview index file, I ended up with manually modify Enactor.css.

@jobara Could you take a look at this too? Thanks!

You are right, @jobara. Changing that.root to that.container in fluid.prefs.enactor.textSize.set only applies to the case when applying the text size change to a specific container rather than the separated panel case.


Instance args categories

Adds equivalences of large precategories.
Adds notation using instance arguments.

It causes an error in the category of groups. That kind of confirms my worry about instance arguments. Maybe it works well in an abstract setting, eg categories for their own sake, but we want the categories to be as functional as possible for our applications. So if the instance arguments don't work with the applications, I'd definitely say it's not worth the trouble to have them.

I'll see if I can make the instance arguments work for the category of groups. Otherwise I'll just remove all uses of instance arguments.

The error was just that I had renamed some arguments and I forgot to change them in the category of groups.

Ok, glad it's working now. It seems that you are convinced that instance arguments are a good thing:)
Can we rename the weird composition symbol to something more conventional though? We shouldn't try to come up with the weirdest symbols. Perhaps _∘Precat_ could do, which also leaves the door open for _∘Cat_.

Yes, I think instance arguments are a good thing :) I can change the notation. The ¤ symbol was the closest thing I could come up with that wasn't the usual composition symbol.

I would also like to suggest that we should try to stick to one kind of symbol for evaluation of functors. It's a bit of a stretch to use ⟨_⟩ for evaluation, because it is very unconventional, and I think it is a bit too much if we use a whole range of variations of angled brackets. So I suggest _obj-⟨_⟩ for the action on objects of functors and _hom-⟨_⟩ for the action on morphisms of functors. But it will still look weird to me.
When we write obj-functor-Precat F x, then we actually have F x in the notation if you ignore the word obj-functor-Precat, so to me that's closer to the usual way of writing F applied to an object x.

We'll go with instance arguments then:)
Martin Escardo told me a horror story though, where his whole library once broke because the Agda implementers changed how instance arguments work. So this feature is still a bit experimental, it seems.

By the way, I saw that you also removed the category from the type of objects. Are you sure that won't give trouble?
In an adjunction we say that for every x : obj C and every y : obj D there is an equivalence Hom(x,G(y)) ≃ Hom (F(x),y). With the new notation, we will have x : obj and y : obj. Will Agda be able to figure out that x is in C and y is in D?
I'm doing adjunctions now in the other pull request, that's why I ask.

Where in the code do you mean?

In the Ljubljana branch in a new file called adjunctions.lagda.md.
But I'll wait with merging the pull request, because I don't want to give you the headache of these merge conflicts. I'll deal with them. I'm afraid they will be quite severe, because we've both been working on large-categories.lagda.md

Anyway, for objects it seems not unreasonable to mention the category, even if Agda can be set up so that this is not technically necessary.

Oh, I meant, where in the code do you mean I have removed the category for objects? In the large or small categories?

Ah, it looks like that only happened in categories.lagda.md

Do you mean obj-Precat? This takes the category as an explicit argument

So does obj-Cat

Ay sorry, my bad. It is in a module. I only saw from the git diff that the category was removed, but I missed the module. I said nothing:)

I have removed the mention of C in the Precat functions below associative-composition-Precat because I put them into a module with the category as a parameter. But this parameter is explicit :)

Ah, I see how you mistook that :) no worries!

Sorry for giving you a hard time. Let me know when the PR is ready to be merged.

Don't worry, I gladly take input on my code :) I don't know how much time I'll have to work on the code over the holidays so you can merge your work on adjunctions to master and don't have to wait for me. Then I can sort out any merge conflicts later when I'm done :)

Cool! I'll merge it tomorrow, and try to set up the instance arguments for the adjunctions too. Happy holidays Elisabeth!

Happy holidays!


static luajit link failed

with a error  _luaL_loadbufferx symbol redefined .
i am use msvc 2015 update3
luajit version is 2.1 beta2 with build command msvcbuild.bat static

I found some compatible code for old version lua in sol.hpp
inline int luaL_loadbufferx(lua_State* L, const char* buff, size_t size, const char* name, const char*) {
    kepler_lua_compat_get_string_view ls;
    ls.s = buff;
    ls.size = size;
    return lua_load(L, kepler_lua_compat_get_string, &ls, name/*, mode*/);
}

But this function has already defined in lj_load.c file at 137 lines .

Well, I added some guards to prevent this from happening, so you should be all set.
Get it here: https://github.com/ThePhD/sol2/releases/tag/v2.12.3

thanks.


logrotate: do not add mail if 'mail = false' is specified

Reported-by: Ricardo M. Correia<EMAIL_ADDRESS>@wizeman
Description of changes
module would incorrectly pull in sendmail if mail was explicitly disabled
Things done

Built on platform(s)

[x] 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/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I hadn't noticed tests failed: n.mail is not necessarily a bool (it'll often be a string in practice, the destination mail address) so adjusted the fold to work with strings.

Merging because @wizeman approved. Thanks!


Add gir

Fix #53
@GuillaumeGomez, @sdroege, @pwil3058
PixbufFormat has comment "It's owned by the library and we never free it" but I don't see why.
Don't generated:
PixbufAnimationIter etc. can't be fully generated due glib.TimeVal (also see #8) and I don't want add xxManualExt here.
Pixbuf has getters from constant self pointer (same as https://github.com/gtk-rs/glib/pull/249)

Generated Pixbuf.
Due const IsA errors many getters like get_colorspace stays manual, but also have get_property_colorspace generated.
Its was not removed to generate connect_property_colorspace_notify.

Looks generally good to me

For me as well so let's merge.
@EPashkin: Please add the missing implementation that @sdroege pointed.

All good for me to go btw, I was just pointing out things that could be improved on top of this


Cookie.java Removing a leading dot from cookie domain makes it unable to be used for subdomains

https://github.com/square/okhttp/blob/59e70dad8193ad5f8cd79749fc00be7dcf0308d1/okhttp/src/main/java/okhttp3/Cookie.java#L421
On this line the leading dot of the cookie domain is removed permanently, and we are unable to see if the cookie should be used on subdomains.
Example:
I perform a request for www.site.com/login. In response it returns me Set-Cookie with domain=.site.com.
After parsing, cookie is stored with a domain site.com, which makes it non-usable even for www.site.com
AFAIK, it does not match cookie specification.

The specs were revised and the leading dot was dropped as a feature. Today cookies whose domain is example.com will domain match foo.example.com dot or no dot.
https://tools.ietf.org/html/rfc6265#section-<IP_ADDRESS>
https://tools.ietf.org/html/rfc6265#section-5.1.3

BTW, you can use this to check for a domain match. It’s a bit subtle.
https://square.github.io/okhttp/3.x/okhttp/okhttp3/Cookie.html#matches-okhttp3.HttpUrl-

then it themes like Android SDK bug.
I've opened this issue because I was trying to use JavaNetCookieJar, with a default CookieManager. And it will not use cookies for foo.example.com, if cookie domain was originally .example.com

Got it. If there's a failing test we can fix, I'm interested.

@metalurgus ,  I meet the same problem，if the cookie's domain is not hostOnly, the OKHttp will remove the leading dot, but the android's CookieManager do not implement the new RFC specs. So what your work around?


create job

i have some xml data,
how to create a job by xml data?

From the response I'm getting when trying to create a job I'd say it doesn't handle http -> https redirect (301). Is there a way to make POST requests follow redirects? Looks like GETs work just fine.


fix: padding and positional embeddings

Issue
When using the default example from train.py and setting pos_emb=True, the following line fails:
if exists(pos_emb):
    positions = pos_emb(torch.arange(tokens.shape[-2], device = device))
    tokens = tokens + positions # error here

with the error The size of tensor a (256) must match the size of tensor b (768) at non-singleton dimension 4.
To fix this:

Positional embeddings need to be created in reverse order, analogously to how self.token_embs are created. I'm not exactly sure why this is done in reverse to begin with but the correct module order is restored via tokens_at_stages.insert(0, tokens) again
The positional embedding at each stage should be of dimension of the transformer sequence length of that stage

Example
Before, the positional and token embeddings were in the form
self.pos_embs=ModuleList(
  (0): Embedding(512, 768)
  (1): Embedding(4, 512)
  (2): Embedding(4, 256)
)
self.token_embs=ModuleList(
  (0): Embedding(256, 256)
  (1): Sequential(
    (0): Embedding(256, 256)
    (1): Rearrange('... r d -> ... (r d)')
    (2): LayerNorm((1024,), eps=1e-05, elementwise_affine=True)
    (3): Linear(in_features=1024, out_features=512, bias=True)
    (4): LayerNorm((512,), eps=1e-05, elementwise_affine=True)
  )
  (2): Sequential(
    (0): Embedding(256, 256)
    (1): Rearrange('... r d -> ... (r d)')
    (2): LayerNorm((4096,), eps=1e-05, elementwise_affine=True)
    (3): Linear(in_features=4096, out_features=768, bias=True)
    (4): LayerNorm((768,), eps=1e-05, elementwise_affine=True)
  )
)

With this change, they are:
# same as before
self.pos_embs=ModuleList(
  (0): Embedding(4, 256)
  (1): Embedding(4, 512)
  (2): Embedding(512, 768)
)
self.token_embs=ModuleList(
  (0): Embedding(256, 256)
  (1): Sequential(
    (0): Embedding(256, 512) # fixed
    (1): Rearrange('... r d -> ... (r d)')
    (2): LayerNorm((2048,), eps=1e-05, elementwise_affine=True)
    (3): Linear(in_features=2048, out_features=512, bias=True)
    (4): LayerNorm((512,), eps=1e-05, elementwise_affine=True)
  )
  (2): Sequential(
    (0): Embedding(256, 768) # fixed
    (1): Rearrange('... r d -> ... (r d)')
    (2): LayerNorm((12288,), eps=1e-05, elementwise_affine=True)
    (3): Linear(in_features=12288, out_features=768, bias=True)
    (4): LayerNorm((768,), eps=1e-05, elementwise_affine=True)
  )
)

Further changes:

Set correct padding id in the forward pass
Enhance example with bits-per-byte loss (#14)
Format train.py - The rest of the files have not been formatted for the sake of correct diffing

@lucidrains let me know if you agree with the changes. Thanks!

@eegli hey Eric, thanks for raising the issue and for writing up the PR
i went ahead and fixed it another way, mainly because there are bunch of unrelated changes in train.py
do you want to see if it looks ok to you?

@lucidrains the diff looks good to me, thanks for the quick fix!


Action Creators: Releasing XHR Control

I'm not sure how I feel about that title. But I didn't want to bikeshed it too much because what I really want to bikeshed is this stuff down there.
⬇
Currently, if you leverage methods from the resourceful-action-creators, it does two really neat things for you. For me, those things are in this order of value:

Help enforce resourceful-normalized API responses (either naturally or with transformData so that they can be properly dispatched into the resourceful-redux reducers. The major value add for me here comes from being able to reason about a given resource slice (e.g. books) in my React applications, especially when coupled with getStatus() to enhance resources in UIs with spinners, retries, cancels, etc.
Automatically manage the XHR requests for you with an easy-to-proxy API to the underlying xhr module for making resource related requests.

There's obviously more to it than that, but that is where I typically reach for this module when building. One downside to this however, is that if you ever want to take back control of that second portion (e.g. by using something like WebSockets, observables/streaming responses, or even just your own specific XHR edge cases or preferred libraries), you lose out on the benefits of the first portion.
Users are definitely still able to make their own resource requests, and then invoke dispatch(...) on their own. The only tricky part is, that they need to really understand how the reducers expect action objects to be formed. It's not impossible of course, it looks something like this:
dispatch({
  ...options,
  type: actionTypes[`${crudType}_RESOURCES_SUCCEEDED`],
  statusCode,
  resources,
  res
});

You can see more about what the options object is here as I think that's probably a good API to try to model from when creating what I think could be called: resourceful-dispatch-creators that could possible be it's own module for the ecosystem.
Imagine an action creator that did not want to loose control of it's own XHR (say it could manage cancel-ability, duplicate request logic, retries, etc). Right now, you'd have to make sure that you dispatch the correct shapes for the correct reducers. But many of those options could be provided for you by default - then you'd really only need to worry about passing the resources or the error from an app developer standpoint.
Here's an example of how I think it could work, compared with the example here
import { dispatchCreators } from 'resourceful-dispatch-creators';

function readManyBooks(pageNumber) {
  // instead of configuring `readResources` here, we're providing the same info,
  // but this will be used to create our final `dispatch(...)` that our reducers pick up
  const dispatchers = dispatchCreators({
    crudAction: 'read',
    resourceName: 'books',
    label: 'homePageBooks',
    mergeLabelIds: false,
  });

  // we make our own preferred XHR request (or open a stream, etc) here
  // and then we use the same `.{status}` method that we'd expect to see results
  // from in `getStatus()` calls in our UIs

  dispatchers.pending();

  return fetch('/books')
    .then(res => dispatchers.succeeded(res)
    .catch(err => dispatchers.failed(err);
}

Not 100% sure of the naming, or API, or if this would be useful for folks. But thought of it today while talking with @jmeas about #185 and how that may path us toward building out something really robust like ember-concurrency - which may still be useful for the fully formed action creators library, but would further lock folks into our XHR decisions and patterns.
This could also exist and be useful for folks who want more control, simple APIs, and flexibility to do things how they really need to in their edge cases.
Thoughts? /cc @vinodkl @marlonpp @tbranyen

This seems like a good idea to me. I can imagine refactoring action-creators to use this lower-level lib.
My only bit of feedback is that we shouldn’t use the word “dispatch” unless they’re bound, and they probably shouldn’t be bound. So maybe something like crudActionHelpers or something (to distinguish them from the action-creators)
:+1:

Updated your comment to fix some syntax errors @sprjr, I like the API :+1:

@jmeas agreed, I am not sold on the same either. The weird thing is, these are really just action creators with a resourceful API over them. You give them some data, and they return an object that you'd probably dispatch yourself now that I think about it. So I was kinda confused because the current action creators are like, super-duper ones that do thunking for you.
@tbranyen thanks! I'm terrible at pseudo. Well, and real code too.

Also just noticing that my code doesn't actually work here at all. It would need to be more like:

export action = (dispatch) => (opts) => {
  return fetch('/foo')
    .then(res => dispatch(dispatchers.succeeded(res));
}

The existing resourceful-action-creators lib has been renamed and published as resourceful-xhr to allow the lib described in this issue to use the name resourceful-action-creators.
When we publish this lib, we'll just need to make sure that the first release is2.0.0.
:v:

Bike shed gist of the API is here. I will try to get an early PR out so we can try an -alpha.1 release in a few apps/plugins and see how that works In The Real World™

Resolved in #250


MSID: 95313 Version: 1 DOI: 10.1101/2<IP_ADDRESS>4143

MSID: 95313
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>4143
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=95313
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>4143

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/95313v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>4143
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/95313/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Not sure what's going on here, but it's looks awful and needs resolving:

(Plus sign is missing in start+methods)

Need to wait for this to update.

We're good to go!


Merge Dev into Staging

Update staging with the latest version of dev.
Description
List of proposed changes:

What to test for/How to test
Additional Notes

Closing


请问需求的PyQt版本是多少？

猜测可能是版本不对，启动不了🥺
(base) PS C:\Users\l\Desktop\8086-emulator-master> python mainUI.py
Traceback (most recent call last):
  File "C:\Users\l\Desktop\8086-emulator-master\ui\codeeditor.py", line 150, in paintEvent
    self.codeEditor.lineNumberAreaPaintEvent(event)
  File "C:\Users\l\Desktop\8086-emulator-master\ui\codeeditor.py", line 52, in lineNumberAreaPaintEvent
    painter.drawText(0, top, areaWidth - rightMargin,
TypeError: arguments did not match any overloaded call:
  drawText(self, p: Union[QPointF, QPoint], s: str): argument 1 has unexpected type 'int'
  drawText(self, rectangle: QRectF, flags: int, text: str): argument 1 has unexpected type 'int'
  drawText(self, rectangle: QRect, flags: int, text: str): argument 1 has unexpected type 'int'
  drawText(self, rectangle: QRectF, text: str, option: QTextOption = QTextOption()): argument 1 has unexpected type 'int'
  drawText(self, p: QPoint, s: str): argument 1 has unexpected type 'int'
  drawText(self, x: int, y: int, width: int, height: int, flags: int, text: str): argument 2 has unexpected type 'float'
  drawText(self, x: int, y: int, s: str): argument 2 has unexpected type 'float'

已在pyqt5==5.15.1下成功启动


Small changes to repair existing functional tests

Made a few small changes to the functional tests (making larger ones as we speak).
Changes
Capitalise 's' to find the element we're looking for
Notification messages for completed digital services now refer to
thems 'Digital Services', not 'Digital services.'
Remove check for next link in service A-Z listing
Checking for a 'next' link on the supplier A-Z page doesn't really tell
us anything about the A-Z page itself.  If we don't have a 'Next, we might still
on an A-Z page.
Removed it because it seemed besides the point.
Refactor notification-selector to actually find an element
Also remove a redundant check.
Stop testing the G7 SSP stuff
G-Cloud-7 applications are long finished, so running the suite of
unit tests (which break by the way) over the G7 SSP seems a bit pointless.

:+1:


Bring build system up to date as much as possible

https://github.com/shana/webkit-sharp/commit/c7961d700d1d07956e1daf588c354a1d2fdf7e61 seems to be a "brown paper bag" commit - it broke things badly, and probably need to be reverted:
./FunnyBrowser.cs(190,42): error CS0246: The type or namespace name `TitleChangedArgs' could not be found. Are you missing an assembly reference?
./FunnyBrowser.cs(198,46): error CS0246: The type or namespace name `HoveringOverLinkArgs' could not be found. Are you missing an assembly reference?
./FunnyBrowser.cs(206,43): error CS0246: The type or namespace name `LoadCommittedArgs' could not be found. Are you missing an assembly reference?
./FunnyBrowser.cs(214,42): error CS0246: The type or namespace name `LoadFinishedArgs' could not be found. Are you missing an assembly reference?
Compilation failed: 4 error(s), 0 warnings
Makefile:625: recipe for target 'funnybrowser.exe' failed
make[1]: *** [funnybrowser.exe] Error 1

Once I revert that, the funny browser example builds, but fails at runtime with:
$ mono funnybrowser.exe
Marshaling notify::load-status signal
Exception in Gtk# callback delegate
  Note: Applications can use GLib.ExceptionManager.UnhandledException to handle the exception.
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.InvalidCastException: Specified cast is not valid.
  at FunnyBrowser.MainWindow.<CreateWebView>m__0 (System.Object o, WebKit.LoadStatusChangedArgs args) <0x4204f300 + 0x00044> in <filename unknown>:0

So I just quickly commented out the LoadStatusChanged delegate, and it runs fine.
The dom sample is badly broken and seems to be badly broken for a while

The runtime problem with load-status may be due to difference between the allerged api (1.3.10/1.2.5) vs the version of my local library, just in case it is important, webkitgtk-2.4.11 .


[sdl2-gfx] build failure

Host Environment

OS: [Ubuntu]
Compiler: 7.5.0
install-x64-linux-dbg-out.log

To Reproduce
Steps to reproduce the behavior:
./vcpkg install sdl2-gfx
Failure logs
buildtrees/sdl2-gfx/src/1.0.4-86ce4cf464/SDL2_rotozoom.c
/home/prafulla/vcpkg/buildtrees/sdl2-gfx/src/1.0.4-86ce4cf464/SDL2_rotozoom.c:31:10: fatal error: windows.h: No such file or directory
#include <windows.h>
^~~~~~~~~~~
compilation terminated.
Additional context
I just commented the lines in SDL2_rootzoom.c and the issue was resolved.

Hi @prafulla-kr
Thanks for posting this issue.
I have submitted the PR #10575 to fix it.

This issue should be fixed now. Please try to update vcpkg and rebuild it.


feat: add Jest matcher for nth specific Command

Fix #145

Hey, many thanks for doing the PR! That's, in fact, a useful matcher.
Could you add a unit test for the matcher failure? You can base it on the existing test. I think that should solve the coverage.
If not, let me know and I will do it and merge it.

Hey @m-radzikowski
I have added the missing unit test.

Thank you! Released in v2.1.0 🚀


Switch

Don't pass id as a prop, should be passed via ...props

@vlacher12 It's used to connect label and input via htmlFor={id}


parse json files in Assets/{platform}/{identifier}

new PR to replace #35

you know, i just had a thought. i wonder if it would ever be useful to identify the source of each asset? meaning, did it come from data.json in the Cores directory or from an instance json file
not entirely sure i can think of a usecase haha. but i guess its easy enough to add if i think of one


ModuleNotFoundError: No module named 'submodule'

I followed all your instructions but I still get the above error when I run the following command in Anaconda
python ./psmnet/submission.py --loadmodel ./psmnet/kitti_3d/finetune_300.tar --datapath ./media/sarim/7AF2CEB6F2CE7643/datasets/kitti/object/training/ --save_path ./media/sarim/7AF2CEB6F2CE7643/datasets/kitti/object/training/predict_disparity
The full error log is:
Traceback (most recent call last):
  File "./psmnet/submission.py", line 20, in <module>
    from models import *
  File "/home/sarim/pseudo_lidar/psmnet/models/__init__.py", line 1, in <module>
    from .basic import PSMNet as basic
  File "/home/sarim/pseudo_lidar/psmnet/models/basic.py", line 8, in <module>
    from submodule import *
ModuleNotFoundError: No module named 'submodule'

Hi, did you solve this problem?

Hi @Teeemio, could you check your python version? This code needs python 2.7

@mileyan @Teeemio You can fix this by using relative import. Nothing to do with python version.
Change to "from .submodule import *"


RegionalDR VP docs

Including docs and diagrams for the RegionalDR VP

This is an automated message:
You can preview this docs PR at http://481.docs-pr.validatedpatterns.io Note that they get generated every five minutes, so please wait a bit.

@mbaldessari @r2dedios
I'm sorry, but I haven't found time to review this yet. I want to start today. Do you think we can still use the preview site?

@mbaldessari Ready to review.
The "HTMLTest" actions is failing due to a redirect loop on "blog/2024-01-16-deploying-mcg-with-cisco-flashstack-portworx/index.html" Should we ignore it?

@mbaldessari Ready to review.
The "HTMLTest" actions is failing due to a redirect loop on "blog/2024-01-16-deploying-mcg-with-cisco-flashstack-portworx/index.html" Should we ignore it?

Yeah that was just a temporary fluke. You do need to drop the aliases: /agof/ though, that is wrong

Hi @mbaldessari I dropped the alias as you said, but still failing.. I'm not sure If I need to modify something else.
Thanks

@mbaldessari ?

Thank you guys!


Add Getdeck Beiboot to Adopters

This is lovely, I hate to ask but can you please re-commit with -s to make the DCO check pass

Sorry, as a CNCF project we can't merge PR's with a failing DCO check, if you're okay with it @Schille I can add this into a PR I'm working on now to get it added?

Sorry I didn't wait for your answer before merging, but I did give you a shoutout @Schille! thanks again for this, but I'll close this out for now

Thank you @cwayne18 I'm sorry, this slipped my mind.

No worries! Happens to me all the time 😄 thanks again for the addition!


[BUG (?)] 'boundary' condition unconsistent between e.g. grid.diff and grid.cumsum

What happened:
When I wrote the documentation about the boundary condition, I focused on the diff operator (see https://xgcm.readthedocs.io/en/latest/boundary_conditions.html or PR #325 ). What I understood was that the boundary condition is applied before the operation (i.e. it adds an extra phantom point before computing the operation). I checked the code today, and that is was it does for diff.
However, while working on #271 , I started some trials of an example using the grid.cumint (and grid.cumsum) to explain the metrics. But I was very surprised to not see the boundary conditions work as I though! In this case, the boundary condition is applied after having computed the cumsum.
If feels to me that it is an incoherent behavior.
What you expected to happen:
I expected that the boundary condition for grid.cumsum would be applied before the operation, so that having a fill value of e.g. 27.6 would then add 27.6 to all values (as if the integral would start with this value).
Proposed solutions:

Clarify this in the doc or,
The boundary condition needs to be applied in cumsum in these cases: (center to left, center to outer, right to center, inner to center). In all these cases, the physical meaning seems to always be boundary='fill', fill_value=0 if we look at cumsum as an integral. So the boundary condition could be removed for cumsum and cumint in favor of always filling with 0

I both cases I can work on a PR.
Minimal Complete Verifiable Example:
import xarray as xr
import numpy as np
from xgcm import Grid
N = 4
zC = xr.DataArray(np.arange(N)+0.5, dims=['zC'])
zF = xr.DataArray(np.arange(N), dims=['zF'])
tracer = xr.DataArray(np.arange(1, N + 1, dtype=np.float64)*10+5, dims=['zC'])
vel = xr.DataArray(np.ones(N,), dims=['zF'])
ds = xr.Dataset({'tracer':tracer, 'vel':vel}, coords={'zF':zF, 'zC':zC})
print('dataset: \n', ds, '\n\n')

grid = Grid(
    ds,
    periodic=False,
    coords={'Z':{'center':'zC', 'left':'zF'}},
)
print('grid: \n', grid, '\n\n')

print('** cumsum **')
print('fill value 0 \n', grid.cumsum(ds.tracer, 'Z', boundary='fill', fill_value=0), '\n')

print('fill value 27.6\n', grid.cumsum(ds.tracer, 'Z', boundary='fill', fill_value=27.6),'\n\n')

print('** diff **')
print('fill value 0 \n', grid.diff(ds.tracer, 'Z', boundary='fill', fill_value=0), '\n')

print('fill value 27.6\n', grid.diff(ds.tracer, 'Z', boundary='fill', fill_value=27.6))

Output:
dataset:
 <xarray.Dataset>
Dimensions:  (zC: 4, zF: 4)
Coordinates:
  * zF       (zF) int64 0 1 2 3
  * zC       (zC) float64 0.5 1.5 2.5 3.5
Data variables:
    tracer   (zC) float64 15.0 25.0 35.0 45.0
    vel      (zF) float64 1.0 1.0 1.0 1.0

grid:
 <xgcm.Grid>
Z Axis (not periodic, boundary=None):
  * center   zC --> left
  * left     zF --> center

** cumsum **
fill value 0
 <xarray.DataArray (zF: 4)>
array([ 0., 15., 40., 75.])
Coordinates:
  * zF       (zF) int64 0 1 2 3

fill value 27.6
 <xarray.DataArray (zF: 4)>
array([27.6, 15. , 40. , 75. ])
Coordinates:
  * zF       (zF) int64 0 1 2 3

** diff **
fill value 0
 <xarray.DataArray (zF: 4)>
array([15., 10., 10., 10.])
Coordinates:
  * zF       (zF) int64 0 1 2 3

fill value 27.6
 <xarray.DataArray (zF: 4)>
array([-12.6,  10. ,  10. ,  10. ])
Coordinates:
  * zF       (zF) int64 0 1 2 3

Anything else we need to know?:
xgcm version and environment:

0.6.1

Thanks for opening this issue @rcaneill. I agree this needs some discussion.
In my recollection, the idea is that cumsum should be the inverse of diff. So whatever order the operations are applied with diff, they should be applied in the opposite order for cumsum. For diff we have

pad
difference

So for cumsum, we should have

cumsum
pad

I believe that is what is implemented, e.g. in 0.7.0
https://github.com/xgcm/xgcm/blob/cd87c1153b7f213f5fd5da28fa00bb1e27ce6080/xgcm/gridops.py#L222-L229
However, I can see how the results from
grid.cumsum(ds.tracer, 'Z', boundary='fill', fill_value=27.6)

gave an unintuitive result--you were expecting the boundary value to propagate all the way through the cumsum. I'm curious if switching to pad_before_func=True fixes the issue. If so, perhaps there could be a flag to toggle this in cumsum?


Thoughts on traits selection in ValidateSetPropertiesByPath() and SetPropertiesByPath()

I suggest renaming includedPropertyTraits to propertyTraitsFilter and make it nullable (i.e. sequence<NcPropertyTrait>? propertyTraitsFilter).
Then describing propertyTraitsFilter as:
propertyTraitsFilter - If non-null, specifies that only role paths with the traits specified by this property are to be validated / restored.  If the parameter is null, the operation should proceed as if this property was set to "General", "Structural", "InstanceSpecific", and "DeviceGenerated" thus invoking a full restore.

We have since removed the concept of traits and made it the responsibility of a client to manipulate the backup data set for any intended use.


chore(BUILD): fix broken build (#3285)

TypeScript seemed to have an issue with XHR all of a sudden, I presume it's some TS update, quick fix any for now to fix the build
cherry-pick of #3285 to stable branch
/cc @chuckjaz

Coverage remained the same at 97.4% when pulling 372e2b54b4792956a343986a788d5838ef2df981 on alexeagle:stable into c009ac68b0e567eedb837a71858531593a8b207b on ReactiveX:stable.

FYI @kwonoj if Angular 6 ships with RxJS 5.5 we'd need to be able to compile it with TS 2.7, so we'd like to cherry-pick Ben's fix to the stable branch


Getting TypeError: stream.destroy is not a function when installing create-eth-app

I am on Windows 10.
Installed yarn globally, created a folder called createethapp
Opened it in VSCode.
Opened VSCode terminal (powershell) and entered the command
createethapp> yarn create eth-app my-eth-app
yarn create v1.22.19
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[4/4] Building fresh packages...

success Installed<EMAIL_ADDRESS>with binaries:
      - create-eth-app

Creating a new Ethereum-powered React app in D:\Knowledge\Blockchain\Projects_Solidity\createethapp\my-eth-app.

Downloading template files. This might take a moment.

Aborting installation.
Unexpected error. Please report it as a bug:
TypeError: stream.destroy is not a function
    at C:\Users\MYUSERNAME.COMPANY\AppData\Local\Yarn\Data\global\node_modules\create-eth-app\dist\index.js:32416:18
    at Array.forEach (<anonymous>)
    at C:\Users\MYUSERNAME.COMPANY\AppData\Local\Yarn\Data\global\node_modules\create-eth-app\dist\index.js:32414:18
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async Promise.all (index 0)

error Command failed.
Exit code: 1
Command: C:\Users\MYUSERNAME.COMPANY\AppData\Local\Yarn\bin\create-eth-app
Arguments: my-eth-app
Directory: D:\Knowledge\Blockchain\Projects_Solidity\createethapp
Output:

info Visit https://yarnpkg.com/en/docs/cli/create for documentation about this command.

What do I need to do.

Hi! Thanks for opening the issue. I'm afraid that this is a known issue: https://github.com/paulrberg/create-eth-app/issues/149.
What version of Node.js are you using?


Fixed tab-channel-group-sso-quickstart auth issues samples

Build Pipeline -
https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/7725593209


make approval mode dynamic instead of hardcoded

Signed-off-by: Ben Luzarraga<EMAIL_ADDRESS>

/retest


Does it support wss?

I am working on a project, which need to handle wss instead of ws. Do I need additional work for wss? Moreover, does Swamp 0.2.0 support WAMP 1.0 server?

I am using wss with no problems


Introduce a "preview" setting so that you can "opt into" a preview features for a given release

/area runtime

Describe the feature
There are times when preview features can be shipped so that people can try these out, give feedback and these can then either be changed, or entirely removed if they did not make sense. However preview features should not affect or be used in production scenarios. This feature request is to introduce a preview capability settings that requires you to explicitly opt-in to using preview features.
We could decide to have a preview setting on a per feature basis, which is good in that this allows you to be specific about your choice of preview feature and makes it explicit via a list of setting flag. If we choose to have a single flag for all preview features then there needs to be a way to determine the list of preview features in a release. Having this in documentation may be the best start, however if the list grows long and varies a CLI option may be better.
Release Note

RELEASE NOTE: ADDED a "preview" setting so that you can opt into preview features for a given release

Looks like I duplicated this here: https://github.com/dapr/dapr/issues/3048. There's a PR for it there as well.

closed as duplicate.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-26-main" 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 2570

Questions ?
Please refer to the Backport tool documentation


Add account setup special folder mapping screen

This is the special folder mapping feature.
The actual saving of the selected folders will be added in a follow up pull-request.
I'm very sorry for the size 😞 the next feature will be sliced into small pieces.
Depends on: #7272

Showing SpecialFoldersFormContent when at least one special folder is set to "None" is currently not working.

For the next feature, it would be good to just sit down and sketch the requirements. Would simplify things a lot.

Showing SpecialFoldersFormContent when at least one special folder is set to "None" is currently not working.

I tried and I could select none as special folder for one and all folders in any combination.

I tried and I could select none as special folder for one and all folders in any combination.

Apologies. Apparently, I a) misread mapToDefaultFolder() in SpecialFoldersFormStateMapper.kt and b) didn't test this with an unmodified branch. When changing mapToDefaultFolder() to return the first element of the list, the screen behaves like I though we agreed upon (see second screenshot). However, then the validation always succeeds and never shows the manual config screen even when some of the special roles have no folders assigned.
With an unmodified branch and a server where only some special folder have been set, we get this:

At first glance, this might seem reasonable. However, we can't tell whether a special folder was not assigned on purpose. We'd be punishing users who don't want to use a certain type of special folder and who configured their server accordingly. That's why we should default to this:

We still want to show the manual config screen because the unassigned special folders might also be a unconfigured/misconfigured server. But at least those who have done everything correctly can just hit the Next button and be done with the config instead of having to manually change from Please select to select None.
The only situation where the Please select default might be appropriate is when the server doesn't support the SPECIAL-USE extension to mark special folders at all. However, FolderFetcher currently doesn't communicate this property. And I don't think it's worth adding support for it. Most servers support SPECIAL-USE, just don't assign special roles (leaving us unable to distinguish "server misconfiguration" from "user deliberately chose not to use any special folders").

Also we are promoting none at the second position of the selection. This could suggest that this is a viable selection. Maybe we should move it to the end.

Not wanting to use e.g. an archive or spam folder is a valid and not that uncommon choice. If a user has a lot of folders, moving "None" to the bottom makes that option quite undiscoverable.

The retry button doesn't work. I created issue #7435 to keep track of this.


Failed to profile torchscript models

I tried to apply this nice profiler to torchscript models but it failed:
Traceback (most recent call last):
  File "/home/guikarist/projects/test-torch/profiler/test_flops_profile.py", line 48, in <module>
    flops, macs, params = get_model_profile(
  File "/home/guikarist/.miniconda3/envs/pytorch-2.0.1/lib/python3.9/site-packages/flops_profiler/profiler.py", line 1505, in get_model_profile
    prof.start_profile(ignore_list=ignore_modules)
  File "/home/guikarist/.miniconda3/envs/pytorch-2.0.1/lib/python3.9/site-packages/flops_profiler/profiler.py", line 152, in start_profile
    self.model.apply(
  File "/home/guikarist/.miniconda3/envs/pytorch-2.0.1/lib/python3.9/site-packages/torch/nn/modules/module.py", line 884, in apply
    module.apply(fn)
  File "/home/guikarist/.miniconda3/envs/pytorch-2.0.1/lib/python3.9/site-packages/torch/nn/modules/module.py", line 885, in apply
    fn(self)
  File "/home/guikarist/.miniconda3/envs/pytorch-2.0.1/lib/python3.9/site-packages/flops_profiler/profiler.py", line 104, in register_module_hooks
    module.__pre_hook_handle__ = module.register_forward_pre_hook(
  File "/home/guikarist/.miniconda3/envs/pytorch-2.0.1/lib/python3.9/site-packages/torch/jit/_script.py", line 942, in fail
    raise RuntimeError(name + " is not supported on ScriptModules")
RuntimeError: register_forward_pre_hook is not supported on ScriptModules

Is there anyway to solve the problem? This issue may be related.

FYI, register_forward_pre_hook post-scripted models has not been supported yet: https://github.com/pytorch/pytorch/issues/29037#issuecomment-792868448


Suggestion: Option to disable the 5 tick shield feature from the mod

It would be really nice to be able to disable the 5 tick shield feature from the mod as having it on actually makes it harder to disable shields

Added


Separate endpoint from processing step logic

Firs part of code for https://github.com/huggingface/datasets-server/issues/755
Since we are going to have two different processing steps for one same endpoint:

Changed the dependency between endpoint-cache_kind/job_type/name in ProcessingStep

Moved configuration for endpoints to api service, it should not exist in other layers only in api service

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Codecov Report
Base: 90.66% // Head: 88.80% // Decreases project coverage by -1.86% :warning:

Coverage data is based on head (4561afc) compared to base (f43a0d2).
Patch coverage: 91.62% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #817      +/-   ##
==========================================
- Coverage   90.66%   88.80%   -1.86%
==========================================
  Files         105       70      -35
  Lines        5292     2939    -2353
==========================================
- Hits         4798     2610    -2188
+ Misses        494      329     -165

Flag
Coverage Δ

jobs_mongodb_migration
80.29% <ø> (ø)

libs_libcommon
92.63% <92.24%> (-0.19%)
:arrow_down:

services_admin
86.76% <78.57%> (-0.09%)
:arrow_down:

services_api
89.26% <93.87%> (-0.02%)
:arrow_down:

services_worker
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

libs/libcommon/src/libcommon/config.py
77.96% <ø> (ø)

libs/libcommon/src/libcommon/simple_cache.py
97.54% <ø> (ø)

services/admin/src/admin/app.py
88.63% <ø> (ø)

services/admin/src/admin/routes/cancel_jobs.py
55.00% <0.00%> (ø)

services/admin/src/admin/routes/jobs_duration.py
55.00% <0.00%> (-2.90%)
:arrow_down:

libs/libcommon/src/libcommon/processing_graph.py
91.66% <50.00%> (ø)

services/api/src/api/utils.py
90.90% <66.66%> (-1.40%)
:arrow_down:

services/api/src/api/routes/endpoint.py
63.79% <83.33%> (ø)

libs/libcommon/src/libcommon/queue.py
94.54% <87.87%> (-1.54%)
:arrow_down:

libs/libcommon/tests/test_queue.py
98.83% <98.57%> (-0.55%)
:arrow_down:

... and 49 more

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

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

https://github.com/huggingface/datasets-server/pull/817#issuecomment-1431477692

created #835


Changed unnecessary int -> Long -> int conversion

merged to master and respectively branches!.

@marcosnils @HeartSaVioR
This PR may not be appropriate for 2.9 and 2.10. db is long in those branches.


No username/password cred info in Waston services

Hi,
I am trying this demo for Call of Code, I see we need to input username/password for Waston services, like:
  "CONVERSATION_API_URL": "",
  "CONVERSATION_API_USER": "",
  "CONVERSATION_API_PASSWORD": "",
  "CONVERSATION_CLASSIFICATION_WORKSPACE_ID": "",

  "NLU_API_USER": "",
  "NLU_API_PASSWORD": "",

  "TONE_ANALYZER_USER": "",
  "TONE_ANALYZER_PASSWORD": "",

But after I created Waston service in bluemix, and check their cred info, there is no username and password info any more:
I can only get this kind of info:

KEY NAME | DATE CREATED | ACTIONS |
-- | -- | -- | --
Auto-generated service credentials | Jul 19, 2018 - 04:08:43 | View credentials |
Credentials-1 | Jul 19, 2018 - 05:30:48 | View credentials |
{   "apikey": "xxx",   "iam_apikey_description": "Auto generated apikey during resource-key operation for Instance - crn:v1:bluemix:public:conversation:au-syd:a/xxx:xxx-74406a2b668b::",   "iam_apikey_name": "auto-generated-apikey-xxx",   "iam_role_crn": "crn:v1:bluemix:public:iam::::serviceRole:Manager",   "iam_serviceid_crn": "crn:v1:bluemix:public:iam-identity::a/xxx::serviceid:ServiceId-95f12db5-xxxx",   "url": "https://gateway-syd.watsonplatform.net/assistant/api" }

I tried to input iam_apikey_name as username and apikey as password, but after start the node app and check the tweet, I cannot see anything and the Waston service should not work.
Have you tried in the latest Waston service , or did I operate any wrong?
Thanks a lot!

I got the error message from cf logs --recent:
2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR Unhandled error for request POST /watson-conversation/send?access_token=50SUwZo9d6HyZo8IBxzWPKsUtD2xAcBXuuNqfnYcdDZgq8jcMj3opDK7aZG4Rsqh: Error: Unauthorized
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at Request._callback (/home/vcap/app/node_modules/watson-developer-cloud/lib/requestwrapper.js:87:15)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at Request.self.callback (/home/vcap/app/node_modules/watson-developer-cloud/node_modules/request/request.js:188:22)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at emitTwo (events.js:106:13)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at Request.emit (events.js:194:7)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at Request.<anonymous> (/home/vcap/app/node_modules/watson-developer-cloud/node_modules/request/request.js:1171:10)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at emitOne (events.js:96:13)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at Request.emit (events.js:191:7)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at IncomingMessage.<anonymous> (/home/vcap/app/node_modules/watson-developer-cloud/node_modules/request/request.js:1091:12)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at Object.onceWrapper (events.js:293:19)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at emitNone (events.js:91:20)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at IncomingMessage.emit (events.js:188:7)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at endReadableNT (_stream_readable.js:975:12)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at _combinedTickCallback (internal/process/next_tick.js:80:11)
   2018-07-19T17:26:19.86+0800 [APP/PROC/WEB/0] ERR     at process._tickCallback (internal/process/next_tick.js:104:9)

You are correct. The Watson service has changed from username/password to apikey. We are in the process of updating this code pattern to use the new apikey. Please watch for this issue to be fixed in the future.

Thanks! @scottdangelo , we are investigating this demo and maybe many other Call for Code users are trying it, hope it can be ready soon. Thanks! :)

@zhangtbj the code should work if you create credentials other than us-east or sydney region, which will give you username+password as oppose to IAM api key.

HI @sanjeevghimire Thanks for the help, yes! after I change to US-South, I can get username and password. Now my demo works fine, thanks!


Console device is relabeling 24-bit SGR color values if they match a 16-color primary in the Windows color scheme.

Environment
Windows build number: 10.0.18363.0
Windows Terminal version (if applicable):

Any other software?

Steps to reproduce
Various example means of triggering are shown in issues libuv/libuv#2766 and mintty/mintty#981
Expected behavior
Truecolor ANSI SGR codes should never be relabeled as 16-color primary codes. Not all downstream consumers use colormaps that will render these values equivalently. (In fact, many environments have chosen markedly different RGB values for their SGR primaries.ref)
Actual behavior
All 24-bit SGR color sequences are remapped to shorter 16-color SGR primaries if the 24-bit RGB intensity values happen to match a primary’s RGB intensity values in the Windows color scheme.
For example, the value \e[48;2;118;118;118m gets relabeled \e[100m because the RGB intensities that Windows uses for "bright black" happen to match those given by the somewhat longer 24-bit ANSI string. But not all environments map "bright black" to the same 24-bit RGB intensity values; see screenshots in the mintty issue, referenced above.
Note: this applies to 8-bit SGR codes, too, if the canonical RGB values for the 8-bit color match what Windows has chosen for one of their primaries: \e[48;5;243m => \e[100m

Thanks for the suggestion! This is actually already being tracked by another issue on our repo - please refer to #2661 for more discussion.
/dup #2661


RFE: night mode

my eyes!

Suggestion: Maybe easier to have a script that detects the system color-scheme and change the Ui automagically by default... something like this. However, the functionality can/should also be customised on a User Preferences/Settings

Good idea, we could certainly add a setting for that to the database.

In the meantimes, there is browser extensions that can do that like Dark Reader.

I would like to bump this issue. It would be really nice to see this added. This would decrease the number of mosquitos attracted by my monitor at night by a lot!


[sdk] Support React Native

Issue To Be Solved
The latest @onflow/sdk is currently unusable by React Native apps. Its required @onflow/send uses the gRPC NodeHttpTransport which does not work with React Native.
SDK calls like sdk.getLatestBlock() fail with the error:
[Unhandled promise rejection: Error: Response closed without grpc-status (Headers only)]
- node_modules/@onflow/send/dist/send.js:1:315 in n.grpc.unary$argument_1.onEnd
- node_modules/@improbable-eng/grpc-web/dist/grpc-web-client.umd.js:1:25082 in t.unary
[...]

Using versions:

@onflow/sdk 0.0.28
@onflow/send 0.0.15
flow version 0.6.0

Suggest A Solution
Allow optional use of the gRPC ReactNativeTransport.
Example commit here showing how we got @onflow/sdk calls working from a React Native app. That hardcodes the ReactNativeTransport but it should probably be a parameter somewhere.
The primary change, from packages/send/src/unary.js:
+ import {ReactNativeTransport} from "@improbable-eng/grpc-web-react-native-transport"

- grpc.setDefaultTransport(NodeHttpTransport())
+ grpc.setDefaultTransport(ReactNativeTransport({}))

All the SDK calls from the example app (getAccount, getEvents, getLatestBlock, getTransactionStatus, ping, script, transaction) worked fine via RN using the above change, though more thorough testing will be needed to identify other potential issues from using RN.
I'm happy to submit a PR for the above if helpful. Open to suggestions on where to put the RN flag, or other considerations re React Native integration unsolved by the above.

@Entryist Good news on this front, we are working at removing gRPC completely from the JS-SDK, and will instead be using standard http requests, so this and an entire other suite of issues should go away naturally. I can't give any promises as to timelines, but I can say it has a fairly high priority from the Access Node side of things and once its an available transport option for us, it will be our highest priority.

I am trying to listen to a flow smart contract event from my Node.js Application.
This is my code for getting the said events:
const getEvents = async (params) => {
  // Define event type from params
  const { contractAddress, contractName, eventName } = params;
  const eventType = `A.${contractAddress}.${contractName}.${eventName}`;

  const { from = 0, to } = params;
  let toBlock;
  if (to === undefined) {
    // Get latest block
    const blockResponse = await fcl.send(
      await sdk.build([sdk.getLatestBlock()])
    );
    toBlock = blockResponse.latestBlock.height;
  } else {
    toBlock = to;
  }

  const response = await fcl.send(
    await sdk.build([sdk.getEvents(eventType, from, toBlock)])
  );

  // Return a list of events
  return response.events;
};

However, I am having this issue.
Error: Response closed without headers

I am using these dependency versions:
@onflow/sdk 0.0.44
@onflow/send 0.0.35
@onflow/fcl 0.0.67
Please help.

I am trying to listen to a flow smart contract event from my Node.js Application.
This is my code for getting the said events:
const getEvents = async (params) => {
  // Define event type from params
  const { contractAddress, contractName, eventName } = params;
  const eventType = `A.${contractAddress}.${contractName}.${eventName}`;

  const { from = 0, to } = params;
  let toBlock;
  if (to === undefined) {
    // Get latest block
    const blockResponse = await fcl.send(
      await sdk.build([sdk.getLatestBlock()])
    );
    toBlock = blockResponse.latestBlock.height;
  } else {
    toBlock = to;
  }

  const response = await fcl.send(
    await sdk.build([sdk.getEvents(eventType, from, toBlock)])
  );

  // Return a list of events
  return response.events;
};

However, I am having this issue.
Error: Response closed without headers

I am using these dependency versions:
@onflow/sdk 0.0.44
@onflow/send 0.0.35
@onflow/fcl 0.0.67
Please help.

House keeping, tagging and closing in favour of: #321

Same issue like @swarlezbarkley here. How did you solve it?


öäü Domain not found in the scanning IIS Site Bindings list.

If present in Domain Name umlauten ( üöä ) they will not appear in the list . e.g. üöä.com

Let's Encrypt does not support Internationalized Domain Names (IDN). They were hidden in the tool in #13.

If your host is: www.üöä.com and you add www.üöä.com as an alias (eg See here for a translator) will it work ? Would you have to remove the IDN first?

That depends on how you run it.
If you use manual mode, then no, as it only knows what you tell it. However there is no renewal support for manual right now and it won't install it in IIS for you.
If you run it for --san , then you would have to remove it as San requires every hostname binding to pass authorization or it won't continue.
If you run it without arguments, then you select what binding to generate a cert for, so you just don't select any IDN bindings.

I don't know if it would work even with the IDN characters removed. If someone has an IDN and wants to try it with it translated, please let us know if it works.

IDN support should start in about 2 months:
https://letsencrypt.org/upcoming-features/
Maybe you can just wait until it's implemented, don't know if the effort is worth it.

I'm going to close this issue for now. If someone wants IDN support after Let's Encrypt supports it, please open up an issue for that as an enhancement then.

Hi!
I have this Issue, IIS doesn't allow puny code in bindings (only real domain name), but Lets encrypt Win simple doesnt list åäö .


Fix Cardano CLI tests on MacOS:

I've confirmed the tests work on my Mac.

Superseded by https://github.com/input-output-hk/cardano-node/pull/1410


KAFKA-14492: Extract a method to create LogManager, in order to be overrided by subclass of KafkaServer

KAFKA-14492
In our scene, we want to implement a subclass of LogManager, add do some interception like encrypting/auditing.
Even more, we want enhance the write log performance in the future.
Split alone a create method can provide an extension point to allow user define the action from outside kafka core.
Base kafka, entend it, rather than modification.

hi, those failed checks are not caused by my modification in this time.
what can i do next ?

Thanks for the PR. It is important to note that KafkaServer is not considered as a public interface in Kafka. Hence, there is not guarantee that the interface will remain like this in the future. It can basically change anytime. Therefore, I am personally not comfortable with this change. Have you considered alternative approaches? @ijuma What do you think?

Yes, a KIP would be required to turn this into a public API. For non public APIs, the code can change at any time.

Thanks for your reply.
I agree that the KafkaServer is not designed for extend, attempt to replace the LogManager by override a createLogManager, maybe not a permanently method.
As your said, a KIP is needed to propose a complete approache that specially focus on which components can be extended in KafkaServer. Design an interface describe which components support extend is obviously a better way.
Before an api or interface be provided, allow to extend an object's method properly can be a temporary way with less drawback. At least is better than that fork a standalone repository, then modify with the code. The extend way is just add a kafka-core.jar, then override inside classes.
Except the extend side, provide a createLogManager method is coincide with createReplicaManager below those code. It's also a good style only by the readability.
Further on, only this modification is negligible, we could meet more problems in the way to extend LogManager. In that time, i am willing to provide a comprehensive plan as a KIP to resolve the extension problem.
Thank you both again.


JAVA-3328: Excessive memory allocation due to unwrapping of updates in bulk ops

Change removes check and call to isEmpty() which causes an unnecessary unwrapping.
IT has been relaxed to allow for server-side failure for this case.

@astral303 we don't want to relax the check so ended up going with a BsonWriter decorator approach that you tried initially.  There is a problem with pipe() as you recognized, but it's rare that pipe() will be used in this context so better to check in most cases and miss the ones where pipe() is used than not to check at all.


Partial fix to landsat example; selected band even when only 1

I can't tell what changed to require this PR, but the landsat example is broken in current master. From the error messages, it looks like xarray used to ignore the band dimension when it has only a single value in a TIFF file, but now we have to explicitly select the value for that dimension if we want to get down to an x,y array.
Still WIP, because the orientation of the image is now completely wrong.  It should be like:

But instead it's:

@jbcrail and @philippjfr , any idea why these changes were needed?  Any idea why the rotation is mixed up now (could it be a side effect of my "fixes"?)? Something to do with the new rasterio backend for xarray? The new regridding code? Any idea how to get it working?

Superseded by #453.


what is the FLOPs and parameters of your model?

The proposed AiATrack has 15.79M model parameters and 9.43G FLOPs model complexity according to the THOP tool (https://github.com/Lyken17/pytorch-OpCounter). If you want to verify it in this project, please refer to the open-source code of STARK (https://github.com/researchmm/Stark) as a quick reference.


[1.10.2] Randomly generated dimension void with gravel and sand

As stated, I just randomly generated a dimension (blank dimension card in a builder), upon entering it around 29k sand and gravel (EntityFallingBlock) blocks started falling towards the void.
This caused the tickrate of the server to fall down to >1 (0.22 TPS was the last I noticed before nuking the dimension).
This is now the 3rd time this has happened on our server in a relatively short time, is there a way to blacklist sand and gravel on any dimensions that contain void in any way?
Modpack: Age of Engineering 1.0.4
RFTools Dimensions version: rftoolsdim-1.1x-5.04

Assuming it's feasible from a coding perspective a config option to replace gravity affected blocks with the inverted world equivalents might be a suitable way to get around it, generate levarG and dnaS instead of Gravel/Sand

BTW. Please report dimension problems to the right mod (RFTools Dimensions is a separate mod)

As to this problem. What kind of world was it? The issue is that gravel and sand can be generated by biomes. Say you have a floating islands world and it is a desert biome then there is a big chance that many islands will be made out of sand. There is nothing that I can do about that as that is done by the biomes. Replacing with the 'non-gravity' versions is really not an option as those are really different blocks

Seemed to me to be a plain void world, no terrain generation, with a random feature deciding to be sand/gravel.

Can you show me the dimension info? (sneak-right click with the dimension tab in your hand)

The dimension in question was nuked (safedel), and I'm quite sure I don't have the dimension tab anymore at hand, though I'm unable to check currently as I'm away from home.
Will see what I can do once I get back

Nope, don't have the dimension tab anymore :(

Idea: add a post-generation option to custom dimensions so that if sand/gravel have a block of air below them and a sand/gravel above them (should match all these free-floating sand spheres and gravel islands), they are replaced with sandstone/stone respectively. Or, with a special Static Sand/Gravel blocks added by the mod for this specific purpose - they are like their respective counterparts in every other aspect - texture, type, mining level - drop them when mined, just do not fall. Hey, they could even be automatically generated to support similar blocks from other mods. This should definitely help combat the lag associated with collapsing blocks like these two.

Scanning all generated chunks for this would add some performance penalties though. i.e. it would slow down chunk generation a bit

Some performance penalties would still be better than a server death from lag. In addition, this option could be limited to specific generation settings, something along the lines of (("FallingBlock Sphere"||"FallingBlock Scattered Sphere")&&(Type!=Solid)||(Type==Solid && Material==FallingBlock)||...). Given that there is a quite small list of existing features and terrain generation options, this predicate could be easily fine-tuned.
Also, spheres, tendrils and the like could be generating supports themselves, further reducing the need for a complete chunk scan.

Well falling blocks like sand are already excluded from worldgen. Except that I have no control over biomes. So in practice these falling blocks can occur anywhere

Thank you for taking the time to create this issue, as we're no longer supporting versions below 1.12, I'm going to go ahead and close it but if this bug seems to continue over to newer versions, please do report it again so we can investigate the issue further.


protoc-gen-go-grpc: program not found or is not executable when using APIv2

I've been searching #1053 and #3453 searching, finally I am here. When trying to $ make build I get the error:

protoc -I . speak.proto --go_out=plugins=grpc:.
protoc-gen-go: program not found or is not executable
Please specify a program using absolute path or make sure the program is available in your PATH system variable
--go_out: protoc-gen-go: Plugin failed with status code 1.
make: *** [build] Error 1

I have tried following and installing and going on and on but can't seem to get this error to go away.
my make file is:
build:
	protoc -I . speak.proto --go_out=plugins=grpc:.

I have tried installing with
go get google.golang.org/grpc/cmd/protoc-gen-go-grpc
go install google.golang.org/grpc/cmd/protoc-gen-go-grpc

and my paths look like
export GOPATH=$HOME/Documents/goworkspace
export PATH=$PATH:$HOME/go/bin
export PATH=$PATH:/usr/local/go/bin

I'm not sure if that's correct as there have been so many conflicting responses as to how it should be set.
What else am I missing? I have sourced my .zshrc, restarted my terminal, and restarted my laptop but to no avail

Executing go get google.golang.org/grpc/cmd/protoc-gen-go-grpc installs protoc-gen-go-grpc, not protoc-gen-go.
Perhaps you meant go get google.golang.org/protobuf/cmd/protoc-gen-go instead?

I was looking at that, and there have been very recent changes to the protobuf package. I have looked at the error and have about 20 tabs open with the same issue and 20 different tabs saying 20 different things. I have already go get google.golang.org/grpc/cmd/protoc-gen-go-grpc as well as done go install google.golang.org/protobuf/cmd/protoc-gen-go
same issue persists

If you already ran go install google.golang.org/protobuf/cmd/protoc-gen-go and protoc still can't find the command, then the issues likely due to the PATH environment variable not set up.

 Thanks a bunch!!!  \ʕ•ᴥ•ʔ/

run :
PATH="${PATH}:${HOME}/go/bin"

RUN :-
echo 'export PATH="${PATH}:${HOME}/go/bin"' >> ~/.bashrc
source ~/.bashrc

Why are we adding gopath if were using go mod?

Will share here, maybe it will help someone: if you install go with asdf, don't forget to run asdf reshim after installing go program. Sometimes the issue, that bin is there and all envs set up correctly, but installed bin still not visible.


Add NSSP data as an endpoint

https://covid.cdc.gov/covid-data-tracker/#ed-visits

Nice! This particular link looks like it would be compatible with the covidcast schema, but NSSP provides way, way more than that through ESSENCE. Are you just looking for the COVID-related ED visits, or something more detailed that would require its own endpoint?

Ah sorry, by "endpoint" I mean the following: Each endpoint of the API returns data in a different format. For example, the covidcast endpoint is largely designed for sample-based estimates, and includes only the value, stdev, and sample size in each row returned; by contrast, the covid_hosp endpoint returns over 60 different fields in each row, most of them raw counts; the fluview endpoint returns 5 different fields plus an age-stratified count.
Making a new endpoint means we can support whatever output format we want, but you have to use the delphi_epidata client to access the data, which is rather bare-bones at the moment. Adding it into the covidcast endpoint restricts the output format, but you can use the covidcast client libraries to access the data, which do things like automatically format the results as a data frame, plot choropleth maps, and compute correlation analyses.
If you need a small number of time series (like <5?) or you don't know exactly which ones you want yet, then adding this into covidcast is the most expedient.
So far I have:

COVID-related ED visits
Influenza-related ED visits

Anything else?

TL;DR: I can't find any regularly updated NSSP ILI or influenza time series data.  That would be an issue for potential usage for flu forecasting.
There are multiple ways that NSSP data extracts are currently published; here are three:

State&agegroup-level (linked above): diagnosed COVID-19, reference dates from 2020-06-16, regular version history from WebArchive from 2020-08-24, seems to be regularly updated
National+HHS-level: CLI and ILI, reference weeks from 201940, spotty version history from WebArchive from 2021-04-11, maybe because the time series ends with 2020w14 (2020, not 2021) and isn't regularly updated
National? via MMWR: COVID-19 and influenza, reference times from Jun 2018, maybe just a one-off image graphic rather than something regularly scrape-able

The state&agegroup-level "diagnosed COVID-19" series is the only one I have seen with ongoing updates.  If this is to be used for ILI situational awareness or forecasting, there will need to be regularly published/shared ILI time series as well.

This may be achieved, at least partially, by https://github.com/cmu-delphi/covidcast-indicators/pull/1952


Iter kmers performance

100% speedup for optimized path (ie no positional metadata) when combined with #976.
@jairideout @ebolyen ready for review

@anderspitman Done reviewing looks good just a stylistic comment.

Can you add a few test cases for the optimized path? All of the unit tests but one have sequences with positional metadata (taking the unoptimized path).

@anderspitman a couple of comments and then this is ready to go!

I just duplicated the normal tests for iter_kmers. Didn't have time for anything fancier. Should be able to reconcile once we have strided pandas hopefully.

Ready for another review

Thanks @anderspitman! This is ready for merge once tests pass (there are pep8 failures currently).

Hopefully good to go now.

@gregcaporaso, just FYI, the performant kmers stuff we discussed yesterday did make it in

ok, thanks!


Review module structure and class names

Since we are preparing for a "full" release, now is the chance to once again review the module structure and / or class and method names.
In particular, we should review the class names added since the last beta release.
I would tend to think some of the enum class names could use being made a bit more specific.

@greschd , here are my inputs
Proposed changes

[ ] UnitType instead of DimensionType
[ ] Unify names: should we use Cutoff or CutOff, Dropoff or DropOff. My proposal is CutOff and DropOff...

Other remarks

[ ] Remove ConnectLaunchConfig from ansys.acp.core?


bug:  linux wheel Import Error

Describe the bug
The package installed from pypi causes the following erros:
Python 3.11.2 (main, Mar 23 2023, 17:12:29) [GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from whispercpp import Whisper
>>>
>>> w = Whisper.from_pretrained("tiny.en")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.11/site-packages/whispercpp/__init__.py", line 105, in from_pretrained
    context = api.Context.from_file(
              ^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/whispercpp/utils.py", line 144, in __getattr__
    self._module = self._load()
                   ^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/whispercpp/utils.py", line 122, in _load
    module = importlib.import_module(self.__name__)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1206, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 676, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 573, in module_from_spec
  File "<frozen importlib._bootstrap_external>", line 1233, in create_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
ImportError: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /usr/local/lib/python3.11/site-packages/whispercpp/api_cpp2py_export.so)

Then I rebuild the package using the latest source, the program creashed.
Python 3.11.2 (main, Mar 23 2023, 17:12:29) [GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from whispercpp import Whisper
>>>
>>> w = Whisper.from_pretrained("tiny.en")
Illegal instruction (core dumped)

To reproduce

docker run -i -t python:3 bash
pip install whispercpp

root@495bbb253133:/# pip install whispercpp
Collecting whispercpp
  Downloading whispercpp-0.0.17-cp311-cp311-manylinux2014_x86_64.whl (1.5 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.5/1.5 MB 2.5 MB/s eta 0:00:00
Installing collected packages: whispercpp
Successfully installed whispercpp-0.0.17

Run codes in readme.
Rebuild and reinstall package using python3 -m build -w
Run codes in readme.

Expected behavior
No response
Environment
python: 3.11
platform: x86-64 Debian GNU/Linux 11 (bullseye)

Had that one too. This bug is not caused by the bindings but by whisper.cpp I suppose. It is simply linked to your Distro-Version being outdated by now. I had this first on Linux Mint Una, it was fixed by upgrading to 21.

Had that one too. This bug is not caused by the bindings but by whisper.cpp I suppose. It is simply linked to your Distro-Version being outdated by now. I had this first on Linux Mint Una, it was fixed by upgrading to 21.

Hello,what does the 21 version here refer to.

The linux mint version 21, i.e. Vanessa. This bug should relate to some .so files being outdated. You could either manually upgrade this or dump your distro.

I had the same problem. Whenever I attempted loading a model - I got "Illegal instruction (core dumped)".
Here I found the the issue could be in the CPU and it worked for me.
I got this error on a server with Xeon E5-2687W CPU (manufactured back in 2012)
When I changed it to AMD Ryzen Threadripper PRO 5955WX (2022) - all worked fine.
If that doesn't help you, I also found that it could be a bug with numpy, try downgrading in to 1.19.4


Add centralized logging proposal

Proposal for initial implementation of centralized logging deployed by kismatic.

Some discussion out of band around log rotation: We came to the conclusion that we don't have to expose the log rotation settings given that the logs are being shipped out to a central store.


Added 2darraySprialFashion.cpp

Added '2darraySpiralFashion.cpp' inside 2d array folder of C++.
Refer to issue #469

Thank you

Please merge it to the main branch.


Second attempt This is a great course

Originally posted by @demetrius99 in https://github.com/demetrius99/github-slideshow/issues/1#issuecomment-445573711

We are good to go

thank you for all your contributions!

Hi there we are re-opening this issue

Hi there we are re-opening this issue

ok great what do you need help with?

[x] @mentions, #refs, links, formatting, and tags supported
[x] list syntax required (any unordered or ordered list supported)
[x] this is a complete item
[ ] this is an incomplete item

thank you for all your contributions!

welcome


Can't run Three-DXF

Getting this error trying to run this in Angular, tried installing types but like other posts had no success.
 You may need an appropriate loader to handle this file type.
|
| 		this.texture = source.texture.clone();
| 		this.texture.image = { ...this.texture.image }; // See #20328.
|
| 		this.depthBuffer = source.depthBuffer;
 @ ./~/three-dxf-loader/dist/three-dxf-loader.js 16:82-98
 @ ./src/app/dxf-viewer/dxf-viewer.component.ts
 @ ./src/app/app.module.ts
 @ ./src/main.ts

Here's the TS file I'm calling it in:
import { Component, OnInit } from '@angular/core';
// import * as THREE from 'three'
import { ThreeDxf } from 'three-dxf-loader'
import DxfParser from 'dxf-parser'

@Component({
  selector: 'app-dxf-viewer',
  templateUrl: './dxf-viewer.component.html',
  styleUrls: ['./dxf-viewer.component.css']
})
export class DxfViewerComponent implements OnInit {
  private file: Blob;
  public dxf;
  public cadCanvas;
  constructor() { }

  ngOnInit() {
  }

  public handleFileInput(fileInput) {
    this.file = fileInput.item(0) as Blob;
    console.log(this.file instanceof Blob);
    let fileReader = new FileReader();
    fileReader.onload = function(e) {
      const parser = new DxfParser();
      try {
        let dxf = parser.parseSync(e.target.result);
        return dxf
      } catch (err) {
        return console.error(err.stack);
      }
    }
    this.dxf = fileReader.readAsText(this.file);
  }

  public renderDxf() {
    // var font;
    // var loader = new THREE.FontLoader();
    // loader.load( 'fonts/helvetiker_regular.typeface.json', function ( response ) {
    //     font = response;
    // this.cadCanvas = new ThreeDxf.Viewer(this.dxf, document.getElementById('cad-view'), 1000, 800);
    // });
  this.cadCanvas = new ThreeDxf.Viewer(this.dxf, document.getElementById('cad-view'), 400, 400);

  }

}

Any tips on what I should do? Thanks!

Sorry, TypeScript definitions are next on my todo list, but I haven't had the time to get to it yet. You'll need to somehow manually define them I believe, but I don't know myself how to do that yet.

Great, thanks for the response! If I'm not mistaken, React is the stack where this would work?

Sorry, I thought this was more about TS definitions. I realize now you are using another lib, probably forked from this one called three-dxf-loader. I dunno what exactly that lib is doing different so I won't be much help.

I see, so I'll try to go with three-dxf for now. Is React supported? Or do you recommend I just go with js?

Awesome, thanks for all your help!


[HUDI-1984] Support independent flink hudi compaction function

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
Support independent flink hudi compaction function, which can do hudi compaction by a independent flink task.
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3046 (74c1610) into master (08464a6) will increase coverage by 13.18%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #3046       +/-   ##
=============================================
+ Coverage     49.86%   63.04%   +13.18%
+ Complexity     3527      346     -3181
=============================================
  Files           488       54      -434
  Lines         23618     2016    -21602
  Branches       2528      241     -2287
=============================================
- Hits          11777     1271    -10506
+ Misses        10802      621    -10181
+ Partials       1039      124      -915

Flag
Coverage Δ

hudicli
?

hudiclient
?

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
?

huditimelineservice
?

hudiutilities
63.04% <ø> (+53.76%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ies/exception/HoodieSnapshotExporterException.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/hudi/utilities/HoodieSnapshotExporter.java
5.17% <0.00%> (-78.45%)
:arrow_down:

...hudi/utilities/schema/JdbcbasedSchemaProvider.java
0.00% <0.00%> (-72.23%)
:arrow_down:

...he/hudi/utilities/transform/AWSDmsTransformer.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...ache/hudi/exception/MetadataNotFoundException.java

...rg/apache/hudi/io/storage/HoodieParquetReader.java

...common/table/log/HoodieMergedLogRecordScanner.java

...udi/timeline/service/handlers/TimelineHandler.java

...n/java/org/apache/hudi/cli/HoodieSplashScreen.java

.../apache/hudi/common/model/CompactionOperation.java

... and 463 more


features and classifier

This task requires a top-n classification (multi-classes) because taking the top class is not sufficient due to the complex nature of the sounds (spectrums). You can have an environment where multiple sounds can be mixed (which is technically a convolution ) .

This issue should aim at increasing the dimension of the features that we extract from the dataset (see if we can use other techniques).

Add a multi-class top-n classifier to the model instead of a top-1.

The loss converges! now let's aim for performance :)


fix(web-sdk): include pathname for page id

Why
I found a potential fix for https://github.com/grafana/faro-web-sdk/issues/702
As I could not find the code that actually propagates the id in the first place (I assume it's done in Grafana?)
What
I included a default value for the parameter id of the page meta object.
Links
https://github.com/grafana/faro-web-sdk/issues/702
Checklist

[ ] Tests added
[ ] Changelog updated
[ ] Documentation updated

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @undefinedhuman thank you so much for the PR but unfortunately we can't solve it that way.
The pageID set in Faro overrides the pageID creation which is done in the receiver.
Even when using location.pathname we can end up with long  paths for exampe which may contain "gibberish" which can't be shortened when overridden via Faro.
A shallow description how the page_id parser's gibberish detection works:
The gibberish detector is based on a likeliness of two letters being next to each other in western European languages, so for example "abc" is gibberish because it doesn't happen often in words.
Note:
In a future release we want make it easier to override page_ids in Faro but it needs some sync and internal discussion before.
Cheers and thank you so much again.
Marco

Sounds good - looking forward! And thanks a lot for the explanation!