Update Sailthru's JS to latest version and include it on all pages

Addresses: https://artsyproduct.atlassian.net/browse/FX-1812
TL;RD: In order to leverage more features in Sailthru (browse abandon, abandoned cart emails, etc.), we need to update the JS snippet that we're pulling in on our pages.
In addition... these features aren't useful unless we actually include it on all of our pages.
This PR:

Updates desktop and mobile web to include the Sailthru JS snippet on all pages.
Includes config options for the SAILTHRU_CUSTOMER_ID. Will need to add this to staging and production for the events to actually fire.

There are a couple of places where we're using this JS to explicitly track something, and many places where we include specific sailthru meta tags. It doesn't look like the API for any of this has changed in the new version, but I'll monitor.
I verified that we're successfully authenticating using a correct customer ID, but won't be able to verify pageviews/etc. until we specify allowed domains in Sailthru's UI.
Some notes:

This script is loaded async to hopefully minimize any perf impact (but it's something we need to be aware of).
We don't intend to use this script/Sailthru's functionality to explicitly send track calls, as we use Segment for that. However, whatever default tracking Sailthru includes in this script should be enough to allow our marketing team to have greater freedom when constructing their campaigns.

Follow ups:

Remove places where we explicitly INCLUDE_SAILTHRU in force's code, and potentially remove the custom tracking calls that we have (in case they can be replaced by the defaults).

Warnings

:warning:
The staging version of Metaphysics has breaking changes, this might not affect your build, but could mean your queries don't work the way you expect. See below for breaking changes:

:warning: Type for argument 'ids' on field 'Query.submissions' changed from '[ID]' to '[ID!]'
:warning: Argument 'user_id: [ID]' was removed from field 'Query.submissions'
:warning: Field 'additional_info' was removed from object type 'ConsignmentSubmission'
:warning: Field 'artist_id' was removed from object type 'ConsignmentSubmission'
:warning: Field 'authenticity_certificate' was removed from object type 'ConsignmentSubmission'
:warning: Field 'created_at' was removed from object type 'ConsignmentSubmission'
:warning: Field 'dimensions_metric' was removed from object type 'ConsignmentSubmission'
:warning: Field 'edition_number' was removed from object type 'ConsignmentSubmission'
:warning: Field 'edition_size' was removed from object type 'ConsignmentSubmission'
:warning: Field 'location_city' was removed from object type 'ConsignmentSubmission'
:warning: Field 'location_country' was removed from object type 'ConsignmentSubmission'
:warning: Field 'location_state' was removed from object type 'ConsignmentSubmission'
:warning: Field 'minimum_price_dollars' was removed from object type 'ConsignmentSubmission'
:warning: Field 'user_id' was removed from object type 'ConsignmentSubmission'
:warning: Field 'asset_type' was removed from object type 'ConsignmentSubmissionCategoryAsset'
:warning: Field 'gemini_token' was removed from object type 'ConsignmentSubmissionCategoryAsset'
:warning: Field 'image_urls' was removed from object type 'ConsignmentSubmissionCategoryAsset'
:warning: Field 'submission_id' was removed from object type 'ConsignmentSubmissionCategoryAsset'
:warning: Type for argument 'input' on field 'Mutation.addAssetToConsignmentSubmission' changed from 'AddAssetToConsignmentSubmissionInput' to 'AddAssetToConsignmentSubmissionInput!'
:warning: Type for argument 'input' on field 'Mutation.createConsignmentSubmission' changed from 'CreateSubmissionMutationInput' to 'CreateSubmissionMutationInput!'
:warning: Type for argument 'input' on field 'Mutation.updateConsignmentSubmission' changed from 'UpdateSubmissionMutationInput' to 'UpdateSubmissionMutationInput!'

  Generated by :no_entry_sign: dangerJS against 78cc40209e22321422ef198831120cb7b2885a83

Our front-end event architecture utilizes Segment as a broker. We try to avoid adding third party JS directly to our site to reduce maintenance costs. I've actually ripped out a lot of third party JS includes over the years – Google Ads, FB, Criteo, etc. because Segment has integrations for those.
We send all front-end events, including page views, to Sailthru via Segment. These can already be used in Lifecycle Optimizer currently – e.g. to trigger Abandoned Cart emails. You do not need the Sailthru JS script for those.
If the Browse Abandon LO requires this JS then I guess we can integrate it. I don't see why it should though, given that Sailthru receives all browsing data already, albeit with a slight lag perhaps.

Sailthru tech support just confirmed they can't support the Browse Abandon feature specifically without this JS, something to do with the fact they have their own session system distinct from Segment's. Abandoned Cart works fine right now however.
So we can merge this in. In terms of confirming pageviews on the Sailthru side, they usually show up on the Content Catalog next to an artwork url.


Allow Owners but not members

We use to be able to Create a Group > add an owner without being a member.
Then the owner could manage members though Groups on Outlook Web/front end and would not receive the group emails. Which was good for delegation of admin without dashboard access.
In this later version, this seems to have been removed, although you can still apply it in the Admin dashboard.
I see in the later version they've split Members & Owners under People > Groups but the Group doesn't appear if you are only a Owner, You have to be a Member or Member & Owner for the group to appear.
I don't know whether this is a bug or by design.
Either way its very annoying.

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

ID: 96891122-3c0b-1335-35be-556b5681b0ca
Version Independent ID: 7c567123-f4c0-497c-7d52-21f8bef66040
Content: Add or remove members from Office 365 groups
Content Source: Office365-Admin/create-groups/add-or-remove-members-from-groups.md
Service: o365-administration
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@andyleese - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


Allow BSLs to specify restic repo prefixes, error if the prefix can't be determined

Fixes #939
Hold for merge until after v1.1
This PR does two things:

If Velero can't determine the restic repo identifier for a given restic repository/BackupStorageLocation, it leaves the field blank and sets the repo to "NotReady" rather than putting an invalid repo identifier in the field and attempting to initialize it
Addresses #939 by allowing an optional resticRepoPrefix config key/val to be specified in a BSL, which if specified takes precedence over the auto-determined one.

Coupled with previously merged PRs around failing faster if a restic repo can't be initialized, this should make the UX much clearer when something goes wrong around creating a repo, particularly for third-party object store plugins.

@nrb @carlisia @prydonius this one is ready for review.


Search is not working

Not?

It's working for me. Please provide a specific search term that isn't working, and include a screenshot of any errors you see.


How to customize the foreground notification UI

This plugin currently does not support customizing the notification UI.

Oke thank you. I have tried it by fork this lib and customize the notification and it work. Thank you for your response


Add README Information; Closes #20

The README only included some badging and was not informative to consumers
of the application. This adds some information about the what and why of the
repo.

Simple Merge


error with webpack v5

[webpack-cli] Promise rejection: Error: chunk.files was changed from Array to Set (using Array method 'splice' is not possible)

180
[webpack-cli] Error: chunk.files was changed from Array to Set (using Array method 'splice' is not possible)
181
at Set.set. [as splice] (/home/runner/work/reacnetgenerator/reacnetgenerator/reacnetgenerator/static/webpack/node_modules/webpack/lib/util/deprecation.js:117:10)
182
at /home/runner/work/reacnetgenerator/reacnetgenerator/reacnetgenerator/static/webpack/node_modules/style-ext-html-webpack-plugin/lib/removeFile.js:10:19
183
at Set.forEach ()
184
at deleteFileFromCompilation (/home/runner/work/reacnetgenerator/reacnetgenerator/reacnetgenerator/static/webpack/node_modules/style-ext-html-webpack-plugin/lib/removeFile.js:7:22)
185
at Set.forEach ()
186
at StyleExtHtmlWebpackPlugin.emitCallback (/home/runner/work/reacnetgenerator/reacnetgenerator/reacnetgenerator/static/webpack/node_modules/style-ext-html-webpack-plugin/lib/plugin.js:134:26)
187
at _next0 (eval at create (/home/runner/work/reacnetgenerator/reacnetgenerator/reacnetgenerator/static/webpack/node_modules/tapable/lib/HookCodeFactory.js:33:10), :17:1)
188
at eval (eval at create (/home/runner/work/reacnetgenerator/reacnetgenerator/reacnetgenerator/static/webpack/node_modules/tapable/lib/HookCodeFactory.js:33:10), :31:1)
189
at /home/runner/work/reacnetgenerator/reacnetgenerator/reacnetgenerator/static/webpack/node_modules/html-webpack-plugin/index.js:319:11
190
at processTicksAndRejections (internal/process/task_queues.js:93:5)
191
Done in 16.85s.

The errors comes from https://github.com/numical/style-ext-html-webpack-plugin/blob/b6dda6bac6f251ffefa9a373184b1f0bfd1e368b/lib/removeFile.js#L10

I was able to fix it locally by replacing L8-11
https://github.com/numical/style-ext-html-webpack-plugin/blob/b6dda6bac6f251ffefa9a373184b1f0bfd1e368b/lib/removeFile.js#L8-L11
with this
if (Array.isArray(chunk.files)) {
  const fileIndex = chunk.files.indexOf(filename);
  if (fileIndex > -1) {
    chunk.files.splice(fileIndex, 1);
  }
} else {
  chunk.files.delete(filename);
}

This should allow for backward compatibility with older versions of Webpack that use an Array for chunk.files as well as Webpack 5+ that uses a Set.
@numical happy to send pull request if you’re still maintaining this.

@adamchal - be warned that I found further problems with Webpack 5 in my regression tests.  However if it works for your use case, excellent!
Please note the next comment.

Hello - I'm sorry to say that this plugin is no longer being maintained. I have tried to close all issues ahead of doing this but instead have found that the issue list just keeps growing. I simply do not have the time to keep up. Sorry.
Please keep an eye on the README where I will record anyone who forks this repo with the intention to supporting it in the future.


[CMAKE] Add cppcheck target to CMakeLists.txt

Scope & Purpose
Adds a cppcheck target to CMakeLists.txt; enables the user to, after running cmake, run make cppcheck to get their source checked using cppcheck (if that tool is installed)

There is also an option to, while CMake builds to project, to just run cppcheck on every file and error out if there's an issue.
That approach would have the obvious downside of slowing down compilation by quite a bit.

The target now has the problem that one has to run a full build to run cppcheck, because some generated files do not exist yet, and cppcheck insists on loading them from the build database; not sure yet whether one can ignore those somehow (-i doesn't work).


Cannot match implicit arg using constructor

The second match fails with 'syntax error', but the fact that the argument is implicit should not make a difference.
noeq
type t2 = | Mk2 : _:int -> t2

let test2 (x:t2) : int =
  match x with
  | Mk2 0 -> 0
  | _ -> 1

noeq
type t1 = | Mk1 : #_:int -> t1

let test1 (x:t1) : int =
  match x with
  | Mk1 #0 -> x
  | _ -> 1

To fix this, we should probably move the aqual in PatVar into the list in PatApp. Currently PatConst cannot be marked implicit.


Controls not displaying correctly if hidden on load

I am using fairly standard Bootstrap 2.3 tabs to hide a tab pane containing a ME.js audio or video player.
When you click the tab and reveal the player, the duration bar doesn't take up as much width as it should:

If you load the page so that the tab contents are displayed on page load, the player displays fine.

I am using the latest stable release of ME.js (2.13.1), and I've witnessed this behavior in Mac Chrome and Firefox. The <audio> tag pictured above is fairly standard as far as I can tell:
<audio id="resource-preview-13" width="400" height="30" style="width: 400px; height: 30px;" class="preview" controls>
  <source src="/admin/resources/13/web_mp3" type="audio/mpeg">
  <source src="/admin/resources/13/web_oga" type="audio/ogg">
  <object type="application/x-shockwave-flash" data="/assets/mediaelement_rails/flashmediaelement.swf" width="400" height="30">
    <param name="movie" value="/assets/mediaelement_rails/flashmediaelement.swf">
    <param name="flashvars" value="controls=true&file=/admin/resources/13/web_mp4">
    <img alt="No audio playback capabilities" height="30" src="/assets/disabled-audio-player.png" width="400" />
  </object>
</audio>

<video> tags are behaving pretty similarly.
You'll see that I've even tried forcing in the width and height via tag attributes and inline CSS.
Any ideas on how I could fix, or is this a bug? I was using a year-old version of ME.js (used in https://github.com/tobsch/mediaelement_rails) before trying to upgrade tonight, and it was working OK. Not sure what all has changed since then, but I may need to revert until this can be resolved.
Thanks for this fantastic library. I hope I can assist and upgrade soon!

@chrisdpeters Closing old issues. If you'd like to reopen this, let us know. Thanks!


[KEBAB] add example to create bot

Add example to create a bot (lots of people ask for help on discord, now we can point them here)
Update README on bot example (code switched to api key, but readme did not)

waiting on you @overlisted

ok, changing back now

@overlisted restored

All green checks make me happy :)


[bug]: SuiString does not have epoch property (JSON RPC)

When trying to call a public read-only function, dev inspect works fine without epoch params, but fails with a non null value:
sample code:

    cfg = SuiConfig.user_config(rpc_url="https://rpc-mainnet.suiscan.xyz:443")
    client = SyncClient(cfg)

    sender = SuiAddress(
        "0x0000000000000000000000000000000000000000000000000000000000000000"
    )

    txn = SyncTransaction(client=client)#, initial_sender=sender)

    txn.move_call(
        target="0x7f6ce7ade63857c4fd16ef7783fed2dfc4d7fb7e40615abdb653030b76aef0c6::staked_sui_vault::afsui_to_sui",
        arguments=[
            ObjectID(
                "0x2f8f6d5da7f13ea37daa397724280483ed062769813b6f31e9788e59cc88994d"
            ),
            ObjectID(
                "0xeb685899830dd5837b47007809c76d91a098d52aabbf61e8ac467c59e5cc4610"
            ),
            SuiU64(1000000000),
        ],
    )

    tx_bytes = txn.build_for_inspection()
    inspect_tx = _DebugInspectTransaction(
            sender_address=sender, tx_bytes=tx_bytes, epoch="500"
        )
    req = client.execute(inspect_tx)

    if req.is_ok():
        result = TxInspectionResult.factory(req.result_data)

This generates an error:
build_parm = <pysui.sui.sui_types.scalars.SuiString object at 0x7f24f7944550>
api_parm = SuiApiParam(name='epoch', schema=SuiJsonString(type='string', type_path=['BigInt_for_uint64', 'string']), required=False, description='The epoch to perform the call. Will be set from the system state object if not provided')

    @versionchanged(version="0.24.0", reason="Moved from list to dict for RPC params")
    @deprecated(
        version="0.65.0", reason="Transition to pysui GraphQL SuiClients and QueryNodes"
    )
    def __validate_parameter(
        build_parm: Any, api_parm: SuiApiParam
    ) -> Union[str, SuiRpcApiInvalidParameter]:
        """Validate the specific parameter."""
>       att = getattr(build_parm, api_parm.name)
E       AttributeError: 'SuiString' object has no attribute 'epoch'

.venv/lib/python3.10/site-packages/pysui/sui/sui_txn_validator.py:60: AttributeError

It seems that the issue is that the "epoch" is defined as a string, but the SuiString is missing epoch as a key. After adding it as a key, it passes the validator, but then RPC call returns an error:
'error': 'ExecutionError: ExecutionError { inner: ExecutionErrorInner { kind: VMVerificationOrDeserializationError, source: Some(VMError { major_status: FAILED_TO_DESERIALIZE_ARGUMENT, sub_status: None, message: None, exec_state: None, location: Undefined, indices: [], offsets: [] }), command: Some(0) } }'

So it seems possible that it expects U64 as per RPC docs instead of String. After changing the _DebugInspectTransaction epoch type to SuiU64, and adding epoch as an property, running gets yet another error:
{'code': -32602,
 'message': 'invalid type: integer `500`, expected a string at line 1 column 4'}

Any pointers would be appreciated!

You should not be using the private class which is for the SuiTransaction.
Instead, in the builders package exec_builders.py use InspectTransaction. Pass in SuiString("500") as the epoch argument.
If it is still an issue I will address, let me know.

Yes,
it's still the same error, here's the standalone code blurb:
from pysui import SuiConfig, SyncClient, SuiAddress
from pysui.sui.sui_txn import SyncTransaction

from pysui.sui.sui_builders.exec_builders import InspectTransaction
from pysui.sui.sui_txresults.complex_tx import TxInspectionResult
from pysui.sui.sui_types.scalars import SuiString, ObjectID

cfg = SuiConfig.user_config(rpc_url="https://rpc-mainnet.suiscan.xyz:443")
client = SyncClient(cfg)

sender = SuiAddress("0x0000000000000000000000000000000000000000000000000000000000000000")

txn = SyncTransaction(client=client, initial_sender = sender)

txn.move_call(target="0x7f6ce7ade63857c4fd16ef7783fed2dfc4d7fb7e40615abdb653030b76aef0c6::staked_sui_vault::afsui_to_sui",
              arguments = [ObjectID("0x2f8f6d5da7f13ea37daa397724280483ed062769813b6f31e9788e59cc88994d"),
                          ObjectID("0xeb685899830dd5837b47007809c76d91a098d52aabbf61e8ac467c59e5cc4610"),
                          SuiString(1000000000)]
         )

tx_bytes = txn.build_for_inspection()

txn2 = InspectTransaction(sender_address=sender, tx_bytes = tx_bytes, epoch = SuiString("500"))

res= client.execute(txn2)

res.result_data

and the full output. To note, it is clearly using executed_epoch = 601 so it's just not paying attention to the epoch param
TxInspectionResult(effects=Effects(status=Status(status='failure', error='VMVerificationOrDeserializationError in command 0', succeeded=False), message_version='v1', gas_used=GasCostSummary(computation_cost=750000, non_refundable_storage_fee=42712, storage_cost=5259200, storage_rebate=4228488), transaction_digest='FdHhKpL2XRK5hifFneeC1tdqQ3MZBekjgbwLHJiX4rBC', gas_object=GenericOwnerRef(reference=GenericRef(object_id='0xa650dcd8569d4fe66e8d69f4d9382dc4f6795c16e46c3d8f1757537ef8776c28', version=438523917, digest='ABdfUGSo62PSf4YwihzmpiHuS7oVbmAHdiUHKRAP3VWh'), owner='0x0000000000000000000000000000000000000000000000000000000000000000'), modified_at_versions=[{'objectId': '0x2f8f6d5da7f13ea37daa397724280483ed062769813b6f31e9788e59cc88994d', 'sequenceNumber': '438523916'}, {'objectId': '0xa650dcd8569d4fe66e8d69f4d9382dc4f6795c16e46c3d8f1757537ef8776c28', 'sequenceNumber': '1'}, {'objectId': '0xeb685899830dd5837b47007809c76d91a098d52aabbf61e8ac467c59e5cc4610', 'sequenceNumber': '438523916'}], dependencies=['7aZJpvLooTaCg538EwLwotYHuuxHWR9eMjCwsX8d5eaq', '9Zpr9ReEJrwwQYwUt6NyQ71vHWLS69gnPSzNEnFmp2iA'], mutated=[GenericOwnerRef(reference=GenericRef(object_id='0x2f8f6d5da7f13ea37daa397724280483ed062769813b6f31e9788e59cc88994d', version=438523917, digest='6qga3wdBobnwRtN4vro5DEV4KruQssFxi8qhZxrwdZga'), owner={'initial_shared_version': 32696040}), GenericOwnerRef(reference=GenericRef(object_id='0xa650dcd8569d4fe66e8d69f4d9382dc4f6795c16e46c3d8f1757537ef8776c28', version=438523917, digest='ABdfUGSo62PSf4YwihzmpiHuS7oVbmAHdiUHKRAP3VWh'), owner='0x0000000000000000000000000000000000000000000000000000000000000000'), GenericOwnerRef(reference=GenericRef(object_id='0xeb685899830dd5837b47007809c76d91a098d52aabbf61e8ac467c59e5cc4610', version=438523917, digest='G7MCGvpCDRUfRMLquNzGSJtEHFjZccjntuCXvk5ufBMB'), owner={'initial_shared_version': 32347695})], created=[], deleted=[], wrapped=[], unwrapped=[], unwrapped_then_deleted=[], shared_objects=[GenericRef(object_id='0x2f8f6d5da7f13ea37daa397724280483ed062769813b6f31e9788e59cc88994d', version=438523916, digest='H1g6GWaCmLwM5Db4zobEbzm64uiyfoH47UcZG1hTP6Fh'), GenericRef(object_id='0xeb685899830dd5837b47007809c76d91a098d52aabbf61e8ac467c59e5cc4610', version=438523916, digest='ANssAZwEcTQiPCzYZ6QTC7133Uz8dh74n5eQqn7XHnPK')], executed_epoch=601, events_digest='0'), events=[], results=[], error='ExecutionError: ExecutionError { inner: ExecutionErrorInner { kind: VMVerificationOrDeserializationError, source: Some(VMError { major_status: FAILED_TO_DESERIALIZE_ARGUMENT, sub_status: None, message: None, exec_state: None, location: Undefined, indices: [], offsets: [] }), command: Some(0) } }')

Yes, I added the fix in a branch and will be merging and publishing to PyPi soon

Just to clarify. There are 2 issues:

SuiString.epoch is missing, which I think you are addressing in a separate issue.
once 1 is fixed, i still get serialization error above, I'm not exactly sure what's the fix for that.

So, the epoch missing on SuiString is this issue.
First:
The serialization error is not related to pysui as (with the fix) this works:
def epoch_test():
    cfg = SuiConfig.default_config()
    client = SyncClient(cfg)
    txn = SyncTransaction(client=client, initial_sender=cfg.active_address)
    scoin = txn.split_coin(coin=txn.gas, amounts=[1000000000])
    txn.transfer_objects(transfers=[scoin], recipient=cfg.active_address)

    tx_bytes = txn.build_for_inspection()
    istxn = InspectTransaction(
        sender_address=cfg.active_address, tx_bytes=tx_bytes, epoch=SuiString("25")
    )
    res = client.execute(istxn)
    print(res.result_data.to_json(indent=2))

Second:
Even though the above returns success on the inspect, the executed transaction does not match the epoch I passed in (the above test is against devnet by the way). I will research this with Mysten
Finally:
The serialization error is something different that may be related to the target function you are calling.

Ok great, I guess we are uncovering a bunch of things from this rarely used
feature!
On Thu, Dec 5, 2024 at 3:39 AM Frank V. Castellucci <
@.***> wrote:

So, the epoch missing on SuiString is this issue.
First:
The serialization error is not related to pysui as (with the fix) this
works:
def epoch_test():
cfg = SuiConfig.default_config()
client = SyncClient(cfg)
txn = SyncTransaction(client=client, initial_sender=cfg.active_address)
scoin = txn.split_coin(coin=txn.gas, amounts=[1000000000])
txn.transfer_objects(transfers=[scoin], recipient=cfg.active_address)
tx_bytes = txn.build_for_inspection()
istxn = InspectTransaction(
    sender_address=cfg.active_address, tx_bytes=tx_bytes, epoch=SuiString("25")
)
res = client.execute(istxn)
print(res.result_data.to_json(indent=2))

Second:
Even though the above returns success on the inspect, the executed
transaction does not match the epoch I passed in (the above test is against
devnet by the way). I will research this with Mysten
Finally:
The serialization error is something different that may be related to the
target function you are calling.
—
Reply to this email directly, view it on GitHub
https://github.com/FrankC01/pysui/issues/244#issuecomment-2519627422,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM6CQTD5HFPDK3LCPZA3UL2EAGM5AVCNFSM6AAAAABSXCJRDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJZGYZDONBSGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Ivan

I confirmed with Mysten that epoch is ignored so you don't have to provide the argument. Updated example:
def epoch_test():
    cfg = SuiConfig.default_config()
    client = SyncClient(cfg)
    txn = SyncTransaction(client=client, initial_sender=cfg.active_address)
    scoin = txn.split_coin(coin=txn.gas, amounts=[1000000000])
    txn.transfer_objects(transfers=[scoin], recipient=cfg.active_address)

    tx_bytes = txn.build_for_inspection()
    istxn = InspectTransaction(sender_address=cfg.active_address, tx_bytes=tx_bytes)
    res = client.execute(istxn)
    print(res.result_data.to_json(indent=2))
    ```

Your issue, however, is still related to your move call and not pysui. I would check the arguments of the function you are calling to ensure they are corrects as well as adding any `type_arguments` it requires.

Yea, I realized epoch doesn't actually do anything. Thanks for looking into
it!
On Mon, Dec 9, 2024, 11:02 AM Frank V. Castellucci @.***>
wrote:

Closed #244 https://github.com/FrankC01/pysui/issues/244 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/FrankC01/pysui/issues/244#event-15581375286, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM6CQVK7TCGMTMLGWQP2ED2EV2FZAVCNFSM6AAAAABSXCJRDWVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGU4DCMZXGUZDQNQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Link Under Test Website is returning an empty page

In the setup the link under Test Website returns an empty page.
Following the steps, I receive the same blank page in my own deployments.

UPDATE: It comes up in Chrome and Edge. Firefox still failing.

Thanks.  Let me verify and make changes for Firefox.  Did test only with Chrome/Edge


Create 0000-01-02-sepi5533-curo.md

this is second step complete

test pull request

close request


Being assigned 1-2-1 users when not logged into New Dot Reported by Guides Team

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

signed out of New Dot yesterday around 6 PM EST

Expected Result:
To not be assigned a user while out of New Dot
Actual Result:
I was assigned several users from 6pm until 5 am this morning
Workaround:
unknown
Platform:

Where is this issue occurring?

Web

Version Number:
Reproducible in staging?: need reproduction
Reproducible in production?: need reproduction
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: Guides Team
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1668102491670949
View all open jobs on GitHub

Not overdue. Just assigned last night.

Not sure this is actually a bug. Chatting with the guide in the slack thread where this was raised. If it is actually a bug I will re-open.


A double space in the Beginning may count as a piece of evidence and will display wrong results.

is this still open, or did @Hiimphteve 's trim fix it and it just hasn't been resolved? That would have been my first solution as well


Add leaflet example

Added leaflet widget as an example.
The example

shows a map in a xul window
shows how to add and remove layers
contains checkboxes for showing the layer control and scale
reponds to mouse clicks by adding markers to the map

I intended to make the widget reflect more of leaflet's API, but found it quite hard to come up with a generic way to do so, in particular because different applications would probably want a different level of Py <-> JS communication. I was thinking of creating a dictionary in JS with GUID->JavascriptObject entries and using the GUIDs as a reference to these objects in python, but I don't know if that's the way to go. Maybe you have some ideas on that. I am going to leave the example as it is for now.

This is great. I think we should merge this as soon as that one comment is covered.
In the mean time, in light of maybe a flexx.ui.MapWidget how do you think that leaflet compares to other frameworks such as those listed here: http://wiki.openstreetmap.org/wiki/Frameworks?

Depends a bit on the goal imho. It seems current development leans towards rendering vector maps with WebGL, which makes sense, but I don't have the impression it's really there yet.
For 'simple' tile maps, the only reasonable options appear to be openlayers, leaflet and Google maps. See e.g. http://wiki.openstreetmap.org/wiki/Deploying_your_own_Slippy_Map
I haven't really looked into google maps, but between openlayers and leaflet I found in my little research:

openlayers is GIS oriented with an extensive API, fairly steep learning curve, and large code base (~1M)
leaflet is web oriented, focused on simplicity, extensible with plugins, easier to get started with and smaller (~150K)
As a general advice I found: if you want to build an extensive mapping application, take the openlayers pill, if you just want to fling together some mapping stuff, leaflet is your best friend. Opinions differ: http://gis.stackexchange.com/questions/33918/should-i-use-openlayers-or-leaflet

For a vector based approach, we might have a look at https://github.com/mapbox/mapbox-gl-js

It seems current development leans towards rendering vector maps with WebGL, which makes sense, but I don't have the impression it's really there yet.

I can imagine that geometry shaders that come with WebGL2 will make this more feasible.

Thanks for the awesome contribution!

You´re very welcome. I hope to able to contribute some more in the future. If you have a specific task that you would like me to take on. Let me know.

I can imagine that geometry shaders that come with WebGL2 will make this more feasible.

Ah. You probably know more about the details of the actual rendering than I do. I don't know if technical feasibility is still an issue. I meant "ready" in the sense of e.g. availability and adoption of a standard for http://wiki.openstreetmap.org/wiki/Vector_tiles and stuff like that.


Handle batchSizes correctly

FerretDB version
Refs #79.
package.txt: docker-local
commit.txt: 4a69d977535fc0a67e523095b4ddb22cf26d9079
branch.txt: batchsizes
version.txt: v1.2.0-32-g4a69d977-dirty
Backend
PostgreSQL
Environment

OS: Ubuntu 22.10 x86_64
Deployment: v1.2.0-32-g4a69d977-dirty
Deployment details: 23.0.3

What did you do?
See repro in #2761
What did you expect to see?
Cursors using the batch size correctly.
What did you see instead?
Cursor returning all documents regardless of a batch size.

Duplicate of #1733


Truncate search results

Is your feature request related to a problem? Please describe.
Truncate search strings that are exceeding the size of search box
Describe the solution you'd like
Add ellipse ...
Describe alternatives you've considered
N/A
Additional context

:tada: This issue has been resolved in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Write high-level introduction to MagicBox and how it's built

Summary
Write one or two page doc to explain MagicBox projects, their relationships, and how it all fits together (complements #15)
Background
There's a lot of repos and they're listed in the project index, but that page isn't helpful. It gives us a bunch of "dots" but not how they're connected. It's confusing for a newcomer to understand how the projects fit together, what each one does, and why they are important.
Details
A one or two page document to explain that pipeline, or flow, of how the repos are connected and how the data is processed will be helpful. I am writing a short document to give that explanation / introduction. The purpose is that we could share this with anyone and they could get a brief understanding of the project and how it fits together.
I plan to use past Medium articles and some of the info on the unicefstories.org site to create this.
Outcome
Easier to understand the project, relationship among repos is more clear, easier to introduce someone to the projects

I shared a survey about MagicBox with people I met abroad in Europe over the last two weeks. From their feedback, I've accumulated a "question queue" of things people were confused by or were curious about (also found in the documentation project board). I plan to map out a strategy to answer these questions over the next week.
Big picture

What is MagicBox?
Is MagicBox a public API?
How do I access MagicBox?
What's the two-year vision for MagicBox?

Use cases

What data does MagicBox provide?
Who would find MagicBox useful?

Who are our current users?

What are some live examples that use it?

What use cases has MagicBox actually been used for? Zika?
Are we able to track the Zika virus risk factor now? Or is it manually generated with our current data?

Technical

How to integrate more countries?
What are the "core" repos to MagicBox?

Which repos play a supporting role to MagicBox, but aren't critical?
How do the projects come together?

Where does the data come from?

Contributing / community

How do I contribute?
What actual development work needs to be finished?

I reviewed many of the Medium blog posts that @mikefab has written and I think a lot of the high-level overview exists throughout those posts. I want to help lift some of those things out of Medium and into documentation proper so they're public and centralized to a single place. This will likely be some of my work for the rest of this week and into next week.

Today, I took a deep dive into one of the longer, private posts on Medium and did a cross-section with the questions generated above. I grouped my research and questions into content sections listed below. Each section represents a unique article.
About MagicBox

What MagicBox is for
How do projects come together

MagicBox data

Where does data come from?
How do we get / ingest data?
Managing large sets of school data

Our CSV files are huge!

Data ingestion rules
Project Connect

Ruby on Rails CRUD (create, read, update, delete) app to upload data
PostgreSQL with single table
File name dichotomy
Data inserted when imported
* How we add new columns to school data

How to integrate more countries (help us!)

Data validation

magicbox-latlon-admin-server

Why we need to validate data
How we validate data today

Serving data

Schools / mobility served via magicbox-open-api

School data is sensitive (e.g. terrorism)

API has three endpoints

Response: country list
Response: schools in country
Response: school

API problems in hindsight
Access control to-dos

Data visualization

Visualizing data in OSM Leaflets
Why we map the way we do
Authentication: Auth0

Tokens with API queries

Documentation progress is in #21, #22, #23, and #24. Two candidate articles remain, one for access control and one for rules. Aiming to finish this by the end of the week.

Rules are explained in #25 and access control in #26.
Marking ticket as blocked until @mikefab returns and has a chance to review the above PRs. Once he reviews the existing content, we can look at expanding our coverage and writing about some of the missing sections.

Each article on the current Docs is wonderful in itself - so detailed, carefully curated and thoughtfully visual. However, as of now, the Docs as a whole is a bit difficult to read because there is no clear pointer as to how each article connects to one another or how should a reader flows from one to another. (If you approach the Docs from the Navigation Menu on the left, you'll get what I mean.)
However, I think the outlines @jwflory has above (on Feb 8 and Feb 21) provide an excellent flow, and they have been broken into tasks here and here. Once the tasks there are largely completed, the Docs will read more intelligibly.


Permission denied(public key)

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:1.68.1
OS Version:Windows 11

Steps to Reproduce:
1.
2.
My issue:
 I generated a ssh key and added it to my Github account. Then I cloned my repository in windows terminal. It worked fine. Then I tried to clone the same repo in the vscode. It shows the error "Permission denied(public key)". Then I searched Internet and got many related results. But those results didn't solve the issue. How can I rectify the issue? I have attached the screenshots of (error message) and (successfully cloned repo in windows terminal). Kindly solve this issue. Thank you in advance.

Now I installed VSCode in Debain GNU/Linux. Now the git commands are working fine. But I don't know why the error "Permission denied(Public key)"has been shown in vscode in Windows 11. Any way thanks. Now I'm going to close this issue.


Climatisation not working

Hi Till,
great job in creating this. I am not an expert in coding, but I did manage to get it running thanks to your good instructions :-).
I started looking for this kind of code because I want to be able to trigger the auxiliary heating (Standheizung) from outside the app. Ideally I would like to be able to us an iOS shortcut action to trigger this, but for now I am happy if I get it up and running using my computer.
I have a Tiguan from 2019 (Model 2020).
When I want to start the climatisation I get the following error:
python3 weconnect-cli --username<EMAIL_ADDRESS>--password ................ -s **** set /vehicles/WVGZZZ5NZLW......./controls/climatisation start
id /vehicles/WVGZZZ5NZLW....../controls/climatisation cannot be set: Could not control climatisation (Not Found - Upstream service responded with an unexpected status. If the problem persists, please contact our support. - Please retry in a moment)
Using the Volkswagen App I am able to start the auxiliary heating.
Is there anything you can do to make this work or am I doing something wrong?
Thank you in advance.

controls for auxiliary heating are in auxiliaryHeating and not climatisation

You can send start or a number representing minutes.
and you have to add the —spin for the auxiliary heating

Try to use the list -s command to see what is available.

Hi Till,
it shows me this list:
/vehicles/WVGZZZ5NZLW313335/controls/access
/vehicles/WVGZZZ5NZLW313335/controls/climatisation
/vehicles/WVGZZZ5NZLW313335/controls/honkAndFlash
/vehicles/WVGZZZ5NZLW313335/controls/wakeup
/controls/spin

Hi Till,
any alternative suggestions I could try?
BR

I have a suspicion. Can you have a look if the controls appear after you configured a timer for the auxiliary heating?

Hi Till,
sorry for maybe a stupid question: Do you mean with timer the departure time or how long the heater is supposed to run?
Should I set the timer in the car or using the Volkswagen App?
Or using your utility? If so, how would I do that?
And what to you mean with "if the controls appear"? Where are they supposed to appear?
Does that referred to the "list -s" command?
Sorry 🙈

No reason to be sorry, my message was cryptic!
I meant setting a departure timer in the car. It should be enough to have an entry, it does normale to be enabled.
Afterwards there is a chance that a new entry for auxiliary heating might appear in the controls section. The reason is: I only add the controls when I believe the car has auxiliary heating. And I detect that by looking for the Timer Section.
If that still does not help I will need to do some changes in the code.

Hi Till,
thank you for the explanation.
So I set a timer using the volkswagen app for tomorrow morning.
then I did the "list -s" command again and got the following response:
2023-12-06T21:01:30+0100:ERROR:/vehicles/WVGZZZ5NZLW313335/domains/vehicleHealthWarnings/warningLights/warningLights/0xA227: warning light icon is not a base64 encoded png
/vehicles/WVGZZZ5NZLW313335/controls/access
/vehicles/WVGZZZ5NZLW313335/controls/climatisation
/vehicles/WVGZZZ5NZLW313335/controls/honkAndFlash
/vehicles/WVGZZZ5NZLW313335/controls/wakeup
/controls/spin
I don't think the warning light has anything to do with it - at least I hope so^^

@klucon2 Perhaps it would be better to not post your full VIN?
@tillsteinbach: I just came across this program and decided to try it. Thanks for putting this out there! :heart:
With regards to this issue. I was under the impression turning on climatisation didn't work for either. But turns out I was simply looking a cached results. Because I had this issue open in a browser tab I would like to add a positive experience. Climatisation works for me. On a ID.3 Business with the ID.Software 2.4 on it.
Command I issued, same as in the opening post:
weconnect-cli set /vehicles/$VIN/controls/climatisation start

So your current implementation does work for some.

Hi Both,
@stejoo  thank you.
@tillsteinbach apparently there are differences depending on the models. Do you think you can look into where the difference lies?


Can't Query 70B LLama Model With Large Prompt Inputs, Even Though It Was Finetuned With These Sizes

System Info
I am using the docker image.  I have tried the latest version as well as older versions.  I am using an 80GB A100.
What I am trying to do is inference a Llama 70B based model.  I am trying to do so with bitsandbytes int4.  I am able to do so for small inputs, but I want inputs over 2k.  Trying to do so leads to OOM issues.
What I am especially confused about is that I finetuned this model using Qlora and flash attention, both of which are used here(at least is my understanding).   In that case, I did not get OOMs and had additional processing associated with training.
I have tried changing many flags to no avail, so I am hoping for some help.
Small prompts with large outputs seem to work still.
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction

Use a Llama 70B model with A100 80GB using bitsandbytes int4
Use a large prompt(2k or more)
OOM

Expected behavior
I expect this to work since finetuning was possible with a full 4k context.

I will add that I have tried many different docker image versions and they all had this issue.
Oddly enough, I tried a version I had on another server with 2 rtx 3090s and it is working.  I am gonna try to export the docker image and import it and see if that fixes it.  If it does, then there is something broken in the docker image.  Perhaps some upstream dependency is broken.
Otherwise, if the issue persists, perhaps it is a CUDA issue on my end.  Or perhaps it is a hardware issue with A100s vs RTX 3090s.  Or perhaps its something to do with sharding vs not sharding.  I will update.


Proposal for a git 'command'

Problem Description
The idea is that forcing the models to use git or any other versioning system will trigger more of the programming training and thus improve logical reasoning? Ideally, the agents would use git history to determine if they're going in the right direction, and possibly undo bad decisions. And the ultimate version would be having the agents be able to interpret git history of existing projects
Proposed Solution
I propose making a 'VersionControl' command with these (hopefully self-explanatory) functions:

git_init(optional readme_content: string) # I assume the model will like to use the readme as a sort of "description of objective"?
git_write_and_commit(text: string, mode: (ReWrite,Append), commit_message: string)
git_cherrypick(commit_hash: string)
git_history() # show last 10 commits
git_search_history(filter: string) # show last 10 matching commits, different command as to not confuse the model into using filters when not needed
git_list_commit(commit_hash: string)
git_diff_last_commit() # will feed the model peacemeal acc to context size
git_status()
git_commit_as(agent_name: string)
git_switch_branch(branch_name: string) # switches or creates
git_ignore(regex: string)

git will be local only (no pushing), all commands will git_init if not already initialised
I also assume, this would be primarily used for code & (story)writing. (off-topic, for code, a companion 'CodeTools' command containing stuff like 'cloc' and linters)
Alternatives Considered
No response
Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

This looks interesting.
Can this be somehow be combined with the concepts of recursion and backtracking? / similar to the tree of thoughts paper?
https://arxiv.org/abs/2305.10601
There are two implementations:
Official: https://github.com/princeton-nlp/tree-of-thought-llm
unofficial: https://github.com/kyegomez/tree-of-thoughts / with a monte-carlo approach also.


[Vue warn]: Failed to mount component: template or render function not defined.

Error when parent component finish rendering. Last version works perfectly.

Hi @mateusgrubermade ,
this issue is fixed. it was a problem with the compiler.
you need to update to v0.3.1
Thanks,


[3.x] Make overridden properties link to parent definition

3.x version of https://github.com/godotengine/godot/pull/55559.

Thanks!

This makes the help page look badly on small screens or when scaled (im on 1920 x 1080 with 125% scaling)
Without this PR

With this PR


Templates

Allow the Collection Creator to choose the type of its New Collection from a list of Template Collections
Examples :

Default Collection (current one !)
Collections with inside means to exchange NFTs

via Royaltie fees (multiple template possible)
via Whitelists

Generative NFT Collections (10000 NFTs at a time)
Personal Collection : NFT dedicated to one account, not transferable (for POAP for exemple)
etc...

Use Cases :

Collection Creator to choose Template from a list  while creating a new Collection
Kredeum Admin to add a Template Collection (optional, as it can be done manually or via scripts)

TODO

Add Template choice in Create Collection popup  (and optionally template addition form)

Design
HTML Integration
Svelte Integration

SmartContract: add templates list to NFTsFactory to choose from when cloning, and develop various NFT templates

done with NFTsFactory V2


Mono: Error message about getter/setter flipped

Godot version:

3.2.3.stable.mono.official
OS/device including version:

Windows 10 x64
Issue description:

When exporting a get-only property, the error message that gets printed mentions this wrongly:
"modules/mono/csharp_script.cpp:2618 - Write-only property (without getter) cannot be exported[..]"

On it.

With get-only did you mean "Read-only"?

yes, meaning a property with only a getter.

I was looking through the code and noticed the following:
if (!property->has_getter()) {
    #ifdef TOOLS_ENABLED
    if (exported) {
        ERR_PRINT("Read-only property cannot be exported: '" + MEMBER_FULL_QUALIFIED_NAME(p_member) + "'.");
    }
    #endif
    return false;
}
if (!property->has_setter()) {
    #ifdef TOOLS_ENABLED
    if (exported) {
        ERR_PRINT("Write-only property (without getter) cannot be exported: '" + MEMBER_FULL_QUALIFIED_NAME(p_member) + "'.");
    }
    #endif
    return false;
}

So, apparently the code logic says you can't export a variable that do not have a setter (a get-only property), probably because you wouldn't be able to import it back.
The only thing that caught my attention in the code snippet was that the text without a getter probably should be a without a setter, right?

I'd opened a PR with the respective fix: https://github.com/godotengine/godot/pull/42414


CLI: add validate option

https://michael-simons.github.io/neo4j-migrations/current/#usage_common_validate

Hello! We've started using this library (thank you very much!) and were looking at various error codes. I am wondering if this issue might resolve our current quandry. It looks like morpheus will always return a success code regardless if the migrations complete successfully or not. Would that be something that using validation would address? We're looking at workaround for now, but wanted to start a conversation here too.

This should be added after the Nestjs migration #14


remove production dependency on pug

-compile pug templates as a part of the build job.
fixes #70

Coverage remained the same at 100.0% when pulling ef9593dd22c303a9b1200bf1b67d2a51b7b5fc29 on fixes-#70 into 34e94bcb33da0c63ac31e7fdf58949f3b36b21cd on master.


Local Descriptors SIFT/SURF

Hi @kasinath , I think we are supporting some features as AUTO_COLOR_CORRELOGRAM, BINARY_PATTERNS_PYRAMID, CEDD..... But this plugin is not supports Feature Sift/surf. So could you instruct me how to make this plugin support 2 features sift/surf?
net.semanticmetadata.lire.imageanalysis.features.local.opencvfeatures.CvSiftFeature/CvSurfFeature

Hello @handsetvn, Currently we only support Color and Edge Directivity Descriptor(CEDD).
Here's what you can do to add your own features -
You can extend the features you want - CvSiftFeature/CvSurfFeature and implement org.elasticsearch.metadatasearch.lire.feature.ImageLireFeature (for example - check out org.elasticsearch.metadatasearch.lire.feature.ImageCEDD)
Finally, add an entry in org.elasticsearch.metadatasearch.index.mapper.image.FeatureEnum , and you should be all set!

@kasinath  I cannot find any way to implement CvSiftFeature/CvSurfFeature. Can you please help me with this issue ?


Shouldn't checking .enabled? on an actor work the same way as on a user?

user = User.find 1
actor = Flipper::Actor.new("User:${user.id}")
user.flipper_id == actor.flipper_id
 => true

name = :my_feature
Flipper[name].enabled? user
 => true
Flipper[name].enabled? actor
 => false

🤯
Shouldn't this work?
I must be doing something wrong here 🤔
Flipper version is 0.20.4

You are using $ to interpolate but in Ruby it is #.
Try this:
actor = Flipper::Actor.new("User:#{user.id}")

Also the separator is a semicolon instead of a colon by default.

@jnunemaker actually a typo in the Issue I opened - I did use a # and the code, and it shows this odd behavior:
> user = User.find 1
> Flipper[:tax_doc_show_2021].enabled? user
=> true
> Flipper[:tax_doc_show_2021].enabled? Flipper::Actor.new("User:#{user.id}")
=> false

@jnunemaker actually a typo when I typed the Issue last night 🤦 - in the code a # was used, and it shows this odd behavior:
> user = User.find 1
> Flipper[:my_feature].enabled? user
=> true
> Flipper[:my_feature].enabled? Flipper::Actor.new("User:#{user.id}")
=> false

Gotcha. Your issue is the colon instead of semi-colon then. Did you change the colon?


feat(TFD-146): Add tcomp jms

Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
There is no component available for the JMS technology.
What is the new behavior?
Starting point for the design-time properties of the JMS datastore, dataset and input/output components.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:
The runtime work is in progress, so the contents of the jsm-runtime_1_1 module are works in progress -- we'd really like to get a review on the jms-definition at this point and include it in the REST API to be consumed by the UI (for design-only at this point).

Hello!  We redid the PR #292 with a "clean" squashed branch.  My apologies for the first one -- it was a mistake on my part to base it off a branch in Talend/components instead of the fork!
As a result we can't see the comments and the diffs from the PR #292, but you can review the last changes from your last comments on PR #292 from the command line:
git difftool --dir-diff pbailly/feature/TFD-146-components-jms 133091a91b23fb4b83d863bca409b13dad9f259d

Pull request test result: failure :x:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: success :star:

Pull request test result: failure :x:

Hello!  We're hitting salesforce limits -- the build worked when master was merged into this branch.  It should be equivalent.


feat: rename photos

Context
Renaming photo right into zoomed photo card
Implemented solution
Text field added to edit name by click on edit button potionned inline after title, new action created in the store to make an HTTP PATCH request and http request handler writed in backend to update name in database
Issues (closes #197 ):

Add a rule on the form to avoid user entering void name


helm: prometheus secrets default is empty map, should be empty list

What did you do?
Used kube-prometheus helm chart as dependency in a tailored monitoring system. In its values.yaml, added a secret for prometheus:
---
kube-prometheus:
  prometheus:
    secrets:
      - etcd-certs

What did you expect to see?
Helm will install and configure the chart without complaining.
What did you see instead? Under which circumstances?
The end result is working as expected, but helm issues a warning:
warning: cannot overwrite table with non table for secrets (map[])
I created a pull request, #1112. Should be an easy fix.
Environment

Kubernetes version information:
Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.3", GitCommit:"d2835416544f298c919e2ead3be3d0864b52323b", GitTreeState:"clean", BuildDate:"2018-02-09T21:51:54Z", GoVersion:"go1.9.4", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.3+coreos.0", GitCommit:"f588569ed1bd4a6c986205dd0d7b04da4ab1a3b6", GitTreeState:"clean", BuildDate:"2018-02-10T01:42:55Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:
kubespray v2.4.0

Closed by #1112


feat(ffprobe): FEAT-0001 get file duration by ffprobe

hello @slhck
Thank you for your work. I just wan to a bit improve the progress tool.
The main reason to get duration by FFprobe is the FFmpeg error log level.
If the ffmpeg log level sets to error so stdout and sdterror pipes will catch only processing output without the info banner.
P.S to be honest it's my first MR please say if something wrong

Thanks for the pull request and helping make this library better.
Some feedback for future PRs:

It would be better for pull requests to only cover small, atomic changes, not multiple changes mixed into one commit.
I see you've made a lot of improvements to the overall handling. Your PR message should ideally motivate those additional changes. (I had to go look at the code to understand what else you changed.)

I will probably merge this with a few added changes.

Merged with smaller changes to fix the types (mypy) and formatting (black).

Wow, great.
To be honest I started from tests to understand how it should works and it was broken, I spend some time to understand your flow and tried to follow by it. After, I started implement the idea to use ffprobe
BTW, thank you very much for your feedback and your time @slhck


feat: Introduce activationTargetValue for MS SQL Scaler

Provide a description of what has been changed
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added

Related to https://github.com/kedacore/keda/issues/2800

/run-e2e mssql*


Ability to decompile VMix graphs

Is your feature request related to a problem? Please describe.
At the moment, we can't decompile .vmix_c files.
Describe the solution you'd like
Since we can now make our own .vmix and .vmix_c files, would it be possible to reverse-engineer and decompile/examine vmix_c files?
Describe alternatives you've considered
No response

It should be fine now.


key_pressed is true for 2 frames

Traced it back to winit: https://github.com/rust-windowing/winit/issues/4048

Re-opening because this might be something that needs handling on the egui side.

I'm seeing this on text input. I can't log into my program because, when I type the user name, each letter is duplicated. So is backspace.
But my other GUI test program does not have this problem. Checking versions on everything now.
Fails (Sharpview metaverse client):

egui 0.30.0
winit 0.30.6

Works (GUI test program ui-mock):

egui 0.30.0
winit 0.30.5

Hm.

Rebuilt ui-mock after a "cargo update", so it now uses winit 0.30.6. It now fails, duplicating each character on input.
So something changed between winit 0.30.5 and 0.30.6
Reproduce by:
https://github.com/John-Nagle/ui-mock.git
cd ui-mock
git checkout wgpu23safe
cargo build --release --examples
cd target/release/examples
./ui-mock


LoL Version detected stuck in 6.9

The application is detecting a 6.9 Lol version while my LoL version is : 6.10.16_05_13_13_53
I'm on Mac 10.11.5 with MBP 15inch Mid 2010.

I have the same issue.

Same here, also on Mac (10.9.5).

Yes, the issue is on Riot's end, they've forgotten to update the realms endpoint which is where Championify gets the LoL version from. This has happened once before, and they should have it updated within the next two days.

Does this affect Championify's functionality other than just displaying the wrong version?

Nope!


Override reply (assertion customer service) doesn't work.

routes.php in this bundle use hard-coded paths. The bundle configuration allows us to specify an ACS URL, but this doesn't override the default /saml/acs route. I need this to be /saml2. How should I approach this problem?
Edit:
I've worked around the issue by manually defining routes (based on the ones from the bundle). I don't think this is a good solution though. In case routes get added or removed in the future, this would break future updates on my end.
saml_metadata:
    path: '/saml/metadata'
    controller: Nbgrp\OneloginSamlBundle\Controller\Metadata
    defaults: {'idp': ~}

saml_acs:
    path: '/saml2' # <--- This is the one I changed.
    controller: Nbgrp\OneloginSamlBundle\Controller\AssertionConsumerService
    defaults: {'idp': ~}
    methods: ['POST']

saml_login:
    path: '/saml/login'
    controller: Nbgrp\OneloginSamlBundle\Controller\Login
    defaults: { 'idp': ~ }

saml_logout:
    path: '/saml/logout'
    controller: Nbgrp\OneloginSamlBundle\Controller\Logout
    defaults: { 'idp': ~ }

@haroldiedema hi!
I have no idea how to customize paths in routes configuration at the moment. But I don't think that manual routes definition (from bundle's origin routes) is a bad idea. I may be wrong, but I've been doing the same many times. :smiley:


Chore(design-tokens): Fix and rename Vite config

Description

The CJS build of Vite's Node API is deprecated.

Use Node module resolution to fix the problem with Vite build
Additional context

https://vitejs.dev/guide/troubleshooting.html#vite-cjs-node-api-deprecated
Issue reference

coverage: 78.435% (-0.6%) from 79.015%
when pulling 5c5f4e2bcbe305ce631006f6b0f0b092bb6be14c on chore/design-tokens-vite-config
into 77a5ec583c1990dd330105c1e56704f1b2356f3d on main.

Do we plan to do this in all packages?

Thanks for your note, I have fixed this in all vite.config files.


Issue with accessing MegaDetector weight file

I am experiencing difficulties accessing the MegaDetector weight file when running the following command in Google Colab notebook:
ruby
Copy code
!wget https://lilablobssc.blob.core.windows.net/models/camera_traps/megadetector/md_v4.1.0/md_v4.1.0.pb
Previously, this command worked fine, allowing me to download the MegaDetector weight file (md_v4.1.0.pb) from the specified URL. However, I now receive the following error message:
vbnet
Copy code
--2023-07-03 09:36:45-- https://lilablobssc.blob.core.windows.net/models/camera_traps/megadetector/md_v4.1.0/md_v4.1.0.pb
Resolving lilablobssc.blob.core.windows.net (lilablobssc.blob.core.windows.net)... <IP_ADDRESS>
Connecting to lilablobssc.blob.core.windows.net (lilablobssc.blob.core.windows.net)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 409 Public access is not permitted on this storage account.
2023-07-03 09:36:45 ERROR 409: Public access is not permitted on this storage account.
The error indicates that public access is not allowed on the storage account hosting the MegaDetector weight file. I tried this command around a month ago, and it worked without any issues.
I kindly request assistance in resolving this issue. Is there an alternative source where I can access the MegaDetector weight file (md_v4.1.0.pb), or could you provide guidance on how to obtain it?
If needed, I can share my Google Colab notebook via email for further investigation.
Thank you for your support!

Thank you for your valuable input. We've moved to a new codebase with the release of Pytorch-Wildlife v1.0. The issue you mentioned might not exist anymore or it might be resolved in the new codebase. We encourage you to try the new release here and share any further feedback or concerns. If the issue still persists, please let us know through this channel and we will work on it!
If you're still interested in working with the previous version, you can take a look at Dan Morris' fork. The issue may or may not be resolved in that codebase.


Problem with DQfD doc

[x] I have marked all applicable categories:

[x] exception-raising bug
[ ] RL algorithm bug
[ ] system worker bug
[ ] system utils bug
[ ] code design/refactor
[x] documentation request
[ ] new feature request

[x] I have visited the readme and doc
[x] I have searched through the issue tracker and pr tracker
[x] I have mentioned version numbers, operating system and environment, where applicable:import ding, torch, sys
print(ding.__version__, torch.__version__, sys.version, sys.platform)

v0.4.5 1.12.1+cu102 3.9.16 (main, Jan 30 2023, 17:28:55)
[GCC 12.2.0] linux
Hi, I'm new here. I'm trying to run DQfD. The command ding -m serial_dqfd -c cartpole_dqfd_config.py -s 0 is not working. So I tried to run inside dizoo/classic_control/cartpole/config, and it pops out with Enter the name of the config you used to generate your expert model: What should I type? And How can I replicate the benchmark shown in the doc?
And another thing is that the benchmark compares DI-Engine with sb3 and tianshou on DQfD, but I can not find the algorithm in sb3 and tianshou. This confuses me.

We have polished DI-engine's DQfD doc. The DQN results of SB3 and tianshou were used in this doc before, which is indeed easy to cause ambiguity. Now it is removed in latest version.

I have recorded a naive demo about how to run cartpole + DQfD in container (link), you can refer to this demo. If you have other problems, you can continue to feedback in this issue.

I check out the video and thanks I get it. There is still one thing about the doc. The evaluation graph contains several curves. What algorithm the color stands for?

I check out the video and thanks I get it. There is still one thing about the doc. The evaluation graph contains several curves. What algorithm the color stands for?

Different curves means different random seed.

Okay, thanks!


Make kill confirm message not ephemeral

Let me have my fun, boy!

I'm not ashamed of me but my mum is.


Give MouseHandlers definitive mouseInWindow state on initialise

This is necessary to give MouseHandlers a correct state when initialised post-game-start. On the first call, they get a MouseEnter event fired once (per chance), but this makes sure they still have a correct state without relying on that fact.


Optimized the images for the fast loading of site

Optimized the images for the fast loading of the site.
If merged please add the label of SWOC.

@raman08  brother can you please give this PR two weeks before the program ends. Now the website will undergo multiple iterations of content and code change

@raman08  brother can you please give this PR two weeks before the program ends. Now the website will undergo multiple iterations of content and code change

okk!!!

okk!!!


getting audio of specific youtube url fails, others work fine

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--format', 'best', '-x', '--verbose', 'https://www.youtube.com/watch?v=VBHr0faDCoQ']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.10.11 (CPython) - macOS-11.7-x86_64-i386-64bit
[debug] exe versions: ffmpeg 6.0, ffprobe 6.0, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] VBHr0faDCoQ: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.10/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

youtube-dl told me to help by writing a bug report, so that's what I'm doing, just being nice, personally I don't care if this gets fixed or not.
I tried to get the audio from https://www.youtube.com/watch?v=VBHr0faDCoQ which didn't work
So please try yourself and if it works just close the issue.

It said to report the issue on the bug tracker, not just visit it and randomly open an issue without following the reporting procedure.
Follow #30839; complete the checklist before posting the issue.


response: Remove "result: partially_completed" for success responses.

In feature levels 153 and 154, a new value for result was added for two endpoints that process messages in batches: /api/delete-topic and /api/mark-all--as-read.
Prior to these changes, result was either "success" or "error" for all responses, which was a useful API invariant to have for clients.
So, here we remove "partially_completed" as a potential value for "result" in a response. And instead, for the two endpoints noted above, we return a boolean field "complete" to indicate if the response successfully deleted/marked as read all the targeted messages (complete: true) or if only some of the targeted messages were processed (complete: false).
When only some messages were processed, the code field with the timeout error string is still returned in the response, which is consistent with the changes in feature levels 153 and 154.
The web app does not currently use the /api/mark-all-as-read endpoint, but it does use the /api/delete-topic endpoint, so these changes update that to check the complete boolean instead of the string value for result.
See relevant CZO discussion.
Notes:

I tried to make the API changelog notes for the relevant feature levels (210, 154, 153, 147) for the related changes here reference the previous change so that it might be easier for someone reading to get the full context of these updates.
See inline comments for a few notes on the backend changes.

Screenshots and screen captures:

API changelog
Current documentation

Delete topic
Current documentation

Mark all read
Current documentation

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Updates:

API changelog entry for 210 updated for @gnprice's feedback in this comment with some small revisions.
Revised Changes notes in both endpoints main descriptions to step back through each change.

Note that the /api/delete-topc endpoint has the change of batching messages and sending a success response with "result": "partially_completed" in two feature levels (154 and 147), while the /api/mark-all-as-read endpoint has those changes both in feature level 153. Therefore, the Changes notes in the endpoint descriptions are not completely symmetrical.

There are no updates to the backend code/test changes.

Updated screenshots:

API changelog
Current documentation

Mark all as read
Current documentation

Delete a topic
Current documentation

Update: Bumped the API feature level change to 211. Screenshots above are still accurate in regards to documentation text changes, except for the feature level numbers.

Updates:

Removes the code field with the error string from the success response as well. This was part of the result: "partially_completed" implementation. With the new complete boolean field in the success response data, clients shouldn't need to reference any other information to know whether to retry the request. See relevant CZO discussion for more context on this change.

Updated screenshots:

API changelog entries

Delete a topic
Current documentation

Mark all messages as read
Current documentation

Merged, thanks @laurynmm!


ExoPlayerImplInternal Source error

Hello, can you help me for solve the problem?

Can you share the video stream url which you are trying to play ?

Closing the issue due to inactivity. Open it again if the issue is persistent

Closing the issue due to inactivity. Open it again if the issue is persistent


Release 0.17.2

Removed 12 configuration files and 5 components (1498, 1499)
SuperGLUE models updated

Approve


bugfix: fix pouch not restore containerd when containerd killed unexpectly

fix pouch not restore containerd when it get accident，refactor
the way in wait task exit, detail is in
https://github.com/alibaba/pouch/issues/1460
Fixes: #1460
Signed-off-by: Ace-Tang<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Detail design logic is locate at https://github.com/alibaba/pouch/issues/1460

Codecov Report

Merging #1475 into master will decrease coverage by 1.69%.
The diff coverage is 45.94%.

@@            Coverage Diff            @@
##           master    #1475     +/-   ##
=========================================
- Coverage   40.31%   38.61%   -1.7%
=========================================
  Files         251      251
  Lines       16416    17255    +839
=========================================
+ Hits         6618     6663     +45
- Misses       8956     9740    +784
- Partials      842      852     +10

Impacted Files
Coverage Δ

ctrd/container.go
50.18% <ø> (ø)
:arrow_up:

ctrd/watch.go
54.11% <39.53%> (-20.89%)
:arrow_down:

ctrd/client.go
54.83% <54.83%> (-0.1%)
:arrow_down:

apis/server/container_bridge.go
53.8% <0%> (-29.09%)
:arrow_down:

apis/server/volume_bridge.go
63.1% <0%> (-23.56%)
:arrow_down:

apis/server/utils.go
45.45% <0%> (-18.84%)
:arrow_down:

apis/server/network_bridge.go
44.87% <0%> (-18.77%)
:arrow_down:

daemon/mgr/container_types.go
61.16% <0%> (-16.62%)
:arrow_down:

daemon/mgr/spec.go
40% <0%> (-14.55%)
:arrow_down:

daemon/mgr/container.go
35.39% <0%> (-13.9%)
:arrow_down:

... and 4 more

ping @fuweid @HusterWan Can you review this pr?

@Ace-Tang Thanks a lot for your work to enhance the management of containerd process instance.
There is indeed a shortcut that when containerd process existed unexpected, we will not pull it up again before, so i think this pr is wonderful !!
there has one thing we should discuss, the old grpc client's stream client has 100 limitation, so we use a grpc clients pool to break the limitation.
so if we restart containerd, we also should notice the stream client counter.

@Letty5411 , from the test result, seems I kill containerd in test will influence cri test, maybe another containerd is need for the test.


Visual Studio Code Download Failed

VS Code Version: 1.65
OS Version: Windows 7 x64

Steps to Reproduce:

Access the website https://code.visualstudio.com/Download
Click the download button Windows 7, 8, 10, 11
Error "Cannot reach this site az764295.vo.msecnd.net took too long to respond." will be shown.

@joaomoreno I tried again now and the download site is working again. I think there was some problem between the server and me.