Ardublockly server and Upload sketch error

1)When I Run Ardublockly, Initially It gives me error in browser console.

It gives no error on command prompt where I run server

I have designed one sensor block. When I drag it on editor and click on Upload and compile sketch
It gives me error in browser console. It launches Arduino software but does not upload code to it.

Command Prompt Error where I have run server

This is an issue with Chrome that should be delivered soon: https://bugs.chromium.org/p/chromium/issues/detail?id=593273

About the second error I will need more info than that, since it is a custom block. Could you please provide the Arduino generated code at the very least? Having the block definition and generators would be useful as well.

I tried Using arduino_debug.exe it giving me same error
Block Definition for Sensor(sensor.js)
'use strict';
goog.provide('Blockly.Blocks.sensor');
goog.require('Blockly.Blocks');
goog.require('Blockly.Types');
Blockly.Blocks.io.HUE = 20;
Blockly.Blocks['flex_sensor'] = {
init: function() {
this.appendDummyInput()
.appendField("FLEX SENSOR")
.appendField("Connector")
.appendField(new Blockly.FieldDropdown([["1", "1"], ["2", "2"], ["3", "3"]]), "connectorselection")
.appendField("Operater")
.appendField(new Blockly.FieldDropdown([["<", "<"], [">", ">"], ["==", "=="]]), "operaterselection");
this.appendDummyInput()
.appendField("resistance")
.appendField(new Blockly.FieldTextInput("Enter Value between 0 - 1023"), "resistance");
this.appendStatementInput("doflex")
.setCheck(null);
this.setColour(20);
this.setTooltip('');
this.setHelpUrl('http://www.robolab.in/');
}
};
Block Generated Code(sensor.js)
'use strict';
goog.provide('Blockly.Arduino.sensor');
goog.require('Blockly.Arduino');
Blockly.Arduino['flex_sensor'] = function(block) {
var connector_sel = block.getFieldValue('connectorselection');
var operator = block.getFieldValue('operaterselection');
var resistance = block.getFieldValue('resistance');
var statements_name = Blockly.Arduino.statementToCode(block, 'doflex');
// Include Macrodefinitions
var flexcode = '#define flexPin_1 A0\n' + '#define flexPin_2 A2\n' + '#define flexPin_3 A5\n';
var sensormacro = 'flexsensor';
Blockly.Arduino.addInclude(sensormacro, flexcode);
// Declare Global Variable
var flexvariable = 'int connector_sel = 0;\n' + 'char operator;\n' + 'char resistance;\n';
var sensorvariable = 'variable';
Blockly.Arduino.addVariable(sensorvariable, flexvariable, false);
// Add code to Setup function
var flexsetup = 'Serial.begin (9600);\n' + 'pinMode(flexPin_1, INPUT);\n' + 'pinMode(flexPin_2, INPUT);\n' + 'pinMode(flexPin_3, INPUT);\n';
var flexsetuptag = 'flexsettag';
Blockly.Arduino.addSetup(flexsetuptag, flexsetup, false);
// Add code to Loop function
var code = 'connector_sel = ' + connector_sel + '\n' + 'if(connector_sel == 1)\n' +
'{\n' +
'int flexposition = analogRead(flexPin_1);\n' +
'delay(1);\n' +
'if(flexposition' + ' ' + operator + ' ' + resistance + ')\n' +
'{\n' + statements_name + '\n}\n' + '}\n' +
'else if(connector_sel == 2)\n' +
'{\n' + 'int flexposition = analogRead(flexPin_2);\n' +
'delay(1);\n' +
'if(flexposition'+ ' ' + operator + ' ' + resistance + ')\n' +
'{\n' + statements_name + '\n}\n' + '}\n' +
'else if(connector_sel == 3)\n' +
'{\n' + 'int flexposition = analogRead(flexPin_3);\n' +
'delay(1);\n' +
'if(flexposition' + ' ' + operator + ' ' + resistance + ')\n' +
'{\n' + statements_name + '\n}\n' + '}\n';
return code;
};

I hope you don't mind I edited your comment to format the code correctly (you can use the github comment editor to achieve the same effect)

I tried Using arduino_debug.exe it giving me same error

Of course, this error is not related to the version of the .exe used, it's just that you will need the debug.exe version to get proper console output displayed in Ardublockly and captured by the log.
But most importantly, could you please copy and paste the Arduino generated code on the Ardublockly application that causes this issue?

#define flexPin_1 A0
#define flexPin_2 A2
#define flexPin_3 A5
int connector_sel = 0;
char operator;
char resistance;
void setup() {
Serial.begin (9600);
pinMode(flexPin_1, INPUT);
pinMode(flexPin_2, INPUT);
pinMode(flexPin_3, INPUT);
}
void loop() {
connector_sel = 1
if(connector_sel == 1)
{
int flexposition = analogRead(flexPin_1);
delay(1);
if(flexposition < Enter Value between 0 - 1023)
{
//statement code goes here
//you can attach led block here
}
}
else if(connector_sel == 2)
{
int flexposition = analogRead(flexPin_2);
delay(1);
if(flexposition < Enter Value between 0 - 1023)
{
//statement code goes here
//you can attach led block here
}
}
else if(connector_sel == 3)
{
int flexposition = analogRead(flexPin_3);
delay(1);
if(flexposition < Enter Value between 0 - 1023)
{
//statement code goes here
//you can attach led block here
}
}
}

Nothing quite obvious there, I'm going to need to debug this locally. Do you have a full source code copy I could just copy and test in my computer? It doesn't look like you are using github for your version control system.

You can download the source code on this link https://drive.google.com/open?id=0B9YgAk1hTD6gdGw3ZDl6YWlNd3c

does this happen every time? I tried to replicate using the blocks from below and it works fine on
Running Python 2.7.10 (32 bit) on Windows-7-6.1.7601-SP1 (64-bit windows)
<xml xmlns="http://www.w3.org/1999/xhtml">
  <block type="flex_sensor" id="xR|D4kkkWG1M]f;MY*Eb" x="55" y="90">
    <field name="connectorselection">1</field>
    <field name="operaterselection"><</field>
    <field name="resistance">Enter Value between 0 - 1023</field>
    <statement name="doflex">
      <block type="io_digitalwrite" id="e1;[XGTa/5q6DkWhZ/CD">
        <field name="PIN">13</field>
        <value name="STATE">
          <block type="io_highlow" id="?jo-DvY_*8kS}ny^(9,0">
            <field name="STATE">HIGH</field>
          </block>
        </value>
      </block>
    </statement>
  </block>
</xml>

Hold, which language have you set up your Arduino IDE?

I am running Python 2.7.11 (32 bit) on 64 bit windows 10 OS.
I am using an english language In Arduino IDE.
I am using Arduino Nano Board which was not included in ardublockly then I write the code for adding Nano board
If I upload only led block then it works fine but when upload my custom made sensor block then it gives me error

The issue is not with the front end javascript code, but with the server parsing of the sketch file, so the console output is the part I need to be able to debug this.
I've been using your zipped version of Ardublockly and having a hard time replicating this, the application seems to work on Win7 64bits Py2.7.10 and Win10 64 bits Py2.7.11 both using Arduino 1.6.9.
Does the python output always give you the same error?
Which version of the Arduino IDE are you using? Does the same problem happen if using arduino.exe and arduino_debug.exe?
When you the error happens, could you copy the file ArdublocklySketch\ArdublocklySketch.ino and upload it somewhere so that I could try that one directly?

Hello Sir,
Code Uploaded Successfully.
thanks for help.
There was syntax error in sketch.

Hi Rimish,
Yes, the generated code had a few issues, but the server was unable to process the output from the Arduino IDE to be displayed in the Ardublockly interface (it failed to parse the std error output), so this problem can still occur if there is any other issue on the compilation.
Could you try to compile the problematic sketch on the Arduino IDE without ardublockly, and then copy and paste here the Arduino IDE console ouput from the bottom of the Arduino window? Hopefully that way I could see the type of text that was causing the issue on Ardublockly.

Actually the generated code from blocks in right panel has some syntax error. When I copy this code into Arduino Software it was giving me syntax error.
So I corrected the generated code from blocks and then uploaded again.
It has uploaded the code successfully to arduino Nano and output was getting correctly.

Yes, but even if the code had a syntax error, Ardublockly should have processed the Arduino IDE output properly and displayed such error in the "Arduino IDE output" box, just like it does when the code compiles or is uploaded. Therefore, there is an issue with Ardublockly that makes it crash when this particular Arduino IDE output is processed, which is why it'd be useful if you'd be able to compile that code in your particular computer (as it seems to work in mine) and see what the Arduino Output is.

Ok Now I got it.
So What should I do for this thing because when there is a syntax error it does not show me error in Ardublockly Output Box but when it is correct it compiles and shows output.

Thanks I really appreciate your help here.
If you still have the broken implementation (I believe the zip file you linked before should be a version of the code with the issue), you could then put together the blocks causing the problem, but instead of "uploading the code" or "verifying the code", you can use the option to "Open in IDE" (the yellow button). Then there are four things that if you could capture I would find quite useful:

If you could "save as" the Arduino sketched opened in the Arduino IDE and save that
If you could try to upload the sketch and copy-paste the Arduino output from the Arduino IDE it into a text file (it should be all printed in the console box at the bottom of the Arduino IDE)
If you could verify the code and also copy and paste the output from the Arduino IDE as well
If you could find the arduino "preferences.txt" file (this link shows where you can find it: https://www.arduino.cc/en/Hacking/Preferences) and save that too

Hopefully with that information I could see what Ardublockly was processing and hopefully try to fix it, or at least be able to replicate the same problem to that I can try to debug it.

Step 1) created sketch in ardublockly and opened it in Arduino software IDE

Step 2) Save as arduino sketch in arduino IDE.
Step 3) Upload the sketch and copied the output from console to txt file.
console output.txt
Step 4) Verify sketch and copied output from console to txt file
verify console output.txt
step 5) preferences.txt file
preferences.txt

Hello Sir, Have you looked into my above comment. I have uploaded the files you wanted

Hi Rimish,
I'm currently on holiday, so unfortunately I won't be able to look into
much for a little while. I'll try to spend some time on Ardublockly as soon
as I'm back.
Regards,
Carlos
On 9 Jul 2016 06:49, "RimishRobolab"<EMAIL_ADDRESS>wrote:

Hello Sir, Have you looked into my above comment. I have uploaded the
files you wanted
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/carlosperate/ardublockly/issues/96#issuecomment-231515184,
or mute the thread
https://github.com/notifications/unsubscribe/AD_sTmhuz3Aku9RdVmUvLN8fSdMeANsuks5qTyhFgaJpZM4I8sxg
.

