Documentation would benefit from example configurations

Hi,
I found the discussion in the HA community. Although it is a great resource it is also more of a log of things people tried instead a clear how-to.
Things that could for example be added:

how to setup a card with the latest picture. Download/ or camera definition (not clear to me)
What is the effect of the collect postcard service? Won't I see and have to check it in my BB app. And what is the effect if I do not collect the sighting in HA.

Maybe even:

Downloading all images
a card with rotation of the downloaded images

Furthermore, I think there are some entities missing from the documentation.
I'm currently struggling to get it working in my HA. If I have a working configuration I'm happy to help with a PR.

Lots of questions here, I'll try to address them all.

how to setup a card with the latest picture. Download/ or camera definition (not clear to me)

That's a good idea to document it, perhaps as a cookbook recipe or something. Likewise for using Downloader to save or archive the most recent media (or all media if you have a need for that).
I had started to toy around with the idea of adding our own camera entity that just shows the last seen photo, or even last N photos as a slideshow, or the videoMedia from the most recent visit. But it feels like anything more than just the last photo is a lot of complexity for something that is really a matter of preference and choice, which causes complexity to increase dramatically. That leaves us with a few options for the implementation:

Single photo: very simple (works just like the Generic Camera), and allows us to work around the expiring AWS link.
Multiple photos: more complex because now we have to manually maintain the list of media and rotate between them.

We have to decide or expose configurations for how many images to cycle, and whether the images should be from only the most recent postcard visit, or maintain the most recent N photos across multiple visits, or a single photo from each of the last N visits, etc. Those choices could be represented with just 2 options: number of visits/postcards to rotate, and number of images per visit; such that recent_visits: 1 and images_per_visit: 1 effectively emulates the (1) single photo option.
Since all of these are a matter of preference, I decided that it's probably better suited to using another integration that is designed for your specific requirements, coupled with manually downloading the exact images you want in your slideshow. This is what many users in the HA forum decided to do, because there's really no one-size-fits-all solution. Rather than forcing my interpretation of one or more of these options, I leave it as an exercise for the user to decide what your requirements are, and then utilize existing integrations to satisfy those requirements.

Video: this is a whole other beast, because Generic Camera does not support streaming a single mp4 video file, only an HTTP image URL, or an RTSP camera feed.

FFMPEG does have this capability (looping input from an mp4 video file), and in theory you could probably get the FFMPEG Camera to do this for you. I don't know what would happen in this case, if the input video file gets replaced -- does ffmpeg pick up the new file and switch it in its loop, or will it see that as a broken file and stop working? There's also an issue that I recall reading a while back, about how the FFMPEG integration will not allow looping the video file, because the Stream integration still sees the "end of stream" marker, and effectively closes the stream when that is encountered. I'm not sure if that is still an issue, and/or if the FFMPEG -stream_loop option works around that issue.
Additionally, Frigate has support for this in much the same way, where Frigate can expose a looping mp4 video file as if it is a camera. This is usually used for testing so that you can have a single clip taken from a camera, and test different motion and object detection settings until you get the exact behavior you want for a given clip. Same question applies here: what happens when the underlying video file gets overwritten? I don't know, and if it doesn't already handle that, I doubt that the maintainers would entertain the ability to pick up file changes in a reliable way, given this capability is advertised only for development and testing purposes.

Due to all of these possibilities, and the fact that there are multiple ways it could be implemented depending on the user's preference, I've been hesitant to actually add official support for any of it, or as you're suggesting, even to add documentation advocating for one option over another. Even the (2) scenario doesn't really make sense for us to recreate the "slideshow" capability here, when there are already multiple implementations of this available in the community. Recreating it would just create Yet Another implementation that won't have all the support necessary to please everyone. The one scenario that could be most universally useful would be the (1) scenario of a single image file, because that's simple to implement and doesn't raise all the other questions and options; plus the "Recent Visitor" entity already gets us just about the whole way there.
Which brings me to: we do already have the "Recent Visitor" entity, which adds the most recent media as its entity_picture attribute, and the bird species name as the state value. That allows the entity itself to display the image as its "picture" (usually a small circle to the left of the name, like what you would see when viewing a Person entity), and this works automatically. To convert that into a larger image that can be used with the Picture Entity card, you can set up a Generic Camera using this template as the "Still Image URL":
# Replace "my_bird_buddy" with the actual name of your own feeder for this entity
{{ state_attr('sensor.my_bird_buddy_recent_visitor', 'entity_picture') }}

and enable the "Limit refetch to URL change" setting. Then you can get a large version of the image appearing using the picture-entity card, for example:
type: picture-entity
show_state: true
show_name: false
camera_view: auto
# The "Generic Camera" entity
camera_image: camera.bird_buddy_recent_visitor
# Your "Recent Visitor" Bird Buddy entity.
# This is used to show the bird species name at the bottom of the card.
entity: sensor.my_bird_buddy_recent_visitor
aspect_ratio: 5x6

The "Recent Visitor" entity has its own challenges, because the URL we get from the Bird Buddy API is an AWS link which has an expiring access token. Once the image is downloaded for that URL, the expiration doesn't affect it anymore because the image bytes will remain in memory in most cases - but there are scenarios where the URL expires and we don't have the image bytes in memory anymore, and in that case the image will fail to load.

What is the effect of the collect postcard service?

The collect postcard service effectively removes (or replaces) the need to manually open the BB app and save the media for a new postcard. It is not very useful by itself, because you can't really call it with manual inputs, unless you rebuild the majority of the JSON payload that it needs in order to emulate the BB app in this process. It is really only usable in conjunction with the "new_postcard" event or device trigger. To remove that confusion, I recently added a Blueprint that combines that trigger with the service call, using the UI to configure the Blueprint. If all you want to do is automate the opening/saving of new postcards, this is probably the best way to do that, because it removes the need to manually configure the automation yaml, because you can use the Blueprint UI to configure it.

Won't I see and have to check it in my BB app.

When you use this blueprint or combine the new_postcard trigger with the collect_postcard service, then you do not have to use or open the BB app at all. Based on how you configured the Blueprint or service inputs, it will behave as documented for the service, which means it will either accept the BB identification for the species if the BB AI is confident in its selection, accept the highest-confidence best guess identification for it, or fall back to mystery visitor. Be aware that if you select recognized and BB is not confident in the identification, that sighting will be discarded (no way to get it back - the same as if you remove a sighting or media from the BB app when opening a postcard), and the same is true if you use best_guess and none of the suggested species meets the configured best_guess_confidence.

And what is the effect if I do not collect the sighting in HA.

If you do not configure an automation that combines the trigger and service call, or the Blueprint, then the effect is simply that the integration will not attempt to automatically process a new postcard. The postcard remains unopened in the BB app, and will not appear in the Media Browser in HA.

Furthermore, I think there are some entities missing from the documentation.

You're right. There are 10 entities (per feeder) exposed by the integration. We document 6 of them. Of the remaining 4, 3 of them are disabled or hidden by default, because they are effectively "incubating", meaning either their function is unknown, or they do not function properly because Bird Buddy has not enabled those features yet. Those 3 disabled entities are Food Level (not functional on BB API yet), Temperature (not functional on BB API yet), and Frequency (functional as far as the API is concerned, but actual effect is dubious, or has no known effect).
That leaves the "Recent VIsitor" entity that has not been documented. Thanks for pointing that out.

I'm currently struggling to get it working in my HA.

Sorry but I haven't actually seen a question in your description that points to something that is not working? As far as I can tell, your confusion is about documentation, or possibly not knowing what to expect from the integration?
If you're actually having trouble getting the integration to work, then please reframe that specific question.
It sounds like maybe you have it working, but don't know what to do with it next? And that's not really something I can answer for you - it has to be the other way around: determine what you want from the integration, and reframe your question to ask if that is possible or how to achieve it.

If I have a working configuration I'm happy to help with a PR.

Again, getting mixed signals here. This implies that you do not have a working integration? If that's the case, I need more information.

I'm sorry for the confusion.
The integration is (and was) working. I could not get the results shown on my dashboard.
The biggest thing I was missing here was the camera. When adding the integration in the UI I was not allowed to put the template in and in de configuration I made some small (but alas important) mistakes in the beginning.
When I got the camera working I could easily create a card. I did not have any visits yet since but I hope the downloading will also work and I'll be able to try using https://github.com/jodur/imagesdirectory-camera#camera-entity
What I was actually missing was an example of the camera config (and I find a lot of places where such example would have helped me)
Something like:
camera:
  - platform: generic
    name: birdbuddy
    limit_refetch_to_url_change: true
    still_image_url: "{{ state_attr('sensor.<<YOUR__BIRD_BUDDY_ENTITY>>_recent_visitor', 'entity_picture') }}"

Another thing I noticed is that the charging entity is not named similar to the rest and does not contain the name of the birdbuddy.

I made some small (but alas important) mistakes in the beginning.

Do you mind sharing what those small/important mistakes were? Maybe I can make changes to prevent those kinds of mistakes in the future.

What I was actually missing was an example of the camera config
Something like:

Unfortunately HA is moving away from requiring (or even allowing) manual configuration.yaml entries for many integrations like this. Your example generic camera configuration might work for now, but it will likely generate a warning in HA logs, as things like that should be converted to being configured from the UI rather than from YAML.
Your code block does make a good, easy to understand way of just copy/pasting a config; but I would rather not propose a configuration that is ultimately deprecated and may be removed in the future. The Generic Camera documentation (linked above) does not even list manual YAML configuration as an option.
I agree that documenting the "Generic Camera + Recent Visitor + Picture Entity card" recipe would be useful. You should not have to sift through the entire HA community thread to find several examples of things that didn't work, and ultimately one hard-to-find example of something that did work.
I updated the README to add some points of clarification, and to add the missing documentation for things like Recent Visitor. And I will work on surfacing those other common example ideas as cookbook recipe ideas - perhaps linking to the Github wiki. There were several examples in that thread of things that other community users accomplished, that could be useful to surface here.

