Proposal: NFT Staking Platform

Description:
NFT Staking is dedicated to nurturing and expanding the blockchain community by providing a secure, accessible platform for users to engage with digital assets.
Bounty track

Link github: https://github.com/olym3labs/non-custodial_nft_staking_platform
Demo: https://non-custodial-nft-staking-v7d-baobap.vercel.app/

Link github: https://github.com/olym3labs/non-custodial_nft_staking_platform
Demo: https://non-custodial-nft-staking-v7d-baobap.vercel.app/

Wow, very cool. In general, your contract is quite good and runs stably.
I think I can assist you with some more smart contract issues.
I see that you give users an OLB token every time they stake an NFT. But I really haven't seen the claim token function in the NONCUSTODIALNFTSTAKING contract, but the claim function is in the OLYM3LABSTOKEN contract.
For example, in case a user wants to claim the token reward but they do not want to unstake their NFT, do you have any solution? I think you should make more improvements here,

@FucktheKingcode That's a good idea for a feature addition. Please see this feature in the next updated version.

@FucktheKingcode implement it to win a bounty


Denial of Service via Arbitrary Allocation

There are many places containing codes pattern: read length then allocate array or map, like:
count, err := leb128.ReadVarUint32(r)
if err != nil {
	return err
}
s.Bodies = make([]FunctionBody, count)

So the attacker can construct a invalid wasm file, set the count to 2^32-1, causing the Wagon interpreter to allocate more memory than is available and irrecoverably crash the entire process.
We may need to change it to something like:
initialCap := min(count, maxInitialCap)
s.Bodies = make([]FunctionBody, 0, initialCap)
....
s.Bodies = append(s.Bodies, body)

good point.
perhaps a helper function like this?
func readCount(r io.Reader, cap int) (int, error) { ... }

Since the Reader is a stream, so theoretically it could be very large, until we read it, if encountering eof be we read out count values, then we can say it is invalid. Although adding a max size limit works, but it is cumbersome, since different use cases may have different value, it will end up with a lot of limit parameters the user need to fill in (like max function count, max string size etc).
The problem is not caused by the count, but the call to make([]Type, count), we just need to replace that from:
arr := make([]Value, count)
for i:=0; i< count; i++ {
    val := read and decode Value
   arr[i] = val
}

to :
var arr []Value // or initialCap := min(count, maxInitialCap); arr := make([]Value, 0, initialCap) to avoid some reallocation
for i:=0; i< count; i++ {
    val := read and decode Value // if the count is too large we will encounter eof here finally.
   arr = append(arr, val)
}


Can one determine whether heap is normal or large somehow from installed release [Windows]

I need to determine whether installed release is of normal- or large-heap type.
Is it possible somehow, for instance, from a release file?

@pshipton Do you know of any option? If not, would probably a good idea to the version output.

You can't tell from the release file output. The -version out does indicate "Compressed References" or not. The difference in the build is the directory structure contains either a compressedrefs directory or a default directory (default for 32-bit). Where exactly that is in the structure depends on whether it's Java 8 or Java 11+, and whether it's Windows or some other platform. On Windows the directory is in the jre/bin directory, or in bin if there is no jre directory.
You could open an issue at https://github.com/eclipse/openj9/issues to see about adding something to the release file, but note the OpenJ9 project is actively working on not having a separate large heap build. It's functional but not yet performant.

Thank you very much! This is enough for my task.

@pshipton  Still need one more clarification - does default for 32-bit tells that it's normal heap?
i.e., the algorithm looks like:
if x64 => presence of  compressedrefs = normal heap; default = large heap
if x32 => presence of  default = normal heap

Correct, there is only one type for 32-bit.

@pshipton Thank you!


Unable to assign current timestamp and sequenceID during live capturing.

Brief description:
Using pyshark, we are able to capture the live logs. But here, current timestamp and sequenceID parameter values are getting assigned inappropriately from server side to the client side, due to the delay of live capturing.
Server side, we are running gptp daemon_cl.
Client side, we are pumping the dummy packets.
Environment:
Scapy version: 2.4.0
Python version: 3.6.6
Operating System: Ubuntu18.04
Expected Result:
Server side timestamp and sequenceID should match with the Client side timestamp and sequenceID parameters

I'm sorry, I don't quite understand what you are expecting and which timestamps you are referring to. It might help to include a packet capture along with a detailed description of how it is different from what you expected. Please also let us know what NICs you are using on your gPTP endpoints and what switches, if any, are between them. It might also help to give some context as to why you are trying to capture these logs and what you are hoping to learn or verify with them.


Visual Script – Godot 4 – AutoConstruct int to string not working

Godot version
v4.0.dev.calinou [6d98f84ab]
System information
Ubuntu 20.10/Atermiter dual Xeon motherboard, gtx 970 nvidia package 460
Issue description
When hooking a non string type to a set text label the autoconstructor does not appear. See attached GIFs for current vs expected behavior.

Steps to reproduce
Drag a connection from a variable output to the connection on a set text label and notice the auto constructor does not appear.
Minimal reproduction project
Will create if requested.

VisualScript is removed in 4.0, closing.


Set DNS trusted servers using FelixConfig instead of FELIX_ env var

For Rancher and OpenShift the operator needs to set a non-default value for DNS trusted servers, and it was previously doing that by setting a FELIX_ env var in the node daemonset.  But that makes it impossible for any further tweaks to that using FelixConfiguration - because FelixConfiguration settings are always overridden by FELIX_ env vars.
To allow later tweaks, change the operator to create or patch FelixConfiguration instead.
References:
https://tigera.atlassian.net/browse/RS-897
https://tigera.atlassian.net/browse/EV-3193
Description

For PR author

[x] Tests for change.
[x] If changing pkg/apis/, run make gen-files
[x] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[x] Milestone set according to targeted release.
[x] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

I think the CI failure is unrelated.  Will wait for review feedback, in case there are other issues to address, before hitting Rerun.


Enable parallelism for cmake generated projects building on windows platforms.

It looks like improved tools exist for parallel building that an be enabled at the project level: https://devblogs.microsoft.com/cppblog/improved-parallelism-in-msbuild/
I'm opening this issue to track suggestions here: https://github.com/BabylonJS/BabylonReactNative/pull/144#discussion_r564120706

@syntheticmagus I don't think your link is talking about the same thing as @chrisfromwork's link.
There are 3 types of parallelism when building on Windows that I'm aware of.

Tell the VC++ compiler to use multiple processors (/MP)
Tell MSBuild to use multiple processors (the /m switch).
The one that @chrisfromwork mentioned is different than both of these but it appears to work together with 1.

None of these are guaranteed to make the build faster as they all depend on the build dependencies and the code base. It would be good to measure the performance to know whether it actually helps or not.


feature : configurable lazy loaded modules

Prod build now uses systemjs at runtime
Prod build extracts libraries in a separate bundle
New gulp task : build.modules, it builds the lazy modules that can be  configured in project config
Lazy modules are loaded through system js and a virtual component configured in the routes

This pull-request is related to improvements proposed here : https://github.com/mgechev/angular2-seed/issues/864

I have the feeling that the system js configuration should be refreshed, for example by removing defaultJSExtensions which is deprecated, but I didn't want to mix multiple purposes to this PR.

Hi @taguan, first of all: 👏  👏  👏  nice work!
Could you provide a separate PR for the problem you described here:

I have the feeling that the system js configuration should be refreshed, for example by removing defaultJSExtensions which is deprecated, but I didn't want to mix multiple purposes to this PR.

That'd be awesome!
Second, i have a few more questions:
How is this change related towards the angular2-rc.1 router? The rc1 router is currently not documented yet, so is this change of how to lazy load components the preferred way to implement this functionality?
Is this a SystemJS specific implementation? The Styleguide states here: Styleguide Style 04-12 & Style 04-13:

Does your implementation suffice to that rules? Since the .rc1 Router is not yet documented (Router Interface) i have not seen a reference implementation for it.
Thank you again on that awesome work of yours! 🎉

Hi @TheDonDope
For the systemjs improvement, I will make a PR later if I have the time, it is not critical at all.
Regarding those rules, I have to admit I am a bit lost with the definition of lazy loading. Comments on the existing files explains that +home and +about are lazy loaded, but in my understanding, they just use the default router configuration.
My lazy loading goes beyond that, it actually downloads the "js" only on demand.
I am not an angular expert, in my own project I am not even yet migrated to the new router, so I really don't know where my code falls regarding conventions.

Hi @taguan,

Regarding those rules, I have to admit I am a bit lost with the definition of lazy loading.

If you have a look at the screenshot where it states:
// AVOID: app/app.component.ts
import { HeroesComponent } from './+heroes';

Your implementation satisfies this rule, since: https://github.com/taguan/angular2-seed/blob/77fcaa744c917fb1daebd35cf7e618fdc68adc50/src/client/app/app.component.ts#L33 here you really do lazy load on demand. You do not have an import on LazyModule. So, if i understand correctly, this then is the right implementation of this rule. Or maybe better to say one correct implementation for it, which uses SystemJS.
Regarding the router: If you take a look at the Angular Weekly Meeting Notes you can see, that they are currently still at work with the router, so we may see another round of changes for rc2 (which is slated for Monday, 2016-06-06).
// cc @mgechev @ludohenin @NathanWalker @Shyam-Chen

Ok, nice to read :-)
I am quite confident changes to the router won't break too much the code, because it uses very few features from it...

@taguan looks great!
I believe in the next release the core team will introduce asynchronous routing, which solve similar problem with less typing. On the other hand, you really did an awesome job. What do you think of adding a wiki page with this virtual proxy you did?

I'll do that!
I don't think asynchronous routing will help with the way js are bundled in the prod build, I think this is the most interesting part of my contribution. The virtual proxy will probably be replaced by the feature you mention.

