DefaultAzureCredential() not being used with anon=False keyword passed in storage_options

Versions

adlfs: 2023.1.0
fsspec: 2023.4.0

Summary:
Hello,
I am not experiencing the expected behaviour introduced in #262 and documented in the project's README > Details > Setting credentials > 2:
"2. Auto credential solving using Azure's DefaultAzureCredential() library: storage_options={'account_name': ACCOUNT_NAME, 'anon': False} will use DefaultAzureCredential to get valid credentials to the container ACCOUNT_NAME. DefaultAzureCredential attempts to authenticate via the mechanisms and order visualized here."
The following code snippet outputs the expected return of the containers list:
from azure.storage.blob import BlobServiceClient
from azure.identity import DefaultAzureCredential
name : str = "<redacted>"
print([a for a in BlobServiceClient(f"https://{name}.blob.core.windows.net/", DefaultAzureCredential()).list_containers()])

verifying that the managed identity for this VM has the right permissions (Storage Blob Data Contributor).
However, the following code
import fsspec
container : str = "<redacted>"
subpath : str = "<redacted>"
fallback_options = {"account_name":f"{name}", "anon": False}
fsspec.filesystem("az", storage_options=fallback_options)
fsspec.get_mapper(f"az://{container}/{subpath}", storage_options = fallback_options})

run in the same environment throws the error:
ValueError: unable to connect to account for Must provide either a connection_string or account_name with credentials!!
Is anyone able to identify why the DefaultAzureCredential fallback is not being triggered even though I have specified the anon=False keyword?
Thanks for any help.

I encountered the same problem when running code that uses adlfs on ComputeInstance (CI) in AzureML with User-managed identity.
The identity has correct permission, which I can confirm running:
az login --identity --username xxx
az storage blob list --account-name SANAME --container-name MYCONTAINER --output table

However, it seems that automatic credentials resolution takes SystemAssigned Identity instead of User-manged identity assigned to the CI. Looking into DefaultCredentials Resolution Order Managed-identity should be correctly resolved, but it is not.
It seems like CI always have SystemAssigned Identity (?) and it may take precedence over User-managed identity. Digging into Azure identity python SDK it seems like setting a single environment variable should work and it indeed does:
import os
os.environ['AZURE_CLIENT_ID'] = 'xxx'
storage_options = {'account_name': SANAME, 'anon': False}
ddf = dd.read_parquet('az://MYCONTAINER/*.csv', storage_options=storage_options)

What would be nice for adlfs is an option to provide two arguments to storage_options, namely: storage_options = {'account_name': SANAME, 'client_id': 'xxx'} and as a result passed client_id should be used to fetch credentials. Currently such combination results in error: ValueError: secret should be an Azure Active Directory application's client secret


Fix dry_run issues

This MR does two things:

Fixes an exception generated due to a removed library import in contentctl (yaml library was removed when merging a previous MR by accident)
remove docker pull when doing a dry_run to save time.

LGTM!


Use Kalman filter for state estimation

Currently, the Oracle observer is used and results in differences between measured steer angle and the observer virtual steer angle even with a correct initial value.

Kalman filter matrices still need to be tuned but this is implemented here: https://github.com/oliverlee/phobos/pull/66

Timing info using test branch https://github.com/oliverlee/phobos/tree/test-use-kalman-filter-observer:
[71c9bcb] system computation time - best: 340   worst: 345      last: 343 us
        dynamics computation time - best: 252   worst: 259      last: 252 us
[71c9bcb] system computation time - best: 340   worst: 345      last: 343 us
        dynamics computation time - best: 252   worst: 259      last: 252 us
[71c9bcb] system computation time - best: 340   worst: 345      last: 343 us
        dynamics computation time - best: 252   worst: 259      last: 252 us

Merged: https://github.com/oliverlee/phobos/commit/ae78115521668cc2a305daaafd45ada47ab3123f


How to troubleshoot failed update

It looks like I'm not able to update my Application via AppImageUpdate but it used to work. Now I get following when I try to update it:
$ ./AppImageUpdate-x86_64.AppImage ./ksnip-1.6.0-continuous.AppImage
AppImageUpdate version 1-alpha (commit 6f2d028), build 400 built on 2019-02-07 22:44:08 UTC
Fetching release information for tag "continuous" from GitHub API.
None of the artifacts matched the pattern in the update information. The pattern is most likely invalid, e.g., due to changes in the filenames of the AppImages. Please contact the author of the AppImage and ask them to revise the update information.
ZSync URL not available. See previous messages for details.

How can I troubleshoot such an issue? The zsync file is there on the release page: https://github.com/DamirPorobic/ksnip/releases/tag/continuous

Get appimageupdatetool and run it with -d on the AppImage. It should display the update information. Please post the output here. It's probably wrong.

https://github.com/DamirPorobic/ksnip/releases/download/continuous/ksnip-1.6.0-continuous.AppImage has gh-releases-zsync|DamirPorobic|ksnip|continuous|ksnip*-x86_64.AppImage.zsync as a quick wget "https://github.com/DamirPorobic/ksnip/releases/download/continuous/ksnip-1.6.0-continuous.AppImage" -O - | strings | grep zsync shows.

I see, yeah that was a recent change. And there is no way to drop that suffix, it must be present?
Anyways, thanks for the investigation.

Well, you can drop it, sure, but you shouldn't. It's very important for users to see what platforms an AppImage will be running on. There's a growing community of ARM users (especially through chromebooks and that stuff), and there's a lot of people who still use i386 for some reason...

Ok, thanks for the help and feedback.


Client can only connect locally

For some reason, the HTML client can only connect to a local server; when ever a connection attempt is made to dwarftowers.com or zomis.net, the client will never connect.
However, when a local server is running, the client connects just fine.

I may be remembering incorrectly, but I do not think this problem was here before.

+1, client appears to time out after a period of time. No immediate error, connection just isn't established.

zomis.net is not running a webserver at the moment. dwarftowers.com might be running an outdated one.
What happens in the client? Is a "Incompatible server" or "Server not responding" message or anything shown?

@Zomis at the time there was no error message. The client would (eventually) time out trying to connect to the websocket.

Then I believe the real issue here is that there are no good error messages shown. The client can't really control which servers are running and which version they are running. If the servers are up and running a matching version, then the client can connect.


Compile CSharp project on script change only

This implement the proposal https://github.com/godotengine/godot-proposals/issues/2231
When "Run project" is triggered from the editor, a check is added to compare the last write times of all .cs files against the last build time. If any file is newer, the build occurs as usual. Otherwise, the build is skipped, and the project starts immediately.
This should reduce iteration times for C# developers, considering many project runs do not involve C# modifications.
Without this PR:
https://github.com/godotengine/godot/assets/81109165/7008b579-6f4f-4972-b023-65e9e099dd18
With this PR:
https://github.com/godotengine/godot/assets/81109165/9e71f94f-5d34-4173-b61d-25732944cf8b

I am not sure checking time on *.cs is always sufficient. For instance: one could have changes in their NuGet dependencies, or changes in a separate local C# project that'd need to be rebuilt, etc. A quite common case would be the following project structure, where I would not want the editor to not rebuild when I am working in ./Foo.SourceGenerators/.
.
├── Foo
│   ├── ...
│   ├── Foo.csproj
│   └── project.godot
├── Foo.SourceGenerators
│   ├── ...
│   └── Foo.SourceGenerators.csproj
└── Foo.sln

Fair point!

NuGet dependencies: A check for the last write time of the .csproj file is already implemented.
For code outside the project folder, I could implement a scan of the .sln file to find other projects and scan those folders, or I could simply scan the folder containing the .sln file when a Solution Directory is defined in the project settings.
I could also implement an opt in/out option to keep the "always rebuild" behavior.

What do you think of this suggestion?

I don't think checking the last modified time on the .csproj is sufficient either. The user could have an MSBuild task that generates code based on other non-C# files, so we'd need to know if the input files for that task changed. Trying to cover every possible scenario seems to be more trouble than it's worth.
.NET builds are already incremental by default, so it should already only be doing the minimum necessary to figure out what needs to be rebuilt. It's possible that this could be improved upstream, you're welcomed to make a PR in the .NET repos if so. But it's preferable to have a build process that always works reliably, rather than a fast build that breaks for many user scenarios.
Perhaps we could have an option to disable building on launching the game entirely, and then users would have to trigger a build manually pressing the build button.

Yes, I definitely feel like thinking we'd have better ways of knowing when to do what than the toolchain itself is kind of a wild goose chase.

Another approach:
dotnet watch build -- /bl:build.binlog and watching the binary build log file build.binlog.
This way, we don't need to guess what the tooling does and still have a way to quickly and reliably detect changes.

Interesting suggestion. I did not thought using dotnet watch. I'll check that thanks.


Pressing "Enter" when prompted after trying to remove rule will prompt the message again

The re-prompted message will overlay the older message and this can be continued indefinitely.
The "enter" is probably hitting the "X" button on the rule and not the "Ok"/"Cancel" button on the modal.

Can you write the exact steps to reproduce the bug?

Press "X" to delete rule
Press "Enter"
Repeat step 2


Extract news archive pages metadata which can be used for search engines like Algolia

Extract archived news page metadata

Title
Description
Content
Link
Thumbnail URL
Tags
Topics
Country
Category

Fixing the following issues

Update page link in search metadata with /archive
Update asset processor to add news page banner images with list transformer
Fix asset and image download issue for archived pages

@nilsml Done with the extraction of news page search metadata. This is initial version and this needs to be smoke tested when we have search engine service is in place.


Will there be a qlauncher for the steam version?

I just want the function to limit the fps,
It's much better than nvidia profile inspector.

It is possible to use the same mechanism that QCLauncher uses to limit FPS on the Steam version. Go to your Quake Champions properties in Steam, click the SET LAUNCH OPTIONS... button and enter: --set /Config/CONFIG/maxFpsValue n replacing n with a number. As for a Steam version of QCLauncher, it should not be needed, however, I will continue to support the software for the considerable number of people who still play via Bethesda.

thanks, man.


[BUG] size has no effect when creating a terminal and recording?

Describe the bug
See #67 (same code):
from io import StringIO
import pytermgui as ptg

code = """
while True:
    if condition:
        print("Hello")
    else:
        print("Goodbye")

    input()
"""

with ptg.Terminal(stream=StringIO(), size=(20, 20)).record() as recorder:
    recorder.write(ptg.tim.parse(ptg.highlight_python(code)))
    print(recorder.export_svg(inline_styles=True))

I can't seem to get the desired size for the generated SVG. It seems it uses my actual terminal size (vertical screen) rather than what I'm providing using the size parameter (zoomed out to see the full size):

