Issue with k8s.io/zh/docs/tasks/extend-kubernetes/configure-multiple-schedulers/

This is a Bug Report

Problem:
When I create a custom scheduler using the page's YAML, I get the following error
kubectl logs -f my-scheduler-64b9ffdbf-6clcg -n kube-system
I1021 07:51:49.297528       1 registry.go:150] Registering EvenPodsSpread predicate and priority function
I1021 07:51:49.297595       1 registry.go:150] Registering EvenPodsSpread predicate and priority function
I1021 07:51:50.371878       1 serving.go:313] Generated self-signed cert in-memory
W1021 07:51:52.038924       1 authentication.go:349] Unable to get configmap/extension-apiserver-authentication in kube-system.  Usually fixed by 'kubectl create rolebinding -n kube-system ROLEBINDING_NAME --role=extension-apiserver-authentication-reader --serviceaccount=YOUR_NS:YOUR_SA'
 Error looking up in-cluster authentication configuration: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot get resource "configmaps" in API group "" in the namespace "kube-system"
W1021 07:51:52.038975       1 authentication.go:298] Continuing without authentication configuration. This may treat all requests as anonymous.
W1021 07:51:52.038986       1 authentication.go:299] To require authentication configuration lookup to succeed, set --authentication-tolerate-lookup-failure=false
W1021 07:51:52.040797       1 options.go:324] Neither --kubeconfig nor --master was specified. Using default API client. This might not work.
I1021 07:51:52.058298       1 registry.go:150] Registering EvenPodsSpread predicate and priority function
I1021 07:51:52.058331       1 registry.go:150] Registering EvenPodsSpread predicate and priority function
W1021 07:51:52.062732       1 authorization.go:47] Authorization is disabled
W1021 07:51:52.062773       1 authentication.go:40] Authentication is disabled
I1021 07:51:52.062812       1 deprecated_insecure_serving.go:51] Serving healthz insecurely on [::]:10251
I1021 07:51:52.065505       1 configmap_cafile_content.go:202] Starting client-ca::kube-system::extension-apiserver-authentication::client-ca-file
I1021 07:51:52.065543       1 shared_informer.go:223] Waiting for caches to sync for client-ca::kube-system::extension-apiserver-authentication::client-ca-file
I1021 07:51:52.066167       1 secure_serving.go:178] Serving securely on [::]:10259
I1021 07:51:52.066606       1 tlsconfig.go:240] Starting DynamicServingCertificateController
E1021 07:51:52.068652       1 reflector.go:178] k8s.io/apiserver/pkg/server/dynamiccertificates/configmap_cafile_content.go:206: Failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"
E1021 07:51:53.350381       1 reflector.go:178] k8s.io/apiserver/pkg/server/dynamiccertificates/configmap_cafile_content.go:206: Failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"
E1021 07:51:55.293749       1 reflector.go:178] k8s.io/apiserver/pkg/server/dynamiccertificates/configmap_cafile_content.go:206: Failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"
E1021 07:51:59.618667       1 reflector.go:178] k8s.io/apiserver/pkg/server/dynamiccertificates/configmap_cafile_content.go:206: Failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"
E1021 07:52:10.488750       1 reflector.go:178] k8s.io/apiserver/pkg/server/dynamiccertificates/configmap_cafile_content.go:206: Failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"
E1021 07:52:23.561973       1 reflector.go:178] k8s.io/apiserver/pkg/server/dynamiccertificates/configmap_cafile_content.go:206: Failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"

It appears that the ClusterRole for system: kube-scheduler does not have access to configMap resources
Proposed Solution:
Page to Update:
https://kubernetes.io/...

In order to run your scheduler in a Kubernetes cluster, create the deployment specified in the config below in a Kubernetes cluster:
kubectl create -f my-scheduler.yaml
Verify that the scheduler pod is running:
kubectl get pods --namespace=kube-system
NAME                                           READY     STATUS    RESTARTS   AGE
....
my-scheduler-lnf4s-4744f                       1/1       Running   0          2m
...
You should see a "Running" my-scheduler pod, in addition to the default kube-scheduler pod in this list.
Enable leader election
To run multiple-scheduler with leader election enabled, you must do the following:
First, update the following fields in your YAML file:
--leader-elect=true
--lock-object-namespace=
--lock-object-name=
Note: The control plane creates the lock objects for you, but the namespace must already exist. You can use the kube-system namespace.
If RBAC is enabled on your cluster, you must update the system:kube-scheduler cluster role. Add your scheduler name to the resourceNames of the rule applied for endpoints and leases resources, as in the following example:
kubectl edit clusterrole system:kube-scheduler

@moluzhui if the issue is resolved, please close it.

/close

@sbarikdev This issue isn't resolved. Following the instructions on the (English) documentation precisely leads to the following errors repeated constantly in the my-scheduler logs:
requestheader_controller.go:193] Unable to get configmap/extension-apiserver-authentication in kube-system.  Usually fixed by 'kubectl create rolebinding -n kube-system ROLEBINDING_NAME --role=extension-apiserver-authentication-reader --serviceaccount=YOUR_NS:YOUR_SA'
reflector.go:138] runtime/asm_arm64.s:1133: Failed to watch *v1.ConfigMap: failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"
reflector.go:324] runtime/asm_arm64.s:1133: failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"

Running this (not mentioned anywhere in the documentation) resolves it:
kubectl create rolebinding -n kube-system my-scheduler-extension-apiserver-authentication-reader --role=extension-apiserver-authentication-reader --serviceaccount=kube-system:my-scheduler

Yes, your solution is correct. The error info  in my k8s cluster is as follows.
W1109 15:30:32.624055       1 reflector.go:424] pkg/server/dynamiccertificates/configmap_cafile_content.go:206: failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system". E1109 15:30:32.624081       1 reflector.go:140] pkg/server/dynamiccertificates/configmap_cafile_content.go:206: Failed to watch *v1.ConfigMap: failed to list *v1.ConfigMap: configmaps "extension-apiserver-authentication" is forbidden: User "system:serviceaccount:kube-system:my-scheduler" cannot list resource "configmaps" in API group "" in the namespace "kube-system"
Yes, we can solve the problem by creating new clusterrolebinding for your own serviceaccount  related to your customized kube-scheduler.
The clusterrolebinding creating command is the following command in  k8s in higher version.
kubectl create clusterrolebinding \     -n kube-system my-scheduler-extension-apiserver-authentication \     --clusterrole=extension-apiserver-authentication-reader \     --serviceaccount=kube-system:my-scheduler


WIP: New document search

WARNING: This is very busted do not merge :exclamation:
I'm long overdue to get something out for https://github.com/seongjaelee/nvatom/issues/35... I started with great intentions but life got the best of me :/ Also an apology up front, I did everything in javascript, mostly so I could take advantage of ES6 features and because I find no joy writing coffeescript. I realize the rest of nvatom is in coffeescript and having a mix of the two probably isn't best, but I wasn't getting any momentum with coffeescript so I had to try ES6 for my own sanity.
I started out by trying to implement each component separately. I should have known better, shame on me :grin:
Eventually I came back to my senses and started at the existing NotationalVelocityView and worked my way down into the Documents model. It kinda sucks in its current state tbh, but I figured getting something out for folks to see was better than continuing to procrastinate.

[x] Remove dependency on docquery
[x] Use search-index instead of lunr.js
[x] Search function for full-text document searches
[x] Recent function for getting a list of recently updated documents
[ ] Profile and tune memory usage (tweaking search-index where possible)
[ ] Intelligently remove documents from search index that no longer exist in your Notes folder (was thinking about hooking into the search function to filter out dead documents from search results and update the index at the same time)
[ ] Benchmark performance and then improve it
[ ] Fix all of the other bugs I haven't found yet :grin:

So yeah, this is in a pretty rough state, but maybe some folks will enjoy reading it? I'm about to head on holiday for a few weeks so I won't be pushing this forward until mid to late November. Feel free to pick parts of it, all of it, or ditch the entire thing :)

:+1: Thanks for your work on this. Looking forward to seeing progress. :trophy:

Imma close this as I just haven't had time to work on it.

I don't blame you. For what it's worth I've been working on a little way to duplicate nvatom functionality with a mixture of native Atom, atom-fuzzy-grep and one or two other plugins.


How to load quantized model with flash_attn?

Hi, I have saved a quantized model using AutoHQQHFModel.save_quantized(model, save_dir).
Now I want to load the quantized model using model = AutoHQQHFModel.from_quantized(save_dir). What should I do if I want to set params like low_cpu_mem_usage,attn_implementation and torch_dtype to conduct training.

Hi @mxjmtxrm
low_cpu_mem_usage: you can't, memory is managed automatically. If you want to load on a specific gpu device, you can pass device='cuda:1' for example
torch_dtype: just use compute_dtype=torch.bfloat16 for example.
attn_implementation: there's no direct way of doing it for the moment. Here are 2 solutions:

edit the config here: https://github.com/mobiusml/hqq/blob/master/hqq/models/hf/base.py#L17
edit the config file directly (config.json)

Note: the quantized model is always frozen, you need to add trainable LoRA on top of it. If you add LoRA via ,HQQ's lora implementation, you can switch the LoRA type via: PeftUtils.cast_lora_weights(model, dtype=train_dtype)
Let me know if that solves your issue!

This should fix it: https://github.com/mobiusml/hqq/commit/474f09b11c8bb791f10d1aa6b9e3277197b4a0af


Observers not called when the activity is in background

Before you say it: I know this is a possible dupe of #1, but I'm not able to reopen it.
Following situation:
I have a fragment, which listenes to a boolean preference, which can only be changed by another activity. Whenever the first activity gets paused, the observers are not active any more. When then the first activity gets resumed, the observer gets called in every case. Even if the preference did not change.
This might be a possible bug.

Can this be confirmed?

Hello @marcauberer you may check the Release 1.9. Please let me know if the problem still occurs.

Okay. The observers are now called correctly. They won't be called until the first activity resumes, but that is okay, cause it only has to redraw while its on-screen.
Thanks for solving this. Awesome lib!

Sweet man glad you figured it out


Privatesend-multisession-enabled

Enable privatesend multisession by default. Intending to merge this into the 15 branch.

See issue 3339
Not including automatic backup on low keys.

