Using trimmed DLog keys doesn't work for Darlin proving system

It seems that the latest version of CCTP Lib, that removed the possibility to initialize the DLog keys several time at runtime, makes the MC Crypto Lib to fail some unit tests. See this comment for further details.
As a brief description:

Generate small DLog Keys (2^9 segment size) and generate proving and verification keys
Generate large DLog keys (2^18 segment size) and try to generate a proof with the proving key of the last step using the small segment size (2^9) so that the DLog keys get trimmed
Generate large DLog keys (2^18 segment size) and try to verify the generated proof with the verification key from step 1

These steps work correctly when using Coboundary Marlin, while we get an error during the proof generation at step 2 for Darlin.
The behavior can be reproduced by launching the unit test in the mcTest folder on the branch pointed by #58.

@i-Alex in reality it's a different issue here (also involving DLOG keys, these objects are very delicate :D)

The issue is due to an error fixed in https://github.com/HorizenOfficial/zendoo-mc-cryptolib/commit/540fe8b0d8fd6d072ea82588c6830d4237198ed0 as the segment_size parameter was not passed to the generate_params function from the create_test_cert_proof and create_test_csw_proof functions


Removed shops reappear on reload

Server version:
git-Paper-1618 (MC: 1.12.2)
Plugin version:
ShopChest-1.13-SNAPSHOT
Description:
Removed shops will reappear if the chests have remained.
Time of occurrence:
The server is (re)loading the plugin.
Effect(s):

Players may not able to use the chest if it was a shop.
Admin shops cannot move their location and change their price at the same time, as the players will abuse them to launder money.

Possible reason(s):

The database doesn't remove the shop data when removing a shop.

Temporary solution(s):

Commit changes to the database manually to remove the shops.

I suppose there haven't been any messages somewhere in the console indicating that the database connection is instable?

I suppose there haven't been any messages somewhere in the console, indicating that the database connection is instable?

The console doesn't show any error message when removing shops, even if I enabled the debug log.
I forgot to point out that the plugin will show 'No Chest found in world {world} at location: {location}' if the chest isn't there anymore. Therefore, I am pretty sure that the database remains the record of removed shops for some reasons.
Although I am facing this problem, the function of creating shops works perfectly. I guess the connection doesn't do a commit to the database when the removal is done?


[BUG] [Dev Branch] ACF entities unweld themselves when updated

Short Description
When an ACF entity is updated, it loses any weld constraints it had.
How to Reproduce
Spawn an ACF engine or gearbox, weld it to the ground or a prop. Update said engine or gearbox and the weld is gone.
Additional Information
This seems to only happen with weld constraints specifically.

It seems to be a problem related to how constraints are saved, noticed that it happened once to me due to the constraint table missing it's Type value on one of the entries. It's also a bit inconsistent from what I could tell, I couldn't replicate it on my server with the given instructions, so it's definitively an interesting issue.

Added something on 07bd535 that should help with this issue. Noticed certain stuff such as weld latches would create a constraint but change their Type value to an empty string, which could actually break the ACF.RestoreEntity function. Still no luck at trying to replicate this issue with regular welds.

Tested again just now on the procedural-armor branch (which I assume is based off dev?) and it's still happening - video https://streamable.com/cg77ot

Tested again just now on the procedural-armor branch (which I assume is based off dev?) and it's still happening - video https://streamable.com/cg77ot

Pretty sure I got it right this time on c58f8be. Constraints with the world seem to be restoring just fine now. It should now be able to deal with wire hydraulics aswell.
Not gonna close it instantly, since as you mentioned it's a really inconsistent issue.


Add slide "End of Fellowship" and remove shiields

Add slide link to the End of Fellowship Report and remove shields when linking presentations.

Accepting PR #36.


Update Chrysler Pacifica fingerprint values

add a few more values based on work by @arne182
Combine fingerprints for Pacifica 2019 Hybrid.

@adhintz are there cars with those fingerprints or they are just a superset (union) of different models?

@rbiasini They're from Arne's message reporting of people running his fork.
We had a new person today that was unable to get the main repo to work for their 2019 Pacifica Hybrid, but Arne's fork worked fine. Unsure if it's due to a fingerprint issue or a Panda issue, but figured I'd add in the values from his fork that the main repo does not have.

mmm... I see. The best solution would be to add a new fingerprint for this new car. Is there a cabana drive?
The problem with expanding existing fingerprints is that at certain point we won't be able to distinguish new car models because their fingerprints risk to be included in the existing giant fingerprints we are making.
At that point, trying to correct the issue would be very hard.
On the other hand, I understand this is a simple way to fix fingeprinting issues. Let's keep this pr open.

@rbiasini I'm the user in question Re: this fingerprint.  Here's a gist of a small snapshot from the fingerprint dump from my vehicle.
https://gist.github.com/mwilliams/bd12e3e589c86cb83096e0116d73724b
That said - once added, I was up and running with ease, so seeing it upstream in some capacity would be nice.  Thank you @adhintz for beating me here!

@mwilliams , great, we'll upstream it. To confirm, you have a '19 pacifica hybrid, correct? Could you please PM your dongle ID on Discord? Or post it here. Thanks.

@rbiasini What's your handle on the Discord?

@mwilliams ,. Riccardo

@rbiasini Still not seeing you on Discord, the ID however is 3c7ce223e3571b54.
And correct, it's a 2019 Pacifica Hybrid (Plug-in).

@mwilliams , looking at your routes I'm actually getting the following:
168: 8, 257: 5, 258: 8, 264: 8, 268: 8, 270: 8, 274: 2, 280: 8, 284: 8, 288: 7, 290: 6, 291: 8, 292: 8, 294: 8, 300: 8, 308: 8, 320: 8, 324: 8, 331: 8, 332: 8, 344: 8, 368: 8, 376: 3, 384: 8, 388: 4, 448: 6, 456: 4, 464: 8, 469: 8, 480: 8, 500: 8, 501: 8, 512: 8, 514: 8, 520: 8, 528: 8, 532: 8, 544: 8, 557: 8, 559: 8, 560: 8, 564: 8, 571: 3, 579: 8, 584: 8, 608: 8, 624: 8, 625: 8, 632: 8, 639: 8, 653: 8, 654: 8, 655: 8, 658: 6, 660: 8, 669: 3, 671: 8, 672: 8, 678: 8, 680: 8, 701: 8, 703: 8, 704: 8, 705: 8, 706: 8, 709: 8, 710: 8, 719: 8, 720: 6, 729: 5, 736: 8, 737: 8, 746: 5, 752: 2, 754: 8, 760: 8, 764: 8, 766: 8, 770: 8, 773: 8, 779: 8, 782: 8, 784: 8, 792: 8, 799: 8, 800: 8, 804: 8, 808: 8, 816: 8, 817: 8, 820: 8, 825: 2, 826: 8, 832: 8, 838: 2, 848: 8, 853: 8, 856: 4, 860: 6, 863: 8, 878: 8, 882: 8, 897: 8, 906: 8, 908: 8, 924: 8, 926: 3, 929: 8, 937: 8, 938: 8, 939: 8, 940: 8, 941: 8, 942: 8, 943: 8, 947: 8, 948: 8, 958: 8, 959: 8, 962: 8, 969: 4, 973: 8, 974: 5, 979: 8, 980: 8, 981: 8, 982: 8, 983: 8, 984: 8, 992: 8, 993: 7, 995: 8, 996: 8, 1000: 8, 1001: 8, 1002: 8, 1003: 8, 1008: 8, 1009: 8, 1010: 8, 1011: 8, 1012: 8, 1013: 8, 1014: 8, 1015: 8, 1024: 8, 1025: 8, 1026: 8, 1031: 8, 1033: 8, 1050: 8, 1059: 8, 1082: 8, 1083: 8, 1098: 8, 1100: 8, 1562: 8, 1570: 8

Not sure how you obtained the one you posted

@rbiasini Hmmm I followed https://community.comma.ai/wiki/index.php/Creating_Fingerprints to collect it.  And my values.py entry below is straight from my EON right now, I pulled out the other fingerprints and this is what's there now and is what seemingly got my OP install working vs faulting every attempt.

@rbiasini looks like those values come from my fork. The cumulative effort to merge all values from all cars using my fork to reduce Sentry unsupported car faults to 0.

@rbiasini Any direction you'd prefer to see this PR go to get it merged upstream?  Would like to get support for my vehicle in master to prevent any unnecessary hackery every time there's a release (and the Pacifica Hybrid is on the supported vehicle list).
Happy to do cleanup wherever necessary if that helps it move.  First introduction to the codebase, pardon any oversight if I'm missing any preferred PR guidelines etc etc.

went over these changes again:

2018 Pacifica change: it has been merged internally. It will be in next release.
2018 Pacifica Hybrid change: the extended fingerprint is already in master.
2019 Pacifica Hybrid change: rejected. @mwilliams , I've looked again at some of your recent drives and the fingerprint in master should solve for your car. Doesn't it?

thanks!

@rbiasini My '19 Pacifica Hybrid has been in the shop for 87 days awaiting a new battery pack, I got the vehicle back today and everything ran smoothly on 0.6.1, no fingerprint issues.  Issue resolved and nice to be running stock OP :+1:

87 days in the shop for a battery pack?!
Sell Chrysler > buy Toyota

Chrysler is buying it back, total 🍋, and I've got 4 kids, so I need a family hauler.  Pacifica plug-in Hybrid was awesome, pretty much drive it as an EV and use gas on weekend trips, but all week long is purely EV mode with in town driving trips and I have a Level 2 charger at home.

Sell kids > buy Prius Prime?


feat: uniform id generate for table/stream source and make it global unique

What's changed and what's your intention?
As title, generate row id in SourceExecutor for both table and stream source:

Init SourceExecutor with step info passed from meta when create and recovery.
Generate row id with embed epoch and different local stepped id in each source executor.

