[UI] Fixed pagination issue in database overview

Notes for Reviewers
This PR fixes #
server-side pagination is changed to indexing 1,
for pages 1 & 2, the server returns the same result
Signed commits

[ ] Yes, I signed my commits.

@MUzairS15 I recorded the issue
https://github.com/meshery/meshery/assets/67836725/be952b07-d254-45bf-8562-e800238f10bd
DataTable doesn't have a reference to page state, its page starts from 0, but server indexing is based 1. so for pages 0 and 1 it returns the first-page result

@UsmanDev09 will you give this PR a review?

Thanks @Meyazhagan. This PR LGTM.

Yay! 🎈


Worked on Issue#60

Added a black layer with 0.15 opacity for better text visibility.

Fix it

 Fix it

Sir can you please explain what i have to fix i am unable to understand?


FHIR version

Hello @Chinlinlee , I was wondering which FHIR release (DSTU2, DSTU3, or R4) does this schema generator correspond to?

This generator use FHIR R4 version

Thank you!


DO NOT MERGE! Fast Forward stable to<PHONE_NUMBER>38d6f3c32eb1234eadfd4bc462279a.

This PR will be merged automatically once checks are successful.

Jenkins job istio-testing/pr-stable passed


Create Student Group Page

4/8 Friday Deadline
Please check in with Fiona and Serena when completed

Create student group page as seen on figma
Communicate with Create Student & Student Group Modals team to connect the edit group button
Communicate with Student Page team to connect the view profile button

/cib student-group-page

Branch 88-student-group-page created!


Added SPM Support

This is a really simple library, so adding Swift Package Manager support is really easy even though it's all ObjC.
If you merge this in, all you need to do is create a tag for each release (which you do for Cocoapods anyway) and SPM handles the rest. Don't need to publish anywhere else.

This does build, but imports don't work correctly. Will submit a new one once it's fixed.

Okay, this works in my local test project now so I think we're good.

This is great, thanks very much for your work here, much appreciated!


Facebook Version Deprecation

Do we still use this version? If so, we need to upgrade before July 10.

Nevermind, I ran the upgrade tool and it said:

Your app hasn't made enough calls to the Graph API to show any info for the methods you selected between v2.3 and v2.9


[Feature Request] Like Songs in YouTube music

Option for adding imported songs from Spotify to be added to "Liked Music" playlist of YouTube music instead of "Liked Music (Spotify)". Meaning like songs as they getting imported. All this with the spotify_to_ytmusic liked option.

see #38

I've seen that PR but how come it isn't implemented in the main repo yet?

Any updates to the question from @MuhammadBayiz ?

Sure can be merged, just needs some conflicts to be resolved. Feel free to merge/resubmit


🪸 implement Json preview component to be used in "Request new schema" [epic 350]

Notes

layout is not ready yet, but there's a very high possibility that we need that
if no immediate implementation in DS is planned, we can add it and see how we can integrate that in our form component
could be a readonly textarea with formatting? not sure how accessible that is

Solved in this issue: https://github.com/aiven/klaw/issues/370


New UI: add copy branch button

Add a button to copy the branch name of a given pull to the clipboard.
Old UI has this, but you had to hover, then press Ctrl-C. Most of the time it worked, but there was no feedback.
Hovering the pull shows the button:

Button hovered:

After Successful Copy:

CR 👍 with a question

CR :goat: through d9a2a16

QA 🎬
Copy branch name button works.

deploy_block :+1:  on #257


As a project coordinator, I want to see if a report is received, late or upcoming

Description:
Custom widget for the Received / Late / Upcoming date of a report. This widget should use data from the reporting_requirement table created in #501 to determine whether it is received, late or upcoming.
Acceptance Criteria:
Given I am viewing the milestone reporting page
When a report has been received
Then I can see a widget showing that it has been received
And the widget shows a green check mark with the date it was received
Given I am viewing the milestone reporting page
When a report is late
Then I can see a widget showing how many days overdue the report is
And a warning icon beside the widget should be visible
Given I am viewing the milestone reporting page
When a report is not received or late
Then I can see a widget showing how many days until it is due and the due date

Figma design can be found at https://www.figma.com/file/PzOY8RyQnVXnjZJdSv2AqQ/Wireframing_Figma?node-id=989%3A2796 all 3 states of this widget are shown in the Quarterly Reports wireframe

Development Checklist:

[ ] A widget exists that can display the 3 states of  a report (received, late, pending)
[ ] The widget uses data from the reporting_requirement table to determine what to display
[ ] The widget is implemented on the Milestone Reporting page
[ ] Meets the DOD

Definition of Ready (Note: If any of these points are not applicable, mark N/A)

[ ] User story is included
[ ] User role and type are identified
[ ] Acceptance criteria are included
[ ] Wireframes are included (if required)
[ ] Design / Solution is accepted by Product Owner
[ ] Dependencies are identified (technical, business, regulatory/policy)
[ ] Story has been estimated (under 13 pts)

·Definition of Done (Note: If any of these points are not applicable, mark N/A)

[ ] Acceptance criteria are tested by the CI pipeline
[ ] UI meets accessibility requirements
[ ] Configuration changes are documented, documentation and designs are updated
[ ] Passes code peer-review
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Ticket is ready to be merged to main branch
[ ] Can be demoed in Sprint Review
[ ] Bugs or future work cards are identified and created
[ ] Reviewed and approved by Product Owner

Notes:

Do we need screens for the different states for this custom control?

received
late
"not yet received but not late"

yes, I was going to ask about this in our planning session

Hey team! Please add your planning poker estimate with ZenHub @matthieu-foucault @Sepehr-Sobhani @mikevespi

Please add your planning poker estimate with ZenHub @pbastia

Looks great @BCerki ! This widget helps make it clear when a report is received or late. I tested this is dev by creating a project and adding in various milestone, quarterly and annual reports and then playing around with date fields. I also tested by editing existing report due dates to reflect when a project has been received. Meets acceptance criteria.


How to generate synset.txt files?

We can use im2rec according to the lst file to generate rec files. But how could we generate synset.txt files? And if we use NDArrayIter, how to find the relation between the sequence of the prediction and label file?
Many thanks.

For example, for a 2 class classification problem. The label is 0 or 1. And we get the prediction results from the network which is [0.5, 0.4], what's the possibility of getting result of 1 and what's the possibility of getting result 0.

@lincolnBush is this what you are after? http://data.dmlc.ml/mxnet/models/imagenet/synset.txt
The mapping from index to label in the synset.txt file is done outside the training/evaluation process, and the actual labels are not encoded in the mxnet symbol or data iterators as it is not required.  Does this clarify your question?

Thank you very much for your reply. I mean how to generate this file by programme. How does this file related to the label by programme.

This file is supplied together with the training dataset, and is not created by any script in the MXNet repo.  You can find out about the origin of the synset.txt here: http://image-net.org/challenges/LSVRC/2015/browse-synsets

@lincolnBush
After creating .lst and .rec files you can make your own synset (example in R for cifar10 dataset):
labels <- read.table("cifar_train.lst")
labels <- labels[!duplicated(labels$V2), ]
class_labels <- data.frame(response = labels$V2,
                                          label = sapply(tmp, function(x) x[1]))
class_labels <- class_labels[order(class_labels$response), ]
write.table(class_labels, "synset.txt", row.names = FALSE, col.names = TRUE)

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!

I have the same problem with that. What's the mean of the id(e.g. n01440764) in front of the class name? And, how can I do to make my own synset.txt?