Another thing I noticed is that the charging entity is not named similar to the rest and does not contain the name of the birdbuddy.

I'm seeing this named correctly in my integration: my Feeder is named "Elm Bird Buddy", and my entities are named:

Elm Bird Buddy Battery (sensor.elm_bird_buddy_battery)
Elm Bird Buddy Charging (binary_sensor.elm_bird_buddy_charging)
and so on...

When viewing the Device Info page, it automatically drops the redundant device name from the entity name (so it appears there as "Battery", "Charging", etc).
Depending on when you first installed the integration, you may very well have had an early version where that naming was not done following current best practices, so it might have been named differently at that time. That naming convention was updated to match best practices in v0.0.6 on Jan 1, 2023.
And another possibility is you may have manually updated the name of the entity. If you open the entity, and then click the ⚙ cog, do you see a value in the "Name" field? If so, that will override the name provided by the integration. Just remove the name override and it will go back to default HA behavior described above.

I installed the integration last Saturday. And I did not change any of the names. At least not knowingly, and there is now override on the name.
All the devices names are sensor.XXX_feeder_state etc, except the charging which is binary_sensor.bird_buddy_charging
Not sure what is the reason.
Regarding the camera. I could not add {{ state_attr('sensor.<<YOUR__BIRD_BUDDY_ENTITY>>_recent_visitor', 'entity_picture') }} in the first field of the camera UI. So I had to do it in the configuration. I'm not sure what I tried that failed before it worked. But I had some tries before the platform was correct and the the correct field still_image_url with the correct value was there :)
When I had that setting up a card was some work to get it as I wanted, but it was kinda straightforward from there.
The only thing I can't seem to find is how to get the translated values of the state. They are visible in the details and the history. But I can't seem to get them on a card :)

That's exactly how I added it, and it works:

However, the 2nd screen of the initial Generic Camera config flow probably requires showing and confirming the image works as you expect. So if your Recent Visitor entity doesn't currently have an entity_picture attribute, or that attribute's URL has expired, it's possible that configuring the Generic Camera might not work correctly if it cannot download the URL to show in the 2nd screen preview.
If your Recent Visitor entity has a bird species name identified, then it should have what's needed. It should look like this:

But if the picture beside the entity is blank, then the URL might have expired, or it might not have had a value yet. This is not uncommon for very new feeders (i.e., if you have not received any visitors yet). But if you have at least one postcard in your feed for that feeder, it should populate the name and image URL when the integration starts up - which you can do by going to Settings > Integrations > clicking the 3-dots overflow menu for Bird Buddy integration, and then select "Reload". It will try to re-initialize the Recent Visitor attributes from the current Bird Buddy feed.

The only thing I can't seem to find is how to get the translated values of the state. They are visible in the details and the history. But I can't seem to get them on a card

If you see the correct translations in the entity dialog, but not in a lovelace card, then it might be an incorrect lovelace card implementation. What card are you trying to use? Also which entity is not being translated? is it the "Feeder State" entity?
The entity name itself ("Feeder State") does not appear to be using translations currently, but not because it can't -- it just wasn't set up (and therefore we don't have translations contributed currently). But the actual value options (i.e., "Sleeping", "Ready", etc) should have translations for de, nl, and sv. If it shows values like "DEEP_SLEEP" or "READY_TO_STREAM" (or lowercase versions of those), then it means it isn't using the translated state value, but instead using the symbolic enumerated states:
class FeederState(Enum):
    """Feeder states"""

    DEEP_SLEEP = "DEEP_SLEEP"
    FACTORY_RESET = "FACTORY_RESET"
    FIRMWARE_UPDATE = "FIRMWARE_UPDATE"
    OFFLINE = "OFFLINE"
    OFF_GRID = "OFF_GRID"
    ONLINE = "ONLINE"
    OUT_OF_FEEDER = "OUT_OF_FEEDER"
    PENDING_FACTORY_RESET = "PENDING_FACTORY_RESET"
    PENDING_REMOVAL = "PENDING_REMOVAL"
    READY_TO_STREAM = "READY_TO_STREAM"
    STREAMING = "STREAMING"
    TAKING_POSTCARDS = "TAKING_POSTCARDS"

    UNKNOWN = "UNKNOWN"

those then get mapped to these strings (or translations):
    "sensor": {
      "feeder_state": {
        "state": {
          "deep_sleep": "Sleeping",
          "firmware_update": "Updating firmware",
          "factory_reset": "Reset to Factory",
          "offline": "Offline",
          "off_grid": "Off-grid",
          "online": "Online",
          "out_of_feeder": "Out of feeder",
          "pending_factory_reset": "Factory reset pending",
          "pending_removal": "Pending removal",
          "ready_to_stream": "Ready",
          "streaming": "Streaming",
          "taking_postcards": "Taking postcards",
          "unknown": "Unknown"
        }

All the devices names are sensor.XXX_feeder_state etc, except the charging which is binary_sensor.bird_buddy_charging
Not sure what is the reason.

Also not sure why this would happen. Mine is named binary_sensor.elm_bird_buddy_charging. The only thing I can think is that when the sensor was added, the integration didn't have the Feeder name yet, but that doesn't make sense why it would be the only entity that didn't.
When creating the entity, we set has_entity_name = True and set the name with name = "Charging". It then uses the device.name and appends the entity.name, so because my Feeder.name is "Elm Bird Buddy", the entity gets created and shows the name "Elm Bird Buddy Charging", which gets converted to entity id "elm_bird_buddy_charging". In device.name, it uses the feeder.name attribute from the BB API response, but if that attribute was missing, it falls back to a default of "Bird Buddy". So if the initial integration was added before it was able to get the name of your feeder, then it might get initialized as "Bird Buddy Charging", which would get converted to binary_sensor.bird_buddy_charging.

Did you add the integration before you paired your Bird Buddy to your account?
Are you logging in as the Bird Buddy feeder owner account, or as a member account? In other words, did you have to use an invitation code in order to grant permission to the email+password account that you created for the integration -- usually because you initially created your Bird Buddy account using SSO like Google/Apple/Facebook, which we are not able to use for this integration?
If you had to use an invitation code, did you confirm the claimed invitation from the owner account before or after configuring the integration in HA?

I ask because if you used an invitation code, there is a possible half-state where Bird Buddy tells us that we have access to a Feeder, but it is in a "pending approval" state, and we can't really access any of the data until the owner confirms the invitation. We should still have access to "FeederForMemberPending.name" though, even in this case, so it still should have created the entity with the correct name.

That's exactly how I added it, and it works:  However, the 2nd screen of the initial Generic Camera config flow probably requires showing and confirming the image works as you expect. So if your Recent Visitor entity doesn't currently have an entity_picture attribute, or that attribute's URL has expired, it's possible that configuring the Generic Camera might not work correctly if it cannot download the URL to show in the 2nd screen preview.
If your Recent Visitor entity has a bird species name identified, then it should have what's needed. It should look like this:

The recent visit was showing. And adding again seems to work. Don't know why. I would swear I now try the same string. I do not have the two checkboxes in the bottom of the screen by the way.
Regarding the translations. I guess it is the card. I think I used a mushroom card. The entity card seems to work now.
regarding the charging entity. I'm using my own owner account and the bird buddy was already paired. But looking at your comments, it looks like some kind of race-condition

I did not change or delete the integration. 🤔

So you have also issues with entity_picture ? The methods with sympke camera was working upmto recently but now not anymore and of a check the recent visitor entity there is no more the attribute entity_picture within

False alarm ...was something transitiory, pictures are now back


Change standard color back to black

Hello,
its a horrible idea to use grey as standard color for the editor and menu (and results panel) text.
Please change to black.
People can see text if its black and on a white background.
Regards

@maximilian22x could you please attach a screenshot?

The colors are controlled by the selected Color Theme.  Please look through the themes to find one that matches your preference.  Alternatively you can load any themes from the VS Code marketplace by downloading and then installing the VSIX in the extension manager.


[Feature request] Preview action for QLab

The Preview action is available with a "V" keyboard shortcut in QLab and triggers the selected cue without advancing to the next one.  Would be pretty convenient to have accessible through Companion.
Thanks!

Selected  or Playhead cue? Depending on preferences, they aren't always the same.

Selected.


JavaScript Code Injection

Using the a field type "text" in the config.json file allows entry of javascript, which gets rendered upon saving.
Reproduce:
Add <script>alert('hi')</script> as the setting for a text field, press save, and an a javascript alert pops up
It is possible that a malicious user could inject code which other users might execute without knowing it.
Solution:
Sanitize all inputs.
Precedence: It is already done for text fields in instruments.  It should be done for all text fields in External Module Settings.
Possible Exception: Create a field type "script" which is available to super users only.

It is a good point, but there needs to be a way to permit javascript from EM's too.  So, perhaps two different field types makes the most sense...
-A

I think I may need more information to help with this.  I tried inserting a javascript alert as described on a few of our modules, but it was not executed.  I could see this happening if a module was inserting a setting value into the DOM, but I don't know of a module off the top of my head that does that in order to think through the use case.  Any information related to goals(s) for inserting javascript in setting values might be helpful as well.  I don't believe I've encountered a need to do that before.

I can no longer reproduce it.  The alert no longer comes up.  It appears that single and double quotes are now being replaced by the HTML entity.


Create Download Release Python Script

Create a standalone python 3 script to enumerate and download a published release for use on destination nodes.

For now, we are going to just use gsutil cp and rsync.


OpenSubtitles

They still offer big data dumps

They also lie about the data being licensed... I've seen scripts for major blockbusters posted here within a week of release. This is a textbook example of data laundering.


AOT Fails to compile

The component is working well except for when I do the AOT compile with following exception: Error encountered resolving symbol values statically. Calling function 'makeDecorator', function calls are not supported. Consider replacing the function or lambda with a reference to an exported function, resolving symbol Injectable... in<EMAIL_ADDRESS>resolving symbol OpaqueToken

Can you show your package.json?

Check this comment.

Closed by #30


fix single non-ASCII character

Similiar to https://github.com/example42/puppet-puppet/pull/56
More info
https://projects.puppetlabs.com/issues/20897
https://tickets.puppetlabs.com/browse/PUP-1386
Error:
Error: Could not retrieve catalog from remote server: Error 400 on SERVER: invalid byte sequence in US-ASCII at /etc/puppet/modules/supervisor/manifests/init.pp:1 on node test
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run

Thanks, @applewiskey ! :-)


[release-2.3] Update org references to stolostron

Signed-off-by: Yu Cao<EMAIL_ADDRESS>

Image built and tested in https://app.travis-ci.com/github/stolostron/governance-policy-framework/builds/244402810


Use the atom-workspace tag instead of the workspace class.

In index.less: Use the atom-workspace tag instead of the workspace class.

duplicate of #27


[AutoPR cognitiveservices/data-plane/ContentModerator] Add SSN parameter to Content Moderator Screen text API

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2815

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/1516


Implementations I've been thinking of, and hopeful fix of path traversal bugs?

So firstly, I am going to be selling parts of my software on the WIi No Ma store, or pre-compiled versions for that matter. iOS jailbreaks are the main thing.
Also, some of the other things I am putting on the store are My Aquarium tanks, mystery randomly generated ones. This is why my scripts pre-support these two things.
Speaking of which though, the scripts have a lot of path traversal bugs, including mine. I am hoping we can fix these.

lmfao

As for liking aquariums, yeah I do. Heck, me and my Dad would have fun at Petsmart a lot in the summer looking at the fish and aquarium equipment. I'm not saying the My aquarium tanks are filler, but I don't want the shop to be useless, ya know?

I don't understand why you're working on the shop for Wii no Ma though? I thought that Snoot and the others said we weren't bringing that back.

The end goal is to provide a shop in some way - from what it seems, we can leverage their Mii Stamp assets which will be quite nice. However I am not sure how the shop, running on a Wii within Flash Player, will provide these DLC/exploits. It might be worth separating and having a unique QR that can be sold or something - another method of access.

It will actually email them Spotlight via sendgrid with the DLC/exploits, digitalcontentsender.py is the script I made for that :)

You can send any file with digitalcontentsender.py, PDF's, images, c files if you want to, zip files, anything :) Support is even there to compress files.

Not to mention I can reuse our sendgrid api key for our Mail-Go instance if needed too :D

Doesn't the game require the DLC to be hosted? I don't think email will be a solution.
(Additionally, if this is true, this code should be in another repository and not integrated to room-server.)

Nah, it sends it locally from a path on the server :)