@mgechev I finished implementing that lazy loading feature in my own project. Actually, the bundling works very well, but not the virtual component, because it does not work with nested routes (it expects some annotations on the virtual component it cannot find).
I am still using the deprecated router in my project, and I made it work perfectly with the AsyncRoute (actually, very few code change compared to this PR).
I propose I wait for Async implementation in the new router and some documentation and they I update this PR with implementation using the async feature of the router.

@taguan sounds good. I'm still using fork of the deprecated router myself, that I described here http://blog.mgechev.com/2016/05/21/angular2-router-implementing-missing-resolve-feature-deprecated-defer/.


Module not found "./index.js" on bun run start

After build with vite and start app by bun run start
showing error error: Module not found "./index.js" in build dir.

When i try to run
bun build/server/index.js
or
bun run start build/server/index.js
show errror here.

This app create by default todo app from https://kit.svelte.dev/

Yes, it's my mistake. i was published a packet of without files directory... I fixed it, update the packet version to 0.2.2-3


Safe to include token?

Sanity Studio is a client side application, so anything we put into it must be non-sensitive. If I enter my imageshop token into the config, it'll show up in the client bundle, which in turn will make it available to any attacker that looks through that bundle.
In other words, if we include the imagebank token, it's available to "everyone".
Is there a way to avoid this? And if not, do you offer a read-only token, that only lets us read from our database?

Administrators in Imageshop can themselves issue tokens and change access rights for token. It is possible to prohibit uploading for the token, which will make them in effect read only and no files can then be uploaded or replaced with the token. If you in addition restrict the token only to interfaces which are public or a special interface intended only to be used with web images, the risk is minimal. It is recommended to restrict the access of the token as much as possible. Read more here about how to issue tokens in Imageshop.

Thank you for reporting :) fixed in 1.3.0


atom:// url package installation

Elaborating on #2037:
To allow some interesting integrations with atom.io, we should allow users to initiate package installation from the atom:// url handler. There are a number of questions to answer:

[ ] What work in atom core & atom-shell is required to support this on all 3 platforms?

Windows registry doc: https://msdn.microsoft.com/en-us/library/aa767914(v=vs.85).aspx
The current top answer to this is relevant, showing how to use gconf on a system with gnome: http://stackoverflow.com/questions/1947209/registering-a-url-protocol-handler-in-a-multiple-platforms

[ ] What are the user concerns around security and predictability and how can we address them?

Users should always know what is being installed and have to explicitly confirm installations initiated outside the editor

[ ] What's a general-purpose UI look like for this, given the editor can be in any state when installation is initiated?

Probably a modal in the active window?

Atom already has atom:// registered in Info.plist:
	<key>CFBundleURLTypes</key>
	<array>
		<dict>
			<key>CFBundleURLSchemes</key>
			<array>
				<string>atom</string>
			</array>
			<key>CFBundleURLName</key>
			<string>Atom Shared Session Protocol</string>
		</dict>
	</array>

And it seems that there is already some meaning of atom:// url handler:
https://github.com/atom/atom/blob/master/src/browser/atom-application.coffee#L402

@zcbenz That's mac-only though, we don't register the url with the Windows registry / try to set gnome/kde config for it do we?

@thedaniel Right, we only have it on Mac.
Registering URL on other platforms are quite similar, they are implemented usually by writing some settings to some registry, and the program specified will be called with the URL appended in command line. The work will mostly be in installers/packages.

Fortunately we can write and test all the app code for this without having to wait for the cross-platform URI handler stuff to be done. I probably won't have time this week but I'll wireframe or prototype this.

Btw GitHub Desktop use that feature for the Clone In Desktop feature. So you may have some solution in house.

Btw GitHub Desktop use that feature for the Clone In Desktop feature. So you may have some solution in house.

There are a bunch of pieces of this that have been moving forward here and there. e.g. the settings-view now supports opening a package via URI

cc @BinaryMuse for context

Any update on this? From what I gathered the work is pretty much done in #14243

@uzitech I'll be handling this as part of the larger effort around https://github.com/atom/atom/pull/11399. Stay tuned!

Closed by https://github.com/atom/atom/pull/11399 and https://github.com/atom/settings-view/pull/1011


MCPWM 5.0

The beginning of a very very rough draft of IDF 5.0 style MCPWM. (mostly reused from #93, thus a lot of code will change)
Counter proposal for #93
Note: This depends on esp-idf/esp-idf-sys#133 - Merged
Status as of 2022-12-10

Does compile
examples/mcpwm-simple tested and working on ESP32-S3

@usbalbin @MabezDev Folks, I'm missing a little bit the context of what is going on here. Can somebody summarize it for me? I'm struggling with the following:

We have #93 now. However it does not explain what it is improving upon? Is it about the fact that you can somehow support two duty signals with it?
Then we have this one (#132). It is named "MCPWM 5.0". Also no explanations. Is this also supporting two duty signals, but only for ESP IDF 5? If so, what would be our story for ESP IDF 4.4, which would be around for quite some time?

@usbalbin I think it would be very useful if you create a comment that summarizes what these changes are about, and what is their relation to ESP IDF4 vs ESP IDF 5. I think you might be assuming that we have a bit more context than what we actually know about this stuff. :) Thanks!

This is still quite the mess, and there are a lot of things I am unsure of. However before I get too far down the road, may I request a very light review of your thoughts of the overall picture of this PR?

Does the API make sense?
Does the usage in the example mcpwm-simple make sense?
Any ideas as for how to reduce the perhaps a little bit too heavy use of generics :)
Naming of things...

I just noticed there is https://github.com/esp-rs/esp-hal/pull/255 , perhaps there could be some inspiration to take from there?

Rebased over master and removed duplicate commits from #183

I have now tested examples/mcpwm-simple.rs on an ESP32-S3 and it does (after a lot of fixes) seem to work.

Yes - will look at it over the weekend. @MabezDev if you can take a look too, that would be appreciated!

Thanks a lot for the review!

I have now made the two Comparators of every Operator mandatory. That way we get rid two of Operator's type parameters. Same thing could be done for the Generators, however there is, as far as I know, no way to create a generator without assigning a pin. Since not all users need both pins, I would prefer if this was optional. I suppose do that by Optional<Generator<.., PIN>>. However is there any good way to not force the user to specify the type of PIN when passing None?

I have now made the two Comparators of every Operator mandatory. That way we get rid two of Operator's type parameters.
Same thing could be done for the Generators, however there is, as far as I know, no way to create a generator without assigning a pin. Since not all users need both pins, I would prefer if this was optional. I suppose do that by Optional<Generator<.., PIN>>. However is there any good way to not force the user to specify the type of PIN when passing None?

const NO_PIN: Option<AnyIOPin> = None. And then you can pass NO_PIN everywhere.

I have now made the two Comparators of every Operator mandatory. That way we get rid two of Operator's type parameters.
Same thing could be done for the Generators, however there is, as far as I know, no way to create a generator without assigning a pin. Since not all users need both pins, I would prefer if this was optional. I suppose do that by Optional<Generator<.., PIN>>. However is there any good way to not force the user to specify the type of PIN when passing None?

And BTW you can get rid of the P generic in your generator struct. I.e. from
pub struct Generator<G, P: OutputPin>

into
pub struct Generator<G>

Why don't you look into how the other drivers do it? As in the SPI and I2C ones? The idea is that only the constructor of the struct is generified by P but the struct itself isn't. You just keep the pin number in the struct, not the actual pin peripheral.

There are also other issues, like the fact that you need to acquaint yourself with the PeripheralRef and pin: impl Peripheral<P = impl OutputPin> + d syntax. Where I'm getting at, is that your current Generator struct in its current shape has also other issues, like the fact that it keeps the pin as P: OutputPin, while it should keep it as PeripheralRef<P> where P: OutputPin. That is, if it was necessary to keep the peripheral in the Generationr struct in the first place, which it isn't.

 I hope you plan to get rid of these as well. If not, let me know, and we can discuss why not (as in what difficulties you see.)

 I hope you plan to get rid of these as well. If not, let me know, and we can discuss why not (as in what difficulties you see.)

Yes, I know, I think I should be able to fix most/all of those now...

const NO_PIN: Option = None. And then you can pass NO_PIN everywhere.

Oh, that is very nice.

There are also other issues, like the fact that you need to acquaint yourself with the PeripheralRef and pin: impl Peripheral<P = impl OutputPin> + d syntax. Where I'm getting at, is that your current Generator struct in its current shape has also other issues, like the fact that it keeps the pin as P: OutputPin, while it should keep it as PeripheralRef where P: OutputPin. That is, if it was necessary to keep the peripheral in the Generator struct in the first place, which it isn't.

Thanks, I'll look into that

The main types the user will be passing around as of right now looks like this
struct TimerDriver<const N: u8, G: Group> {...}
struct Operator<'d, const N: u8, G: Group> {...}
struct TimerConnection<const N: u8, G: Group, O0, O1, O2> {...}

I suppose TimerDriver(and therefore also TimerConnection) will also get a lifetime parameter once i have done the impl Peripheral<T = Timer> thing for that too.
What types should get the Driver suffix? All of Operator, TimerConnection, Generator and Comparator?

The main types the user will be passing around as of right now looks like this
struct TimerDriver<const N: u8, G: Group> {...}
struct Operator<'d, const N: u8, G: Group> {...}
struct TimerConnection<const N: u8, G: Group, O0, O1, O2> {...}

I suppose TimerDriver(and therefore also TimerConnection) will also get a lifetime parameter once i have done the impl Peripheral<T = Timer> thing for that too.
What types should get the Driver suffix? All of Operator, TimerConnection, Generator and Comparator?

For now, let's just have TimerDriver suffixed with *Driver. Once this lands, we can revisit shortly after that.
Question about the remaining generics:

Why do we need N?
Why do we still need O0, O1 and O2?

IF that brings additional clarity: we are continuing to (ab)use traits to express patterns where I hoped a simple Option would do. As in OptionalOperator. Why do we need to express that an operator is optional with a dedicated trait?

