Post core-esm migration - Migrate test-utils

This issue is to track the work involved in migrating test-utils.

chai globals/namespaces are involved, which need to be migrated to vitest
mockInstrumenter may need to be modified
changes similar to @azure/core-tracing maybe needed (cc @jeremymeng )
or make a new major version like the recorder v3 -> v4 to capture the differences of cjs vs esm

@maorleger
Standalone, the tracing test at template fails with the following..

from the PR #29016

I won't have bandwidth to look at this for a while, so if there's urgency Matt said he might take a stab at it next week. Worse case scenario maybe we can switch that test to a mocking approach?
The only thing I can think of is that dual-package hazard issue combined with side-by-side versions of core-tracing 🤷
I don't know about the rest of the items on the list

@mpodwysocki added test-utils-vitest which addresses this issue. An example for using it can be found in https://github.com/Azure/azure-sdk-for-js/pull/30760


versions: Update cri-containerd  yaml

Use only one version to install cri-contaienrd

version could be a containerd version or a commit of the cri
repository.

Fixes: #1464

@jcvenegas nudge.

@jcvenegas - Travis is complaining as you need to add a has to your fixes:
Fixes: #1464

/test

/test

@devimc @jodh-intel branch updated.


Fixed #32786 -- Moved subquery ordering clearing optimization to the _in lookup.

Ticket: https://code.djangoproject.com/ticket/32786

buildbot, test on oracle.

Looks like we need to add the clearing to RelatedIn somehow.

I assume the failure is due to RelatedIn's usage of SubqueryConstraint
https://github.com/django/django/blob/7e51893911237dfca9294e3ca12163ff813fb656/django/db/models/fields/related_lookups.py#L82-L86

@hannseman any thoughts about doing the ordering clearing in as_subquery_condition instead since that's where the EXISTS is generated?
https://github.com/django/django/blob/7e51893911237dfca9294e3ca12163ff813fb656/django/db/models/sql/compiler.py#L1227-L1238

@hannseman any thoughts about doing the ordering clearing in as_subquery_condition instead since that's where the EXISTS is generated?

as_subquery_condition is overridden for MySQL so we would need to repeat the clearing for that backend. Another alternative could be to do it in SubqueryConstraint?
We'd also probably want to move the whole is_sliced, distinct_fields, select_for_update checks to a utility method on Query as you suggested in the ticket.

as_subquery_condition is overridden for MySQL so we would need to repeat the clearing for that backend. Another alternative could be to do it in SubqueryConstraint?

Both work for me.

Maybe Query.can_clear_ordering or Query.needs_ordering

can_clear_ordering seems like a better candidate due to its symmetry with clear_ordering. Another alternative would be to change the signature of clear_ordering to (force=False, force_empty=None) and change its implementation to
def clear_ordering(self, force=False, clear_default=True):
    if not force and (self.is_sliced or self.distinct_fields or self.select_for_update):
        return
    self.order_by = ()
    self.extra_order_by = ()
    if clear_default:
        self.default_ordering = False

And change all clear_ordering calls accordingly.

Looked at the other Oracle test failures and they appear to be caused by the ordering not being cleared for other lookups than __in.
For example:
order_with_respect_to.tests.OrderWithRespectToBaseTests.test_previous_and_next_in_order fails due to it using a subquery along with the lt/gt lookups.
SELECT "order_with_respect_to_answer"."id",
       "order_with_respect_to_answer"."text",
       "order_with_respect_to_answer"."question_id",
       "order_with_respect_to_answer"."_order"
FROM "order_with_respect_to_answer"
WHERE ("order_with_respect_to_answer"."question_id" = 1 AND
       "order_with_respect_to_answer"."_order" > (SELECT U0."_order"
                                                  FROM "order_with_respect_to_answer" U0
                                                  WHERE U0."id" = 1
                                                  ORDER BY U0."_order" ASC))
ORDER BY "order_with_respect_to_answer"."_order" ASC
LIMIT 1

expressions.tests.FTimeDeltaTests.test_date_subquery_subtraction fails due to the subquery being annotated and then used on the lhs along with a direct value.
SELECT (1) AS "a"
FROM "expressions_ExPeRiMeNt"
WHERE (interval '1 day' * ((SELECT U0."completed"
                            FROM "expressions_ExPeRiMeNt" U0
                            WHERE U0."id" = ("expressions_ExPeRiMeNt"."id")
                            ORDER BY U0."name" ASC) -
                           "expressions_ExPeRiMeNt"."completed")) =
      '0 days 0.000000 seconds'::interval
LIMIT 1

The real reason for the crashes is Oracle not accepting ORDER BY clauses (ORA-00907: missing right parenthesis) in sub queries while other backends most likely ignores these clauses
Tricky stuff, not really sure how to proceed. Feels like we can't remove the ordering since it will crash queries on Oracle against models with a defined Model.ordering with no way around it other than forcing them to remove the model defined ordering. One workaround to make it work for array producing subqueries could be to add some attribute on Query which could be used along with the other order-clearing conditions, Query.ordered_subquery or something like that..

Thanks for digging into that @hannseman.
Another alternative could be add a supports_ordered_subquery feature to backends and have Query.as_sql branch which already special case self.subquery do a self.clear_ordering(clear_default=True) if not connection.features.supports_ordered_subquery
https://github.com/django/django/blob/0d67481a6664a1e66d875eef59b96ed489060601/django/db/models/sql/query.py#L1067-L1068

Another alternative could be add a supports_ordered_subquery feature to backends and have Query.as_sql branch which already special case self.subquery do a self.clear_ordering(clear_default=True) if not connection.features.supports_ordered_subquery

That feels like a good workaround. I'll give it a shot! Thanks!

buildbot, test on oracle.

buildbot, test on oracle.

@hannseman thanks for the patch. At this point I think the only thing missing is additional tests proving that ordering is not cleared on engine that supports it.
I guess declaring an AggregateSubquery in postgres_tests solely for tests purposes should do for now?

Thanks both :rocket: Great work :star:

buildbot, test on oracle.

@charettes @felixxm thanks for the reviews!


BIGTOP-2866:Fix rmr and people directory error

