Can StructScan decode to an interface given the reflect.Type and a table name?

Hi,
I'm trying to use sqlx like gob; I want register concrete types and then want "decode" them from SQL back into a slice of interfaces.  For example, I want to transform this:

type Event interface{
	ID() int
}

type S struct {
	Id int
}

func (s *S) ID() int {
	return s.Id
}

func add(s reflect.Value, idx int) {

	entry := reflect.New(reflect.TypeOf(S{}))
	field := entry.Elem().FieldByIndex([]int{0})
	field.SetInt(int64(idx))

	s.Index(idx).Set(entry)
}

func main() {
	var a []Event

	s := reflect.ValueOf(&a).Elem()
	s.Set(reflect.MakeSlice(reflect.TypeOf(a), 2, 2))

	add(s, 0)
	add(s, 1)

	fmt.Println(a[0], a[1])
}

into somthing that uses sqlx in the add() function.  This is the closest I could get:
func add(s reflect.Value, idx int) {
	rows, _ := db.Queryx("select * from [main.S]")
	for rows.Next() {
		v0 := reflect.New(reflect.TypeOf(S{}))
		v1 := s0.Elem().Interface()
		_ = rows.StructScan(&s)
		s.Index(idx).Set(v0)
		idx++
	}
}

(This panics with "TraversalsByName on interface Value")
Is this possible?
Thanks!
(Working example of first snippet: https://play.golang.org/p/EKZSxKvt3i).

I don't quite know what you're trying to do.  Your original code seems to be basically re-implementing the append builtin.
Your sqlx sample code has many, many errors in it;  why pass a reflect.Value if you're just going to take the TypeOf(S{})?  Why are you trying to scan into a reflect.Value?  Where is the s0 variable coming from?
That said, you should be able to scan into an interface{} if it has an addressable struct inside it (without passing a pointer to it;  pointer to an interface isn't what you want).
If you want to select a bunch of events from SQL, you could easily just do
var a []Event
err := db.Select(&a, "SELECT * FROM events;")

I'm so deep in the weeds on this one, I'm not surprised my intent wasn't clear.  Let me rephrase.
Given:

a sql statement that selects all MyEvent from a table
an interface Event
the reflect.Type of MyEvent (which implements the interface Event)

I am trying to write this function:
func getEvents(sqlx.DB *db, sql string, typ reflect.Type) []Event {
}

The original code I posted is a boiled down version of what encoding/gob does when it decodes serialized structs into an slice of interfaces, which is pretty much the same problem I am trying to tackle with sqlx.
It would be awesome if I am missing something really simple ... :)

This does what I want.
func test12(db *sqlx.DB) {
	ptr := reflect.New(reflect.SliceOf(reflect.TypeOf(&MyEvent{})))
	iface := ptr.Interface()
	err := db.Select(iface, "select * from [main.MyEvent]")
	if err != nil {
		panic(err)
	}
	x0 := ptr.Elem().Index(0)
	var i Event = x0.Interface().(Event)
}```

* Event is the interface
* &MyEvent implements the interface
* all we have to work with is TypeOf(&MyEvent) and Event.

Closing ticket, thanks.


update(): add charset to html page

add charset to html page

Would love to see a PR from you @Lordnoname


Which words are reserved under ViewComponent?

I've just encountered an issue whereby having a slot like renders_many :tags results in a conflict between the tag slot and the tag view helper. It'd be good to keep track of which other words, if any, are likely to cause issues with ViewComponent.

@boardfish I agree. Our work on reducing the number of potential instance variable clashes was a good step in this direction. How do you think we should keep track of these words?

I wonder if tests would be a good way to do this. In the same way that the test for the trailing newline issue uses something peripheral to ViewComponent to document the solution to the issue, we could do something similar for namespace clashes.
For example, in the case I had above, we could have passing tests to suggest alternative names like topics or categories for the slot. attributes is another namespace clash, now that I think about it, if you're using Active Model on top of your base component class.

@boardfish hmm. I wonder if we could enforce this at the framework level. For example, we could check slot names against a list of reserved words at compile time and raise an error.

That could be really good for addressing this in a way that's more developer-friendly.
I wonder if reserved words could affect the experience in other ways, perhaps such as:

defining a method that's the same as a slot name
doing something similar with some of ViewComponent's convenience features like collection parameters
accidentally overriding private methods
I'm interested in experimenting with these to see what kinds of situations can arise.

I suppose the logic we'd use to raise errors in case of reserved words would be common to all these cases, but the words themselves may differ between each case.

accidentally overriding private methods
Oh, that is an interesting one. I hadn't thought of namespacing our private methods.
Would you be up for taking on the slot naming issue and seeing what you can come up with?

Yeah, I'll see what I can whip up there.

Turns out there's a validation in place for if renders_one :content is called, but there isn't for renders_many :contents, so I'll try to address that in isolation first.
I'll see how it interfaces with other things too, but this is one well-defined area we can tackle for sure.

I've had a look at this just now - I've tried using slots to override existing methods at various levels. Not sure how confident I feel in this, so if anyone's got more knowledge of how to break Ruby and libraries, I'd be interested to know if there's anything else I can try around here. So far I've tried slots named text_field_tag (a view helper), freeze (a method on the component), methods, and initialize_parameters (a private method on the component). I've also tried calling these via a template and via call. These don't seem to mess with rendering the component at all.
I guess the usual warning of "avoid overriding library methods" flies, but I don't think there's really much more we can do here without evidence of a conflict. content was something there was evidence for, but I can't really think of anything else that would trigger this. I think I'll close this, but I'm willing to investigate if something else to this effect comes up.


Loss (loss_rpn_box_reg) is nan from RPN

My RPN is defined as:
rpn = rpn.RegionProposalNetwork(rpn_anchor_generator, rpn_head,rpn_fg_iou_thresh, rpn_bg_iou_thresh, rpn_batch_size_per_image, rpn_positive_fraction,rpn_pre_nms_top_n, rpn_post_nms_top_n, rpn_nms_thresh,training) rpn.cuda()
Here's my loss:
proposals, proposal_losses = rpn(src_images,src_features,src_targets)
proposal_losses gives:
{'loss_objectness': tensor(0.6926, device='cuda:2', grad_fn=<BinaryCrossEntropyWithLogitsBackward>),  'loss_rpn_box_reg': tensor(nan, device='cuda:2', grad_fn=<DivBackward0>)}
I'm getting the backbone features from Resnet (as in docs) which in turn gives an OrderedDict. Image is an object from ImageList.
My targets are :
[{'boxes': tensor([[1324.5386,  275.7656,  699.6948,   19.5117],           [1324.5386,  275.1152,  702.2983,   19.5117],           [1307.6157,  184.0605, 1000.4009,   22.7637],           [1310.8701,  133.9805,  345.6167,   26.0156]], device='cuda:2'),   'labels': tensor([0., 0., 5., 5.], device='cuda:2'),   'image_id': tensor([438.], device='cuda:2'),   'area': tensor([378240., 375708., 117056., 246178.], device='cuda:2'),   'iscrowd': tensor([0, 0, 0, 0], device='cuda:2')}]
I've tried many ways to prevent nan (normalizing images, low lr) , but it still persists. I've tried with two different datasets. Both of them give nan loss in first epoch.  Only 'loss_rpn_box_reg' becomes nan.  What can be the reason ? I also discussed this issue here. Another thread.

@prajjwal1 in my case (and I think it might be the same in your case), the problem for NaN in the RPN was due to degenerate boxes.
For example, I would have a box where the width would be 0 or negative (i.e., x0 > x1). So I remove all degenerate boxes from COCO in https://github.com/pytorch/vision/blob/d84fee6d2a64b1a253de89f40e0d3995e6b83578/references/detection/coco_utils.py#L82
This seems to be the case for you, as in your first box, you have
[1324.5386, 275.7656, 699.6948, 19.5117]

which seems to indicate that you might be using the boxes in x0,y0,w,h format, instead of returning it as x0,y0,x1,y1 format.
Note that the boxes that the algorithm expects are x0, y0, x1, y1.
I'm closing this issue as this seems to be user-error on the box format, but let me know if you have further questions.

all the values in output dictionary from the model presents nan .
Why its happen?

Hi, I encountered the nan issue with my faster_rcnn_resnet50 model, turns out I was using Adam optimizer which led to the values going to Nan, I changed it back to SGD with momentum and weight decay (to which the original architecture was trained) and the results were better, Hope this helps


Execute nested-jobs best-effort. Individual exception should not block t...

If a smaller job from a NestedJob fails with an exception, Gizzard currently gives up on the rest of the jobs without trying and send them to the error queue. When error queue is backed up, this causes innocents jobs significant delay. Note that a blocked shard could cause this problem. This will make Gizzard's NestedJobs best-effort.
This change may affect previous usages, if they were relying on the serial execution of NestedJob.

Thanks for the reviews. I changed the nulls to options and added some comments. Let me know how it looks.

shipit

thanks!


Scrollbars not considered in detecting dashboard orientation

https://youtu.be/3yaT9BmIgWw
Based on the video I recorded, it seems the dashboard changed on how large is the width / height. Also using more than 3 girls, this happens as well :- http://i.imgur.com/oqNUa7R.png (This only happens when the windows are wider than 817px).
Well even with this "surprise", the dashboard still fine.

This looks funny to me xD Actually the threshold to change the dashboard is 800px and that 17px comes from the scrollbar of the browser, which we haven't taken into account.
Thanks for reporting, anyway.
@dragonjet I think we should increase it to 817px or a little more, i guess?

ahh then I guess we should use document.width instead of window.width in detectOrientation. Let me see

I've been playing around with the orientation detection mechanism, and it seems the current way is still the best way. The desired effect is in between to only show it if the window is wide enough, and for the horizontal scrollbar to never be activated.
Adjusting the threshold higher or lower will generate mroe undesired results, showing horizontal scrollbar or using the wrong layout even if desired conditions are satisfied.
Currently the desired conditions are:

Vertical: Show only if there is enough width and will not result to a horizontal scrollbar for the panel
Horizontal: Show in any case the vertical panel conditions are not satisfied

Thus, I might mark this as wontfix if there are no more concerns about the method of changing layouts.
About the vertical surprise layout, it is known that it's not yet fully finished and is tracked on these two entries #157  #158


fix: always load the default certs on our custom SSL context

This is a follow-up PR of this one. In the linked PR we had to specify and older version of the requests package to avoid a regression.
This PR contains the fix for that, so we can continue using always the latest version. Furthermore, the HTTPS related tests have been improved and moved to the directory of the unit tests, to always include them in our builds.

:tada: This PR is included in version 3.20.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


dart2native: reverse -o & -k in help

The help text for dart2native is almost but not quite in alpha order (2.6.0-dev.8.1). Move the -o line after -k, and we'll be good.
$ dart2native --help
Usage: dart2native <main-dart-file> [<options>]

Generates an executable or an AOT snapshot from <main-dart-file>.

-D, --define=<key=value>       Set values of environment variables. To specify
                               multiple variables, use multiple options or use
                               commas to separate key-value pairs.
                               E.g.: dart2native -Da=1,b=2 main.dart

    --enable-asserts           Enable assert statements.
-h, --help                     Display this help message.
-o, --output=<path>            Set the output filename. <path> can be relative
                               or absolute.
                               E.g.: dart2native main.dart -o ../bin/my_app.exe

-k, --output-kind=<aot|exe>
          [aot]                Generate an AOT snapshot.
          [exe] (default)      Generate a standalone executable.

-p, --packages=<path>          Get package locations from the specified file
                               instead of .packages. <path> can be relative or
                               absolute.
                               E.g.: dart2native --packages=/tmp/pkgs main.dart

-v, --verbose                  Show verbose output.

This ordering is intentional since it is alpha ordered by the (full) flag name:
    ..addFlag('enable-asserts', ...
    ..addFlag('help', ...
    ..addOption('output', ...
    ..addOption('output-kind', ...
    ..addOption('packages', ...

The confusion I believe comes from some of the flags having an optional abbreviation ('o', 'k', ...), which doesn't follow the same order.
But since not all flags have an abbreviation that does not make a good sorting key.
I would suggest we leave the order as is.

Unfortunately the abbreviations come first, so it's easy to think it's a mistake. (I actually missed the -k flag when I first wrote the docs, possibly because of this.)
One more thing... Some flags have blank lines above them in the help, and some don't. Can we make them all have a blank line separator?

I personally prefer the current order, but I'll leave it up to you.
The blank lines, like some of the previous discussion about formatting choices and key=value pairs, is ultimately up to the args package.
In https://github.com/dart-lang/args/issues/48 it's stated that the forced newline after all multiline flags is WAI. It's been suggested to make it configurable, but I don't think it's been implemented yet.


Auto-download executable drivers?

Auto-download executable drivers?

It's already implemented in https://github.com/bonigarcia/webdrivermanager that can be used in pair with webdriver-factory


Clarify documentation about customMapStyles being only effective with Google provider

Does any other open PR do the same thing?

No
What issue is this PR fixing?
customMapStyle has no effect when using Apple maps as provider #4948
How did you test this PR?

This is a change in the Readme.

@gkpo would be nice if you update the documentations instead in two ways:

the section title: Customizing the map style could have (Provider: GoogleMaps) as part of the title
mention that this is a depricated way of customyzing style instead using google maps cloud and using mapId is the way to go (this feature was introduced recently but not well documented and doesn't work yet for iOS).


[Bug]: Fix spec deviations in handling ambiguities in cloneWithType

Description
$title
The spec for cloneWithType says this.
Constructs a value with a specified type by cloning another value.
# + v - the value to be cloned
# + t - the type for the cloned to be constructed
# + return - a new value that belongs to type `t`, or an error if this cannot be done
#
# When `v` is a structural value, the inherent type of the value to be constructed
# comes from `t`. When `t` is a union that includes more than one type descriptor
# that can be used as the inherent type of values with `v`'s basic type, then
# the inherent type used will be the first (leftmost) such type descriptor such that a value
# belonging to that type can be constructed from `v`.

Currently the inherent type is not chosen properly. We need to fix this such that it produces the below examples.

First (leftmost) such type descriptor is considered only when v is a structural value.

json j = { x: 1 };
record { float x; } | record { int x; } v = checkpanic j.cloneWithType(); // should give { x: 1.0 } inherent type is record { float x; }

This is recursively applicable for nested fields also.
json j = {y: {x: 1}};
record{record {float x;}|record {int x;} y;} v = checkpanic j.cloneWithType(); // should give {y: {x: 1.0}}, if nested field is a structural value, should consider the first match policy

json j = { x: 1, y: 2 };
record { float x; decimal|int y; } | record { int x; decimal y; } v = check j.cloneWithType(); // should give { x:1.0, y: 2 }  inherent type is record { float x; decimal|int y; }

If it is not a structural value, the first match policy is not considered.

float x = 12.0;
int|float y = checkpanic x.cloneWithType(); // should give 12.0 float

This is recursively applicable for nested fields also.
json j = { x: 6.0, y: 12 };
record { int|float x; float y; } = j.fromJsonWithType(); // should give { x: 6.0, y: 12.0 } if nested field is not a structural value, the first match policy is not considered.

json j = { x: 6.0, y: 12 };
record { int|float x; int y; } = j.fromJsonWithType(); // should give { x: 6.0, y: 12 } if nested field is not a structural value, the first match policy is not considered.

Steps to Reproduce
No response
Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Runtime
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

@hasithaa @gimantha Currently because the spec on the mapping constructor says the below
https://github.com/ballerina-platform/ballerina-spec/blob/dc03364c958b93612cdc3c0a93bac6e0fbbf886c/lang/spec.html#L4913-L4918
I think we do not allow the following.
type PetByAge record {
    int age;
    string nickname?;
};

type PetByType record {
    "Cat"|"Dog" pet_type;
    boolean hunts?;
};

type Pet PetByAge|PetByType;

public function main() {
     Pet dog = {"age": 4, "nickname": "Fido", "pet_type": "Dog", "hunts": true}; // compilation error: ambiguous type 'Pet'(BCE2523)
}

But with the first match policy described above, since now we are allowing
    json dog = {"age": 4, "nickname": "Fido", "pet_type": "Dog", "hunts": true};
    Pet pet = checkpanic dog.cloneWithType(); // not an error, inherent type will be PetByAge, because it is the first match
    io:println(pet is PetByAge); // true
    io:println(pet is PetByType); // false

do we need to discuss on the spec, whether we should allow the mapping constructor, not to give an ambiguity error during the compilation, like in the above mentioned case, and to choose the inherent type using the first match policy?

If we are considering this, other structured types' constructors will need to be considered also. I think a different first match policy should be considered without any numeric conversions, unlike in fromJsonWithType.

do we need to discuss on the spec, whether we should allow the mapping constructor, not to give an ambiguity error during the compilation, like in the above mentioned case, and to choose the inherent type using the first match policy?

James has considered and rejected this idea. See https://github.com/ballerina-platform/ballerina-spec/issues/1226#issuecomment-1465470314.


error with SSL

I get this error when evilginx2 is requesting for certificate

The error in the Screenshot shows you the issue: temporary failure in name resolution. Your DNS resolution is not working correctly.

Hello! If you were already able to resolve your doubts and achieve your goals, close the issue so that we know which ones are pending.
Thank you!

Sorry, that was a while ago. I left it open for other users as reference. I’ll close now.

@Support-1535 i haven’t been able to get it to work yet, is there any way you can help?

If you want expedited help and have the assistance of many professional people, join our private group. Contact us<EMAIL_ADDRESS>to give you access to the group.


Token in User

Hi. First thanks for sharing your work.Its been a great inspiration for learning koa and vue.
I don't really get what this token in the user model/controller is doing? Maybe I missed something but it seems that it is genreated but not used.
https://github.com/johndatserakis/koa-vue-notes-api/blob/5e5121dafb05dbf981163da36c7a5439759da465/src/controllers/UserActionController.js#L94-L95

Good question - I generate a token there to give the user a unique value that's separate from the autogenerated incrementing id.
The reason I do this is because sometimes you may want to identify a user, or any object from the database, in a way that does not give away any database details. For instance, if you refer to a user in a public url and use their id, you are publicly displaying how many users your program has.
Now, this isn't actually the biggest deal, but what I started doing a while back to mitigate any negative effects was to create a unique token alongside the newly created user that could be used to identify the user just like an id, but not give away any further details. In this program I don't think I used it further, but if there was a public route I may have elected to use it there.

Closing for now


Problem with reading ZIP file - "Compression method not supported"

I have this ZIP file containing heightmap data, but for some reason unzipping it using zip-rs returns:

"Error while reading zip file "N48E008.SRTMGL3.hgt.zip": Unsupported Zip archive: Compression method not supported"

I don't know too much about ZIP compression - it did work a few weeks ago, but now I can't read the file anymore:
N48E008.SRTMGL3.hgt.zip
Do you have any idea what's wrong with the ZIP file?

With the default options, this file should read just fine as it is the default 'deflate' compression.
You can test this by downloading the repository and running cargo run --example extract -- N48E008.SRTMGL3.hgt.zip.
How are you trying this?

fn get_unzipped_heightmap(input: Vec<u8>, file_path: PathBuf) -> Result<Vec<u8>, Error> {
    use std::io::Read;

    // Unzip the file
    let reader = ::std::io::Cursor::new(input);
    let mut zip = ::zip::ZipArchive::new(reader).map_err(|e| Error::Zip(file_path.clone(), e))?;

    // Take the first file (each heightmap zip file only contains one file)
    let mut file = zip.by_index(0).map_err(|e| Error::Zip(file_path.clone(), e))?;
    let mut file_buf = Vec::<u8>::new();
    file.read_to_end(&mut file_buf).map_err(|e| Error::UnzipError(file_path, e))?;
    Ok(file_buf)
}

If I run the extract sample, it does work, I can unzip the file. If I use the above code, it doesn't. Strange.

What do you have for zip-rs in your Cargo.toml file?

zip = { version = "0.5.5", default-features = false }
Oh, I think it's because of the default-features = false. I usually add that automatically for all crates.
Thanks for the hint.


Shiftkey + multiselect

Congratulations guys for this amazing library.
This is not an issue, but a proposal / question.
Is there a way to make a multiple selection by holding the shift key of the keyboard and selecting the items to include?
Thank you,
Marios

@dipson88 is there a way to develop it?
Thank you very much!

Hi @makatsis, It is a nice feature, but it is too difficult to implement in the current code.
I worked with treeselect on Vue and React, but I didn't face this functional. It is the reason why I skip functional like this.


Set Content-Length header when posting a form

Some servers will reject a POST when there is no content-length header (or if it is inaccurate). This header cannot be set from the header options within lua (I tracked "why" down when I committed this, but have since forgotten).

I think this can be merged safely.

Thanks for the patch. Makes sense to me. We might want to add a test for it before merging?

Hi,
Currently on vacation but I will add a test when I get back next week.

On Jun 25, 2016, at 9:14 AM, Christian Joudrey<EMAIL_ADDRESS>wrote:
Thanks for the patch. Makes sense to me. We might want to add a test for it before merging?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@chrsm I created a pull request including tests, if @cjoudrey and you don't mind I'll merge this one.

Forgot to link the new pull: https://github.com/cjoudrey/gluahttp/pull/14

Closing this in favour of #14. Thanks @chrsm for pointing this out.


Move this repo under the OpenShift org

Once we have something basic running, we can migrate the repo.

Created a repo, but need to migrate the code.

https://github.com/openshift/console-operator/issues/5


Signature length/algorithm

The current document states that signature is 64 bytes but also that it is RSA. This would correspond to a 512-bit RSA signature, which is very weak.
If size is an issue, then you should use EC signatures, which are much stronger at an equivalent length (64 bytes). If there is a very strict size constraint, you could use BLS which allows for aggregating multiple signatures from different signers.

Thank you. We will switch to EC signatures with 64 byte length prior to commercial trials to improve the solution. We'll update this issue when the documentation and code has been modified. The reference implementation will use the EC implementation from Go and within the Crypto API of web browsers.
As a reminder for those new to SWAN. The OWIDs will be transmitted and stored many trillions of times a day. As such there is a primary requirement to reduce the number of bytes used considering the risks associated with spoofing, and the other non technical methods available to uncover bad acts and actors.

Obviously I can't say for sure without looking in detail, but it should fix
the one we flagged.
On Thu, Sep 9, 2021 at 8:08 AM James Rosewell @.***>
wrote:

The solution and trials have been switched to ECDSA P-256. @ekr
https://github.com/ekr Do you consider this change addresses the issue
related to the cryptographic method used to sign OWIDs?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SWAN-community/owid/issues/1#issuecomment-916187561,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIPLIKDRZGJQ2EVPUZHA7TUBDEXLANCNFSM5BNXPFAQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Understood. Thank you. I'll close the issue.
Note: For those that read this n the future here are the links to the interfaces used.
Go
.NET
JavaScript


Array parameter issue in Incidents api

The following code fails with an error:
pypd.Incident.find(statuses=['triggered', 'acknowledged'])
And the error is:
BadRequest: BadRequest (2001): Invalid Input Provided - Statuses must be in triggered, acknowledged, or resolved.
It works when passing just one value in the array so I suspect there's a problem with the url generated for the request when an array has more than one item.

This should fix your issue with multiple values in kwargs.
BTW, this project os howling for tests ;-)

Please accept my apologies for being so late to this issue. Somehow I unsubbed from my own project. At any rate, I'll verify the fix and maybe even make a new test for this today. Thanks so much for your contributions :) please feel free to bug me about it.


influxdb: 1.4.1 -> 1.6.3

Motivation for this change
Wanted to use InfluxDB but saw that the version in Nixpkgs was almost a year old (2017-11).
I'm not sure if InfluxDB can be bumped just like this, or if both versions 1.4.1 and 1.6.3 should be made available and a switch is made (in the NixOS InfluxDB service) based on stateVersion.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build influxdb

Success on aarch64-linux (full log)
Attempted: influxdb
Partial log (click to expand)
shrinking /nix/store/b2a0zzch00a7rbi2xqrcdhq7sv0mpid2-influxdb-1.6.3-bin/bin/influxd
shrinking /nix/store/b2a0zzch00a7rbi2xqrcdhq7sv0mpid2-influxdb-1.6.3-bin/bin/store
shrinking /nix/store/b2a0zzch00a7rbi2xqrcdhq7sv0mpid2-influxdb-1.6.3-bin/bin/influx_tsm
shrinking /nix/store/b2a0zzch00a7rbi2xqrcdhq7sv0mpid2-influxdb-1.6.3-bin/bin/influx_stress
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/b2a0zzch00a7rbi2xqrcdhq7sv0mpid2-influxdb-1.6.3-bin/bin
patching script interpreter paths in /nix/store/b2a0zzch00a7rbi2xqrcdhq7sv0mpid2-influxdb-1.6.3-bin
checking for references to /build in /nix/store/b2a0zzch00a7rbi2xqrcdhq7sv0mpid2-influxdb-1.6.3-bin...
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
/nix/store/b2a0zzch00a7rbi2xqrcdhq7sv0mpid2-influxdb-1.6.3-bin

Success on x86_64-linux (full log)
Attempted: influxdb
Partial log (click to expand)
shrinking /nix/store/rp32456slbx712x9480166p72qg75fq9-influxdb-1.6.3-bin/bin/influxd
shrinking /nix/store/rp32456slbx712x9480166p72qg75fq9-influxdb-1.6.3-bin/bin/influx
shrinking /nix/store/rp32456slbx712x9480166p72qg75fq9-influxdb-1.6.3-bin/bin/influx_inspect
shrinking /nix/store/rp32456slbx712x9480166p72qg75fq9-influxdb-1.6.3-bin/bin/influx_tsm
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/rp32456slbx712x9480166p72qg75fq9-influxdb-1.6.3-bin/bin
patching script interpreter paths in /nix/store/rp32456slbx712x9480166p72qg75fq9-influxdb-1.6.3-bin
checking for references to /build in /nix/store/rp32456slbx712x9480166p72qg75fq9-influxdb-1.6.3-bin...
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
/nix/store/rp32456slbx712x9480166p72qg75fq9-influxdb-1.6.3-bin

Success on x86_64-darwin (full log)
Attempted: influxdb
Partial log (click to expand)
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
find: '/nix/store/p1h1p6a14v8xvcjx1b8akq73824a7lp2-influxdb-1.6.3': No such file or directory
/nix/store/wy8mn1fp4jyba2lfpvsxy0axr0x9ahp0-influxdb-1.6.3-bin

@markuskowa beegfs ist the only user of this in nixpkgs.  I tried to build it on top of this PR and got 1 failed test. Not sure if that is related to the influxdb bump. Can you please have a look before this is merged?

@xeji I can build beegfs as well as the test nixos/tests/beegfs.nix successfully for this PR. Influxdb is only used in the monitoring service in beegfs. I think the update is fine.

Thank you  @markuskowa!

I'm not sure if InfluxDB can be bumped just like this, or if both versions 1.4.1 and 1.6.3 should be made available ...

@c0deaddict  we generally try to keep only the latest version of a package in nixgpks, unless there's a specific reason to keep older versions, like other packages depending on them. In this case it's fine to bump.


Escpos protocol additional styles and printer specific alignment settings

Luke,
As requested, the commits related only to protocol related updates for font styles, alternate line feeds and standard mode settings.
Kind regards,
Adrian.

Thanks for the contribution @hollandar !  Can you remove these changes from the other PR, and let me know your thoughts on my comments as well!

Thoughts provided over there, let me know if/how we should proceed and I'll tear that PR apart as necessary.


fix(host): load existing guests desc

init device desc after load pci infos
What this PR does / why we need it:

Does this PR need to be backport to the previous release branch?:

/hold

/hold cancel


Naming changes

limitX + limitY -> boundaries
snapTo -> snapPoints
springs -> springPoints
gravity -> gravityPoints
friction -> frictionAreas
gravity.limitX + gravity.limitY -> gravity.influenceArea
onSnap + onStop -> onStop and combine both, to always include the closest point (snap / spring / gravity) with its id
drag -> dragWithSpring + dragToss

done


Initial aarch64-apple-darwin support

Adds initial support for Mac M1 ARM CPUs.
This may not be correct as had problems running our pxbind structgen utility on Mac M1 so for now just copy'n'pasted the x86_64 Mac structgen files. Does seem to run but could be subtle differences (that could cause catastrophic failures)?
Resolves: #92

I'm not actually sure what would be different between archs here. It's just a bunch of structs with repr(C), and all the platforms we support are 64-bit and, I think, pretty much entirely layouts structs the same way? Do we really need platform specific structgen.rs ?

Yeah I'm not fully sure either, I've done it this way before also of just copying it over from other platforms, @h3r2tic do you know more specifically where compiler/arch differences could affect the structgen?

@h3r2tic what do you think of the risks of just getting this in vs generating a proper definition?
Seems to be working well, been using it for a couple of months and no seen physx issues

I spent a couple of evenings trying to replace the bindgen stuff with an easier to maintain Python version but the clang-bindings lacked some things, so getting a 1-1 match in output wasn't possible.

Added an issue here: #127, if we want to tackle it.

@h3r2tic what do you think of the risks of just getting this in vs generating a proper definition?
Seems to be working well, been using it for a couple of months and no seen physx issues

In reality: who knows! If we reuse x64 struct definitions for arm64, we are crossing fingers and relying on C++ ABI compatibility between those (we rely on repr(C) between C and Rust, but the generated struct layout matches the C++ ABI). In theory we shouldn't be relying on pre-generated struct layouts at all, although they are de-facto stable.
So we should run structgen on aarch64-apple-darwin, but the build process has changed for clang/libtooling, and I'm at a loss trying to get it to work. I'd much rather go with @tgolsson's idea of relying on the devs of Python's or Rust's bindings, and using a sane build system.
Seeing as that's not something we have right now, and considering you've been testing aarch64-apple-darwin for a while, we could just merge this, and at some point replace the generator with one written in not-C++.

Thanks! and agreed, think we should get this in @hrydgard @tgolsson


ppc64el fails with clang : too few template arguments for class template 'batch'

xsimd 12 is failing to build tests with clang on ppc64el architecture. It was previously fine with xsimd 10.
gcc is still passing.  Other "unsupported" architectures like s390x are passing,
An except from a test log is
223s cd /tmp/autopkgtest-lxc.1ndnhv8d/downtmp/build.R7D/src/test/run_test_clang/architectures && /usr/bin/clang++   -O3 -DNDEBUG -march=knl -MD -MT architectures/CMakeFiles/test_knl.dir/dummy.cpp.o -MF CMakeFiles/test_knl.dir/dummy.cpp.o.d -o CMakeFiles/test_knl.dir/dummy.cpp.o -c /tmp/autopkgtest-lxc.1ndnhv8d/downtmp/build.R7D/src/test/architectures/dummy.cpp
223s clang: warning: argument unused during compilation: '-march=knl' [-Wunused-command-line-argument]
224s /tmp/autopkgtest-lxc.1ndnhv8d/downtmp/build.R7D/src/test/architectures/dummy.cpp:4:8: error: too few template arguments for class template 'batch'
224s xsimd::batch<int> come_and_get_some(xsimd::batch<int> x, xsimd::batch<int> y)
224s        ^
224s /usr/include/xsimd/arch/xsimd_scalar.hpp:30:11: note: template is declared here
224s     class batch;
224s           ^
224s /tmp/autopkgtest-lxc.1ndnhv8d/downtmp/build.R7D/src/test/architectures/dummy.cpp:4:44: error: too few template arguments for class template 'batch'
224s xsimd::batch<int> come_and_get_some(xsimd::batch<int> x, xsimd::batch<int> y)
224s                                            ^
224s /usr/include/xsimd/arch/xsimd_scalar.hpp:30:11: note: template is declared here
224s     class batch;
224s           ^
224s /tmp/autopkgtest-lxc.1ndnhv8d/downtmp/build.R7D/src/test/architectures/dummy.cpp:4:65: error: too few template arguments for class template 'batch'
224s xsimd::batch<int> come_and_get_some(xsimd::batch<int> x, xsimd::batch<int> y)
224s                                                                 ^
224s /usr/include/xsimd/arch/xsimd_scalar.hpp:30:11: note: template is declared here
224s     class batch;
224s           ^
224s 3 errors generated.
224s make[2]: *** [architectures/CMakeFiles/test_knl.dir/build.make:76: architectures/CMakeFiles/test_knl.dir/dummy.cpp.o] Error 1
224s make[2]: Leaving directory '/tmp/autopkgtest-lxc.1ndnhv8d/downtmp/build.R7D/src/test/run_test_clang'

Are there assumptions used with the batch class for ppc64le or other unsupported architectures that aren't consistent with clang?
armel, for comparison is passing, but isn't trying to compile architectures/dummy.cpp (I'm not sure, is armel considered "unsupported")?  s390x is also not compiling architectures/dummy.cpp

I can reproduce!


[VS Code] Starting collaboration session: Cannot start 'ShareWorkspaceAsync' while 'ShareWorkspaceAsync' is running.

Error:
Starting collaboration session: Cannot start 'ShareWorkspaceAsync' while 'ShareWorkspaceAsync' is running.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.1151

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.30.2

platformName
MacOS

platformVersion
17.7.0

Closing due to inactivity. Please re-Open if you’re still encountering this issue. Thanks!

I am getting the same issue. I can join someone else's session though. But I don't get any options to join their audio.
Here are my logs:
LiveShareLogs.zip

Thanks for reporting, we'll look into fixing this issue.

This issue has been closed due lack of any recent activity. We've made lots of improvements to Live Share since this issue. Please re-open the issue, if this problem persists.
🤖Live-Share-Clean-Up-Bot🤖


Update deps/k_release

https://github.com/runtimeverification/hs-backend-booster/actions/runs/6380667892/job/17315497608?pr=320#step:8:709
The blockchain-k-plugin_release needs to be updated in tandem with the k_release here, to account for a changed symbol name in the llvm-backend that it uses.


add Auth to .spelling for blog/2017/0.1-auth translate lint issue

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rshriram
If they are not already assigned, you can assign the PR to them by writing /assign @rshriram in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @DavadDi. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

I don't think we want this. The translated post needs to be fixed instead.

@geeknoid The word Istio Auth will cause spelling-related errors in articles that have been translated into Chinese, and personally think it would be a good idea to add Auth to the .spelling file. The current period is temporarily working through Isito Auth, but it is very unattractive.


GTK3 Theming does not seem to work

I've seen it mentioned by mmstick that gtk3 theming should be suported in COSMIC if adw-gtk3 is also installed, however this does not seem to currently be the case in NixOS. Now whether is this a NixOS or a COSMIC issue I don't know how to check myself, so I thought I should report this here.

Should this module include an option for gtk3 support?

This really sounds mostly irrelevant to COSMIC (e.g. if this is a libadwaita thing, I don't see why other libadwaita-based environments wouldn't suffer similarly), if you have config options that are useful to add for an "out of box" COSMIC experience, feel free to share them or make a PR
Only trouble is this would likely be most correctly set via GSettings overrides or dconf, and both are still a bit awkward with NixOS options (perhaps COSMIC should be setting/managing GTK theme itself even?)


Add thermostatOperatingState to Climate Device

Adds the value of hvac_action (idle/heating/cooling) as HE attribute thermostatOperatingState.

Also request modifying line 84 of the app code to include "climate" domain.  This looks to be necessary in order for "climate" devices to populate into the discovery section of the app.  I did this on my local version and appears to be working.

I allready added your request (this one) in the current testing file.


removing integrity attr

It seems like the same hash algo gives different results on the GHA runner and on my Windows locals, which causes the auto-built prod-deploy to fail the browser's integrity checks. So I'm removing the integrity attributes for now, because it's better that they work with a trivial reduction in security vs. not work at all.


Add support for automated login and multiple usernames

I am not sure if this PR is going to be merged as it diverges a lot from the original interface. But I don't want people to have to go to forks for this functionality. There are two main features in this PR:

Originally, InstaRaider only allows downloading images for one user at a time. This PR allow giving a json file containing usernames and downloading photos/videos for them. Now it does not take username as command argument.
Due to above feature, it is not ideal for the end-user to keep manually logging-in everytime a private profile is found. I have added two optional arguments: username and password, which will be used to login into instagram automatically.

Also, I had to change a check for private profile.
Let me know your thoughts

FYI, there is some discussion here: https://github.com/akurtovic/InstaRaider/issues/46

something for consideration: I'd rather add the json-method as an additional option and leave the default method intact (downloading for one username at a time) to avoid breaking everyone's implementation.

Sorry, I haven't had a lot of time to devote to this project. I'll review the PR as soon as I find some time. Thanks for the contribution.


fix/doc-jetson-installation

This PR fixes the installation for jetson, which is currently broken for the second part. It also has some minor editorial changes to enhance readability.

Besides the fix, 99% of the changes in this PR consists of editorial changes in order to satisfy the markup rules...the diff looks like as if there are many content changes, but this is not the case. You can verify that using the markdown preview in e.g. VS Code.

Well, so please just extract the fix into a separate commit for review.
Searching for a needle in a haystack in not an efficient way to review PRs :)

alright, so it is now split into 'fix' and 'editorial' changes.

hm, what exactly is jenkins complaining about?


Combine onDidInputData and onDidSendText

ITerminalInstance.onDidInputData added in https://github.com/microsoft/vscode/pull/200107

Actually after looking deeper they are slightly different since onDidInputData also fires on xterm.onData, so we shouldn't merge them


[fix][gws][monitor] unable to show proper created/updated time if there are multiple answers

Coverage increased (+0.07%) to 84.643% when pulling bcc41244f78ef53d2c33f098100326d8aebc0a7b on fix-gws-monitor-created-updated into 10e4d10ba4daf7ce184e0faf706f29efb9a58cd8 on master.


Add the side-loaded parl and tag data as JSON files

We created an extrameta CouchDB database.  As we don't intend to create editors for these custom substitutions, one cleanup is to add them to this repository where they can be manipulated by staff in the same way any other static text or data might be.

An FYI on the current size.
russell@russell-XPS-13-7390:~/git/cihm-metadatabus/data$ du -sh *
30M	parl
40M	tag
russell@russell-XPS-13-7390:~/git/cihm-metadatabus/data$

I can use https://prettier.io/ on the files right away, as that is the code formatting we are using for VSCode by default.   This will make reading the .json files much easier.


Creates CODEOWNERS

This will subsequently require the master branch protection rule to be updated to enable "Require review from Code Owners".

@mellis481 i still don’t understand the value of this or even the context in relation to our current flow. So if this is important, we’ll need to set up a mtg so I can show you what the administration of a public Github repo that isn’t part of the Vertex org. Or you can start your own public repo to see if you can find what you’re looking to do. But I personally am not a fan of code owner files and I don’t want to merge it until I know what problem it solves.

We can certainly discuss. Ultimately, I'm the only member of UIEN (currently) who can approve a PR in this repo which is a problem. I'm not sure what permissions you've set, but an approval by Kate currently does not result in the ability to merge. I know the CODEOWNERS file allows for the ability to define people whose approval can result in a merge regardless of their role in the repo.


Increase memory for FlexCI

/test

/test

Merging as the test passed. I think this should prevent flaky failures regarding distributed tests.


Make Last Row uneditable or block any row to make their cells uneditable.

Hi, right now we can make just the columns editable, but I need to make the last row uneditable. ¿Is there any way to achieve this?

@kabala, sorry for lately reply. Use editable={ false } can solve your issue. Thanks


FIX:  KIT reference channels

For UMD files from system ID 52 reference channels seem to be labeled differently. Is it worth adding another small raw file for the tests?
The actual Fix is lines 608-15, everything below is just refactoring I did while debugging and could be reverted.
CC @teonbrooks

Current coverage is 87.38% (diff: 55.55%)

Merging #3721 into master will decrease coverage by <.01%

@@             master      #3721   diff @@
==========================================
  Files           343        343
  Lines         60986      60967    -19
  Methods           0          0
  Messages          0          0
  Branches       9328       9329     +1
==========================================
- Hits          53299      53277    -22
- Misses         4928       4930     +2
- Partials       2759       2760     +1

Powered by Codecov. Last update 31883df...609710c

thanks @christianbrodbeck. I need to dig through the c++ code to check why the sens_type was set to 257. I think I got the check from there. It's on my hard drive, which is currently somewhere in my luggage ;)

I just did a search and it looks like the assignment of sens_type to sqd['i'] isn't even used and the coordinate of the reference channels are defined with just zeros. I think it was just a way to filter for the ref channels.

I just did a search and it looks like the assignment of sens_type to sqd['i'] isn't even used and the coordinate of the reference channels are defined with just zeros. I think it was just a way to filter for the ref channels.

Thanks! Good to know, then that modification should be safe. The problem was just that if a channel had sens_type 0 it was skipped, leading to an erraneous sensor count later.
Do we know the coordinates of the reference channels, which is set to zero currently? I assume if it were in the file you would have used it...

Do we know the coordinates of the reference channels, which is set to zero currently? I assume if it were in the file you would have used it...

I don't know if the position of the reference is written in the file. when I have a chance I can dig into the old code. it seems that they are just set to zero origin.

@agramfort some people here at UMD expressed interest in looking at older data, would it be a lot of work to do a maintenance release if I cherry-picked these commits into the maint/0.13 branch?

yes we can do a maint release
PR on maint branch welcome


Possible in memory DB issue on OSX

MacOS Version :10.15.7
Python Version: 3.7
Neoexpress version: 2.0.23-preview
When following the tutorial here: https://dojo.coz.io/neo3/mamba/tutorial/index.html I seem to encounter the following error if the chain is longer than a few blocks:

2021-05-19 07:49:38,889 DEBUG - node:105 Connected to /Neo:3.0.0-rc1/ @ <IP_ADDRESS>:50013: 1420.
connection_done!
2021-05-19 07:49:38,889 DEBUG - node:396 Waiting for a message.
2021-05-19 07:49:38,939 DEBUG - syncmanager:60 Starting services
2021-05-19 07:49:38,942 DEBUG - syncmanager:254 Asking for blocks 1 - 500 from DRCvlWGQ (blocks in cache: 0).
block received! []
block received! []
...
 Python(45970,0x10d7d7dc0) malloc: Incorrect checksum for freed object 0x7fae946049f8: probably modified after being freed.
Corrupt value: 0x0
Python(45970,0x10d7d7dc0) malloc: *** set a breakpoint in malloc_error_break to debug
[1]    45970 abort      python3.7 main.py

code:
import asyncio

# web specific imports
from quart import Quart, websocket
import logging

# neo3 specific imports
from neo3 import settings, blockchain
from neo3.network import convenience, payloads
from neo3.core import msgrouter
from neo3.storage import implementations as db

msg_queue = asyncio.Queue()

# web specific part
app = Quart(__name__)

with open('./index.html', 'r') as f:
    html_page = f.read()

@app.route('/')
async def index():
    return html_page

@app.websocket('/ws')
async def ws():
    while True:
        msg = await msg_queue.get()
        await websocket.send(msg)

def enable_network_logging():
    stdio_handler = logging.StreamHandler()
    stdio_handler.setLevel(logging.DEBUG)
    stdio_handler.setFormatter(logging.Formatter(
        "%(asctime)s %(levelname)s - %(module)s:%(lineno)s %(message)s"))

    network_logger = logging.getLogger('neo3.network')
    network_logger.addHandler(stdio_handler)
    network_logger.setLevel(logging.DEBUG)

def connection_done(node_client, failure):
    print('connection_done!')
    if failure:
        asyncio.create_task(
            msg_queue.put(f"Failed to connect to {failure[0]} reason: {failure[1]}."))
    else:
        asyncio.create_task(
            msg_queue.put(f"Connected to node {node_client.version.user_agent} @ {node_client.address}"))

def block_received(from_nodeid: int, block: payloads.Block):
    print('block received!', block.transactions)
    asyncio.create_task(msg_queue.put(
        f"Received block with height {block.index} and hash {block.hash()}"))

async def run_neo():
    # set network magic to NEO TestNet
    settings.network.magic = 692804366

    # add a node to test against
    settings.network.seedlist = ['<IP_ADDRESS>:50013']

    # listen to the connection events broad casted by the node manager
    msgrouter.on_client_connect_done += connection_done

    # listen to block received events
    msgrouter.on_block += block_received

    node_mgr = convenience.NodeManager()
    node_mgr.start()

    sync_mgr = convenience.SyncManager()
    await sync_mgr.start()

async def main():
    settings.network.magic = 692804366
    settings.network.seedlist = ['<IP_ADDRESS>:50013']
    node_mgr = convenience.NodeManager()
    node_mgr.start()

    sync_mgr = convenience.SyncManager()
    await sync_mgr.start()

    async def print_height():
        while True:
            print(f"Local chain height: {bc.height}")
            await asyncio.sleep(2)

    # Start an endless loop informing us about our local chain height
    await print_height()

if __name__ == "__main__":
    enable_network_logging()
    loop = asyncio.get_event_loop()
    loop.create_task(run_neo())
    app.run(loop=loop)

Can you please paste the output of python --version.  I'm interested in the minor version (e.g. 3.7.1, 3.7.2 ... 3.7.9, or 3.7.10)

I can't reproduce and we've also dropped 3.7 support. Let me know if this happens on 3.8 and I'll re-open


Convertion functions (bin2dec,hex2dec,oct2dec) does not work in PowerToys Run

Environment
Windows build number: 10.0.18363.836
PowerToys version: 0.18.1
PowerToy module for which you are reporting the bug (if applicable): Microsoft.Plugin.Calculator

Steps to reproduce

Open Launcher (alt+space)
write in the windows "hex2dec(20)"

Expected behavior

UI should show 32
Actual behavior

UI showing nothing
Screenshots

According to the code, the calculator should support bin2dec,hex2dec,oct2dec
but I seem like MagesEngine.Interpret() does not process it.

@baget, MAGES doesn't support it.
https://github.com/FlorianRappl/Mages/blob/master/doc/functions.md

this work item may solve this: https://github.com/microsoft/PowerToys/issues/2265 which will encompass https://github.com/microsoft/calculator/issues/526
Need to validate what items it will enable.


add admin and manager function list team match result

get "api/teams" can get the team match results.


Engineer input prompts

Engineer input prompts
User Story
AS A manager
I WANT to generate a webpage that displays my team's basic info
SO THAT I have quick access to their emails and GitHub profiles
Acceptance Criteria
GIVEN a command-line application that accepts user input
WHEN I select the engineer option
THEN I am prompted to enter the engineer’s name, ID, email, and GitHub username, and I am taken back to the menu

This has been completed


feat: added support for c language

Added support for C language
Added a new color for numeric literals prefixes and suffixes  #90c0c9

Looks good!


Add ssl protocols in system properties to avoid ssl handshake problem

In recent indy prod logs, I found that indy met some "ssl handshake peer shutdown" problem. After google & SO searching, seems this can solve the problem: System.setProperty("https.protocols", "TLSv1,TLSv1.1,TLSv1.2");
So added this property to indy startup script.
P.S: SO discussion here: https://stackoverflow.com/questions/28908835/ssl-peer-shut-down-incorrectly-in-java?utm_medium=organic&utm_source=google_rich_qa&utm_campaign=google_rich_qa

retest this please

+1On May 4, 2018 3:51 AM, Gang Li<EMAIL_ADDRESS>wrote:In recent indy prod logs, I found that indy met some "ssl handshake peer shutdown" problem. After google & SO searching, seems this can solve the problem: System.setProperty("https.protocols", "TLSv1,TLSv1.1,TLSv1.2");
So added this property to indy startup script.
You can view, comment on, or merge this pull request online at:
  https://github.com/Commonjava/indy/pull/945
Commit Summary
Add ssl protocols in system properties to avoid ssl handshake problem
File Changes
M
deployments/launcher/src/main/bin/indy.sh
(2)

Patch Links:
https://github.com/Commonjava/indy/pull/945.patchhttps://github.com/Commonjava/indy/pull/945.diff
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.


Allow adding an arrow to the MouseCoordinateX

Hey, I need to have an arrow displayed where the line meets X axis on MouseCoordinateX, like here:

How can I do that? Thank you!

Also, it would be great to have an option to add padding inside the rectangle so that there is more space between borders and the text

done, available in - http://rrag.github.io/react-stockcharts/documentation.html#/interactive_y_coordinate


Fix certificate generation

Problem statement:

In Cloudmos we have our own certificate generation which has been working for many years without issues: https://github.com/akash-network/cloudmos/blob/main/deploy-web/src/utils/certificateUtils.ts
We need to merge or replace the code here: https://github.com/akash-network/akashjs/blob/main/src/certificates/generate509.ts

An error from a user:

"TypeError: Cannot read properties of undefined (reading 'Integer')" , at createCSR function of /node_modules/@akashnetwork/akashjs/build/certificates/generate509.js:39:46
Cannot read properties of null (reading 'getAlgorithmParameters')
const cert = await createCertificate(address);

TODO:

[ ] Merge/replace certificate generation code into AkashJS
[ ] Add tests

Any updates? or a workaournd for now?

@Charizard13 this should fix the issue. We're still working to sync implementation with akash-network/cloudmos


Enable autofocus on searchbar in ReturnDetails, PODetail, and ShipmentDetails page

What is the motivation for adding/enhancing this feature?

Add autofocus on ion-inputs of all details page
What are the acceptance criteria?

[ ] ...

Can you complete this feature request by yourself?

[ ] YES
[ ] NO

Additional information

Closing. Duplicate issue.


Switched to AsyncKeyedLock library

This gives better performance and reduced memory allocations.

@donaldgray will this need to wait just like https://github.com/dlcs/protagonist/pull/435 ? They're very related.

@donaldgray ?

Morning @MarkCiliaVincenti - Thanks for opening this PR, it is appreciated but we have discussed this change internally and decided against pulling it in.
The AsyncKeyedLock class we have is an intentionally naive implementation that has been used in a very high-traffic production system for almost 2 years without issue so we will stick with this for now and avoid pulling in an additional dependency.

Morning @MarkCiliaVincenti - Thanks for opening this PR, it is appreciated but we have discussed this change internally and decided against pulling it in.
The AsyncKeyedLock class we have is an intentionally naive implementation that has been used in a very high-traffic production system for almost 2 years without issue so we will stick with this for now and avoid pulling in an additional dependency.

Does this mean the same fate for https://github.com/dlcs/protagonist/pull/435 ?
I understand the "if it ain't broke..." but I wonder if you did run some benchmarks to see how much of a 'real life difference' it'd make.


docs - Fixed typo

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

Looks good; thanks, @sethflowers.


RC4 not available on client

I use ubuntu 18.04
my target is windows 10 64bit
when I run the script by the following command I got those errors:
#./seth.sh eth1 192.168.10.{59,57,10} calc
███████╗███████╗████████╗██╗  ██╗
██╔════╝██╔════╝╚══██╔══╝██║  ██║   by Adrian Vollmer
███████╗█████╗     ██║   ███████║<EMAIL_ADDRESS>╚════██║██╔══╝     ██║   ██╔══██║   SySS GmbH, 2017
███████║███████╗   ██║   ██║  ██║   https://www.syss.de
╚══════╝╚══════╝   ╚═╝   ╚═╝  ╚═╝
[] Linux OS detected, using iptables as the netfilter interpreter
[] Spoofing arp replies...
[] Turning on IP forwarding...
[] Set iptables rules for SYN packets...
[] Waiting for a SYN packet to the original destination...
[+] Got it! Original destination is <IP_ADDRESS>
[] Clone the x509 certificate of the original destination...
[] Adjust iptables rules for all packets...
[] Run RDP proxy...
Listening for new connection
Connection received from <IP_ADDRESS>:49802
Warning: RC4 not available on client, attack might not work
Listening for new connection
Downgrading authentication options from 11 to 3
Enable SSL
test::DESKTOP-V50VN9K:b6f9cc86441228f9:bea6860fecda0cd86cd5b23b4216b602:010100000000000022972acf6b8ad60193e25ae7f86e77d20000000002001e00570049004e002d003700370032004400320052003600470033005300370001001e00570049004e002d003700370032004400320052003600470033005300370004001e00570049004e002d003700370032004400320052003600470033005300370003001e00570049004e002d00370037003200440032005200360047003300530037000700080022972acf6b8ad601060004000200000008003000300000000000000001000000002000002ebd4ef70e6bb9b6b73d66d1da7808e5aba82dc2bcb98547d30c8db0de025fdc0a0010000000000000000000000000000000000009002a005400450052004d005300520056002f003100390032002e003100360038002e00310030002e0031003000000000000000000000000000
Tamper with NTLM response
Downgrading CredSSP
Connection lost ([Errno 104] Connection reset by peer)
Connection received from <IP_ADDRESS>:49809
Warning: RC4 not available on client, attack might not work
Listening for new connection
Server enforces NLA; switching to 'fake server' mode
Enable SSL
Connection lost on enableSSL: [Errno 104] Connection reset by peer
Connection lost on run_fake_server
Connection received from <IP_ADDRESS>:49810
Warning: RC4 not available on client, attack might not work
Listening for new connection
Enable SSL
'NoneType' object has no attribute 'getsockopt'
Hiding forged protocol request from client
.\test:test123
[] Cleaning up...
[] Done

Hey there,
it's just a warning. You can ignore it, especially if the password appears in the output, which means the attack was successful.


GSSoC : Link the options on hero section to the respective pages

Description
The options given in the hero section should be clickable and link to the respective page.
Changes You need to make
I will direct the links to the respective pages.

file1
file2
file3

Screenshots

Checklist
Before submitting your pull request, ensure that you have completed the following tasks:

[x] Gone through the rules and guidelines of the project.
[x] When you are creating PR, your PR should title follows the specified format: [Feature/Enhancement/Bug Fix]: Brief description of the change.
[x] Attached OSSI or GSSoC in the heading of the Issue.
[x] I have checked this issue is not already covered in another issue.

Expected Time to Complete
1 day
Additional Notes
Add any additional information, concerns, or considerations that may be relevant to your pull request. This could include performance implications, future enhancements, or potential risks associated with the changes.

I also felt the same! Go ahead.


Increment Version

@JuliaRegistrator register() SequentialSamplingModels v0.1.3

Registration pull request created: JuliaRegistries/General/35018
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.3 -m "<description of version>" 9f510d32d821bd02ca7feca816759e5a6469b272
git push origin v0.1.3


I am not able to add v7.0.0 as a working dependency, 6.1.1 works

Is this a regression?
Yes
Description
If I add v7.0.0 as a Package Dependency, it never fully resolves, leaving me in a state where I can't link it to my target, or import it into Swift files.
This works fine with v6.1.1
Please provide the exception or error you saw
When adding the Swift SDK as a Package Dependency, it should resolve all the packages and it should show the display name of the package.  The package list should allow me to open each package and see its contents. This does not happen with v7.0.0 for me.

v6.1.1 works fine.

Please provide the environment you discovered this bug in
Using from the 7.0.0 Tag in Xcode Package Dependency screen

Anything else?
Here is how my Package Dependency screen looks when I tag it to 7.0.0:

You can see that the packages aren't fully resolved:

v6.1.1 works fine, shows up as it's name "EdgeAgentSDK" (not the slug name) :

And 6.1.1 can be found when I add it to my Target, 7.0.0 does not load fully so it's not available to this menu.

Note, that v6.1.1 does not actually compile, but the SPM appears to install correctly.

I deleted old 7.0.0 and released a new one with the dependency bug solved. Should work ok now.

This works now, I was able to include v7.0.0 successfully in Xcode.  thank you!  I will close.


WIP: Replace nose with pytest, refs bug 1194993

This references https://bugzilla.mozilla.org/show_bug.cgi?id=1194993 and uses my fork of https://github.com/EnTeQuAk/unittest2pytest for an mostly-automatic conversation.
This isn't ready yet as the conversion leaves a few code-style issues and it's not yet complete.
Todo:

[x] Replace common unittest assert* statements to py.test assert statements
[x] Replace eq_ with py.test assert statements
[ ] Replace ok_
[ ] Replace assert_not_equal
[ ] Replace assert_raises
[ ] Replace assert_false
[ ] Fix code-style issues (lines too long) from automatic conversion and remove exclude from make flake8
[ ] Cleanup imports, remove nose.tools

I'll try to fix the code-style issues to that this passes and move all other points to separate issues/branches/pull requests to keep the level of review managable. This PR is already huge.

Closing this in favor of restarting the work on this branch with a clean base from master some time later and step-by-step.


Add action + output

added ses:SendEmail action
added secret var output

Please see my comment here. For creating IAM users, I'd like to point you to this other module: https://github.com/Flaconi/terraform-aws-iam-user


Resizing multiple animations on the same page

I have three animations that I'd like to resize when the window size changes. I use the code below:
  var anim = []; // collecting animation instances in this array
   /**
    *  Animations
    */
   $('.bodymovin').each(function(index, el) {

   	var params = {
   		container: $(this)[0],
   		renderer: 'canvas',
   		loop: true,
   		autoplay: false,
   		autoloadSegments: true,
   		path: $(this).data('path')
   	};

   	anim[index] = bodymovin.loadAnimation(params);
   	bodymovin.setQuality(1);
   	anim[index].playSegments([[1,2],[$(this).data('segment-0'),$(this).data('segment-1')]],true);
   });

   $(window).resize( function(){
   	$.each(anim, function(index, el) {
   		console.log(el); // this outputs the correct animation instance
   		el.resize.bind(el);
   	});
   })

This will not resize any animation canvases even though I keep a separate instance of each animation in a global array and console.log outputs the correct animation instance. An earlier version was more similar to #106 but was only resizing the last element on the page.

on your iteration you're binding the resize method to the element but you're not actually invoking the function.
you should do
el.resize.bind(el)();
but it would be better if you do the binding outside the window.resize so you're not creating the binding function each time.
var anim = []; // collecting animation instances in this array
   /**
    *  Animations
    */
   $('.bodymovin').each(function(index, el) {

   	var params = {
   		container: $(this)[0],
   		renderer: 'canvas',
   		loop: true,
   		autoplay: false,
   		autoloadSegments: true,
   		path: $(this).data('path')
   	};
   	anim[index] = bodymovin.loadAnimation(params);
   	bodymovin.setQuality(1);
   	anim[index].playSegments([[1,2],[$(this).data('segment-0'),$(this).data('segment-1')]],true);
        anim[index].resize.bind(anim[index]);
   });
   $(window).resize( function(){
   	$.each(anim, function(index, el) {
   		console.log(el); // this outputs the correct animation instance
   		el.resize();
   	});
   })

On an unrelated issue keep in mind that the playSegments needs the animation to be loaded in order for it to work.

Thank you for your quick answer. The edits you suggested now produce the following Uncaught TypeError when resizing the window:

Cannot read property 'canvas' of undefined

can you share a link with the code implemented?

I have sent you the link via email as the site is not yet lanched


The left side of the "Skip to content" button's outline is not visible

Describe the Issue
It seems that the left side of the "Skip to content" button's outline is extending off the screen.
Affected Page
Pages with Header component
Steps to Reproduce

Go to /learn page
Press Tab key once to be focused on the "Skip to content" button and make it visible
See the outline of the button

Expected behavior
The position of the button should be properly aligned and the outline should be visible.
Screenshots

"Skip to content" button in /learn page

System

Device: Laptop
OS: Windows 11 Home
Browser: Chrome
Version: 123.0.6312.58 (Official Build) (64-bit)

Additional context
With reffering this fCC's article, the button and its outline should probably be aligned to the left edge of the search bar (on screens wider than or equal to 980px) or the fCC logo (on screens less than 980px).

hello @gikf,
Could you please explain the expected behavior where the "skip to button" will be aligned correctly on the page, and also confirm if my understanding is correct that the skip to button should only be visible when the tab is clicked?


unbound-anchor: prioritize system-wide DNS servers over public roots

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
We use the unbound resolver in forwarding mode on several OPNsense boxes hosted in internal networks. These unbound instances are used as caching DNS servers, and have no connectivity to the root DNS servers living on the Internet.
At unbound startup, unbound-anchor is run to update DNS roots. By default, unbound-anchor tries to contact DNS root servers. Since the box has no connectivity to foreign DNS servers, unbound startup is delayed until unbound-anchor gives up (timeout).
In such a context, running unbound-anchor with the -Rf /etc/resolv.conf argument is appropriate, as it would prioritize system-wide DNS servers from resolv.conf (which typically are upstream servers) over public roots to update the keys, while still allowing falling back to public roots on failure.
This is the only way to perform successful anchor updates at startup in offline network environments such as ours, and avoid a 60-second delay each time the unbound process is restarted.
Describe the solution you like
Modify unbound startup script at /usr/local/opnsense/scripts/unbound/start.sh to make unbound-anchor use the servers set in resolv.conf, still allowing a fallback to public roots on resolution failure:
--- /usr/local/opnsense/scripts/unbound/start.sh        2022-01-27 11:25:21.000000000 +0100
+++ start.sh    2022-02-14 13:45:50.232509000 +0100
@@ -30,7 +30,7 @@

 # prepare and startup unbound, so we can easily background it

-chroot -u unbound -g unbound / /usr/local/sbin/unbound-anchor -a /var/unbound/root.key
+chroot -u unbound -g unbound / /usr/local/sbin/unbound-anchor -a /var/unbound/root.key -Rf /etc/resolv.conf

 if [ ! -f /var/unbound/unbound_control.key ]; then
     chroot -u unbound -g unbound / /usr/local/sbin/unbound-control-setup -d /var/unbound

Describe alternatives you considered
Replace the startup script with our own amended version.
Additional context

Already reported in issue #4427 (now closed)
Same issue as #5441 (but different cause)

This is a good solution for an offline environment, and I agree that it makes sense to use /etc/resolv.conf in this case. Especially since you're assuming a pure forwarding environment you wouldn't expect Unbound to make any recursive queries itself to bootstrap. However, I'm worried as to how generic this solution is.
When specifiying -Rf /etc/resolv.conf, regardless of whether query forwarding is enabled, Unbound will forward to any nameserver listed in resolv.conf in order - including localhost (which of course hasn't been bootstrapped yet). Besides it maybe being unwanted behaviour for security reasons, any nameserver listed here not responding in time will create a delay in the completion of unbound-anchor. This is even worse if you factor in purely validated environments in which all UDP port 53 traffic might be blocked and instead DNS-over-TLS is used. If it's blocked upstream, it will take even longer and make it very hard to debug.
I'm honestly wondering about the usefulness of unbound-anchor as before this program was shipped, by default, a root.key file was shipped with it instead. Also, the failover implemented in it writes out a hardcoded root.key file which gets updated as soon as Unbound has started - via the correct channels (be it forwarding, or DoT setups).
Ideally this addition would be dependent on the fact that query forwarding is enabled. Also, as a general optimization, I'd make unbound-anchor run only if no root.key file is present, or is empty.


Remove deprecated API paths

and add some missing swagger docs

Removing /hook probably needs a migration for old webhooks to work

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

Comparison is base (31240b5) 34.09% compared to head (d2a2ca7) 34.06%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2639      +/-   ##
==========================================
- Coverage   34.09%   34.06%   -0.03%
==========================================
  Files         204      204
  Lines       13109    13109
==========================================
- Hits         4469     4466       -3
- Misses       8288     8291       +3
  Partials      352      352

Files
Coverage Δ

server/api/stream.go
0.00% <ø> (ø)

... and 2 files with indirect coverage changes

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

We could think about adding a repair button to the admin settings

Something like this? Or do you mean a button for each repo on this page?


Is there any way to add some kind of @Tags in testcafe tests, and we can use these tags to run different combination of tests. it will help in maintain different suite

Are you requesting a feature or reporting a bug?
Feature
What is the current behavior?
What is the expected behavior?
some kind of tags can be added in test meta section, which can categories test while creating , later we can use these tags to run different combination of tests, like smoke, regression etc.
How would you reproduce the current behavior (if this is a bug)?
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code

Specify your

operating system:
testcafe version:
node.js version:

Hi @vivekpant28, currently TestCafe doesn't have such feature, but there is a feature request for it: https://github.com/DevExpress/testcafe/issues/2527. So I close this issue as a duplicate of https://github.com/DevExpress/testcafe/issues/2527.


Getting undefined when trying to login

when I try to login with the following script:
const Instagram = require('instagram-web-api')

const client = new Instagram({ username:"kc.spnl", password:"password" })

client.login().then(() => {
    client.getProfile().then(console.log()))
})

It just shows me undefined

Same problem i can't event login

You need to do this in asynchronous function.

You need to do this in asynchronous function.
I eventually did that like this:

(async () => {
    const Instagram = require('instagram-web-api')

const client = new Instagram({ username: 'kc.spnl', password: "FietsBel2312:" })

client
  .login()
  .then(() => {
    client
      .getProfile()
      .then(console.log)
  })

})();(async () => {
    const Instagram = require('instagram-web-api')

const client = new Instagram({ username: 'kc.spnl', password: "PASSWORD" })

client
  .login()
  .then(() => {
    client
      .getProfile()
      .then(console.log)
  })

})();

Still no luck


empty prefix declarations

swagger generates empty turtle prefix generations. I tried this with the example turtle input for the calls for e-entity, e-link and e-terminology. The generated CURL request starts in the data provided via -d like this

@prefix itsrdf:  .
@prefix xsd:  .

Seems to be a bug in swagger. I raised https://github.com/swagger-api/swagger-ui/issues/1978

Maybe then this section must remove at next release while not resolved. Better is no information than wrong information.

+1 to @andish . @jnehring , please consider removing the part that Andish mentioned if no fix is available.

The problem has been fixed by swagger: https://github.com/swagger-api/swagger-ui/issues/1978 but we need to update when we want to use this.

This is works since the recompile of the swagger-ui. Using the newest swagger-client mentioned here: swagger-api/swagger-ui#1978  destroys it again.

Can we close this?

From my side: yes


MODE-2444 Teiid DDL Importer Processor Error: "Unparsable table body constraint" when importing from SAP HANA

There were 2 issues: (1) INDEX was used as a column name in a column definition but was being parsed as a table element constraint, and (2) the parser was not handling embedded quotes in string literals. The parser was modified to handle the way the SQL spec allows for escaping embedding quotes (which is 2 consecutive quotes). Also the parser was changed to look ahead at the next token when it finds a keyword like INDEX when parsing table bodies. Added test cases and the full HANA DDL. This DDL file is huge so only the statements that test the 2 issues are left uncommented.

looks good, thanks @elvisisking

@rhauch thoughts ?

Looks good.


Merge cfg-if into std?

Seeing as search engines are useless for searching for if this has been answered before I'll ask the question directly here. I apologize if it has been answered elsewhere.
crates.io (total):

lib.rs (monthly):

This crate is being used by an absurd amount of users and according to lib.rs >68 000 crates. Which states quite clearly that cfg-if is sorely wanted by Rust users. This crate is also very simple only supplying a single macro that is below a hundred lines of code.
There is thus a quite obvious question to ask: Why is this not part of the standard library? Why is it instead maintained as a separate crate by the Rust team?
I'd love to see this as part of the standard library as it'd be one less dependency one has to worry if it is going to be xz'ed in the dependency tree or not.

See https://github.com/rust-lang/rust/issues/115585

Thank you. <3
My searching definitely failed me.  I'll close then.
Hopefully it'll be implemented soon.


Allow spell check for Internet Explorer 11

Currently, Draft explicitly excludes IE11 (and below) from the spell check feature. The reason is that the input event is not fired when a misspelled word is corrected, so Draft doesn't know that it needs to update its internal state.
Technical reasons notwithstanding, it's still a shame that this doesn't work. Isn't there some way to work around this?

Hey @robbertbrak
It sucks that we lack support for spellcheck in IE11, but so far we haven't be aware of a good solution for it.
We'd be open to a patch if there is a technical path someone sees that doesn't add a lot of overhead

Just hitting this limitation now.
I wonder how the Gmail editor gets around this?  It seems to be contentEditable=true + spellchecker enabled with no issues?


Fix for React/Node

Is there a way to use this (and packery) in a React/NodeJS environment?

Thanks!

Hello! Thanks for reporting this issue.
Draggabilly was designed for the browser. See https://github.com/desandro/draggabilly/pull/101#issuecomment-140589998

I feel like the conditional should be higher up — this module shouldn't be loaded in the first place

That said, I'm open to improving support for server-side rendering, so these errors don't get in the way of your development. Could you help me understand what you're looking for?

I'm not sure what the fix is. But the error I get is:
packery.js:481
})( window );
^
ReferenceError: window is not defined
I've tried some node/react ports but they don't seem to work. Would be great if it was compatible.

@jrock17 you can add the following in your entry point:
if (typeof window === 'undefined') {
    global.window = global
}

This is more of a hack and it should work for your situation.

Draggabilly v3 has been released that checks for window and falls back to this.


[FeatureRequest] Decode gzipped files in tf.FixedLengthRecordReader

My dataset is stored in binary format that is easily compressible (from 100 MB down to ~0.5 MB), but I currently cannot make use of this in TF, unless I write a TFRecords file, which is way more cumbersome and leads to a lot of duplication -- I need to work with those files in other programs as well, so the binary format is the easiest solution for me. It would be nice if  tf.FixedLengthRecordReader would support on-the-fly decompression.

We would happy to accept a batch that allows this behavior, but we are unlikely to work on it ourselves, since we either use TFRecords or other data formats internally. It shouldn't be super difficult to do yourself. @saxenasaurabh can probably give you some direction on how to use the existing gzip classes, if you'd be willing to help out.

This shouldn't be too hard imho. You should be able to swap out the input_buffer_ in FixedLengthRecordReader with the ZlibInputStream. You may need to subclass the InputBuffer from InputStreamInterface though.
You can specify the compression_type in a manner similar to the TFRecordReader and add it to the op registration similar to here.

@untom @aselle @saxenasaurabh I created a PR #8901 to add gzip and zlib compression support for FixedLengthRecordReader. Please take a look.


Using keyboard to go down list for add members causes members to resort

Rancher versions: 2.0 master 3/3
Steps to Reproduce:

Create 5 users (testuser1 - testuser5)
Create a cluster
Add member and type in test
Now use your keyboard to try and go down the list

Results:  The list resorts everytime I hit the down arrow key making it very difficult to pick a specific user.

Version - 2.0 master 3/3
Verified fixed


Add Talkdesk

We at Talkdesk are rolling out our first system written in Elixir.

Woohoo :tada:!  Congrats @tjsousa.  Thanks for the contribution :+1:


fix!: Cap merkelized gas used to be gas wanted + 1

Description
Cref https://github.com/cosmos/cosmos-sdk/issues/12788
Caps the maximum gas_used field that gets merkelized to be gas_wanted + 1. It emits an event so folks can still use their old debugging flows (granted the entire gas debugging flow should be improved, using gas usage traces, and by getting better visualizers for them)
This should really be a sentinel value, but thats not currently possible without more involved tendermint & ecosystem changes.
In general, I think the design of the ABCI TxResults is pretty wonky, and needs to be rethought. Its unclear to me why gas used amounts are merkelized in Tendermint -- they should be deterministic across versions, but this should be checked / ensured at the application layer.
(This PR still needs tests added)

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[x] added ! to the type prefix if API or client breaking change
[x] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[x] added a changelog entry to CHANGELOG.md
[x] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Going to close this, I think were in favor of no longer merkelizing this tendermint side


Outdated Documentation

Issue in ./help/c-implementing-target/c-implementing-target-for-client-side-web/t-mbox-download/target-download-config-mbox.md
We need to upgrade this document to reflect both mbox.js as well as AT.js download instructions.

That topic is specifically for mbox.js and will go away as soon as that library is deprecated. The instructions for at.js are in the at.js section: https://docs.adobe.com/content/help/en/target/using/implement-target/client-side/deploy-at-js/how-to-deployatjs.html


Add dependabot

To add Dependabot to a project, create a dependabot.yml file in project root. Configure it to check for updates in  project's dependencies, and set up notifications for automated pull requests. Dependabot will then help  keep  dependencies up-to-date, enhancing project stability and security.

Hey thanks for raising the issue you have been assigned this @Akshayp02 you can start working on it 😄


default Spectral coloramp in smudge plots not Colour Blind Friendly

The default brewer coloramp (get_default_col_ramp.R) in the smudge plots is very problematic for color blind individuals can you please change it to a perceptually uniform, qualitative, colour blind friendly palette like viridis (https://cran.r-project.org/web/packages/viridis/vignettes/intro-to-viridis.html).
> brewer.pal.info["Spectral",]
         maxcolors category colorblind
Spectral        11      div      FALSE

Thanks

Uf, I thought that Spectral is colorblind friendly. Sorry, my bad. I will update it.

Thanks for that.
Best wishes,
Graeme

Alright, we made two changes.

First we changed the transformation we use for the kmer frequencies from quadratic to log scale. To make sure that people get a good idea about the genome we now report also non-transformed smudgeplot.
we use viridis palette


Feature: Use F-Droid for deploying

As this is a FOSS App, you can  publish it in f-droid.
Have a look at their Inclusion Policy.

There have been multiple pushes to get this in to the f droid repo, held up by problems with one of the included libraries, bypass. The problem is with the build chain, specifically around Maven and gradle. Check out the issue for more info. https://github.com/ekx/GitLabAndroid/issues/10

:+1: Still waiting :P - or a deploying on a share - way from play.store (i am using a Jolla)

Sorry guys, certainly working on this. Trying to get a version of the app in a working state, then I will deploy the new version to the Play Store and go for getting it on f-droid as well.

Hey peeps, submission has been made. But who knows how that will go. For now, if you want to download on devices without the Play Store, I will be adding the release APKs to the releases on GitHub. See:
https://github.com/Commit451/GitLabAndroid/releases/tag/2.0.0

Wow thank you. This app is great ;)

afair it cannot be added to fdroid as long as it uses Fabric and requires key to build. I might be wrong, but it was added to Fdroiddata and never enabled:
https://gitlab.com/fdroid/fdroiddata/commits/master/metadata/com.bd.gitlab.txt
@mvdan?

Using a third party repository is not allowed right now. We won't sign up for api keys for apps, so if the app requires one to function and doesn't include it, we can't publish it.

Hmm, its unfortunate that fdroid has to be so restrictive. I mean, it doesn't need to be a valid API key if you do not care about tracking crashes. Nonetheless, thanks @mvdan for looking into it.

Hmm, its unfortunate that fdroid has to be so restrictive

That's implied by name (FOSS-Droid) and it's a feature.

I mean, it doesn't need to be a valid API key if you do not care about tracking crashes

just switch to ACRA. ACRA is OSS and is used in many apps on FDroid, difference is we ahve to host webservice ourself to browse reports. I was going to fork Gitlab-android anyway because it's buggy, doesn't support system themes and its UI is terrible. I can help you with this in my free time.

If the api key is for reporting crashes, I assume not having one won't keep the app from building and working. That would be fine by us.
If not having an api key renders the app useless, that's when we won't package it, because it would make no sense.
Either way, requiring an API key to report crashes seems overly complex to me. You can use ACRA, like @agilob suggests.

Any progress?

Marvellous to see this included in f-droid this week, good work folks.

Yeay cool, thanks :+1:


Always run onSet and onRemove regardless of environment

This partly includes the fix from https://github.com/reactivestack/cookies/pull/94 without the update to the tests, so ideally we should merge that in first and rebase this branch.
I would like to run onSet and onRemove irrespective of the environment, I couldn't think of any reason to only run it on node since the browser does not provide hooks to support hook.
Also note that the onSet/onRemove is before setting the cookie, this is to allow the onSet to manipulate the options which is passed by reference. It shouldn't cause too much issue of existing usages however it may be considered as a breaking change if a usage of onSet is also subsequently has a reference to the cookies instance and is querying it.

I like the idea of always running hooks. However, I think changing the option object using onSet is an undesirable side-effect that would make the code less intuitive to read. It's a bad practice to change an input object. Thus I don't consider this change a breaking change.
Do you have use-cases where it is necessary or practical?

The reason I am using onSet/onRemove is to override default settings. For example, onSet in node.js I want to check if the request is secure and equally set the cookie as secure alternatively you may want to always set the cookie as HttpOnly for all cookie requests and having to pass it in manually every time is error prone. This also applies on the browser where I want to set the path to be "/". Perhaps a global defaults would be a better solution?


[FEATURE] update model group

In 2.8, we introduces model access control in ml-commons. But we didn't extend this functionality in opensearch-py-ml yet.
We want to add update_model_group: https://opensearch.org/docs/latest/ml-commons-plugin/model-access-control/#updating-a-model-group
request parameters: https://opensearch.org/docs/latest/ml-commons-plugin/model-access-control/#request-fields-1
example request: https://opensearch.org/docs/latest/ml-commons-plugin/model-access-control/#example-request-1
example response: https://opensearch.org/docs/latest/ml-commons-plugin/model-access-control/#example-response

Can I work on this issue?

Sure, assigned it to you.

@wuzhijing0127 Do you have any update on this issue?

@wuzhijing0127 Due to the urgency and no update from you, I'm going to assign this issue to @rawwar
@rawwar could you please comment in this issue, so that I can assign it to you?


Update schema reloading section

Right now it only mentions it's needed for resource embedding. But now it's needed for other things.
See https://github.com/PostgREST/postgrest/issues/1376.

Also add how to send docker a SIGUSR1

Oh and just as a note, the existing example for pg_listen on the 5.2 docs no longer works
pg_listen <db-uri> ddl_command_end "killall -SIGUSR1 postgrest"
should be changed to
pg_listen <db-uri> ddl_command_end /usr/bin/killall -SIGUSR1 postgrest
I think PATH support went away around pg_listen commit #6 and I did not notice until this week.


Extended spritesheet

New features:

Takes default pivot points from sprite sheet if present in the data
Sprite sheet has function to receive animations defined in a sprite sheet

Tests and updated documentation included.

Btw, I've added polygon-packing feature in pixi-heaven plugin, but changes are too big, need to consult with @bigtimebuddy @GoodBoyDigital if we add it in v4 or v5.

I assume you've made changes in pixi template for texturePacker?

We've added a property called "anchorPoint" that holds the anchor point if enabled in TexturePacker.
We've also added a map of animations.
Both are optional in the loader and only used if present.
Do you have an ETA for the next Pixi release with these features? We'll have to wait with the next TexturePacker release until the feature is available in Pixi.
About the Polygon-Packing: This is great news. We are happy to enable the feature with one of the next updates. Let's coordinate this after the current release.

it would also be great to see the option "PACK WITH SAME LAYOUT" .
I made my own hack to use it, but it not very operational.
https://github.com/pixijs/pixi-display/issues/30

Is the release going to be 4.9.0 or 4.8.2?
We have to know this for the TexturePacker release notes... people should know which version of Pixi they have to use.

looks like 4.8.2

Might be just me, but a Texture having a property called anchor, and a Sprite having a property called anchor... isn't this ripe for some confusion? Won't people think about changing the texture anchor and expecting changes for existing sprites?
Could it be called defaultAnchor or something like that?

@CodeAndWeb
First , thank a lot for this new update it just very nice for me and my workFlow.
I have a little problem with auto-animations.
It possible to add a special suffixe ? or regex ? for autoDetect from the end if we add a suffix ?
Because it not work in context  of my animation sequence have a special suffixe at the end.
How i can manage with this ?
I don't use Pack with same layout for now, because it not yet implement in PIXI loader.

thanks a ;lot

Hey @CodeAndWeb quick question, is the anchor optional? As in if i don't specify it in Texturepacker it will not export an anchor point?

@GoodBoyDigital You only get anchor points in the file if you specify them in TexturePacker.
The pixi loader applies them when found while creating a new sprite. They can be overwritten from code at any time.

@CodeAndWeb thanks, just made a few doc changes and resolved conflicts.
I want to double-check that TexturePacker generates anchor values (float from 0-1) and not pivot (real pixel values), is this correct? There were some places in the your docs and in your description in this PR where you refer to it as a "pivot". Sprite has both anchor and pivot.

@bigtimebuddy TexturePacker exports anchor values between 0..1 (or bigger, if anchor is outside of the sprite), the key in the JSON format is also called "anchor"

I'd like to notify that in my case, upgrading from 4.8.1 to 4.8.2 broke loading textures with PIXI.loader.add(path), as they will miss the defaultAnchor property, throwing an exception (Cannot read property 'x' of undefined) when creating a sprite via new Sprite(texture). When created with PIXI.Texture.fromImage they instead have defaultAnchor and sprite creation works.
This might be or not an issue, perhaps my mistake but unfortunately i don't have more time to investigate/debug/open a real issue, so i thought i'd just leave a comment here.

@djmisterjon or @pietrovismara could either of you create a jsfiddle or pixiplayground.com that reproduces this? I've been unable to reproduce and I'm not sure if I'm missing something.


Add support to import Blocklist

First iteration of blocklist support

Adds support to simple domain list file (one domain per line)
Adds support to Oliphant blocklist format (https://writer.oliphant.social/oliphant/the-oliphant-social-blocklist)

In this version it blocks all domains in the list since we don't have yet support to globally silencing domains.
relates to #613

Thanks for you work on this!


[wip] [backport 7258] add .fa margin like .octicon

Backport of #7258
My local config seems to not generate the same css.
I try to clean it but it still have some browser specific css that wasn't there before.
I will retrieve the content genretate from CI and duplicate (that why the WIP).

I can't seem to be able to build the correct css and CI doesn't output the full diff. If someone can rebuild it and push it that would be perfect. I will also try to do it on another computer later.