Basically, I actually figured out the DLC isn't hosted on NWCS, it's actually something you inject into your savefile. So the script generates the DLC, saves it to the same directory, and sends it off from a local file on the server :) It doesn't need to delete it as it will automatically wipe the file each time.

This has one slight flaw however. They overlap. I need to seperate and terminate each instance, in case two people want tanks at the same time.

My Aquarium allows you to share tanks with your Wii Friends, but I don't think sending them via an actual email address will be something the Wii will parse.

Yeah I know Larsen, people will have to inject the dat file into their own savefiles. Although Larsen, similar to how you distributed a mii for the RC24 christmas present, is it possible to distribute a0014682.dat to all wii's too?

Honestly I don't see much of a userbase for My Aquarium, it's a very niche thing.
We do Animal Crossing: City Folk DLC for instance because we believe there's a lot of people still playing it on the Wii. Even though New Horizons is on the Switch, some people don't have that and are still getting enjoyment.

Agreed yeah, I guess it's kinda because I don't know where else to put it?

https://github.com/WiiLink24-Extensions/File-Maker Here is where the My Aquarium and Animal Crossing DLC WiiLink24 uses is. I slightly modified/customized some of RiiConnect24's tools to get this the way I like it :)

@larsenv So Larsen, you can implement Wii No Ma in the RC24 patcher if you want :) I do wanna however do the UNIX version myself, as I have been working on a much more improved UNIX patching engine.

This issue seems off-topic, and I'm not certain as to its actual topic.


Add MultiHead Attention

After reading
- cudnn
- pytorch/fairseq (the base impl. of pytorch/pytorch)
- dmlc/gluon-nlp
I chose pytorch's impl as a reference.
TODO

[x] test
[x] documentation

Since I'm not familiar with Attention enough, any comment is highly welcomed.

Now I'm done with porting pytorch implementation.
@himkt @keisuke-umezawa Could you skim this?

Sorry for being late, we want to merge this PR as the experimental functionality
before merging https://github.com/cupy/cupy/pull/2215.
So, could do please resolve conflicts and add a statement to make MHA modules experimental?
example: https://github.com/chainer/chainer/blob/721986ebfdf9375f52ef71f885ad8ea6402a5a03/chainermn/links/n_step_rnn.py#L84
ref. https://docs.chainer.org/en/stable/reference/generated/chainer.utils.experimental.html

Could you elaborate on the motivation behind experimental?
On Tue, Aug 6, 2019 at 19:23 Makoto Hiramatsu<EMAIL_ADDRESS>wrote:

Sorry for being late, we want to merge this PR as the experimental
functionality
before merging cupy/cupy#2215 https://github.com/cupy/cupy/pull/2215.
So, could do please resolve conflicts and add a statement to make MHA
modules experimental?
example:
https://github.com/chainer/chainer/blob/721986ebfdf9375f52ef71f885ad8ea6402a5a03/chainermn/links/n_step_rnn.py#L84
ref.
https://docs.chainer.org/en/stable/reference/generated/chainer.utils.experimental.html
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chainer/chainer/pull/7066?email_source=notifications&email_token=AD3Q7U47GUY72SHOYX64Z2LQDFGJ7A5CNFSM4HKSECM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3UVY2A#issuecomment-518610024,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD3Q7UZIGEY7PB2IBOQOQELQDFGJ7ANCNFSM4HKSECMQ
.

Ah, I made a big mistake in resolving conflicts.

workin' on fixing this mistake.

Why this pull request is closed? What's the problem of using cudnn for multihead attention?

@2sin18 Chainer has been in maintenance, not actively developed anymore. So they and I decided not to make this pull request & #7879 merged. Thank you for your interest and sorry not to have made this happen.


Add JOINs piece to relational databases section

Hi @nikolasburk, I implemented the changes last Friday EOD - let me know if you think it's ready to merge. Thanks!


[Backport release-22.05] cfitsio: 4.1.0 -> 4.2.0

Description of changes
Backport #202171 to release-22.05 due to security fixes.
(cherry picked from commit f8eea402836c5f7f538b2b380c977503873b306a)
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

nixpkgs-review happy, aarch64-linux, nixos x86_64 & macos 10.15.


How to build and import a local repo of MRTK into my Unity project

Describe the issue
I'd like to make my own personal change to the MRTK staying on the version I am currently on (not upgrading to MRTKv3). I've forked the repo and made my change but I'm unsure on how to go about building it so I can use it as a package in Unity.
Feature area
It would be great if there were steps available on how to get a local version of the MRTK into my Unity project without just dragging the assets in. Preferably how it's currently done with making each part into a .tgz file.
Existing doc link
N/A
Additional context
This would help for future custom development anyone else may want.

Thank you for reporting this issue.  Unfortunately, we are only fixing high priority issues for MRTK2.  There is currently no active development on MRTK2.
Moving this issue to the forums.
If you need further assistance, please reach out to our support team at https://aka.ms/hldevsupport


Breaking Change: Fix param name in ActivityContext..ctor (traceOptions -> traceFlags)

Fix param name in ActivityContext..ctor (traceOptions -> traceFlags)
The ActivityContext API was added earlier in 5.0, but there was a mistake in the constructor's argument name where traceOptions was used instead of traceFlags.  The implementation is correct but the reference assembly was wrong. dotnet/runtime#41843 fixed the argument name.
Version introduced
.NET 5.0 Preview 4
Old behavior
The previous constructor signature was the following, with an argument named traceOptions.
public ActivityContext(System.Diagnostics.ActivityTraceId traceId, System.Diagnostics.ActivitySpanId spanId, System.Diagnostics.ActivityTraceFlags traceOptions, string? traceState = null)

New behavior
The updated constructor signature is the following, with the argument renamed to traceFlags.
public ActivityContext(System.Diagnostics.ActivityTraceId traceId, System.Diagnostics.ActivitySpanId spanId, System.Diagnostics.ActivityTraceFlags traceFlags, string? traceState = null, bool isRemote = false)

Reason for change
This corrects a mistake in the argument name and will improve the API, avoiding confusion by users.
Recommended action
If the argument was being used as a named argument, update calling code to correct the argument name to match the renamed traceFlags name.
Category

[ ] ASP.NET Core
[ ] C#
[ ] Code analysis
[x] Core .NET libraries
[ ] Cryptography
[ ] Data
[ ] Debugger
[ ] Deployment
[ ] Globalization
[ ] Interop
[ ] JIT
[ ] LINQ
[ ] Managed Extensibility Framework (MEF)
[ ] MSBuild
[ ] Networking
[ ] Printing
[ ] Security
[ ] Serialization
[ ] Visual Basic
[ ] Windows Forms
[ ] Windows Presentation Foundation (WPF)
[ ] XML, XSLT

Affected APIs
ActivityContext(ActivityTraceId, ActivitySpanId, ActivityTraceFlags, String, Boolean)

Issue metadata

Issue type: breaking-change

I'm not sure we need to document breaking changes from earlier preview versions. @PriyaPurkayastha ?

I thought we needed to document these since RC1 is Go-Live and these changes land in RC2.

I updated the description to call out that the fix goes in with RC2.

Yes, any change in behavior from Go-Live release should be documented. In this case, breaking change is being introduced in .NET 5 RC2. So Version introduced should mention .NET 5 RC2 (not the version in which the API was introduced).

Thanks, @PriyaPurkayastha; I'll update that.


failed: error: UnexpectedQuit (used Azure AD, discovery not completed)