The below error is issued while running the spark smoke
tests on AArch64 architecture.  The patch will fix the issues.
...
....
org.apache.bigtop.itest.spark.TestSpark STANDARD_OUT
Failed command: hdfs dfs -rmr people* examples
error code: 1
stdout: [Deleted examples]
stderr: [rmr: DEPRECATED: Please use 'rm -r' instead.,
rmr: `people*': No such file or directory,
...
.....
Signed-off-by: Naresh Bhat<EMAIL_ADDRESS>

That must break the test everywhere, ie independent of a platform. +1


Serve static images from Docker setup

As per the Slack discussion, the metadata_url should correctly resolve to the absolute URL of the service in a Docker environment (it is otherwise pointing to AWS, but that doesn't work with our Docker setup).

c514a7d21dd326b072a522cc8619c3bd54f7f763


Treat . as a trivial atom too

Not handling the OCaml in https://github.com/facebook/hhvm/commit/7b8687d28d49e1d7c13d5fe59496d0af86672cf0 correctly yet.


Why not use this data format？

Example:
filename \t 23 45 67 88 99 3
This makes it easy to handle special characters.

You can try it, or you can use escape character to handle special characters.


Generate snap packages

It would be useful to have a generator for the snapcraft package/container format. It's a basic yaml syntax. We could then register and publish automatically to the public snap package registry.
See: https://github.com/prometheus/node_exporter/pull/1033

I've changed my mind on this. Snap doesn't have enough useful adoption to be worth the overhead of having another system.


add some logic to frontend to show better instructions

Adds quick and dirty instruction formatting to the route frontend to visualise walk/transfer segments. Will be replaced by a functional frontend at some point.

Codecov Report

Merging #170 into master will not change coverage.
The diff coverage is n/a.


conversations_list does not return newly created public channels

Reproducible in:
The Slack SDK version
slack-sdk==3.13.0
Python runtime version
Python 3.9.7+
OS info
Linux gpad 5.15.12-arch1-1 #1 SMP PREEMPT Wed, 29 Dec 2021 12:04:56 +0000 x86_64 GNU/Linux
Steps to reproduce:

Create an app add it to the workspace with the following scope

Fetch the channels using the below code

client = WebClient(token=bot_token)
        client.retry_handlers.append(RateLimitErrorRetryHandler(max_retry_count=3))
        response = client.conversations_list(
            exclude_archived=True,
            limit=200,
            types="public_channel, private_channel, mpim, im",
        )

Observe that all the newly(1 hr ago) created channels are missing

Expected result:
Should return all the channels
Actual result:
Only returns some public channels

Hi @gagantrivedi! I went ahead and spent some time trying to reproduce the issue you've described above. The only case in which channels returned were "missing" were those channels that were 1.) private and 2.) the app hadn't been added to.
Of the ones that you notice are missing, do they all share the two characteristics listed above?

@gagantrivedi
To add to the things Alissa mentioned, I guess the Slack API pagination might confuse you.
Your newly created channel can be included in a following page of the API response pagination. If response_metadata.next_cursor in an API response is not empty, conversations.list API has more to return. See also: https://github.com/slackapi/python-slack-sdk/issues/1074#issuecomment-888706495
This Python SDK offers a handy way to iterate with the Slack API pagination. Checking this test code should be helpful to know how to use it.

Yeah, it was related to pagination. Thanks, @seratch @misscoded


Search Case Insensitive

Love this project! For the life of me I cannot figure out how to search case insensitive. It will only search case sensitive. Many thanks in advance.

I have found the solution.
I removed the iteration for "each" word search. See below
$searchinput = (string)$this->input('search')['value'];
$length = strlen($searchinput);
$allcolumns = $this->input('columns');
    if ($searchinput == '' || $length < 2)
    {
        return null;
    }

    $search = [];
    $searchinput = preg_replace("/[^\wá-žÁ-Ž]+/", " ", $searchinput);
    $new_search = strtoupper($searchinput);

    $lookfor = [];
    foreach ($this->columns as $key => $column) {

        if(array_key_exists($key,$allcolumns)){

            if ($allcolumns[ $key ]['searchable'] == 'true') {
                $lookfor[] = "UPPER(".$column . ") LIKE  '%". $new_search . "%'  ";
            }
        }
    }
    $search[] = "(" . implode(" OR ", $lookfor) . ")";

I know I need to escape...But this gives the general direction.

And I am using a DB2, ibmi

As I thought, it is something related to DB. There is an article in this post about db2 which is you may have already be read.
it says:

The short answer is that DB2 for Linux®, UNIX®, and Windows® Version 7.2 is case sensitive: BLAIR is not the same as Blair.

Yep. Thanks for the follow up...great package. Love it.


Milestones

Implement Main Website
Reach 10 athletes
Launch Website Officially
Reach 100 athletes
Implement Sponsorship Revenue model

if reached 100 athlete make it so that the application process creates a github pr and new athletes are discussed there and this could be a way of verifying proofs with the community

after 100 athletes reach out to journalist who has interviewed athletes before to suggest the media contact list on the website they may be interested in that

how to reach the first 10 athletes?
getting the first 10 athletes will be one-on one stuff

hopefully some of the RO athletes will be willing to join if asked
get people who work with me on this to join to fill the spaces
crypto pirate promised he and his gf will join

after reached 10 athletes

add podcast episode link to athlete dialog (json) and a podcast icon into the row itself next to athlete's name, next to the personal link. these little links next to athlete names could serve as the badges?


Wireguard Peer Generator: Field "Allowed IPs" always errors with "A value is required."

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[ ] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
When using the WireGuard Peer Generator I am unable to use "Store and generate next" because the field "Allowed IPs" always shows the error "A value is required." no matter what I enter. If the field requires a special value / notation the error message should be changed.

To Reproduce
Steps to reproduce the behavior:

Go to 'VPN -> WireGuard -> Peer Generator'
type network (e.g. <IP_ADDRESS>/24) in 'Allowed IPs'
Click on the check button 'Store and generate next'
Allowed IPs gets a red box and the error message "A value is required."

Expected behavior
Store and generate next
Describe alternatives you considered
No way found
Screenshots
See above
Relevant log files
This shows in log files everytime trying to click 'Store and generate next':
2024-05-19T19:53:11	Error	config	[OPNsense\Wireguard\Server:servers.server.a6d636b1-XXX.peers] Choose a peer.{01cd3b71-YYY}
Additional context
First time using WireGuard. Don't know if this error happens in prior versions, too.
Environment
OPNsense 24.1.7-amd64

can't reproduce on a clean install, but looking at the message in the log, the spurious message would better be omitted as referential integrity checks are not possible yet at that point.  https://github.com/opnsense/core/commit/611ee6a4a23f0a297d23efe544340f909e9c1199 silences the message.
It might be a good idea to do a firmware health check and perhaps check any browser plugins installed (if any).

I got this on two separate boxes with two different clients. Both installations where updated from an older version to 24.1.7
Anyway, I tried the peer generator in a chromium without any plugins (at least I haven't installed any ;)) with the same result:

BTW: the initial view of the peer generator page looks like this:

As you can see, the Allowed IPs field is already not shown as it should, right?
I will look into the firmware health check and come back with the results. Thanks so far :)

No errors in Health Check:
***GOT REQUEST TO AUDIT HEALTH***
Currently running OPNsense 24.1.7 at Mon May 20 20:54:03 CEST 2024
>>> Root file system: /dev/gpt/rootfs
>>> Check installed kernel version
Version 24.1.5 is correct.
>>> Check for missing or altered kernel files
No problems detected.
>>> Check installed base version
Version 24.1.5 is correct.
>>> Check for missing or altered base files
No problems detected.
>>> Check installed repositories
OPNsense
>>> Check installed plugins
os-ddclient 1.21_2
os-squid 1.0_1
os-zabbix64-agent 1.13_2
os-zabbix64-proxy 1.10_1
>>> Check locked packages
No locks found.
>>> Check for missing package dependencies
Checking all packages: .......... done
>>> Check for missing or altered package files
Checking all packages: .......... done
>>> Check for core packages consistency
Core package "opnsense" has 68 dependencies to check.
Checking packages: ..................................................................... done
***DONE***

Checked all request on reloading page (ignoring cache), but all files are getting loaded with HTTP 200. Two issues regarding "img-src self" in the console but I think don't matter here.

can't reproduce on a clean install, [...]

Did a clean install of 24.1 by myself. As peer generator is not available I updated to 24.1.7 and the result is:

Double checked this with chromium. Same result.

On a second server I discovered that Proxy Auto Configuration is not working anymore. [...]

This is broken because of the commit https://github.com/opnsense/core/commit/67f6aeed2b40e1336c5c883b76486b2b41d4a17b which changed the returning object class of the helper function. I don't know if this is related to the WireGuard Peer Generator issue described here. So I will create a new issue on opnsense/plugins for squid proxy.

The error you see is when the instance has no tunnel address set in the instance. We're working on a fix.

right error, wrong field indeed. https://github.com/opnsense/core/commit/4e161348982e7b730505fa4485129128974a5119 should fix this


Performance on RLBench is quite different with proposed results

Hi,
Thank you for the kind introduction for your code!
By the way, I've reproduced 3D Diffuser Actor on RLBench, obtaining slightly different performance with the results in the paper.
Well, it could infer well most of the tasks included here, but there were some performance drop in some tasks (ex. Place cups, Place wine at rack location, Stack blocks, Stack cups).
Here is my reproduction results below.

Could you please explain why the model's performance is not as expected?

Hi,
Could you illustrate what are the setup you used to reproduce the results?  We've been testing the model multiple times and didn't find the performance drop.

@twke18 Hello, I would like to know how to obtain the above data and which script needs to be run.

@twke18
Hi,
Thank you very much for your contribution to this work.
I encountered similar problem. I’ve been trying to reproduce the results from your paper, but noticed that the accuracy from my inference is lower than reported. Could you clarify if there are any specific settings or steps I might have missed during the inference process?


fix: revert double-spends releases used outputs

What happened?
a. Transaction T is in an unconfirmed transaction in DB and bitcoind's mempool.
b. Block B containing Tds, a double spend of T, enters bitcoind.
c. Transaction T is being dropped from bitcoind.
d. Process 'SCANNER' - starts reverting transactions which are unconfirmed in DB, but no longer in bitcoind's mempool, including transaction T.
Here, it reaches revert_vin() where it sees the outputs in the utxos collection, which were used as the inputs of transaction T - as {usedTxid: T.txid}.
Thus, it prepared a bulk update where it sets those outputs to have {used: false, usedTxid: null, usedBlockheight: null, lastUsedTxid: T.txid}.
e. Process 'FIXER' - starts "fixing" transaction Tds from block B.
Here, it reaches add_remove_to_bulk() as part of parse_vin(), and sees the outputs in the utxos collection, which were used as the inputs of transaction T - as {usedTxid: T.txid}.
So, it changes them to have {usedTxid: Tds.txid, usedBlockheight: B.height, lastUsedTxid: T.txid, used: true}.
f. Process 'SCANNER' finishes his bulk updates after process FIXER has finished his bulk updates - leaving the outputs as {used: false}.
What should have happened?
Process SCANNER should not be able to revert a utxo from used: true to used: false if usedTxid !== T.txid at the moment of the update.
Meaning, there should be an atomic update ensuring no race condition when coming to change the output usage properties.
How this is solved?
The bulk update to set used: false in revert_vin() should consider another condition usedTxid: T.txid, which will be atomic at the document level - either it succeeds executing before FIXER so eventually the utxo is kept as used: true, or it fails executing after FIXER so the utxo is not moved to used: false at the first place.

Coverage increased (+0.06%) to 5.7% when pulling d6132466f713542dd8b73988daad0e1ea2032449 on fix_revert_double_spend into 217a79f85890c36058ff50fd19feb054b6a956d2 on master.

Coverage increased (+0.05%) to 5.694% when pulling 99fbeb71df583495db401a2dd4a7d016023ba41c on fix_revert_double_spend into 217a79f85890c36058ff50fd19feb054b6a956d2 on master.

Coverage increased (+0.04%) to 5.683% when pulling 12300c9c068ef977fd3f837db8bba88f8564f8c9 on fix_revert_double_spend into 217a79f85890c36058ff50fd19feb054b6a956d2 on master.

Coverage increased (+0.04%) to 5.683% when pulling 68e09872be0cc185da183c707a6a9cad85d1c310 on fix_revert_double_spend into 217a79f85890c36058ff50fd19feb054b6a956d2 on master.

Coverage increased (+0.04%) to 5.683% when pulling 224f9e3676a0508a47f3e764792bcea21482706b on fix_revert_double_spend into 217a79f85890c36058ff50fd19feb054b6a956d2 on master.


Add option to sort fields.List when presented with a set

Currently the code produces the following:
https://github.com/flask-restful/flask-restful/blob/f6921bbcc3a392cd92a1381357b5efa069cc55ea/flask_restful/fields.py#L181-L183
which has undefined ordering

Proposal:

introduce a sort keyword-only argument to fields.List, defaulting to False
(note, it must default to False even for sets as you could potentially be marshalling unsortable data)

Workaround
class SortedList(fields.List):
    def format(self, value):
        return super(SortedList, self).format(sorted(value))

This is a pretty niche use case that is better supported by the custom solution you've come up with.


Fix URL parse pwd failed when the pwd contains '#'

What is the purpose of the change
close https://github.com/apache/dubbo/issues/11422
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

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

Comparison is base (26eac84) 70.38% compared to head (259d482) 70.40%.
Report is 5 commits behind head on 3.2.

Additional details and impacted files
@@            Coverage Diff             @@
##              3.2   #13631      +/-   ##
==========================================
+ Coverage   70.38%   70.40%   +0.02%
==========================================
  Files        1606     1606
  Lines       69999    70002       +3
  Branches    10098    10099       +1
==========================================
+ Hits        49268    49287      +19
+ Misses      16103    16085      -18
- Partials     4628     4630       +2

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


Get error with func column of matrix

Please help fix the error at  A.column(1) when building this
let A = Matrix([
[1, 2],
[3, 4]
])
var colValueZero = A.column(0)
print(colValueZero)
var colValueOne = A.column(1)
print(colValueOne)

Try now, let me know if that works.

It works perfectly now, thank you so much!


codemirror broken when div is hidden or placed in a bootsrap tab

When you show the div parent for a codemirror or set a tab active. then codemirror show as plain textarea until you click it.

This is by design. You have to call the refresh method when you show the editor, so that it can display itself.


when will you release the code?

Really expect your work

expect your work TOO!!

expect the training code!


how to access each constraint after mas_make called

I made constraints using mas_make function like
make.left.top.right.bottom.equalTo(self.view);
but not sure how to access topConstraint.
Please help, thanks

If you want to hold onto a reference to one particular constraint you will need to seperate your constraints
self.topConstraint = make.top.equalTo(self.view);
make.left.right.bottom.equalTo(self.view);

please have a look at https://github.com/SnapKit/Masonry#1-references

thanks

I need to get NSLayoutConstraint Object


ValueError: Booster.get_score() results in empty.  This maybe caused by having all trees as decision dumps.

param = {
    "max_depth": 5,
    "eta": 1,
    "objective": "reg:squaredlogerror",
}
num_rounds = 1000
evallist = [(dval, "val"), (dtrain, "train")]

bst = xgb.train(
    param, dtrain, num_boost_round=num_rounds, evals=evallist, early_stopping_rounds=10
)

xgb.plot_importance(bst)

This problem only arises when I set objective to "reg:squaredlogerror". This error doesn't appear when using "reg:squarederror".

See https://discuss.xgboost.ai/t/reg-squaredlogerror-causes-empty-get-score/1987/2


Convert to Portable library

for Windows RT, Windows Phone and Desktop support

Portable version (at least when targetting Win RT) of HtmlAgilityPack does not have a XPath feature so it appears that it's currently to much work to implement that... :(


RISC-V image for beta channel

Now dart has experimental support for RISC-V in beta channel. Having the RISC-V container image for beta channel would be nice.

It looks like debian:bullseye-slim doesn't yet have OS/ARCH support for linux/riscv64 so there's an upstream dependency issue to be dealt with (and although Debian can be run on RISC-V boards/emulators the architecture isn't yet listed as one of their official images).

It also turns out that RISC-V shouldn't have appeared in the beta channel - https://github.com/dart-lang/sdk/issues/50548#issuecomment-1326763431
For my own efforts I think I'm going to hold off until it does make it to beta.

As I noted above it's been in 'sid' (unstable) for a little while now, but the Dockerfiles for stable and beta start FROM debian:bullseye-slim, which is the (now old) stable release (Debian 11).
Debian have announced riscv64 is now an official architecture, but that came after the release of Debian 12 "Bookworm" as the latest stable.
If things run their normal course this means that we'll be waiting until Debian 14 "Forky" for RISC-V to make it into a stable release as a supported architecture, and that will be years away (~summer of 2027). I'm hopeful though that we won't have to wait that long, and that we'll start seeing RISC-V show up in Debian 13 "Trixie" and maybe even Debian 12.
As a point of comparison, Ubuntu already have a bunch of 23.04 "Lunar Lobster" releases available for the popular RISC-V dev boards (and QEMU) - Download Ubuntu for RISC-V Platforms, which hints at Ubuntu on RISC-V being ready for the 24.04 LTS release next year. Though there aren't yet any linux/riscv64 official Ubuntu images on Docker Hub.
SPECULATION what I think will happen here is that Ubuntu will fully support RISC-V from 24.04 LTS (including Docker images). Debian will likely then catch up with a 12 stable release (and 13 testing release) that also supports RISC-V. So we have about a year to wait./SPECULATION The reason for optimism is that substantial resource is being poured into RISC-V dependencies from companies (including Google) through the RISC-V Software Ecosystem (RISE) project. Keep an eye on the RISE Debian wiki for updates.
Meanwhile the images here need to catch up to Debian 12...

Returning to this thread (a year and a bit later).
RISC-V Docker images are indeed available in Ubuntu 24.04

and also in Debian 13 "Trixie", which is still at the 'testing' stage and not yet a stable release

but no sign of it yet in the stable Debian 12 images, and maybe that will not happen, which leaves us waiting until next year when "Trixie" becomes stable.


feat(effects): explicitly set true disabled/transfer values

Plutonium's AE handling, when it comes to compatibility with
MidiQOL/DAE, is currently abysmal. As part of efforts to straighten that
out moving forward, be more explicit about which effects have which
transfer/disabled values.
Additionally, avoid using MidiQOl-specific versions of these flags, as
they don't seem to take precedence over the real transfer/disabled,
which Plutonium always sets to a non-null value internally.
Note that for all "addon" effects, the implicit default for both values
is false. This also applies to the foundry-*.jsons/etc. in the
5etools source, so copy-pasting between the two won't require any
changes.

This probably still needs tweaks to the AE parser tool, but I don't have nice example source data handy. @Spappz if you've got some you could throw at me, that'd be perfect 😊


update record with a WHERE statement

Trying to update a record in a collection where i match a field which is not the RECORD_ID
not sure if this is possible is the javascript-sdk
Kindly requesting help
Big fan of the project by the way

You can update a record only by its ID.
If you want to update by ussing different field, you can query first the record and then extract its id and send the update request.
In future we will add support for bulk update/create/delete which will allow providing a filter query, but that will be after the users and profiles refactoring (you could also check this discussion - https://github.com/pocketbase/pocketbase/discussions/545).

This should not be marked as complete IMHO. Keeping it open makes sense in this case.

@hood bulk update/create/delete will be handled together with the data import feature. You could subscribe to issue #48.

@0x1337z If you are asking for updating a record using a WHERE/filter clause with the Web APIs, then No and there are no longer plans for this as it comes with too many edge cases. If you really need it you'll have to register your own route and operate with db helpers (using Go or the JSVM).
If you are asking for the bulk update/create/delete actions, then this is already implemented as part of the ongoing refactoring but again the individual operation would be similar to the existing APIs, aka. they require to provide the id of the record to update.


Icon Request: cross

Icon Request

Icon name: cross
Use case: health or pharmacies
Screenshots of done icon:

Could submit this as a PR

Will submit as soon as I can 👍

Any progress on this @lscheibel

Hey, sorry for being inactive for so long. I sorta forgot about this 😅

I prefer B with a border-radius of 1 px on the outer edges. What do you think?

Hey, sorry for being inactive for so long. I sorta forgot about this 😅

I prefer B with a border-radius of 1 px on the outer edges. What do you think?

I prefer A

I prefer A

I prefer B

I prefer B

100% View

Pixel preview

Maybe this helps with deciding 😄

100% View

Pixel preview

Maybe this helps with deciding 😄

@lscheibel Nice thanks for sharing. I think A or B.
What's your own opinion?

@lscheibel Nice thanks for sharing. I think A or B.
What's your own opinion?

I think B actually I changed my mind to A. While B imo preserves the "cross" a bit better, the more round one seems to fit a bit better in the overall asthetic? Everytime I look at it my opinion changes 😄

I think B actually I changed my mind to A. While B imo preserves the "cross" a bit better, the more round one seems to fit a bit better in the overall asthetic? Everytime I look at it my opinion changes 😄

I'm with you, both looks have their advantages.
I let you choose which one you want to add!

I'm with you, both looks have their advantages.
I let you choose which one you want to add!

Choose A now and updated the PR

Choose A now and updated the PR


chore: upgrade deno to 1.11.1

The deployment is failing with strange error...
00:37:16.980 | TypeError: Cannot read property 'length' of undefined
-- | --
00:37:16.982 | at Object.build (/vercel/d670d8cb963ab198/.build-utils/.builder/node_modules/vercel-deno/src/index.ts:261:50)
00:37:16.982 | at buildStep (/var/task/webpack:/@zeit/api/scripts/build-container/src/utils/purpose-build.ts:164:1)

The deployment is failing with strange error...
00:37:16.980 | TypeError: Cannot read property 'length' of undefined
-- | --
00:37:16.982 | at Object.build (/vercel/d670d8cb963ab198/.build-utils/.builder/node_modules/vercel-deno/src/index.ts:261:50)
00:37:16.982 | at buildStep (/var/task/webpack:/@zeit/api/scripts/build-container/src/utils/purpose-build.ts:164:1)

Is this issue of vercel-deno?

Upgrading vercel-deno fixed the issue.


Could you include AngelLoader.ico

Hi Fen,
could you include the AngelLoader.ico in the release zip file?
It looks like some applications like steam for linux can't use the icon within the executable.

Sure, done.


[FLINK-34643] Fix concurrency issue in LoggerAuditingExtension

What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluster with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Can you elaborate a bit why this would solve the issue? Isn't beforeEach called only once per test method? :thinking:

Sure, my theory is that although the order is correct, the write to this field (loggingEvents) might not be visible to other threads, including threads started by Flink and logging threads (if any).
So some threads might see null there, catch NPE and log it to ConsoleLogger.
I'm not 100% sure that this is the issue, but it seems more likely than async logging or buffering of log records.
The fix is to use volatile or avoid using field (use closure). I'm using both (just in case some new usage will be added later).

Thanks for reviewing! Merging


Update CSV mime type

The mime-types 3.2.0 gem has changed the type for .csv files. This is needed to get the builds passing again.

Is this the correct changelog entry for mime-types 3.2.0? I don't see the change listed there.

Yes, that is the correct changelog entry. See the issue the changelog references, it describes the change using the CSV mime types.
https://github.com/mime-types/ruby-mime-types/pull/132

One such example are "text/comma-separated-values" and "text/csv" MIME types. The former is obsolete and unregistered, while the latter is not obsolete and registered, but mime-types would still rank "text/comma-separated-values" higher because it just looks at the alphabetic order.


Hotfix for failing linking to project search

Tackles https://www.wrike.com/open.htm?id=1408704796

Testing the changes locally, and this does not work. It still shows 11 results when navigating the the link /about/projects?consortiaType=SPARC&search=implantable&selectedProjectsAnatomicalFocusIds=Vagus

I thought it was working, sorry... I made another change to avoid executing the consortiaType watcher
https://github.com/nih-sparc/sparc-app-2/assets/4764217/113ccd24-d167-4600-928d-69ec43a494c7
Note that I cannot load the home page...


DRM Widevine L3 decryption key parameters can be placed into youtube-dl ?

Checklist

[X ] I'm asking a question
[X ] I've looked through the README and FAQ for similar questions
[X ] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE
I have decryption keys for Udemy website, I would like to know if I can download Udemy content using the decryption keys I have.
I have paid subscription videos which I would like to download that is DRM protected.
Udemy.com utilizes, L3 Widevine DRM protocol. Wondering if it is possible to download Udemy videos using this open source git.
If so, how do I place the syntax parameters?
Thank You.

 What you are asking for is illegal even if you have the decryption keys.
You know that this is PUBLIC. Do you think they will support it specially it's DRM related?

yt-dl can't help with this.
If you would like to download encrypted media, say to test your hard disk, yt-dlp has the --allow-unplayable-formats option that enables that, if the yt-dlp Udemy extractor supports it.

I feel like this conversation was shut down prematurely.
Yes, I can understand YouTube-DL doesnt want to engage in piracy, or openly break US law. I don't think anyone here is suggesting we do that. Tools are currently available [1] that allow people to decrypt Widevine and similar encryption methods, assuming people have the keys.
This is perfectly legal. You could create a video today using your phone, then encrypt it yourself using Widevine. then you could decrypt the same video, using your own keys that you created. None of this is against the law. This is similar to file sharing, in that its not against any law to file share something you created or own. If people choose to use a tool to break the law, that's their choice. But we should not shut down a conversation, because the possibility exists that someone could misuse a tool.
Similar to how YouTube-DL calls FFmpeg, YouTube-DL could call mp4decrypt, or even implement its own decryption, if user supplies the keys. Personally, I'm not sure how you can say using FFmpeg with encrypted HLS is OK (which YouTube-DL currently does), but any conversation about other encryption methods is shut down. Encryption is not going away, so while I am not arguing for openly breaking the law, I think we should be discussing handling encryption more, as its not unreasonable to think of a future where most sites (including YouTube) are using it.

https://bento4.com/documentation/mp4decrypt

It can be argued that

material offered for you to view or hear is yours to remember;
those who lack eidetic memory should not be prevented from using technical means to aid their memory
so-called effective technical mechanisms that claim to protect content are a legal fiction, or the content could never be accessed.

yt-dl avoids such controversy generally by reproducing what is presented to a browser. Although it might be possible to connect yt-dl to the browser's (eg) WideVine EME, that looks like a research project.
Meanwhile yt-dl and yt-dlp have completely functional mechanisms (more extensive in the latter case) for post-processing the downloaded media, or just piping the output to a further processing stage. For the moment that is where the user should cast any magic spell to make the download usable; some people may wish to make an archive of unplayable content in case it may become playable in future. So I would be open to back-porting --allow-unplayable-formats and modifying relevant extractors to avoid skipping DRM formats under that option.


Button: Focus lost when text of button is changed on click

Expected behavior
Focus is not lost when the contents of a button are updated
Actual behavior
Focus is lost and sent to the document body
Steps to reproduce the issue

Create pharos button
Add JS to change the text of the button on click (like a toggle)
Click button or tab to button and hit enter and see the focus is lost

Screenshots or code
Enter button clicked and focus is lost
https://user-images.githubusercontent.com/13315416/158630980-da4f441e-a689-4942-ba45-086b59e1b397.mov
<pharos-button
       @click="handleToggleClicked"
>
    {{ buttonText }}
</pharos-button>

Pharos version
11.2.2
Your environment

OS: macOS
Browser: Chrome (not Firefox)
Version: 99

Additional information
Can get around this by adding <span> around the text element
<pharos-button
       @click="handleToggleClicked"
>
    <span>{{ buttonText }}</span>
</pharos-button>

@michael-iden I spent a bit of time finally getting a StackBlitz sandbox set up to reproduce Pharos issues more readily. I wasn't able to reproduce the behavior here. Can you try creating a reproduction of the issue using the sandbox as a starting place?

I can no longer reproduce this in the latest version of Chrome or Arc (also Chromium-based), and reconfirmed that this doesn't happen in Firefox or Safari.


Use node 4.0 in Travis

As of #123, npm install crashes on versions of node < 4

By default Travis uses 0.10.36 our server currently has 0.10.25. I'm updating our server now.
FYI @spicyj

Oops, okay!

Upgrading to node 4 via apt-get proved difficult. I've opted instead to take the approach of just not installing it on the prod sever: #131


Reproducer for when expression must be exhaustive.

Might be fixed some day: https://youtrack.jetbrains.com/issue/KT-8781


Notes

XTC Release Notes

Very nice, that's pretty much how I imagined it to look!

I just merged the changes that split the doc into XLT/XTC doc, which also includes the XTC release notes you did here (see https://xltdoc.xceptance.com/xtc/xtc-release-notes/), so I will close this pull request and you may delete this branch.


Maintain text casing from the Google API

Currently, the tool upper cases each word in text fields. Can we remove automatic casing of any text? The Google automatic casing should take care of casing in instances where the text is all lower case.
For example, LA submitted data in which their Early Vote Site days_times_open was submitted as:
8:30 a.m. to 6 p.m. on early voting days. Closed on Sundays.
The tool changes this to:
8:30 A.M. To 6 P.M. On Early Voting Days. Closed On Sundays.
Thank you!

I just wanted to bump this @alexkonrad Thank you!

@jen-tolentino Is there a specific LA address we can use for testing? Thanks!

An example test address:
130 Vital St
Layfayette, LA 70506
Thanks!

Updated.
Alex Konrad
On Thu, May 14, 2015 at 2:43 PM, nomadaisy<EMAIL_ADDRESS>wrote:

An example test address:
130 Vital St
Layfayette, LA 70506
Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/votinginfoproject/vip-embeddable-tool/issues/120#issuecomment-102132221
.

Thank you!


The prediction result is not reset after training again.

System Information (please complete the following information):

Model Builder Version (available in Manage Extensions dialog): latest main (<IP_ADDRESS>7601)
Microsoft Visual Studio Enterprise 2022 (64-bit) Version 17.0.5

Describe the bug

On which step of the process did you run into an issue: Display the previous Predication result on Evaluate page after training again.

To Reproduce
Steps to reproduce the behavior:

Select Create a new project from the Visual Studio 2022 start window;
Choose the C# Console App (.NET Core) project template with .Net 6.0;
Add model builder by right click on the project;
Select Forecasting scenario>Local (CPU);
Input the csv data source, select the predict and time series columns;
Navigate to Evaluate page to predict the model after training completed;
Back Train page, update Horizon to 20, complete train again;
Then navigate to Evaluate page again, see that the previous Predication result displayed there.

Expected behavior
The prediction result should be reset after training again.
Screenshots
If applicable, add screenshots to help explain your problem.

Not repro this issue on latest main: <IP_ADDRESS>0302

But find another issue is that the table frame is truncated after train again, @LittleLittleCloud please take a look?

track in #2028


Not affected by either splitright, botright

Botright should make it open on the right side imho.

I don't think it's possible to use botright or vertical with user-defined commands (correct me if I'm wrong).
command! NEW new
" Try :vertical NEW

You can instead of set up g:plug_window. See https://github.com/junegunn/vim-plug#global-options


Able to apply extra texture in fonts.

Able to apply extra texture in fonts.

And might notice that texture should be relative to the target character (main character only for now.)


Update troubleshooting-info.md

Adding new error code descriptions to SMS troubleshooting

@Prakulka
Can you review the proposed changes? IMPORTANT: 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

Learn Build status updates of commit 706a0d5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/communication-services/concepts/troubleshooting-info.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 4efa408:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/communication-services/concepts/troubleshooting-info.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#sign-off


(docs) Update fallback navigation.

Per puppet-docs PR 595, update the fallback sidebar in this repo to add links to new configuration file docs.

+1


Missing keycode 0

Tell me what key it was or even better, submit a Pull request!
A key on my laptop, second row, 1st left
row 1: esc f1 f2 ...
row 2: [(normal) ` or (shift) ~]  1 2 3 ...
row 3: tab q w e ...