NACK for 0.15 since it is already released and this isn't a bug fix. Regarding the reason why this hasn't been done already (and probably still shouldn't be done) is described here https://github.com/dashpay/dash/pull/2065#issuecomment-386602564

@PastaPastaPasta As far as merging into 15, I understand.
As for the reasoning, I agree with @akijuh's reply
We can have the wallet generate more keys and do an automatic backup. If we want to attract normal users, privatesend must be fast and grandma proof. Having privatesend work slowly, then eventually produce some scary warning about keys will be sure to scare them off.
When it comes to user interface, think like a user, not like a developer. Do we want mass adoption or not? We have the technology, let's use it.


Fix bug in PartialRange#removeWorker

Previously it never did any such thing.
Bug revealed by the upcoming staleness auditing (#261) under the recently added
peer churn test with partial affinity enabled.

nice find. lgtm.


Roll Out Pin Expiration Notification Service

We have a pin expiration notification system in tutil,  with the revamp of the playground it would be a good time to roll this out. We should do a monthly notification where at the first of every month verified emails are sent a list of pins that will expire that month.
Although the code implementation is pretty simple, we still need to ensure that we get the consent of the users who wish to participate in this system. This means that we need to expand the user database table with a field that says they consent to this pin notification system.
We also need to make sure that existing users who have verified their emails also get asked for consent to participate in this system. To do this I'll send out a BCC (very important as to make sure user emails are hidden from one another) email asking their consent to participate in this system explaining what it is, and I'll feed the responses into our database.

Done


Exclude more directories in Cargo.toml

This doesn’t significantly affect the size of tarballs, but make them not include some files with very long paths like
gm/rebaseline_server/testdata/outputs/expected/compare_rendered_pictures_test.CompareRenderedPicturesTest.test_endToEnd/compare_rendered_pictures.json

This will hopefully help on Windows:
http://logs.glob.uno/?c=mozilla%23servo#c403696
https://i.imgur.com/JeBRLlE.png
https://github.com/rust-lang/cargo/issues/2516
http://logs.glob.uno/?c=mozilla%23servo#c403699
The longest path from the crate root is now 65, down from 150.
third_party/freetype/include_overrides/freetype/config/ftoption.h

I’ve checked that cargo build --target arm-linux-androideabi still succeeds when run on the result of extracting the tarball.
r? @mbrubeck

This change is

@bors-servo r+

:pushpin: Commit f7ef1d8 has been approved by mbrubeck

:hourglass: Testing commit f7ef1d8 with merge 25296a4...

:sunny: Test successful - travis


Please invite me to the Recode-Hive GitHub Community Organization

Name
Shubham Pandey
Discord Username (if applicable)
No response
Additional Context
Just curious.

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people

Thank you


fix: healthcheck not waiting for all services to start

fix daytona healthcheck not waiting for all services to start
Description
This PR updates the Daytona health check to ensure all services are ready before the command exits.

[ ] This change requires a documentation update
[ ] I have made corresponding changes to the documentation

Related Issue(s)
closes #1357

@quest-bot loot #1357

On it!

@tarunrajput there's something wrong with the project, I'm having trouble running dtn serve on this PR. It's "hanging". See below.

@mojafa, could you pls check now?

On it now!

Hi @Tpuljak, I am having a tough time with my mental health right now and don't think I'll be able to complete this. therefore, I am closing this PR.

Hi @Tpuljak, I am having a tough time with my mental health right now and don't think I'll be able to complete this. therefore, I am closing this PR.

@tarunrajput I am very sorry to hear that. Everyone from the Daytona team wishes you the best. Mental health should be everyone's top priority.
You are always welcome as a contributor to Daytona ❤


Can't access relationships on Ember Data models

I have a simple scenario like this:
// user.js
DS.Model.extend({
  project: DS.belongsTo('project', {async: true}),
});

// acceptance test
test('visiting /some/url', async (assert) => {
  const project = server.create('project'),
        user = server.create('user', {project_id: project.id});

  await visit('/some/url');
});

/// and the route
Ember.Route.extend({
  model() {
    return this.store.find('user', 1).then(user => {
      // project is always empty. Created projects exist in the mirage db but the
      // user record doesn't have the relationship. user._data.project_id does exist
      console.log(user.get('project'));
    });
  },
});

Is this supposed to work and I'm doing something wrong or are relationships not possible yet? Does the user factory need to know that project is a relationship? I'm on Ember 1.12 and using the default ED serializers and etc.

Depends on how your routes (/mirage/config.js) are configured, does it return related resources? If you want async, it may not include it until/unless your template requests (e.g. user.project).
If you want GET /users/1 to return the user and their project, you'd do something like this
this.get('/users/:id', function(request, db) {
  let user = db.users.find(request.params.id);
  return {
    user: user,
    projects: db.projects.where({ id: user.project_id })
  };
});

depending on your adapter.

Thanks so much Sam. Sorry for opening an issue for what ended up being user error :/ You helped me out though by confirming that it should be working.
For anyone else that may run into this: the main thing I had wrong was using server.create('user', { project_id: project.id }) when it should have been server.create('user', { project: project.id }) for a belongsTo. Ember Data wasn't actually getting the project's id so it naturally wasn't hooking up the relationship. It doesn't appear to matter if the factory includes the property to start with or not, whatever you pass will get added to the mock instance.

Yep that's correct, think of Factory's as more of a "minimal generic set of attrs" rather than a full-blown model definition.
No need to apologize at all, Issues are a great place to get help!


Reduce object construction when object parsing

After CPU profiling an sbt update run on a our multi-project setup, I observed ~20% time spent in map/Vector creation/destruction when parsing objects. This MR reduces the amount of objects created in that tight inner loop (our sbt update command went from 33s to 25s)

Thanks for the contribution!


Yeoman generator for new feature

Is your feature request related to a problem? Please describe.
As a engineer trying to add a new implementation for my problem space, I'd like a generator to automatically scaffold out the files for me
Describe the solution you'd like
Say california state taxes require similar information to HMRC about the user's device. You want to leverage the work in js/common, but you need to format it differently to how HMRC want it.
Essentially you need to clone the HMRC folder https://github.com/intuit/user-data-for-fraud-prevention/tree/master/src/js/hmrc but with your own name and then rewrite the functions.
It would be good to have a yeoman generator script you can call which automatically scaffolds this all out.
https://yeoman.io/learning/index.html
Describe alternatives you've considered
A good MVP would be a README section for this specific problem more clearly detailing out the structure required.
Additional context
Add any other context or screenshots about the feature request here.

@reubenae Raised changes for this here: https://github.com/intuit/user-data-for-fraud-prevention/pull/72


[feature request] Option to switch off update check for individual novel

Option to switch off update check for individual novel.
I think about plan to move 120+ novels
but most of them are complete. so instance of check all of them every time. it is good to switch off check for completed one manually.

this is more flexible then option to switch off update for tab (category) #45
just exertion at start.

That the completed novel bust status is N/A
Some Novels are completed status but it is ongoing.
There are some Ongoing novels with Completed Status.
Able to edit status will solve this ploblem.


[FEATURE]: Get unflagged comments #10

What does this PR do?
It proposes a controller and an endpoint for getting all unflagged comments to a report
What is the link to the issue on Github?
Issue #10

As I have explained in the link issue, there have been some major changes to simplify how the API works and this endpoint is now not required.


Python: Unable to select, enable or run linting

code-server version: 2.1698-vsc1.41.1
OS Version: Ubuntu 18.04.3 LTS
Extension: Python 2020.1.58038

Description
Unable to select a linter, get the error "Unable write to Folder Settings because no resource is provided"
Steps to Reproduce
From Command Pallet, select "Python: enable linting" and then selecting "on" gives me the above error. The same scenario will happen if I "Enable Linting" or "Run Linting"

Works for me. Do you have a folder open?

code-server version : 2.1698-vsc1.41.1
OS version: CentOS 7
Extension: Python 2020.2.63072

I'm running this in singularity, not docker fyi
Hi, I'm Having this same issue. I have a workspace open, and within settings it says linting is on both globally and in the workspace. But when I try to enable listing I get the error message Unable to write to Folder Settings because no resource is provided.

Would you be able to try this out in v3 and see if the problem persists?

That did the trick. Thank you!


typescript error when trying to patch a column with a null value

When doing the following query :
Session.query().findById(s.id).patch({ name: null });

I get the following typescript error :
[0] server/src/sessions.ts(180,62): error TS2322: Type 'unknown' is not assignable to type 'string | RawBuilder | Raw<any> | ReferenceBuilder | AnyQueryBuilder | ValueBuilder | undefined'.
I have to either use raw('null') or force the type like this : null as unknown as string.  Is that intentional ?
(objection 2.1.2)

If that value can be null, the type for that field should also be nullable. Simply update your typings for that field.

If that value can be null, the type for that field should also be nullable. Simply update your typings for that field.

indeed, I completely forgot the type taken by patch was deduced from my Session type. Sorry about that :)


Release Google.Cloud.SecretManager.V1Beta1 version 3.0.0-beta05

Changes in this release:
New features

Add IServiceCollection extension methods for client registration where an IServiceProvider is required. (commit 022fab2)

Created release for Google.Cloud.SecretManager.V1Beta1-3.0.0-beta05


The edges argument to Histogram is supposed to be optional

This is supposed to work without requiring the edges argument:
hv.Histogram([1,2,3])

Right now this generates TypeError, probably because it is trying to check the len of None:
TypeError: len() of unsized object

Once this is fixed, we must make sure to add a unit test for this version of the constructor.

Philipp fixed this in f2c2dcc and then added the unit tests in 13149df.


Enum tab completion isn't sorted

When you do tab completion on a parameter that is an enum, the values are not sorted.
Get-ChildItem -ErrorAction
SilentlyContinue
Stop
Continue
Inquire
I think it makes more sense to have them sorted.

Enums are sorted in a specific way and we shouldn't change that.


Add Ollama or any local LLM models

Hi,
i really loved the idea, can we also have ollama or any other locally hosted llms. As it might get very costly very fast if it keeps using openai API

Good idea. Alright, I'm going to add different models, in such a way the user can choose the one he/she have, in order to used

Now it has been solved


Allow pass ExpressionInterface as alias to QueryPartsInterface::withQuery()

It was removed before minor release here: https://github.com/yiisoft/db/pull/775

Moved to #767


footer widget is not showing in .builder

footer widget is not showing in .builder but it's showing in .count
footer: [
Card(
child: Center(
child: Icon(Icons.add),
),
),
],

.builder is nto support footer, because I can't footer in .builder
But maybe you can try Custom like in demo https://github.com/huhuang03/reorderable_grid_view/blob/master/example/lib/demo/demo_custom.dart

@huhuang03 How to set dragWidgetBuilder in demo_custom.dar


MS.AAD.1.1: Legacy authentication SHALL be blocked

We have a conditional access policy in place which blocks legacy auth as per the docs on Microsoft learn. However the MS.AAD.1.1 test shows that we do not have this

Hi @Wtcheesecakes you can use the following lines to determine which configuration item is not properly tagged.
The policy would need to be enabled, for exchangeActiveSync and Other, and blocks for all users.
https://github.com/maester365/maester/blob/main/powershell/public/cisa/entra/Test-MtCisaBlockLegacyAuth.ps1#L20-L29

Hi, we got the same issue.
I tried this:
$result = Get-MtConditionalAccessPolicy | Where-Object {$.state -eq "enabled" -and $.grantControls.builtInControls -contains "block" -and $.conditions.clientAppTypes -contains "exchangeActiveSync" -and $.conditions.clientAppTypes -contains "other" -and $_.conditions.users.includeUsers -contains "All" }
Returns correct CA policy for my tenant.
$result.count gives no value, but if i try $result.displayName.Count it returns 1.
So i think line 29 is the issue, for some reason it count on $result, but need $result..count

Yes i did it in Windows Powershell, did not try Powershell core unforntunately.

Fra: Michael @.>
Sendt: torsdag, juni 6, 2024 3:23 pm
Til: maester365/maester @.>
Kopi: Petter Fauske @.>; Mention @.>
Emne: Re: [maester365/maester] MS.AAD.1.1: Legacy authentication SHALL be blocked (Issue #212)
Interesting, I had a hunch and was able to recreate this. @PetterFauskehttps://github.com/PetterFauske are you using Windows PowerShell rather than PowerShell Core?
I was able to recreate this in Windows PowerShell as it doesn't appear count is an implicit property on the PSCustomObject. I will stage a PR around this shortly.
—
Reply to this email directly, view it on GitHubhttps://github.com/maester365/maester/issues/212#issuecomment-2152478566, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD6OVDOJ3FJIJA6NBKTZC53ZGBPG5AVCNFSM6AAAAABIYZUGJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJSGQ3TQNJWGY.
You are receiving this because you were mentioned.Message ID: @.***>

This change have to be done on many scripts, or you may add an alert if you are not using Powershell 7.x. With Powershell 7.x it works out of the box.

Definitely, each of the CISA test cmdlets that rely on that count structure will need to be updated. This one is a good first issue for anyone wanting to begin contributing, but if it is still unresolved by the end of this weekend I will likely update the code and resolve this issue.

I have never contributed, but will see if I understand how.  If so, I can get it done before the end of this weekend :)
Mvh
Petter Fauske

Fra: Michael @.>
Sendt: Monday, June 10, 2024 6:42:09 PM
Til: maester365/maester @.>
Kopi: Petter Fauske @.>; Mention @.>
Emne: Re: [maester365/maester] MS.AAD.1.1: Legacy authentication SHALL be blocked (Issue #212)
Definitely, each of the CISA test cmdlets that rely on that count structure will need to be updated. This one is a good first issue for anyone wanting to begin contributing, but if it is still unresolved by the end of this weekend I will likely update the code and resolve this issue.
—
Reply to this email directly, view it on GitHubhttps://github.com/maester365/maester/issues/212#issuecomment-2158844502, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD6OVDM47A4QDC53667BGZ3ZGXJODAVCNFSM6AAAAABIYZUGJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJYHA2DINJQGI.
You are receiving this because you were mentioned.Message ID: @.***>

@PetterFauske thanks for the contribution. I saw your PR was merged so I am closing this issue.


Historical battery discharge data missing from sofar_hyd3k-6k-es after entity rename in v1.5.0

Since v1.5.0, I am missing battery discharge information.
I believe this is a side effect of the typo fix in  https://github.com/StephanJoubert/home_assistant_solarman/pull/332/ which renamed "Battery daily Discharge" to "Battery Daily Discharge" (note the case change of the 'd' in Daily).
This caused HA to create a new entity called  sensor.solar_battery_daily_discharge_2 which now holds current battery discharge data, with historical data remaining in sensor.solar_battery_daily_discharge.

Slightly tangential from your comment (well very tangential, sorry), but did you have to do anything special to make the sofar_hyd3k-6k-es parameters work with v1.5.x....when I've updated from 1.43 to 1.5.x I'm getting every sensor unavailable using the same name parameter file.....
(same happening with a custom_parameters file too)

@Gavin-A92 Nope, apart from changing the battery daily discharge entity I was using in the energy dashboard, all the other entities are still working for me after the upgrade, so sadly I can't help you.

thanks @leeky.....appreciate you taking the time to come back to me.......something on my end so 🙁


Modify testrender to work with triangle meshes

Description
testrender was originally envisioned as a tiny example renderer that should only handle spheres/planes. Over time several groups have expressed the wish for it to handle arbitrary geometry instead. This PR replaces the sphere and quad primitives with triangle meshes.
I am making use of the (embedded) rapidobj library to load models in the .obj format. The original .xml scene format remains, so that you can combine several models together as well as declare and assign the osl shader networks you want to these meshes. For backwards compatibility, spheres and planes are still supported via tessellation (you specify how many subdivisions you want).  In the case of .obj scenes, the shader assignment can be done either by the mesh name or the material name.
I am submitting this to get the review kickstarted. The history of commits includes some fairly large test scenes that should be squashed away to go into the main repo. Files over 50Mb are recommended to use git lfs, but we probably want to figure that out as a different task (possibly handle it via a separate repo).
The handling of derivatives is not totally correct yet, but the behavior is compatible with the previous (incorrect) handling of derivatives we had before.
Tests
Existing tests are passing (I will push updated reference images shortly).
Checklist:

[x] I have read the contribution guidelines.
[x] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite (adding new test cases if necessary).
[x] My code follows the prevailing code style of this project. If I haven't
already run clang-format v17 before submitting, I definitely will look at
the CI test that runs clang-format and fix anything that it highlights as
being nonconforming.

I think I have the CI passing now (except for the cuda/optix portion which @timgrant's patch will address).

Chris, I just rebased my changes on your branch and there were no conflicts and the OptiX path still builds and passes all tests. So that should be GTG.

Two interesting CI failures

Two interesting CI failures

The cuda failures are expected and will be fixed when tim submits the Optix version.
I'm not sure why icx (but not icc) is having trouble with the BVH code.  I'm not sure how to debug this :(

Two interesting CI failures

The cuda failures are expected and will be fixed when tim submits the Optix version.
I'm not sure why icx (but not icc) is having trouble with the BVH code.  I'm not sure how to debug this :(

Could you try adding "__INTEL_LLVM_COMPILER" to the definition of OSL_LIKELY in platform.h? Looks like OSL_LIKELY is not using __builtin_expect for icx, unlike clang and icc.

I added a few print statements around one of the assertions that was hitting:
at line 151:
                    if (!(binID >= 0 && binID < NumBins)) {
                        print("binID {} NumBins {}  i={}\n", binID, NumBins, i);
                        print("current.left {} right {}\n", current.left,
                              current.right);
                        print(
                            "center {}, current.centroid.min[bestAxis={}] = {}, sub = {}, binFactor[bestAxis] {}\n",
                            center, bestAxis, current.centroid.min[bestAxis],
                            (center - current.centroid.min[bestAxis]),
                            binFactor[bestAxis]);
                    }
                    OSL_ASSERT(binID >= 0 && binID < NumBins);

And I get the following output:
binID -2147483648 NumBins 16  i=13222
current.left 13222 right 16384
center -nan, current.centroid.min[bestAxis=2] = -0.06956504, sub = -nan, binFactor[bestAxis] 114.8853

Does that help you in any way?

This is just me adding printf debugging and then pushing to GH to trigger a CI run and seeing what the icx case does.

Thanks Larry. Do you have this debug from the icc run?

I guess the question is how icx gets nans in that spot. Given that the other compilers are fine, I am guessing maybe icx has some overly aggressive (ie: non-IEEE conforming) floating point optimizations enabled? @steenax86 do you know the right flag we could add to see if this is the case?

I believe adding -fp-model=precise should let us test this theory?
I'll dig around the cmake file and see if I can sort out where to add this for testrender.

Line 62 in src/testrender/CMakeLists.txt has fp-model=precise already, and the precise model has fhonor-nans on by default. I don't see fp-model=precise for the non-Intel clang compiler, which maybe why the assert is not firing for clang. There is a nan calculation happening somewhere higher up--binID is very odd, and depends on the nan-valued center via Box3.
Could you add fp-model=precise for a clang compilation in the CMakeList.txt and check for nans.
(Probably not needed, but for purposes of debug only, you could add fno-honor-nans and fno-honor-infinities This should disallow optimizations that assume results/operands are not nans. )

Crossing my fingers that this will fix the icx issue.
Chris, can you also look at the GPU test failure? (It's a build failure, not a run failure.)

Oh, N/M, the GPU one is where you said that Tim's fix immediately after will address it, right?

Yes, I'll open a PR for the OptiX fixes after this PR is merged.

looks like it's still hitting assertions

I am very tempted to merge this as-is, knowing that there will be icx test failures (or maybe you'd prefer to disable the render tests just for icx, for now?) and assume that we'll get to the bottom of it and soon have what will probably be a very small follow-on PR to directly address the issue. I think failing tests for one compiler is a lesser evil that entering Dev Days with this unmerged, since I think some people want to do work on testrender and it seems pointless for them to start with the obsolete code. What do you think, @fpsunflower ?

I agree. I think its worth merging now and we can investigate further as a follow up.

OK, merging!
As a reminder, there are two outstanding issues here:

[ ] The icx test, alone, is failing by hitting assertions in the BVH code and we don't understand why yet. Expect some debugging and a follow-up PR to address the issue. Bug in the new bvh code? Need some kind of clamp to deal with compiler-induced floating point slop? Legit icx compiler bug? Will be exciting to find out!
[ ] The GPU test is failing to build. We understand that Tim Grant's immediate follow-on PR that adds OptiX functionality should take care of that.

@timgrant You're up next!

Diagnosed the issue to be undefined behavior in the [] operator in float center = bbox.center()[bestAxis] in bvh.cpp's build_bvh function. IMath library's[] operator on a  Vec3<float> dereferences only the first component, here float x, and every access beyond that (to y, or z) is out of bounds. Refining the fix and should be able to submit a PR shortly.

Thanks for tracking that down! It's too bad that icx miscompiles this particular idiom. Also, I believe the same operator is used in the ray/triangle test, so perhaps there are more bugs lurking once this first one is fixed?

I'm also wondering what the ideal way of expressing this idiom would be for icx? Is there a solution we could recommend for the Imath folks that would make all compilers happy?

Wow, great detective work!
That idiom is unfortunately used in a number of places in OpenEXR. I feel like I've run into that problem before, back when we were realizing that it had a hard time vectorizing those classes. I thought I remembered fixing it on the openexr side, but maybe not?
What's the best way to fix that will not be UB? We should fix it in OpenEXR for sure.

I mean Imath, not OpenEXR.

Unfortunately, we also need a workaround on the OSL side, since we can't count on an Imath version being newer than whenever we can get it fixed there.

It was an interesting experience! XD.  I examined values of all locals in build_bvh for icc 2022.0.0 and icx 2023.1.0. Turns out the value of center (for binID calculation) never changed and caused incorrect values to be propagated. Bringing bbox.center() on the stack and then indexing into bestAxis, solved the issue but then identified the core problem.
In it's current form, Imath's [] operator is breaking icx's strict aliasing rules.
The fix is a workaround in OSL. Replaced [] operator with a local getter to return vec3.x, vec3.y, and vec3.z, via depending on bestAxis value.
I will do a pass on all testrender files (including ray/triangle tests) to replace all occurrences of Imath::Box3f [] with the new getter.

Replacing that memory access with a switch statement (I'm guessing?) seems like it will make things slower. I'll be curious to see the performance impact of the change.
Is there any flag we could pass to icx for it to match the aliasing rules of the other compilers?

Could you use a trick like
union shim {
    Box3D b;
    float f[6];
};

Box3f thebox;
float component = ((const shim*)&thebox)->f[i];

Or is that also the same kind of UB?

@fpsunflower , Imath breaks C/C++ language aliasing rules, so its not an icx specific issue.
https://github.com/AcademySoftwareFoundation/Imath/blob/main/src/Imath/ImathVec.h#L1593
template <class T>
IMATH_HOSTDEVICE IMATH_CONSTEXPR14 inline T&
Vec3<T>::operator[] (int i) IMATH_NOEXCEPT
{
    return (&x)[i]; // NOSONAR - suppress SonarCloud bug report.
}

When an address of an object is taken, it is illegal/undefined behavior to use that pointer to access memory outside the sizeof the object.  In this case the object is float x, yes it happens to be a data member, but the object type that lifetime is marked for is 4 bytes of a float.  In practice what this means is a temporary Vec3 with x, y, z is considered as only having x be used and y and z might be removed.  Later when &x[i] happens with i > 0, the data being accessed won't be there, undefined behavior.
A simple correctness fix for this would be to take the address of the Vec3 object itself to mark all 12 bytes as potentially used, then cast to float to index.
template <class T>
IMATH_HOSTDEVICE IMATH_CONSTEXPR14 inline T&
Vec3<T>::operator[] (int i) IMATH_NOEXCEPT
{
    return reinterpret_cast<float *>(this)[i]
}

There are many good reasons to not use dynamic indices, but that is a longer conversation

We are literally talking about this and looking at this ticket at this moment in the OpenEXR/Imath TSC meeting.

The reinterpret_cast version looks good to me if it workss, though I believe its technically also UB, so is using a union. Its a shame there isn't a good way to do this in C++.
As far as this particular issue goes, we could switch from using Imath::Vec3 to using a plain old float[3] maybe? We would loose the overloaded operators but it might not be that bad for the handful of use cases that need indexing.

Belatedly catching up on this. I reposted the issue at https://github.com/AcademySoftwareFoundation/Imath/issues/446 for wider discussion, comments there are welcome!


Kafka limits for eval instances are not properly handled in the CLI

We need to detect this error type and return message to user.
Error details
{"id":"24","kind":"Error","href":"/api/kafkas_mgmt/v1/errors/24","code":"KAFKAS-MGMT-24","reason":"only one eval instance is allowed","operation_id":"c5641htrvpbniolbrat0"}

https://github.com/redhat-developer/app-services-cli/pull/1128


Add aliases to class and package Naming rules

There are similar rules in IDEA inspections and Detekt, but these rules have different names.
It would be great to add aliases.
Expected Behavior of the rule

ClassNaming - should have an alias ClassName
PackageNaming - should have an alias PackageDirectoryMismatch

Also RedundantVisibilityModifierRule should have alias RedundantVisibilityModifier


Fe background load

@twinge

This PR is a little hard to read because the white space changes come after the other changes, but I don't see any obvious issues. I'm assuming you did plenty of testing since this is a pretty critical part of the app :)

:+1:


ci: setup e2e ci pipeline

Description
Added e2e tests pipeline in CI
Related Issue

Part of https://github.com/JankariTech/web-app-presentation-viewer/issues/16

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Tests only (no source changes)
[ ] Documentation only (no source changes)
[ ] Maintenance (e.g. dependency updates or tooling)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation updated

Looks Good


How to test trained stage 1 and stage 2 model?

I have trained model in specific lr images.
How can i infer the models .prh generated?
Thanks

Use this command for custom-trained model inference :)
Since DiffBIR needs to support multiple tasks, the inference code may appear somewhat complex. Feel free to ask if you have any questions about the code.

First of all, thanks for all the work you have done.
I did the process, and the result is a very black image. I just tested it for 1000 iter on 20 train images, but the result is all black.
Is there something that i missing?

Can you provide more detailed information?,
such as: (1) your training configuration file; (2) whether your training logs are normal; (3) your inference command.


page tracking doesn't work as intended in v1.15.6 (using CDN), but works in v1.15.4 (using npm)

Describe the issue
the library at version 1.15.6 (using the CDN) stopped working as intended. when I reversed the version back to 1.15.4, the issue was resolved. Previously, we tracked pages using the following code snippet:
pageView: function (from, to) {
  window.analytics.page('', to.name, {
    name: to.name,
    path: to.path,
    referrer: from.fullPath,
    search: to.query || undefined,
    title: document.title,
    url: document.href
    // keywords: ''
  })
}

we're using vue2, vue-router and we're making the above pageView call on every page move in router/index.js
To Reproduce
Steps to reproduce the behavior:

Include the library via CDN with version 1.15.6.
Use the provided code snippet for tracking page views.
Navigate between pages on the website.
Observe that page tracking no longer works as expected.

Additional context
Please let me know if you need any more information or if there's a new recommended method for tracking page views in v1.15.6. thank you.

Hey, @ktseo41
I think you're missing a URL, correct? document.href is not a property -- I think you meant window.location.href?
The new behavior means we support proper overriding, so:
window.analytics.page('', 'foo, { url: undefined })

Will actually result in an object without a URL, rather than us attempting to set a default.
  "page": {
            "path": "my path",
            "referrer": "fullPath",
            "search": "query",
            "title": ""
        },

If you omit the URL property, this should work as expected.

@silesky Thanks for your response. first of all, I apologize for the confusion caused by not specifying what result I was expecting. and as you mentioned, I was incorrectly binding the url. I have now properly bound the url, but our problem has not been resolved.
The issue that occurred in version 1.51.6 is as follows:
Firstly, the page tracking numbers have significantly decreased. below are the results of querying the segment logs being loaded in our service by date:

Even after correctly binding the url and using the 1.51.6 version through the CDN for page tracking, it still seemed like the page views were decreasing. I wasn't able to verify this properly, but it seemed like the page views were decreasing after deploying to production, so I quickly rolled back and couldn't measure it accurately.
Secondly, some values are missing when viewed in the segment debugger. when tracking with version 1.51.4 via npm, the debugger shows the following:

And below is the result when loading segment with version 1.51.6 (through CDN) and mapping the url to location.href. unlike above screenshot, the name are not visible. url is now displayed correctly

The reduced page tracking numbers and the missing name values are the issues we are facing when using version 1.51.6.

@ktseo41 Thank you for more information.

The missing name in the debugger -- I'm having trouble reproing. What does your call look like?

// when I do
window.analytics.page('ArticlesArticle1', {
    path: 'foo',
    referrer: 'bar',
    search: 'baz',
    title: 'title',
    url: 'http://foo.com'
  })

I get:

Reduced event -- This is vexing; between 1.54-1.56, there's  been no changes to the pipeline that would have affected event velocity. The only big change is a refactor of the page-enrichment plugin, but it wouldn't affect the number of page calls you saw. Are you sure there's been no changes on your end?

https://github.com/segmentio/analytics-next/compare/%40segment/analytics-next%401.51.4...%40segment/analytics-next%401.51.6


Encoding issues and 'SET NAME '

Hello,
I've encountered the dreaded encoding errors dumping/importing using this library, and finally fixed them by manually adding 'SET NAME utf8mb4' to the top of the file.
Would it be appropriate to somehow automate this process?
For example retrieving the desired encoding from the ConnectionOptions.charset option?
Or running some sort of 'GET NAME' query and parsing its' results?

Happy to accept a PR


KMS key rotation

Would it work to make the KMS key auto rotate?

@sherryxcheng Thanks for reaching out to us.
Key rotation changes only the CMK's backing key. The CMK is the same logical resource, regardless of whether or how many times its backing key changes. The properties of the CMK do not change. So enabling auto-rotation should work.
For more details please refer this document: https://docs.aws.amazon.com/kms/latest/developerguide/rotate-keys.html

@groverlalit Thanks!


New Follower Alert

It seems like there is no New Follower Alert. I was used to use such a thing before on different platforms. Now it's an issue. I don't seem to find it in your settings.  Or if not, can you implement it. Or give me information of how to implement it. Thank you in advance.

At the moment this is not possible, unless odysee add it to their API.


Pyenv is used to set up the exact version of Python

Goal: anyone who participated in this project MUST use the exact and the same Python.
pyenv is proposed as a solution.
What should be done:

a doc how to start should be added
pyenv should be integrated
feature is manually tested


setup.sh raises an exception

Exception:
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/pip/basecommand.py", line 215, in main
status = self.run(options, args)
File "/usr/lib/python2.7/dist-packages/pip/commands/install.py", line 353, in                                                                                run
wb.build(autobuilding=True)
File "/usr/lib/python2.7/dist-packages/pip/wheel.py", line 749, in build
self.requirement_set.prepare_files(self.finder)
File "/usr/lib/python2.7/dist-packages/pip/req/req_set.py", line 380, in prepa                                                                                re_files
ignore_dependencies=self.ignore_dependencies))
File "/usr/lib/python2.7/dist-packages/pip/req/req_set.py", line 554, in _prep                                                                                are_file
require_hashes
File "/usr/lib/python2.7/dist-packages/pip/req/req_install.py", line 278, in p                                                                                opulate_link
self.link = finder.find_requirement(self, upgrade)
File "/usr/lib/python2.7/dist-packages/pip/index.py", line 465, in find_requir                                                                                ement
all_candidates = self.find_all_candidates(req.name)
File "/usr/lib/python2.7/dist-packages/pip/index.py", line 423, in find_all_ca                                                                                ndidates
for page in self._get_pages(url_locations, project_name):
File "/usr/lib/python2.7/dist-packages/pip/index.py", line 568, in _get_pages
page = self._get_page(location)
File "/usr/lib/python2.7/dist-packages/pip/index.py", line 683, in _get_page
return HTMLPage.get_page(link, session=self.session)
File "/usr/lib/python2.7/dist-packages/pip/index.py", line 792, in get_page
"Cache-Control": "max-age=600",
File "/usr/share/python-wheels/requests-2.12.4-py2.py3-none-any.whl/requests/s                                                                                essions.py", line 501, in get
return self.request('GET', url, **kwargs)
File "/usr/lib/python2.7/dist-packages/pip/download.py", line 386, in request
return super(PipSession, self).request(method, url, *args, **kwargs)
File "/usr/share/python-wheels/requests-2.12.4-py2.py3-none-any.whl/requests/s                                                                                essions.py", line 488, in request
resp = self.send(prep, **send_kwargs)
File "/usr/share/python-wheels/requests-2.12.4-py2.py3-none-any.whl/requests/s                                                                                essions.py", line 609, in send
r = adapter.send(request, **kwargs)
File "/usr/share/python-wheels/CacheControl-0.11.7-py2.py3-none-any.whl/cachec                                                                                ontrol/adapter.py", line 47, in send
resp = super(CacheControlAdapter, self).send(request, **kw)
File "/usr/share/python-wheels/requests-2.12.4-py2.py3-none-any.whl/requests/a                                                                                dapters.py", line 423, in send
timeout=timeout
File "/usr/share/python-wheels/urllib3-1.19.1-py2.py3-none-any.whl/urllib3/con                                                                                nectionpool.py", line 643, in urlopen
_stacktrace=sys.exc_info()[2])
File "/usr/share/python-wheels/urllib3-1.19.1-py2.py3-none-any.whl/urllib3/uti                                                                                l/retry.py", line 315, in increment
total -= 1
TypeError: unsupported operand type(s) for -=: 'Retry' and 'int'

I am unable to reproduce the error, but as this StackOverflow link question suggests, it seems to be fixable by either re-installing pip or setting the correct proxy settings.

indeed, the problem is gone after updating pip. thank you.


Automation update 2021.05.20

Added several functions, reorganized into folder. Added an instructions page for a simple automation to gauge usefullness and interest before doing the extra work.

@garrett-wood, All looks good! Awesome stuff!


Add ϟ Enlightning host to profile page

Hey @wiggitywhitney! Just have a small tweak to get this PR working, I've opened up a PR against your branch that just moves this up to the front matter of the bio page, once that's merged we can get this merged to main :) Thanks!
https://github.com/wiggitywhitney/tanzu-dev-portal/pull/11


Issue with _fromData

I am implementing this after upgrading from an old version of a signature pad and I ran across a minor annoyance. When you run fromData on a series of points it will draw it as normal, but it will not add them to the _data variable. It was a simple fix and I've fixed it on my local site, but it might be worth adding an update so that the issue is avoided in the future.
In short, for every pointGroup found in the parameter pointGroups, it pushes an empty array onto _data, just like in _strokeBegin and for every point in that group it adds it to the last possible group in _data just like in _strokeUpdate.
SignaturePad.prototype._fromData = function (pointGroups, drawCurve, drawDot) {
  for (var i = 0; i < pointGroups.length; i += 1) {
    var group = pointGroups[i];
// like _strokeBegin
    this._data.push([]);
    if (group.length > 1) {
      for (var j = 0; j < group.length; j += 1) {
        var rawPoint = group[j];
        var point = new Point(rawPoint.x, rawPoint.y, rawPoint.time);
// start like _strokeUpdate
        this._data[this._data.length - 1].push({
            x: point.x,
            y: point.y,
            time: point.time
        });
// end like _strokeUpdate

// . . .

closing this as stale


WIP Removing traceur, which is no longer needed for Angular2

WIP, this fails, so it must be fixed before it should be merged
See https://github.com/angular/angular/issues/3737#issuecomment-132877615

"now that Traceur is gone we should be able to fix this"

According to https://github.com/mgechev/angular2-seed/issues/137#issuecomment-147175881

Seems like PhantomJS2 cannot work without Traceur - after restoring its
dependency (which was removed in  Update to<EMAIL_ADDRESS>commit: mgechev@b94031c)

Which means that we cannot remove Traceur anyway.


Translate Synthetic Event

Hi, kindly check / comment on the translation. Thanks!

hi @regalius !
sure, I'll check the suggestions by today or tomorrow night ya, will come back to you soon!

since pointer event is not translated, I think that "focus event" translation should also be retained. wdyt @regalius ?

Agreed, for now I think it's a little bit weird to translate the event names, let's keep it that way so later when we find the need to translate the event names, we can do it accordingly 😄

hi @regalius , looks like I'm finally done with the revision :smile: . sorry I'm kinda busy last week :disappointed:


can I change "localtunnel" to other word ?

for example:
https://yunzhuiyue.localtunnel.me
instead of
https://yunzhuiyue.google.me

I don't understand the question.


graph-toolkit-one-productivity-hub

Added One Productivity Hub sample under the following folder: graph-toolkit-one-productivity-hub.

@aycabas ,  thank you very much for your contribution to our sample gallery, I leave some comments for your PR~


feat: lake exe refactor, initial framework

This is the initial framework code for lake exe refactor. To use it, you add your refactoring to Refactor/Main.lean (note, you have to actually write some metaprogramming code here for the refactoring itself), and the harness will run it on specified files and apply the generated edits.

@adomani can you say more about what you used this for/other branches with this?

I just linked one of the tests that I did with refactor.  The extra metacode that I wrote for that tests is this declaration.


Add support for resources that have no dryrun possible (#3084)

Pull request opened by the merge tool on behalf of #3084

Merged in 0b717575c49f91252686807230d99a086561965c


Nested anonymous components default props overwrites given attributes

Laravel Version: 8.17.0
PHP Version: 7.4.12

Description:
Nested anonymous components can pass down attributes with de Attribute bag. A anonymous component can have a @props directive with defaults. If you pasdown the attributes bag with nested component, the default is used and not the passed attribute.
Steps To Reproduce:
Anonymous component button(components/button.blade.php):
@props(['tag' => 'button'])

<{{ $tag }} {{ $attributes->merge(['class' => 'inline-flex items-center px-4 py-2 border text-sm font-medium rounded-md shadow-sm focus:outline-none focus:ring-2 focus:ring-offset-2']) }}>
    {{ $slot }}
</{{ $tag }}>

Anonymous component button blue(components/button-blue.blade.php):
<x-button {{ $attributes->merge(['class' => 'border-transparent text-white bg-indigo-600 hover:bg-indigo-700 focus:ring-indigo-500']) }}>
    {{ $slot }}
</x-button>

Use somewhere in blade template:
<x-button-blue tag="a">Button text</x-button>

Expect:
<a class="inline-flex items-center px-4 py-2 border text-sm font-medium rounded-md shadow-sm focus:outline-none focus:ring-2 focus:ring-offset-2 border-transparent text-white bg-indigo-600 hover:bg-indigo-700 focus:ring-indigo-500">
    Button text
</a>

Given:
<button class="inline-flex items-center px-4 py-2 border text-sm font-medium rounded-md shadow-sm focus:outline-none focus:ring-2 focus:ring-offset-2 border-transparent text-white bg-indigo-600 hover:bg-indigo-700 focus:ring-indigo-500">
    Button text
</button>

Workaround to get expected, change the button component to:
<{{ $attributes->get('tag', 'button') }} {{ $attributes->merge(['class' => 'inline-flex items-center px-4 py-2 border text-sm font-medium rounded-md shadow-sm focus:outline-none focus:ring-2 focus:ring-offset-2']) }}>
    {{ $slot }}
</{{ $attributes->get('tag', 'button') }}>

Yeah, you have to be explicit about the tag prop you pass down.

Oke, why not pass it down? I'm just curious. With passing it down the component explicitly, we get some extra line of code that doesn't do anything besides passing it down.
By the way, thank you for creating laravel and doing all this work for the community!


Allow Markdown syntax for comments

The title says it all : it would be nice to have Markdown syntax for comments.

OK for markdown syntax ; many thanks :-)
But some style are not visible (for example, <em> and <code>) : the reset.css overrides the default font-style of the browser, so <em> and <code> are displayed as normal text.


No way to handle failed b2PolygonShape::Set

When b2PolygonShape::Set fails, whether it be because N<3, the polygon is degenerate, or otherwise, the function currently fails an assertion and calls SetAsBox. This makes it impossible to check if we've passed a bad polygon to the function, which is crucial in projects where polygons can be fully defined by the user.
Right now the only way around this is to copy the logic from within b2PolygonShape::Set to check the validity of our input before actually calling b2PolygonShape::Set.
I propose that instead of calling b2Assert and failing, b2PolygonShape::Set should simply return false (and continue to call SetAsBox as currently implemented - that part is good)

In the case where the polygon is defined by the programmer, an assert is desirable. With your proposed changes, that would require changing this
shape.Set(vertices, count);

to this
[[maybe_unused]] const bool valid = shape.Set(vertices, count);
b2Assert(valid);

to maintain the same behaviour. I think the current Set function should be kept as it as with a separate function for validation.

I'd generally argue against using asserts in functions with nontrivial failure cases. There's no reason to assume that every polygon is going to be made using predefined data. However, I can also see many reasons why it is done that way, so I agree that adding a function to pre-validate a set of points would be the best solution to this.

I will add a function to check for validity.

I will add a function to check for validity.

Heyhey, just stumbled across this issue. Looking at the b2PolygonShape::Set code I'm kinda curious how this is going to work out because Setbasically runs 2 internal algorithms that can lead to function abort: unique-folding and this Gift Wrap thingy.
So the polygon validator would have to run those 2 as well and then you can call Set with your validated poly which runs them again? (because it doesn't/can't rely on only getting valid polys)


feat(node_framework): add timeouts for running tasks

What ❔

Why ❔

Checklist

[ ] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[ ] Tests for the changes have been added / updated.
[ ] Documentation comments have been added / updated.
[ ] Code has been formatted via zk fmt and zk lint.
[ ] Spellcheck has been run via zk spellcheck.
[ ] Linkcheck has been run via zk linkcheck.

bad approach


Configuration could not be found 12.2.1

Description

Steps to reproduce
I have a monorepo, with frontend and backend directories. In the root folder I have husky with this pre-commit script:
#!/bin/sh
. "$(dirname "$0")/_/husky.sh"

cd frontend
npx lint-staged

In frontend/package.json this is the config:
"lint-staged": {
    "src/**/*.+(ts|tsx)": [
      "eslint --fix"
    ],
    "src/**/*.{js,jsx,ts,tsx,json,css,scss,md}": [
      "prettier --write"
    ],
    "*.js": "eslint --cache --fix"
  }

If I commit, for example, a .tsx file in frontend/src everything is working fine. If I commit for example README.md, no problem. But committing .vscode/settings.json in the root folder gives me an error.
Debug Logs

expand to view
> git -c user.useConfigOnly=true commit --quiet --allow-empty-message --file -
2022-01-20T11:54:20.322Z lint-staged:bin Running<EMAIL_ADDRESS>2022-01-20T11:54:20.323Z lint-staged:bin Options parsed from command-line: {
  allowEmpty: false,
  concurrent: true,
  configPath: undefined,
  debug: true,
  maxArgLength: 131072,
  stash: true,
  quiet: false,
  relative: false,
  shell: false,
  verbose: false
}
2022-01-20T11:54:20.324Z lint-staged:validateOptions Validating options...
2022-01-20T11:54:20.324Z lint-staged:validateOptions Validated options!
2022-01-20T11:54:20.324Z lint-staged Unset GIT_LITERAL_PATHSPECS (was `undefined`)
2022-01-20T11:54:20.325Z lint-staged:runAll Running all linter scripts
2022-01-20T11:54:20.325Z lint-staged:resolveGitRepo Resolving git repo from `/Users/ambes000083/Desktop/Ergo/zahn-ppz-otr/frontend`
2022-01-20T11:54:20.325Z lint-staged:resolveGitRepo Unset GIT_DIR (was `undefined`)
2022-01-20T11:54:20.325Z lint-staged:resolveGitRepo Unset GIT_WORK_TREE (was `undefined`)
2022-01-20T11:54:20.325Z lint-staged:execGit Running git command [ 'rev-parse', '--show-prefix' ]
2022-01-20T11:54:20.338Z lint-staged:resolveGitRepo Resolved git directory to be `/Users/ambes000083/Desktop/Ergo/zahn-ppz-otr`
2022-01-20T11:54:20.338Z lint-staged:resolveGitRepo Resolved git config directory to be `/Users/ambes000083/Desktop/Ergo/zahn-ppz-otr/.git`
2022-01-20T11:54:20.338Z lint-staged:execGit Running git command [ 'log', '-1' ]
2022-01-20T11:54:20.346Z lint-staged:execGit Running git command [ 'diff', '--staged', '--diff-filter=ACMR', '--name-only', '-z' ]
2022-01-20T11:54:20.353Z lint-staged:runAll Loaded list of staged files in git:
[
  '/Users/ambes000083/Desktop/Ergo/zahn-ppz-otr/.vscode/settings.json'
]
2022-01-20T11:54:20.354Z lint-staged:loadConfig Searching for configuration from `/Users/ambes000083/Desktop/Ergo/zahn-ppz-otr/.vscode`...
Configuration could not be found.
husky - pre-commit hook exited with code 1 (error)

Environment

OS: macOS Monterey
Node.js: v16.13.1
lint-staged: 12.2.1

Thanks, kind regards.

I guess we should also include any config from the cwd. Currently your file doesn't have any config in its directory path.
Thanks for the issue!

I think you are not asking for additional info. Are you talking about you should add additional feature to solve this behaviour? Thanks for the prompt response!

@javifm86 yes, I think I understood your issue and will create a fix!


Possible RemoveGraph  bug wrt Accumulo tables & changing schemas

From a non-github user:
"If I execute RemoveGraph, then by default the underlying Accumulo table will remain. If I then execute AddGraph using the existing table name then happy days, I can re-access my data. We've been using this mechanism to make subtle changes to the schema. Obviously needs a certain level of compatibility or bad things will happen but e.g. for additive changes this approach works well.
Turns out, however, there's a problem with this approach. In this scenario the original schema remains set as Accumulo configuration properties and is hence what is enforced e.g. at compaction. This can then vary from the schema that's being applied on ingest (via the API). I can't understand how that is a good place to be, so I consider this a bug, but is it actually intended behaviour? It seems sufficiently bad to diverge schemas like this I'd suggest one of two alternatives: A) when you re-add the graph, and Gaffer notices there's an existing Accumulo table there, it compares the schema and gets grumpy with you if it's not the same as that set on the existing table. B) when you re-add the graph, and Gaffer notices there's an existing Accumulo table there, it updates the version of the schema held in it's config properties (exactly the same as it would have set it in the first place if it was a new graph). I'd strongly favour B). Any thoughts?"

@d21211122 suggested: "... longer term, I'd prefer it if we didn't allow the update at all, and provided a specific 'update schema' operation for this which does all the compatibility checking for you... and then we can make 'remove graph' actually remove the graph..."
"

The motivation behind all this is a need to be able to mark records for deletion. The process above is a scary way of going about this. Instead we will look at a better solution to that problem before deciding how to proceed here.

See #3004 for where this deletion concept is explored further.


"GitHub authorization failed" when trying to reconnect to a repository using github organization

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
new_app
AWS Region
us-east-1
Amplify Hosting feature
One click deploy
Frontend framework
Next.js
Next.js version
10.1.3
Next.js router
Pages Router
Describe the bug
Existing amplify deployment can't reconnect after turning on mandtory 2fa for github users.  Revoked tokens and stil lgetting "GitHub authorization failed" on aws when connecting github repo.
Expected behavior
Should connect to github.
Reproduction steps
existing amplify on github organization connected and working.
add mandatory 2fa for github users.
need to reconnect amplify to github but when I try just get errors GitHub authorization failed
Build Settings
just regulat stuff

Log output

# Put your logs below this line

I'm not reall a dev
Additional information
I'm not super devy.

I ended up revoking the AWS token, and then turning on ALL repo access instead of two of my repos, then reconnected and it all worked.  Users should still be able to share only select repos with aws.

Hi @phelix001 👋 , thanks for raising this!
Amplify uses the GitHub Apps feature to authorize Amplify read-only access to GitHub repositories. Can you try the following steps to provide read-only access to only select repositories:

Within your GitHub profile, navigate to Installed GitHub apps: https://github.com/settings/installations/
Select the AWS Amplify (us-east-1) GitHub app
Select Configure
Within the Repository access section, choose only select repositories option
Click Save

Alternatively, you can also uninstall the AWS Amplify (us-east-1) GitHub app and can restart the GitHub authorization process from the Amplify Console. During this process, you will be prompted to:

Select where you want to install the GitHub app
Install and authorize all repositories or only select repositories

Yes, I tried both those methods, and ended up not being able to select a single or few repos but had to disconnect completely and ONLY choose ALL repos.

Having this same issue, on an IAM account, can access Amplify dashboard and have access to a private organization repository. Attempting to connect and every time receiving a Github authorization error with no further explanation

@phelix001 @kensiej 👋 , I was not able to reproduce this behavior. Here are the flows I tried:

2FA was already enabled on my GitHub profile
Enabled the requirement of 2FA for users on an existing repository which is linked to a GitHub organization
On an existing app, I used the Reconnect this repository button in the Amplify console and was able to re-authenticate to the GitHub repository successfully
I was able to modify the GitHub app to only allow access to a single repository and was able to successfully reconnect following the above step.
Also, created a new app altogether connected to the same repository and was able to authenticate as expected.

Are you using the 1-click deploy feature? Can you share the detailed flow of steps/events using which you can reproduce this behavior?

Can you try again? if you are still experiencing an issue please open an aws support case


[WIP] CEL matcher

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/retest

/assign @htuch @wbpcode @yanavlasov @kyessenov

/lgtm deps

@kyessenov @yanavlasov  Thanks for review! PTAL. Thanks.

Waiting for @kyessenov approval

/wait-any for @kyessenov review

/retest


CompletionItemProvider locating currentprocedure in wrong source

Describe the bug
If you use copybooks with procedure prototypes inside you'll find vs code intellisense behaving strange: the list of suggestions doesn't match with your source at all.
To Reproduce
Define a copybook with a procedure prototype definition on a specific range of code lines, e.g. from line no. 10 to 12:
**free

dcl-pr SomeProcedure;
  SomeParameter char(1);
end-pr;

Use a /copy statement to include the copybook and place a procedure inside, where the procedure body covers the same line number:
**free

/copy 'QCPYLESRC/somecopybook.rpgle'

dcl-proc ExportedProcedure export;
  dcl-pi *n;
    ExportParameter1 char(1);
    ExportParameter2 char(1);
  end-pi;

end-proc;

Now if you start typing on said lines (e.g. 10-12) in your rpg source, you won't be able to find local definitions (e.g. "ExportParameter1"), but will find parameter definitions from your copybook procedure prototype instead (e.g. "SomeParameter") - as if you were typing on the same line but in the copybook source.
If you instead start typing on another line where there's no definition (e.g. 14), you'll find the suggestions are correct.
Expected behavior
I'd expect to find the same local definitions as suggestions on every line of code in a procedure.
Screenshots

Environment

Extension version v0.26.8
IBM i OS version 7.4

Additional context
I found out that in completionItem.ts, currentProcedure is only found by comparing line numbers of definitions, but file path isn't being compared.
After I added the following code inside find() on line 34, the suggestions worked again for me:
&& currentPath === proc.position.path
I'm not sure if there's a reason why this check is missing but maybe you can consider this for a future fix.

Related PR release in 0.28.9.


Reconcile in case the snapshot in use in ovirt

When creating multiple snapshots, it is possible forklift is too fast and ovirt is still creating the previous snapshot. In that case we hit:
Fault reason is "Operation Failed". Fault detail is "[Cannot create Snapshot. Snapshot is currently being created for VM golden_env_mixed_virtio_2_1.]". HTTP response code is "409". HTTP response message is "409 Conflict".
This also may happen when deleting the snapshots. We better reconcile in such case and not fail the whole migration.

It's unlikely that when triggering the snapshot creation (by default every hour) it conflicts with the previous snapshot creation (which is supposed to be almost immediate - as it doesn't include memory). I'm closing this in favor of #283 that provides a broader scope of this issue


Fix enabling motion bias estimation

motionBiasEstEnabled should be true if VQF_NO_MOTION_BIAS_ESTIMATION isn't defined

I generally agree.
So that change will enable that feature for all. So is that a improvement?
@nekomona whats your meaning, can we just add it?

To be clear this doesn't enable motion bias estimation - the user still needs to undefine VQF_NO_MOTION_BIAS_ESTIMATION, this only fixes enabling it


[low-n-slow-totaltime] Out of memory

@nilsbenson is the Out of Memory related to the lines of code, the dynamic memory allocation, or both? The low-n-slow-totaltime runs out of memory in several instances. This comes randomly.
What can I do to reduce the memory footprint?

I just saw your comment on WelcomeView related to the logoImage. I'll create the smaller size logos.
Error: Out Of Memory Error
Details: Failed invoking
Stack:

onShow() at /Users/canedo/eclipse-workspace/barbecue_app/source/WelcomeView.mc:33 0x10000657

added #90 as one of the steps to take to reduce memory use

going to close this for now. We have some extra space free now for the 64kb devices and it should be enough if we're careful.


Does not crawl if config provided

ucss -h http://www.host.com -c http://www.host.com/path/to.css crawls the page and outputs the urls as expected. however there are links to subdomains and youtube so this option is not suitable.
ucss -g config_ucss.js doesn't seem to crawl beyond the pages.crawl option in the config
content of config_ucss.js
module.exports = {
    "pages": {
        "crawl": "http://www.host.ca/",

        "exclude": [
            "http://www.youtube.com/",
            "http://sub1.host.ca/*",
            "http://sub2.host.ca/*"
        ]
    },

    "css": [
        "http://www.host.ca/css/style-min.css"
    ]
};

Thanks for the bug report, and sorry for the slow response. I will look into this as soon I get the chance, but I can not make any promises about when.

I've finally been able to have a look at this; sorry that it took so long!
From what I understand, you expect the crawler to go beyond the domain to crawl. But it should never do this: Links to e.g. YouTube will never be followed. This is because your CSS is not relevant for YouTube.
Thus, the links you have in pages.exclude are not needed: pages.exclude is only for pages on your crawled domain that is to be skipped by the crawler.
Let me know if I've misunderstood. Also, if you have suggestions for how to make the documentation/example clearer, please let me know.

Here's some sample output from using this command ucss -h http://www.shaw.ca -c http://www.shaw.ca/uploadedfiles/ecomm/css/shaw-min.css

As you can see it's crawling subdomains (would like to disable that as they are entirely separate sites) as well as Facebook, YouTube and iTunes.
Using just ucss with essentially the same config as above does produce output although it doesn't appear to crawl beyond the specified page. It's flagging styles that I know for sure are used on other pages (315 used, 3352 unused?... nope!)

Thanks, I can reproduce this. I will try to look into it when I'm back from vacation next week.

Seems like there may be two separate issues:

Sometimes, pages outside of given domain is visited by crawler.
The used selectors count seems too low for the specified page. This may indicate that not all pages are visited.

I think I've fixed the first issue (in a separate branch, 82b66787dff7b41cd796ef8852482719a849b967). It was caused by protocol relative URIs (e.g. "//newsroom.shaw.ca/") being handled as domain relative URLs, which is clearly wrong.
As for the second issue: I ran ucss towards your site and got the following (both with config and from command line):
3674 (1303 used, 2364 unused, 304 duplicates, 2 ignored)

This is a higher number of used selectors than you got. Does it seem correct?
(Btw, it seems like you have a broken link somewhere, pointing to hhttps://www.shaw.ca/store/cart/orderLeadForm2.jsp)

There's a good possibility that those results are more accurate. We still have a legacy stylesheet that's still in use (which is the root of this ol' project of mine). I'll take a closer look when I have some time (hopefully next week).
Thanks for pointing out the broken link. I'm surprised our marketing dept. hasn't pointed that out yet... lol

Great! I will try to integrate and launch the changes mid next week, when I am back from travel. Until then, please use the branch mentioned above.

I've integrated and released this now. Very sorry that it took longer than expected.


bugfix: fix build for C NRIWrapperVK.h

I was running into build issues on linux related to NRIWrapperVK.

Thanks. I thought I had found all such cases...


[WIP] Build automation

This PR contains:

Migration to xUnit 2.1
Single build script for win, linux and osx
Travis CI execution matrix that run test on  (OSX & Linus)x(Mono Latest & Beta & Alpha & Nighly)
Nuspec and automated nuget package creation.
Latest 2.* NuGet.exe

In order to release new version you need to

Update RELEASE_NOTES.md
Execute build release

@EricWhiteDev Please merge #79  and #80 first and share your thought about this one.
Then I will rebase & cleanup this PR if required.

Hello, what is the current status on this?

@japj I still wait any kind of response from maintainers about their plans about this repo.

@sergey-tihon we have a system in place for CI. you might want to rework this with the new build system for the cross platform aspects.

@tomjebo this PR contains "nuget package creation". If the package already created in another way, please give me a link to related info. If package not created, is there any plans to implement this feature? If there is no such plans, I suppose this PR should be reopened, reworked by author and then accepted.

@resnyanskiy Please refer to the readme for the repo. There are instructions to retrieve the nuget package there.

@tomjebo thank you very much 👍


ag -i doesn't find special chars

echo "b€" | ag -i € doesn't find €, why?

This works fine with ag 2.2.0 on my machine:

Note that € is highlighted, which is perhaps not easy to see with your color scheme?

@dabba39 Are you running this on Windows or on Unix?
This is indeed not working with the default Windows version, and I built one that works with non-ASCII characters there.
Note also that for this to work in Windows, you need to have your console configured to use a code page that contains the € character. (The fact that cmd.exe can display the € character is no proof of that: cmd.exe is a Unicode application that bypasses code pages for displaying to the screen, but uses them when piping data to another application!) And unfortunately the default 437 code page used in US versions of Windows consoles does not include the € character. In that case, change to the 1252 or 65001 code page.


when release update version to PYPI?

@maxcountryman Hi, when do you plan on making a new release? The old one is broken in all sorts of ways with recent flask versions.

Well I’d like to release soon, however I’m busy and getting PyPI setup is kind of a pain. Also please bear in mind I don’t get paid for this work, so it’s unfortunately not a super high priority. In the meantime you can also install from git.

Of course, I understand that. It's just that packaging the git version for us (Gentoo) isn't as straightforward as using a PyPI release (but it's doable :smiley:).

That makes sense—I’ll try to get PyPI setup in the near future.

@maxcountryman actually I might not need the new release so soon, I misunderstood the goal of the package, what I wanted is https://github.com/pallets/flask-sphinx-themes/.
I now understand that your package serves a wholly different purpose.


The division of the dataset into training, testing, and validation sets.

Hello, I have a question. I noticed that you divided the data into training, testing, and validation sets in the code, but during the training process, I didn't see any differentiation. Instead, all the data was used for training the model, and during testing, all the data was used as well. Why is that?

It's called transductive learning different from the inductive learning.

Hello, I have the same question. Can you answer it now


Add method to get/set georeference scale by blueprint

Fixes #1124

Thanks for the pull request @joseph-kaile!

:x: Missing CLA.

Please send in a Contributor License Agreement (CLA) and comment back here to let us know to check this!
Maintainers, here's how to check for a submitted CLA.

Reviewers, don't forget to make sure that:

[ ] Cesium for Unreal Samples works.


Add option for price axis overlay the pane

Add an option so that the price bar is on top of the chart.

That's a good suggestion, thank you. I believe we can add new positions for price axis something like overlay-left and overlay-right.

any update?

+1

Thank you for your feature request for Lightweight Charts. In order to keep the library lightweight and maintain a narrow focus, we strive to include only strictly required features in the core library. As a result, we have decided that this requested feature should be implemented as a plugin instead, and will be closing this issue as a result.
We recently introduced the ability to write plugins for Lightweight Charts, allowing users like you to extend the functionality of the library without adding it to the core. We highly recommend implementing your feature request as a plugin to achieve the desired functionality while keeping the core bundle size smaller for all users.
To get started with plugins, please refer to our Plugins Documentation for a better understanding of what is possible and how plugins work. You can also explore our collection of plugin examples (with a preview hosted here) for inspiration and guidance on implementing specific functionality.
For your particular feature request, we suggest taking a look at the following example:
•	Overlay Price Scale: This example demonstrates how to achieve x, y, and z. Demo
To help you get started quickly, we have created an NPM package called create-lwc-plugin, which sets up a plugin project for you. This way, you can hit the ground running with your plugin development.
If you do create a plugin based on your feature request, we would appreciate it if you could publish it on GitHub or NPM and let us know. This way, we can share it with the rest of the community. It would be helpful if you could prefix your NPM package name with lwc-plugin- for easier discoverability by others.
Thank you for your understanding and support. We believe implementing your feature request as a plugin will greatly benefit both you and the Lightweight Charts community.


Update Zenodo DOI

This issue is duplicate. So we won't fix it.

Okay. SUre. Let's not do redundant work.


animation and player speed enhancement

Co-authored-by: choonhongyeoh0241<EMAIL_ADDRESS>Walking Animation fixed

Animator fix walking exit time

Uncheck exit time and fixed duration for both transition
Transition Duration set to 0

Player Script fixed

FixedUpdate()  method implemented

RigidBody component used in Player, FixedUpdate() method used for maintaining RigidBody.MovePosition
delta assigned to hold player movement calculation

They fix the issues perfectly. Thank you very much
Thank you for explaining FixedUpdate and RidigBody


React development warning for componentWillMount

Using v6?
Yes
Describe the bug
Using React table 6.10.3 with React 16.9.0, the following warning is shown:
Warning: componentWillMount has been renamed, and is not recommended for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

Move code with side effects to componentDidMount, and set initial state in the constructor.
Rename componentWillMount to UNSAFE_componentWillMount to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run npx react-codemod rename-unsafe-lifecycles in your project source folder.

Please update the following components: ReactTable
Expected behavior
No warning

Please feel free to submit a PR to fix this issue :)

Closing this issue since it is v6 related.


Math error in Soln to problem 3.5: Gaussian vs jointly Gaussian

Eq (56) in the solutions to Book 1 states
$$E[XY] = ... = -1 \cdot 0.5 \cdot E[X \cdot -X] + 1 \cdot 0.5 \cdot E[X \cdot X] = 0$$
But I think the $-1$ and $1$ are erroneous and should not be there.

you are right - fixed


Calls to DBContext in OnParametersSet causes Errors when moving between pages.

Describe the bug
Moving between pages where both have calls to the dbContext in their OnParametersSet method causes concurrency errors.
Notes
All calls should be moved to OnInitalized methods instead.
Moving items to the OnIntialized creates a new problem - the content is called twice. This can be mitigated by either moving it to the OnAfterRender(bool firstRender) method, or using the example here about stateful reconnection to prevent multiple calls to the dbContext or userManager.

Going to go with OnAfterRenderAsync as it has less changes for similar functionality. This may be changed later to use OnInitalized but currently the services do not support using the MemoryCache.

A further update. We are using the Blazor preview and have access to ApplicationState helpers - this will be used instead where needed.

Some parts of data can't be saved, large collections such as full roster lists, and must be registered after the first render and only the first render.

Fixed as of 02ddb8c and new policies for data will need to go into effect for pages.


is there a way to contribute to improve your training dictionaries

Of course just the fact that you made it so far with purely automated translation is amazing! Could we try to get you even better training database by manually vetting your 10K lists? I could check en, ru, de, es in any combination, for example.

Cheers Motorrat!
I think there's a lot more we could do on the methods side to improve the alignment. I wouldn't want to use up people's time before doing some more work myself!
We would like to open source the training/test dictionaries, but we haven't done so yet since we aren't completely sure legally whether the Google Translate API allows this. We're going to try to find out though!

Hi Samuel,
any chance you can release just a few of the training/test dictionaries? I guess you might have some non Google dicts that you probably have used for calculating the matrices?
Would be really nice to be able to go through and reproduce the calculations in your paper with real data. Who knows? Maybe someone else might find alternative methods :)
All the best,
Ajay

Hi Ajay,
All the training/test dictionaries for the matrices in this repository were constructed from Google Translate, but the training/test dictionaries and the word vectors for English and Italian used in our paper are all freely available online from this link:
http://clic.cimec.unitn.it/~georgiana.dinu/down/
When we get back from holiday next week we'll investigate open-sourcing the dictionaries used in this repository, and we will also provide some training code (it was just a rush getting the repository out before the ICLR conference...). We are definitely keen to see people improve on our work :)
Best wishes,
Sam

Thank you :)


Update Emoji Search in Examples

@ahfarmer does not update its repository for 1 year.
My repository packages always contain the latest versions.

I’d be cool with it if you can specify @ahfarmer as the original author somewhere in the README. :-)

FYI I just updated my repo to the latest packages.
@alik0211 you can always file an issue or a pull request in my repositories when they aren't up-to-date.

👍

@ahfarmer You ignore pull requests, therefore I did pull request

@alik0211 ah, good point. Thanks for bringing that to my attention. Must have had notifications not set up correctly. Mentioning me always gets my attention in case that happens again.

@ahfarmer Ok. No problem. ;)


playground: typeless map keys output format errors, but compiles

Example: http://play.golang.org/p/DuxMMkl3TF
Running the formatter on it, it complains that the map key does not have the type declaration [2]int, but it runs just fine. Is this the expected behavior?

cc @griesemer

I assume you mean that the playground "format" button doesn't work, because gofmt (from Go 1.5) does work fine.

Sorry I think this actually a goimports issue. I first noticed this using the vim go plugin.. and I realized that I override the plugin to use goimports. I presume the playground also uses this?

I don't know what the playground uses.
goimports works for me too.  Note that omitting the map key type is new for Go 1.5, and that goimports is not part of the Go 1.5 release.  Perhaps you simply need to update your goimports binary?

I think @adg forgot to update something when deploying the playground for 1.5.

Perhaps you simply need to update your goimports binary?

Thanks, that was it! I thought I had done that, but apparently not.

It's because App Engine is running Go 1.4.2 and the playground is using its go/format package.

There are plan to upgrade Gae to 1.5
Thanks
El 17/9/2015 2:42, "Andrew Gerrand"<EMAIL_ADDRESS>escribió:

It's because App Engine is running Go 1.4.2 and the playground is using
its go/format package.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/12644#issuecomment-140934227.

This is fixed.


test-hdfs

test-hdfs
Closes https://github.com/dask/dask/issues/5230

Fixed I think. Just need to explicitly close the pool that we create in test_read_bytes. Thanks to whoever setup those CI scripts, it was relatively painless to debug this locally.
Merging on green.


Fix project loading with bad text-number connections

Fixes #1213
Change-Id: I78c1666fd34346e46e68e112d02603e4de197fa5

This bug has come up a number of times on the forum. Here's the most recent example: https://community.appinventor.mit.edu/t/the-blocks-area-did-not-load-properly-changes-to-the-blocks-for-screen-5690035441500160-scrhoy-will-not-be-saved/7729?u=ewpatton

I like the tests :D


Create SASS Mixins for Icons

Hi Guys
I'm thinking of using this repo for the icons in my app but it seems to be written to only support declarative usage. I'd prefer to use the icons directly in my own SASS files.
Would you be open to me refactoring the SASS file to use mixins so you can call them directly in another SASS file, or output a CSS file exactly the same as the current one.
It'd be based roughly on (a simpler version of) this: http://jaydenseric.com/blog/fun-with-sass-and-font-icons
Alternatively, if you have a better idea for how to do this, I'd be glad to hear it.

That sounds good to me.

Thanks, Geordan. I'll see what I can do...

Done.
The output CSS file looks the same to me.
I'm new to Grunt so you might want to check my Gruntfile. And I used SCSS instead of SASS - does that matter? I was having trouble writing the mixin in SASS mode. :/
https://github.com/geordanr/xwing-miniatures-font/pull/22

Do you want me to write a short how to guide for using the SASS files for inclusion in the readme?

Yes that would be great! Thanks for contributing, btw!
Den 23 mar 2017 11:40 skrev "Fergus Hadley"<EMAIL_ADDRESS>
Do you want me to write a short how to guide for using the SASS files for
inclusion in the readme?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/geordanr/xwing-miniatures-font/issues/21#issuecomment-288679965,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHCfJfBVjwuI53yzV-vEr9r1cxht0jcPks5rokwJgaJpZM4Mjqsm
.


Autocomplete for deployments, pods and namespaces

When user adds a new service we could fetch the list of namespaces, deployments and pods from Kubernetes API, and instead of typing in the input user will be able to pick one of the options

Remember to correctly handle permissions. In some cases, user might not have permission to list ns, deploy or pods. There should be combo boxes, so user can try to manual enter input in all cases.

@Adiqq good point! Thank you!

I have started to do.
https://github.com/pixel-point/kube-forwarder/commit/566cc51f58d1b012ad133f57e42f8da671dc0de8

Added in v1.2.0


InstallLanguageKey Issue in Installer.cs

//Strip 2digit langcode from filename
The line should be updated to:
var langcode = Path.GetFileNameWithoutExtension(languagefile);  since the Umbraco XML files "en_us.xml" or "zh_tw.xml" would currently break the code.

Done...thank you!


All differences are only detected when info.version is equal for compared documents

Looks like for the same specification using different versions give different results.
Any logic behind that?

Thank you for your feedback! Could you elaborate on what you're getting and what you're expecting? Swagger::Diff doesn't take version into consideration, so it's possible there's another culprit. To confirm, I just tried:

comparing the same specification with a different version
comparing a specification with a changed attribute and a different version
comparing a specification with a changed attribute and the same version

and all of the results were as expected.

@jeffreyc I'm very sorry but I made a mistake when choosing github project... There are two projects with the same name...
Btw... I'm searching for a swagger diff tool to integrate with Maven...


Hacktoberfest Page UI Fixes

Fixed 3 Spacing Issues in the Hacktoberfest page

Congrats 🎉, your PR got merged. To be eligible for the swags from iNeuron X ClueLess. Follow this steps.

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


Using Oracle Vector store

Discussed in https://github.com/arc53/DocsGPT/discussions/980
Can be added by creating a file here
https://github.com/arc53/DocsGPT/tree/main/application/vectorstore
Will require concrete class similar to others.
Also looks like langchain supports it, which will make integration extra easy.
https://python.langchain.com/v0.2/docs/integrations/vectorstores/oracle/#load-documents

Happy to do this, please assign me!

Hello @dartpain & @SDanielDev , actually been working on this issue and almost have a pull request ready. Would you mind taking a look and offering any feedback?

Hello @dartpain I want to contribute into this can you assign this to me?

Hi @dartpain Can you please describe the issue?

@eeshan15
You can check out existing PR, and try to fix the remaining issues there. #995
thank you!

Hello Alex @dartpain if you could help telling me a simplified version of this issue it would be great because I am new to open source


Bridge should use colors stored in each body

The simulator now has bodies that generate and store their colors so that they can be persistent and customizable. The bodies directive should use this to determine the colors to draw.

This will be resolved by pending PR #126


rollup-plugin-index-html handle inline scripts and its imports.

I am often doing single file components where i inline everything into the html even
and also cjs
i would love to see this case handled with index-html
**importent is ***
cjs needs to stay cjs and esm needs to stay esm they have diffrent loading algos that trigger in diffrent dom state.
it should only inline or build the import from statements from esm and the dynamic imports from cjs
it should in general sanitize any urls referencing js
rollup has now also support for data urls that should also get used here.
i created a little https://github.com/direktspeed/esm-loader that allows isomorphic loading via data url feature
here is the rollup support for it https://github.com/rollup/plugins/pull/150

@LarsDenBakker ok i agree we should not handle the scripts and modules them self we should handle the imports they are doing so static and dynamic import