For now, let's just have TimerDriver suffixed with *Driver. Once this lands, we can revisit shortly after that. [...]

Ok

Why do we need N?

TimerConnection is used to ensure the operators' won't outlive the Timer. TimerConnection::split() gives us mutable references to both timer and operators. If we allowed multiple instances of the same operator type, the user could use mem::swap to move the Operator out of the TimerConnection which would be bad. Could this be checked at runtime? Maybe, have not given it enough thought..
Other than that, there is also the issue of IDF4 only supporting Timer<A> connecting to Operator<B> iff A=B.

Why do we still need O0, O1 and O2?

Thanks to that, TimerConnection::split() can return the real types without the user having to unwrap any Options. I was also considering adding TimerConnection::timer() and TimerConnection::operator{N} methods at some point(nice to have when only needing access to one object at a time). TimerConnection::operator{N} would only be implemented if O{N} is a real Operator. In my mind this would be really nice for discoverability when using something like vs code + rust analyzer or similar.
Is it worth the three type parameters? Not sure... :)

Hey @usbalbin I'm looking to implement/leverage this as a pwm capture method, any pointers on where you left off?

Hey @usbalbin I'm looking to implement/leverage this as a pwm capture method, any pointers on where you left off?

I have not really looked at the capture thing. However most of the types so far are quite close to how the diagram are drawn here, so I would probably start there and look through the diagrams and functions/structures to figure out how things relate.
From a quick glance, the capture thing seems to be quite independent having its own internal timer etc, yet it seems it can be synced with other timers from the same mcpwm unit.
As to the state of the code here, I do not quite remember. I have not worked on this for quite a while. I believe it should work but there is likely a lot of room for improvement :)

I am not working at this right now, and I do not see myself doing so anytime soon. So feel free to use this/finish it as you wish :)

Hey, I'd very much try to use this as I need the new MCPWM api (5.x) for my application.
I tried making esp-idf-hal a git dep to your fork, however am running into build issues where it seems like the bindings have changed since you last worked on this. Would you maybe know know which version of ESP-IDF your bindings were getting compiled from and how I could change the target esp idf version used to create the bindings ?

It was some quite some time since I did anything with this...
However, digging through my files:
.cargo/config.toml:
...
[env]
ESP_IDF_VERSION = { value = "branch:release/v5.0" }

Cargo.toml:
...
#esp-idf-svc = { version = "0.45.0" } # <--- Not sure if this is what I based my local version on?
esp-idf-svc = { path = "..." }
esp-idf-sys = { version = "0.32.1", default-features = false, features = ["binstart", "panic_handler", "alloc_handler"] }
...

I have changed mcu since, so no guarantees, but I believe this worked when it was written.


DM-84 - produce prototype of more efficient compression calculation

This aligns with the proposal in DM-86.
NOTE: if we go this direction, then we need to make sure we have access to all of the sizes for the various data types provided by DuckDB, as noted by FIXME comments in the source.
This prototype only has a handful, so we can test with the nyc311 dataset.

Going to close this for now with the understanding that the PR contains a lot of useful information in it. We can always search for it in the future.


Missed parameters "layoutSizing" for FrameNode

FrameNode type does not include parameters layoutSizingHorizontal and layoutSizingVertical

Released in 1.69.0


ZEPPELIN-442: utilize space between paragraphs to add new paragraphs

This PR relates to: https://issues.apache.org/jira/browse/ZEPPELIN-442 Quickly add new paragraphs.
By utilizing the space between the paragraphs, thought it'd be a good place to add new paragraphs.
Thoughts/suggestions?
Current User Experience of Adding New Paragraph (no direct response if paragraph covers screen):

After Pull Request (adds above the paragraph):

I think this is going to be really useful, thanks. One suggestion is, if it can keep space between unchanged, it will be better. Because of i've seen some use cases that have many paragraph with complex layout, that carefully adjusted to fit in the screen for dashboard purpose.

I like the idea, it will improve user experience a lot.
I found there are few things need to be handled.
There 3 different look&feel mode in Zeppelin which are default, simple, report.
After this PR paragraph's button group overlaps paragraph border in simple mode.
Before:

After:

And in report mode Zeppelin doesn't have ability to add new paragraph, so it will be better to disable this feature for consistency.

@Leemoonsoo - Great suggestion. I've decreased the height of the spacing and should be around the same (see image below) .
@minahlee - Thank you for pointing that out. The newest PR should fix both the simple and report modes.
Let me know if there are more suggestions/thoughts.