Hey - thanks a lot. Keycode 0 means there is no keyCode associated with that key. I will update the app to reflect that.


refactor: remove old Xtext-based DSL component

Summary of Changes
Remove the old Xtext-based DSL component to prevent confusion. It is still available on the old_dsl branch until the new Langium-based DSL component has feature parity.

:tada: This PR is included in version 0.2.0 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


ERROR cargo_tarpaulin: Failed to compile tests!

Describe the bug
Error while building:
Dec 19 12:50:49.869 ERROR cargo_tarpaulin: Failed to compile tests! Error: redis-module: cannot find value `ptr` in this scope

To Reproduce
> git clone git@github.com:RedisJSON/RedisJSON.git
> cd RedisJSON
> cargo tarpaulin

Expected behavior
Successful build

When trying to build
> git clone git@github.com:RedisLabsModules/redismodule-rs.git
> cd redismodule-rs
> cargo tarpaulin

Got the following error:
0 ERROR cargo_tarpaulin: Failed to compile tests! Error: bitvec: cannot find attribute `skip` in this scope
Error: "Failed to compile tests! Error: bitvec: cannot find attribute `skip` in this scope"

ref https://github.com/xd009642/tarpaulin/issues/756

So for RedisJSON, I got the SIGILL instruction, so I checked the readme and did cargo tarpaulin --features test and it works. Maybe you're missing the feature?
Well when I cloned redismodule-rs fresh it's using nom 7 and bitvec isn't in the dependency tree, I'd recommend upgrading whatever package is bringing it in cargo tree -i bitvec, but I can't see what is doing it... In the meantime --avoid-cfg-tarpaulin is a flag to go back to the legacy tarpaulin attribute behaviour.
I then ran tarpaulin using cargo tarpaulin --features test,experimental-api as this matched the features tested under in CI, and it compiled and was running fine until this error which I imagine is just due to my system setup - I'm not sure if I have redis installed on this machine:

running 2 tests
test test_keys_pos ... FAILED
test test_hello ... FAILED

failures:

---- test_keys_pos stdout ----
Error: failed to start redis server

Is there any more information that could help me recreate this? Compiler versions, any configuration etc? From what I see currently it all seems to work as expected

So for RedisJSON, I got the SIGILL instruction, so I checked the readme and did cargo tarpaulin --features test and it works. Maybe you're missing the feature?

I just ran it with cargo tarpaulin --features test and got the same error:
$ cargo tarpaulin --features test
...

...
   Compiling chrono v0.4.19
   Compiling rand v0.7.3
   Compiling env_logger v0.9.0
   Compiling strum_macros v0.23.1
   Compiling redis-module v0.25.0
error: Broken pipe (os error 32)
warning: build failed, waiting for other jobs to finish...
error: Broken pipe (os error 32)
error: build failed
Dec 19 14:39:13.835 ERROR cargo_tarpaulin: Failed to compile tests! Error: redis-module: cannot find value `ptr` in this scope
Error: "Failed to compile tests! Error: redis-module: cannot find value `ptr` in this scope"

Strange, I just tried it with tarpaulin 0.18.5, with rustc 1.57.0 on ubuntu 18.04 and it worked fine. Can you do a cargo clean before you run tarpaulin?