Storage Explorer Version:   1.10.0/ 1.15.0
Build Number:
Platform/OS:  windows 10 x64
Architecture:  x64
Regression From:
Bug Description

Can't download and upload file from Azure storage server ,
failed: error: UnexpectedQuit (used Azure AD, discovery not completed)
But same account login other computer , it download and upload normal
Steps to Reproduce

select file to download , it will received error message
Expected Experience

Download/upload successful
Actual Experience

occur error : failed: error: UnexpectedQuit (used Azure AD, discovery not completed)
Additional Context

Tried reinstall Azure storage App ,Reset , Reload , issue still occur in my PC , when i change a new PC , issue gone ,

Error message
{
"message": ""UnexpectedQuit""
}

@linyu220 There is an AzCopy log in its logs directory (C:/Users/username/.azcopy). Could you clear the directory, reproduce the failed transfer and share the log file with us? The log file already have credentials redacted.
Given that the error only occur on one PC, it may be related to firewall setting. Does the storage account you are accessing have firewall enabled? Is your computer within the allowed IP range?

@JasonYeMSFT ,
Did we got any resolution here? I am also getting same exception, I have also tried with different version(Old and latest), still no luck, And surprisingly, It was working before, only recently this issue occurred.
Please suggest any resolution for this.

@saurabh2335 Unfortunately no resolution so far. We use AzCopy's output to show the error. The "Unexpected quit" will be shown when AzCopy fails to give a legitimate output before quitting. Please try to get the log file or use "Copy command to clipboard" and run the command on command line using AzCopy to see what the actual error is.
You can download AzCopy from here: https://docs.microsoft.com/en-us/azure/storage/common/storage-use-azcopy-v10#download-azcopy

@JasonYeMSFT,
I only got this details in project explorer.
{
  "message": "\"UnexpectedQuit\""
}

```An in AzCopy, it is not returning any error
Command which i used in AzCopy
`azcopy.exe copy "https://*****/data-dct/UAT3/database/cpspms/adhoc/id/archive/360_Adhoc_0062232050_100074288_1589448361697.CSV?sig=***********************&st=2020-11-05T14%3A44%3A00Z&se=2020-11-06T15%3A44%3A00Z&sv=2017-04-17&spr=https%2Chttp&sp=racwdl&sr=c" "C:\Users\*****\New folder\360_Adhoc_0062232050_100074288_1589448361697.CSV" --overwrite=prompt --check-md5 FailIfDifferent --from-to=BlobLocal --recursive;`

@saurabh2335 I believe it is caused by your sas resource type "sr=c". Please try creating a SAS with at least "sr=co".

@JasonYeMSFT , Also tried with sr=co, still no output.
This might be because I tried to login with SAS url with sr=c and I copied the command from "Copy AzCopy Command to Clipboard"
I also tried to login with account name and key, still getting the same error.

@saurabh2335 Could you share the command when using account name and key? Also, could you do me a favor to add an argument "--output-type=json" to the end of the command and share the json output with us?
For account level SAS, we expect you to choose "Use a connection string" to connect to the storage account with account level SAS. If your resource type is "c", we don't allow you to move forward. The SAS url is usually used for connecting with a container level SAS.

This command I got with account name and account key, but still no output.
$env:AZCOPY_CRED_TYPE = "Anonymous"; $env:AZCOPY_CONCURRENCY_VALUE = "AUTO"; ./azcopy.exe copy "https://**********.blob.core.windows.net/data-cps/*****/activityfile/ID/archive/activity_file_0000110005_1192735_20200723071953.csv?sv=2019-10-10&se=2020-12-05T18%3A37%3A10Z&sr=c&sp=rl&sig=HanAPNgvk1uE8PxbtW%2BekKl%2F6JLuZvRHXs704tPNxZM%3D" "C:\Users\s*****\New folder\activity_file_0000110005_1192735_20200723071953.csv" --overwrite=prompt --check-md5 NoCheck --from-to=BlobLocal --recursive --output-type=json; $env:AZCOPY_CRED_TYPE = ""; $env:AZCOPY_CONCURRENCY_VALUE = "";

Are you saying that there is no output at all? Like when you execute the command, the command line looks like:
user$ azcopy cp ......
user$

If that is the case, could you schedule a call with us so we can investigate it together? You can send your preferred way of contact to sehelp at microsoft dot com.

@JasonYeMSFT,  I think we are one step closer to resolution,
I downloaded AzCopy 32 bit(previously using 64 bit), Now at least I am getting  some output,
So I tried below command in AzCopy (without any parameter), files is downloaded successfully,
azcopy.exe copy "https://*********.blob.core.windows.net/****/database/activityfile/ID/archive/activity_file_0000110005_1192735_20200723071953.csv?sv=2019-10-10&se=2020-12-05T18%3A37%3A10Z&sr=c&sp=rl&sig=*******" "C:\Users\**********\New folder\activity_file_0000110005_1192735_20200723071953.csv";`
but for previous command I got this error 'Error: couldn't parse "json;" into a "OutputFormat"', that means it is not working with command that I got from Storage Explorer.
I also tried with '--overwrite=prompt;' from AzCopy, got error like "failed to parse user input due to error: couldn't parse "prompt;" into a "OverwriteOption""
Also same with other parameter from original query I got the parsing error.

@JasonYeMSFT , I think it has something to do with that parameter which are part of query from Storage Explorer (Same with AzCopy), as it is not able to parsed that and failing in between. But not sure why it occurs recently.

@saurabh2335 Good to know that the command itself works. The error you mentioned is caused by AzCopy thinks "json;" or "prompt;" are the argument value instead of "json" and "prompt". The ; is used to separate commands. Not sure why it happens since the cmd tool should be able to tell and exclude it when passing arguments to the executable. The only way I can reproduce is to pass the argument like --overwrite="prompt;". Removing the ; may help mitigate this issue. At the moment, if AzCopy doesn't work in Storage Explorer, please try to use the command line to workaround we look into it.
Also are you using "Command Prompt" instead of powershell? That cmd tool has this problem

@JasonYeMSFT Because it prevent download and upload file from Azure storage . and the Azure storage work normal at other PC .
So I try to reinstall image and Azure storage , issue gone ,

@linyu220 , Can you please elaborate the meaning of reinstall image? Because I also tried reinstall Azure Storage several times, but it doesn't works in my case.

@saurabh2335 I get this failure in my surface studio 2 .And I also try to reinstall Azure storage explorer 1.10 or 1.15.1 when issue happen . Issue still happen
I try to reinstall customer image for my set .  It work normal now.

@saurabh2335 you said that running on command line using 32 bit AzCopy worked, but it doesn't look like you did it with --output-type=json. Can you try it again with that option? Also, Jason asked:
Are you saying that there is no output at all? Like when you execute the command, the command line looks like:
user$ azcopy cp ......
user$

Is that indeed the case when you try to use 64 bit AzCopy?

Closing due to lack of responses. For the future, if you're having issues doing an upload or download, we recommend:

try running the transfer on the command line using the command from the "Copy AzCopy Command to Clipboard" action
if that works, but you still want to use Storage Explorer in the future, then we'll ask you to do these troubleshooting steps:

provide us the log file from "Goto AzCopy Log File"
run the transfer on the command line, but add --output-type=json to the command

If y'all are still having problems, please open a new issue. Thanks!

Closing due to lack of responses. For the future, if you're having issues doing an upload or download, we recommend:

try running the transfer on the command line using the command from the "Copy AzCopy Command to Clipboard" action
if that works, but you still want to use Storage Explorer in the future, then we'll ask you to do these troubleshooting steps:

provide us the log file from "Goto AzCopy Log File"
run the transfer on the command line, but add --output-type=json to the command

If y'all are still having problems, please open a new issue. Thanks!


document that json_error_code will be added in version 2.11

Coverage decreased (-0.2%) to 90.012% when pulling 24d45272a7553da609e747ef7deb1cf2c3d5beb7 on phst:doc-error-code into 9e5af7c3b77f81a7d1720073dca600e51fe9962d on akheron:master.


Add File Picker documentation for "viewType" configuration parameter

This adds documentation for the viewType configuration parameter which will be supported by v7.2 of the OneDrive File Picker (pending release).
The viewType parameter allows developers to scope the selection of item types to one of "all" | "files" | "folders" with the default value being "files" which is the existing default behavior.
An example usage which scopes item selection to only folders is:
var options = {
    clientId: '<client_id>',
    viewType: 'folders',
    success: (files) => console.log('success', files)
};

OneDrive.open(options);

Related to enabling support for #352.

OPS Build status updates of commit af48e91:
:clock10: Preparing: average preparing time is 1 min(s) 8 sec(s)

OPS Build status updates of commit af48e91:
:clock10: Incremental building: average incremental building time is 28 sec(s)

OPS Build status updates of commit af48e91:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/controls/file-pickers/js-v72/open-file.md
:white_check_mark:Succeeded
View (>= odsp-graph-online)

docs/controls/file-pickers/js-v72/save-file.md
:white_check_mark:Succeeded
View (>= odsp-graph-online)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

  This is the expected behavior of the build system.

OPS Build status updates of commit 90df9e2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/controls/file-pickers/js-v72/open-file.md
:white_check_mark:Succeeded
View (>= odsp-graph-online)

docs/controls/file-pickers/js-v72/save-file.md
:white_check_mark:Succeeded
View (>= odsp-graph-online)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


feat(Petals): Add support to Petals

https://github.com/bigscience-workshop/petals

It needs specific version of CUDA which I do not have. So, maybe hold on.

This is huge as it would enable the use of biggest models deployed on community swarms using inexpensive hardware.
I think this is a critical milestone in democratizing AI.


create dashboard for openmetrics v2

What does this PR do?
Create a CoreDNS dashboard for openmetrics V2 metrics
Motivation
https://datadoghq.atlassian.net/browse/AGENT-12365
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

/merge

/merge


sd_setImageWithURL  can't show the image at the moment

sd_setImageWithURL  With tableViewCell's imageView can't show the image at moment,and u have to touch cell, then the image show

please use the example project as reference. you might have missed something in your code.