Can we also cover insertion via keyboard shortcuts, ideally the same as ipython (http://johnlaudun.org/20131228-ipython-notebook-keyboard-shortcuts/) ?

About keyboard shortcuts, it's better to be discussed in https://issues.apache.org/jira/browse/ZEPPELIN-391. But i think it make sense to have ipython or jupyter inspired keyboard shortcut.

It would be nice to have one under the last paragraph also.
One question about Multiple paragraphs in the same row.
In this PR, we have this:

Do we want to keep that behavior, or have only one + button bar per row?

@corneadoug great suggestion for having one on the last paragraph. The latest PR address that.
Regarding multiple paragraphs in a row, I think we should keep it as is because it allows users to add specifically where they want(if they want to add between col 1 & 2, they can). Thoughts?

Okay, I tested it again and saw things to improve.
Originally the insertParagraph method (used in the menu) is adding a paragraph under the actual one, however with your changes, it always add that paragraph before. And while changing that could be discussed, it would be good for now to not change the behavior users are used to.
So, instead of having that lastPragraph parameter, we could have a position parameter with values like under and upper. This way, if we call insertParagraph('upper') it would do newIndex = i; and `insertParagraph('under')would donewIndex = i + 1;``

@corneadoug: That's a great improvement. Thank you for suggesting it. I have updated the PR to address that. It makes insertNew a lot clearer. Really appreciate it.
Let me know if you see anything else!

Here is screenshots for the top difference:
Master Branch

This PR

@corneadoug : I've reverted it back to 16px. Should be the same as the master branch now. Really appreciate it. Thanks!

Nice, I think it's Ready For Merge then :)

Tested and works greatly. Very useful improvement. Thanks @MikeTYChen.
+1 for merge.

Merging if there's no more discussions then


Issue 90

Issue 90
GIVEN: I’m logged into the application
WHEN: I need to add a league into the database.
THEN: I want to be able to store all the information for that league
Added CreateLeagues function in db/league.go
AddedPostLeagues Handles in apis/leagues
Added documentation in the leagues.yaml file

A couple nitpicky things, but looks great!


SECURITY.md is missing

Please add a SECURITY.md file as a security policy.
The content is in the README.md, but it's a better practice to have a separate document (even if it is duplicate info).

I will argue that this is not required.
The purpose of SECURITY.md is to highlight how the OSS project/organisation prefers to have the issues from the codebases they are storing reported.
I didn't check all OpenPrinting repositories, but some of them (for example, cups) already have a security policy in place. This is a separate discussion.
In the context of this repository, the section that you are referencing describes how the issues discovered with the embedded harnesses should not be reported in GitHub issues. IMO, a SECURITY.md in the context of this repository will guide the security researchers on how they can report security issue in the harness themselves (which will be Inception-like, but not the case here).

The purpose of SECURITY.md is to highlight how the OSS project/organisation prefers to have the issues from the codebases they are storing reported.

Agreed. That's why I thought info in https://github.com/OpenPrinting/fuzzing?tab=readme-ov-file#bug-reporting could be moved or duplicated in a SECURITY.md.

I didn't check all OpenPrinting repositories, but some of them (for example, cups) already have a security policy in place. This is a separate discussion.

It would be really cool if they all had them with GitHub Private Security Reporting \o/

In the context of this repository, the section that you are referencing describes how the issues discovered with the embedded harnesses should not be reported in GitHub issues. IMO, a SECURITY.md in the context of this repository will guide the security researchers on how they can report security issue in the harness themselves (which will be Inception-like, but not the case here).

Yeah, it is unlikely that this info is needed for the harness (any real issues are likely in oss-fuzz or other upstream). But the SECURITY.md may still be checked by fuzzing researchers and it's useful to have (like it is in the README.md). Security Policy info can apply to the whole org generally in addition to a specific project.
Please feel free to close this if needed. I don't mean to bike shed :pray:


Revamp library as new fs2-pubsub

💻 How to review this PR?
This PR was created with the idea of being reviewed commit by commit. Each commit contains an incremental change that makes it easier to review. Also some of the commits contain additional information in their description to help understand why the change was made.
I also recommend checking "Hide whitespace" when reviewing this PR!

🚀 What's included in this PR?
This PR revamps the library as a new fs2-pubsub.
The new library includes several improvements over the previous one, such as builder-pattern for creating the publishers/subscribers, better utilities around message or error handling or pureconfig support.
I recommend checking out the README.md added in the last step to see how the new library works.

👋 What was the motivation for switching from wrapping the Google provided Java SDK to the generated Scala gRPC sources? Was it just to have pure Scala all the way down? Or were their other motivating factors?
What types of testing have been done comparing two implementations? Under load as well?

👋 What was the motivation for switching from wrapping the Google provided Java SDK to the generated Scala gRPC sources? Was it just to have pure Scala all the way down? Or were their other motivating factors?

Hey Zan! Nice to see you here 😸 Main motivation was using pure Scala to avoid dependency hell when two Google libraries are being used and bring different versions of their internal dependencies.

What types of testing have been done comparing two implementations? Under load as well?

There hasn't been any actual testing comparing the two implementations. But the new library is already in use and we haven't observed any degradation on performance.


Do not set code when recasting PDOException

Fixes #2098
The PDOException class has the signature int|string $code, which makes it incompatible with most other stdlib exceptions that only allow int $code. While we could probably make a custom exception class or whatever for the recast, I don't think it's worth the hassle, and looking at the DB ecosystem, some amount of it also just drops the code in favor of just using 0 (e.g. cakephp/Database in Database/Driver and have for years that I'm not sure people will really miss it.
The PDOException recast has been happening for the entire existence of this project that I'm somewhat risk adverse to remove the recast.


Updating the npmjs.org to the latest version?

Are you guy planning on updating the npm?
https://www.npmjs.com/package/coinbase-exchange

done! sorry about the delay


Cannot find module 'layouts/IconButtonElement'.

I updated package.json to latest "node": "10.1.0"
Now after running npm start I am getting the below
Failed to compile
/home/paul/PAUL/H/Web/R/R-Vid/2018/mern-stack-front-to-back-DEVELOPER-SOCIAL-NETWORK/OTHER-GIT-CLONED/react-social-network-110star/src/components/circle/CircleComponent.tsx
(24,31): Cannot find module 'layouts/IconButtonElement'.
This error occurred during the build time and cannot be dismissed.

Thanks for reporting. I have just updated the package 8184eb0  please update your repository and try again. Let me know if any issue.

Now I am getting
Failed to compile
./src/components/comment/CommentComponent.tsx
Module not found: Can't resolve 'material-ui' in '/home/paul/PAUL/H/Web/R/R-Vid/2018/mern-stack-front-to-back-DEVELOPER-SOCIAL-NETWORK/OTHER-GIT-CLONED/react-social-network-110star-Problem/src/components/comment'

I guess you have not used git pull from repository to update your local repository. As I have upgrade material-ui to @material-ui/core you can't have material-ui as a reference any more. Please pull repository again and check if you can see @material-ui/core in CommentComponent.tsx. Thanks!

I did a fresh git-clone of this repo and ran npm i and npm start and getting the below error again . My environment is Ubuntu 16.04
Failed to compile
/home/paul/PAUL/H/Web/R/R-Vid/2018/mern-stack-front-to-back-DEVELOPER-SOCIAL-NETWORK/OTHER-GIT-CLONED/react-social-network-110/src/components/circle/CircleComponent.tsx
(24,31): Cannot find module 'layouts/IconButtonElement'.

I have done 3 steps:
git clone git@github.com:Qolzam/react-social-network.git
yarn install or (npm install)
yarn start or (npm start)

Everything seems well but I can't reproduce your issue. Please help to find if the file exist in layouts folder.

No reproduce. I'm going to close. Let me know if any issue.


perf: schedule inital routing and fetching

This PR tries to chunk up the bootstrap phase of angular.
Before:

The following things are implementes and measured:

Schedule platformBrowserDynamic().bootstrapModule(AppModule) on setTimeout to split work

Schedule initial navigation on setTimeout to split work

Schedule Angular bootatrapping over APP_INITIALIZER on setTimeout to split work

After:

Hi @hoebbelsB I added comments. Let me know what you think.


feat: Extend Datagrid

I Extended DataGrid component with new features :
1- make it editable with some default columns like TextboxColumn, CheckboxColumn ... with validation functionality
2- new embedded default ColumnOptions based on column binding data type, for example, if column data is string, then display a textbox to search, or if datatype is 'Date' then display two datepicker for "from" and "to" filter
3- extend sort capability, so we can sort by a column, then by another column, then by another ...
4- user can remove sort from a column
5- auto generate column header by [Display] attribute on model
6- another way to sort ColumnOptions dropdown header similar to excel sorting added
7- split sort,filter,display property of column. so a column can display a property, sort by another property, filter by another property, or use same property, if not setted
8- Impoved render performance by setting all parameters of Row and Cells by one setter
if you have a plan or agree to add all these features or some of them, I update my fork and create a PR
BR

Discussion on this has moved to #283


feat: disable sound in xcode preview

Detailed description of changes
sekarang di xcode preview udah ga ngeluarin suara
Screenshots or videos (before and after if appropriate)
Recommendations on how to test the changes
Checklist for developer

[x] The code builds clean successfully
[x] The code does not log anything to console
[x] I have thoroughly tested the new code and any adjacent features it may affect

Link to related issue(s)

ini adalah fitur yg paling membantu


Docs appear to be down

https://vuepress-theme-blog.ulivz.com/ is no longer loading.

Thanks. Check https://vuepress-plugin-blog.billyyyyy3320.com/.


Can't use Create to insert record into table if field is composite primary key, is an integer and has a value of zero

go version go1.9 linux/arm64
MySQL 5.7
package main

import (
	"fmt"
	"github.com/coreos/go-log/log"
	"github.com/jinzhu/gorm"
	_ "github.com/jinzhu/gorm/dialects/mysql"
)

type SomeFile struct {
	ID     uint   `gorm:"primary_key" json:"-"  sql:"type:int(10) unsigned;"` // Foreign key to another table
	FileID uint   `json:"id" sql:"type:int" xml:"id,attr"`
	Name   string `json:"name" xml:"name"`
	Url    string `json:"url" xml:"url"`
}

var(
	DB *gorm.DB
	working = &SomeFile{
		ID: 12345,
		FileID: 1,
		Name: "image.jpeg",
		Url: "http://somesite.com/images/image.jpeg",
	}

	notWorking = &SomeFile{
		ID: 12345,
		FileID: 0,
		Name: "image.jpeg",
		Url: "http://somesite.com/images/image.jpeg",
	}
)

func main() {
	DB, err := gorm.Open("mysql", fmt.Sprintf("%v:%v@tcp(%v:%v)/%v?parseTime=True&loc=Local", "root", "password", "<IP_ADDRESS>", "3306", "database"))
	if err != nil {
		panic(err)
	}
	DB = DB.Set("gorm:table_options", "CHARSET=utf8")
	DB = DB.Debug()
	DB.LogMode(true)
	DB.AutoMigrate(&SomeFile{})

	if err := DB.Create(working).Error; err != nil {
		log.Error(err)
	}
	if err := DB.Create(notWorking).Error; err != nil {
		log.Error(err)
	}
}

When running the above code the first record with a FileID of 1 inserts correctly, but the second insert fails. It appears Gorm ignores zero integer values.
(main.go:42)
[2018-02-21 17:18:22]  [21.74ms]  CREATE TABLE `some_files` (`id` int(10) unsigned,`file_id` int,`name` varchar(255),`url` varchar(255) , PRIMARY KEY (`id`,`file_id`)) CHARSET=utf8
[0 rows affected or returned ]

(main.go:44)
[2018-02-21 17:18:22]  [1.54ms]  INSERT INTO `some_files` (`id`,`file_id`,`name`,`url`) VALUES ('12345','1','image.jpeg','http://somesite.com/images/image.jpeg')
[1 rows affected or returned ]

(main.go:47)
[2018-02-21 17:18:22]  Error 1364: Field 'file_id' doesn't have a default value

(main.go:47)
[2018-02-21 17:18:22]  [0.87ms]  INSERT INTO `some_files` (`id`,`name`,`url`) VALUES ('12345','image.jpeg','http://somesite.com/images/image.jpeg')
[0 rows affected or returned ]
Feb 21 17:18:22.192 [    ERROR] - Error 1364: Field 'file_id' doesn't have a default value

I've managed to work around this by setting a default value of -1 for the column, but this is not ideal, neither is incrementing the FileID as the data is defined externally and the FileID if used to link the file to a record in another table.

Solved. Just needed to use a field with Scanner & Value functions as mentioned here. I'd tried this earlier but forgot to specify the field as a pointer and therefore it didn't match the method signatures I'd defined & I got the error as shown above.

@joesteel2010  I am facing this issue. Would you please elaborate what you did to avoid this issue. Sorry I couldn't make it out of the provided link.

@joesteel2010 I am facing this issue. Would you please elaborate what you did to avoid this issue. Sorry I couldn't make it out of the provided link.

Hi @rreubenur, yes it wasn't very obvious from the documentation how to solve this, it does take a bit of research to get the full picture. If you are using a custom type as your PK, the functions that you would need to implement are the Scanner and  Valuer interfaces.
It looks like there are now types provided in the sql package you can use rather than implementing these interfaces yourself, i.e.: https://golang.org/pkg/database/sql/#NullInt64 - I've not tested these but they look like they should work perfectly.
Hope that helps.


Shortcut key for "Close tab and save to corral"

I'm finding that I use the feature to close a tab and move it to the corral a lot - it's helpful when I want to deal with X in a couple hours - worth getting out of my way, but not worth creating a bookmark for.
It's currently a lot of clicks to find.  Would be great if there could be a keyboard shortcut for that.  Since CTRL-W is "Close Tab," maybe "CTRL-SHIFT-W"?

I know this issue was opened a long time ago, but since it was referenced recently I believe it's worth mentioning that Ctrl+Shift+W is the shortcut for closing the current window. Also, according to the docs,

Certain Chrome shortcuts (e.g. window management) always take priority over Extension Command shortcuts and can not be overwritten.

@edjroot Ctrl+Shift+W would work on MacOS because the shortcut for closing the current window is Cmd+Shift+W.  What would be appropriate on those platforms?

Hmm, I didn't know that! I use both Linux and Windows and all shortcuts are the same for them (and I guess Mac too, except it's always Cmd in place of Ctrl). I don't have any particular suggestion, but it's probably a good idea to use the same combinations across OSs anyway.

I'm going to go with Ctrl+Space+W. The hand position feels quite natural since your thumb is resting on the spacebar.

I chose the following shortcuts to start:
Mac: Ctrl+Shift+W
Other: Ctrl+Shift+E
These are configurable by going to chrome://extensions/configureCommands in Chrome. If you have a more suitable suggestion for a default for Windows or Linux, I'm happy to change the default.

It's fine by me!


Post-Hacktoberfest Cleanup

After the close of hacktoberfest, run through the koans and fix up:

[ ] comments
[ ] method names
[ ] path to enlightenment

Specifically  ensure that the flow is still coherent, reads well and can be completed.

I might look into this in the next couple of weeks 😊


Microsoft.Azure.CosmosDB.BulkExecutor package depends on wrong version of Microsoft.Azure.DocumentDB

Your Microsoft.Azure.CosmosDB.BulkExecutor package claims to depend on Microsoft.Azure.DocumentDB, v1.21.1 but in fact it fails at compile time if you call particular methods because in fact (and your own samples corroborate this) it depends on v2.0.0-preview of the Microsoft.Azure.DocumentDB package.
Can you push a new Microsoft.Azure.CosmosDB.BulkExecutor package that fixes the dependency information so it isn't broken by default?

I thought maybe it was just the project file that was referencing the higher version, so I went ahead and converted the project to use packagereferences instead. Sure enough, despite the nuget dependencies showing that the library only requires Microsoft.Azure.DocumentDB (>= 1.21.1), it failed to compile.
It's truly sad that once again this team...

Completely ignores the community (no response to this question in six days)
Seems unfamiliar with the latest Microsoft technologies (netstandard, packagereferences)
Publishes a 100% garbage Nuget

Where's the testing? I've said it before, and I'll say it again, if I had a team that displayed this level of workmanship, someone would have been fired ages ago.

@AArnott : Thanks for reporting this. We will fix this soon and will release an updated BulkExecutor package by this week. Meanwhile, please upgrade the DocumentDB SDK version to 2.0.0-preview in your application to workaround this issue.

@AArnott : We have released a new version(1.1.0) of BulkExecutor that fixes this issue. It now depends on 2.0.0 version of .NET SDK. The sample also has been updated. We will be updating the Release Notes shortly to mention What's new in this version.

Using the latest version of your package, I can now compile without other package refs, but at runtime it still fails:
var graphBulkImporter = new GraphBulkExecutor(this.docClient, this.graph);
await graphBulkImporter.InitializeAsync(); // exception thrown here:

System.MissingMethodException: 'Method not found: 'System.Threading.Tasks.Task`1<Microsoft.Azure.Documents.Routing.CollectionRoutingMap> Microsoft.Azure.Documents.Routing.PartitionKeyRangeCache.TryLookupAsync(System.String, Microsoft.Azure.Documents.Routing.CollectionRoutingMap, System.Threading.CancellationToken)'.'

What's going on here? Are you still compiling against a different version than you have a package reference to?

I get the same error running the sample at https://github.com/Azure/azure-cosmosdb-bulkexecutor-dotnet-getting-started.
I've updated the packages version as follow, but still facing the exception during InitializeAsync.
<packages>
  <package id="Microsoft.Azure.CosmosDB.BulkExecutor" version="1.1.0" targetFramework="net452" />
  <package id="Microsoft.Azure.DocumentDB" version="2.0.0" targetFramework="net452" />
</packages>

Any suggestion?


BLEU metric card

Add BLEU metric card

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

Some thoughts:

For values, e.g. "Defaults to False", I would put False in code: False. Same for : "Defaults to 4."
I would put the following remark in "Limitations":

"BLEU's output is always a number between 0 and 1. This value indicates how similar the candidate text is to the reference texts, with values closer to 1 representing more similar texts. Few human translations will attain a score of 1, since this would indicate that the candidate is identical to one of the reference translations. For this reason, it is not necessary to attain a score of 1. Because there are more opportunities to match, adding additional reference translations will increase the BLEU score."

Add some values from the original BLEU paper (https://aclanthology.org/P02-1040.pdf)


[Bug]: Could not find expected browser locally

Bug description
I started noticing this issue as soon as I updated to v19. I didn’t change any of my code as the list of breaking changes didn’t make it seem like I had to but if that’s the issue please let me know. The issue occurs as soon as I called puppeteer.launch().
Steps to reproduce the problem:

Install v19
Launch a new chrome browser using puppeteer.launch({ args: ["args: ["--no-sandbox"], headless: true })
Observe the error

Puppeteer version
19.0.0
Node.js version
18.10.0
npm version
Yarn 1.22.19
What operating system are you seeing the problem on?
Windows
Relevant log output
/app/node_modules/puppeteer-core/lib/cjs/puppeteer/node/ChromeLauncher.js:69
                  throw new Error(missingText);
                        ^
  Error: Could not find expected browser (chrome) locally. Run `npm install` to download the correct Chromium revision (1045629).
      at ChromeLauncher.launch (/app/node_modules/puppeteer-core/src/node/ChromeLauncher.ts:118:15)

Same error for me, going to revert upgrade for now.
Could not find expected browser (chrome) locally. Run npm install to download the correct Chromium revision (1045629)

Yarn typically has issues due to caching. Could you clean yarn's cache and try again? It would also make sense to "git clean -X" to remove any gitignored files and try a clean install. If this does not work, is your home directory accessible? Since the browser binaries are now placed in a home directory .cache folder, puppeteer needs permission to create directories there. To verify if this is the case, set PUPPETEER_CACHE_DIR to your current working directory and see if puppeteer starts downloading there.

@jrandolf I'm getting this error on my host which performs a clean install on every push, so I doubt that would change anything. As for the home directory, I probably don't have access to that and it would be very dumb to require that as that would fall outside of the directory of the app which most hosts don't let you access, so that should be changed if that's the case.

In general, a cloud host provider will create a user that is specific to the app or your account. Sometimes a home directory is created, other times not, but I wouldn't say it's dumb since a home directory really means "the base directory that we allow the user to have". In any case, we plan to use 'XDG_CACHE_HOME' in the future which may be defined on your provider, but I'd recommend using PUPPETEER_CACHE_DIR as this will give you the most control regarding where you download your browser.

What I'm saying is that it doesn't make sense to download the browser outside of the app's directory by default. I understand I can control it like that but the default should remain within the app and not require us to set an environment variable for it to work which wasn't even documented or alerted in the changelog

This was documented:

use ~/.cache/puppeteer for browser downloads

If you inspect the PR, we also document how to work with it if that directory is problematic.
In any case, there are a few solutions that we are discussing regarding this issue (we know using environment variables is a bad solution). Most likely we will have something like .puppeteerrc, but for now, we recommend using the environment variable. Does this solve your issue?

Im not sure because I added the variable and the issue was fixed but then I removed it and it was still working but I’ll keep the environment variable just in case. I really think you should change the default back to what it was before so that it doesn’t require a folder outside of the app’s directory.

@jrandolf seems like that didn’t fix it after all. Could you reopen this?
I set the env var to the same as my WORKDIR in my dockerfile, any tips?

As you are using a Dockerfile, our support for setup is limited there. We ask that you use the docker image we provide as a base/inspiration since it's tested in our CI. See pptr.dev/guides/docker.

This broke our jenkins build server

@jrandolf the dockerfile is not the cause of the issue. I'm no longer using one and the issue is still happening and clearly it is for other people too. Please reopen this

I got this problem as well and I'm not using cloud function, not lambda and not even docker. I also use standard "npm" (i.e. not yarn or pnpm). I started "node" and ran "os.homedir().toString()" in the REPL and it finds my homedir and my homedir is readable. I'm using ubuntu 22.04 on this computer. I looked in ~/.cache/puppeteer and there was a copy of chrome in that directory but the app still crashed with the message "Could not find expected browser locally" when it ran the puppeteer.launch() code.
However, then I ran:
rm -rf ~/.cache/puppeteer/ ; rm -rf node_modules/
....and then "npm install" and after this everything worked again.


Added hkumar to sudo  admin list

This was missed before, so adding myself in sudo admin list.

merging it since it will not affect anything else.


Xcrypto Issue: #4277

Title and Issue number

Xcrypto App (ReactJs) #4277
Close #4277

Checklist:

[] I have mentioned the issue number in my Pull Request.
[] I have commented my code, particularly in hard-to-understand areas
[] I have created a helpful and easy to understand README.md

@MohitGupta121 ,@TusharKesarwani, @pranjay-poddar, @Saurabh-kumar09
Please merge it with gsocc label.


Not sure how to write tasks that clean up

I'm having a hard time finding a way to run cleanup tasks for old deployments. I can add a cleanup task, but I'm not sure how to tell which release is being cleaned up?

I referenced the existing cleanup task to find my answer.


FetchAssets

@greenealexander having some issues:

with the .env file displaying secrets.
and all videos fetching.


Add getting started  section to the README

README update
Please add a getting started section to the README so folks know how to start with this project.

Added Getting Started section in README file.


Add note about archiving prisma-client-js repository

 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.

 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.


Views404 page

404 page created in views folder and app.all added to server.js to catch and send 404's to page


add dependency - [merged]

In GitLab by @NicolasDenoyelle on May 2, 2019, 14:10
Merges doc -> master

In GitLab by @NicolasDenoyelle on May 2, 2019, 14:17
added 1 commit
22f0459f - add tag for runners
Compare with previous version

In GitLab by @NicolasDenoyelle on May 2, 2019, 14:19
added 3 commits
7e2cf8e5 - add tag for runners8818d069 - add tag for runners13ef74d6 - reorganize tags
Compare with previous version

In GitLab by @NicolasDenoyelle on May 2, 2019, 14:27
merged

In GitLab by @NicolasDenoyelle on May 2, 2019, 14:27
mentioned in commit 8abeb0a25ff434a73919e0fd4b602092e12f2202


CupertinoSliverNavigationBar throws exception back button only when used in routes that can be popped

It seems that the CupertinoSliverNavigationBar is not correctly implying its back button.
I have a route that is using the widget, and when I pushAndPopUntil to it, popping until it is the first route on the page, I see in my logs an exception complaining that the back button is failing an assertion. I think this is due to the check not updating after the pop happens in navigation. The ui seems to be fine and the app functions correctly.
Steps to Reproduce
Navigate to a page with a CupertinoSliverNavigationBar using a pushAndPopUntil that clears out the stack fully

Hi @rmargolinross
Can you please provide your flutter doctor -v?
Thank you

[✓] Flutter (Channel stable, 2.2.2, on macOS 11.4 20F71 darwin-x64, locale en-US)
    • Flutter version 2.2.2 at /Users/rmargolinross/dev/flutter
    • Framework revision d79295af24 (2 weeks ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/rmargolinross/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.23.0

[✓] Connected device (2 available)
    • iPhone (mobile) • 07da1517197bd6155d47de15a47759f492e42c8d • ios            • iOS 14.6
    • Chrome (web)    • chrome                                   • web-javascript • Google Chrome 91.0.4472.114

• No issues found!

Hi @rmargolinross

Navigate to a page with a CupertinoSliverNavigationBar using a pushAndPopUntil that clears out the stack fully

Just tried to reproduce the issue based on your description on the flutter channel
After pushAndPopUntil and no errors are thrown when doing so

minimal code sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      title: 'Cupertino App',
      initialRoute: '/',
      routes: {
        // When navigating to the "/" route, build the FirstScreen widget.
        '/': (context) => Home(),
        // When navigating to the "/second" route, build the SecondScreen widget.
        '/second': (context) => Second(),
      },
    );
  }
}

class Home extends StatelessWidget {
  const Home({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      navigationBar: CupertinoNavigationBar(
        middle: Text('Cupertino App Bar'),
      ),
      child: Center(
        child: Container(
          child: CupertinoButton(
            child: Text("Go to next screen"),
            onPressed: () {
              Navigator.pushAndRemoveUntil(
                context,
                CupertinoPageRoute(
                  builder: (_) => Second(),
                ),
                ModalRoute.withName('/second'),
              );
            },
          ),
        ),
      ),
    );
  }
}

class Second extends StatelessWidget {
  const Second({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return CustomScrollView(
      slivers: [
        CupertinoSliverNavigationBar(
          largeTitle: Text('CupertinoSliverNavigationBar'),
        ),
        SliverFillRemaining(
          child: FlutterLogo(size: 300),
        )
      ],
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.2, on macOS 11.4 20F71 darwin-x64, locale
    en-US)
    • Flutter version 2.2.2 at /Users/tahatesser/Code/flutter_stable
    • Framework revision d79295af24 (3 weeks ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.23.0

[✓] Connected device (4 available)
    • Android SDK built for x86 (mobile) • emulator-5554                        •
      android-x86    • Android 10 (API 29) (emulator)
    • iPhone 12 Pro (mobile)             • 4819C924-80DB-4DE5-9093-71A234590ABB • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)                    • macos                                •
      darwin-x64     • macOS 11.4 20F71 darwin-x64
    • Chrome (web)                       • chrome                               •
      web-javascript • Google Chrome 91.0.4472.114

• No issues found!

Im not sure if it is the same, but I used a predicate of (_) => false, and I am using a Material app for instead of a cupertino one. I will work to try to get you a minimal code example.

hmm, I was unable to reproduce it. I will work more to try to figure out the real cause of my issue

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


fix: method POST must have a request body

This is a fix for the issue found here:
https://github.com/bf2fc6cc711aee1a0c2a/e2e-test-suite/pull/502#discussion_r1136044789
// Unexpected exception thrown: java.lang.IllegalArgumentException: method POST must have a request body.

@baywet should be ready to go 🙂

@baywet I missed an android linting thing, should be good now 👍 Looking for the release, and thanks a lot!

thanks for making the update, I didn't know you could put the attribute inline for the return type like you can for parameters.

Oh, sorry, I haven't noticed that you are using a different style in this codebase, do you want me to follow up on this?
Also, since we are here 🙂 , do you have any opinionated automatic formatting tool for Java already in place somewhere?

I personally don't care about those things.
But we have some rules in spotbugs that do a very basic level of linting, I think that's about it for the linting today.
If you have spare time, setting up something like checkstyles or equivalent would be appreciated.

I personally don't care about those things.

Forgot about it 😅 Right!

If you have spare time, setting up something like checkstyles or equivalent would be appreciated.

Deal, next time I jump on this codebase and feel bad because of the formatting I'll do it 🙂


If last player leaves lobby it should be deleted and thus be unjoinable

Implemented in:
https://github.com/sopra-fs23-group-13/meme-it-server/commit/87bac67b0216b67495b3d31d2abde75580803d89
Had small bug, fixed in:
https://github.com/sopra-fs23-group-13/meme-it-server/commit/4a71cddae042736e8f8814b007c8d9b0820b86f2


Error compiling on Raspberry Pi 4

Required Info

Camera Model
{ D400 } although it's N/A

Firmware Version
(N/A)

Operating System & Version
{Raspbian from NOOBS 3_2_0

Kernel Version (Linux Only)
Linux caremon5 4.19.58-v7l+ #1245 SMP Fri Jul 12 17:31:45 BST 2019 armv7l GNU/Linux

Platform
Raspberry Pi 4

SDK Version
{ just downloaded today }

Language
{python3 }

Segment
{libRealsense build }

Issue Description
Linux caremon5 4.19.58-v7l+ #1245 SMP Fri Jul 12 17:31:45 BST 2019 armv7l GNU/Linux
I did get an error message with this step earlier:
root@caremon5:/srv/www/webroot# sudo apt-get install -y libdrm-amdgpu1 libdrm-amdgpu1-dbg libdrm-dev libdrm-exynos1 libdrm-exynos1-dbg libdrm-freedreno1 libdrm-freedreno1-dbg libdrm-nouveau2 libdrm-nouveau2-dbg libdrm-omap1 libdrm-omap1-dbg libdrm-radeon1 libdrm-radeon1-dbg libdrm-tegra0 libdrm-tegra0-dbg libdrm2 libdrm2-dbg
Reading package lists... Done
Building dependency tree
Reading state information...  -I../src -I/usr/include/python3.7m -c google/protobuf/pyext/extension_dict.cc -o build/temp.linux-armv7l-3.7/google/protobuf/pyext/extension_dict.o -Wno-write-strings -Wno-invalid-offsetof -Wno-sign-compare
google/protobuf/pyext/extension_dict.cc: In function ‘PyObject* google::protobuf::python::extension_dict::_FindExtensionByName(google::protobuf::python::ExtensionDict*, PyObject*)’:
google/protobuf/pyext/extension_dict.cc:59:45: error: invalid conversion from ‘const char*’ to ‘char*’ [-fpermissive]
        ((*(charpp) = PyUnicode_AsUTF8AndSize(ob, (sizep))) == NULL? -1: 0): \
                      ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~
google/protobuf/pyext/extension_dict.cc:187:7: note: in expansion of macro ‘PyString_AsStringAndSize’
   if (PyString_AsStringAndSize(arg, &name, &name_size) < 0) {
       ^~~~~~~~~~~~~~~~~~~~~~~~
error: command 'arm-linux-gnueabihf-gcc' failed with exit status 1

same here, following this guide:
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation_raspbian.md
pi@raspberrypi:~ $ uname -a
Linux raspberrypi 4.19.75-v7l+ #1270 SMP Tue Sep 24 18:51:41 BST 2019 armv7l GNU/Linux

pi@raspberrypi:~ $ gcc -v
Using built-in specs.
 Done
Building dependency tree
Reading state information... Done
E: Unable to locate package libdrm-amdgpu1-dbg
E: Unable to locate package libdrm-exynos1-dbg
E: Unable to locate package libdrm-freedreno1-dbg
E: Unable to locate package libdrm-nouveau2-dbg
E: Unable to locate package libdrm-omap1-dbg
E: Unable to locate package libdrm-radeon1-dbg
E: Unable to locate package libdrm-tegra0-dbg
E: Unable to locate package libdrm2-dbg

has someone figured out where this libraries can be downloaded from ? or any link we have add to apt source.list? i am also getting the same error

Having the same issue!

I have solved the problem.Here is some change based on origin install command.
compiling on Raspberry Pi 4
$uname -a
::Linux raspberrypi 4.19.75-v7l+ #1270 SMP Tue Sep 24 18:51:41 BST 2019 armv7l GNU/Linux
$gcc -v
::gcc version 8.3.0 (Raspbian 8.3.0-6+rpi1)
$cmake --version
::cmake version 3.13.4

Install packages
you will get an error message for example:Unable to locate package libdrm-amdgpu1-dbg.
ignore the error and continue the next install command.

install protobuf
change the protobuf version v3.5.1 to v3.10.0
so change  git clone command for install  the packages :
git clone --depth=1 -b v3.10.0 https://github.com/google/protobuf.git

install OpenCV
I didn't install opencv,this package  seem didn't affect correct open the camera.

4.change pi settings (enable OpenGL)
insatll pyopengl_accelerate will see an error ,change the command
$ sudo -H pip3 install pyopengl_accelerate
to $ sudo -H pip3 install pyopengl_accelerate==3.1.3rc1
Connected D415 to the pi and open terminal
$ realsense-viewer
Good luck

@suyuan945, thank you very much for sharing and glad to see D415 working on Pi 4.
Closing this ticket if no further question on the same topic.
Thank you.

When will the official documentation be updated?

The simplest solution
Go to scripts inside repo and download this script-
libuvc_installation.sh
Then
Chmod +x libuvc_installation.sh
./libuvc_installation
(Will take some time to compile)
Realsense will work for sure

That may be true, but the official docs are really out of date and need to be changed.
Otherwise it's an exercise in frustration for other people to try to make this package work, which is a shame as the Pi 4 is a beast of a machine.
It won't work on a Pi 3 either unless on a very old version of Raspbian.
The docs need to be fixed, and any extra scripts need to be included in the repository.
I realize open source can be really slow to update as it's a DIY mentality, but we paid for this hardware product, and it should run on modern hardware.

The simplest solution
Go to scripts inside repo and download this script-
libuvc_installation.sh
Then
Chmod +x libuvc_installation.sh
./libuvc_installation
(Will take some time to compile)
Realsense will work for sure

I used libuvc_installation.sh to install successfully on TX2 device, but would like to ask how to install pyrealsense2？

same here, following this guide:
https://github.com/IntelRealSense/librealsense/blob/master/doc/installation_raspbian.md
pi@raspberrypi:~ $ uname -a
Linux raspberrypi 4.19.75-v7l+ #1270 SMP Tue Sep 24 18:51:41 BST 2019 armv7l GNU/Linux

pi@raspberrypi:~ $ gcc -v
Using built-in specs.
 Done
Building dependency tree
Reading state information...  -I../src -I/usr/include/python3.7m -c google/protobuf/pyext/extension_dict.cc -o build/temp.linux-armv7l-3.7/google/protobuf/pyext/extension_dict.o -Wno-write-strings -Wno-invalid-offsetof -Wno-sign-compare
google/protobuf/pyext/extension_dict.cc: In function ‘PyObject* google::protobuf::python::extension_dict::_FindExtensionByName(google::protobuf::python::ExtensionDict*, PyObject*)’:
google/protobuf/pyext/extension_dict.cc:59:45: error: invalid conversion from ‘const char*’ to ‘char*’ [-fpermissive]
        ((*(charpp) = PyUnicode_AsUTF8AndSize(ob, (sizep))) == NULL? -1: 0): \
                      ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~
google/protobuf/pyext/extension_dict.cc:187:7: note: in expansion of macro ‘PyString_AsStringAndSize’
   if (PyString_AsStringAndSize(arg, &name, &name_size) < 0) {
       ^~~~~~~~~~~~~~~~~~~~~~~~
error: command 'arm-linux-gnueabihf-gcc' failed with exit status 1

I used apt-cache search for libdrm-amdgpu1-dbg it returned libdrm-amdgpu1-dbgsym and libdrm-amdgpu1 so I went on to run command Sudo apt-get install libdrm-amdgpu1-dbgsym which got installed without errors.
I am trying other steps to install librealsense completely will update if any mistake and solution are found.

This issue should not be closed. The docs are still out of date and were never changed.

I have solved the problem.Here is some change based on origin install command.
compiling on Raspberry Pi 4
$uname -a
::Linux raspberrypi 4.19.75-v7l+ #1270 SMP Tue Sep 24 18:51:41 BST 2019 armv7l GNU/Linux
$gcc -v
::gcc version 8.3.0 (Raspbian 8.3.0-6+rpi1)
$cmake --version
::cmake version 3.13.4

Install packages
you will get an error message for example:Unable to locate package libdrm-amdgpu1-dbg.
ignore the error and continue the next install command.
install protobuf
change the protobuf version v3.5.1 to v3.10.0
so change  git  command for install  the packages :
git clone --depth=1 -b v3.10.0 https://github.com/google/protobuf.git
install OpenCV
I didn't install opencv
change pi settings (enable OpenGL)
install pyopengl_accelerate will see an error ,change the command
$ sudo -H pip3 install pyopengl_accelerate
to $ sudo -H pip3 install pyopengl_accelerate==3.1.3rc1

Connected D415 to the pi and open terminal
$ realsense-viewer
Good luck

copied your all the work and in the end getting this fault

Segmentation fault

I have solved the problem.Here is some change based on origin install command.
compiling on Raspberry Pi 4
$uname -a
::Linux raspberrypi 4.19.75-v7l+ #1270 SMP Tue Sep 24 18:51:41 BST 2019 armv7l GNU/Linux
$gcc -v
::gcc version 8.3.0 (Raspbian 8.3.0-6+rpi1)
$cmake --version
::cmake version 3.13.4

Install packages
you will get an error message for example:Unable to locate package libdrm-amdgpu1-dbg.
ignore the error and continue the next install command.
install protobuf
change the protobuf version v3.5.1 to v3.10.0
so change  git  command for install  the packages :
git clone --depth=1 -b v3.10.0 https://github.com/google/protobuf.git
install OpenCV
I didn't install opencv
change pi settings (enable OpenGL)
install pyopengl_accelerate will see an error ,change the command
$ sudo -H pip3 install pyopengl_accelerate
to $ sudo -H pip3 install pyopengl_accelerate==3.1.3rc1

Connected D415 to the pi and open terminal
$ realsense-viewer
Good luck

I copied all your work but in the end getting this error
when i runned the following command
realsense-viewer
'Segmentation fault'


Built file size is confusing

💡 Feature request
Overview and problem statement
The built file size language is a little confusing.
 Uploading...
 Uploading...
 Success
⬇ Installing wranglerjs...
⬇ Installing wasm-pack...
 Built successfully, built project size is 11 KiB.

"Built project size" can be interpreted as the size of your built bundle.js in webpack terms, which can be confusing when the user notices that bundle.js is significantly larger than than the reported built project size.
The reported value appears to be the gzipped project size.  Adding this would clarify the difference:
 Built successfully, compressed project size is 11 KiB.

for anyone stumbling across this issue, the built size is computed after gzip compression

related issue #1227

Has something changed on how the worker size is calculated in wrangler 1.9? I have a worker that with wrangler 1.8.4 builds into a file of ~635KiB. But when built with wrangler 1.9 I get the warning of the over 1Mb file and of course cannot be published:
❯❯❯ wrangler --version                                                                           16:12:54
👷 ✨  wrangler 1.8.4

~/C/worker master*
❯❯❯ wrangler build                                                                               16:12:58
⬇ Installing wranglerjs...
⬇ Installing wasm-pack...
Warning: webpack's output filename is being renamed to worker.js because of requirements from the Workers runtime
✨  Built successfully, built project size is 635 KiB.

~/C/worker master*
❯❯❯ npm install -g<EMAIL_ADDRESS>                                                    16:13:05
/usr/local/bin/wrangler -><EMAIL_ADDRESS>
><EMAIL_ADDRESS>postinstall /usr/local/lib/node_modules/@cloudflare/wrangler
> node ./install-wrangler.js

Downloading release https://workers.cloudflare.com/get-npm-wrangler-binary/1.9.0/x86_64-apple-darwin
wrangler has been installed!
+<EMAIL_ADDRESS>updated 3 packages in 2.534s

~/C/worker master*
❯❯❯ wrangler build                                                                               16:13:15
⬇ Installing wranglerjs...
⬇ Installing wasm-pack...
Warning: webpack's output filename is being renamed to worker.js because of requirements from the Workers runtime
✨  Built successfully, built project size is 1 MiB. ⚠  Your built project has grown past the 1MiB size limit and may fail to deploy. ⚠

@jorgelbg nothing should have changed, and we haven't been able to reproduce. is this still happening for you? if so, it needs its own issue, i don't want to lose it under this one.

@ashleymichal Thanks for the reply!
I tested with the latest release (v1.9.2) and I see the same thing happening. I will open a new issue and put the details there.

We just landed bundle size reporting in Wrangler 2. See https://github.com/cloudflare/wrangler2/pull/1272
Please upgrade to that version of Wrangler if this is still a problem for you.


How to compile

hello,
I'd like to cross compile this web server for a riscv64 architecture but I have some problems. Current instructions are a bit unclear to me.
HOWEVER you must provide functions to access your sockets. A Posix/Berkeley sockets example is provided (as well as a WinSock example). Therefor you must be able to compile on a Posix system with Berkeley sockets.
what does that mean?
when you say:
So to compile a basic server:
gcc main.c SocketsCon.c FileServer.c WebServer.c -o example.exe

where should I place myself? in src/ there is a main.c but there is not FileServer.c so.. this command will not work. If I move to the hello world example this command is also incorrect. If I use the one on the bat file, I do manage to cross compile it but when I run it I get:
# sh hello_world_server
hello_world_server: line 3: syntax error: unexpected word (expecting ")")

If would be great if you could clarify. Thanks a lot

Hi fabriziotappero,
I have updated the readme to try to make it a bit clearer.
However here is the answers to your questions directly :)
Question 1
HOWEVER you must provide functions to access your sockets. A Posix/Berkeley sockets example is provided (as well as a WinSock example). Therefor you must be able to compile on a Posix system with Berkeley sockets.
what does that mean?
Bitty HTTP is not locked into one socket library, if you have some other kind of sockets library (other than Posix sockets for example) you can connect in to them.  I have seen SPI based chips that had very different socket API's.  Bitty HTTP was build to support these type of things.
I did provide example use standard Berkeley sockets (and a different one for Windows sockets) in the src/SocketsCon.c file.  If you are running on a Linux or other Unix like system you can just use them and not have to worry about the details.
Question 2
when you say:
So to compile a basic server: gcc main.c SocketsCon.c FileServer.c WebServer.c -o example.exe
where should I place myself?
This was a generic example that doesn't work with any of the provided examples.  It was assuming that you would copy the needed files to your own directory before compiling.
So if you wanted to use it from the bittyhttp directory you would use:
gcc src/main.c src/SocketsCon.c src/Examples/HelloWorld/FileServer.c src/WebServer.c -o example.exe
This would be for a Linux build.
For windows it would be
gcc src\main.c src\SocketsConWin.c src\Examples\HelloWorld\FileServer.c src\WebServer.c -o example.exe -lws2_32
Question 3
# sh hello_world_server
hello_world_server: line 3: syntax error: unexpected word (expecting ")")

Not sure what's going on here.  sh is a shell and I'm not sure where hello_world_server came from.
To run the bat / .sh file (for hello world):
Linux:
cd src/Examples/HelloWorld
./build.sh
./HelloWorld.exe

Windows:
cd src/Examples/HelloWorld
build
HelloWorld

The should be all there is to it (assuming that gcc is installed and setup)
Hope this helps.

Oh, you should also check out http://bittyhttp.com/Development.html for more documentation.


Add DX11 support

The library currently supports OpenGL, OpenGL Core profile, and Cg.
It would be great to add the Dx11 support (i.e. Maya needs it).

How to start with this one?

Hi @mundane140 ,
The best way to start the task is to implement a GPU unit test framework using Dx11 (with the existing OpenGL unit test framework as example). With the recent work done on master the HLSL support is now in without having anyway to test it.
Refer to #526.

Closing the defect as the HLSL support is now in.


[DNM] Proposal for Adding local model loading and automatically dependency installation in webui

add auto dependencies installation
add local path for loading 'sentence-transformers' model
authentication update
TODO: If it makes sense, we can add configuration for User to decide private key path

cc @KepingYan

Hi, @KepingYan and @carsonwang , I don't meant to propose a merge in the PR.
The modification I made in the PR are something I used to fix the codes so it can run in my local ENV.
I believe others may encouter same issue as I did, so I want to use this PR to have some codes discussion .


Failed to download all of CC12M

I have been trying to download CC12M, using the same instructions you provided, however, the download is not complete.
I am getting the following error for a significant number of iterations:
total   - success: 0.792 - failed to download: 0.195 - failed to resize: 0.013 - images per sec: 485 - count: 12423374
After the code is done, I end up with 1243 tar files. How can I solve this to get the full CC12M dataset?

Hey, did you set up knot resolver for DNS resolving?
This is really important to avoid overloading your DNS and hence having a
low success rate
On Wed, Dec 14, 2022, 15:34 AlaaKhaddaj @.***> wrote:

I have been trying to download CC12M, using the same instructions you
provided, however, the download is not complete.
I am getting the following error for a significant number of iterations:
total - success: 0.792 - failed to download: 0.195 - failed to resize:
0.013 - images per sec: 485 - count: 12423374
After the code is done, I end up with 1243 tar files. How can I solve this
to get the full CC12M dataset?
—
Reply to this email directly, view it on GitHub
https://github.com/rom1504/img2dataset/issues/242, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437WB34WFXPJEZ2J74PDWNHLIFANCNFSM6AAAAAAS6R3NSI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@rom1504 Hi, is there any way to re-download only the failed data?

+1  is there method?

You can read the output parquet files and select only the samples that are
failed status, write that as parquet (you can do that with pandas or spark)
Then rerun img2dataset on it
On Fri, May 26, 2023, 03:42 Oliver Wei @.***> wrote:

+1 is there method?
—
Reply to this email directly, view it on GitHub
https://github.com/rom1504/img2dataset/issues/242#issuecomment-1563703809,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAR437TO7VERUZX34437QLTXIADAJANCNFSM6AAAAAAS6R3NSI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Rework events to include conventions

The awesome folks at fancons.com kindly allowed us to use their list of furry conventions to put them on Howlr. The Events tab would be a great place for them, but it needs to be reworked first:

The event tab was made to handle a small amount of local events with a small amount of participant, nothing is paginated and this could cause some performance issue (especially with loading the list of participants).
Local events are more important that conventions, so we there must be a way for them to stand out and not get drowned in the list of conventions.
It should be possible to search for people inside an event, like searching for "Users into [some_kink] and [some_other_kink] going to [this_convention]"

The new event interface was pushed (with quite a few bugs x3), we will import the conventions tomorrow.


Signal's providing_args removal in 4.0

providing_args arg has been deprecated since 3.1.
https://docs.djangoproject.com/en/dev/releases/3.1/#id2
I can draft a PR provided some insights on:

how to document the arguments
whether to keep the providing_args for when version 2.2 and 3.0 (my suggestion would be no)

Since 4.0 beta 1 is out, should we add it to the test matrix?

@g-as thank you, go ahead!

@g-as have you seen this draft?
https://github.com/IdentityPython/djangosaml2/pull/292
consider to work on top of this if possibile, thank you

I'll take a look at it tomorrow!

Any chance of a release including this any time soon?

1.3.6 was released two weeks ago:
https://github.com/IdentityPython/djangosaml2/releases/tag/v1.3.6
https://pypi.org/project/djangosaml2/

Whoops! Was looking for a "version X" commit message, missed it. Sorry!

cheers


Update interfaces.json

The ";" isnt needed. It causes these errors:
*** Started working on sourcesWithNeoForge
 Compiling 5364 source files
 ERROR Line: 50, > or ',' expected in /net/minecraft/core/component/DataComponentType.java
 ERROR Line: 50, illegal start of type in /net/minecraft/core/component/DataComponentType.java
 ERROR Line: 50, initializers not allowed in interfaces in /net/minecraft/core/component/DataComponentType.java
 ERROR Line: 55, illegal start of expression in /net/minecraft/core/component/DataComponentType.java
 ERROR Line: 55, = expected in /net/minecraft/core/component/DataComponentType.java
 ERROR Line: 107, class, interface, enum, or record expected in /net/minecraft/core/component/DataComponentType.java
 ✓ Completed sourcesWithNeoForge in 2.51s
Total runtime: 8.79s

net.neoforged.neoform.runtime.graph.NodeExecutionException: Node action for recompile failed
	at net.neoforged.neoform.runtime.engine.NeoFormEngine.runNode(NeoFormEngine.java:458)
	at net.neoforged.neoform.runtime.engine.NeoFormEngine.lambda$getWaitCondition$3(NeoFormEngine.java:404)
	at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1804)
	at java.base/java.util.concurrent.ThreadPerTaskExecutor$TaskRunner.run(ThreadPerTaskExecutor.java:314)
	at java.base/java.lang.VirtualThread.run(VirtualThread.java:329)
Caused by: java.io.IOException: Compilation failed
	at net.neoforged.neoform.runtime.actions.RecompileSourcesActionWithJDK.run(RecompileSourcesActionWithJDK.java:89)
	at net.neoforged.neoform.runtime.engine.NeoFormEngine.runNode(NeoFormEngine.java:447)
	... 4 more

Dont merge yet, im still porting AT to neoforge 😄

On line 2 of accesstransformers.cfg
public can be changed to protected to avoid this error, dont know if it would cause issues
> Task :createMinecraftArtifacts
Loaded 124 artifacts from C:\Users\vyrek\Desktop\Vyrek\Development\mods\phasorite-networks\build\moddev\nfrt_artifact_manifest.properties
*** Started working on downloadJson
 ✓ Completed downloadJson in 0.01s
*** Started working on downloadServer
*** Started working on downloadClient
*** Started working on downloadClientMappings
*** Started working on listLibraries
 ♻ Used cache of listLibraries in 0.00s
 ✓ Completed downloadServer in 0.02s
 ✓ Completed downloadClientMappings in 0.02s
 ✓ Completed downloadClient in 0.02s
*** Started working on mergeMappings
 ♻ Used cache of mergeMappings in 0.00s
*** Started working on stripClient
 ♻ Used cache of stripClient in 0.00s
*** Started working on extractServer
 ♻ Used cache of extractServer in 0.00s
*** Started working on stripServer
 ♻ Used cache of stripServer in 0.00s
*** Started working on merge
 ♻ Used cache of merge in 0.00s
*** Started working on rename
 ♻ Used cache of rename in 0.00s
*** Started working on decompile
 ♻ Used cache of decompile in 0.00s
*** Started working on inject
 ♻ Used cache of inject in 0.00s
*** Started working on patch
 ♻ Used cache of patch in 0.00s
*** Started working on applyNeoforgePatches
 ♻ Used cache of applyNeoforgePatches in 0.00s
*** Started working on transformSources
 ♻ Used cache of transformSources in 0.00s
*** Started working on recompile
 124 items on the compile classpath
 1 items on the sourcepath
*** Started working on sourcesWithNeoForge
 ♻ Used cache of sourcesWithNeoForge in 0.00s
 Compiling 5364 source files
 ERROR Line: 337, <T>addRenderableWidget(T) in net.minecraft.client.gui.screens.worldselection.CreateWorldScreen cannot override <T>addRenderableWidget(T) in net.minecraft.client.gui.screens.Screen
  attempting to assign weaker access privileges; was public in /net/minecraft/client/gui/screens/worldselection/CreateWorldScreen.java
Total runtime: 15.44s

net.neoforged.neoform.runtime.graph.NodeExecutionException: Node action for recompile failed
	at net.neoforged.neoform.runtime.engine.NeoFormEngine.runNode(NeoFormEngine.java:458)
	at net.neoforged.neoform.runtime.engine.NeoFormEngine.lambda$getWaitCondition$3(NeoFormEngine.java:404)
	at java.base/java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1804)
	at java.base/java.util.concurrent.ThreadPerTaskExecutor$TaskRunner.run(ThreadPerTaskExecutor.java:314)
	at java.base/java.lang.VirtualThread.run(VirtualThread.java:329)
Caused by: java.io.IOException: Compilation failed
	at net.neoforged.neoform.runtime.actions.RecompileSourcesActionWithJDK.run(RecompileSourcesActionWithJDK.java:89)
	at net.neoforged.neoform.runtime.engine.NeoFormEngine.runNode(NeoFormEngine.java:447)
	... 4 more
Caused by: java.io.IOException: Compilation failed

> Task :createMinecraftArtifacts FAILED

Execution failed for task ':createMinecraftArtifacts'.
> Process 'command 'C:\Program Files\Eclipse Adoptium\jdk-<IP_ADDRESS>-hotspot\bin\java.exe'' finished with non-zero exit value 1

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.
BUILD FAILED in 15s
3 actionable tasks: 1 executed, 2 up-to-date

Closed as its already updated


Enhancement: restrict material lists to specific ontologies

Specify further that we use UBERON and ENVO ontologies specifically for body and sediment related sample lists (currently allow any ontology, as these are often linked but this has lead to confusion at points).
Should also check all categories are currently valid

Some interesting things here: http://www.obofoundry.org/
e.g. BCO for museum collections


According to the document, the bound event does not take effect

According to the Swiper document, the bound event does not take effect。
Hope to update the document to describe specifically how to use event binding。
onClick: () => {
console.log('【click】')
},
onSlideChangeEnd:() => {
console.log('【changeEnd】')
}
Events must be used like this, and many events do not take effect。

I'm blind.