Module repeatedly tries to turn the tv on

The module sends the on command repeatedly which leads to the HDMI1 message in the corner of the TV.  My Javascript is not as strong and I can't do it myself but I think it would be better if the module only sends the command once to turn it on and once to turn it off. The PI can still be scanning the network every 5 seconds. I tried to implement it but failed. Please let me know what you think of this and if you would be able to help with this or if I need to start learning some more javascript.

I Will check the code so it won't sent the command all the time.


torchserve docker build fail

not torch version is specified, in my case, it is download pytorch 2.1.0, which seems not compatible with mmcv 1.7.0
Change mmcv to 1.7.2 may lead the success of the building, but fail during runnig
Change torch to 1.13.1 may lead to the error of "distribution not found"
Set torch to 2.0.0 seems ok in building and "python image_to_animation.py drawings/garlic.png garlic_out" works fine

Had same error, fixed by pinning torchserve to this version:
torch==2.0.1

Merged PR #275, which addressed docker build fail


Refactor Code

Throughout the development process we disregarded coding principles in many of the documents. The goal of this issue is the following:

[x] Remove old comments
[ ] Delete unused variables, functions, code, etc.
[x] Move certain components into their own function
[x] Move related components into sub-folders
[ ] Obfuscate API addresses through .env.local

I have refactored the main note component by divvying its states and its handlers to their own function.
Additionally all files related to the primary note component are now stored in the NoteElements folder within the components folder. Testing still needs to be done, to ensure no logic was lost in the process of refactoring.

done


SUP-1085: Cluster Queue resource implementation

PR checklist:

[ ] docs/ updated
[ ] tests added
[ ] an example added to examples/ (useful to demo new field/resource)
[ ] CHANGELOG.md updated with pending release information

Reviews have been requested but this is in draft, should it be moved to ready for review?


Add namespace query parameter for registry mirroring/proxying

This adds the ability to set a mirror to the Reference instance, which is then used to pull the image, instead of the registry that was parsed from the image name. And it then forwards the original registry with the ns query parameter to the mirror registry, so the mirror registry can use this information to decide from which registry you wanted to pull something.
This is currently not an officially documented standard feature yet, but it's probably going to be eventually, it's just taking a really long time (https://github.com/opencontainers/distribution-spec/pull/66). But containerd is already using the ns query parameter and I think I read somewhere that nexus is also already using the parameter when used as pull through cache (but I haven't verified that). But I think it's a really useful feature (when working with mirrors/proxies), even if it's not officially part of the spec yet, so I also implemented it to the library here.
Some additional notes:

It currently doesn't really handling pushing after you set the mirror registry (as it would also try to push through the mirror, which probably wouldn't work). But my assumption was, that somebody who set the mirror registry doesn't also use the same Reference instance also for pushing the same image again (would that even make sense if you just pulled it). So I decided to not add additional complexity for that use-case in the code for now. For people who don't set a mirror registry, everything works the same as before and pushing works. And if it's really needed, this logic can always be added later, but I wanted to keep the code simpler. But I'm open to feedback.
I wasn't sure if I should make the parameter for the set_mirror_registry() function an Option<String>, so you can also unset the mirror again, but for my use-case I don't need it to be unset. But let me know what you think and I can change it.
I have no idea why cargo fmt formats the rstest at the test_mirror_registry like that, but doesn't do it for the other tests :man_shrugging:

Also: The last commit doesn't have anything to do with this feature, but when scrolling through the code while adding this feature, I saw the FIXME and thought "why not just fix that as well?" :smiley:

I saw the FIXME and thought "why not just fix that as well?"

❤


V4 | The buyer cannot select the quantity purchased from the order

Hello everyone!
I noticed that the user, when he bought an ERC1155 against an ERC20 could not choose the quantity bought; If the offer is 5 ETH against 5 ERC1155, it is impossible from the SDK to buy 1 ERC1155 for 1 ETH.
I have fork your project and create a fix, not being allowed to make a PR, here is a screen of the patch :
(src/sdk/v4/NftSwapV4.ts): https://prnt.sc/c1G4i5T-Z13m
(src/sdk/v4/types.ts) : https://prnt.sc/B4Fw-ZR2Zd2w

Would want this to be integrated

I agree. Since 0x allows a taker to take fewer than the maker offered, the NftSwap SDK should support this and take an optional quantity in the options when filling a sell order.

I have fork your project and create a fix, not being allowed to make a PR, here is a screen of the patch :
(src/sdk/v4/NftSwapV4.ts): https://prnt.sc/c1G4i5T-Z13m (src/sdk/v4/types.ts) : https://prnt.sc/B4Fw-ZR2Zd2w

@boulinb Have you tested that your fix works? If it does then I agree this would be amazing to merge.

Hey @jinsley8 !
Yes I use it in production on my project ;)

What happens to the original order after a partial buy?
Can you still do another order from it or do you just cancel the remaining order?

In my case, the order is partially filled and waiting to be completely filled: If I put 2 ERC-1155s up for sale, I can buy 1 and then buy the rest back with another transaction


Support for FoundationDB 7.1

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
12 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.2%) to 74.937%

Files with Coverage Reduction
New Missed Lines
%

foundationdb/src/directory/mod.rs
1
93.38%

foundationdb/src/tuple/pack.rs
1
92.38%

foundationdb/src/directory/directory_layer.rs
4
92.9%

foundationdb/src/directory/directory_partition.rs
6
76.87%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
4165

Relevant Lines:
5558

💛  - Coveralls

Codecov Report

Merging #61 (b2ae4f1) into main (7a838a5) will decrease coverage by 0.72%.
The diff coverage is 55.38%.

:exclamation: Current head b2ae4f1 differs from pull request most recent head 88b61ea. Consider uploading reports for the commit 88b61ea to get more accurate results

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
- Coverage   83.52%   82.79%   -0.73%
==========================================
  Files          24       24
  Lines        4806     4936     +130
==========================================
+ Hits         4014     4087      +73
- Misses        792      849      +57

Impacted Files
Coverage Δ

foundationdb-gen/src/lib.rs
0.44% <ø> (ø)

foundationdb/src/lib.rs
100.00% <ø> (ø)

foundationdb/src/future.rs
65.90% <44.76%> (-8.99%)
:arrow_down:

foundationdb/src/transaction.rs
90.02% <100.00%> (+0.78%)
:arrow_up:

foundationdb/src/directory/directory_partition.rs
76.86% <0.00%> (-4.48%)
:arrow_down:

foundationdb-bindingtester/src/main.rs
89.43% <0.00%> (-0.26%)
:arrow_down:

foundationdb/src/directory/mod.rs
94.11% <0.00%> (ø)

foundationdb/src/directory/directory_layer.rs
94.32% <0.00%> (+2.23%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7a838a5...88b61ea. Read the comment docs.

I'm still not features/macros are the best way to handle fdb versions.
I wonder if version features should be additives,
For example:
710, adds 700, 630, 620, 610, 600, 520, 510
In the code we would have to care about cfg! ordering and exclusion.
Example:
// From v600 to v630 included (i.e v600 to v700 not included)
cfg!(all(feature = "fdb-6_0", not(feature = "fdb-7_0")))
// v600 only
cfg!(all(feature = "fdb-6_0", not(feature = "fdb-6_1")))

PR ready to be reviewed,I will add tenant support in a separate PR


[Request] ATT&CK stix2 classes

Is your feature request related to a problem?
No, but for background, I've rolled custom STIX and Navigator layer generation scripts for use with an ATT&CK Navigator fork.
Describe the solution you'd like
My "to-stix" Python script includes the definition of ATT&CK stix2.v20 classes for the creation of tactics and matrices.  Would it be useful to have these classes as a utility in this repository for broader use?
Describe alternatives you've considered
Looked through this repo for similar classes, but didn't find a match.
Additional context
Happy to chat about other functionality overlaps and ways to contribute back to ATT&CK.

Thanks for the suggestion! I think this would be a good place to add that sort of functionality. Feel free to add a pull request referencing this issue. I think a halfway decent spot to put them might be something like
|-- mitreattack/
    |-- stix/
        |-- __init__.py
        |-- README.md
        |-- v20.py
        |   |-- AttackTactic()
        |   |-- AttackMatrix()
        |   |-- AttackTechnique()
        |   |-- <etc.>
        |-- v21.py
        |   |-- AttackTactic()
        |   |-- AttackMatrix()
        |   |-- AttackTechnique()
        |   |-- <etc.>

Since they wouldn't be used immediately by anyone, i'm open to suggestions on structure of where they would fit in the library as well.


hangs inside jest test

ipos.new never resolves when run inside jest test

Inside jest test, the process gets a process.send function and ipos.new acts as if in a child process. But it never gets a "ready" nor a "sync" signal, so it can never sync up and then return.


'Group' Table Name Make Issue on MariaDB

I know group, when are rezerved keyword to sql but I can't see warning when generating sql or model classes. Therefore this is should be a bug.
PHPUnit 8.5.15 by Sebastian Bergmann and contributors.

Runtime:       PHP 8.0.6
Configuration: /opt/lampp/htdocs/sholist-api/phpunit.xml

......E.....................                                      28 / 28 (100%)

Time: 10.86 seconds, Memory: 8.00 MB

There was 1 error:

1) GroupTest::testGetsByShoppingCardId
Propel\Runtime\Exception\PropelException: Unable to execute INSERT statement [INSERT INTO group (id, shopping_card_id, member_id) VALUES (:p0, :p1, :p2)]

/opt/lampp/htdocs/sholist-api/propel/model/Base/Group.php:819
/opt/lampp/htdocs/sholist-api/propel/model/Base/Group.php:745
/opt/lampp/htdocs/sholist-api/propel/model/Base/Group.php:690
/opt/lampp/htdocs/sholist-api/vendor/propel/propel/src/Propel/Runtime/Connection/TransactionTrait.php:35
/opt/lampp/htdocs/sholist-api/propel/model/Base/Group.php:703
/opt/lampp/htdocs/sholist-api/propel/model/Group.php:23
/opt/lampp/htdocs/sholist-api/tests/unit/Model/GroupTest.php:16

Caused by
PDOException: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'group (id, shopping_card_id, member_id) VALUES (?, ?, ?)' at line 1