Uploading WebWork.zip…

you did not upload the zip correctly. try again

WebWork.zip

WebWork.zip

Why it can't work without a placeholderImage?
when I use  sd_setImageWithURL:  instead of sd_setImageWithURL:placeholderImage, the image also can't show at the moment , and need to click the cell

you should use with custom subclass if you want to use without a placeholder image.
or you can make a transparent placeholder image and use that one. this way the user will not see the placeholder at all

yeah，thank u


The Strategic Defense Initiative

This was an idea america had in the 80s to prevent nuclear annihilation but did not come to fruition. maybe make it a decision or have a event that gives the possibility for it to succeed?
http://www.coldwar.org/articles/80s/SDI-StarWars.asp
https://en.wikipedia.org/wiki/Strategic_Defense_Initiative
https://www.youtube.com/watch?v=XI-YkEIiUXY

I like the idea. Maybe not as a viable program but more as a way to force expenditures on the Soviet Union by forcing them to take counter measures.


bg- disabled not working on some element

Hi,
I can use bg-orange with well
well well-sm bg-orange
but I can use bg-orange disabled
Maybe disabled color class should be renamed
Any clue?

Make sure you add the btn class to it. As shown below:
<button class="btn well well-sm bg-orange disabled">Test</button>
I did it first without the btn class the disabled class did not work and adding btn to it worked.

In my case I'm using Bootstrap well element, so I don't want to use btn class for this.
I guess the color pallet was more for button than other Bootstrap elements.
I have created new class for the disabled color with specific color and I'm not using disabled attribute.
This may be a suggestion for future update.

So in what case would you need a well disabled?

Oh, it was just for a  background color of a well.
It is ok to use the disabled attribute on button, but not really for other element.
But we may like to use the lighter color for other background element.

If you're trying to set the color of a well, doing it through CSS would be your better option than adding a disabled class since the class will also affect the colors of everything inside the well and not just the background.

This what I have done.
My suggestion is this will be nice to have a complete pallet color that we can reuse in other element without using attribute like disabled.

Maybe rather than having a complete pallet color, there could be .light and .dark which would adjust the opacities or overlay a white or black over it? Not sure how this can be proposed as a new feature.
That way it would allow for more colors on the palette without needing to add 10+ more color classes.

Sound good to me


Feat: Add progress tracking breadcrumb component - 213

This pull request introduces the ProgressBreadcrumb component, designed to effectively track and display progress stages on the new transfer page.
Closes #213

Looks great!

Thank you for reviewing my code!


Get rid of styled components styling & switch to MUI in ClientListTable.jsx and ClientListTableRow.jsx

Issue:
currently have some leftover styling in ClientListTable.jsx and ClientListTableRow.jsx from first implementation

Direction:
change to MUI components and styling instead

Hi
Where are those file?
Would you please mention dir to these two .jsx file

Hi Where are those file? Would you please mention dir to these two .jsx file

On Development branch...  src/components/Clients

Hey @xscottxbrownx @Sky-De , is this being worked on? If not, could I have it assigned to me? Doesn't look like I have permissions to self-assign.

^ Thanks andy, I missed this in email.

Hi @masenmatthews. Just checking in. Have you made progress with this?

Hey @andycwilliams, apologies for the delayed response! Yeah I've made some progress but have gotten a bit sidelined due to work and travel demands in the last couple of weeks. I'm out of town this week, but should be able to continue working on this next week.
If this has increased in urgency, then it might be worth reassigning if there are resources available. I should have a lot more bandwidth to work on tasks and attend meetings consistently after roughly 7/25.

Hey @masenmatthews was a bit curious as to the progress of this issue since it's related to PR #376. Let us know if you're still up to working on this issue. Thanks!

Unassigned, been 2.5months with no seen progress and lack of responses

@leekahung @xscottxbrownx
Apologies for missing your previous message. Confirming that this can be reassigned.

This is a good issue for a newer team member to take on but I can pick it up if nobody else wants to.

I'll take a shot at this if no one else is working on it.

I'll take a shot at this if no one else is working on it.

You can but just so you know I'm currently converting ClientListTable.jsx and ClientListTableRow.jsx to MUI as part of #343. I can leave the refactoring of this styling itself to you. Just wanted to let you know in order to avoid conflicting branches.

@andycwilliams thanks for the info. I have no emotional attachment to this particular issue, and don't want to risk any unnecessary thrash. I will look for something else.

I am pretty sure this issue has been solved by this PR.
Here is the commit that has the changes to the two jsx files mentioned above.

Think we can close this as complete.

Sry I thought that my comment on the PR would auto close this ticket.

Sry I thought that my comment on the PR would auto close this ticket.

It's due to our default branch being Master, so issues won't close until code is merged into that branch...
I rose an issue for this suggesting we switch default branch to Development so issues close when PR is created to Development as is our typical flow.   Issue #430


Add upgrade task

I've extended the provided upgrade tasks by creating an abstract class with all the plumbing needed for it to work, and a couple of functions that help detect when we are dealing with a fresh install, as some upgrades might not be needed in that case.
It also add the first migration, that deals with the removed priority option and sets it in the workflow.

Good work! Merging


⚠ Bofrost.be has degraded performance

In 55078c3, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1461 ms

Resolved: Bofrost.be performance has improved in 1293c32 after 8 minutes.


Does scredis support readonly to only read from slave and not master ?

I didn't find support for readonly in scredis but just wanted to double-check in case i missed it.
Is there a way i can do hmget only from slaves.I don't want it to read from master as those are getting throttled.So I want to mark it readonly when creating a RedisCluster to read from slaves.
Do you have suggestions/idea?

Hi @amishra123, this repository is no longer maintained.
Can you reopen your issue in https://github.com/scredis/scredis?

Ok Thanks will do.


#30 Add extended codec information

Resolves #30
It would help collections to add extended codec information to the metadata output.

[x] Add codec_long_name and codec_tag_string to the video and audio codec output

Testing

Update your local storage volume location in docker-compose-dev.yml
Add locations to your dev.env
Add a video file to the watch folder
Re-build the transcoder: make build
Note the csv in the output folder now has extended video_codec and audio_codec information
Note tests pass: docker exec -it transcoder make test
Note linting and tests pass docker exec -it transcoder make lint
Note we now have GitHub Actions as well: https://github.com/ACMILabs/transcoder/actions

Example output:
{
  "title": "ACMI_COLLECTIONS_SEARCH_PROMO_draft1_GL_23052024",
  "width": 1920,
  "height": 1080,
  "checksum": "754b5d21c6327291388b54e874c4c0a2",
  "filetype": "m",
  "mime_type": "video/mp4",
  "vernon_id": "",
  "audio_codec": "AAC (Advanced Audio Coding) (mp4a)",
  "video_codec": "H.264 / AVC / MPEG-4 AVC / MPEG-4 part 10 (avc1)",
  "duration_hms": "00:00:35:01",
  "duration_secs": 35.029333,
  "audio_bit_rate": 317219,
  "audio_channels": 2,
  "video_bit_rate": 19878604,
  "file_size_bytes": 88465999,
  "overall_bit_rate": 20203867,
  "video_frame_rate": 25.0,
  "audio_sample_rate": 48000,
  "creation_datetime": "2024-05-23 08:02:35+00:00",
  "audio_max_bit_rate": null,
  "video_max_bit_rate": null
}

@rubyquaildesign Here's how to update your local repo from master to main too:
git branch -m master main
git fetch origin
git branch -u origin/main main
git remote set-head origin -a


Not able to build with VS2017

I get the following errors: (VS 2017 Professional)

Do you have the latest net core 1.1 and net core 2.0 SDK's installed?
And what is your VS 2017 version?

Now I could build. I've to install net core 1.1 sdk from https://github.com/dotnet/core/blob/master/release-notes/download-archive.md
I did select net core 1.e support when installing VS, but this seems to be not enough...
But how should I get from this error, to I need another SDK...

Ah I see the errors now.
Also net core 2.0 is not yet added as a target ....
Some work to do for me...

building UAP 10 is fixed


which python version, i got 2.7 and python3 installed.

python b2fuse.py /home/nodejs/bucket
Traceback (most recent call last):
File "b2fuse.py", line 499, in
main(args.mountpoint, args.account_id, args.application_key, args.bucket_id)
File "b2fuse.py", line 491, in main
with B2Fuse(account_id, application_key, bucket_id) as filesystem:
File "b2fuse.py", line 93, in init
config = load_config()
File "b2fuse.py", line 86, in load_config
return yaml.load(f.read())
File "/usr/lib/python2.7/dist-packages/yaml/init.py", line 71, in load
return loader.get_single_data()
File "/usr/lib/python2.7/dist-packages/yaml/constructor.py", line 37, in get_single_data
node = self.get_single_node()
File "/usr/lib/python2.7/dist-packages/yaml/composer.py", line 35, in get_single_node
if not self.check_event(StreamEndEvent):
File "/usr/lib/python2.7/dist-packages/yaml/parser.py", line 98, in check_event
self.current_event = self.state()
File "/usr/lib/python2.7/dist-packages/yaml/parser.py", line 143, in parse_implicit_document_start
StreamEndToken):
File "/usr/lib/python2.7/dist-packages/yaml/scanner.py", line 116, in check_token
self.fetch_more_tokens()
File "/usr/lib/python2.7/dist-packages/yaml/scanner.py", line 257, in fetch_more_tokens
% ch.encode('utf-8'), self.get_mark())
yaml.scanner.ScannerError: while scanning for the next token
found character '\t' that cannot start any token
in "", line 1, column 23:
accountId:SecretXXXXXXX
^

I am afraid your going to have to be a bit more detailed on the problem your facing. It does however seem that it has problems parsing you config file.
This is the correct format:

accountId:
applicationKey:
bucketId:

Note that there is a space between ":" and each value.

From you error message it appears you are using python2.7. This should work fine.

its working, fused into folder  , space seemed like the problem

Excellent, I'll consider this issue resolved. Please open another issue if you have additional problems ;)