cargo clean didn't help

Haven't been able to reproduce, still works fine my end and now there's --engine llvm which further increases compat with other projects. So closing this as likely fixed.


[sram_ctrl/dv] Fix sec_cm assertions

Similar to lowRISC/opentitan#23609, this PR adapts the sec_cm DV tests to also consider the enabled security counters for the u_reqfifo and u_sramreqfifo FIFOs.

I am setting this PR to a draft because we should adapt #23627 if we decide not to touch the FIFO RTL.

This got superseded by #23629. @nasahlpa , are you okay to close this?


Add brewers friend as endpoint.

I dont have an account for brewers friend so I have not been able to verfy this completely. Its based on the available documentation. Need help to verfiy these instructions.
Brewer's friend is an all in one service that allows you to manage you recepies and brews.
You can find you API key when logged in to the service. Follow these instructions https://docs.brewersfriend.com/devices/ispindel
Note there are different URLs if you are using plato or specific gravity!
http://log.brewersfriend.com/ispindel/[API KEY]
http://log.brewersfriend.com/ispindel_sg/[API KEY]
From what I can read in the documentation you need to add the API key as a token as well. This can be done using a custom
format for the endpoint. Just add you API key after token.
{
"name" : "${mdns}",
"ID": "${id}",
"token" : "[API KEY]",
"interval": ${sleep-interval},
"temperature": ${temp},
"temp-units": "${temp-unit}",
"gravity": ${gravity},
"angle": ${angle},
"battery": ${battery},
"rssi": ${rssi}
}

Closing this, if there are issues we will handle that as a bug.


Add YQ to modify knative and istio YAML before cluster setup

Summary
Previously, we stored Knative/Istio YAML files with our modifications in place, which is prone to errors. Hence, I wrote a generator, which is based on YQ YAML processor, that applies various updates to YAML files. For example, setting CPU and memory quotas, nodeSelector, CIDR.
Implementation Notes

Add rewrite_yaml_files.sh to modify the yaml files in vhive.
Modify create_multinode.sh and create_singlenode_container.sh to use rewrite_yaml_files.sh to modify the yaml files.
Move stabilize.sh and turbo_boost.sh from vhive to invitro.

First, please state that with this pr we are bumping knative, k8s, istio and metallb versions.
Second, open this pr as a draft pr to master branch instead of this yq_dev branch.

In the description state clearly which versions are now used, as a list, not the sidenote. As well mention that we are disabling flannel.


Add some comments for daemon/graphdriver/aufs/aufs.go

When I read the file aufs.go, I realized some comments missed for the function createDirsFor. So I add some explanation.
Please review, thanks.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:hsinko/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~8
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Can you "squash", and sign-off your commits, otherwise we won't be able to review


Ghci Script Target parses space in Filepath with space

Issue in hie: https://github.com/haskell/haskell-ide-engine/issues/1669
Example: some ghci script:
:add Lib Lib2 "/home/user/Has kell/Test.hs"

is wrongfully parsed into targets: ["Lib", "Lib2", "\"home/user/Has", "kell/Test.hs\""].
Offending code: https://github.com/mpickering/hie-bios/blob/9fd8af352317490f1c68cb628b2ce2a8733b63e5/src/HIE/Bios/Environment.hs#L223

Proposed Solution: use attoparsec to parse these targets correctly.
attoparsec is already in the project dependency closure.

For Documentation:
This happens when we are using a Stack cradle and have a executable component.
Assuming we have only an executable, then stack will produce a ghci-script with the following contents:
:add "<root-path>/Main.hs"

The previous parsing logic just used words to split this line into targets that we can load.
However, this breaks if the path contains a space.
The patch in #163 uses the parser from base to parse whitespace within quotes and allow escaped ".


Require Java 11.0.7

Earlier versions are affected by JDK-8206955.

Duplicate of my PR https://github.com/prestosql/presto/pull/5956 ... maybe use some of that content?


Make the website pretty

We should do this on the demo app. The UI library should ideally be kept quite simple so that other users can add their own styling to the components.

we need to deploy this repo into a temp hosting to test and review. Can you help us with this @mihaisc ?


Configure Redis to cache session

Feature Request
Is your feature request related to a problem or unsupported use case? Please describe.
Update the backend used by the session engine, to improve system performances.
Describe the solution you'd like
Use Redis to handle session.

This was done in #130 - per Django docs, having both SESSION_ENGINE set to django.contrib.sessions.backends.cache (since #59) and Redis setup in CACHES (in settings.py) means sessions are (non-persistently) cached in Redis.


Incorrect Ordinals

I'm submitting a ...  (check one with "x")
[ x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request

Current behavior

Ordinals from 11 to 13 are not correct (11st, 12nd, 13rd)
Expected behavior

They should be 11th, 12th, 13th
Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

I'll try to make a fix ASAP

@fknop I can take care of it if you like

@christopheelkhoury I would appreciate it, thanks

@fknop on it. Should be done by 6 PM GMT 😃

Nvm done


Update Facebook SDK to 4.36.0

Description
LEARNER-5572

Facebook SDK has been updated to 4.36.0 in this PR.

Please refer to Jira story for more details.
Docs followed:
https://developers.facebook.com/docs/facebook-login/android/
Sample project followed:
https://github.com/facebook/facebook-android-sdk/tree/master/samples/FBLoginSample
Testing:

Facebook login button should work accordingly for all cases (Existing user and a new user).
Register with Facebook button should work accordingly for all cases (Existing user and a new user).
When user will try to register with new Facebook account, email and name fields should be populated properly (In this case user facebook profile data will be collected through graph api).
App should build and work smoothly if Facebook configuration is missing in config file.
Gradle shouldn't make the build if Facebook is enabled in configuration but key is missing.
App should behave accordingly when Facebook is enabled but wrong key is provided in congifuration.

Pending:
- Discussion on zero network check in pending, should we control facebook sdk integration with that or not.

cc: @staubina , @miankhalid

Can you please write some instructions documented in Confluence on how to test Facebook login? I want to make sure we test all these changes thoroughly.

FB test cases are already written in doc: https://openedx.atlassian.net/wiki/spaces/MA/pages/773816340/Regression+Test+Cases+edX+Apps

Some minor style nits. Are we using the Google Android and Java Style guides?
There are a couple of locations where it looks like we might have used 8 spaces where we could use 4 and it would be more clear. I will take a look again and try to comment on the lines. This is not blocking just and observation.

We are following Google Android guides which is almost same as of Java Style guide, wandering there are some naming convetion differences.
In short just press the short cut key Command+Option+L and it will let you know the correct indentation recommended by Google team.
Other than that we are also following some of our own convetions define for project like not start class variables names with 'm'

@miankhalid its ready for next pass. Only zero rate network check discussion is pending which i have mentioned in PR description as well.
Added more testing scenarios in PR description as well.

I like to hear this. We should consider adding a linter to the PR tests to help keep the OSPRs in check.

@staubina Yeah, I like the idea here. Especially incase of OSPRs, this is a major problem. We should have some code quality checks built into our PRs build line. Any suggestions? or something we might already be using in other Open edX repos?

@miankhalid ready for next pass.

@miankhalid We would need to use a Java linter.  I will look up what the currently preferred one is for Android. No one else will be using a Java linter at edX right now.

Cool, I'll try to give one a look as well.


Formatting updates to improve readability.

It might look like a lot, but most changes involve returning early to reduce nesting.

I can merge this into master, but it isn't going to do any good once v2 comes out. If you want this merged into v2, please update this PR or submit a new one.

I'll wait until v2 comes out then.


[RFC] Contrib test suite + tests for timm and sentence_transformers

First discussed in https://github.com/huggingface/huggingface_hub/issues/1190. Goal is to detect proactively breaking changes and deprecation warnings in downstream libraries. This is a very first implementation with tests for timm and sentence_transformers libraries to validate the concept. I think for a start we can have a github workflow only triggered
manually or contrib_ci_* branches. It doesn't really make sense to run it on each PR or on main.
How it works ?
The contrib folder contains simple end-to-end scripts to test integration of huggingface_hub in downstream libraries. The main goal is to proactively notice breaking changes and deprecation warnings. Each library is tested in its own virtualenv (with its own dependencies). Here is the workflow for "timm" library:

Create a virtualenv under ./contrib/timm/.venv
Install common requirements  ./contrib/requirements.txt (e.g. pytest)
Install lib requirements  ./contrib/timm/requirements.txt: install timm from main branch. Configurable for each lib.
Uninstall huggingface_hub (if any)
Install huggingface_hub from source code to test against latest version
Run timm tests pytest ./contrib/timm

See https://github.com/huggingface/huggingface_hub/pull/1200#issuecomment-1322444901 for more details.
How to add a new library ?
To add another contrib lib, one must:

Create a subfolder with the lib name. Example: ./contrib/transformers
Create a requirements.txt file specific to this lib. Example ./contrib/transformers/requirements.txt
Implements tests for this lib. Example: ./contrib/transformers/test_push_to_hub.py
Edit makefile to add the lib to CONTRIB_LIBS variable. Example: CONTRIB_LIBS := timm transformers
Edit .github/workflows/contrib-tests.yml to add the lib to matrix.contrib list. Example: contrib: ["timm", "transformers"]

Run contrib tests in CI
Contrib tests can be manually triggered in GitHub with the Contrib tests workflow. CI is also triggered on branches starting by ci_contrib_*
Tests are not run in the default test suite (for each PR) as this would slow down development process. The goal is to notice breaking changes, not to avoid them. In particular, it is interesting to trigger it before a release to make sure it will not cause too much friction.
Run contrib tests locally
Tests are separated to avoid conflicts between version dependencies. Before running tests, a virtual env must be setup for each contrib library. To do so, run:
# Run setup in parallel to save time
make contrib_setup -j4

# Run tests
# Optional: -j4 to run in parallel. Output will be messy in that case.
make contrib_test -j4

# Run only "timm" tests
make contrib_setup_timm
make contrib_test_timm

See https://github.com/huggingface/huggingface_hub/pull/1200#issuecomment-1323405061 for more details.
Todo:

[x] separated virtual envs
[x] working CI
[x] working makefile with multiple contrib
[X] README
[x] from_pretrained test for timm

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@osanseviero thanks for your feedback, it helped a lot !
I make some changes to the contrib structure. Now I have a common requirements.txt. For each lib, I have a folder with a test file and another requirements.txt. The workflow is now to launch 1 process/job per dependent library:

create virtualenv for timm
install common requirements
install timm requirements
uninstall huggingface_hub (if any)
install huggingface_hub from source code
run timm tests

This way we don't have to handle conflicts between different dependencies / versions. It makes local tests a bit more complex as it requires 1 env per contrib. Hopefully not much people will really need to run those stuff locally. What I plan to do is to add a script (either separate, either in makefile) to handle all the stuff with the venvs. WDYT in general ?

Refactored a bit the makefile. Now possible to do:
# Setup all virtualenvs
make contrib_setup

# Run all tests
make contrib_tests

# Setup and run all tests at once
make contrib

# Delete all virtual envs (if corrupted)
make contrib_clear

And for a specific lib:
# Setup timm tests
make contrib_setup_timm

# Run timm tests
make contrib_test_timm

# Setup and run timm tests at once
make contrib_timm

# Delete timm virtualenv
make contrib_clear_timm

Thanks for the review @osanseviero. I made some changes and addresses all of your comments.
Let's have a final review from @LysandreJik once he is back from holidays and then merge the PR.

@osanseviero @LysandreJik thanks for the last reviews :)
I've made some last minute changes:

added a script to list contrib test and update Makefile/github action. It's now integrated to make style and make quality.
added a cron to run tests every Saturday midnight. Let's see how it goes. Later work should be to keep us updated about the cron result (slack message for instance ?). This can be done later as my main concern is to actually get more contrib tests first.
removed the contrib/requirements.txt in favor of .[testing]. Let's see if at some point we need a more specific requirement.

So I think we are now finally good to go :smile: :fire:


LDAP password store in db as plain text

But if I create new user, password stored hashed.
In code:
    def create_user(self):
        """
        If user logged in successfully via LDAP in the first time
        We will create a local user (in DB) in order to manage user
        profile such as name, roles,...
        """
        db.session.add(self)
        db.session.commit()

I fix this by add
self.password = self.get_hashed_password(self.plain_text_password)
before
db.session.add(self)

The password-logic should probably be removed from LDAP-users completely? Or is the idea that LDAP-users should be able to log in with whatever passord is stored locally if LDAP suddenly becomes unreachable? (provided that they've logged in before and the user was created).

User's LDAP passwords should never be stored in local DB whatsoever - plain or hashed. That's one of the point of centralized identity management, that even when the app is breached, passwords are still safe.. This issue is a security vulnerability.

@pschiffe, never thought about it from that angle before, but I guess most people utilize LDAP to be able to have the same password "everywhere". I know of several applications that caches users/passwords locally in order to facilitate login in case LDAP is unreachable (if user is deactivated in LDAP, and LDAP is reachable, the users won't be able to log in, of course -- so only in the event of an unreachable LDAP server).
The biggest issue here is that it's stored in plaintext, but I have no issue with removing it completely...

@jallakim LDAP is used in that way because it's easy to set up and use that way. Caching passwords is definitely a wrong work-around to a problem, which should be solved differently (LDAP replicas). But again, because it's possible to cache it, and relatively easy to do, people do it. But that doesn't mean it's OK, it's definitely insecure.
When you look at the modern standards for "one password everywhere", or single sign on - an oauth or openid connect, you see that in this case, the app never sees the password, so it's not possible to cache it. And that's how it should be.
Please, if it's possible, don't store the LDAP password in pdns-admin db in any way.

I agree with @pschiffe . The User which created from LDAP or openID should never have a password. LDAP's "local user" in the pdns-admin is only used for role/permission setting. The current code should be changed to remove LDAP user password completely.


Create draft store client module

Please set up CI for this repo

Worth having a look at ExpressJs template app, which contains all the basic elements (quality checks, docker, travis, basic security setup, etc.) your application should have.


Dev: Update readme file - changes to the SW download section

Expand and clarify the section describing the software downloads.
List all 3 file names

marc is closing this duplicate PR.


Create emacs-console

(See the guidelines for contributing, linked above)

Collapse this into the change in #3982.


Blog app: add devise

Project Requirements
Authentication Refactoring

Remove the current_user method in ApplicationController, as Devise will provide the necessary functionality.

Devise Integration

Install and set up Devise for user authentication.

User Registration

Implement the ability for users to register for a new account.

User Login

Users should be able to log in using a combination of email and password.

Password Storage

Ensure that hashed passwords are securely stored in the database.

Email Confirmation

Implement a mechanism to confirm user email addresses.

Password Reset

Provide functionality for users to reset their passwords.

View Modification

Modify the views in Devise for registration and login to match the wireframes provided in the Sneak Peek.
Apply styling to the views to maintain consistency with the overall project design.

Hi @obibaadoma, @EvansSnave and @fmanimashaun,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request

The last two team members did not make meaningful contributions to this project. Kindly redo the project with meaningful commits related to the project guidelines.

In the new PR, also ensure there are not 5000+ files changed. You should use .gitignore properly, as files of that amount make review next to impossible.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


[release-branch.go1.22] Update to github.com/golang-fips/openssl/v2 v2.0.1; increment rev

Incrementing rev manually should allow the release process to build this commit as is without running through PR validation again.

/azp run microsoft-go


[OFFICIAL/TV] — Mr. OlYmPiA 2024 LiVe 𝐒𝐭𝐫𝐞𝐚𝐦𝐬 FrEe BrOaDcAsT oN tV cHaNnEl 12 October 2024 #509 #22413

The highly anticipated 60th edition of the Mr. Olympia competition is scheduled to take center stage during the 2024 Olympia weekend in Las Vegas, NV. Known as the pinnacle of professional bodybuilding, the event will span two nights, beginning with the pre-judging rounds on Friday, Oct. 11, 2024, and culminating with the finals on Saturday, Oct. 12, when the new Mr. Olympia will be crowned.
 Click here to access Mr. Olympia PPV Live access from anywhere.
Originally, 19 elite competitors qualified for the 2024 event. However, Tonio Burton, a rising star and strong contender, has opted not to participate, narrowing the field to 18 athletes. Despite his absence, the competition remains fierce, with bodybuilders vying for not only the title of Mr. Olympia but also the impressive $400,000 cash prize— the largest payout in bodybuilding this year. Fans and experts alike are eager to see which athlete will rise to the top and etch their name into the sport’s storied history.
How to Watch 2024 Mr. Olympia Official PPV Anywhere (Low Cost)
Fans who are unable to attend the event in person can still catch all the action through a pay-per-view livestream. The livestream will offer comprehensive coverage of both the professional and amateur competitions, ensuring viewers don’t miss any of the excitement across the event’s two-day schedule.
Details on how to access the livestream, including pricing and streaming options, will be available on the official Olympia website, allowing fans to enjoy the experience from the comfort of their homes, no matter where they are.
2024 Mr. Olympia Qualifying
According to IFBB Pro League rules, former Mr. Olympia champions have lifetime eligibility to compete, though after five years, they need permission from IFBB President Jim Manion.
The top three finishers from the previous Mr. Olympia automatically qualify for the next year, while others must win a pro show within the qualification season, which ended on Sept. 15, 2024. No special invitations are expected this year.
2024 Mr. Olympia Competitors
Here are the athletes, listed by their qualification order:
Derek Lunsford (USA) – Defending Champion
Hadi Choopan (Iran)
Samson Dauda (UK)
Theo Leguerrier (France)
Rafael Brandao (Brazil)
Martin Fitzwater (USA)
Nicholas Walker (USA)
Akim Williams (USA)
Brandon Curry (USA)
William Bonac (Netherlands)
Behrooz Tabani (Iran)
Nathan De Asha (UK)
John Jewett (USA)
Mohamed Foda (Egypt)
Jonathan Delarosa (USA)
Andrew Jacked (UAE)
Hunter Labrada (USA)
Bruno Santos (Brazil)
Past Olympia Champions
This year’s lineup includes three previous winners: 2023 champion Derek Lunsford, 2022 titleholder Hadi Choopan, and 2019 victor Brandon Curry, who remains eligible under a five-year rule. Notably, 2020 and 2021 winner Mamdouh “Big Ramy” Elssbiay will not compete this year.
The last time three former champions shared the Mr. Olympia stage was in 2020, when Curry, seven-time champion Phil Heath, and 2008 winner Dexter Jackson competed. That year, Elssbiay won his first title.
As 2023’s finalists, Lunsford and Choopan are widely considered top contenders, while Choopan and Curry are both aiming to regain the title, a feat only Jay Cutler accomplished in 2009.
Though some experts doubt Curry’s chances of a high placing in 2024, especially after a fourth-place finish in 2023 following hospitalization, he’s still a factor.
Mr. Olympia Top Contenders
Outside of the past champions, a few athletes stand out as major threats for the title. Samson Dauda, who placed third at the 2023 Mr. Olympia, finished second behind Choopan at both the 2024 Arnold Classic and Arnold Classic UK. While Dauda’s size is impressive, Choopan and Lunsford have historically outmatched him in conditioning.
Nick Walker is another strong contender. Though he missed the 2023 Olympia due to a hamstring injury, he has a history of defeating both Dauda and Curry. After winning the 2024 New York Pro, Walker feels confident about his chances in Las Vegas, even going so far as to practice his victory speech. His conditioning could rival Choopan and Lunsford.
Andrew Jacked and Hunter Labrada have also been top-five finishers in the past. Both athletes qualified for 2024 late in the season, and if they bring their best, they could be contenders for the first callouts during pre-judging.
Mr. Olympia Dark Horses
The remaining spots in the top ten are wide open, with several athletes capable of making it into the top six if any favorites falter. Martin Fitzwater, making his Olympia debut, is one to watch, as cited by four-time Mr. Olympia Jay Cutler.
Nathan De Asha, who hasn’t been on the Olympia stage since 2018, has been successful in other competitions but has faced injuries and travel issues. If he competes, he’s likely to be a top-10 contender.
Veteran bodybuilder William Bonac, a two-time Arnold Classic winner and 2019 Mr. Olympia runner-up, is still going strong after a decade of competing at the Olympia.
2024 Olympia Schedule
Locations & times subject to change
Monday, 10/7/2024
Amateur Olympia Athlete Registration
Location: Resorts World Las Vegas (Primrose Meeting Room)
Time: 4:00 PM
Tuesday, 10/8/2024
Amateur Olympia – Women’s Judging
Location: Resorts World Las Vegas (The Olympia Pavilion)
Time: 8:00 AM
Amateur Olympia – Women’s Finals
Location: Resorts World Las Vegas (The Olympia Pavilion)
Time: TBD
Wednesday, 10/9/2024
Amateur Olympia – Men’s Judging
Location: Resorts World Las Vegas (The Olympia Pavilion)
Time: 8:00 AM
Amateur Olympia – Men’s Finals
Location: Resorts World Las Vegas (The Olympia Pavilion)
Time: TBD
Thursday, 10/10/2024
Olympia Press Conference (Presented by WOLFpak)
Location: Resorts World Las Vegas (The Olympia Pavilion)
Time: 12:00 PM
(11:30 AM – VIP Entry
11:45 AM – General Public Entry)
Meet the Olympians (Presented by BSN)
Location: Resorts World Las Vegas (The Olympia Pavilion)
Time:
7:00 PM – VIP Entry
8:00 PM – General Public Entry
Friday, 10/11/2024
Olympia World Fitness Expo
Location: Las Vegas Convention Center (South Hall, Floors 1 & 2)
Time:
8:00 AM – VIP Entry
9:00 AM – General Public Entry
Olympia Pre-Judging
Location: Las Vegas Convention Center
Time: 9:30 AM
Fitness Olympia
212 Olympia
Figure Olympia
Women’s Physique Olympia
Ms. Olympia
Wellness Olympia
Olympia VIP Behind-the-Scenes Theater Tour
Location: Resorts World Theatre (Outside Main Doors)
Time: 3:00 PM sharp
(Gold VIP & Silver VIP Plus Only)
Olympia Friday Finals
Location: Resorts World Theatre
Time: 6:00 PM
212 Olympia
Fitness Olympia
Figure Olympia
Women’s Physique Olympia
Ms. Olympia
Wellness Olympia
Plus Mr. Olympia Pre-Judging
Saturday, 10/12/2024
Olympia World Fitness Expo
Location: Las Vegas Convention Center (South Hall, Floors 1 & 2)
Time:
8:00 AM – VIP Entry
9:00 AM – General Public Entry
Olympia Pre-Judging
Location: Las Vegas Convention Center
Time: 9:30 AM
Classic Physique Olympia
Men’s Physique Olympia
Bikini Olympia
Wheelchair Olympia Pre-Judging & Finals
Official Olympia Watch Party (Presented by BodyDream)
Location: Resorts World (Zouk Nightclub)
Time: Doors open at 6:30 PM
Olympia Saturday Finals
Location: Resorts World Theatre
Time: 7:00 PM
Mr. Olympia
Men’s Physique Olympia
Bikini Olympia
Classic Physique Olympia
Olympia VIP Rooftop After Party (Presented by ESN)
Location: Resorts World Las Vegas Rooftop
Time: 11:00 PM (VIPs Only)
Sunday, 10/13/2024
Olympia Champions Seminar
Location: Resorts World Las Vegas (The Olympia Pavilion)
Time: 11:00 AM

dsfsdfe
📺📱  Mr OlYmPiA 2024

.
.
.
.
.
.
📺📱  Mr OlYmPiA 2024

.
.
.
.
..
📺📱  Mr OlYmPiA 2024

.
.
.
.
.
..
.
.
📺📱  Mr OlYmPiA 2024
.
.

.
.
.
.
.
.
.
.
.
📺📱  Mr OlYmPiA 2024

.
.
.
.
.
.
.
xfvedw


NativeApp: Enable/disable default hotkeys

Create AppBase::EnableHotkeys() method to enable/disable default hotkeys (such as exit from app on Esc and change fullscreen mode on Shift+Enter in UWP).

This implementation seems to result in incorrect handling of the shift key in UWP. Also, toggling all hot keys on/off at the same time is not very flexible.
I would recommend reworking this change as follows:

Add HOT_KEY_FLAGS flag enum with HOT_KEY_FLAG_ALLOW_EXIT_ON_ESC and HOT_KEY_FLAG_ALLOW_FULL_SCREEN_SWITCH values.
Add HOT_KEY_FLAGS GetHotKeyFlags() method instead of EnableHotkeys()

Hmm.. Nice idea. Maybe also add HOT_KEY_FLAG_NONE = 0 (or HOT_KEY_FLAG_ALLOW_NONE)?


c/driver/postgresql: compiler warning -Wnarrowing

In file included from /adbc/c/driver/postgresql/statement.h:29,
                 from /adbc/c/driver/postgresql/postgresql.cc:27,
                 from /adbc/build/arm64/driver/postgresql/CMakeFiles/adbc_driver_postgresql_objlib.dir/Unity/unity_0_cxx.cxx:7:
/adbc/c/driver/postgresql/postgres_copy_reader.h:35:73: error: narrowing conversion of ‘'\377'’ from ‘char’ to ‘int8_t’ {aka ‘signed char’} [-Wnarrowing]
   35 |                                           '\n', '\377', '\r', '\n', '\0'};
      |

Oh, this might be causing builds to fail, too.


Adding entry sensor to Simplisafe Homebridge doesn't see the new sensor

Just added a new sensor to my Simplisafe system, works fine but doesn't show in Homebridge. Any suggestions?

Hi @cwmn , as the bot stated, there is not nearly enough info here to be able to help. Please provide more details about what happened, what you’ve tried to do to fix it, and logs etc. if appropriate. And, have you restarted it?

Michael,
Sorry I had never seen a firm like that for reporting issues. Must admit I wasn’t sure of the syntax etc. the good news is that I solved my issue with and o do mean my issue not yours. Here’s what I did.
Went into Homebridge, Accessories, saw the sensor, clicked the gear in the upper right corner, selected show in dashboard widget, everything is working fine now! Problem solved!
Thanks for your plug-in and for reaching out to me.
Chuck
Sent from my iPhone

On May 12, 2021, at 6:11 PM, Michael Shamoon @.***> wrote:

Hi @cwmn , as the bot stated, there is not nearly enough info here to be able to help. Please provide more details about what happened, what you’ve tried to do to fix it, and logs etc. if appropriate. And, have you restarted it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


feat: remote template repositories

Adds support for remote Git template repositories.
Accessible using the --repository flag on the CLI, and WithRepository client Option.
Specifying this option takes precidence over both extensible (on-disk) template repositories and embedded.

/kind enhancement

I've added the following issues to address comments in this PR. @lkingland if I have missed any, shout out!

https://github.com/knative-sandbox/kn-plugin-func/issues/451
https://github.com/knative-sandbox/kn-plugin-func/issues/452
https://github.com/knative-sandbox/kn-plugin-func/issues/453
https://github.com/knative-sandbox/kn-plugin-func/issues/454

one or more commits were authored or co-authored by someone other than the pull request submitte

Dear Google-bot.  I rebased on main such that only myself and Lance exist in this PR.  Seeing as the commits I added from Lance were suggested changes, doesn't that imply consent?   @lance you may need to @googlebot I consent. until we find the "I" in "AI".

@googlebot I consent

/lgtm


Specify backing file format

System Information
libvirt 6.0.0+
Linux distribution
openSUSE Tumbleweed
Terraform version
v0.12.13
Provider and libvirt versions
# terraform-provider-libvirt -version
terraform-provider-libvirt was not built correctly
Compiled against library: libvirt 6.0.0
Using library: libvirt 6.0.0
Running hypervisor: QEMU 4.2.0
Running against daemon: 6.0.0

If that gives you "was not built correctly", get the Git commit hash from your local provider repository:
Installed from systemsmanagement:sumaform OBS repo

Description of Issue/Question
Starting with version 6.0.0 libvirt requires the backing file format to be provided either in the qcow2 metadata or in the XML for security reasons. See this page for more informations.
When using a base image, the generated libvirt XML does not include the <backingStore> element and the qcow2 overlay is not generated with -F option. Note that in order to avoid the security problems in the terraform libvirt provider, the properties may be extended with a base_volume_format.
Setup
Just ensure the VM has a base_volume_name pointing to a disk image

Additional information:
The implementation of the fix would ideally add both the use of the qemu-img create -F parameter and add something like this to the XML definition of the disk:
<backingStore type='volume'>
  <format type='qcow2'/>
  <source pool='mypool' volume='myvolume.qcow2'/>
</backingStore>

However this is not yet handled by libvirt and a fix is on the way. The following is working though:
<backingStore type='type'>
  <format type='qcow2'/>
  <source file='/path/to/myvolume.qcow2'/>
</backingStore>

Just tested recreating resources from scratch with libvirt 6.0.0. The created volume XML is:
<volume>
    <name>test-main-disk</name>
    <capacity unit="bytes">214748364800</capacity>
    <target>
        <format type="qcow2"></format>
        <permissions>
            <mode>644</mode>
        </permissions>
    </target>
    <backingStore>
        <path>/public/vms/sles15sp1</path>
        <format type="qcow2"></format>
    </backingStore>
</volume>

And qemu-img info /public/vms/test.qcow2 | grep backing outputs backing file: /public/vms/sles15sp1.
So all is fine. Closing


What version number should have the first joint release?

I would suggest 0.18:  we have 0.12 from WorldSynth + 0.5 from Lands, it means we are currently at 0.17

We agreed to go for 0.18 (please correct me if I misunderstood!)


Nautilis iRelay support?

I'm currently using a Nautilis iRelay with my iSpindel. I was wondering if the hardware could be reused for the GravityMon Gateway?
It has a (Wemos) Lolin D1 mini with an OLED screen.

In principle yes, but there is currently no build for that hardware combinations since the esp8266 is quite old and most of the software frameworks are no longer maintained.
I would recommend to go for one of the newer hardware models, preferably an S3 based board with much more compute power. You dont really need the display since you an also view data via the web interface.
The main target for this project is to support the Bluetooth transmissions from the Gravitymon 2.x version.


Bug/switch merge lead ids

Are you a current Fivetran customer?

Fivetran made PR
What change(s) does this PR introduce?

switches merge_lead_id and lead_id, which appeared to be mixed up! merged_lead_id can be a comma-separated list of IDs, whereas lead_id is an integer
updates changelog
updates package version

Does this PR introduce a breaking change?

[ ] Yes (please provide breaking change details below.)
[x] No  (please provide explanation how the change is non breaking below.)

Is this PR in response to a previously created Issue

[x] Yes, Issue https://github.com/fivetran/dbt_marketo/issues/17
[ ] No

How did you test the PR changes?

[x] CircleCi
[ ] Other (please provide additional testing details below)

Select which warehouse(s) were used to test the PR

[x] BigQuery
[x] Redshift
[x] Snowflake
[ ] Postgres
[ ] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

🍋
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

Looks like it's failing dbt test. Let me know when you want me to re-review! Outside of that, everything else looks good.


can't build on windows7 32-bit

I'm a rookie. I try to use 'npm install pdfium' command, but always error.
Info:
2421 info preinstall<EMAIL_ADDRESS>2422 silly install resolved []
2423 verbose about to build C:\Users\xiaoai\node_modules\pdfium
2424 info build C:\Users\xiaoai\node_modules\pdfium
2425 verbose linkStuff [ false, false, false, 'C:\Users\xiaoai\node_modules' ]
2426 info linkStuff<EMAIL_ADDRESS>2427 verbose linkBins<EMAIL_ADDRESS>2428 verbose linkMans<EMAIL_ADDRESS>2429 verbose rebuildBundles<EMAIL_ADDRESS>2430 info install<EMAIL_ADDRESS>2431 verbose unsafe-perm in lifecycle true
2432 info<EMAIL_ADDRESS>Failed to exec install script
2433 verbose unlock done using C:\Users\xiaoai\AppData\Roaming\npm-cache_locks\pdfium-791539a3aa48c796.lock for C:\Users\xiaoai\node_modules\pdfium
2434 verbose stack Error<EMAIL_ADDRESS>install: node-gyp rebuild
2434 verbose stack Exit status 1
2434 verbose stack     at EventEmitter. (D:\Tools\nodejs\node_modules\npm\lib\utils\lifecycle.js:213:16)
2434 verbose stack     at EventEmitter.emit (events.js:110:17)
2434 verbose stack     at ChildProcess. (D:\Tools\nodejs\node_modules\npm\lib\utils\spawn.js:14:12)
2434 verbose stack     at ChildProcess.emit (events.js:110:17)
2434 verbose stack     at maybeClose (child_process.js:1008:16)
2434 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1080:5)
2435 verbose pkgid<EMAIL_ADDRESS>2436 verbose cwd C:\Users\xiaoai
2437 error Windows_NT 6.1.7600
2438 error argv "D:\Tools\nodejs\\node.exe" "D:\Tools\nodejs\node_modules\npm\bin\npm-cli.js" "install" "pdfium"
2439 error node v0.12.1
2440 error npm  v2.5.1
2441 error code ELIFECYCLE
2442 error<EMAIL_ADDRESS>install: node-gyp rebuild
2442 error Exit status 1
2443 error Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
2443 error This is most likely a problem with the pdfium package,
2443 error not with npm itself.
2443 error Tell the author that this fails on your system:
2443 error     node-gyp rebuild
2443 error You can get their info via:
2443 error     npm owner ls pdfium
2443 error There is likely additional logging output above.
2444 verbose exit [ 1, true ]
2445 verbose unbuild node_modules\pdfium
2446 info preuninstall<EMAIL_ADDRESS>2447 info uninstall<EMAIL_ADDRESS>2448 verbose unbuild rmStuff<EMAIL_ADDRESS>from C:\Users\xiaoai\node_modules
2449 info postuninstall<EMAIL_ADDRESS>2450 verbose gentlyRm vacuuming C:\Users\xiaoai\node_modules\pdfium
2451 silly gentlyRm purging C:\Users\xiaoai\node_modules\pdfium
2452 silly gentlyRm quitting because other entries in C:\Users\xiaoai\node_modules
node-gyp rebuild is failed.what need tools for it?Thank you

