Opens PTC Window but then nothing?

I'll admit, I am 100% new in using ChromeDriver, so I'm sorry if this is a stupid-easy fix.
When I run "ptc2" with or without -- commands, the PTC website opens in a new window but... that's it. None of the info is filled in, it's just the Create Account page. Here is what is in my prompt:
Susan@Susan-HP MINGW64 ~/Desktop/chromedriver_win32
$ ptc2
Creating new account:
Attempting to create user GeXMiiXGOojRF96:uSuZ5eyhZHTp5LU. Opening browser...
Step 1: Verifying age using birthday: 1959-01-15
Traceback (most recent call last):
File "C:\Python27\Scripts\ptc2-script.py", line 9, in
load_entry_point('PTCAccount2==2.0.2', 'console_scripts', 'ptc2')()
File "c:\python27\lib\site-packages\ptcaccount2\console.py", line 47, in entry
account_info = ptcaccount2.random_account(args.username, args.password, args.email, args.birthday)
File "c:\python27\lib\site-packages\ptcaccount2\accounts.py", line 210, in random_account
account_created = create_account(try_username, password, try_email, try_birthday)
File "c:\python27\lib\site-packages\ptcaccount2\accounts.py", line 125, in create_account
elem.clear()
File "c:\python27\lib\site-packages\selenium\webdriver\remote\webelement.py", line 87, in clear
self._execute(Command.CLEAR_ELEMENT)
File "c:\python27\lib\site-packages\selenium\webdriver\remote\webelement.py", line 461, in _execute
return self._parent.execute(command, params)
File "c:\python27\lib\site-packages\selenium\webdriver\remote\webdriver.py", line 236, in execute
self.error_handler.check_response(response)
File "c:\python27\lib\site-packages\selenium\webdriver\remote\errorhandler.py", line 192, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.InvalidElementStateException: Message: invalid element state: Element must be user-editable i                                                                                n order to clear it.
(Session info: chrome=51.0.2704.103)
(Driver info: chromedriver=2.23.409699 (49b0fa931cda1caad0ae15b7d1b68004acd05129),platform=Windows NT 10.0.10586 x86_6                                                                                4)
Thanks for everything!

Hi!
selenium.common.exceptions.InvalidElementStateException: Message: invalid element state: Element must be user-editable i n order to clear it.
That's a really weird error, since the code explicitly removes the readonly attribute before trying to clear the element. Is the error consistent on every run, or does it only happen sporadically?

May be related to #15, could you verify? Thanks!

Sorry for being MIA. THe update fixed it, and it's running perfectly! Thanks so much!


add functionality to specify temp directory location when creating memory mapped temp arrays

this allows you to also utilize memory mapped temp arrays in multithreaded processes by explicitly passing the directory name without needing to rely on the global TEMP_DIR_NAME variable still being accurate

@sophiamaedler We now have quite restrict PR rules for alphaX packages. Please git pull before coding/pushing:)

@sophiamaedler We now have quite restrict PR rules for alphaX packages. Please git pull before coding/pushing:)

@jalew188 Sure :) I had done a git pull before opening this PR, should I still do anything else next time?

@sophiamaedler We now have quite restrict PR rules for alphaX packages. Please git pull before coding/pushing:)

@jalew188 Sure :) I had done a git pull before opening this PR, should I still do anything else next time?

I think the only thing is to fix this error in this PR: https://github.com/MannLabs/alphabase/actions/runs/9000693343/job/24725402319?pr=162#step:4:246

@sophiamaedler We now have quite restrict PR rules for alphaX packages. Please git pull before coding/pushing:)

@jalew188 Sure :) I had done a git pull before opening this PR, should I still do anything else next time?

I think the only thing is to fix this error in this PR: https://github.com/MannLabs/alphabase/actions/runs/9000693343/job/24725402319?pr=162#step:4:246

I installed the ruff linter and let it run over the code. I hope that should fix it and should avoid this problem for the future.

I also noticed that the global definition of TEMP_DIR_NAME was removed and added it back in. This otherwise throws errors in some rare cases.

tempmmap was designed long time ago by Sander, we can refactor it if necessary in the near future.
The same for alphabase.io.hdf.py and so on.

In general I think there is a lot of potential here for additional use cases e.g. I have currently been playing around with Dask arrays and found a way to use a tempmmap compatible HDF5 in the backend of a memory mapped Dask array (see here : src/sparcstools/base/daskmmap.py) but I am not sure where the best place to include it would be and if alpha base is the correct location to add something like that. Might be worth a more in depth discussion at some point.


parsers.Readline is not a constructor

SerialPort Version:
npm info serialport version
6.0.4

NodeJS Version:
node -v
v8.9.0

Operating System and Hardware Platform:
ver
Microsoft Windows [Version 10.0.15063]
wavecom modem

Have you checked the right version of the api docs?:
yes i copy from example code for test
https://github.com/node-serialport/node-serialport/blob/v6.0.4/examples/readline.js

Are you having trouble installing and you checked the Installation Special Cases docs?
no i guess
i follow https://github.com/johnny-five-io/electron-serialport

other comand work

Are you using Electron and have you checked the Electron Docs?:
npm info electron version
1.7.9

Summary of Problem
Uncaught TypeError: parsers.Readline is not a constructor
at index.html:11
Steps and Code to Reproduce the Issue
i wanna try readline method
but showing errors
is this  bugs ?
thank you
<script>
/* eslint-disable node/no-missing-require */
'use strict';

// Use a Readline parser

const SerialPort = require('serialport');
const parsers = SerialPort.parsers;

// Use a `\r\n` as a line terminator
const parser = new parsers.Readline({
  delimiter: '\r\n'
});

const port = new SerialPort('COM1', {
  baudRate: 115200
});

port.pipe(parser);

port.on('open', () => console.log('Port open'));

parser.on('data', console.log);

///// port.write('ROBOT PLEASE RESPOND\n');

// The parser will emit any string response
</script>

fom https://github.com/node-serialport/node-serialport/issues/937
i make mistake with comand  npm info serialport version
its print 6.0.4 but i check in  package.json from https://github.com/johnny-five-io/electron-serialport
it set 4.0.7
"dependencies": {
    "data-table": "^2.0.0",
    "serialport": "^4.0.7"
  }

so i change to "serialport": "^6.0.4" and rebuild install
works no error  but no console output
change to version 5.0.0 works no error  with console output
now I try on version 5.0.0 first


Uncaught TypeError: Cannot read property 'naturalHeight' of undefined - On VDC list

I have one deployed VDC Right now

Browse to VDC list with console open

results in
Uncaught TypeError: Cannot read property 'naturalHeight' of undefined
Every new browse throws the error again.
Issue was logged and closed in the past for a different page.
See https://github.com/threefoldtech/js-sdk/issues/1029

As mentioned in here: https://github.com/threefoldtech/js-sdk/issues/1029#issuecomment-688791578
problem is with vue engine, maybe it could be fixed in coming versions