Hi Carlos Sir,
Have you tried above uploaded files and get any solution

Apologies, I must admit I had forgotten about this, I'll try to give it a good look this week.

Hello Sir,
I have solved this issue yesterday. There was a problem with my arduino.exe file. I have downloaded the arduino software again and it works for me


Modal name seems not to matter

The name parameter doesn't seem to matter.  If I have multiple modals in one template they all open anyways unless I give them different classes and call those in my function.  So in my component template I have:
<button class="ui green button" {{action 'openModal' 'create-project'}}>Open Modal</button>
In my component.js file:
actions: {
        openModal: function(name) {
            console.log(name);
            $('.ui.modal.create.project').modal('setting', 'closable', false).modal('show');
        },

The first tag of my modal:
{{#ui-modal name="create-project" class="create project"}}
I have also tried $('.ui.modal.create.project').modal('setting', 'closable', false).modal('show', name);.  If I remove the .create.project class names, then the function opens all my other modals.  Is there something I am missing?

Based on your selector this should work, its hard to tell without looking at all the code. Did you find any solutions? Can we close this ticket?

@aaronbhansen Sorry for the late response.
The only thing that seemed to work was adding a defining class to the modal, such as {{#ui-modal name="response-message" class="response message"}}, and then calling it like $('.ui.modal.response.message').modal().modal('show');
So this shouldn't be closed.  This is still an issue.  What other code do you need to see?


Feature/source freshness hooks

resolves #5609
Problem
Currently, on-run-* hooks are not supported in the dbt source freshness command, this PR makes it so it does.
Solution
Changed the FreshnessTask class to inherit from RunTask (as all other task types that support hooks do).
Alternative solutions:

Move the hook logic to GraphRunnableTask and disabling in CompileTask - feels a little messier (disabling parent functionality in inheriter).
Create a HookTask in the middle of the inheritance tree - seems like a lot more work when every other hook-able task just inherits from RunTask.

Additionally, this change has been made backwards-compatible with previous versions of dbt which did not include project hooks by default for dbt source freshness commands via a project flag + tests added across the different scenarios (default configuration, flag explicitly disabled, flag explicitly enabled)
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX
[x] This PR includes type annotations for new and modified functions

Initial review covering first 2 commits of this change: https://github.com/dbt-labs/dbt-core/pull/9322#issuecomment-1879504718

As things for possible future improvements, that whole flags processing thing is getting complicated. We should look for an opportunity to refactor it.

Yeah, I agree. It's definitely got a lot of responsibilities and is quite aware of dbt-specific CLI flags even though it's meant to be more generalized. In the meantime, I've added an additional unit test for the project-only flag behaviour so it'll be safer to refactor in the future.

Also mixing in different Status enums is something that's not wonderful. We need to do a rethink of all of those status codes sometime. It might be better to have a single enum instead of all of these different ones.

We may be able to take a stab at this with some of the ongoing dbt/artifacts refactoring, but agree it shouldn't block this work.

Opened a new issue in dbt-labs/docs.getdbt.com: https://github.com/dbt-labs/docs.getdbt.com/issues/4787


Update carto-package.json

Update gdal required version to >=2.2.0 <2.4.0, we are running it with 2.2.2 but we are also running the tests against 2.3.2 in the Postgres 11 CI so this version is supported.
Update nodejs required version to >=10.3.0 <11.0.0 to match the current requirement of Node.js 10 and npm 6.

retest this please


How to install/run the project on Windows10

Hi, this is completely noob question. I have installed Docker Desktop(I guess it also contains docker-compose).
I also downloaded the whole repository.
But I couldn't make it work. I have tried to run make file but it is working(even though I have installed WSL2 with ubuntu). What should I do to make it run? Is there any easy way :) thank you

Sorry @kkmb34 we only provide this docker-compose for development purposes. If you want to use it in production, some adaptations are necessary. For example the error you get can be fixed by removing the crowdin service from the docker-compose.yml file since it is only needed to generate translation files in development.

Thank you @sampaccoud for your prompt reply and clarification!


⚠ GitHub has degraded performance

In 200d4b2, GitHub (https://www.githubstatus.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 313 ms

Resolved: GitHub performance has improved in 8e07bf1.


VideoPlayer can't show

if change the videoEl.style.zIndex = '1' to videoEl.style.zIndex = '2', the video on Edge can show, the RDK web still can't show.

By design the VideoPlayer is placed below the canvas. This enables you to create an overlaying video player UI (i.e. progress bar, control buttons, etc.) in Lightning.
Whenever you're playing video in your App, you should manually make the background color of the main component (partially) transparent to make the video-tag visible

Ok , Thanks.


Create CONTRIBUTING.MD

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[x] No

ℹ Additional Information

Summary by CodeRabbit

Documentation

Introduced a comprehensive contributing guide for the project.
Provided detailed instructions for forking, cloning, and development processes.
Emphasized coding conventions, testing procedures, and integration with the OpenAI API.
Outlined the submission process and included a pull request checklist.
Encouraged reporting issues and community engagement for support.

@gstraccini codacy bypass


Column type names not displaying correctly in alter table GUI

Incompatible type names between old implementation, using information_schema, and the new one, using pg_catalog. Alter table GUI display things like int4, float8, varchar instead of integer, double precision and character varying, e.g.

Hi @barthisrael ,
I'm closing this. Thanks for your contribution.
Regards,
William Ivanski


Edges of shapes are not rendered well

Black rectangle expected to be invisible:
    let black = vger.color_paint(Color{r: 0.0, g: 0.0, b: 0.0, a: 1.0});
    vger.fill_rect(euclid::rect(100.0, 100.0, 100.0, 100.0), 10.0, black);

(5x)
Two rectangles expected to be seamless：
    let cyan = vger.color_paint(Color::CYAN);
    vger.fill_rect(euclid::rect(100.0, 100.0, 100.0, 100.0), 0.0, cyan);
    vger.fill_rect(euclid::rect(200.0, 100.0, 100.0, 100.0), 0.0, cyan);

(5x)
I guess these artifacts are caused by anti-aliasing.

Thanks for the report!
I'll take a closer look at the black rectangle example.
For the two rectangles, perhaps changing the AA to use a box filter would fix it, since the overlapping falloffs would sum to one (unlike smoothstep).

Thanks for the report!
I'll take a closer look at the black rectangle example.
Edit: actually two smoothsteps in opposite directions do sum to one, so I'll have to look into that in more detail as well.

How to solve this problem?


Updates for MAX78002 SDK; minor logging improvements

@rotx-maxim I don't have write access to unblock merging


detect encrypted DOCX files

This pull request detects encrypted DOCX files per these requirements.

Could you give some test files names from the test_files/ dir I can test with?

Test with
test_files/docx/apachepoi/bug53475-password-is-pass.docx
test_files/docx/apachepoi/bug53475-password-is-solrcell.docx

LGTM 👍


Add clone

This adds basic support for jQuery.clone. The jQuery.clone method accepts an optional second parameter deepWithDataAndEvents, not currently implemented. I'm open to suggestions how to handle that on the PureScript side.

Thanks!

https://pursuit.purescript.org/packages/purescript-jquery/3.1.0


readme suggestion on Advanced integration

I'm working on a module package that generates text files, and I'm appending the current module version to the generated files. This package helped me to deal with reading the package.json file as expected, I'm really grateful for it.
I'm using TypeScript, and wanted to, as the "Advanced Integration" documentation suggests, generate the version file after the patch.
I've tried to use a postversion.sh as indicated but this caused some issues. For example, this scripts requires you to fiddle with the git tags after creating, and also for my case of using Typescript it wasn't so useful because at this step the code has already been built and this file wasn't there in the process.
I've found a practical way to use it, which I think other people might find useful, which is:

create a "gv" script to generate the file and add it to the build process:

// file: package.json

{
  ...
  "scripts": {
  ...
    "gv": "genversion --esm --property name,version,homepage src/constants/project.ts",
    "build": npm run gv && tsc"
  ...
  }
}

Add a "version" script which runs after the version was bumped but before the tagging which I use to build the module and to add the generated file to the commit:

// file: package.json

{
  ...
  "scripts": {
  ...
    "gv": "genversion --esm --property name,version,homepage src/constants/project.ts",
    "build": npm run gv && tsc",
    "version": "npm run build && git add -A",
  ...
  }
}

And now, each time you run npm version patch for example, the file is generated and pushed along with the package.json update. Besides that, you've already built the project with the recently generated version, so you can just publish it right away.
I'm not an expert on this, so I might have overlooked something, but this was really helpful on my project, and I didn't had to create a shell file, made it executable and so on, it just works right out of the clone.
Cheers!

Thanks for the approach, definitely worth it. I might adapt this into the advanced integration docs at some point. Related to https://github.com/axelpale/genversion/issues/7


Does PUK security note apply given guide references OpenPGP applet vs PIV applet?

Security Note: If you followed this guide before Jan 2021, your PUK (Pin Unblock Key) may be set to its default value of 12345678. An attacker can use this to reset your PIN and use your Yubikey. Please see the Change PUK section for details on how to change your PUK.

From YubiKey docs, it appears PUK is part of PIV applet.

I came here to ask the same question. There is no PUK OpenPGP Card spec.

See also #248

Here is Yubico’s answer:

Hello Sun,
Thank you for contacting Yubico support.
It looks like that guide conflated the PIV functions with the GPG functions.
The PUK is just for the PIV functionality. On OpenPGP, the admin PIN (or a reset code, if one is set) can be used to reset the OpenPGP PIN. The PUK command listed is just for the PIV Smart card PIN, and does not reset the OpenPGP PIN.
Sincerely,
Jane Doe* | Customer Support Specialist at Yubico

*I obfuscated customer support specialist name for privacy reasons.

Maybe we can close this one and go to #248?


Clicking TicketCenter by HelpDesk user gives NullReferenceException

Version: 2.5 Beta (tag td2-v2.5.0)
Download: TicketDesk.2.5.0-beta.Compiled.zip from https://github.com/NullDesk/TicketDesk/releases/tag/td2-v2.5.0
Issue Description:
When logged on to the system as HelpDesk User, on clicking TicketCenter the site errors out. Attached the error shown on a browser. When Logged in as Internal User or Admin the issue is not seen.
Steps to Reproduce:

Using Internal User create a ticket
Click on TicketCenter, a list of Tickets will be shown
Logout
Login as HelpDesk user
Click on the TicketCenter link on the top menu
Browser shows error. Object reference not set to an instance of an object.

TicketDeskError.html.txt

This appears to have been related to problems with multi-project support. Should be fixed in release td2-v2.5.1


Training the model

Thank you sharing your code
I would like to train the model by my own data.
Can you give me the steps please to train the model?
Thank you,

Hello!
You can use the run_training_seq2seq function to train your model on any Sequence-to-Sequence dataset you have, the only hard requirement is to specify existing source and target columns


Mappable split, refactoring of transaction mapping

Distinguish between a mappable entity and a transaction:

the mapping of an entity depends only on its properties
the mapping of a transaction depends on its properties, the operation type (required) and the parent transaction type (optional)

Transaction->mappedProperties

made the parameter operation required (remove the default value of null)
refactored the method to the template method pattern

Coverage remained the same at 100.0% when pulling f8f79b0513f77f84ed0e7beabb60f25c55afe0e9 on reka:mappable-split into 65334f275e9a6b902e2a3763a2e8a092df3e0a35 on wirecard:59-sepa.

Coverage remained the same at 100.0% when pulling 30ec36ce132cfb6ecc6de4154934416e1420138f on reka:mappable-split into 65334f275e9a6b902e2a3763a2e8a092df3e0a35 on wirecard:59-sepa.

I'll open a new pull request for this branch.


Implement extensible() support

extensible(prop) { return false; } could be simply converted to {"$other": {".validate": false}}
extensible(prop) { return prop == 'p1' || prop == 'p2'; } is converted to {"$other": {".validate":"$other == 'p1' || $other == 'p2'"}}
There could also be simpler (CoffeeScript) syntax extensible(prop) { return prop in ['p1', 'p2']; }

I'm working on this now.  Actually, extensible() is being removed, to be replaced with something like this:
type Type {
  validate() { return $other.length < 10; }
  prop: String,
  $other: String
}

so no additional properties would be $other: Null
and mixed property types would be with $other: String|number
and more complex could be validate() { return this.other1.isString() && (this.other2 == null || this.other2.isNumber()); }
?
I would say with 'safe by default' there should be no additional properties by default.

I like that line thinking @mckoss, pushing the validation into the Type system. however it's not clear as to whether the validate applies to fixed props or not. Also it seems at the wrong level. Shouldn't it be
validate() {
  return this[$other].length < 10;
}

However that runs into difficulty too because you might be setting several $other keys at the same time. You can't really mix the enclosing type's validate with the $other validator. You could push it all into the $other's type though, which is pretty neat I think.
However, that means you can't perform validation on the key used. Type script solves typing dictionaries by typing the key too. So it would be more like:
type ParentType {
  validate() { /*parentType validation */}
  prop: String,
  $other: KeyType => ChildType
}

I like the direction as well @mckoss and @tomlarkworthy. The typing dictionary for additional properties seems like it would help considerably. With dynamic properties it seems possible to bind several:
type DynamicType {
  prop: String,
  $otherObject: String => Object,
  $otherStringList: String => String[]
}

Dynamic types that contain many ignored properties seem like they might be harder to work with, but maybe I'm missing some aspect of the change.

FB keys are always String.
Also we need to have an option to allow only specific key names. For example.
type Email extends String {
  validate() { return this.indexOf('@') > 0; }
}
type Address {
  line1: String
  country: String
}

type Person {
  name: String
  $emails: Email
  $address: Address|Null
  $age: Number

  validate () { return $emails.startsWith('email') && $address.startsWith('address') && $age == 'age'; }
}

So that following object would be valid:
{
  "name": "John Smith",
  "email1"<EMAIL_ADDRESS>  "age": 42,
  "address1": {
    "line1": "Smith Str.",
    "country": "CA"
  }
}

Doh.  Quite right, @tomlarkworthy - the validate is not scoped at the right level to include $other in the rule (even with this[$other] - which wasn't my intention for the sample - I was trying to limit the key length).
How about:
type Type {
prop: String,
$other: String {
validate() { return $other.length < 10; }
}
}
BTW: this validation rule gets run (on the server) once for each property not matching the explicit property names of the Type.

I prefer the dictionary semantic which is matched with what a dynamic type is trying to achieve. (I also looked up how to do it properly in TS)
// create a type encoding custom key validation logic
type SmallString extends String {
   validate() {return this.length < 10;}
}

//now use that custom type as the key type in a dictionary
//typescript style embedded dictionaries
type Type {
  prop: String
  [key: SmallString]: Any
}

I like this because its a prexisting concept from Typescript, and it doesn't require a different code path for key validation vs. value validation i.e. it leverages the type validation to both situations. It does require a new dictionary concept though. (I am not convinced Typescript has the best syntax, I think I prefer => but that's not the point, the point is expressing key validation as just a parameter choice for construction of a parametric dictionary)

So how would one define type Person where two dynamic properties are related?
type Person {
  name: String
  [email: SmallString]: Email
  [address: SmallString]: Address
  validate() { return this.email.endsWith('google.com') && this.address.zip == 94043; }
}

My plan is to implement  collection-like syntax as a valid property type.  This would not give access to writing validation rules based on the $key directly in the type statement (you'd have to write a parallel validation rule in a path for that).
To rewrite matjaz's example:
type Email extends String {
  validate() { return this.indexOf('@') > 0; }
}

type Address {
  line1: String,
  country: String
}

type Person {
  name: String,
  emails: Emails[],
  addresses: Addresses[],
  age: Number | Null,
}

path /people is Person[] {}

I think following safe by default, types should be by default freezed (no additional properties), but still allowing type to be easily extensible.
For example for game board we have (X&Y) coordinates. from 1,1 to 10,10. I should be able to easily check if coordinate is already taken. So I need a hash.
type BoardMove {
  userId: UserId
  ts: Timestamp
}

path /gamesBoard/$gameId/moves is BoardMove[] {} // no need for that
path /gamesBoard/$gameId/moves/$coordinate is BoardMove {
  validate() { return $coordinate.test(/^\d,\d$/); }
}

I think of a two different scenarios for dynamic properties (ones not explicitly defined).
To validate if dynamic property key is acceptable. (above example)
To validate if dynamic property value is acceptable. (for example in above example if timestamp is less than 10seconds before previous.)

@matjaz
So obviously that example is not great as you shouldn't mix dynamic properties, but here is the dictionary approach for comparison.

note I assume we are using the email as the key in emails, but there is no clear value, so I will jsut put a boolean in it (validated or not?).

type Email extends String {
  validate() { return this.indexOf('@') > 0; }
}
type Address {
  line1: String,
  country: String
}

type Person {
  name: String,
  {Email => Boolean} |    // the type of this, is the union of two dynamic types (+ fixed fields)
  {String => Address}
  age: Number | Null,
}

path /people/$name is Person {}
OR
path /people is {String => Person}

I kinda think using arrays is the wrong thing as we don't use arrays in Firebase. We use dictionaries.

Firebase supports generic type Object<String, Any>.  Developers use this as the way to support internal collections (when just defined in a property but not in their own path statement).
But emails: Object<String, Email> seemed rather verbose for a common paradigm (though I do support this type expression in my latest PR)....perhaps we should prefer Map<String, Email>?
So I propose to make Email[] an alias for that type expression. Other aliases options: List<Email>, Collection<Email>, Object<Email>.
You'll note that Firebase does allow for Javascript Arrays to be written to a Object<String, Map> (and in fact, Javascript actually implements the internal Array type as just that!).
I looked at Swift type expressions - they use [Email] for Array and [String: Email] for Dictionary<String, Email> - I'm not a fan.

Here's another idea...
type BoardCoordinate extends String { // in Firebase keys are strings
  validate() { return this.test(/^\d,\d$/); }
}

path /gamesBoard/$gameId/moves is BoardCoordinate => BoardMove {
  validate() { return this.numChildren() < 42 && this.hasChild('0,0'); }
}

It's not enough to validate hash values but ALSO keys. And as in above example also some additional control for how many children hash can have or that some key is mandatory etc...

FYI - Firebase has no numChildren function available - (presumably because it could be a very expensive operation in the general case).
I'm not a fan of the Key => Value notation (as we may use this for "arrow functions") - I assume what is meant here is Object<BoardCoordinate, BoardMove> (using generics notation).

there is https://www.firebase.com/docs/web/api/datasnapshot/numchildren.html
what about
type BoardMoves {
  '0,0': BoardMove
  $coordinate: Map<BoardCoordinate, BoardMove>
  validate() { return this.numChildren() < 42 }
}

I stand corrected!

We DO have numChildren - implemented in the clients - it copies all data in child node to execute it.  But no such method in the Security and Rules API.

Question is - is $prop: Type necessary?  Your example could be:
path /gamesBoard/$gameId/moves is Map<BoardCoordinate, BoardMove>;

It's not the same.
With my example you have two additional conditions:

property '0,0' is mandatory
numChildren() (or any other) additional validation (e.g. sum of fields)

numChildren just isn't supported on the server Rules API (to which we are translating), I'm afraid.
What do you think of this for the required child case?
path /gamesBoard/$gameId/moves is Map<BoardCoordinate, BoardMove> {
  validate() { return this['0,0'] != null; }
}

yeah doable. But I would really like to see this in type. For code reuse.

Except we don't have $prop: Type.

You should be able to reuse a type like this:
type Moves extends Map<BoardCoordinate, BoardMove> {
  validate() { return this['0,0'] != null; }
}

path /gamesBoard/$gameId/moves is Moves;

But this is Not Yet Implemented ... but I THINK this will work.

That's also great.

Yeah the parametric Map works as well. I think that looks quite understandable for most people. +1

@matjaz - See https://github.com/firebase/bolt/pull/57 - generic types support (most of) what we discussed here.

Resolved by f0b108fcdae66ffdee82f6e01b77589d59ddc572


Frequently stop searching

Before submitting a bug report...

[X] This bug wasn't already reported.
(I have checked every bug report on GitHub)

[X] I've cleared the sessions folder.

Title

[X] The title is no longer "Title" and I edited it with the right error name.

Branch
master
Commit
Latest
Describe the bug
Actually, I have frequently a timeout error during the search process, both desktop or mobile. Also, after that the script exit but no notificaton sended, either wit ALWAYS conf.
Copy and paste your error
2024-08-11 22:58:02,348 [INFO] [BING] 12/14
2024-08-11 22:58:02,947 [DEBUG] terms=['stephen curry', 'stephen curry stats', 'stephen curry height', 'stephen curry underrated', 'stephen curry wallpaper', 'stephen curry shoes', 'stephen curry wallpaper 4k', 'stephen curry shooting form', 'stephen curry age', 'stephen curry under armour shoes', 'stephen curry net worth', 'stephen curry daughter']
2024-08-11 22:58:02,947 [DEBUG] rootTerm=stephen curry
2024-08-11 22:58:23,140 [DEBUG] in exit exc_type=<class 'selenium.common.exceptions.TimeoutException'> exc_value=Message:
Stacktrace:
#0 0x5582c6a956aa
#1 0x5582c6766600
#2 0x5582c67b5bcb
#3 0x5582c67b5eb1
#4 0x5582c67f9b24
#5 0x5582c67d88cd
#6 0x5582c67f704a
#7 0x5582c67d8643
#8 0x5582c67a8d31
#9 0x5582c67a979e
#10 0x5582c6a5d23b
#11 0x5582c6a611d2
#12 0x5582c6a4a5f5
#13 0x5582c6a61d62
#14 0x5582c6a2f23f
#15 0x5582c6a84e48
#16 0x5582c6a85020
#17 0x5582c6a9447c
#18 0x7f62a2efb134
traceback=<traceback object at 0x7ff320451900>
2024-08-11 22:58:23,569 [ERROR]
Traceback (most recent call last):
File "/home/deb/MS-Rewards-Farmer-master/main.py", line 39, in main
earned_points = executeBot(currentAccount, args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/deb/MS-Rewards-Farmer-master/main.py", line 271, in executeBot
searches.bingSearches()
File "/home/deb/MS-Rewards-Farmer-master/src/searches.py", line 138, in bingSearches
self.bingSearch()
File "/home/deb/MS-Rewards-Farmer-master/src/searches.py", line 174, in bingSearch
searchbar = self.browser.utils.waitUntilClickable(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/deb/MS-Rewards-Farmer-master/src/utils.py", line 74, in waitUntilClickable
return WebDriverWait(self.webdriver, timeToWait).until(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/deb/.local/lib/python3.11/site-packages/selenium/webdriver/support/wait.py", line 105, in until
raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:
Stacktrace:
#0 0x5582c6a956aa
#1 0x5582c6766600
#2 0x5582c67b5bcb
#3 0x5582c67b5eb1
#4 0x5582c67f9b24
#5 0x5582c67d88cd
#6 0x5582c67f704a
#7 0x5582c67d8643
#8 0x5582c67a8d31
#9 0x5582c67a979e
#10 0x5582c6a5d23b
#11 0x5582c6a611d2
#12 0x5582c6a4a5f5
#13 0x5582c6a61d62
#14 0x5582c6a2f23f
#15 0x5582c6a84e48
#16 0x5582c6a85020
#17 0x5582c6a9447c
#18 0x7f62a2efb134
2024-08-11 22:58:23,816 [INFO] [POINTS] Data saved for the next day.
Screenshots
NA
Value of dashboard variable
NA

Try upping the timeout time at that call

You mean higher than the 20 seconds default? Would try, but it seems a consistently time already!


Investigate crypto-cpp builds on MacOS Ventura

    Wild guess, but it might be a problem with crypto-cpp. MacOS Ventura has bumped the version to 13, and we probably lack compiled version so wheel tried to build from source. Sadly this is something that is at the moment impossible to automate using gh actions.

It would be great if someone that sports older version of macos could confirm. Alternatively, we can check using gh actions. Meanwhile, I will add it to my list to add compiled version for new system.
Originally posted by @THenry14 in https://github.com/software-mansion/starknet.py/pull/598#discussion_r1050593147

At the time of having this issue, fastecdsa indeed needed cmake in order to install on new macos. Since then however, a new version was released containing a needed wheels.
Therefore my recommendation is to inform users (in the docs under mac section - both amd and apple sillicon) that if installation fails, they should install cmake and try again (we could maybe add it to the dev/some other dependencies in the project?)


[vscode-graphql] 🐞 Language Server crashes when presented with Svelte files containing any JS/TS code

Edit: just realized this is of an issue in vscode-graphql and updated the issue title, but I can't relabel anymore now
Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Since the latest update of graphql.vscode-graphql, the LSP started crashes immediately when a Svelte file containing any code in a <script> tag is opened in VSCode, regardless of the actual code. The stack trace is:
TypeError: Ji.getModifiers is not a function
    at B7t (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4631:148)
    at age.handleVariableStatement (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4636:8914)
    at Q (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4647:2226)
    at /Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4647:3559
    at f (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4467:30007)
    at Object.p [as forEachChild] (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4467:33871)
    at Z.forEachChild (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4498:84222)
    at K4n (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4647:3543)
    at y9t (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4693:3638)
    at gFn (/Users/user/.vscode/extensions/graphql.vscode-graphql-0.8.22/out/server/index.js:4700:2873)

Expected Behavior
The LSP should be able to process Svelte files correctly, or at least it should preferably not crash when a parser fails and be able to serve completions for unaffected files.
Steps To Reproduce

Create a new project in VSCode with graphql.vscode-graphql enabled
Create a new Svelte file anywhere in the project
Add some JavaScript to the file:<script>
  let foo = 'bar'
</script>

Restart the language server by Show All Commands > VSCode GraphQL: Manual Restart, or alternatively, by closing VSCode and re-opening the project
The language server crashes 5 times and VSCode gives up restarting it

Environment

LSP Server Version: latest / 2.11.7 in local tests
OS: Ubuntu 22.04 / Windows 11 / macOS Sonoma 14.1.1
LSP Client: vscode-graphql at version v0.8.22

Anything else?
After a bit of digging around, the issue is that svelte2tsx, used by graphql-language-service-server, requires TypeScript as a peerDependency with the version spec ^4.9.4 || ^5.0.0. This project's root package.json requires TypeScript in version ^4.6.4 only, but svelte2tsx uses TypeScript compiler API not yet available in that version (namely, the missing getModifiers function mentioned in the stack trace, among a few others).
Testing with a clean reproduction project let me reproduce both, the error with an "outdated" TS version as well as confirm upgrading TS to at least 4.8.4 fixed it for me, even though I suppose since ^4.9.4 is required, it should also be used.
When then looking for a minimally invasive way to fix it for the language server, upgrading TypeScript for the entire project broke the build, but explicitly requesting Version ^4.9.4 in the vscode-graphql's package.json solves the problem.
If desired, I can provide a PR with this fix right away. Just wanted to check if there may be any concerns.

hey thanks for looking into this!
it appears this bug was introduced by this PR
https://github.com/graphql/graphiql/pull/3204
it isn't detectable in our tests because of the esbuild bundle it would seem. a PR would be awesome! you may need to add typescript to the esbuild externals list


DataFrance Store does not update on cell edit

Main Issue
Cell editing does not update the sheet.dataFrame.store. For example:

the 3 console logs on the store you see are before I hit clear, after, and after I entered the "AAA" value into cell 0,0

@dkrasner I have just pushed a fix for this straight to master. Let me know if there's any trouble. Otherwise feel free to close.

@darth-cheney cool closing tu


I'm following an old tutorial (https://www.youtube.com/watch?v=mScd-Pc_pX0), so stuff might be outdated.

In particular, I'm not sure the docker compose notation is still modern. We'll see!

Note: this is for the new-django-project branch.

Okay, I've made it to the point where we're adding static and media files. I'm facing a bug where the files are being saved in the wrong place (./app/vol/web/media vs. ./data/web/media). I don't know what's going on there.
From my perspective, there are two reasonable paths forward. One is to proceed with the tutorial. I iron out the bugs with static and media files and then I proceed to set up the reverse proxy stuff. I stop short of uploading to AWS because AWS doesn't support docker compose any more, but otherwise everything is production ready.
Another approach is to stop here. Revert back to the pre-static/media commits to when I have a barebones working Django project. Then start setting up things like Angular and the actual backend. I can't think of any static or media files that I'll have to work with, at least for now. The deployment configuration is going to have to be different because the big cloud providers aren't working with docker compose anymore. Kubernetes is something that I don't want to have to set up right now.

I fixed the bug (I had missed some '/' characters somewhere). I think that I should proceed with the tutorial. No matter which deployment setup I go with, it will need something like nginx. Better to design my application around it now than to try to integrate it after the fact.

Seems to work locally, I think this is fine.


Enforce invariants during kpt fn

There are two parts of this task:

Define the invariants that needs to be held as functions are applied during kpt fn render.
Implement checks for invariants

Duplicate resource invariant
A resource is identified by the following properties:

GVKNN (group, version, kind, namespace (if namespaced resource), name)
filepath of the resource

kpt fn render should fail with error whenever it detects duplicate resource in the input or output of the function in the pipeline. Alternative behavior such as merging the two resources if part of a same file or overwriting the resource with the copy that appears later in the resourceList.items can also be supported with some additional configuration. But being conservative here and enforcing this invariant seems like a good starting point
Validators should not mutate resources
validators functions should not mutate resources. One way to ensure this will be to discard the resourceList.items from the output of validators.
Meta resources should not be mutated
Kptfile and function config files should not be mutated.
Any other ?
/cc @frankfarzan @Shell32-Natsu @mortent @phanimarupaka

Re "filepath of the resource": Orchestrator must handle multi-object files, so filepath is not a primary key.

Are all resources required to have metadata.name? How about function configs?

Generalizing this to both fn render and fn eval. For example, with eval, if you have two function configs with same GKNN, you get such an error today:
"failed to run function: may not add resource with an already registered id: ~G_v1_ConfigMap|~X|setterValues"

if you have two function configs with same GKNN, you get such an error today:
"failed to run function: may not add resource with an already registered id: ~G_v1_ConfigMap|~X|setterValues"

Can you share more about the scenario here ? (especially how were you invoking these)
Are these functionConfigs referred in the pipeline ? If yes, then kpt fn eval should exclude those by default.  @Shell32-Natsu

pipeline:
  mutators:
    - image: gcr.io/kpt-fn/apply-setters:v0.1
      configPath: setters.yaml
    - image: gcr.io/kpt-fn/apply-setters:v0.1
      configPath: setters2.yaml

Where both setters.yaml and setter2.yaml are identical (Same GVKNN):
This succeeds:
$ kpt fn render

This fails:
$ kpt fn source \
  | kpt fn eval --image gcr.io/kpt-fn/set-namespace:v0.1 - -- namespace=mywordpress \
  | kpt fn sink
failed to run function: may not add resource with an already registered id: ~G_v1_ConfigMap|~X|setterValues

@frankfarzan @Shell32-Natsu The use-case above reveals another issue. fn eval and fn render excludes meta resources from the input unless specified. So in this case fn source whose primary job is to generate input for the function execution should be consistent with fn eval and fn render and should exclude meta resources by default.

Filed https://github.com/GoogleContainerTools/kpt/issues/2033 for the fn source consistency.

created a task bug for enforcing metadata.name for functionConfig: https://github.com/GoogleContainerTools/kpt/issues/2105

We have settled on new behavior in kpt where kpt fn commands include meta resources to be included in the function input and their modification is allowed.
kpt fn render ignores the mutations done by validators.
Since we have settled on new behavior, closing this issue. We can reopen this issue if we have any requirements missing.


[Bug]: References are not displayed in Author Guidelines page

Is there an existing issue for this?

[X] There are no existing issues

Priority of this issue

[ ] high
[X] medium
[ ] low

What browsers are you seeing the problem on?
No response
Contact Details
No response
Describe the current behaviour
Current behaviour:

References are not getting displayed
Describe the expected behaviour
Display the expected result!
Steps to Reproduce

Go to
Click on
Enter ...

Add more steps here,

try this
https://bit.ly/4gwC82r
I put the necessary dlls in the archive

The issue is fixed and merged to main


[backend] make deployment field optional in API and DB

For v1 we don't need to be concerned about deployment type. To unblock FE I've made it optional here and defaulting the DB value to COHERE_PLATFORM and updated the unit tests accordingly. Also generating new client for FE.
AI Description

This PR introduces changes to the Agent model and associated tests. The deployment field is now optional and can be set to null. Additionally, a new tools field has been added, which is an array of ToolName enums. This enum includes options such as 'Wikipedia', 'search_file', and 'Python_Interpreter'.
The following changes have been made:

The deployment field in the Agent model is now optional and can be null.
A new tools field has been added to the Agent model, which is an array of ToolName enums.
Updated the CreateAgent and UpdateAgent types to include the new tools field and make the deployment field optional.
Removed the Auth type and added new types for JWTResponse, ListAuthStrategy, and Logout.
Updated the DefaultService class to use the new types and handle the optional agentId parameter in the listToolsV1ToolsGet method.
Removed the test_create_agent_missing_deployment test cases as they are no longer applicable with the optional deployment field.

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

Please upload report for BASE (main@24c4164). Learn more about missing BASE report.

:exclamation: Current head 38b2ff0 differs from pull request most recent head 9f72f89
Please upload reports for the commit 9f72f89 to get more accurate results.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #213   +/-   ##
=======================================
  Coverage        ?   88.14%
=======================================
  Files           ?      137
  Lines           ?     4545
  Branches        ?        0
=======================================
  Hits            ?     4006
  Misses          ?      539
  Partials        ?        0

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


Request for Final year project

How are you? I have seen your videos on YouTube. Physics simulation videos are very beautiful and I am a final semester student of bachelor degree in physics and my final year project is also on physics simulation. It didn't work. Can you help me? Make me project at reasonable price.I share more details of the project with you if you want to make it.

Hi, I can collaborate on research projects but I do not make bachelor's thesis

The simulations with time dependent Schrodinger equation  in your repositories are not running. You have updated a lot of things in it. Please tell me the solution. Explain in detail what needs to be done.

Ok kindly share your what's app number or other contact number I will
discuss you about my project kindly response it I will be very thankful to
you.
The simulations with time dependent Schrodinger equation in your
repositories are not running. You have updated a lot of things in it.
Please tell me the solution. Explain in detail what needs to be done.
On Sat, 24 Feb 2024, 10:19 am Rafael de la Fuente, @.***>
wrote:

Hi, I can collaborate on research projects but I do not make bachelor's
thesis
—
Reply to this email directly, view it on GitHub
https://github.com/quantum-visualizations/qmsolve/issues/31#issuecomment-1962259688,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BF67U76Z6WLFLOW2WDGD44DYVFZ7XAVCNFSM6AAAAABC5UXNZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRSGI2TSNRYHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Why Code not woring on second vue instance ?

Hello! Please use our forum or Gitter for questions like this. The repo issue list is strictly for bug reports and feature requests, as specified in the contributing guidelines. If you're sure this is a Vue's bug, please create a reproducing fiddle.

Thanks


Join/Parts not in sync with mattermost

Every so often I will see people leaving a channel and just after that it will be marked as they joined it again, followed by a message from System saying that person has left the channel. And later when that person actually rejoins the channel I will only get the message from System and no part/join dance again.
19:47 -!- XXX [foo@bar] has left #off-topic []
19:47 -!- XXX [foo@bar] has joined #off-topic
19:47 < XXX> XXX has left the channel.
Day changed to [..]
15:34 < XXX> XXX has joined the channel.

I am not sure how to debug this, I suppose there is some event sent and I should be able to verify which is which. Please tell me if there is an easy way to debug this on production, without taking down the server or running it in debug mode. If not I'll try to spawn a test environment.

Fixed in master, release v0.12.1 soon

Hi! I'm running 9b09123 (master) and I'm still observing similar issues.
The joins are displayed right before a user writes their first message in a channel, and not when they actually join the channel.

What mattermost version ?
What total amount of users on the server ?

Mattermost 4.8.1, about 130 users

Ok, 130 users isn't hitting a user limit of 50000 users :)
So happens this:

when the user is already on the channel and then types a message ?
OR
when the user joins the channel (not on the channel before) and types a message for the first time ?

On 2018/05/01, Wim wrote:

Ok, 130 users isn't hitting a user limit of 50000 users :)
So happens this:

when the user is already on the channel and then types a message ?
OR
when the user joins the channel (not on the channel before) and types a message for the first time ?

The latter!
A user joins a channel, nothing happens. He then sends a message, and
what is displayed in the bridge is:

-!- foo [..] has joined #channel
< foo> message

As a side effect, the user doesn't appear in the userlist of the client
until the join action is sent, which is also annoying.
--
Maxime “pep” Buquet

Fixed in master

Thanks!


Add support for the new AWS Bedrock Nova models

Hey, I am not too deep into the bedrock ai model design in spring ai but I have seen that bedrock models need special care codewise: https://github.com/spring-projects/spring-ai/tree/main/models/spring-ai-bedrock/src/main/java/org/springframework/ai/bedrock
Is it possible to add the support for the newest model Nova?
Expected Behavior
Being able to select one of the nova models (Nova Micro, Nova Lite, Nova Pro, Nova Premier) for usage in spring ai projects.
Current Behavior
As seen in the current repo state, and as stated in the documentation, nova is currently not listed as a supported ai model to work with.
Context
In the official Kotlin SDK provided by AWS it is just necessary to change the model id to use. Why is it with spring ai that complicated?
Looking forward to discuss this.

Im also very interested in support of Nova. I cannot understand why spring.ai has to be so bulky.

+1

@Exceptionflug The essential Bedrock Nova support is already available with Spring AI 1.0.0-M4 via the Spring AI Bedrock Converse API integration.

Following the Bedrock recommendations, Spring AI is transitioning to using Amazon Bedrock's Converse API for ALL Chat conversation implementations in Spring AI.

You can consider the old, InvokerModel API-based Spring/Bedrock integrations for Chat Completion as deprecated. However, since the Converse API does not support embedding operations, these will remain in the current API, and the embedding model functionality in the existing InvokeModel API will be maintained.
Here is the list of all Converse API supported models that includes the latest Nova models as well.
So you can use the Spring AI Bedrock Converse API to connect to Nova Pro, Nova Lite, and Nova Micro like this:
<dependency>
   <groupId>org.springframework.ai</groupId>
   <artifactId>spring-ai-bedrock-converse-spring-boot-starter</artifactId>
</dependency>

application.properties :
spring.ai.bedrock.aws.region=<YOUR REGION>
spring.ai.bedrock.aws.access-key=${AWS_ACCESS_KEY_ID}
spring.ai.bedrock.aws.secret-key=${AWS_SECRET_ACCESS_KEY}
spring.ai.bedrock.aws.session-token=${AWS_SESSION_TOKEN} # only if you use temp sessions.

spring.ai.bedrock.converse.chat.options.model=amazon.nova-pro-v1:0

spring.ai.bedrock.converse.chat.options.temperature=0.8
spring.ai.bedrock.converse.chat.options.max-tokens=1000

Text completion:

String response = chatClient.prompt("Who are you?").call().content()

Text + image:

String response = chatClient.prompt()
   .user(u -> u.text("Explain what do you see on this picture?")
	.media(MimeType.valueOf("image/png"), new ClassPathResource("/test.png")))
   .call()
   .content();

Tool calling

record WeatherRequest(String city, String unit) { }
record WeatherResponse(int temperature, String unit) {}

String toolResponse = chatClient.prompt()
   .user("what to wear in Amsterdam today?")
   .functions(FunctionCallback.builder()
	.description("Gets current weather by city name")
	.function("getCurrentWeather", (WeatherRequest request) -> new WeatherResponse(15, request.unit()))
	.inputType(WeatherRequest.class)
	.build())
   .call()
   .content();

Finally, the https://github.com/spring-projects/spring-ai/pull/1888 will improve the multimodality support, allowing you to use the Video and Document input modalities as well.
Hope this helps!

Thank you for your detailed response! This clarifies a lot. I will close this now since the feature is already usable via the Converse API.

Glad it helped.
The https://github.com/spring-projects/spring-ai/pull/1888 will bring full multimodality support for Nova (e.g. video, pdf, html, md , docs, xslx ...) and improved docs.


Fix keyboard reload issue in TextInput component

Summary:
This pull request addresses a keyboard mode resetting issue in the TextInput component that occurs when the value prop is programmatically updated. Previously, upon entering the first character in number mode, the keyboard would reload and revert to its default letter mode, disrupting the user input experience. This fix ensures that the keyboard remains in the expected mode, improving usability for scenarios that require sustained number input.
Changelog:
[GENERAL] [FIXED] - Prevent TextInput keyboard from resetting to default mode on value update.
Test Plan:
To verify the changes, the following steps were taken:

Environment Setup: The test was conducted in a standard development environment using a React Native project where the TextInput component is extensively used.
Test Execution:

A TextInput component was set up to accept numeric input with an initial state.
The value prop was programmatically updated to simulate user input.
Observed that the keyboard remains in numeric mode after the first character is entered and does not reset to the default alphanumeric keyboard.

Result:

Before the change: The keyboard would switch back to letter mode after the first input when the value prop is updated.
After the change: The keyboard stays in number mode, reflecting the user's choice throughout the input session.

Screenshots/Videos:

https://github.com/AliRezaBeigy/react-native/assets/46004092/b66837f2-8280-48ab-b32f-ca440d984c6b
https://github.com/AliRezaBeigy/react-native/assets/46004092/a37d2196-34e4-4085-aa8d-a32edc3549bc
These tests confirm that the code changes effectively prevent the keyboard from resetting when the value prop changes, maintaining the desired keyboard state across updates.

i found this related issue on iOS too while the multiline props is true
https://github.com/facebook/react-native/assets/46004092/b238beab-e7dd-4655-8c55-7bd047014baf
https://github.com/facebook/react-native/assets/46004092/6205ee8b-afae-4973-be05-d9314c25031a


Adjust splaying behavior

added parameter distanceIncrement to SpreadClusterSplayDelegate which increases the splay radius depending on the amount of markers
added function _modifyDisplacedMarker to SuperclusterLayerImpl which updates the DisplacedMarkers when a ModifyMarkerEvent is fired

I need to cherry-pick the changes not related to flutter_map upgrade


Add new event and functions

Recent versions of libgphoto2 added a new camera event and three additional functions.

Are we able to get this merged? It would be a shame if this repository / the crates.io crate is no longer maintained.


feat: create alpha-test-app

closes #39
Description
During the alpha phase of onyx, we can make use of a bare vitest starter app to try out onyx components in a user-near environment and detect problems that Storybook can not show.
The alpha-test-app will be removed once onyx leaves the alpha phase. As it is temporary, it will not be mentioned in our documentation app.
Checklist

[x] The added / edited code has been documented with JSDoc
[x] All changes are documented in the documentation app (folder apps/docs)
[x] If a new component is added, at least one Playwright screenshot test is added
[x] A changeset is added with npx changeset add if your changes should be released as npm package (because they affect the library usage)

Seems like the pnpm lock file is broken on the branch, can you run pnpm install to fix it?

Seems like the pnpm lock file is broken on the branch, can you run pnpm install to fix it?

can you try again with the latest version? did that already

Seems like the pnpm lock file is broken on the branch, can you run pnpm install to fix it?

can you try again with the latest version? did that already

Yes it works now 👍

not again 😭
what should I do? restart it?

Yes you can restart for now...I am working on a fix or maybe we should just remove the contributor count

Yes you can restart for now...I am working on a fix or maybe we should just remove the contributor count

restart worked :)
yeah, disabling at least until we have it stable would make sense


tests.vsprojects/x64/Debug/h2_census_nosec_test.exe binary_metadata (c,windows_x64_vs2015)

https://grpc-testing.appspot.com/job/gRPC_portability_master/974/language=c,scenario=windows_x64_vs2015/testReport/junit/(root)/tests/vsprojects_x64_Debug_h2_census_nosec_test_exe_binary_metadata/

The build has disappeared from Jenkins since then. Please copy the relevant logs to the issue comments, or hit the "Keep this build forever" button on Jenkins.


[MM-42443]: fix selectionRange updates in edit_post component

Summary
Fixed updating the inputs selectionRange by using the state to determine whenever it has to be manually updated.
This should go hand in hand with #10034 since changes in this PR might affect it directly.
I discovered that only italic markdown is being handled differently and the italic markers are excluded from the selection. I would propose to add other markers as well (e.g. bold, strikethrough, etc.) and let the manual update on selectionRanges be handled by the component (maybe write a getter function in Utils, but let the actual updates to the input be handled inside the component)
Ticket Link
MM-42443
Related Pull Requests
n/a
Screenshots
n/a
Release Note
fixes markdown insert link behavior in edit post input field

@amyblais 2/3 for cloud release

Triggering e2e testing with options:
<nil>

Thanks @michelengelen
Tested, looks good to merge - pending e2e report

Verified insert link keyboard shortcut is working as expected in edit post textbox.

@michelengelen
E2E report  looks good overall
One test failure may be related - this test case is passing when testing manually
MM-T2139

hi @jgilliam17 ... I adjusted the script to wait a bit before searching for the postMesage_<postId> container ... it now should not fail anymore.

Thanks @michelengelen Can you please update the branch so I can re-run the e2es. Thanks

@jgilliam17 merged master just now ... feel free to re-run whenever you like!

/e2e-test

The command /e2e-test requires the build docker check to pass.

@michelengelen unfortunately, the same test is failing again. See report here

@jgilliam I increased the wait time for the test and it runs on my local machine just fine.

/e2e-test

The command /e2e-test requires the build docker check to pass.

/e2e-test

The command /e2e-test requires the build docker check to pass.

E2E test running here

Thanks @michelengelen I am starting to think this may be a flaky test MM-T2139, full report here
@saturninoabril thoughts?

I think most of those were recently addressed, especially by https://github.com/mattermost/mattermost-webapp/pull/10053. I'll update this PR then will run E2E again. Thanks Jelena!

/update-branch

Oopss, sorry, I thought other failed tests from the report. I'll take a look what's going on with MM-T2139.

E2E test at https://automation-dashboard.vercel.app/cycles/241b1375-0313-4088-81ec-4908c92abf34

@jgilliam17 MM-T2139 should be good now. Please take a look again.

Thanks @michelengelen - looks good to merge.
And thank you @saturninoabril for taking care of the flaky test.
E2E report looks good.

/cherry-pick cloud

Cherry pick is scheduled.


Refactoring indexer: cleaned up options and logging

Per thread logging is very verbose for collections that have lots of file segments. Added option to control, off by default.
Got ride of dryrun option... never used; simplifies code path.

Hi @r-clancy - I adopted your suggestion of setting different log levels.
I was going through the code again, and decided to remove the "unindexed" counter. The rationale here is that the counter code path was really confusing. Some things were being doubled counted - for example, a "skipped" doc inside a generator was also counted as an "unindexed". But a whitelisted doc was counted as "skipped" but not "unindexed".
With "unindexed" counts removed, everything is only counted in one category, which I believe is the desired behavior.
What do you think?
Also added more tests.

@x389liu @bazingagin can you follow along also please to start learning about Anserini's indexing internals?

I was going through the code again, and decided to remove the "unindexed" counter. The rationale here is that the counter code path was really confusing. Some things were being doubled counted - for example, a "skipped" doc inside a generator was also counted as an "unindexed". But a whitelisted doc was counted as "skipped" but not "unindexed".
With "unindexed" counts removed, everything is only counted in one category, which I believe is the desired behavior.
What do you think?

Agreed, the counters are less confusing now.
LGTM, verbose/quiet logging is much cleaner and simpler now.

hi @r-clancy I rewrote all the index usage descriptions to be consistent grammatically, with correct metavars, etc.
I rewrote all -es options to begin with "Elasticsearch" and all -solr options to begin with "Solr" so it's very clear.
A second pair of eyes would be appreciated.
$ target/appassembler/bin/IndexCollection
Option "-collection" is required
 -bm25.accurate               : Boolean switch to use AccurateBM25Similarity (computes accurate
                                document lengths). (default: false)
 -collection [class]          : Collection class in package 'io.anserini.collection'.
 -es                          : Indexes into Elasticsearch. (default: false)
 -es.batch [n]                : Elasticsearch batch index requests size. (default: 1000)
 -es.connectTimeout [ms]      : Elasticsearch (low level) REST client connect timeout (in ms).
                                (default: 600000)
 -es.hostname [host]          : Elasticsearch host. (default: localhost)
 -es.index [name]             : Elasticsearch index name.
 -es.password [password]      : Elasticsearch password. (default: changeme)
 -es.poolSize [num]           : Elasticsearch client pool size. (default: 10)
 -es.port [port]              : Elasticsearch port number. (default: 9200)
 -es.socketTimeout [ms]       : Elasticsearch (low level) REST client socket timeout (in ms).
                                (default: 600000)
 -es.user [username]          : Elasticsearch user name. (default: elastic)
 -generator [class]           : Document generator class in package 'io.anserini.index.generator'.
 -index [path]                : Index path.
 -input [path]                : Location of input collection.
 -keepStopwords               : Boolean switch to keep stopwords. (default: false)
 -language [language]         : Analyzer language (ISO 3166 two-letter code). (default: en)
 -memorybuffer [mb]           : Memory buffer size (in MB). (default: 2048)
 -optimize                    : Boolean switch to optimize index (i.e., force merge) into a single
                                segment; costly for large collections. (default: false)
 -quiet                       : Turns off all logging. (default: false)
 -shard.count [n]             : Number of shards to partition the document collection into.
                                (default: -1)
 -shard.current [n]           : The current shard number to generate (indexed from 0). (default: -1)
 -solr                        : Indexes into Solr. (default: false)
 -solr.batch [n]              : Solr indexing batch size. (default: 1000)
 -solr.commitWithin [s]       : Solr commitWithin setting (in seconds). (default: 60)
 -solr.index [name]           : Solr index name.
 -solr.poolSize [n]           : Solr client pool size. (default: 16)
 -solr.zkChroot [path]        : Solr ZooKeeper chroot (default: /)
 -solr.zkUrl [urls]           : Solr ZooKeeper URLs (comma separated list).
 -stemmer [stemmer]           : Stemmer: one of the following {porter, krovetz, none}; defaults to
                                'porter'. (default: porter)
 -storeDocvectors             : Boolean switch to store document vectors; needed for (pseudo)
                                relevance feedback. (default: false)
 -storePositions              : Boolean switch to index store term positions; needed for phrase
                                queries. (default: false)
 -storeRawDocs                : Boolean switch to store raw document text. (default: false)
 -storeTransformedDocs        : Boolean switch to store transformed document text. (default: false)
 -threads [num]               : Number of indexing threads.
 -tweet.deletedIdsFile [file] : File that contains deleted tweet ids (longs), one per line; these
                                tweets will be skipped during indexing. (default: )
 -tweet.keepRetweets          : Boolean switch to index retweets. (default: false)
 -tweet.keepUrls              : Boolean switch to keep URLs. (default: false)
 -tweet.maxId [id]            : Max tweet id to index (long); all tweets with larger tweet ids will
                                be skipped. (default:<PHONE_NUMBER>854775807)
 -tweet.stemming              : Boolean switch to apply Porter stemming while indexing tweets.
                                (default: false)
 -uniqueDocid                 : Removes duplicate documents with the same docid during indexing.
                                This significantly slows indexing throughput but may be needed for
                                tweet collections since the streaming API might deliver a tweet
                                multiple times. (default: false)
 -verbose                     : Enables verbose logging for each indexing thread; can be noisy if
                                collection has many small file segments. (default: false)
 -whitelist [file]            : File containing list of docids, one per line; only these docids
                                will be indexed.
Example: IndexCollection -collection [class] -generator [class] -input [path] -threads [num]


OCPBUGS-13021: Add internal/external elb tags to subnets

What this PR does / why we need it:
Internal/external tags are used by the aws cloud provider to select subnets assigned to load balancers. In standalone OCP, the installer sets these tags on subnets it creates. This commit sets the same tags for hypershift clusters.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #OCPBUGS-13021
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.

/retest-required

/lgtm

/cherry-pick release 4.13
For consistency in our cli/e2es

/cherry-pick release-4.13


[HTTP][Caching] Original request from the client disregarded when sending a validation request

The current behaviour of the HTTP caching implementation is to send a new HTTP request when the response needs to be validated. Instead of doing this, need to add the relevant precondition header(s) to the original request itself and forward it to the origin server.
See the following:
https://tools.ietf.org/html/rfc7234#section-4.3
https://tools.ietf.org/html/rfc7232#section-3.2
https://tools.ietf.org/html/rfc7232#section-4.1

Closing since this was fixed for 1.1.7, 1.2.6 and in master.


Fuel example is out of date

Didn't get updated after Model/GP switch

@tonystao

(also should be updated w/ working interactives...)

Shoot everything's changed in iPython widgets. They're not even bundled by default anymore???

This example only exists as an ipython notebook at the moment, correct?

Yup.

Working Fuel example is currently living here: http://nbviewer.ipython.org/github/convexopt/gpkit/blob/master/docs/source/ipynb/Fuel/Fuel.ipynb.
This may change.


Eclipse V18

Hi Dominic, is CC Generator available for Eclipse V18?
Thanks

Hi there, we are on v18 and this script worked without recompiling. Are you running into an issue?

Hi Dominic, I downloaded the source code (you do not provide the compiled script dll right?) but using Visual Studio 2022 I cannot compile the solution because some files (like AriaConnection.cs) are missing. Am i doing something wrong?
Thanks!

When you downloaded the source did you pull from the "releases" section?
The OSU Helpers are required for building the application, and sounds like
those are missing from your build.
Written on my mobile device. Please excuse any typographical errors.
On Thu, Dec 19, 2024, 4:27 AM ste-agolabs @.***> wrote:

Hi Dominic, I downloaded the source code (you do not provide the compiled
script dll right?) but using Visual Studio 2022 I cannot compile the
solution because some files (like AriaConnection.cs) are missing. Am i
doing something wrong?
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/ddicostanzo/CC-Gen/issues/2#issuecomment-2553179630,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE4YIFVKYZASYEIKTEZCMML2GKGP5AVCNFSM6AAAAABT2QTQPSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJTGE3TSNRTGA
.
You are receiving this because you commented.Message ID:
@.***>

Hi Stefano, if you look in the project OSU_Helpers_Public, there are some files "Aria_Connection" among them that were purposefully excluded from the uploaded version. They show as dotted box outlines and aren't able to be found. If you exclude those from the project and try to build, I was able to get the released version to build. You can also build against the v18 ESAPI DLLs if you prefer. Hope this helps. I'll work on updating the release to rectify the problem.

Hi Stefano, I've updated the release to v1.0.1 to exclude the files and it should be able to be built now without issues. Thanks for bringing this up.


Doc: Add a new Extention Feature(extentio8.kt)

Description:

add a new extention8.kt

PR Instruction:

Issue will be assigned on a first come first serve basis.
If you are thinking to make a pr then at first be sure to get assigned.
before pushing your PR please make sure to fetch upstream.
your pr should be pushed on the develop branch only.
try to complete your assigned task within 3 to 5 days and if you need more time then please let me know. As the program has limited time only so please try to complete as fast as you can.
With your PR try to send a screenshot or screen recording of your work

I want to work on this issue

Sure go for it.


Exporting table to .sql script for Postgres with NaN values

System information:

Windows 10
21.0.0.

Connection specification:

Postgres 12

Describe the problem you're observing:
When using the export table to .sql script I have a column of type numeric which has multiple rows with the value NaN.  The export script generated creates an output like:
INSERT INTO public.loads(interval_timestamp,load_id,forecast_timestamp,status,fixed_value,value) VALUES
('2022-04-02 07:00:00-07',9971,'2021-04-01 07:00:00-07',false,false,NaN);
the script above will provide an error at NaN.  If the script is modified to below:
INSERT INTO public.loads(interval_timestamp,load_id,forecast_timestamp,status,fixed_value,value) VALUES
('2022-04-02 07:00:00-07',9971,'2021-04-01 07:00:00-07',false,false,'NaN');
it will insert as expected.

thanks for the report

Fixed for PostgreSQL:
select 'NaN'::float4, '-Infinity'::float4, '+Infinity'::float4;

Produced SQL script:
INSERT INTO "select 'NaN'::float4, '-Infinity'::float4, '+Infinity'::float4" ("float4","float4","float4") VALUES
	 ('NaN','-Infinity','Infinity');

And for Oracle:
SELECT BINARY_DOUBLE_NAN, -BINARY_DOUBLE_INFINITY, BINARY_DOUBLE_INFINITY FROM DUAL;

Produced SQL script:
INSERT INTO DUAL (BINARY_DOUBLE_NAN,"-BINARY_DOUBLE_INFINITY",BINARY_DOUBLE_INFINITY) VALUES
	 ('NaN','-Infinity','Infinity');

verified


batch % im2col_step_ == 0INTERNAL ASSERT FAILED

good job！
When I train bevformer_tiny on two a800s, samples_per_gpu=16.
i got an error
RuntimeError: batch % im2col_step_ == 0INTERNAL ASSERT FAILED at "/tmp/mmcv/mmcv/ops/csrc/pytorch/cuda/ms_deform_attn_cuda.cu":255, please report a bug to PyTorch. batch(%d) must divide im2col_step( %d)9664
May I ask how to solve this problem?

I meet the same problem. Have you solved this problem?


Ensuring inline citations are displayed correctly

Make sure that Document PDF references are displayed in a functional way.
Acceptance Criteria

[ ] Cross linking (link from inline <-> bibliography)

Hey @bwbaker1, this ticket is meant to outline work related to references in the Document PDF. Right now, we're displaying in-line citations as simply a superscript of the reference number. For example:
PDF:

This is the same as the view mode, but without the tooltip:

We're currently thinking of adding in cross linking between the in-line citation and the reference section – clicking on the in-line citation would bring you to the reference section, and clicking on the reference would bring you to where it's located.
We'll investigate to make sure this is standard practice. If the users have any requirements here, please let us know.
cc @batpad

@wrynearson @batpad I think linking the in-line and reference sections as you explained would be excellent. This is standard practice for online accessible articles - though I don't recall seeing the linkage from the reference (in ref section) to where it is located in the document.  This would be very useful!
I do want to note that the in-line citation must be (author, year) and not a superscript number. For example: (Doe & Smith, 2023)

Duplicate of #651, closing


Submit Transactions Agent Pallet proposal

Project Abstract
The Transactions Agent Pallet is designed to provide a proxy signing tool for the Substrate ecosystem, aiming to enhance the user interaction experience on Substrate chains. It includes signer accounts and proxy accounts, allowing a user’s wallet address (signer account) to perform on-chain transactions via a token-holding wallet address (proxy account) even without holding tokens.
Grant level

[x] Level 1:  Up to $10,000, 2 approvals
[ ] Level 2:  Up to $30,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (project_name.md).
[x] I have read the application guidelines.
[x] Payment details have been provided (Polkadot AssetHub (USDC & DOT) address in the application and bank details via email, if applicable).
[x] I understand that an agreed upon percentage of each milestone will be paid in vested DOT, to the Polkadot address listed in the application.
[x] I am aware that, in order to receive a grant, I (and the entity I represent) have to successfully complete a KYC/KYB check.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[ ] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @_______:matrix.org (change the homeserver if you use a different one)

I have read and hereby sign the Contributor License Agreement.

recheck in this Pull Request

Thanks for the application @OIyoung we don't pay to ETH mainnet addresses anymore, please see our payment options and let us know if you are willing to be paid in 50% vested DOT?

The system you're describing sounds very similar to pure proxies. What are the differences? Are you hearing from others that there is a great need in the ecosystem for this?
What advantages does a system like this have over using the existing proxy pallet?

We are willing to change the payment method and accept 50% vested DOT.
In fact, we have used pure proxies before, and it is indeed a great feature. Although Transactions Agent Pallet, pure proxies, and proxy pallet may sound similar, they serve different functions and aim to achieve different goals.
Their differences are as follows:
	1.	Problems They Solve
	•	Pure proxies reduce the exposure of the primary account’s private key.
	•	Transactions Agent Pallet improves the conversion rate from acquiring users to user registration by allowing users to use Substrate applications without needing tokens.
	2.	Functions of Accounts
	•	Pure proxies are new accounts created (not assigned) by a primary account. The primary account acts as a proxy on behalf of the pure proxy. Pure proxies are keyless, non-deterministic accounts, meaning they do not have private keys but possess randomly generated addresses.
	•	The proxy accounts in the Transactions Agent Pallet are controlled accounts with private keys. Their primary tasks include not only signing transactions on behalf of others but also paying gas fees for the signed accounts and implementing various payment strategies.
	3.	User Entry Barriers
	•	Proxy pallet requires users to pay at least one transaction’s gas fee for authorization.
	•	Transactions Agent Pallet is barrier-free for users, allowing them to make calls without gas.
In simple terms:
	•	Pure proxies enhance the security of the primary account’s signature, serving as a security tool that allows users to interact with DAPPs more safely.
	•	Proxy pallet addresses the issue of gas fees during transactions but has an entry barrier requiring authorization.
	•	Transactions Agent Pallet is a tool designed for operational purposes, enabling more users to utilize DAPPs.
These features are not mutually exclusive; they can even complement each other. For instance, a sub-key can be generated via pure proxies, and Transactions Agent Pallet can then be used to call the proxy pallet for authorized transactions.
Regarding the project’s advantages:
We are aware that the Substrate ecosystem has many proposals aimed at solving technical and security issues. However, few consider where the users are and how to acquire them. Transactions Agent Pallet provides a marketing tool for the Substrate ecosystem, attracting more users and converting a higher proportion of them into paying users.


SyntaxError in /

Error in ClashLeaders
SyntaxError in /
missing ) after argument list
View on Bugsnag
Stacktrace
SyntaxError: missing  after argument list:1 -

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
SyntaxError in /


ENH Increase rate limit

Issue https://github.com/silverstripe/silverstripe-framework/issues/9474
Related PR for limiting rate limit to only POST requests https://github.com/silverstripe/silverstripe-framework/pull/9982

This seems good to me. As you can see below, we double the number of necessary requests (assuming only POST is required) for a single good verification, so doubling the number of allowed failures before rate limiting kicks in feels okay too, though 20 also feels like maybe it's too many (and if we haven't had lots of complaints, maybe it's not really that big of a deal and we could just leave this alone - given if only POST requests count now then it's far less likely to cause issues already)?
Coupled with silverstripe/silverstripe-framework#9982, where now only POST requests would be counted, does that change how many requests MFA would make? As I see it, here's what happens (for TOTP at least, I don't know if other methods like WebAuthN require additional calls):

GET request to /Security/login (previously was counted, now won't be)
POST request to /Security/login/default/LoginForm (1 POST request towards rate limit)
GET request to /Security/login/default/mfa (previously was counted, now won't be)
GET request to /Security/login/default/mfa/verify/totp (previously was counted, now won't be)
POST request to /Security/login/default/mfa/verify/totp with TOTP code for example (1 POST request towards rate limit)
(More singular POST requests to the verify endpoints if the first verification fails)
GET request to /Security/login/default/mfa/complete once MFA is complete (previously was counted, now won't be)

So all in all, previously there were 6 requests minimum that counted, now with these two changes there would only be two requests that count (assuming the user correctly enters their request) - so I'd say this almost feels unnecessary.

My thinking here is:

The previous max limit of 10 pre-dates MFA, and my assumption of the intention was "10 login attempts from a single IP" rather than "10 requests"
MFA doubles the required number of POST requests, for increase this to 20 requests still retains the "10 login attempts from a single IP" - though unfortunately this also allows "1 correct user/pass + 19 attempts at MFA"

so I'd say this almost feels unnecessary.

So what you're saying is
a) do keep the PR that restricts rate-limiting to POST requests
b) discard the PR that bumps the limit when MFA is installed from 10 to 20
?

Basically I think that implementing both changes feels like a bit of overkill, so my opinion is to do one or the other, but not both. I've flip-flopped on this a bit, so I feel like maybe another opinion is needed.
I think the best option is to merge this change and not merge silverstripe/silverstripe-framework#9982. This means that both GET and POST requests will be counted toward the limit, and therefore we increase the number of allowed attempts when using MFA (because more requests are needed to successfully login).
Keen for your thoughts on this too!

Only bumping it to 20 on MFA seems good.  It does mean less code changes which is a good thing
20 is still probably on the low end specifically for scenario of shared IPs within an office
There is a risk that increasing this too high opens the door for brute force attacks, however there is an additional layer of limiting on MFA as mentioned on the parent issue:
On my local when I repeatedly enter the wrong TOTP code, eventually I get:
"Your account is temporarily locked. Please try again later." with a 403 status code
This codes from silverstripe-mfa/src/Authenticator/LoginHandler.php

This comes from a private static on Member $lock_out_after_incorrect_logins = 10;
That said I'm OK with 20, though 50 is also probably fine to.
Let me know what you think the final number should be

IMO let's do 20. I don't think we've had any actual complaints about the feature yet, so let's not go too overboard with this.
Thanks!

Cool, thanks for you assistance on this one @madmatt
Someone else in the team will pick this up for review


Cherry-pick PR  luci-flutter tree state is red even though tree for the candidate branch is open

PR is https://github.com/flutter/flutter/pull/109475

Tree for both framework master and the base flutter-3.2-candidate.5 branch is open.

The root cause is https://github.com/flutter/flutter/issues/106816. It lists recent 30 based on created time.
As a workaround, could you try landing a new PR?

As a workaround, could you try landing a new PR?

Here: https://github.com/flutter/flutter/pull/110045
luci-flutter isn't in the checks list.

luci-flutter isn't in the checks list.

This WAI. Tree status is intended for master branch only.
https://github.com/flutter/cocoon/blob/main/app_dart/lib/src/request_handlers/push_build_status_to_github.dart#L57

Then why did my original PR ever have that check?

Then why did my original PR ever have that check? https://github.com/flutter/flutter/pull/109475

Seems a race condition: that PR was switched to master by the bot at 08/12 1:43pm, and then switched back to release branch at 1:45pm. The status should have been injected in between.

I guess now the release PRs are not following tree status and auto-submit logics. They are simply manually landed manually. /cc @CaseyHillers @godofredoc

I suspect this happened as the flutter-dashboard bot changed the PR back to master, and then it was swapped back to the release branch.
The tree status shouldn't be reported on release branches, but GitHub doesn't clear statuses on changes to the base ref.

Ahh that makes sense.

Moving to P3 while we decide if we want to support autosubmit in release branches.

Release branches are now supported by autosubmit and tree-status is not taken into account by autosubmit for release branches.
https://github.com/flutter/cocoon/pull/2779
Can we close this or is there other work that is needed?
@jmagman @godofredoc @CaseyHillers ^


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative/operations-wg-leads
/assign knative/operations-wg-leads
Produced by: knative-extensions/knobots/actions/update-deps

/test eventing-upgrade-tests

/test eventing-upgrade-tests


Changes keep effect and planning code

This PR introduces two main changes:

pre-image backchaining at planning time: instead of running the expected_atoms check on the atoms sequence yielded by planning, we use a simple backchaining strategy to compute a 'necessary' atoms sequence by computing the necessary image at every timestep in the plan. This is exactly the number of atoms that need be true at a certain timestep for the rest of the plan to succeed (consequently, this is the smallest set on which we can run the expected_atoms check).
no ignoring add-effects at learning time: previously, we would make ignore effects from any unmodeled add or delete effects. However, because of positive preconditions, we don't need to do this for add effects. So we now just make ignore effects from unmodeled delete effects. As a corollary to this change, we modify the keep effects procedure to not just check ignore effects but also delete effects.

 Specifically, we need to modify task_plan_with_option_plan_constraint to not check the expected atoms after every action execution, but rather compute an entire task plan, then find the necessary atoms sequence, then check that for every timestep, the necessary atoms are a subset of the actually-observed atoms.
I will think about the easiest way to make this change and try implementing sometime later today/tomorrow!

Okay - so I'm actually pretty stuck on how to do this. Let's chat about it during our meeting tomorrow @tomsilver ?

Yeah it looked tricky to me too, chatting sounds good

@tomsilver change made!


Argparse V2

Added command line inputs to parallel reach test file.

@jhrehanoaa
We could probably set this as draft while finishing the input file setup.


pulling stays before buckling but not after

Description

pull thing
buckle to chair
still pulling thing

basically you can make train with this so i am opening issue so nobody evil can

Description
1. pull thing

2. buckle to chair

3. still pulling thing

Intended to some degree due to vehicles existing and not being able to be separated from buckling.

I think there's still some unintended funnies you can get up to so I'll leave it up so someday 2 years from now I remember and check.


Max, Min, and Close icons aren't visible in Windows (and possibly Linux?)

Currently the window buttons to maximize, minimize, and close aren't shown on Windows (and likely also aren't shown on Linux).

This is fixed in latest version on Windows

While the application toolbars aren't looking great on windows and linux, I'll track that as a different issue and close this one.


After removing SAML auth, user should be able to login via password directly

ISSUE TYPE

Bug Report

COMPONENT NAME
API via cmk

CLOUDSTACK VERSION
<IP_ADDRESS>

CONFIGURATION
N/A
OS / ENVIRONMENT
N/A
SUMMARY

STEPS TO REPRODUCE
step1: add user, add password for this user, play with this user.
step2: enable SAML SSO authentication for this user, either by webui or API
step3: When you choose to remove the SAML SSO authentication, via cmk : authorize samlsso enable=false userid=myuser id
step4: Try to login on webui with failure :)

EXPECTED RESULTS
User should be able to login on cloudstack web UI
When SSO is disable the field "source" on user table is SAML2DISABLED
When SSO has never been activated (and user is able to login via cloudstack directly) this field must be UNKNOWN.
ACTUAL RESULTS
User can't login on cloudstack web UI

This is done so if some security issue happens, SSO authorised SAML account/users don't become active for normal auth access. Consider/think this like an ldap account, you can't change the source or change their auth mechanism too (I think cc @DaanHoogland to confirm). I think maybe only the root admin can do something like that.

I will have to investigate, but both premisses seem reasonable from a functional point of view:

a user that gets saml enabled looses its status as direct login user, or
a user has an underlaying account that remains available for normal login while its enabled for sso.
The source of a user is however only one and cannot be changed by normal interaction with the system. That part is correct. I don´t know enough of the saml implementation to say if this is a bug or on purpose.

I think this is a valid issue  and can be considered as improvement request
The root admin  should be able to take a call to allow login via normal ldap credentials when saml auth is disabled
We could Introduce  a new  parameter "usersoruce" in the updateuser api call
https://cloudstack.apache.org/api/apidocs-4.20/apis/updateUser.html
Currently its set to saml disabled


Update netty-codec, ... to 4.1.71.Final

Updates

io.netty:netty-codec
io.netty:netty-transport-native-epoll
io.netty:netty-transport-native-kqueue

from 4.1.70.Final to 4.1.71.Final.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.netty" } ]

labels: library-update

Superseded by #88.


[FEATURE] Generic OIDC login auth

Is your feature request related to a problem? Please describe.
Anyone self hosting this would have to store yet another username/password combination for their local user(s), like many others of us who self host apps, we might be running something like Keycloak or Authelia to centralise the authentication of our users.
Describe the solution you'd like
Ability to configure a Client ID & Client Secret as well as an OIDC auth domain that is used instead of Google, users who log in with this method and are not already registered will be automatically created (similar to the Google login functionality currently).
Describe alternatives you've considered
Possibly add a OIDC proxy/login page that forces user to be authenticated before being allowed to navigate to the website, but that authentication cannot be passed in/etc to the app, so it's kinda pointless.

Not sure what the process to track feature requests is, but are github issues the right place?

This is most definitely the place, but I'm just not sure about what OIDC Auth is well enough to implement it within Reactive Resume's current authentication system. And also, it should be known that I would prioritize to implement features that would be useful on the both versions (managed + self-hosted) of the site than just the self-hosted one. If this does have more folks peeking interest, will reconsider implementing it.

Sorry late reply :)
Yeah that makes sense, having a quick poke around the code what we'd be after is a similar approach to https://github.com/AmruthPillai/Reactive-Resume/blob/main/server/src/auth/auth.service.ts#L110 but just using a more generic OIDC library than google's one, so that you can plug in a few extra variables like the auth endpoint!
If I get some time I can have a go, but I wouldn't hold my breath :D

Hi, @AmruthPillai
Have you had a chance to implement this feature?
Thanks!

Not going to lie, I very much want to self host this for both myself and friends, but my own rule is that I won't host anything that doesn't have the ability to use SSO, so I hope this feature gets implemented at some point. It genuinely looks like an incredibly helpful and promising web app.

Also interested in a OIDC. My concern is that anyone with the public url can create an account. Or is there a way to restict that?

It would be fantastic to see this feature coming in the next releases!

yeah OIDC would be nice to have

Would love to see generic OIDC support implemented. Existing users in IDPs (Authentik here) would be able to login immediately, without having to create an account for something they may only use a handful of times.

As a Authelia user, definitely a nice feature

Also keen on this feature!

@AmruthPillai please have a look at the linked PR! Looks good.

They there! I'd also really love to self-host this for my friends using Authentik as SSO. Hoping this someday becomes a thing with that PR already up. I did a quick review with my own feedback, but ofc this requires project owners to take a deeper look than I can provide. Thanks for the starting work!