/opt/lampp/htdocs/sholist-api/vendor/propel/propel/src/Propel/Runtime/Connection/PdoConnection.php:183
/opt/lampp/htdocs/sholist-api/vendor/propel/propel/src/Propel/Runtime/Connection/StatementWrapper.php:81
/opt/lampp/htdocs/sholist-api/vendor/propel/propel/src/Propel/Runtime/Connection/ConnectionWrapper.php:394
/opt/lampp/htdocs/sholist-api/propel/model/Base/Group.php:799
/opt/lampp/htdocs/sholist-api/propel/model/Base/Group.php:745
/opt/lampp/htdocs/sholist-api/propel/model/Base/Group.php:690
/opt/lampp/htdocs/sholist-api/vendor/propel/propel/src/Propel/Runtime/Connection/TransactionTrait.php:35
/opt/lampp/htdocs/sholist-api/propel/model/Base/Group.php:703
/opt/lampp/htdocs/sholist-api/propel/model/Group.php:23
/opt/lampp/htdocs/sholist-api/tests/unit/Model/GroupTest.php:16

I think you can get Propel to put quotes around the table name by enabling identifier quoting. You can enable it for the table or the whole database by adding the identifierQuoting attribute to the respective tag in schema.xml.
I have not tested this, so if you do, please let me know if it works.
Still, I would just rename the table. I suspect the name will keep causing problems somewhere.
If you believe Propel should show a warning, when a table uses a reserved keyword as name, it is probably best if you create a PR for that.

I think you can get Propel to put quotes around the table name by enabling identifier quoting. You can enable it for the table or the whole database by adding the identifierQuoting attribute to the respective tag in schema.xml.
I have not tested this, so if you do, please let me know if it works.
Still, I would just rename the table. I suspect the name will keep causing problems somewhere.
If you believe Propel should show a warning, when a table uses a reserved keyword as name, it is probably best if you create a PR for that.

I tried identifierQuoting  attribute to fix it. But it didn't work as expected. I changed schema.xml and generated sql file using propel sql:build --output-dir="propel" --overwrite
If you mean contribute propel that saying Pull Request, I am not sure that I can do