As mentioned in here: https://github.com/threefoldtech/js-sdk/issues/1029#issuecomment-688791578
problem is with vue engine, maybe it could be fixed in coming versions


Bump brew to point to latest release

Description of changes:
Bump brew to point to latest release
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

/lgtm
/approve


Support for images/textures

Are there any plans to support images/textures on the GPU, with hardware based linear filtering, edge clamping, etc?

Unfortunately not.  Since Futhark is a platform-agnostic high-level language, GPU features are not directly exposed.  GPUs just happen to be the most interesting platform currently targeted by the compiler.
It might be possible to come up with a high-level model that can also be translated to something reasonable on non-GPUs, but I'm not quite sure what it would look like.  Maybe an abstract data type that is somehow specially recognised by the compiler, and turned into texture memory if the backend supports it...

I was envisioning something like functions sample2D, sample3D, etc that takes an array, a floating point tuple index and a "sampler descriptor" record (filter mode, edge mode, etc). This could be emulated on CPUs (like how OpenCL does it), but on GPU it would cause the array to be materialized as a texture instead of a buffer (or copied into such if used as both).
The built in linear sampling is one thing (its part of the die that now goes unutilized), but the biggest issue is the cached uncoalesced access that textures provide. How would you e.g. write a futhark program that rotates an image an arbitrary amount of degrees efficiently?

Thanks for your reply, much appreciated! I hope you'll be able to find a solution that fits other design requirements in the future.


Add tts test single

@rachmadaniHaryono It has our older commit history as well the merging of which will confuse contributors. It would be better if maybe you could update you could send in just the changes that you made.

moved #119


One to many w/ paging

Any guidance on using one to many helpers as well as paging?  I haven't had much success coming up with a reliable way to make it happen.

you would have to write your own query here where you write a paged query then join that query to the many table on the outside. Its doable but not automatic.


Message missing data: "{Mining ship} has remote mined {Planet name} extracting no minerals."

All my miners now send this message, but it's not correct, I can collect minerals from these planets. They're not occupied.

Ah dang, I saw an issue with remote mine messages and "fixed" it, but didn't test it. Your miners are working according to the logs though:
2024-11-02 23:04:18 DBG planet remote mined Fleet="Mini-Miner v4 #47" GameID=343 GameName="Game 05" Minerals="59kT ironium, 64kT boranium" Planet="Autumn Leaves" Player=1 Year=2490
2024-11-02 23:04:18 DBG planet remote mined Fleet="Mini-Miner v4 #37" GameID=343 GameName="Game 05" Minerals="129kT ironium, 113kT boranium, 116kT germanium" Planet=Vega Player=1 Year=2490
2024-11-02 23:04:18 DBG planet remote mined Fleet="Mini-Miner v4 #23" GameID=343 GameName="Game 05" Minerals="128kT ironium, 2kT boranium, 141kT germanium" Planet=Skunk Player=1 Year=2490
2024-11-02 23:04:18 DBG planet remote mined Fleet="Mini-Miner v4 #46" GameID=343 GameName="Game 05" Minerals="133kT ironium, 145kT boranium, 23kT germanium" Planet=Phad Player=1 Year=2490
2024-11-02 23:04:18 DBG planet remote mined Fleet="Mini-Miner v4 #10" GameID=343 GameName="Game 05" Minerals="54kT ironium, 66kT boranium" Planet=Equuleus Player=1 Year=2490
2024-11-02 23:04:18 DBG planet remote mined Fleet="Mini-Miner v4 #12" GameID=343 GameName="Game 05" Minerals="66kT ironium, 64kT boranium, 7kT germanium" Planet=Penance Player=1 Year=2490
2024-11-02 23:04:18 DBG planet remote mined Fleet="Mini-Miner v4 #55" GameID=343 GameName="Game 05" Minerals="66kT ironium, 63kT boranium, 7kT germanium" Planet=Penance Player=1 Year=2490
2024-11-02 23:04:18 DBG planet remote mined Fleet="Mini-Miner v4 #9" GameID=343 GameName="Game 05" Minerals="54kT ironium, 65kT boranium" Planet=Equuleus Player=1 Year=2490


[PHPUnit 10]   DataProviderAnnotationToAttributeRector does not "find"  tests class  not inheriting directly from PHPUnit\Framework\TestCase

i.e it works fine on class like this
use PHPUnit\Framework\TestCase;

class DateTest extends TestCase
{
}

but it does not work on
  use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;

  class BoostTest extends WebTestCase
  {

}

the issue seems to be that TestsNodeAnalyzer->isInTestClass  does not consider them as test class
I'm trying to dig more in the issue
Note: I'm not sure if of any importance but my rector is installed in a different directory than my actual projet (i.e I have a common "rector" tool for all my projects , with its own composer.json )

the reason I don't want to do that is that  I don't want my project to be blocked on some dependencies because I need  library foobar at version X and a tool I use need it at version < X

I also need a version phpstan, so I can't use a version below that yours  (I also use phpstan)  therefore my point stands
I however understand that it's a use case you don't want to support and I'm fine with it.

@allan-simon That's ok. PHPStan is also scoped and downgraded :) Upgrade to the latest PHPStan and you're good to go.

https://packagist.org/packages/phpstan/phpstan

How you install rector? rector-phpunit is included in rector/rector, so you need to install rector/rector only:
composer require rector/rector --dev

that, rector-phpunit is included inside /vendor/rector/rector-phpunit
https://github.com/rectorphp/rector/tree/main/vendor/rector/rector-phpunit

yes I installed only  rector/rector , in a tools directory  with its own composer.json
and when I run it   precise  the sources path  of the project I want to run rector on

on  " Upgrade to the latest PHPStan and you're good to go." is not always that easy
I give you an exemple I rely on phpstan extensions not made by me ,  that requires phpstan ^1.0  , it works well and I don't need to upgrade it.
Tomorrow phpstan upgrade to 2.0 with a lot of awesome feature and you decide to switch to it, as it is a major version, it comes with backward compat breaking changes so now you support only phpstan 2.0 , and now I'm between rock and stone , i.e I can't upgrade rector because I can't upgrade the extension

ah thanks I will try this then !

awesome, it works with  -a

Is there a way to configure this directly in the rector.php ?