Expected behavior
Respect the size value of the terminal when exporting a record to SVG.
System information
PyTermGUI version 6.2.1

System details:
    Python version: 3.8.13
    $TERM:          xterm-256color
    $COLORTERM:     truecolor
    Color support:  ColorSystem.TRUE
    OS Platform:    Linux-5.17.9-arch1-1-x86_64-with-glibc2.35

There is a bit of weirdness with how Terminals are handled in the library, and it will be focused on in the new documentations. Basically, there exists a global terminal that basically everything uses to write/read information to, and it can be set using the ptg.set_global_terminal function.
This shouldn't affect your use case, but it seems it does, since changing it to this:
terminal = ptg.Terminal(stream=StringIO(), size=(20, 20))
ptg.set_global_terminal(terminal)

with terminal.record() as recorder:
    recorder.write(ptg.tim.parse(ptg.highlight_python(code)))
    recorder.save_svg(inline_styles=True)

Seems to work perfectly well. Will look into it further, but using it this way should help for now.


cfn-nag issues

when cfn-nag is ran against the template aws/FSS-Scanner-Stack.template this error occurs:
| FAIL FATAL
|
| (): found unexpected ':' while scanning a plain scalar at line 228 column 69
Failures count: 1
This happens on all AWS pseudo parameters that are used.  They need to be in double quotes for cfn-nag to not report a failure.  This is important because we are trying to deploy this through Control Tower customizations to member accounts and it runs through a cfn-nag validation before it deploys.  We've had to modify these pseudo parameters to pass the scan:
!Ref AWS::NoValue
!Ref AWS::StackId

Hello ✋🏽 @richardsonky
I couldn't find this error here, I ran the cfn-nag and had some warnings but not the same results. Could you double-check if you're using the latest version of the template and if there any additional parameter being used in the cfn-nag to scan the template?
@jack-c-tang

This happens on all AWS pseudo parameters that are used. They need to be in double quotes for cfn-nag to not report a failure.
Does the tool report any reason to follow this rule? I don't see any advantage to enforce it.

Hi @richardsonky,
I ran the cfg_nag scan with the latest scanner template and it didn’t report any failure. Could you share more information and the command/parameters are used?

it looks like aws control tower customizations is currently using an old version to scan (0.7.2).  i'll contact support to try and get it upgraded.  thanks for checking.


Polkadot Protocol > Architecture > System Chains > Overview

Migrates the https://wiki.polkadot.network/docs/learn-system-chains Wiki page

@eshaben @dawnkelly09 Do you think replacing the table with just the headings, with "Encointer"  being a sub-section under collections, as this is the only exception for Kusama. WDYT?

@eshaben @dawnkelly09 Do you think replacing the table with just the headings, with "Encointer" being a sub-section under collections, as this is the only exception for Kusama. WDYT?

Yeah, I think the tabs get lost between the headers. I'd move it out of the tabbed element. It might be nice to have a mermaid diagram that has a Polkadot element and a Kusama element that list the supported system chains for each, so users can quickly glance at it and understand what system chains belong to what ecosystem. Then yeah have the following sections go over each system chain


New icon based on the Angler fish

People have commented that they like that angler fish icon we have on Slack. I personally like it more than the trowel. Let's try to build our own angler fish icon and put it into a few spots on the site.

I'm happy with this for now. Closing.


Migrate to typescript 12481

Summary

Migrate utils/markdown module and tests to TypeScript
Ticket Link

Fixes https://github.com/mattermost/mattermost-server/issues/12481

Thanks @karlmarxlopez for the pull request!
Per the CONTRIBUTING.md file displayed when you created this pull request, we need to add you to the list of approved contributors for the Mattermost project.
Please help complete the Mattermost contribution license agreement?
This is a standard procedure for many open source projects. Your form should be processed within 24 hours and reviewers for your pull request will be able to proceed.
Please let us know if you have any questions.
We are very happy to have you join our growing community! If you're not yet a member, please consider joining our Contributors community channel to meet other contributors and discuss new opportunities with the core team.

@devinbinnie readyfor review

@devinbinnie Hi, I've fixed the TS errors.

@karlmarxlopez You have some tests failing:
Summary of all failing tests
 FAIL  utils/text_formatting_search_highlighting.test.jsx
  ● TextFormatting.searchHighlighting › search term highlighting in a code block

    expect(received).toEqual(expected) // deep equality

    Expected: "

words in a sentence
words in a sentence
"
    Received: "

words in a sentence
words in a sentence
"

      122 |             const output = TextFormatting.formatText(testCase.input, options).trim();
      123 |
    > 124 |             expect(oneLineTrim(output)).toEqual(oneLineTrim(testCase.expected));
          |                                         ^
      125 |         });
      126 |     }
      127 |

      at Object.toEqual (utils/text_formatting_search_highlighting.test.jsx:124:41)

  ● TextFormatting.searchHighlighting › search match highlighting in a code block

    expect(received).toEqual(expected) // deep equality

    Expected: "

words in a sentence
words in a sentence
"
    Received: "

words in a sentence
words in a sentence
"

      122 |             const output = TextFormatting.formatText(testCase.input, options).trim();
      123 |
    > 124 |             expect(oneLineTrim(output)).toEqual(oneLineTrim(testCase.expected));
          |                                         ^
      125 |         });
      126 |     }
      127 |

      at Object.toEqual (utils/text_formatting_search_highlighting.test.jsx:124:41)

 FAIL  utils/message_html_to_component.test.jsx
  ● messageHtmlToComponent › latex

    expect(received).toMatchSnapshot()

    Snapshot name: `messageHtmlToComponent latex 1`

    - Snapshot
    + Received

    @@ -3,16 +3,14 @@
          This is some latex!

,
        "
      ",
        <LatexBlock
    -     content="x^2 + y^2 = z^2
    - "
    +     content="x^2 + y^2 = z^2"
        />,
        <LatexBlock
    -     content="F_m - 2 = F_0 F_1 \\dots F_{m-1}
    - "
    +     content="F_m - 2 = F_0 F_1 \\dots F_{m-1}"
        />,

          That was some latex!

