Check infrastructure prerequisites as part of join operation

Currently, we check a few things when joining a cluster:

Minimal K8s version.
Valid customCoreDNSConfig.
Overlapping GlobalnetCIDRs when the user explicitly supplies a globalCIDR

Expand subctl join to check additional prerequisites from the infrastructure itself, and make sure it supports installing Submariner:

[ ] Gateway nodes are set-up -  Kube proxy, if there, doesn't run in IPVS mode - as it's currently not supported.
[ ] The broker is reachable from the cluster.
[ ] Check if CNI as detected by us is supported (from a list of plugins).
Warn that Submariner might not work if the CNI not detected (but don't fail the operation).
[ ] Extra checks for:

[ ] OVN-K -  Version of nothbound database is supported.
[ ] Calico - IP Pools are configured in the cluster to instruct Calico not to perform NAT on the subnets associated with the Pod and Service CIDRs of the remote clusters.

[ ] Conflicts with existing clusters in the clusterset.

[ ] CIDRs should be distinct, unless the clusterset is using globalnet.

[ ] The necessary ports are open.

[ ] On the gateway, when not using Load Balancer, the cable driver port(s).
[ ] On the nodes, when not using OVN, the vxlan port.

Optionally add a --dry-run flag in order to allow running just the validations, without installing Submariner on the cluster.
To see proposed implementation details, check out https://github.com/submariner-io/enhancements/pull/134/

Shouldn't this be in the enhancements or subctl repo?


feat: Auto-generate embedding when inserting the row

It would be better if we could support generating embeddings in the DB for the text field.
CREATE TABLE items (
    ...,
    content text,
    embedding vector(n) GENERATED ALWAYS AS (generate_embedding(content)) STORED
);

For example, create a table called "items" with a column named "content" of type "text", and another column named "embedding" of type "vector(n)" that is generated based on the "content" column using a function called "generate_embedding". The "embedding" column is marked as [https://www.postgresql.org/docs/current/ddl-generated-columns.html]("GENERATED ALWAYS AS STORED").
Not sure if generated column supports a func call. We can create an AFTER INSERT OR UPDATE trigger on the table that calls the function to calculate the value and updates the column accordingly if it is not supported.
CREATE OR REPLACE FUNCTION update_embedding_column()
RETURNS TRIGGER AS
$$
BEGIN
    NEW.embedding := generate_embedding(NEW.content);
    RETURN NEW;
END;
$$
LANGUAGE plpgsql;

CREATE TRIGGER calculate_embedding
BEFORE INSERT OR UPDATE ON items
FOR EACH ROW
EXECUTE FUNCTION update_embedding_column();

We could first implement the func generate_embedding and corresponding config for OpenAI embedding model. Then write a doc to show how to do it.

Generated column needs an immutable expression. Vectorizers must be marked volatile because there are IO operations. We may only take advantages of triggers.

As Part of this work, #350 implement new functions:
text2vec_openai('input', 'model-name') -> vector for user specified model.
text2vec_openai_v3('input') -> vector for most modern official model text-embedding-3-small

We may need to guide users to create another table to insert vectors.
See also #346.

After #350 ,there seems not worthing for E2E embedding retrieval at database itself. They will be implemented outside.


Improve the performance of hqps http handler

The query performance appears to be influenced by hqps http handler setup.

Closed since resolved


about training loss

The loss of training is very small and  decreases slowly  when i finetune model on cocodatasets. Is't normal？
and how many epoches do you set?

taining from scatch on cocodatasets, not finetune

We used 140 epochs. You can find the settings that we used exactly in the experiments folder.

what about the loss of training

The heatmap loss is usually on the order of 1e-4 and the offset loss is usually on the order of 1e-5.

Thanks,
By the way, What the loss of training did you get when you finish the 140 epochs of training?

For the model with HRNet-W48 backbone, the AP on COCO validation with multi-scale testing is 72.3, the heatmap loss is 1.752e-4 and the offset loss is 4.764e-5.

ok , thank you!


Add support for capturing interimResults (Real Time Hypothosis)

It would be great to have an event listener which allows me to get hypothosis in real time, allowing the user to see what they are saying in real time, similar to how Android's speech input works.
I know the webkitSpeechRecognition allows this and I would be willing to add it once I have time.

yes it is possible. why would you want to do that?

In order to allow

the user to see what they are saying in real time, similar to how Android's speech input works.

Basically to indicate that it's working.
Why would you question that giving a user real-time feedback is good?

piwik image tracker no javascript

That reply has nothing to do with this thread.
@TalAter I don't know if the above comment is from a bot or a person trolling. In either case I will stop responding to it and defer to you to moderate.

@alanjames1987 This is now possible in annyang v2.0.0
Since v2.0.0, the result, resultNoMatch, resultMatch callbacks are now called with a list of phrases the user said.
So you can do something like:
annyang.addCallback('result', function(phrases) {
  console.log('Speech recognized. Possible sentences said:');
  console.log(phrases);
});

Does the 2.0.0 release allow for continuous / real time results?
I don't see a way to enable that.

Sorry, I was meaning interimResults. I don't see a way to enable that and that was what I was referring to when I called it "real time".

No.
I think I didn't add it originally because I didn't want to try and match interim results with commands.
But with the new callbacks that can be called with parameters, we could maybe implement interim results as a callback.
We'll need to check if we can enable it without affecting current functionality...
What do you think about this approach?

I think that would be a great approach.

Hi, I wanted to use your awesome library but I need interimResults on my project :/
Is it possible to implement that ? No news on this issue since 2015 :(
Currently, it's very difficult for somebody to know if the application listen him without interimResult.
I only need a simple render like this one : https://www.google.com/intl/fr/chrome/demos/speech.html (grey interim results, black final results) :)

Thanks @Kant73 ! I used that code to implement interim results as well and it's working great.

The @Kant73 solution for this issue worked perfectly for me.

I'm trying to get interim results.  I see two people here that say they used the above method and it worked.  I can't seem to get it to work.  Has anything changed since this was posted.  Here is my exact code.  I had to change it slightly from Kant73's original just because I think he only posted a snippet.
`

    Annyang! Speech Test

No results yet

`
Thanks.

Argh. Sorry I figured it out 2 minutes after posting this.  The interim went so fast that it only showed up in the console but never in the <div> on the screen.  I figured I'd leave my above comment in case someone wants a full example that works (at least in the console).

Thanks!!!! It saves me a lot :)


[Bug Fix] codeunit 90 "Purch.-Post"

There is a bug in codeunit 90 "Purch.-Post" as follows
Original Code
                OnDivideAmountOnBeforeTempVATAmountLineRemainderModify(PurchHeader, PurchLine, TempVATAmountLine, TempVATAmountLineRemainder, Currency);
                TempVATAmountLineRemainder.Modify();
                if "Deferral Code" <> '' then
#if not CLEAN20
                    if UseLegacyInvoicePosting() then
                        CalcDeferralAmounts(PurchHeader, PurchLine, OriginalDeferralAmount)
                    else begin
#endif
                        GetInvoicePostingSetup();
                        InvoicePostingInterface.CalcDeferralAmounts(PurchHeader, PurchLine, OriginalDeferralAmount);
#if not CLEAN20
                    end;
#endif
            end;

        OnAfterDivideAmount(PurchHeader, PurchLine, QtyType, PurchLineQty, TempVATAmountLine, TempVATAmountLineRemainder);

Bug fix
                OnDivideAmountOnBeforeTempVATAmountLineRemainderModify(PurchHeader, PurchLine, TempVATAmountLine, TempVATAmountLineRemainder, Currency);
                TempVATAmountLineRemainder.Modify();
                if "Deferral Code" <> '' then begin //it needs a begin here               //FIX
#if not CLEAN20
                    if UseLegacyInvoicePosting() then
                        CalcDeferralAmounts(PurchHeader, PurchLine, OriginalDeferralAmount)
                    else begin
#endif
                        GetInvoicePostingSetup();
                        InvoicePostingInterface.CalcDeferralAmounts(PurchHeader, PurchLine, OriginalDeferralAmount);
                    end;  //and close the begin here         //FIX
#if not CLEAN20
                    end;
#endif
            end;

        OnAfterDivideAmount(PurchHeader, PurchLine, QtyType, PurchLineQty, TempVATAmountLine, TempVATAmountLineRemainder);

because of the missing begin end GetInvoicePostingSetup() is executed only when "Deferral Code" <> '' but  InvoicePostingInterface.CalcDeferralAmounts is executed always resulting in runtime error.

This bug also affects  version 19

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


allow GET requests

needed for lokid pinging on uptime proofs

We are going to have the storage server ping Lokid, not the other way around, so closing this PR.


TypeError: not all arguments converted during string formatting

I used orator 0.9.8 and Python version 3.7
When I try to insert data, I get the following exception
python code
from orator import DatabaseManager

config = {
    'mysql': {
        'driver': 'mysql',
        'host': 'localhost',
        'database': 'test',
        'user': 'root',
        'password': '123',
        'prefix': '',
        'log_queries': True
    }
}

db = DatabaseManager(config)

db.insert('insert into user (username,password,email) values (?,?,?)'<EMAIL_ADDRESS>
exception info
Traceback (most recent call last):
  File "/Users/fangzheng/Library/Python/3.7/lib/python/site-packages/orator/connections/connection.py", line 31, in _run
    result = wrapped(self, query, bindings, *args, **kwargs)
  File "/Users/fangzheng/Library/Python/3.7/lib/python/site-packages/orator/connections/connection.py", line 251, in statement
    return self._new_cursor().execute(query, bindings)
  File "/Users/fangzheng/Library/Python/3.7/lib/python/site-packages/pymysql/cursors.py", line 168, in execute
    query = self.mogrify(query, args)
  File "/Users/fangzheng/Library/Python/3.7/lib/python/site-packages/pymysql/cursors.py", line 147, in mogrify
    query = query % self._escape_args(args, conn)
TypeError: not all arguments converted during string formatting

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/fangzheng/PycharmProjects/pydemo/ORM/orator/test.py", line 39, in <module>
    db.insert('insert into user (username,password,email) values (?,?,?)'<EMAIL_ADDRESS>  File "/Users/fangzheng/Library/Python/3.7/lib/python/site-packages/orator/connections/connection.py", line 236, in insert
    return self.statement(query, bindings)
  File "/Users/fangzheng/Library/Python/3.7/lib/python/site-packages/orator/connections/connection.py", line 34, in _run
    e, query, bindings, wrapped
  File "/Users/fangzheng/Library/Python/3.7/lib/python/site-packages/orator/connections/connection.py", line 341, in _try_again_if_caused_by_lost_connection
    raise QueryException(query, bindings, e)
orator.exceptions.query.QueryException: not all arguments converted during string formatting (SQL: insert into user (username,password,email) values (?,?,?) (['John', 'John', 'John@qq.com']))

Process finished with exit code 1

You only need to specify the table and the arguments: https://orator-orm.com/docs/0.9/query_builder.html#inserts
# With keywords arguments
<EMAIL_ADDRESS>votes=0)

# Or with dictionary
db.table('users').insert({
    'email'<EMAIL_ADDRESS>    'votes': 0
})


fix Schema.parse to new Schema.Parser().parse

minor modify info:

Schema.parse() deprecated function to  new Schema.Parser().parse()
Extract method from duplicated code.

@ambition119 Could you please rebase and resubmit this PR ?

@ambition119 Could you please rebase and resubmit this PR ?

done!


Select Field : is it possible to reload via ajax on editTemplate?

Is it possible to reload select field items on it field self when editTemplate is called?
I try to do like this but not running
sorry my English is not good
 {title:"Category",type:"select", name:"prodCatId", items:dataCategory,valueField:"prodCatId",textField:"prodCatDesc",
  editTemplate : function(){
   var sbuField = this._grid.fields[0];
   var $editControl = jsGrid.fields.select.prototype.editTemplate.call(this,arguments);

    $editControl.on("change", function(){
     var selectedSbuId = sbuField.editControl[0].value;
     var selectedprodCatId = $(this).val();
     $.ajax({
      type: "POST",
       url: "./getDataCategoryList",
       data:{"sbuId":selectedSbuId,"prodCatId":selectedprodCatId},
       success: function(data){
        dataCategory =  data
       },
       error : function(error){alert('error')}
      });
    });
    return $editControl;
 },
}

Hi,
Use select2 instead, it has ajax loading option.

@carlozSantiago, I know it's not a direct answer to your question, but have you checked out these issues: #46 #71?

@tabalinas , @heiovsky
I have try it,
but would only work to change items for other field (Dependency Dropdown),
not working when i want change or reload items for select itself,
i just want to change items for select itself not items for other field.

{title:"Category",type:"select", name:"prodCatId", items:dataCategory,valueField:"prodCatId",textField:"prodCatDesc",
editTemplate : function(){
var sbuField = this._grid.fields[0];
var $editControl = jsGrid.fields.select.prototype.editTemplate.call(this,arguments);
$editControl.on("change", function(){
 var selectedSbuId = sbuField.editControl[0].value;
 var selectedprodCatId = $(this).val();
 $.ajax({
  type: "POST",
   url: "./getDataCategoryList",
   data:{"sbuId":selectedSbuId,"prodCatId":selectedprodCatId},
   success: function(data){
   $("#jsGrid").jsGrid("fieldOption", "prodCatId", "items", data)     <-----------------------------
   },
   error : function(error){alert('error')}
  });
});
return $editControl;

},
}
hope it helps

@heiovsky , The code make table reload , i want it no reload ~

Can you explain your case? Why you need this behavior?

I have 3 dependency dropdown,
categoryA, categoryB, categoryC.
the items of categoryC(items:dataCategory) depends on what i chose in the categoryA and categoryB,
i use ajax to hit server and get list, it works and items for categoryC is change,
The problem when i chose dropdown on the other row into editing mode where the value(valueField:"prodCatId") of the dropdown was not found in list(dataCategory) which i got before, so the dropdown showing empty,
ex:

i try to reload just items (items:dataCategory) on $editControl, i use ajax again to hit server . In the hope of  items(items:dataCategory) will be load again based on value of categoryA and categoryB in that row...

found this fiddle http://jsfiddle.net/tabalinas/31cug9sa/

@heiovsky , yes
Right exactly like that

select country canada
select state A2
add

so we have 1 record, and add new record

select country United States
select state A1
add

now we hae 2 different record, then click on Canada record, state field is empty. .
how do i showed the data and choice ??

I have no idea at the moment, I think, I will face this issue in couple of weeks and maybe then I find a solution

I have similar type of issue when using select dropdowns please share if you have any solution


Merge two layers, But backprop only through one

Let's say I have code to merge two layers, like this :
left = Sequential()
left.add(Dense(784, 50))
left.add(Dense(50, 50))
left.add(Activation('relu'))

right = Sequential()
right.add(Dense(784, 50))
right.add(Dense(50, 50))
right.add(Activation('relu'))

model = Sequential()
model.add(Merge([left, right], mode='concat'))

model.add(Dense(100, 10))
model.add(Activation('softmax'))

Now, it's apparent that concat(left,right) is used to generate the softmax output. But while training, I want to network to treat "left" as a constant , and not backprop through it, while "right" can be "backproped" through. Is this possible ?

Just record the output of left (in test mode, i.e. using .predict()) then use this as the input to your 2-branch model. Do not include into your model layers that you don't intend on training.

I want to do the same experiment as @superhans , but I'm not quite understand  your ideas @fchollet   Can you explain it with more specific details.
Thanks.

right.trainable = False

@fchollet Wouldn't this work?

@farizrahman4u Thanks very much.


[ANDROID] Crushes at initSdk

Report
Plugin Version
v5.4.3 (Latest on android)
Platform
android
What did you do?
Run gradleW assembleRelease,
Run gradleW assembleDebug
What did you expect to happen?
Run as expected.
What happened instead?
crushed at the start of the init .
10-07 17:28:23.774 26354 26400 E AndroidRuntime: com.facebook.react.common.JavascriptException: TypeError: t.initSdk is not a function. (In 't.initSdk(n,o,u)', 't.initSdk' is undefined)
Please provide any other relevant information.
ℹ Please replace these two lines with more information.
The issue started when we upgraded the plugin to the latest version

Problem has fixed by this steps:
1.yarn remove react-native-appsflyer
2.remove node module.
3.yarn add react-native-appsflyer


TTY device names under /dev for USART6 and RS-485

What are try device names under /dev for USART6 and RS-485?
can that be added to the documentation somewhere?
thanks

Thank you very much for your valuable feedback. The document team has been notified to add descriptions of the relevant pins in Linux. Please stay tuned for updates on the CoreMP135 official website.

The request wasn’t about pins.
it was about device names on Debian image.
/dev/tty???
How usart6 and rs485 are named under
/dev/  filesystem?
thanks

yes,i see.The CoreMP135 document has been updated.

Ok.
So it’s
/dev/ttySTM0 for usart6
and
/dev/ttySTM3 for Rs-485
thanks


[AzureRM] The version '5.8.3' of the module 'AzureRM.Profile' being installed is not catalog signed.

Trying to install AzureRM.Profile version 5.8.4  and 5.8.3 side by side results in the following error. It can be reproed with the following two liners.
PS C:> Install-Module AzureRM.Profile
PS C:> Install-Module AzureRM.Profile -RequiredVersion 5.8.3
PackageManagement\Install-Package : The version '5.8.3' of the module 'AzureRM.Profile' being installed is not catalog signed. Ensure that the version '5.8.3' of the module
'AzureRM.Profile' has the catalog file 'AzureRM.Profile.cat' and signed with the same publisher 'CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US' as the
previously-installed module 'AzureRM.Profile' with version '5.8.4' under the directory 'C:\Program Files\WindowsPowerShell\Modules\AzureRM.profile\5.8.4'. If you still want to install or
update, use -SkipPublisherCheck parameter.
At C:\Program Files\WindowsPowerShell\Modules\PowerShellGet\2.2.5\PSModule.psm1:9711 char:34

... talledPackages = PackageManagement\Install-Package @PSBoundParameters

                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (Microsoft.Power....InstallPackage:InstallPackage) [Install-Package], Exception
FullyQualifiedErrorId : ModuleIsNotCatalogSigned,Validate-ModuleAuthenticodeSignature,Microsoft.PowerShell.PackageManagement.Cmdlets.InstallPackage

Some of our customers are hitting this issue as our AzureRm version 2.5.0 is taking a >= dependency on AzureRm.Profile 5.8.3 and our AzureStack module 1.8.2 is taking = RequiredVersion dependency on 5.8.3.

Released AzureStack module version 1.8.3 to fix the issue


Wrong method used in Python SDK examples when getting planner plan by plan id

In the below documentation page "by_plan_id" is used as the method to get a planner plan by plan id.
However this method doesn't exist. The proper method should be using "by_planner_plan_id".
So the code example should be changed from
# THE PYTHON SDK IS IN PREVIEW. FOR NON-PRODUCTION USE ONLY

graph_client = GraphServiceClient(request_adapter)

result = await graph_client.planner.plans.by_plan_id('plannerPlan-id').buckets.get()

To:
# THE PYTHON SDK IS IN PREVIEW. FOR NON-PRODUCTION USE ONLY

graph_client = GraphServiceClient(request_adapter)

result = await graph_client.planner.plans.by_planner_plan_id('plannerPlan-id').buckets.get()

Instead.
If I read the contribution guidelines correctly these examples are auto generated (python wasn't explicitly mentioned but I assume python is included), so I haven't changed it myself.
I assume the same mistake might be happening in other auto generated samples using this method.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 41fcd104-c491-d371-6417-bb46c9445d7c
Version Independent ID: 778d4ad7-fb16-660c-f7b8-dea16652f42f
Content: List buckets - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/plannerplan-list-buckets.md
Product: planner
Technology: microsoft-graph
GitHub Login: @TarkanSevilmis
Microsoft Alias: MSGraphDocsVteam

Closing as this seems to be fixed now. Thanks.


Langchain-Community LLM TextGen has wrong API endpoint

System Info
Windows 10 & Ubuntu 22.04
langchain==0.0.354
langchain-community==0.0.8
langchain-core==0.1.5
Python 3.10.13
https://github.com/langchain-ai/langchain/blob/master/libs/community/langchain_community/llms/textgen.py
Who can help?
@agola11 @hwchase17
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
With the latest TextGen WebUI install the API endpoints are openAI
Lanchain has the endpoints:
Example Line 213
url = f"{self.model_url}/api/v1/generate"
when used this returns a 404.
What works is
url = f"{self.model_url}/v1/chat/completions"
A fix was attempted:  https://gist.github.com/ddomen/8eaa49879d42a4a42a243437b5ddfa83
It works for me if I set  legacy_api=False,
but his truncates responses to about 20 or so characters.
Expected behavior
certianly not a 404.
My app that is using langchain was working with an install of TextGen fomr like amonth ago.  I went to deploy in a new environment and pulled the latest TextGen and Langchain stopped working.  When I dug  into the problem I saw they now force an OpenAI API interface.

I didn't write the referenced code.  This was another persons suggested work around.  When I use it, the code connects to the TextGen instance, however the responses are truncated.   It seems as if there is more to it than just changing the URL.  I think the interaction between the code and the server needs to be handled differently and thats not someinth I could sort out.
This still needs to be looked at.


wip k3s support

Very very draft k3s support. This adds a distro setting to the test, allowing you to swap between nixos and k3s. k3s is mostly working, however there are a few issues:

k3s allows you to pass in--docker and use docker as the container runtime. I was unable to get this working, when docker would go to run an image it would print an error about to many symbolic link layers. This is now working.
I do not know how the local dns should work, so I have been unable to get the curl request to nginx to actually succeed. This is now working.

I think other than that the bones of k3s is working. Happy to work on this more to get it up to snuff.

Containerd worked great, I didn't want to break compatibility with any of your existing tests that were using docker commands though. Swapping back to containerd is easy, just have to remove --docker and switch over to the containerd commands for loading images.
Kubetest and telepresence look interesting, I don't think I have used kubenix enough yet to give an educated opinion though. I know what I really love about how things currently work is not having to worry about any state carrying over between test runs. It has made learning kubernetes so much easier.

The more I use the current testing framework, the more frustrated I become with it. The current issue I am having, is k3s does not include all images needed to use local-path storage classes in its airgap image set. This took a while to discover though, because NixOS testing library disables the airgap firewall rules when you are using .driver to run the tests with a debug repl. It feels like for a pleasant user experience, first class support for running tests on a real kubernetes cluster is critical. Otherwise there will be a constant friction between the kubernetes distro you use for testing, and the kubernetes distro you are deploying to. This is probably just reiterating thoughts you have already had yourself.

I'd love to see this move forward as k3s is (not only) my go to choice for k8s deployments.

I am not working on this actively, as I use other tooling for kubernetes deployments currently (pukumi), but I am happy to review and merge if  some has interest in this project

I am not working on this actively, as I use other tooling for kubernetes deployments currently (pukumi), but I am happy to review and merge if some has interest in this project

I see, would you be open to cede maintainership / ownership on kubenix to somebody willing to evolve this further?

Yes, I would be very open for that, and I would would be very happy someone continues efforts 🙂
I would be also open to discuss ideas, as I have some experience building and refactoring this project and different issues I had.

@offlinehacker did you mean https://www.pulumi.com? May I ask what was the reason for choosing it over kubenix? Thanks!

Btw I’d be happy to work on developing kubenix as I use it at work and should be able to commit a significant amount of time to it. Happy to hop on a call to discuss this further.

@offlinehacker I'm also curious if you're using Pulumi + Nix in any novel way, or just regular Pulumi+TS to manage resources?
(re: kubenix: I've always been on the edge of adopting kubenix but wasn't sure if it had other (non-@offlinehacker) users or a future. Knowing others are interested, even just this much, helps alleviate some of that fear.

I'm going to go all in on kubenix. It is "brilliant" (quote zimbatm). A generalized nix (and later nickel) DSL is strategically superior to (special purpose) pulumi ts based "DSL", I think.

Btw I’d be happy to work on developing kubenix as I use it at work and should be able to commit a significant amount of time to it. Happy to hop on a call to discuss this further.

@offlinehacker You could just transfer the repo to @adrian-gierakowski (Adrian would need to rename his current fork first). Would that be something? I'm currently very engaged in the offline world, but you can definitely expect input similar to what I'm currently doing on divnix/devos.

@offlinehacker Does this operator run within the cluster? I think anything that goes into the direction of an operator is out-of-reach for a (declarative) configuration language, while deploying that operator would probably be in-scope.

@blaggacao I've renamed my fork, but maybe it would be better to create a kubenix org? Unfortunately the name seems to be taken. @offlinehacker have you created that org? Btw. I'd be able to start working on this next week. Shall we arrange a call to discuss the direction in which we'd like to take this project?

Shall we arrange a call to discuss the direction in which we'd like to take this project?

Plese set a time that is ok for you, I can adapt completely.
Click the following link to join the meeting:
https://meet.jit.si/kubenix
=====
Click this link to see the dial in phone numbers for this meeting:
https://meet.jit.si/static/dialInInfo.html?room=kubenix

Maybe it could go under nix-community?

Maybe it could go under nix-community?

sounds ok, what do you think @offlinehacker

Shall we arrange a call to discuss the direction in which we'd like to take this project?

Plese set a time that is ok for you, I can adapt completely.

I very flexible as well. @offlinehacker do you think you'd be able to find some time for this anytime this or next week? If so, I'd defer to you regarding picking the time for the meeting.

https://nix-community.org/#how-do-i-get-my-project-included

@offlinehacker I’m planning to dedicate 2-3 days next week to working on kubenix and I think it would be really helpful to get some input from you before kicking off. Do you think we could have a chat sometime next Tuesday/Wednesday? Thanks!

@offlinehacker I understand that you might be busy, so it would be great if you could at least give your blessing for me to post a message on NixOS Discourse announcing that I’m going to work on developing kubenix and asking for feedback from the community regarding the roadmap.
I also really don’t mind in which repo the development continues. Nix-community or kubenix gh org seem preferable, but I’d be just as happy for the project to stay where it is, as long as I’m added as a maintainer.
Thanks!

I combined this with #27: https://github.com/xtruder/kubenix/pull/29

@xavierzwirtz What was the original reason not to use nixpkgs' services.k3s? Didn't exist at the time of writing?

Been awhile, I don't remember a good reason either way.
On Wed, May 5, 2021, at 8:39 PM, David Arnold wrote:

@xavierzwirtz https://github.com/xavierzwirtz What was the original reason not to use nixpkgs' services.k3s https://github.com/NixOS/nixpkgs/blob/nixos-unstable/nixos/modules/services/cluster/k3s/default.nix? Didn't exist at the time of writing?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/xtruder/kubenix/pull/18#issuecomment-833162211, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADTZZ25BKB3HO2QXVLZABTTMHXOLANCNFSM4KM3MOGQ.

This repo has been deprecated, since I stopped maintaining it some time ago. There is a fork maintained by @hall available at https://github.com/hall/kubenix, that has better documentation and looks like a way further.


React v15 upgrade branch using moment().format() would fail BDD test on Travis CI

Description of the Issue and Steps to Reproduce:
I previously reported this issue in the moment channel on Gitter. Although I've found a workaround, I still wish to understand the real cause of the issue.
When upgrading ReactJS from v0.14 to v15 in our app, one particular Python BDD browser test had consistently failed on Travis CI (in both Chrome and Firefox), while this very same BDD test passed both locally and directly piped to Sauce Labs.
The assertion was to test a JavaScript functionality by passing a string value in a  element and expect it to invoke the onChange event handler. This assertion had no failures on Travis CI prior to the ReactJS v15 upgrade branch.
Eventually I’d found that the failing test was related to moment (well, together with React v15 and Travis CI). The instances of moment(utc_date_string).format(‘YYYY MMM DD’) appeared to be causing JavaScript to fail on Travis CI. Another unique thing about the failing test is that it only happened on pages in which moment().format() is used in a loop within the render() method, similar to the following:
render() {
  return (

      {items.map(item => {

{item.status} - {moment(item.timestamp).format(‘YYYY MMM DD’)}

      })}

  );
}

We've had the code (similar to the above) in our app all along to display the local time in the UI, and it hasn't been an issue until now with the React v15 upgrade.
Purely for debugging attempts, the following would allow the test to pass on Travis CI:

Removed the instances of moment(item.timestamp).format(‘YYYY MMM DD’) in the loop.
Added utc() into moment(item.timestamp).format(‘YYYY MMM DD’), such as moment(item.timestamp).utc().format(‘YYYY MMM DD’).
Changed moment(item.timestamp).format(‘YYYY MMM DD’) to JSON.stringify(moment(item.timestamp)).

The workaround
I found this stackoverflow post (http://stackoverflow.com/questions/38251763/moment-js-to-convert-date-string-into-date) and had tried passing the date string into the new Date(utc_date_string) first and the passed it to moment(), such as the following:
render() {
  return (

      {items.map(item => {
        let date = new Date(item.timestamp);

{item.status} - {moment(date).format(‘YYYY MMM DD’)}

      })}

  );
}

Unfortunately, the above attempt alone still didn't allow the BDD test to pass on Travis CI. So eventually, I took the date parsing and formatting out of the loop in the render() method and put them into a function which returns a JSON object for the render() method to consume. With that, the BDD test passed on Travis CI.
Environment:
React 15.4.2, moment 2.18, Node.JS 6.10.1, Python 3.4.2, Chrome 48, Chrome Driver 2.21, Linux 3.13, Ubuntu 12.04
Other information:

Travis CI server runs on UTC.
Example date string to be passed into moment(): 2015-05-04T08:11:18.000000+00:00

Is the timestamp you are dealing with in ISO8601 format? If it isn't, then you could simply be getting different results for your formatted time depending on the browser.

Yes, the timestamp I am dealing with is in ISO 8601 format (see the Example date string I provided above under Other information).
The issue I was having problem with has nothing to do with the results of time formatting (if I am not mistaken). It has to do with calling the moment(...).format(...) in a loop when rendering the DOM. The result was seemingly some script error that prevented the event handler from being invoked in a BDD test on Travis CI.

This is hard to debug from where we're sitting. If I understand correctly, you're not actually suggesting that this...
moment('2015-05-04T08:11:18.000000+00:00').format('YYYY MMM DD');

...returns the wrong string in your tests. That makes sense, because Moment does not know that it's being run by React, not to mention by a specific version being run by Travis. I think you're saying that there's some other, unspecified (or perhaps invisible?) error.
Now, I could guess: maybe React sets a timeout for each iteration of its loop and Travis is kind of slow, for example. But I don't know how I could do better than guess, because I don't have any special knowledge of React and because Moment is a comparatively simple library that is--as far as anyone can tell--not actually doing anything unexpected here.
I know nebulous things-aren't-working errors are frustrating, especially from fickle CI tools, but I don't see if how there's a path forward from Moment's end. If you do find a way in which Moment is misbehaving, I will be eager to know about it and help.


[FEATURE REQ] Change feed processor should accept a publisher of JsonNodes instead of consumer of JsonNodes to be reactive in true sense

Is your feature request related to a problem? Please describe.
I am trying to integrate cosmos CFP in a java based spring webflux microservice. But on starting the coding, I found that handleChanges() method expects a Consumer. I feel that in order to be reactive, this method should expect a Publisher from the caller. Otherwise, how does a caller even know that he/she is supposed to write reactive code here?
Or am I seeing it completely in a wrong way and I am not supposed to write reactive code inside handleChanges() method?

handleChanges is not meant to be reactive - successful completion of the handleChanges invocation is used as a signal that the changes were successfully processed - and result in bookmark making progress. The model you mentioned above wouldn't allow tracking the bookmarking this way. For custom handling of change feed processing you can always move to the pull-model (CosmosAsyncContainer.queryChangeFeed) if the ChangeFeedProcessor abstraction doesn't meet your requirements.

@FabianMeiswinkel but there are methods available in project reactor like then() where u can wait for a previous publisher to finish and then proceed to next publisher. Also, on a more simpler level, u can expect a Publisher from caller and put a block() inside the library code. By this, you can achieve two objectives -:

U can change the library code tomorrow to pure reactive and caller will not have to change his/her code.
It will be clear to caller that they should use reactive programming.


WIP -- specialization constant support

Patches 1-5 do some housekeeping in the test framework in order to make writing this test easier.
Patches 6-7 are adding new test cases (which currently fail).
Still to come: actual implementation to make the tests pass.

@tobine perhaps. It's easy to add one.

Note there's still some issues here:

We're not using the optimized module yet.
Need the SPIRV-Tools team to fix their opt pass (currently doing string parsing that we don't want)
cmake needs some more work

@tobine What do you think about landing 1-5 now, and the rest later?

@chrisforbes Yes, landing 1-5 asap makes sense. Spin off into separate PR and run through our farm to be safe. Assuming clean it has my +1.

cmake for this is no good. I need to look at it a bit more.

Can anything be done with the work here, or should we just archive this for now?

@chrisforbes,  this repository will close for write access on Sunday, 5/13/2018.  If it is pushed before that time it will be present in the follow-on Vulkan-ValidationLayers repository on Monday, otherwise a new PR will be required in the new repo.


Support CMap-data with only strings, when parsing TrueType composite fonts (bug 920426)

In the referenced bug, the embedded fonts contain custom CMap-data that only include strings. Note how for embedded composite TrueType fonts we're using the CMap-data when building the glyph mapping, and currently we end up with a completely empty map because the code expects only CID numbers.
Furthermore, just fixing the glyph mapping alone isn't sufficient to fully address the bug, since we also need to consider this "special" kind of CMap-data when looking up glyph widths.
Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=920426

/botio-linux preview

/botio test

/botio browsertest

/botio-linux browsertest

/botio-linux preview

I think this may be a bug in the cmap parser. Looking at the spec on beginbfchar, it says the second value can be either a "code or name".
Here we could instead check if the first char of the string is a / if it isn't then convert it to an int.

Nevermind about checking the /. If that's the case the lexer will return a name obj.

One thing that complicates all of this somewhat is that depending on how/where we're invoking the CMapFactory in our existing code, we actually expect the returned data to look subtly different:

In the src/core/fonts.js file, when accessing the cMap-data, the surrounding code always expect to get a CID number in return. The situation fixed in this patch should thus be very rare, since this is the first case we know of.
In the PartialEvaluator.readToUnicode-method, please see this code in particular, we're actually (by default) expecting that the CMapFactory will return string data. (As mentioned in a comment there, I recently added a code-path to handle essentially the "inverse" of this bug to fix another older issue.)

My initial thought when looking at both this bug, and issue 9367, was indeed that we should perhaps (somehow) change the CMapFactory to fix these bugs. However, since I noticed that we have different expectations for the return values depending on how/where we're using that code, it wasn't at all clear (to me) how to reconcile those two cases in the CMapFactory directly.
Also, I should probably add that the situation fixed here seems extremely rare in practice, since we've seen a total of two cases where the existing assumptions regarding numbers/strings for the CMap-data doesn't hold.
@brendandahl Given the above, how should we proceed here?

I'd lean towards an integer and then convert that to a string if we need to for the unicode stuff.

For the PartialEvaluator.readToUnicode case, wouldn't that mean that we'd need to essentially convert the data twice then?
Also, looking at this code, how would you even get the CMap.forEach-method to return one integer value for the string-case?
I might be missing something obvious here, but I really don't understand how you'd actually implement that.

Yeah, forgot about how the ToUnicode cMaps work. For regular font ones the charcode should always fit into a 32bit int. What about always returning a string, that conversion should be pretty fast. Or alternatively differrent forEach methods for strings and ints?

Or alternatively differrent forEach methods for strings and ints?

I've tried to implement this, which also required some changes to the lookup-methods as well, and while it seem to work locally (all tests pass) it does feel a little clunky. @brendandahl Feedback please :-)

/botio test

/botio-linux browsertest

/botio-linux browsertest

/botio makeref

/botio-linux makeref


Where are the mainnet launch instructions?

Where are the mainnet launch instructions?
I'm looking for documentation but can't find it.
I would like an example like this.
https://github.com/flare-foundation/flare#connecting-to-coston
https://github.com/flare-foundation/flare#connecting-to-songbird

I'm looking for it too.


Fix ts0601_rcbo.py after #1999

This PR is to fix the quirk for Tuya ts0601 _TZE200_hkdl5fmv circuit breaker for the changes made in #1999
See #1991 for more details.

There's a more clean way to do that, please hold on

Codecov Report
Base: 83.09% // Head: 83.53% // Increases project coverage by +0.44% :tada:

Coverage data is based on head (11d8e3f) compared to base (792d7e2).
Patch coverage: 98.72% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2009      +/-   ##
==========================================
+ Coverage   83.09%   83.53%   +0.44%
==========================================
  Files         248      249       +1
  Lines        7860     8016     +156
==========================================
+ Hits         6531     6696     +165
+ Misses       1329     1320       -9

Impacted Files
Coverage Δ

zhaquirks/tuya/ts0601_rcbo.py
98.62% <98.62%> (ø)

zhaquirks/heiman/smoke.py
100.00% <100.00%> (ø)

zhaquirks/tuya/mcu/__init__.py
99.51% <100.00%> (+2.46%)
:arrow_up:

zhaquirks/tuya/ts001x.py
100.00% <100.00%> (ø)

zhaquirks/tuya/ts0121_plug.py
100.00% <100.00%> (ø)

zhaquirks/tuya/__init__.py
72.94% <0.00%> (+1.04%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

The code doesn't seem like it's going to break anything implemented, so it looks good to me.
The only thing to comment would be that this part here:
t.uint32_t.deserialize(x.serialize()[::-1])[0]

seems to me to be the same as what this code does (with a lot of validations):
t.LVBytes.deserialize(val)[0]

So not sure if in some way that would be simplified, but if code works for you is fine to me.
I suggest that we release the current version 'as is' and if you want we can comment on the rest and assess if another PR is necessary.


Request for Cupertino-flavored-Segmented-Control

It would be nice for my app if flutter provides a Cupertino-flavored Segmented Control like this one on the Apple Developper page https://developer.apple.com/ios/human-interface-guidelines/controls/segmented-controls/
I hope you can make this possible.
Thanks!

Please +1 to #7509


Remove patches from dotnet/templating

Remove all from https://github.com/dotnet/source-build/tree/dev/release/2.0/patches/templating

This was completed for 2.1 with #406


Implement analyzers for StyleCop+ rules (SPxxxx)

It would be nice if you could add analyzers for StyleCop+ rules (SPxxxx). You can find the source code here on github: https://github.com/shuruev/StyleCopPlus.

It will be difficult to implement these rules as described on the StyleCopPlus page, because Roslyn analyzers are not easily configured.
Each of these rules would need to be submitted as a new rule proposal, and then discussed to determine if we want to add it. Note that some of the rules have already been proposed, and one of them even has a pull request with an implementation under review. As of right now, I believe the status is as follows:

SP2000 (CodeLineMustNotEndWithWhitespace): #729
SP2001 (CheckAllowedIndentationCharacters): #33, #730
SP2002 (CheckWhetherLastCodeLineIsEmpty): No proposal
SP2100 (CodeLineMustNotBeLongerThan): No proposal
SP2101 (MethodMustNotContainMoreLinesThan): No proposal
SP2102 (PropertyMustNotContainMoreLinesThan): No proposal
SP2103 (FileMustNotContainMoreLinesThan): No proposal

I'm marking this as a closed question regarding the proposal process. Please feel free to comment on existing issues or submit new proposals if there are any of these you particularly want to see included.

Thanks for the clarification. We also use StyleCop+ StyleCopPlus.dll http://stylecopplus.codeplex.com/  I'm not sure which rules though

For anyone else looking for StyleCop+ rules, I missed them enough that I wrote analyzer equivalents for several of them in my new [Menees.Analyzers] (http://www.menees.com/Analyzers.htm) assembly.  It has rules similar to SP2001 (require tabs), SP2100 (long lines), SP2101 (long methods), SP2102 (long properties), and SP2103 (long files).

@bmenees, what was your reason for creating a separate analyser to StyleCopAnalyzers, rather than including them here?
@sharwell, your position on StyleCop+ seems a little unclear. For me at least (and I'm sure many others), StyleCop+ transformed StyleCop and it's vital that all its rules be implemented to make StyleCopAnalyzers a viable replacement tool. Here you seem to suggest that we can propose those rules. Elsewhere (https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/422#issuecomment-128703160) you state that StyleCop+ rules won't be implemented, even if they are proposed. Also I'm puzzled by the idea that proposed rules like this need to be approved. Are you saying that if, for example, I implement SP2103 (files must not be more than n lines long) and submit a pull request, then it could be rejected simply because you or other existing contributors don't like it?

your position on StyleCop+ seems a little unclear

The only rules "grandfathered in" to this project are the ones that existed in StyleCop "classic" at the time this project was started. All other rules, regardless of origin, need to be proposed and then accepted before an implementation will be merged in. While we don't have an official documented procedure for accepting new rules, I can say that to date, no rules have been accepted that had an active objection from any code contributor to this project. In addition, rules which clearly worked well with another existing rule (e.g. #653 clearly works well with other rules regarding the order of using directives) needed at least two :+1: votes, and rules which represented new styles (e.g. #1451) tended to take three.

Elsewhere (#422 (comment)) you state that StyleCop+ rules won't be implemented, even if they are proposed.

A more accurate statement is the rules would be unlikely to be accepted. Multiple contributors to this project have expressed interest in avoiding rules that do not have clear benefits under a fixed configuration. We already permanently disabled 5 StyleCop "classic" rules for this reason (#1057).

Are you saying that if, for example, I implement SP2103 (files must not be more than n lines long) and submit a pull request, then it could be rejected simply because you or other existing contributors don't like it?

This is correct. And it's already happened for at least three of my own proposals which included fully-tested, working implementations along with code fixes. But the beauty of Roslyn is you can always create an analyzer that implements just SP2103 and use it along with StyleCop.Analyzers for a complete solution that works for you. :smile:

I created my own set of rules because I know how to write them, and I know exactly what I want them to do.  Some of them conflict with existing StyleCop.Analyzers rules, and they're all coded in a different style (e.g., using tabs and using #regions).  I like my rules the way I write them, and I didn't want to have to debate them with others.
The beauty of analyzers and rulesets is that it's not an all or nothing proposition with an analyzer library.  I'm currently using 100% of the Menees.Analyzers rules, 80% of the StyleCop.Analyzers rules, and about 70% of the Code-Cracker rules.

@bmenees is it possible to use your Menees.Analyzers in dotnet core projects?

Probably not.  I don't use .NET Core at this point, and there's one place where I used Registry access in Menees.Analyzers to look up a setting.  So that required a reference to the .NET Framework.

ok, got it, thanks for feedback. Just wondering why file max length and method max length still not implemented here. Because they are rather important one. So at least pull request for these rules should be acceptable.


[DOC] Search request slow logs

Description
Documentation for new search request slow log feature. Target version is 2.12.
OpenSearch PR: https://github.com/opensearch-project/OpenSearch/pull/10650
GitHub Issue: https://github.com/opensearch-project/OpenSearch/issues/9642
Checklist

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

@natebower @kolchfa-aws I rebased this PR. Can you please take a look again? The dev PR is now merged so documentation is good to go

@dzane17 Looks good. Thanks!


The dependency for json-path creates duplicate libraries of asm packages

The dependency for json-path (see below) causes after 'mvn eclipse:eclipse' two libraries with the asm packages (accessors-smart-1.1.jar and asm-5.0.3.jar). Both are providing the packages org.objectweb.asm and org.objectweb.asm.signature.
So it is impossible to make a runnable JAR file from Eclipse (but maybe I am unexperienced and there is a workaround for it, I don't know).
<dependency>
    <groupId>com.jayway.jsonpath</groupId>
    <artifactId>json-path</artifactId>
    <version>2.2.0</version>
</dependency>

A similar issue is posted in the issue management of json-smart-v2: https://github.com/netplex/json-smart-v2/issues/20

Check out the comments here #224

Dear kallestenflo, thanks for the hint. I will try out to exclude it from my Maven pom.
    <dependency>
        <groupId>com.jayway.jsonpath</groupId>
        <artifactId>json-path</artifactId>
        <version>2.2.0</version>
        <exclusions>
            <exclusion>
                <groupId>org.ow2.asm</groupId>
                <artifactId>asm</artifactId>
            </exclusion>
        </exclusions>
    </dependency>

The exclusion within the jsonpath dependency worked out for myself.
You should definitely exclude this library from your build environment or update it to accessors-smart 1.1.
See https://github.com/netplex/json-smart-v2/issues/20.


Add env. vars. for CGW push notifications

Summary
In accordance with the implementation of Firebase to the CGW, this adds the required env. vars. for push notifications.
Changes

Add example env. vars.:

PUSH_NOTIFICATIONS_API_BASE_URI (defaulted)
PUSH_NOTIFICATIONS_API_PROJECT
PUSH_NOTIFICATIONS_API_SERVICE_ACCOUNT_CLIENT_EMAIL
PUSH_NOTIFICATIONS_API_SERVICE_ACCOUNT_PRIVATE_KEY

I have read the CLA Document and I hereby sign the CLA


Auto corrected by following Lint Ruby Parentheses

Auto corrected by following Lint Ruby Parentheses
Click here to configure it on awesomecode.io

Coverage remained the same at 96.13% when pulling 9d3fde70a82534ac3d3627d97982a14dbca648ae on awesomecode-lint-ruby-parentheses-47815 into 99aa892c07cfc221480991741f56e2ccdf4fb464 on master.

Coverage remained the same at 96.13% when pulling 9d3fde70a82534ac3d3627d97982a14dbca648ae on awesomecode-lint-ruby-parentheses-47815 into 99aa892c07cfc221480991741f56e2ccdf4fb464 on master.


Payment method nonce represents an un-vaulted payment instrument

General information

java pom: 3.12.0
Sandbox

Issue description
in sandbox create subscription failed
Payment method nonce represents an un-vaulted payment instrument

👋  Subscriptions have to be created with a vaulted payment method token, there are only certain specific conditions that you can create a subscription with a payment method nonce. For more information see our developer docs.
If you have any additional questions, please contact Technical Support with timestamps and your sandbox merchant ID.


PDI-16032 - Clicking on Row Numbers in certain steps causes new ones …

…to be created

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -f ui/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  ui/src/org/pentaho/di/ui/core/widget/TableView.java


No 'Powerflow' or 'Current State'

Problem Description
New, clean installation, does not display anything for 'Current State' and the Powerflow animation is absent.
In Chrome, the Powerflow pane reports 'x.x.x.x refused to connect', in Safari it shows a looping progress bar.
I've tried the manual install and the auto-install. I am using the cloud connection rather than local (not clear if this could be the reason?)
verify.sh looks clean
I can't see port 8675 open on the machine at all - from what I could find, this is required for at leat the Powerflow - I presume also for the current state
Any help would be very much appreciated!
netstat -tl

Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 <IP_ADDRESS>:ssh             <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:9000            <IP_ADDRESS>:*               LISTEN
tcp        0      0 localhost:domain        <IP_ADDRESS>:*               LISTEN
tcp        0      0 localhost:36077         <IP_ADDRESS>:*               LISTEN
tcp        0      0 <IP_ADDRESS>:8086            <IP_ADDRESS>:*               LISTEN
tcp6       0      0 [::]:ssh                [::]:*                  LISTEN
tcp6       0      0 [::]:8581               [::]:*                  LISTEN
tcp6       0      0 [::]:51764              [::]:*                  LISTEN
tcp6       0      0 [::]:9000               [::]:*                  LISTEN
tcp6       0      0 [::]:9273               [::]:*                  LISTEN
tcp6       0      0 [::]:8086               [::]:*                  LISTEN

System Details
Describe the host system you are trying to use for the Dashboard:
Ubuntu 22.04.3 VM (under Proxmox) (4 CPU, 4GB) kernel 5.15.0-91-generic
Verify Tool Output
Verify Powerwall-Dashboard 3.0.8 on Linux - Timezone: Europe/London
----------------------------------------------------------------------------
This script will attempt to verify all the services needed to run
Powerwall-Dashboard. Use this output when you open an issue for help:
https://github.com/jasonacox/Powerwall-Dashboard/issues/new

Checking configuration
----------------------------------------------------------------------------
 - Dashboard configuration: solar-only
 - EnvVar COMPOSE_PROFILES: solar-only

Checking pypowerwall
----------------------------------------------------------------------------
 - Skipped: Only required in 'default' configuration

Checking telegraf
----------------------------------------------------------------------------
 - Skipped: Only required in 'default' configuration

Checking influxdb
----------------------------------------------------------------------------
 - Config File influxdb.conf: GOOD
 - Environment File influxdb.env: GOOD
 - Container (influxdb): GOOD
 - Service (port 8086): GOOD
 - Filesystem (./influxdb): GOOD
 - Version: InfluxDB shell version: 1.8.10

Checking grafana
----------------------------------------------------------------------------
 - Config File grafana.env: GOOD
 - Container (grafana): GOOD
 - Service (port 9000): GOOD
 - Filesystem (./grafana): GOOD
 - Version: Grafana CLI version 9.1.2

Checking tesla-history
----------------------------------------------------------------------------
 - Config File tools/tesla-history/tesla-history.conf: GOOD
 - Auth File tools/tesla-history/tesla-history.auth: GOOD
 - Container (tesla-history): GOOD
 - Version: 0.1.4

Checking weather411
----------------------------------------------------------------------------
 - Skipped: weather411 not set up

All tests succeeded.

Hi @llamakarma !
The installation method you picked is for Tesla Solar-Only. This current solar-only version does not use pypowerwall to pull the data so you will not see pypowerwall running on port 8675. That means that the power flow animation or live data meters will not show. There is a solar-only dashboard you can use that removes these panels.  However, there is now another option in development.
We have a new version of pypowerwall that @mcbirse and I are working on that will use pypowerwall for solar-only installs (also works for Powerwall 3 owners which do not get a local API).  Details are here: https://github.com/jasonacox/Powerwall-Dashboard/pull/414
What type of system do you have?  Solar-only, Powerwall 2/+ or Powerwall 3?  If you have a Powerwall, PW2 or PW+, you can use the "local mode" and I would recommend it. If you have Solar-only or. Powerwall 3, you will need to use cloud mode. If you want to try to new setup (beta mode), you can follow the directions here: https://github.com/jasonacox/Powerwall-Dashboard/pull/414#issuecomment-1882355993

Hi @llamakarma
Since you were starting with a clean install anyway, you may want to test a fresh install of the v4.0.0 pre-release. You can do so following the instructions here: https://github.com/jasonacox/Powerwall-Dashboard/pull/414#issuecomment-1897867180
Any feedback or issues you find can be posted in that thread and would be most welcome!

Hi both,
Thank you for your quick reply to what probably seems like a dumb question!
I misinterpreted the setup script:
Current: Tesla Cloud

 1 - Local Access (Powerwall 1, 2, or + using extended data from Tesla Gateway on LAN) - Default
 2 - Tesla Cloud  **(Solar-only, Powerwall 1, 2, +, or 3 using data from Tesla Cloud)**

It reads to me that option 2 is suitable for all options and my search-fu didn't come up with any other answers.
Anyway - my setup is PW2 with Backup Gateway2, my solar input is non-Tesla. I have just re-setup with the local only connection and things are working just fine - thank you!
I will take a look at v4.0 over the weekend.
Thanks again for the quick assistance

I'll flag this to update documentation and possibly the setup script.
We should probably switch to this to indicate that the 2nd option is only need if you don't have local access. Perhaps something like this:
1 - Local Access (Powerwall 1, 2, or + using the Tesla Gateway on LAN) - Default
2 - Tesla Cloud  (Solar-only systems or Powerwalls without LAN access)


Windows 11 Tweaks

[ ] Removing wallpaper doesn't stick after restart
[ ] Hybernation needs to be turned off powercfg /h off
[ ] Double check if Windows update checker and service get properly disabled (was set to manual)

Output when running the script the second time (service was set to manua)
Disable update checks? (y/n) y
Running script ./scripts/windows/disable_update_check.ps1
Disable update service? (y/n) y
Running script ./scripts/windows/disable_update_service.ps1
Successfully disabled the 'wuauserv' service on DESKTOP-FHU6E5S.
Successfully stopped the 'wuauserv' service on DESKTOP-FHU6E5S.

Could possibly be that the Windows Update Medic Service restores the Windows Update Service after a while? Need to investigate


Add introduction flow

intro screen explaining no data leaves phone

also include information about swipe actions
and concept about "x weeks buckets" (tell that it syncs back to phone)
matrix integration

Implemented in 1.2


MAINT: Add codecov.yml config file to reduce noise in PRs

Removes the "project" target from comments in PRs because the segregated
testing of niworkflows (masks, pytests, reportlets, ...) makes almost
all PRs to be under a decent project threshold (and hence show up as
failed).

Could you show an example of what you're trying to avoid?

Example #506 - this will hide the codecov/project check from the list and show up as building okay in github.

Oh, got it. Right, I forgot that we don't run all the tests on PRs.

This is fine by me.


Connect to mutliple projects using google BigQuery service account

Hi,
I'm trying to access multiple projects available in the provided google bigquery service account.
I'm able to access only the project we can see in bold. I can select the other available projects from the dropdown but still the query gets executed against the project that we see in bold.
Need help.
Thanks is advance.
Regards,
Revan

thanks for the report

@r1evan Hi, I tried to reproduce this issue, with a select query and it worked correctly when the full path to schema was written. Could you describe your issue in more detail?

Hi George,
But it does not work when we select the required database from the database list as shown in the below screenshot.
@.***
It still queries the default database given while defining the connection. See the below screenshot:
@.***
Can you help me a way of passing all the available projects.
Regards,
Revan
From: George @.>
Sent: 18 January 2022 18:04
To: dbeaver/dbeaver @.>
Cc: Revan Tadikonda @.>; Mention @.>
Subject: Re: [dbeaver/dbeaver] Connect to mutliple projects using google BigQuery service account (#13916)
@r1evanhttps://github.com/r1evan Hi, I tried to reproduce this issue, with a select query and it worked correctly when the full path to schema was written. Could you describe your issue in more detail?
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/13916#issuecomment-1015367769, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVUQ7ASRZT7IWRXDVU7O4KTUWVM2XANCNFSM5ECKQ67Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@r1evan  I think screenshots are failed to load correctly, I think you need to resend them.

Attached the screenshots as images
From: George @.>
Sent: 18 January 2022 18:29
To: dbeaver/dbeaver @.>
Cc: Revan Tadikonda @.>; Mention @.>
Subject: Re: [dbeaver/dbeaver] Connect to mutliple projects using google BigQuery service account (#13916)
@r1evanhttps://github.com/r1evan I think screenshots are failed to load correctly, I think you need to resend them.
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/13916#issuecomment-1015385781, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVUQ7ARTYVYJT522HURQSCLUWVPX7ANCNFSM5ECKQ67Q.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@r1evan, I think the Github email reply doesn't support attachments. At least I couldn't find a way to see them.

@r1evan If possible, can you send screenshots directly via Github?

Attached the images. Can you confirm if you're able to view them.

Yes, everything is good

@r1evan Can you tell, are you writing the full path to the schema in a query?

No, I'm just selecting the required schema from the dropdown.

@r1evan As I see, you have projects which are not added as additional projects and are still shown. I will look into it.

@r1evan I think if you pass needed projects as additional projects you will be able to query them.

I did it that way as well, but it's not working. Can you check and send me of how to add multiple projects??

@r1evan Sorry for waiting. Can you send your connection settings and driver properties?

Any location from where I can send you these?

And one more question. Have you tried to use the full table name in the SELECT statement? With project name + schema name + table name?
SELECT * FROM bigquery-public-data.austin_311.311_service_requests`` works in my case.

It does work in that case. But it's quite tedious to give fully qualified name for each table every time rite?

Not sure if there is any way we can connect over a call and I can show you the exact issue / requirement which I'm looking for.

@r1evan Sorry, this is not possible, here is information from driver documentation

For queries, tables in the projection list must be fully qualified, in the format of
catalog.schema.table. If the catalog is not specified, the driver will assume the
project specified by the projectId connection option

Great thanks!

But, when it provided the list of schemas available, there should be a way to run the queries by just selecting the schema from the dropdown. Should be taken as an enhancement. It really tough to give the fully qualified name for table every time.

@r1evan The only way to do this is by modifying the user query. This is bad practice due to the possibility of incorrect queries being formed.

verified


TypeError in authWithPassword() if server reponse is 308 "Permanent Redirect"

Hi everyone,
I tried to log in to a pocketbase server via the authWithPassword() method.
The firewall in front of the PB instance redirected me and I got a _TypeError (type 'String' is not a subtype of type 'Map<String, dynamic>?' in type cast) exception.
I tracked it down and found that the following cast is causing the issue:
https://github.com/pocketbase/dart-sdk/blob/d7c0c877587befce623608c899be657bd2f710e2/lib/src/services/record_service.dart#L226
This happens because client.send() can also return a string due to this line:
https://github.com/pocketbase/dart-sdk/blob/d7c0c877587befce623608c899be657bd2f710e2/lib/src/client.dart#L232
I will submit a pull request shortly with a proposed solution.
Best and thank you

I don't exactly understand your use case but usually I'd expect the Dart HTTP client to automatically follow redirects. If the resulting page of the redirect is a plain text and not a json then Yes, it would fail but I'm not sure if this is something that the SDK should deal with.
As mentioned in your PR, I'll think a little more on it after the refactoring (there are no ETAs).

I guess you are right with the automatic follow part, but in my case the server responded with a different (default) endpoint, therefore following the redirect ends in a crash (_TypeError from above) caused by the PB-sdk. Since this a lib internal error, I would expect to get something "outside" to deal with it, other than trying to catch a cast-error that should not happen anyways 🙈
Best

The fix in your PR is not OK because the above error is not related to the 308 redirect. It could happen every time when you have a reverse-proxy or anything else intercepting the requests and that returns a non standard PocketBase response.
The more proper fix for the TypeError I guess would be to wrap all type assertions in a helper like assertOrThrowClientException in order to normalize the thrown exceptions, but again this is not a standard operational behavior as the SDK expects all API responses to be a valid PocketBase json corresponding to the specific endpoint.

Graceful fallback value assert was implemented in the develop branch together with some other changes related to the upcoming PocketBase v0.23.0 release.
Note that even with the above change there is no guarantee that this will cover every case because there is nothing preventing the proxy returning valid json but with custom/unknown fields (and I'm not willing to add checks whether each response matches with the expected PocketBase API data as this will be unmaintainable and completely unnecessary in my opinion).


Should work fine in 3.6.


Airflow

Consider moving to Apache airflow as workflow framework
https://airflow.apache.org/docs/apache-airflow/stable/index.html

Or prefect


Is this project dead?

It seems like there is very little activity in this repo in last months.
Can we get an update on the status of the project? Is the development stopped?

Hey all, short answer no. I'll have some more news on this in the next few days :)

See #1144


Invoke ORCA Load Report callbacks from Router::Filter.

Commit Message:
Invoke ORCA Load Report callbacks from Router::Filter.

Add LoadBalancerContext::setOrcaLoadReportCb(OrcaLoadReportCb callback) method.
Allow LoadBalancer to set the callback to be invoked when ORCA load report is received.

Risk Level: Low
Release Notes: N/A
Platform Specific Features: N/A
#34777

/assign @adisuissa

I am back. Will take a look at this tomorrow. :)

Thank you for your feedback! I was OOO last week and will address your comments shortly.

/retest

LGTM, Thanks. I think it would be pretty quick to see our new LB, haha.

Thank you for quick reply and approval!
I have the LB PR next, I just need to update it with the new callbacks interface.


Prevent a potential RCE attack vector using confused packages method

Description
Our team received a report from Traficom, related to a potential security vulnerability. According to the given description and the resulting research, since we had not claimed the "organisation"-scope that we used in 1 of our internal packages from the public NPM -package repository, there was a possible remote code execution (RCE) attack vector, using a "confused packages" attack.
Our monorepo includes some common packages, which are currently all meant to be private-only-packages and are so used only by the monorepo itself. It should be noted that we have already been using the Yarn workspaces and set the internal packages as private. This should already lesser the possibilities for the confusion attack.

We have now claimed the organisation name events-helsinki from the NPM. By using the scoped package names, the attack has less chance of success.
The internal packages of the monorepo are now renamed so that they all include the new organisation scope events-helsinki in their names. Since that, they are maintainable only by the members of the events-helsinki -organisation.

All the package names under /packages -directory are now
renamed so that they include the new organisation scope:

events-helsinki-components -> @events-helsinki/components
events-helsinki-common-i18n -> @events-helsinki/common-i18n
events-helsinki-common-tests -> @events-helsinki/common-tests
events-helsinki-graphql-proxy-server
-> @events-helsinki/graphql-proxy-server

Issues
A potential security vulnerability (Remote Code Execution "RCE") with a package dependency confusion attack.
We got a following note:

Based on the package.json file from this repo, you have the following NPM dependencies: "@events-helsinki/eslint-config-bases": "workspace:^", However, according to the NPM registry, these packages are not claimed yet: https://www.npmjs.com/org/events-helsinki

A dangerous actor could register the used organisation scope for himself and "hijack" the package's management:

As I mentioned, the current configuration can be considered as secure, but as for the government, I would suggest preventing a potential risk. I can register the scoped package with this name which can contain some malicious code inside. Once someone will try to build this source code it may lead to remote code execution (RCE) on that host. Please note, that I didn't claim this package, so you can register by yourself or you can remove this dependency from your source code."

Therefore:
We should not use any unclaimed package-names:
- we should not use unclaimed organization name in monorepo's internal packages. E.g, `@events-helsinki/eslint-config-bases`
- we should not use unclaimed package names that are not registered in the NPM

Closes
LIIKUNTA-453: Security issue: do not use unclaimed organization or package names in monorepo's internal packages
and the subtasks:
LIIKUNTA-456: Remove the events-helsinki-core from the NPM since it's not in any use anymore
LIIKUNTA-457: Remove any events-helsinki-core deps
LIIKUNTA-458: Investigate the security issue and it's scope
LIIKUNTA-459: Contact the security issue reporter with the pull request
LIIKUNTA-460: Claim the "events-helsinki" organisation in the NPM
LIIKUNTA-461: Rename and publish the internal packages in a way that they are organised consistently
Related

The changeset tool that was earlier used in our monorepo, still had some tails there in the content of the repo. They are now cleaned.
The old proof of concept package, events-helsinki-core still had some tails. They are now cleaned.
The internal packages version numbers are set to 1.0.0 since they are already in real production use.
Some circular imports were found when the components-package was built. It's strange that these issues were not raised earlier.

Testing
Manual testing
Use the Confused tool for checking for lingering free namespaces for private package names referenced in dependency configuration.
Note that we will get the false positives from our private repos which are never thought to be published. However, they should now be all scoped with the organisation name events-helsinki.
Examples
events-helsinki-monorepo/apps/sports-helsinki
❯ cd apps/sports-helsinki
❯ ~/Code/tools/confused/confused -l npm package.json
Issues found, the following packages are not available in public package repositories:
 [!] @events-helsinki/common-i18n
 [!] @events-helsinki/common-tests

events-helsinki-monorepo/packages/components
❯ cd packages/components
confused -l npm package.json
Issues found, the following packages are not available in public package repositories:
 [!] @events-helsinki/eslint-config-bases
 [!] @events-helsinki/common-i18n

events-helsinki-monorepo/packages/common-tests
❯ cd packages/common-tests
❯ ~/Code/tools/confused/confused -l npm package.json
Issues found, the following packages are not available in public package repositories:
 [!] @events-helsinki/eslint-config-bases

Screenshots

Additional notes
Related articles:

<EMAIL_ADDRESS>https://fossa.com/blog/dependency-confusion-understanding-preventing-attacks/
https://blog.includesecurity.com/2021/02/dependency-confusion-when-are-your-npm-packages-vulnerable/
https://github.com/x1337loser/Dependency-Confusion
https://security.stackexchange.com/questions/244866/how-to-protect-from-dependency-confusion-attacks
https://azure.microsoft.com/en-us/resources/3-ways-to-mitigate-risk-using-private-package-feeds/

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr289.dev.hel.ninja :rocket::rocket::rocket:

Hobbies-Helsinki branch is deployed to platta: https://harrastukset-pr289.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://harrastukset-pr289.dev.hel.ninja :laughing::tada::tada::tada:

Sports-Helsinki branch is deployed to platta: https://liikunta-pr289.dev.hel.ninja :rocket::rocket::rocket:

VENUE GRAPHQL PROXY is deployed to platta: https://venue-graphql-proxy-pr289.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://liikunta-pr289.dev.hel.ninja :laughing::tada::tada::tada:

There was already a circular dependency issue in the EventsFederationApolloClient that needed a fix. Here is the result of the build before the changes https://github.com/City-of-Helsinki/events-helsinki-monorepo/pull/289/commits/e4ecc42b0b5df38d28c14a4524585b564b61e8b5:
Failed to resolve the module https imported by src/apollo/EventsFederationApolloClient.ts
Is the module installed? Note:
 ↳ to inline a module into your bundle, install it to "devDependencies".
 ↳ to depend on a module via import/require, install it to "dependencies".
Circular dependency: src/components/errorPages/ErrorFallback.tsx -> src/components/errorPages/UnknownError.tsx -> src/components/index.ts -> src/components/errorPages/index.ts -> src/components/errorPages/ErrorFallback.tsx
Circular dependency: src/components/index.ts -> src/components/errorPages/index.ts -> src/components/errorPages/NotFound.tsx -> src/components/index.ts
Circular dependency: src/components/errorPages/UnknownError.tsx -> src/components/index.ts -> src/components/errorPages/index.ts -> src/components/errorPages/UnknownError.tsx

Like I said in the description already, I have no idea how we have missed that earlier.

Went through the changes, looks ok, checked out the branch and searched with "events-helsinki-", most are "events-helsinki-monorepo", some are "events-helsinki-*" (i.e. literally with the asterisk), then there's some container names in docker-compose.events.yml and documentation stuff in README.md, CHANGELOG.md and how-to.md. Looks ok to merge for me, what about the docs/howto/how-to.md? Are all the places where "events-helsinki-" prefix exists in it after this pull request ok in your view?

I used the search and replace for the exact package names and they always included more than just the prefix events-helsinki. I can self go the changes through once again before the merge, but at least all the CI processes passed ok. It's probably easier to go through the changes a commit by commit.

Went through the changes, looks ok, checked out the branch and searched with "events-helsinki-", most are "events-helsinki-monorepo", some are "events-helsinki-*" (i.e. literally with the asterisk), then there's some container names in docker-compose.events.yml and documentation stuff in README.md, CHANGELOG.md and how-to.md. Looks ok to merge for me, what about the docs/howto/how-to.md? Are all the places where "events-helsinki-" prefix exists in it after this pull request ok in your view?

I used the search and replace for the exact package names and they always included more than just the prefix events-helsinki. I can self go the changes through once again before the merge, but at least all the CI processes passed ok. It's probably easier to go through the changes a commit by commit.

No need to go through all the changes from my part. Just looking at what's left after this PR i.e. with events-helsinki- search from this branch.


Enable notification dispatch for system and staging tests

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@mpolden - poke


[BUG]: stable diffusion example

🐛 Describe the bug
when installing stable diffusion environments, everything works fine until doing below:
**

git clone https://github.com/Lightning-AI/lightning && cd lightning && git reset --hard b04a7aa
pip install -r requirements.txt && pip install .

**
the requirements.txt in lightning will change lots of things(like changing torchmetrics==0.6.0 to torchmetrics>=0.7.0, <0.10.1)
Environment
No response

Those specific versions of packages are required by Lightning. There may raise compatibility problem if you install a mismatched version.

Hi @yli1994 ,
Thanks for your issue. I've already updated environment.yaml.

Hi @yli1994,
could you show me more about your error and you environment.
you can use the scripts in pytorch issues to get your environment. just open an issue in pytorch and you should know how to get your environment.


Remove npm update; add release-2.3 to branches

This command should do nothing immediately following an npm install, but causes failures due to tslint having a dependency on typescript.
Also add release-2.3 to our covered branches

🔔


[GHA] add cache timeout

according to https://github.com/actions/cache/blob/main/RELEASES.md#308

Codecov Report

Merging #9524 (4b4c8b4) into master (38ac70a) will decrease coverage by 35.29%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #9524       +/-   ##
=============================================
- Coverage     69.96%   34.67%   -35.30%
+ Complexity     4871      198     -4673
=============================================
  Files          1925     1925
  Lines        102555   102555
  Branches      15559    15559
=============================================
- Hits          71752    35560    -36192
- Misses        25734    63993    +38259
+ Partials       5069     3002     -2067

Flag
Coverage Δ

integration1
?

integration2
24.59% <ø> (-0.05%)
:arrow_down:

unittests1
?

unittests2
15.55% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...in/java/org/apache/pinot/spi/utils/BytesUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/BaseRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/spi/trace/NoOpRecording.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/config/table/FSTType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/config/user/RoleType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/data/MetricFieldSpec.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/stream/StreamMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/common/tier/TierFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/spi/config/table/TableType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../org/apache/pinot/spi/data/DimensionFieldSpec.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1111 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Create getUserMeId to get a userId

Changed:

Added new function getUserMeId.ts and his test.
Added new type UserMeResponse.ts
Added new type UserMeSerializedResponse.ts

I've removed resetAllMock from test file


How to do AR like Living Wine labels ?

Hi Guys,
How todo a AR app like the one in this link on the bottle via Viromedia Framework ?
https://tactic.studio/living-wine-labels/

Hi @mkharibalaji, you can use our ARCarDemo as a template. The Wine Label app is an example of image recognition. Rather than use the entire wine label as the marker, you probably want to use a cropped image of a portion of the label so that it is flatter and easier to track. You can use a curved OBJ and apply a video texture on it, so it matches the curve of the bottle (example code below). You you could try using alpha video -> https://docs.viromedia.com/v2.7.3/docs/video#alpha-video-chroma-key-filtering
        <ViroNode position={[0, 0, -.008]} rotation={[-90, 0, 0]}>
          <Viro3DObject
            source={require('./res/winebottle_label.obj')}
            materials="bottle"
            type="OBJ"
            scale={[.28,.28,.28]}
            />
          <ViroMaterialVideo material="bottle" paused={this.state.pauseVideo} loop={true} />
        </ViroNode>
        </ViroARImageMarker>```

Thanks @dam00n Will try and let you know.

Hi @dam00n It's working and got issues on marker traking which i will raise it as a separate issue.We can close this issue


PML1: Typo in 13.2.1 The XOR problem

Footnote 1:

(There are some exceptions, such as [Tol+21], where unstructured models can learn to perform well on image and text data, but such models need massive datasets to overcome their lack of inductive bias .

The closing parenthesis is missing.
PS amazing book, keep up the good work. This will become a classic, like the first edition did.

fixed, and thanks for the praise :)


Unable to register mythx account

Hello, I am a researcher from China. I want to analyze some smart contracts through mythril. Unfortunately, I couldn't register for a mythx account. I entered the correct mailbox and subscribed to the update message, but the registration function was never available. When I use Amazon Cloud, the registration interface always prompts "There was a error, please retry". How can I register my mythx account? I noticed that many functions of mythril depend on this account. I look forward to your reply.

Hi @xf97, I think mythril doesn’t need registration to run, if you have problems with MythX registration post the query on discord, it will be resolved quickly.

Closing, as the question is unrelated to Mythril itself.


refactor(molecules): SfBanner

Related issue
https://github.com/DivanteLtd/storefront-ui/issues/805
Scope of work
refactor of Sfbanner adjusting to new design, updating font variables and modifiers added: --right and --square
Screenshots of designs:

Checklist

[x] No commented blocks of code left
[x] Run tests and docs
[x] Self code-reviewed

If applicable:

[x] I followed composition rules for my component
[x] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)

Fixes after CR made

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: AdamPawlinski:x: pspaczekYou have signed the CLA already but the status is still pending? Let us recheck it.

@AdamPawlinski


feat: evaluator integration

Integrate the Evaluator in finetuner.fit()

@tadejsv if we do it as a callback the user will need to pass many arguments of fit in the callback. Example
callbacks=[
    Evaluation(
        query_data,
        index_data,
        embed_model,
        batch_size=batch_size,
        device=device,
        preprocess_fn=preprocess_fn,
        collate_fn=collate_fn,
    )
]

I don't think this is very usable. Another option is to add these arguments to the state, so that the callback can retrieve them, but having these in the state doesn't make much sense to me. What do you think?
Also what happens if the user uses to_embedding_model=True? The model that should be used for evaluation is not the same that the user pass in fit

@gmastrapas

I don't think this is very usable. Another option is to add these arguments to the state, so that the callback can retrieve them, but having these in the state doesn't make much sense to me. What do you think?

So of the arguments you have shown, only query_data and index_data (and metrics related args) would need to be passed to the callback. For the model and the rest, we can set them as a (private) property of the tuner (I agree they don't really belong to the state), and the callback can get them from there - it has access to tuner.

we can set them as a (private) property of the tuner (I agree they don't really belong to the state), and the callback can get them from there - it has access to tuner.

This makes perfect sense, I'll proceed this way

minor comment, besides, can you try if the callback progress bar looks correct in notebook?

@bwanglzu It works if I install ipywidgets. A warning however from the same package is constantly being refreshed with the progress bar


field.String/field.Text generates int8 type field

Hello.
I am trying to generate code by example schemas and all string fields are generated as int8. As I know it begins to happen after this commit.
entc describe output for examples/start.

Can you upload please your schema?

Another question please - are you working with go modules?
I assume your ent version and entc are not identical.
Please try to follow this guide, and let me know if it does fix your problem.

I assume your ent version and entc are not identical.

Thank you, I just forgot to update entc:
go get -u github.com/facebookincubator/ent/cmd/entc


tailscale: 1.62.1 -> 1.64.0

Description of changes
Diff: https://github.com/tailscale/tailscale/compare/v1.62.1...1.64.0

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

2 packages built:

tailscale
trayscale


convert from CommonJS to imports

Now that it's using TypeScript, use imports rather than requires to get better type inference.

Duplicate of #143.


It started to give CORS error.

No issue to me?

test at: https://freegpt.js.org/example.html

@anil-turk Actually you might not be able to fetch from there from http, try the same thing on a delpoyment (Https)

Yes it fixed. Maybe there was a protection mode was active. Because I was tried 30 prompts one after another.

Can we do it with node. I tried other projects, but it doesn't work well as this.I tried but I couldnt scrape from you.com

I already said this in README. The site is protected by Cloudflare and you cannot easily scrape it.
Consider just running this script on the client-side I know there are limits.

Yes it fixed. Maybe there was a protection mode was active. Because I was tried 30 prompts one after another.

It wasn't a issue.


Revert "Shutdown reconfiguration thread in a more controlled way [run-systemtest]"

Reverts vespa-engine/vespa#21072

Looks like this broke the system tests on factory


Add an integration test for forcerelay

Closes: #XXX
Description

PR author checklist:

[ ] Added changelog entry, using unclog.
[ ] Added tests: integration (for Hermes) or unit/mock tests (for modules).
[ ] Linked to GitHub issue.
[ ] Updated code comments and documentation (e.g., docs/).
[ ] Tagged one reviewer who will be the one responsible for shepherding this PR.

Reviewer checklist:

[ ] Reviewed Files changed in the GitHub PR explorer.
[ ] Manually tested (in case integration/unit/mock tests are absent).

I find you record so many JSON files for mocking beacon header, but you can combine them into one JSON file with map format for storing these headers, and when you read them from mock rpc, you just need to read from the BTreeMap.

some clippy errors need to solve which are not able to be ignored.


the size of Room B

The width and length of room B are the same, which is not consist with the rectangular shape of this room.

You are right.And the length obviously wrong because the listener position add 1.5m is beyond the length.


Use arrow to define components

Just idea:
Managing feedback like Yampa in haskell.

If I do this, I will do it in another repository, not in Jelly.


minor fix and make align with float->float32

Which issue does this PR close?

minor fix and followup for https://github.com/apache/arrow-datafusion/pull/1390
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Yes.
In the datafusion README.md, the SQL float is corresponding to the arrow-FLOAT32.
@alamb

Thank you @liukun4515


add spike_aggregation rule mapping

Fix issue #2422
Add value for spike aggregation in rules_mapping map

The content of this pull request has already been merged with the following pull request.
updated rules_mapping to include spike_aggregation #2570


authorize_access_token() unintentionally send oauth_callback

Describe the bug
For oauth1 client with flask integration, OAuth.providername.authorize_access_token() send oauth header with oauth_callback field to access_token_url.
The uri in the unnecessary oauth_callback field was stored in flask session from previous authorize_redirect(redirect_uri) call.
Since there is no way to utilize oauth_callback after access_token_url step, the server SHOULD return 400 bad request for unsupported parameter according to rfc 5849, Section 3.2.  Verifying Requests.
Error Stacks
I found hatena (Japanese social bookmark service) follows this SHOULD statement and fails to complete authorization.
File "./oauth.py", line 143, in hatena_authorized
  token = oauth.hatena.authorize_access_token()
File "/home/ubuntu/.pyenv/versions/3.9.1/envs/fav/lib/python3.9/site-packages/authlib/integrations/flask
_client/remote_app.py", line 76, in authorize_access_token
  token = self.fetch_access_token(**params)
File "/home/ubuntu/.pyenv/versions/3.9.1/envs/fav/lib/python3.9/site-packages/authlib/integrations/base_
client/remote_app.py", line 102, in fetch_access_token
  token = client.fetch_access_token(token_endpoint, **kwargs)
File "/home/ubuntu/.pyenv/versions/3.9.1/envs/fav/lib/python3.9/site-packages/authlib/oauth1/client.py",
line 142, in fetch_access_token
  return self._fetch_token(url, **kwargs)
File "/home/ubuntu/.pyenv/versions/3.9.1/envs/fav/lib/python3.9/site-packages/authlib/oauth1/client.py",
line 159, in _fetch_token
  token = self.parse_response_token(resp.status_code, resp.text)
File "/home/ubuntu/.pyenv/versions/3.9.1/envs/fav/lib/python3.9/site-packages/authlib/oauth1/client.py",
line 170, in parse_response_token
  self.handle_error('fetch_token_denied', message)
File "/home/ubuntu/.pyenv/versions/3.9.1/envs/fav/lib/python3.9/site-packages/authlib/integrations/reque
sts_client/oauth1_session.py", line 58, in handle_error
  raise OAuthError(error_type, error_description)
authlib.integrations.base_client.errors.OAuthError: fetch_token_denied: Token request failed with code 40
0, response was 'oauth_problem=parameter_rejected&oauth_parameters_rejected=oauth_callback'.

To Reproduce
Just call authorize_access_token() from flask integration.
# oauth = authlib.integrations.flask_client.OAuth()
# oauth.register("hatena", ...)
@app.route('/authorized')
def hatena_authorized():
    # session.pop("_hatena_authlib_redirect_uri_") # workaround
    token = oauth.hatena.authorize_access_token()

Expected behavior
oauth_callback field (or any unnecessary fields) should not be sent for accesss_token_url.
Possible workaround is delete _providername_authlib_redirect_uri_ from session before every authorize_access_token() call.
Environment:

OS: Ubuntu 20.04 with pyenv-install/virtualenv
Python Version: 3.9.1
Authlib Version: 0.15.2

Additional context
Add any other context about the problem here.

Try with v0.15.3 release.

Try with v0.15.3 release.


Create the structs definitions

Create the following structs:

A table struct
A row struct
A cell struct
בכבוד אלחי

In no2, do you mean a column struct?

Yes. Sorry. I will fix that.


[autoGuides] End date and start date are computed wrong for each guide

The weekend guides are not computed correctly. The problem seems to appear from this pice of code:
      // calculate beginning and end of day
      var start = new Date( date );
      start.setHours( 0, 0, 0 );
      var end = new Date( date );
      start.setHours( 23, 59, 59 );

it should be like:
      // calculate beginning and end of day
      var start = new Date( date );
      start.setHours( 0, 0, 0 );
      var end = new Date( date );
      end.setHours( 23, 59, 59 );

Hi @bogdanalexe90, sorry for not getting back to you sooner on this.
On April 21st we made some changes to the autoGuides plugin.
After upgrading to the latest version, do you still have this problem?


#874 changes autocomplete="off" to autocomplete="false"

changes autocomplete="off" to autocomplete="false" for not letting the browser autocomplete in chrome and firefox new versions

Will be solved at https://github.com/angular-ui/ui-select/pull/1210 Thanks!


Adding issue forms for the repo

I want to add a few issue forms so that it will be convenient for new contributors to open issues. There are already issue templates for this project but they are deprecated now, issue forms instead work way better. I plan to create  Bug Report, Documentation,  and Feature Request forms.
The Bug Report issue form will have the following fields

Description of the bug
How to Reproduce
Expected behavior
Urgency indicator
Desktop Info (OS, Version, Browser, etc):

The Documentation issue form will have the following fields

Description for documentation

The Feature Request issue form will have the following fields

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Describe alternatives you've considered

@MonarchGitHub do you mean a PR template?

No, issue templates are similar to PR templates but they are used to create issues. I will attach a screenshot of the issue templates below:-