Update containerImage loading to not rename on empty string dst

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 10 (80.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 58.483%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

deploy/deploy.go
8
10
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
2144

Relevant Lines:
3666

💛  - Coveralls


PPC: e_li D, SIMM20 incorrectly calculates the immediate value

Describe the bug
SIMM20 only exports the lowest 16 bits of a 20-bit immediate value. This causes any immediate greater than 0xFFFF or less than 0x0000 to be incorrect.
https://github.com/NationalSecurityAgency/ghidra/blob/5234839b24c418219e35d0902161ca47ce98ce91/Ghidra/Processors/PowerPC/data/languages/ppc_vle.sinc#L48
To Reproduce
Find any e_li instruction that uses an immediate larger than 16 bits.
Ghidra's assembler works for testing negative values, just remember to set the VLE register to 1.
Expected behavior
SIMM20 should export all 20 bits of the immediate. Values like -0x1 and 0x12345 should work.
Actual:        70 7f 07 ff     e_li       r3, 0xffff
Expected:      70 7f 07 ff     e_li       r3, -0x1

Actual:        70 82 52 34     e_li       r4, 0x1234
Expected:      70 82 52 34     e_li       r4, 0x12345

Environment (please complete the following information):

OS: Windows 10 build 19041
Java Version: 14.0.2
Ghidra Version: 9.2.0
Ghidra Origin: official ghidra-sre.org distro

Additional context
Changing the size of the export to 3 bytes instead of 2 seems to fix the problem, but I'm not sure this is the best solution.

@GhidorahRex if you're looking into PPC fixes, any chance for some attention on 1672?

Fixed by b7499e1bc1c4f097de2312fad3acae990fb2ac43


Error: Non-base58 character

when i run this code i am facing this error
/Users/webskitters/Desktop/raydium-sdk-swap-example-typescript/node_modules/base-x/src/index.js:113
throw new Error('Non-base' + BASE + ' character')
^
Error: Non-base58 character
at Object.decode (/Users/webskitters/Desktop/raydium-sdk-swap-example-typescript/node_modules/base-x/src/index.js:113:11)
at new RaydiumSwap (/Users/webskitters/Desktop/raydium-sdk-swap-example-typescript/src/RaydiumSwap.ts:33:73)
at /Users/webskitters/Desktop/raydium-sdk-swap-example-typescript/src/index.ts:14:23
at Generator.next ()
at /Users/webskitters/Desktop/raydium-sdk-swap-example-typescript/src/index.ts:8:71
at new Promise ()
at __awaiter (/Users/webskitters/Desktop/raydium-sdk-swap-example-typescript/src/index.ts:4:12)
at swap (/Users/webskitters/Desktop/raydium-sdk-swap-example-typescript/src/index.ts:10:25)
at Object. (/Users/webskitters/Desktop/raydium-sdk-swap-example-typescript/src/index.ts:67:1)
at Module._compile (node:internal/modules/cjs/loader:1376:14)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
how to solve it ?

It looks like it's throwing this error while trying to read the private key from .env here:
  const raydiumSwap = new RaydiumSwap(process.env.RPC_URL, process.env.WALLET_PRIVATE_KEY);

Make sure your key is in base58, which looks like this 5Kd3NBUAdUnhyj4E7Ct5oqL8hM2FfGtLhawt4XPP9ZZZkqJE7FB. This is a random dummy key.


Error in FR and EN Privacy Policy and terms of use link

There is problem in loading Privacy Policy page both in EN and FR.
In iPad and Android and Windows.

@navpreetkaur051 please fix it in terms of use page ASAP right after dr. Cleo's confirmation before having any tests OR updating the production!
Thanks

it has successfully tested.


Expose every HTTP response

Expose every HTTP response even if a 400 case
:heavy_check_mark: Checklist

[ ] Added tests for new functionality and regression tests for bug fixes
[ ] Added changeset (run yarn changeset in the root)
[ ] Screenshots of before and after attached (for UI changes)
[ ] Added or updated documentation (if applicable)

I opened this instead so will close this one


Moved files are not detected any more

Describe the bug
If a file is moved, Aves does not see it any more. It sees it at the old location, just unavailable.
To Reproduce
Steps to reproduce the behavior:

Download an image.
Open Aves, check that it appears.
Using a file manager, rename the directory that contains the image, or move the image in another directory.
Aves now says that the image does not exist.
Rescanning from the hamburger menu, and force-closing the app from the Android settings, do not solve it.

Expected behavior
Aves its info automatically or when rescanning.
System information and logs:
Aves 1.9.7 (10804) from F-droid on Android 7.1

https://github.com/deckerst/aves/wiki/FAQ#why-does-aves-show-thumbnails-with-a-broken-image-symbol
Isn't the file manager app that's moving the file without updating the Media Store, as it should?

Hello, I now changing to Simple File Manager and I get the same result. Simple Gallery shows the images at the new locations.

Duplicate:

#760


remove function part 1, 2 naming

replaces functions part II with delegates - a bit more expressive.

Good suggestion. Just needs rebasing.

Good suggestion. Just needs rebasing.

Done.


Missing xRDServer resource?

Is this resource actually missing from the module?  It is in the readme, but when I check it is not there?
PS C:\> Get-DscResource -Module xremotedesktopsessionhost

ImplementedAs   Name                      ModuleName                     Version    Properties
-------------   ----                      ----------                     -------    ----------
PowerShell      xRDRemoteApp              xRemoteDesktopSessionHost      <IP_ADDRESS>    {Alias, CollectionName, DisplayN...
PowerShell      xRDSessionCollection      xRemoteDesktopSessionHost      <IP_ADDRESS>    {CollectionName, SessionHost, Co...
PowerShell      xRDSessionCollectionCo... xRemoteDesktopSessionHost      <IP_ADDRESS>    {CollectionName, ActiveSessionLi...
PowerShell      xRDSessionDeployment      xRemoteDesktopSessionHost      <IP_ADDRESS>    {ConnectionBroker, SessionHost, ...

I'm looking to use this to configure my hosts that won't be part of an RDS Deployment..
Thanks,
David F.

For some reason, the edit on my post is being eaten/rejected, etc..
I am specifically looking for the xRDServer resource..
David F.

It do exist in the code https://github.com/PowerShell/xRemoteDesktopSessionHost/tree/dev/DSCResources/MSFT_xRDServer, not sure why you are not seeing it though.

Hi @figueroadavid, the xRDServer resource was just added and will be released in the next version of xRemoteDesktopSessionHost. If you want to use it right now, just clone the repository or download the zip of the dev branch, this will include xRDServer.

I was just about to say what @danielboth said ☺ thanks Daniel!

Great!  thank you so much.. I was really scratching my head on this one.
I am trying to build out DSC for RDS hosts that could be used in Citrix or
VMware, and in those circumstances, the collections are just not
necessary.   One thing I hope you guys will add, is that when using DSC to
enable RDS, the option to turn off the requirements for NLA as part of that
step.
Thanks again,
David F.
On Tue, Jul 10, 2018 at 10:37 AM Daniël<EMAIL_ADDRESS>wrote:

Hi @figueroadavid https://github.com/figueroadavid, the xRDServer
resource was just added and will be released in the next version of
xRemoteDesktopSessionHost. If you want to use it right now, just clone the
repository or download the zip of the dev branch, this will include
xRDServer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/xRemoteDesktopSessionHost/issues/40#issuecomment-403867844,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB8n71xj6yaJia4P1zbU58kFCqBC74Itks5uFMojgaJpZM4VJoP0
.

@figueroadavid please submit a new issue, for the suggested enhancement.
Leaving this open until we release if others have the same question.

This is released now, so closing this issue.


fix: 000011_workspace_no_outputs_test test

It used the same resource name as 000010 test.
We requested resources to be deleted, but we didn't wait them to be deleted.

fixes #908
References:

https://github.com/weaveworks/tf-controller/issues/908

I wonder if we could rely on namespaces to avoid having to clean up things after we are done with the tests. Having to wait those deletes will slow down our tests quite a bit.

I do that elsewhere, and clean them up. Tests should in general be able to run in parallel, and using separate namespaces helps with that. (Running in parallel also means doing cleanup on each test doesn't add up to lots of running time)

I wonder if we could rely on namespaces to avoid having to clean up things after we are done with the tests. Having to wait those deletes will slow down our tests quite a bit.

I think then we would have to define a lot of namespaces when we create the controller as it does not process resources in namespaces we didn't define in allowedNamespaces and generate the role bindings stuff for the runner for all namespaces we intend to use. I think that's the reason why @chanwit used only flux-system everywhere.
Note:
We create a lot of resources and if they are not cleaned up properly before we go to the next test, we may end up 20-30 Terraform + Source objects and the controller will reconcile each of them until they are deleted (I think, take it with a grain of salt). That can also cause random failures on the CI runner (everything slows down with each tests).


Is it possible to use tidis as a replica of Redis server?

Is it possible to use tidis as a replica of Redis server for below's purpose?

Hot data backup
Read scalability

Not support directly, but you can try to use third-party migrate tools to do so.


Use SVG Coveralls badge

For consistency with Travis and it looks better.
Before:

After:

Coverage remained the same at 30.064% when pulling 84e38d6b14f1eb260863e16078d7e2a74b6bdd95 on hugovk:patch-3 into 66fff5e6511956f2107edcc14072d417982e5e37 on sixohsix:master.


release v0.1.1

Pull Request Test Coverage Report for Build e3243f917e4e0e903063b96c042f605015109e44-PR-44

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 46.765%

Totals

Change from base Build 7159f4aaab16002723039340d30b83594f0a2a8b:
0.0%

Covered Lines:
159

Relevant Lines:
340

💛  - Coveralls


Better error messages for missing options

Thanks @pik694 exw3 will throw an error message if you don't include gas, from, or contract address in a transaction. Which helps with debugging.
Right now the error messages look like this:
** (MatchError) no match of right hand side value: {:error, :missing_gas}

Which is useful, and does tell what is going wrong. But I think it would be better if we could have a more readable and clear error message like
(MissingOptionsError) transaction request is missing the :gas field

Hi, new to project (would like to contribute). So this would just be a simple modification to the handle_call function, just with a more human readable description ?

Hi there! It's a bit more complicated than that. Its essentially modifying some of the behavior for check_option, example used here:
https://github.com/hswick/exw3/blob/master/lib/exw3.ex#L665
To be more human readable like you said.


[Team] Consider PDF output with summary of all plots and things that we do

Howdyyy, not sure anybody if familiar with python module for creating pdf reports but Tay and I were chatting yesterday about this. Might be nice to summarize everything we are doing into a report output

Do you mean running all the functions on the input data set and giving out a report kind of thing?

I think this is out of scope at this time, we can delete this comment


Properties with reserved keywords cause inserts to fail

 Issue by Wilson-Ding1
Thursday Sep 28, 2017 at 03:49 GMT
Originally opened as https://github.ibm.com/IBM-Swift/Kassandra/issues/30

 Issue by jesspittman
Friday Sep 02, 2016 at 17:26 GMT
Originally opened as https://github.com/IBM-Swift/Kassandra/issues/30

If i have a model called 'todo' and it has a property named 'order', todo.save() will fail when I try to call it. Renaming the 'order' field listed in the 'Field' enum to something like 'rank' lets you create the table and fetch from the table, but save will still fail.
Why does save fail?
The name 'order' is reserved by Cassandra for sorting. When it finds the word in the query, it gets confused and returns an error. I tried to workaround this by changing the Field name to 'rank'.
Save uses reflection to get the properties and values from the 'todo' object. The other functions use the 'Field' enum to get the field names. This means that every other function will use 'rank' except save, which uses the real property name.
How do we fix this then?
Allow for custom field names
We could allow people to map the property name to a field name and use it when saving. This definitely allows for more flexibility.
Escaping column names
This works:
CREATE TABLE IF NOT EXISTS fakeitem(documentID uuid primary key, userID text, title text, "order" int, completed boolean);
This doesn't:
CREATE TABLE IF NOT EXISTS fakeitem(documentID uuid primary key, userID text, title text, order int, completed boolean);
Suggested Reading
Cassandra Keywords

 Comment by Wilson-Ding1
Thursday Sep 28, 2017 at 03:49 GMT

 Comment by jesspittman
Friday Sep 02, 2016 at 17:52 GMT

Small side effect of escaping column names. They become case sensitive. If they aren't escaped, they are transformed to lowercase.


Fix a couple of VS Code format problems

@randomir, did you add ipympl to the docker file in the end? When I run https://github.com/dwave-examples/structural-imbalance-notebook in Leap IDE, it give the notification:
Widgets require us to download supporting files from a 3rd party website. Click [here](https://command:jupyter.enableLoadingWidgetScriptsFromThirdPartySource/) to enable this or click [here](https://aka.ms/PVSCIPyWidgets) for more information. (Error loading jupyter-matplotlib:^0.10.5). Widgets require us to download supporting files from a 3rd party website. Click [here](https://command:jupyter.enableLoadingWidgetScriptsFromThirdPartySource/) to enable this or click [here](https://aka.ms/PVSCIPyWidgets) for more information. (Error loading jupyter-matplotlib:^0.10.5). Widgets require us to download supporting files from a 3rd party website. Click [here](https://command:jupyter.enableLoadingWidgetScriptsFromThirdPartySource/) to enable this or click [here](https://aka.ms/PVSCIPyWidgets) for more information. (Error loading jupyter-matplotlib:^0.10.5). Widgets require us to download supporting files from a 3rd party website. Click [here](https://command:jupyter.enableLoadingWidgetScriptsFromThirdPartySource/) to enable this or click [here](https://aka.ms/PVSCIPyWidgets) for more information. (Error loading jupyter-matplotlib:^0.10.5).

When you click "here to enable this", it enables download from 3rd party websites, and it works. Not sure I like this overall, though. We could enable it by default, but should check with legal first.


Question: Intended use with venvs

Hi,
this are questions concerning the PDM_MULTIRUN_USE_VENVS environment variable.

What is the default setting here?
Guess this should be 0. Correct?

Usage with pyenv/venvs
Situation:
Normally I would go with one venv, building on one or multiple python versions (build with pyenv).
I don't like my pyenvs to be bloated by packages from projects - Therefore I use venvs (saved in package folder or App folder). How would I have to setup pdm-multirun to follow that direction?

Is this good practice/what is your practice here?
In copier-pdm setup script you are using __pypackages__ by default, but the PEP582 was rejected. How is this intended?

On further notice to  copier-pdm setup script
When using in a project:
export PDM_MULTIRUN_USE_VENVS=1
export PDM_MULTIRUN_VERSIONS="3.10 3.11 3.12"
pdm multirun -v pdm install -G:all

The 3.10 and 3.12 versions/interpreters are skipped. (having set 3.11 to global with pyenv)
I would have expected to have created three venvs for different pyenv versions.
When creating the venvs first and setting pdm config venv.in_project = false it is working that the three venvs are filled.
Not working when set true, but this is likely an issue with pdm.

What is the default setting here?

The default setting is unset. The option is activated only if the value is 1. Any other value (0, empty, 2), and the venv feature will be disabled.

I don't like my pyenvs to be bloated by packages from projects

Definitely something to avoid indeed.

How would I have to setup pdm-multirun to follow that direction?

The idea is to declare and create a few venvs with PDM, then use the same names with PDM Multirun:
# Unfortunately I don't think PDM supports creating multiple venv in the project folder,
# so we have to tell it to create them elsewhere. But maybe I'm wrong.
pdm config venv.in_project false

pdm venv create -n 38 python3.8
pdm venv create -n 39 python3.9
pdm venv create -n 310 python3.10
pdm venv create -n 311 python3.11
pdm venv create -n 312 python3.12

export PDM_MULTIRUN_VERSIONS="38 39 310 311 312"
export PDM_MULTIRUN_USE_VENVS=1

Is this good practice/what is your practice here?
In copier-pdm setup script you are using __pypackages__ by default, but the PEP582 was rejected. How is this intended?

That's a good question. I started using PEP 582's __pypackages__ when it was not yet rejected, and came to like it a lot, so I'm still using __pypackages__ today (though sometimes I wish I could just pip install stuff without having to also pass the -t option). That's why the setup script of my template uses __pypackages__ by default. But since PEP 582 was rejected, I wouldn't say it is good practice: __pypackages__ should only be used if you know what you're doing.

The 3.10 and 3.12 versions/interpreters are skipped. (having set 3.11 to global with pyenv)
I would have expected to have created three venvs for different pyenv versions.

If neither 3.10 or 3.12 are listed in pyenv's global setting, then these two versions are actually not exposed in your PATH, and PDM can't find them. Therefore they are skipped when running PDM Multirun. My recommendation here is to add all your Python versions installed through pyenv to its global setting: pyenv global 3.12.x 3.11.x 3.10.x 3.9.x 3.8.x. This way all their shims are in the PATH, and you can call python3.12, python3.11, python3.10, etc., and PDM will find them.

When creating the venvs first
Not working when set true, but this is likely an issue with pdm.

Yep, I think you always need to create the venvs first, because when you set venv.in_project to true, PDM will always create the venv as .venv in the project folder. It will not suffix it with the Python version, nor even use the name you provided. The names are just used to map venvs location IIUC, but in this case the location is always the same, whatever the Python version. This is not ideal IMO. What I would prefer, if I were to use venvs again instead of __pypackages__, is that PDM automatically suffixes my .venv folder with 38, 39, etc., and also automatically create it when I install dependencies, so that I don't have to manually create all venvs with pdm venv create.


bug(terraform): false positive of "CloudWatch Without Retention Period Specified"

When retention-in-days is set using a variable (in my case a map) I get the warning "CloudWatch Without Retention Period Specified"
Expected Behavior
No warning as retention period is set to a valid value
Actual Behavior
Warning "CloudWatch Without Retention Period Specified"
Steps to Reproduce the Problem
variable "cloudwatch_log_retention_period" {
  description = "Cloudwatch log retention period in days"
  type        = map(string)
  default    = {
    feature    = 1
    dev        = 5
    test       = 5
    stage      = 30
    production = 90
  }
}

locals {
  environment = "dev"
}

resource "aws_cloudwatch_log_group" "negative2" {
  name = "Yada"

  tags = {
    Environment = "production"
    Application = "serviceA"
  }

  retention_in_days = var.cloudwatch_log_retention_period[local.environment]
}

Specifications
N/A

I don't know if this will help, but from the logs:
Terraform will perform the following actions:
  # aws_cloudwatch_log_group.negative2 will be created
  + resource "aws_cloudwatch_log_group" "negative2" {
      + arn               = (known after apply)
      + id                = (known after apply)
      + log_group_class   = (known after apply)
      + name              = "Yada"
      + name_prefix       = (known after apply)
      + retention_in_days = 5
      + skip_destroy      = false
      + tags              = {
          + "Application" = "serviceA"
          + "Environment" = "production"
        }
      + tags_all          = {
          + "Application" = "serviceA"
          + "Environment" = "production"
        }
    }
Plan: 1 to add, 0 to change, 0 to destroy.

The generated plan
{
  "format_version": "1.2",
  "terraform_version": "1.9.5",
  "variables": {
    "cloudwatch_log_retention_period": {
      "value": {
        "dev": "5",
        "feature": "1",
        "production": "90",
        "stage": "30",
        "test": "5"
      }
    }
  },
  "planned_values": {
    "root_module": {
      "resources": [
        {
          "address": "aws_cloudwatch_log_group.negative2",
          "mode": "managed",
          "type": "aws_cloudwatch_log_group",
          "name": "negative2",
          "provider_name": "registry.terraform.io/hashicorp/aws",
          "schema_version": 0,
          "values": {
            "kms_key_id": null,
            "name": "Yada",
            "retention_in_days": 5,
            "skip_destroy": false,
            "tags": {
              "Application": "serviceA",
              "Environment": "production"
            },
            "tags_all": {
              "Application": "serviceA",
              "Environment": "production"
            }
          },
          "sensitive_values": {
            "tags": {},
            "tags_all": {}
          }
        }
      ]
    }
  },
  "resource_changes": [
    {
      "address": "aws_cloudwatch_log_group.negative2",
      "mode": "managed",
      "type": "aws_cloudwatch_log_group",
      "name": "negative2",
      "provider_name": "registry.terraform.io/hashicorp/aws",
      "change": {
        "actions": [
          "create"
        ],
        "before": null,
        "after": {
          "kms_key_id": null,
          "name": "Yada",
          "retention_in_days": 5,
          "skip_destroy": false,
          "tags": {
            "Application": "serviceA",
            "Environment": "production"
          },
          "tags_all": {
            "Application": "serviceA",
            "Environment": "production"
          }
        },
        "after_unknown": {
          "arn": true,
          "id": true,
          "log_group_class": true,
          "name_prefix": true,
          "tags": {},
          "tags_all": {}
        },
        "before_sensitive": false,
        "after_sensitive": {
          "tags": {},
          "tags_all": {}
        }
      }
    }
  ],
  "configuration": {
    "provider_config": {
      "aws": {
        "name": "aws",
        "full_name": "registry.terraform.io/hashicorp/aws",
        "version_constraint": "5.66.0",
        "expressions": {
          "region": {
            "constant_value": "eu-west-2"
          }
        }
      }
    },
    "root_module": {
      "resources": [
        {
          "address": "aws_cloudwatch_log_group.negative2",
          "mode": "managed",
          "type": "aws_cloudwatch_log_group",
          "name": "negative2",
          "provider_config_key": "aws",
          "expressions": {
            "name": {
              "constant_value": "Yada"
            },
            "retention_in_days": {
              "references": [
                "var.cloudwatch_log_retention_period",
                "local.environment"
              ]
            },
            "tags": {
              "constant_value": {
                "Application": "serviceA",
                "Environment": "production"
              }
            }
          },
          "schema_version": 0
        }
      ],
      "variables": {
        "cloudwatch_log_retention_period": {
          "default": {
            "dev": "5",
            "feature": "1",
            "production": "90",
            "stage": "30",
            "test": "5"
          },
          "description": "Cloudwatch log retention period in days"
        }
      }
    }
  },
  "timestamp": "2024-09-13T07:47:38Z",
  "applyable": true,
  "complete": true,
  "errored": false
}


Scout and Beautify

Hello. I am using Scout to make the compiling of sass/scss easier.
I am wondering what path to put in and how, to make beautify work with this app (if possible).
Is it even possible to pair these?
/DrMowinckel

Closing older issues.  Let me know if we need to re open.


Fix upload progress for multipart uploads for issue #566

This change provides upload progress callbacks for multipart uploads, similar to how the non-multipart progress is updated.

Thank you. Will merge it shortly.


High-level bindings for Bitwuzla

Hi,
I noticed that you've updated bitwuzla-sys recently. If you're interested in high-level bindings (and don't have something else/better working already), I tried updating boolector-rs to the Bitwuzla APIs (incl. FP support) a while ago: https://github.com/Mrmaxmeier/bitwuzla-rs/commits/main/
My port is a bit messy, but it works fine for my use-cases. Many of the tests fail because it seems like Bitwuzla is less eager in rewriting constant terms to constant values.
Let me know if this is interesting to you and I'll try cleaning up my branch :slightly_smiling_face:

Hi!
Sorry for the late reply, been away.  Thanks for sharing your project, looks nice!  I personally don't have use for it, though, as in my (closed-source) projects I'm relying instead on an abstraction layer which translates a generic SMT-like intermediate representation into the actual ASTs for different solvers.  One of the reasons for doing it this way is related to what you have mentioned: I needed to have consistent rewriting rules for constant folding and propagation.
However, I believe that publishing your crate on crates.io would be beneficial to the community.


New instructions for 1-proposal SNS flow

This PR changes the instructions in README.md to document the new, 1-proposal SNS flow. The legacy flow will still be available to users who follow the instructions from here.
This PR should be merged after the NNS FE dapp's RC is released via proposal.
To simulate the minimal number of contributors required for completing the swap, run
./participate_sns_swap.sh \
    100 `# one hundred swap participants` \
    10000 `# each contributes 10_000 ICP (i.e., the Maximum Commitment)`

There are still occurrences of dfx canister --network "${NETWORK}" call sns_governance and similarly for other SNS canisters. This won't work unless we add the SNS canister IDs to the JSON files used by DFX (sns_canister_ids.json is not enough).

The legacy flow will still be available to users who follow the instructions from here.

Shouldn't we recommend using that other v1-legacy branch also for running the scripts (now it sounds like you can use the scripts from this branch). And maybe also remark that in the README.md on this branch?

To simulate the minimal number of contributors required for completing the swap, run

This note from the PR's description refers to the default configuration (I suppose) - shouldn't this be made more explicit?

./upgrade_test_canister.s

Good catch. I would suggest that we pass explicit arguments to DFX in these cases. If you agree, I'll clean up to make sure we're consistent in this respect.

@mraszyk

The legacy flow will still be available to users who follow the instructions from here.

Shouldn't we recommend using that other v1-legacy branch also for running the scripts (now it sounds like you can use the scripts from this branch). And maybe also remark that in the README.md on this branch?

The current idea is to not mention the legacy flow in this document at all.
Which exact scripts would not work with the new solution?
Generally, I suggest that from now on we only support small set of well-defined flows in this repo. I don't want us to spend too much time trying to create fully modular scripts, only to realize that they are never going to be truly modular, and that having a munch of scripts without telling how exactly they are supposed to be used will lead to a maintenance nightmare for us.
If this doesn't sound right to you, please reach out to discuss this 1:1!

The current idea is to not mention the legacy flow in this document at all.

You're mentioning it in the PR description. If that's not supposed to be read, then maybe we should drop it from there. And then if the "legacy" branch is effectively hidden, what's the point of having it? Is it for expert users only?

Generally, I suggest that from now on we only support small set of well-defined flows in this repo.

This makes sense: the question is how we arrive at this state. My proposal was to use the "legacy" branch for the "legacy" flow as the CI confirmed that it supports the "legacy" flow. On this branch, the CI is updated to confirm that it works for the "new" flow. If we want to support both flows on the "new" branch, we should also add CI to the "new" branch confirming that the "legacy" flow works on the "new" branch. Does that make sense?

This makes sense: the question is how we arrive at this state. My proposal was to use the "legacy" branch for the "legacy" flow as the CI confirmed that it supports the "legacy" flow. On this branch, the CI is updated to confirm that it works for the "new" flow.

Agreed.

If we want to support both flows on the "new" branch, we should also add CI to the "new" branch confirming that the "legacy" flow works on the "new" branch.

This would make sense in general, but our plan is not to advertise the legacy flow at all, since it will soon be entirely deprecated, and since it is only needed for the few projects that have already started testing with the legacy flow before the 1-prop flow will have been released.
So, the only reason the legacy flow is mentioned in the description of this PR is because the reviewers might have the question, i.e., what should we provide to the above-mentioned category of users (since this PR is deprecating the legacy flow in sns-testing).
Please let me know if this doesn't make sense, @mraszyk

In other words, why would we tell those (few) users to use the latest master and a separate document on a branch to tell them how to use the new code for the old flow?

The current idea is to have the old code with the old flow and the new code with the new flow, that's it.
We don't need a branch that supports both flows.


Facing issues with codegen

Description
I am facing weird issue with codegen when trying to use direct cpp bindings for ios.
Getting below error when building.
Undefined symbol: facebook::react::NativeCodegenExampleCxxSpecJSI::NativeCodegenExampleCxxSpecJSI(std::__1::shared_ptr<facebook::react::CallInvoker>)

When I remove outputDir from codegen config android build starts breaking. (which I think is optional)
If I turn the includesGeneratedCode to false cmake is no longer able to link the specs generated 'react_codegen_RNCodegenExampleSpec'

Reproducer Repo  Added  below
Steps to reproduce

Create a c++ module using create react native library
Use pure c++ module turbo module for both android and ios
Write native cpp code .
Write onLoad.mm
Run yarn example ios

React Native Version
0.76.3
Affected Platforms
Runtime - iOS, Build - MacOS
Areas
Codegen
Output of npx react-native info
System:
  OS: macOS 15.1
  CPU: (8) arm64 Apple M2
  Memory: 194.16 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 22.7.0
    path: ~/.nvm/versions/node/v22.7.0/bin/node
  Yarn:
    version: 3.6.1
    path: ~/.nvm/versions/node/v22.7.0/bin/yarn
  npm:
    version: 10.8.2
    path: ~/.nvm/versions/node/v22.7.0/bin/npm
  Watchman: Not Found
Managers:
  CocoaPods:
    version: 1.15.2
    path: /Users/riteshshukla2/.rvm/gems/ruby-2.7.5/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 24.1
      - iOS 18.1
      - macOS 15.1
      - tvOS 18.1
      - visionOS 2.1
      - watchOS 11.1
  Android SDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.18034.62.2412.12266719
  Xcode:
    version: 16.1/16B40
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.12
    path: /usr/bin/javac
  Ruby:
    version: 2.7.5
    path: /Users/riteshshukla2/.rvm/rubies/ruby-2.7.5/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react: Not Found
  react-native: Not Found
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: true
iOS:
  hermesEnabled: true
  newArchEnabled: true

Stacktrace or Logs
#If outputDir is removed

info A dev server is already running for this project on port 8081.
info Installing the app...

> Task :app:invokeLibraryCodegen
ℹ Building target codegen
✖ Cannot read properties of undefined (reading 'android')

> Task :app:invokeLibraryCodegen FAILED
17 actionable tasks: 5 executed, 12 up-to-date

Reproducer
https://github.com/riteshshukla04/rn-codegen-repro
Screenshots and Videos

@riteshshukla04 if you are using the create-react-native-library, I suggest you first to open an issue in their repository.
If they end up finding that the problem is in react native core, we will investigate further. This looks more a configuration issue on the template created by the tool.

Hey @cipolleschi  , Thanks for the reply.
Yes the repro example uses create-react-native-library but I am not using any of the cpp code they provided as they are not using direct cpp bindings to JS. So opening it there wont help either.
It would be really helpful if someone from RN core can look into this

I think you are doing something that you should not do:

When I remove outputDir from codegen config android build starts breaking. (which I think is optional)
This is not optional. If you are using codegen, you can't remove the folder

If I turn the includesGeneratedCode to false cmake is no longer able to link the specs generated 'react_codegen_RNCodegenExampleSpec'

If you turn includesGeneratedCode to false, the Codegen will run in the app. You can't build your library in isolation because there are some missing pieces.

In the beginning of your message you mention:

I am facing weird issue with codegen when trying to use direct cpp bindings for ios.

But then you talk about Android and Cmake.
Which platform is having issues?

I strongly suggest to start by looking into the website guides and to stick with the recommendations provided by create-react-native-library. Moving away from those binaries is a pretty advanced setup and a not recommended one.

You are right, The issue is with react native builder bob. I will raise a issue there. Thanks @cipolleschi


feat(LOC-2234): purge stale site data

Summary
Currently stale/deleted site data is not getting removed. This changes that!
Technical

remove any SiteData that no longer has an associated Site within in Local
adds unit tests for main/index to ensure that it is calling the correct functions with the correct args

Note that this does have some overlap and extends some patterns introduced in #42. Before merging, this should be rebased on top of that work once it is merged.


Radarbot calls my phone

Hi,
Since BMW Navi doesn't have speed camera alert I have used radarbot for alerts only.
This worked great as I use my Samsung watch for the notifications and therefore don't need the phone to be visible.
But ever since I started to use AAIdrive Radarbot starts a hands-free call from my phone to my phone (calls my number) when ever there's a speed cam.
I'm not sure if the call is triggered from the car or from the phone. But it's just as if I made a call using handsfree.
If I end the call it starts a new one emediatly and this repeats until I've passed the camera.
Very strange and extremely annoying.
I really like BMWs Navi but do need something to alert me about cameras, so for now I use Waze in parallell.
Is there a logical reason for this and is there a get around? Something else I can use for speed camera alert?.

That's quite a puzzle! AAIdrive doesn't set up any phone calls or anything like that, and the Reading Out of Phone Notifications relies on the car's TTS support as opposed to using a Bluetooth phone call.
Is it just coincidence? Perhaps Radarbot updated to add this feature right around the same time as when you started using AAIdrive; does it still do it even if AAIdrive isn't connected? You can Force Stop AAIdrive and it will stay disconnected until the next time you open the app or until MyBMW next reconnects to the car (if Notification support is enabled).

With both AAIdrive and myBMW apps forced off Radarbot still calls me.
So something probably happen to Radarbot app in some upgrade since this has not always been the case.
Since Radarbot  seems to be a popular app I'm surprised to not find any information from other users though.
I'll continue to investigate...

I don't use Radarbot but in Germany there is a similar App BlitzerPlus. In this App it is possible to change the Bluetooth Audio Profile. With A2DP I get the Radar Warnings with the "normal" Audio Context and with Profile HFP the App sends a call / simulate an audio call and use this way to play Audio. Perhaps this can help you.

I don't use Radarbot but in Germany there is a similar App BlitzerPlus. In this App it is possible to change the Bluetooth Audio Profile. With A2DP I get the Radar Warnings with the "normal" Audio Context and with Profile HFP the App sends a call / simulate an audio call and use this way to play Audio. Perhaps this can help you.

Thanks for the tip. But unfortunately I can not find any similar options I Radarbot. I'll keep trying alternatives...

I believe this is good to close, it sounds like a car interaction with Radarbot and not related to AAIdrive. Hopefully the Radarbot team can figure it out with you!


[ISSUE #6863] SQL expression CONTAINS support

Which Issue(s) This PR Fixes

Fixes #6863
Brief Description

For a long time, Alibaba Group has used SQL CONTAINS expressions internally to support complex business filtering logic.
Now we plan to introduce this feature into open source.
How Did You Test This Change?

The newly added code is covered by a large number of unit tests, and we have also conducted end-to-end testing to ensure everything works fine.

Codecov Report

Merging #6864 (e71efee) into develop (bee5077) will increase coverage by 0.09%.
The diff coverage is 41.27%.

@@              Coverage Diff              @@
##             develop    #6864      +/-   ##
=============================================
+ Coverage      42.65%   42.74%   +0.09%
- Complexity      9093     9108      +15
=============================================
  Files           1121     1121
  Lines          79775    79766       -9
  Branches       10409    10430      +21
=============================================
+ Hits           34027    34099      +72
+ Misses         41462    41386      -76
+ Partials        4286     4281       -5

Impacted Files
Coverage Δ

.../apache/rocketmq/filter/parser/ParseException.java
56.71% <ø> (+7.46%)
:arrow_up:

...ocketmq/filter/parser/SelectorParserConstants.java
100.00% <ø> (ø)

...pache/rocketmq/filter/parser/SimpleCharStream.java
37.94% <ø> (ø)

.../java/org/apache/rocketmq/filter/parser/Token.java
61.53% <ø> (ø)

...g/apache/rocketmq/filter/parser/TokenMgrError.java
0.00% <ø> (ø)

.../apache/rocketmq/filter/parser/SelectorParser.java
45.63% <35.49%> (+5.52%)
:arrow_up:

...etmq/filter/parser/SelectorParserTokenManager.java
47.36% <45.78%> (+0.33%)
:arrow_up:

...cketmq/filter/expression/ComparisonExpression.java
47.55% <73.33%> (+3.96%)
:arrow_up:

... and 36 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Enabled bitcode

Setting BITCODE_GENERATION_MODE is necessary to ensure that Carthage builds frameworks with bitcode. See this issue for details.

:+1:

I took the liberty of publishing a new release with this change. Hope that's alright! :+1:

@Thomvis: Thank you very much for doing so!


fix(linter): useSortedClasses false positive #3394

Summary
Supplementary test case，and fixed
<>
  <div class={`${variable}-bar ${variable}`}>

  <div class={`${variable}-bar    ${variable}`}>

</>

Expected behavior
<>
  <div class={`${variable}-bar ${variable}`}>

  <div class={`${variable}-bar ${variable}`}>

</>

What happens
<>
  <div class={`${variable}-bar${variable}`}>

  <div class={`${variable}-bar${variable}`}>

</>

Test Plan

@ematipico I have discovered some uncovered use cases, fixed them, and added supplementary test cases. Can you help review them? Thank you.


Fixed #29148 -- Explain how to use QuerySet.get_or_create() with Q objects

modified:   docs/topics/db/queries.txt

Try to explain the use case. What's the purpose of the example code?

Em 05-03-2018 14:53, Tim Graham escreveu:

Try to explain the use case. What's the purpose of the example code?

First it will try to get a record with 'first_name' Bryan or Bruce.
If gets nothing it will create a record with 'last_name' Harrison.
But 'first_name' will be empty. There is no information about the
'first_name'.
I copy/paste the example code from the "reborn" ticket on the thread.
I should put '.get_or_create({'first_name': choice_here})
Maybe a better example:

d = User.objects.all()
d
<QuerySet [<User: admin>, <User: teste>]>

First a query to get a 'username' with value 'blabla' or 'John' and 'last_name'
with value Harrison. If the query does not get anything with these values, it
will be created a record with value for 'username' John and value for
'last_name' Harrison: (manually formatted)

d = User.objects.filter(
... Q(username='blabla')|Q(username='John') & Q(last_name='Harrison')
... ).get_or_create({'username':'John', 'last_name':'Harrison'})
d
(<User: John>, True)

The word 'True' indicate that the record did not exist and was just
created

d = User.objects.all()
d
<QuerySet [<User: admin>, <User: teste>, <User: John>]>

If the query is executed once more:

d = User.objects.filter(
... Q(username='blabla')|Q(username='John') & Q(last_name='Harrison')
... ).get_or_create({'username':'John', 'last_name':'Harrison'})
d
(<User: John>, False)

The query returns what it was looking for and the word 'False' indicate that
the record already exists previously.

My question was meant to suggest that the documentation should answer those questions.

Humm, I think I misunderstood the ticket.
The .get_or_create() without .filter() is the common usage. The ticket is about .get_or_create() with Q module. That's why .filter().  It is possible to use get_or_create() with Q module without .filter()? If yes I am sorry.
The '&' is not necessary, by the way. .filter() does not need it. But the second example does not show how to use it?

I misunderstood this paragraph, I think though it's written in a confusing way. Actually I don't think this documentation belongs under |Q|. It should be under |get_or_create| docs. The end user would start by saying "I wish to use get_or_create(), but I can't pass it a Q obj as an arg. What should I do? I will look at get_or_create() docs." Another note that it's not about using |filter| method, it's about applying |get_or_create| to a filtered queryset, which may be obtained with a filter, exclude, etc, with or without a |Q| object.

The .filter() is because of the Q module. I did not say different.
The Title is "Explain how to use QuerySet.get_or_create() with Q objects".
What I see in the documentation is that complex queries with OR statements
you can use Q objects. Q objects has a link to 'construction of complex queries'
(Section 'Complex lookups with Q objects'). Which is the case.
.get() is there. And also an explanation why .filter() does not need a &
and the translation to SQL. .exclude() is there too.
I do not think is good to split the subject.

20 days.
Busy days, but I have to say that I am unable to write a good documentation.

Hi.
This is my first time contributing so please forgive any clumsiness. I wrote a small patch, but was unsure about how to move forward since the ticket was assigned and a pull request already opened. Hence my posting it here, hoping it can help move this forward.


V5 to revolution !

Hi everyone, I plan to completely revisit the way to use that library with more POO compliant usage where everythings is represented as object. :boom:
You can provide help about that and suggest many change with more modern usage of PHP. :rocket:
@zzantares If you authorize me to do that things ? :pray:

Have you seen Saleh7/ProxmoxVE_PHP_API? Don't mean to take attention aware from this repo, but more people working together can be good in the long run.

Interesting but very static usage of php :disappointed:

Hi @GothShoot, I'm sorry it took forever for me to reply; please feel free to improve this library as you may see fit, you're already a collaborator in the repo and shouldn't face any issues in getting your own code merged or integrating the outstanding PRs.
The truth is that I'm no longer working with PHP for a long time now and honestly I do not desire to come back to it (I'm trapped in FP languages land :D). I've been wanting for someone else to take over the project, atm I'm just waiting to see some contributor actively engaged with the it to "leave it in good hands".
Thank you for your interest!


Adding tooltip functionality

@tonsky, I got around to writing tooltips:
(ui/dynamic ctx [{:keys [fill-dark-gray fill-yellow]} ctx]
  (ui/valign 0.5
    (ui/row
      [:stretch 1 nil]
      [:stretch 2
       (ui/column
         (custom-ui/tooltip {:anchor :top-left :shackle :top-right
                             :left -10 :up 0}
           (ui/fill fill-dark-gray
             (ui/row
               (ui/width 120
                 (ui/height 25
                   (ui/label "TOOLTIP")))))
           (ui/column
             (ui/row
               (ui/width 200
                 (ui/height 300
                   (ui/fill (paint/fill 0xFFEFFAFF)
                     (ui/fill fill-yellow
                       (ui/label "TOOLTIP TEST")))))))))]
      [:stretch 1 nil])))

Code above produces on hover:

Let me know if the api isn't to your liking.

Found a bug where the hoverable area can be larger than the final child element. This fixes that, but not 100% sure it's generic based on your alignment system.
So I used valign/halign as suggested.

Thanks! Merged. In the future, I might extend it to do automatic positioninig based on window boundaries. Not sure if we have enough API right now to do that. Thanks for the contribution!