,
      ]

      26 |         const html = TextFormatting.formatText(input);
      27 |
    > 28 |         expect(messageHtmlToComponent(html)).toMatchSnapshot();
         |                                              ^
      29 |     });
      30 |
      31 |     test('link without enabled tooltip plugins', () => {

      at Object.toMatchSnapshot (utils/message_html_to_component.test.jsx:28:46)

Snapshot Summary
 › 1 snapshot failed from 1 test suite. Inspect your code changes or run `npm run test-ci -- -u` to update them.

Looks like there's some conflicts from the recent Markdown changes I made. I was worried that was going to happen. Unfortunately, we needed that PR in for our October release, so it ended up going in at a bad time.
Please let us know if you need any help or have any questions about how to resolve those conflicts.

Hi @hmhealey, sorry for the delay. I'm busy this past few days, but I'll finish this hopefully on weekend.

No worries if it takes some more time. Let us know if you run into any issues.

Hi @hmhealey , I think this is ready for review

@karlmarxlopez Just so that it doesn't get lost, I responded above with a link to a branch I made that fixes the type of the modified methods in the Renderer class without having to fight with ESLint about it. Let me know if that works for you, but if it doesn't I'm fine with sticking with what you currently have

Hi @hmhealey, please take a look if this is good.

Hi @hmhealey, I'm getting on :install stage
npm ci
cd node_modules/mattermost-redux && npm i && npm run build
npm ERR! Unexpected end of JSON input

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2019-10-30T21_57_24_120Z-debug.log

I think something got merged incorrectly in the package-lock.json. I get that same error when I try to run npm ci locally. I think it's happening because these lines were removed

@hmhealey I need help passing the install stage of the CI

I'll take a look. That error is really strange since I don't get that locally at all

@karlmarxlopez The error was being caused because the hashes for those packages in package-lock.json didn't match the actual hashes of those files. I don't know why the error that was happening on CircleCI doesn't reproduce locally, but updating those fixed that part of the build at least.

Now that that passes, there's still some linting and type checking errors occurring if you could take a look at those

@hmhealey Hi, please check if this can now be merged.

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3845.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Thank you @karlmarxlopez
@hmhealey I ran through some markdown tests as we discussed and see that some /code text is not rendering as expected. You can log in as sysadmin on the test server to see the markdown posts I am referring to (see Basic Markdown Testing post)

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@jgilliam17 Which tests in particular are incorrect? I copied that text to community-daily, and everything seems to be rendering the same for me there

/update-branch

New commit detected. SpinWick will upgrade if the updated docker image is available.

Mattermost test server updated with git commit 91a24acf3b361b08e78d0adb6836a3e20610aa9d.
Access here: https://mattermost-webapp-pr-3845.test.mattermost.cloud

@jgilliam17 Which tests in particular are incorrect? I copied that text to community-daily, and everything seems to be rendering the same for me there

It was the test for carriage return and carriage return code block. Unless that's also broken on daily. I'll have to check there and if test data I am using is correct.
Expected:

Actual:

New commit detected. SpinWick will upgrade if the updated docker image is available.

New commit detected. SpinWick will upgrade if the updated docker image is available.

Test server creation failed. See the logs for more information.

Mattermost test server updated with git commit 09cbed8318f56adc7b66c01b900f2bbd6cc8e286.
Access here: https://mattermost-webapp-pr-3845.test.mattermost.cloud

Test server destroyed


Modify ingest.py to call pip installed insert code & build test

The code to do Cassandra inserts is pulled almost verbatim from my gkdbutils code.  I (Ken) need to modify the gkdbutils code so that the inserter code can be included directly (saves having to maintain two identical code bases).  Then modification of ingest.py code required to include the relevant function.  It would also be useful to have a unit test, though the test is really an integration test (since it requires access to a test Cassandra instance).

Superseded


AzureOpenAI features not supported and are hard coded

🐛 Describe the bug
In the documentation and the code base there seems to be a discrepancy. In the documentation, it says to use the following environment variables while using AzureOpenAI services, but the name of the environment variables differs. Secondly, I was looking at the source code and it seems that the api version was hard coded. It would be nice to pull the from yaml config. Thirdly, I couldn't see the embedder for AzureOpenAI in the source code.
Thank you!

Facing same issue

I was able to load the version from the YAML config, in the latest version, below an example how it worked.
os.environ["OPENAI_API_VERSION"] = "2024-02-01"

I will take this up

@neilbhutada
My PR will fix the mismatch in the documentation for azure openai environment variables, and let you add api_version from the config.
Azure OpenAI embedder is already present in the code and docs. You can put your deployment name in config and use it.
However, I feel it should be in a different file such as embedders/azure_openai.py, and self.config.deployment_name should not be used to differentiate between azure and openai embedded. @Dev-Khant let me know if you think the same, will make that change.
Please let me know if you have any other issues.


to-do: dark mode

implement a dark mode functionality (and subsequent light mode button) and animations
https://icons.getbootstrap.com/icons/moon-stars-fill/
https://icons.getbootstrap.com/icons/sun-fill/

implemented! ✅


Cannot read property 'displayName' of undefined

i am getting undefined error on itemComponent.
the code from view.jsx
`const componentName = itemComponent.type.displayName || itemComponent.type.name;`

if you can explain MyMasonryItem class it would be really helpful right now

Hey layout engine is now working fine now but the items keep disappearing from the page it happens sometime on scroll and almost always on window resize i don't have infiniteLoading just finite set items.
any idea what could be causing this?

i removed the onclickListeners. now scrolling doesn't make the images go away but now the most images doesn't even load on mobile devices!!! will on desktops it does.
what could be causing this? really could use your help......

Please provide a REPL or example app where the issue is occurring and I will take a look 👍

I am new to REACT. I can't understand, how the MyMasoneryItems are instantiated. I do have to provide an array of objects of this type, or any other type that implements that specification, right? How do I technically do that? Many greetings in advance from Berlin!

@matthiaskaemi there is an example in the file https://github.com/coleturner/react-components/blob/cf3533afb5384267890d7930d2dc8fef3d45a278/Masonry/View.jsx#L21

@matthiaskaemi you can use whatever properties you like so long as they fit the shape of the props expected by MyMasonryItem or whatever you prefer to call it :)
Side note: this issue has run it's course. Folks please feel free to open a separate issue


chore: upgrade action release

otherwise the release action fails. includes an arbitrary formatting change in the values file to trigger a new release.

:tada: This PR is included in version 2.0.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


Raven freezes hapi server and converts 500 error to empty 200

Do you want to request a feature or report a bug?
Bug
What is the current behavior?

With raven enabled, a hapi server hard freezes for 5 seconds and sends an empty 200 response

What is the expected behavior?

hapi server should send a 500 response immediately and not freeze

Which versions of Raven.js, and which browser and OS are affected by this issue? Did this work in previous versions of Raven.js? Are you using the CDN (http://ravenjs.com)? Are you using hosted Sentry or on-premises? If on-premises, which version (e.g. 8.7.0)?
Everything is documented in detail in this git repository. It includes a fully working Minimal Complete Verifiable Example, all version details, etc.

Hey @focusaurus, thanks for the very useful repository, it definitely sped up the process.
As you already noticed, it's caused by calling stringify on a whole kwargs object, which also includes context data. However, the main issue here is that hapi request object is using a lot of references, to other objects and doesn't only contain request information we need.
We extract things like host, port, path etc., however, we parsed only req object and mentioned in the docs that request object should follow http interface https://docs.sentry.io/clientdev/interfaces/http/ but it's apparently not always the case. We'll try to guard against it in next patch version. I already tested it with your repro case and it works fine.
Fixed in https://github.com/getsentry/raven-js/pull/1404

@focusaurus released as 2.6.3


Fix buggy init method

Describe your changes
It's a silly little thing that I noticed when looking at something else.  The facet class in ggplot/facet_wrap.py was missing the self argument in its __init__ method.  This wasn't an issue, since this class is never used directly.  Only its subclass, facet_wrap is used, which has its own functional __init__ method, which doesn't call the superclass method.  Since the method supposed to do anything, I just removed it.
I'm not sure why flake8 or black didn't raise a fuss about this.
Issue number
Closes #X
Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[x] Added docstring documentation and update the changelog (when needed)

📚 Documentation preview 📚: https://jupysql--999.org.readthedocs.build/en/999/

I added a changelog entry in order to let testing continue, but this fix doesn't deserve that!  I seem to be unable to add the relevant tag to turn off this requirement.
I've been seeing the failure of test-sql-alchemy-v1 in my own testing.  Four tests are failing, all related to persisting with DuckDB.  They complain about needing a list of parameters, and they seem to be provided with a (nested) tuple of parameters.  This seems completely unrelated to my change.  I wonder if some requirement released a new version, and now it's pickier about data types.
The Py3.9 on Windows failure is about some .tcl file not being present.  No idea what's up there, but again it seems completely unrelated to these changes.
The integration test seemed to fail at the very end, with errors about what seem like internal tables not existing.  Could an over-eager teardown have started before the tests were done?

I've added the no-changelog label, can you remove the changelog entry? once that's done, we can merge. as you suggested, the test issues are not related to your change

I force-pushed to the branch to remove the changelog commit.  Hope that didn't break anything too badly.

thanks for your contribution!


Release version is missing for conan-integration

Hi Robert,
Could you please create a release or at least a (1.0.0?)-tag for vsgImGui that preferably works with vsg-1.0.3, so I can create the conan-integration for this component?
Apart from this it would be really nice if the submodules referenced could point to releases/tags. This make managing them less awkward and doesn't give that "random version"-taste.
A little background: conan bases its dependencies on tar-balls/zips in conjunction with the specific sha256 to be used as a version.

Would it need to be 1.0.s or could it be a 0.x.x release?
vsgImGui pulls in ImGui and ImPlot as giit submodules, is his compatible?

Hi Robert,
it can be any release version. The submodules are ok, but it would be much better if they'd reference a tag or branch.
I've already moved towards vsg 1.0.5 for testing. So maybe we could stick to 1.0.5 and use the current master as a "stable" reference.
Cheers
Sebastian

Closing as we now have a 0.1.0 release:
https://github.com/vsg-dev/vsgImGui/releases/tag/v0.1.0


Console task completion messages are not meaningful

For example, the above console output after hitting "Clear RACK" do not convey if the action was actually completed or if there were unexpected errors

We need to switch to RackConsole and pin it, see if you are able to pin the RackConsole successfully
Clicking on the highlighted button in yellow shows you the list of consoles available, see if you are able to pin it

Can we make it so that after every operation we get a "Operation Successfully Completed" or "Operation Ended with Errors" in a default console without the user having to deal with adjusting the console?

No need to automatically pi a console, just print an "<OPERATION_NAME> Completed" message on RACK console

There is a done message now for operations, will be merged with latest PR


MorePowerTool no longer available

What am I supposed to use when MPT is no longer available? Where do I get a safe unmodified archive of it? as even the installer isn't offline, it downloads a "MorePowerToolCurve_1_3_8_final.zip" which again is also no longer available.

resolved, the websites set up in a blog manner, and the links are all broken unless you find the proper executable.
https://www.igorslab.de/installer/MorePowerTool_Setup.exe

Sorry @royalgraphx for my delayed answer.
This issue had gone unnoticed by me. I just read it now.
I see that Igor's Lab site has changed its design, as you say, and that he has separated downloads of stable versions from beta versions. It can be a bit complicated to get to the MorePowerTool download. At least it's still available.
Thank you for your notice even though it is very late.


Fix issues when failing to setItem

Some browsers like Firefox can raise an exception when the localStorage quora is exceeded.

I think this one can be safely closed, as the plugin is now returning the Error provided by the browser, if the quota is exceeded for example.
It is true that the plugin could simply throw on its initialization if there is no space left in the selected storage. I ended up wrapping the plugin instantiation in a try/catch in some cases. @robinvdvleuten, do you think this should be better documented?

It is not a good idea to return undefined when setItem() fails. When this is required for your specific use case, you can just create your own wrapper around localStorage and pass that to the plugin.


The customerSuccess is offline

I try to send a long message and there will feedback "The customerSuccess is offline"

We has fix bug today , please git pull and update your code.

Ok, I pull the code and refresh to try to send a long message. This time the message is success send.


Don't allow skip mnemonic confirmation for Mainnet

Please remove "Skip" option on mnemonic confirmation when creating Mainnet profile (for Testnet is OK to keep it).

Also "Backup Phrase" allow you to click "Next" without checking and making backup of mnemonic.

I think there is a second option which is on the mainnet if a person chooses to skip, then there is a big warning message that they can't get rid of until they prove they have backed it up.
The reason is that some users want to just "check out a wallet" to look around before they buy or test. To go through the process of backing up and re-entering the 24 words stops some people from checking out the wallet.
But I agree with the general point that it is important to make people back up their wallet. Otherwise people don't do it and lose funds.

Fixed.


Fix preferred-versions format

Cabal can not parse the preferred-versions format as written by foliage before this PR as the preceding package name is missing, so all deprecations were ignored before this PR. E.g. the mtl/preferred-versions file from the Hackage index looks like this:
mtl <2.1 || >2.1 && <2.3 || >2.3

Relevant links to the Cabal source code:

The contents of the preferred-versions files are parsed here. It is a line-wise format with simple comments, where lines are parsed via the Parsec instance for Dependency.
Sadly, for secure repos, Cabal uses parsePreferredVersions, which silently ignores parse errors. This could be reported upstream, especially as warnings are emitted for RepoLocalNoIndex.

I used this code to create this repo (src) for https://github.com/haskell/cabal/pull/8997; this can be used to verify that deprecations now work with this PR.

Thank you!
@michaelpj @yvan-sraka can you review (and merge)? It's public holiday on Monday.
@amesgen do we have already wrongly formatted preferred-versions on chap? If yes, this means we need to reset the index (😥) as the fix will change old entries.

do we have already wrongly formatted preferred-versions on chap? If yes, this means we need to reset the index (disappointed_relieved) as the fix will change old entries.

No, CHaP does not yet have any deprecations, e.g. this returns no entries:
curl -sL https://input-output-hk.github.io/cardano-haskell-packages/01-index.tar.gz \
  | tar tzf - | rg preferred-versions

Note for future ourselves (and new visitors :wave:). A deprecated package is avoided if possible but it will still be choosen if it's the only solution.
Consider the following setup:
❯ cat cabal.project
extra-packages: tagged

repository cabal-repo-override-example
  url: file:/home/andrea/cabal-repo-override-example-repo/_repo
  secure: True

active-repositories: hackage.haskell.org, cabal-repo-override-example:override

❯ tar tzf /home/andrea/cabal-repo-override-example-repo/_repo/01-index.tar.gz --full-time --utc | grep tagged
-rw-r--r-- foliage/foliage 2770 2023-06-02 18:05:25 tagged/<IP_ADDRESS>/tagged.cabal
-rw-r--r-- foliage/foliage  805 2023-06-02 18:05:25 tagged/<IP_ADDRESS>/package.json
-rw-r--r-- foliage/foliage   27 2023-06-02 18:37:52 tagged/preferred-versions

With this PR, foliage produces the correct preferred-version
❯ tar xzf /home/andrea/cabal-repo-override-example-repo/_repo/01-index.tar.gz -O tagged/preferred-versions
tagged <<IP_ADDRESS> || ><IP_ADDRESS>

but then cabal-install picks tagged-<IP_ADDRESS> anyway!
❯ cabal clean && cabal build tagged --dry-run -v3 | grep tagged
Warning: There are no packages or optional-packages in the project
targets: tagged
  tagged >=0 (user target)
  tagged <<IP_ADDRESS> || ><IP_ADDRESS>
[__0] trying: tagged-<IP_ADDRESS> (user goal)
[__1] trying: base-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged)
[__5] trying: tagged:+deepseq
[__6] trying: deepseq-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged +deepseq)
[__8] trying: tagged:+transformers
[__9] trying: transformers-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged +transformers)
[_10] trying: template-haskell-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged)
Component graph for tagged-<IP_ADDRESS>: component lib
...