sorry for silly question: do you have python 2.7.x installed and configured correctly?

I installed python, because it use npm install printer command is true
npm install printer   and    npm install pdfium,are they neeed the same tools?

Sorry,i think maybe it is build pdfium's question.I don't kwon pdfium is Chromium's open souce.
For build instructions visit https://code.google.com/p/pdfium/wiki/Build
Diff:

removed v8 folder

this info put tojocky/node-pdfium maybe good for rookie?


Merge 0.23.26 -> main

No conflicts, but I haven't tried to build it.

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

Project coverage is 77.61%. Comparing base (be57ab5) to head (cef754e).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7404      +/-   ##
==========================================
+ Coverage   77.60%   77.61%   +0.01%
==========================================
  Files         335      335
  Lines       10264    10268       +4
  Branches     1146     1141       -5
==========================================
+ Hits         7965     7970       +5
+ Misses       2299     2298       -1

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

Are we going to draft the successive milestone release when this gets merged?

That sounds good to me, but I'm unlikely to get to it before Monday at the earliest.


Bonfire: Title Case a Sentence

Challenge Bonfire: Title Case a Sentence has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
please help with this code ,it give true result but I can't pass issue :angry:
this is the issue ::
Return the provided string with the first letter of each word capitalized. Make sure the rest of the word is in lower case.
this is My code:
    function titleCase(str) {
      var titCa,word,firstLitter;
      word="";
      str=str.toLowerCase(' ');
      titCa=str.split(" ");
      for(var i=0;i<titCa.length;i++){
        titCa[i]=titCa[i].split("");
      firstLitter =  titCa[i][0].toUpperCase('');
        titCa[i][0]= firstLitter;

     word= word+" "+ titCa[i].join('');
      }

 return word;
}