Storing step info in AddMutation seems weird here, make it run first and I think its time to refactor Mutation and Barrier definition now.
Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Please note that for chunks from the delete executor, we do not need to generate a row_id column. Maybe we need some mechanism to tell the source executor whether to generate an id.

Please note that for chunks from the delete executor, we do not need to generate a row_id column. Maybe we need some mechanism to tell the source executor whether to generate an id.

Yes, this is the reason why e2e failed. I'm working on it.

For append-only sources, the row-id can be generated either by SourceExecutor or Source itself, while for table source and CDC sources, it gets complex:

Table source must provide the row-ids internally, considering updates & deletes
CDC sources do not have row-ids. Instead, CDC tables must have PKs.

Therefore, I think it's more reasonable to let the source to decide how to generate all columns including row-id columns, instead of doing this in SourceExecutor.

For append-only sources, the row-id can be generated either by SourceExecutor or Source itself, while for table source and CDC sources, it gets complex:

Table source must provide the row-ids internally, considering updates & deletes
CDC sources do not have row-ids. Instead, CDC tables must have PKs.

Therefore, I think it's more reasonable to let the source to decide how to generate all columns including row-id columns, instead of doing this in SourceExecutor.

For table source, the row-id is necessary:

Insert, it should be generated and filled same with append-only sources.
Others (Delete, Update), it will be queried and filled in the source side.
For CDS source, since it has Pk already and the row_id_idx will be None, no row-id will be generate and filled.

Besides, generate row-id in Source seems hard to guarantee unique in current implementation, since we use epoch and step info here to generate it. Unless we generate row-id in another way like timestamp(same with epoch) + worker_id + local_incremental_id. Do you think this solution acceptable?

Just come up with an idea:
how about letting the Source tells SourceExecutor which column is the row-id column? It should be an Option<usize>, where None means no row-id column needed to be assigned i.e. the PK is always provided by Source itself. For Some(usize), for rows with op == Op::Insert, we fill the generated row-id in the specified row-id column.

Just come up with an idea:
how about letting the Source tells SourceExecutor which column is the row-id column? It should be an Option<usize>, where None means no row-id column needed to be assigned i.e. the PK is always provided by Source itself. For Some(usize), for rows with op == Op::Insert, we fill the generated row-id in the specified row-id column.

+1. Currently, this info is fixed for a specific source.


Various test clean ups

Various test clean ups. Easiest to review by commit.
Updates #995

It seems that Azure pipelines rebases the pr on master, which had an additional commit compare to my local branch. That is why the CI is failing.

After a rebase on master the CI is green now.

@kleimkuhler I agree that when using unwrap having an indirect/incorrect line number is a bit annoying. I always run with RUST_BACKTRACE=full (or 1), so the problem is less pressing. I don't really agree that having two macros for is worth it. Also hopefully something like https://github.com/rust-lang/rfcs/pull/2091 would fix the problem in future.
I also fixed the  expect_read docs.

@Thomasdezeeuw Even with assert_ok!, I do usually end up switching to RUST_BACKTRACE=1 once I need to actually see why a Result is not the expected value, so that is a good point.
There are going to be some conflicts with the UDS tests getting split up, but I think most will just be replacing the macros with unwrap. Once that is taken care of I'll be good with this change!

Rebased this on master @kleimkuhler can you take another look? I'll to merge this quickly as rebasing this is a pain in the.. you know what :)


update rbranch of california

go get github.com/edgexfoundry/edgex-go
above command, It pull out master branch.
please, you check if pull out california branch source code.

https://stackoverflow.com/questions/42761820/how-to-get-another-branch-instead-of-default-branch-with-go-get


Regeneration Blessing's healing isn't showing in other languages

When the Regeneration Blessing is active, it displays "+10 Health" in English instead of French when we gain +10 Health at the beginning of a level.
I suppose it happens with other languages because "health" is well-translated on my file.

Thanks! This will be fixed in the next version


Save, Pause, and Load Sessions

For the discussions related to the development of the Save, Pause, and Load Sessions in SeneseMap.

Update:

Made Meeting Timeline for 17/10/17, you can find this also on the SideBar, but for those who want a direct link. Meeting 17/10/17.
Cleaned UP, Code for Loading and Saving are now in Separate Documents.
Removed Node OFF, also removed code from browser provenance to the Saving Document.
Fixed Bugs with saving and pausing.
Even more buttons.... Sorry, One for retrieving User Session Information and one to load the actual selected Session.
Users can now Select their desired session that they want to load in the User Profile Box.
Closed Login ISSUE for now.
Made a new Branch for SaveLoad called rebuild-SaveLoad.
Pushed current state of SaveLoad to rebuild-Login.

Points of Discussion for meeting:

Update in regards of Code
Question for "nodes" to backend
The Bug where nodes do not get added to the new session API bug yet still remains, Can we investiage this