This because cabal-repo-override-example:override has masked any other possible choice of tagged so tagged-<IP_ADDRESS> has to be.
If we add tagged-0.8.6 to the repository, we can see the prefereces taking effect:
❯ tar tzf /home/andrea/cabal-repo-override-example-repo/_repo/01-index.tar.gz --full-time --utc | grep tagged
-rw-r--r-- foliage/foliage 2509 2023-06-02 17:05:25 tagged/0.8.6/tagged.cabal
-rw-r--r-- foliage/foliage  804 2023-06-02 17:05:25 tagged/0.8.6/package.json
-rw-r--r-- foliage/foliage 2770 2023-06-02 18:05:25 tagged/<IP_ADDRESS>/tagged.cabal
-rw-r--r-- foliage/foliage  805 2023-06-02 18:05:25 tagged/<IP_ADDRESS>/package.json
-rw-r--r-- foliage/foliage   27 2023-06-02 18:37:52 tagged/preferred-versions
❯ tar xzf /home/andrea/cabal-repo-override-example-repo/_repo/01-index.tar.gz -O tagged/preferred-versions
tagged <<IP_ADDRESS> || ><IP_ADDRESS>

cabal-install is not picking the latest version of tagged anymore, because it's now deprecated.
❯ cabal clean && cabal build tagged --dry-run -v3 | grep tagged
Warning: There are no packages or optional-packages in the project
targets: tagged
  tagged >=0 (user target)
  tagged <<IP_ADDRESS> || ><IP_ADDRESS>
[__0] trying: tagged-0.8.6 (user goal)
[__1] trying: base-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged)
[__5] trying: tagged:+deepseq
[__6] trying: deepseq-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged +deepseq)
[__8] trying: tagged:+transformers
[__9] trying: transformers-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged +transformers)
[_10] trying: template-haskell-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged)
Component graph for tagged-0.8.6: component lib
...

Setting index-state to just before the deprecation makes tagged-<IP_ADDRESS> available again.
❯ cabal clean && cabal build tagged --dry-run -v3 --index-state='cabal-repo-override-example 2023-06-02T18:37:51Z' | grep tagged
...
targets: tagged
  tagged >=0 (user target)
[__0] trying: tagged-<IP_ADDRESS> (user goal)
[__1] trying: base-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged)
[__5] trying: tagged:+deepseq
[__6] trying: deepseq-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged +deepseq)
[__8] trying: tagged:+transformers
[__9] trying: transformers-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged +transformers)
[_10] trying: template-haskell-<IP_ADDRESS>/installed-<IP_ADDRESS> (dependency of tagged)
Component graph for tagged-<IP_ADDRESS>: component lib
...

Thank you @amesgen <3


[SDK ask] we should force typespec conversion without vertical changes

For vertical change definition, refer to TypeSpec Brownfields Conversion (Specification).  We should ensure no vertical changes when brownfield service is working on TypeSpec conversion, by ensure

The TypeSpec conversion should be implemented in the latest stable version. This means that the latest stable version needs to be rewritten using TypeSpec.
The rewritten of the latest stable version should introduce no functional breaking changes.

a related ask https://github.com/Azure/azure-sdk-tools/issues/8617


patchance, raysession: fix pipewire compatibility

TL;DR: this seems to make these apps work with PipeWire-emulated JACK.
Currently, running patchance or raysession with a setup like the following, where PipeWire emulates JACK,
{
  services.pipewire = {
    enable = true;
    jack.enable = true;
  }
}

these apps are unable function (a "JACK is not started !" message is shown).
This seems to happen because the wrapper in their derivations prefixes the LD_LIBRARY_PATH environment variable with ${libjack2}/lib. Since services.pipewire.jack.enable also functions by putting a path to its own version of libjack into this environment variable:
https://github.com/NixOS/nixpkgs/blob/080166c15633801df010977d9d7474b4a6c549d7/nixos/modules/services/desktops/pipewire/pipewire.nix#L408C34-L409
the wrappers for patchance / raysession end up clobbering them. The apps then load the libraries from libjack2 and as such try to connect to a (non-existent) "real" JACK server.
Instead, my PR makes the wrapper suffix the environment variable with libjack2's path. If the machine is running pipewire-jack, PW libs will be preferred, otherwise, the "traditional" JACK will be used.
My PR seems to fix the issue in a pipewire.jack setup. I have not tested with normal JACK, but it should keep working.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

patchance
raysession

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4795


OpsGenie integration

Adding OpsGenie service integration.  This is a slightly modified version of PR #58.

:+1:


more darkcoin->dash

note: old settings are preserved and converted into new one https://github.com/dashpay/dash/compare/v0.12.1.x...UdjinM6:darkcoindash?expand=1#diff-bd81cc35fa9249ec9e6a2ace6d3a4d59R78

Merged, thanks!


Cannot add a varient for a propduct in permission based dashboard of partners

This is explicitly mentioned in the documentation:

The permission-based dashboard currently does not support parent or child products. Supporting this requires a modelling change.

There's a ticket open (https://github.com/django-oscar/django-oscar/issues/675) to address improvements to variant handling, so closing this as a duplicate.


FullScreen Control is not working

Hello, I followed the documentation to add the fullscreen control but the control doesn't appear on the map.
HTML:
<agm-map [latitude]="latitude" [longitude]="longitude" [zoom]="15"
 [fullscreenControl]="true"
 [fullscreenControlOptions]="screenOptions">
</agm-map>

Typescript
Steps to reproduce and a minimal demo of the problem
https://stackblitz.com/edit/angular-google-maps-demo-bkmqme?file=app%2Fapp.component.html

Fullscreen works for me on Desktop and Android (Chrome/FF), but not on IOS. Same issue or should I open a new one?

Per Google documentation, iOS is not supported.

How to handle click event into map to show full screen map? help me please

Is full screen control still supported in v3? I am getting an error trying to use fullscreenControl property:
Can't bind to 'fullscreenControl' since it isn't a known property of 'agm-map'.

it is now done differently. Instead of an attribute, it's a sub element. <agm-fullscreen-control> For further questions, head toward our discord, or stack overflow


[Question]: Does this graphrag support multi-hop queries? It seems that the code doesn't seem to cover this aspect.

Describe your problem
Does this graphrag support multi-hop queries? It seems that the code doesn't seem to cover this aspect.

It's more like knowledge graph which is adopted from Microsoft graph rag


Support activity types

Discord supports multiple activity types like watching, and listening. Currently this library defaults to the Game activity type.
More info: https://discord.com/developers/docs/topics/gateway-events#activity-object-activity-types

Would be a really addition, atm apps that use this lib like spotube have to use the game activity type which doesn't make much sense


Bluetooth: tests: Fix config option dependencies

With the introduction of controller's advanced features
Kconfig option, some of the dependent options where not
enabled in the init tests build. Fixed by enabling the
advanced features Kconfig options.
Also, updated conf files to reflect latest set of Kconfig
options supported by the Bluetooth controller subsystem.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

I pushed a fix to this PR for the compiler warning that was causing CI to fail

Ok, seems there are more warnings, but this time in the controller side. @cvinayak I'll leave this one to you:
subsys/bluetooth/controller/ll_sw/ctrl.c:2613:29: error: ‘LLCP_ENCRYPTION’ undeclared (first use in this function)          (conn->llcp_type != LLCP_ENCRYPTION)) ||

@jhedberg I will handle the errors and push updates.

@cvinayak : IMHO this should not wait for the -Ofast fix. Turns out there is no trivial solution, so I need some time to fix it.

so, ok to merge then?

Wait, this might be broken ...

I expected
cmake -DCONF_FILE=prj_controller_dbg.conf -DBOARD=nrf51_pca10028 ~/zephyr/tests/bluetooth/init
to produce a .config that had CONFIG_BT_CTLR_FAST_ENC=y, but it didn't.
Commenting out this in prj_controller_dbg.conf
#CONFIG_BT_CTLR_PHY_2M_NRF=y
fixed the issue.
Perhaps the root cause is here:
config BT_CTLR_LE_ENC
	bool
	depends on !BT_CTLR_DATA_LENGTH_CLEAR && !BT_CTLR_PHY_2M_NRF

LE_ENC does not depend on ! PHY_2M_NRF does it?
I'm not familiar enough with bluetooth to be sure what's going wrong.

@SebastianBoe encryption and 2Mbit/s PHY are mutually exclusive in nRF51 at least

Hmm, okay, so I guess it is prj_controller_dbg.conf that is broken then.
It should not be both enabling FAST_ENC and 2M at the same time.

I'll leave this to you @cvinayak , I have my own bugs to deal with :)

@SebastianBoe in a nrf51_pca10028 FAST_ENC is disabled when PHY_2M_NRF is enabled... but on nrf51_pca10040 FAST_ENC is enabled as PHY_2M_NRF is invalid for nRF52.
This PR is good as-is, as these conf files are meant to cover Kconfig options under sanitycheck.

I don't like that a user can enable FAST_ENC and then it builds without FAST_ENC and there is no feedback to the user about something being amiss. But this is not my subsystem, so if this is okay for you guys then it is OK for me.

@SebastianBoe I don't like it either but this is how it was before CMake. If we want to fix that it should be a separate PR

Interestingly enough, the linux kernel's merge_config behaves as I would expect,
it warns you if you tried to enable something but it wasn't enabled.
https://github.com/torvalds/linux/blob/master/scripts/kconfig/merge_config.sh#L157
https://github.com/zephyrproject-rtos/zephyr/pull/5086#issuecomment-346063707


Classical Bluetooth SPP send throughput is very slow to Android (IDFGH-466)