Hey @abdyek , does this still occur on a more recent version? If so, you could perhaps help us with debugging this by telling which methods correspond to the referenced lines in the base-class.
I recently bumped into a similar issue (#1769 ) in our table/models named order, but the only occurrence of the text INSERT in the base class now has the table name properly quoted.

Hey @nederdirk :smiley: , do you want to me to test order, group etc. table names with propel master? If so, should i use identifierQuoting attribute?

Hey @nederdirk smiley , do you want to me to test order, group etc. table names with propel master? If so, should i use identifierQuoting attribute?

Thanks for your reply! Yeah, definitely with the identifierQuoting attribute enabled! (that should have been the default, in my opinion, but that's not the issue at hand) A single table with the name of a SQL keyword would be enough to test. Don't hesitate to give plenty of information, like the schema definition, which functions  show up in stack traces, and the contents of the referenced lines (and perhaps some lines before and after those, to complete the picture)

Hey @nederdirk smiley , do you want to me to test order, group etc. table names with propel master? If so, should i use identifierQuoting attribute?

Thanks for your reply! Yeah, definitely with the identifierQuoting attribute enabled! (that should have been the default, in my opinion, but that's not the issue at hand) A single table with the name of a SQL keyword would be enough to test. Don't hesitate to give plenty of information, like the schema definition, which functions show up in stack traces, and the contents of the referenced lines (and perhaps some lines before and after those, to complete the picture)

I want to test all sql reserved keyword so I created a tiny phpunit generator to test it. But I couldn't complete because it return issue. Issue is here
PHP Fatal error:  Uncaught ParseError: syntax error, unexpected 'And' (T_LOGICAL_AND), expecting identifier (T_STRING) in /opt/lampp/htdocs/deneme/propel/model/Map/AndTableMap.php:5
Stack trace:
#0 /opt/lampp/htdocs/deneme/vendor/composer/ClassLoader.php(346): Composer\Autoload\includeFile('/opt/lampp/htdo...')
#1 [internal function]: Composer\Autoload\ClassLoader->loadClass('Map\\AndTableMap')
#2 /opt/lampp/htdocs/deneme/vendor/propel/propel/src/Propel/Runtime/Map/DatabaseMap.php(115): spl_autoload_call('Map\\AndTableMap')
#3 [internal function]: Propel\Runtime\Map\DatabaseMap->addTableFromMapClass('\\Map\\AndTableMa...')
#4 /opt/lampp/htdocs/deneme/vendor/propel/propel/src/Propel/Runtime/ServiceContainer/StandardServiceContainer.php(264): array_map(Array, Array)
#5 /opt/lampp/htdocs/deneme/generated-conf/loadDatabase.php(6): Propel\Runtime\ServiceContainer\StandardServiceContainer->initDatabaseMaps(Array)
#6 /opt/lampp/htdocs/deneme/propel/config.php(33): require_once('/opt/lampp/htdo...')
#7 /opt/lampp/htdocs/deneme/tests/M in /opt/lampp/htdocs/deneme/vendor/phpunit/phpunit/src/TextUI/Command.php on line 98

But I don't know that is it about sql reserved keyword. Maybe it is my setup issue. I will review again and try to complete tomorrow. Repository link is here

Hello @nederdirk again!! :smile:
I finished testing code in the repository.
and, as, case, default, or and primary_key keywords are making issue I think because these keywords also are php reserved keywords. Other than these passed perfectly as table name and column name.
I hope my answer satisfied you.
Finally, reason of the fatal error in my previous comment this php reserved keywords.

Hey @abdyek, great researching work! It's a sad thing you can't name a class And etc. in PHP, (but I understand it from the parser perspective...)
Would you consider 'I can't give propel classes names of PHP keywords' a new definition of this bug report, or can we close this issue?

Hey @abdyek, great researching work! It's a sad thing you can't name a class And etc. in PHP, (but I understand it from the parser perspective...)
Would you consider 'I can't give propel classes names of PHP keywords' a new definition of this bug report, or can we close this issue?

Thank You!! :smiley: We can close this issue :+1:
I want to ask a question before close this issue. Can I pull request for Propel to show notice that if developers use the keywords? If I can do, how do I keep in roadmap of Propel? I've never done a pull request before for large projects. So I think must be rules for it.

I want to ask a question before close this issue. Can I pull request for Propel to show notice that if developers use the keywords? If I can do, how do I keep in roadmap of Propel? I've never done a pull request before for large projects. So I think must be rules for it.

@dereuromark Motion to close this issue?
Regarding submitting a PR: there are no global rules for contributing, and in the Propel2 project folks are generally pretty relaxed. Many projects provide a CONTRIBUTING.md file in the root of the project to lay out the rules, or a 'Contributing' section in the README.md. For Propel2 you'll find that at https://github.com/propelorm/Propel2#contribute . That section specifically states that everyone is welcome to submit a contribution; generally project maintainers will be grateful, especially if you write some paragraphs about why you think the changes are necessary and/or helpful. If you have any open questions don't be afraid to ask for help (e.g. I've asked for help about 'what would be a good method to test this' and have received great answers!)
Don't stress too much about the coding standards, as those will be automatically checked when you submit a contribution.
Peace and all good,
Dirk


Add Menu::get_current_item() helper

Ticket: #1704
Issue
Current solutions for getting the current menu item outside the main nav loop involve looping over all menu items explicitly. This is clunky.
Solution
Add a Menu::get_current_item() method that returns the current item, or the closest ancestor.
Impact
Timber is more awesome now.
Usage Changes
No breaking changes; strictly new code. I did add a note to the changelog for 2.0. Oh and MOAR DOCS. :)
Considerations
It'd be nice to add a $depth parameter in the future, to limit the depth of the tree traversal: e.g. calling get_current_item(1) (or 0?) would only scan the top menu level for the current/ancestor item. (I think the depth constructor option is 1-based? In any case it should be consistent with that.)
Once we have that it'd be trivial to implement a get_current_top_level_item helper, which is a use-case I encounter all the time. It might also be nice to add a get_current_item_parent helper.
Testing
https://www.daedtech.com/wp-content/uploads/2012/12/TestAllTheThings.jpg

Coverage increased (+0.01%) to 94.184% when pulling 93e210695269b52b8beda1f915e1aa1d35be7d3f on sitecrafting:feature/get-current-item into dd03fd94f13838b6972cd7f62c319c1100ab2360 on timber:master.

Coverage increased (+0.01%) to 94.184% when pulling 93e210695269b52b8beda1f915e1aa1d35be7d3f on sitecrafting:feature/get-current-item into dd03fd94f13838b6972cd7f62c319c1100ab2360 on timber:master.

Coverage increased (+0.01%) to 94.184% when pulling 93e210695269b52b8beda1f915e1aa1d35be7d3f on sitecrafting:feature/get-current-item into dd03fd94f13838b6972cd7f62c319c1100ab2360 on timber:master.

Coverage increased (+0.01%) to 94.184% when pulling 93e210695269b52b8beda1f915e1aa1d35be7d3f on sitecrafting:feature/get-current-item into dd03fd94f13838b6972cd7f62c319c1100ab2360 on timber:master.

I just knocked out the $depth param and get_current_top_level_item(). I may circle back to this to refactor for readability (recursion, anyone?), but unit tests are passing and I'm feeling good about the documentation. Still might be nice to have a get_current_item_parent() helper at some point.

Thanks @acobster! Taking a look at this now ...

Thanks again @acobster ! I made a few tweaks in #1724 ...

Removed get_ from public function names
Made null the default to return all values instead of -1

... this is now merged this into 2.x!


Fix broken Dockerfile code sample syntax

At the moment, the Dockerfile syntax highlighting on this page is broken because it was generated using an outdated version of Pygments, which is used by Hugo to generate those snippets. This PR changes the website setup script to include installation of Pygments 2.1.3, which supports the Dockerfile syntax.

ship it 👍


run presidio on UpVoteWeb

asked at https://huggingface.slack.com/archives/C01GSG1QFPF/p1720525385703189?thread_ts=1720079065.204769&cid=C01GSG1QFPF

can you make sure the dataset author(s) posts about it when it's live?

yup !


The website [boilerplatejs.org] has numerous spelling and grammatical errors

Is there a repository for the boilerplatejs.org website? If so the community might be able to help with some basic copy editing.

Also, I assume it's no longer hosted at boilerplatejs.org?


No emoji support

Needs emojis

To reproduce:

Add 💢 to the document (anger emoji
It will look fine for you:
..but when replicated to others, it will get borked:

Replicated to self also has the problem, can just reload.


Update container-insights-prometheus-logs.md

The link was sending to the same page

@bwren
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Impressions from channel but no spend information.

Project Robyn
Describe issue
Not really an error, but I'm putting together the dataset to use for Robyn. If I have data on a channel's impressions but not the channel's spend -- do I include the impression variable?
A similar question I have is: If they're organic impressions, do I just leave them in?
Wondering what people have done in this situation.
Thanks!

Hi @hd-fernandez
Are you referring to a paid channel that you don't have the spend information of? If that's the case, you must have spend data for it to make sense. If it's organic, it's OK to pass it via the organic variables. Keep in mind we will be able to use the budget allocator afterwards only with paid media variables.

@laresbernardo Got it. So if it's a paid channel, it must have spend data if I am going to include the impressions. If it's organic, put it into the organic variable argument. Thank you!

So in the case described in this thread paid channel without known spend but with known impression data should be excluded from the model? Or is it better to treat it as if it was organic?


Fix flaky tests

Currently, our test suite actually downloads the Deno CLI binaries as part of each run. This makes the tests flaky and they frequently fail due to network/IO errors. We should consider mocking the HTTP call and using a local version of the Deno CLI.

Still not fixed, sadly.

Hopefully #139 fixes this. Let's reopen if we see failures again.


Add type safety via enums

We have slowly done this over time in a lot of places, closing this out as no longer relevant.


LXD Driver Support

Would LXD as a Driver for Molecule provide any value compared with the existing Docker Driver? I have been using LXD on Ubuntu 16.04 with a ZFS backend now for awhile and have been amazed at the performance - usually about 1.6 seconds from lxc launch to when the container is up and running. Leveraging this for playbook testing would be great! Also, there might be some other advantages with LXD since it is intended to run "full-server" containers whereas Docker is more geared towards running a single app. Thoughts on this?

Hi @asmartin that sounds like a great suggestion.  However, we will not be able to implement the work any time soon.  Please feel free to contribute and add if you feel strongly.  This is how we obtained the docker and openstack drivers.

I'm also quite busy at the moment so won't have time to dive in and right an LXD driver right now, but would be happy to help test if someone else can help to write it

I'm also quite busy at the moment so won't have time to dive in and right an LXD driver right now, but would be happy to help test if someone else can help to write it

This likely will not get implemented then.  Will close this out next week, unless someone decides to pick it up.

Is it possible to utilize vagrant-lxc as a means of provisioning LXC containers?

Is it possible to utilize vagrant-lxc as a means of provisioning LXC containers?

That would need implemented into the Vagrantfile much like the other providers we have.
This is being reworked in #582, #582, and #463.

Closing this issue.  Reporter does not have time to write this feature, and we don't have a need for it, which prevents us from putting resources on it.
@asmartin Please feel free to reopen when you have cycles to implement it.


contract: Build & typecheck docs

Add doctests
Build with asciidoc

Yeah it's a stepping stone. I'm trying to do smaller, more frequent PRs.


[Feature Reqeust]can custom debounce value

Existing Component
Yes
Component Name
el-select
Description
In version 1.4.2, el-select can't set custom debounce value but others can.

What is custom value?

@Leopoldthecoder  the debounce is a fixed value for now.

Sorry, we have no plan to add this API.

this sucks! Why can I not control the time that the remote-method takes to be dispatched?


SQL table support

Hi Doug,
So I use Invoke-Sqlcmd to insert data to Azure SQL databases.
I used a connection string like this one:
$ConnectionString = "Data Source=tcp:$($SqlServer).database.windows.net;Initial Catalog=$($SqlDatabase);Authentication=Active Directory Password;User Id='$($account)';Password='$($SqlCredsSecretValue)'"
And then I saw the ConvertFromExcelToSQLInsert cmdlet that helps write the INSERT commands to SQL.
Obviously the combination of the two will insert the data to the table.
Do you have anything that would CREATE the table in the database from an Excel sheet?
Or similarly to ConvertFromExcelToSQLInsert, generate the sql commands to create the table with the right column names and types?
Thank you!

No, don't have that. Interesting idea though.

Nice! Does the ConvertTo-DbaDataTable infer the datatypes too?


installation: pin invenio-celery <5

I guess I will close the one I made yesterday https://github.com/inveniosoftware/invenio-celery/pull/69

@lnielsen unfortunately Travis if failing because of a version conflict. And also, why removing isort checks?

@lnielsen unfortunately Travis if failing because of a version conflict. And also, why removing isort checks?

I'm not removing isort, but instead moving them into pytest as part of the check (the --isort in pytest.ini).


Fix aggregation

Codecov Report
Attention: Patch coverage is 88.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 73.16%. Comparing base (257e2d2) to head (10c2127).

Files with missing lines
Patch %
Lines

...nt/torch_bench/_bash_bench_benchmark_runner_agg.py
88.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #223      +/-   ##
==========================================
- Coverage   73.16%   73.16%   -0.01%
==========================================
  Files         143      143
  Lines       20693    20699       +6
==========================================
+ Hits        15140    15144       +4
- Misses       5553     5555       +2

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


Feature/auto defer many relations

Ember Data gets confused when many-relations return IDs, and there is also a link object. This PR adds an option to automatically defer many-relations, unless deferred=False is explicitly set on the field. The option is disabled by default for now for backward compatibility.

:+1:


add moveit stack and related packages

I know especially I should have done that already, but I had other priorities in ROS-O for now.

Currently somehow the version of MoveIt (and all subpackages) is detected as 0 which makes it fail because in some cases a minimum version is required
ros-one-moveit-ros-planning : Conflicts: ros-one-moveit-ros-planning-interface (< 1.1.2) but 0-2<IP_ADDRESS>.48 is to be installed

https://github.com/isys-vision/ros-one-moveit-overlay/actions/runs/4133800349/jobs/7144142434

The fork you selected is missing tags:
https://github.com/v4hn/moveit/tags

hmm, shouldn't the version be defined by package.xml?

We use git describe to include the number of commits and hash to allow nightly builds with incremented version numbers:
https://github.com/jspricke/ros-deb-builder-action/blob/main/build.sh#L97

The only way I see around this is, instead of relying on the git tag, to identify the latest commit that changed the package.xml's version tag and count above that one instead.
However, that is quite heavy-weight and less intuitive than simply using the latest tag. git tags with version numbers for releases are standard across FOSS git projects and even default with ROS bloom releases.

done


Fix NetworkTopology breadcrumbs link

Purpose or Intent
Fix NetworkTopology breadcrumbs link
Steps for Testing/QA
Fixes BZ:
https://bugzilla.redhat.com/show_bug.cgi?id=1347241

Darga Backport conflict:
$ git cherry-pick -x -m 1   d524643
error: could not apply d524643... Merge pull request #10770 from Ladas/fix_network_topology_breadcrumb_link
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

$ git status
On branch darga
Your branch is up-to-date with 'upstream/darga'.
You are currently cherry-picking commit d524643.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)

        both modified:   app/controllers/network_topology_controller.rb

$ git diff
diff --cc app/controllers/network_topology_controller.rb
index 05f9545,97ae7c5..0000000
--- a/app/controllers/network_topology_controller.rb
+++ b/app/controllers/network_topology_controller.rb
@@@ -9,15 -10,9 +9,21 @@@ class NetworkTopologyController < Appli
      # all previous navigation should not be displayed in breadcrumbs as the user could arrive from
      # any other page in the application.
      @breadcrumbs.clear if params[:id].nil?
++<<<<<<< HEAD
 +    drop_breadcrumb(:name => _('Topology'), :url => '')
 +  end
 +
 +  def index
 +    redirect_to :action => 'show'
 +  end
 +
 +  def data
 +    render :json => {:data => generate_topology(params[:id])}
++=======
+     drop_breadcrumb(:name => _('Topology'), :url => "/network_topology/show/#{params[:id]}")
+     @lastaction = 'show'
+     @display = @showtype = 'topology'
++>>>>>>> d524643... Merge pull request #10770 from Ladas/fix_network_topology_breadcrumb_link
    end

    private

@Ladas Please run the same cherry-pick command locally, resolve conflicts, validate, and make a Darga PR from it.

@chessbyte done here https://github.com/ManageIQ/manageiq/pull/11005

Backported to Darga via #11005
$ git log

commit 7361adfc8525c83c0bd526e60396b34ce8c291ba
Merge: 57949a9 df9fc87
Author: Oleg Barenboim<EMAIL_ADDRESS>Date:   Mon Sep 5 15:46:09 2016 -0500

    Merge pull request #11005 from Ladas/darga_fix_network_topology_breadcrumb_link

    [DARGA] Fix NetworkTopology breadcrumbs link


[Ripple network that can sign] (Version: [1.11.0])

hello.
I have a question regarding Ripple nodes.
Currently, I am running a Ripple node.
But here's where a problem arose.
I created a wallet address using C# language.
I tried to synchronize that address to the node through the RPC "submit" function.
However, the results were as follows.
{
"result": {
"error": "notSupported",
"error_code": 75,
"error_message": "Signing is not supported by this server.",
"request": {
"command": "sign",
"fee_mult_max": 1000;
"offline": false,
"secret": "",
"tx_json": {
"Account": "r5LWzT19hrddhReiM1yVBFki4AuaF9dUF",
"Amount": {
"currency": "XRP",
"issuer": "r5LWzT19hrddhReiM1yVBFki4AuaF9dUF",
"value": "1000"
},
"Destination": "rB2TiKtjqdoHhekTwL2JdCKDsxg276J7eJ",
"TransactionType": "Payment"
}
},
"status": "error"
}
}
Can't we create a node that can sign?
I think that in order to create a signing node, you need to create a full node. Is this correct?
I would like to solve this problem.
Experts, please help.

You need an admin connection to the rippled node. Which connections are considered "admin" depends on your rippled.cfg.
Alternatively, you could enable public signing.


custom API provider usage

can anthracite be used with other API endpoints than firebase?

No. It'll be a big task, I cut scope by only supporting Firebase in order to meet internal deadlines.
If there's other real-time document stores that are desired I'm open to PRs adding support.

i belive that Horizon.js could fit for


update sum.js

wrote an arrow function to find sum of two numbers.

wrote an arrow function to find sum of two numbers.

Looks good! When using arrow functions, the brackets and return keyword is usually omitted, so you can just simply write const sum = (a,b) => a + b.


Loader tasks occasionally hang

Today I discovered a handful of loaders that had somehow gotten stuck hanging for more than a month. Unfortunately, there’s no ECS-native way to add timeouts to tasks (it’s been a planned feature for 2.5 years, though! https://github.com/aws/containers-roadmap/issues/572), so we should consider doing something to handle this ourselves.
Some options:

Use the timeout command as part of the loader entrypoint. For example, instead of: https://github.com/usdigitalresponse/univaf/blob/a5622335ddf5bff353250133ce9b4eba9806909d/loader/Dockerfile#L24-L25
ENTRYPOINT ["timeout", "1800", "node", "./bin/univaf-loader", "--send", "--compact"]

(May also require installing timeout with sudo apt-get install timeout)

Add some JS to the CLI entrypoint that starts a timeout that kills the process, for example:
setTimeout(() => process.exit(1), 30 * 60 * 1000).unref()

Something with AWS Step Functions (I didn’t even know this existed) as described here: https://github.com/aws/containers-roadmap/issues/572#issuecomment-805996037

A CloudWatch alarm that notifies us of jobs that have run too long so we can manually kill them.

The first option seems like the most straightforward to me.

We have not seen this issue recur since I added timeouts in #880, so I’m marking it as done.


Open Dev14 solution with xproj AspNetCoreWebApp_NoAuth_1 using VS, global.json shows up twice in Solution Explorer

Steps to reproduce
Using VS 2017 RC.3 (Build d15rel 26109.02) with Web workload
git clone https://github.com/balaTest/ProjectKTestProjects
Open solution under Dev14_K101/Tooling202/AspNetCoreWebApp_NoAuth_1
Actual behavior
global.json shows up twice in Solution Explorer

Expected behavior
global.json shows up only once in Solution Explorer

And the sln file doesn't list it twice? We don't add the global.json at all. Tagging @mlorbetske

I just checked the sln file and there does seem to be 2 entries there
Project("{2150E333-8FDC-42A3-9474-1A3956D46DE8}") = "Solution Items", "Solution Items", "{23CBDAA2-0C75-456E-91D4-EE16A435FB82}"
	ProjectSection(SolutionItems) = preProject
		..\AspNetCoreWeb_IndAuth_WithAI_1\global.json = ..\AspNetCoreWeb_IndAuth_WithAI_1\global.json
		global.json = global.json
	EndProjectSection
EndProject

The global.json is added by the web tooling and is planned to be removed.


Replace Foundation SPI exports with local implementations

Re-implement _NSErrorWithFilePath and _NSErrorWithFilePathAndError locally so that apps using the overlay don't have to get permission to use SPI.
Resolves rdar://problem/30541957

@swift-ci please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - dc3bda86ff591934a2e4661b1cf90ae2e4778c0d
Test requested by - @kperryua

@swift-ci please test

This found its way into master in dc783c0


fix: set size-limit to 1kb;

Опишите проблему
Size limit показывает табличку изменений после рефакторинга, по пакетам.

Понизил уровень с 2кб -> 1кб
Добавл иконку под положительные изменения

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.005%) to 82.269%

Totals

Change from base Build<PHONE_NUMBER>2:
0.005%

Covered Lines:
10620

Relevant Lines:
11781

💛  - Coveralls


Smt-lib Function Specifications

We have the capability to add custom smtlib pre and postconditions to the main function we are actually querying, but it seems like it would be useful to add the capability for the user to add custom smtlib pre/post conditions as function summaries for functions encountered within the main function.
This capability seems like it might extend to an interesting scaling story in which we verify these summaries actually hold and then use them as summaries in verification of other functions. In a related but more aggressive idea, we may wish to attempt to synthesize these function summaries off of templates or some other means.

I propose we create a flag --fun-spec="<function_name>,pre:<smtlib-precondition>,post:<smtlib-postcondition>" that would allow the user to supply a function and respective specs. The user could also choose to only supply a precondition or a postcondition. It would act similarly to the flags precond and postcond, just with the added detail that this would apply to a specific function-name <function_name>, whose contents will be treated as a black box.


[Server side] Doing a blocking call on IJSRuntime.InvokeAsync ends up locking the server

Super nitpicky, but doing a .Result or 'Wait on the task returned by IJSRuntime locks up the server. E.g.
void DoLog()
{
    JSRuntime.Current.InvokeAsync<string>("console.log", "Hello world").Wait();
}

The server does not respond to any further actions.

Correction: It locks up the current connection. New connections continue work.

Same when calling .Result btw no error message anywhere

Just had something similar on my chat sample app, https://github.com/conficient/BlazorChatSample
I was upgrading to use async invoke, and it was hanging and not returning when I called
var tmp = await JSRuntime.Current.InvokeAsync<bool>("ChatClient.Start", ....`

The JS method returns the promise returned by SignalR's connection.start() method, but this wasn't working. The method returns a promise but there isn't a return value in the promise (at least one I don't need).
Changing this to
var tmp = await JSRuntime.Current.InvokeAsync<object>("ChatClient.Start", ...

fixed the problem and it worked, so I suggest you try that. I also amended my C# methods to be async as well (see https://github.com/conficient/BlazorChatSample/blob/master/BlazorChatSample.Client/ChatClient.cs )

Making a blocking call on a task can deadlock in ASP.NET too, this is because of synchronization context, I recommend using await and not blocking at all if possible.

@pranavkm as @psibernetic said, blocking asynchronous calls in code running on any server-side ASP.NET Core stack will cause a deadlock; however, if you are unable to await the call, you can get it's result by using Task.Run(async () => await JSRuntime.Current.InvokeAsync<string>("console.log", "Hello world").ConfigureAwait(false)).Result, but it may or may not launch the call into a new thread. As far as I know, that should allow for the blocking of a thread for the purposes of synchronously waiting for the result of an asynchronous call, without deadlocking the process. Please let me know if it works.

@TheFanatr the bug was discovered \ filed as part of verifying the 0.5.0 release. Not doing blocking calls are always recommended, but oftentimes developers or even frameworks write code that blocks. The current experience around it isn't super great.

We need to do something to address this even if it's throwing an exception to report the deadlock. Threads are a limited resource.

Shouldn't it be possible to easily fix this just by adding ConfigureAwait(false) to the awaited async calls in the InvokeAsync method itself to avoid deadlocks?

@pranavkm as @psibernetic said, blocking asynchronous calls in code running on any server-side ASP.NET Core stack will cause a deadlock

No absolutely not. This deadlock is blazor specific behavior.
A custom awaiter backed by ValueTask may be the best way to achieve this.

I have the same problem with 0.5.1.
My scenario: I get the JWT token from the local storage in my App.cshtml. This operation should complete before OnInitAsync ends because pages need this token for authenticating the API calls.

@NikolayIT Does awaiting the async call not work for your scenario?

@danroth27 awaiting the async call works as expected. But as I explained I need to execute it synchronously as its result is needed by the child components and pages.

@NikolayIT Ah, I see. Yes, even if you await the call to retrieve the JWT in OnInitAsync the component rendering will proceed. You'll need to have logic in the child components that checks to see if the JWT is available, and if not display some sort of "Loading..." message. We do rerender once OnInitAsync completes. If your setting up the JWT in the App component and using routing, then that complicates things further. We probably should have better docs on how to do this: https://github.com/aspnet/Blazor.Docs/issues/240

Hi,
try to use it
string tnkn=(JSRuntime.Current as IJSInprocess.Runtime).Invoke("GetTkn",...)
I used it and it worked for me in synchronous code. but this is not my personal hack. I saw this in some of the Blazors extensions unfortunately I don’t remember in which

@SteveSandersonMS  - I'm looking into fixing this. This would be cleanest to solve if DotNetDispatcher.EndInvoke were an intrinsic rather than just another usage of JS Interop. Put another way, the fact that DotNetDispatcher.EndInvoke is called via JS Interop by the client means that it's functionally indistinguishable from other JS Interop calls.
The problem with this is that I need to be able to break the rules when completing an interop operation. Interop calls need to acquire the sync context in order to call user code. In this case acquiring the sync context is what causes the deadlock.

I also tried implementing detection for deadlocks instead, but it ends up being a very hardcore solution.  Basically I can make it so that the sync context is notified whenever blocking occurs, but that includes all blocking, locks, events, tasks, etc - http://joeduffyblog.com/2008/02/27/hooking-clr-blocking-calls-with-synchronizationcontext/

Here's what I'd like to do. Change DotNetDispatcher.EndInvoke to not be JS Invokable, and make it another method exposed by DotNetCallDispatcher - then replace the usage of it in the .ts code with the direct call.
Then we can implement this on the .NET side by making it another virtual method on JSRuntimeBase. The circuit host will then set JSRuntime.Current but not acquire the sync context before calling EndInvoke.

@rynowak why not detect and throw, why "fix" it?

What's your suggestion on how to detect a task.Result but not a random lock? Using SynchronizationContext.Wait seems like too big of a hammer.


vm command modules: revisit all artifacts marked as preview

Log this as the check-point so we can revisit routinely to remove those flags once GA, and apply necessary adjustment if the GA version has breaking change:

~VMSS Rolling upgrade~
~Availability Zone~
~Run-Commands~
~Disk Encryption~
~OS-disk swap~

@ejarvi, is single pass disk encryption still under feature flag for VM and VMSS?

Single VM - No
VMSS - Yes

@ejarvi just to confirm vmss disk encryption is no longer in preview right?

That's right...

Announcement:  https://azure.microsoft.com/en-us/blog/extending-azure-security-center-capabilities/
Windows: https://docs.microsoft.com/en-us/azure/security/azure-security-disk-encryption-windows#encrypt-virtual-machine-scale-sets
Linux: https://docs.microsoft.com/en-us/azure/security/azure-security-disk-encryption-linux#encrypt-virtual-machine-scale-sets

Thanks a lot @ejarvi

Will close this as this is more of a reminder and not an issue. We will always have features in preview. I have removed. Moreover the features mentioned in the issue have had "(PREVIEW)" removed from their help text.

Some "preview" text in compute help text has been removed in #9306


Compile command for new file incorrectly being taken from a different folder's source file.

For context, my project has this structure:
src/
  |- foo.cpp    #     in compile_commands.json.
  \- bar.cpp    #     in compile_commands.json.
test/
  |- foo.cpp    #     in compile_commands.json.
  |- bar.cpp    # not in compile_commands.json.
  \- xyz.cpp    # not in compile_commands.json.

All of the source files are in compile_commands.json except for bar.cpp and xyz.cpp which are new files created after compile_commands.json was generated.  Also, the source files in test/ have slightly different compile flags as compared with the ones in src/.
Expected behavior: when opening either test/bar.cpp or test/xyz.cpp in the editor, clangd will "guess" the compile commands by using those of test/foo.cpp since it is in the same folder.
Actual behavior:

For test/xyz.cpp clangd does what I expect, and uses the compile commands from test/foo.cpp, and thus compilation of test/xyz.cpp succeeds.
For test/bar.cpp, however, it uses those from src/bar.cpp, presumably because it has the same file name. This causes the wrong compile flags to be used and, as a result, compilation of test/bar.cpp fails.

Request: If the above is indeed the current behavior of clangd, then I would request that when guessing the compile command for a file that does not have an entry in compile_commands.json, clangd should look first to other files in the same folder, and only if no such file is found, should it look to files in other folders.
Logs: I won't paste them here because they are large, but I did look at the Neovim trace-level LSP logs and that is how I determined which compile commands clangd is using for each file.
System information
Output of clangd --version:
clangd version 16.0.0 (https://github.com/llvm/llvm-project.git 296f7fbbb5c48a961aa31b2ffa1d048e6bfbd7a1)
Features: linux
Platform: x86_64-unknown-linux-gnu

Editor/LSP plugin: Neovim v0.9.0 with builtin LSP
Operating system: Pop!_OS 22.04

Preferring a file with the same basename in a different directory is deliberate.
The motivating scenario for that is the following very common setup:
include/
    Widget.h
src/
    Widget.cpp

Most build system do not generate CDB entries for header files, so Widget.h will not have an entry, and Widget.cpp is the logical file whose entry to use for it, but it may be in a different directory.
I can see an argument to limiting this strategy to header files only, and for source files preferring other source files in the same directory.


Segmentation fault when (OpenFAST with tower) + (AMR-Wind without tower)

Bug description
When OpenFAST is used with AMR-Wind, a segmentation fault occur if AMR-Wind is not using a tower, but AeroDyn15 is configured with a tower.
To Reproduce
Using AMR-Wind with OpenFAST 3.5.0.  In AMR-Wind, turn off the tower, but in AeroDyn15 in OpenFAST leave the tower on (TwrShadow, TwrPotent, or TwrAero turned on)
OpenFAST Version
This is known to happen with 3.5.0, but may happen with earlier versions as well.
Additional context
The issue is with the lack of error handling in the OpenFOAM module during initialization.  Currently there is no logic to catch when AMR-Wind does not have a tower, but AD15 does.  I am uncertain exactly how catch this at the moment given how the meshes are built and transferred.  The error is occurring in a call to Transfer_Line2_to_Point in either SetOpFMPositions or SetOpFMForces called by OpFM_SetInputs.  However this really should be remedied by logic during initialization

Thanks to @shenpai35 for finding this issue.


select first voice by defult

hi thanks for awesome package
how i can select first voice after stop recording?

@farhadf246 Hi, thnx, just click by mouse


Renaming catalog makes it unable to remove the catalog

Rancher versions:
rancher/server: v1.6.14
After renaming the catalog name from foo-bar to foo/bar it is not possible to delete the catalog.
The DELETE Request returns 404
Request URL:http://rancher/v1-catalog/catalogs/foo/bar
Request Method:DELETE

Thanks for your report!
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
If you think we should keep this issue open, please let me know.


Add missing word "like"

Should be: Looks like we've got no publications

Sorry for the messy commit history. I didn't see an easy way to add commits to a pull request via GitHub, so I merged them in via additional pull requests.


Am I using the parser correctly or not? Validation issue.

My question is simply how the public func parse(numberToParse: String!, defaultRegion: String!) throws -> NBPhoneNumber works? More specifically, I'm having a parsing issue in my app. When the user enters numbers it's all fine, but if the user starts adding letters the validation is suddenly not working so great anymore. The user can add up to two letters and the parser will still return a correct NBPhoneNumber, unless the user adds the letter "ö" which, for some reason, will always return a validated NBPhoneNumber.
Clarification: If the user enters "0706348441ee" the parser will return "0706348441", but if the user enters "0706348441eee" it will return "070634844133", which is too long for a number of my region thus failing validation. Or the user can enter "0706348441öööööööööööööööööööö" which will return just the numbers. The discomforting part is mainly that if you feed the parser letters it will start returning numbers, how can three "e"'s turn into the number 33?
So my question is wether you are supposed to send in the user's entered value to the parser, or if you are supposed to send it numbers only?
My snippet of code:

func validate(value: String?) -> Bool {
    var valid = false

    do {
        let phoneUtil = NBPhoneNumberUtil()
        let phoneNumber = try phoneUtil.parse(value, defaultRegion: "SE")
        valid = phoneUtil.isValidNumberForRegion(phoneNumber, regionCode: "SE")
    }
    catch let error as NSError {
        logger.debug(error.localizedDescription)
    }

    return valid
}

Thanks!

libphonenumber can parse character as phonenumber
EX>
NBPhoneNumber *tollfreeNumber = [[NBPhoneNumber alloc] init];
[tollfreeNumber setCountryCode:@64];
[tollfreeNumber setNationalNumber:@800332005];
XCTAssertTrue([tollfreeNumber isEqual:[_aUtil parse:@"0800 DDA 005" defaultRegion:@"NZ" error:&anError]]);

You can test it using libphonenumber's demo page.
https://rawgit.com/googlei18n/libphonenumber/master/javascript/i18n/phonenumbers/demo-compiled.html
And, how about to use below methods ?

(NSString*)normalizePhoneNumber:(NSString*)phoneNumber;
(NSString*)normalizeDigitsOnly:(NSString*)number;

Thanks, that's exactly what I was looking for. Will rewrite my validation a bit then, thanks for the quick response! :)


Revert "(packaging) Use CMake project to specify version"

Reverts puppetlabs/facter#1376
The original commit missed that the gemspec used the version variables. Also, somehow we have no testing that covers the gem.

:(

I can fix this up, I'd just like to look into whether we should be testing the specfile somehow.


Replace voting option with serial number in mobile layout

Figma link
https://www.figma.com/file/vqpglMGW8psHKB00eIVDUV/OpenSquare?node-id=3658%3A22300
Preview

done


Allow resolvers to return null instead of Task

Async resolver methods are allowed to return null to bail out early, like this:
ctx.Field("somefield").Resolve(ctx => {
  var condition = ...;
  if(!condition) {
    return null;
  }
  return _service.RunSomeTask(); // returns Task
});

However, the typings of the Resolve methods involved here do not allow returning null instead of Task<TResult>, even when the code handles null fine. I've extended the typings to allow null explicitly.

/AzurePipelines run


Added Jclouds module with jclouds Actions (openstack+Amazon compute) Signed-off-by: Alex Persa<EMAIL_ADDRESS>

@oritstone @meirwah @Alex-Persa
Who is supposed to review this PR?

I review it, but there are some more code changes that Alex has to made.After Alex will fix them, I'll made the merge.

Ok. Just wanted to make sure it's not hanging :smile:


Move defaultNamespace under application

In its current location, it is almost always missed.

@rushtehrani Do I need to change
common/onepanel/base/configmap-onepanel-onepanel.yaml?
The
data:
  # This key right here?
  applicationDefaultNamespace: $(applicationDefaultNamespace)

@rushtehrani Do I need to change
common/onepanel/base/configmap-onepanel-onepanel.yaml?
The
data:
  # This key right here?
  applicationDefaultNamespace: $(applicationDefaultNamespace)
  # from defaultNamespace: $(defaultNamespace)

Might as well get it done, you'd have to update that key in onepanelio/core too I believe.

I'll dig around

I can't find it as well.
We do pull that configMap, onepanel, for database connection.. But not the default namespace as far as I can tell.
I'll remove it for now.


Readme not clear

This sentence in the README.md is not clear:
Models that are loaded via either transformers.AutoModelForCausalLM (autoregressive, decoder-only GPT style models) or transformers.AutoModelForSeq2SeqLM (such as encoder-decoder models like T5) in Huggingface are supported via Support for this model type is currently pending.

what does it mean by "via Support for this model type is currently pending" -- perhaps this could be rewritten to be more clear.

Thanks! I've fixed this.


Minor: Fixed conn is busy issue with mssql

Describe your changes:
Minor: Fixed conn is busy issue with mssql

Type of change:

[x] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

needs to be against main


Socket::IPAddress constructors

The only ways to construct a Socket::IPAddress are by parsing a string or using the C representation that is not publicly documented:
Socket::IPAddress.new("#{192}.#{168}.#{1}.#{3}", 1234) # => Socket::IPAddress(<IP_ADDRESS>:1234)

addr = LibC::SockaddrIn.new(
  sin_family: LibC::AF_INET,
  sin_port: 1234.to_u16.byte_swap,
  sin_addr: LibC::InAddr.new(s_addr: 0x03_01_a8_c0),
)
Socket::IPAddress.from(pointerof(addr).as(LibC::Sockaddr*), sizeof(typeof(addr))) # => Socket::IPAddress(<IP_ADDRESS>:1234)

There should be simpler constructors to do this:
struct Socket::IPAddress
  def self.v4(x0, x1, x2, x3, *, port) : self
  end

  def self.v6(x0, x1, x2, x3, x4, x5, x6, x7, *, port) : self
  end

  def self.v4_mapped_v6(x0, x1, x2, x3, *, port) : self
  end
end

@straight-shoota This would mean doing the same for every caller, sounds like avoidable PITA :)

Yeah, that might be too annoying for using it with arbitrary numbers.
However maybe we could have an overload for UInt8 to skip the validation and conversion when the types already match.

For #13191 it'll be nice to have StaticArray overloads as well because I am doing something like this currently:
# port is Int32 because that's what the existing constructor uses

if v4_fields = parse_v4_fields?(address)
  addr = v4(v4_fields[0], v4_fields[1], v4_fields[2], v4_fields[3], port: port.to_u16!)
elsif v6_fields = parse_v6_fields?(address)
  addr = v6(v6_fields[0], v6_fields[1], v6_fields[2], v6_fields[3], v6_fields[4], v6_fields[5], v6_fields[6], v6_fields[7], port: port.to_u16!)
else
  raise Error.new("Invalid IP address: #{address}")
end

Maybe static array should be supported for splatting: v4(*v4_fields, port: port.to_u16!) 😄
Alternatively, StaticArray#to_tuple could be a useful enhancement as the opposite of Tuple#to_static_array (#12930). In contrast to the latter, it should always be just require a cast for reinterpretation because the memory layout is identical.

It would be too easy to instantiate Tuple types with an extremely large number of generic arguments. I think #to_tuple gives people the wrong idea about when to or not to use StaticArray.
So I think only these overloads are necessary:
struct Socket::IPAddress
  def self.v4(x0 : Int, x1 : Int, x2 : Int, x3 : Int, *, port : Int) : self
  end

  def self.v4(fields : UInt8[4], *, port : UInt16) : self
  end

  # ditto for `.v6` and `.v4_mapped_v6`
end

I don't follow the argument- why did the parameters end up as Int and not UInt8? These are bytes. Why is there a run-time check when just specifying the correct type would ensure static safety?

The idea was to have a convenience constructor that receives Int where the caller does not need to cast the type. And there's also an efficient constructor which receives UInt8 (in form of a static array), so you can avoid the runtime check. This constructor is less convenient to call, though. I think that can be acceptable.
However I'm wondering about concrete use cases for the convenience constructor with Int. I can't think of many, so maybe UInt8 would really be better?
Regarding the representation of IPv6 I suppose there are good reasons for both. The general concept is based on 16 bytes, but typical string representations use 8 numbers.
Accordingly, there are APIs which use one or the other (e.g. https://doc.rust-lang.org/std/net/struct.Ipv6Addr.html, https://pkg.go.dev/net#IP, https://docs.oracle.com/javase/8/docs/api/java/net/InetAddress.html#getByAddress-java.lang.String-byte:A-).


SOX-15988: Camel case transformed variable name and reduce them to three words

For future reference, I opted to go with the first two words plus the last word because some translations require different fields from the same object, so if we have something like model.label.name and model.label.description, they'll be properly disambiguated.


'All Exceptions' and 'Uncaught Exceptions' options disappear (C# - Mac)

Using VS Code and vscode-mono-debug on a macOS 10.12 for a C# project. When the project is started, the debug panel briefly lists 'All Exceptions' and 'Uncaught Exceptions' options. But these disappear moments later when the debugger attaches to the xsp process.

The scripts/configurations which we use for starting the debugger are attached:
Debug scripts.zip
project/launch.js
project/startServer.sh
project/.vscode/launch.json

This works as designed: mono-debug has no exception filter options.


Terraform template initial commit

Initial PR for adding a GHA that will:

Setup Infra via Terraform
Run the Liquibase Test Harness
Destroy Infra via Terraform

This is a first draft and will likely require some tweaks including a harness-config.yml

This was done in a separate PR


Fix _query_is_cached to actually enable caching

_query_is_cached will always returns false because _cache.get expects key to be in a tuple. This renders the caching useless.

I'm surprised that this PR receives no attention because it improves performance of our service by a large margin. To understand the effect of this change, creates a Magnitude class with lazy_loading greater than 0 (e.g. 1000), and observe _vector_for_key_cached._cache.hits and  _vector_for_key_cached._cache.misses when you query a list of embeddings. Without this change, _vector_for_key_cached._cache.hits will remain 0 even after calling query(['hello', 'world']) more than once.

Hi @zfang,
Thanks for this PR, this likely broke at some point when I modified the underlying LRU cache. Sorry, I've been on travel for the last week or so, I'll get around to reviewing this tonight and merging this in the next few days.
I'll also add some tests to make sure the cache works and prevent regressions in the future.


Connector naming convention

I noticed all connectors are named in a different way, and don't follow general rules. I propose a naming convention for connector filenames, which should help to keep the source code in the same style.
https://gist.github.com/alexesprit/f86391c1d72de44146b57e8acdddd6d6
Not sure if all the things are described clearly in the convention. Also, not sure if it's useful. Any feedback is welcome.
P.S. I know that some connectors don't match the rules mentioned in convention; they should be updated if the convention will be accepted.

I'd also suggest to include top-level domain if it's part of the service name(e.g. listen.moe)

@f1u77y yes, that makes a sense. I updated the document according your suggestion.

Conventions FTW - I think including the tld could be useful. Although how would this work for sites which use multiple? Can't think of any off the top of my head. They normally redirect to the .com.

Although how would this work for sites which use multiple? Can't think of any off the top of my head.

Yes, that's pretty rare case, but I read this recently comment and decided to mention such cases in the convention.

I'm all for this - what you documented seems good to me :+1:
However what would the file name be for this
	{
		label: 'Wonder.fm',
		matches: ['*://wonder.fm/*', '*://white-label.fm/*', '*://primary.fm/*'],
		js: ['connectors/wonder.fm.js'],
	},

@inverse it should be called either "wonder.js" (their primary service) or "hydricmedia.js" (the company owns these service). The same story with "bemusic", "radiotunes" connectors and some others. That's one more case should be noted in the convention.

Added to "Connectors development" wiki page.


AddGTFStoaNetworkDataset: Improve memory management in step 1

The stop_times.txt file is often huge, and you either have to read it all into memory or write it to a query-able sql database so you can figure out which stop times belong to which trip.  The input file is not required to be ordered, so you can't make any assumptions about whether you've found adjacent stops until you've read in the entire file.  The current strategy fails for really large datasets, like De Lijn.

Had hoped pandas could solve this problem, but although it might have improved memory management a little, De Lijn still broke my computer.

I think it's time to revisit indexing and ORDER BY on the stop_times table and iterating through the cursor rather than reading in all of stop_times into a variable.  See also https://github.com/Esri/public-transit-tools/issues/84

Problem solved by making better use of sqlite functionality.  DeLijn actually finishes now (though would probably run out of memory later during schedule caching anyway).


New CI/CD docs

Describe changes
I tried to make the guide for CI/CD more actionable.
Pre-requisites
Please ensure you have done the following:

[ ] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] If my changes require changes to the dashboard, these changes are communicated/requested.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

Here's a first draft @htahir1 - I'm sure we can improve this and add more text/images, I just wanted to get this out in a better shape quickly


An unhandled exception occured.

Hello,
i have downloaded the plugin via nuget, implemented the interface and inizialized afre on create on android. In the cross platform OnStart i register, when happen i get "An unhandled exception occured." with no detail on VS log or android logcat. I have VS 2015, the last nuget plugin on Mono Android 7 Xamarin forma 2.3.3. Any ideas?
Thanks

try this version:
https://www.nuget.org/packages/Xam.Plugin.PushNotification/1.2.5-beta

i have the same problem when i call register, i am using the 1.2.5-beta

the error only give in debug mode, what could be?

only in debug mode and android, in ios no problem


bug: com.intellij.diagnostic.PluginException: ActionUpdateThread.OLD_EDT is deprecated and going to be removed soon. 'com.sourcegraph.cody.CodyActionGroup' must override getActionUpdateThread and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: com.sourcegraph.jetbrains]

Plugin version: 5.2.20247
IDE version: IU-241.9959.31
Exception: com.intellij.diagnostic.PluginException: `ActionUpdateThread.OLD_EDT` is deprecated and going to be removed soon. 'com.sourcegraph.cody.CodyActionGroup' must override `getActionUpdateThread` and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: com.sourcegraph.jetbrains]
Stacktrace:
com.intellij.diagnostic.PluginException: `ActionUpdateThread.OLD_EDT` is deprecated and going to be removed soon. 'com.sourcegraph.cody.CodyActionGroup' must override `getActionUpdateThread` and chose EDT or BGT. See ActionUpdateThread javadoc. [Plugin: com.sourcegraph.jetbrains]
	at com.intellij.diagnostic.PluginProblemReporterImpl.createPluginExceptionByClass(PluginProblemReporterImpl.java:23)
	at com.intellij.diagnostic.PluginException.createByClass(PluginException.java:90)
	at com.intellij.diagnostic.PluginException.reportDeprecatedUsage(PluginException.java:125)
	at com.intellij.openapi.actionSystem.ActionUpdateThreadAware.getActionUpdateThread(ActionUpdateThreadAware.java:21)
	at com.intellij.openapi.actionSystem.AnAction.getActionUpdateThread(AnAction.java:198)

Thanks for reporting!

It seems that we are using something that will be removed soon. However, I'm wondering why we are getting exceptions instead of deprecation warnings.

I sincerely apologise for the duplicates, these get automatically reported in my case. I am checking how to turn those off.
This is a warning as far as I am concerned, just with a stack trace for devs to fix such issues. There is no issue with the plugin's working. Also users can choose to not report or not disable the plugin.
What I have seen is Jetbrains deprecates and removes its APIs very often with its versions, so IMO plugin may require active fixing at all times.
This deprecation notice from Jetbrains might cause compatibility issues with future stable versions of Jetbrains products.

closing in favor of https://github.com/sourcegraph/jetbrains/issues/610


[Request]: Allow deletion of account via Clerk

What happened?
You use Clerk for auth which I appreciate, however, you disabled the option to allow your account deletion through the  component.
At least in EU this is not legal sadly so I'd appreciate if you'd add that option :)
thanks!!
What did you expect to happen?
Allow account deletion through clerk user account.
Version
latest
Reproduction Steps
No response
Reproduction Repository
No response
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Not sure how we missed that one 😅 I'm on it!

Account deletion should now be enabled 👍


ImageFit.none doesn't recenter when NetworkImage size changes.

You mean when a network image changes size in layout?

Do you have a test case that shows the issue?

Not sure how to make progress here without an example that shows the issue.

Going to close because we're not sure how to make progress. Please reopen if we have a repo or test case. Thanks for the feedback!


Feature/guest reservation

PR Description

Description of what is contained in pull request?
Completed the shell of Guest Check-in. Counter is working with hardcoded data, will need to add new api for it to work. List of members who are reserved array completed (Still need to update api for data to flow properly). Displayed checkbox for members staying with increment and decrement features to total beds in shelter.
Description of why this change/addition is needed?
In order for the Guest check in to be fully functional, we will need to pull in data from the api and also add data (members and total amounts) to the api. Guest should be able to make reservations to stay at the shelter and also be put on wait list.
Which user story or task is this PR tied to?

As a Guest, I can reserve a bed in the shelter using my Guest profile and log how many beds I need
How many files and/or lines were changed?
6 files, 375 lines

Type of Change

Types of Change:

[x] New feature
[x] Refactoring / Bug fix
[] Documentation update
[] Executive Change (technical decision change overwrite)

Reason for change (required):

Testing

[] Yes
[] No
[] Not needed

Completeness
Is this feature complete

[] Feature Complete
[x] Feature Incomplete
[] User story/trello card linked here

PR Requirements

[] Clean and concise commenting
[] Clean and comprehensive documentation included
[] Conforms to engineering standards and checks
[] There are no merge conflicts
[] There are no warnings
[] Corresponding documentation elsewhere has been updated to reflect these changes

Requester Questions

Add any questions, comments, or concerns you have as the person initiating the pull request for your reviewers

Other
Rubric
Template Guidelines

looks good! we worked on this PR together

looks good! we worked on this PR together


Issue 39, allow users to switch channels

fixes #39

moved docopt_cmd to it's own files
started main_menu code
users can enter into channels
users can leave channels back to main_menu

Coverage decreased (-0.4%) to 48.684% when pulling 525aed95c8c5e53a2f964c157717f4934d878b3f on issue-39 into 1e121d480d86d7afdc683c1abaeb732c2af00b18 on master.

Coverage decreased (-1.4%) to 47.742% when pulling 4b36362aaf2c3f921c68ed1f65038edc03951028 on issue-39 into 1e121d480d86d7afdc683c1abaeb732c2af00b18 on master.

Coverage increased (+6.3%) to 55.405% when pulling 808e693812aec3682eeec3d862e0fa96d7d66af5 on issue-39 into 1e121d480d86d7afdc683c1abaeb732c2af00b18 on master.


#[cfg(debug_assertions)] print!(..) should be replaced by tracing logging

https://github.com/0xPolygonZero/zk_evm/blob/02e258a58b904441ab357acb1d207739612b81cb/evm_arithmetization/src/cpu/kernel/interpreter.rs#L415-L424

This can probably be even removed instead, it's not really useful anymore (and never used)


Python Website Scrapper & Email Automation

Tasks : :desktop_computer:

Pulls Headlines from hackerNews webpage lists them with their respective links.
Distributes the links through email in automated way.

All the above tasks are completed in a single click.
Required Libraries-

requests
bs4
smtplib
email

Please Consider it for hacktoberfest

adding your project


2 tables in the same grid

Hi!
I searched on this section, but i don't find a solution for this type of problem.
I have 2 tables which are connected by the ID(TbTask is Primary and TbTask_Details is Foreign).
This is an example of the schema:

I want to display both tables in the same grid like this:

Is this possible at this moment?
I tried to use the tutorial, but is not giving me any good examples for this.
I followed the tutorial like you added the genre, but now i am stacked.
I want the insert to be done from one popup only.
This is a part of the code in the TbTaskRow which i modified without any success:
        [DisplayName("ID"), ForeignKey("dbo.TbTask_Details", "ID"), LeftJoin("g")]
        public Int32 ID
        {
            get { return (Int32)Fields.TaskId[this]; }
            set { Fields.TaskId[this] = value; }
        }
        [DisplayName("Action"),DatabaseAlias("g"), Column("Action") ]
        public String Action
        {
            get { return Fields.Action[this]; }
            set { Fields.Action[this] = value; }
        }

I am still a beginner in C#, so I am pretty sure that is something which i missed and the solution is pretty simple.
Thank you for your time!

See mapping guide for joins. Where did you get DatabaseAlias etc??
You need to show task details in a grid, you can't both. It is not supported.

Thank you for the answer. I found another way to do it, but i have a problem with insertion.
I created a view with those tables and in this way the gird will be updated with all the necessary information. The think is that the insert command can not be done as it is in the current configuration made by sergen. Sergen is trying to add files into the view directly which is not possible, because the view contain more than one table.
            public RowFields() : base("[dbo].[View]")
            {
                LocalTextPrefix = "Project.ViewTask";
            }

I studied the entire code today and i see that the database is selected from the RowFields class ctor:
            public RowFields() : base("[dbo].[TbTask_Details]")
            {
                LocalTextPrefix = "Project.ViewTask";
            }

The Code seems to work in this case, but is not good because is using only one Table and is not able to find some fields in that Database.
I want to know if there is any possibilities to select only some fields that i will add in one database and the other fields in the other database or another way to achieve that.
Serenity is a great powerful tool and i would like to use it forward. Thank you for your time!

Why you needed to declare a view? Serenity rows can do almost anything a view can do. Anyway, if you insist, you may open another dialog type, e.g. original table, for your view grid. getDialogType method is the one you set it.

Thank you for your help and patience.
I worked on it using your guidance and now it show me something like I wanted to do.
Maybe i will need your guidance again, but i hope that i will not need any help in the future.:)
You've been very helpful. Thank you again!
Have a nice day!


Get data context reactively

I've added this to my project in order to get the data context of the current template instance reactively:
ViewModel.prototype.getData = function () {
  return Blaze.getData(this.templateInstance.view)
};

I think this should be part of the package, but I wonder whether there is a smoother way of doing it.
Any thoughts?

Looks nice but wouldn't any writes to this data bypass the ViewModel event system and cause the templating to break?

I'm not sure, but would like to understand any caveats with this method. Writing to the data... The dependency that getData registers gets invalidated every time the data context is updated, like {{> mySubtemplate myData=theValueOfThisHelperMayChangeForSomeReason}}.

Let me ask you this: what use cases have you encountered for this pattern?

For instance, I have a paginator widget on my page, which is instantiated like this: {{> paginator items=mycollection.count}}

Computed properties make perfect sense for this.

Could you be more specific? Computed properties?
I'm talking about the need for any component that is configured with named template parameters to be able to reactively update when those parameters change.

computed == functions with dependencies

Yes, exactly. If one of the dependencies is a named parameter, that one needs to be retrieved reactively, like in my example. It's not enough to write this.templateInstance.data if the data changes.

I found what in my view is the more canonical way:
ViewModel.prototype.getData = function () {
  return this.templateInstance.view.templateInstance().data;
};

It seems clear that ViewModel's this.templateInstance should have been equal to the view's templateInstance() method from the beginning, so that the data retrieved is reactive.

@dalgard Wow. I've spent almost whole day today to get reactive data context...
Template.parentData() sometimes throws 'There is no current view'-error.
this.templateInstance.view.templateInstance().data works nicely.
I'm interested in some helper for this from-the-box too.


Library Creates Tables but Provides no (Documented) Way to Remove Them

I saw a reference to a dbCleanup function in one of the issues (I guess it wasn't working in SQLite?), but no other mention of what it is or how it works anywhere.
Beyond that, I also can't find any reference anywhere to a command to clear or remove the tables this library creates.
It's pretty standard practice in Knex to use Knex migrations to change the database, so it seems like this library really should have a way to generate a migration to add the table, rather than just adding them automatically within the app itself.
But separate from that, there absolutely should be some way to "tear down" the DB work done by the library (and that way should be documented).

I think I wrote this before knex had migrations. A MR would be welcome!


[release/1.2] cherry-pick: Bump ttrpc

cherry-pick #3278 for release/1.2

Codecov Report

Merging #3316 into release/1.2 will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release/1.2    #3316   +/-   ##
============================================
  Coverage        43.74%   43.74%
============================================
  Files              101      101
  Lines            10761    10761
============================================
  Hits              4707     4707
  Misses            5319     5319
  Partials           735      735

Flag
Coverage Δ

#linux
47.38% <ø> (ø)
:arrow_up:

#windows
40.84% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 367c672...d2d36e5. Read the comment docs.

LGTM


[1.x] Allow ignoring route registration

Similiar to other first pary packages i thought it would be great to allow ignoring route registration, in order to be able to register the routes yourself. A more practical use case might be if you use something like tenancy for laravel where registring the routes only on central domains, etc.. can be beneficial and easir to be created that way

Has conflicts.

Has conflicts.

Resolved, there seems to be an issue with the static code analysis from the 1.x branch right now

rebased this one against a fix version


Adding islington college

Adding Islington College, where I am currently doing my undergraduate Networking and IT Security degree!
Islington College is located at Kamal Marg, Kamal Pokhari, Kathmandu, Nepal and is affiliated to London Metropolitan University, London.
The official domain of the institution is islington.edu.np, and the website homepage is located at https://islington.edu.np/.

You can validate that this is the official domain of the institution by visiting the webpage containing list of International collaborative partners in the London Met website: https://www.londonmet.ac.uk/courses/our-collaborative-partners/
I am looking forward to seeing this Pull Request being merged.

@pr0d33p Pull request merged. Thank you!


--install fails due to issue with kisielk/errcheck dependency

I've opened an issue on the errcheck repo: https://github.com/kisielk/errcheck/issues/83

... and it's totally my fault. Sorry about the noise!

👍


Série C teams + Gauchão state championship

Great. Thanks. Obrigado. Cheers.
PS: If you're interested I send you a github team invite so you can commit without pull requests. Cheers.

I think it will be great, please send me the team invite.


Passing a JSON-Schema in create_json_agent

Issue you'd like to raise.
It seems the default prompts do not support passing a JSON-Schema in create_json_agent to provide description to each field and nested fields. Wanted to understand if this hasn't been deemed necessary?
In addition, if there is a list of JSON objects, with some fields in each JSON object having nested objects or arrays, is the JSON agent the right fit, or should one try to use custom agents (perhaps Pandas) for such cases?
Suggestion:
No response

Issue you'd like to raise.
It seems the default prompts do not support passing a JSON-Schema in create_json_agent to provide description to each field and nested fields. Wanted to understand if this hasn't been deemed necessary?
In addition, if there is a list of JSON objects, with some fields in each JSON object having nested objects or arrays, is the JSON agent the right fit, or should one try to use custom agents (perhaps Pandas) for such cases?
Suggestion:
No response

Im also interested in this, did you find any answers?


Phaser3.60-beta22: What's your experience of using ESM? I see no difference

Phaser 3.60 beta 22 is installed
production mode in webpack.
es2015 for "module" in tsconfig.ts
Some simple code to test:

import { AUTO, Scene, Game } from "phaser";
class Test extends Scene {
    constructor () { super(); }
    public create () {
        this.add.text(10, 10, 'Welcome to Phaser ESM');
    }
}
const config = {
    type: AUTO,
    width: 800,
    height: 600,
    parent: 'phaser-example',
    scene: [ Test ]
};
const game = new Game(config);

Webpack's build log:
asset index.js 13 MiB [emitted] [minimized] [big] (name: main) 1 related asset
runtime modules 663 bytes 3 modules
cacheable modules 7 MiB
  ./src/index.ts 438 bytes [built] [code generated]
  ../../node_modules/phaser/dist/phaser.js 7 MiB [built] [code generated]

WARNING in asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
This can impact web performance.
Assets:
  index.js (13 MiB)

WARNING in entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.
Entrypoints:
  main (13 MiB)
      index.js

WARNING in webpack performance recommendations:
You can limit the size of your bundles by using import() or require.ensure to lazy load some parts of your application.
For more info visit https://webpack.js.org/guides/code-splitting/

webpack 5.72.0 compiled with 3 warnings in 42477 ms

And I get a js bundle of size: 1.1mb for such a simple code. Looks it gets all the Phaser's functions without properly making the shaking tree feature, it just pushes the whole source code. But what's the profit in using ESM then? 😀

I never claimed the ESM version would tree shake. It won't, and it can't. That's nothing to do with the module system being used and everything to do with the structure of the modules. It's also why Phaser 4 exists, as that was its primary goal.
However, you cannot import Phaser as a module directly in a web page using the CJS version. Browsers simply don't support that. So by producing an ESM version, you can now simply import it directly and start coding immediately without any packager or bundler required at all, which is significantly useful for testing things fast, producing Codepens, etc.


Minor spelling error on death save - successes

Successes is spelled sucesses

Hi @elikennedy, thanks for pointing that out, I'll get that corrected shortly.


fix: bunotel add set attributes to query metrics

Currently, even if we set attributes with WithAttributes, they are not set for query metrics, only pool/connections metrics. This PR fixes this issue.

Thanks


Lambda function creation failed

When I run the command to deploy I get this error
Specified  account's UnreservedConcurrentExecution below its  minimum value of [50].
How can I solve it please ?

I have the same issue, did you find a workaround?


[Docker] create a Docker File

The following can be used as inspiration.
# ~~ Generated by projen. To modify, edit .projenrc.js and run "npx projen".
FROM public.ecr.aws/docker/library/node:20.9.0-slim as builder
ARG AWS_ACCESS_KEY_ID
ARG AWS_SECRET_ACCESS_KEY
ARG AWS_SESSION_TOKEN
ARG AWS_REGION=ap-southeast-2

RUN apt-get update && \
    apt-get install -y python3-launchpadlib software-properties-common && \
    add-apt-repository -y ppa:deadsnakes/ppa && \
    apt-get update && \
    apt-get install -y python3 python3-pip

RUN python3 -V &&  pip3 --version && \
    pip3 install awscli --break-system-packages

WORKDIR /app
COPY . .
RUN yarn codeartifact:login
RUN yarn install --frozen-lockfile && yarn build

FROM public.ecr.aws/docker/library/node:20.9.0-slim as runner
COPY --from=public.ecr.aws/awsguru/aws-lambda-adapter:0.8.1 /lambda-adapter /opt/extensions/lambda-adapter
ENV PORT=3000 NODE_ENV=production
ENV AWS_LWA_ENABLE_COMPRESSION=true
ENV AWS_LWA_READINESS_CHECK_PROTOCOL=tcp
WORKDIR /app
COPY --from=builder /app/package.json ./package.json
COPY --from=builder /app/public ./public
COPY --from=builder /app/.next/standalone ./
COPY --from=builder /app/.next/static ./.next/static
COPY --from=builder /app/run.sh ./run.sh
RUN ln -s /tmp/cache ./.next/cache

RUN chmod 777 ./run.sh
EXPOSE 3000
CMD exec ./run.sh

Here is the NextJS documentation for Docker.  https://nextjs.org/docs/app/building-your-application/deploying#docker-image

Hi @nabarunchakma , I’ve added the Dockerfile in the add-docker branch.
You can find the  Dockerfile here. Let me know if any changes are required before merge.

add-docker branch has been successfully merged with master branch. @nabarunchakma


Update schema.js

MySQL 5.7 support JSON column type, and Sails too: http://sailsjs.org/documentation/concepts/models-and-orm/attributes

Published!


Rope is missing the tilePosition property

The Rope class uses acts upon a tilePosition property, probably for the advertised 'texture scrolling', but it isn't defined, so calling Rope.reset() which references it will crash.
Is this a feature awaiting implementation or pending removal?
If it's to stay, then probably best to hide all references to tilePosition in the interim to stop crashes.

Hmm it was used in several places. I have removed it for now, the demos still work, so I'll leave it "left out" for the time being.

Correct me if I'm wrong, but that property was only there because the Rope class was created from the Tilesprite class template

Quite possibly, although it did mention it in the jsdocs. Right now I don't have time to go back through earlier builds to figure out why, so am happy to leave it as-is (as it works and doesn't break now).


Create attachment_html_emoji_map.yml

Description
Detects inbound messages containing HTML attachments with an unusually high number of emojis in a list, sent from untrusted or suspicious senders who lack an established sending history or have previous malicious behavior.
Associated samples

https://platform.sublime.security/messages/e5b6be9c54d5177afc4aa3f7f1e572a7229b9e3001c751d2494065aff959970e

No hits in test rules.


Asset Loading - loading multiple times

Would it make sense for the kontra.assets.loadImage function to resolve with the existing image if it has already been loaded into the assets? Same with the other load functions.

Yeah, the way I'm setting up my games is it has a reset function, which loads the assets, and in the return creates new sprites. I'd like to let this function be smart enough to use existing image assets if they are already in memory, rather than loading new Images every time.


Mailchimp Environmental Variables

We're going to be deploying the mobile app splash page and we need to make sure these two variables are populated. I have the list ID and assume the User ID is our API key.
<form class="email-signup" action="//dosomething.us4.list-manage.com/subscribe/post?u={{ site.mailchimp.user-id }}&id={{ site.mailchimp.list-id }}" method="post" novalidate>

@lkpttn You can just hard-code these in your splash page. No need for environment variables!

@DFurnes hard-code BAD!! :no_good: It's better to have as an env variable so it's not in a public repo.

They're just going to be output into a plain HTML tag. There's nothing "secret" about them.

@DFurnes is :100: smart on this. Info isn't any sort of disclosure. Talked to @lkpttn as well.
@angaither I think it's best if this lives in Phoenix repo, but can be persuaded to put it elsewhere. Your thoughts on best location? (We can still put it as resolveable to ds.org without it being part of Phoenix).

it's just a one page app, right? doesn't @lkpttn have it living elsewhere currently?
@sheyd I would say if we don't need the code to live in phoenix, since it has nothing to do with the main drupal app, let's keep it outta here.

I've had really good experiences using Github pages for static sites. That's what we're using for tech.dosomething.org and forge.dosomething.org.

@angaither Roger, I'll put it elsewhere. @DFurnes I'd prefer to track log hits to this, etc. (and have a little more control over downtime). I'll have it served up out of another repo, and post where here.

:github: has no downtime.

yeah it does lol, just last week (or was it this week)  me & tong noticed for a few minutes

Can we just make a json file on amazon S3 and make a public URL for it? You could easily layer Amazon's CDN on that as well

I looked around at :allthethings: that would work and make this easiest. GitHub is our best option, but with a caveat. @lkpttn Let's discuss in the morning.

@lkpttn I think we're all setup and on our way to completion on this. Infrastructure is setup, and you and Dee are working out the form details.


[FLINK-2125][streaming] Delimiter change from char to string

I tried to change based on the previous comments. I hope it is sufficient enough.

Hi,
Generally this is a good idea to receive all buffer once instead get a char every time.
And you can see my PR:https://github.com/apache/flink/pull/992.
There will be some changes in SocketTextStreamFunctionTest.java.
And These changes is just "maybe".
I am not sure since that PR has not been merged yet.
Just provide you an info.

I changed unit tests as HuangWHWHW suggested, added a simple test server socket and refactored test methods.

I changed static port and added assertions for full content

Hi, sorry for reply late.
The file SocketTextStreamFunction.java has been changed recently.
Can you update the code and squash your commits into one for the CI re-running?
Since there was an issue in travis before.
And generally this is good from my side.
Maybe you need to @somecommiter to take a look.

@ogokal, please ping me when you have rebased and updated the PR and I will have a look.
Thanks

Hi @ogokal, do you plan to update this PR? If not, could you please close it? Thank you.

OK, I will close this PR for now.
Please reopen if you would like to continue to work on this issue.
Thank you.


APIError: No media found

Getting a No media found error from Openrouter when I use the google/gemini-pro-vision model to query an image. See pictures below(Also, kindly note that I have set payment for Openrouter and when I query the image using Bing it works):

I used the !change google/gemini-pro-vision command to change the model

Currently no model from openrouter support images by our side, only bing.

Ok, thanks for the assistance :)


Do not overwrite machine state with nil data

How to categorize this PR?

/area control-plane-migration
/kind bug
What this PR does / why we need it:
This PR fixes an issue which was causing the machine-state to be overwritten with nil data if the migrate phase of control plane migration errored and was retried after the MachineDeployment, MachineSet and Machine resources were deleted.
The issue would occur because the computed machineState is empty if there are no machine objects. This would then result in a nil machineStateJSONCompressed  and this nil data is added as a machine-state entry in the gardener data list here. The nil data then overwrites the existing machine-state data in the ShootState when gardenerData.Upsert(...) is called.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/cc @rfranzke
Release note:

Fixed an issue that would cause the entry for the `machine-state` in the `ShootState` to be overwritten with nil data during control plane migration, if the `migrate` phase errored and was retried after the `MachineDeployment`, `MachineSet` and `Machine` objects were deleted, which would result in the Shoot's nodes to be recreated during Control Plane Migration.

/assign

/cherry-pick release-v1.106

/cherry-pick release-v1.105

/cherry-pick release-v1.104


Irby/create dotnet core implementation

Issue: N/A
Description
Added in a .NET Core implementation of the project so it could run on Linux / macOS. I noticed that when running the unit tests I was getting an SSL issue despite the ServicePointManager.ServerCertificateValidationCallback assignment. What I noticed is that the options passed into the clients needed to have their HttpMessageHandler written with the ServerCertificateValidationCallback assignment. I'm not sure why this is necessary, it's perhaps because of how the HTTP client interacts with Docker in .NET Core.
Entry point
dotnet-core/lowkey-vault-example-dotnet-core.sln
Checklists

[x] I have rebased my branch
[x] My commit message is meaningful
[x] The changes are focusing on the issue at hand
[x] I have maintained or increased test coverage

Notes
N/A

Made the suggested changes, and re-worked the .NET Core unit test code to make it cleaner.

Made the suggested changes, and re-worked the .NET Core unit test code to make it cleaner.

Thank you! The workflow has a small issue, but that is the only outstanding change we need.