Commit 630e88e92f4ca9482b6611849ada709b1abb0126 now uses UUID for node id (https://www.npmjs.com/package/uuid), so it is very unlikely that two nodes will have the same Id.

Save the historyMap.model to the database instead of nodes and tree.


update list of control deprecations

added several items to the list of upcoming changes due to reliability

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


Updated ReadMe.md

Various grammatical and formatting edits to were made to the ReadMe.
A screen reader, with text to speech and spell check, was used to edit this document.
A link to the copyright indemnity notice was included in this document, and links to the current and development versions of the RTDs are provided below the notice, to encourage reading this.

You can copy and paste my edits into https://stackedit.io/ for previewing.


Research what is available for static code analysis on go code that can be used in concourse

Research what tools are available for static code analysis on go code, and how to use them. Next step would be to find one that can be used in the concourse build pipeline as a part of the build process
Document the findings, as comments

coveralls and go test -cover are chosen tools


Push to Hub

Hi All, great paper btw that came in an extremely great time for me personally.
I just want to inquire about the possibility to push the adapter to the hub, is it supported in your code base ?
Also for the pretraining code, i would like to know what does the 250m means in the --pretrain 250m ? and since i see no column mapping, i believe the code expect data with text column only ... If you can please answer/confirm these points i would greatly appreciate it and again, great work, congratulations

Hello, thank you for your interest in our work.
It is possible to push our adapter to the hub, but it requires our PEFT library to load it.
Regarding the pretraining, 250M refers to the number of parameters in the model. We follow the ReLoRA preprocessing method for the C4 dataset (which can be found in its repository) and directly load the tokenized dataset in our training script.

Thanks for the quick response
I'm just curious what do you mean exactly with this requires our PEFT library to load it ? like is there an argument i can pass to the training args like --hf_token "xxx" --push_hub true --push_private true ? Also now i'm thinking about merging the adapters back to the base, at first i thought it shouldn't be a problem, but now i think about the merge_and_unload() function expect an A and B matrices, so will it have problemes with MoRA ? I still think that it could be feasible but would love to hear a confirmation from assuming you already done it. Also if the current script support --merge_adapter argument.
About the second point, I believe it is okay if i pass an untokinezed C4-like dataset, the script should tokenize it right ? at least that's how i think it is done by the original peft ...

We supports merging (we have modified merge_and_unload to merge correctly) and loading MoRA via our PEFT.
But current training script provided does not support pushing to the hub or merging by flags. You  can add corresponding code or other training scripts to achieve this. To load the pushed MoRA from the hub, you need to use our PEFT library installed via pip install -ed ./peft-mora.
For the second point, we only support loading tokenized C4 datasets because tokenizing them can take a lot of time (I don't think it can  be done by original PEFT . ). To tokenize, you can use this script from ReLoRA.

Ok I see, Thank you,you have been really helpful to get a bigger picture about the current state of the code. I will close this issue now and maybe re-open it later if i got any further questions ...
Thanks again 🤗


modify house-test.go comments

The description at the top of house_test.go can be confusing
and ultimately distracting from the problem at hand. this commit
attempts to replace the (unnecessarily) academic language with
something plainer and more to the point.
I noticed I'm not the only one who had this issue:
http://exercism.io/submissions/3b5b6003f03248f6a402859df52b6381

Seems good, thanks. I honestly wonder why Embed has to be exported + tested at all. I would remove it.


Make coverage report pattern configurable

This makes the coverage report pattern configurable by the user if they do not use any code coverage jenkins plugins.
This way, the plugin does not scan the entire workspace to find coverage files as it was doing before. Scanning the workspace fully can lead to slowdowns in large workspaces.

Looks like tests are failing?

Ill take a look at the tests

Coverage increased (+0.03%) to 91.329% when pulling 62f2b09873d7c57e799b75192afe3ef8e3c5cfb4 on coverage_report_pattern into b2a1d4cc84484f383b2512e9501f5df00dcdbbbb on master.

Test failue seems to only affect openjdk7 for some reason
FAILURE: Build failed with an exception.
* What went wrong:
Gradle build daemon disappeared unexpectedly (it may have been killed or may have crashed)

The remaining ci jobs pass. @jjx any ideas?

bump @jjx

@sectioneight tests seem to be failing on openjdk only for some reason. Any idea why that might be?
https://github.com/uber/phabricator-jenkins-plugin/pull/216#issuecomment-300568102

@kageiit Regarding issues with openjdk - I've faced them as well when tried to get my pull request approved. Had to make some changes in .travis.yml to fix that:
https://github.com/uber/phabricator-jenkins-plugin/commit/b5a1fd97baab67536f60b70764e9932d51cbac3f#diff-354f30a63fb0907d4ad57269548329e3
These changes are already in master, so you could try rebasing from master and see if it helps.

Coverage increased (+0.03%) to 91.365% when pulling 427d69534fbbc8435a15d7e99f72f23e6efd3d03 on coverage_report_pattern into b5a1fd97baab67536f60b70764e9932d51cbac3f on master.

Thanks @mfesenko rebase did the trick.


http3: don't modify any fields of the http.Request when doing 0-RTT

Depends on #4378.

It should be possible to issue the same request twice.


Added Chatbox

Description
[Please include a brief description of the changes or features added]
Added ChatBox
Related Issues
Issue Number #44
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]

Closes #44

Type of PR

[ ] Bug fix
[ X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]

Checklist

[X ] I have gone through the contributing guide
[ X] I have updated my branch and synced it with project main branch before making this PR
[X ] I have performed a self-review of my code
[ X] I have tested the changes thoroughly before submitting this pull request.
[ X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ ] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

What's about API Key?

What's about API Key?

Currently API is not working
ignore that key.
I have added predefined Questions/Answers Instead of it.

@varshith257 I removed that key


[Update Snap] Starknet Snap

Checklist
All items in the list below needs to be satisfied.

[x] Is the summary of the change documented in this ticket: minor bug fix, https://github.com/Consensys/starknet-snap/pull/364
[ ] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[ ] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

Change here: https://github.com/Consensys/starknet-snap/compare/starknet-snap-v2.10.0...starknet-snap-v2.10.1


Improve stdlib compatibility

Null values for primitive types no longer clear the original value in
the destination object.

Dereference multiple levels of pointers in the destination interface{}
type before unmarshaling into it. This is needed to match stdlib
behavior when working with nested interface{} fields. If the destination
object is a pointer to interface{} then the incoming nil value should
nil out the destination object but keep the reference to that nil value
on its parent object. However if the destination object is an
interface{} value it should set the reference to nil but keep the
original object intact.

Correctly handle typed nil decode destinations.

please fix the conflict

@taowen Done


[GTD][Code Sample] React SPA Code Sample Update (ADO-54223)

ID
Title
Assigned To
Iteration Path

54223
[GTD][Code Sample] React SPA Code Sample Update
Owen Richards
Content\Production\Security Compliance and Identity\Microsoft Identity\GTD

@henrymbuguakiarie please verify this is correct, pretty sure I have done this correctly.

@cilwerner @Dickson-Mwendia

#assign-reviewer: @henrymbuguakiarie

#sign-off


Feature/yac storage cleanup

Description of the Change
Add storage cleanup for old rounds.
Benefits
Improves stability of work
Possible Drawbacks
There is an absent strategy for preventing DOS-attacks from the future
Usage Examples or Tests
Pls, test Irohad some time and check that Iroha doesn't occupy lots of memory.
Alternate Designs
Point of the discussion: side effect in VoteStorage::store method.

@muratovv what is the status?


A file title with special characters, like #, stalls the process

I'm trying to push an image and I want it's name to be "Affiche_205_Nuit_#1_Fr.jpg".
But it stalls, without giving an error.
If I try to import it thru the original importer directly on Commons, the file gets changed with a confirmation form to "Affiche_205_Nuit_-1_Fr.jpg".
Regards, Antoine

The same for [ ].

Same for all forbidden characters in Mediawiki title (see https://www.mediawiki.org/wiki/Manual:Page_title for full list, maybe not all can be dealt with but at least the more common).

Thanks @belett, that list is very useful.


Vote for feature

Hey. I created a feathub for AlaSQL
That way we can let more people vote for what features to implement
http://feathub.com/agershun/alasql
The idea is that feature requests that can't be done right now have a place to stay where people can direct focus to the areas where there are needs

Superb! I dreamt about this for last monthhs. :+1: Already added one :)


Add ability to create docs for store.js and .js files.

This PR adds the ability for Vuese to be used with the store.js file and any mixins. I didn't set these to be automatically included, so this functionality would be completely optional. Adding **/*.js to the included will allow these to be picked up.
I am not too familiar with TypeScript, so if there a better way to accomplish any of this, or if you see anything you would like changed, let me know and I will update this PR.
Thanks!

Thanks to this PR, but there are many ways to write Store, and there are many ways to organize files. They may manage State/Actions in different ways in different files, so I think it's not enough just to deal with it as shown in PR. More importantly, vuese is concerned with the component itself and does not care about the global state.
Tip: It is best to open an issue before submitting PR.
Thanks again.

I've been using Vuese to document my Vue plugins and some of them contain store modules, mixins, and "utils" files. Currently I have to write the markdown for these files by hand, but thought Vuese could be enhanced to account for this use case as well. I definitely understand what you are saying about the store, there are many ways to manage it.
Are you open to simplifying this PR to drop the logic that documents the store, but keeping the ability to document js files for mixins?

I agree. Being able to document mixins would be very valuable. I think that adding a includeMixins: ['mixins/**/*.js'] to the config that has to be set explicitly by the user would be very useful.

@ppisecky I am happy to update this PR. @HcySunYang Is that a PR you would accept?

Just to add onto the discussion - the reason why I believe that mixins are a significantly better fit than the store documentation is because they very closely match the structure of components which are already handled by the current implementation so there is a significant amount of overlap there which means the implementation can probably be reused to a large extent. As a result not much overhead would be put on maintenance by adding this feature.

@TayHobbs I am sorry for the late reply, yes, if you update it, then I am willing to accept this PR.

Closed in favor of #111


How should I get the "value" of altitude or airmass?

Astroplan's plot_altitude gives us a plot of altitude and/or airmass.
However, I would like to get the value of the altitude on the specific time.
How should I get this value?

See the docs here:
observer.altaz(time, target).secz # returns airmass approximation

Great!
Thank you!!


Fixes #103

Not sure if this is due to 103 or a prior commit, but I get an empty editor when pulling up the map-2 dozerBeanMapping.xml with a build of your 103 branch.  Here's the associated error:
org.eclipse.swt.SWTException: Failed to execute runnable (java.lang.ClassCastException: org.eclipse.jface.layout.GridDataFactory cannot be cast to org.eclipse.swt.layout.GridData)
at org.eclipse.swt.SWT.error(SWT.java:4441)
at org.eclipse.swt.SWT.error(SWT.java:4356)
at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:139)
at org.eclipse.swt.widgets.Display.runAsyncMessages(Display.java:3983)
at org.eclipse.swt.widgets.Display.readAndDispatch(Display.java:3660)
at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine$9.run(PartRenderingEngine.java:1151)
at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:332)
at org.eclipse.e4.ui.internal.workbench.swt.PartRenderingEngine.run(PartRenderingEngine.java:1032)
at org.eclipse.e4.ui.internal.workbench.E4Workbench.createAndRunUI(E4Workbench.java:148)
at org.eclipse.ui.internal.Workbench$5.run(Workbench.java:636)
at org.eclipse.core.databinding.observable.Realm.runWithDefault(Realm.java:332)
at org.eclipse.ui.internal.Workbench.createAndRunWorkbench(Workbench.java:579)
at org.eclipse.ui.PlatformUI.createAndRunWorkbench(PlatformUI.java:150)
at org.eclipse.ui.internal.ide.application.IDEApplication.start(IDEApplication.java:135)
at org.eclipse.equinox.internal.app.EclipseAppHandle.run(EclipseAppHandle.java:196)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.runApplication(EclipseAppLauncher.java:134)
at org.eclipse.core.runtime.internal.adaptor.EclipseAppLauncher.start(EclipseAppLauncher.java:104)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:380)
at org.eclipse.core.runtime.adaptor.EclipseStarter.run(EclipseStarter.java:235)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at org.eclipse.equinox.launcher.Main.invokeFramework(Main.java:648)
at org.eclipse.equinox.launcher.Main.basicRun(Main.java:603)
at org.eclipse.equinox.launcher.Main.run(Main.java:1465)
at org.eclipse.equinox.launcher.Main.main(Main.java:1438)
Caused by: java.lang.ClassCastException: org.eclipse.jface.layout.GridDataFactory cannot be cast to org.eclipse.swt.layout.GridData
at org.eclipse.swt.layout.GridLayout.layout(GridLayout.java:208)
at org.eclipse.swt.layout.GridLayout.layout(GridLayout.java:197)
at org.eclipse.swt.widgets.Composite.updateLayout(Composite.java:1201)
at org.eclipse.swt.widgets.Composite.layout(Composite.java:710)
at org.eclipse.swt.custom.CTabFolder.runUpdate(CTabFolder.java:3755)
at org.eclipse.swt.custom.CTabFolder$10.run(CTabFolder.java:3741)
at org.eclipse.swt.widgets.RunnableLock.run(RunnableLock.java:35)
at org.eclipse.swt.widgets.Synchronizer.runAsyncMessages(Synchronizer.java:136)
... 24 more

I actually get the same error on master.  Are you able to open the config in examples/map-2 with the latest changes to the editor?

Not sure what’s happening to you with master.  That change was only introduced with 103.  It’s fixed now in the PR.

On Jan 22, 2015, at 10:42 AM, Keith Babo<EMAIL_ADDRESS>wrote:
I actually get the same error on master. Are you able to open the config in examples/map-2 with the latest changes to the editor?
—
Reply to this email directly or view it on GitHub https://github.com/fabric8io/data-mapper/pull/104#issuecomment-70994782.

Looks like the merge got ugly when trying to fix this on your branch - it’s three commits now and can’t be rebased cleanly on top of master:
https://github.com/jpav/data-mapper/commits/103
~ keith
On Jan 22, 2015, at 10:57 AM, John Verhaeg<EMAIL_ADDRESS>wrote:

Not sure what’s happening to you with master. That change was only introduced with 103. It’s fixed now in the PR.

On Jan 22, 2015, at 10:42 AM, Keith Babo<EMAIL_ADDRESS>wrote:
I actually get the same error on master. Are you able to open the config in examples/map-2 with the latest changes to the editor?
—
Reply to this email directly or view it on GitHub https://github.com/fabric8io/data-mapper/pull/104#issuecomment-70994782.

—
Reply to this email directly or view it on GitHub.

Forgot to mention that I built the changeset and it works great. :-)  If the git surgery is too nasty, you can always open a new branch off of master and then just copy your changed files to that new branch.  The diff will end up being the same.

I got it cleaned up and did the push to master

On Jan 22, 2015, at 11:04 AM, Keith Babo<EMAIL_ADDRESS>wrote:
Looks like the merge got ugly when trying to fix this on your branch - it’s three commits now and can’t be rebased cleanly on top of master:
https://github.com/jpav/data-mapper/commits/103
~ keith
On Jan 22, 2015, at 10:57 AM, John Verhaeg<EMAIL_ADDRESS>wrote:

Not sure what’s happening to you with master. That change was only introduced with 103. It’s fixed now in the PR.

On Jan 22, 2015, at 10:42 AM, Keith Babo<EMAIL_ADDRESS>wrote:
I actually get the same error on master. Are you able to open the config in examples/map-2 with the latest changes to the editor?
—
Reply to this email directly or view it on GitHub https://github.com/fabric8io/data-mapper/pull/104#issuecomment-70994782.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub https://github.com/fabric8io/data-mapper/pull/104#issuecomment-70997406.

This has been pushed - closing the PR.


getFollowers do not work!

I get this result. Object data is empty
{ "count": 4, "page_info": { "has_next_page": false, "end_cursor": null }, "data": [] }

web api has limit 150 requests per hour count*150 requests= max scraped users, another thing to consider is that your account will get web api block eventually if used just for scraping.

getFollowers and getFollowings is broken

Yes, having the same problem with both getFollowers and getFollowings. It's not API limit problem, because all other requests works.

Some idea to solve the problem?

There were some changes on the Instagram API, I guess we have to wait.

Im trying to create a function to get followers given a userInformation (which is the information we get from getUserbyUsername) and i have this exact problem. but if I run api.getFollowers outside that function, I have no problems with that. any idea of why?


Properly handle nil Data from HEAD requests

This PR adds support for treating HEAD requests as successful when they have nil response data. It also refactors our existing treatment of nil data response codes and adds tests for this new behavior.

Hey @jshier, I had a chance to review this more in-depth and I'm afraid we'll have to wait until AF5 to make these changes. Are there any changes in here that are backwards compatible that should still make it into AF 4.x? I remember you saying you found a couple small issues that you thought were incorrect. Maybe with the tests?
I'm going to close this PR out for now. We can revisit in AF 5. I've added a card to our Trello board.


[BUG] Can't deploy project serverless-with-lambda-layer

When I clone project serverless-with-lambda-layer and run serverless deploy
https://github.com/Sparticuz/chromium/tree/master/examples/serverless-with-lambda-layer
I got this error, Am I missing anything? Thank you for your help!

For that specific example, you need to run make chromium.zip in the root directory first to generate the layer.

Thank you very much


Search Failing on HasOne Relationship

Summary of problem or feature request
Just trying to solve relationship searching, it is a HasOne relationship, a project has a project creator. The Ajax pagination works great, but as soon as the 'project_created_by.first_name' field has searchable:false removed, it breaks. I've looked all through the doc examples but couldn't piece it together. The values come across successfully, it just throws a 500 exception when doing the query
Code snippet of problem
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'project_created_by.first_name' in 'where clause' (SQL: select count(*) as aggregate from (select '1' as `row_count` from `projects` where `projects`.`deleted_at` is null and (LOWER(`quote_id`) LIKE %%k%% or LOWER(`purchase_order`) LIKE %%k%% or LOWER(`site_street_address`) LIKE %%k%% or LOWER(`builder_name`) LIKE %%k%% or LOWER(`project_created_by`.`first_name`) LIKE %%k%% or LOWER(`created_at`) LIKE %%k%% or LOWER(`due_date`) LIKE %%k%%)) count_row_table)

"columns": [
            {data: 'quote_id', name: 'quote_id'},
            {data: 'purchase_order', name: 'purchase_order'},
            {data: 'site_street_address', name: 'site_street_address'},
            {data: 'builder_name', name: 'builder_name'},
            {data: 'project_created_by.first_name', name: 'project_created_by.first_name'},
            {data: 'created_at', name: 'created_at'},
            {data: 'due_date', name: 'due_date'},
        ]

return Datatables::of($this->project->with(['ProjectCreatedBy', 'ManagerStatus' => function ($query) {
                return $query->where('user_id', Auth::user()->id);
            }])->select('projects.*'))
            ->editColumn('created_at', '{!! $created_at->format("d/m/Y") !!}')
            ->editColumn('due_date', '{!! $due_date->format("d/m/Y") !!}')
            ->make(true);

System details

Operating System - OSX
PHP Version - Homestead
Laravel Version - 5.2.39
Laravel-Datatables Latest Stable (Tonight from composer)

I just tested this and this is the way you should declare your column:
{data: 'project_created_by.first_name', name: 'ProjectCreatedBy.first_name'},

Notice that name should be equal to the relationship method name.


Fixed #6147 -- Show pages in PageField only once, regardless of translations count

Fixed #6147

Coverage remained the same at 79.182% when pulling 8c274ebf05834588858cf1836b4def4e77ed529a on czpython:fixes/3.5.x/6147 into a019a48d722c6051d612dde7065f81414024b1c6 on divio:develop.


Tackle some simple clippy lints.

Lints attempted:
https://rust-lang.github.io/rust-clippy/master/index.html#/semicolon_if_nothing_returned
https://rust-lang.github.io/rust-clippy/master/index.html#/manual_string_new
https://rust-lang.github.io/rust-clippy/master/index.html#/map_unwrap_or
https://rust-lang.github.io/rust-clippy/master/index.html#/implicit_clone

Thanks for holding up code quality, will add these warnings permanently so they won't happen again!


Configure max build file number

Hitting a max limit on the number of build files (20000) on cloudflare pages.
Is there a way to configure the number of generated files?
Alternatively I'd like to explore a way to upload the generated assets to r2 or other other destinations if possible.

The best issue to follow for this would be #74 — though there isn't work underway on that yet.
There shouldn't be a barrier to uploading Pagefind's files to another destination. You'd just need to run Pagefind, upload those files, and then remove the index from disk before finishing so Cloudflare Pages doesn't encounter them.

for fragment hosted on R2 upload is real slow. 9934 fragments around 3MB in total size. Upload take 25 minutes.

I'll close this issue as a duplicate, but make sure to follow #74 where I'll post any updates :)


Compilation: perform newline splicing on input files

Newline splicing happens in "phase 2" of translation, before everything
except character set mapping and trigraph replacement
Fixes #210

Note: This stackoverflow answer indicates that this approach will not match the behavior of clang/gcc with regard to __LINE__ if a file has line splicings (gcc/clang preserve the original pre-splice line number; this approach will produce the post-splice line number)

Only real issue with this is that it doesn't support \r\n (and just \r) newlines. It would also be nice to allow whitespace before the newline and give a warning about it like gcc and clang.
Also the relevant part from the Tokenizer should now be deleted as dead code.

I can make those changes. I could also incorporate the utf8 validation in the loop if you think that's a good idea.

A future improvement would be for the Tokenizer to keep track of the spliced lines since it feels pretty weird to store a line that isn't actually meaningful in any way.

Maybe I'm misunderstanding but that seems like a much more invasive change, since anything that uses token slicing would need to account for embedded spliced newlines, right?

Only real issue with this is that it doesn't support \r\n (and just \r) newlines. It would also be nice to allow whitespace before the newline and give a warning about it like gcc and clang.
Also the relevant part from the Tokenizer should now be deleted as dead code.

I can make those changes. I could also incorporate the utf8 validation in the loop if you think that's a good idea.

That does sound like a good idea, less looping over the same bytes is always good.

A future improvement would be for the Tokenizer to keep track of the spliced lines since it feels pretty weird to store a line that isn't actually meaningful in any way.

Maybe I'm misunderstanding but that seems like a much more invasive change, since anything that uses token slicing would need to account for embedded spliced newlines, right?

Slicing uses Token.byte_offset, Token.line is pretty much only used for diagnostics and __LINE__. But this is more of an enhancement to the API and not necessary for this pr since I expect this isn't going to be quite as straightforward to implement.
Alternatively Token.line could be renamed to Token._line with a comment explaining that it's not the actual line as in the file and to use Token.line() for that.

Currently the line number and context for the backslash_newline_escape warning are incorrect; I think this warning will have to be handled as a special case, since it needs to print the buffer as it existed before splicing occurred.
UTF-8 validation is still a separate loop over the bytes; I figured I could add that as an enhancement if this looks good so far.