When sending data using esp_spp_write() at more than ~ 5KB/s to an Android phone, it triggers a lot of ESP_SPP_CONG_EVT events and the communication slow down, this problem doesn't occur in other systems, like a Windows PC with the same ESP code
Commenting out this line as proposed by Markus Becker, this issue seems less frequent and no more ESP_SPP_CONG_EVT events with a high throughput
More discussion about this problem, and workaround: https://www.esp32.com/viewtopic.php?t=5613

Hi @copercini
Please pull the latest IDF and have a try. We have merged two MR related to SPP flow control. It will take several days to sync on Github.
Thanks
Weitianhua

Thanks for reporting and sorry for slow turnaround, the fix has been available at https://github.com/espressif/esp-idf/commit/8e4b919391eb5592f11438e76202265e6820abe0, feel free to reopen if the issue still happens. Thanks.


Declare documentRangeFormatting in JSON LS

See reason in https://github.com/Microsoft/vscode/pull/72124

We have a setting json.fomatter.enabled to configure whether the formatter is on or off. This needs to continue to work.
Fix has to:

pass in the initial setting through the InitializeParams (e.g. InitializeParams.initializationOptions.settings)
register the formatter like that:

		foldingRangeProvider: enableFormatter ? { id: 'json.Formatter ' } : false ;
	};

use RegistrationRequest and UnregistrationRequest to register/unregister by id when the setting changes.

Why not just enabling it by default? AFAIK, the capability here is for the manual formatting, not the automated one. I don't get what's the value of making the manual one off by default and -as mentioned in #72124- I see it introduce complexity? Is it because VSCode uses another formatter?

Enabling by default on the server means the language client will register the formatter with VSCode. If you have multiple formatters registered for a given document (e.g. also prettier registers a JSON formatter), we now need to prompt the user which formatter or sometimes even pick just one.
Until the last VSCode build, the only way to avoid the conflict was to configure wach formatter and e.g. use the 'json.formatter.enabled' setting to disable the built in formatter. When false, the sever deregisters, causing the client to deregister.
Even there are now new ways of solving formatter conflicts in VSCode, we can't remove the setting in the LS server. Not only to be backward compatible to VSCode users but also to JSON LS clients.

It's a client issue (how to dispatch a request to multiple possible provider) leaking into a server implementation. Dealing with enablement in the server isn't a proper solution when dealing with conflicts, the remediation of the conflict must be the client informed to pick one provider or the other.
I think it's much simpler as a contributor and as an "embedder" that I just tweak the JSON Language Server on my side. I don't plan to implement the described workflow, so let's close this PR.


zebra listings

this changes make my listings appear zebra-styled

my tex skills are limited, thus you probably want to adapt my changes to your quality standards ;)

merge for now, optionalize then


Add example of HTML5 scaffolding for an app template

Template 1
We need a starting point to give scaffolding a leg up.
This template should fit the current Material design app layout. Potentially, I would like to use it as the base for the next color editor similar to the one I built for Bootstrap at: Color Editor
View app template
UI Kit?

Added a new layout attribute am-slide</code for figure element to slide in figcaption on hover.

sand boxing (testing) HTML5 template 2

We have a portrait and a landscape template.Closing this.


Make package compatible with league/oauth2-client v2.0

Do you have plans on releasing a version compatible with league/oauth2-client v2.0
We have v2.0 included in our app because it's required by league/oauth2-linkedin and league/oauth2-google.
By looking at the change list https://github.com/thephpleague/oauth2-client/blob/master/CHANGELOG.md it seems like the only breaking change is "Rename getResponse() to getParsedResponse()" used in Slack.fetchAuthorizedUserDetails

@vkulov I'll have a look at implementing the new version at some point today. Thanks for using the library.

Hi @adam-paterson :) Thanks for your oauth provider. :+1:
I also need a new release with league's v2.0 compatibility because of a PHP 7.1 deprecation thrown with league/oauth2-client in v1. Are you still working on it? Do you need some help to get the fix done? I can make a PR if you does not have time to do it.
Thanks!

@pyrech I'm afraid I haven't had time to look into this yet. If you have the time feel free to submit a PR. All contributions are welcome.


Allow stopping blinking cursor in terminal

Check for existing issues

[X] Completed

Describe the feature
I cannot change blinking cursor in my Fish shell itself, it has to be done

Warp allows you to do this.
Please allow turning off blinking cursor in terminals in Zed too.
Similar to how you already have "cursor_blink": false for Editor cursor.
Thanks.

Although on 2nd thought the blinking part is useful as it shows that the terminal is in focus.
Still would be nice to have option for non blinking cursor in terminal.

Hi Nikita,
Similar to the keymaps, we already have this setting available, like so:
{
  "terminal": {
    "blinking": "off"
  }
}

You can use the command zed: open default settings to see the initial settings we think are sensible defaults in Zed, + documentation for each one :)


SyntaxError: Unexpected token 'export'

hello i have cloned vitesse project and then added gsap on it and very thing is fine unless i decided to build project
i have got this error
node_modules/gsap/ScrollToPlugin.js:127
export var ScrollToPlugin = {
^^^^^^

SyntaxError: Unexpected token 'export'

this link helped me to solve my problem
https://greensock.com/forums/topic/24799-syntaxerror-unexpected-token-export-on-npm-import/?do=findComment&comment=118873
i hope you check the problem

I guess gsap is aim to work client only? If so, you should use <ClientOnly> component to avoid including it in your SSG build.

no i just import it in my page and use it in mounted hook, it's not about the way i am using it's about the way i imported,
i have used it before for nuxt2 ssr and it was totally okay with that

@vuejstime add async callback on mount hook, use dynamic import to import the lib and remove the import: see the comment at top and the install method code here https://github.com/antfu/vite-ssg/issues/129#issuecomment-969149950


Move uninstall command to new subctl implementation

Signed-off-by: Janki Chhatbar<EMAIL_ADDRESS>

If this "moves" the command, shouldn't it be removed from where it was before? Is that planned for later?

If this "moves" the command, shouldn't it be removed from where it was before? Is that planned for later?

yeah it will be once the rest of the commands are "moved".


[jenkins-jobs] Simplify organization using matrix jobs

http://ci.hibernate.org/job/debezium-postgresql-matrix-test/
http://ci.hibernate.org/view/Debezium/job/debezium-mongodb-matrix-test/

Great stuff with the Matrix build. Should have done that long time ago :)


Can't install due to cltl.triple_extraction UnicodeDecodeError

When trying to install cltl-chatbots with py -m pip install -r requirements.txt, I get the following output:
Collecting cltl.brain
  Using cached cltl.brain-0.0.dev7.tar.gz (253 kB)
Collecting cltl.asr[impl]
  Using cached cltl.asr-0.0.dev5.tar.gz (5.1 kB)
Collecting cltl.vad[impl]
  Using cached cltl.vad-0.0.dev1.tar.gz (5.6 kB)
Collecting cltl.backend[host,impl]
  Using cached cltl.backend-0.0.dev1.tar.gz (11 kB)
Collecting cltl.reply-generation
  Using cached cltl.reply_generation-0.0.dev8.tar.gz (20 kB)