titleCase("sHoRt AnD sToUt");

Your codes create an extra space in front of the first word.
This is what comes out of the console: => ' Short And Stout'

How to solve please me  more  details>>

@MariamShqair I don't think this is a bug in the challenge. In the future, I would ask other students on the live FreeCodeCamp/Help chat.
In the meantime, I hope this helps:  REPL.IT

@pmbenjamin
I'm trying to solve this issue again and again and it give me the same problem , It put the sentence between ' 'single quotation marks..
:tired_face:

Finally it's worked the final code:
function titleCase(str) {
var pieces = str.split(" ");
for ( var i = 0; i < pieces.length; i++ )
{    pieces[i]=pieces[i].toLowerCase();
var j = pieces[i].charAt(0).toUpperCase();
pieces[i] = j + pieces[i].substr(1);
}
return pieces.join(" ");}
titleCase("sHoRt AnD sToUt");

@MariamShqair Please close this issue if your problem is resolved. Thanks.

function titleCase(str)  { return str.split(' ').map(function(a){return a.toLowerCase();}).map(function(a){return a.replace(a[0],a[0].toUpperCase());}).join(' '); }


fix b.transform('sheetify/transform') in README

Current coverage is 95.89% (diff: 100%)

Merging #101 into master will not change coverage

@@             master       #101   diff @@
==========================================
  Files             2          2
  Lines           146        146
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            140        140
  Misses            6          6
  Partials          0          0

Powered by Codecov. Last update 5ca8973...504e985

yep yep, thanks @sethvincent !


Problems with module import when running RTD inside docker container

Details
I am trying to run RTD inside docker container. I have spent countless hours of trying making it work. I have succeeded once, but then run into python version issue when building the documentation.
I am using docker-compose file originally based on https://hub.docker.com/r/moul/readthedocs/ and then customized (the base image is now python:2.7-stretch).
I believe everything is installed correctly, but then when RTD is started I get
  File "/app/readthedocs/config.py", line 1, in <module>
    from readthedocs.settings.dev import *
ImportError: No module named readthedocs.settings.dev

That configuration file is a part of the project and is being added from the host system when the image is built (as /etc/default/rtd-config.py) and then copied to app root when installation script is executed
cp -f /etc/default/rtd-config.py $APPDIR/readthedocs/config.py

I know this is not an application related issue but more an environment setup issue, but I will greatly appreciate any help.
The project is available here for anyone willing to help.
Thank you
Expected Result
I'd like to have it up and running.
Actual Result
It does not work and the container does not start. I even tried removing that step, but then the app did not launch since configuration settings were empty.
PS: I have noticed the docker build images, but is there an image just for running RTD?

PS: I have noticed the docker build images, but is there an image just for running RTD?

No, we don't run the application with docker in production.

Now the problem is
  File "/venv/lib/python2.7/site-packages/django/conf/__init__.py", line 129, in __init__
    raise ImproperlyConfigured("The SECRET_KEY setting must not be empty.")

and it is driving me crazy :)

@kfrajtak looks like you should change the secret_key setting with something safe. Also, we will be deprecating python 2 very soon.

I have fixed the problem! The config file was not loaded. Please, close this issue. Thank you
BTW is there a way how to find out why my build is not working (i.e. log files, etc.)?
Thank you

We save all the output on the db on the Build model, we also do some logging, that should be a django thing to see where the logs are saved.

Thank you a lot.