Hi! I was stucking with the same problem and solved with this way : creating a .lst file from my dataset. Convering the .lst file to the synset file with this python code :
f= open("C:/Users/SAHINA/Desktop/mxnet_sample/mxnet_sample/OKNOK.lst","r")
f2 = open("C:/Users/SAHINA/Desktop/mxnet_sample/mxnet_sample/OKNOKsynset.txt", "w")
for line in f:
line = line[line.find("\t")+1:len(line)]
line = line[line.find("c"):len(line)-5] + " " + line[line.find("\t")+1:line.find("\")]
f2.write(line)
f2.write("\n")
f.close()
f2.close()
So you just need to change the file path and the characters to find and move for having the synset order. An example line from my .lst file to synset file is :
315	0.000000	NOK\crop53_3906_2.jpg   ---> crop53_3906_2 NOK

@lincolnBush is this what you are after? http://data.dmlc.ml/mxnet/models/imagenet/synset.txt
The mapping from index to label in the synset.txt file is done outside the training/evaluation process, and the actual labels are not encoded in the mxnet symbol or data iterators as it is not required. Does this clarify your question?

Hi,
The domain may changed,
May i have the latest domain of
http://data.dmlc.ml/mxnet/models/imagenet/synset.txt
TY


Address Validation Reconciliation Component

Address Validation Reconciliation
Overview / Summary
Create a component that will permit a user to pick from an entered address or a automatically validated suggested address. The entered address should have the fields that may be incorrect or incomplete highlighted. The component should provide a button to edit the entered address and a button to confirm a selected address.
Summary description of UI component
This component is essentially a form with a radio input and styled labels.
Rationale for why this component is necessary
This component is necessary for reconcilling a customer entered address with an address validation service and providing a way for the customer to edit their address, keep their address as it was entered, or to use a suggested address from an address validation service.
Expected use cases
This component will be used by the address book when an address validation service plugin is in use. When a customer enters an address that cannot be validated, the address validation reconciliation component should be shown. Once a customer has selected a valid address or edited an address to make it valid, the address should be added to the address book or cart as necessary
Images / Designs of UI component in context

JSX
This component exists in the Meteor Reaction app already and was updated in #4086 to match these designs. I believe we will just need to port the existing component to the Component Library
See https://github.com/reactioncommerce/reaction/pull/4086#issuecomment-405134774
// Invalid field information could potentially be passed inside of the address object if that makes more sense here.
propTypes: {
  addressEntered: Reaction.PropTypes.address,
  addressSuggested: Reaction.PropTypes.address,
  invalidFields: ["region", "postal"]
  onSubmit: PropTypes.func,
  onEdit: PropTypes.func,
  warningDescription: PropTypes.string
}

Render Criteria
This component should be rendered when a user attempts to create a new address and and the address creation is rejected and a suggested address correction is supplied instead.
UI States
Normal
The normal state for this component is really a specific error state for the address book involving a new address that has been entered but caught during a validation process
https://app.zeplin.io/project/5afc82f5e8d73e250ff9d855/screen/5b197a1ca2d5663159876954
It appears that the "Suggested address" should be selected by default when this component is initially rendered
There are no additional states for this component in the designs.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Only build master branch on travis

As @maxpoulin64 pointed out, we pointlessly trigger twice for pull requests done from this repository.

The problem with this is that we don't build until you create a PR, so if you are developing something on a branch, you won't see failures on travis (if you forget to lint locally for example) until you create a PR.

That's true, but is it a big problem? Ideally you should do npm run test before creating a pull request. And this change only affects the maintainers, we can probably live with it.

That's true, but is it a big problem?

I think it's a bigger problem than just travis running 2 builds. Especially as it runs them in parallel, so it's not like it increases the time.

Ideally you should do npm run test before creating a pull request.

Especially as it runs them in parallel, so it's not like it increases the time.

That. I would say until this is a problem (I don't see why it could be right now, but we never know, Travis CI can evolve, ...), it's not one.
Also, one detail: the two builds are not the exact same. According to the Travis CI doc, while the master build tests the branch itself, the PR one tests the merge of the PR and the master branch:

Rather than test the commits from the branches the pull request is sent from, we test the merge between the origin and the upstream branch.

I think having both is interesting.

Fair enough.


Support authentication & authorization in cvdr

Implement this to avoid store tokens in CO.
For more information, please refer to b/353884144#comment13

LGTM. Perhaps we need @ser-io's review unless waiting for a long time.

@chyijia thanks for resolving existing comments, I'll be reviewing the PR tomorrow.


added Thang Names

Coverage remained the same at 55.826% when pulling 104a87d3c40bc44dbe6273ad3c8f2af9c6fe97a5 on xjasminelu:master into 3665a8a1011828d1af7ac9d1a40cd0bf23e093a8 on codecombat:master.

Thanks @xjasminelu, good names!


Create Sales Order App with SAP RAD by Mendix

Tutorials: https://developers.sap.com/tutorials/mendix-salesorders.html
On the step 13, Studio pro 9.3.0 has a different header from the picture on the tutorial.

When I click Publish, "SalesOrders210723-Development" application is deployed and started in my trial account.

SalesOrders210723 is my app name.

However when I click Responsive Web, I'm getting an error in the browser.

If I click on "Run on Cloud Foundry" instead of "Publish", Studio Pro will pop up with the following error.

My questions are the following two.
Should I go to "Publish" or "Run on Cloud Foundry"?
How can I fix the error?
Best Regard,
Masahiro Obata

Hi Masahiro,
Sorry for the delay in my reply.
I am looking into it and will get back to you soon.
Regards,
Pao


The name 'App' does not exist in the current context

I'm having trouble building the sample project or the freshly created projects, when adding the line App.Configure() in FinishedLaunching. Is there an additional step (from the "getting started" steps) that I'm missing?

The docs are incorrect.  You need to use:
Firebase.Analytics.App.Configure();

In my, firebase.analytics.app.configure() i give error, app nota found in analytics

App.Configure method was moved from Firebase.Analytics namespace to Firebase.Core namespace in the mayor release from v3.x to v4.x. Please, import Firebase.Core to your libraries to use this method.

Add Firebase.FirebaseApp to the scope of the app. For example:
`
    Firebase.FirebaseApp.CheckAndFixDependenciesAsync().ContinueWith(task => {
        var dependencyStatus = task.Result;
        if (dependencyStatus == Firebase.DependencyStatus.Available)
        {
            Firebase.FirebaseApp // I mean here
            // Create and hold a reference to your FirebaseApp,
            // where app is a Firebase.FirebaseApp property of your application class.
            app = Firebase.FirebaseApp.DefaultInstance;

            // Set a flag here to indicate whether Firebase is ready to use by your app.
        }
        else
        {
            UnityEngine.Debug.LogError(System.String.Format(
              "Could not resolve all Firebase dependencies: {0}", dependencyStatus));
            // Firebase Unity SDK is not safe to use here.
        }
    });

`


DIVER: Update Readme.md and create a /docs folder

He plan is to simplify the readme and move the majority of documentation to multiple markdown files in ./docs.

Completed with PR #98, hopefully it will make it more simple for people to keep the docs up to date.


C# '@' special character for multi-lines strings is not displayed correctly like in visual studio editor

Godot version:

3.2.1 Mono
OS/device including version:

Windows 10
Issue description:

Using '@', the verbatim identifier, for multi-lines strings the godot editor is not able to know when the strings ends.
Exemple in godot editor:
//displayed as code (normal)
class Foo
{

    public void foo()
    {
           string str =
// displayed as string content (normal)
           @" line 1
           line 2
           line 3
           ...."
// displayed as string content (bad)
           ;
     }
}

Visual studio editor knows when strings ends when started with '@'.
Exemple in vs editor:
//displayed as code (normal)
class Foo
{

    public void foo()
    {
           string str =
// displayed as string content (normal)
           @" line 1
           line 2
           line 3
           ...."
//displayed as code (normal)
           ;
     }
}

Duplicate of #32336 (fixed in 3.2.2 by #37484).


Deleted repo still pending in the list

We have enabled greenkeeper on our public example repos (thanks!) but one of the repos initially enabled was deleted (it was an experiment no longer needed). But it still shows up under account and we cannot delete it from greenkeeper!
The yellow repo in the image

Our account is cypress-io and we appreciate help with this little issue

Unfortunately GitHub does not send a webhook when a repo was deleted, so Greenkeeper does not get notified and got the chance to delete the documents regarding the pull request and repository in the database.
If Greenkeeper would try to communicate (create a new branch ect.) to the repo it would get an error from GitHub but since the initial pull request was not merged, Greenkeeper ignores this repo (and never knows about it's deletion)
I'll delete it from the database and contact GitHub about how useful a webhook like this could be.
Thanks for reporting!

Thank you for the resolution. Would it make sense to have a "sweeper" task that goes and checks repos existence every day for example? Then it could delete the project if repo is gone
Sent from my iPhone

On Oct 24, 2017, at 08:53, anderspree<EMAIL_ADDRESS>wrote:
Closed #541.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

oh I found out it does send a webhook but Greenkeeper doesn't have actions for it!  I'll make a ticket about handling those events!

https://github.com/greenkeeperio/greenkeeper/issues/543


Fixed crash on empty terain

Copy of #39 but coming from a new branch instead of main.

@natebot13 should this be closed?

Sure, seems like the latest way tiled uses terrain is with wangsets. Older tilesets will crash if they're attempted to be parsed, but they can simply be saved with a newer version of tiled to change to wangsets.


Cannot read property of 'GitHub' of undefined

When I install the package and start using it, I get: Uncaught TypeError: Cannot read property 'Github' of undefined. I'm not sure where this could be going wrong.

Similar problem "Cannot read property 'Google' of undefined". My code was working when I checked last time.

@yinnovationlab good news, version 1.2.10 fixes this

I still had this problem in 1.2.1 for Meteor <IP_ADDRESS> I believe it's due to Meteor using the github-oauth package instead of github, similiar to this issue: https://github.com/percolatestudio/meteor-google-api/issues/40#issuecomment-280662638
I've fixed In my project by referencing the new package. I'll put in a PR if you'd like to check it out, but I haven't checked for older Meteor versions.

I had this issue, got round it by running meteor add github

Not sure why this was closed, but I've got it to work by importing Github via es6 instead of using Package.github.Github. I can create a PR with refactors for all of the services and I plan to add Bitbucket and Gitlab if those are of interest


Sklearn: support multiclass classification

Pull Request Test Coverage Report for Build 86

31 of 60 (51.67%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 67.589%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xgensemble_io.go
0
1
0.0%

internal/pickle/sklearn.go
14
22
63.64%

skensemble_io.go
17
37
45.95%

Files with Coverage Reduction
New Missed Lines
%

skensemble_io.go
1
75.33%

Totals

Change from base Build 82:
-0.4%

Covered Lines:
1441

Relevant Lines:
2132

💛  - Coveralls

Pull Request Test Coverage Report for Build 86

31 of 60 (51.67%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 67.589%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xgensemble_io.go
0
1
0.0%

internal/pickle/sklearn.go
14
22
63.64%

skensemble_io.go
17
37
45.95%

Files with Coverage Reduction
New Missed Lines
%

skensemble_io.go
1
75.33%

Totals

Change from base Build 82:
-0.4%

Covered Lines:
1441

Relevant Lines:
2132

💛  - Coveralls


Visible parameters of the embedding method

Is this something that could be considered?
Looking ahead into having other embedding methods, or more "sophisticated" Composites that can make use of the flexibility of minorminer, it would be very useful to be able to read out the parameters and eventually add them to the sampler parameters. e.g:
class ???EmbeddingComposite(...
...
param = self.child.parameters.copy()
param['embedding_parameters'] = self._embedding_method.parameters.copy()
...

# Example usage
import minorminer

print(minorminer.parameters)
>>> {'chainlength_patience',
 'fixed_chains',
 'initial_chains',
 'inner_rounds',
 'max_beta',
 'max_fill',
 'max_no_improvement',
 'random_seed',
 'restrict_chains',
 'return_overlap',
 'skip_initialization',
 'suspend_chains',
 'threads',
 'timeout',
 'tries',
 'verbose'}

I am wondering if we should just be relying on Python's inspect module for this sort of thing. I am worried about the maintenance. @boothby what do you think?

I agree, the way I did it would be hard to maintain. It's mostly an example of an implementation and how to use it. Unless it only has one dictionary, and not two, i.e. names and parameters.
Would inspect work? I think Cython isn't supported in inspect, and also, the parameters are only parsed from **params not as implicit function arguments.

Sorry for the delay, I've been bogged down with other work... I don't mind this change at all.  When I add a parameter, it requires changing about a dozen lines of code across something like 5 files.  So it's already a nightmare; this PR only moves a line and my workflow already begins with grep.  My temptation is to pitch the review back to @arcondello -- how do you feel about exposing parameters to EmbeddingComposite in this way?

Okay this issue came to a head today... again I apologize for the extended delay.  Alex and I had a chat and I decided that this was a perfect grenade-jumping exercise.  Cython doesn't work with inspect, but inspect does the right thing.  So I wrote a horrible python wrapper for the cython function, with all the parameters.  Yay.
https://github.com/dwavesystems/minorminer/pull/86


Witherspoon MFG Stage related Test

https://github.com/openbmc/openbmc/issues/664

This includes both January and April deliverables.

Please feel free to change the estimate.

https://github.com/openbmc/openbmc-test-automation/issues/297
https://github.com/openbmc/openbmc-test-automation/issues/308

[ ] Please list out all TCs associate with it. ( Send via email or put it in internal repo)

[ ] Evaluate Automation can be done or need manual execution

[ ]  Create stories phase wise

[ ] Create stories for automation and check-in it in internal repo.

Closing tracked differently


feat: pfe-tabs style updates

Component updates
Update pfe-tab component style to match design system specs
Preview
Link(s) to demo page(s) where this element can be viewed:

Link to pfe-tabs

What has changed and why
Summarize files edited as part of this MR along with a brief description of what was changed/why.

Testing instructions
Be sure to include detailed instructions on how your update can be tested by another developer.

Browser requirements
Your component should work in all of the following environments:

[ ] Latest 2 versions of Edge
[ ] Internet Explorer 11 (should be useable, not pixel perfect)
[ ] Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
[ ] Firefox 68 (or latest version for Red Hat Enterprise Linux distribution)
[ ] Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
[ ] Latest 2 versions of Safari
[ ] Android mobile device (such as the Galaxy S9)
[ ] Apple mobile device (such as the iPhone X)
[ ] Apple tablet device (such as the iPhone Pro)

Ready-for-merge Checklist
Check off items as they are completed.  Feel free to delete items if they are not applicable.

[ ] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
[ ] Tests have been updated to cover these changes.
[ ] Browser testing passed.
[ ] Repository compiles and tests pass.
[ ] Changelog updated.
[ ] Documentation (README.md, WHY.md, etc.) updated or added.
[ ] Link to the demo recording:
[ ] Approved by designer.

Merging
Please squash when merging and ensure your commit message uses conventional commit formatting.
Be sure to share your updates with the<EMAIL_ADDRESS>mailing list!
https://rally1.rallydev.com/#/detail/userstory/378976493960/testcases?fdp=true

It looks like the right border is too small on any tab that isn't the last child.


The url inside the composer.json should be in their secure version?

Hi,
I think the url should be written with https;// instead of http://.
https://github.com/amphp/byte-stream/blob/77decf18efbabf23e3c35e16946322d6d621fa61/composer.json#L3
What do you think?
Regards

Yes, it should, do you want to send a PR?

Sure, no problem


Drag end event

Hi,
Just started using this excellent library and everything is working as expected although I am having difficulty trapping the pm:markerdragend event. This is my code which otherwise works ok:
L.geoJSON(g, {
        onEachFeature: function (feature, layer) {

            layer.options.id = feature.properties.id

            layer.on("dblclick", function(e) {

                if (editingLayer == drawnLayer) {

                    if (this.pm.enabled()) {
                        this.pm.disable();
                        editing = false
                        console.log(this.options.id)
                    } else {
                        if (editing == false) {
                            this.pm.enable()
                            editing = true
                        }
                    }
                }

            })

            layer.on("pm:markerdragend", function(e) {
                this.pm.disable()
                console.log(this.options.id)
            })

        }

    })

Double-clicking on a marker enables and disables the editing but drag end doesn't. Could you suggest how I might improve this?

You need to use pm:dragend.
pm:markerdragend is for the vertex marker on a line

Excellent - thank you


Update TO_DATE, TO_TIMESTAMP scalar functions to support Utf8View

Is your feature request related to a problem or challenge?
Part of https://github.com/apache/datafusion/issues/11752 and https://github.com/apache/datafusion/issues/11790
Currently, a call to TO_DATE or TO_TIMESTAMP* UDFs with a Utf8View datatypes fails. After the change that fixes this issue, it should not.
We are working to add complete StringView support in DataFusion, which permits potentially much faster processing of string data. See https://github.com/apache/datafusion/issues/10918 for more background.
Today, most DataFusion string functions support DataType::Utf8 and DataType::LargeUtf8 and when called with a StringView argument DataFusion will cast the argument back to DataType::Utf8 which is expensive.
To realize the full speed of StringView, we need to ensure that all string functions support the DataType::Utf8View directly.
Describe the solution you'd like
Update the functions to support DataType::Utf8View directly
Describe alternatives you've considered
No response
Additional context
The typical steps are:
Write some tests showing the function doesn't support Utf8View (see the tests in [string_view.slt](https://github.com/apache/datafusion/blob/main/datafusion/sqllogictest/test_files/string_view.slt) to ensure the arguments are not being cast
Change the Signature of the function to accept Utf8View in addition to Utf8/LargeUtf8
Update the implementation of the function to operate on Utf8View

Example PRs
Update to use an arrow kernel that already supports StringView:

https://github.com/apache/datafusion/pull/11787
Change the implementation to support StringView directly:
https://github.com/apache/datafusion/pull/11676
Change implementation (option 2):
https://github.com/apache/datafusion/pull/11556

take


🤫 Secrets commands

Secrets create, list and delete commands all added.
Note includes:

Alew's fix for cloudflare-rs http update https://github.com/cloudflare/wrangler/pull/994/commits/90dbe12ce34c33abe6e9bbbc55c36ff8af226382
Moved KV::interactive_delete to utils to be used across commands

TODOs:

[ ] Verify error codes and messages (e.g. message for 10007)
[ ] [Nice to have] Check 200 v. 201 and give appropriate messaging for create
[x] Remove debug print outs from format_error

closing in favor of #1045


Update service-validate.js

Solves the issue :
Error:<PHONE_NUMBER>03136:error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure:../deps/openssl/openssl/ssl/s23_clnt.c:757

From what I understand, some ciphers are known to be exploitable.  Allowing all ciphers might not be appropriate here I think, sorry


deps: Remove expo-splash-screen.

Empirically, this is no longer needed: we added it in c4fca9d9d to
satisfy a peer-dependency constraint, but now I'm not getting a
peer-dep warning if I remove it.
See #4279 and discussion at
https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/React.20Navigation.20v5/near/1042787
for some warnings from Greg about the design of expo-splash-screen,
should we choose to actually use it one day.

Closing as superseded by #4949.


Update konstellation chain name

Update konstellation chain name

Why are the image links changing?

Why are the image links changing?

Updated. Please check.


Getsidechaininfo fix

Made a new branch and PR to not risk messing up rebase.
Ive added test for range of prefix values, PUBKEY_ADDRESS reconstruction of address and SCRIPT_ADDRESS reconstruction of address. Not sure if theres a way to test BLINDED_ADDRESS and SECRET_KEY?

Rebase is fine as long as you haven't edited the same files with the latest changes in master. If you have then you need to carefully edit these, which is the same with merge anyway. In this case rebase should work instantly. Then you will need to force push your branch though because rebase will change the history. Merge on the other hand creates a commit that reapplies all the changes from master, which is a bit weird because you are committing them again. Usually neither are required but in this case it is due to the master test breaking.

Cheers, the problem though was laurence updating master after I had merged

I think i had trouble there because Laurence was changing things after I had merged. I see what you mean with force push now also.

Yeah not sure what happened. I think you started from a branch that wasn't merged, then it got merged in master and you mixed both rebase and merge. 14 commits for a test addition is too much. Can you create a clean branch? Take some time to understand the merge/rebase difference, it's important.


Create Your SAP Fiori Launchpad Project and its Module

Tutorial URL: https://developers.sap.com/tutorials/cp-workflow-2-create-module-cf.html
Unfortunately I cannot deploy to SCP.
Application "workflowtilesFLP" stagedExecuting task "deploy" on application "workflowtilesFLP"...Execution of task "deploy" failed. Download the application logs "workflowtilesFLP", via the dmol command, and check them for more information.A step of the process has failed. Retrying it may solve the issue.
I was following the tutorial very strictly. It would be nice if someone knew what's wrong.

Hi,
Check your changes in step 2. See whether the instance name has been updated in all places, for example, in the Code Editor view. If everything is OK, try to deploy again.
If the error persists, you should check the log file. Use the command prompt to log in to your Cloud Foundry account: cf login and enter your credentials. Then use the cf dmol command shown in the Web IDE console to download the log. Check whether there is more information on the workflowtilesFLP application.
Best regards,
Astrid

@sudoManci
Thank you for your feedback. We haven't heard from you in a while, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


Rename standard?

I do not thing it is was good idea to name standard standard because it causes an ambiguity.
If one talks about the standard library of Nit, does one talk about /lib or about /lib/standard?
For instance, even http://nitlanguage.org/doc/stdlib/ it titled Nit Standard Library but in fact talks about /lib and not only about /lib/standard
One solution to avoid this communication issue is to rename the library standard as something else.
Some proposals

use the name nit (like in java and other). but this does not completely solve  the communication issue. eg "nit is the default a library in the standard libraries of Nit."
use the name default because it is the library that is imported by default.
use the name implicit because it is the library that is implicitly imported.
use the name base or basic because it is the basic library that contains basic classes and services.
keep the name standard because this issue is just pure nitpicking.

Ideally, http://nitlanguage.org/doc/stdlib/ should point to standard only, though right now since we lack packages, we tend to use anything in lib as part of standard.
This should be fixed once we have a proper package distribution system, hence the standard can keep its name imho, as all that is in lib should be hosted someplace else than the compiler itself
That, and some modules in lib could be migrated to standard once they have reached a certain maturity level, like socket for instance

What about core?
My idea is the following:

core library equals the current standard, it's the minimum and default set of functionnalities
standard, is the set of core + some libraries maintained inside the nit project (because used within src for example), thus given in standard with Nit
everything else is migrated to external package once we have a packet manager

core is nice.
For @R4PaSs the other issue with standard is that "does this come bundled with nit in the standard install" or "does this is imported by default in standard Nit programs",

Going once, going twice, sold!


General improvements in string handling from config check (Fixes #4)

General improvements in string handling from config check:

Use regular expression to exclude colorlog install attempts which sometimes appear
Remove escape characters (not needed and causing issues in service call)
Remove everything before string 'Failed config' (it is repeated anyway after)
Make sure text is not removed if it contains string 'Invalid config'
Remove '(See' (lost after split by ?)

Fixes #4

https://github.com/custom-components/config_check/blob/master/custom_components/config_check/manifest.json#L2 bump this to 0.1.1 as well

https://github.com/custom-components/config_check/blob/master/custom_components/config_check/manifest.json#L2 bump this to 0.1.1 as well

Done, thanks.

You added a duplicate of the file in the root, that's not right

You added a duplicate of the file in the root, that's not right

Sorry, not very used to do this, but still wanting to contribute. Should be OK now.


Sample application update - from Beta to V1.0 Graph APIs

Updated the sample application class files, and minor edits to the readme.md

@Arvind-Ravi looks good. I added the other folks as reviewers for you.


fix: int tests

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Bug fix

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist

[x] My commit message is conventional
[x] I have run pre-commit on all files before creating the PR
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

:tada: This PR is included in version 1.5.10-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Spring Boot Database: refactoring SQL to use a common data base domain service

fix #419
Still not finished.
Has to do same with postgre sql and fix IT orders properties.

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

relates #419

@seraphinandrieux :

you need to rebase against upstream remote, as your branch is not up-to-dated
after that, you need to solve some conflict
can you update the title too plz?

@seraphinandrieux :

you need to rebase against upstream remote, as your branch is not up-to-dated
after that, you need to solve some conflict
can you update the title too plz?

Totally agree,
I just pull requested if some people want to check => I will do it in the day.
Sorry about title, I'm fixing that right now

PR still not ready for reviewing.
I still have to fix 1 IT for PostgreSQL when initializing => It seems to return a 500 have to investigate why.
Could extract some duplicated code between MySQL.java and PostgreSql.java into CommonDatabase

Thanks @seraphinandrieux : I'll review it soon

As discussed together:

the logic should stay in PostgreSQL and MySQL
only extract in Sql Common the properties (jpa, hibernate, etc), the common lib

The work can be done in different smalls PR.
The reason is mainly for maintenance:

when someone wants to improve or fix PostgreSQL part, he/she doesn't need to worry about other database (MySQL, MariaDB etc)
it's important to not do the same mistakes than in generator-jhipster by regrouping all these technologies together, and having complex templates / complex logics, forcing the contributor to test all databases when only improving 1 single technology

@pascalgrimaud I did as discussed in this MR the refacto.
For now, it should be ok, please could you review it and check if I missed something.
Thx :)

Thanks @seraphinandrieux I'll try to review it asap :-)

@seraphinandrieux : the coverage is at 99.97%, probably missing private constructor in SQLCommon :)

U right, intelliJ didn't detected, next time I'll run sonar analysis :)

$200 bounty claimed : https://opencollective.com/generator-jhipster/expenses/64890

@seraphinandrieux : approved


BUGFIX #141, missing a round to define the seconds.

Minor bugfix. It shouldn't compromise the previous results.

Thanks for picking this up!


Add support for a per-input configuration file

Allow per-input customization (disable certain optimizations, change tolerances, etc) via an optional configuration file.

I'm not sure this is valuable.


Performance degradation in version 1.0

The performance of the java client has dramatically decreased from v0.7.3 to v1.0. The NatsBench utility reports a 30% performance reduction for the production of a million 100-byte messages and my own applications run at one quarter of the speed when producing messages.

Whoa. That's really surprising. What gnatsd version? I'll look into it.
On Tue, Feb 7, 2017 at 1:09 PM, bagguley<EMAIL_ADDRESS>wrote:

The performance of the java client has dramatically decreased from v0.7.3
to v1.0. The NatsBench utility reports a 30% performance reduction for the
production of a million 100-byte messages and my own applications run at
one quarter of the speed when producing messages.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nats-io/java-nats/issues/99, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGq4LJShe2oJw9dl6x3ZtDHsX2HCptmBks5raN1_gaJpZM4L6CUC
.

--
Larry McQueary | NATS.IO Product Management
@mcqueary | github.com/mcqueary

I used it with the latest Linux version 0.9.6 of gnatsd, 64bit.

@bagguley Thanks for raising this issue. I see how the problem was introduced and I'll get it fixed ASAP.

No problem, I look forward to the fix!

Closed by #101

Hi, it's in the current master 1.1-SNAPSHOT if you would like to try it out.
On Tue, Feb 7, 2017 at 3:46 PM bagguley<EMAIL_ADDRESS>wrote:

No problem, I look forward to the fix!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nats-io/java-nats/issues/99#issuecomment-278168810,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGq4LOic_3z-_C4GHnLc5AtUdJvG2pEXks5raPRRgaJpZM4L6CUC
.

--
Larry McQueary | NATS.IO Product Management
@mcqueary | github.com/mcqueary

Sorry for the delay in getting back to you. The performance looks good now. Any idea when it might be released? Also, GitHub is showing the latest release as 0.5.3 -> https://github.com/nats-io/java-nats/releases

@bagguley , Good to hear about the performance.  About the github release - the current SNAPSHOT is pretty far ahead of the github releases.  I plan to normalize those at the next stable (non-snapshot) release.
There are a few other things we are looking at before another release - is this blocking you?
Thanks,
Colin

Yes, it would be good to sort out the versioning, it's not clear which ones are snapshots. Plus. the nats.io website says that the latest version is 0.7.1, which is confusing.
It's not blocking me at the moment as I'm using a client version prior to 1.0.


schema errors on webpack.config.js

I am trying to use the ExtractTextPlugin for my webpack but I am getting an code ELIFECYCLE errno 1
I am unable to see any resolution.. please help.. here's my code:
  rules: [
     {
        test: /\.scss$/,
        use: ExtractTextPlugin.extract({
           fallback: 'style-loader',
           use: ['css-loader', 'sass-loader'],
           publicPath: '/Users/AJ/Desktop/Simple\ Websites/Webpack/dist'
        })
     }
  ]

  new ExtractTextPlugin({
     filename: 'app.css',
     disabled: false,
     allChunks: true
  })

"devDependencies": {
"css-loader": "^0.28.4",
"extract-text-webpack-plugin": "^2.1.2",
"html-webpack-plugin": "^2.28.0",
"node-sass": "^4.5.3",
"sass-loader": "^6.0.5",
"style-loader": "^0.18.2",
"webpack": "^2.6.1",
"webpack-dev-server": "^2.4.5"
}

  new ExtractTextPlugin({
     filename: 'app.css',
-    disabled: false,
+    disable: false,
     allChunks: true
  })

If you don't intend to disable the plugin this option is redundant anyways, so maybe remove it entirely

@michael-ciniawsky
Great! I have removed it entirely and the build runs now.. thanks so much!


Defect: IphoneSE (responsive)  'Tutorial and More' section has incorrect text size, no background image

'Tutorial and More' section has incorrect text size, no background image
Steps:

Navigate to https://main--vg-partsasist--hlxsites.hlx.page/ from index.docx by clicking preview
Validate the 'Tutorial and More' section in responsive mode using 'IphoneSE'

Expected:

Actual:

Hi @girishdigrajkar
The issue should be fixed. Could you retest it?

Defect has been fixed as able to see correct details post go live in partsassist.com hence, closing the defect:


[File Watcher (parcel)] Unexpected error: No such file or directory

Does this issue occur when all extensions are disabled?: Yes/

VS Code Version: 1.74.3 (Universal)
OS Version: MacOS Ventura Version 13.0.1

Steps to Reproduce:

Start VS code and open any folder tracked with GIT
Modify any of the code and the source control won't reflect the changes made in the code

Developer logs > WIndows, shows the below error:
2023-01-24 20:03:13.166 [error] [File Watcher (parcel)] Unexpected error: No such file or directory (EUNKNOWN) (path: /Users/UserName/Desktop/folder)
2023-01-24 20:03:13.167 [error] [File Watcher (universal)] restarting watcher after error: No such file or directory
2023-01-24 20:03:13.918 [error] [File Watcher (parcel)] Unexpected error: No such file or directory (EUNKNOWN) (path: /Users/UserName/Desktop/folder)
2023-01-24 20:03:13.919 [error] [File Watcher (universal)] restarting watcher after error: No such file or directory

The notifications section shows this error:

Can you try to reproduce with our nightly insider builds? You can give our preview releases a try from: https://code.visualstudio.com/insiders/

Same issue with the insider version @bpasero

@nosisky anything special you can share about the file system / hard drives / network shares used?

Nothing special just a regular booted MacOS laptop. It used to work fine the issue just started unexpectedly when I noticed the source control was no longer showing the error. It has to do with the Parcel watcher, somehow it's not seeing any GIT-activated folders on my device.

I wonder if this issue is being caused by an installed extension. Can you try to run VS Code without extensions? From the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If you see it is an issue with the extension, please file it against the extension repository itself.

I have tried that, I also disabled all extensions still the same issue. I tried downloading an older version(1.63.0 (Universal)) but the logs show the below error:
[File Watcher (parcel)] correcting a path to watch that seems to be a symbolic link (original: /Users/Username/Desktop/App, real: /Users/username/Desktop/App)

Hm, so it maybe possible that symbolic links are involved or the casing as it is used is different, does that ring any bells?
To see exactly how parcel operates, some logs would be needed. You can enable verbose logging when you run code-insiders --verbose and then from the command palette pick the command "Output: Show output channels" and then pick the "Window" logs. There anything that has "parcel" in the name if of interest.

Thanks, @bpasero  I had two folders with the same name but different casing inside the /Users folder. Somehow, The watcher was tracking the /username instead of /Username which is the currently logged-in user, I had to rename the lowercase home folder and the error seems to disappear for now.


Use asynchronous copy in cupy.copyto

Fix #2159.
cupy.copyto currently uses synchronous DtoD copy in one of its branches, eventually using the default CUDA stream. This PR fixes it to use asynchronous DtoD copy with the current CUDA stream that CuPy manages.

Thanks for the PR.
Does it fix cupy.dot (as reported at https://github.com/cupy/cupy/issues/2159#issuecomment-484938869) as well?

And why "no-compat"?

Does it fix cupy.dot (as reported at #2159 (comment)) as well?

No. Let me check it as another problem.

And why "no-compat"?

It's because that I suppose if a function performs synchronously or asynchronously is contained in its specification. It is not the case here?

The H2D copy of cupy.dot reported at https://github.com/cupy/cupy/issues/2159#issuecomment-484938869 is triggered by initializing CUBLAS. I suppose it's fairly tiny, ~1us or so, and is not an essential issue to address.

Does it fix cupy.dot (as reported at #2159 (comment)) as well?

No. Let me check it as another problem.

OK, then this PR should not close #2159.

And why "no-compat"?

It's because that I suppose if a function performs synchronously or asynchronously is contained in its specification. It is not the case here?

I don't think so. It only affects performance and it won't be imcompatible.

The H2D copy of cupy.dot reported at https://github.com/cupy/cupy/issues/2159#issuecomment-484938869 is triggered by initializing CUBLAS. I suppose it's fairly tiny, ~1us or so, and is not an essential issue to address.

Is it CPU time?
If host-device synchronization occurs, it's more problematic than any CPU time.
I don't think that has to be done in this PR though.

Jenkins, test this please

Jenkins CI test (for commit f1fa586b4e4cf4584876348dfe5ca9f03d2bf268, target branch master) succeeded!

👍

Is it CPU time?
No, it's GPU time that I collected with nvprof and visualized with nvvp. Let me supply its screen shot.

I don't think so. It only affects performance and it won't be incompatible.

Okay!


Just multiply confidence and initial depth together while propagating!

Hi bro, I have the same question about why the confidence should be propagated, the function ModulatedDeformConvFunction looks quite complex, .
Did you get the result of just  multiply confidence and initial depth?


update default cluster version to 1.27

We use K8s version 1.27 in tests, while the default version in CLI is set at 1.26. We should use 1.27 for all.
1.27.3 was taking from Kind release that we use: https://github.com/kubernetes-sigs/kind/releases/tag/v0.20.0

We could. But our controller-runtime is a bit old and technically they are not compatible as-is. 1.27 seems ok from my testing. I think I am more inclined to upgrade to 1.28 once we have the controller-runtime dependencies figured out.

We could. But our controller-runtime is a bit old and technically they are not compatible as-is. 1.27 seems ok from my testing. I think I am more inclined to upgrade to 1.28 once we have the controller-runtime dependencies figured out.

I think as long as that is apart of the controller-runtime dependency work then that sounds fine!

Is it documented on the README.md that idpbuilder should be deployed on kind >= 1.27 since release 0.x ?
@nabuskey

Lets just merge this - I dont think theres a user facing version dependency here we need to worry about documenting since we vendor our kind dep. If a user has a different kind version IME it "just works".


mgm module: tracking issue for Management Key (MGM) support (e.g. get, set, derive)

This issue is for tracking work on the mgm module.

I think it makes sense for the management key and its metadata to be in one of four states:

The default key.
A user-supplied custom key.
A derived key.
A PIN-protected key (optionally custom, but defaulting to a new random key).

All of these could have optional touch requirements.
The current APIs are:

MgmKey::set(yubikey, touch: Option<u8>)

Replaces the existing management key (using the given touch requirement).

MgmKey::set_protected(yubikey)

Replaces the existing management key (using no touch requirement).
Saves the new key in the PIN-protected metadata.
Sets the corresponding admin metadata flag bit.
Removes any existing derived mgm salt.

MgmKey::set_protected(yubikey) provides an effective transtion from any existing state to "PIN-protected", but MgmKey::set(yubikey, touch: Option<u8>) leaves the YubiKey in an indeterminate metadata state (the newly-set MgmKey will work regardless).
We should either have MgmKey::set take a MgmType parameter, or (my preference) have separate APIs for setting each of the four states.

I think it makes sense for the management key and its metadata to be in one of four states:

The default key.
A user-supplied custom key.
A derived key.
A PIN-protected key (optionally custom, but defaulting to a new random key).

All of these could have optional touch requirements.
The current APIs are:

MgmKey::set(yubikey, touch: Option<u8>)

Replaces the existing management key (using the given touch requirement).

MgmKey::set_protected(yubikey)

Replaces the existing management key (using no touch requirement).
Saves the new key in the PIN-protected metadata.
Sets the corresponding admin metadata flag bit.
Removes any existing derived mgm salt.

MgmKey::set_protected(yubikey) provides an effective transtion from any existing state to "PIN-protected", but MgmKey::set(yubikey, touch: Option<u8>) leaves the YubiKey in an indeterminate metadata state (the newly-set MgmKey will work regardless).
We should either have MgmKey::set take a MgmType parameter, or (my preference) have separate APIs for setting each of the four states.

Sounds good to me

Sounds good to me

Closing these issues as the API has been flattened in #274.


Flip arguments for power

Currently the order is power x base but I suggest power base x is more coherent and provably be renamed to exponent.
The inverse, logarithm, has already this order. Power or exponent and logarithm functions are 2-args and non-commutative, just like subtract and divide, but those are rather a 1-arg function happens to manifest its base.
Note that changing base is just a multiplication of its input, just like cosine is defined as adding a constant to the input of sine: Cosine ← ○ + η.
Exp ← ⁿ∶e
Exp 1
ExpTwo ← Exp × ₙe2
ExpTwo 1
ExpTen ← Exp × ₙe10
ExpTen 1

I mean the exponent function deserves to be a builtin primitive than power function.

current order of power makes sense for alot things power: square ⁿ2 square, cube ⁿ3,  sqrm ⁿ0.5 sqrt, cbrt ⁿ ÷3 1,  polynomials, & what not for both scalars & vectors alike.
so, massive dislike for that aspect of issue description
i have no say on exp being a separate builtin

This is interesting to me.
I think that in practice, you don't actually need power as it is right now.
×. and √ handle the most common cases.
Also, them being inverses would have some nice consequences for invert and under I think.

While this initially appealed to me, I now think that it is useful  to have power and logarithm not express two sides of the same relationship.
In particular, invert power gives an nth root, which is not as nice with exponent.
I am closing this for now.

invert power gives an nth root

⍘ⁿ 3 27

Error:
27
3

Error: No inverse found
   ╭─[:1:1]
 1 │ ⍘ⁿ 3 27
   │ ─
───╯


add initial HTTP proxy support

I have added some basic support for using the WebSocket client with an HTTP proxy. Developers will pass a TunnelingAgent object to WebSocketClient.prototype.connect. TunnelingAgent is generated with node-tunnel, which does all the proxy work.
Example
var WebSocketClient = require('websocket').client;
var client = new WebSocketClient();
var tunnel = require('tunnel');

var tunnelingAgent = tunnel.httpOverHttp({
  proxy: {
    host: 'proxy.host.com',
    port: 8080
  }
});

client.connect('ws://echo.websocket.org/',null,null,null,tunnelingAgent);

While this may seem more complicated than passing a "http://proxy.host.com:8080" string argument, I believe this gives developers the flexibility of defining all aspects of TunnelingAgent. (It gets complicated if you are dealing with SSL, defined certificates, and authentication.). Developers designing user applications may choose to parse out a string argument to build the TunnelingAgent object to fit their needs.
Thoughts?

so does this work? how would i use proxies? its not in the docs


Add a hyperlink + correct spelling error

Create a hyperlink in "License" section: word "LICENSE" should link to website: https://github.com/PhazeRoOman/thawani-for-woocommerce/blob/master/LICENSE.md

Spelling error in "Frequently asked questions" section: word "teh" > correction "tech"

mam can i ?

Greetings @PuneetDabral this issue has been assigned to @jeanansabry However, be on the lookout. We will be adding more issues and would love your contribution!

ok sir i am eagerly waiting to work with you....! give me a chance so i can contribute my first contribution and learn more ...


Automated cherry pick of #5336: fix: keystone user password history check should skip system account

Cherry pick of #5336 on release/3.1.
#5336: fix: keystone user password history check should skip system account

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.1@627495d). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             release/3.1   #5338   +/-   ##
=============================================
  Coverage               ?   7.25%
=============================================
  Files                  ?     572
  Lines                  ?   97303
  Branches               ?       0
=============================================
  Hits                   ?    7064
  Misses                 ?   89671
  Partials               ?     568

Flag
Coverage Δ

#aFlag
7.25% <0%> (?)

Impacted Files
Coverage Δ

pkg/keystone/models/users.go
1.72% <0%> (ø)

pkg/keystone/models/passwords.go
11.68% <0%> (ø)

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 627495d...03cd5dc. Read the comment docs.


Unable to match results on code generation

I am unable to get a decent (close to test set performance reported in paper) performance on the validation set for code generation using your fine-tuned checkpoint. I am getting a bleu score of 29.49 and EM of 12.65. Here is my code. Am I doing something wrong here?
from datasets import load_dataset

class Example(object):
    def __init__(self, idx, source, target ):
        self.idx = idx
        self.source = source
        self.target = target

def read_examples(split):
    dataset = load_dataset('code_x_glue_tc_text_to_code')[split]
    examples = []
    for eg in dataset:
        examples.append(Example(idx = eg['id'], source=eg['nl'], target=eg['code']))
    return examples

examples = read_examples('validation')

from transformers import RobertaTokenizer, T5ForConditionalGeneration
import torch
from tqdm import tqdm
import os

os.environ["CUDA_VISIBLE_DEVICES"]="0"

tokenizer = RobertaTokenizer.from_pretrained('Salesforce/codet5-base')
model = T5ForConditionalGeneration.from_pretrained('Salesforce/codet5-base')

model.load_state_dict(torch.load('finetuned_models_concode_codet5_base.bin'))

device = torch.device("cuda" if torch.cuda.is_available() else "cpu")
print (device)
model.to(device)

preds = []
for eg in tqdm(examples):
    input_ids = tokenizer(eg.source, return_tensors="pt").input_ids.to(device)
#     print (len(input_ids[0]))
    generated_ids = model.generate(input_ids, max_length=200, num_beams=5)
    preds.append(tokenizer.decode(generated_ids[0], skip_special_tokens=True))

import sys
import numpy as np
from bleu import _bleu

accs = []
with open("test.output",'w') as f, open("test.gold",'w') as f1:
    for ref,gold in zip(preds,examples[:len(preds)]):
        f.write(ref+'\n')
        f1.write(gold.target+'\n')
        accs.append(ref.strip().split()==gold.target.split())

print (np.mean(accs), _bleu('test.gold', 'test.output'))

Hi, before we get time to examine your code to figure out where the problem comes from, we suggest you to first employ the the run_gen.py script to reproduce the results. You can pass the do_test argument at here and pass the finetuned checkpoint to load at here.

thanks, that worked. I got test BLEU 39.6 which is close to but not exactly the same reported in paper. Probably the paper used a different checkpoint?


Running on mac: "make test" getting an error

bash-3.2$ make test
SKIP_SQUASH=0 VERSIONS="1.4 1.5 1.6" OS=centos7 VERSION= BASE_IMAGE_NAME=go OPENSHIFT_NAMESPACES="1.4 1.5 1.6" TAG_ON_SUCCESS= TEST_MODE=true hack/build.sh
hack/build.sh: line 85: conditional binary operator expected

Closing in favor of https://github.com/sclorg/golang-container


Support multiple backend refs

Currently, this plugin only modifies the weight of the first matching backend refs. I changed the setHTTPRouteWeight and setTCPRouteWeight functions to modify the weights of all matching backend refs.

Hi @tommy351 ! Thank you for contribution, please

Resolve conflicts
Remove empty lines
When you will finish, please ping me

@Philipp-Plotnikov Fixed in 9eded55


Provide ability to manage port-forwards

Is your feature request related to a problem? Please describe.
I am using some other Javascript/Typescript tooling for e2e testing (Playwright) on UIs in my k8s cluster. Some UIs are not externally exposed and I would like to start and stop port-forwards as needed during my tests.
Describe the solution you'd like
Provide a new K8s.PortForward type function that could be used to portforward the given pod/service and port. Also ideally provide an easy way to start and stop the portforward.
Describe alternatives you've considered
Manage the port-forward with kubectl or other tooling outside of KFC and potentially outside of TS.

@jeff-mccoy asked us to put this on hold for now, he mentioned that this is something Zarf can do and that he may push back. Waiting for further instruction around prioritization.


Code breaks when using curly braces for single import

Describe the bug
this broke my entire app:
import { Modal } from '../components/Modal.svelte';
removing the braces fixed it:
import Modal from '../components/Modal.svelte';
but for some reason it works with npm run preview but not npm run build
the component is literally just <slot></slot> and not currently used in the page
The error I get when I have the curly braces is: __layout.svelte:7 Uncaught (in promise) SyntaxError: The requested module '/src/components/Modal.svelte' does not provide an export named 'Modal'
Reproduction
Basic svelte kit boilerplate plus an empty component
Logs
No response
System Info
C:\Users\Justin>npx envinfo --system --binaries --browsers --npmPackages "{svelte,@sveltejs/*,vite}"

  System:
    OS: Windows 10 10.0.19042
    CPU: (12) x64 AMD Ryzen 5 1600 Six-Core Processor
    Memory: 7.09 GB / 15.95 GB
  Binaries:
    Node: 14.17.5 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 7.5.4 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 92.0.4515.159
    Edge: Spartan (44.19041.1023.0), Chromium (92.0.902.78)

C:\Users\Justin>

Severity
annoyance
Additional Information
When using the component (again, even if it's entirely empty) the entire app refuses to load and I get the error that it's not a valid SSR component. I have other components that work perfectly fine. Took a few hours to debug this lol. I tried clearing cache and hard refreshing, incognito, stopping and starting npm run dev, the works...
Thanks for looking into this

This works as designed, and the error is right. The Svelte component itself is the default export, named exports are exports from the module script.

This works as designed, and the error is right. The Svelte component itself is the default export, named exports are exports from the module script.

But the errors given are completely unrelated. I get anything from it's not a valid SSR component (and led down the path that I need to make it work with SSR) and that it can't find a modal component. I went on the svelte discord and spent hours debugging this with several experienced svelte devs. If the error message made any sense at all then maybe this would be a thirty second fix.
I understand why it broke after the fact now, but the error message should be changed imo.

You only posted the error message __layout.svelte:7 Uncaught (in promise) SyntaxError: The requested module '/src/components/Modal.svelte' does not provide an export named 'Modal' which is very descriptive in my opinion.

My mistake for not mentioning, the first error I came across was:
Error: <Modal> is not a valid SSR component. You may need to review your build config to ensure that dependencies are compiled, rather than imported as pre-compiled modules
    at Proxy.validate_component (C:\Users\Justin\Documents\GitHub\desktopclock\node_modules\svelte\internal\index.js:1654:15)
    at eval (C:\Users\Justin\Documents\GitHub\desktopclock\src\routes\__layout.svelte:49:29)
    at Object.$$render (C:\Users\Justin\Documents\GitHub\desktopclock\node_modules\svelte\internal\index.js:1677:22)
    at root.svelte:36:44
    at $$render (C:\Users\Justin\Documents\GitHub\desktopclock\node_modules\svelte\internal\index.js:1677:22)
    at Object.render (C:\Users\Justin\Documents\GitHub\desktopclock\node_modules\svelte\internal\index.js:1685:26)
    at render_response (file:///C:/Users/Justin/Documents/GitHub/desktopclock/node_modules/@sveltejs/kit/dist/ssr.js:472:28)
    at async respond_with_error (file:///C:/Users/Justin/Documents/GitHub/desktopclock/node_modules/@sveltejs/kit/dist/ssr.js:1108:10)
    at async respond$1 (file:///C:/Users/Justin/Documents/GitHub/desktopclock/node_modules/@sveltejs/kit/dist/ssr.js:1336:10)
    at async render_page (file:///C:/Users/Justin/Documents/GitHub/desktopclock/node_modules/@sveltejs/kit/dist/ssr.js:1386:19)
    ```

Which is what I was stuck on for several hours.

@justingolden21 as mentioned in discord, that's because the library did not properly export the svelte component. If any, you should open an issue in the library's repo instead of here.


Implement testing for the frontend.

The front currently does not have any tests, and we urgently need to add some. Jest and React Testing Library should be used to test all our pages and components to ensure their functionality and good code coverage.

Testing Branch was merged which implemented testing libraries and the beginning of tests for most pages and components.


Stop importing cockpit's base1/patternfly.css

Thank you :heart:

This is deprecated API and will be dropped at some point, in favor of projects shipping their own CSS.

Do you already know when this "some point" will be?

Thank you heart

This is deprecated API and will be dropped at some point, in favor of projects shipping their own CSS.

Do you already know when this "some point" will be?

We discussed about fedora-33 and rhel-8-4 but there are not yet any changes in the spec files in cockpit. We are trying to keep track of all projects using this API, and I am trying to do the same change to all of them. So the above seem reasonable to me assuming that I am taking over the responsibility to port to the new PF consumption pattern.

Thanks for making these changes! I poked around and most things look good, but there are a couple of issues with font styling and spacing. I'm happy to work on the updates needed for those issues that I found. Let me know if you have suggestions on how to move forward with changes (i.e. push additional commits on top of this PR, or some other approach?).

Let me know if you have suggestions on how to move forward with changes

Doesn't matter where we fix those, but I wouldn't land this PR before we do. I think I'd work in a new PR based on @KKoukiou's commits.

Thanks for making these changes! I poked around and most things look good, but there are a couple of issues with font styling and spacing.

Can you point the exact problems? This solution should work for all cockpit projects without workarounds, so if something is broken here, it will probably be problematic in other projects  as well and I would like a unified solution.

I'm happy to work on the updates needed for those issues that I found. Let me know if you have suggestions on how to move forward with changes (i.e. push additional commits on top of this PR, or some other approach?).

You can push seperate commit in this PR, or just send the screenshots where the problems appear and I can see what I can do.

@jgiardino I checked your comments regarding users table header fonts, and this is not a regression caused by this PR - it's the same on master.
But since we saw that issue I sent a PR which converts that table to PF4 so the fonts will be picked up correctly now.
See the relevant PR in https://github.com/osbuild/cockpit-composer/pull/1013

There's a trivial conflict. Force-pushed to fix that, as Katerina is on holidays now.
@jgiardino : As #1013 landed now, this seems ready?

Thanks, Martin!
I took another look and the only big difference I see now is with the font size in the empty state. I included a before/after of one of the empty states below. I would like to wait until we have updates to the Empty State available before merging this. Is there any concern with waiting to merge this until a later cycle?

@jgiardino : Not a big concern, it's just a relatively large PR, so it tends to bitrot quickly. But of course it shouldn't be merged with regressions. Thanks for thorough testing!

Thanks, Martin!
I took another look and the only big difference I see now is with the font size in the empty state. I included a before/after of one of the empty states below. I would like to wait until we have updates to the Empty State available before merging this. Is there any concern with waiting to merge this until a later cycle?

@jgiardino this in unfortunately not reproducible on my side. I see the 'before' screenshot even with my PR. Can you please recheck? And if you still see it show me the screenshot from your dev tools on the title element that has different font weight and size for you.

@KKoukiou, here is a screenshot:

@jgiardino I saw the issue you noticed, I could either override the font-size for that h1 element, but instead I thought it would be nicer to fix the EmptyState component to be actually PF4 and user smaller font size for the header.
Let me know if you like it now.

👍 I think your suggestion makes sense. I also was thinking that  updating the EmptyState should happen before/with these changes.
Looking across different empty states, the only remaining issues I see are the following:

The spacing for the actions seem off. The styles for spacing  seem dependent on the presence of the EmptyStatePrimary component, which is being pulled into the Blueprints page, but not the examples shown below.

What are your thoughts on this? Should this update be included here or a separate PR?
As a side note, the version we're pulling in is different than the latest examples on pf.org, which no longer include the use of the EmptyStatePrimary component, but does require the Button component to be used to get the expected spacing. So if we did stick with the current version of pf-react, and made this change, we'd need to pull it out once we update to the latest version.
🤷

There's some inconsistency with which variant is being used for the action that displays (i.e. as a link or secondary instead of primary). I think this can definitely be handled in a separate PR, but wanted to mention it here since it is somewhat related to the previous item.

+1 I think your suggestion makes sense. I also was thinking that updating the EmptyState should happen before/with these changes.
Looking across different empty states, the only remaining issues I see are the following:
* The spacing for the actions seem off. The styles for spacing  seem dependent on the presence of the EmptyStatePrimary component, which is being pulled into the Blueprints page, but not the examples shown below.

  * What are your thoughts on this? Should this update be included here or a separate PR?
  * As a side note, the version we're pulling in is different than the latest examples on pf.org, which no longer include the use of the EmptyStatePrimary component, but does require the Button component to be used to get the expected spacing. So if we did stick with the current version of pf-react, and made this change, we'd need to pull it out once we update to the latest version.
  * shrug'

For that you will need to start using PF4 buttons, now you use PF3 only. And of course the spacing rule does not apply with these.
Let's keep it like this, add a CSS rule for the spacing and change your PF3 buttons to PF4 in another PR and then we can delete the WA and remove also the EmptyStatePrimary wrapper component.

* There's some inconsistency with which variant is being used for the action that displays (i.e. as a link or secondary instead of primary). I think this can definitely be handled in a separate PR, but wanted to mention it here since it is somewhat related to the previous item.

@jgiardino here are the screenshots with the updated code:

This works for me. Thanks for making these updates!
@jkozol are you good with this?


Unexpected provenance in boringtun main crate

On boringtun f672bb6c1e1e371240a8d151f15854687eb740bb and charon 827ee91c945717ca19ae9c3d1cdfa591d0d5e0d9, repo can be found: https://github.com/RaitoBezarius/boringtun/tree/aeneas-fv with a flake.nix for repro.
I tried to build the boringtun crate (not the CLI) with charon --cargo-no-rust-version (that is, using the same rustc as the one charon was built with) and got this error:
error[HaxFront]: Expected provenance to be GlobalAlloc::Static, got Memory(
                     ConstAllocation { .. },
                 ) instead
  |
  = note: ⚠ This is a bug in Hax's frontend.
          Please report this error to https://github.com/hacspec/hax/issues with some context (e.g. the current crate)!

error: could not compile `boringtun` (lib) due to previous error

Minimal reproducer:
pub(crate) const LABEL_MAC1: &[u8; 8] = b"mac1----";

fn main() {}

Thanks for the bug report!
That's a bug in the MIR extraction of constants, I am going to look into it. Should be easy to fix though, this code extracts without error when exporting THIR.

I think this was fixed by https://github.com/hacspec/hax/pull/579. Please re-open otherwise.


azure devops nuget pack dependency version

Hi,
I'm not entirely convinced this is the right place to ask my question.
    - task: NuGetCommand@2       displayName: 'NuGet pack'       inputs:         command: pack         packagesToPack: '**\Database\**\*.nuspec;**\*.csproj;!**\*Host.csproj;!**\Tests\**;'         packDestination: '$(build.ArtifactStagingDirectory)\nuget'         includeReferencedProjects: true            versioningScheme: byEnvVar         versionEnvVar: 'GitVersion.NuGetVersion'
Looking at the logs Foo.nuspec creates a nupkg with a version like  1.1.0-feature0098.nupkg
Further in the logs another project Bar gets created with a version 1.1.0-feature0098.nupkg however this project has a dependency to Foo.
From the logs it appears to be adding the dependency with a version 1.1.0-feature98.nupkg   "Note the dropped leading 0"
This causes problems as when consuming Foo it cannot find the dependency.
Any ideas what is causing this?

Hey,
We would recommend you use nuget authenticate
See this link for more info
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/package/nuget-authenticate?view=azure-devops

Hey,
We would recommend you use nuget authenticate
See this link for more info
https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/package/nuget-authenticate?view=azure-devops

Hi,
How would that help?


There is a 'Leased' icon for the snapshot when creating the snapshot for one leased blob under snapshot view

Storage Explorer Version: 1.21.0-dev
Build Number: 20210803.3
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Big Sur 11.4
Architecture: ia32\x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create a blob container -> Upload a blob.
Right click the blob -> Click 'Manage History -> Manage Snapshots'.
Acquire lease for the blob -> Create snapshot for the leased blob.
Check there is no leased icon for the snapshot.

Expected Experience
There is no leased icon for the snapshot.
Actual Experience
There is a leased icon for the snapshot.

Additional Info
The leased icon disappears after manfully refreshing.

If the fix is fast go for it otherwise won't fix for now.

@craxal Verified this issue on the build 20210810.8.

Fixed on the normal blobs.
Reproduces on the ADLS Gen2 blobs.


Ensure blas is statically linked

Description
This fixes a bug where we set BLA_STATIC after we find the package, leading OpenBlas to be dynamically linked instead of statically linked.
Bug was introduced in https://github.com/opensearch-project/k-NN/pull/1636
Issues Resolved
Related https://github.com/opensearch-project/opensearch-build/issues/4379#issuecomment-2083623882. This may resolve the issue with binutils because we are statically linking.
Check List

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

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

These failures for linux are related to using gcc 12 (https://github.com/opensearch-project/opensearch-build/issues/4379#issuecomment-2083623882). We are moving to gcc 11, so we need to ignore these failures for now.

@jmazanec15 Have you tested it on Windows?

@naveentatikonda the CI passed for windows


should we print another deprecation on a rebuild?

cc @rwjblue

No, shouldnonky deprecate once for the duration of the plugin.


Bug in 4-c DFT treatment of the magnetic spin of the small component

There is the inconsistency between the spin magnetization vector (\beta\Sigma) with the potential (\Sigma).

Fixed in PR #1256


get the proxy video file from dataplane bucket instead of the source bucket for the input media

Issue #, if available:
Closes #270
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

skipping code review since this is a trivial change


[labs/motion] Add Spring and Spring2D controllers

Adds SpringController and SpringController2D that you can use to simulate springs that drive host updates, like so:
@customElement('goo-element')
export class GooElement extends LitElement {
  _mouse = new MouseController(this);
  _spring1 = new SpringController2D(this, fast);

  render() {
    // Set the spring to go to the mouse
    this._spring1.toPosition = this._mouse.position;
    // Position a div based on the current position of the spring.
    return html`
      <div
        class="b1"
        style=${positionStyle(this._spring3.currentPosition)}
      ></div>
    `;
  }
}

const fast = {
  stiffness: 1200,
  damping: 400,
};

const positionStyle = ({x, y}: Position2D) =>
  styleMap({
    transform: `translate3d(${x}px,${y}px,0) translate3d(-50%,-50%,0)`,
  });

@AndrewJakubowicz thanks for the review from so long ago!
I addressed you comments and then rebased onto the 3.0 branch.
I have one important change I want to make to the 2D controller before I think this is mergable.

Very cool controller and demo! Code looks great. Documentation is thorough enough that I could confidently use the controller.


Add server-side conveyor belt movement to V3 client players

This is mainly for conveyor belt functionality, and is supposed to work for both players and NPCs.

If a player or entity is standing on a front item with a move: true use, they will be pushed with power that is proportional to the power level of the item.

Problems

[ ] I would like a general code review.
[x] Maybe a player shouldn't be pushed if they have god mode.
[x] Players get pushed into the conveyor belt for some reason.

There is a bug. Both directions increase the velocity in the same direction.

I will be closing this pull request because I don't think a good, purely server-side implementation is possible for this.

Player movement is very choppy. The server can only update the player's position 8 times a second, and for smooth movement it would have to do it at the framerate the game runs at, which isn't practical for multiple reasons.
There is no real collision testing being done on the server, which, along with other factors, make it difficult (if at all possible) to determine whether the player is actually standing on a conveyor belt, and not just slightly hovering above it.


All resources in Schema::Container should be added previously

They're just mapped into the container during the construct phase.
This is basically just a shortcut for hot adding the resources using ModifySettingRequest.HostedSettings after construct.

This has been completed.


Switch to Twemoji

Overview
Twemoji is

Twitter’s open source emoji has you covered for all your project's emoji needs. With support for the latest Unicode emoji specification, featuring 3,071 emojis, and all for free.
https://twemoji.twitter.com/

prawn-emoji will use the Twemoji's emoji since v3.0.

v3.0 (Twemoji 12.04)
v2.1 or older (EmojiOne 2.0)

See also

https://github.com/hidakatsuya/prawn-emoji/blob/master/test/pdf/emoji_rendering/expect.pdf
https://github.com/hidakatsuya/prawn-emoji/blob/master/test/pdf/prawn_integration/expect.pdf

Status
Fixed 5ae4c5bc4fde929ce199b0b33db6991fd377960b but not released. The release works in progress.
Consequences

More emojis supported
Emoji expression changes

Related Issue/PR
https://github.com/hidakatsuya/prawn-emoji/issues/20 https://github.com/hidakatsuya/prawn-emoji/pull/21

Released in v3.0!!!
https://github.com/hidakatsuya/prawn-emoji/blob/master/CHANGELOG.md#300


docs: Descriptive Javadoc, Updated README, Fixed workflow to use codecov

Descriiptive Javadoc.
Fixed github action to use codecov for coverage reporting.
Enhanced readme documentation.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@f3d99ea). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage        ?   91.66%
  Complexity      ?      363
=======================================
  Files           ?       43
  Lines           ?      996
  Branches        ?       54
=======================================
  Hits            ?      913
  Misses          ?       74
  Partials        ?        9

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 f3d99ea...ec8d3d9. Read the comment docs.


use the correct initializer for the final layer of class_net

Hi @VicariNishimura  This might fix that test case that was failing in https://github.com/google/automl/pull/376 . Please try it out.
It looks like the keras implementation was using the wrong initializer for the final classifier layer. Also, the keras implementation has slightly fewer operations than the legacy version. That is a good thing if they both produce the same results.

Wow Thanks, I was sure I made the correct inits and was absolutely not looking there !

I think the test must be written as self.assertContainsSubset(legacy_names, keras_names) no ? As legacy_names is our expected results. The initializer is now correct but there is still the same naming issues.

Since the keras version has ~120 fewer operations, the keras version is a subset of the non-keras version. If they are producing the same output, it should be good. I checked and it looks like all the weight names are the same which is the goal.


Adding and removing class in same digest cycle causes class to still be added

I have created a simplified plunker to demonstrate this issue:
http://plnkr.co/edit/sVomGmPvI2wMgaoh0MsA?p=preview
The second remove class call appears to cancel the first animation, however when the canceled animation is processed post digest, the class is still added.
The actual issue in my application arises in slightly more complicated circumstances.  A property is changing multiple times during the digest loop, causing the watch handler for ng-show to process multiple times, first adding the "ng-hide" class, and then removing it.
This appears to be a regression in the 1.4.x animate code, as changing the angular and angular-animate versions to 1.3.4 in the plunker will no longer produce this issue.

It does look fixed in master: http://plnkr.co/edit/TJZC4p0jyZZgKrHhe3hr?p=preview


bytes exhausted BUG

when i copy big buildings the server goes crashing
Fatal error: Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 134217736 bytes) in /home/minecraft/plugins/BuilderTools-main/src/czechpmdevs/buildertools/blockstorage/BlockArray.php on line 100
To Reproduce
Steps to reproduce the behavior:

Copy a Big Bulding
Paste this
Server crash
See error

System information (please complete the following information):

OS: Win10, IOS
PHP Version: 8
BuilderTools Version: [1.3.1]

Pretty sure this is just a ram issue. Try to allocate more RAM in pocketmine.yml settings (on every single RAM settings, dont miss one). Yeah BuilderTools is using a lot of RAM for big schematics.
But this stay a bug and maybe BuilderTools should not save everything in RAM but use other ways if the copy clipboard is too large.

when i copy big buildings the server goes crashing
Fatal error: Allowed memory size of<PHONE_NUMBER> bytes exhausted (tried to allocate 134217736 bytes) in /home/minecraft/plugins/BuilderTools-main/src/czechpmdevs/buildertools/blockstorage/BlockArray.php on line 100
To Reproduce
Steps to reproduce the behavior:

Copy a Big Bulding
Paste this
Server crash
See error

System information (please complete the following information):

OS: Win10, IOS
PHP Version: 8
BuilderTools Version: [1.3.1]

I got same crash message. But in my case. I had big schematic to load. and it crash like you had.


apple home remove device (CON-354)

when apple home remove device，device will call  esp_matter_command_callback_remove_fabric ，if i want application know this event，what should I do.

esp-matter exposes that as an event kFabricRemoved https://github.com/espressif/esp-matter/blob/d886f9c76513417abab80ccbb25d3a730a2bbbd9/components/esp_matter/esp_matter.h#L128-L129
Find the usage in light example, here. (NOTE: code inside the case starts the commissionable advertising on dns-sd on removal of last fabric).
If you are curious about the implementation, please find below more details.
You have to implement the chip::FabricTable::Delegate class and register it with fabrics table.
https://github.com/espressif/esp-matter/blob/d886f9c76513417abab80ccbb25d3a730a2bbbd9/components/esp_matter/esp_matter_core.cpp#L121-L147
https://github.com/espressif/esp-matter/blob/d886f9c76513417abab80ccbb25d3a730a2bbbd9/components/esp_matter/esp_matter_core.cpp#L797

@shun-dgut any update on this?

I will close this as the fabric removed event is already present and received. Please re-open if you still see an issue.


Updated minedojo example

Hi!
It's my first time trying to use sheeprl, and I'm interested in using it with MineDojo. After installing all the dependencies, I still get errors with both Dreamerv2 and Dreamerv3:
For instance, when running:
xvfb-run python sheeprl.py exp=dreamer_v3 fabric.devices=1 env=minedojo env.id=open-ended algo.cnn_keys.encoder=[rgb] algo.actor.cls=sheeprl.algos.dreamer_v3.agent.MinedojoActor
I get:
sheeprl/sheeprl/algos/dreamer_v3/agent.py", line 916, in forward
mask["mask_destroy"][t, b] = mask["mask_destroy"].expand_as(logits)
RuntimeError: expand(torch.FloatTensor{[4, 392]}, size=[392]): the number of sizes provided (1) must be greater or equal to the number of dimensions in the tensor (2)

Could you share a working config example for both dreamer v2 and dreamer v3 pls? :)
Also, is there any plan to support PPO and other algorithms in MineDojo? I think that would be great! Would that imply only implementing a MineDojoActor for PPO?
Thank you!

Hi @roger-creus, than you for using SheepRL! Do you have any feedback on the library?
BTW, i'll look into the issue tomorrow asap.
Thank you for your patience.

Btw I think the KeyError problem only happens with env.num_envs > 1. With 1 env it seems to work although it might be very slow because I don't see a single value logged.

I was looking into that yesterdya but i had a problem installing MineDojo: i had to clone the repo and update a malmo bundle.gradle file following https://github.com/microsoft/malmo/issues/942.

Btw I think the KeyError problem only happens with env.num_envs > 1

Exactly, I've noted the same. I''l try in the next few days with multiple envs

Yup! Same experience here with build.gradle. I solved it following this comment. Looking forward to this! If you manage to solve it, I might work on enabling PPO+MineDojo compatibility

Hi @roger-creus,

The task_specs.pop("prompt") KeyError: 'prompt' error is due to a Minedojo problem, they remove the prompt key the first time the environment is created, so the second environment, does not find that key in their configuration dictionary.
Instead the KeyError: KeyError(124) is a sheeprl problem

I created a branch (https://github.com/Eclectic-Sheep/sheeprl/tree/fix/minedojo) in which I fixed these two problems. There is also a configuration file as an example (dreamer_v3_minedojo.yaml in the sheeprl/configs/exp) that you can take a cue from.
Let me know if it works, thanks.
PS. Minedojo is a very heavy environment, resetting can take minutes. It may be that your experiment is slower than others run with other environments for this reason.


Added 8 bit weights compression by default for decoders larger than 1B

Tested on Dolly-3B and Opt-125m

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


patsubst doesn't work

$ mmk/id
%MMK-I-IDENT, this is the MMK Make Utility V5.1
-MMK-I-COPYRIGHT, Copyright (c) 2008, Matthew Madison.
    Copyright (c) 2014, Endless Software Solutions.
  See LICENSE.TXT in distribution kit for license information.
$ type descrip.mms
ARCH=$(patsubst %1.0,%,$(VERSION))
TMPSRC=$(subst V,RELEASE,$(ARCH))
.if $(ARCH) .eq $(TMPSRC)
TMPSRC=$(subst T,TEST,$(ARCH))
.endif
all :
        @ write sys$output "VERSION: $(VERSION), ARCH: $(ARCH), TMPSRC: $(TMPSRC
)"
$ mmk/ext /macr=VERSION=V1.0
VERSION: V1.0, ARCH: , TMPSRC:
$

Expected output:
$ mms/id
%MMS-I-IDENT, MMS V4.0 � Copyright 2015 VMS Software, Inc and Hewlett-Packard De
velopment Company, L.P.
$ mms/ext /macr=VERSION=V1.0

VERSION: V1.0, ARCH: V, TMPSRC: RELEASE
$

and with GNU Make
# make -v
GNU Make 4.1
Built for i686-pc-linux-gnu
...
# cat Makefile
TYPE=$(patsubst %1.0,%,$(VERSION))
TMPSRC=$(subst V,RELEASE,$(TYPE))
ifeq ($(TYPE),$(TMPSRC))
TMPSRC=$(subst T,TEST,$(TYPE))
endif
all :
        @ echo "VERSION: $(VERSION), TYPE: $(TYPE), TMPSRC: $(TMPSRC)"
# make VERSION=V1.0
VERSION: V1.0, TYPE: V, TMPSRC: RELEASE
#

Probably related:
$ type descrip.mms
all :
        @ write sys$output "$(patsubst %.c,%.o,a.c b.c foobar.c c.x.c)"
$ mmk/ext
a.o b.o f o a . c. .
$

Expected output:
# cat Makefile
all :
        @ echo "$(patsubst %.c,%.o,a.c b.c foobar.c c.x.c)"
# make
a.o b.o foobar.o c.x.o
#

Probably unrelated:
$ type descrip.mms
all :
        @ write sys$output "$(patsubst %.c,%.o,a.c b.c foobar.c c.x.c,)"
$ mmk/ext
%MMK-W-TOOMANYARGS, too many function parameters for 'PATSUBST'

$

Expected output:
# cat Makefile
all :
        @ echo "$(patsubst %.c,%.o,a.c b.c foobar.c c.x.c,)"
# make
a.o b.o foobar.o c.x.c,
#

I learned the other day, that MMS' "patsubst" is not compatible with GNU Make's one. And because MMK claims to be compatible with MMS, I should remove all the "expected output" from GNU Make.
Probably related:
$ type descrip.mms
all :
        @ write sys$output "$(patsubst %.c,%.o,a.c b.c foobar.c c.x.c)"
$ mmk/ext
a.o b.o f o a . c. .
$

Expected output:
$ mms/ext

a.o b.o foobar.c c.x.c
$

The documentation for MMS talks about two wildcard characters: '%' and '*'. GNU Make's patsubst accepts only one: '%'. And the meaning of '%' is different in MMS and GNU Make. Unfortunately the meaning of the wildcard characters isn't made clear in the MMS documentation.
This gives more test cases:
$ type descrip.mms
all :
        @ write sys$output "$(patsubst *.c,%.o,a.c b.c foobar.c c.x.c)"
$ mmk/ext
a.o b.o f.o c. .c
$

Expected output:
$ mms/ext

a.o b.o foobar.o c.x.o
$

And
$ ty descrip.mms
all :
        @ write sys$output "$(patsubst *.c,*.o,a.c b.c foobar.c c.x.c)"
$
$
$ mmk/ext
a.o b.o foobar.o c. .c
$

Which shows that the two wildcard characters of MMS are supported but work differently when used in the replacement.
Probably unrelated:
$ type descrip.mms
all :
        @ write sys$output "$(patsubst %.c,%.o,a.c b.c foobar.c c.x.c,)"
$ mmk/ext
%MMK-W-TOOMANYARGS, too many function parameters for 'PATSUBST'

$

Expected output:
$ mms/ext

a.o b.o foobar.c c.x.c,
$

There are probably more differences between MMS and MMK in this area:
$ type descrip.mms
all :
        @ write sys$output "$(patsubst *%.c *.z,*.*.o,a.c b.c foobar.c c.x.c x.y.z)"
$
$ mmk/ext
a.c b.c foobar.c c.x.c x.y.z
$
$ mms/ext

.a.o .b.o fooba.r.o c..x.o x.y.*.o
$

And yes, that's the way it is in MMS, whether MM% should do the same as GNU Make is a different question.


python310Packages.pydaikin: 2.7.0 -> 2.7.1

Description of changes
Update to latest upstream release 2.7.1

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

1 package failed to build:

home-assistant-component-tests.daikin

2 packages built:

python310Packages.pydaikin
python39Packages.pydaikin


Keeps running after JDBC signon cancel

When running the code below I'm presented with a popup asking for my username and password.

If I press cancel then an SQLException is raised: java.sql.java.sql.SQLException: java.sql.SQLException: The application server rejected the connection. (Signon was canceled.). I added a try block and allow it to pass on exception, so the print statement is executed. However the python script never quits.
import os
import jpype
import jpype.dbapi2

import urllib.request
from pathlib import Path

jt400_version = "11.0"
jt400_jar_filename = f"jt400-{jt400_version}.jar"
jt400_jar_url = f"https://repo1.maven.org/maven2/net/sf/jt400/jt400/{jt400_version}/{jt400_jar_filename}"

classpath = os.path.join(os.getcwd(), "jt400-11.0.jar")

if Path(classpath).is_file():
    print(f"Already exists: {jt400_jar_filename}")
else:
    urllib.request.urlretrieve(jt400_jar_url, classpath)

jpype.startJVM(classpath=classpath)

try:
    jpype.dbapi2.connect("jdbc:as400://localhost", driver="com.ibm.as400.access.AS400JDBCDriver")
except:
    pass

print("test")

How can I allow the python script to exit when the end of the file is reached?
I've tried adding this to the bottom of the script:
jpype.java.lang.Thread.detach()
jpype.shutdownJVM()

jpype.java.lang.Thread.detach()
jpype.java.lang.System.exit(0)

But still the python script never exits.

I wrote back on this topic this morning, but it appears that it didnt get posted.
The most likely issue us that the driver started a non daemon thread that must be terninated or set to daemon before proceeding to JVM shutdown.   If the driver is open source you should find the thread launch.  If not you will need to call Java to get the list of available running threads.  Hopefully the driver writers did not eat the interrupt message, or you will find this very difficult (though not impossible depending your technical skill.)  If they did eat interrupt, your options are limited to calling eith the Java exit method or the Java halt/terminate method.  These will exit without returning to python so they are generally discouraged.
This is not a JPype bug as JPype is following the defined JVM protocols.  You can find the shutdown sequence as well as the available user hooks in the org.jpype module under natice/java in the JPypeContext object.   In there we attempt to interrupt every non deamon thread which should have allowed shutdown to proceed.  If they ate it then they have to provide a way to safely terminate in their documentation or you will need to find a way to covert it to a daemon.

Thanks a lot for replying, glad it's not a but with jpype! I found a workaround: not showing the popup at all by setting prompt to false: jdbc:as400://$AS400_HOST;naming=sql;errors=full;libraries=$AS400_DBNAME;prompt=false according to this article.


Add "sassImport" alias wrapper task

This adds the ability to use sassImport as the task name, whilst retaining the old name to avoid breaking previous installs. This is required as jscs requires names to be in camelCase by default. The default installation of the Angular Yeoman generator (a very popular setup) scans the Gruntfile.js file. Using the underscored name causes an exception.

Thank you! 🏆

You're welcome :)