Collecting cltl.triple-extraction
  Using cached cltl.triple_extraction-0.0.dev3.tar.gz (82.8 MB)
    ERROR: Command errored out with exit status 1:
     command: '(...)\cltl-chatbots\venv\Scripts\python.exe' -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\...\\AppData\\Local\\Temp\\pip-install-wv1monyx\\cltl-triple-extraction_0bf0c5efb60442338d7916b55f661f41\\setup.py'"'"'; __file__='"'"'C:\\Users\\...\\AppData\\Local\\Temp\\pip-install-wv1monyx\\cltl-triple-extraction_0bf0c5efb60442338d7916b55f661f41\\setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(__file__) if os.path.exists(__file__) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base 'C:\Users\...\AppData\Local\Temp\pip-pip-egg-info-zg6zfxj_'
         cwd: C:\Users\...\AppData\Local\Temp\pip-install-wv1monyx\cltl-triple-extraction_0bf0c5efb60442338d7916b55f661f41\
    Complete output (7 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "C:\Users\...\AppData\Local\Temp\pip-install-wv1monyx\cltl-triple-extraction_0bf0c5efb60442338d7916b55f661f41\setup.py", line 4, in <module>
        long_description = fh.read()
      File "C:\Users\...\AppData\Local\Programs\Python\Python39\lib\encodings\cp1252.py", line 23, in decode
        return codecs.charmap_decode(input,self.errors,decoding_table)[0]
    UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 1390: character maps to <undefined>
    ----------------------------------------
WARNING: Discarding https://files.pythonhosted.org/packages/38/20/3f597645c874ec9ae57315d9e6009e22b767bb1ca90e113a980e8b40557c/cltl.triple_extraction-0.0.dev3.tar.gz#sha256=8cb45ce3ac544d663548236c6aa473fbf1b4372d3c8adb2d3cbbbf754a029c1e (from https://pypi.org/simple/cltl-triple-extraction/) (requires-python:>=3.7). Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
ERROR: Could not find a version that satisfies the requirement cltl.triple-extraction (from versions: 0.0.dev3)
ERROR: No matching distribution found for cltl.triple-extraction```

Hey,
I guess you are using a Windows machine.
Unfortunately, I haven't tested anything on Windows, so I can't help with you so much at this moment.

Otherwise, you'll have to google or find someone who can help you with running things on a Windows machine.

@a-r-t-i-f-i-c-i-a-l
One of the students managed to run it on Windows.
Check this out: https://github.com/leolani/cltl-chatbots/blob/main/TROUBLESHOOTING.md

I've tried the steps outlined there, but the encoding is already set in setup.py. Trying to change it to # encoding: utf-8 does nothing either.

I've tested it yesterday, and now it supports sound output as well. They say USB devices might work too but it might be a bit hacky at this point.


PT-1316 | Add language column to occurrences table in event page

Description :sparkles:

Add language column to occurrences table in event page

Issues :bug:
Closes :no_good_woman:
PT-1316:
Related :handshake:
Testing :alembic:
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:

Additional notes :spiral_notepad:

Codecov Report

Merging #184 (ec6c5cd) into master (4ccd4aa) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #184      +/-   ##
==========================================
+ Coverage   83.40%   83.41%   +0.01%
==========================================
  Files         111      111
  Lines        3278     3280       +2
  Branches      729      730       +1
==========================================
+ Hits         2734     2736       +2
  Misses        536      536
  Partials        8        8

Impacted Files
Coverage Δ

src/domain/event/occurrences/OccurrencesTable.tsx
96.85% <100.00%> (+0.05%)
:arrow_up:

Continue to review full report at Codecov.

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


fix(core): exclude Lambda metrics if TG target is not Lambda

LambdaUserError and LambdaInternalError metrics were being included for ALB TargetGroup resources, even if the target type was not Lambda. This was previously fixed for alarms, but not dashboard.
How Has This Been Tested?
Unit test added
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING.md document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Coverage remained the same at 100.0% when pulling df250ad0273f14189d1779c2cacc2164c5f7e2d3 on alb-tg-omit-lambda-metrics into e9e518aa8c067f53934887d4ac091a60a0b97edd on main.


Multiple warnings while concurrently edges insertion

I written small piece of code, which concurrently inseting edges between nodes.
I've got multiple messages like these in logs:
Apr 18, 2015 2:40:51 PM com.orientechnologies.common.log.OLogManager log
WARNING: Update of collection pointer is received but collection is not registered
Apr 18, 2015 2:40:51 PM com.orientechnologies.common.log.OLogManager log
WARNING: Update of collection pointer is received but collection is not registered
Apr 18, 2015 2:40:51 PM com.orientechnologies.common.log.OLogManager log
WARNING: Update of collection pointer is received but collection is not registered
Apr 18, 2015 2:40:51 PM com.orientechnologies.common.log.OLogManager log
WARNING: Update of collection pointer is received but collection is not registered

A key idea is to commit oftenly in multiple threads:
for (x <- 0 to settings.initialHospitals / concurrency) {
          val hospitalProps = generator.generateHospital()
          hospitalProps.put("name", s"${settings.prefix}_hospital_${x}_$opNumber")
          val hospital = countingAddVertex(graph, "class:hospital", hospitalProps)
          graph.commit()

          for (y <- 0 to settings.initialDoctorsPerHospital) {
            val doctorProps = generator.generateDoctor()
            doctorProps.put("name", s"${settings.prefix}_doctor_${y}_from_hospital_${x}_$opNumber")
            val doctor = countingAddVertex(graph, "class:doctor", doctorProps)
            doctor.addEdge("employed_at", hospital)
            graph.commit()

          }
        }

Here is my test suite for OrientDB evaluation: https://github.com/FeiWongReed/odbtest
Here is the branch demostraiting this issue: https://github.com/FeiWongReed/odbtest/tree/collection_not_registered
Also could you hint me if it's good idea to commit oftenly or to do many operations then commit once?

Which version do you use, this bug was fixed in latest hot fix version.

2.1-SNAPSHOT )

Sorry, I do not know Scala is it possible to make the same test on Java ?

Moved to 2.2 because of no reply

Closed because of no reply


DutyCycle to be updated at the end current PWM period

Original post from @ggmichael  in Change Duty Cycle #1

Thanks for this! Your library has fixed a problem I was having with inconsistent calibration, which I now think arose from the changing timing of the analogWrite pwm.
One more feature that would be nice would be to be able to command a duty cycle update to occur at the end of the next full period instead of immediately (as I think happens now). If you're trying to create a continuously variable mean output voltage, resetting the channel at a frequency that approaches the PWM frequency causes a bias towards whichever part of the duty cycle comes first. This wouldn't happen if the update occurred at the end of the cycle. It may be that there are other applications (long cycles) where you would prefer the update straight away. So perhaps a flag to set either mode.

HI  @ggmichael
I'll publish a new release to add this enhancement within half an hour for you to test the new feature. Be ready.
Regards,

Hi @ggmichael
The new ESP8266_PWM releases v1.2.2 has just been published. Your contribution is noted in Contributions and Thanks. Please test and verify the new feature is supported and working OK now.
The library is better and better everyday, thanks to contributing users like you, who spot bugs, test and request enhancements. We, the library writers, normally can't never have enough time, knowledge and experience to know all the use-cases in the world to add the new features.
Best Regards,

Releases v1.2.2

DutyCycle to be optionally updated at the end current PWM period instead of immediately. Check DutyCycle to be updated at the end current PWM period #2

I can confirm that the new code gets rid of the 'glitch' on changing duty cycle - that's great - thanks. I think you're right that analogWrite() doesn't do this, but there's no harm in having a better implementation. (However, something is now causing the ESP8266 to reboot periodically - I have to check over my integration of the library)

HI @ggmichael

However, something is now causing the ESP8266 to reboot periodically - I have to check over my integration of the library

If you still have that problem, I suggest you try these mods to see if it's any better

Increase ESP8266 clock to 160MHz from 80MHz, if not yet
Increase the HW_TIMER_INTERVAL_US from 20uS to higher, such as 30, 50, 100uS. For example
ISR_Modify_PWM.ino#L46

#define HW_TIMER_INTERVAL_US      50L

The reason is the ISR-based function run() now take more time to process (double accuracy, dutycycle check, etc.)
I'm thinking, in future releases, about letting users optionally select double, float or uint32_t for dutycycle / frequency / period, depending on the use-case.

Hi Khoi,
I got a previous version to work by back-integrating the main parts of your
new updates. I did use floats for the duty cycle instead of double (I think
double is more precision than you could ever need). The period remained
uint32_t (this also makes sense to me, since this is the actual count).
Frequency could also be a float, since it is always converted. So maybe
this is the reason. I'll try your new suggestions when I get a chance -
probably only next week.
On Tue, 1 Feb 2022, 14:59 Khoi Hoang, @.***> wrote:

HI @ggmichael https://github.com/ggmichael
However, something is now causing the ESP8266 to reboot periodically - I
have to check over my integration of the library
If you still have that problem, I suggest you try these mods to see if
it's any better

Increase ESP8266 clock to 160MHz from 80MHz, if not yet
Increase the HW_TIMER_INTERVAL_US from 20uS to higher, such as 30,
50, 100uS. For example
ISR_Modify_PWM.ino#L46
https://github.com/khoih-prog/ESP8266_PWM/blob/main/examples/ISR_Modify_PWM/ISR_Modify_PWM.ino#L46

#define HW_TIMER_INTERVAL_US      50L
The reason is the ISR-based function run() now take more time to process
(double accuracy, dutycycle check, etc.)
I'm thinking, in future releases, about letting users optionally select
double, float or uint32_t for dutycycle / frequency / period, depending
on the use-case.
—
Reply to this email directly, view it on GitHub
https://github.com/khoih-prog/ESP8266_PWM/issues/2#issuecomment-1026872180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR5ZQ2XLYMJDCQALK3GE77LUY7RLBANCNFSM5NEVDMVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

I did use floats for the duty cycle instead of double (I think double is more precision than you could ever need). The period remained uint32_t (this also makes sense to me, since this is the actual count). Frequency could also be a float, since it is always converted.

I agree with you and will make float for DC and Freq, period uint32_t. Sorry for to be too unreasonably aggressive.
Will have a new release within today.

Hi @ggmichael
Please test the new ESP8266_PWM releases v1.2.3

Releases v1.2.3

Use float for DutyCycle and Freq, uint32_t for period.
Optimize code by not calculation in ISR

I confirm that update works, and also fixes the reboot problem.

Glad to know it's working better now, thanks to your contribution anyway.


how im can make the window longer please help

how im can make the window longer please help
reply PLEASEEEE ;(

you can't, only when adding more keys

Okay
Em qui, 13 de ago de 2020 05:19, Robin<EMAIL_ADDRESS>escreveu:

you can't, only when adding more keys
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yugecin/osukps/issues/32#issuecomment-673335571, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APSZA3BEOXKY64A2AM4TAD3SAOO2LANCNFSM4P5P222Q
.


B_Sweep broken?

When we run B_Sweep, we get a figure that contradicts the figure shown in your paper, Preventing Extreme Polarization of Political Attitudes. The only code we changed was the intervals for the Tolerance and Exposure (for the sake of running the experiment more quickly) -- we changed line 364 in arm_exp.py to
T, E = np.arange(0.05, 1.01, 0.2), [[e] for e in np.arange(0.05, 0.51, 0.1)]

The figure is attached below.

.

Thanks for bringing this to my attention! It seems like there's a transposition of the tolerance and exposure axes. I'll look into this, but in the meantime, do you see a similar inversion with A_sweep (which produces Fig. 3)?


Edit category modal

Currently categories are edited "in place" by checking an "edit mode" checkbox. This is a bit clumsy, and with new features like #407 and #414 the interface starts to look a bit bloated.
The idea here is to have an editing modal for categories, similar to the one for tracks.

There is some general discussion about this in #396 already, but we should probably summarize it here and specify exactly what should and what shouldn't be in the new modal.
@dagraf would you be willing to do that in a comment here?

This should probably be deferred until after #414 and #407 are merged.

There is some general discussion about this in #396 already, but we should probably summarize it here and specify exactly what should and what shouldn't be in the new modal.
@dagraf would you be willing to do that in a comment here?

Yes, I am willing to do this here. :) But it has to wait till beginning of next week (at latest).

See also maybe #373? :thinking:

Short update: I'm working on the specs and will post them here this week.

I suggest creating 4 different modals (create category, edit catgory, create scale, edit scale) and a new 'action-dropdown' in the Public- and Mine-Tab. See mockups and specs for details. I think we should discuss about this in a meeting. What do you think? Please send me some possible dates we could meet online.
Mockup_Specs Kateogrie-Modal.pdf

See new proposals for...

... how to ged rid of the checkbox "Edit"
... deal with missing values in mandatory boxes
... deal with the "scales are not possible with series-cateogries"-issue

Mockup Specs_Kategorie und Skala Modal_v3.pdf

As far as I can tell this is basically done now. Of course we can always improve things further, but the feature in its basic form has landed.


Bug fix npe when one row relation contain aggregate function

Proposed changes
mysql [test]>select sum(1);
ERROR 1105 (HY000): errCode = 2, detailMessage = Unexpected exception: null

Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

run buildall

run p1


Package directory is configurable

I've changed the parts of the plugin which load the language syntax files to read a setting called git_package_dir and use that as the directory to load from. This is relative to the packages folder.
For example, if you have the plugin installed in /path/to/Packages/sublime-text-2-git, then you should change Base File.sublime-settings to add this:
{
    "git_package_dir": "sublime-text-2-git"
}

The default value is "Git", so if the plugin was installed via Package Control, you do not need to do anything.

Note: I'm a total newbie with python, and I wasn't able to find any information about how to get the file system path of the currently-executing module, hence the hack workaround with using settings. If someone else knows a better way, please do let me know.

Not merging this, for the reason given in issue #23.
It's actually sort of fiddly to get the directory of the currently executing file. __file__ is a python constant that (normally) exists, and contains either an absolute or relative path to the current file, depending on whether it's in sys.path or not.
Unfortunately... __file__ is set at the time of import, so if you have '.' in your path and the import happened while cd'd into a directory then __file__ will be ./git.py, which is not terribly helpful if you've moved since then. Sublime loads its plugins by changing into their directories and then importing the .py file, so that's what it winds up being. I did thus work out that the way to know your plugin directory is to do os.getcwd at the top level of the plugin module, since that guarantees it'll run while sublime's execution is actually in the plugin directory.

Ah, excellent. My approach did feel very hacky...


clp-bench: Refactor and Add New Results

This PR does a major refactoring. The main change is that we lock the clp-bench executor's code logic, for each target tool, the user shall provide corresponding scripts that to execute tasks of benchmarking.
Benefits:

No need to modify the clp-bench package's code everytime we add a new target tool to benchmark against.
The scripts of each target tool are stored in a asset folder, which is clear and easy to manage.

Sorry, was sick over the weekend so didn't get a chance to look at this. Will review at least the docs today.

Sorry, was sick over the weekend so didn't get a chance to look at this. Will review at least the docs today.

Take care! It becomes very vold suddenly these days.

Is the first flowchart in the methodology being displayed as you expect?

Is the first flowchart in the methodology being displayed as you expect?

Apparently no lol, so weird. It should look like this:

Don't know why it gets rendered so weirdly on github


[T4A3][W14-B2] Seow Wei Jie

Ready for review

@weijieseow I added some comments. Please close the PR after reviewing them.


navigation styling

Task
It would be great to have a transition for the navigation. Something like a toggle for a slide in to view transition. The
Requirements

Keep the design as  accessibility friendly as possible.
Don't stray too far from the current design. (In this case avoid strong contrasting colors or over the top details, as the current styling is simplistic)

Files effected
The navigation is going to be available on all pages, so the components/global should be the only folder changed in this task. As to how you implement the changes, it is entirely up to you 😄

can you assign this to me. i am new to open source contributions but i have worked o next js projects

can you assign this to me. i am new to open source contributions but i have worked on next js projects

Absolutely! Thank you :)

@iamshobhraj Any update on this?

@iamshobhraj Any update on this?

i raised a PR #2


Running with coverage option throws "Cannot find module 'lcov'"

🐛 Bug Report
Running with jest --coverage throws the following error:
yarn run v1.19.1
$ jest --coverage
 PASS  src/index.test.js
  add
    ✓ should work (3ms)

----------|---------|----------|---------|---------|-------------------
File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
----------|---------|----------|---------|---------|-------------------
All files |     100 |      100 |     100 |     100 |
 index.js |     100 |      100 |     100 |     100 |
----------|---------|----------|---------|---------|-------------------

        Failed to write coverage reports:
        ERROR: Error: Cannot find module 'lcov'
Require stack:
- /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/istanbul-reports/index.js
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/jest-cli/build/cli/index.js
- /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/jest-cli/bin/jest.js
- /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/jest/bin/jest.js
        STACK: Error: Cannot find module 'lcov'
Require stack:
- /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/istanbul-reports/index.js
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/jest-cli/build/cli/index.js
- /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/jest-cli/bin/jest.js
- /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/jest/bin/jest.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:892:15)
    at Function.Module._load (internal/modules/cjs/loader.js:785:27)
    at Module.require (internal/modules/cjs/loader.js:956:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.create (/Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/istanbul-reports/index.js:19:20)
    at /Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/@jest/reporters/build/coverage_reporter.js:300:20
    at Array.forEach (<anonymous>)
    at CoverageReporter.onRunComplete (/Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/@jest/reporters/build/coverage_reporter.js:298:25)
    at ReporterDispatcher.onRunComplete (/Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/@jest/core/build/ReporterDispatcher.js:71:10)
    at TestScheduler.scheduleTests (/Users/shihsyuanyan/Desktop/test/jest-coverage-issue/node_modules/@jest/core/build/TestScheduler.js:280:5)

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        0.48s, estimated 1s
Ran all test suites.

To Reproduce

Create a new folder and initialize with yarn init
yarn add --dev jest
yarn run jest --init
write some tests
yarn run jest --coverage
It throws errors

Expected behavior
I checked the coverageReports option in the documentation. It should generate lcov related reports without any error, I guess?
Link to repl or repo (highly encouraged)
jest-coverage-issue. Simply run yarn run test:cov to reproduce the error.
envinfo

  System:
    OS: macOS Mojave 10.14.1
    CPU: (8) x64 Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz
  Binaries:
    Node: 13.1.0 - ~/.nvm/versions/node/v13.1.0/bin/node
    Yarn: 1.19.1 - /usr/local/bin/yarn
    npm: 6.12.1 - ~/.nvm/versions/node/v13.1.0/bin/npm
  npmPackages:
    jest: ^25.1.0 => 25.1.0

I cannot reproduce this from your reproduction

I'm able to reproduce.
System:
  Model Name:	MacBook Pro
  Model Identifier:	MacBookPro12,1
  Processor Name:	Intel Core i5
  Processor Speed:	2.7 GHz
  OS: macOS Movaje 10.14.6
Binaries:
  Yarn: 1.3.2
  Node: 13.0.0
  npm: 6.14.3
Packages:
  Jest v25.1 .0

Stack trace:
Failed to write coverage reports:
        ERROR: Error: Cannot find module 'lcov'
Require stack:
- /Users/famoraes/Documents/Projects/React/telescopio/node_modules/istanbul-reports/index.js
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /Users/famoraes/Documents/Projects/React/telescopio/node_modules/jest-cli/build/cli/index.js
- /Users/famoraes/Documents/Projects/React/telescopio/node_modules/jest-cli/bin/jest.js
        STACK: Error: Cannot find module 'lcov'
Require stack:
- /Users/famoraes/Documents/Projects/React/telescopio/node_modules/istanbul-reports/index.js
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /Users/famoraes/Documents/Projects/React/telescopio/node_modules/jest-cli/build/cli/index.js
- /Users/famoraes/Documents/Projects/React/telescopio/node_modules/jest-cli/bin/jest.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:795:15)
    at Function.Module._load (internal/modules/cjs/loader.js:688:27)
    at Module.require (internal/modules/cjs/loader.js:850:19)
    at require (internal/modules/cjs/helpers.js:74:18)
    at Object.create (/Users/famoraes/Documents/Projects/React/telescopio/node_modules/istanbul-reports/index.js:19:20)
    at /Users/famoraes/Documents/Projects/React/telescopio/node_modules/@jest/reporters/build/coverage_reporter.js:300:20
    at Array.forEach (<anonymous>)
    at CoverageReporter.onRunComplete (/Users/famoraes/Documents/Projects/React/telescopio/node_modules/@jest/reporters/build/coverage_reporter.js:298:25)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at ReporterDispatcher.onRunComplete (/Users/famoraes/Documents/Projects/React/telescopio/node_modules/@jest/core/build/ReporterDispatcher.js:71:10)

Solved upgrade node version to v13.11.0.

That's weird, some bug in node then? Closing, but can reopen if it's reproducible in current versions of node

I have the same problem. I tried to clone my project to another folder and it worked for the first time and never succeeded again.


React step 5

https://carbon-tutorial-crystalgxh.au-syd.mybluemix.net/#/
Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

pls help to review react-step-5


Out of Memory Error when using flutter_uxcam 2.4.5

I encountered a fatal Out of Memory error while using the flutter_uxcam plugin in my Flutter project. This error occurred with the following versions:

flutter_uxcam version: 2.4.5
Flutter SDK version: 3.10.6

Error Message:
Fatal Exception: java.lang.OutOfMemoryError: Failed to allocate a 75497480 byte allocation with 8388608 free bytes and 69MB until OOM, target footprint 204333984, growth limit 268435456
       at java.util.Arrays.copyOf(Arrays.java:3766)
       at java.lang.AbstractStringBuilder.ensureCapacityInternal(AbstractStringBuilder.java:125)
       at java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:449)
       at java.lang.StringBuilder.append(StringBuilder.java:137)
       at org.json.JSONStringer.close(JSONStringer.java:205)
       at org.json.JSONStringer.endArray(JSONStringer.java:153)
       at org.json.JSONArray.writeTo(JSONArray.java:618)
       at org.json.JSONStringer.value(JSONStringer.java:242)
       at org.json.JSONArray.writeTo(JSONArray.java:616)
       at org.json.JSONStringer.value(JSONStringer.java:242)
       at org.json.JSONArray.writeTo(JSONArray.java:616)
       at org.json.JSONStringer.value(JSONStringer.java:242)
       at org.json.JSONArray.writeTo(JSONArray.java:616)
       at org.json.JSONStringer.value(JSONStringer.java:242)
       at org.json.JSONObject.writeTo(JSONObject.java:734)
       at org.json.JSONStringer.value(JSONStringer.java:246)
       at org.json.JSONArray.writeTo(JSONArray.java:616)
       at org.json.JSONStringer.value(JSONStringer.java:242)
       at org.json.JSONObject.writeTo(JSONObject.java:734)
       at org.json.JSONObject.toString(JSONObject.java:702)
       at com.uxcam.internals.bm.a(bm.java:1206)
       at com.uxcam.internals.fs.a(fs.java:12)
       at com.uxcam.internals.fs.a(fs.java:18)
       at com.uxcam.internals.fj.a(fj.java:20)
       at com.uxcam.internals.gl$ac.run(gl.java:197)

Additional Information:

The issue occurs consistently with the specified versions.
Please provide guidance or updates on how to resolve this Out of Memory error when using the flutter_uxcam plugin.

In Crashlytics, we have observed the Out of Memory error reported on the following devices versions:

Moto E5 Play
Pixel 5
Galaxy S22 Ultra
Moto G Stylus 5G
Moto G 5G (2022)
Galaxy Note20 Ultra 5G

These devices running Android 13 primarily experience this issue. This information can help in pinpointing the problem and providing a more targeted solution.
Thank you for your attention to this issue.

Hi @aovooyov Diego here from UXCam, First of all, I apologize for the delay in this response.
In the other hand, I wanted to give you good news, We have implemented a fixed for this specific crash which will be available in the upcoming release of the flutter_uxcam package.
Will keep you updated, but please keep an eye at this page for upcoming releases of our package


varnish connection refused

I think it's the same kind of problem in varnish than in redis :
Wed Dec 11 16:53:30 UTC 2019 - Processing <IP_ADDRESS>:6081
Wed Dec 11 16:53:30 UTC 2019 - Error connecting to <IP_ADDRESS>:6081 : Connection refused (111)

plugin used : Varnish Wordpress Version 1.7 | By AdminGeekZ

Is this plugin logs? I see you're connecting <IP_ADDRESS>, varnish host is varnish by default

Yes, thank you.


Remove undo/redo buttons from rich text toolbar

Is your proposal related to a problem?
Currently we show undo-redo buttons in the toolbar, which don’t feel particularly useful, and add to the clutter of icons:

Describe the solution you'd like
In https://github.com/wagtail/wagtail/issues/4997#issuecomment-577705230, @benenright suggests removing them. This would just remove the buttons, but people would still be able to use OS-level controls to undo/redo (keyboard shortcuts, virtual keyboard on mobile devices). I can see a few different ways to go with this:

Remove the buttons, but add a new undo-redo rich text feature, not enabled by default, which makes both buttons appear as they do currently.
Remove the buttons, adding another way to make their presence configurable.
Remove the buttons, without any option to re-add them.

I’d be interested in feedback on which of the 3 ways to implement this sounds best.

Just to note that as part of the page editor 2022 project we’re going to go for option 3 – remove the buttons altogether (but undo-redo is still possible). Implementing this will just be a matter of removing two lines: https://github.com/wagtail/wagtail/blob/ab92256b093b3817ac58aff5c1e2b1b04af8c10f/client/src/components/Draftail/index.js#L116-L117
and updating any related test cases (if any, not sure).

Done in #8806, shipping in Wagtail 4!


command line option to exclude files

Add a command line option that will read in one or more UNIX-style patterns. Any file name matching this pattern will be ignored. No license will be incorporated.
licenseheaders ... -x "_version.py,*not_this_one*"

Hello. Was the function mentioned aforementioned implemented in addition finally?
Thanks

No, I forked the repository and that's why I closed the issue. I'll try to implement that functionality in the forked repo and then open a PR to your repo. The biggest hurdle is finding time, as usual 😄

Done
https://github.com/johann-petrak/licenseheaders/pull/35

Thanks, I have merged the PR, but I am going to leave this open since it only checks for literal matches, not REs or globs.

OK, it should be possible now to add glob patterns, e.g. -x 'somedir/*' '*.ext' to ignore files in the directory somedir and ignore all files with extension ext
More testing required.

Seems to work.


RuntimeError: File files/default/plugins/README does not exist for cookbook ohai

Version:
Using chef-solo 12.2 or 12.3
Environment: [Details about the environment such as the Operating System, cookbook details, etc...]
ubuntu 14.04
chef-solo 12.3
application cookbook : nginx
wrapper cookbook : mycompany-nginx
Scenario:
Using nginx wrapper cookbook to install nginx(nginx cookbook). This works with Test Kitchen(Vagrant driver and chef-solo provisioner).  When trying to use with Packer(amazon-ebs builder, chef-solo provisioner), I get the following
amazon-ebs: [2015-04-27T12:22:41+00:00] ERROR: remote_directory[/etc/chef/ohai_plugins for cookbook ohai] (ohai::default line 33) had an error: RuntimeError: cookbook_file[/etc/chef/ohai_plugins/README] (dynamically defined) had an error: RuntimeError: File files/default/plugins/README does not exist for cookbook ohai

Steps to Reproduce:
Install nginx 2.7.6 cookbook via a wrapper cookbook with Packer 0.7.5(amazon-ebs builder, chef-solo provisioner)
Expected Result:
Expecting nginx to be installed, since it's working in Test Kitchen.
Actual Result:
I get the following error
amazon-ebs: [2015-04-27T12:22:41+00:00] ERROR: remote_directory[/etc/chef/ohai_plugins for cookbook ohai] (ohai::default line 33) had an error: RuntimeError: cookbook_file[/etc/chef/ohai_plugins/README] (dynamically defined) had an error: RuntimeError: File files/default/plugins/README does not exist for cookbook ohai

Note the source  and destination dirs exist.
-rw-r--r-- 1 ubuntu ubuntu 49 Apr 29 15:08 /tmp/packer-chef-solo/cookbooks-1/ohai-2.0.1/files/default/plugins/README

drwxr-xr-x 2 root root 4096 Apr 29 15:08 /etc/chef/ohai_plugins

When I log into the packer node and run with chef-solo manually, I get the same error.
sudo chef-solo -l debug -c /tmp/packer-chef-solo/solo.rb -j /tmp/packer-chef-solo/node.json

$ cat /tmp/packer-chef-solo/solo.rb

cookbook_path 	["/tmp/packer-chef-solo/cookbooks-0","/tmp/packer-chef-solo/cookbooks-1"]

cat /tmp/packer-chef-solo/node.json
{
  "run_list": [
    "recipe[mycompnay-nginx::default]"
  ]

I added some puts to debug the  get_filename function in /opt/chef/embedded/apps/chef/lib/chef/cookbook/file_system_file_vendor.rb. I see  the ohai version number is omitted in the path to source file
----------------------- /tmp/packer-chef-solo/cookbooks-0/ohai/files/default/plugins/README -------------------
----------------------- /tmp/packer-chef-solo/cookbooks-1/ohai/files/default/plugins/README -------------------

I ran kitchen converge in debug mode, and noticed it was consolidating the cookbooks(wrapper and application) in one directory.  I did this to my repo also , and now it's working.

What about the case you have to store your wrapper and upstream cookbooks in separate repositories (i.e. upstream cookbook is managed by berkshelf)? I am also affected by this by using chef-solo alone.

Another issue could be Berkshelf isn't loaded properly

I'm sorry--what is the actual fix here? I'm having this issue and I don't know how to solve it.

This fix was to put all cookbooks into one directory. Although not a best practice.  The provisioners section of packer template.json file looked like this
"provisioners": [{
    "type": "chef-solo",
    "cookbook_paths": ["/Users/username/cookbooks"],
    "run_list": [
      "recipe[apt::default]",
       "recipe[mycompany-nginx::default]"
    ]
  }],

It could very well be Berkshelf isn't loaded properly, and I'm curious if anyone can elaborate.

Cookbooks are all in the same dir and it's still not working for me. :(

It appears that the issue is that cookbook_file is unaware that the directory the cookbook is in contains a version. The code in file_system_watcher does correctly identify berkshelf cookbook root paths, but it does not correctly identify the cookbook path because the cookbook path is "name+version" and file_system_watcher expects "name."


introduce timeout factor to controll invoker queue behavior

This PR introduces a configurable multiplicator to influence the forced active ack timeout period.
Description
In rare condition one can see that the load balancer put activations into queues of invokers falsely
identified to have free capacity because forced active acks  are send out for activation that are waiting in the queue to be executed. With increasing  the timeout the loadbalancer can be influenced to distribute the load wider in this cases.
Related issue and scope
My changes affect the following components

[ ] API
[x] Controller
[ ] Message Bus (e.g., Kafka)
[x] Loadbalancer
[x] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #3767 into master will increase coverage by <.01%.
The diff coverage is 50%.

@@            Coverage Diff            @@
##           master   #3767      +/-   ##
=========================================
+ Coverage    75.4%   75.4%   +<.01%
=========================================
  Files         132     132
  Lines        6172    6173       +1
  Branches      393     397       +4
=========================================
+ Hits         4654    4655       +1
  Misses       1518    1518

Impacted Files
Coverage Δ

...e/loadBalancer/ShardingContainerPoolBalancer.scala
32.54% <0%> (ø)
:arrow_up:

...a/src/main/scala/whisk/core/entity/TimeLimit.scala
95.65% <100%> (+0.19%)
:arrow_up:

Continue to review full report at Codecov.

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

PG3 2561 🔵


Ability to create calendar display by manually passing dates collection

Currently, to create a calendar table, you have to enter start / end dates.
Going forward with events, it may be beneficial to have the ability to retrieve calendar dates collection and then be able to build a MyCal table from the collection rather than start / end dates

Fixed in 0.5.0 release


Switch snapping to Rtree package?

This tests out switching the Rtree implementation to that provided by libspatialindex. This is the only place that Spaghetti uses an Rtree. This resolves issues reported by libpysal#126 in the downstream location, rather than upstream with the rtree package. Given the existing soft dependencies on shapely & geopandas in the module, I think this might be a simpler fix to consider.

Pull Request Test Coverage Report for Build 1412

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.4%) to 97.443%

Files with Coverage Reduction
New Missed Lines
%

spaghetti/tests/test_network.py
2
99.3%

spaghetti/tests/test_network_api.py
2
99.17%

spaghetti/util.py
3
98.14%

Totals

Change from base Build 1359:
-0.4%

Covered Lines:
1346

Relevant Lines:
1367

💛  - Coveralls

Pull Request Test Coverage Report for Build 1412

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
7 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.4%) to 97.443%

Files with Coverage Reduction
New Missed Lines
%

spaghetti/tests/test_network.py
2
99.3%

spaghetti/tests/test_network_api.py
2
99.17%

spaghetti/util.py
3
98.14%

Totals

Change from base Build 1359:
-0.4%

Covered Lines:
1346

Relevant Lines:
1367

💛  - Coveralls

Potential for solution of #196

Given the existing soft dependencies on shapely & geopandas in the module, I think this might be a simpler fix to consider.

shapely & geopandas are soft dependencies and all core functionality of spaghetti can be performed without them. However, the bug fixed by introducing Rtree is within a core function of spaghetti (util.snap_points_to_links), making Rtree a hard dependency.
@ljwolf @sjsrey @weikang9009 @knaaptime
What are your thoughts on this?

pysal/libpysal#126

I am concerned about making RTree a hard dependency, but would like feedback on this from @ljwolf and other pysal folks.

It looks like rtree is widely used: https://anaconda.org/conda-forge/rtree
so I'm ok with the dependency.

@sjsrey ready for review.


Write nextCell

Hi
I've done a TODO

What about scrolling animation?
Which package do you prefer to use?
Personally I think scroll-to-element is a good choice

Fixed,
Ok no problem also I started to write another local Notebook (Something like Jupyter) with React, publish the beta next week after doing some works :)

I close it because my local branch is shitty and I couldn't make things right
An straight & clean PR is on the way with all your requests


layers: Distinguish never-bound from un-bound memory cases

Fixes #964
Added special MEMORY_UNBOUND handle (0xF..FE) that indicates that memory
bound to an object has been freed. When attempting to bind memory or
checking for bound memory, distinguish the never-bound case from the
memory un-bound case.
Update tests to account for new error messages.
There's a sliver of exposure here if an actual memory handle is MEMORY_UNBOUND.
We could remove that exposure by never having unique_objects return MEMORY_UNBOUND
as a handle. I believe the exposure is small enough that we don't need to do that,
but am open to other opinions.

cherry-picked and pushed to master