folder tree structure not showing

VSCode Version: Code 1.15.0 (8b95971d8cccd3afd86b35d4a0e098c189294ff2, 2017-08-09T20:06:21.685Z)
OS Version: Darwin x64 16.7.0
Extensions:

Extension
Author (truncated)
Version

html-class-suggestions
And
1.0.7

ng-template
Ang
0.1.4

beautify
Hoo
1.1.1

html-css-class-completion
Zig
1.8.0

npm-intellisense
chr
1.3.0

vscode-eslint
dba
1.2.11

javadebugger
don
0.1.3

tslint
eg2
0.17.0

auto-close-tag
for
0.4.3

html-tag-wrapper
hwe
0.1.7

Steps to Reproduce:

Reproduces without extensions: Yes/No

Please provide more information, including screenshots.

In the sidebar when I open folder the folder tree is not showing. I shows the folder name in title bar only.

sorry solved


test/432-puppeteer-fix-puppeteer-timeout-errors

…for jest and upgraded puppeteer @types version to 1.0.0

 Puppeteer: Fix Puppeteer timeout errors


Filter + Search together throws type error or resets the search text

I created my own toolbar, containing a switch and a searchbar.
The switch calls this.refs.table.handleFilterData while the searchbar makes use of this.refs.table.handleSearch
The problems arise during 2 scenario where you filter first, then search or when you search first then filter.
I've included my work arounds, and am wondering anyone had encountered these issues before and what the recommended way is to fix these issues before starting on a PR.

Filter then search will throw:

TableDataStore.js?0c9e:527 Uncaught TypeError: Cannot read property 'filter' of undefined
Which is the line:
const source = this.filterObj !== null ? this.filter(this.filterObj) : this.data;

I can work around this by using the filtered data if a filter object is present
But I have to make sure I set the filerObj back to null in my component when I want to reset the filter
var source = _this4.filterObj !== null ? _this4.filteredData : _this4.data;

Search then filter will ignore the search, this is due to the filter always using the complete data set and not the filtered data set:

TableDataStore:264
this.filteredData = data.filter(function (row) {

I can work around it by using the filtered data if searchText is set.
Again I have to make sure I set the searchText to '' or null when I clear it.
var data = this.searchText ? this.filteredData : this.data;
this.filteredData = data.filter(function (

edit: This workaround doesn't solve the scenario:

Sucesfull search
Filter that returns 0 results
Turn off filter (should return previous succesfull search, but won't )

I will have another look tomorrow

@shotor, thanks your feedback, I'll check this out :)

I just ran across the same issue.

Fix on v2.3.3.


Simplified Web Page of Examples

Proposal to add an abridged version of all current and future path tracing examples.

@svnkomo
Hello Sakhile!  This is fantastic!  I will definitely be merging this, as I think that this is a perfect addition to the path tracing repo here.  I happened to check my email late tonight right before bed (ha) which had the notification about your PR.  If you don't mind, let me get some sleep and go over everything in detail tomorrow (Friday).  But from what I've seen and tested out so far, this is amazing!
Thank you so much, and talk to you soon! :-)
-Erich

@erichlof
Hi Erich! No problem, it's something I've been thinking of doing for a while, finally got down to it. Glad you like it.
If there are any suggestions or improvements to be made, I'd be glad to do it. Definitely a WIP.

Hi again, Sakhile!
For the most part everything looks great - there are just a couple of issues I have found and a couple of questions/requests that I want to ask.
I think all of the demos are listed on the examples page - the only one I couldn't find is Geometry_Showcase.html (that's the main default demo for my repo - it has a link at the top right when you visit the GitHub repo). Could you please add a .png card for that as well, maybe like you did for Quadric_Geometry_Showcase.html, which is very similar?
The only other issue I found was that the buttons don't appear correctly anymore when viewing on my mobile device. I'm including a screenshot:

The buttons on the right appear shifted down and off the viewing area, so that mobile users can't navigate the camera. The good news is that I'm sure this can be fixed with a 'one-shot-fixes-all' solution (maybe in the default CSS?), but I'm not very knowledgeable about CSS and not very up-to-date with how tailwind works. Those triangular buttons for mobile might have to be 'fixed' in their position. Long ago I cobbled together the correct spacing for both portrait and landscape mode for mobile, so that no matter which way you turn your phone or tablet, the buttons are visible and accessible in the corner. There's a small room for error when dealing with mobile buttons like these, because you don't want them too spread out where they won't all fit on the page,  but then if they're too close together, some people's thicker thumbs accidentally press 2 buttons at once, which is frustrating. I'm sure you and I can figure this issue out.
This reply is getting long so I'll put the questions in the next one... Thank you! 😊

Hi again,
I wanted to ask some questions about the new links and files. Is it possible to just include the tailwind.js file on the repo here, instead of dynamically linking to a cdn? As you know, I like to have everything available in one place on the repo here, even three.js, so that when someone forks or downloads the source code, they can quickly just click the demos on their own local machine (with a minimal local server running of course, for images and shader file loading) and it'll just work. If they happen to be not connected to the internet, I would like them still to be able to start a local simple server and run everything right off the bat. The cdn link to tailwind wouldn't work in that situation, unless I'm mistaken. I know that adds a little size to the repo, but I would just like all dependencies in one place, so that if a user is offline, or tailwind goes away or has breaking changes, then this tailwind file we're including right now is a 'frozen' version that correctly works out of the box. I looked at the cdn link and it does appear already minified, so this current version and file size is good to go in my eyes. Again, open to suggestions on how to handle this dependency.
Lastly, will everything be hosted from my repo here? I noticed that the examples page is currently hosted by your GitHub pages, but unless I'm mistaken, that's just so you can show the functionality while you're working on your fork of my repo - do I have this correct in my mind?
Again, thank you so much for putting all of this together- I know it must have been a lot of work! Once we get some of these issues and details worked out, your new examples system will take this GitHub repo to the next level! 😃
Talk to you soon,
Erich

Oh one last thing (ha), I probably will be removing 2 demos from my repo - namely the MultipleSamples Per Frame and the MultiSPF Dynamic Scene demos.  These were just a proof of concept for a multiple samples per frame technique that was suggested by someone else long ago.  Back when path tracing was just getting started on the web, multiple samples were required to deal with the noise, but a high-end graphics card (like NVIDIA RTX) was required to keep the framerate acceptable.  However, the modern approach has shifted away from multi-sample loops in the same animation frame, to doing only 1 sample per frame (which is very noisy), but then cleaning up with some kind of denoising post-processing.  The nice thing about doing only 1 sample per frame is that it goes super fast, even on the web, and even on mobile.  And now that I have a simple denoiser, both spatially (pixel and its surrounding neighbor pixels ) and temporally (pixel color samples are averaged over several frames in a row), there is really no need for these demos and their outdated technique.
Plus, you might be able to take the .png card for MultiSamples Per Frame, and use it for Geometry_Showcase.html instead (which is exactly the same scene setup), as I really like that .png card snapshot - I think it looks really cool!
So if you want to remove the 2 links mentioned above from your example web page, that would be great!  I will be removing those files (and their spot on my README) on my own end very soon, maybe even today.  Maybe after these changes, you can re-fork, or update your own local fork, so that yours reflects my latest changes - that way, there won't be any merge conflicts later on down the line.
Thanks again!


Nat hole punching for discv5.2

Closes https://github.com/sigp/discv5/issues/133. This implements a new protocol with the same goal as the rendezvous protocol outlined as draft in the issue, that is to hole punch NATs, but with a different method. The protocol is tailored with more precision after discv5's design to remove identified attack vector's that the rendezvous protocol introduced like the 2-for-1 message triggering and adding mechanisms to make NAT hole punching produce almost only necessary overhead for the network at whole by taking parameters like connectivity between relays and targets into account, for example that as the time an initiator stores a relay to a target increases, the connectivity guarantee between that relay and the target decreases. The new discv5.2 spec contains the new hole punching protocol and the rationale is still in review.
Testing with my discv5-cli fork.

I've addressed you review @divagant-martian. I'm unsure about the need of the constant assertion on the MESSAGE_NONCE_LENGTH you suggest since it seems it is checked anyway as part of how the compiler works by reading this?

I've addressed you review @divagant-martian. I'm unsure about the need of the constant assertion on the MESSAGE_NONCE_LENGTH you suggest since it seems it is checked anyway as part of how the compiler works by reading this?

That's known wrong behaviour for that lint https://github.com/rust-lang/rust-clippy/issues/8159

@divagant-martian I have addressed all your comments with code or answers

Another partial review to get this moving.
Code in improving after the last pass, good work.
It's likely I need to still dig deeper into the code to understand this but I'll rather ask early. There are a couple of parts of the code where we drop sessions. What's the rationale behind this? I seem to remember one is when our external ip address is updated and the other when a session message can't be decrypted

when our external ip address is updated: Sessions in this code default to 24 hours, and answering PING from peers not in k-buckets was deactivated, as it was found to be an attack vector I remember being told. Combining these two facts means a node without a contactable ENR on session establishment, won't be given a second chance to get into the k-buckets of a sigp/discv5 node until 24 hours later, unless they remove all their sessions and this way trigger new session establishment.
when a session message can't be decrypted: refer to discv5.2 specs

Thanks for the review @divagant-martian, I have now addressed your latest remarks. Excuse me for drive by fixes, especially structural ones like moving functions, I realise from your review that makes it difficult to review the diff. I strongly advise you to read the discv5.2 spec at this point, it will answer some of your logic questions.

Sorry for the delay on this. Have been waiting for others to complete their reviews and dealing with some other things. Reviewing this will be my sole focus next week, so we can get some cool hole punching in soon :)

Ok, I see, no need to apologise. Sounds awesome. There is only one breaking change: using the news SessionMessage container for a response. I could revert that in anticipation for the rest of the discv5.2 protocol, and then nat hole punching could be used already on mainnet as any SessionMessage would then only be notifications related to hole punching and can be dropped by other clients for now. Let me know what you think.

it says I deleted a comment from age, sorry! was supposed to be a comment from myself. @emhane I could help you with the conflicts if you need it. Let me know

for using nat:ed peers discovered by this devp2p nat protocol in lighthouse, we will have to modify these lines in libp2p to iterate through all the tcp connections instead of just the relay's reservations, so that libp2p's hole punching code will work with relays passed on from discv5. here in discv5, I will add the relay used into the RequestCall struct, and add the event "discovered nat:ed peer", to pass the relay back to lighthouse along with the nat:ed peer upon established connection. lighthouse peer manager will have to combine the two peers into a p2p-circuit mulitaddr @AgeManning
https://github.com/libp2p/rust-libp2p/blob/30024f38d41804599a64d00934bf169b146e3ea3/protocols/relay/src/behaviour.rs#L498C16-L502C18

adressed about half of your review @AgeManning with either comments or code. diff gets quite large in latest commit 32321a70 due to layout change of hole punch code to make it more idiomatic, but before that at 7cde68c6 the diff shouldn't be too big and easier to review.

Awesome thanks! Currently fighting some libp2p things. But know this has not been forgotten and i'm still eager to get it in. Let me re-review soon and maybe I"ll make some changes as I go (which you can revert if you disagree with) to speed up the process :)

Awesome thanks! Currently fighting some libp2p things. But know this has not been forgotten and i'm still eager to get it in. Let me re-review soon and maybe I"ll make some changes as I go (which you can revert if you disagree with) to speed up the process :)

nice! sure just open a pr to my branch

for using nat:ed peers discovered by this devp2p nat protocol in lighthouse, we will have to modify these lines in libp2p to iterate through all the tcp connections instead of just the relay's reservations, so that libp2p's hole punching code will work with relays passed on from discv5. here in discv5, I will add the relay used into the RequestCall struct, and add the event "discovered nat:ed peer", to pass the relay back to lighthouse along with the nat:ed peer upon established connection. lighthouse peer manager will have to combine the two peers into a p2p-circuit mulitaddr @AgeManning
https://github.com/libp2p/rust-libp2p/blob/30024f38d41804599a64d00934bf169b146e3ea3/protocols/relay/src/behaviour.rs#L498C16-L502C18

@jxs nice that you want to get involved. perhaps this is smthg you could dig into?


Menu Design

Info
For larger displays is it possible to make the menu width smaller
UI

Solve
Make the menu width smaller
Thanks Team, also loving the new design looks amazing 🙌

Thanks for the feedback @WillHick, glad you like the new menu, and you spotted really quick the change. About the width, it's a design decision to use that width, I can share feedback but I'm closing the issue as it's not a bug.


Readme, Window.initialize, and MultilineEntry.text

The main thing is the draw method in Window.initialize. It is called right after initialization, and doesn't allow the user to pass any callbacks into the window.

Thanks for the changes, I haven't had the chance to work on Hedron as of late due to schoolwork. I'm currently rewriting the entirety of the plugin to hopefully clean things up.

Awesome! I took a look at the rebuild branch, and it looks like it is much more intuitive and flexible. Please also can you add an example of how to use Area and Image?


Tape tool does not work from map view

When trying to measure a distance from the map view, no measurement is made.
Any save.zip
Select the tape tool, go to the map, and then attempt to use the tool.
Username: AmericanPatriot
factorio-current.log
This is probably because the tape tool is a grenade(why?) and not a selection box.

Duplicate of #3.


Later rollback of transactions

Concrete Request:
I would appreciate the possibility to rollback a transaction after doing the commit by myself.
My cause:
That would allow me to test the outcome of a transaction (To do reads after the writes) and decide by myself if the transaction was successful or not.
Alternative
Another solution for my problem would be to have another callback function which will be executed after the commit but could result in a rollback. Could be called a "test" callback.
Possible solution:
We could achieve this by making the Transaction class accessable:
https://github.com/googleapis/nodejs-firestore/blob/56933478d8633759d853a8ed578e8204f8714eed/dev/src/transaction.ts#L67
At the very moment the transaction object and its transaction ID seems to be unreachable:
https://github.com/googleapis/nodejs-firestore/blob/56933478d8633759d853a8ed578e8204f8714eed/dev/src/transaction.ts#L72
But there already are commit and rollback functionality, unfortunately also hidden:
https://github.com/googleapis/nodejs-firestore/blob/56933478d8633759d853a8ed578e8204f8714eed/dev/src/transaction.ts#L384
https://github.com/googleapis/nodejs-firestore/blob/56933478d8633759d853a8ed578e8204f8714eed/dev/src/transaction.ts#L398

@nicolaiwolko Thanks for your suggestions!
We currently don't hav plans to make our internal transaction API public, as this will expose our user to its retry and error handling logic. That being said, you might be able to achieve what you suggested if you can transform your transaction into a WriteBatch. Note that you can optimistically lock documents in a WriteBatch by using Preconditions:
const docSnap = await docRef.get();
const batch = firestore.batch();
batch.update(docRef, 'field', 'value', {lastUpdateTime:docSnap.updateTime});
await batch.commit();


add basic test suite

This will probably require either dynamically generating repos or bundling them like pkgcheck.

Something super basic, but I've added self test by running over guru repository https://github.com/pkgcore/pkgcheck-action/commit/18799ee3e6f42f7beb5ab87d8a2cb56074fb00e8.


chore: update readme

Please describe the changes this PR makes and why it should be merged:
Update README.md with a little infromation by @almostSouji so people can see what this repository is at a glance, without having to go to Discussions.
Status and versioning classification:

This PR only includes non-code changes, like changes to documentation, README, etc.

The discord.js version 13 text needs updating since it's released! Also, changing the contributing link to use the main branch would be a bonus!


copykat py2rpy error

Hi,
When I run copykat method with the latest version 0.3.0, the following error occured:
NotImplementedError: Conversion 'py2rpy' not defined for objects of type '<class 'pandas.core.indexes.base.Index'>'
What should I do?
Best

HI @diyang1354,
can you try
...
norm_cell_names = norm_cell.obs.index.tolist()
...

Hi @diyang1354,
What helped me was to run this after importing and before running a tool:
from rpy2.robjects import pandas2ri

pandas2ri.activate()


Pass-through of unhandled MIDI data

Hi,
I've written a template for FL Studio that makes great use of this plugin but it seems the newer versions change the behavior of MIDI data. Before, anything unhandled would pass through, but now MSS doesn't forward along unhandled MIDI messages. Can this behavior be restored, so that MIDI messages will continue along to the out port by default?
Thanks

This change of behaviour was not intentional. I am wondering what has caused this to be a problem. Maybe the dependency updates.

My report was not super helpful, I realize. This was initially noticed on version 1.0.11, from version 1.0.10 in 2013 (Beta was possibly different as well but I didn't test it). I hadn't reported it as I was trying to use an alternative due to needing 64 parameters for MPE.


call soup.find_all only once in backcompat.apply_rules

22 calls to soup.find_all turned out to be very expensive, taking ~90%
of the total time spent parsing. instead we can just call it once and
collect all the classic microformats roots at once.
cc: @snarfed @KevinMarks

woo awesome! thanks @kylewm!
i take it that 90% was from light benchmarking?

yes to light benchmarking. I profiled the parser while processing a few different pages, and eyeballed the amount of time spent on apply_rules compared to everything else.

empirically this has mostly sped up parsing in mf2py on my hovertest page:
url time before time after
http://about.me/thisisdeb 3.69 2.11 57%
http://cascadesf.com/ 13.62 5.43 40%
http://chocolateandvodka.com/ 14.61 11.73 80%
http://epeus.blogspot.com 11.24 6.72 60%
http://giudici.us/ 0.425 1.22 287%
http://kathyems.wordpress.com/ 24.92 9.13 37%
http://kevinmarks.com 2.63 5.06 192%
http://known.kevinmarks.com 9.13 4.43 49%
http://laurelschwulst.com/ 1.74 2.31 133%
http://ma.tt 1.61 4.64 288%
http://pmckay.com 0.431 1.73 401%
http://rickydesign.me/ 5.13 1.33 26%
http://tantek.com 3.43 4.45 130%
http://twitter.com/kevinmarks 5.53 2.32 42%
http://werd.io 11.75 12.11 103%
http://www.katiejohnson.me/whatimthinking.html 31.34 10.53 34%
https://kylewm.com/ 40.2 9.62 24%
https://plus.google.com/+KevinMarks 1.35 1.62 120%
https://snarfed.org/ 16.68 5.31 32%
On Sun, Feb 28, 2016 at 10:28 AM, Kyle Mahan<EMAIL_ADDRESS>wrote:

yes to light benchmarking. I profiled the parser while processing a few
different pages, and eyeballed the amount of time spent on apply_rules
compared to everything else.
—
Reply to this email directly or view it on GitHub
https://github.com/tommorris/mf2py/pull/61#issuecomment-189917808.

bridgy saw this speed up its mf2py parse times an average (mean) 3-5x during a 4h test run just now. woo!

friendly nudge @tommorris. itching to see this get released so i can use it properly instead of hackily. :P

Away from laptop for the next ~12 hours but will take a look tomorrow. I am as excited by the performance improvents!
--
Tom Morris
http://tommorris.org/

On 1 Mar 2016, at 19:52, Ryan Barrett<EMAIL_ADDRESS>wrote:
friendly nudge @tommorris. itching to see this get released so i can use it properly instead of hackily. :P
—
Reply to this email directly or view it on GitHub.

@tommorris OK with you if @snarfed or @Kevinmarks reviews, and I merge and upload a new pypi package pending their approval?

Go for it @kylewm.
--
Tom Morris
http://tommorris.org/

On 5 Mar 2016, at 17:51, Kyle Mahan<EMAIL_ADDRESS>wrote:
@tommorris OK with you if @snarfed or @Kevinmarks reviews, and I merge and upload a new pypi package pending their approval?
—
Reply to this email directly or view it on GitHub.

i'm not familiar with mf2py, but this lgtm! more importantly, mf2py has backcompat tests which (i assume :P) pass, and bridgy and granary's do too.

1.0.3 is up here https://pypi.python.org/pypi/mf2py/1.0.3
thanks all

yay, thank you!


[poco] Remove unneeded dependencies from core feature

poco core feature should not depend on libpq and libpng, but add openssl. Then an equivalent set of files is produced.
After vcpkg install poco, disk space used: 2.2GB (original) vs 1.4GB (without libpq) which saves 800MB!
To verify this change makes equivalent files, here are the 2 scripts to print the file lists and hashes in packages/poco_x64-linux directory.
Dockerfile 1
FROM ubuntu:focal

RUN apt-get update && apt-get install -y curl tar unzip git g++ make && rm -rf /var/lib/apt/lists/*
WORKDIR /src
RUN git clone git://github.com/Microsoft/vcpkg.git . && git checkout 2020.04
RUN ./bootstrap-vcpkg.sh
RUN apt-get update && apt-get install -y bison flex && rm -rf /var/lib/apt/lists/*
RUN ./vcpkg install poco
RUN find packages/poco_x64-linux -type f ! -name vcpkg_abi_info.txt ! -name CONTROL | sort | sha1sum `xargs` | tee sum.txt && sha1sum sum.txt

Dockerfile 2
FROM ubuntu:focal

RUN apt-get update && apt-get install -y curl tar unzip git g++ make && rm -rf /var/lib/apt/lists/*
WORKDIR /src
RUN git clone git://github.com/Microsoft/vcpkg.git . && git checkout 2020.04
RUN ./bootstrap-vcpkg.sh
RUN sed -ie 's/, libpq,/, openssl,/g' ports/poco/CONTROL && sed -ie 's/, libpng//g' ports/poco/CONTROL
RUN ./vcpkg install poco
RUN find packages/poco_x64-linux -type f ! -name vcpkg_abi_info.txt ! -name CONTROL | sort | sha1sum `xargs` | tee sum.txt && sha1sum sum.txt

Hi @linquize
Thanks for posting this issue.
It seems that libpq is required for feature postgresql and libpng is required for feature pdf.
So they can be removed from the dependency lists.
But as for openssl , it looks like optional instead of required, which can be referred here.
OpenSSL headers and libraries (optional, but recommended)

However, poco 's CMakLists.txt uses find_package to detect SSL present.

However, poco 's CMakLists.txt uses find_package to detect if OpenSSL present.
If installed directory already has openssl, find_package will return openssl is present.

Yes, you can create a separate feature call openssl that depends on openssl

Since it's related to #10108, we can track this issue there #10108.
I'm closing this issue for now.


update CI to ODF 4.14 release

/cherry-pick release-4.14

/cherry-pick release-4.14-compatibility

/override Build Test / resolve-dependencies (18) (pull_request)

/override resolve-dependencies (18)

/override "resolve-dependencies (18)"

/override "resolve-dependencies (14)"

/override "resolve-dependencies (16)"

/cherry-pick release-4.14


Link the utilities js file in src/helpers to js pages

The functions written in the utilities file should be exported such that they can be accessible to the other files on the server side.

Resolved with commit a098f4055c811b59bfa466f47c5a8a3cb781d9b2

There are two ways to do it, implemented and tested them both and they work. See:
http://stackoverflow.com/questions/5311334/what-is-the-purpose-of-node-js-module-exports-and-how-do-you-use-it


Migration script won't work with Askar, as it uses the wallet export

Not sure what to do here. Maybe we can do an sql dump and export that? We would still need to support that in Askar though, otherwise we need an sql client

Could you please provide more detail on this?
What does this block, and where do you expect the solution to come?
Askar’s export/import functionality is being expanded to match that of indy-sdk — does that help?

Askar’s export/import functionality is being expanded to match that of indy-sdk — does that help?

Absolutely. I'm looking forward to get the new functionality regarding queries without categories so we can easily retrieve all records from the database and do a dump to JSON or any other convenient format that can allow us to properly manipulate and transform to other backends.
If this new feature of Askar is going to get ready soon, we can wait for it.

@andrewwhitehead — I think you have this almost ready, right?  The handling of the release was holding it up, right?

Solved for SQLite by copying the database file. For Postgres we need to be able to query all records, or just make a dump of the postgres database itself.

This is getting a higher priority now, as in 8cc499d133f67be391a7956ac51f6078af78f69d the storage version was updated to 0.5, making any existing Askar wallet coming from 0.4 to execute migration script.
When running it in one of our applications using postgres DB backend, I found the unfortunate error:
2024-02-01 21:53:51.675 DEBUG /build/config/logger.js:61 Wallet '2060-cloud-agent-dev' opened with handle '1'
2024-02-01 21:53:51.688 INFO /build/config/logger.js:61 Agent storage is not up to date.
2024-02-01 21:53:51.690 INFO /build/config/logger.js:61 Starting update of agent storage with updateIdentifier<PHONE_NUMBER>689
2024-02-01 21:53:51.699 INFO /build/config/logger.js:61 Starting update process. Total of 1 update(s) will be applied to update the agent storage from version 0.4 to version 0.5
2024-02-01 21:53:51.702 ERROR /build/config/logger.js:58 Error updating storage (updateIdentifier:<PHONE_NUMBER>689) {
  cause: {
    name: 'WalletError',
    message: 'Export is only supported for SQLite backend',
    stack: 'WalletError: Export is only supported for SQLite backend\n' +
      '    at AskarWallet.export (/www/node_modules/@credo-ts/askar/build/wallet/AskarWallet.js:222:19)\n' +
      '    at WalletApi.export (/www/node_modules/@credo-ts/core/build/wallet/WalletApi.js:89:33)\n' +
      '    at UpdateAssistant.createBackup (/www/node_modules/@credo-ts/core/build/storage/migration/UpdateAssistant.js:168:39)\n' +
      '    at UpdateAssistant.update (/www/node_modules/@credo-ts/core/build/storage/migration/UpdateAssistant.js:96:24)\n' +
      '    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n' +
      '    at async CloudAgent.initialize (/www/node_modules/@credo-ts/core/build/agent/BaseAgent.js:106:13)\n' +
      '    at async CloudAgent.initialize (/www/node_modules/@credo-ts/core/build/agent/Agent.js:120:9)\n' +
      '    at async initCloudAgent (/www/build/agent/initCloudAgent.js:59:5)\n' +
      '    at async run (/www/build/index.js:13:9)'
  }
}
ERROR: StorageUpdateError: Error updating storage (updateIdentifier:<PHONE_NUMBER>689): Export is only supported for SQLite backend

As now we are using copyTo method from Askar, I think it can be possible to export from postgres database (and even do a dump to sqlite), so there are big chances that this wallet backup procedure can work by just removing the line where the error is thrown. However we'll need to test it because the whole update process (which does wallet export + delete + import in case of an error) is originally meant for file system-based DBs and may have issues in a 'real' database.
I think that for this particular case it is OK to dump the backup to an sqlite file and then import it in case of error during migration. Not sure if this will be a good solution in long-term, thinking on cloud wallets with a massive amount of data (either mediators or multi-tenancy agents). I guess in those cases users may prefer to disable autoUpdateStorageOnStartup and run the update process manually to get more control.

I think we should maybe add a config option that indicates you have created a backup yourself. For postgres it makes sense if you create regular backups anyway, and e.g. with neon.tech (which we use for Paradym) we can go back to any point in time from the last X days because they use WAL to keep track of changes to the DB over time.
So i think we shouldn't support a backup strategy for non file based dbs

This has been adressed. New issues can focus on improveents / extensions


Release Date for Commander 2014 and FtV:Annihilation

The cards from these set are listed as having a null release date.
"From the Vault: Annihilation" Per the official site, the release date for this set was August 22, 2014. Link: http://archive.wizards.com/Magic/magazine/article.aspx?x=mtg%2Fdaily%2Farcana%2F1426
"Commander 2014" Per the official site, the release date for this set was November 7, 2014. Link: http://magic.wizards.com/en/articles/archive/news/announcing-magic-gathering-commander

Unfortunatley, the service will be taken offline and I will no longer be working on this code, August is the deadline for this.

Any reason why you are having to take this service offline?


Align the base64 feature branch with develop

What does this PR do?
Resolve conflicts between develop and the base64 feature branch
Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1594 (50f22a9) into feature/base64-jmoskovich (d4c8e5b) will increase coverage by 0.15%.
The diff coverage is 90.42%.

@@                      Coverage Diff                      @@
##           feature/base64-jmoskovich    #1594      +/-   ##
=============================================================
+ Coverage                      83.44%   83.59%   +0.15%
=============================================================
  Files                            446      451       +5
  Lines                          15181    15426     +245
  Branches                        2288     2306      +18
=============================================================
+ Hits                           12667    12894     +227
- Misses                          1912     1919       +7
- Partials                         602      613      +11

Files Changed
Coverage Δ

...roid/core/internal/data/upload/NoOpDataUploader.kt
0.00% <0.00%> (ø)

...tadog/android/core/internal/persistence/Storage.kt
100.00% <ø> (ø)

...ain/kotlin/com/datadog/android/rum/CloseableExt.kt
90.91% <ø> (ø)

...lin/com/datadog/android/rum/resource/ContextExt.kt
83.33% <ø> (ø)

...com/datadog/android/rum/resource/InputStreamExt.kt
50.00% <ø> (ø)

...droid/rum/tracking/FragmentViewTrackingStrategy.kt
87.04% <ø> (+6.00%)
:arrow_up:

...om/datadog/android/telemetry/internal/Telemetry.kt
12.50% <0.00%> (-7.50%)
:arrow_down:

...c/main/kotlin/com/datadog/android/trace/SpanExt.kt
88.24% <ø> (ø)

.../datadog/android/trace/sqlite/SqliteDatabaseExt.kt
91.67% <ø> (ø)

...m/datadog/android/okhttp/trace/OkHttpRequestExt.kt
100.00% <ø> (ø)

... and 50 more

... and 10 files with indirect coverage changes