Local gives problem in edit form.

    in DateTextField
    in WithUtils(DateTextField) (created by WithStyles(WithUtils(DateTextField)))
    in WithStyles(WithUtils(DateTextField)) (created by ModalWrapper)
    in ModalWrapper (created by RenderPropsComponent)
    in RenderPropsComponent (created by withHandlers(RenderPropsComponent))
    in withHandlers(RenderPropsComponent) (created by lifecycle(withHandlers(RenderPropsComponent)))
    in lifecycle(withHandlers(RenderPropsComponent)) (created by withState(lifecycle(withHandlers(RenderPropsComponent))))
    in withState(lifecycle(withHandlers(RenderPropsComponent))) (created by BasePicker)
    in BasePicker
    in WithUtils(BasePicker) (created by DatePickerWrapper)
    in DatePickerWrapper (created by ForwardRef)
    in MuiPickersUtilsProvider (created by _makePicker)
    in div (created by _makePicker)
    in _makePicker (created by ConnectedField)
    in ConnectedField (created by Connect(ConnectedField))
    in Connect(ConnectedField) (created by Field)
    in Field (created by FormFieldView)
    in FormFieldView (created by DefaultValue)
    in DefaultValue (created by Connect(DefaultValue))
    in Connect(DefaultValue) (created by WithFormField)
    in WithFormField (at HealthCareEdit.js:56)
    in div (created by FormInput)
    in FormInput (created by WithStyles(FormInput))
    in WithStyles(FormInput) (created by FormTab)
    in span (created by FormTab)
    in FormTab (created by translate(FormTab))
    in translate(FormTab) (at HealthCareEdit.js:51)
    in Route (created by TabbedForm)
    in div (created by TabbedForm)
    in form (created by TabbedForm)
    in TabbedForm (created by WithStyles(TabbedForm))
    in WithStyles(TabbedForm) (created by Form(WithStyles(TabbedForm)))
    in Form(WithStyles(TabbedForm)) (created by Connect(Form(WithStyles(TabbedForm))))
    in Connect(Form(WithStyles(TabbedForm))) (created by ReduxForm)
    in ReduxForm (created by translate(ReduxForm))
    in translate(ReduxForm) (created by Connect(translate(ReduxForm)))
    in Connect(translate(ReduxForm)) (created by Route)
    in Route (created by withRouter(Connect(translate(ReduxForm))))
    in withRouter(Connect(translate(ReduxForm))) (at HealthCareEdit.js:50)
    in div (created by Paper)
    in Paper (created by WithStyles(Paper))
    in WithStyles(Paper) (created by Card)
    in Card (created by WithStyles(Card))
    in WithStyles(Card) (created by EditView)
    in div (created by EditView)
    in EditView (created by EditController)
    in EditController (created by translate(EditController))
    in translate(EditController) (created by Connect(translate(EditController)))
    in Connect(translate(EditController)) (created by Edit)
    in Edit (at HealthCareEdit.js:49)
    in HealthCareEdit (at App.js:75)
    in Unknown (created by WithPermissions)
    in WithPermissions (created by Connect(WithPermissions))
    in Connect(WithPermissions) (created by getContext(Connect(WithPermissions)))
    in getContext(Connect(WithPermissions)) (created by Route)
    in Route (created by Resource)
    in Switch (created by Resource)
    in Resource (created by Connect(Resource))
    in Connect(Resource) (created by AdminBuilder)
    in Route (created by RoutesWithLayout)
    in Switch (created by RoutesWithLayout)
    in RoutesWithLayout (created by Route)
    in div (created by Layout)
    in main (created by Layout)
    in div (created by Layout)
    in div (created by Layout)
    in Layout (created by WithStyles(Layout))
    in WithStyles(Layout) (created by Route)
    in Route (created by withRouter(WithStyles(Layout)))
    in withRouter(WithStyles(Layout)) (created by Connect(withRouter(WithStyles(Layout))))
    in Connect(withRouter(WithStyles(Layout))) (created by LayoutWithTheme)
    in MuiThemeProvider (created by LayoutWithTheme)
    in LayoutWithTheme (created by Route)
    in Route (created by CoreAdminRouter)
    in Switch (created by CoreAdminRouter)
    in div (created by CoreAdminRouter)
    in CoreAdminRouter (created by Connect(CoreAdminRouter))
    in Connect(CoreAdminRouter) (created by getContext(Connect(CoreAdminRouter)))
    in getContext(Connect(CoreAdminRouter)) (created by Route)
    in Route (created by CoreAdmin)
    in Switch (created by CoreAdmin)
    in Router (created by ConnectedRouter)
    in ConnectedRouter (created by CoreAdmin)
    in TranslationProvider (created by withContext(TranslationProvider))
    in withContext(TranslationProvider) (created by Connect(withContext(TranslationProvider)))
    in Connect(withContext(TranslationProvider)) (created by CoreAdmin)
    in Provider (created by CoreAdmin)
    in CoreAdmin (created by withContext(CoreAdmin))
    in withContext(CoreAdmin) (created by AdminBuilder)
    in AdminBuilder (created by _class)
    in _class (at App.js:147)
    in Unknown (at index.js:7)```

@thecassion Are you setting the locale prop and is this the error message you get (before the stack trace)?

RangeError: locale must contain localize property.

RangeError: locale must contain localize property.

I have this problem and cannot see the DatePicker at all.
I tried to set prop
locale="en-us"
or
import enLocale from 'date-fns/locale/en-US';
and then
locale={enLocale}
but none of it worked for me.

This is problematic for us too.

I'll be able to take a look at this next week. Meanwhile if someone has a
fix feel free to submit a PR.
On Mon, 13 Aug 2018, 22:39 Jeffry Evans<EMAIL_ADDRESS>wrote:

This is problematic for us too.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vascofg/react-admin-date-inputs/issues/11#issuecomment-412674071,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmuSd1L6iR-tSzskB0Q90fsTHbv6ZDaks5uQfH3gaJpZM4VvLVT
.

This is fixed in 1.0.20.
The problem was that the locale prop already existed in react-admin, so, when not defined in the picker component, instead of being undefined inside react-admin-date-inputs, it was receiving the value of the react-admin locale.
Anyway, please refer to the README.md for the new way to specify custom locales and util libs.


Slack URL and Channel IDs should be modifiable

Great script; useful and works as intended.
There are hard-coded slack subdomains and channel ids. These should be changed for each different instance. Otherwise you may see this error on your spreadsheet {"ok":false,"error":"channel_not_found"}.

@drwl there are now functions up top to centralize where to change these things.

Using latest code, I still had to edit the payload.channels line with the channel ID, otherwise I got the error mentioned above.


charts,manifests: Update the upstream and downstream CSV descriptions.

Superseding #1303

/override ci/prow/metering-e2e-aws
/override ci/prow/metering-upgrade-aws


Remove python 3.6 support, and add python 3.10.

Signed-off-by: Rodney Richardson<EMAIL_ADDRESS>Fixes #204

Please review.
I'm not familiar with the build process, so I'm not sure why linux_python_3_6 is tagged as Required, and linux_python_3_10 is not.

@goneall I don't appear to have access to the branch protection area for this repository.  Can you turn off all the required checks for branch protection?  I don't believe those have a lot of value, especially since we're likely to need to make frequent changes to the list of required checks.

Please keep 3.6 support for now unless you have a good reason not too. This is still supported in several linux distros.

Can you turn off all the required checks for branch protection? I don't believe those have a lot of value, especially since we're likely to need to make frequent changes to the list of required checks.

Can you elaborate why you don't believe protecting branches from mistakes does not have a lot of value?

@licquia Are there any specific checks of concern?  Currently, the following checks are made:

Pull request
1 approval
Status Checks

continuous-integration/appveyor/pr
ci/circleci: linux_python_3_9 through ci/circleci: linux_python_3_6

@licquia Are there any specific checks of concern? Currently, the following checks are made:

Status Checks

ci/circleci: linux_python_3_9 through ci/circleci: linux_python_3_6

linux_python_3_6 has been removed, linux_python_3_10 has been added.

Right now, those checks are acting as a barrier to accepting code from a contributor.  That's fine, as long as:

it is clear why the contribution is bad
in cases where the contribution is not bad, there is an easy way to fix it

In this case, we don't have a policy for how long we support old Python releases that I know of, and we don't seem to have a way to remove required tests when the Python version becomes obsolete.  I wouldn't have a problem with keeping the requirements if those specific issues are addressed.
@goneall, specifically, I've only got an issue with the per-version tests.
As for Python 3.6: we have been forced to drop testing support on MacOS because it's no longer supported by the upstream testing framework.  The release itself is 5.5 years old, and hasn't been supported with security or other patches for about 3.5 years.  Do we have a specific reason to continue supporting 3.6?  If so, how long will we need to continue that support?

I think following the official python support lifecycle would make for a sensible policy. This is a tool used to help manage software inventory, and hence cybersecurity issues. It would seem inconsistent supporting potentially insecure versions of python.
It also makes practical sense if you follow the same policy as your build infrastructure, and circleci seems to follow the python support lifecycle.

@goneall, can we go ahead and remove the requirements for the ci/circleci: linux_python_3 checks?

@pombredanne Thoughts?  Seems reasonable to drop 3.6 at this point.
@licquia Before changes, can you create a branch so that those wanting to continue 3.6 can have a branch to work from?  Not quite sure what a good name would be - perhaps Python3_6Support?

I'm happy to create such a branch -- if we have someone willing to maintain it, and a plan for how the work in that branch becomes usable for others.  Frankly, though, folks using 3.6 are already having to either use old releases or do their own forking and maintenance work, since 3.6 is now unsupported upstream and is getting dropped by the tools and vendors.  Do you know of anyone who can take up this work?

I'm happy to create such a branch -- if we have someone willing to maintain it, and a plan for how the work in that branch becomes usable for others.

@licquia based on the previous comments, it sounds like @pombredanne is interested in keeping the 3.6 version available.
IMHO - I don't think we need a commitment in maintaining the branch to create it since this is easy to do now, a bit more difficult to do after several commits (someone would have to research where the incompatibility was introduced to create the branch).
Creating the branch would give anyone interest in maintaining it the option.
I think of it as a nice compromise between dropping support from 3.6 and not taking in any PR's that break 3.6 compatibility.
Let me know if you disagree.

@licquia Since I haven't heard any objections from @pombredanne I removed the linux_python_3_6 requirement. I've also created a tag python3.6 which we can use to create a branch if there is interest.  Without at least a tag, it would be very difficult to find the point of divergence for creating a branch for supporting Python 3.6.

Is there's anything preventing this being merged? The "changes requested" (i.e. leave in python 3.6) has been addressed by tagging a branch.

@pombredanne I tagged the release - so we can create a 3.6 support branch if needed.  OK to merge?  Please resolve your comment or add a comment as @RodneyRichardson is looking for this to be merged.

@pombredanne Any chance this can be merged?

@RodneyRichardson - I just pinged @pombredanne as well - if we don't hear back in a week or so, I'll override and merge the PR.

Since we now have a tagged version, we can always create a separate branch which supports Python 3.6 or even revert this PR if the community deems it necessary.
Based on this, I'll go ahead and override the check and merge the PR since it has been outstanding for quite some time.


Mark game as modded in PUN metadata

When creating a game with Photon, it's possible to set extra properties for use by the client.  PPL should add an is_modded flag or similar metadata to help with future management of modded games.

Added with c858ddf


shortcuts for timeline navigation as of issue #46

This PR implements two parts of the issue #46
selecting previous and next event in the timeline with CTRL + UP and CTRL + DOWN
moving currently selected event up and down the timeline ALT + UP and ALT + DOWN
This PR aswell automatically focuses the textbox when you select a TextEvent
This PR aswell automatically selects the event when you click into a TextEvent textbox
Preview Video:
https://streamable.com/i6nvah
(yes, I know there are typos in the video :P)
Edit:
Indentation is now working on shortcut movement
CTRL DELETE removes the currently selected event node and selects the next/last event node
CTRL T creates a new text event node below the currently selected and focus it's textbox to continue writing

Indendation is missing when you move the block with shortcuts. I will fix this real quick

Would it be possible to remove a selected item with the delete key? Or would it interfere with the rest of the editor?

Would it be possible to remove a selected item with the delete key? Or would it interfere with the rest of the editor?

The delete key is a tricky one. A lot of interactions withing the editor uses the delete key and, inside a Text Event the delete key is part of a normal text edit workflow. Since this commit automatically selects the event panel in the timeline when you enter a Text Event text field, I dont know if it would be a good choice to chose the delete key.

Do not merge: Something is odd with the character selection and portrait selection of text events. Don't know if its my fault or the problem comes from somewhere else

Added CTRL T for creating a new text node below the current selected node and focusing it's textbox

There is a wonky behaviour with the alt down sometimes switching from the timeline down to the characters. not that big of a deal tho and i think the issue is within a default control behaviour of another element, not touched by this PR

Thank you so much!


[4.2][stdlib] Add KeyValuePairs typealias as prep for 5.0 rename

Note, this is a 4.2-only PR. The 5.0 version renames the type and type aliases the old type, here: #16577.
Adding this typealias now will allow people to make the change on code that will still compile on the (future) old compiler, allowing us to deprecate the old name in 5.0 mode.

@swift-ci please test


[Feature] Do not allow players to re-enter played games in 2020

So that they cannot retry to get a higher score.

Done.


Document @private class inheritance behavior.

Denied to enforce encapsulation

:tada: This issue has been resolved in version 1.0.1 :tada:
The release is available on:

1.0.1
GitHub release

Your semantic-release bot :package::rocket:


fix: update to eye v10.30.11

fix: update to eye v10.30.11

:tada: This PR is included in version 17.0.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v17.0.10

Your semantic-release bot :package::rocket:


Permission Denied on production tunnel

When tunneling into containers that have a registered port below 1000 nanobox cannot open a proxy.

Messaging added to nanobox (#520 #521)


V1.1 update save, edit data file commands and command summary in user guide

Codecov Report

Merging #42 (835c3e4) into dev (1964b75) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev      #42   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1964b75...835c3e4. Read the comment docs.

LGTM!


hpl.load_database(), hpl.get_parameters

Why am I getting an error of module not found, when I type the above code

Hello Jalal-Khan0,
how did you import the library into your file?
It is supposed to be: from hplib import hplib as hpl


fix: prepareAddrs trims /128 suffixes for ipv6 filters

What this PR does / why we need it:
Single-host IPv6 addresses may show as 2401:4900:33d5:4afa:9d59:6c45:239f:8ead/128 on the filter list but as 2401:4900:33d5:4afa:9d59:6c45:239f:8ead on the ip route output.
This can cause problems where such addresses are falsely recognized as a diff that needs to be added to the blackholed routes, even though it is already blackholed.
The symptom shows as
Checking ipv6 egress filter list with 995 entries against current settings...
Currently applied filter list contains 995 entries
Adding 1 routes.
Error updating blackhole routes: updateRoutes failed for IPv6: error adding blackhole routes: exit status 1

So both current list and new list have the same number of entries, but still one entry is to be added.
Special notes for your reviewer:

Updated tests to reflect the change.

Release note:

fixed a bug where non-suffixed IPv6 addresses could cause the egress filter applier to crash.

/lgtm


Display error message when trying to delete a referenced resource

Describe the bug
I have a Claim in FHIR Server which has a Patient , 2 Organizations and 1 Practitioner resources.
When I try to delete referenced Patient or Organizations, server deletes without warning.
FHIR Version?
R4
Data provider?
CosmosDB
To Reproduce
Steps to reproduce the behavior:

Add a Patient
Add 2 organizations
Add a Practitioner
Add a Claim referencing above Patient ID, Organization IDs and and Practitioner ID.
Try to delete Patient
Your Patient will be deleted
Claim is still available even your Patient referencing is missing
Same steps for Organization and Practitioner

Expected behavior
When you try to delete Patient it should warn user to tell, it's already used by other resource type.
Actual behavior
You can delete Patient and your Claim remains even your Patient referencing is missing

AB#87249


version 1.74

Type: Bug
i just dowloaded vsCode version 1.74.0
and all my Extensions  are not showing in Extension tabs , event tho they are still running . And the searching for the extension is running fine and showing the results.
VS Code version: Code 1.74.0 (5235c6bb189b60b01b1f49062f4ffa42384f8c91, 2022-12-05T16:38:16.075Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz (8 x 1800)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.91GB (5.64GB free)

Process Argv
--folder-uri=vscode-remote://wsl+Ubuntu/home/ammar/kutubeeweb21 --remote=wsl+Ubuntu --crash-reporter-id be4286ca-e7f8-46c9-8cde-d69ac1243dbc

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: Ubuntu

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz (8 x 1799)

Memory (System)
7.71GB (5.57GB free)

VM
0%

Extensions (81)

Extension
Author (truncated)
Version

Bookmarks
ale
13.3.1

es7-react-js-snippets
dsz
4.4.3

python-ultisnippets
eli
2.7.0

auto-close-tag
for
0.5.14

auto-rename-tag
for
0.1.10

vsc-python-indent
Kev
1.18.0

theme-monokai-pro-vscode
mon
1.2.0

jupyter-keymap
ms-
1.0.0

remote-wsl
ms-
0.72.0

material-icon-theme
PKi
4.22.0

vscode-icons
vsc
12.0.1

jinja
who
0.0.8

Bookmarks
ale
13.3.1

vscode-sqlite
ale
0.14.1

arepl
alm
2.0.4

jsoncrack-vscode
Ayk
1.0.8

vscode-django
bat
1.10.0

better-toml
bun
0.3.2

turbo-console-log
Cha
2.6.2

path-intellisense
chr
2.8.3

regex
chr
0.4.0

coderoad
Cod
0.17.2

vscode-markdownlint
Dav
0.48.1

vscode-eslint
dba
2.2.6

python-preview
don
0.0.4

githistory
don
0.6.19

python-environment-manager
don
1.0.4

python-extension-pack
don
1.7.0

typescript-notebook
don
2.0.6

es7-react-js-snippets
dsz
4.4.3

gitlens
eam
13.1.1

vscode-html-css
ecm
1.13.1

EditorConfig
Edi
0.16.4

python-ultiextension-pack
eli
1.6.0

python-ultisnippets
eli
2.7.0

prettier-vscode
esb
9.10.3

auto-close-tag
for
0.5.14

auto-rename-tag
for
0.1.10

code-runner
for
0.11.8

copilot
Git
1.62.7527

vscode-pull-request-github
Git
0.56.0

live-sass
gle
5.5.1

gc-excelviewer
Gra
4.2.56

grepper
Gre
0.0.7

todo-tree
Gru
0.0.220

rest-client
hum
0.25.1

nbpreviewer
jit
1.2.2

vscode-text-pastry
jkj
1.3.1

svg
joc
1.4.23

vscode-colorize
kam
0.11.1

vsc-python-indent
Kev
1.18.0

calculator
lix
0.1.1

rainbow-csv
mec
3.4.0

python-path
mge
0.0.14

mongodb-vscode
mon
0.9.5

theme-monokai-pro-vscode
mon
1.2.0

isort
ms-
2022.8.0

python
ms-
2022.20.0

vscode-pylance
ms-
2022.12.20

jupyter
ms-
2022.11.1003412109

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.12

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

cpptools
ms-
1.13.6

autodocstring
njp
0.6.1

indent-rainbow
ode
8.3.1

material-icon-theme
PKi
4.22.0

heroku-command
pko
0.0.8

polacode
pnp
0.3.4

vscode-css-peek
pra
4.2.0

vscode-thunder-client
ran
2.1.2

LiveServer
rit
5.7.9

git-autoconfig
shy
0.0.2

wav-preview
suk
2.0.4

intellicode-api-usage-examples
Vis
0.2.6

vscodeintellicode
Vis
1.2.29

vscode-icons
vsc
12.0.1

quokka-vscode
Wal
1.0.515

jinja
who
0.0.8

vscode-import-cost
wix
3.3.0

(6 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vscaat:30438848
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1:30587005
282f8724:30602487
gswce1:30612156
iaj6b796:30613358
dbltrim-noruby:30604474
89544117:30613380
fim-prod:30623723
ejf25101:30620729

Can you please reload the window and see if it works?

Can you please reload the window and see if it works?

i did reload , and i closed the editor and re-open it  ( all local installed = 0 , WSL installed =0 , recommended =0 )
and if you go to {Show all running extension} you will be able to see them.

Do you see any errors in the following logs, also please share them?

F1 > Open View... > Shared
F1 > Open View... > Window
F1 > Open View... > Remote
F1 > Toggle Developer Tools > Console/Network

all extensions started working

Do you mean you see extensions now?

all extensions started working

Do you mean you see extensions now?

yes i can see them.

Can you please share a gif of reloading the window...
Please enable log level to trace and share all following logs?
F1 > Open View... > Shared
F1 > Open View... > Window
F1 > Open View... > Remote
F1 > Open View... > Extension Host
F1 > Open View... > Remote Extension Host
F1 > Toggle Developer Tools > Console/Network
It seems you are using remote, may i know if you are not seeing remote extensions or also local extensions?

what it seems like the problem was closing VsCode and re-opening it , doesnt solve the problem ,
but when i opened Console/Network  , i was able to press f5 or ctrl + r to do a refresh or reload , and thats when my problem was solved .

also my other pc had the same problem .. i did the same steps i just mentioned above and solved

May I know if the extensions views/sections are collapsed? Can you please do following and let me know if it worked?

Expand the extensions views/sections and reload the window?

/duplicate


Test failure for 0.7.0 on Arch Linux

Hey, I'm getting the following test failure while building the latest version for Arch Linux (in a clean chroot):
failures:

---- test_ignore_target stdout ----
thread 'test_ignore_target' panicked at src/main.rs:310:5:
assertion failed: entries.unwrap().is_empty()
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    test_ignore_target

test result: FAILED. 12 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 1.98s

error: test failed, to rerun pass `--bin cargo-machete`

Do I need any preparation before running the tests? Any ideas about it? 🤔
As a workaround I currently --skip this test in the package.

Thanks for the report. That's weird, I can't repro on my (MacOS) machine; are you using the version from a github release, or installed from cargo, or building from a specific git hash?

I'm pulling the release from GitHub (0.7.0) and building in an isolated environment. See the PKGBUILD for build steps.
Let me know if there is any other information is needed!

Thanks for the details; unfortunately I can't reproduce on my machine. If anyone can reproduce and track the test failure, that would be greatly appreciated 🙏

Maybe you can tell me what that test is doing and why it might fail. Then I can dig into it a bit :)

Okay I found the culprit. The downloaded archive from GitHub is not a git repository (does not contain .git folder) so I'm guessing the underlying library does not take .gitignore into account in that case.
The test passes when I rename .gitignore to .ignore 🫠

Thanks for the investigation! I think we could rename .gitignore to .ignore and not lose any testing coverage, since this is testing that any kind of ignore files is taken into account. Thoughts?

Yup, sounds good to me 👍🏼


vnext

adopt to go-aserto@vnext changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

3 of 8 (37.5%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 69.681%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

logger/self/logger.go
2
4
50.0%

scribe/scribe_client_factory.go
1
4
25.0%

Totals

Change from base Build<PHONE_NUMBER>8:
-0.4%

Covered Lines:
262

Relevant Lines:
376

💛  - Coveralls


Resetting stack shows an older page in stack for a split second.

I have the following stack navigation set up:
const Routes = StackNavigator({   	Login: { screen: Login }, 	Main:{screen:Main} }, { 	headerMode:'none' })
const Main=StackNavigator({ PhoneInputPage: { screen: PhoneInputPage }, OTP:{screen:OTP} })
when I am in my Routes stack and I reset my stack to point to the Main stack I see the Login screen briefly in Android.  It doesn't look appealing at all. How do I solve this?

Do you have some example code of resetting your stack?

 const resetAction = NavigationActions.reset({       index: 0,       actions: [NavigationActions.navigate({ routeName: 'Main' })]});
this.props.navigation.dispatch(resetAction);

I have the exact same problem and need help aswell.

Related: https://github.com/react-community/react-navigation/issues/2084

I have this same problem. I'm attempting to reset from a nested navigator. It briefly shows the bottom screen before properly navigating to the dispatch target.

Similar issue here.. I have a Stacks over Tabs setup.
const MainTabNavigator = TabNavigator( {
  connectionList: { screen: ConnectionListNavigator },
  addConnectionMain: {  screen: AddConnectionNavigator    },
  drafts: { screen: DraftsNavigator },
  settings: { screen: SettingsNavigator },
} )

const MainNavigator = StackNavigator( {
  root: { screen: MainTabNavigator },
  addConnectionViaSms: { screen: AddConnectionViaSmsScreen },
  addConnectionViaEmail: { screen: AddConnectionViaEmailScreen },
  addConnectionDetails: { screen: AddConnectionDetailsScreen }
}, {
  mode: 'modal'
} )

I navigate from addConnectionMain -> addConnectionViaSms -> addConnectionDetails then after a form is submitted on that page, I do a reset back to connectionList.
      const resetAction = NavigationActions.reset( {
        index: 0,
        actions: [
          NavigationActions.navigate( { routeName: 'root' } )
        ]
      } )
      await navigation.dispatch( resetAction )

When I do the reset, it distinctly  navigates be back through each route: addConnectionViaSms -> addConnectionMain -> connectionList.
Sometimes it's fairly slow (especially on the iOS simulator) and actually hangs on one or two of the screens for a few seconds.

Hi! In an effort to get the hundreds of issues on this repo under control I'm closing issues tagged as questions. Please don't take this personally - it's simply something we need to do to get the issues to a manageable point. If you still have a question I encourage you to re-read the docs, ask on StackOverflow, or ask on the #react-navigation Reactiflux channel. Thank you!


Cleanup: fix some typos in code comment

Signed-off-by: Guangming Wang<EMAIL_ADDRESS>
[] CHANGELOG entry if change is relevant to the end user.

Changes
just fix typo words in codebase
Verification

go report misspell
https://goreportcard.com/report/github.com/thanos-io/thanos#misspell

OOOO, adding this now!

here u go: https://github.com/thanos-io/thanos/pull/1446


write video with opencv

Hello
I tried to write a video using a sequence of images using : out = cv2.VideoWriter('sequencemjpg.mp4',cv2.VideoWriter_fourcc('P','I','M','1'), 5 , frameSize)
The video is saved correctly with 5 fps as I mentioned.
The problem occurs when I try to load it back, It plays too fast.
This is the code I am using for loading:
**import cv2
import numpy as np
Create a VideoCapture object and read from input file
If the input is the camera, pass 0 instead of the video file name
cap = cv2.VideoCapture(r"sequence.mp4")
Check if camera opened successfully
if (cap.isOpened()== False):
print("Error opening video stream or file")
Read until video is completed
while(cap.isOpened()):
Capture frame-by-frame
ret, frame = cap.read()
if ret == True:
# Display the resulting frame
cv2.imshow('Frame',frame)

# Press Q on keyboard to  exit
if cv2.waitKey(25) & 0xFF == ord('q'):
  break

Break the loop
else:
break
When everything done, release the video capture object
cap.release()
Closes all the frames
cv2.destroyAllWindows()**
I just want to understand how to create the video and load it back with the same fps rat I fixed during creation

Hey I think I can complete it could you assign it to me

if cv2.waitKey(25) & 0xFF == ord('q'): this line makes 25ms delay between frames. OpenCV itself does not introduce delays.


WIP:  Multiple simultaneous NSM Forwarders

Signed-off-by: denis-tingajkin<EMAIL_ADDRESS>
Description

Motivation and Context
https://github.com/networkservicemesh/networkservicemesh/issues/1565
How Has This Been Tested?

[ ] Covered by existing integration testing
[ ] Added integration testing to cover
[ ] Tested locally
[ ] Have not tested

Types of changes

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

Checklist:

[X] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

depends on https://github.com/networkservicemesh/networkservicemesh/pull/1626. https://github.com/networkservicemesh/networkservicemesh/pull/1745

@denis-tingajkin the dependencies are merged a long time ago. Are you still considering this PR?

Reason: multiple forwarders will be supported with SDK


How to open google authentication in a new window?

Is it even possible?

@raphadko This was closed because its not possible, right?


Repo sync for protected CLA branch

The pull request is created from main637794217549425783 to main to fix git push error for protected CLA branch

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


feat: stateless FlightSQL prepared statements

implements client changes for stateless management of FlightSQL prepared statement handles based on the design proposal described in apache/arrow#37720

Marking as draft as my understanding of the status of https://github.com/apache/arrow/issues/37720 is that it is a protocol proposal that has yet to go through a mailing list approval process.
Feel free to unmark as a draft if I am mistaken here.

Marking as draft as my understanding of the status of apache/arrow#37720 is that it is a protocol proposal that has yet to go through a mailing list approval process.
Feel free to unmark as a draft if I am mistaken here.

That's fair. I just marked it as ready to indicate that it's ready for review, but it does still need to undergo the formal spec change process. These PRs are intended to serve as conversation starters around specific implementation details of the spec change.

:+1: I just wanted to avoid this accidentally getting merged early

Mailing list discussion thread: https://lists.apache.org/thread/3kb82ypx99q96g84qv555l6x8r0bppyq

TODO: update protobuf docs with latest changes to apache/arrow#40243 once the documentation in that PR has finalized

The proposal for this specification change was approved on the Arrow dev mailing list. See thread here I've moved this PR out of draft status.

I've updated the protobuf docs to match the docs that were merged into arrow here https://github.com/apache/arrow/pull/40243
Let me know if there's anything else left to address.


Deckard-gradle, failing with space in the folder name

On Windows 7, issuing "gradlew clean test" command, got
java.lang.RuntimeException: java.lang.IllegalArgumentException: java.net.URISyntaxException: Illegal character in path at index 22: file:/C:/Users/Dmitriy Kur/.m2/repository/org/robolectric/android-all/4.3_r2-robolectric-0/android-all-4.3_r2-robolectric-0.jar
at org.robolectric.RobolectricTestRunner$2.evaluate(RobolectricTestRunner.java:226)
...
Are we back in DOS times)? No spaces in the path allowed)
As for workaround: any ideas? It's very hard to change user folder name on windows (requires horsing around with regedit).
I'm not sure how to change the default path for maven default repository folder either. I think this would help to solve the problem for now. Any advices on how to do this? Thank you.

I have the same problem with the robolectric 2.4 ('org.robolectric:robolectric:2.4').
Cheers

@shimel1 @southerton81 Yeah this looks like an issue with Robolectric rather than Deckard. Have you created an issue on here? I had a look but couldn't find one.
I imagine the path is just being constructed wrong in Robolectric (in a way that assumes no spaces).


adding some standard stuff:

rubygems key to travis
github pr templates
CHANGELOG
Rakefile
travis
gemspec
add dev deps
appeased the cops
standard way of managing versions
badges

Signed-off-by: Ben Abrams<EMAIL_ADDRESS>

Thank you for your work. That's so good!


Some cleaning in ansible

While trying to find why we get a weird "artifacts/artifacts" empty
directory, I found some other inconsistent content, like a double "/"
and a parameter that doesn't seem to be used anywhere.
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running
[X] Appropriate documentation exists and/or is up-to-date:

[X] README in the role

/approve

recheck

/approve


where condition + hierarchy creation inconsistencies

Steps to reproduce
Run the notebook below and see the differences between:

V1 and V2: doing the same computation but using the same data in two ways of creating a hierarchy gives us a different outcome
Where condition in both V1 and V2: we try to do measure=constant, level=constant and level=measure and we don't have the same results

notebook.zip
Actual Result
See the pivot tables in the notebook
Expected Result
Everything to be consistent
Environment

atoti: 0.6.4
Python: 3.10.2
Operating system: win32

Logs (if relevant)
N/A

Hi Xavier,
Coming back with some information regarding the first case (Product V1).
First of all, when calling cube.create_parameter_hierarchy_from_column("strike v1", products_table["strike"]), you create a special type of hierarchy that is a factless hierarchy.
You then create a hierarchy that is composed of some of the initial levels, and the strike v1 that was just created. Seems that at this point, since the strike level comes from the base store anyway, we end up with a regular axis hierarchy. (to confirm)
Now with the issue with Product V1, we have tt.agg.sum(tt.where(m['Parent strike min to 1 v1'] == 9515, m["Historical VaR 95%"], 0), scope=tt.scope.origin(lvl[("Products", 'Product v1', 'strike v1')])) that yields values for locations where m['Parent strike min to 1 v1'].
Your notebook states:
One with a where between a level and a constant = OK
One with a where between a measure and a constant = FAIL
One with a where between a level and a measure = OK
Now that I think about it, I have to disagree: where between measure and 9515 works, because the measure used is Parent strike min to 1 v1 which value is always 9515, so they are equal for all locations. Am I wrong ?
If I am right though, that means that "One with a where between a level and a measure" is actually Failing, because the measure Parent strike min to 1 v1 is always 9515, but the level's members aren't.

"Now that I think about it, I have to disagree: where between measure and 9515 works, because the measure used is Parent strike min to 1 v1 which value is always 9515, so they are equal for all locations. Am I wrong ?"
I get it: yes, I should use m['strike with min to 1 v1'] instead of the Parent value
That explains the case V1
BUT
If you change m['Parent strike min to 1 v1'] to m['strike with min to 1 v1'] in line THREE, it doesn't work... Why is that ?

Because you shouldn't change it:
You want to have values where the equality between the members of a level is equal to the evaluation of a measure at this location. But all the while, you want it only for the value 9515. So in this case, you need the parent value, else, the two are equals everywhere. So you see non zero values everywhere.

Make sense. Thank you !
I changed on my notebook and all works OK for the case V1.
Do we have something on the case V2 ?

Same thing for V2, if you use
- m['Portfolio Ref 1 Eq to Measure v2'] = tt.agg.sum(tt.where(m['Parent strike min to 1 v2'] == 9515, m["Historical VaR 95%"], 0), scope=tt.scope.origin(lvl[("Products", 'Product v2', 'strike v2')]))
+ m['Portfolio Ref 1 Eq to Measure v2'] = tt.agg.sum(tt.where(m['strike with min to 1 v2'] == 9515, m["Historical VaR 95%"], 0), scope=tt.scope.origin(lvl[("Products", 'Product v2', 'strike v2')]))

m['Portfolio Ref 1 Eq to Measure v2'] was already working
The issue was on m['Portfolio Ref 1 Measure Eq To Level v2'] and it's not working with both 'Parent...' and 'strike...'

The reason for that was a difference in scope definition:
- scope=tt.scope.origin(lvl[("Products", 'Product v1', 'strike v1')])
+ scope=tt.scope.origin(lvl[("Products", 'strike v2', 'strike')])

It should be scope=tt.scope.origin(lvl[("Products", 'Product v2', 'strike')]) if you want the comparison to be relevant.
To prevent having this kind of mistake, you'd need to drop the hierarchies that you don't need anymore :

h.pop('strike v1')
h.pop('strike v2')
h.pop('optionType')
h.pop('prod')
h.pop('maturity')

scope=tt.scope.origin(lvl[("Products", 'Product v2', 'strike')]) doesn't work in my notebook as it says KeyError: 'No level with name strike found in cube.'
Can you show me how you made it work ?

This version of the notebook
Aurora project.zip

All good, got my mistake. Thanks @Acriss for the investigation
All makes sense

Seems that at this point, since the strike level comes from the base store anyway, we end up with a regular axis hierarchy. (to confirm)

It looks like there's a bug / problem here. While strike v1 is on its own in a single level hierarchy, it behaves as expected and replicated data as an analysis hierarchy.
However, once it becomes part of a multi-level hierarchy, it behaves like a regular hierarchy and relies on fatcs.

Should we keep the ticket open ? Do you need another ticket ?


Fixes #1210. Create a new service scope when cloning RequestContext

Create a new service scope when cloning RequestContext
Doing so ensures that any queries that use the cloned context get their
own scoped service instances instead of using the scoped service instances
of the original RequestContext.
This solves the issue of inadvertently having subscription event queries
use the service scope of the originating long-lived subscription request.
Addresses #1210

Hi @rafd123,
had to really look at the impact on this one, but it looks good.
will push my test in a minute.


Fabric: DatePicker

Summary

Add DatePicker to Fabric. This is a wip. Outstanding question around how we deal with native date primitives in fabric, this means the date prop / onDateChanged props / events are not functioning.
Changelog

[iOS] [Added] - Fabric: DatePicker
Test Plan

Run RNTester with Fabric, observe that DatePickerIOS example works correctly.

Thanks for doing this. Since we are actually extracting this component to a separate repo as part of Lean Core, I don't think we'll merge this.

@ericlewis please let me know where I can follow this work further! Its interesting to see how the actual component migration looks like after all the Fabric internal discussions.
I am particularly interested in other files that need to be added. Maybe we can have a chat on Discord? :D

@grabbou more than happy to chat on Discord. It’s been quite a lot of fun playing with the system, especially with how easy it is to get something bound and working fairly quickly. I’ll continue to post on Twitter / discord my discoveries.

Also, for anyone who runs across this in the future: the answer to date primitive in fabric (as of this comment) is you can pass in dates as floats (FloatTypeAnnotation) because they are represented by JSI as floats. I haven’t figured out how to get it to work in the opposite direction however. Heheh.

Also yeah, just wanna make clear: I love that you are doing this :)

@cpojer no worries! I have figured out a way to be able to “sideload” components that are missing, so dev is still easy :) trying to come up with a plan for how to apply this to components for 3rd parties & thing I have a fairly workable system going.


Possible memory leak when initialising child element via constructor

Hi,
While initialising a child element as bellow, it seems like a memory gets created:
import { LitElement, html, css, property } from 'lit-element';
import { TextInput } from './text-input';

export class SearchAutocomplete extends LitElement {
  searchInput: TextInput = new TextInput(); // <-- creates memory leak
  ...

Chrome isn't able to GC my web-app, so after 4-5 page reload I end up with around ~300MB of memory that's stuck and can't even manually be cleared (by clicking the bin - top left or switching tabs) but via closing and reopening the web-app in a new tab.
If I create a plain class:
export class TextInputTest {
  sayHello() {
    console.log('hello world');
  }
}

than use it for the initialisation, the leak is gone:
export class SearchAutocomplete extends LitElement {
  searchInput: TextInputTest = new TextInputTest();

git repo to demo:
https://github.com/vedtam/lit-leak-test
I have created a demo that can be tested on stackblitz as well, but for most accurate testing the code from the repo would be more appropriate:
https://stackblitz.com/edit/lit-element-leak-demo?file=app-main.ts
Regards,
vedtam.

I'm able to replicate this on Windows/Chrome 74

Thanks for the repro. A memory leak across page refreshes would be a browser related bug. It's unclear right now if this is actually a leak. I modified your index.html to include the following to be able to check memory with the Chrome dev tools open. Careful as if there's a leak, this will eventually crash the browser:
<script>
    document.title = new Intl.NumberFormat().format(performance.memory.totalJSHeapSize);
    window.location.reload();
  </script>

When I run your app with this addition, I see the usage never go above about 350MB, cycling down after that. With the dev tools open, however, the number keeps going up.
I'll try to check with a v8 expert to see if this is an issue that should be filed. It does seem, at least, like there's a Chrome dev tools issue.

Filed as https://bugs.chromium.org/p/chromium/issues/detail?id=968603.

Hi,
This becomes more and more interesting, I haven't tried with dev tools closed. Will follow closely the issue you filed on chrome.

not sure if related or helpful here, but I was experiencing a similar issue across page refreshes. My use case had an async/await resolving inside a reoccurring animationFrame request. I ended up boiling my example down until there was no more lit-element and the problem persisted until I closed and reopened the tab.
Seems to validate the issue with progressive memory leak from chrome tab process.

Looks like this was fixed in Chrome.


[release-4.10]: UPSTREAM: 121196: Prevent rapid reset http2 DOS on API server

The upstream PR 121196 was only backported to Kubernetes 1.25, so we need to port it to 1.23 (4.10) ourselves.
proof: https://github.com/openshift/cluster-policy-controller/pull/141

Closing as 4.10 is EOL.


pipelines-ascode to pipelines-as-code

/retest

I think we need https://github.com/openshift-pipelines/pipelines-as-code/pull/86 first so we can test that pr (or add you to openshift-pipelines org to be allowed)

/ok-to-test

cluster will be automatically reinstalled at noon CET so hopefully the new ok-to-test shningan would work then,

/ok-to-test

/lgtm 👍

/ok-to-test

ah nice /ok-to-test works now!

/ok-to-test
(there is a bit more work to do with /ok-to-test if there was a previous /ok-to-test the new commit should be allowed)

This commit sounds good to me, just need a rebase with a proper commit message and we are good to go  ! 🚀

/ok-to-test

Thanks 🙇🏻
I have updated https://github.com/openshift-pipelines/tekton-asa-code-infra so the reinstall works properly with the namespace change


Add a maximumNumberOfThings property

It'd be nice as an addition for DogeKit to trigger stop automatically when a certain number of labels are reached, exposing such number as a maximumNumberOfThings property.

@DavdRoman PRs welcome! :)